Re: [PATCH v1 01/10] arm64: dts: colibri-imx8x: Add usb support

From: Francesco Dolcini
Date: Mon Aug 26 2024 - 18:28:17 EST


On Mon, Aug 26, 2024 at 06:14:58PM -0400, Frank Li wrote:
> On Mon, Aug 26, 2024 at 11:59:13PM +0200, Francesco Dolcini wrote:
> > From: Emanuele Ghidoli <emanuele.ghidoli@xxxxxxxxxxx>
> >
> > Add USB HOST and OTG support to Colibri-iMX8X carrier boards.
> >
> > Signed-off-by: Emanuele Ghidoli <emanuele.ghidoli@xxxxxxxxxxx>
> > Signed-off-by: Francesco Dolcini <francesco.dolcini@xxxxxxxxxxx>
> > ---
> > .../dts/freescale/imx8x-colibri-aster.dtsi | 26 +++++++++++++
> > .../dts/freescale/imx8x-colibri-eval-v3.dtsi | 26 +++++++++++++
> > .../dts/freescale/imx8x-colibri-iris.dtsi | 26 +++++++++++++
> > .../boot/dts/freescale/imx8x-colibri.dtsi | 37 +++++++++++++++++--
> > 4 files changed, 111 insertions(+), 4 deletions(-)

...

> > diff --git a/arch/arm64/boot/dts/freescale/imx8x-colibri.dtsi b/arch/arm64/boot/dts/freescale/imx8x-colibri.dtsi
> > index 49d105eb4769..1199e311d6f9 100644
> > --- a/arch/arm64/boot/dts/freescale/imx8x-colibri.dtsi
> > +++ b/arch/arm64/boot/dts/freescale/imx8x-colibri.dtsi
> > @@ -23,12 +23,31 @@ key-wakeup {
> > };
> > };
> >
> > + extcon_usbc_det: usbc-det {
> > + compatible = "linux,extcon-usb-gpio";
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_usbc_det>;
> > + id-gpio = <&lsio_gpio5 9 GPIO_ACTIVE_HIGH>;
>
> Not sure if I missed something, CHECK_DTBS report below warning.
>
> arch/arm64/boot/dts/freescale/imx8qxp-colibri-iris.dtb: usbc-det: 'id-gpio' does not match any of the regexes: 'pinctrl-[0-9]+'
> from schema $id: http://devicetree.org/schemas/extcon/linux,extcon-usb-gpio.yaml

This should be `id-gpios`. The binding for "linux,extcon-usb-gpio" was recently
added, and this was not tested on the latest next (my fault), but on shawn imx branch and
the old txt doc documented just `id-gpio`.

We'll fix it. And thanks for catching it.


> > + reg_usbh_vbus: regulator-usbh-vbus {
> > + compatible = "regulator-fixed";
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_usbh1_reg>;
> > + gpio = <&lsio_gpio4 3 GPIO_ACTIVE_LOW>;
> > + regulator-always-on;
>
> Needn't regulator-always-on, because reg_usbh_vbus referece by other node.

Is this an issue? we expect this regulator to be always on, therefore this is
described this way in the DT.

Francesco