Re: [PATCH 4/5] arm64: dts: renesas: r8a77995: draak: Add backlight

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Dec 10, 2018 at 02:33:08PM +0200, Laurent Pinchart wrote:
> Hi Geert,
> 
> On Monday, 10 December 2018 14:30:22 EET Geert Uytterhoeven wrote:
> > On Tue, Dec 4, 2018 at 6:36 PM Geert Uytterhoeven wrote:
> > > On Sun, Nov 25, 2018 at 3:40 PM Laurent Pinchart wrote:
> > >> Add the backlight device for the LVDS1 output, in preparation for panel
> > >> support.
> > >> 
> > >> Signed-off-by: Laurent Pinchart
> > >> <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx>
> > > 
> > > Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> > 
> > Oops, seems I missed the backlight node should be moved up, to preserve
> > sort order.
> 
> I assumed that aliases and chosen should be kept at the top of the file.
> Maybe we don't want to keep the tradition :-)

There is precedence for this tradition in salvator-common.dtsi.
I am ambivalent at this point.



[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux