From: kernel test robot <lkp@intel.com>
To: Dave Penkler <dpenkler@gmail.com>,
gregkh@linuxfoundation.org, linux-staging@lists.linux.dev,
linux-kernel@vger.kernel.org
Cc: oe-kbuild-all@lists.linux.dev, Dave Penkler <dpenkler@gmail.com>
Subject: Re: [PATCH v2] staging: gpib: Modernize gpib_interface_t initialization and make static
Date: Thu, 9 Jan 2025 22:48:28 +0800 [thread overview]
Message-ID: <202501092221.k9rTlGeN-lkp@intel.com> (raw)
In-Reply-To: <20250108174121.23044-1-dpenkler@gmail.com>
Hi Dave,
kernel test robot noticed the following build errors:
[auto build test ERROR on staging/staging-testing]
url: https://github.com/intel-lab-lkp/linux/commits/Dave-Penkler/staging-gpib-Modernize-gpib_interface_t-initialization-and-make-static/20250109-014247
base: staging/staging-testing
patch link: https://lore.kernel.org/r/20250108174121.23044-1-dpenkler%40gmail.com
patch subject: [PATCH v2] staging: gpib: Modernize gpib_interface_t initialization and make static
config: loongarch-allyesconfig (https://download.01.org/0day-ci/archive/20250109/202501092221.k9rTlGeN-lkp@intel.com/config)
compiler: loongarch64-linux-gcc (GCC) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250109/202501092221.k9rTlGeN-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/202501092221.k9rTlGeN-lkp@intel.com/
All errors (new ones prefixed by >>):
drivers/staging/gpib/cb7210/cb7210.c: In function 'cb7210_line_status':
drivers/staging/gpib/cb7210/cb7210.c:383:30: warning: variable 'nec_priv' set but not used [-Wunused-but-set-variable]
383 | struct nec7210_priv *nec_priv;
| ^~~~~~~~
drivers/staging/gpib/cb7210/cb7210.c: In function 'cb_gpib_config':
drivers/staging/gpib/cb7210/cb7210.c:1265:28: warning: variable 'dev' set but not used [-Wunused-but-set-variable]
1265 | struct local_info *dev;
| ^~~
drivers/staging/gpib/cb7210/cb7210.c:1264:31: warning: variable 'handle' set but not used [-Wunused-but-set-variable]
1264 | struct pcmcia_device *handle;
| ^~~~~~
drivers/staging/gpib/cb7210/cb7210.c: At top level:
>> drivers/staging/gpib/cb7210/cb7210.c:1360:25: error: static declaration of 'cb_pcmcia_unaccel_interface' follows non-static declaration
1360 | static gpib_interface_t cb_pcmcia_unaccel_interface = {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from drivers/staging/gpib/cb7210/cb7210.c:8:
drivers/staging/gpib/cb7210/cb7210.h:42:25: note: previous declaration of 'cb_pcmcia_unaccel_interface' with type 'gpib_interface_t' {aka 'struct gpib_interface_struct'}
42 | extern gpib_interface_t cb_pcmcia_unaccel_interface;
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/staging/gpib/cb7210/cb7210.c:1388:25: error: static declaration of 'cb_pcmcia_interface' follows non-static declaration
1388 | static gpib_interface_t cb_pcmcia_interface = {
| ^~~~~~~~~~~~~~~~~~~
drivers/staging/gpib/cb7210/cb7210.h:40:25: note: previous declaration of 'cb_pcmcia_interface' with type 'gpib_interface_t' {aka 'struct gpib_interface_struct'}
40 | extern gpib_interface_t cb_pcmcia_interface;
| ^~~~~~~~~~~~~~~~~~~
>> drivers/staging/gpib/cb7210/cb7210.c:1416:25: error: static declaration of 'cb_pcmcia_accel_interface' follows non-static declaration
1416 | static gpib_interface_t cb_pcmcia_accel_interface = {
| ^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/staging/gpib/cb7210/cb7210.h:41:25: note: previous declaration of 'cb_pcmcia_accel_interface' with type 'gpib_interface_t' {aka 'struct gpib_interface_struct'}
41 | extern gpib_interface_t cb_pcmcia_accel_interface;
| ^~~~~~~~~~~~~~~~~~~~~~~~~
vim +/cb_pcmcia_unaccel_interface +1360 drivers/staging/gpib/cb7210/cb7210.c
1359
> 1360 static gpib_interface_t cb_pcmcia_unaccel_interface = {
1361 .name = "cbi_pcmcia_unaccel",
1362 .attach = cb_pcmcia_attach,
1363 .detach = cb_pcmcia_detach,
1364 .read = cb7210_read,
1365 .write = cb7210_write,
1366 .command = cb7210_command,
1367 .take_control = cb7210_take_control,
1368 .go_to_standby = cb7210_go_to_standby,
1369 .request_system_control = cb7210_request_system_control,
1370 .interface_clear = cb7210_interface_clear,
1371 .remote_enable = cb7210_remote_enable,
1372 .enable_eos = cb7210_enable_eos,
1373 .disable_eos = cb7210_disable_eos,
1374 .parallel_poll = cb7210_parallel_poll,
1375 .parallel_poll_configure = cb7210_parallel_poll_configure,
1376 .parallel_poll_response = cb7210_parallel_poll_response,
1377 .local_parallel_poll_mode = NULL, // XXX
1378 .line_status = cb7210_line_status,
1379 .update_status = cb7210_update_status,
1380 .primary_address = cb7210_primary_address,
1381 .secondary_address = cb7210_secondary_address,
1382 .serial_poll_response = cb7210_serial_poll_response,
1383 .serial_poll_status = cb7210_serial_poll_status,
1384 .t1_delay = cb7210_t1_delay,
1385 .return_to_local = cb7210_return_to_local,
1386 };
1387
> 1388 static gpib_interface_t cb_pcmcia_interface = {
1389 .name = "cbi_pcmcia",
1390 .attach = cb_pcmcia_attach,
1391 .detach = cb_pcmcia_detach,
1392 .read = cb7210_accel_read,
1393 .write = cb7210_accel_write,
1394 .command = cb7210_command,
1395 .take_control = cb7210_take_control,
1396 .go_to_standby = cb7210_go_to_standby,
1397 .request_system_control = cb7210_request_system_control,
1398 .interface_clear = cb7210_interface_clear,
1399 .remote_enable = cb7210_remote_enable,
1400 .enable_eos = cb7210_enable_eos,
1401 .disable_eos = cb7210_disable_eos,
1402 .parallel_poll = cb7210_parallel_poll,
1403 .parallel_poll_configure = cb7210_parallel_poll_configure,
1404 .parallel_poll_response = cb7210_parallel_poll_response,
1405 .local_parallel_poll_mode = NULL, // XXX
1406 .line_status = cb7210_line_status,
1407 .update_status = cb7210_update_status,
1408 .primary_address = cb7210_primary_address,
1409 .secondary_address = cb7210_secondary_address,
1410 .serial_poll_response = cb7210_serial_poll_response,
1411 .serial_poll_status = cb7210_serial_poll_status,
1412 .t1_delay = cb7210_t1_delay,
1413 .return_to_local = cb7210_return_to_local,
1414 };
1415
> 1416 static gpib_interface_t cb_pcmcia_accel_interface = {
1417 .name = "cbi_pcmcia_accel",
1418 .attach = cb_pcmcia_attach,
1419 .detach = cb_pcmcia_detach,
1420 .read = cb7210_accel_read,
1421 .write = cb7210_accel_write,
1422 .command = cb7210_command,
1423 .take_control = cb7210_take_control,
1424 .go_to_standby = cb7210_go_to_standby,
1425 .request_system_control = cb7210_request_system_control,
1426 .interface_clear = cb7210_interface_clear,
1427 .remote_enable = cb7210_remote_enable,
1428 .enable_eos = cb7210_enable_eos,
1429 .disable_eos = cb7210_disable_eos,
1430 .parallel_poll = cb7210_parallel_poll,
1431 .parallel_poll_configure = cb7210_parallel_poll_configure,
1432 .parallel_poll_response = cb7210_parallel_poll_response,
1433 .local_parallel_poll_mode = NULL, // XXX
1434 .line_status = cb7210_line_status,
1435 .update_status = cb7210_update_status,
1436 .primary_address = cb7210_primary_address,
1437 .secondary_address = cb7210_secondary_address,
1438 .serial_poll_response = cb7210_serial_poll_response,
1439 .serial_poll_status = cb7210_serial_poll_status,
1440 .t1_delay = cb7210_t1_delay,
1441 .return_to_local = cb7210_return_to_local,
1442 };
1443
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2025-01-09 14:49 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-01-08 17:41 [PATCH v2] staging: gpib: Modernize gpib_interface_t initialization and make static Dave Penkler
2025-01-09 14:48 ` kernel test robot [this message]
2025-01-09 15:46 ` kernel test robot
2025-01-09 20:09 ` kernel test robot
2025-01-10 15:03 ` Greg KH
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=202501092221.k9rTlGeN-lkp@intel.com \
--to=lkp@intel.com \
--cc=dpenkler@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-staging@lists.linux.dev \
--cc=oe-kbuild-all@lists.linux.dev \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox