:::::: 
:::::: Manual check reason: "low confidence static check warning: 
drivers/tty/serial/lantiq.c:149:8: sparse:    int static [signed] [toplevel] 
__raw_writeb( ... )"
:::::: 

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-rc5 next-20220912]
[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/20220906-185236
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/tty.git 
tty-testing
:::::: branch date: 6 days ago
:::::: commit date: 6 days ago
config: arc-randconfig-s053-20220911 
(https://download.01.org/0day-ci/archive/20220913/[email protected]/config)
compiler: arceb-elf-gcc (GCC) 12.1.0
reproduce:
        wget 
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O 
~/bin/make.cross
        chmod +x ~/bin/make.cross
        # apt-get install sparse
        # sparse version: v0.6.4-39-gce1a6720-dirty
        # 
https://github.com/intel-lab-lkp/linux/commit/7a2fb33f6cabf44f85ffba57d6487529ce598d13
        git remote add linux-review https://github.com/intel-lab-lkp/linux
        git fetch --no-tags linux-review 
Jiri-Slaby/tty-TX-helpers/20220906-185236
        git checkout 7a2fb33f6cabf44f85ffba57d6487529ce598d13
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross C=1 
CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=arc 
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/lantiq.c:149:8: sparse: sparse: Expected ) in function 
declarator
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: got __VA_OPT__
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Expected ) in function 
declarator
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: got ;
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Trying to use reserved 
word 'for' as identifier
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Expected ; at end of 
declaration
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: got &&
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Expected ) in nested 
declarator
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: got {
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Expected ) in nested 
declarator
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: got {
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Expected ) in function 
declarator
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: got ->
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Expected ; at the end of 
type declaration
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: got }
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Expected ; at end of 
declaration
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: got ->
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Trying to use reserved 
word 'continue' as identifier
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Expected ; at the end of 
type declaration
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: got }
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Expected ) in nested 
declarator
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: got {
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Expected ) in function 
declarator
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: got ->
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Expected ; at the end of 
type declaration
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: got }
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Expected ; at end of 
declaration
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: got ->
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Expected ; at the end of 
type declaration
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: got }
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Expected ) in function 
declarator
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: got <
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Trying to use reserved 
word 'if' as identifier
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Expected ) in function 
declarator
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: got ==
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Trying to use reserved 
word 'if' as identifier
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Expected ; at the end of 
type declaration
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: got }
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: Expected ; at the end of 
type declaration
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: got }
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: symbol 'true' redeclared 
with different type (different signedness):
   drivers/tty/serial/lantiq.c:149:8: sparse:    int [addressable] [toplevel] 
true
   drivers/tty/serial/lantiq.c: note: in included file (through 
include/uapi/linux/posix_types.h, include/uapi/linux/types.h, 
include/linux/types.h, ...):
   include/linux/stddef.h:12:9: sparse: note: previously declared as:
   include/linux/stddef.h:12:9: sparse:    unsigned int enum <noident> 
[toplevel] true
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: symbol '__raw_writeb' 
redeclared with different type (incompatible argument 1 (different base types)):
>> drivers/tty/serial/lantiq.c:149:8: sparse:    int static [signed] [toplevel] 
>> __raw_writeb( ... )
   drivers/tty/serial/lantiq.c: note: in included file (through 
include/linux/io.h):
   arch/arc/include/asm/io.h:128:20: sparse: note: previously declared as:
   arch/arc/include/asm/io.h:128:20: sparse:    void static inline [gnu_inline] 
[toplevel] __raw_writeb( ... )
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: undefined identifier 
'xmit'
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: undefined identifier 
'xmit'
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: symbol 'ch' redeclared 
with different type (different type sizes):
   drivers/tty/serial/lantiq.c:149:8: sparse:    int [addressable] [toplevel] ch
   drivers/tty/serial/lantiq.c:149:8: sparse: note: previously declared as:
   drivers/tty/serial/lantiq.c:149:8: sparse:    unsigned char [addressable] 
[toplevel] [usertype] ch
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: symbol '__raw_writeb' 
redeclared with different type (incompatible argument 1 (different base types)):
>> drivers/tty/serial/lantiq.c:149:8: sparse:    int static [signed] [toplevel] 
>> __raw_writeb( ... )
   arch/arc/include/asm/io.h:128:20: sparse: note: previously declared as:
   arch/arc/include/asm/io.h:128:20: sparse:    void static inline [gnu_inline] 
[toplevel] __raw_writeb( ... )
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: cannot dereference this 
type
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: cannot dereference this 
type
   drivers/tty/serial/lantiq.c:149:8: sparse: sparse: symbol 'pending' 
redeclared with different type (different signedness):
   drivers/tty/serial/lantiq.c:149:8: sparse:    int [addressable] [toplevel] 
pending
   drivers/tty/serial/lantiq.c:149:8: sparse: note: previously declared as:
   drivers/tty/serial/lantiq.c:149:8: sparse:    unsigned int [addressable] 
[toplevel] pending

vim +149 drivers/tty/serial/lantiq.c

7a2fb33f6cabf4 Jiri Slaby 2022-09-06  148  
7a2fb33f6cabf4 Jiri Slaby 2022-09-06 @149  static 
DEFINE_UART_PORT_TX_HELPER(lqasc_tx_chars,
7a2fb33f6cabf4 Jiri Slaby 2022-09-06  150               lqasc_tx_ready(port),
7a2fb33f6cabf4 Jiri Slaby 2022-09-06  151               writeb(ch, 
port->membase + LTQ_ASC_TBUF));
7a2fb33f6cabf4 Jiri Slaby 2022-09-06  152  

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp
_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to