Re: [PATCH 01/10] backlight: Add backlight_device_get_by_name()

Noralf Trønnes
2020-04-30T10:42:25Z

Den 30.04.2020 12.15, skrev Lee Jones:
> On Thu, 30 Apr 2020, Noralf Trønnes wrote:
>
>>
>>
>> Den 30.04.2020 10.32, skrev Lee Jones:
>>> On Wed, 29 Apr 2020, Noralf Trønnes wrote:
>>>
>>>> Add a way to lookup a backlight device based on its name.
>>>> Will be used by a USB display gadget getting the name from configfs.
>>>>
>>>> Cc: Lee Jones <lee.jones@linaro.org>
>>>> Cc: Daniel Thompson <daniel.thompson@linaro.org>
>>>> Cc: Jingoo Han <jingoohan1@gmail.com>
>>>> Signed-off-by: Noralf Trønnes <noralf@tronnes.org>
>>>> ---
>>>> drivers/video/backlight/backlight.c | 21 +++++++++++++++++++++
>>>> include/linux/backlight.h | 1 +
>>>> 2 files changed, 22 insertions(+)
>>>
>>> Once reviewed, can this patch be applied on its own?
>>>
>>
>> If you can apply it for 5.8, then we're good. DRM has cutoff at -rc5 and
>> the driver won't be ready for that. This patch has this dependency
>> chain: usb -> drm -> backlight. So if you can apply it for 5.8, things
>> gets easier.
>>
>>> My guess is that it can't, as the other patches in this set depend on
>>> it, right? If this assumption is true, you need to send me the rest
>>> of the set.
>>>
>>> FYI: It's normally better to send the whole set to everyone, as it
>>> provides visibility on current review (or lack there of) status of the
>>> other patches and allows each of the maintainers to discuss possible
>>> merge strategies.
>>
>> dri-devel is the ML for backlight so I assumed you got the full set.
>
> dri-devel isn't the ML for Backlight. It's an interested party.

Oh, I thought it was strange, but kernel development has all kinds of
things that seems strange to me, so I just went with it.

>
> I certainly have no intention of subscribing to it.
>
>> I have had trouble in the past with my email provider dropping parts of
>> a series when I had to many recipients.
>
> Without visibility into the other patches in the set, things become
> more difficult. Maybe use a different/better email provider.
>

Yeah, you need to have context, I have resent the series to you, Daniel
and Jingoo.

Noralf.