Skip to content
This repository has been archived by the owner on Dec 11, 2020. It is now read-only.

Add rtw_led_ctrl module parameter. Set so led blinks normally or is a… #6

Merged
merged 1 commit into from Aug 31, 2017

Conversation

smitgd
Copy link
Contributor

@smitgd smitgd commented Aug 30, 2017

…lways on or off.

Also adds example /etc/modprobe.d/ configuration file (rtl8812au.conf) that can
be used to set this new parameter option value automatically at boot or when
command "modprobe rtl8812au" is done with no module parameter.

Changes to be committed:
modified: hal/led/hal_usb_led.c
modified: include/drv_types.h
modified: os_dep/linux/os_intfs.c
new file: rtl8812au.conf

…lways on or off.

Also adds example /etc/modprobe.d/ configuration file (rtl8812au.conf) that can
be used to set this new parameter option value automatically at boot or when
command "modprobe rtl8812au" is done with no module parameter.

Changes to be committed:
	modified:   hal/led/hal_usb_led.c
	modified:   include/drv_types.h
	modified:   os_dep/linux/os_intfs.c
	new file:   rtl8812au.conf
@smitgd
Copy link
Contributor Author

smitgd commented Aug 30, 2017

This requests the addition of a module parameter that allows the LED on the usb adapter to be configured as always OFF or always ON or (default) normally blinking. It works well with the TP-LINK T4UH that I have. Based on code inspection, it appears that it will also work with other adapters supported by this driver that use software LED control. But other adapters have not been tested.

@xxNull-lsk xxNull-lsk merged commit 7ebffab into xxNull-lsk:master Aug 31, 2017
@smitgd
Copy link
Contributor Author

smitgd commented Aug 31, 2017

Thanks for merging. I should repeat that there is no guarantee this works with all of the supported wifi adapters. However, it should not break anything with the default install, i.e., the new parameter is not explictly set but remains at the default value of 1.

@xxNull-lsk
Copy link
Owner

I know. And it's worked fine in Tenda U12.

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

Successfully merging this pull request may close these issues.

None yet

2 participants