[PATCH] fbtft-bus.c:

linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed

* [PATCH] fbtft-bus.c:
@ 2020-06-27  4:50 B K Karthik
  2020-06-27  5:12 ` Greg Kroah-Hartman
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: B K Karthik @ 2020-06-27  4:50 UTC (permalink / raw)
  To: Greg Kroah-Hartman, Jan Sebastian Götte,
	Nicolas Saenz Julienne, dri-devel, linux-fbdev, devel,
	linux-kernel

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

fbtft-bus.c:

fixing ERROR: space prohibited before that close parenthesis ')' by removing that space and ',' in line 65 and 67.

Signed-off-by: B K Karthik <karthik.bk2000@live.com>
---
 drivers/staging/fbtft/fbtft-bus.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/fbtft/fbtft-bus.c b/drivers/staging/fbtft/fbtft-bus.c
index 63c65dd67b17..847cbfbbd766 100644
--- a/drivers/staging/fbtft/fbtft-bus.c
+++ b/drivers/staging/fbtft/fbtft-bus.c
@@ -62,9 +62,9 @@ out:									      \
 }                                                                             \
 EXPORT_SYMBOL(func);
 
-define_fbtft_write_reg(fbtft_write_reg8_bus8, u8, u8, )
+define_fbtft_write_reg(fbtft_write_reg8_bus8, u8, u8)
 define_fbtft_write_reg(fbtft_write_reg16_bus8, __be16, u16, cpu_to_be16)
-define_fbtft_write_reg(fbtft_write_reg16_bus16, u16, u16, )
+define_fbtft_write_reg(fbtft_write_reg16_bus16, u16, u16)
 
 void fbtft_write_reg8_bus9(struct fbtft_par *par, int len, ...)
 {
-- 
2.20.1


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

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

* Re: [PATCH] fbtft-bus.c:
  2020-06-27  4:50 [PATCH] fbtft-bus.c: B K Karthik
@ 2020-06-27  5:12 ` Greg Kroah-Hartman
  2020-06-27  7:43 ` kernel test robot
  2020-06-27  8:41 ` kernel test robot
  2 siblings, 0 replies; 4+ messages in thread
From: Greg Kroah-Hartman @ 2020-06-27  5:12 UTC (permalink / raw)
  To: B K Karthik
  Cc: Jan Sebastian Götte, devel, linux-fbdev, linux-kernel,
	dri-devel, Nicolas Saenz Julienne

On Sat, Jun 27, 2020 at 12:50:04AM -0400, B K Karthik wrote:
> fbtft-bus.c:
> 
> fixing ERROR: space prohibited before that close parenthesis ')' by removing that space and ',' in line 65 and 67.
> 
> Signed-off-by: B K Karthik <karthik.bk2000@live.com>
> ---
>  drivers/staging/fbtft/fbtft-bus.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)


Your subject line is really odd :(

Please read the following text from my patch-bot:

- You did not specify a description of why the patch is needed, or
  possibly, any description at all, in the email body.  Please read the
  section entitled "The canonical patch format" in the kernel file,
  Documentation/SubmittingPatches for what is needed in order to
  properly describe the change.

- You did not write a descriptive Subject: for the patch, allowing Greg,
  and everyone else, to know what this patch is all about.  Please read
  the section entitled "The canonical patch format" in the kernel file,
  Documentation/SubmittingPatches for what a proper Subject: line should
  look like.

And fix up both the subject lines, and the changelog body text so that
it looks correct.  Look at the other patches for this code that have
been accepted as examples of what to do.

thanks,

greg k-h

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

* Re: [PATCH] fbtft-bus.c:
  2020-06-27  4:50 [PATCH] fbtft-bus.c: B K Karthik
  2020-06-27  5:12 ` Greg Kroah-Hartman
@ 2020-06-27  7:43 ` kernel test robot
  2020-06-27  8:41 ` kernel test robot
  2 siblings, 0 replies; 4+ messages in thread
From: kernel test robot @ 2020-06-27  7:43 UTC (permalink / raw)
  To: B K Karthik, Greg Kroah-Hartman, Jan Sebastian Götte,
	Nicolas Saenz Julienne, dri-devel, linux-fbdev, devel,
	linux-kernel
  Cc: kbuild-all

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

Hi K,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on staging/staging-testing]
[also build test ERROR on v5.8-rc2 next-20200626]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use  as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/B-K-Karthik/fbtft-bus-c/20200627-125213
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git 92cd1b5d65f5c67147c7da39a3c2ad7e6ff81027
config: m68k-randconfig-r016-20200624 (attached as .config)
compiler: m68k-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
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=m68k 

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

All errors (new ones prefixed by >>):

   In file included from include/linux/build_bug.h:5,
                    from include/linux/bits.h:23,
                    from include/linux/gpio/consumer.h:5,
                    from drivers/staging/fbtft/fbtft-bus.c:4:
   include/linux/scatterlist.h: In function 'sg_set_buf':
   arch/m68k/include/asm/page_mm.h:169:49: warning: ordered comparison of pointer with null pointer [-Wextra]
     169 | #define virt_addr_valid(kaddr) ((void *)(kaddr) >= (void *)PAGE_OFFSET && (void *)(kaddr) < high_memory)
         |                                                 ^~
   include/linux/compiler.h:78:42: note: in definition of macro 'unlikely'
      78 | # define unlikely(x) __builtin_expect(!!(x), 0)
         |                                          ^
   include/linux/scatterlist.h:143:2: note: in expansion of macro 'BUG_ON'
     143 |  BUG_ON(!virt_addr_valid(buf));
         |  ^~~~~~
   include/linux/scatterlist.h:143:10: note: in expansion of macro 'virt_addr_valid'
     143 |  BUG_ON(!virt_addr_valid(buf));
         |          ^~~~~~~~~~~~~~~
   drivers/staging/fbtft/fbtft-bus.c: At top level:
>> drivers/staging/fbtft/fbtft-bus.c:65:53: error: macro "define_fbtft_write_reg" requires 4 arguments, but only 3 given
      65 | define_fbtft_write_reg(fbtft_write_reg8_bus8, u8, u8)
         |                                                     ^
   drivers/staging/fbtft/fbtft-bus.c:14: note: macro "define_fbtft_write_reg" defined here
      14 | #define define_fbtft_write_reg(func, buffer_type, data_type, modifier)        \
         | 
>> drivers/staging/fbtft/fbtft-bus.c:65:23: error: expected ';' before 'void'
      65 | define_fbtft_write_reg(fbtft_write_reg8_bus8, u8, u8)
         |                       ^
         |                       ;
   drivers/staging/fbtft/fbtft-bus.c:67:57: error: macro "define_fbtft_write_reg" requires 4 arguments, but only 3 given
      67 | define_fbtft_write_reg(fbtft_write_reg16_bus16, u16, u16)
         |                                                         ^
   drivers/staging/fbtft/fbtft-bus.c:14: note: macro "define_fbtft_write_reg" defined here
      14 | #define define_fbtft_write_reg(func, buffer_type, data_type, modifier)        \
         | 
   drivers/staging/fbtft/fbtft-bus.c:67:23: error: expected ';' before 'void'
      67 | define_fbtft_write_reg(fbtft_write_reg16_bus16, u16, u16)
         |                       ^
         |                       ;
      68 | 
      69 | void fbtft_write_reg8_bus9(struct fbtft_par *par, int len, ...)
         | ~~~~                   

vim +/define_fbtft_write_reg +65 drivers/staging/fbtft/fbtft-bus.c

    64	
  > 65	define_fbtft_write_reg(fbtft_write_reg8_bus8, u8, u8)
    66	define_fbtft_write_reg(fbtft_write_reg16_bus8, __be16, u16, cpu_to_be16)
    67	define_fbtft_write_reg(fbtft_write_reg16_bus16, u16, u16)
    68	

---
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: 31023 bytes --]

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

* Re: [PATCH] fbtft-bus.c:
  2020-06-27  4:50 [PATCH] fbtft-bus.c: B K Karthik
  2020-06-27  5:12 ` Greg Kroah-Hartman
  2020-06-27  7:43 ` kernel test robot
@ 2020-06-27  8:41 ` kernel test robot
  2 siblings, 0 replies; 4+ messages in thread
From: kernel test robot @ 2020-06-27  8:41 UTC (permalink / raw)
  To: B K Karthik, Greg Kroah-Hartman, Jan Sebastian Götte,
	Nicolas Saenz Julienne, dri-devel, linux-fbdev, devel,
	linux-kernel
  Cc: kbuild-all

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

Hi K,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on staging/staging-testing]
[also build test ERROR on v5.8-rc2 next-20200626]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use  as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/B-K-Karthik/fbtft-bus-c/20200627-125213
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git 92cd1b5d65f5c67147c7da39a3c2ad7e6ff81027
config: i386-randconfig-a005-20200624 (attached as .config)
compiler: gcc-7 (Ubuntu 7.5.0-6ubuntu2) 7.5.0
reproduce (this is a W=1 build):
        # save the attached .config to linux build tree
        make W=1 ARCH=i386 

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/fbtft/fbtft-bus.c:65:53: error: macro "define_fbtft_write_reg" requires 4 arguments, but only 3 given
    define_fbtft_write_reg(fbtft_write_reg8_bus8, u8, u8)
                                                        ^
>> drivers/staging/fbtft/fbtft-bus.c:15:1: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'void'
    void func(struct fbtft_par *par, int len, ...)                                \
    ^
>> drivers/staging/fbtft/fbtft-bus.c:66:1: note: in expansion of macro 'define_fbtft_write_reg'
    define_fbtft_write_reg(fbtft_write_reg16_bus8, __be16, u16, cpu_to_be16)
    ^~~~~~~~~~~~~~~~~~~~~~
   drivers/staging/fbtft/fbtft-bus.c:67:57: error: macro "define_fbtft_write_reg" requires 4 arguments, but only 3 given
    define_fbtft_write_reg(fbtft_write_reg16_bus16, u16, u16)
                                                            ^
   drivers/staging/fbtft/fbtft-bus.c:69:1: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'void'
    void fbtft_write_reg8_bus9(struct fbtft_par *par, int len, ...)
    ^~~~

vim +15 drivers/staging/fbtft/fbtft-bus.c

c296d5f9957c03 Thomas Petazzoni   2014-12-31   7  
c296d5f9957c03 Thomas Petazzoni   2014-12-31   8  /*****************************************************************************
c296d5f9957c03 Thomas Petazzoni   2014-12-31   9   *
c296d5f9957c03 Thomas Petazzoni   2014-12-31  10   *   void (*write_reg)(struct fbtft_par *par, int len, ...);
c296d5f9957c03 Thomas Petazzoni   2014-12-31  11   *
c296d5f9957c03 Thomas Petazzoni   2014-12-31  12   *****************************************************************************/
c296d5f9957c03 Thomas Petazzoni   2014-12-31  13  
8d8825b420ffb3 Alfonso Lima Astor 2017-10-17  14  #define define_fbtft_write_reg(func, buffer_type, data_type, modifier)        \
c296d5f9957c03 Thomas Petazzoni   2014-12-31 @15  void func(struct fbtft_par *par, int len, ...)                                \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  16  {                                                                             \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  17  	va_list args;                                                         \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  18  	int i, ret;                                                           \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  19  	int offset = 0;                                                       \
8d8825b420ffb3 Alfonso Lima Astor 2017-10-17  20  	buffer_type *buf = (buffer_type *)par->buf;                           \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  21  									      \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  22  	if (unlikely(par->debug & DEBUG_WRITE_REGISTER)) {                    \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  23  		va_start(args, len);                                          \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  24  		for (i = 0; i < len; i++) {                                   \
cc1c0eea8527bd Renato Soma        2018-04-17  25  			buf[i] = modifier((data_type)va_arg(args,             \
cc1c0eea8527bd Renato Soma        2018-04-17  26  							    unsigned int));   \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  27  		}                                                             \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  28  		va_end(args);                                                 \
cc1c0eea8527bd Renato Soma        2018-04-17  29  		fbtft_par_dbg_hex(DEBUG_WRITE_REGISTER, par,                  \
cc1c0eea8527bd Renato Soma        2018-04-17  30  				  par->info->device, buffer_type, buf, len,   \
cc1c0eea8527bd Renato Soma        2018-04-17  31  				  "%s: ", __func__);                          \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  32  	}                                                                     \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  33  									      \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  34  	va_start(args, len);                                                  \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  35  									      \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  36  	if (par->startbyte) {                                                 \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  37  		*(u8 *)par->buf = par->startbyte;                             \
8d8825b420ffb3 Alfonso Lima Astor 2017-10-17  38  		buf = (buffer_type *)(par->buf + 1);                          \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  39  		offset = 1;                                                   \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  40  	}                                                                     \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  41  									      \
8d8825b420ffb3 Alfonso Lima Astor 2017-10-17  42  	*buf = modifier((data_type)va_arg(args, unsigned int));               \
cc1c0eea8527bd Renato Soma        2018-04-17  43  	ret = fbtft_write_buf_dc(par, par->buf, sizeof(data_type) + offset,   \
cc1c0eea8527bd Renato Soma        2018-04-17  44  				 0);                                          \
e70065fdc11d86 Heiner Kallweit    2017-03-02  45  	if (ret < 0)							      \
e70065fdc11d86 Heiner Kallweit    2017-03-02  46  		goto out;						      \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  47  	len--;                                                                \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  48  									      \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  49  	if (par->startbyte)                                                   \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  50  		*(u8 *)par->buf = par->startbyte | 0x2;                       \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  51  									      \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  52  	if (len) {                                                            \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  53  		i = len;                                                      \
e70065fdc11d86 Heiner Kallweit    2017-03-02  54  		while (i--)						      \
cc1c0eea8527bd Renato Soma        2018-04-17  55  			*buf++ = modifier((data_type)va_arg(args,             \
cc1c0eea8527bd Renato Soma        2018-04-17  56  							    unsigned int));   \
e70065fdc11d86 Heiner Kallweit    2017-03-02  57  		fbtft_write_buf_dc(par, par->buf,			      \
8d8825b420ffb3 Alfonso Lima Astor 2017-10-17  58  				   len * (sizeof(data_type) + offset), 1);    \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  59  	}                                                                     \
e70065fdc11d86 Heiner Kallweit    2017-03-02  60  out:									      \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  61  	va_end(args);                                                         \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  62  }                                                                             \
c296d5f9957c03 Thomas Petazzoni   2014-12-31  63  EXPORT_SYMBOL(func);
c296d5f9957c03 Thomas Petazzoni   2014-12-31  64  
31b9aef1a7ba07 B K Karthik        2020-06-27 @65  define_fbtft_write_reg(fbtft_write_reg8_bus8, u8, u8)
8d8825b420ffb3 Alfonso Lima Astor 2017-10-17 @66  define_fbtft_write_reg(fbtft_write_reg16_bus8, __be16, u16, cpu_to_be16)
31b9aef1a7ba07 B K Karthik        2020-06-27  67  define_fbtft_write_reg(fbtft_write_reg16_bus16, u16, u16)
c296d5f9957c03 Thomas Petazzoni   2014-12-31  68  

---
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: 32688 bytes --]

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

end of thread, other threads:[~2020-06-27  8:41 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-06-27  4:50 [PATCH] fbtft-bus.c: B K Karthik
2020-06-27  5:12 ` Greg Kroah-Hartman
2020-06-27  7:43 ` kernel test robot
2020-06-27  8:41 ` 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).