2020-05-29 21:39:14

by kernel test robot

[permalink] [raw]
Subject: drivers/gpu/drm/nouveau/nvkm/nvfw/acr.c:49:1: warning: no previous prototype for 'lsb_header_tail_dump'

Hi Ben,

FYI, the error/warning still remains.

tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 75caf310d16cc5e2f851c048cd597f5437013368
commit: 22dcda45a3d1dfe6eeb4ab0a3b9aaa2333cb649d drm/nouveau/acr: implement new subdev to replace "secure boot"
date: 5 months ago
config: arm64-sof-customedconfig-sst-defconfig (attached as .config)
compiler: aarch64-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
git checkout 22dcda45a3d1dfe6eeb4ab0a3b9aaa2333cb649d
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arm64

If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <[email protected]>

All warnings (new ones prefixed by >>, old ones prefixed by <<):

>> drivers/gpu/drm/nouveau/nvkm/nvfw/acr.c:49:1: warning: no previous prototype for 'lsb_header_tail_dump' [-Wmissing-prototypes]
49 | lsb_header_tail_dump(struct nvkm_subdev *subdev,
| ^~~~~~~~~~~~~~~~~~~~

vim +/lsb_header_tail_dump +49 drivers/gpu/drm/nouveau/nvkm/nvfw/acr.c

47
48 void
> 49 lsb_header_tail_dump(struct nvkm_subdev *subdev,
50 struct lsb_header_tail *hdr)
51 {
52 nvkm_debug(subdev, "lsbHeader\n");
53 nvkm_debug(subdev, "\tucodeOff : 0x%x\n", hdr->ucode_off);
54 nvkm_debug(subdev, "\tucodeSize : 0x%x\n", hdr->ucode_size);
55 nvkm_debug(subdev, "\tdataSize : 0x%x\n", hdr->data_size);
56 nvkm_debug(subdev, "\tblCodeSize : 0x%x\n", hdr->bl_code_size);
57 nvkm_debug(subdev, "\tblImemOff : 0x%x\n", hdr->bl_imem_off);
58 nvkm_debug(subdev, "\tblDataOff : 0x%x\n", hdr->bl_data_off);
59 nvkm_debug(subdev, "\tblDataSize : 0x%x\n", hdr->bl_data_size);
60 nvkm_debug(subdev, "\tappCodeOff : 0x%x\n", hdr->app_code_off);
61 nvkm_debug(subdev, "\tappCodeSize : 0x%x\n", hdr->app_code_size);
62 nvkm_debug(subdev, "\tappDataOff : 0x%x\n", hdr->app_data_off);
63 nvkm_debug(subdev, "\tappDataSize : 0x%x\n", hdr->app_data_size);
64 nvkm_debug(subdev, "\tflags : 0x%x\n", hdr->flags);
65 }
66

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]


Attachments:
(No filename) (2.43 kB)
.config.gz (40.47 kB)
Download all attachments

2020-05-29 23:39:39

by Ben Skeggs

[permalink] [raw]
Subject: Re: drivers/gpu/drm/nouveau/nvkm/nvfw/acr.c:49:1: warning: no previous prototype for 'lsb_header_tail_dump'

On Sat, May 30, 2020 at 7:00 AM kbuild test robot <[email protected]> wrote:
>
> Hi Ben,
>
> FYI, the error/warning still remains.
Thanks, I've got a patch in my tree.

Ben.

>
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head: 75caf310d16cc5e2f851c048cd597f5437013368
> commit: 22dcda45a3d1dfe6eeb4ab0a3b9aaa2333cb649d drm/nouveau/acr: implement new subdev to replace "secure boot"
> date: 5 months ago
> config: arm64-sof-customedconfig-sst-defconfig (attached as .config)
> compiler: aarch64-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
> git checkout 22dcda45a3d1dfe6eeb4ab0a3b9aaa2333cb649d
> # save the attached .config to linux build tree
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arm64
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kbuild test robot <[email protected]>
>
> All warnings (new ones prefixed by >>, old ones prefixed by <<):
>
> >> drivers/gpu/drm/nouveau/nvkm/nvfw/acr.c:49:1: warning: no previous prototype for 'lsb_header_tail_dump' [-Wmissing-prototypes]
> 49 | lsb_header_tail_dump(struct nvkm_subdev *subdev,
> | ^~~~~~~~~~~~~~~~~~~~
>
> vim +/lsb_header_tail_dump +49 drivers/gpu/drm/nouveau/nvkm/nvfw/acr.c
>
> 47
> 48 void
> > 49 lsb_header_tail_dump(struct nvkm_subdev *subdev,
> 50 struct lsb_header_tail *hdr)
> 51 {
> 52 nvkm_debug(subdev, "lsbHeader\n");
> 53 nvkm_debug(subdev, "\tucodeOff : 0x%x\n", hdr->ucode_off);
> 54 nvkm_debug(subdev, "\tucodeSize : 0x%x\n", hdr->ucode_size);
> 55 nvkm_debug(subdev, "\tdataSize : 0x%x\n", hdr->data_size);
> 56 nvkm_debug(subdev, "\tblCodeSize : 0x%x\n", hdr->bl_code_size);
> 57 nvkm_debug(subdev, "\tblImemOff : 0x%x\n", hdr->bl_imem_off);
> 58 nvkm_debug(subdev, "\tblDataOff : 0x%x\n", hdr->bl_data_off);
> 59 nvkm_debug(subdev, "\tblDataSize : 0x%x\n", hdr->bl_data_size);
> 60 nvkm_debug(subdev, "\tappCodeOff : 0x%x\n", hdr->app_code_off);
> 61 nvkm_debug(subdev, "\tappCodeSize : 0x%x\n", hdr->app_code_size);
> 62 nvkm_debug(subdev, "\tappDataOff : 0x%x\n", hdr->app_data_off);
> 63 nvkm_debug(subdev, "\tappDataSize : 0x%x\n", hdr->app_data_size);
> 64 nvkm_debug(subdev, "\tflags : 0x%x\n", hdr->flags);
> 65 }
> 66
>
> ---
> 0-DAY CI Kernel Test Service, Intel Corporation
> https://lists.01.org/hyperkitty/list/[email protected]