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

From: Philip Li
Date: Fri Oct 02 2020 - 01:39:51 EST

Next message: Philip Li: "Re: [PATCH 4/5] clk: qcom: Add SDM660 GPU Clock Controller (GPUCC) driver"
Previous message: Yun Hsiang: "Re: [PATCH 1/1] sched/uclamp: release per-task uclamp control if user set to default value"
In reply to: Scott Branden: "Re: [PATCH v5 03/15] misc: bcm-vk: add autoload support"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

On Thu, Oct 01, 2020 at 10:48:08AM -0700, Scott Branden wrote:
>
>
> On 2020-09-30 9:07 p.m., kernel test robot wrote:
> > 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.
> Yes, this patch series will not compile as described in the cover letter without Kees' patches:
>
> This patch series drops previous patches in [1]
> that were incorporated by Kees Cook into patch series
> "Introduce partial kernel_read_file() support" [2].
>
> Remaining patches are contained in this series to add Broadcom VK driver.
> (which depends on request_firmware_into_buf API addition in
> other patch series [2] being applied first).
>
> Please note this patch series will not compile without [2].
Thanks for the detail info, and sorry for the wrong report. Currently
we are not able to analyz the info in cover letter which may lead to
wrong base used. If possible, you can give a try to use --base option
when git format patch, this provides more hints to us when looking
for a base.

>
> [1] https://lore.kernel.org/lkml/20200706232309.12010-1-scott.branden@xxxxxxxxxxxx/
> [2] https://lore.kernel.org/lkml/20200729175845.1745471-1-keescook@xxxxxxxxxxxx/
>
>
>
> > 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
>

> _______________________________________________
> kbuild-all mailing list -- kbuild-all@xxxxxxxxxxxx
> To unsubscribe send an email to kbuild-all-leave@xxxxxxxxxxxx

Next message: Philip Li: "Re: [PATCH 4/5] clk: qcom: Add SDM660 GPU Clock Controller (GPUCC) driver"
Previous message: Yun Hsiang: "Re: [PATCH 1/1] sched/uclamp: release per-task uclamp control if user set to default value"
In reply to: Scott Branden: "Re: [PATCH v5 03/15] misc: bcm-vk: add autoload support"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]