* Re: [PATCH net-next 10/13] net: airoha: Introduce PPE initialization via NPU
[not found] <20250205-airoha-en7581-flowtable-offload-v1-10-d362cfa97b01@kernel.org>
@ 2025-02-07 1:21 ` kernel test robot
0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2025-02-07 1:21 UTC (permalink / raw)
To: Lorenzo Bianconi, Andrew Lunn, David S. Miller, Eric Dumazet,
Jakub Kicinski, Paolo Abeni, Felix Fietkau, Sean Wang,
Matthias Brugger, AngeloGioacchino Del Regno, Philipp Zabel,
Rob Herring, Krzysztof Kozlowski, Conor Dooley
Cc: llvm, oe-kbuild-all, netdev, linux-arm-kernel, linux-mediatek,
devicetree, upstream
Hi Lorenzo,
kernel test robot noticed the following build warnings:
[auto build test WARNING on 135c3c86a7cef4ba3d368da15b16c275b74582d3]
url: https://github.com/intel-lab-lkp/linux/commits/Lorenzo-Bianconi/net-airoha-Move-airoha_eth-driver-in-a-dedicated-folder/20250206-022555
base: 135c3c86a7cef4ba3d368da15b16c275b74582d3
patch link: https://lore.kernel.org/r/20250205-airoha-en7581-flowtable-offload-v1-10-d362cfa97b01%40kernel.org
patch subject: [PATCH net-next 10/13] net: airoha: Introduce PPE initialization via NPU
config: hexagon-allyesconfig (https://download.01.org/0day-ci/archive/20250207/202502070935.LuHfHz3M-lkp@intel.com/config)
compiler: clang version 18.1.8 (https://github.com/llvm/llvm-project 3b5b5c1ec4a3095ab096dd780e84d7ab81f3d7ff)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250207/202502070935.LuHfHz3M-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/202502070935.LuHfHz3M-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> drivers/net/ethernet/airoha/airoha_npu.c:201:30: warning: format specifies type 'long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
200 | dev_err(dev, "%s: fw size too overlimit (%ld)\n",
| ~~~
| %zu
201 | NPU_EN7581_FIRMWARE_RV32, fw->size);
| ^~~~~~~~
include/linux/dev_printk.h:154:65: note: expanded from macro 'dev_err'
154 | dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ~~~ ^~~~~~~~~~~
include/linux/dev_printk.h:110:23: note: expanded from macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ~~~ ^~~~~~~~~~~
drivers/net/ethernet/airoha/airoha_npu.c:221:30: warning: format specifies type 'long' but the argument has type 'size_t' (aka 'unsigned int') [-Wformat]
220 | dev_err(dev, "%s: fw size too overlimit (%ld)\n",
| ~~~
| %zu
221 | NPU_EN7581_FIRMWARE_DATA, fw->size);
| ^~~~~~~~
include/linux/dev_printk.h:154:65: note: expanded from macro 'dev_err'
154 | dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ~~~ ^~~~~~~~~~~
include/linux/dev_printk.h:110:23: note: expanded from macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ~~~ ^~~~~~~~~~~
2 warnings generated.
vim +201 drivers/net/ethernet/airoha/airoha_npu.c
187
188 static int airoha_npu_run_firmware(struct airoha_npu *npu, struct reserved_mem *rmem)
189 {
190 struct device *dev = &npu->pdev->dev;
191 const struct firmware *fw;
192 void __iomem *addr;
193 int ret;
194
195 ret = request_firmware(&fw, NPU_EN7581_FIRMWARE_RV32, dev);
196 if (ret)
197 return ret;
198
199 if (fw->size > NPU_EN7581_FIRMWARE_RV32_MAX_SIZE) {
200 dev_err(dev, "%s: fw size too overlimit (%ld)\n",
> 201 NPU_EN7581_FIRMWARE_RV32, fw->size);
202 ret = -E2BIG;
203 goto out;
204 }
205
206 addr = devm_ioremap(dev, rmem->base, rmem->size);
207 if (!addr) {
208 ret = -ENOMEM;
209 goto out;
210 }
211
212 memcpy_toio(addr, fw->data, fw->size);
213 release_firmware(fw);
214
215 ret = request_firmware(&fw, NPU_EN7581_FIRMWARE_DATA, dev);
216 if (ret)
217 return ret;
218
219 if (fw->size > NPU_EN7581_FIRMWARE_DATA_MAX_SIZE) {
220 dev_err(dev, "%s: fw size too overlimit (%ld)\n",
221 NPU_EN7581_FIRMWARE_DATA, fw->size);
222 ret = -E2BIG;
223 goto out;
224 }
225
226 memcpy_toio(npu->base + REG_NPU_LOCAL_SRAM, fw->data, fw->size);
227 out:
228 release_firmware(fw);
229
230 return ret;
231 }
232
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] only message in thread