On 09/02/2017 08:48 AM, Scott Branden wrote: > Looks good > > On Sep 1, 2017 7:10 PM, "Florian Fainelli" <f.fainelli@xxxxxxxxx > <mailto:f.fainelli@xxxxxxxxx>> wrote: > > > > On 08/31/2017 01:16 PM, Scott Branden wrote: > > Hi Eric, > > > > mode is correct, location in file needs to be moved. > > > > > > On 17-08-31 11:54 AM, Eric Anholt wrote: > >> This doesn't yet enable it on any particular platform, as we still > >> need a panel driver for bcm911360_entphn. > >> > >> Signed-off-by: Eric Anholt <eric@xxxxxxxxxx <mailto:eric@xxxxxxxxxx>> > >> --- > >> > >> These bits are just carving off a little bit of my 911360_entphn > panel > >> series, to reduce conflicts when rebasing (which I just did for > >> testing pl111 changes for cygnus regressions). I'm waiting to get my > >> current RPi panel driver in before working on the 911360 panel again. > >> > >> arch/arm/boot/dts/bcm-cygnus.dtsi | 10 ++++++++++ > >> 1 file changed, 10 insertions(+) > >> > >> diff --git a/arch/arm/boot/dts/bcm-cygnus.dtsi > >> b/arch/arm/boot/dts/bcm-cygnus.dtsi > >> index 7c957ea06c66..74f73ff24aec 100644 > >> --- a/arch/arm/boot/dts/bcm-cygnus.dtsi > >> +++ b/arch/arm/boot/dts/bcm-cygnus.dtsi > >> @@ -575,6 +575,16 @@ > >> status = "disabled"; > >> }; > >> + clcd: clcd@180a0000 { > > please place in correct address ordered location in file > > I moved it right above the v3d node to keep the nodes ordered by unit > address, please check the result here: > > https://github.com/Broadcom/stblinux/commit/4d1e42c6b9d28ce7b74d92258435f9d16834ae75 > <https://github.com/Broadcom/stblinux/commit/4d1e42c6b9d28ce7b74d92258435f9d16834ae75> > > > With Moved location > Acked-by: Scott Branden <scott.branden@xxxxxxxxxxxx > <mailto:scott.branden@xxxxxxxxxxxx>> Pushed in place with your tag added, thanks! -- Florian -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html