> > 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 maintain > >>> 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@xxxxxxx> > >>> > >>> --- > >>> 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-usb2-imx.yaml > >>> > >>> diff --git a/Documentation/devicetree/bindings/usb/ci-hdrc-usb2-imx.yaml > 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 [...] > > > If I only create imx specific yaml file and remove all compatilbles from common > > 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
