From: kernel test robot <lkp@intel.com>
To: 412574090@163.com, sudipm.mukherjee@gmail.com
Cc: oe-kbuild-all@lists.linux.dev, linux-kernel@vger.kernel.org,
xiongxin@kylinos.cn, weiyufeng <weiyufeng@kylinos.cn>
Subject: Re: [PATCH 2/4] parport: change struct parport member to iobase
Date: Sat, 26 Oct 2024 05:58:37 +0800 [thread overview]
Message-ID: <202410260507.aPp3LOTd-lkp@intel.com> (raw)
In-Reply-To: <20241025082128.60034-2-412574090@163.com>
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@intel.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
next prev parent reply other threads:[~2024-10-25 21:59 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-25 8:21 [PATCH 1/4] parport: use standard kernel printing functions 412574090
2024-10-25 8:21 ` [PATCH 2/4] parport: change struct parport member to iobase 412574090
2024-10-25 21:58 ` kernel test robot [this message]
2024-10-25 22:29 ` kernel test robot
2024-10-25 8:21 ` [PATCH 3/4] parport: add parport_data struct 412574090
2024-10-25 8:21 ` [PATCH 4/4] parport: add iomem type for parport 412574090
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202410260507.aPp3LOTd-lkp@intel.com \
--to=lkp@intel.com \
--cc=412574090@163.com \
--cc=linux-kernel@vger.kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=sudipm.mukherjee@gmail.com \
--cc=weiyufeng@kylinos.cn \
--cc=xiongxin@kylinos.cn \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.