Re: [PATCH v5 03/15] misc: bcm-vk: add autoload support

From: kernel test robot
Date: Thu Oct 01 2020 - 00:07:39 EST

Next message: Kees Cook: "Re: [PATCH V3 1/8] sysfs: Add sysfs_emit and sysfs_emit_at to format sysfs output"
Previous message: Dmitry Osipenko: "Re: [PATCH v3 2/3] iommu/tegra-smmu: Rework .probe_device and .attach_dev"
Next in thread: Scott Branden: "Re: [PATCH v5 03/15] misc: bcm-vk: add autoload support"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Scott,

I love your patch! Yet something to improve:

[auto build test ERROR on char-misc/char-misc-testing]
[also build test ERROR on soc/for-next kees/for-next/pstore linus/master v5.9-rc7 next-20200930]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url: https://github.com/0day-ci/linux/commits/Scott-Branden/Add-Broadcom-VK-driver/20201001-093119
base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git c471bf4b11c7df0f0f9f42b5aeec424dc62d0c7e
config: powerpc-allyesconfig (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/f04a1673933a1af94e44bc650533bbb271e3dcc1
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Scott-Branden/Add-Broadcom-VK-driver/20201001-093119
git checkout f04a1673933a1af94e44bc650533bbb271e3dcc1
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=powerpc

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@xxxxxxxxx>

All errors (new ones prefixed by >>):

drivers/misc/bcm-vk/bcm_vk_dev.c: In function 'bcm_vk_load_image_by_type':
>> drivers/misc/bcm-vk/bcm_vk_dev.c:284:8: error: implicit declaration of function 'request_partial_firmware_into_buf'; did you mean 'request_firmware_into_buf'? [-Werror=implicit-function-declaration]
284 | ret = request_partial_firmware_into_buf(&fw, filename, dev,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| request_firmware_into_buf
cc1: some warnings being treated as errors

vim +284 drivers/misc/bcm-vk/bcm_vk_dev.c

206
207 static int bcm_vk_load_image_by_type(struct bcm_vk *vk, u32 load_type,
208 const char *filename)
209 {
210 struct device *dev = &vk->pdev->dev;
211 const struct firmware *fw = NULL;
212 void *bufp = NULL;
213 size_t max_buf, offset;
214 int ret;
215 u64 offset_codepush;
216 u32 codepush;
217 u32 value;
218 dma_addr_t boot_dma_addr;
219 bool is_stdalone;
220
221 if (load_type == VK_IMAGE_TYPE_BOOT1) {
222 /*
223 * After POR, enable VK soft BOOTSRC so bootrom do not clear
224 * the pushed image (the TCM memories).
225 */
226 value = vkread32(vk, BAR_0, BAR_BOOTSRC_SELECT);
227 value |= BOOTSRC_SOFT_ENABLE;
228 vkwrite32(vk, value, BAR_0, BAR_BOOTSRC_SELECT);
229
230 codepush = CODEPUSH_BOOTSTART + CODEPUSH_BOOT1_ENTRY;
231 offset_codepush = BAR_CODEPUSH_SBL;
232
233 /* Write a 1 to request SRAM open bit */
234 vkwrite32(vk, CODEPUSH_BOOTSTART, BAR_0, offset_codepush);
235
236 /* Wait for VK to respond */
237 ret = bcm_vk_wait(vk, BAR_0, BAR_BOOT_STATUS, SRAM_OPEN,
238 SRAM_OPEN, LOAD_IMAGE_TIMEOUT_MS);
239 if (ret < 0) {
240 dev_err(dev, "boot1 wait SRAM err - ret(%d)\n", ret);
241 goto err_buf_out;
242 }
243
244 max_buf = SZ_256K;
245 bufp = dma_alloc_coherent(dev,
246 max_buf,
247 &boot_dma_addr, GFP_KERNEL);
248 if (!bufp) {
249 dev_err(dev, "Error allocating 0x%zx\n", max_buf);
250 ret = -ENOMEM;
251 goto err_buf_out;
252 }
253 } else if (load_type == VK_IMAGE_TYPE_BOOT2) {
254 codepush = CODEPUSH_BOOT2_ENTRY;
255 offset_codepush = BAR_CODEPUSH_SBI;
256
257 /* Wait for VK to respond */
258 ret = bcm_vk_wait(vk, BAR_0, BAR_BOOT_STATUS, DDR_OPEN,
259 DDR_OPEN, LOAD_IMAGE_TIMEOUT_MS);
260 if (ret < 0) {
261 dev_err(dev, "boot2 wait DDR open error - ret(%d)\n",
262 ret);
263 goto err_buf_out;
264 }
265
266 max_buf = SZ_4M;
267 bufp = dma_alloc_coherent(dev,
268 max_buf,
269 &boot_dma_addr, GFP_KERNEL);
270 if (!bufp) {
271 dev_err(dev, "Error allocating 0x%zx\n", max_buf);
272 ret = -ENOMEM;
273 goto err_buf_out;
274 }
275
276 bcm_vk_buf_notify(vk, bufp, boot_dma_addr, max_buf);
277 } else {
278 dev_err(dev, "Error invalid image type 0x%x\n", load_type);
279 ret = -EINVAL;
280 goto err_buf_out;
281 }
282
283 offset = 0;
> 284 ret = request_partial_firmware_into_buf(&fw, filename, dev,
285 bufp, max_buf, offset);
286 if (ret) {
287 dev_err(dev, "Error %d requesting firmware file: %s\n",
288 ret, filename);
289 goto err_firmware_out;
290 }
291 dev_dbg(dev, "size=0x%zx\n", fw->size);
292 if (load_type == VK_IMAGE_TYPE_BOOT1)
293 memcpy_toio(vk->bar[BAR_1] + BAR1_CODEPUSH_BASE_BOOT1,
294 bufp,
295 fw->size);
296
297 dev_dbg(dev, "Signaling 0x%x to 0x%llx\n", codepush, offset_codepush);
298 vkwrite32(vk, codepush, BAR_0, offset_codepush);
299
300 if (load_type == VK_IMAGE_TYPE_BOOT1) {
301 u32 boot_status;
302
303 /* wait until done */
304 ret = bcm_vk_wait(vk, BAR_0, BAR_BOOT_STATUS,
305 BOOT1_RUNNING,
306 BOOT1_RUNNING,
307 BOOT1_STARTUP_TIMEOUT_MS);
308
309 boot_status = vkread32(vk, BAR_0, BAR_BOOT_STATUS);
310 is_stdalone = !BCM_VK_INTF_IS_DOWN(boot_status) &&
311 (boot_status & BOOT_STDALONE_RUNNING);
312 if (ret && !is_stdalone) {
313 dev_err(dev,
314 "Timeout %ld ms waiting for boot1 to come up - ret(%d)\n",
315 BOOT1_STARTUP_TIMEOUT_MS, ret);
316 goto err_firmware_out;
317 } else if (is_stdalone) {
318 u32 reg;
319
320 reg = vkread32(vk, BAR_0, BAR_BOOT1_STDALONE_PROGRESS);
321 if ((reg & BOOT1_STDALONE_PROGRESS_MASK) ==
322 BOOT1_STDALONE_SUCCESS) {
323 dev_info(dev, "Boot1 standalone success\n");
324 ret = 0;
325 } else {
326 dev_err(dev, "Timeout %ld ms - Boot1 standalone failure\n",
327 BOOT1_STARTUP_TIMEOUT_MS);
328 ret = -EINVAL;
329 goto err_firmware_out;
330 }
331 }
332 } else if (load_type == VK_IMAGE_TYPE_BOOT2) {
333 unsigned long timeout;
334
335 timeout = jiffies + msecs_to_jiffies(LOAD_IMAGE_TIMEOUT_MS);
336
337 /* To send more data to VK than max_buf allowed at a time */
338 do {
339 /*
340 * Check for ack from card. when Ack is received,
341 * it means all the data is received by card.
342 * Exit the loop after ack is received.
343 */
344 ret = bcm_vk_wait(vk, BAR_0, BAR_BOOT_STATUS,
345 FW_LOADER_ACK_RCVD_ALL_DATA,
346 FW_LOADER_ACK_RCVD_ALL_DATA,
347 TXFR_COMPLETE_TIMEOUT_MS);
348 if (ret == 0) {
349 dev_dbg(dev, "Exit boot2 download\n");
350 break;
351 } else if (ret == -EFAULT) {
352 dev_err(dev, "Error detected during ACK waiting");
353 goto err_firmware_out;
354 }
355
356 /* exit the loop, if there is no response from card */
357 if (time_after(jiffies, timeout)) {
358 dev_err(dev, "Error. No reply from card\n");
359 ret = -ETIMEDOUT;
360 goto err_firmware_out;
361 }
362
363 /* Wait for VK to open BAR space to copy new data */
364 ret = bcm_vk_wait(vk, BAR_0, offset_codepush,
365 codepush, 0,
366 TXFR_COMPLETE_TIMEOUT_MS);
367 if (ret == 0) {
368 offset += max_buf;
369 ret = request_partial_firmware_into_buf
370 (&fw,
371 filename,
372 dev, bufp,
373 max_buf,
374 offset);
375 if (ret) {
376 dev_err(dev,
377 "Error %d requesting firmware file: %s offset: 0x%zx\n",
378 ret, filename, offset);
379 goto err_firmware_out;
380 }
381 dev_dbg(dev, "size=0x%zx\n", fw->size);
382 dev_dbg(dev, "Signaling 0x%x to 0x%llx\n",
383 codepush, offset_codepush);
384 vkwrite32(vk, codepush, BAR_0, offset_codepush);
385 /* reload timeout after every codepush */
386 timeout = jiffies +
387 msecs_to_jiffies(LOAD_IMAGE_TIMEOUT_MS);
388 } else if (ret == -EFAULT) {
389 dev_err(dev, "Error detected waiting for transfer\n");
390 goto err_firmware_out;
391 }
392 } while (1);
393
394 /* wait for fw status bits to indicate app ready */
395 ret = bcm_vk_wait(vk, BAR_0, VK_BAR_FWSTS,
396 VK_FWSTS_READY,
397 VK_FWSTS_READY,
398 BOOT2_STARTUP_TIMEOUT_MS);
399 if (ret < 0) {
400 dev_err(dev, "Boot2 not ready - ret(%d)\n", ret);
401 goto err_firmware_out;
402 }
403
404 is_stdalone = vkread32(vk, BAR_0, BAR_BOOT_STATUS) &
405 BOOT_STDALONE_RUNNING;
406 if (!is_stdalone) {
407 ret = bcm_vk_intf_ver_chk(vk);
408 if (ret) {
409 dev_err(dev, "failure in intf version check\n");
410 goto err_firmware_out;
411 }
412
413 /* sync & channel other info */
414 ret = bcm_vk_sync_card_info(vk);
415 if (ret) {
416 dev_err(dev, "Syncing Card Info failure\n");
417 goto err_firmware_out;
418 }
419 }
420 }
421
422 err_firmware_out:
423 release_firmware(fw);
424
425 err_buf_out:
426 if (bufp)
427 dma_free_coherent(dev, max_buf, bufp, boot_dma_addr);
428
429 return ret;
430 }
431

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx

Attachment: .config.gz
Description: application/gzip

Next message: Kees Cook: "Re: [PATCH V3 1/8] sysfs: Add sysfs_emit and sysfs_emit_at to format sysfs output"
Previous message: Dmitry Osipenko: "Re: [PATCH v3 2/3] iommu/tegra-smmu: Rework .probe_device and .attach_dev"
Next in thread: Scott Branden: "Re: [PATCH v5 03/15] misc: bcm-vk: add autoload support"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]