Re: [PATCH RESEND 02/16] Documentation: dt-bindings: backlight: add TI LMU backlight binding information

Rob Herring
2015-11-03T15:32:52Z

On Tue, Nov 3, 2015 at 1:13 AM, Kim, Milo <milo.kim@ti.com> wrote:
>
> On 11/3/2015 12:02 AM, Rob Herring wrote:
>>
>> On Sun, Nov 1, 2015 at 11:24 PM, Milo Kim <milo.kim@ti.com> wrote:
>>>
>>> LM3532, LM3631, LM3632, LM3633, LM3695 and LM3697 use common dt-bindings
>>> for describing device.
>>>
>>> Cc: devicetree@vger.kernel.org
>>> Cc: Jingoo Han <jingoohan1@gmail.com>
>>> Cc: Lee Jones <lee.jones@linaro.org>
>>> Cc: linux-kernel@vger.kernel.org
>>> Signed-off-by: Milo Kim <milo.kim@ti.com>
>>> ---
>>> .../bindings/video/backlight/ti-lmu-backlight.txt | 67
>>> ++++++++++++++++++++++
>>
>>
>> Please move to bindings/leds/backlight/
>
>
> There are backlight bindings under video/backlight. I'd like to know why

Not if you look in linux-next.

> this 'led' location is preferred. My guess is most of properties are from
> common LED properties. Any other reasons?

Yes, that is the main reason.

Rob