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
