RE: [PATCH v1 3/4] arm64: dts: ls1028a: fix little-big endian issue for dcfg

From: Y.b. Lu
Date: Mon Dec 09 2019 - 21:34:49 EST


+ Shawn,

> -----Original Message-----
> From: Michael Walle <michael@xxxxxxxx>
> Sent: Tuesday, December 10, 2019 8:06 AM
> To: Yinbo Zhu <yinbo.zhu@xxxxxxx>
> Cc: Ashish Kumar <ashish.kumar@xxxxxxx>; Alexandru Marginean
> <alexandru.marginean@xxxxxxx>; Alison Wang <alison.wang@xxxxxxx>;
> Amit Jain (aj) <amit.jain_1@xxxxxxx>; catalin.horghidan@xxxxxxx; Claudiu
> Manoil <claudiu.manoil@xxxxxxx>; devicetree@xxxxxxxxxxxxxxx; Jiafei Pan
> <jiafei.pan@xxxxxxx>; Leo Li <leoyang.li@xxxxxxx>;
> linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
> linuxppc-dev@xxxxxxxxxxxxxxxx; mark.rutland@xxxxxxx;
> rajat.srivastava@xxxxxxx; Rajesh Bhagat <rajesh.bhagat@xxxxxxx>;
> robh+dt@xxxxxxxxxx; Vabhav Sharma <vabhav.sharma@xxxxxxx>; Xiaobo Xie
> <xiaobo.xie@xxxxxxx>; Y.b. Lu <yangbo.lu@xxxxxxx>; Michael Walle
> <michael@xxxxxxxx>
> Subject: Re: [PATCH v1 3/4] arm64: dts: ls1028a: fix little-big endian issue for
> dcfg
>

[Y.b. Lu] Acked-by: Yangbo Lu <yangbo.lu@xxxxxxx>

Hi Shawn, could you help to review and merge the two dts patches of this patch-set?
Thanks.

> > dcfg use little endian that SoC register value will be correct
> >
> > Signed-off-by: Yinbo Zhu <yinbo.zhu@xxxxxxx>
> > ---
> > arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
>
> This patch is still missing. Any news?
>
> Tested-by: Michael Walle <michael@xxxxxxxx>
>
> > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
> b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
> > index b0d4f8916ede..5538e8e354b2 100644
> > --- a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
> > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
> > @@ -162,7 +162,7 @@
> > dcfg: syscon@1e00000 {
> > compatible = "fsl,ls1028a-dcfg", "syscon";
> > reg = <0x0 0x1e00000 0x0 0x10000>;
> > - big-endian;
> > + little-endian;
> > };
> >
> > scfg: syscon@1fc0000 {
> > --
> > 2.17.1
> >