lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <202410260507.aPp3LOTd-lkp@intel.com>
Date: Sat, 26 Oct 2024 05:58:37 +0800
From: kernel test robot <lkp@...el.com>
To: 412574090@....com, sudipm.mukherjee@...il.com
Cc: oe-kbuild-all@...ts.linux.dev, linux-kernel@...r.kernel.org,
	xiongxin@...inos.cn, weiyufeng <weiyufeng@...inos.cn>
Subject: Re: [PATCH 2/4] parport: change struct parport member to iobase

Hi,

kernel test robot noticed the following build errors:

[auto build test ERROR on linus/master]
[also build test ERROR on v6.12-rc4 next-20241025]
[cannot apply to deller-parisc/for-next mkp-scsi/for-next jejb-scsi/for-next tiwai-sound/for-next tiwai-sound/for-linus]
[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/412574090-163-com/parport-change-struct-parport-member-to-iobase/20241025-162302
base:   linus/master
patch link:    https://lore.kernel.org/r/20241025082128.60034-2-412574090%40163.com
patch subject: [PATCH 2/4] parport: change struct parport member to iobase
config: m68k-allmodconfig (https://download.01.org/0day-ci/archive/20241026/202410260507.aPp3LOTd-lkp@intel.com/config)
compiler: m68k-linux-gcc (GCC) 14.1.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241026/202410260507.aPp3LOTd-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202410260507.aPp3LOTd-lkp@intel.com/

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

   drivers/parport/parport_mfc3.c: In function 'mfc3_write_data':
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:84:17: note: in expansion of macro 'pia'
      84 |         dummy = pia(p)->pprb; /* clears irq bit */
         |                 ^~~
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:86:9: note: in expansion of macro 'pia'
      86 |         pia(p)->pprb = data;
         |         ^~~
   drivers/parport/parport_mfc3.c: In function 'mfc3_read_data':
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:92:16: note: in expansion of macro 'pia'
      92 |         return pia(p)->pprb;
         |                ^~~
   drivers/parport/parport_mfc3.c: In function 'mfc3_write_control':
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:126:9: note: in expansion of macro 'pia'
     126 |         pia(p)->ppra = (pia(p)->ppra & 0x1f) | control_pc_to_mfc3(control);
         |         ^~~
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:126:25: note: in expansion of macro 'pia'
     126 |         pia(p)->ppra = (pia(p)->ppra & 0x1f) | control_pc_to_mfc3(control);
         |                         ^~~
   drivers/parport/parport_mfc3.c: In function 'mfc3_read_control':
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:132:35: note: in expansion of macro 'pia'
     132 |         return control_mfc3_to_pc(pia(p)->ppra & 0xe0);
         |                                   ^~~
   drivers/parport/parport_mfc3.c: In function 'mfc3_read_status':
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:167:36: note: in expansion of macro 'pia'
     167 |         status = status_mfc3_to_pc(pia(p)->ppra & 0x1f);
         |                                    ^~~
   drivers/parport/parport_mfc3.c: In function 'mfc3_interrupt':
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:180:29: note: in expansion of macro 'pia'
     180 |                         if (pia(this_port[i])->crb & 128) { /* Board caused interrupt */
         |                             ^~~
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:181:41: note: in expansion of macro 'pia'
     181 |                                 dummy = pia(this_port[i])->pprb; /* clear irq bit */
         |                                         ^~~
   drivers/parport/parport_mfc3.c: In function 'mfc3_enable_irq':
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:189:9: note: in expansion of macro 'pia'
     189 |         pia(p)->crb |= PIA_C1_ENABLE_IRQ;
         |         ^~~
   drivers/parport/parport_mfc3.c: In function 'mfc3_disable_irq':
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:194:9: note: in expansion of macro 'pia'
     194 |         pia(p)->crb &= ~PIA_C1_ENABLE_IRQ;
         |         ^~~
   drivers/parport/parport_mfc3.c: In function 'mfc3_data_forward':
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:200:9: note: in expansion of macro 'pia'
     200 |         pia(p)->crb &= ~PIA_DDR; /* make data direction register visible */
         |         ^~~
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:201:9: note: in expansion of macro 'pia'
     201 |         pia(p)->pddrb = 255; /* all pins output */
         |         ^~~
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:202:9: note: in expansion of macro 'pia'
     202 |         pia(p)->crb |= PIA_DDR; /* make data register visible - default */
         |         ^~~
   drivers/parport/parport_mfc3.c: In function 'mfc3_data_reverse':
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:208:9: note: in expansion of macro 'pia'
     208 |         pia(p)->crb &= ~PIA_DDR; /* make data direction register visible */
         |         ^~~
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:209:9: note: in expansion of macro 'pia'
     209 |         pia(p)->pddrb = 0; /* all pins input */
         |         ^~~
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:210:9: note: in expansion of macro 'pia'
     210 |         pia(p)->crb |= PIA_DDR; /* make data register visible - default */
         |         ^~~
   drivers/parport/parport_mfc3.c: In function 'mfc3_save_state':
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:223:27: note: in expansion of macro 'pia'
     223 |         s->u.amiga.data = pia(p)->pprb;
         |                           ^~~
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:224:9: note: in expansion of macro 'pia'
     224 |         pia(p)->crb &= ~PIA_DDR;
         |         ^~~
>> drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:225:30: note: in expansion of macro 'pia'
     225 |         s->u.amiga.datadir = pia(p)->pddrb;
         |                              ^~~
   drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:226:9: note: in expansion of macro 'pia'
     226 |         pia(p)->crb |= PIA_DDR;
         |         ^~~
   drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:227:29: note: in expansion of macro 'pia'
     227 |         s->u.amiga.status = pia(p)->ppra;
         |                             ^~~
   drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:228:9: note: in expansion of macro 'pia'
     228 |         pia(p)->cra &= ~PIA_DDR;
         |         ^~~
   drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:229:32: note: in expansion of macro 'pia'
     229 |         s->u.amiga.statusdir = pia(p)->pddrb;
         |                                ^~~
   drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:230:9: note: in expansion of macro 'pia'
     230 |         pia(p)->cra |= PIA_DDR;
         |         ^~~
   drivers/parport/parport_mfc3.c: In function 'mfc3_restore_state':
   drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:235:9: note: in expansion of macro 'pia'
     235 |         pia(p)->pprb = s->u.amiga.data;
         |         ^~~
   drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:236:9: note: in expansion of macro 'pia'
     236 |         pia(p)->crb &= ~PIA_DDR;
         |         ^~~
   drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:237:9: note: in expansion of macro 'pia'
     237 |         pia(p)->pddrb = s->u.amiga.datadir;
         |         ^~~
   drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:238:9: note: in expansion of macro 'pia'
     238 |         pia(p)->crb |= PIA_DDR;
         |         ^~~
   drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:239:9: note: in expansion of macro 'pia'
     239 |         pia(p)->ppra = s->u.amiga.status;
         |         ^~~
   drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:240:9: note: in expansion of macro 'pia'
     240 |         pia(p)->cra &= ~PIA_DDR;
         |         ^~~
   drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:241:9: note: in expansion of macro 'pia'
     241 |         pia(p)->pddrb = s->u.amiga.statusdir;
         |         ^~~
   drivers/parport/parport_mfc3.c:77:39: error: 'struct parport' has no member named 'base'; did you mean 'iobase'?
      77 | #define pia(dev) ((struct pia *)(dev->base))
         |                                       ^~~~
   drivers/parport/parport_mfc3.c:242:9: note: in expansion of macro 'pia'
     242 |         pia(p)->cra |= PIA_DDR;
         |         ^~~
   drivers/parport/parport_mfc3.c: In function 'mfc3_read_control':
   drivers/parport/parport_mfc3.c:133:1: warning: control reaches end of non-void function [-Wreturn-type]
     133 | }
         | ^
   drivers/parport/parport_mfc3.c: In function 'mfc3_read_data':
   drivers/parport/parport_mfc3.c:93:1: warning: control reaches end of non-void function [-Wreturn-type]
      93 | }
         | ^

Kconfig warnings: (for reference only)
   WARNING: unmet direct dependencies detected for GET_FREE_REGION
   Depends on [n]: SPARSEMEM [=n]
   Selected by [m]:
   - RESOURCE_KUNIT_TEST [=m] && RUNTIME_TESTING_MENU [=y] && KUNIT [=m]


vim +77 drivers/parport/parport_mfc3.c

^1da177e4c3f41 Linus Torvalds 2005-04-16   76  
^1da177e4c3f41 Linus Torvalds 2005-04-16  @77  #define pia(dev) ((struct pia *)(dev->base))
^1da177e4c3f41 Linus Torvalds 2005-04-16   78  static struct parport_operations pp_mfc3_ops;
^1da177e4c3f41 Linus Torvalds 2005-04-16   79  
^1da177e4c3f41 Linus Torvalds 2005-04-16   80  static void mfc3_write_data(struct parport *p, unsigned char data)
^1da177e4c3f41 Linus Torvalds 2005-04-16   81  {
46aea0d41cf836 Joe Perches    2020-04-03   82  	pr_debug("write_data %c\n", data);
^1da177e4c3f41 Linus Torvalds 2005-04-16   83  
^1da177e4c3f41 Linus Torvalds 2005-04-16   84  	dummy = pia(p)->pprb; /* clears irq bit */
^1da177e4c3f41 Linus Torvalds 2005-04-16   85  	/* Triggers also /STROBE.*/
^1da177e4c3f41 Linus Torvalds 2005-04-16   86  	pia(p)->pprb = data;
^1da177e4c3f41 Linus Torvalds 2005-04-16   87  }
^1da177e4c3f41 Linus Torvalds 2005-04-16   88  
^1da177e4c3f41 Linus Torvalds 2005-04-16   89  static unsigned char mfc3_read_data(struct parport *p)
^1da177e4c3f41 Linus Torvalds 2005-04-16   90  {
^1da177e4c3f41 Linus Torvalds 2005-04-16   91  	/* clears interrupt bit. Triggers also /STROBE. */
^1da177e4c3f41 Linus Torvalds 2005-04-16   92  	return pia(p)->pprb;
^1da177e4c3f41 Linus Torvalds 2005-04-16   93  }
^1da177e4c3f41 Linus Torvalds 2005-04-16   94  
^1da177e4c3f41 Linus Torvalds 2005-04-16   95  static unsigned char control_pc_to_mfc3(unsigned char control)
^1da177e4c3f41 Linus Torvalds 2005-04-16   96  {
^1da177e4c3f41 Linus Torvalds 2005-04-16   97  	unsigned char ret = 32|64;
^1da177e4c3f41 Linus Torvalds 2005-04-16   98  
^1da177e4c3f41 Linus Torvalds 2005-04-16   99  	if (control & PARPORT_CONTROL_SELECT) /* XXX: What is SELECP? */
^1da177e4c3f41 Linus Torvalds 2005-04-16  100  		ret &= ~32; /* /SELECT_IN */
^1da177e4c3f41 Linus Torvalds 2005-04-16  101  	if (control & PARPORT_CONTROL_INIT) /* INITP */
^1da177e4c3f41 Linus Torvalds 2005-04-16  102  		ret |= 128;
^1da177e4c3f41 Linus Torvalds 2005-04-16  103  	if (control & PARPORT_CONTROL_AUTOFD) /* AUTOLF */
^1da177e4c3f41 Linus Torvalds 2005-04-16  104  		ret &= ~64;
1d6b84d29779f8 Arnd Bergmann  2024-04-05  105  	/* PARPORT_CONTROL_STROBE handled directly by hardware */
^1da177e4c3f41 Linus Torvalds 2005-04-16  106  	return ret;
^1da177e4c3f41 Linus Torvalds 2005-04-16  107  }
^1da177e4c3f41 Linus Torvalds 2005-04-16  108  
^1da177e4c3f41 Linus Torvalds 2005-04-16  109  static unsigned char control_mfc3_to_pc(unsigned char control)
^1da177e4c3f41 Linus Torvalds 2005-04-16  110  {
^1da177e4c3f41 Linus Torvalds 2005-04-16  111  	unsigned char ret = PARPORT_CONTROL_STROBE 
^1da177e4c3f41 Linus Torvalds 2005-04-16  112  			  | PARPORT_CONTROL_AUTOFD | PARPORT_CONTROL_SELECT;
^1da177e4c3f41 Linus Torvalds 2005-04-16  113  
^1da177e4c3f41 Linus Torvalds 2005-04-16  114  	if (control & 128) /* /INITP */
^1da177e4c3f41 Linus Torvalds 2005-04-16  115  		ret |= PARPORT_CONTROL_INIT;
^1da177e4c3f41 Linus Torvalds 2005-04-16  116  	if (control & 64) /* /AUTOLF */
^1da177e4c3f41 Linus Torvalds 2005-04-16  117  		ret &= ~PARPORT_CONTROL_AUTOFD;
^1da177e4c3f41 Linus Torvalds 2005-04-16  118  	if (control & 32) /* /SELECT_IN */
^1da177e4c3f41 Linus Torvalds 2005-04-16  119  		ret &= ~PARPORT_CONTROL_SELECT;
^1da177e4c3f41 Linus Torvalds 2005-04-16  120  	return ret;
^1da177e4c3f41 Linus Torvalds 2005-04-16  121  }
^1da177e4c3f41 Linus Torvalds 2005-04-16  122  
^1da177e4c3f41 Linus Torvalds 2005-04-16  123  static void mfc3_write_control(struct parport *p, unsigned char control)
^1da177e4c3f41 Linus Torvalds 2005-04-16  124  {
46aea0d41cf836 Joe Perches    2020-04-03  125  	pr_debug("write_control %02x\n", control);
^1da177e4c3f41 Linus Torvalds 2005-04-16  126  	pia(p)->ppra = (pia(p)->ppra & 0x1f) | control_pc_to_mfc3(control);
^1da177e4c3f41 Linus Torvalds 2005-04-16  127  }
^1da177e4c3f41 Linus Torvalds 2005-04-16  128  	
^1da177e4c3f41 Linus Torvalds 2005-04-16  129  static unsigned char mfc3_read_control( struct parport *p)
^1da177e4c3f41 Linus Torvalds 2005-04-16  130  {
46aea0d41cf836 Joe Perches    2020-04-03  131  	pr_debug("read_control\n");
^1da177e4c3f41 Linus Torvalds 2005-04-16  132  	return control_mfc3_to_pc(pia(p)->ppra & 0xe0);
^1da177e4c3f41 Linus Torvalds 2005-04-16 @133  }
^1da177e4c3f41 Linus Torvalds 2005-04-16  134  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ