> From: gregkh@xxxxxxxxxxxxxxxxxxx <gregkh@xxxxxxxxxxxxxxxxxxx> > Sent: Wednesday, February 23, 2022 3:56 PM > To: Tao Wang (Consultant) (QUIC) <quic_wat@xxxxxxxxxxx> > Cc: mka@xxxxxxxxxxxx; balbi@xxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; > dianders@xxxxxxxxxxxx; frowand.list@xxxxxxxxx; hadess@xxxxxxxxxx; > krzk@xxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; linux-usb@xxxxxxxxxxxxxxx; > mathias.nyman@xxxxxxxxx; michal.simek@xxxxxxxxxx; > peter.chen@xxxxxxxxxx; ravisadineni@xxxxxxxxxxxx; robh+dt@xxxxxxxxxx; > rogerq@xxxxxxxxxx; stern@xxxxxxxxxxxxxxxxxxx; swboyd@xxxxxxxxxxxx; > Linyu Yuan (QUIC) <quic_linyyuan@xxxxxxxxxxx> > Subject: Re: 回复: 回复: Re: [PATCH v20 3/5] usb: misc: Add > onboard_usb_hub driver > > On Wed, Feb 23, 2022 at 07:40:31AM +0000, Tao Wang (Consultant) (QUIC) > wrote: > > Ok, thanks your reply. > > > > Here is my question, we must modify the driver "onboard_usb_hub.c" if > we want to use it. But it's hard to complete because it's an opensource code. > > I do not understand. We do not deal with code that is not in the kernel > source tree, as we have no idea what is out there. Please just submit > your changes to be merged into the tree and all will be fine. Hi Greg and mka, Let's make it clear that we are talking about once this driver is approved into usb tree, If we use different USB HUB which have VID/PID not defined in this driver, We need to update this driver. But if we defined VID/PID in device tree(for a specific board, manufacture should know VID/PID from HUB it used), dynamic parsed by the driver, then we don't need to change this driver (increase VID/PID table). > > thanks, > > greg k-h