2013/1/16 Gabor Juhos <[email protected]>: > The currently used 'extern inline' directive causes > the following compiler warnings if CONFIG_SWAP_IO_SPACE > is defined: > > <...>/include/asm/io.h:345:1: warning: '__fswab32' is static but used in > inline function '__outlc_p' which is not static [enabled by default] > <...>/include/asm/io.h:345:1: warning: '__fswab32' is static but used in > inline function '__outl_p' which is not static [enabled by default] > <...>/include/asm/io.h:345:1: warning: '__fswab32' is static but used in > inline function '__outlc' which is not static [enabled by default] > <...>/include/asm/io.h:345:1: warning: '__fswab32' is static but used in > inline function '__outl' which is not static [enabled by default] > <...>/include/asm/io.h:344:1: warning: '__fswab16' is static but used in > inline function '__outwc_p' which is not static [enabled by default] > <...>/include/asm/io.h:344:1: warning: '__fswab16' is static but used in > inline function '__outw_p' which is not static [enabled by default] > <...>/include/asm/io.h:344:1: warning: '__fswab16' is static but used in > inline function '__outwc' which is not static [enabled by default] > <...>/include/asm/io.h:344:1: warning: '__fswab16' is static but used in > inline function '__outw' which is not static [enabled by default] > <...>/include/asm/io.h:341:1: warning: '__fswab32' is static but used in > inline function '__inlc_p' which is not static [enabled by default] > <...>/include/asm/io.h:341:1: warning: '__fswab32' is static but used in > inline function '__inl_p' which is not static [enabled by default] > <...>/include/asm/io.h:341:1: warning: '__fswab32' is static but used in > inline function '__inlc' which is not static [enabled by default] > <...>/include/asm/io.h:341:1: warning: '__fswab32' is static but used in > inline function '__inl' which is not static [enabled by default] > <...>/include/asm/io.h:340:1: warning: '__fswab16' is static but used in > inline function '__inwc_p' which is not static [enabled by default] > <...>/include/asm/io.h:340:1: warning: '__fswab16' is static but used in > inline function '__inw_p' which is not static [enabled by default] > <...>/include/asm/io.h:340:1: warning: '__fswab16' is static but used in > inline function '__inwc' which is not static [enabled by default] > <...>/include/asm/io.h:340:1: warning: '__fswab16' is static but used in > inline function '__inw' which is not static [enabled by default] > > Signed-off-by: Gabor Juhos <[email protected]> > Cc: Daniel Schwierzeck <[email protected]> > > --- > NOTE: checkpatch.pl throwns some errors/warnings for the patch: > > 0002-MIPS-convert-IO-port-accessor-functions-to-static-in.patch:42: ERROR: > open brace '{' following function declarations go on the next line > 0002-MIPS-convert-IO-port-accessor-functions-to-static-in.patch:51: ERROR: > open brace '{' following function declarations go on the next line > 0002-MIPS-convert-IO-port-accessor-functions-to-static-in.patch:60: > WARNING: line over 80 characters > 0002-MIPS-convert-IO-port-accessor-functions-to-static-in.patch:60: ERROR: > "foo * bar" should be "foo *bar" > 0002-MIPS-convert-IO-port-accessor-functions-to-static-in.patch:60: ERROR: > open brace '{' following function declarations go on the next line > 0002-MIPS-convert-IO-port-accessor-functions-to-static-in.patch:69: > WARNING: line over 80 characters > 0002-MIPS-convert-IO-port-accessor-functions-to-static-in.patch:69: ERROR: > "foo * bar" should be "foo *bar" > 0002-MIPS-convert-IO-port-accessor-functions-to-static-in.patch:69: ERROR: > open brace '{' following function declarations go on the next line > > However those errors are already present if the > 'arch/mips/include/asm/io.h' file is checked > with checkpatch.pl. The cleanup should be done > in a separare patch. > --- > arch/mips/include/asm/io.h | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-)
applied to u-boot-mips/master, thanks -- Best regards, Daniel _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
