* arch/powerpc/sysdev/fsl_rmu.c:544: warning: Function parameter or member 'pw' not described in 'fsl_rio_port_write_init'
@ 2023-09-26 23:49 kernel test robot
2023-09-27 8:33 ` Christophe Leroy
0 siblings, 1 reply; 2+ messages in thread
From: kernel test robot @ 2023-09-26 23:49 UTC (permalink / raw)
To: Christophe Leroy; +Cc: oe-kbuild-all, linux-kernel, Michael Ellerman
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 0e945134b680040b8613e962f586d91b6d40292d
commit: c265735ff5b1f13272e2bfb196f5c55f9b3c9bac powerpc/85xx: Mark some functions static and add missing includes to fix no previous prototype error
date: 5 weeks ago
config: powerpc-randconfig-003-20230927 (https://download.01.org/0day-ci/archive/20230927/202309270747.cirkf8k8-lkp@intel.com/config)
compiler: powerpc-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20230927/202309270747.cirkf8k8-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/202309270747.cirkf8k8-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> arch/powerpc/sysdev/fsl_rmu.c:544: warning: Function parameter or member 'pw' not described in 'fsl_rio_port_write_init'
>> arch/powerpc/sysdev/fsl_rmu.c:544: warning: Excess function parameter 'mport' description in 'fsl_rio_port_write_init'
>> arch/powerpc/sysdev/fsl_rmu.c:1028: warning: Function parameter or member 'dbell' not described in 'fsl_rio_doorbell_init'
>> arch/powerpc/sysdev/fsl_rmu.c:1028: warning: Excess function parameter 'mport' description in 'fsl_rio_doorbell_init'
vim +544 arch/powerpc/sysdev/fsl_rmu.c
6ec4bedbf153a8 Liu Gang 2011-11-12 533
6ec4bedbf153a8 Liu Gang 2011-11-12 534 /**
6ec4bedbf153a8 Liu Gang 2011-11-12 535 * fsl_rio_port_write_init - MPC85xx port write interface init
6ec4bedbf153a8 Liu Gang 2011-11-12 536 * @mport: Master port implementing the port write unit
6ec4bedbf153a8 Liu Gang 2011-11-12 537 *
6ec4bedbf153a8 Liu Gang 2011-11-12 538 * Initializes port write unit hardware and DMA buffer
6ec4bedbf153a8 Liu Gang 2011-11-12 539 * ring. Called from fsl_rio_setup(). Returns %0 on success
6ec4bedbf153a8 Liu Gang 2011-11-12 540 * or %-ENOMEM on failure.
6ec4bedbf153a8 Liu Gang 2011-11-12 541 */
6ec4bedbf153a8 Liu Gang 2011-11-12 542
abc3aeae3aaa5c Liu Gang 2011-11-12 543 int fsl_rio_port_write_init(struct fsl_rio_pw *pw)
6ec4bedbf153a8 Liu Gang 2011-11-12 @544 {
6ec4bedbf153a8 Liu Gang 2011-11-12 545 int rc = 0;
6ec4bedbf153a8 Liu Gang 2011-11-12 546
6ec4bedbf153a8 Liu Gang 2011-11-12 547 /* Following configurations require a disabled port write controller */
abc3aeae3aaa5c Liu Gang 2011-11-12 548 out_be32(&pw->pw_regs->pwmr,
abc3aeae3aaa5c Liu Gang 2011-11-12 549 in_be32(&pw->pw_regs->pwmr) & ~RIO_IPWMR_PWE);
6ec4bedbf153a8 Liu Gang 2011-11-12 550
6ec4bedbf153a8 Liu Gang 2011-11-12 551 /* Initialize port write */
abc3aeae3aaa5c Liu Gang 2011-11-12 552 pw->port_write_msg.virt = dma_alloc_coherent(pw->dev,
6ec4bedbf153a8 Liu Gang 2011-11-12 553 RIO_PW_MSG_SIZE,
abc3aeae3aaa5c Liu Gang 2011-11-12 554 &pw->port_write_msg.phys, GFP_KERNEL);
abc3aeae3aaa5c Liu Gang 2011-11-12 555 if (!pw->port_write_msg.virt) {
6ec4bedbf153a8 Liu Gang 2011-11-12 556 pr_err("RIO: unable allocate port write queue\n");
6ec4bedbf153a8 Liu Gang 2011-11-12 557 return -ENOMEM;
6ec4bedbf153a8 Liu Gang 2011-11-12 558 }
6ec4bedbf153a8 Liu Gang 2011-11-12 559
abc3aeae3aaa5c Liu Gang 2011-11-12 560 pw->port_write_msg.err_count = 0;
abc3aeae3aaa5c Liu Gang 2011-11-12 561 pw->port_write_msg.discard_count = 0;
6ec4bedbf153a8 Liu Gang 2011-11-12 562
6ec4bedbf153a8 Liu Gang 2011-11-12 563 /* Point dequeue/enqueue pointers at first entry */
abc3aeae3aaa5c Liu Gang 2011-11-12 564 out_be32(&pw->pw_regs->epwqbar, 0);
abc3aeae3aaa5c Liu Gang 2011-11-12 565 out_be32(&pw->pw_regs->pwqbar, (u32) pw->port_write_msg.phys);
6ec4bedbf153a8 Liu Gang 2011-11-12 566
6ec4bedbf153a8 Liu Gang 2011-11-12 567 pr_debug("EIPWQBAR: 0x%08x IPWQBAR: 0x%08x\n",
abc3aeae3aaa5c Liu Gang 2011-11-12 568 in_be32(&pw->pw_regs->epwqbar),
abc3aeae3aaa5c Liu Gang 2011-11-12 569 in_be32(&pw->pw_regs->pwqbar));
6ec4bedbf153a8 Liu Gang 2011-11-12 570
6ec4bedbf153a8 Liu Gang 2011-11-12 571 /* Clear interrupt status IPWSR */
abc3aeae3aaa5c Liu Gang 2011-11-12 572 out_be32(&pw->pw_regs->pwsr,
6ec4bedbf153a8 Liu Gang 2011-11-12 573 (RIO_IPWSR_TE | RIO_IPWSR_QFI | RIO_IPWSR_PWD));
6ec4bedbf153a8 Liu Gang 2011-11-12 574
446957ba512714 Adam Buchbinder 2016-02-24 575 /* Configure port write controller for snooping enable all reporting,
6ec4bedbf153a8 Liu Gang 2011-11-12 576 clear queue full */
abc3aeae3aaa5c Liu Gang 2011-11-12 577 out_be32(&pw->pw_regs->pwmr,
6ec4bedbf153a8 Liu Gang 2011-11-12 578 RIO_IPWMR_SEN | RIO_IPWMR_QFIE | RIO_IPWMR_EIE | RIO_IPWMR_CQ);
6ec4bedbf153a8 Liu Gang 2011-11-12 579
6ec4bedbf153a8 Liu Gang 2011-11-12 580
6ec4bedbf153a8 Liu Gang 2011-11-12 581 /* Hook up port-write handler */
abc3aeae3aaa5c Liu Gang 2011-11-12 582 rc = request_irq(IRQ_RIO_PW(pw), fsl_rio_port_write_handler,
abc3aeae3aaa5c Liu Gang 2011-11-12 583 IRQF_SHARED, "port-write", (void *)pw);
6ec4bedbf153a8 Liu Gang 2011-11-12 584 if (rc < 0) {
6ec4bedbf153a8 Liu Gang 2011-11-12 585 pr_err("MPC85xx RIO: unable to request inbound doorbell irq");
6ec4bedbf153a8 Liu Gang 2011-11-12 586 goto err_out;
6ec4bedbf153a8 Liu Gang 2011-11-12 587 }
6ec4bedbf153a8 Liu Gang 2011-11-12 588 /* Enable Error Interrupt */
6ec4bedbf153a8 Liu Gang 2011-11-12 589 out_be32((u32 *)(rio_regs_win + RIO_LTLEECSR), LTLEECSR_ENABLE_ALL);
6ec4bedbf153a8 Liu Gang 2011-11-12 590
abc3aeae3aaa5c Liu Gang 2011-11-12 591 INIT_WORK(&pw->pw_work, fsl_pw_dpc);
abc3aeae3aaa5c Liu Gang 2011-11-12 592 spin_lock_init(&pw->pw_fifo_lock);
abc3aeae3aaa5c Liu Gang 2011-11-12 593 if (kfifo_alloc(&pw->pw_fifo, RIO_PW_MSG_SIZE * 32, GFP_KERNEL)) {
6ec4bedbf153a8 Liu Gang 2011-11-12 594 pr_err("FIFO allocation failed\n");
6ec4bedbf153a8 Liu Gang 2011-11-12 595 rc = -ENOMEM;
6ec4bedbf153a8 Liu Gang 2011-11-12 596 goto err_out_irq;
6ec4bedbf153a8 Liu Gang 2011-11-12 597 }
6ec4bedbf153a8 Liu Gang 2011-11-12 598
6ec4bedbf153a8 Liu Gang 2011-11-12 599 pr_debug("IPWMR: 0x%08x IPWSR: 0x%08x\n",
abc3aeae3aaa5c Liu Gang 2011-11-12 600 in_be32(&pw->pw_regs->pwmr),
abc3aeae3aaa5c Liu Gang 2011-11-12 601 in_be32(&pw->pw_regs->pwsr));
6ec4bedbf153a8 Liu Gang 2011-11-12 602
6ec4bedbf153a8 Liu Gang 2011-11-12 603 return rc;
6ec4bedbf153a8 Liu Gang 2011-11-12 604
6ec4bedbf153a8 Liu Gang 2011-11-12 605 err_out_irq:
abc3aeae3aaa5c Liu Gang 2011-11-12 606 free_irq(IRQ_RIO_PW(pw), (void *)pw);
6ec4bedbf153a8 Liu Gang 2011-11-12 607 err_out:
abc3aeae3aaa5c Liu Gang 2011-11-12 608 dma_free_coherent(pw->dev, RIO_PW_MSG_SIZE,
abc3aeae3aaa5c Liu Gang 2011-11-12 609 pw->port_write_msg.virt,
abc3aeae3aaa5c Liu Gang 2011-11-12 610 pw->port_write_msg.phys);
6ec4bedbf153a8 Liu Gang 2011-11-12 611 return rc;
6ec4bedbf153a8 Liu Gang 2011-11-12 612 }
6ec4bedbf153a8 Liu Gang 2011-11-12 613
:::::: The code at line 544 was first introduced by commit
:::::: 6ec4bedbf153a8ef71aeba99a40efef556b57798 fsl-rio: Split rio driver into two parts, RapidIO endpoint and message unit
:::::: TO: Liu Gang <Gang.Liu@freescale.com>
:::::: CC: Kumar Gala <galak@kernel.crashing.org>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: arch/powerpc/sysdev/fsl_rmu.c:544: warning: Function parameter or member 'pw' not described in 'fsl_rio_port_write_init'
2023-09-26 23:49 arch/powerpc/sysdev/fsl_rmu.c:544: warning: Function parameter or member 'pw' not described in 'fsl_rio_port_write_init' kernel test robot
@ 2023-09-27 8:33 ` Christophe Leroy
0 siblings, 0 replies; 2+ messages in thread
From: Christophe Leroy @ 2023-09-27 8:33 UTC (permalink / raw)
To: kernel test robot, Liu Gang, Li Yang
Cc: oe-kbuild-all@lists.linux.dev, linux-kernel@vger.kernel.org,
Michael Ellerman
Le 27/09/2023 à 01:49, kernel test robot a écrit :
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head: 0e945134b680040b8613e962f586d91b6d40292d
> commit: c265735ff5b1f13272e2bfb196f5c55f9b3c9bac powerpc/85xx: Mark some functions static and add missing includes to fix no previous prototype error
> date: 5 weeks ago
> config: powerpc-randconfig-003-20230927 (https://download.01.org/0day-ci/archive/20230927/202309270747.cirkf8k8-lkp@intel.com/config)
> compiler: powerpc-linux-gcc (GCC) 13.2.0
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20230927/202309270747.cirkf8k8-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/202309270747.cirkf8k8-lkp@intel.com/
>
> All warnings (new ones prefixed by >>):
>
>>> arch/powerpc/sysdev/fsl_rmu.c:544: warning: Function parameter or member 'pw' not described in 'fsl_rio_port_write_init'
>>> arch/powerpc/sysdev/fsl_rmu.c:544: warning: Excess function parameter 'mport' description in 'fsl_rio_port_write_init'
>>> arch/powerpc/sysdev/fsl_rmu.c:1028: warning: Function parameter or member 'dbell' not described in 'fsl_rio_doorbell_init'
>>> arch/powerpc/sysdev/fsl_rmu.c:1028: warning: Excess function parameter 'mport' description in 'fsl_rio_doorbell_init'
Those warnings have nothing to do with the above mentioned commit.
They were introduced by commit abc3aeae3aaa ("fsl-rio: Add two ports and
rapidio message units support") in 2011.
>
>
> vim +544 arch/powerpc/sysdev/fsl_rmu.c
>
> 6ec4bedbf153a8 Liu Gang 2011-11-12 533
> 6ec4bedbf153a8 Liu Gang 2011-11-12 534 /**
> 6ec4bedbf153a8 Liu Gang 2011-11-12 535 * fsl_rio_port_write_init - MPC85xx port write interface init
> 6ec4bedbf153a8 Liu Gang 2011-11-12 536 * @mport: Master port implementing the port write unit
> 6ec4bedbf153a8 Liu Gang 2011-11-12 537 *
> 6ec4bedbf153a8 Liu Gang 2011-11-12 538 * Initializes port write unit hardware and DMA buffer
> 6ec4bedbf153a8 Liu Gang 2011-11-12 539 * ring. Called from fsl_rio_setup(). Returns %0 on success
> 6ec4bedbf153a8 Liu Gang 2011-11-12 540 * or %-ENOMEM on failure.
> 6ec4bedbf153a8 Liu Gang 2011-11-12 541 */
> 6ec4bedbf153a8 Liu Gang 2011-11-12 542
> abc3aeae3aaa5c Liu Gang 2011-11-12 543 int fsl_rio_port_write_init(struct fsl_rio_pw *pw)
> 6ec4bedbf153a8 Liu Gang 2011-11-12 @544 {
> 6ec4bedbf153a8 Liu Gang 2011-11-12 545 int rc = 0;
> 6ec4bedbf153a8 Liu Gang 2011-11-12 546
> 6ec4bedbf153a8 Liu Gang 2011-11-12 547 /* Following configurations require a disabled port write controller */
> abc3aeae3aaa5c Liu Gang 2011-11-12 548 out_be32(&pw->pw_regs->pwmr,
> abc3aeae3aaa5c Liu Gang 2011-11-12 549 in_be32(&pw->pw_regs->pwmr) & ~RIO_IPWMR_PWE);
> 6ec4bedbf153a8 Liu Gang 2011-11-12 550
> 6ec4bedbf153a8 Liu Gang 2011-11-12 551 /* Initialize port write */
> abc3aeae3aaa5c Liu Gang 2011-11-12 552 pw->port_write_msg.virt = dma_alloc_coherent(pw->dev,
> 6ec4bedbf153a8 Liu Gang 2011-11-12 553 RIO_PW_MSG_SIZE,
> abc3aeae3aaa5c Liu Gang 2011-11-12 554 &pw->port_write_msg.phys, GFP_KERNEL);
> abc3aeae3aaa5c Liu Gang 2011-11-12 555 if (!pw->port_write_msg.virt) {
> 6ec4bedbf153a8 Liu Gang 2011-11-12 556 pr_err("RIO: unable allocate port write queue\n");
> 6ec4bedbf153a8 Liu Gang 2011-11-12 557 return -ENOMEM;
> 6ec4bedbf153a8 Liu Gang 2011-11-12 558 }
> 6ec4bedbf153a8 Liu Gang 2011-11-12 559
> abc3aeae3aaa5c Liu Gang 2011-11-12 560 pw->port_write_msg.err_count = 0;
> abc3aeae3aaa5c Liu Gang 2011-11-12 561 pw->port_write_msg.discard_count = 0;
> 6ec4bedbf153a8 Liu Gang 2011-11-12 562
> 6ec4bedbf153a8 Liu Gang 2011-11-12 563 /* Point dequeue/enqueue pointers at first entry */
> abc3aeae3aaa5c Liu Gang 2011-11-12 564 out_be32(&pw->pw_regs->epwqbar, 0);
> abc3aeae3aaa5c Liu Gang 2011-11-12 565 out_be32(&pw->pw_regs->pwqbar, (u32) pw->port_write_msg.phys);
> 6ec4bedbf153a8 Liu Gang 2011-11-12 566
> 6ec4bedbf153a8 Liu Gang 2011-11-12 567 pr_debug("EIPWQBAR: 0x%08x IPWQBAR: 0x%08x\n",
> abc3aeae3aaa5c Liu Gang 2011-11-12 568 in_be32(&pw->pw_regs->epwqbar),
> abc3aeae3aaa5c Liu Gang 2011-11-12 569 in_be32(&pw->pw_regs->pwqbar));
> 6ec4bedbf153a8 Liu Gang 2011-11-12 570
> 6ec4bedbf153a8 Liu Gang 2011-11-12 571 /* Clear interrupt status IPWSR */
> abc3aeae3aaa5c Liu Gang 2011-11-12 572 out_be32(&pw->pw_regs->pwsr,
> 6ec4bedbf153a8 Liu Gang 2011-11-12 573 (RIO_IPWSR_TE | RIO_IPWSR_QFI | RIO_IPWSR_PWD));
> 6ec4bedbf153a8 Liu Gang 2011-11-12 574
> 446957ba512714 Adam Buchbinder 2016-02-24 575 /* Configure port write controller for snooping enable all reporting,
> 6ec4bedbf153a8 Liu Gang 2011-11-12 576 clear queue full */
> abc3aeae3aaa5c Liu Gang 2011-11-12 577 out_be32(&pw->pw_regs->pwmr,
> 6ec4bedbf153a8 Liu Gang 2011-11-12 578 RIO_IPWMR_SEN | RIO_IPWMR_QFIE | RIO_IPWMR_EIE | RIO_IPWMR_CQ);
> 6ec4bedbf153a8 Liu Gang 2011-11-12 579
> 6ec4bedbf153a8 Liu Gang 2011-11-12 580
> 6ec4bedbf153a8 Liu Gang 2011-11-12 581 /* Hook up port-write handler */
> abc3aeae3aaa5c Liu Gang 2011-11-12 582 rc = request_irq(IRQ_RIO_PW(pw), fsl_rio_port_write_handler,
> abc3aeae3aaa5c Liu Gang 2011-11-12 583 IRQF_SHARED, "port-write", (void *)pw);
> 6ec4bedbf153a8 Liu Gang 2011-11-12 584 if (rc < 0) {
> 6ec4bedbf153a8 Liu Gang 2011-11-12 585 pr_err("MPC85xx RIO: unable to request inbound doorbell irq");
> 6ec4bedbf153a8 Liu Gang 2011-11-12 586 goto err_out;
> 6ec4bedbf153a8 Liu Gang 2011-11-12 587 }
> 6ec4bedbf153a8 Liu Gang 2011-11-12 588 /* Enable Error Interrupt */
> 6ec4bedbf153a8 Liu Gang 2011-11-12 589 out_be32((u32 *)(rio_regs_win + RIO_LTLEECSR), LTLEECSR_ENABLE_ALL);
> 6ec4bedbf153a8 Liu Gang 2011-11-12 590
> abc3aeae3aaa5c Liu Gang 2011-11-12 591 INIT_WORK(&pw->pw_work, fsl_pw_dpc);
> abc3aeae3aaa5c Liu Gang 2011-11-12 592 spin_lock_init(&pw->pw_fifo_lock);
> abc3aeae3aaa5c Liu Gang 2011-11-12 593 if (kfifo_alloc(&pw->pw_fifo, RIO_PW_MSG_SIZE * 32, GFP_KERNEL)) {
> 6ec4bedbf153a8 Liu Gang 2011-11-12 594 pr_err("FIFO allocation failed\n");
> 6ec4bedbf153a8 Liu Gang 2011-11-12 595 rc = -ENOMEM;
> 6ec4bedbf153a8 Liu Gang 2011-11-12 596 goto err_out_irq;
> 6ec4bedbf153a8 Liu Gang 2011-11-12 597 }
> 6ec4bedbf153a8 Liu Gang 2011-11-12 598
> 6ec4bedbf153a8 Liu Gang 2011-11-12 599 pr_debug("IPWMR: 0x%08x IPWSR: 0x%08x\n",
> abc3aeae3aaa5c Liu Gang 2011-11-12 600 in_be32(&pw->pw_regs->pwmr),
> abc3aeae3aaa5c Liu Gang 2011-11-12 601 in_be32(&pw->pw_regs->pwsr));
> 6ec4bedbf153a8 Liu Gang 2011-11-12 602
> 6ec4bedbf153a8 Liu Gang 2011-11-12 603 return rc;
> 6ec4bedbf153a8 Liu Gang 2011-11-12 604
> 6ec4bedbf153a8 Liu Gang 2011-11-12 605 err_out_irq:
> abc3aeae3aaa5c Liu Gang 2011-11-12 606 free_irq(IRQ_RIO_PW(pw), (void *)pw);
> 6ec4bedbf153a8 Liu Gang 2011-11-12 607 err_out:
> abc3aeae3aaa5c Liu Gang 2011-11-12 608 dma_free_coherent(pw->dev, RIO_PW_MSG_SIZE,
> abc3aeae3aaa5c Liu Gang 2011-11-12 609 pw->port_write_msg.virt,
> abc3aeae3aaa5c Liu Gang 2011-11-12 610 pw->port_write_msg.phys);
> 6ec4bedbf153a8 Liu Gang 2011-11-12 611 return rc;
> 6ec4bedbf153a8 Liu Gang 2011-11-12 612 }
> 6ec4bedbf153a8 Liu Gang 2011-11-12 613
>
> :::::: The code at line 544 was first introduced by commit
> :::::: 6ec4bedbf153a8ef71aeba99a40efef556b57798 fsl-rio: Split rio driver into two parts, RapidIO endpoint and message unit
>
> :::::: TO: Liu Gang <Gang.Liu@freescale.com>
> :::::: CC: Kumar Gala <galak@kernel.crashing.org>
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-09-27 8:34 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-09-26 23:49 arch/powerpc/sysdev/fsl_rmu.c:544: warning: Function parameter or member 'pw' not described in 'fsl_rio_port_write_init' kernel test robot
2023-09-27 8:33 ` Christophe Leroy
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox