Re: [PATCH v8 04/10] dt-bindings: usb: chipidea,usb2-imx: move imx parts to dedicated schema

From: Rob Herring
Date: Tue Mar 12 2024 - 10:51:42 EST



On Tue, 12 Mar 2024 17:16:57 +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, also add a common schema.
>
> 1. Copy common ci-hdrc-usb2.yaml properties to a new shared
> chipidea,usb2-common.yaml schema.
> 2. Move fsl,* compatible devices and imx spefific properties
> to dedicated binding file chipidea,usb2-imx.yaml.
>
> Signed-off-by: Xu Yang <xu.yang_2@xxxxxxx>
>
> ---
> Changes in v6:
> - new patch
> Changes in v7:
> - not remove ci-hdrc-usb2.yaml and move imx parts to ci-hdrc-usb2-imx.yaml
> Changes in v8:
> - rename yaml name to chipidea,usb2-common.yaml and chipidea,usb2-imx.yaml
> - remove common properties from ci-hdrc-usb2.yaml
> - add new end line to chipidea,usb2-common.yaml
> - remove fsl,anatop since it's not needed by controller
> - fix pinctrl-names formats
> ---
> .../bindings/usb/chipidea,usb2-common.yaml | 200 ++++++++++
> .../bindings/usb/chipidea,usb2-imx.yaml | 193 ++++++++++
> .../devicetree/bindings/usb/ci-hdrc-usb2.yaml | 360 +-----------------
> 3 files changed, 396 insertions(+), 357 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/usb/chipidea,usb2-common.yaml
> create mode 100644 Documentation/devicetree/bindings/usb/chipidea,usb2-imx.yaml
>

Reviewed-by: Rob Herring <robh@xxxxxxxxxx>