linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] lib: fix for_each_child.cocci warnings
@ 2021-05-11 20:46 Julia Lawall
  2021-05-17 22:08 ` Linus Walleij
  0 siblings, 1 reply; 2+ messages in thread
From: Julia Lawall @ 2021-05-11 20:46 UTC (permalink / raw)
  To: Julian Braha
  Cc: Andrew Morton, Linux Memory Management List, linux-kernel, kbuild-all

From: kernel test robot <lkp@intel.com>

For_each_available_child_of_node should have of_node_put() before return
around line 1121.

Generated by: scripts/coccinelle/iterators/for_each_child.cocci

Fixes: 7d37cb2c912d ("lib: fix kconfig dependency on ARCH_WANT_FRAME_POINTERS")
CC: Julian Braha <julianbraha@gmail.com>
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: kernel test robot <lkp@intel.com>
Signed-off-by: Julia Lawall <julia.lawall@inria.fr>
---

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   1140ab592e2ebf8153d2b322604031a8868ce7a5
commit: 7d37cb2c912dc5c25ffac784a4f9b98c06c6bd08 lib: fix kconfig dependency on ARCH_WANT_FRAME_POINTERS
:::::: branch date: 15 hours ago
:::::: commit date: 5 weeks ago

 mcde_dsi.c |    1 +
 1 file changed, 1 insertion(+)

--- a/drivers/gpu/drm/mcde/mcde_dsi.c
+++ b/drivers/gpu/drm/mcde/mcde_dsi.c
@@ -1118,6 +1118,7 @@ static int mcde_dsi_bind(struct device *
 			bridge = of_drm_find_bridge(child);
 			if (!bridge) {
 				dev_err(dev, "failed to find bridge\n");
+				of_node_put(child);
 				return -EINVAL;
 			}
 		}


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] lib: fix for_each_child.cocci warnings
  2021-05-11 20:46 [PATCH] lib: fix for_each_child.cocci warnings Julia Lawall
@ 2021-05-17 22:08 ` Linus Walleij
  0 siblings, 0 replies; 2+ messages in thread
From: Linus Walleij @ 2021-05-17 22:08 UTC (permalink / raw)
  To: Julia Lawall
  Cc: Julian Braha, Andrew Morton, Linux Memory Management List,
	linux-kernel, kbuild-all

On Tue, May 11, 2021 at 10:46 PM Julia Lawall <julia.lawall@inria.fr> wrote:

> From: kernel test robot <lkp@intel.com>
>
> For_each_available_child_of_node should have of_node_put() before return
> around line 1121.
>
> Generated by: scripts/coccinelle/iterators/for_each_child.cocci
>
> Fixes: 7d37cb2c912d ("lib: fix kconfig dependency on ARCH_WANT_FRAME_POINTERS")
> CC: Julian Braha <julianbraha@gmail.com>
> Reported-by: kernel test robot <lkp@intel.com>
> Signed-off-by: kernel test robot <lkp@intel.com>
> Signed-off-by: Julia Lawall <julia.lawall@inria.fr>

Acked-by: Linus Walleij <linus.walleij@linaro.org>

Shall I apply it to the DRM tree or do you plan on funneling it
some other way?

Yours,
Linus Walleij


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-05-17 22:08 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-11 20:46 [PATCH] lib: fix for_each_child.cocci warnings Julia Lawall
2021-05-17 22:08 ` Linus Walleij

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).