Re: [PATCH 1/3] staging: wfx: Make function 'sram_write_dma_safe', 'load_firmware_secure' static

2019-10-09 Thread Dan Carpenter
On Wed, Oct 09, 2019 at 09:03:37PM +0800, zhengbin wrote: > -int sram_write_dma_safe(struct wfx_dev *wdev, u32 addr, const u8 *buf, > size_t len) > +static int > +sram_write_dma_safe(struct wfx_dev *wdev, u32 addr, const u8 *buf, size_t > len) Either declaration style is fine, but keep it

[PATCH 1/3] staging: wfx: Make function 'sram_write_dma_safe', 'load_firmware_secure' static

2019-10-09 Thread zhengbin
Fix sparse warnings: drivers/staging/wfx/fwio.c:83:5: warning: symbol 'sram_write_dma_safe' was not declared. Should it be static? drivers/staging/wfx/fwio.c:229:5: warning: symbol 'load_firmware_secure' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: zhengbin