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.
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux