:::::: :::::: Manual check reason: "low confidence static check warning: drivers/tty/serial/sifive.c:766:28: sparse: sparse: too many arguments for function __ssp_transmit_char" ::::::
BCC: [email protected] CC: [email protected] In-Reply-To: <[email protected]> References: <[email protected]> TO: Jiri Slaby <[email protected]> Hi Jiri, I love your patch! Perhaps something to improve: [auto build test WARNING on tty/tty-testing] [also build test WARNING on usb/usb-testing shawnguo/for-next linus/master v6.0-rc3 next-20220901] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch#_base_tree_information] url: https://github.com/intel-lab-lkp/linux/commits/Jiri-Slaby/tty-TX-helpers/20220901-190806 base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/tty.git tty-testing :::::: branch date: 15 hours ago :::::: commit date: 15 hours ago config: i386-randconfig-s001 (https://download.01.org/0day-ci/archive/20220902/[email protected]/config) compiler: gcc-11 (Debian 11.3.0-5) 11.3.0 reproduce: # apt-get install sparse # sparse version: v0.6.4-39-gce1a6720-dirty # https://github.com/intel-lab-lkp/linux/commit/f6f99189d77ea45bc72efe7393bb91f4eb46a2fb git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Jiri-Slaby/tty-TX-helpers/20220901-190806 git checkout f6f99189d77ea45bc72efe7393bb91f4eb46a2fb # save the config file mkdir build_dir && cp config build_dir/.config make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=i386 SHELL=/bin/bash drivers/tty/serial/ If you fix the issue, kindly add following tag where applicable Reported-by: kernel test robot <[email protected]> sparse warnings: (new ones prefixed by >>) drivers/tty/serial/sifive.c:280:8: sparse: sparse: Expected ) in function declarator drivers/tty/serial/sifive.c:280:8: sparse: sparse: got __VA_OPT__ drivers/tty/serial/sifive.c:280:8: sparse: sparse: Expected ) in function declarator drivers/tty/serial/sifive.c:280:8: sparse: sparse: got ; drivers/tty/serial/sifive.c:280:8: sparse: sparse: Trying to use reserved word 'for' as identifier drivers/tty/serial/sifive.c:280:8: sparse: sparse: Expected ; at end of declaration drivers/tty/serial/sifive.c:280:8: sparse: sparse: got && drivers/tty/serial/sifive.c:280:8: sparse: sparse: Expected ) in nested declarator drivers/tty/serial/sifive.c:280:8: sparse: sparse: got -- drivers/tty/serial/sifive.c:280:8: sparse: sparse: Expected ) in function declarator drivers/tty/serial/sifive.c:280:8: sparse: sparse: got ( drivers/tty/serial/sifive.c:280:8: sparse: sparse: undefined identifier 'port' drivers/tty/serial/sifive.c:280:8: sparse: sparse: undefined identifier 'port' drivers/tty/serial/sifive.c:280:8: sparse: sparse: Expected ) in nested declarator drivers/tty/serial/sifive.c:280:8: sparse: sparse: got sifive_serial_port drivers/tty/serial/sifive.c:280:8: sparse: sparse: Trying to use reserved word 'struct' as identifier drivers/tty/serial/sifive.c:280:8: sparse: sparse: Expected ; at the end of type declaration drivers/tty/serial/sifive.c:280:8: sparse: sparse: got } drivers/tty/serial/sifive.c:280:8: sparse: sparse: Expected ; at end of declaration drivers/tty/serial/sifive.c:280:8: sparse: sparse: got -> drivers/tty/serial/sifive.c:280:8: sparse: sparse: Trying to use reserved word 'continue' as identifier drivers/tty/serial/sifive.c:280:8: sparse: sparse: Expected ; at the end of type declaration drivers/tty/serial/sifive.c:280:8: sparse: sparse: got } drivers/tty/serial/sifive.c:280:8: sparse: sparse: Expected ) in function declarator drivers/tty/serial/sifive.c:280:8: sparse: sparse: got ( drivers/tty/serial/sifive.c:280:8: sparse: sparse: cannot dereference this type drivers/tty/serial/sifive.c:280:8: sparse: sparse: cannot dereference this type drivers/tty/serial/sifive.c:280:8: sparse: sparse: Expected ) in nested declarator drivers/tty/serial/sifive.c:280:8: sparse: sparse: got sifive_serial_port drivers/tty/serial/sifive.c:280:8: sparse: sparse: Trying to use reserved word 'struct' as identifier drivers/tty/serial/sifive.c:280:8: sparse: sparse: Expected ; at the end of type declaration drivers/tty/serial/sifive.c:280:8: sparse: sparse: got } drivers/tty/serial/sifive.c:280:8: sparse: sparse: Expected ; at end of declaration drivers/tty/serial/sifive.c:280:8: sparse: sparse: got -> drivers/tty/serial/sifive.c:280:8: sparse: sparse: Expected ; at the end of type declaration drivers/tty/serial/sifive.c:280:8: sparse: sparse: got } drivers/tty/serial/sifive.c:280:8: sparse: sparse: Expected ) in function declarator drivers/tty/serial/sifive.c:280:8: sparse: sparse: got < drivers/tty/serial/sifive.c:280:8: sparse: sparse: Trying to use reserved word 'if' as identifier drivers/tty/serial/sifive.c:280:8: sparse: sparse: Expected ) in function declarator drivers/tty/serial/sifive.c:280:8: sparse: sparse: got == drivers/tty/serial/sifive.c:280:8: sparse: sparse: Trying to use reserved word 'if' as identifier drivers/tty/serial/sifive.c:280:8: sparse: sparse: Expected ; at the end of type declaration drivers/tty/serial/sifive.c:280:8: sparse: sparse: got } drivers/tty/serial/sifive.c:280:8: sparse: sparse: Expected ; at the end of type declaration drivers/tty/serial/sifive.c:280:8: sparse: sparse: got } drivers/tty/serial/sifive.c:280:8: sparse: sparse: symbol '__ssp_transmit_char' redeclared with different type (different argument counts): drivers/tty/serial/sifive.c:280:8: sparse: int static [signed] [toplevel] __ssp_transmit_char( ... ) drivers/tty/serial/sifive.c:275:13: sparse: note: previously declared as: drivers/tty/serial/sifive.c:275:13: sparse: void static [toplevel] __ssp_transmit_char( ... ) drivers/tty/serial/sifive.c:280:8: sparse: sparse: undefined identifier 'xmit' drivers/tty/serial/sifive.c:280:8: sparse: sparse: undefined identifier 'xmit' drivers/tty/serial/sifive.c:280:8: sparse: sparse: symbol 'ch' redeclared with different type (different type sizes): drivers/tty/serial/sifive.c:280:8: sparse: int [addressable] [toplevel] ch drivers/tty/serial/sifive.c:280:8: sparse: note: previously declared as: drivers/tty/serial/sifive.c:280:8: sparse: unsigned char [addressable] [toplevel] [usertype] ch drivers/tty/serial/sifive.c:280:8: sparse: sparse: symbol '__ssp_transmit_char' redeclared with different type (different argument counts): drivers/tty/serial/sifive.c:280:8: sparse: int static [signed] [toplevel] __ssp_transmit_char( ... ) drivers/tty/serial/sifive.c:275:13: sparse: note: previously declared as: drivers/tty/serial/sifive.c:275:13: sparse: void static [toplevel] __ssp_transmit_char( ... ) drivers/tty/serial/sifive.c:280:8: sparse: sparse: cannot dereference this type drivers/tty/serial/sifive.c:280:8: sparse: sparse: cannot dereference this type drivers/tty/serial/sifive.c:280:8: sparse: sparse: symbol 'pending' redeclared with different type (different signedness): drivers/tty/serial/sifive.c:280:8: sparse: int [addressable] [toplevel] pending drivers/tty/serial/sifive.c:280:8: sparse: note: previously declared as: drivers/tty/serial/sifive.c:280:8: sparse: unsigned int [addressable] [toplevel] pending drivers/tty/serial/sifive.c:521:37: sparse: sparse: too many arguments for function __ssp_transmit_chars >> drivers/tty/serial/sifive.c:766:28: sparse: sparse: too many arguments for >> function __ssp_transmit_char -- drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: Expected ) in function declarator drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: got __VA_OPT__ drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: Expected ) in function declarator drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: got ; drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: Trying to use reserved word 'for' as identifier drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: Expected ; at end of declaration drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: got && drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: Expected ) in nested declarator drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: got -- drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: Expected ) in function declarator drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: got -> drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: Expected ; at end of declaration drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: got -> drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: Trying to use reserved word 'continue' as identifier drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: Expected ; at the end of type declaration drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: got } drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: Expected ) in function declarator drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: got -> drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: Expected ; at end of declaration drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: got -> drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: Expected ; at the end of type declaration drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: got } drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: Expected ) in function declarator drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: got < drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: Trying to use reserved word 'if' as identifier drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: Expected ) in function declarator drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: got == drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: Trying to use reserved word 'if' as identifier drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: Expected ; at the end of type declaration drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: got } drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: Expected ; at the end of type declaration drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: got } drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: symbol 'writel' redeclared with different type (incompatible argument 1 (different base types)): drivers/tty/serial/altera_jtaguart.c:137:8: sparse: int static [signed] [toplevel] writel( ... ) drivers/tty/serial/altera_jtaguart.c: note: in included file (through include/linux/scatterlist.h, include/linux/kfifo.h, include/linux/tty_port.h, ...): arch/x86/include/asm/io.h:69:1: sparse: note: previously declared as: >> arch/x86/include/asm/io.h:69:1: sparse: void static inline [gnu_inline] >> [toplevel] writel( ... ) drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: undefined identifier 'xmit' drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: undefined identifier 'xmit' drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: symbol 'ch' redeclared with different type (different type sizes): drivers/tty/serial/altera_jtaguart.c:137:8: sparse: int [addressable] [toplevel] ch drivers/tty/serial/altera_jtaguart.c:137:8: sparse: note: previously declared as: drivers/tty/serial/altera_jtaguart.c:137:8: sparse: unsigned char [addressable] [toplevel] [usertype] ch drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: symbol 'writel' redeclared with different type (incompatible argument 1 (different base types)): drivers/tty/serial/altera_jtaguart.c:137:8: sparse: int static [signed] [toplevel] writel( ... ) arch/x86/include/asm/io.h:69:1: sparse: note: previously declared as: >> arch/x86/include/asm/io.h:69:1: sparse: void static inline [gnu_inline] >> [toplevel] writel( ... ) drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: cannot dereference this type drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: cannot dereference this type drivers/tty/serial/altera_jtaguart.c:137:8: sparse: sparse: symbol 'pending' redeclared with different type (different signedness): drivers/tty/serial/altera_jtaguart.c:137:8: sparse: int [addressable] [toplevel] pending drivers/tty/serial/altera_jtaguart.c:137:8: sparse: note: previously declared as: drivers/tty/serial/altera_jtaguart.c:137:8: sparse: unsigned int [addressable] [toplevel] pending drivers/tty/serial/altera_jtaguart.c:150:36: sparse: sparse: too many arguments for function altera_jtaguart_do_tx_chars vim +766 drivers/tty/serial/sifive.c 45c054d0815b15 Paul Walmsley 2019-04-12 760 3f8bab174cb26a Jiri Slaby 2022-03-03 761 static void sifive_serial_console_putchar(struct uart_port *port, unsigned char ch) 45c054d0815b15 Paul Walmsley 2019-04-12 762 { 45c054d0815b15 Paul Walmsley 2019-04-12 763 struct sifive_serial_port *ssp = port_to_sifive_serial_port(port); 45c054d0815b15 Paul Walmsley 2019-04-12 764 45c054d0815b15 Paul Walmsley 2019-04-12 765 __ssp_wait_for_xmitr(ssp); 45c054d0815b15 Paul Walmsley 2019-04-12 @766 __ssp_transmit_char(ssp, ch); 45c054d0815b15 Paul Walmsley 2019-04-12 767 } 45c054d0815b15 Paul Walmsley 2019-04-12 768 -- 0-DAY CI Kernel Test Service https://01.org/lkp _______________________________________________ kbuild mailing list -- [email protected] To unsubscribe send an email to [email protected]
