(switching back to my normal "community" email) On Fri, 2022-06-24 at 09:34 +0300, Dan Carpenter wrote: > > Hrm... what is that driver ? I've missed it ... is the code lifted > > from > > aspeed-vhub ? If yes, should we instead make it a common code base > > ? > > And if there are bug fixes on one they might apply to the other as > > well... > > > No, I'm the person who introduced the bug so it's unique to this > driver. Ok thanks. That said, the code looks fairly similar to the vhub code, so my comment stands, if this is the same IP block, which it appears to be, the driver should be common. IE. The vhub is made of a virtual hub with a bunch of UDCs underneath, this appears to address the ast2600 "new" standalone (no hub) variant of said UDC if I'm not mistaken. The way I structured the code in vhub, it shouldn't be overly difficult to make it standalone. I wrote (and maintain) aspeed-vhub and would be happy to work on this if I got sent an ast2600 board. Cheers, Ben.
