| Message ID | 20170420065513.1930-1-stefan@agner.ch (mailing list archive) |
|---|---|
| State | Not Applicable |
| Delegated to: | Johannes Berg |
| Headers | show |
On Wed, 2017-04-19 at 23:55 -0700, Stefan Agner wrote: > Use type enum nl80211_rate_info for bitrate information. This fixes > a warning when compiling with clang: > warning: implicit conversion from enumeration type 'enum > nl80211_rate_info' > to different enumeration type 'enum nl80211_attrs' I'm pretty sure I just applied exactly these two patches very recently. What made *two* people suddenly see this? johannes
On 2017-04-20 00:22, Johannes Berg wrote: > On Wed, 2017-04-19 at 23:55 -0700, Stefan Agner wrote: >> Use type enum nl80211_rate_info for bitrate information. This fixes >> a warning when compiling with clang: >> warning: implicit conversion from enumeration type 'enum >> nl80211_rate_info' >> to different enumeration type 'enum nl80211_attrs' > > I'm pretty sure I just applied exactly these two patches very recently. > > What made *two* people suddenly see this? There has been renewed work on compiling Linux using clang... Sorry for the clash. -- Stefan
diff --git a/net/wireless/nl80211.c b/net/wireless/nl80211.c index 2312dc2ffdb9..9af21a21ea6b 100644 --- a/net/wireless/nl80211.c +++ b/net/wireless/nl80211.c @@ -4151,7 +4151,7 @@ static bool nl80211_put_sta_rate(struct sk_buff *msg, struct rate_info *info, struct nlattr *rate; u32 bitrate; u16 bitrate_compat; - enum nl80211_attrs rate_flg; + enum nl80211_rate_info rate_flg; rate = nla_nest_start(msg, attr); if (!rate)
Use type enum nl80211_rate_info for bitrate information. This fixes a warning when compiling with clang: warning: implicit conversion from enumeration type 'enum nl80211_rate_info' to different enumeration type 'enum nl80211_attrs' Signed-off-by: Stefan Agner <stefan@agner.ch> --- net/wireless/nl80211.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)