Messages in this thread | | | From | Anand Moon <> | Date | Wed, 28 Dec 2022 15:59:17 +0530 | Subject | Re: [PATCH v1 01/11] dt-bindings: usb: Add device id for Genesys Logic hub controller |
| |
Hi Icenowy,
Thanks for the review comments
On Wed, 28 Dec 2022 at 15:38, Icenowy Zheng <uwu@icenowy.me> wrote: > > 在 2022-12-28星期三的 10:03 +0000,Anand Moon写道: > > Add usb hub device id for Genesys Logic, Inc. GL852G-OHG Hub USB 2.0 > > root hub and Genesys Logic, Inc. GL3523-QFN76 USB 3.1 root hub. > > > > Signed-off-by: Anand Moon <linux.amoon@gmail.com> > > --- > > Documentation/devicetree/bindings/usb/genesys,gl850g.yaml | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git > > a/Documentation/devicetree/bindings/usb/genesys,gl850g.yaml > > b/Documentation/devicetree/bindings/usb/genesys,gl850g.yaml > > index a9f831448cca..db009f3ef438 100644 > > --- a/Documentation/devicetree/bindings/usb/genesys,gl850g.yaml > > +++ b/Documentation/devicetree/bindings/usb/genesys,gl850g.yaml > > @@ -16,6 +16,8 @@ properties: > > compatible: > > enum: > > - usb5e3,608 > > + - genesys,usb5e3,610 > > + - genesys,usb5e3,620 > > I don't think genesys, is needed here because usb5e3 means USB VID > 0x05e3, which is already linked to Genesys Logic. >
Ok, I added this as genesys, is it part of the manufacturer or vendor name which is most commonly used for adding vendor-specific compatible strings.
> In addition, the control logic of these two hubs are needed to be > verified.
We were using gpio-hog to reset the USB hub, so now switch to USB onboard hub
> > And what's the status of the companion hub of the USB3 hub? Is it > really a USB3-only hub, or is its USB2 part just equal to another USB3 > hub? > usb5e3,610 is USB 2.0 port hub present on Odroid C1 and Odroid C2 board
usb5e3,620 is USB 3.1 port hub present on Odroid N2.
> > > > reg: true > > >
Thanks -Anand
| |