Re: [PATCH] dt-bindings: usb: ehci: Increase the number of PHYs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Rob,

On Wed, Jun 15, 2022 at 7:24 PM Rob Herring <robh+dt@xxxxxxxxxx> wrote:
> On Wed, Jun 15, 2022 at 7:53 AM Geert Uytterhoeven
> <geert+renesas@xxxxxxxxx> wrote:
> >
> > "make dtbs_check":
> >
> >     arch/arm/boot/dts/r8a77470-iwg23s-sbc.dtb: usb@ee080100: phys: [[17, 0], [31]] is too long
> >             From schema: Documentation/devicetree/bindings/usb/generic-ehci.yaml
> >     arch/arm/boot/dts/r8a77470-iwg23s-sbc.dtb: usb@ee0c0100: phys: [[17, 1], [33], [21, 0]] is too long
> >             From schema: Documentation/devicetree/bindings/usb/generic-ehci.yaml
> >
> > Some USB EHCI controllers (e.g. on the Renesas RZ/G1C SoC) have multiple
> > PHYs.  Increase the maximum number of PHYs to 3, which is sufficient for
> > now.
>
> Maybe not so 'generic'. Why are there 3? I understand USB3 HCI having
> 2 for SS and HS. Is it for multiple ports? There's a thread about that
> currently (Mediatek IIRC) and how to handle multiple phys.

TBH, I don't know for sure. I've never been much into USB, and the
RZ/G1C USB architecture differs from other Renesas SoCs in the
same family.

This is USB2, not USB3.  There are two channels of OHCI/EHCI, and
two channels of USB OTG, all with corresponding PHYs (4 in total).
The first OHCI/EHCI combo is linked to one of the host PHYs, and one
of the OTG PHYs, so that makes 2 PHYs.
The second OHCI/EHCI combo is linked to the other host PHY, and to
both OTG PHYs, so that makes 3 PHYs. IIRC the third link was due to
some dependency between the 2 OTG PHYs.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux