[ath9k-devel] [PATCH] ath9k_htc: Add a module parameter to disable blink

[Oleksij Rempel <linux@rempel-privat.de>]

Am 11.01.2015 um 08:09 schrieb Hong Xu:
> Add an option "blink" to enable or disable the LED blink. The default
> value is set to 1 so that existing users would not experience any
> unexpected changes.
> ---
>  drivers/net/wireless/ath/ath9k/htc.h          | 1 +
>  drivers/net/wireless/ath/ath9k/htc_drv_gpio.c | 4 ++++
>  drivers/net/wireless/ath/ath9k/htc_drv_init.c | 4 ++++
>  3 files changed, 9 insertions(+)
> 
> diff --git a/drivers/net/wireless/ath/ath9k/htc.h b/drivers/net/wireless/ath/ath9k/htc.h
> index 9dde265..5697097 100644
> --- a/drivers/net/wireless/ath/ath9k/htc.h
> +++ b/drivers/net/wireless/ath/ath9k/htc.h
> @@ -44,6 +44,7 @@
>  
>  extern struct ieee80211_ops ath9k_htc_ops;
>  extern int htc_modparam_nohwcrypt;
> +extern int led_blink;
>  
>  enum htc_phymode {
>  	HTC_MODE_11NA		= 0,
> diff --git a/drivers/net/wireless/ath/ath9k/htc_drv_gpio.c b/drivers/net/wireless/ath/ath9k/htc_drv_gpio.c
> index 50f74a2..998b558 100644
> --- a/drivers/net/wireless/ath/ath9k/htc_drv_gpio.c
> +++ b/drivers/net/wireless/ath/ath9k/htc_drv_gpio.c
> @@ -279,6 +279,10 @@ void ath9k_init_leds(struct ath9k_htc_priv *priv)
>  	else
>  		priv->ah->led_pin = ATH_LED_PIN_DEF;
>  
> +	if (!led_blink)
> +		priv->led_cdev.default_trigger =
> +			ieee80211_get_radio_led_name(priv->hw);
> +
>  	ath9k_configure_leds(priv);
>  
>  	snprintf(priv->led_name, sizeof(priv->led_name),
> diff --git a/drivers/net/wireless/ath/ath9k/htc_drv_init.c b/drivers/net/wireless/ath/ath9k/htc_drv_init.c
> index e8fa944..c762d84 100644
> --- a/drivers/net/wireless/ath/ath9k/htc_drv_init.c
> +++ b/drivers/net/wireless/ath/ath9k/htc_drv_init.c
> @@ -38,6 +38,10 @@ static int ath9k_ps_enable;
>  module_param_named(ps_enable, ath9k_ps_enable, int, 0444);
>  MODULE_PARM_DESC(ps_enable, "Enable WLAN PowerSave");
>  
> +int led_blink = 1;
> +module_param_named(blink, led_blink, int, 0444);
> +MODULE_PARM_DESC(blink, "Enable LED blink on activity");
> +
>  #ifdef CONFIG_MAC80211_LEDS
>  static const struct ieee80211_tpt_blink ath9k_htc_tpt_blink[] = {
>  	{ .throughput = 0 * 1024, .blink_time = 334 },
> 


Nice, thank you!
Acked-by: Oleksij Rempel <linux@rempel-privat.de>
Tested-by: Oleksij Rempel <linux@rempel-privat.de>

Please resend this patch to Kalle Valo.
-- 
Regards,
Oleksij

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 213 bytes
Desc: OpenPGP digital signature
Url : http://lists.ath9k.org/pipermail/ath9k-devel/attachments/20150111/bfc5dca7/attachment.pgp