From: kernel test robot <lkp@intel.com>
To: Raj Kumar Bhagat <quic_rajkbhag@quicinc.com>, ath12k@lists.infradead.org
Cc: oe-kbuild-all@lists.linux.dev,
Johannes Berg <johannes@sipsolutions.net>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Jeff Johnson <jjohnson@kernel.org>,
linux-wireless@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org,
Raj Kumar Bhagat <quic_rajkbhag@quicinc.com>
Subject: Re: [PATCH v6 07/13] wifi: ath12k: add support for fixed QMI firmware memory
Date: Fri, 28 Feb 2025 03:19:00 +0800 [thread overview]
Message-ID: <202502280215.J3T7EWeY-lkp@intel.com> (raw)
In-Reply-To: <20250225064834.2002499-8-quic_rajkbhag@quicinc.com>
Hi Raj,
kernel test robot noticed the following build warnings:
[auto build test WARNING on 13a84b4e7dd5d843dfaa54283adb4bbbe0adee15]
url: https://github.com/intel-lab-lkp/linux/commits/Raj-Kumar-Bhagat/dt-bindings-net-wireless-describe-the-ath12k-AHB-module-for-IPQ5332/20250225-145536
base: 13a84b4e7dd5d843dfaa54283adb4bbbe0adee15
patch link: https://lore.kernel.org/r/20250225064834.2002499-8-quic_rajkbhag%40quicinc.com
patch subject: [PATCH v6 07/13] wifi: ath12k: add support for fixed QMI firmware memory
config: mips-allyesconfig (https://download.01.org/0day-ci/archive/20250228/202502280215.J3T7EWeY-lkp@intel.com/config)
compiler: mips-linux-gcc (GCC) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250228/202502280215.J3T7EWeY-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/202502280215.J3T7EWeY-lkp@intel.com/
All warnings (new ones prefixed by >>):
In file included from drivers/net/wireless/ath/ath12k/qmi.c:11:
drivers/net/wireless/ath/ath12k/qmi.c: In function 'ath12k_qmi_assign_target_mem_chunk':
>> drivers/net/wireless/ath/ath12k/qmi.c:2677:44: warning: format '%lld' expects argument of type 'long long int', but argument 6 has type 'phys_addr_t' {aka 'unsigned int'} [-Wformat=]
2677 | "failed to assign mem type %d req size %d avail size %lld\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
......
2680 | rmem->size);
| ~~~~~~~~~~
| |
| phys_addr_t {aka unsigned int}
drivers/net/wireless/ath/ath12k/debug.h:69:40: note: in definition of macro 'ath12k_dbg'
69 | __ath12k_dbg(ab, mask, fmt, ##__VA_ARGS__); \
| ^~~
drivers/net/wireless/ath/ath12k/qmi.c:2677:100: note: format string is defined here
2677 | "failed to assign mem type %d req size %d avail size %lld\n",
| ~~~^
| |
| long long int
| %d
drivers/net/wireless/ath/ath12k/qmi.c:2707:44: warning: format '%lld' expects argument of type 'long long int', but argument 6 has type 'phys_addr_t' {aka 'unsigned int'} [-Wformat=]
2707 | "failed to assign mem type %d req size %d avail size %lld\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
......
2710 | bdf_size);
| ~~~~~~~~
| |
| phys_addr_t {aka unsigned int}
drivers/net/wireless/ath/ath12k/debug.h:69:40: note: in definition of macro 'ath12k_dbg'
69 | __ath12k_dbg(ab, mask, fmt, ##__VA_ARGS__); \
| ^~~
drivers/net/wireless/ath/ath12k/qmi.c:2707:100: note: format string is defined here
2707 | "failed to assign mem type %d req size %d avail size %lld\n",
| ~~~^
| |
| long long int
| %d
drivers/net/wireless/ath/ath12k/qmi.c:2748:44: warning: format '%lld' expects argument of type 'long long int', but argument 6 has type 'phys_addr_t' {aka 'unsigned int'} [-Wformat=]
2748 | "failed to assign mem type %d req size %d avail size %lld\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
......
2751 | rmem->size);
| ~~~~~~~~~~
| |
| phys_addr_t {aka unsigned int}
drivers/net/wireless/ath/ath12k/debug.h:69:40: note: in definition of macro 'ath12k_dbg'
69 | __ath12k_dbg(ab, mask, fmt, ##__VA_ARGS__); \
| ^~~
drivers/net/wireless/ath/ath12k/qmi.c:2748:100: note: format string is defined here
2748 | "failed to assign mem type %d req size %d avail size %lld\n",
| ~~~^
| |
| long long int
| %d
vim +2677 drivers/net/wireless/ath/ath12k/qmi.c
2659
2660 static int ath12k_qmi_assign_target_mem_chunk(struct ath12k_base *ab)
2661 {
2662 struct reserved_mem *rmem;
2663 phys_addr_t bdf_size;
2664 int i, idx, ret;
2665
2666 for (i = 0, idx = 0; i < ab->qmi.mem_seg_count; i++) {
2667 switch (ab->qmi.target_mem[i].type) {
2668 case HOST_DDR_REGION_TYPE:
2669 rmem = ath12k_core_get_reserved_mem(ab, 0);
2670 if (!rmem) {
2671 ret = -ENODEV;
2672 goto out;
2673 }
2674
2675 if (rmem->size < ab->qmi.target_mem[i].size) {
2676 ath12k_dbg(ab, ATH12K_DBG_QMI,
> 2677 "failed to assign mem type %d req size %d avail size %lld\n",
2678 ab->qmi.target_mem[i].type,
2679 ab->qmi.target_mem[i].size,
2680 rmem->size);
2681 ret = -EINVAL;
2682 goto out;
2683 }
2684
2685 ab->qmi.target_mem[idx].paddr = rmem->base;
2686 ab->qmi.target_mem[idx].v.ioaddr =
2687 ioremap(ab->qmi.target_mem[idx].paddr,
2688 ab->qmi.target_mem[i].size);
2689 if (!ab->qmi.target_mem[idx].v.ioaddr) {
2690 ret = -EIO;
2691 goto out;
2692 }
2693 ab->qmi.target_mem[idx].size = ab->qmi.target_mem[i].size;
2694 ab->qmi.target_mem[idx].type = ab->qmi.target_mem[i].type;
2695 idx++;
2696 break;
2697 case BDF_MEM_REGION_TYPE:
2698 rmem = ath12k_core_get_reserved_mem(ab, 0);
2699 if (!rmem) {
2700 ret = -ENODEV;
2701 goto out;
2702 }
2703
2704 bdf_size = rmem->size - ab->hw_params->bdf_addr_offset;
2705 if (bdf_size < ab->qmi.target_mem[i].size) {
2706 ath12k_dbg(ab, ATH12K_DBG_QMI,
2707 "failed to assign mem type %d req size %d avail size %lld\n",
2708 ab->qmi.target_mem[i].type,
2709 ab->qmi.target_mem[i].size,
2710 bdf_size);
2711 ret = -EINVAL;
2712 goto out;
2713 }
2714 ab->qmi.target_mem[idx].paddr =
2715 rmem->base + ab->hw_params->bdf_addr_offset;
2716 ab->qmi.target_mem[idx].v.ioaddr =
2717 ioremap(ab->qmi.target_mem[idx].paddr,
2718 ab->qmi.target_mem[i].size);
2719 if (!ab->qmi.target_mem[idx].v.ioaddr) {
2720 ret = -EIO;
2721 goto out;
2722 }
2723 ab->qmi.target_mem[idx].size = ab->qmi.target_mem[i].size;
2724 ab->qmi.target_mem[idx].type = ab->qmi.target_mem[i].type;
2725 idx++;
2726 break;
2727 case CALDB_MEM_REGION_TYPE:
2728 /* Cold boot calibration is not enabled in Ath12k. Hence,
2729 * assign paddr = 0.
2730 * Once cold boot calibration is enabled add support to
2731 * assign reserved memory from DT.
2732 */
2733 ab->qmi.target_mem[idx].paddr = 0;
2734 ab->qmi.target_mem[idx].v.ioaddr = NULL;
2735 ab->qmi.target_mem[idx].size = ab->qmi.target_mem[i].size;
2736 ab->qmi.target_mem[idx].type = ab->qmi.target_mem[i].type;
2737 idx++;
2738 break;
2739 case M3_DUMP_REGION_TYPE:
2740 rmem = ath12k_core_get_reserved_mem(ab, 1);
2741 if (!rmem) {
2742 ret = -EINVAL;
2743 goto out;
2744 }
2745
2746 if (rmem->size < ab->qmi.target_mem[i].size) {
2747 ath12k_dbg(ab, ATH12K_DBG_QMI,
2748 "failed to assign mem type %d req size %d avail size %lld\n",
2749 ab->qmi.target_mem[i].type,
2750 ab->qmi.target_mem[i].size,
2751 rmem->size);
2752 ret = -EINVAL;
2753 goto out;
2754 }
2755
2756 ab->qmi.target_mem[idx].paddr = rmem->base;
2757 ab->qmi.target_mem[idx].v.ioaddr =
2758 ioremap(ab->qmi.target_mem[idx].paddr,
2759 ab->qmi.target_mem[i].size);
2760 if (!ab->qmi.target_mem[idx].v.ioaddr) {
2761 ret = -EIO;
2762 goto out;
2763 }
2764 ab->qmi.target_mem[idx].size = ab->qmi.target_mem[i].size;
2765 ab->qmi.target_mem[idx].type = ab->qmi.target_mem[i].type;
2766 idx++;
2767 break;
2768 default:
2769 ath12k_warn(ab, "qmi ignore invalid mem req type %d\n",
2770 ab->qmi.target_mem[i].type);
2771 break;
2772 }
2773 }
2774 ab->qmi.mem_seg_count = idx;
2775
2776 return 0;
2777 out:
2778 ath12k_qmi_free_target_mem_chunk(ab);
2779 return ret;
2780 }
2781
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2025-02-27 19:20 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-02-25 6:48 [PATCH v6 00/13] wifi: ath12k: add Ath12k AHB driver support for IPQ5332 Raj Kumar Bhagat
2025-02-25 6:48 ` [PATCH v6 01/13] dt-bindings: net: wireless: describe the ath12k AHB module " Raj Kumar Bhagat
2025-02-25 8:47 ` Krzysztof Kozlowski
2025-02-25 10:34 ` Krzysztof Kozlowski
2025-02-25 10:42 ` Raj Kumar Bhagat
2025-02-25 6:48 ` [PATCH v6 02/13] wifi: ath12k: fix incorrect CE addresses Raj Kumar Bhagat
2025-02-25 6:48 ` [PATCH v6 03/13] wifi: ath12k: refactor ath12k_hw_regs structure Raj Kumar Bhagat
2025-02-25 6:48 ` [PATCH v6 04/13] wifi: ath12k: add ath12k_hw_params for IPQ5332 Raj Kumar Bhagat
2025-02-25 6:48 ` [PATCH v6 05/13] wifi: ath12k: avoid m3 firmware download in AHB device IPQ5332 Raj Kumar Bhagat
2025-02-25 6:48 ` [PATCH v6 06/13] wifi: ath12k: Add hw_params to remap CE register space for IPQ5332 Raj Kumar Bhagat
2025-02-25 6:48 ` [PATCH v6 07/13] wifi: ath12k: add support for fixed QMI firmware memory Raj Kumar Bhagat
2025-02-27 19:19 ` kernel test robot [this message]
2025-02-25 6:48 ` [PATCH v6 08/13] wifi: ath12k: add AHB driver support for IPQ5332 Raj Kumar Bhagat
2025-02-25 6:48 ` [PATCH v6 09/13] wifi: ath12k: Power up root PD Raj Kumar Bhagat
2025-02-25 6:48 ` [PATCH v6 10/13] wifi: ath12k: Register various userPD interrupts and save SMEM entries Raj Kumar Bhagat
2025-02-25 6:48 ` [PATCH v6 11/13] wifi: ath12k: Power up userPD Raj Kumar Bhagat
2025-02-25 6:48 ` [PATCH v6 12/13] wifi: ath12k: Power down userPD Raj Kumar Bhagat
2025-02-25 6:48 ` [PATCH v6 13/13] wifi: ath12k: enable ath12k AHB support Raj Kumar Bhagat
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202502280215.J3T7EWeY-lkp@intel.com \
--to=lkp@intel.com \
--cc=ath12k@lists.infradead.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=jjohnson@kernel.org \
--cc=johannes@sipsolutions.net \
--cc=krzk@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=quic_rajkbhag@quicinc.com \
--cc=robh@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox