| Message ID | 1458972892-14169-1-git-send-email-bhumirks@gmail.com (mailing list archive) |
|---|---|
| State | Not Applicable |
| Delegated to: | Kalle Valo |
| Headers | show |
Hi All, On Sat, Mar 26, 2016 at 5:14 PM, Bhumika Goyal <bhumirks@gmail.com> wrote: > The functions rtw_get_oper_bw23a and rtw_get_oper_ch23aoffset are never > used anywhere in the kernel. So, remove their definition and prototype. > Grepped to find occurences. > > Signed-off-by: Bhumika Goyal <bhumirks@gmail.com> Looks right to me. Reviewed-by: Julian Calaby <julian.calaby@gmail.com> > --- > drivers/staging/rtl8723au/core/rtw_wlan_util.c | 10 ---------- > drivers/staging/rtl8723au/include/rtw_mlme_ext.h | 2 -- > 2 files changed, 12 deletions(-) > > diff --git a/drivers/staging/rtl8723au/core/rtw_wlan_util.c b/drivers/staging/rtl8723au/core/rtw_wlan_util.c > index cc2b84b..694cf17 100644 > --- a/drivers/staging/rtl8723au/core/rtw_wlan_util.c > +++ b/drivers/staging/rtl8723au/core/rtw_wlan_util.c > @@ -304,21 +304,11 @@ inline void rtw_set_oper_ch23a(struct rtw_adapter *adapter, u8 ch) > adapter_to_dvobj(adapter)->oper_channel = ch; > } > > -inline u8 rtw_get_oper_bw23a(struct rtw_adapter *adapter) > -{ > - return adapter_to_dvobj(adapter)->oper_bwmode; > -} > - > inline void rtw_set_oper_bw23a(struct rtw_adapter *adapter, u8 bw) > { > adapter_to_dvobj(adapter)->oper_bwmode = bw; > } > > -inline u8 rtw_get_oper_ch23aoffset(struct rtw_adapter *adapter) > -{ > - return adapter_to_dvobj(adapter)->oper_ch_offset; > -} > - > inline void rtw_set_oper_ch23aoffset23a(struct rtw_adapter *adapter, u8 offset) > { > adapter_to_dvobj(adapter)->oper_ch_offset = offset; > diff --git a/drivers/staging/rtl8723au/include/rtw_mlme_ext.h b/drivers/staging/rtl8723au/include/rtw_mlme_ext.h > index ea2a6c9..0e7d3da 100644 > --- a/drivers/staging/rtl8723au/include/rtw_mlme_ext.h > +++ b/drivers/staging/rtl8723au/include/rtw_mlme_ext.h > @@ -461,9 +461,7 @@ void Update23aTblForSoftAP(u8 *bssrateset, u32 bssratelen); > > u8 rtw_get_oper_ch23a(struct rtw_adapter *adapter); > void rtw_set_oper_ch23a(struct rtw_adapter *adapter, u8 ch); > -u8 rtw_get_oper_bw23a(struct rtw_adapter *adapter); > void rtw_set_oper_bw23a(struct rtw_adapter *adapter, u8 bw); > -u8 rtw_get_oper_ch23aoffset(struct rtw_adapter *adapter); > void rtw_set_oper_ch23aoffset23a(struct rtw_adapter *adapter, u8 offset); > > void set_channel_bwmode23a(struct rtw_adapter *padapter, unsigned char channel, > -- > 1.9.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-wireless" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
Bhumika Goyal <bhumirks@gmail.com> writes: > The functions rtw_get_oper_bw23a and rtw_get_oper_ch23aoffset are never > used anywhere in the kernel. So, remove their definition and prototype. > Grepped to find occurences. > > Signed-off-by: Bhumika Goyal <bhumirks@gmail.com> > --- > drivers/staging/rtl8723au/core/rtw_wlan_util.c | 10 ---------- > drivers/staging/rtl8723au/include/rtw_mlme_ext.h | 2 -- > 2 files changed, 12 deletions(-) Acked-by: Jes Sorensen <Jes.Sorensen@redhat.com> -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/staging/rtl8723au/core/rtw_wlan_util.c b/drivers/staging/rtl8723au/core/rtw_wlan_util.c index cc2b84b..694cf17 100644 --- a/drivers/staging/rtl8723au/core/rtw_wlan_util.c +++ b/drivers/staging/rtl8723au/core/rtw_wlan_util.c @@ -304,21 +304,11 @@ inline void rtw_set_oper_ch23a(struct rtw_adapter *adapter, u8 ch) adapter_to_dvobj(adapter)->oper_channel = ch; } -inline u8 rtw_get_oper_bw23a(struct rtw_adapter *adapter) -{ - return adapter_to_dvobj(adapter)->oper_bwmode; -} - inline void rtw_set_oper_bw23a(struct rtw_adapter *adapter, u8 bw) { adapter_to_dvobj(adapter)->oper_bwmode = bw; } -inline u8 rtw_get_oper_ch23aoffset(struct rtw_adapter *adapter) -{ - return adapter_to_dvobj(adapter)->oper_ch_offset; -} - inline void rtw_set_oper_ch23aoffset23a(struct rtw_adapter *adapter, u8 offset) { adapter_to_dvobj(adapter)->oper_ch_offset = offset; diff --git a/drivers/staging/rtl8723au/include/rtw_mlme_ext.h b/drivers/staging/rtl8723au/include/rtw_mlme_ext.h index ea2a6c9..0e7d3da 100644 --- a/drivers/staging/rtl8723au/include/rtw_mlme_ext.h +++ b/drivers/staging/rtl8723au/include/rtw_mlme_ext.h @@ -461,9 +461,7 @@ void Update23aTblForSoftAP(u8 *bssrateset, u32 bssratelen); u8 rtw_get_oper_ch23a(struct rtw_adapter *adapter); void rtw_set_oper_ch23a(struct rtw_adapter *adapter, u8 ch); -u8 rtw_get_oper_bw23a(struct rtw_adapter *adapter); void rtw_set_oper_bw23a(struct rtw_adapter *adapter, u8 bw); -u8 rtw_get_oper_ch23aoffset(struct rtw_adapter *adapter); void rtw_set_oper_ch23aoffset23a(struct rtw_adapter *adapter, u8 offset); void set_channel_bwmode23a(struct rtw_adapter *padapter, unsigned char channel,
The functions rtw_get_oper_bw23a and rtw_get_oper_ch23aoffset are never used anywhere in the kernel. So, remove their definition and prototype. Grepped to find occurences. Signed-off-by: Bhumika Goyal <bhumirks@gmail.com> --- drivers/staging/rtl8723au/core/rtw_wlan_util.c | 10 ---------- drivers/staging/rtl8723au/include/rtw_mlme_ext.h | 2 -- 2 files changed, 12 deletions(-)