- Status Closed
- Percent Complete
- Task Type Bug Report
- Category Base system
-
Assigned To
Mathias Kresin - Operating System All
- Severity Very Low
- Priority Very Low
- Reported Version All
- Due in Version Undecided
-
Due Date
Undecided
- Private
Attached to Project: OpenWrt/LEDE Project
Opened by mike - 23.01.2017
Last edited by Mathias Kresin - 01.02.2017
Opened by mike - 23.01.2017
Last edited by Mathias Kresin - 01.02.2017
FS#425 - mvebu led config (wrt1200 v1)
Supply the following if possible:
- Device problem occurs on
- Software versions of LEDE release, packages, etc.
- Steps to reproduce
Device is a Linksys wrt1200 v1 running LEDE Reboot 17.01-SNAPSHOT r3006-ef08595. I noticed the usb2 led wasn’t working, checked the led configs in /etc/config/system and the dev option for config led ‘led_usb3’ was -
option dev ‘3-1’
changed it to -
option dev ‘2-1’
rebooted and usb2 led is working now.
Closed by Mathias Kresin
01.02.2017 06:49
Reason for closing: Fixed
Additional comments about closing:
01.02.2017 06:49
Reason for closing: Fixed
Additional comments about closing:
Fixed with ht tps://git.lede-project.org/726e45dfe956b a6e0814919f2943a6a9d2f529d3
Have you tested with an usb2 or usb3 device? It could be that 2-1 is for usb2 and 3-1 is for usb3.
Would you please run the following command on the wrt1200 and paste the output here:
You're right, the drive that's normally connected to the usb3 port is a usb2 drive, so i left the usb3 led config dev option at '2-1' and connected a usb3 drive, led didn't work, changed the dev option to '3-1' and the led worked with the usb3 drive, but not the usb2 drive. Here's the output you asked for -
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 be fixed as well.
Got the image flashed, settings were deleted, tested usb2 and usb3 drives on both ports and all leds work and both drives work, many thanks! Looking forward to the official 17.01 release.
I was hinted that there is one more USB LED. I've uploaded new images to take account of this extra LED.
The expected behaviour changes to:
The names should be "USB 1" for the first USB LED, "USB 2" for the second USB LED and "USB 2 SS" for the "dot" LED below the second (bar) USB LED.
Would you please test again and confirm that it is working this way on your wrt1200!
mamba:
- OK, so firstboot, with a usb stick on usb2 port, usb3 HD on usb3 port it comes up as above and usb2 led lit dimly, no usb3 led. and this:
Right, terminology usb2 port == usb1 label, usb3 port == usb2 label
So my reference to usb2 led is the one labeled USB 1
Mhh, that is unexpected. Would you please clarify which LED you mean with USB3? Is it the USB2-bar LED or the USB2-dot LED?
According to your first paste, you have the following usb port trigger:
And according to the dts file you should have the following LEDs:
Would you please attach your /etc/config/system! It should look similar to the one in https://bugs.lede-project.org/index.php?do=details&task_id=423#comment1407.
What I've added is the usb2_ss usb led trigger.
Ya, things went South on me trying to change led configs through gui front-end instead of just editing /etc/config/system. If there is anything else you think you need from the mamba let me know, otherwise I can switch it out with a rango to test that device.
https://github.com/openwrt/luci/issues/971
Oh yes, please test it on a rango as well! But better switch to the forum
Would you please test again and confirm that it is working this way on your wrt1200!
Flashed the new image and re-tested usb2 and usb3 drives on both ports and leds are still working correctly. The new USB 2 SS led also works as expected, a dot lights up under the usb2 led when a usb3 drive is connected, that's a nice feature i didn't even know about, thanks again!
LuCI support for "usbport" triggers added in https://github.com/openwrt/luci/commit/1640d141913367827ab442276a943b0a3bba2e20