Re: [PATCH 3/8] mac80211: fix possible compile warning in michael.h

[Johannes Berg <johannes@sipsolutions.net>]

On Wed, 2014-04-09 at 09:28 +0800, Zhao, Gang wrote:
> The warning doesn't show up due to duplicate including, but it's worth
> fixing it.
> 
> net/mac80211//michael.h:22:4: warning: ‘struct ieee80211_hdr’ declared inside parameter list [enabled by default]
>     const u8 *data, size_t data_len, u8 *mic);
>     ^
> net/mac80211//michael.h:22:4: warning: its scope is only this definition or declaration, which is probably not what you want [enabled by default]
> 
> Signed-off-by: Zhao, Gang <gamerh2o@gmail.com>
> ---
>  net/mac80211/michael.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/net/mac80211/michael.h b/net/mac80211/michael.h
> index 3b848da..b259547 100644
> --- a/net/mac80211/michael.h
> +++ b/net/mac80211/michael.h
> @@ -10,7 +10,7 @@
>  #ifndef MICHAEL_H
>  #define MICHAEL_H
>  
> -#include <linux/types.h>
> +#include <linux/ieee80211.h>

This should probably include both now though.

johannes