drivers/net/wireless/ath/ath12k/wifi7/dp_rx.c:1712:3-6: ERROR: invalid reference to the index variable of the iterator on line 1702
kernel test robot <[email protected]>
| Newsgroups | dev.linux.lists.oe-kbuild |
|---|---|
| Message-ID | <[email protected]> |
BCC: [email protected] CC: [email protected] CC: [email protected] TO: Pavankumar Nandeshwar <[email protected]> CC: Jeff Johnson <[email protected]> CC: Ripan Deuri <[email protected]> CC: Vasanthakumar Thiagarajan <[email protected]> CC: Baochen Qiang <[email protected]> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 06cf61899d6498b33e4b7c87d99d5bd471ccc375 commit: 5d2df2aa84303a187b82ca048ae28709bef85f77 wifi: ath12k: Move srng processing to wifi7 directory date: 11 months ago :::::: branch date: 2 days ago :::::: commit date: 11 months ago config: i386-randconfig-052-20260810 (https://download.01.org/0day-ci/archive/20260810/[email protected]/config) compiler: clang version 22.1.3 (https://github.com/llvm/llvm-project e9846648fd6183ee6d8cbdb4502213fcf902a211) 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 | Fixes: 5d2df2aa8430 ("wifi: ath12k: Move srng processing to wifi7 directory") | Reported-by: kernel test robot <[email protected]> | Reported-by: Julia Lawall <[email protected]> | Closes: https://lore.kernel.org/r/[email protected]/ cocci warnings: (new ones prefixed by >>) >> drivers/net/wireless/ath/ath12k/wifi7/dp_rx.c:1712:3-6: ERROR: invalid reference to the index variable of the iterator on line 1702 drivers/net/wireless/ath/ath12k/wifi7/dp_rx.c:1712:29-32: ERROR: invalid reference to the index variable of the iterator on line 1702 vim +1712 drivers/net/wireless/ath/ath12k/wifi7/dp_rx.c 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1645 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1646 void ath12k_dp_rx_process_reo_status(struct ath12k_base *ab) 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1647 { 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1648 struct ath12k_dp *dp = &ab->dp; 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1649 struct hal_tlv_64_hdr *hdr; 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1650 struct hal_srng *srng; 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1651 struct ath12k_dp_rx_reo_cmd *cmd, *tmp; 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1652 bool found = false; 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1653 u16 tag; 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1654 struct hal_reo_status reo_status; 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1655 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1656 srng = &ab->hal.srng_list[dp->reo_status_ring.ring_id]; 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1657 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1658 memset(&reo_status, 0, sizeof(reo_status)); 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1659 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1660 spin_lock_bh(&srng->lock); 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1661 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1662 ath12k_hal_srng_access_begin(ab, srng); 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1663 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1664 while ((hdr = ath12k_hal_srng_dst_get_next_entry(ab, srng))) { 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1665 tag = le64_get_bits(hdr->tl, HAL_SRNG_TLV_HDR_TAG); 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1666 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1667 switch (tag) { 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1668 case HAL_REO_GET_QUEUE_STATS_STATUS: 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1669 ath12k_hal_reo_status_queue_stats(ab, hdr, 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1670 &reo_status); 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1671 break; 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1672 case HAL_REO_FLUSH_QUEUE_STATUS: 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1673 ath12k_hal_reo_flush_queue_status(ab, hdr, 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1674 &reo_status); 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1675 break; 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1676 case HAL_REO_FLUSH_CACHE_STATUS: 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1677 ath12k_hal_reo_flush_cache_status(ab, hdr, 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1678 &reo_status); 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1679 break; 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1680 case HAL_REO_UNBLOCK_CACHE_STATUS: 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1681 ath12k_hal_reo_unblk_cache_status(ab, hdr, 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1682 &reo_status); 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1683 break; 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1684 case HAL_REO_FLUSH_TIMEOUT_LIST_STATUS: 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1685 ath12k_hal_reo_flush_timeout_list_status(ab, hdr, 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1686 &reo_status); 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1687 break; 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1688 case HAL_REO_DESCRIPTOR_THRESHOLD_REACHED_STATUS: 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1689 ath12k_hal_reo_desc_thresh_reached_status(ab, hdr, 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1690 &reo_status); 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1691 break; 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1692 case HAL_REO_UPDATE_RX_REO_QUEUE_STATUS: 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1693 ath12k_hal_reo_update_rx_reo_queue_status(ab, hdr, 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1694 &reo_status); 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1695 break; 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1696 default: 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1697 ath12k_warn(ab, "Unknown reo status type %d\n", tag); 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1698 continue; 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1699 } 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1700 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1701 spin_lock_bh(&dp->reo_cmd_lock); 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 @1702 list_for_each_entry_safe(cmd, tmp, &dp->reo_cmd_list, list) { 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1703 if (reo_status.uniform_hdr.cmd_num == cmd->cmd_num) { 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1704 found = true; 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1705 list_del(&cmd->list); 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1706 break; 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1707 } 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1708 } 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1709 spin_unlock_bh(&dp->reo_cmd_lock); 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1710 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 1711 if (found) { 5d2df2aa84303a Pavankumar Nandeshwar 2025-08-28 @1712 cmd->handler(dp, (void *)&cmd->data, -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki