diff mbox series

[v8,01/10] dt-bindings: usb: usbmisc-imx: add fsl,imx8ulp-usbmisc compatible

Message ID 20240312091703.1220649-1-xu.yang_2@nxp.com
State Superseded
Headers show
Series [v8,01/10] dt-bindings: usb: usbmisc-imx: add fsl,imx8ulp-usbmisc compatible | expand

Commit Message

Xu Yang March 12, 2024, 9:16 a.m. UTC
Add "fsl,imx8ulp-usbmisc" compatible.

Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: Xu Yang <xu.yang_2@nxp.com>

---
Changes in v3:
 - new patch due to missed this little one
Changes in v4:
 - no changes
Changes in v5:
 - add Acked-by tag
Changes in v6:
 - no changes
Changes in v7:
 - no changes
Changes in v8:
 - no changes
---
 Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml | 1 +
 1 file changed, 1 insertion(+)

Comments

Rob Herring (Arm) March 12, 2024, 2:33 p.m. UTC | #1
On Tue, 12 Mar 2024 17:16:54 +0800, Xu Yang wrote:
> Add "fsl,imx8ulp-usbmisc" compatible.
> 
> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Signed-off-by: Xu Yang <xu.yang_2@nxp.com>
> 
> ---
> Changes in v3:
>  - new patch due to missed this little one
> Changes in v4:
>  - no changes
> Changes in v5:
>  - add Acked-by tag
> Changes in v6:
>  - no changes
> Changes in v7:
>  - no changes
> Changes in v8:
>  - no changes
> ---
>  Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 


My bot found new DTB warnings on the .dts files added or changed in this
series.

Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings
are fixed by another series. Ultimately, it is up to the platform
maintainer whether these warnings are acceptable or not. No need to reply
unless the platform maintainer has comments.

If you already ran DT checks and didn't see these error(s), then
make sure dt-schema is up to date:

  pip3 install dtschema --upgrade


New warnings running 'make CHECK_DTBS=y freescale/imx8ulp-evk.dtb freescale/imx93-11x11-evk.dtb' for 20240312091703.1220649-1-xu.yang_2@nxp.com:

arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb: tcpc@50: compatible: ['nxp,ptn5110'] is too short
	from schema $id: http://devicetree.org/schemas/usb/nxp,ptn5110.yaml#
arch/arm64/boot/dts/freescale/imx93-11x11-evk.dtb: tcpc@51: compatible: ['nxp,ptn5110'] is too short
	from schema $id: http://devicetree.org/schemas/usb/nxp,ptn5110.yaml#
Rob Herring (Arm) March 12, 2024, 2:51 p.m. UTC | #2
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@nxp.com>
> 
> ---
> 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@kernel.org>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml b/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
index 2d3589d284b2..0a6e7ac1b37e 100644
--- a/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
+++ b/Documentation/devicetree/bindings/usb/fsl,usbmisc.yaml
@@ -33,6 +33,7 @@  properties:
               - fsl,imx7ulp-usbmisc
               - fsl,imx8mm-usbmisc
               - fsl,imx8mn-usbmisc
+              - fsl,imx8ulp-usbmisc
           - const: fsl,imx7d-usbmisc
           - const: fsl,imx6q-usbmisc
       - items: