Received-SPF: pass (google.com: domain of linux-kernel+bounces-80470-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
From: Xu Yang <xu.yang_2@nxp.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>, Rob Herring
	<robh@kernel.org>
CC: "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
	"krzysztof.kozlowski+dt@linaro.org" <krzysztof.kozlowski+dt@linaro.org>,
	"shawnguo@kernel.org" <shawnguo@kernel.org>, "conor+dt@kernel.org"
	<conor+dt@kernel.org>, "s.hauer@pengutronix.de" <s.hauer@pengutronix.de>,
	"kernel@pengutronix.de" <kernel@pengutronix.de>, "festevam@gmail.com"
	<festevam@gmail.com>, dl-linux-imx <linux-imx@nxp.com>,
	"peter.chen@kernel.org" <peter.chen@kernel.org>, Jun Li <jun.li@nxp.com>,
	"linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
	"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>
Subject: RE: [EXT] Re: [PATCH v6 4/9] dt-bindings: usb: add NXP ChipIdea USB2
 Controller schema
Thread-Topic: [EXT] Re: [PATCH v6 4/9] dt-bindings: usb: add NXP ChipIdea USB2
 Controller schema
Thread-Index: AQHaZJJxwpJRCpYUm0K+YwxPJkmdNbEX+NsAgAAEwLCAAWvdgIACkzrA
Date: Mon, 26 Feb 2024 03:30:16 +0000
Message-ID:
 <DU2PR04MB8822BF43FA0CF416EB72F19A8C5A2@DU2PR04MB8822.eurprd04.prod.outlook.com>
References: <20240221145846.1611627-1-xu.yang_2@nxp.com>
 <20240221145846.1611627-4-xu.yang_2@nxp.com>
 <20240223140256.GA1768266-robh@kernel.org>
 <DU2PR04MB882234D1A9D5718D8355928B8C552@DU2PR04MB8822.eurprd04.prod.outlook.com>
 <6c5c0233-dab8-4bd1-9f08-0b609d01defd@linaro.org>
In-Reply-To: <6c5c0233-dab8-4bd1-9f08-0b609d01defd@linaro.org>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0:
 =?us-ascii?Q?32OWii7duQLMigDwrGO8ixY5OtAXJpSw1BGernbEKBBvT7Q09aNebrY7kO4J?=
 =?us-ascii?Q?O0HPVRIbqpnUjEKTdr81dNXOrv+m76BG9gp2YFe4slaqVk7K3aVTaaZLzxpH?=
 =?us-ascii?Q?b5XYeEoHFcaXm3tcPx8uTcp3sXQDF3DTI4Rr0brVxAghqrS66eKSDVEKuTSC?=
 =?us-ascii?Q?1Ol4VbwNjd2nZ/kTteDoujaJBhe3SQ3/O6k6qPv7aewO4wbvUPJIeqarjReo?=
 =?us-ascii?Q?fLZtHlEbyX3/DJNRi6A79+ycL4ljga+DDQoP5r0WlfONCknq/8NKkh2iUI2z?=
 =?us-ascii?Q?ILILnu08k2lKSt2SKgQxEAp9+n0Coi91ljc1xYAla1jmz6fXBNC+hNJjFA/Z?=
 =?us-ascii?Q?6BFvx7+8dS7TAone1dZuKEaNNwULUEtxN1xtK/KZbdf5/3yTTReQVV+F3tHO?=
 =?us-ascii?Q?iaEJz5nZGL/o81zfMdszSefYDhbsiippYQJLtX+5jUkvnI41Bgfvq4Kds9LV?=
 =?us-ascii?Q?5RkZ/n+BM8fKdonadoLPekI7gAiyepCdrlNvlj1huJ56ypZhcmmZmk6hCglL?=
 =?us-ascii?Q?QEgeTI/PuMSUaQCToJhqS0fBSlNyh4qRWbpNsubJpOX/N6JyZRTfENXwgrGf?=
 =?us-ascii?Q?4Eibzncu1DPyff4cChr+mYrVwwWI1V6JOacWwmwE441oFrRS27tLr8yzMYhB?=
 =?us-ascii?Q?wKFUD4gXHmmcDneDTSsQc5+UcgncYe8SDraRC7P4Oxs05oMbG9PljMlfBaLK?=
 =?us-ascii?Q?0vWIuv0rHjpENi3lq2iSy/sBNVu5ZxWfbJexP+AvU46EOsfqMv5H9V5GlICl?=
 =?us-ascii?Q?wbODkLHr3bnj+7My2TlU4UQEqG0ZNIU4WM3b13Rs7olDkASXpIsHjxlXKVu4?=
 =?us-ascii?Q?bJVwePcT+IBOrB8GsfsLh6EVGmLXVAjSo14VTVuWi8t/8O+gY7Njp6q3gfuY?=
 =?us-ascii?Q?EifwjhRNRNMcgD3lmEGoHPRy98SmUOhHnmt/4yJnp28c3D7BfQIPQhOmdDFh?=
 =?us-ascii?Q?3D+jteWSCphnS0/OXYZt+PMBVbIHDPombyMejK3z8/dStDFdq+mwZnrH5iF5?=
 =?us-ascii?Q?avoUH9cCAjn1jRn0iiagwKDgCMNOaljephjhKt6L3JKT04RXhcIaHCFPUZo1?=
 =?us-ascii?Q?rRfAcrB96KAOy0zVhrUXoPa8GUFjNtcBqsCHb+RhJpBMYbMZXfiYVI+4DQ4N?=
 =?us-ascii?Q?DJZEnTGvIck4X59cBMcc5ueqiC94xuTR1e/tnavTRjy83N3qgW9GSs8svU+h?=
 =?us-ascii?Q?nzq70gNq7YufSetVMeQB0bHeRYu+twuDT2E+sww7pESqkSnNB6sXj6MSNC88?=
 =?us-ascii?Q?voRpTho6XipB99lBvLPaKRbkaIl6e062jcRFFIcuJCPnOr9fmege6owm0TlE?=
 =?us-ascii?Q?S6G5gyIKuK1sr0KaTQ8T76Y1tPPPErxY8sJYLa4SoxM9k4aZcIKTBYlXoQfy?=
 =?us-ascii?Q?wxCMujI1SRUSdKQ9soH4EiH06AwIQoL9zeLb0t/UH42ZsCBP6QdyTa7XAABC?=
 =?us-ascii?Q?AB49ulqlsiBogOI3BuzoQYs3ljIotAlhSZ1SrpVjs998+WHdlo5nWSGUZYq3?=
 =?us-ascii?Q?IVC6DlNMvbGNUvmsffdFkH6L3Gn3d21D2HShEg9sPI+PuoAEz8OZopbsapqR?=
 =?us-ascii?Q?zhs6qf9//NmaVEcop1g=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DU2PR04MB8822.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 0398ddff-7f19-4a23-239f-08dc367b4066
X-MS-Exchange-CrossTenant-originalarrivaltime: 26 Feb 2024 03:30:16.8575
 (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: dyxv7P3DZviGqxz7RwUEMLDpgzbuwV2rgvlqk0imhGVBV9Xr2HMQJcBwWpAQFtqak75Bt2Hnuw4RXoZVToroHA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBBPR04MB7562


>=20
> On 23/02/2024 15:56, Xu Yang wrote:
> > Hi Rob,
> >
> >>
> >> On Wed, Feb 21, 2024 at 10:58:41PM +0800, Xu Yang wrote:
> >>> As more and more NXP i.MX chips come out, it becomes harder to mainta=
in
> >>> ci-hdrc-usb2.yaml if more stuffs like property restrictions are added=
 to
> >>> this file. This will separate i.MX parts out of ci-hdrc-usb2.yaml and=
 add
> >>> a new schema for NXP ChipIdea USB2 Controller.
> >>>
> >>> Signed-off-by: Xu Yang <xu.yang_2@nxp.com>
> >>>
> >>> ---
> >>> Changes in v6:
> >>>  - new patch
> >>> ---
> >>>  .../bindings/usb/ci-hdrc-usb2-imx.yaml        | 75 +++++++++++++++++=
++
> >>>  1 file changed, 75 insertions(+)
> >>>  create mode 100644 Documentation/devicetree/bindings/usb/ci-hdrc-usb=
2-imx.yaml
> >>>
> >>> diff --git a/Documentation/devicetree/bindings/usb/ci-hdrc-usb2-imx.y=
aml
> b/Documentation/devicetree/bindings/usb/ci-
> >> hdrc-usb2-imx.yaml
> >>> new file mode 100644
> >>> index 000000000000..2ec62f564bf5
> >>> --- /dev/null
> >>> +++ b/Documentation/devicetree/bindings/usb/ci-hdrc-usb2-imx.yaml
> >>> @@ -0,0 +1,75 @@
> >>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> >>> +%YAML 1.2

[...]

>=20
> > If I only create imx specific yaml file and remove all compatilbles fro=
m common
>=20
> Why would you remove ALL of others? What is the point of this.

I originally thought ci-hdrc-usb2.yaml should be removed and only
ci-hdrc-usb2-comm.yaml is kept. So I'm not sure where other compatibles
should go. I looked though your Qualcomm PCI dt-bindings series and now
I know how to do.

Thanks,
Xu Yang