From: Alexander Stein <alexander.stein@ew.tq-group.com>
To: shawnguo@kernel.org, Fabio Estevam <festevam@gmail.com>
Cc: linux-arm-kernel@lists.infradead.org, tharvey@gateworks.com,
marcel.ziswiler@toradex.com, Fabio Estevam <festevam@denx.de>
Subject: Re: [PATCH 3/3] arm64: dts: imx8mp-tqma8mpql-mba8mpxl: Fix USB connector description
Date: Mon, 11 Dec 2023 08:42:01 +0100 [thread overview]
Message-ID: <2710885.mvXUDI8C0e@steina-w> (raw)
In-Reply-To: <20231207185543.2699697-3-festevam@gmail.com>
Hi Fabio,
Am Donnerstag, 7. Dezember 2023, 19:55:43 CET schrieb Fabio Estevam:
> From: Fabio Estevam <festevam@denx.de>
>
> The USB connector should not be placed under the dwc3 node.
>
> Move the USB connector out of the SoC level and use port to describe
> the connection to the dwc3 controller.
>
> This fixes the following dt-schema warning:
>
> imx8mp-tqma8mpql-mba8mpxl.dtb: usb@38100000: Unevaluated properties are not
> allowed ('connector' was unexpected) from schema $id:
> http://devicetree.org/schemas/usb/snps,dwc3.yaml#
>
> Signed-off-by: Fabio Estevam <festevam@denx.de>
Reviewed-by: Alexander Stein <alexander.stein@ew.tq-group.com>
Tested-by: Alexander Stein <alexander.stein@ew.tq-group.com>
Thanks
Alexander
> ---
> .../freescale/imx8mp-tqma8mpql-mba8mpxl.dts | 26 ++++++++++++++-----
> 1 file changed, 19 insertions(+), 7 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl.dts
> b/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl.dts index
> 4b161564a284..505b70cb2bd7 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl.dts
> @@ -55,6 +55,21 @@ clk_xtal25: clk-xtal25 {
> clock-frequency = <25000000>;
> };
>
> + connector {
> + compatible = "gpio-usb-b-connector", "usb-b-connector";
> + type = "micro";
> + label = "X29";
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_usbcon0>;
> + id-gpios = <&gpio1 10 GPIO_ACTIVE_HIGH>;
> +
> + port {
> + usb_dr_connector: endpoint {
> + remote-endpoint = <&usb3_dwc>;
> + };
> + };
> + };
> +
> fan0: pwm-fan {
> compatible = "pwm-fan";
> pinctrl-names = "default";
> @@ -625,13 +640,10 @@ &usb_dwc3_0 {
> role-switch-default-mode = "peripheral";
> status = "okay";
>
> - connector {
> - compatible = "gpio-usb-b-connector", "usb-b-connector";
> - type = "micro";
> - label = "X29";
> - pinctrl-names = "default";
> - pinctrl-0 = <&pinctrl_usbcon0>;
> - id-gpios = <&gpio1 10 GPIO_ACTIVE_HIGH>;
> + port {
> + usb3_dwc: endpoint {
> + remote-endpoint = <&usb_dr_connector>;
> + };
> };
> };
--
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-group.com/
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
prev parent reply other threads:[~2023-12-11 7:42 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-12-07 18:55 [PATCH 1/3] arm64: dts: imx8mp-verdin: Fix USB connector description Fabio Estevam
2023-12-07 18:55 ` [PATCH 2/3] arm64: dts: imx8mp-venice: " Fabio Estevam
2023-12-13 16:43 ` Tim Harvey
2023-12-07 18:55 ` [PATCH 3/3] arm64: dts: imx8mp-tqma8mpql-mba8mpxl: " Fabio Estevam
2023-12-11 7:42 ` Alexander Stein [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2710885.mvXUDI8C0e@steina-w \
--to=alexander.stein@ew.tq-group.com \
--cc=festevam@denx.de \
--cc=festevam@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=marcel.ziswiler@toradex.com \
--cc=shawnguo@kernel.org \
--cc=tharvey@gateworks.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox