Re: [PATCH net-next 4/8] net: dsa: microchip: Add support for KSZ8463's PTP interrupts

[kernel test robot <lkp@intel.com>]

Hi Bastien,

kernel test robot noticed the following build warnings:

[auto build test WARNING on 3adff276e751051e77be4df8d29eab1cf0856fbf]

url:    https://github.com/intel-lab-lkp/linux/commits/Bastien-Curutchet-Schneider-Electric/net-dsa-microchip-Add-support-for-KSZ8463-global-irq/20260116-000545
base:   3adff276e751051e77be4df8d29eab1cf0856fbf
patch link:    https://lore.kernel.org/r/20260115-ksz8463-ptp-v1-4-bcfe2830cf50%40bootlin.com
patch subject: [PATCH net-next 4/8] net: dsa: microchip: Add support for KSZ8463's PTP interrupts
config: loongarch-allmodconfig (https://download.01.org/0day-ci/archive/20260116/202601160729.AzeG6b64-lkp@intel.com/config)
compiler: clang version 19.1.7 (https://github.com/llvm/llvm-project cd708029e0b2869e80abe31ddb175f7c35361f90)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260116/202601160729.AzeG6b64-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/202601160729.AzeG6b64-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> drivers/net/dsa/microchip/ksz_ptp.c:1187:40: warning: variable 'p' is uninitialized when used here [-Wuninitialized]
    1187 |         ptpirq->reg_mask = ops->get_port_addr(p, KSZ8463_PTP_TS_IER);
         |                                               ^
   drivers/net/dsa/microchip/ksz_ptp.c:1175:7: note: initialize the variable 'p' to silence this warning
    1175 |         int p;
         |              ^
         |               = 0
   1 warning generated.


vim +/p +1187 drivers/net/dsa/microchip/ksz_ptp.c

  1167	
  1168	int ksz8463_ptp_irq_setup(struct dsa_switch *ds)
  1169	{
  1170		struct ksz_device *dev = ds->priv;
  1171		const struct ksz_dev_ops *ops = dev->dev_ops;
  1172		struct ksz_port *port1, *port2;
  1173		struct ksz_irq *ptpirq;
  1174		int ret;
  1175		int p;
  1176	
  1177		port1 = &dev->ports[0];
  1178		port2 = &dev->ports[1];
  1179		ptpirq = &port1->ptpirq;
  1180	
  1181		ptpirq->irq_num = irq_find_mapping(dev->girq.domain, KSZ8463_SRC_PTP_INT);
  1182		if (!ptpirq->irq_num)
  1183			return -EINVAL;
  1184	
  1185		ptpirq->dev = dev;
  1186		ptpirq->nirqs = 4;
> 1187		ptpirq->reg_mask = ops->get_port_addr(p, KSZ8463_PTP_TS_IER);
  1188		ptpirq->reg_status = ops->get_port_addr(p, KSZ8463_PTP_TS_ISR);
  1189		ptpirq->irq0_offset = KSZ8463_PTP_INT_START;
  1190		snprintf(ptpirq->name, sizeof(ptpirq->name), "ptp-irq-%d", p);
  1191	
  1192		ptpirq->domain = irq_domain_create_linear(dev_fwnode(dev->dev), ptpirq->nirqs,
  1193							  &ksz_ptp_irq_domain_ops, ptpirq);
  1194		if (!ptpirq->domain)
  1195			return -ENOMEM;
  1196	
  1197		ret = request_threaded_irq(ptpirq->irq_num, NULL, ksz_ptp_irq_thread_fn,
  1198					   IRQF_ONESHOT, ptpirq->name, ptpirq);
  1199		if (ret)
  1200			goto release_domain;
  1201	
  1202		ret = ksz8463_ptp_port_irq_setup(ptpirq, port1,
  1203						 KSZ8463_PTP_PORT1_INT_START - KSZ8463_PTP_INT_START);
  1204		if (ret)
  1205			goto release_irq;
  1206	
  1207		ret = ksz8463_ptp_port_irq_setup(ptpirq, port2,
  1208						 KSZ8463_PTP_PORT2_INT_START - KSZ8463_PTP_INT_START);
  1209		if (ret)
  1210			goto free_port1;
  1211	
  1212		return 0;
  1213	
  1214	free_port1:
  1215		ksz8463_ptp_port_irq_teardown(port1);
  1216	release_irq:
  1217		free_irq(ptpirq->irq_num, ptpirq);
  1218	release_domain:
  1219		irq_domain_remove(ptpirq->domain);
  1220	
  1221		return ret;
  1222	}
  1223	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki