Re: [PATCH] [media] sir_ir: annotate hardware config module parameters

[kbuild test robot]

Hi Sean,

[auto build test ERROR on linuxtv-media/master]
[also build test ERROR on v4.12-rc4 next-20170607]
[if your patch is applied to the wrong git tree, please drop us a note to help 
improve the system]

url:
https://github.com/0day-ci/linux/commits/Sean-Young/sir_ir-annotate-hardware-config-module-parameters/20170608-052931
base:   git://linuxtv.org/media_tree.git master
config: x86_64-randconfig-x019-201723 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64 

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

>> drivers/media/rc/sir_ir.c:437:30: error: macro "module_param_hw" requires 4 
>> arguments, but only 3 given
module_param_hw(io, int, 0444);
 ^
>> drivers/media/rc/sir_ir.c:437:1: warning: data definition has no type or 
>> storage class
module_param_hw(io, int, 0444);
^~~
>> drivers/media/rc/sir_ir.c:437:1: error: type defaults to 'int' in 
>> declaration of 'module_param_hw' [-Werror=implicit-int]
   drivers/media/rc/sir_ir.c:440:31: error: macro "module_param_hw" requires 4 
arguments, but only 3 given
module_param_hw(irq, int, 0444);
  ^
   drivers/media/rc/sir_ir.c:440:1: warning: data definition has no type or 
storage class
module_param_hw(irq, int, 0444);
^~~
   drivers/media/rc/sir_ir.c:440:1: error: type defaults to 'int' in 
declaration of 'module_param_hw' [-Werror=implicit-int]
   cc1: some warnings being treated as errors

vim +/module_param_hw +437 drivers/media/rc/sir_ir.c

   431  module_exit(sir_ir_exit);
   432  
   433  MODULE_DESCRIPTION("Infrared receiver driver for SIR type serial 
ports");
   434  MODULE_AUTHOR("Milan Pikula");
   435  MODULE_LICENSE("GPL");
   436  
 > 437  module_param_hw(io, int, 0444);
   438  MODULE_PARM_DESC(io, "I/O address base (0x3f8 or 0x2f8)");
   439  
   440  module_param_hw(irq, int, 0444);

---
0-DAY kernel test infrastructureOpen Source Technology Center
https://lists.01.org/pipermail/kbuild-all   Intel Corporation


.config.gz
Description: application/gzip

Re: [PATCH] [media] sir_ir: annotate hardware config module parameters

[Sean Young]

On Wed, Jun 07, 2017 at 09:38:57AM -0300, Mauro Carvalho Chehab wrote:
> Em Tue,  6 Jun 2017 16:18:19 +0100
> Sean Young <s...@mess.org> escreveu:
> 
> > This module was merged after commit 5a8fc6a3cebb ("Annotate hardware
> > config module parameters in drivers/media/"), so add add the missing
> > hardware annotations.
> 
> This patch seems wrong:
> 
> drivers/media/rc/sir_ir.c:403:1: error: macro "module_param_hw" requires 4 
> arguments, but only 3 given
> drivers/media/rc/sir_ir.c:406:1: error: macro "module_param_hw" requires 4 
> arguments, but only 3 given
> drivers/media/tuners/r820t.c:2341 r820t_attach() error: potential null 
> dereference 'priv'.  (kzalloc returns null)
> drivers/media/rc/sir_ir.c:403:30: error: macro "module_param_hw" requires 4 
> arguments, but only 3 given
>  module_param_hw(io, int, 0444);
>   ^
> drivers/media/rc/sir_ir.c:403:1: warning: data definition has no type or 
> storage class
>  module_param_hw(io, int, 0444);
>  ^~~
> drivers/media/rc/sir_ir.c:403:1: error: type defaults to 'int' in declaration 
> of 'module_param_hw' [-Werror=implicit-int]
> drivers/media/rc/sir_ir.c:406:31: error: macro "module_param_hw" requires 4 
> arguments, but only 3 given
>  module_param_hw(irq, int, 0444);
>^
> drivers/media/rc/sir_ir.c:406:1: warning: data definition has no type or 
> storage class
>  module_param_hw(irq, int, 0444);
>  ^~~
> drivers/media/rc/sir_ir.c:406:1: error: type defaults to 'int' in declaration 
> of 'module_param_hw' [-Werror=implicit-int]
> 
> Please check.

Oh dear, that's embarrassing. :(

Sean

From: Sean Young <s...@mess.org>
Subject: [PATCH] [media] sir_ir: annotate hardware config module parameters

This module was merged after commit 5a8fc6a3cebb ("Annotate hardware
config module parameters in drivers/media/"), so add add the missing
hardware annotations.

Signed-off-by: Sean Young <s...@mess.org>
---
 drivers/media/rc/sir_ir.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/media/rc/sir_ir.c b/drivers/media/rc/sir_ir.c
index 5ee3a23..20234ba 100644
--- a/drivers/media/rc/sir_ir.c
+++ b/drivers/media/rc/sir_ir.c
@@ -400,10 +400,10 @@ MODULE_DESCRIPTION("Infrared receiver driver for SIR type 
serial ports");
 MODULE_AUTHOR("Milan Pikula");
 MODULE_LICENSE("GPL");
 
-module_param(io, int, 0444);
+module_param_hw(io, int, ioport, 0444);
 MODULE_PARM_DESC(io, "I/O address base (0x3f8 or 0x2f8)");
 
-module_param(irq, int, 0444);
+module_param_hw(irq, int, irq, 0444);
 MODULE_PARM_DESC(irq, "Interrupt (4 or 3)");
 
 module_param(threshold, int, 0444);
-- 
2.9.4

Re: [PATCH] [media] sir_ir: annotate hardware config module parameters

[Mauro Carvalho Chehab]

Em Tue,  6 Jun 2017 16:18:19 +0100
Sean Young  escreveu:

> This module was merged after commit 5a8fc6a3cebb ("Annotate hardware
> config module parameters in drivers/media/"), so add add the missing
> hardware annotations.

This patch seems wrong:

drivers/media/rc/sir_ir.c:403:1: error: macro "module_param_hw" requires 4 
arguments, but only 3 given
drivers/media/rc/sir_ir.c:406:1: error: macro "module_param_hw" requires 4 
arguments, but only 3 given
drivers/media/tuners/r820t.c:2341 r820t_attach() error: potential null 
dereference 'priv'.  (kzalloc returns null)
drivers/media/rc/sir_ir.c:403:30: error: macro "module_param_hw" requires 4 
arguments, but only 3 given
 module_param_hw(io, int, 0444);
  ^
drivers/media/rc/sir_ir.c:403:1: warning: data definition has no type or 
storage class
 module_param_hw(io, int, 0444);
 ^~~
drivers/media/rc/sir_ir.c:403:1: error: type defaults to 'int' in declaration 
of 'module_param_hw' [-Werror=implicit-int]
drivers/media/rc/sir_ir.c:406:31: error: macro "module_param_hw" requires 4 
arguments, but only 3 given
 module_param_hw(irq, int, 0444);
   ^
drivers/media/rc/sir_ir.c:406:1: warning: data definition has no type or 
storage class
 module_param_hw(irq, int, 0444);
 ^~~
drivers/media/rc/sir_ir.c:406:1: error: type defaults to 'int' in declaration 
of 'module_param_hw' [-Werror=implicit-int]

Please check.

Regards,
Mauro

Thanks,
Mauro

[PATCH] [media] sir_ir: annotate hardware config module parameters

[Sean Young]

This module was merged after commit 5a8fc6a3cebb ("Annotate hardware
config module parameters in drivers/media/"), so add add the missing
hardware annotations.

Signed-off-by: Sean Young 
---
 drivers/media/rc/sir_ir.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/media/rc/sir_ir.c b/drivers/media/rc/sir_ir.c
index 5ee3a23..f8738e5 100644
--- a/drivers/media/rc/sir_ir.c
+++ b/drivers/media/rc/sir_ir.c
@@ -400,10 +400,10 @@ MODULE_DESCRIPTION("Infrared receiver driver for SIR type 
serial ports");
 MODULE_AUTHOR("Milan Pikula");
 MODULE_LICENSE("GPL");
 
-module_param(io, int, 0444);
+module_param_hw(io, int, 0444);
 MODULE_PARM_DESC(io, "I/O address base (0x3f8 or 0x2f8)");
 
-module_param(irq, int, 0444);
+module_param_hw(irq, int, 0444);
 MODULE_PARM_DESC(irq, "Interrupt (4 or 3)");
 
 module_param(threshold, int, 0444);
-- 
2.9.4