[bvanassche:scsi-const-host-template 10/11] drivers/ata/pata_amd.c:575:50: warning: passing argument 3 of 'ata_pci_bmdma_init_one' discards 'const' qualifier from pointer target type

From: kernel test robot
Date: Tue Aug 09 2022 - 03:04:55 EST

Next message: kernel test robot: "[bvanassche:scsi-const-host-template 10/11] drivers/ata/ahci_seattle.c:169:10: error: passing 'const struct scsi_host_template *' to parameter of type 'struct scsi_host_template *' discards qualifiers"
Previous message: kernel test robot: "[intel-amx:init_fpstate 2/3] arch/x86/kernel/fpu/xstate.c:362:24: warning: format specifies type 'unsigned int' but the argument has type 'unsigned long'"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

tree: https://github.com/bvanassche/linux scsi-const-host-template
head: 86b323713ebb1d9a9b2f35df76be3954ba7f1759
commit: 579544e409bc1067fb7a362db251e6061b2ca2b2 [10/11] scsi: Declare most SCSI host templates const
config: i386-defconfig (https://download.01.org/0day-ci/archive/20220809/202208091539.kxKLGtDn-lkp@xxxxxxxxx/config)
compiler: gcc-11 (Debian 11.3.0-3) 11.3.0
reproduce (this is a W=1 build):
# https://github.com/bvanassche/linux/commit/579544e409bc1067fb7a362db251e6061b2ca2b2
git remote add bvanassche https://github.com/bvanassche/linux
git fetch --no-tags bvanassche scsi-const-host-template
git checkout 579544e409bc1067fb7a362db251e6061b2ca2b2
# save the config file
mkdir build_dir && cp config build_dir/.config
make W=1 O=build_dir ARCH=i386 SHELL=/bin/bash

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@xxxxxxxxx>

All warnings (new ones prefixed by >>):

drivers/ata/pata_amd.c: In function 'amd_init_one':
>> drivers/ata/pata_amd.c:575:50: warning: passing argument 3 of 'ata_pci_bmdma_init_one' discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
575 | return ata_pci_bmdma_init_one(pdev, ppi, &amd_sht, hpriv, 0);
| ^~~~~~~~
In file included from drivers/ata/pata_amd.c:24:
include/linux/libata.h:2001:62: note: expected 'struct scsi_host_template *' but argument is of type 'const struct scsi_host_template *'
2001 | struct scsi_host_template *sht,
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~
--
drivers/ata/pata_oldpiix.c: In function 'oldpiix_init_one':
>> drivers/ata/pata_oldpiix.c:248:50: warning: passing argument 3 of 'ata_pci_bmdma_init_one' discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
248 | return ata_pci_bmdma_init_one(pdev, ppi, &oldpiix_sht, NULL, 0);
| ^~~~~~~~~~~~
In file included from drivers/ata/pata_oldpiix.c:24:
include/linux/libata.h:2001:62: note: expected 'struct scsi_host_template *' but argument is of type 'const struct scsi_host_template *'
2001 | struct scsi_host_template *sht,
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~
--
drivers/ata/pata_sch.c: In function 'sch_init_one':
>> drivers/ata/pata_sch.c:164:50: warning: passing argument 3 of 'ata_pci_bmdma_init_one' discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
164 | return ata_pci_bmdma_init_one(pdev, ppi, &sch_sht, NULL, 0);
| ^~~~~~~~
In file included from drivers/ata/pata_sch.c:21:
include/linux/libata.h:2001:62: note: expected 'struct scsi_host_template *' but argument is of type 'const struct scsi_host_template *'
2001 | struct scsi_host_template *sht,
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~
--
drivers/ata/ata_generic.c: In function 'ata_generic_init_one':
>> drivers/ata/ata_generic.c:209:49: warning: passing argument 3 of 'ata_pci_bmdma_init_one' discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
209 | return ata_pci_bmdma_init_one(dev, ppi, &generic_sht, (void *)id, 0);
| ^~~~~~~~~~~~
In file included from drivers/ata/ata_generic.c:25:
include/linux/libata.h:2001:62: note: expected 'struct scsi_host_template *' but argument is of type 'const struct scsi_host_template *'
2001 | struct scsi_host_template *sht,
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~

vim +575 drivers/ata/pata_amd.c

c48052cc36e02f Alan Cox 2009-02-11 457
669a5db411d85a Jeff Garzik 2006-08-29 458 static int amd_init_one(struct pci_dev *pdev, const struct pci_device_id *id)
669a5db411d85a Jeff Garzik 2006-08-29 459 {
1626aeb881236c Tejun Heo 2007-05-04 460 static const struct ata_port_info info[10] = {
14bdef982caeda Erik Inge Bolsø 2009-03-14 461 { /* 0: AMD 7401 - no swdma */
1d2808fd3d2d5d Jeff Garzik 2007-05-28 462 .flags = ATA_FLAG_SLAVE_POSS,
14bdef982caeda Erik Inge Bolsø 2009-03-14 463 .pio_mask = ATA_PIO4,
14bdef982caeda Erik Inge Bolsø 2009-03-14 464 .mwdma_mask = ATA_MWDMA2,
14bdef982caeda Erik Inge Bolsø 2009-03-14 465 .udma_mask = ATA_UDMA2,
669a5db411d85a Jeff Garzik 2006-08-29 466 .port_ops = &amd33_port_ops
669a5db411d85a Jeff Garzik 2006-08-29 467 },
669a5db411d85a Jeff Garzik 2006-08-29 468 { /* 1: Early AMD7409 - no swdma */
1d2808fd3d2d5d Jeff Garzik 2007-05-28 469 .flags = ATA_FLAG_SLAVE_POSS,
14bdef982caeda Erik Inge Bolsø 2009-03-14 470 .pio_mask = ATA_PIO4,
14bdef982caeda Erik Inge Bolsø 2009-03-14 471 .mwdma_mask = ATA_MWDMA2,
14bdef982caeda Erik Inge Bolsø 2009-03-14 472 .udma_mask = ATA_UDMA4,
669a5db411d85a Jeff Garzik 2006-08-29 473 .port_ops = &amd66_port_ops
669a5db411d85a Jeff Garzik 2006-08-29 474 },
14bdef982caeda Erik Inge Bolsø 2009-03-14 475 { /* 2: AMD 7409 */
1d2808fd3d2d5d Jeff Garzik 2007-05-28 476 .flags = ATA_FLAG_SLAVE_POSS,
14bdef982caeda Erik Inge Bolsø 2009-03-14 477 .pio_mask = ATA_PIO4,
14bdef982caeda Erik Inge Bolsø 2009-03-14 478 .mwdma_mask = ATA_MWDMA2,
14bdef982caeda Erik Inge Bolsø 2009-03-14 479 .udma_mask = ATA_UDMA4,
669a5db411d85a Jeff Garzik 2006-08-29 480 .port_ops = &amd66_port_ops
669a5db411d85a Jeff Garzik 2006-08-29 481 },
669a5db411d85a Jeff Garzik 2006-08-29 482 { /* 3: AMD 7411 */
1d2808fd3d2d5d Jeff Garzik 2007-05-28 483 .flags = ATA_FLAG_SLAVE_POSS,
14bdef982caeda Erik Inge Bolsø 2009-03-14 484 .pio_mask = ATA_PIO4,
14bdef982caeda Erik Inge Bolsø 2009-03-14 485 .mwdma_mask = ATA_MWDMA2,
14bdef982caeda Erik Inge Bolsø 2009-03-14 486 .udma_mask = ATA_UDMA5,
669a5db411d85a Jeff Garzik 2006-08-29 487 .port_ops = &amd100_port_ops
669a5db411d85a Jeff Garzik 2006-08-29 488 },
669a5db411d85a Jeff Garzik 2006-08-29 489 { /* 4: AMD 7441 */
1d2808fd3d2d5d Jeff Garzik 2007-05-28 490 .flags = ATA_FLAG_SLAVE_POSS,
14bdef982caeda Erik Inge Bolsø 2009-03-14 491 .pio_mask = ATA_PIO4,
14bdef982caeda Erik Inge Bolsø 2009-03-14 492 .mwdma_mask = ATA_MWDMA2,
14bdef982caeda Erik Inge Bolsø 2009-03-14 493 .udma_mask = ATA_UDMA5,
669a5db411d85a Jeff Garzik 2006-08-29 494 .port_ops = &amd100_port_ops
669a5db411d85a Jeff Garzik 2006-08-29 495 },
14bdef982caeda Erik Inge Bolsø 2009-03-14 496 { /* 5: AMD 8111 - no swdma */
1d2808fd3d2d5d Jeff Garzik 2007-05-28 497 .flags = ATA_FLAG_SLAVE_POSS,
14bdef982caeda Erik Inge Bolsø 2009-03-14 498 .pio_mask = ATA_PIO4,
14bdef982caeda Erik Inge Bolsø 2009-03-14 499 .mwdma_mask = ATA_MWDMA2,
14bdef982caeda Erik Inge Bolsø 2009-03-14 500 .udma_mask = ATA_UDMA6,
669a5db411d85a Jeff Garzik 2006-08-29 501 .port_ops = &amd133_port_ops
669a5db411d85a Jeff Garzik 2006-08-29 502 },
14bdef982caeda Erik Inge Bolsø 2009-03-14 503 { /* 6: AMD 8111 UDMA 100 (Serenade) - no swdma */
1d2808fd3d2d5d Jeff Garzik 2007-05-28 504 .flags = ATA_FLAG_SLAVE_POSS,
14bdef982caeda Erik Inge Bolsø 2009-03-14 505 .pio_mask = ATA_PIO4,
14bdef982caeda Erik Inge Bolsø 2009-03-14 506 .mwdma_mask = ATA_MWDMA2,
14bdef982caeda Erik Inge Bolsø 2009-03-14 507 .udma_mask = ATA_UDMA5,
669a5db411d85a Jeff Garzik 2006-08-29 508 .port_ops = &amd133_port_ops
669a5db411d85a Jeff Garzik 2006-08-29 509 },
669a5db411d85a Jeff Garzik 2006-08-29 510 { /* 7: Nvidia Nforce */
1d2808fd3d2d5d Jeff Garzik 2007-05-28 511 .flags = ATA_FLAG_SLAVE_POSS,
14bdef982caeda Erik Inge Bolsø 2009-03-14 512 .pio_mask = ATA_PIO4,
14bdef982caeda Erik Inge Bolsø 2009-03-14 513 .mwdma_mask = ATA_MWDMA2,
14bdef982caeda Erik Inge Bolsø 2009-03-14 514 .udma_mask = ATA_UDMA5,
669a5db411d85a Jeff Garzik 2006-08-29 515 .port_ops = &nv100_port_ops
669a5db411d85a Jeff Garzik 2006-08-29 516 },
14bdef982caeda Erik Inge Bolsø 2009-03-14 517 { /* 8: Nvidia Nforce2 and later - no swdma */
1d2808fd3d2d5d Jeff Garzik 2007-05-28 518 .flags = ATA_FLAG_SLAVE_POSS,
14bdef982caeda Erik Inge Bolsø 2009-03-14 519 .pio_mask = ATA_PIO4,
14bdef982caeda Erik Inge Bolsø 2009-03-14 520 .mwdma_mask = ATA_MWDMA2,
14bdef982caeda Erik Inge Bolsø 2009-03-14 521 .udma_mask = ATA_UDMA6,
669a5db411d85a Jeff Garzik 2006-08-29 522 .port_ops = &nv133_port_ops
669a5db411d85a Jeff Garzik 2006-08-29 523 },
669a5db411d85a Jeff Garzik 2006-08-29 524 { /* 9: AMD CS5536 (Geode companion) */
1d2808fd3d2d5d Jeff Garzik 2007-05-28 525 .flags = ATA_FLAG_SLAVE_POSS,
14bdef982caeda Erik Inge Bolsø 2009-03-14 526 .pio_mask = ATA_PIO4,
14bdef982caeda Erik Inge Bolsø 2009-03-14 527 .mwdma_mask = ATA_MWDMA2,
14bdef982caeda Erik Inge Bolsø 2009-03-14 528 .udma_mask = ATA_UDMA5,
669a5db411d85a Jeff Garzik 2006-08-29 529 .port_ops = &amd100_port_ops
669a5db411d85a Jeff Garzik 2006-08-29 530 }
669a5db411d85a Jeff Garzik 2006-08-29 531 };
887125e3740283 Tejun Heo 2008-03-25 532 const struct ata_port_info *ppi[] = { NULL, NULL };
669a5db411d85a Jeff Garzik 2006-08-29 533 int type = id->driver_data;
887125e3740283 Tejun Heo 2008-03-25 534 void *hpriv = NULL;
669a5db411d85a Jeff Garzik 2006-08-29 535 u8 fifo;
f08048e94564d0 Tejun Heo 2008-03-25 536 int rc;
669a5db411d85a Jeff Garzik 2006-08-29 537
06296a1e684bcd Joe Perches 2011-04-15 538 ata_print_version_once(&pdev->dev, DRV_VERSION);
669a5db411d85a Jeff Garzik 2006-08-29 539
f08048e94564d0 Tejun Heo 2008-03-25 540 rc = pcim_enable_device(pdev);
f08048e94564d0 Tejun Heo 2008-03-25 541 if (rc)
f08048e94564d0 Tejun Heo 2008-03-25 542 return rc;
f08048e94564d0 Tejun Heo 2008-03-25 543
669a5db411d85a Jeff Garzik 2006-08-29 544 pci_read_config_byte(pdev, 0x41, &fifo);
669a5db411d85a Jeff Garzik 2006-08-29 545
669a5db411d85a Jeff Garzik 2006-08-29 546 /* Check for AMD7409 without swdma errata and if found adjust type */
44c10138fd4bbc Auke Kok 2007-06-08 547 if (type == 1 && pdev->revision > 0x7)
669a5db411d85a Jeff Garzik 2006-08-29 548 type = 2;
669a5db411d85a Jeff Garzik 2006-08-29 549
ce54d161630211 Tejun Heo 2007-12-18 550 /* Serenade ? */
ce54d161630211 Tejun Heo 2007-12-18 551 if (type == 5 && pdev->subsystem_vendor == PCI_VENDOR_ID_AMD &&
ce54d161630211 Tejun Heo 2007-12-18 552 pdev->subsystem_device == PCI_DEVICE_ID_AMD_SERENADE)
ce54d161630211 Tejun Heo 2007-12-18 553 type = 6; /* UDMA 100 only */
ce54d161630211 Tejun Heo 2007-12-18 554
ce54d161630211 Tejun Heo 2007-12-18 555 /*
ce54d161630211 Tejun Heo 2007-12-18 556 * Okay, type is determined now. Apply type-specific workarounds.
ce54d161630211 Tejun Heo 2007-12-18 557 */
887125e3740283 Tejun Heo 2008-03-25 558 ppi[0] = &info[type];
ce54d161630211 Tejun Heo 2007-12-18 559
ce54d161630211 Tejun Heo 2007-12-18 560 if (type < 3)
9363c3825ea9ad Tejun Heo 2008-04-07 561 ata_pci_bmdma_clear_simplex(pdev);
c48052cc36e02f Alan Cox 2009-02-11 562 if (pdev->vendor == PCI_VENDOR_ID_AMD)
c48052cc36e02f Alan Cox 2009-02-11 563 amd_clear_fifo(pdev);
ce54d161630211 Tejun Heo 2007-12-18 564 /* Cable detection on Nvidia chips doesn't work too well,
ce54d161630211 Tejun Heo 2007-12-18 565 * cache BIOS programmed UDMA mode.
ce54d161630211 Tejun Heo 2007-12-18 566 */
ce54d161630211 Tejun Heo 2007-12-18 567 if (type == 7 || type == 8) {
ce54d161630211 Tejun Heo 2007-12-18 568 u32 udma;
669a5db411d85a Jeff Garzik 2006-08-29 569
ce54d161630211 Tejun Heo 2007-12-18 570 pci_read_config_dword(pdev, 0x60, &udma);
887125e3740283 Tejun Heo 2008-03-25 571 hpriv = (void *)(unsigned long)udma;
ce54d161630211 Tejun Heo 2007-12-18 572 }
669a5db411d85a Jeff Garzik 2006-08-29 573
669a5db411d85a Jeff Garzik 2006-08-29 574 /* And fire it up */
1c5afdf7a629d2 Tejun Heo 2010-05-19 @575 return ata_pci_bmdma_init_one(pdev, ppi, &amd_sht, hpriv, 0);
669a5db411d85a Jeff Garzik 2006-08-29 576 }
669a5db411d85a Jeff Garzik 2006-08-29 577

:::::: The code at line 575 was first introduced by commit
:::::: 1c5afdf7a629d2e77de8dd043b97a33dcd7e6dfa libata-sff: separate out BMDMA init

:::::: TO: Tejun Heo <tj@xxxxxxxxxx>
:::::: CC: Jeff Garzik <jgarzik@xxxxxxxxxx>

--
0-DAY CI Kernel Test Service
https://01.org/lkp