From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 25211C433EF for ; Tue, 28 Jun 2022 04:37:25 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S243950AbiF1EhW (ORCPT ); Tue, 28 Jun 2022 00:37:22 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59218 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S244576AbiF1Egz (ORCPT ); Tue, 28 Jun 2022 00:36:55 -0400 Received: from muru.com (muru.com [72.249.23.125]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id F39181CFF5; Mon, 27 Jun 2022 21:36:51 -0700 (PDT) Received: from localhost (localhost [127.0.0.1]) by muru.com (Postfix) with ESMTPS id 96B228106; Tue, 28 Jun 2022 04:31:39 +0000 (UTC) Date: Tue, 28 Jun 2022 07:36:50 +0300 From: Tony Lindgren To: Laurent Pinchart Cc: Miaoqian Lin , Russell King , Tomi Valkeinen , Sebastian Reichel , linux-arm-kernel@lists.infradead.org, linux-omap@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] ARM: OMAP2+: Fix refcount leak in omapdss_init_of Message-ID: References: <20220601044858.3352-1-linmq006@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Laurent Pinchart [220601 06:46]: > Hello Lin, > > Thank you for the patch. > > On Wed, Jun 01, 2022 at 08:48:58AM +0400, Miaoqian Lin wrote: > > omapdss_find_dss_of_node() calls of_find_compatible_node() to get device > > node. of_find_compatible_node() returns a node pointer with refcount > > incremented, we should use of_node_put() on it when done. > > Add missing of_node_put() in later error path and normal path. > > > > Fixes: e0c827aca0730 ("drm/omap: Populate DSS children in omapdss driver") > > Signed-off-by: Miaoqian Lin > > Reviewed-by: Laurent Pinchart Thanks applying this into omap-for-v5.20/soc. Regards, Tony