From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============1555529380447389732==" MIME-Version: 1.0 From: kernel test robot Subject: [PATCH] v4l: xilinx-vpss-csc: fix for_each_child.cocci warnings Date: Sat, 28 Aug 2021 17:19:36 +0800 Message-ID: <20210828091936.GA36690@72ef8f4dd4c6> In-Reply-To: <202108281703.6bOqPBYe-lkp@intel.com> List-Id: To: kbuild@lists.01.org --===============1555529380447389732== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable CC: kbuild-all(a)lists.01.org CC: linux-arm-kernel(a)lists.infradead.org TO: Rohit Athavale CC: Michal Simek CC: Hyun Kwon CC: Laurent Pinchart CC: Mauro Carvalho Chehab CC: linux-media(a)vger.kernel.org CC: linux-arm-kernel(a)lists.infradead.org CC: linux-kernel(a)vger.kernel.org From: kernel test robot drivers/media/platform/xilinx/xilinx-vpss-csc.c:747:1-23: WARNING: Function= "for_each_child_of_node" should have of_node_put() before return around li= ne 752. drivers/media/platform/xilinx/xilinx-vpss-csc.c:747:1-23: WARNING: Function= "for_each_child_of_node" should have of_node_put() before return around li= ne 758. drivers/media/platform/xilinx/xilinx-vpss-csc.c:747:1-23: WARNING: Function= "for_each_child_of_node" should have of_node_put() before return around li= ne 763. Semantic patch information: False positives can be due to function calls within the for_each loop that may encapsulate an of_node_put. Generated by: scripts/coccinelle/iterators/for_each_child.cocci CC: Rohit Athavale Reported-by: kernel test robot Signed-off-by: kernel test robot --- tree: https://github.com/Xilinx/linux-xlnx master head: 45cd0074cdf1ddd710b28848e6a860b442babfcc commit: 7d47a653da2e81cb73f43e507664d36305819ea1 [10741/12418] v4l: xilinx-= vpss-csc: driver support for xilinx vpss csc :::::: branch date: 5 days ago :::::: commit date: 5 months ago Please take the patch only if it's a positive warning. Thanks! xilinx-vpss-csc.c | 3 +++ 1 file changed, 3 insertions(+) --- a/drivers/media/platform/xilinx/xilinx-vpss-csc.c +++ b/drivers/media/platform/xilinx/xilinx-vpss-csc.c @@ -749,17 +749,20 @@ static int xcsc_parse_of(struct xcsc_dev vip_format =3D xvip_of_get_format(port); if (IS_ERR(vip_format)) { dev_err(dev, "Invalid media pad format in DT"); + of_node_put(port); return PTR_ERR(vip_format); } = rval =3D of_property_read_u32(port, "reg", &port_id); if (rval < 0) { dev_err(dev, "No reg in DT to specify pad"); + of_node_put(port); return rval; } = if (port_id !=3D 0 && port_id !=3D 1) { dev_err(dev, "Invalid reg in DT"); + of_node_put(port); return -EINVAL; } xcsc->vip_formats[port_id] =3D vip_format; --===============1555529380447389732==--