Re: [PATCH] ARM: dts: microchip: at91: align LED node name with bindings

[Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>]

On 02/07/2024 10:51, Alexander Dahl wrote:
>> diff --git a/arch/arm/boot/dts/microchip/at91sam9x5cm.dtsi b/arch/arm/boot/dts/microchip/at91sam9x5cm.dtsi
>> index cdd37f67280b..fb3c19bdfcb6 100644
>> --- a/arch/arm/boot/dts/microchip/at91sam9x5cm.dtsi
>> +++ b/arch/arm/boot/dts/microchip/at91sam9x5cm.dtsi
>> @@ -120,13 +120,13 @@ rootfs@800000 {
>>  	leds {
>>  		compatible = "gpio-leds";
>>  
>> -		pb18 {
>> +		led-pb18 {
>>  			label = "pb18";
>>  			gpios = <&pioB 18 GPIO_ACTIVE_LOW>;
>>  			linux,default-trigger = "heartbeat";
>>  		};
>>  
>> -		pd21 {
>> +		led-pd21 {
>>  			label = "pd21";
>>  			gpios = <&pioD 21 GPIO_ACTIVE_HIGH>;
>>  		};
> 
> In this case these are all gpio-leds and the pattern is in the
> leds-gpio gpio binding.  I'm wondering however why you chose the very
> generic 'led' match over the more strict one requiring the names to

match? Which match? I did not write the match pattern in the binding, did I?

> look like 'led-0', 'led-1' an so forth?  The generic match would also

Works for me too. The easiest was to add led prefix. I am not the
maintainer of this platform, so I am doing just some random cleanups and
prefix is the easiest cleanup.

> match names like 'knowledge' or 'controlled'.  But besides that:

Best regards,
Krzysztof