[PATCH] realtek: update GPIO bindings for DGS-1210-10P

[Sander Vanheule]

Hi Jan-Niklas,

On Fri, 2022-12-02 at 14:55 +0100, Jan-Niklas Burfeind wrote:
> add three missing LEDs
>  - PoE-Max
>  - Link/Act
>  - PoE

Do the latter two LEDs indicate which LED mode is currently selected (on stock
FW)? Users can do with these LEDs what they like, of course, but it would be
good to know what the 'default' behaviour is in case somebody ever wanted to add
a default trigger.

Regarding the changes, I have one comment below.

> 
> add two missing buttons
>  - mode
>  - reset
> 
> The last was dropped in
> commit 61a3d0075b15 ("realtek: update GPIO bindings in the dts files in dts-
> 5.10")
> 
> Signed-off-by: Jan-Niklas Burfeind <git at aiyionpri.me>
> ---
> Hello everyone,
> I just tested the missing GPIO assignments for the DGS-12-10-10P
> and verified the three LEDs as well as the reset button are working.
> 
> The mode button should work, as its adress is 481 compared to resets
> 484, but I haven't found a way to test it yet.

You could change the "linux,code" property to <KEY_RESTART> and verify that it
(also) triggers a reboot.

> 
> Thanks
> Jan-Niklas Burfeind
> 
>  .../dts-5.10/rtl8382_d-link_dgs-1210-10p.dts  | 30 ++++++++++++++++---
>  1 file changed, 26 insertions(+), 4 deletions(-)
> 
> diff --git a/target/linux/realtek/dts-5.10/rtl8382_d-link_dgs-1210-10p.dts
> b/target/linux/realtek/dts-5.10/rtl8382_d-link_dgs-1210-10p.dts
> index 7ab37aaa9f..16934ede3b 100644
> --- a/target/linux/realtek/dts-5.10/rtl8382_d-link_dgs-1210-10p.dts
> +++ b/target/linux/realtek/dts-5.10/rtl8382_d-link_dgs-1210-10p.dts
> @@ -11,12 +11,34 @@
>                 compatible = "gpio-keys-polled";
>                 poll-interval = <20>;
>  
> -               /* is this pin 30 on the external RTL8231 (&gpio1)? */
> -               /*mode {
> +               mode {
> +                       label = "mode";
> +                       gpios = <&gpio1 30 GPIO_ACTIVE_LOW>;
> +                       linux,code = <KEY_LIGHTS_TOGGLE>;

Strictly speaking, this was intended for turning a (reading) light on or off. 
See Linux kernel commit 5a1bbf21325bd4f2641f6141fb8c47f6095578dd. You're just
adding back what was dropped in commit 61a3d0075b15, but maybe we should try to
keep the "turn the lights on/off" semantics for <KEY_LIGHTS_TOGGLE>.

The Engenius EW2910P and Panasonic M*eG switch series have similar LED mode
buttons, which the authors mapped to <BTN_0> [1, 2].

See target/linux/realtek/dts-5.10/rtl83xx_panasonic_mxxeg-pn28xx0k.dtsi
and target/linux/realtek/dts-5.10/rtl8380_engenius_ews2910p.dts

As I'm bikeshedding a bit here, does anyone else on the mailing list have an
opinion on this?

[1] https://github.com/openwrt/openwrt/pull/9896
[2] https://github.com/openwrt/openwrt/pull/4209


Best,
Sander