[PATCH] spi: spi-zynqmp-gqspi: fix for_each_child.cocci warnings
by Julia Lawall
From: kernel test robot <lkp(a)intel.com>
for_each_available_child_of_node should have of_node_put() before break.
Generated by: scripts/coccinelle/iterators/for_each_child.cocci
Fixes: f0fcd7d14036 ("spi: spi-zynqmp-gqspi: Fix suspend/resume")
CC: Amit Kumar Mahapatra <amit.kumar-mahapatra(a)xilinx.com>
Reported-by: kernel test robot <lkp(a)intel.com>
Signed-off-by: kernel test robot <lkp(a)intel.com>
Signed-off-by: Julia Lawall <julia.lawall(a)inria.fr>
---
tree: https://github.com/Xilinx/linux-xlnx xlnx_rebase_v5.15
head: 1183ce490adb103e5e569b8ebd74c50c885ddc05
commit: f0fcd7d140361b1f54cfd76afb8c3afc9bf218a5 [857/872] spi: spi-zynqmp-gqspi: Fix suspend/resume
:::::: branch date: 3 days ago
:::::: commit date: 3 days ago
Please take the patch only if it's a positive warning. Thanks!
spi-zynqmp-gqspi.c | 2 ++
1 file changed, 2 insertions(+)
--- a/drivers/spi/spi-zynqmp-gqspi.c
+++ b/drivers/spi/spi-zynqmp-gqspi.c
@@ -1395,6 +1395,7 @@ static int zynqmp_qspi_probe(struct plat
&rx_bus_width);
if (!ret) {
xqspi->rx_bus_width = rx_bus_width;
+ of_node_put(nc);
break;
}
}
@@ -1407,6 +1408,7 @@ static int zynqmp_qspi_probe(struct plat
&tx_bus_width);
if (!ret) {
xqspi->tx_bus_width = tx_bus_width;
+ of_node_put(nc);
break;
}
}
7 months, 1 week
[ammarfaizi2-block:google/android/kernel/common/android-4.19-stable 95/9999] drivers/nvmem/core.c:347:5: warning: no previous prototype for 'nvmem_add_cells'
by kernel test robot
Hi Bartosz,
FYI, the error/warning still remains.
tree: https://github.com/ammarfaizi2/linux-block google/android/kernel/common/android-4.19-stable
head: 69be2ccce366f9ef3eec5cd1c9b8bfb3e1c2d16d
commit: e96a10625581a499e8a4218ef504f3f53918408b [95/9999] UPSTREAM: nvmem: add support for cell info
config: x86_64-randconfig-a011 (https://download.01.org/0day-ci/archive/20220209/202202091936.rHZ0Thvu-lk...)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
# https://github.com/ammarfaizi2/linux-block/commit/e96a10625581a499e8a4218...
git remote add ammarfaizi2-block https://github.com/ammarfaizi2/linux-block
git fetch --no-tags ammarfaizi2-block google/android/kernel/common/android-4.19-stable
git checkout e96a10625581a499e8a4218ef504f3f53918408b
# save the config file to linux build tree
mkdir build_dir
make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/nvmem/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> drivers/nvmem/core.c:347:5: warning: no previous prototype for 'nvmem_add_cells' [-Wmissing-prototypes]
347 | int nvmem_add_cells(struct nvmem_device *nvmem,
| ^~~~~~~~~~~~~~~
vim +/nvmem_add_cells +347 drivers/nvmem/core.c
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 337
b3db17e4b864e4 Andrew Lunn 2018-05-11 338 /**
b3db17e4b864e4 Andrew Lunn 2018-05-11 339 * nvmem_add_cells() - Add cell information to an nvmem device
b3db17e4b864e4 Andrew Lunn 2018-05-11 340 *
b3db17e4b864e4 Andrew Lunn 2018-05-11 341 * @nvmem: nvmem device to add cells to.
b3db17e4b864e4 Andrew Lunn 2018-05-11 342 * @info: nvmem cell info to add to the device
b3db17e4b864e4 Andrew Lunn 2018-05-11 343 * @ncells: number of cells in info
b3db17e4b864e4 Andrew Lunn 2018-05-11 344 *
b3db17e4b864e4 Andrew Lunn 2018-05-11 345 * Return: 0 or negative error code on failure.
b3db17e4b864e4 Andrew Lunn 2018-05-11 346 */
b3db17e4b864e4 Andrew Lunn 2018-05-11 @347 int nvmem_add_cells(struct nvmem_device *nvmem,
b3db17e4b864e4 Andrew Lunn 2018-05-11 348 const struct nvmem_cell_info *info,
b3db17e4b864e4 Andrew Lunn 2018-05-11 349 int ncells)
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 350 {
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 351 struct nvmem_cell **cells;
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 352 int i, rval;
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 353
b3db17e4b864e4 Andrew Lunn 2018-05-11 354 cells = kcalloc(ncells, sizeof(*cells), GFP_KERNEL);
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 355 if (!cells)
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 356 return -ENOMEM;
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 357
b3db17e4b864e4 Andrew Lunn 2018-05-11 358 for (i = 0; i < ncells; i++) {
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 359 cells[i] = kzalloc(sizeof(**cells), GFP_KERNEL);
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 360 if (!cells[i]) {
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 361 rval = -ENOMEM;
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 362 goto err;
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 363 }
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 364
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 365 rval = nvmem_cell_info_to_nvmem_cell(nvmem, &info[i], cells[i]);
287980e49ffc0f Arnd Bergmann 2016-05-27 366 if (rval) {
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 367 kfree(cells[i]);
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 368 goto err;
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 369 }
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 370
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 371 nvmem_cell_add(cells[i]);
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 372 }
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 373
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 374 /* remove tmp array */
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 375 kfree(cells);
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 376
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 377 return 0;
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 378 err:
dfdf141429f089 Rasmus Villemoes 2016-02-08 379 while (i--)
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 380 nvmem_cell_drop(cells[i]);
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 381
dfdf141429f089 Rasmus Villemoes 2016-02-08 382 kfree(cells);
dfdf141429f089 Rasmus Villemoes 2016-02-08 383
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 384 return rval;
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 385 }
b3db17e4b864e4 Andrew Lunn 2018-05-11 386 EXPORT_SYMBOL_GPL(nvmem_add_cells);
eace75cfdcf7d9 Srinivas Kandagatla 2015-07-27 387
:::::: The code at line 347 was first introduced by commit
:::::: b3db17e4b864e46ad150ebef69c0e0130a1c5fca drivers: nvmem: Export nvmem_add_cells()
:::::: TO: Andrew Lunn <andrew(a)lunn.ch>
:::::: CC: Greg Kroah-Hartman <gregkh(a)linuxfoundation.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
7 months, 1 week
[PATCH] mtd: spi-nor: fix minmax.cocci warnings
by Julia Lawall
From: kernel test robot <lkp(a)intel.com>
Simplify the cide using min().
Generated by: scripts/coccinelle/misc/minmax.cocci
Fixes: 2747bc2106aa ("mtd: spi-nor: Add dual parallel and stacked mode support")
CC: Amit Kumar Mahapatra <amit.kumar-mahapatra(a)xilinx.com>
Reported-by: kernel test robot <lkp(a)intel.com>
Signed-off-by: kernel test robot <lkp(a)intel.com>
Signed-off-by: Julia Lawall <julia.lawall(a)inria.fr>
---
tree: https://github.com/Xilinx/linux-xlnx xlnx_rebase_v5.15
head: 1183ce490adb103e5e569b8ebd74c50c885ddc05
commit: 2747bc2106aa429773d9a932c799aa6229595148 [840/872] mtd: spi-nor: Add dual parallel and stacked mode support
:::::: branch date: 7 days ago
:::::: commit date: 7 days ago
core.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
--- a/drivers/mtd/spi-nor/core.c
+++ b/drivers/mtd/spi-nor/core.c
@@ -2034,10 +2034,7 @@ static int spi_nor_read(struct mtd_info
(offset << nor->shift);
}
}
- if (len < rem_bank_len)
- read_len = len;
- else
- read_len = rem_bank_len;
+ read_len = min(len, rem_bank_len);
/* Wait till previous write/erase is done. */
ret = spi_nor_wait_till_ready(nor);
7 months, 1 week
