Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp999513imm; Fri, 13 Jul 2018 09:44:19 -0700 (PDT) X-Google-Smtp-Source: AAOMgpdGfnbQp1A+T4cPeyHdRUQgGBTuMVvUALtDJru7a4RdhoibJN3bzs1IE8+Kz+Jt9F4pOT3O X-Received: by 2002:a63:8f53:: with SMTP id r19-v6mr6694831pgn.17.1531500259903; Fri, 13 Jul 2018 09:44:19 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1531500259; cv=none; d=google.com; s=arc-20160816; b=HxLT7HKHHFlawhOGsJrXoL2E3pJ7f4Mg7ODTK2dkl65fAZLmFYy44QzUbBIRmHeXFK Qwkr1lroah/Ct7uPLg/zpMOMFvdZ4SO3iy8bm7FLlf2s/E9HNmZRfwhR+aR4FkK3OwNb YL/WAFjDWHOu4+tGmcZIUE9cDzMAtmt5U8tCbHzTFoz4nt/RexetFmKNXInuphBiW/k/ 2W3NYkaqiOgrJo7nGgvEaaYjIE2WjK7t0TaUxGcylpC38FtWrSl2A8UMEZ1IaGjquf5w Gyps4/RF/6XAcmitzij1FkH0Yo3RL0yCRSd4kTs8UE0qH6ontGmH1MBOppNPtZSB0h0R O55w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:arc-authentication-results; bh=KCZeT+GTKTF1/bZUJan0PfsqWQfd5DihlUiUA9BVi2A=; b=GS2pKTeFa+PiJp1iCKZlADokT4EZBC2XPI5JrpZfoEmAlWSlqfvJe9TKV0EpIsnNa3 u5BrskrA5eBIq4dY45nSOf3qzd7/t9MBSmdh+5KLcKNu82KEKEz/b7brvszt81PTOXME yetXxkhxWdT2GInA1Aec+i2iMYz45InIxs2z2LwzQQFdub2/jMVjMJjO76uld8fz9K6Z NHovSkgj8+5Q3G92VwUgiOmcUMLsf0uBGuIY8gQavDVTPw7ZCeK0D1kv8TrQmhSUEFCh HKm1YgqqePy9DrhNejO4QUW41TDoppyA2/HiWhDrj2ceQeF6zeGx8BCo1Gt9QX0SQ5jy +fsw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id f12-v6si22345654pgo.203.2018.07.13.09.44.04; Fri, 13 Jul 2018 09:44:19 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731955AbeGMQ6I (ORCPT + 99 others); Fri, 13 Jul 2018 12:58:08 -0400 Received: from 178.115.242.59.static.drei.at ([178.115.242.59]:45164 "EHLO mail.osadl.at" rhost-flags-OK-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1729935AbeGMQ6I (ORCPT ); Fri, 13 Jul 2018 12:58:08 -0400 Received: by mail.osadl.at (Postfix, from userid 1001) id 8D8F35C0F8B; Fri, 13 Jul 2018 16:41:10 +0000 (UTC) Date: Fri, 13 Jul 2018 16:41:10 +0000 From: Nicholas Mc Guire To: Stephen Boyd Cc: Nicholas Mc Guire , Shawn Guo , Frank Li , Michael Turquette , linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, NXP Linux Team , Pengutronix Kernel Team , Fabio Estevam , Sascha Hauer , linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH] clk: imx6ul: fix missing of_node_put() Message-ID: <20180713164110.GA27943@osadl.at> References: <1531480400-7664-1-git-send-email-hofrat@osadl.org> <153149925072.48062.9395649003513930129@swboyd.mtv.corp.google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <153149925072.48062.9395649003513930129@swboyd.mtv.corp.google.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jul 13, 2018 at 09:27:30AM -0700, Stephen Boyd wrote: > Quoting Nicholas Mc Guire (2018-07-13 04:13:20) > > of_find_compatible_node() is returning a device node with refcount > > incremented and must be explicitly decremented after the last use > > which is right after the us in of_iomap() here. > > > > Signed-off-by: Nicholas Mc Guire > > Fixes: commit 787b4271a6a0 ("clk: imx: add imx6ul clk tree support") > > This is not how a Fixes line works. It doesn't have 'commit'. I'll > remove it when applying so no need to resend. > sorry - guess I got that wrong on a whole set of patches - fill fix that for future patches and thanks for fixing it here thx! hofrat