All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: kbuild@lists.01.org
Subject: Re: [PATCH v2 2/3] tty: serial: use DEFINE_UART_PORT_TX_HELPER()
Date: Fri, 02 Sep 2022 05:01:35 +0800	[thread overview]
Message-ID: <202209020454.Ee5QOZVP-lkp@intel.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 8120 bytes --]

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

BCC: lkp(a)intel.com
CC: kbuild-all(a)lists.01.org
In-Reply-To: <20220901110657.3305-3-jslaby@suse.cz>
References: <20220901110657.3305-3-jslaby@suse.cz>
TO: Jiri Slaby <jslaby@suse.cz>

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: 10 hours ago
:::::: commit date: 10 hours ago
config: loongarch-randconfig-s053-20220901 (https://download.01.org/0day-ci/archive/20220902/202209020454.Ee5QOZVP-lkp(a)intel.com/config)
compiler: loongarch64-linux-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/393b592949b2c633992ae9bdf00d3cffdd9a85d7
        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 393b592949b2c633992ae9bdf00d3cffdd9a85d7
        # 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=loongarch SHELL=/bin/bash drivers/tty/serial/

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>

sparse warnings: (new ones prefixed by >>)
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: Expected ) in function declarator
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: got __VA_OPT__
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: Expected ) in function declarator
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: got ;
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: Trying to use reserved word 'for' as identifier
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: Expected ; at end of declaration
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: got &&
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: Expected ) in nested declarator
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: got {
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: Expected ) in function declarator
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: got ->
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: Expected ; at end of declaration
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: got ->
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: Trying to use reserved word 'continue' as identifier
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: Expected ; at the end of type declaration
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: got }
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: Expected ) in function declarator
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: got ->
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: Expected ; at end of declaration
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: got ->
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: Expected ; at the end of type declaration
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: got }
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: Expected ) in function declarator
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: got <
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: Trying to use reserved word 'if' as identifier
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: Expected ) in function declarator
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: got ==
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: Trying to use reserved word 'if' as identifier
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: Expected ; at the end of type declaration
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: got }
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: Expected ;@the end of type declaration
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: got }
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: symbol 'true' redeclared with different type (different signedness):
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse:    int [addressable] [toplevel] true
   drivers/tty/serial/fsl_lpuart.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/fsl_lpuart.c:751:8: sparse: sparse: symbol 'writeb' redeclared with different type (incompatible argument 1 (different base types)):
>> drivers/tty/serial/fsl_lpuart.c:751:8: sparse:    int static [signed] [toplevel] writeb( ... )
   drivers/tty/serial/fsl_lpuart.c: note: in included file (through arch/loongarch/include/asm/io.h, arch/loongarch/include/asm/pgtable.h, arch/loongarch/include/asm/uaccess.h, ...):
   include/asm-generic/io.h:249:20: sparse: note: previously declared as:
>> include/asm-generic/io.h:249:20: sparse:    void static inline [gnu_inline] [toplevel] writeb( ... )
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: undefined identifier 'xmit'
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: undefined identifier 'xmit'
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: symbol 'ch' redeclared with different type (different type sizes):
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse:    int [addressable] [toplevel] ch
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: note: previously declared as:
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse:    unsigned char [addressable] [toplevel] [usertype] ch
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: symbol 'writeb' redeclared with different type (incompatible argument 1 (different base types)):
>> drivers/tty/serial/fsl_lpuart.c:751:8: sparse:    int static [signed] [toplevel] writeb( ... )
   include/asm-generic/io.h:249:20: sparse: note: previously declared as:
>> include/asm-generic/io.h:249:20: sparse:    void static inline [gnu_inline] [toplevel] writeb( ... )
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: cannot dereference this type
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: cannot dereference this type
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: sparse: symbol 'pending' redeclared with different type (different signedness):
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse:    int [addressable] [toplevel] pending
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse: note: previously declared as:
   drivers/tty/serial/fsl_lpuart.c:751:8: sparse:    unsigned int [addressable] [toplevel] pending

vim +751 drivers/tty/serial/fsl_lpuart.c

c9e2e946fb0ba5 Jingchang Lu 2013-06-07  750  
393b592949b2c6 Jiri Slaby   2022-09-01 @751  static DEFINE_UART_PORT_TX_HELPER(lpuart_transmit_buffer, port, ch,
393b592949b2c6 Jiri Slaby   2022-09-01  752  		lpuart_tx_ready(port),
393b592949b2c6 Jiri Slaby   2022-09-01  753  		writeb(ch, port->membase + UARTDR));
c9e2e946fb0ba5 Jingchang Lu 2013-06-07  754  

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp

             reply	other threads:[~2022-09-01 21:01 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-01 21:01 kernel test robot [this message]
  -- strict thread matches above, loose matches on Subject: below --
2022-09-04 11:44 [PATCH v2 2/3] tty: serial: use DEFINE_UART_PORT_TX_HELPER() kernel test robot
2022-09-03  3:18 kernel test robot
2022-09-03  1:14 kernel test robot
2022-09-02 22:10 kernel test robot
2022-09-02 18:00 kernel test robot
2022-09-02 15:05 kernel test robot
2022-09-02  9:14 kernel test robot
2022-09-02  1:17 kernel test robot
2022-09-01 23:45 kernel test robot
2022-09-01 21:52 kernel test robot
2022-09-01 20:10 kernel test robot
2022-09-01 11:06 [PATCH v2 0/3] tty: TX helpers Jiri Slaby
2022-09-01 11:06 ` [PATCH v2 2/3] tty: serial: use DEFINE_UART_PORT_TX_HELPER() Jiri Slaby
2022-09-02 14:21   ` Ilpo Järvinen
2022-09-06 10:50     ` Jiri Slaby

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202209020454.Ee5QOZVP-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=kbuild@lists.01.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.