OpenWrt/LEDE Project

The renaming should be as follow:
USB2 → USB1
USB3 → USB2

I think the LED names are supposed to denote USB 2.0 vs. USB 3.1, not the number of the USB port.

But that could led to confusion, front in the leds you have USB1 and USB2 and rear in the usb ports itself you also have USB1 and USB2.

I think that I didn't notice it before having an Archer C7 where you have USB1 and USB2 both 2.0. I suppose I understood like you explain it before seeing the C7 with USB1 and 2 matching the router labels.

What you decide it's ok for me as it's cosmetics.

I'm undecided too. If we change it we probably need to change it for all models, not just for the ACS.

I will fix this together with  FS#425 .

howl, would you please try an image from https://www.kresin.me/files/mvebu/. The images are build from the 17.01 release branch and do include LuCI.

Don't preserve your config when upgrading! Please test both usb ports with USB 2.0 and USB 3.0 devices. Check if the correct LED is switched on and report back!

The LED names should match the label printed on the case as well.

Will try to test it today.

Ok, I have tested it and I have found two things.

First one is that the generated led configuration had caiman instead of shelby in the led names, perhaps you hardcoded the other model to test this. I have just edited caiman to shelby and it works, USB 1 brights up when a usb 2 device is connected, and because it's only usb 2 also with a usb 3 device. USB 2 also brights up with usb 2 and 3 devices because the two ports setup in the led configuration.

The second one is that before testing with devices I set up 255 to the brightness of the leds manually in the sysfs, and found that the third led is also working. The led usb3_2 is there just below the usb3_1, it's a little point similar to the one in the ethernet port to indicate if the link is 100 or 1000 mb. Joining that we can detect if the device in the usb 3 port is 2 or 3 and that third led I have done this to the configuration:

config led 'led_usb1'
	option name 'USB 1'
	option sysfs 'pca963x:shelby:white:usb2'
	option trigger 'usbport'
	list port 'usb1-port1'

config led 'led_usb2'
	option name 'USB 2'
	option sysfs 'pca963x:shelby:white:usb3_1'
	option trigger 'usbport'
	list port 'usb2-port1'
	list port 'usb3-port1'

config led 'led_usb2_2'
	option name 'USB 2-2'
	option sysfs 'pca963x:shelby:white:usb3_2'
	option trigger 'usbport'
	list port 'usb3-port1'

With it if you plug a device in USB 2, if it's 2 the bar get bright, but if it's 3 the bar and the dot below gets bright. I suppose the original firmwares does it but I haven't tested. Anyways it seems that usb3_2 is there for that purpose.

I also opened two days ago a bug in luci asking for usbport trigger configuration that is related to this use case https://github.com/openwrt/luci/issues/971.

One suggestion for the naming, I named the dot as 'USB 2-2' but it could be 'USB 2 SS', referencing SuperSpeed also as described in the first paragraph of https://en.wikipedia.org/wiki/USB_3.0 that is used as a standard de-facto like the blue color to distinguish USB 3 from 2.

Indeed, I made a stupid copy/paste mistake and I wasn't ware of the usb3_2 LED.

I've fixed the copy/paste mess, added the usb3_2 LED (which is according to the usermanual used by the linksys firmware the way you supposed already) and uploaded a new images.

Would you please test again!

Tested, and it's ok. Thanks for taking into account the SS suggestion.

Mathias I see that mamba has usb2-port2 for USB 2 instead usb3-port1 or usb2-port1 like the other models: https://git.lede-project.org/?p=source.git;a=blob;f=target/linux/mvebu/base-files/etc/board.d/01_leds;h=d6e42ddfd590ff8e6136bffafd0c57159f1cbb98;hb=9641ceea0ce68d1c507b9d2bbe7cf8da518a2eb8

You changed it just to be like the other models in your patch: https://git.lede-project.org/?p=lede/mkresin/staging.git;a=blob;f=target/linux/mvebu/base-files/etc/board.d/01_leds;h=a147fa46d463d3b3e43d51e7214c67da3e3a75d7;hb=d4ad139874f7c37f36c5d602443d649f874ff346

I don't have a mamba device to test it.