On 10/10/2011 01:43 PM, Jouni Malinen wrote: > The new debugfs files keepalive and disconnect_timeout can be used to > fetch the current values and to change the values for keepalive and > disconnect event timeout (both in seconds). [...] > --- a/drivers/net/wireless/ath/ath6kl/wmi.c > +++ b/drivers/net/wireless/ath/ath6kl/wmi.c > @@ -1940,6 +1940,10 @@ int ath6kl_wmi_disctimeout_cmd(struct wmi *wmi, u8 timeout) > > ret = ath6kl_wmi_cmd_send(wmi, skb, WMI_SET_DISC_TIMEOUT_CMDID, > NO_SYNC_WMIFLAG); > +#ifdef CONFIG_ATH6KL_DEBUG > + if (ret == 0) > + wmi->parent_dev->debug.disc_timeout = timeout; > +#endif /* CONFIG_ATH6KL_DEBUG */ > return ret; > } > > @@ -2524,6 +2528,10 @@ int ath6kl_wmi_set_keepalive_cmd(struct wmi *wmi, u8 keep_alive_intvl) > > ret = ath6kl_wmi_cmd_send(wmi, skb, WMI_SET_KEEPALIVE_CMDID, > NO_SYNC_WMIFLAG); > +#ifdef CONFIG_ATH6KL_DEBUG > + if (ret == 0) > + wmi->parent_dev->debug.keepalive = keep_alive_intvl; > +#endif /* CONFIG_ATH6KL_DEBUG */ Similar comment for these two as well. It would better to have the code in debug.c (or .h) and add functions for providing this information. Kalle -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html