[LEDE-DEV] [PATCH V2] apm821xx: fix USB LED trigger for WNDR4700

[Rafał Miłecki]

On 28 October 2016 at 12:46, Christian Lamparter
<chunkeey at googlemail.com> wrote:
> On Friday, October 28, 2016 12:40:01 PM CEST Rafał Miłecki wrote:
>> From: Rafał Miłecki <rafal at milecki.pl>
>>
>> The old usbdev trigger never supported assigning more than 1 USB port.
>> This code we got was never working as expected and it was missing 2 more
>> ports. Switch to usbport to have LED working with all ports.
>>
> Tested-by: Christian Lamparter <chunkeey at gmail.com>
>> Signed-off-by: Rafał Miłecki <rafal at milecki.pl>
>
> Thanks!
>
> here's a explanation what's going on with the multiple port and usb definitions:
>
> The WNDR4700 has two different USB solutions. A DesignWare DWC2 IP-Core
> in the APM82181-SoC ("usb1") which is wired to the SD-Card reader and needs
> to be ignored (as it's always connected). And a dedicated Renesas uPD720202
> usb 3.0 chip which powers both physical USB 3.0 ports (port1 and port2).
> The uPD720202 has two root hubs. In case of the WNDR4700 these are enumerated
> as "usb2" (for USB 1.x/2.0 devices) and "usb3" (soley for USB 3.0+ devices).
> Hence in order to fully detect any usb 1.x/2.0 or usb 3.0 device being
> plugged in to any of the two ports, the usbport trigger needs to check both
> ports on the "usb2" and "usb3" root hubs.

Thank you! Pushed.

-- 
Rafał