Re: [PATCH] staging: media: omap4iss: Replace macro function by static inline function in file iss_csi2.c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Aline,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on staging/staging-testing]

url:    https://github.com/0day-ci/linux/commits/Aline-Santana-Cordeiro/staging-media-omap4iss-Replace-macro-function-by-static-inline-function-in-file-iss_csi2-c/20210412-213647
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git f2f560e1bdc055a6a306e6b7823ba589794e6564
config: sh-allmodconfig (attached as .config)
compiler: sh4-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/0day-ci/linux/commit/c8106c60a292b7a0fa55aeac1c0910719fc37dec
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Aline-Santana-Cordeiro/staging-media-omap4iss-Replace-macro-function-by-static-inline-function-in-file-iss_csi2-c/20210412-213647
        git checkout c8106c60a292b7a0fa55aeac1c0910719fc37dec
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=sh 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@xxxxxxxxx>

All error/warnings (new ones prefixed by >>):

   drivers/staging/media/omap4iss/iss_csi2.c:600:15: error: return type defaults to 'int' [-Werror=return-type]
     600 | static inline csi2_print_register(iss, regs, name)
         |               ^~~~~~~~~~~~~~~~~~~
>> drivers/staging/media/omap4iss/iss_csi2.c:600:15: error: function declaration isn't a prototype [-Werror=strict-prototypes]
   drivers/staging/media/omap4iss/iss_csi2.c: In function 'csi2_print_register':
>> drivers/staging/media/omap4iss/iss_csi2.c:600:15: warning: old-style function definition [-Wold-style-definition]
>> drivers/staging/media/omap4iss/iss_csi2.c:600:15: warning: type of 'iss' defaults to 'int' [-Wmissing-parameter-type]
>> drivers/staging/media/omap4iss/iss_csi2.c:600:15: warning: type of 'regs' defaults to 'int' [-Wmissing-parameter-type]
>> drivers/staging/media/omap4iss/iss_csi2.c:600:15: warning: type of 'name' defaults to 'int' [-Wmissing-parameter-type]
   In file included from include/linux/printk.h:409,
                    from include/linux/kernel.h:16,
                    from include/linux/delay.h:22,
                    from drivers/staging/media/omap4iss/iss_csi2.c:10:
>> drivers/staging/media/omap4iss/iss_csi2.c:602:31: error: stray '#' in program
     602 |  dev_dbg(iss->dev, "###CSI2 " #name "=0x%08x\n",
         |                               ^
   include/linux/dynamic_debug.h:91:14: note: in definition of macro 'DEFINE_DYNAMIC_DEBUG_METADATA'
      91 |   .format = (fmt),    \
         |              ^~~
   include/linux/dynamic_debug.h:147:2: note: in expansion of macro '__dynamic_func_call'
     147 |  __dynamic_func_call(__UNIQUE_ID(ddebug), fmt, func, ##__VA_ARGS__)
         |  ^~~~~~~~~~~~~~~~~~~
   include/linux/dynamic_debug.h:161:2: note: in expansion of macro '_dynamic_func_call'
     161 |  _dynamic_func_call(fmt,__dynamic_dev_dbg,   \
         |  ^~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:123:2: note: in expansion of macro 'dynamic_dev_dbg'
     123 |  dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
         |  ^~~~~~~~~~~~~~~
   include/linux/dev_printk.h:123:23: note: in expansion of macro 'dev_fmt'
     123 |  dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                       ^~~~~~~
   drivers/staging/media/omap4iss/iss_csi2.c:602:2: note: in expansion of macro 'dev_dbg'
     602 |  dev_dbg(iss->dev, "###CSI2 " #name "=0x%08x\n",
         |  ^~~~~~~
>> drivers/staging/media/omap4iss/iss_csi2.c:602:32: error: expected ')' before 'name'
     602 |  dev_dbg(iss->dev, "###CSI2 " #name "=0x%08x\n",
         |                                ^~~~
   include/linux/dynamic_debug.h:91:14: note: in definition of macro 'DEFINE_DYNAMIC_DEBUG_METADATA'
      91 |   .format = (fmt),    \
         |              ^~~
   include/linux/dynamic_debug.h:147:2: note: in expansion of macro '__dynamic_func_call'
     147 |  __dynamic_func_call(__UNIQUE_ID(ddebug), fmt, func, ##__VA_ARGS__)
         |  ^~~~~~~~~~~~~~~~~~~
   include/linux/dynamic_debug.h:161:2: note: in expansion of macro '_dynamic_func_call'
     161 |  _dynamic_func_call(fmt,__dynamic_dev_dbg,   \
         |  ^~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:123:2: note: in expansion of macro 'dynamic_dev_dbg'
     123 |  dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
         |  ^~~~~~~~~~~~~~~
   include/linux/dev_printk.h:123:23: note: in expansion of macro 'dev_fmt'
     123 |  dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                       ^~~~~~~
   drivers/staging/media/omap4iss/iss_csi2.c:602:2: note: in expansion of macro 'dev_dbg'
     602 |  dev_dbg(iss->dev, "###CSI2 " #name "=0x%08x\n",
         |  ^~~~~~~
   include/linux/dynamic_debug.h:91:13: note: to match this '('
      91 |   .format = (fmt),    \
         |             ^
   include/linux/dynamic_debug.h:127:2: note: in expansion of macro 'DEFINE_DYNAMIC_DEBUG_METADATA'
     127 |  DEFINE_DYNAMIC_DEBUG_METADATA(id, fmt);  \
         |  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/dynamic_debug.h:147:2: note: in expansion of macro '__dynamic_func_call'
     147 |  __dynamic_func_call(__UNIQUE_ID(ddebug), fmt, func, ##__VA_ARGS__)
         |  ^~~~~~~~~~~~~~~~~~~
   include/linux/dynamic_debug.h:161:2: note: in expansion of macro '_dynamic_func_call'
     161 |  _dynamic_func_call(fmt,__dynamic_dev_dbg,   \
         |  ^~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:123:2: note: in expansion of macro 'dynamic_dev_dbg'
     123 |  dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
         |  ^~~~~~~~~~~~~~~
   drivers/staging/media/omap4iss/iss_csi2.c:602:2: note: in expansion of macro 'dev_dbg'
     602 |  dev_dbg(iss->dev, "###CSI2 " #name "=0x%08x\n",
         |  ^~~~~~~
>> drivers/staging/media/omap4iss/iss_csi2.c:602:13: error: invalid type argument of '->' (have 'int')
     602 |  dev_dbg(iss->dev, "###CSI2 " #name "=0x%08x\n",
         |             ^~
   include/linux/dynamic_debug.h:129:15: note: in definition of macro '__dynamic_func_call'
     129 |   func(&id, ##__VA_ARGS__);  \
         |               ^~~~~~~~~~~
   include/linux/dynamic_debug.h:161:2: note: in expansion of macro '_dynamic_func_call'
     161 |  _dynamic_func_call(fmt,__dynamic_dev_dbg,   \
         |  ^~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:123:2: note: in expansion of macro 'dynamic_dev_dbg'
     123 |  dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
         |  ^~~~~~~~~~~~~~~
   drivers/staging/media/omap4iss/iss_csi2.c:602:2: note: in expansion of macro 'dev_dbg'
     602 |  dev_dbg(iss->dev, "###CSI2 " #name "=0x%08x\n",
         |  ^~~~~~~
>> drivers/staging/media/omap4iss/iss_csi2.c:602:31: error: stray '#' in program
     602 |  dev_dbg(iss->dev, "###CSI2 " #name "=0x%08x\n",
         |                               ^
   include/linux/dynamic_debug.h:129:15: note: in definition of macro '__dynamic_func_call'
     129 |   func(&id, ##__VA_ARGS__);  \
         |               ^~~~~~~~~~~
   include/linux/dynamic_debug.h:161:2: note: in expansion of macro '_dynamic_func_call'
     161 |  _dynamic_func_call(fmt,__dynamic_dev_dbg,   \
         |  ^~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:123:2: note: in expansion of macro 'dynamic_dev_dbg'
     123 |  dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
         |  ^~~~~~~~~~~~~~~
   include/linux/dev_printk.h:123:23: note: in expansion of macro 'dev_fmt'
     123 |  dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                       ^~~~~~~
   drivers/staging/media/omap4iss/iss_csi2.c:602:2: note: in expansion of macro 'dev_dbg'
     602 |  dev_dbg(iss->dev, "###CSI2 " #name "=0x%08x\n",
         |  ^~~~~~~
>> drivers/staging/media/omap4iss/iss_csi2.c:602:32: error: expected ')' before 'name'
     602 |  dev_dbg(iss->dev, "###CSI2 " #name "=0x%08x\n",
         |                                ^~~~
   include/linux/dynamic_debug.h:129:15: note: in definition of macro '__dynamic_func_call'
     129 |   func(&id, ##__VA_ARGS__);  \
         |               ^~~~~~~~~~~
   include/linux/dynamic_debug.h:161:2: note: in expansion of macro '_dynamic_func_call'
     161 |  _dynamic_func_call(fmt,__dynamic_dev_dbg,   \
         |  ^~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:123:2: note: in expansion of macro 'dynamic_dev_dbg'
     123 |  dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
         |  ^~~~~~~~~~~~~~~
   include/linux/dev_printk.h:123:23: note: in expansion of macro 'dev_fmt'
     123 |  dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                       ^~~~~~~
   drivers/staging/media/omap4iss/iss_csi2.c:602:2: note: in expansion of macro 'dev_dbg'
     602 |  dev_dbg(iss->dev, "###CSI2 " #name "=0x%08x\n",
         |  ^~~~~~~
   drivers/staging/media/omap4iss/iss_csi2.c:603:32: error: stray '##' in program
     603 |   iss_reg_read(iss, regs, CSI2_##name));
         |                                ^~
   include/linux/dynamic_debug.h:129:15: note: in definition of macro '__dynamic_func_call'
     129 |   func(&id, ##__VA_ARGS__);  \
         |               ^~~~~~~~~~~
   include/linux/dynamic_debug.h:161:2: note: in expansion of macro '_dynamic_func_call'
     161 |  _dynamic_func_call(fmt,__dynamic_dev_dbg,   \
         |  ^~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:123:2: note: in expansion of macro 'dynamic_dev_dbg'
     123 |  dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
         |  ^~~~~~~~~~~~~~~
   drivers/staging/media/omap4iss/iss_csi2.c:602:2: note: in expansion of macro 'dev_dbg'
     602 |  dev_dbg(iss->dev, "###CSI2 " #name "=0x%08x\n",
         |  ^~~~~~~
   drivers/staging/media/omap4iss/iss_csi2.c: In function 'csi2_print_status':
>> drivers/staging/media/omap4iss/iss_csi2.c:616:40: error: 'SYSCONFIG' undeclared (first use in this function); did you mean 'RSZ_SYSCONFIG'?
     616 |  csi2_print_register(iss, csi2->regs1, SYSCONFIG);
         |                                        ^~~~~~~~~
         |                                        RSZ_SYSCONFIG
   drivers/staging/media/omap4iss/iss_csi2.c:616:40: note: each undeclared identifier is reported only once for each function it appears in
>> drivers/staging/media/omap4iss/iss_csi2.c:617:40: error: 'SYSSTATUS' undeclared (first use in this function)
     617 |  csi2_print_register(iss, csi2->regs1, SYSSTATUS);
         |                                        ^~~~~~~~~
>> drivers/staging/media/omap4iss/iss_csi2.c:618:40: error: 'IRQENABLE' undeclared (first use in this function); did you mean 'IF_ENABLED'?
     618 |  csi2_print_register(iss, csi2->regs1, IRQENABLE);
         |                                        ^~~~~~~~~
         |                                        IF_ENABLED
>> drivers/staging/media/omap4iss/iss_csi2.c:619:40: error: 'IRQSTATUS' undeclared (first use in this function)
     619 |  csi2_print_register(iss, csi2->regs1, IRQSTATUS);
         |                                        ^~~~~~~~~
>> drivers/staging/media/omap4iss/iss_csi2.c:620:40: error: 'CTRL' undeclared (first use in this function)
     620 |  csi2_print_register(iss, csi2->regs1, CTRL);
         |                                        ^~~~
>> drivers/staging/media/omap4iss/iss_csi2.c:621:40: error: 'DBG_H' undeclared (first use in this function)
     621 |  csi2_print_register(iss, csi2->regs1, DBG_H);
         |                                        ^~~~~
>> drivers/staging/media/omap4iss/iss_csi2.c:622:40: error: 'COMPLEXIO_CFG' undeclared (first use in this function); did you mean 'CSI2_COMPLEXIO_CFG'?
     622 |  csi2_print_register(iss, csi2->regs1, COMPLEXIO_CFG);
         |                                        ^~~~~~~~~~~~~
         |                                        CSI2_COMPLEXIO_CFG
>> drivers/staging/media/omap4iss/iss_csi2.c:623:40: error: 'COMPLEXIO_IRQSTATUS' undeclared (first use in this function); did you mean 'CSI2_COMPLEXIO_IRQSTATUS'?
     623 |  csi2_print_register(iss, csi2->regs1, COMPLEXIO_IRQSTATUS);
         |                                        ^~~~~~~~~~~~~~~~~~~
         |                                        CSI2_COMPLEXIO_IRQSTATUS
>> drivers/staging/media/omap4iss/iss_csi2.c:624:40: error: 'SHORT_PACKET' undeclared (first use in this function); did you mean 'SHORT_PACKET_IRQ'?
     624 |  csi2_print_register(iss, csi2->regs1, SHORT_PACKET);
         |                                        ^~~~~~~~~~~~
         |                                        SHORT_PACKET_IRQ
>> drivers/staging/media/omap4iss/iss_csi2.c:625:40: error: 'COMPLEXIO_IRQENABLE' undeclared (first use in this function); did you mean 'CSI2_COMPLEXIO_IRQENABLE'?
     625 |  csi2_print_register(iss, csi2->regs1, COMPLEXIO_IRQENABLE);
         |                                        ^~~~~~~~~~~~~~~~~~~
         |                                        CSI2_COMPLEXIO_IRQENABLE
   drivers/staging/media/omap4iss/iss_csi2.c:626:40: error: 'DBG_P' undeclared (first use in this function)
     626 |  csi2_print_register(iss, csi2->regs1, DBG_P);
         |                                        ^~~~~
   drivers/staging/media/omap4iss/iss_csi2.c:627:40: error: 'TIMING' undeclared (first use in this function); did you mean 'TIME_INS'?
     627 |  csi2_print_register(iss, csi2->regs1, TIMING);
         |                                        ^~~~~~
         |                                        TIME_INS
   drivers/staging/media/omap4iss/iss_csi2.c:628:40: error: implicit declaration of function 'CTX_CTRL1'; did you mean 'BTE_CTRL'? [-Werror=implicit-function-declaration]
     628 |  csi2_print_register(iss, csi2->regs1, CTX_CTRL1(0));
         |                                        ^~~~~~~~~
         |                                        BTE_CTRL
   drivers/staging/media/omap4iss/iss_csi2.c:629:40: error: implicit declaration of function 'CTX_CTRL2'; did you mean 'BTE_CTRL'? [-Werror=implicit-function-declaration]
     629 |  csi2_print_register(iss, csi2->regs1, CTX_CTRL2(0));
         |                                        ^~~~~~~~~
         |                                        BTE_CTRL
   drivers/staging/media/omap4iss/iss_csi2.c:630:40: error: implicit declaration of function 'CTX_DAT_OFST'; did you mean 'CSI2_CTX_DAT_OFST'? [-Werror=implicit-function-declaration]
     630 |  csi2_print_register(iss, csi2->regs1, CTX_DAT_OFST(0));
         |                                        ^~~~~~~~~~~~
         |                                        CSI2_CTX_DAT_OFST
   drivers/staging/media/omap4iss/iss_csi2.c:631:40: error: implicit declaration of function 'CTX_PING_ADDR'; did you mean 'CSI2_CTX_PING_ADDR'? [-Werror=implicit-function-declaration]
     631 |  csi2_print_register(iss, csi2->regs1, CTX_PING_ADDR(0));
         |                                        ^~~~~~~~~~~~~
         |                                        CSI2_CTX_PING_ADDR
   drivers/staging/media/omap4iss/iss_csi2.c:632:40: error: implicit declaration of function 'CTX_PONG_ADDR'; did you mean 'CSI2_CTX_PONG_ADDR'? [-Werror=implicit-function-declaration]
     632 |  csi2_print_register(iss, csi2->regs1, CTX_PONG_ADDR(0));
         |                                        ^~~~~~~~~~~~~
         |                                        CSI2_CTX_PONG_ADDR
   drivers/staging/media/omap4iss/iss_csi2.c:633:40: error: implicit declaration of function 'CTX_IRQENABLE'; did you mean 'CSI2_IRQENABLE'? [-Werror=implicit-function-declaration]
     633 |  csi2_print_register(iss, csi2->regs1, CTX_IRQENABLE(0));
         |                                        ^~~~~~~~~~~~~
         |                                        CSI2_IRQENABLE
   drivers/staging/media/omap4iss/iss_csi2.c:634:40: error: implicit declaration of function 'CTX_IRQSTATUS'; did you mean 'CSI2_IRQSTATUS'? [-Werror=implicit-function-declaration]
     634 |  csi2_print_register(iss, csi2->regs1, CTX_IRQSTATUS(0));
         |                                        ^~~~~~~~~~~~~
         |                                        CSI2_IRQSTATUS
   drivers/staging/media/omap4iss/iss_csi2.c:635:40: error: implicit declaration of function 'CTX_CTRL3'; did you mean 'BTE_CTRL'? [-Werror=implicit-function-declaration]
     635 |  csi2_print_register(iss, csi2->regs1, CTX_CTRL3(0));
         |                                        ^~~~~~~~~
         |                                        BTE_CTRL
   drivers/staging/media/omap4iss/iss_csi2.c: In function 'csi2_print_register':
   drivers/staging/media/omap4iss/iss_csi2.c:604:1: error: control reaches end of non-void function [-Werror=return-type]
     604 | }
         | ^
   cc1: some warnings being treated as errors

Kconfig warnings: (for reference only)
   WARNING: unmet direct dependencies detected for SND_ATMEL_SOC_PDC
   Depends on SOUND && !UML && SND && SND_SOC && SND_ATMEL_SOC && HAS_DMA
   Selected by
   - SND_ATMEL_SOC_SSC && SOUND && !UML && SND && SND_SOC && SND_ATMEL_SOC
   - SND_ATMEL_SOC_SSC_PDC && SOUND && !UML && SND && SND_SOC && SND_ATMEL_SOC && ATMEL_SSC


vim +600 drivers/staging/media/omap4iss/iss_csi2.c

   596	
   597	/*
   598	 * csi2_print_status - Prints CSI2 debug information.
   599	 */
 > 600	static inline csi2_print_register(iss, regs, name)
   601	{
 > 602		dev_dbg(iss->dev, "###CSI2 " #name "=0x%08x\n",
   603			iss_reg_read(iss, regs, CSI2_##name));
   604	}
   605	
   606	
   607	static void csi2_print_status(struct iss_csi2_device *csi2)
   608	{
   609		struct iss_device *iss = csi2->iss;
   610	
   611		if (!csi2->available)
   612			return;
   613	
   614		dev_dbg(iss->dev, "-------------CSI2 Register dump-------------\n");
   615	
 > 616		csi2_print_register(iss, csi2->regs1, SYSCONFIG);
 > 617		csi2_print_register(iss, csi2->regs1, SYSSTATUS);
 > 618		csi2_print_register(iss, csi2->regs1, IRQENABLE);
 > 619		csi2_print_register(iss, csi2->regs1, IRQSTATUS);
 > 620		csi2_print_register(iss, csi2->regs1, CTRL);
 > 621		csi2_print_register(iss, csi2->regs1, DBG_H);
 > 622		csi2_print_register(iss, csi2->regs1, COMPLEXIO_CFG);
 > 623		csi2_print_register(iss, csi2->regs1, COMPLEXIO_IRQSTATUS);
 > 624		csi2_print_register(iss, csi2->regs1, SHORT_PACKET);
 > 625		csi2_print_register(iss, csi2->regs1, COMPLEXIO_IRQENABLE);
 > 626		csi2_print_register(iss, csi2->regs1, DBG_P);
 > 627		csi2_print_register(iss, csi2->regs1, TIMING);
 > 628		csi2_print_register(iss, csi2->regs1, CTX_CTRL1(0));
 > 629		csi2_print_register(iss, csi2->regs1, CTX_CTRL2(0));
 > 630		csi2_print_register(iss, csi2->regs1, CTX_DAT_OFST(0));
 > 631		csi2_print_register(iss, csi2->regs1, CTX_PING_ADDR(0));
 > 632		csi2_print_register(iss, csi2->regs1, CTX_PONG_ADDR(0));
 > 633		csi2_print_register(iss, csi2->regs1, CTX_IRQENABLE(0));
 > 634		csi2_print_register(iss, csi2->regs1, CTX_IRQSTATUS(0));
 > 635		csi2_print_register(iss, csi2->regs1, CTX_CTRL3(0));
   636	
   637		dev_dbg(iss->dev, "--------------------------------------------\n");
   638	}
   639	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux