Re: [PATCH net-next v22 1/2] mailbox/pcc: support mailbox management of the shared buffer
From: Dan Carpenter <hidden>
Date: 2025-07-14 19:37:46
Also in:
lkml, oe-kbuild, oe-kbuild-all
Hi, kernel test robot noticed the following build warnings: url: https://github.com/intel-lab-lkp/linux/commits/admiyo-os-amperecomputing-com/mailbox-pcc-support-mailbox-management-of-the-shared-buffer/20250711-031525 base: net-next/main patch link: https://lore.kernel.org/r/20250710191209.737167-2-admiyo%40os.amperecomputing.com patch subject: [PATCH net-next v22 1/2] mailbox/pcc: support mailbox management of the shared buffer config: x86_64-randconfig-161-20250711 (https://download.01.org/0day-ci/archive/20250712/202507120609.Myazax08-lkp@intel.com/config) compiler: clang version 20.1.8 (https://github.com/llvm/llvm-project 87f0227cb60147a26a1eeb4fb06e3b505e9c7261) 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 [off-list ref] | Reported-by: Dan Carpenter [off-list ref] | Closes: https://lore.kernel.org/r/202507120609.Myazax08-lkp@intel.com/ (local) smatch warnings: drivers/mailbox/pcc.c:498 pcc_send_data() error: uninitialized symbol 'ret'. vim +/ret +498 drivers/mailbox/pcc.c 86c22f8c9a3b71 Ashwin Chaugule 2014-11-12 490 static int pcc_send_data(struct mbox_chan *chan, void *data) 86c22f8c9a3b71 Ashwin Chaugule 2014-11-12 491 { c45ded7e11352d Sudeep Holla 2021-09-17 492 int ret; bf18123e78f4d1 Sudeep Holla 2021-09-17 493 struct pcc_chan_info *pchan = chan->con_priv; e332edef98ddac Adam Young 2025-07-10 494 struct acpi_pcct_ext_pcc_shared_memory __iomem *pcc_hdr; e332edef98ddac Adam Young 2025-07-10 495 e332edef98ddac Adam Young 2025-07-10 496 if (pchan->chan.rx_alloc) e332edef98ddac Adam Young 2025-07-10 497 ret = pcc_write_to_buffer(chan, data); Hi Adam! :) ret is uninitialized on the else path. e332edef98ddac Adam Young 2025-07-10 @498 if (ret) e332edef98ddac Adam Young 2025-07-10 499 return ret; 8b0f57889843af Prakash, Prashanth 2016-02-17 500 c45ded7e11352d Sudeep Holla 2021-09-17 501 ret = pcc_chan_reg_read_modify_write(&pchan->cmd_update); c45ded7e11352d Sudeep Holla 2021-09-17 502 if (ret) c45ded7e11352d Sudeep Holla 2021-09-17 503 return ret; c45ded7e11352d Sudeep Holla 2021-09-17 504 e332edef98ddac Adam Young 2025-07-10 505 pcc_hdr = pchan->chan.shmem; e332edef98ddac Adam Young 2025-07-10 506 if (ioread32(&pcc_hdr->flags) & PCC_CMD_COMPLETION_NOTIFY) e332edef98ddac Adam Young 2025-07-10 507 pchan->chan.irq_ack = true; e332edef98ddac Adam Young 2025-07-10 508 3db174e478cb0b Huisong Li 2023-08-01 509 ret = pcc_chan_reg_read_modify_write(&pchan->db); e332edef98ddac Adam Young 2025-07-10 510 3db174e478cb0b Huisong Li 2023-08-01 511 if (!ret && pchan->plat_irq > 0) 3db174e478cb0b Huisong Li 2023-08-01 512 pchan->chan_in_use = true; 3db174e478cb0b Huisong Li 2023-08-01 513 3db174e478cb0b Huisong Li 2023-08-01 514 return ret; 86c22f8c9a3b71 Ashwin Chaugule 2014-11-12 515 } -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki