linux-kernel - Re: [PATCH v2 7/7] arm64: dts: marvell: Add a device tree for the iEi Puzzle-M801 board

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20200926195008.6bd84dd3@nic.cz>
Date:   Sat, 26 Sep 2020 19:50:08 +0200
From:   Marek Behun <marek.behun@....cz>
To:     Luka Kovacic <luka.kovacic@...tura.hr>
Cc:     linux-kernel@...r.kernel.org, linux-hwmon@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org, linux-leds@...r.kernel.org,
        lee.jones@...aro.org, pavel@....cz, dmurphy@...com,
        robh+dt@...nel.org, jdelvare@...e.com, linux@...ck-us.net,
        andrew@...n.ch, jason@...edaemon.net, gregory.clement@...tlin.com,
        luka.perkov@...tura.hr, robert.marko@...tura.hr
Subject: Re: [PATCH v2 7/7] arm64: dts: marvell: Add a device tree for the
 iEi Puzzle-M801 board

On Sat, 26 Sep 2020 15:55:14 +0200
Luka Kovacic <luka.kovacic@...tura.hr> wrote:

> +	leds {
> +		compatible = "gpio-leds";
> +		status = "okay";
> +		pinctrl-0 = <&cp0_sfpplus_led_pins &cp1_sfpplus_led_pins>;
> +		pinctrl-names = "default";
> +
> +		led0 {
> +			function = LED_FUNCTION_STATUS;
> +			label = "p2_act";
> +			gpios = <&cp1_gpio1 6 GPIO_ACTIVE_LOW>;
> +		};

There should be a dash in LED node name, please pass this dts via
dt_binding_check
  led-0 {
    ...
  };

Also why not add the `color` property to the LED? This is DTS for a
specific device, right?
`label` is obsolete. The LED subsystem creates a name in form
  [device:]color:function
If this LED should blink for activity on port 2 (is this an ethernet
port?), the function should be LED_FUNCTION_LAN and function-enumerator
should be <2> (or function should be LED_FUNCTION_ACTIVITY, depending
on how the LED subsystem goes forward with this, but certainly not
LED_FUNCTION_STATUS), and trigger-sources should be set to point to the
ethernet port.

Luka, are you willing to change this once we solve this API properly
in LED subsystem?

> +		led6 {
> +			function = LED_FUNCTION_STATUS;
> +			linux,default-trigger = "disk-activity";
> +			label = "front-hdd-led";
> +			gpios = <&cp0_gpio2 22 GPIO_ACTIVE_HIGH>;
> +		};

led-6. LED_FUNCTION_DISK. `label` deprecated.

> +		leds {
> +			compatible = "iei,wt61p803-puzzle-leds";
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +
> +			led@0 {
> +				reg = <0>;
> +				color = <LED_COLOR_ID_BLUE>;
> +				label = "front-power-led";
> +			};

Again, `label` is deprecated. Rather use function =
<LED_FUNCTION_POWER>;

Marek