2022-05-11 12:06:15

by Dan Carpenter

[permalink] [raw]
Subject: [mtd:spi-nor/next 16/18] drivers/mtd/spi-nor/debugfs.c:84 spi_nor_params_show() warn: '%ph' cannot be followed by 'n'

tree: https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git spi-nor/next
head: cdbc44dbb2c73d8bafa1a8ae0d780608be5dbd40
commit: 0257be79fc4a16a3252ce80aa13b3640f728c425 [16/18] mtd: spi-nor: expose internal parameters via debugfs
config: x86_64-randconfig-m001-20220509 (https://download.01.org/0day-ci/archive/20220511/[email protected]/config)
compiler: gcc-11 (Debian 11.2.0-20) 11.2.0

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

smatch warnings:
drivers/mtd/spi-nor/debugfs.c:84 spi_nor_params_show() warn: '%ph' cannot be followed by 'n'

vim +84 drivers/mtd/spi-nor/debugfs.c

0257be79fc4a16 Michael Walle 2022-04-29 73 static int spi_nor_params_show(struct seq_file *s, void *data)
0257be79fc4a16 Michael Walle 2022-04-29 74 {
0257be79fc4a16 Michael Walle 2022-04-29 75 struct spi_nor *nor = s->private;
0257be79fc4a16 Michael Walle 2022-04-29 76 struct spi_nor_flash_parameter *params = nor->params;
0257be79fc4a16 Michael Walle 2022-04-29 77 struct spi_nor_erase_map *erase_map = &params->erase_map;
0257be79fc4a16 Michael Walle 2022-04-29 78 struct spi_nor_erase_region *region;
0257be79fc4a16 Michael Walle 2022-04-29 79 const struct flash_info *info = nor->info;
0257be79fc4a16 Michael Walle 2022-04-29 80 char buf[16], *str;
0257be79fc4a16 Michael Walle 2022-04-29 81 int i;
0257be79fc4a16 Michael Walle 2022-04-29 82
0257be79fc4a16 Michael Walle 2022-04-29 83 seq_printf(s, "name\t\t%s\n", info->name);
0257be79fc4a16 Michael Walle 2022-04-29 @84 seq_printf(s, "id\t\t%*phn\n", info->id_len, info->id);

Maybe N was intended instead of n? (I have no idea).

0257be79fc4a16 Michael Walle 2022-04-29 85 string_get_size(params->size, 1, STRING_UNITS_2, buf, sizeof(buf));
0257be79fc4a16 Michael Walle 2022-04-29 86 seq_printf(s, "size\t\t%s\n", buf);
0257be79fc4a16 Michael Walle 2022-04-29 87 seq_printf(s, "write size\t%u\n", params->writesize);
0257be79fc4a16 Michael Walle 2022-04-29 88 seq_printf(s, "page size\t%u\n", params->page_size);
0257be79fc4a16 Michael Walle 2022-04-29 89 seq_printf(s, "address width\t%u\n", nor->addr_width);
0257be79fc4a16 Michael Walle 2022-04-29 90
0257be79fc4a16 Michael Walle 2022-04-29 91 seq_puts(s, "flags\t\t");

--
0-DAY CI Kernel Test Service
https://01.org/lkp



2022-05-12 12:49:09

by Michael Walle

[permalink] [raw]
Subject: Re: [mtd:spi-nor/next 16/18] drivers/mtd/spi-nor/debugfs.c:84 spi_nor_params_show() warn: '%ph' cannot be followed by 'n'

Am 2022-05-11 12:17, schrieb Dan Carpenter:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git
> spi-nor/next
> head: cdbc44dbb2c73d8bafa1a8ae0d780608be5dbd40
> commit: 0257be79fc4a16a3252ce80aa13b3640f728c425 [16/18] mtd: spi-nor:
> expose internal parameters via debugfs
> config: x86_64-randconfig-m001-20220509
> (https://download.01.org/0day-ci/archive/20220511/[email protected]/config)
> compiler: gcc-11 (Debian 11.2.0-20) 11.2.0
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <[email protected]>
> Reported-by: Dan Carpenter <[email protected]>
>
> smatch warnings:
> drivers/mtd/spi-nor/debugfs.c:84 spi_nor_params_show() warn: '%ph'
> cannot be followed by 'n'
>
> vim +84 drivers/mtd/spi-nor/debugfs.c
>
> 0257be79fc4a16 Michael Walle 2022-04-29 73 static int
> spi_nor_params_show(struct seq_file *s, void *data)
> 0257be79fc4a16 Michael Walle 2022-04-29 74 {
> 0257be79fc4a16 Michael Walle 2022-04-29 75 struct spi_nor *nor =
> s->private;
> 0257be79fc4a16 Michael Walle 2022-04-29 76 struct
> spi_nor_flash_parameter *params = nor->params;
> 0257be79fc4a16 Michael Walle 2022-04-29 77 struct
> spi_nor_erase_map *erase_map = &params->erase_map;
> 0257be79fc4a16 Michael Walle 2022-04-29 78 struct
> spi_nor_erase_region *region;
> 0257be79fc4a16 Michael Walle 2022-04-29 79 const struct flash_info
> *info = nor->info;
> 0257be79fc4a16 Michael Walle 2022-04-29 80 char buf[16], *str;
> 0257be79fc4a16 Michael Walle 2022-04-29 81 int i;
> 0257be79fc4a16 Michael Walle 2022-04-29 82
> 0257be79fc4a16 Michael Walle 2022-04-29 83 seq_printf(s,
> "name\t\t%s\n", info->name);
> 0257be79fc4a16 Michael Walle 2022-04-29 @84 seq_printf(s,
> "id\t\t%*phn\n", info->id_len, info->id);
>
> Maybe N was intended instead of n? (I have no idea).

Thanks for reporting. Yeah I had %*phN previously but then
I wanted to switch to "xx xx xx" style and i missed checking
printk-formats.rst. %*phn worked as intended ;) I'll send
a fixup patch.

-michael