Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   "Y.b. Lu" <yangbo.lu@nxp.com>
To:     Shawn Guo <shawnguo@kernel.org>
CC:     Ashish Kumar <ashish.kumar@nxp.com>,
        Alexandru Marginean <alexandru.marginean@nxp.com>,
        Alison Wang <alison.wang@nxp.com>,
        "Amit Jain (aj)" <amit.jain_1@nxp.com>,
        "catalin.horghidan@nxp.com" <catalin.horghidan@nxp.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        Jiafei Pan <jiafei.pan@nxp.com>, Leo Li <leoyang.li@nxp.com>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linuxppc-dev@lists.ozlabs.org" <linuxppc-dev@lists.ozlabs.org>,
        "mark.rutland@arm.com" <mark.rutland@arm.com>,
        "rajat.srivastava@nxp.com" <rajat.srivastava@nxp.com>,
        Rajesh Bhagat <rajesh.bhagat@nxp.com>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        Vabhav Sharma <vabhav.sharma@nxp.com>,
        Xiaobo Xie <xiaobo.xie@nxp.com>,
        Michael Walle <michael@walle.cc>, Yinbo Zhu <yinbo.zhu@nxp.com>
Subject: RE: [PATCH v1 3/4] arm64: dts: ls1028a: fix little-big endian issue
 for dcfg
Thread-Topic: [PATCH v1 3/4] arm64: dts: ls1028a: fix little-big endian issue
 for dcfg
Thread-Index: AQHVUnFULi52wD5vaE6y0evvPl+NNKezNm2AgAAo2GA=
Date:   Tue, 10 Dec 2019 02:34:30 +0000
Message-ID: <VI1PR0401MB2237D2D6708807511BDB8788F85B0@VI1PR0401MB2237.eurprd04.prod.outlook.com>
References: <20190814072649.8237-3-yinbo.zhu@nxp.com>
 <20191210000623.22321-1-michael@walle.cc>
In-Reply-To: <20191210000623.22321-1-michael@walle.cc>
Accept-Language: en-US
Content-Language: en-US
received-spf: None (protection.outlook.com: nxp.com does not designate
 permitted sender hosts)
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 6c9cbf6e-5ad0-4730-40cf-08d77d197c45
X-MS-Exchange-CrossTenant-originalarrivaltime: 10 Dec 2019 02:34:30.8062
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: inhC5TjVZnnF1gtcBx27H+ZX/fGc5BR/7k90pnVkFrZwV52MJoF8dmaSCnq+Ei1HZksx58hSycKZfxo05BLgpw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0401MB2335
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

+ Shawn,

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

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

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

> > dcfg use little endian that SoC register value will be correct
> >
> > Signed-off-by: Yinbo Zhu <yinbo.zhu@nxp.com>
> > ---
> >  arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
>=20
> This patch is still missing. Any news?
>=20
> Tested-by: Michael Walle <michael@walle.cc>
>=20
> > 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 =3D "fsl,ls1028a-dcfg", "syscon";
> >  			reg =3D <0x0 0x1e00000 0x0 0x10000>;
> > -			big-endian;
> > +			little-endian;
> >  		};
> >
> >  		scfg: syscon@1fc0000 {
> > --
> > 2.17.1
> >