Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

luci-app-ledtrig-usbport: fix allow multiple selection #5312

Merged
merged 1 commit into from Aug 30, 2021
Merged

luci-app-ledtrig-usbport: fix allow multiple selection #5312

merged 1 commit into from Aug 30, 2021

Conversation

boretom
Copy link

@boretom boretom commented Aug 29, 2021

The kernel module kmod-usb-ledtrig-usbport allows for multiple selections of
USB ports to trigger a single LED.
See https://www.kernel.org/doc/html/latest/leds/ledtrig-usbport.html

This simplest of fixes allows for selection one or more USB ports to trigger a LED.

The ASUS RT-AC88U router which is getting ported to OpenWrt by arinc9 has got an
USB 3.0 port at the front which will use different internal USB ports, depending
on if you connect an USB 2.0 device or an USB 3.0 device.

This fix/patch was tested on the mentioned router.

Signed-off-by: Thomas Kupper thomas.kupper@gmail.com

The kernel module `kmod-usb-ledtrig-usbport` allows for multiple selections of
USB ports to trigger a single LED.
See https://www.kernel.org/doc/html/latest/leds/ledtrig-usbport.html

This simplest of fixes allows for selection one or more USB ports to trigger a LED.

The ASUS RT-AC88U router which is getting ported to OpenWrt by arinc9 has got an
USB 3.0 port at the front which will use different internal USB ports, depending
on if you connect an USB 2.0 device or an USB 3.0 device.

This fix/patch was tested on the mentioned router.

Signed-off-by: Thomas Kupper <thomas.kupper@gmail.com>
@feckert feckert merged commit ab2f8b8 into openwrt:master Aug 30, 2021
@feckert
Copy link
Member

feckert commented Aug 30, 2021

Thanks merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants