Re: [PATCH v2] leds: start numerating duplicated devices from 2

[Jacek Anaszewski <j.anaszewski@samsung.com>]

On 03/26/2015 07:15 PM, Bryan Wu wrote:
> On Thu, Mar 26, 2015 at 7:59 AM, Jacek Anaszewski
> <j.anaszewski@samsung.com> wrote:
>> Duplicated devices are currently given numerical suffixes
>> starting from 1. Since duplication means that one instance
>> of a device already exists in the system, then it is more
>> intuitive for the next device to get the suffix "_2".
>>
>
> No, I prefer to "_1" since normally we treat the first one "_0" or
> empty.

OK, I can live with that :)

> But I suggest we add the "_0" for the first new device then the
> second one will be "_1"

If we changed naming for the device probed as the first one, then
we could break some existing user space applications.

I think that most of time there won't be any device with automagically
added numerical suffix in the LED subsystem, provided that the user
will get acquainted with the DT common leds documentation and will
define labels with different names.

The only case when adding the suffix can't be avoided is the one with
DT overlays.

Adding the suffix should be considered rather a countermeasure than
a convention.


>> Signed-off-by: Jacek Anaszewski <j.anaszewski@samsung.com>
>> Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
>> Cc: Bryan Wu <cooloney@gmail.com>
>> Cc: Richard Purdie <rpurdie@rpsys.net>
>> Cc: Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com>
>> Cc: Sakari Ailus <sakari.ailus@iki.fi>
>> ---
>>   drivers/leds/led-class.c |    4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/leds/led-class.c b/drivers/leds/led-class.c
>> index 4ca37b8..07fcb35 100644
>> --- a/drivers/leds/led-class.c
>> +++ b/drivers/leds/led-class.c
>> @@ -222,14 +222,14 @@ static int match_name(struct device *dev, const void *data)
>>   static int led_classdev_next_name(const char *init_name, char *name,
>>                                    size_t len)
>>   {
>> -       int i = 0;
>> +       int i = 1;
>>
>>          strncpy(name, init_name, len);
>>
>>          while (class_find_device(leds_class, NULL, name, match_name))
>>                  snprintf(name, len, "%s_%d", init_name, ++i);
>>
>> -       return i;
>> +       return i > 1 ? i : 0;
>>   }
>>
>>   /**
>> --
>> 1.7.9.5
>>
>


-- 
Best Regards,
Jacek Anaszewski