From mboxrd@z Thu Jan 1 00:00:00 1970 From: skannan@codeaurora.org Subject: Re: [Patch 2/3] clk: skip re-parenting orphan clk Date: Thu, 2 May 2013 03:09:24 -0700 Message-ID: References: <1367475929-32166-1-git-send-email-ambresh@ti.com> <1367475929-32166-3-git-send-email-ambresh@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7BIT Return-path: In-Reply-To: <1367475929-32166-3-git-send-email-ambresh@ti.com> Sender: linux-kernel-owner@vger.kernel.org Cc: Mike Turquette , Nishanth Menon , Rajendra , linux-kernel@vger.kernel.org, Tero Kristo , Ambresh K , linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org List-Id: linux-omap@vger.kernel.org Ambresh K wrote: > From: Ambresh K > > If clk is same as orphan clk than skip the iteration, there Typo: than => then > by avoiding unnecessary look-up. > > Signed-off-by: Ambresh K > --- > drivers/clk/clk.c | 4 ++++ > 1 files changed, 4 insertions(+), 0 deletions(-) > > diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c > index c187321..f4d2c73 100644 > --- a/drivers/clk/clk.c > +++ b/drivers/clk/clk.c > @@ -1635,6 +1635,10 @@ int __clk_init(struct device *dev, struct clk *clk) > * this clock > */ > hlist_for_each_entry_safe(orphan, tmp2, &clk_orphan_list, child_node) { > + /* Skip if clk is same as orphan clk */ > + if (!strcmp(clk->name, orphan->name)) > + continue; > + > if (orphan->ops->get_parent) { > i = orphan->ops->get_parent(orphan->hw); > if (i < 0) { > -- > 1.7.4.1 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > -- The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation