ARM: OMAP: fix new sparse warnings

Igor Grinberg grinberg at compulab.co.il
Thu Sep 27 09:38:32 EDT 2012


Hi Paul,

On 09/25/12 01:09, Paul Walmsley wrote:
> 
> Commit e54adb1e79762d3591459e5b0e9b9ff578e33fdb ("ARM: OMAP: omap3evm:
> cleanup revision bits") adds a new sparse[1] warning:
> 
> arch/arm/mach-omap2/board-omap3evm.c:90:4: warning: symbol 'get_omap3_evm_rev' was not declared. Should it be static?
> 
> This symbol no longer appears to be used outside this file, so mark it
> static and remove the export.
> 
> Commit 801475ccb2b2c1928b22aec4b9e5285d9e347602 ("ARM: OMAP: move
> debug_card_init() function") results in the following new sparse[1]
> warning:
> 
> arch/arm/plat-omap/debug-devices.c:71:12: warning: symbol 'debug_card_init' was not declared. Should it be static?
> 
> Fix by including the proper header file.
> 
> ...
> 
> 1. https://sparse.wiki.kernel.org/index.php/Main_Page

Yeah, my compiler did not emit those... Thanks for the fix!

> 
> Signed-off-by: Paul Walmsley <paul at pwsan.com>
> Cc: Igor Grinberg <grinberg at compulab.co.il>
> Cc: Tony Lindgren <tony at atomide.com>

Acked-by: Igor Grinberg <grinberg at compulab.co.il>


-- 
Regards,
Igor.



More information about the linux-arm-kernel mailing list