From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH RESEND 02/16] Documentation: dt-bindings: backlight: add TI LMU backlight binding information Date: Tue, 3 Nov 2015 09:32:52 -0600 Message-ID: References: <1446441875-1256-1-git-send-email-milo.kim@ti.com> <1446441875-1256-3-git-send-email-milo.kim@ti.com> <56385E84.2050702@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <56385E84.2050702@ti.com> Sender: linux-kernel-owner@vger.kernel.org To: "Kim, Milo" Cc: "devicetree@vger.kernel.org" , Lee Jones , "linux-kernel@vger.kernel.org" , Jingoo Han List-Id: devicetree@vger.kernel.org On Tue, Nov 3, 2015 at 1:13 AM, Kim, Milo wrote: > > On 11/3/2015 12:02 AM, Rob Herring wrote: >> >> On Sun, Nov 1, 2015 at 11:24 PM, Milo Kim wrote: >>> >>> LM3532, LM3631, LM3632, LM3633, LM3695 and LM3697 use common dt-bindings >>> for describing device. >>> >>> Cc: devicetree@vger.kernel.org >>> Cc: Jingoo Han >>> Cc: Lee Jones >>> Cc: linux-kernel@vger.kernel.org >>> Signed-off-by: Milo Kim >>> --- >>> .../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