Thanks. On 2/26/2019 12:55 PM, Bo YU wrote: > Fix sparse warning: > > drivers/staging/wilc1000/host_interface.c:450:30: warning: incorrect type in > assignment (different base types) > drivers/staging/wilc1000/host_interface.c:450:30: expected restricted > __le16 [usertype] beacon_period > drivers/staging/wilc1000/host_interface.c:450:30: got unsigned short > [usertype] beacon_interval > drivers/staging/wilc1000/host_interface.c:451:25: warning: incorrect type in > assignment (different base types) > drivers/staging/wilc1000/host_interface.c:451:25: expected restricted > __le16 [usertype] cap_info > drivers/staging/wilc1000/host_interface.c:451:25: got unsigned short > [usertype] capability > > Signed-off-by: Bo YU <tsu.y...@gmail.com>
Reviewed-by: Ajay Singh <ajay.kat...@microchip.com> > --- > V2: use cpu_to_le16 assign valid type according to Ajay's suggestions > --- > drivers/staging/wilc1000/host_interface.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/staging/wilc1000/host_interface.c > b/drivers/staging/wilc1000/host_interface.c > index 50dc2dd942f5..20349af2ed30 100644 > --- a/drivers/staging/wilc1000/host_interface.c > +++ b/drivers/staging/wilc1000/host_interface.c > @@ -447,8 +447,8 @@ void *wilc_parse_join_bss_param(struct cfg80211_bss *bss, > if (!param) > return NULL; > > - param->beacon_period = bss->beacon_interval; > - param->cap_info = bss->capability; > + param->beacon_period = cpu_to_le16(bss->beacon_interval); > + param->cap_info = cpu_to_le16(bss->capability); > param->bss_type = WILC_FW_BSS_TYPE_INFRA; > param->ch = ieee80211_frequency_to_channel(bss->channel->center_freq); > ether_addr_copy(param->bssid, bss->bssid); > _______________________________________________ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
