Messages in this thread | | | Subject | Re: [PATCH] staging: rtl8723au: Fix static symbol sparse warning | From | Paul Bolle <> | Date | Fri, 01 Aug 2014 14:11:38 +0200 |
| |
On Fri, 2014-08-01 at 14:06 +0100, Miguel Oliveira wrote: > Fix sparse warning: > drivers/staging/rtl8723au/core/rtw_efuse.c:579:5: warning: symbol 'efuse_GetCurrentSize23a' was not declared. Should it be static? > > Signed-off-by: Miguel Oliveira <cmroliv@gmail.com> > --- > drivers/staging/rtl8723au/include/rtw_efuse.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/staging/rtl8723au/include/rtw_efuse.h b/drivers/staging/rtl8723au/include/rtw_efuse.h > index 07bdc34..304cc71 100644 > --- a/drivers/staging/rtl8723au/include/rtw_efuse.h > +++ b/drivers/staging/rtl8723au/include/rtw_efuse.h > @@ -82,6 +82,7 @@ struct pg_pkt_struct { > /*------------------------Export global variable----------------------------*/ > > u16 efuse_GetMaxSize23a(struct rtw_adapter *padapter); > +int efuse_GetCurrentSize23a(struct rtw_adapter *padapter, u16 *size); > int rtw_efuse_access23a(struct rtw_adapter *padapter, u8 bRead, u16 start_addr, u16 cnts, u8 *data); > int rtw_efuse_map_read23a(struct rtw_adapter *padapter, u16 addr, u16 cnts, u8 *data); > u8 rtw_efuse_map_write(struct rtw_adapter *padapter, u16 addr, u16 cnts, u8 *data);
This function is never used in current linux-next. So I think it might as well be dropped.
Note there's also Efuse_GetCurrentSize23a (with an uppercase 'E'). It's used just once. You could probably submit another patch to remove that from rtw_efuse.h and make it static in rtw_efuse.c The comment above that function needs and update too, but maybe it's also OK to drop that comment.
Paul Bolle
| |