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:   Peng Fan <peng.fan@nxp.com>
To:     Rob Herring <robh@kernel.org>
CC:     "mark.rutland@arm.com" <mark.rutland@arm.com>,
        "shawnguo@kernel.org" <shawnguo@kernel.org>,
        "kernel@pengutronix.de" <kernel@pengutronix.de>,
        "festevam@gmail.com" <festevam@gmail.com>,
        dl-linux-imx <linux-imx@nxp.com>,
        Aisheng Dong <aisheng.dong@nxp.com>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Alice Guo <alice.guo@nxp.com>
Subject: RE: [PATCH] ARM: dts: imx: drop "fsl,aips-bus"
Thread-Topic: [PATCH] ARM: dts: imx: drop "fsl,aips-bus"
Thread-Index: AQHVvu9AXeMfpR61Rke6CNGr+daIZKfZsCkAgANF9OA=
Date:   Mon, 6 Jan 2020 02:38:18 +0000
Message-ID: <AM0PR04MB4481917D6A969053CB85276F883C0@AM0PR04MB4481.eurprd04.prod.outlook.com>
References: <1577696078-21720-1-git-send-email-peng.fan@nxp.com>
 <20200104003634.GA6058@bogus>
In-Reply-To: <20200104003634.GA6058@bogus>
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: 00c7e4fc-4ba0-4415-da5c-08d792517d4b
X-MS-Exchange-CrossTenant-originalarrivaltime: 06 Jan 2020 02:38:18.7104
 (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: QJ+C0IEYdTEE8+dijL6O0ivqG2h76WQnDnylZtiRIg/ml6v2IejhJqXo6uZV3zsbhSJ7un/S+zQanZ9gv489GQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR04MB5682
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Rob,

> Subject: Re: [PATCH] ARM: dts: imx: drop "fsl,aips-bus"
>=20
> On Mon, Dec 30, 2019 at 08:58:05AM +0000, Peng Fan wrote:
> > From: Peng Fan <peng.fan@nxp.com>
> >
> > There is no binding doc for "fsl,aips-bus", "simple-bus" is enough for
> > aips usage, so drop it.
>=20
> NAK. The AIPS bus has registers, so 'simple-bus' alone is not enough.

You mean the "reg" property below, right?
                aips-bus@2000000 { /* AIPS1 */
-                       compatible =3D "fsl,aips-bus", "simple-bus";
+                       compatible =3D "simple-bus";
                        #address-cells =3D <1>;
                        #size-cells =3D <1>;
                        reg =3D <0x02000000 0x100000>;

But the reg property is not required, I think it could be removed.
There is no binding doc and driver for "fsl,aips-bus", so I think
It not make sense to have that compatible in dts.

>=20
> What you should do is change 'aips' node names to 'bus'.

How do you think about below change, with reg removed?
                aips-bus@2000000 { /* AIPS1 */
-                       compatible =3D "fsl,aips-bus", "simple-bus";
+                       compatible =3D "simple-bus";
                        #address-cells =3D <1>;
                        #size-cells =3D <1>;
-                       reg =3D <0x02000000 0x100000>;

Thanks,
Peng.
>=20
> >
> > Scirpt:
> > sed -i 's/compatible =3D "fsl,aips-bus", "simple-bus";/compatible =3D
> "simple-bus";/'
> > arch/arm/boot/dts/imx*
> >
> > Signed-off-by: Peng Fan <peng.fan@nxp.com>
> > ---
> >  arch/arm/boot/dts/imx25.dtsi   | 4 ++--
> >  arch/arm/boot/dts/imx31.dtsi   | 4 ++--
> >  arch/arm/boot/dts/imx50.dtsi   | 4 ++--
> >  arch/arm/boot/dts/imx51.dtsi   | 4 ++--
> >  arch/arm/boot/dts/imx53.dtsi   | 4 ++--
> >  arch/arm/boot/dts/imx6qdl.dtsi | 4 ++--
> > arch/arm/boot/dts/imx6sl.dtsi  | 4 ++--
> > arch/arm/boot/dts/imx6sll.dtsi | 4 ++--  arch/arm/boot/dts/imx6sx.dtsi
> > | 6 +++---  arch/arm/boot/dts/imx6ul.dtsi  | 4 ++--
> > arch/arm/boot/dts/imx6ull.dtsi | 2 +-
> >  arch/arm/boot/dts/imx7s.dtsi   | 6 +++---
> >  12 files changed, 25 insertions(+), 25 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/imx25.dtsi
> > b/arch/arm/boot/dts/imx25.dtsi index 40b95a290bd6..3b7a0b249d80
> 100644
> > --- a/arch/arm/boot/dts/imx25.dtsi
> > +++ b/arch/arm/boot/dts/imx25.dtsi
> > @@ -76,7 +76,7 @@
> >  		ranges;
> >
> >  		aips@43f00000 { /* AIPS1 */
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x43f00000 0x100000>;
> > @@ -333,7 +333,7 @@
> >  		};
> >
> >  		aips@53f00000 { /* AIPS2 */
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x53f00000 0x100000>;
> > diff --git a/arch/arm/boot/dts/imx31.dtsi
> > b/arch/arm/boot/dts/imx31.dtsi index 6b62f0745b82..b0e7e3bf8a1a
> 100644
> > --- a/arch/arm/boot/dts/imx31.dtsi
> > +++ b/arch/arm/boot/dts/imx31.dtsi
> > @@ -64,7 +64,7 @@
> >  		};
> >
> >  		aips@43f00000 { /* AIPS1 */
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x43f00000 0x100000>;
> > @@ -226,7 +226,7 @@
> >  		};
> >
> >  		aips@53f00000 { /* AIPS2 */
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x53f00000 0x100000>;
> > diff --git a/arch/arm/boot/dts/imx50.dtsi
> > b/arch/arm/boot/dts/imx50.dtsi index 0bfe7c91d0eb..961de09b571d
> 100644
> > --- a/arch/arm/boot/dts/imx50.dtsi
> > +++ b/arch/arm/boot/dts/imx50.dtsi
> > @@ -102,7 +102,7 @@
> >  		ranges;
> >
> >  		aips@50000000 { /* AIPS1 */
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x50000000 0x10000000>;
> > @@ -390,7 +390,7 @@
> >  		};
> >
> >  		aips@60000000 {	/* AIPS2 */
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x60000000 0x10000000>;
> > diff --git a/arch/arm/boot/dts/imx51.dtsi
> > b/arch/arm/boot/dts/imx51.dtsi index dea86b98e9c3..86708688371b
> 100644
> > --- a/arch/arm/boot/dts/imx51.dtsi
> > +++ b/arch/arm/boot/dts/imx51.dtsi
> > @@ -159,7 +159,7 @@
> >  		};
> >
> >  		aips@70000000 { /* AIPS1 */
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x70000000 0x10000000>;
> > @@ -441,7 +441,7 @@
> >  		};
> >
> >  		aips@80000000 {	/* AIPS2 */
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x80000000 0x10000000>;
> > diff --git a/arch/arm/boot/dts/imx53.dtsi
> > b/arch/arm/boot/dts/imx53.dtsi index ed341cfd9d09..f46a83c7d2c4
> 100644
> > --- a/arch/arm/boot/dts/imx53.dtsi
> > +++ b/arch/arm/boot/dts/imx53.dtsi
> > @@ -223,7 +223,7 @@
> >  		};
> >
> >  		aips@50000000 { /* AIPS1 */
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x50000000 0x10000000>;
> > @@ -655,7 +655,7 @@
> >  		};
> >
> >  		aips@60000000 {	/* AIPS2 */
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x60000000 0x10000000>;
> > diff --git a/arch/arm/boot/dts/imx6qdl.dtsi
> > b/arch/arm/boot/dts/imx6qdl.dtsi index e6b4b8525f98..9b7635e9cf3c
> > 100644
> > --- a/arch/arm/boot/dts/imx6qdl.dtsi
> > +++ b/arch/arm/boot/dts/imx6qdl.dtsi
> > @@ -295,7 +295,7 @@
> >  		};
> >
> >  		aips-bus@2000000 { /* AIPS1 */
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x02000000 0x100000>;
> > @@ -936,7 +936,7 @@
> >  		};
> >
> >  		aips-bus@2100000 { /* AIPS2 */
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x02100000 0x100000>;
> > diff --git a/arch/arm/boot/dts/imx6sl.dtsi
> > b/arch/arm/boot/dts/imx6sl.dtsi index 59c54e6ad09a..4b4f22217dfe
> > 100644
> > --- a/arch/arm/boot/dts/imx6sl.dtsi
> > +++ b/arch/arm/boot/dts/imx6sl.dtsi
> > @@ -144,7 +144,7 @@
> >  		};
> >
> >  		aips1: aips-bus@2000000 {
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x02000000 0x100000>;
> > @@ -787,7 +787,7 @@
> >  		};
> >
> >  		aips2: aips-bus@2100000 {
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x02100000 0x100000>;
> > diff --git a/arch/arm/boot/dts/imx6sll.dtsi
> > b/arch/arm/boot/dts/imx6sll.dtsi index a1bc5bb31756..fac8f22255aa
> > 100644
> > --- a/arch/arm/boot/dts/imx6sll.dtsi
> > +++ b/arch/arm/boot/dts/imx6sll.dtsi
> > @@ -145,7 +145,7 @@
> >  		};
> >
> >  		aips1: aips-bus@2000000 {
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x02000000 0x100000>;
> > @@ -664,7 +664,7 @@
> >  		};
> >
> >  		aips2: aips-bus@2100000 {
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x02100000 0x100000>;
> > diff --git a/arch/arm/boot/dts/imx6sx.dtsi
> > b/arch/arm/boot/dts/imx6sx.dtsi index 59bad60a47dc..4499be62c8bb
> > 100644
> > --- a/arch/arm/boot/dts/imx6sx.dtsi
> > +++ b/arch/arm/boot/dts/imx6sx.dtsi
> > @@ -236,7 +236,7 @@
> >  		};
> >
> >  		aips1: aips-bus@2000000 {
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x02000000 0x100000>;
> > @@ -831,7 +831,7 @@
> >  		};
> >
> >  		aips2: aips-bus@2100000 {
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x02100000 0x100000>;
> > @@ -1189,7 +1189,7 @@
> >  		};
> >
> >  		aips3: aips-bus@2200000 {
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x02200000 0x100000>;
> > diff --git a/arch/arm/boot/dts/imx6ul.dtsi
> > b/arch/arm/boot/dts/imx6ul.dtsi index d9fdca12819b..63d276fc2477
> > 100644
> > --- a/arch/arm/boot/dts/imx6ul.dtsi
> > +++ b/arch/arm/boot/dts/imx6ul.dtsi
> > @@ -205,7 +205,7 @@
> >  		};
> >
> >  		aips1: aips-bus@2000000 {
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x02000000 0x100000>;
> > @@ -772,7 +772,7 @@
> >  		};
> >
> >  		aips2: aips-bus@2100000 {
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x02100000 0x100000>;
> > diff --git a/arch/arm/boot/dts/imx6ull.dtsi
> > b/arch/arm/boot/dts/imx6ull.dtsi index b7e67d121322..633fa08bc972
> > 100644
> > --- a/arch/arm/boot/dts/imx6ull.dtsi
> > +++ b/arch/arm/boot/dts/imx6ull.dtsi
> > @@ -52,7 +52,7 @@
> >  / {
> >  	soc {
> >  		aips3: aips-bus@2200000 {
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x02200000 0x100000>;
> > diff --git a/arch/arm/boot/dts/imx7s.dtsi
> > b/arch/arm/boot/dts/imx7s.dtsi index 139ab9b98472..552b14be14a1
> 100644
> > --- a/arch/arm/boot/dts/imx7s.dtsi
> > +++ b/arch/arm/boot/dts/imx7s.dtsi
> > @@ -317,7 +317,7 @@
> >  		};
> >
> >  		aips1: aips-bus@30000000 {
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x30000000 0x400000>;
> > @@ -669,7 +669,7 @@
> >  		};
> >
> >  		aips2: aips-bus@30400000 {
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x30400000 0x400000>;
> > @@ -809,7 +809,7 @@
> >  		};
> >
> >  		aips3: aips-bus@30800000 {
> > -			compatible =3D "fsl,aips-bus", "simple-bus";
> > +			compatible =3D "simple-bus";
> >  			#address-cells =3D <1>;
> >  			#size-cells =3D <1>;
> >  			reg =3D <0x30800000 0x400000>;
> > --
> > 2.16.4
> >