On Sat, 2008-11-08 at 23:50 +0200, Tomas Winkler wrote:
> This patch eliminates sparse warnings in pid rate scale algorithm
> 'debugfs: allow access to signed values' patch hit the dead end
> year ago w/o much echo so I guess there is no real need to address this
> properly.
And I guess nobody uses these debugfs knobs anyway :)
Looks like an acceptable workaround.
> Signed-off-by: Tomas Winkler <tomas.winkler@xxxxxxxxx>
Acked-by: Johannes Berg <johannes@xxxxxxxxxxxxxxxx>
> ---
> net/mac80211/rc80211_pid_algo.c | 6 +++---
> 1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/net/mac80211/rc80211_pid_algo.c b/net/mac80211/rc80211_pid_algo.c
> index 2328ba5..96ceb7e 100644
> --- a/net/mac80211/rc80211_pid_algo.c
> +++ b/net/mac80211/rc80211_pid_algo.c
> @@ -403,11 +403,11 @@ static void *rate_control_pid_alloc(struct ieee80211_hw *hw,
> S_IRUSR | S_IWUSR, debugfsdir,
> &pinfo->sampling_period);
> de->coeff_p = debugfs_create_u32("coeff_p", S_IRUSR | S_IWUSR,
> - debugfsdir, &pinfo->coeff_p);
> + debugfsdir, (u32 *)&pinfo->coeff_p);
> de->coeff_i = debugfs_create_u32("coeff_i", S_IRUSR | S_IWUSR,
> - debugfsdir, &pinfo->coeff_i);
> + debugfsdir, (u32 *)&pinfo->coeff_i);
> de->coeff_d = debugfs_create_u32("coeff_d", S_IRUSR | S_IWUSR,
> - debugfsdir, &pinfo->coeff_d);
> + debugfsdir, (u32 *)&pinfo->coeff_d);
> de->smoothing_shift = debugfs_create_u32("smoothing_shift",
> S_IRUSR | S_IWUSR, debugfsdir,
> &pinfo->smoothing_shift);
Attachment:
signature.asc
Description: This is a digitally signed message part
