Hi,
On 12/16/22 14:54, Andy Shevchenko wrote:
> On Fri, Dec 16, 2022 at 12:30:04PM +0100, Hans de Goede wrote:
>> Split out part of of_led_get() into a generic __led_get() helper function.
>>
>> This is a preparation patch for adding a generic (non devicetree specific)
>> led_get() function.
>
> ...
>
>> +static struct led_classdev *__led_get(struct device *led_dev)
>> +{
>> + struct led_classdev *led_cdev;
>> +
>> + if (!led_dev)
>> + return ERR_PTR(-EPROBE_DEFER);
>> +
>> + led_cdev = dev_get_drvdata(led_dev);
>> +
>> + if (!try_module_get(led_cdev->dev->parent->driver->owner)) {
>> + put_device(led_cdev->dev);
>> + return ERR_PTR(-ENODEV);
>> + }
>> +
>> + return led_cdev;
>> +}
>
> If I'm not mistaken, the entire series leaves this function as is.
> At the end I found three _get functions:
> __led_get
> led_get
> __devm_led_get
>
>
> As far as I can see the above can be named more precisely, i.e.
> led_module_get(). Or alike if this sounds not good either.
led_module_get() sounds good, I will rename this for v4.
Regards,
Hans
