From:   Peng Fan <peng.fan@nxp.com>
To:     Marco Felsch <m.felsch@pengutronix.de>,
        "Peng Fan (OSS)" <peng.fan@oss.nxp.com>
CC:     "shawnguo@kernel.org" <shawnguo@kernel.org>,
        "s.hauer@pengutronix.de" <s.hauer@pengutronix.de>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "krzysztof.kozlowski+dt@linaro.org" 
        <krzysztof.kozlowski+dt@linaro.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        dl-linux-imx <linux-imx@nxp.com>,
        "kernel@pengutronix.de" <kernel@pengutronix.de>,
        "festevam@gmail.com" <festevam@gmail.com>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>
Subject: RE: [PATCH] arm64: dts: imx8mq: update usb compatible string
Thread-Topic: [PATCH] arm64: dts: imx8mq: update usb compatible string
Thread-Index: AQHZNVPZKYj2vaLQBEOJVeHt/zuM7K64QRqAgAACFPA=
Date:   Tue, 31 Jan 2023 09:34:49 +0000
Message-ID: <DU0PR04MB941741A921F3953E165BB45F88D09@DU0PR04MB9417.eurprd04.prod.outlook.com>
References: <20230131091140.3696808-1-peng.fan@oss.nxp.com>
 <20230131092455.2s63zr6d56v2u6ka@pengutronix.de>
In-Reply-To: <20230131092455.2s63zr6d56v2u6ka@pengutronix.de>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?+/reNvXYgWlhriIN/trt/QGEjSWl/j6k/I6a10G7/cKHVSZCohEDGCmvntCu?=
 =?us-ascii?Q?/YnAH9JBnhJm1tPDGmOqKdFHHxT/yiIMzklyx5nZeD45ufehZpPrgICTEXOQ?=
 =?us-ascii?Q?QXTmm7VN1e2tBL2OKN4esFMibo9VNHZg9X9mCnyaA1uGTqimNjnQguF+k1dX?=
 =?us-ascii?Q?7VUSMzRZj9PqFL0sM5KnkjLB9wmVLiK1G/H4nRFCdv61Y3PGm3AapETcM9UJ?=
 =?us-ascii?Q?ktuCz8CvYEQfErw5etU8jFh3BMZ36QNZrKg8Kz3O7flkq5wMNcEVuA4Ve1go?=
 =?us-ascii?Q?1FEixx4v24lyiCa4Gz8goeYDcakfuYSK09dV4Ky5Q0ZH1DwXR7Mbg9IYePuA?=
 =?us-ascii?Q?X6SNxGPIruEzHTGRUOGBycpE3Oi0DfhLmvuIsb3NgtuIHt7M9GuTIX1MoEL4?=
 =?us-ascii?Q?PBepQxSrwUtY9IwWEXmNMF7vQYx2NFKDglCxt35skSGhyDj0jhZNzEyOM+Xh?=
 =?us-ascii?Q?Eo4OZZuS5GDQHrbFKeaNO4iSRBDGPVhbGFl9YfwXKP6YgAsfw98gzbRgRbBR?=
 =?us-ascii?Q?LVa51xh/vit6Gy9eNAVakVuXI8ldy4/cF9MHalkyEuHiox5yIzX/e1l8tr5b?=
 =?us-ascii?Q?Lyx0so7PILp59V+emKmFMcidraYBstB5xmyq/ijFomh++cJ1kEqUIwRZQjZ9?=
 =?us-ascii?Q?kZTd+vQT/8z73pFHWpCS7b/QbJbgPONbOEopJGZgvTTs2XydWk2DHN9fVR6Y?=
 =?us-ascii?Q?HcLs8J2Lv9sJYHD7sQmLKp6K7gSj6c3hd83mU5Fmbkv1yLUt2jSsVQcgy6dt?=
 =?us-ascii?Q?NfUm9Oh4u0SljpXMi5XNlP7LxjJKkRPMFn0YcfUdmWWAVElzK5fQZL/i6VpS?=
 =?us-ascii?Q?ZZXFbIHll7G0NsS52ZToqvJUg6uszwtKExIJM2tGpPDRGQOooLix8LfCTR2r?=
 =?us-ascii?Q?Wnu79zyfLHhTAWQXexOYbq6nmD5HkzmSub7v05jajcjwUwtbDdHYdI4aQ0Dn?=
 =?us-ascii?Q?UYNYm2QUrBbQcTUQrLfx0/mUQOAqe5wiOabM+egZyeWzMUksAo+rTqKC9dd1?=
 =?us-ascii?Q?WnHWUXU8KX0MuFYseFhFQBk4lq8IeU+iTq8ex1j39AdT7SnRB77JDl/aUsNY?=
 =?us-ascii?Q?fDYsYce3WM7XGIYw6PrHrz86FS1LqCMN9wd729UL0z8AtgGUfwU6k+9r2Ao4?=
 =?us-ascii?Q?dtPhthVKpotbaB8wZr6Nir55I1LqPcEdxhdKuKR+wmAsDX5RYqn1D77GVFuG?=
 =?us-ascii?Q?2kJ7kLjQqQXhFWxnyUuxhubuB1yAhNxpEbh5+uixOEBciMU0H77MgYZa6x8W?=
 =?us-ascii?Q?t8Bx/SOI7WchT8kfclmjMT5uj2tEK0v3/DUIgsK376e+AxV26rRHeYZnWiGi?=
 =?us-ascii?Q?GF/yNjVgbve4CXVZm7RmUVq+vtxUAzbZjGF4TBkyEuvTtEs5prFve/Ekn5Tr?=
 =?us-ascii?Q?uw537nhxDcLSWH4nbWBCKgjuyDL3aCddVW78ay410vPEyPkTou9ddrbJOlS3?=
 =?us-ascii?Q?7VWiAgXk9nDM9ItfJhvoR38IexmPw5VMgKQEKrQTlOAfiggZO/sfrmcg+GKu?=
 =?us-ascii?Q?TYVlCvnNVm9WNyxOPYgu/kRA4CuoN7wxywk/7Y5DHs20XbebHIrQ3WHTfmTr?=
 =?us-ascii?Q?WgFbG0VlGCHQNBPr3nw=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DU0PR04MB9417.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 96050593-3d3c-41a0-0ac5-08db036e660f
X-MS-Exchange-CrossTenant-originalarrivaltime: 31 Jan 2023 09:34:49.6448
 (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: QGUi9VGnMmAJ+9pcqlz6k9GOgenj4j14vVMUSmyKrDScArQgaj4XxVGQPFFyXrVDHO1+VleyVbmAMiMdMW8vYA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB6989
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> Subject: Re: [PATCH] arm64: dts: imx8mq: update usb compatible string
>=20
> Hi,
>=20
> On 23-01-31, Peng Fan (OSS) wrote:
> > From: Peng Fan <peng.fan@nxp.com>
> >
> > There is no fsl,imx8mq-dwc3 binding doc, no driver use this compatible
> > string. Drop it.
>=20
> please don't. I'm not aware of the driver state but the compatibles are t=
here
> for _possible_ quirk handling, so the driver can add a match on demand.

NXP downstream also not has driver use this compatible string, and pass
several GA releases.

People could add it back when it is needed together with binding doc.

Thanks,
Peng.

>=20
> Regards,
>   Marco
>=20
>=20
> >
> > Signed-off-by: Peng Fan <peng.fan@nxp.com>
> > ---
> >  arch/arm64/boot/dts/freescale/imx8mq.dtsi | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> > b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> > index d59156fdee0b..d308f94a7c52 100644
> > --- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> > +++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
> > @@ -1431,7 +1431,7 @@ gpu: gpu@38000000 {
> >  		};
> >
> >  		usb_dwc3_0: usb@38100000 {
> > -			compatible =3D "fsl,imx8mq-dwc3", "snps,dwc3";
> > +			compatible =3D "snps,dwc3";
> >  			reg =3D <0x38100000 0x10000>;
> >  			clocks =3D <&clk IMX8MQ_CLK_USB1_CTRL_ROOT>,
> >  			         <&clk IMX8MQ_CLK_USB_CORE_REF>, @@ -
> 1463,7 +1463,7 @@
> > usb3_phy0: usb-phy@381f0040 {
> >  		};
> >
> >  		usb_dwc3_1: usb@38200000 {
> > -			compatible =3D "fsl,imx8mq-dwc3", "snps,dwc3";
> > +			compatible =3D "snps,dwc3";
> >  			reg =3D <0x38200000 0x10000>;
> >  			clocks =3D <&clk IMX8MQ_CLK_USB2_CTRL_ROOT>,
> >  			         <&clk IMX8MQ_CLK_USB_CORE_REF>,
> > --
> > 2.37.1
> >
> >
> >