On 08/23/2016 10:39 AM, Simon Horman wrote:
Add support for the general purpose LEDs on the Blanche board. Signed-off-by: Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx> --- This patch is against the 'renesas-devel-20160822-v4.8-rc3' of Simon Horman's 'renesas.git' repo plus the SDHI0/DU patches posted earlier. arch/arm/boot/dts/r8a7792-blanche.dts | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) Index: renesas/arch/arm/boot/dts/r8a7792-blanche.dts =================================================================== --- renesas.orig/arch/arm/boot/dts/r8a7792-blanche.dts +++ renesas/arch/arm/boot/dts/r8a7792-blanche.dts @@ -159,6 +159,27 @@ }; }; + leds { + compatible = "gpio-leds"; + + led17 { + gpios = <&gpio10 10 GPIO_ACTIVE_HIGH>; + label = "LED17";I am unclear of the motivation for using the label property when its value is so similar to the name of the node. The reason I noticed this is that it appears to differ from led nodes for other Renesas ARM SoCs.
I was looking at the Koelsch's DT as an example (and Gose's DT also had labels).
I'm not opposed to using the label property if there is some value to it. But I would prefer to use it consistently. And I'd prefer not to add it to all the other SoCs unless there is a reason to.
Looks like the driver can infer the LED names from the device node names. I'll recast.
[...] MBR, Sergei
