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

[Rafał Miłecki]

On 22 October 2016 at 18:10, Christian Lamparter
<chunkeey at googlemail.com> wrote:
> On Saturday, October 22, 2016 11:22:35 AM 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. Switch to usbport to
>> have LED working with both ports.
>>
>> Signed-off-by: Rafał Miłecki <rafal at milecki.pl>
>
> I was testing this on the wndr4700.
>
> # cat /etc/config/system
> config led 'led_usb3'
>         option name 'USB3'
>         option sysfs 'wndr4700:blue:usb'
>         option trigger 'usbport'
>         list port 'usb2-port1'
>         list port 'usb3-port1'
> ---
>
> However the LED only lights up if a device (in my case two
> usb-sticks one 3.0 and one 2.0) is connected on port
> usb3-port1. No LED activity for usb2-port1 at first.
>
> The next step was to look into the port triggers
> # ls -al /sys/class/leds/wndr4700\:blue\:usb/ports/
> usb1-port1  usb2-port1  usb2-port2  usb3-port1  usb3-port2
>
> # cat /sys/class/leds/wndr4700\:blue\:usb/ports/*
> 0
> 1
> 0
> 1
> 0
> ---
> The triggers are set for usb2-port1 and usb3-port1 but not for any of
> the port2. I had to enable the usb2-port2 and usb3-port2 to get it
> working. Is there a way to specify sth like "usb2-port[2-Adresslimit]"
> and the same for usb3-port?

Thanks for the research, I incorrectly assumed info in 01_leds is
correct and complete. There isn't a way to specify range of ports.