Lots of drivers still register with the LED Class using init_data. Let's
provide some additional Kunit infrastructure to exercise the various
possibilities.

Change log:

v1 => v2:
 - Use devm_* to ensure resources are cleaned-up on exit

Lee Jones (5):
  leds: led-test: Move common LED class registration code into helper
    function
  leds: led-test: Provide test for registration with missing
    default_label
  leds: led-test: Provide test for registration with missing devicename
  leds: led-test: Provide test for registration with a name that is too
    long
  leds: led-test: Provide test for successful registration using
    init_data

 drivers/leds/led-test.c | 86 ++++++++++++++++++++++++++++++++++++++---
 1 file changed, 80 insertions(+), 6 deletions(-)

-- 
2.49.0.1101.gccaa498523-goog

Hi Lee,

On 5/15/25 10:27, Lee Jones wrote:
> Lots of drivers still register with the LED Class using init_data. Let's

What do you mean by "still"? Actually init_data is the new approach,
that turns the LED core DT parser on, and is predestined for use with
{devm_}led_classdev_register_ext() API.

> provide some additional Kunit infrastructure to exercise the various
> possibilities.
> 
> Change log:
> 
> v1 => v2:
>   - Use devm_* to ensure resources are cleaned-up on exit
> 
> Lee Jones (5):
>    leds: led-test: Move common LED class registration code into helper
>      function
>    leds: led-test: Provide test for registration with missing
>      default_label
>    leds: led-test: Provide test for registration with missing devicename
>    leds: led-test: Provide test for registration with a name that is too
>      long
>    leds: led-test: Provide test for successful registration using
>      init_data
> 
>   drivers/leds/led-test.c | 86 ++++++++++++++++++++++++++++++++++++++---
>   1 file changed, 80 insertions(+), 6 deletions(-)
> 

-- 
Best regards,
Jacek Anaszewski

On Sun, 18 May 2025, Jacek Anaszewski wrote:

> Hi Lee,
> 
> On 5/15/25 10:27, Lee Jones wrote:
> > Lots of drivers still register with the LED Class using init_data. Let's
> 
> What do you mean by "still"? Actually init_data is the new approach,
> that turns the LED core DT parser on, and is predestined for use with
> {devm_}led_classdev_register_ext() API.

Thanks for the information.  I'll update the wording.

-- 
Lee Jones [李琼斯]

On 5/19/25 15:33, Lee Jones wrote:
> On Sun, 18 May 2025, Jacek Anaszewski wrote:
> 
>> Hi Lee,
>>
>> On 5/15/25 10:27, Lee Jones wrote:
>>> Lots of drivers still register with the LED Class using init_data. Let's
>>
>> What do you mean by "still"? Actually init_data is the new approach,
>> that turns the LED core DT parser on, and is predestined for use with
>> {devm_}led_classdev_register_ext() API.
> 
> Thanks for the information.  I'll update the wording.

Other than that, for the whole series:

Reviewed-by: Jacek Anaszewski <jacek.anaszewski@gmail.com>

-- 
Best regards,
Jacek Anaszewski