linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] staging: media: omap4iss: Replace macro function by static inline function in file iss_csi2.c
@ 2021-04-12 13:36 Aline Santana Cordeiro
  2021-04-12 16:46 ` kernel test robot
  0 siblings, 1 reply; 2+ messages in thread
From: Aline Santana Cordeiro @ 2021-04-12 13:36 UTC (permalink / raw)
  To: Laurent Pinchart, Mauro Carvalho Chehab, Greg Kroah-Hartman
  Cc: linux-media, linux-staging, linux-kernel, outreachy-kernel

Replace macro function by static inline function.
Issue suggested by Julia.

Signed-off-by: Aline Santana Cordeiro <alinesantanacordeiro@gmail.com>
---
 drivers/staging/media/omap4iss/iss_csi2.c | 49 ++++++++++++++++---------------
 1 file changed, 26 insertions(+), 23 deletions(-)

diff --git a/drivers/staging/media/omap4iss/iss_csi2.c b/drivers/staging/media/omap4iss/iss_csi2.c
index a6dc2d2..cbeba44 100644
--- a/drivers/staging/media/omap4iss/iss_csi2.c
+++ b/drivers/staging/media/omap4iss/iss_csi2.c
@@ -597,9 +597,12 @@ static int csi2_configure(struct iss_csi2_device *csi2)
 /*
  * csi2_print_status - Prints CSI2 debug information.
  */
-#define CSI2_PRINT_REGISTER(iss, regs, name)\
-	dev_dbg(iss->dev, "###CSI2 " #name "=0x%08x\n", \
-		iss_reg_read(iss, regs, CSI2_##name))
+static inline csi2_print_register(iss, regs, name)
+{
+	dev_dbg(iss->dev, "###CSI2 " #name "=0x%08x\n",
+		iss_reg_read(iss, regs, CSI2_##name));
+}
+
 
 static void csi2_print_status(struct iss_csi2_device *csi2)
 {
@@ -610,26 +613,26 @@ static void csi2_print_status(struct iss_csi2_device *csi2)
 
 	dev_dbg(iss->dev, "-------------CSI2 Register dump-------------\n");
 
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, SYSCONFIG);
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, SYSSTATUS);
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, IRQENABLE);
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, IRQSTATUS);
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, CTRL);
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, DBG_H);
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, COMPLEXIO_CFG);
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, COMPLEXIO_IRQSTATUS);
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, SHORT_PACKET);
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, COMPLEXIO_IRQENABLE);
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, DBG_P);
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, TIMING);
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, CTX_CTRL1(0));
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, CTX_CTRL2(0));
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, CTX_DAT_OFST(0));
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, CTX_PING_ADDR(0));
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, CTX_PONG_ADDR(0));
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, CTX_IRQENABLE(0));
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, CTX_IRQSTATUS(0));
-	CSI2_PRINT_REGISTER(iss, csi2->regs1, CTX_CTRL3(0));
+	csi2_print_register(iss, csi2->regs1, SYSCONFIG);
+	csi2_print_register(iss, csi2->regs1, SYSSTATUS);
+	csi2_print_register(iss, csi2->regs1, IRQENABLE);
+	csi2_print_register(iss, csi2->regs1, IRQSTATUS);
+	csi2_print_register(iss, csi2->regs1, CTRL);
+	csi2_print_register(iss, csi2->regs1, DBG_H);
+	csi2_print_register(iss, csi2->regs1, COMPLEXIO_CFG);
+	csi2_print_register(iss, csi2->regs1, COMPLEXIO_IRQSTATUS);
+	csi2_print_register(iss, csi2->regs1, SHORT_PACKET);
+	csi2_print_register(iss, csi2->regs1, COMPLEXIO_IRQENABLE);
+	csi2_print_register(iss, csi2->regs1, DBG_P);
+	csi2_print_register(iss, csi2->regs1, TIMING);
+	csi2_print_register(iss, csi2->regs1, CTX_CTRL1(0));
+	csi2_print_register(iss, csi2->regs1, CTX_CTRL2(0));
+	csi2_print_register(iss, csi2->regs1, CTX_DAT_OFST(0));
+	csi2_print_register(iss, csi2->regs1, CTX_PING_ADDR(0));
+	csi2_print_register(iss, csi2->regs1, CTX_PONG_ADDR(0));
+	csi2_print_register(iss, csi2->regs1, CTX_IRQENABLE(0));
+	csi2_print_register(iss, csi2->regs1, CTX_IRQSTATUS(0));
+	csi2_print_register(iss, csi2->regs1, CTX_CTRL3(0));
 
 	dev_dbg(iss->dev, "--------------------------------------------\n");
 }
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] staging: media: omap4iss: Replace macro function by static inline function in file iss_csi2.c
  2021-04-12 13:36 [PATCH] staging: media: omap4iss: Replace macro function by static inline function in file iss_csi2.c Aline Santana Cordeiro
@ 2021-04-12 16:46 ` kernel test robot
  0 siblings, 0 replies; 2+ messages in thread
From: kernel test robot @ 2021-04-12 16:46 UTC (permalink / raw)
  To: Aline Santana Cordeiro, Laurent Pinchart, Mauro Carvalho Chehab,
	Greg Kroah-Hartman
  Cc: kbuild-all, linux-media, linux-staging, linux-kernel, outreachy-kernel

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

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@intel.com>

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@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 54268 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-04-12 16:50 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-12 13:36 [PATCH] staging: media: omap4iss: Replace macro function by static inline function in file iss_csi2.c Aline Santana Cordeiro
2021-04-12 16:46 ` kernel test robot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).