On Wed, Sep 20, 2017 at 5:18 PM, kbuild test robot <[email protected]> wrote:
> Hi Viswas,
>
> [auto build test WARNING on mkp-scsi/for-next]
> [also build test WARNING on v4.14-rc1 next-20170920]
> [if your patch is applied to the wrong git tree, please drop us a note to
> help improve the system]
>
> url:
> https://github.com/0day-ci/linux/commits/Viswas-G/pm80xx-updates/20170920-202506
> base: https://git.kernel.org/pub/scm/linux/kernel/git/mkp/scsi.git for-next
> config: x86_64-randconfig-a0-09202136 (attached as .config)
> compiler: gcc-4.4 (Debian 4.4.7-8) 4.4.7
> reproduce:
> # save the attached .config to linux build tree
> make ARCH=x86_64
>
> All warnings (new ones prefixed by >>):
>
> drivers/scsi/pm8001/pm80xx_hwi.c: In function 'pm80xx_chip_phy_ctl_req':
>>> drivers/scsi/pm8001/pm80xx_hwi.c:4507: warning: unused variable 'ret'
>
> vim +/ret +4507 drivers/scsi/pm8001/pm80xx_hwi.c
Indeed, ret is unused, Viswas, could you send a update patch to fix
the warning in this patch and patch5.
Thanks
>
> f5860992d Sakthivel K 2013-04-17 4492
> f5860992d Sakthivel K 2013-04-17 4493 /**
> f5860992d Sakthivel K 2013-04-17 4494 * pm80xx_chip_phy_ctl_req - support
> the local phy operation
> f5860992d Sakthivel K 2013-04-17 4495 * @pm8001_ha: our hba card
> information.
> f5860992d Sakthivel K 2013-04-17 4496 * @num: the inbound queue number
> f5860992d Sakthivel K 2013-04-17 4497 * @phy_id: the phy id which we
> wanted to operate
> f5860992d Sakthivel K 2013-04-17 4498 * @phy_op:
> f5860992d Sakthivel K 2013-04-17 4499 */
> f5860992d Sakthivel K 2013-04-17 4500 static int
> pm80xx_chip_phy_ctl_req(struct pm8001_hba_info *pm8001_ha,
> f5860992d Sakthivel K 2013-04-17 4501 u32 phyId, u32 phy_op)
> f5860992d Sakthivel K 2013-04-17 4502 {
> 18e9bdf60 Viswas G 2017-09-19 4503 u32 tag;
> 18e9bdf60 Viswas G 2017-09-19 4504 int rc;
> f5860992d Sakthivel K 2013-04-17 4505 struct local_phy_ctl_req
> payload;
> f5860992d Sakthivel K 2013-04-17 4506 struct inbound_queue_table
> *circularQ;
> f5860992d Sakthivel K 2013-04-17 @4507 int ret;
> f5860992d Sakthivel K 2013-04-17 4508 u32 opc =
> OPC_INB_LOCAL_PHY_CONTROL;
> f5860992d Sakthivel K 2013-04-17 4509 memset(&payload, 0,
> sizeof(payload));
> 18e9bdf60 Viswas G 2017-09-19 4510 rc =
> pm8001_tag_alloc(pm8001_ha, &tag);
> 18e9bdf60 Viswas G 2017-09-19 4511 if (rc)
> 18e9bdf60 Viswas G 2017-09-19 4512 return rc;
> f5860992d Sakthivel K 2013-04-17 4513 circularQ =
> &pm8001_ha->inbnd_q_tbl[0];
> 18e9bdf60 Viswas G 2017-09-19 4514 payload.tag =
> cpu_to_le32(tag);
> f5860992d Sakthivel K 2013-04-17 4515 payload.phyop_phyid =
> f5860992d Sakthivel K 2013-04-17 4516 cpu_to_le32(((phy_op
> & 0xFF) << 8) | (phyId & 0xFF));
> 18e9bdf60 Viswas G 2017-09-19 4517 return
> pm8001_mpi_build_cmd(pm8001_ha, circularQ, opc, &payload, 0);
> f5860992d Sakthivel K 2013-04-17 4518 }
> f5860992d Sakthivel K 2013-04-17 4519
>
> :::::: The code at line 4507 was first introduced by commit
> :::::: f5860992db55c9e36b0f120dff73f0c34abe510d [SCSI] pm80xx: Added SPCv/ve
> specific hardware functionalities and relevant changes in common files
>
> :::::: TO: Sakthivel K <[email protected]>
> :::::: CC: James Bottomley <[email protected]>
>
> ---
> 0-DAY kernel test infrastructure Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all Intel Corporation
--
Jack Wang
Linux Kernel Developer
ProfitBricks GmbH
Greifswalder Str. 207
D - 10405 Berlin
Tel: +49 30 577 008 042
Fax: +49 30 577 008 299
Email: [email protected]
URL: https://www.profitbricks.de
Sitz der Gesellschaft: Berlin
Registergericht: Amtsgericht Charlottenburg, HRB 125506 B
Geschäftsführer: Achim Weiss
