Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752666AbbBZGNS (ORCPT ); Thu, 26 Feb 2015 01:13:18 -0500 Received: from mail-gw2-out.broadcom.com ([216.31.210.63]:21530 "EHLO mail-gw2-out.broadcom.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750768AbbBZGNQ (ORCPT ); Thu, 26 Feb 2015 01:13:16 -0500 X-IronPort-AV: E=Sophos;i="5.09,650,1418112000"; d="scan'208";a="58055283" Message-ID: <54EEB97B.6080006@broadcom.com> Date: Wed, 25 Feb 2015 22:13:15 -0800 From: Ray Jui User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 MIME-Version: 1.0 To: Sascha Hauer CC: Mike Turquette , Stephen Boyd , Matt Porter , Alex Elder , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Russell King , Arnd Bergmann , , , Scott Branden , "Dmitry Torokhov" , Anatol Pomazau , , Subject: Re: [PATCH v5 1/6] clk: add of_clk_get_parent_rate function References: <1423097705-22939-1-git-send-email-rjui@broadcom.com> <1423097705-22939-2-git-send-email-rjui@broadcom.com> <20150226055406.GB28214@pengutronix.de> In-Reply-To: <20150226055406.GB28214@pengutronix.de> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1643 Lines: 38 Hi Sascha, On 2/25/2015 9:54 PM, Sascha Hauer wrote: > Hi Ray, > > On Wed, Feb 04, 2015 at 04:55:00PM -0800, Ray Jui wrote: >> Sometimes a clock needs to know the rate of its parent before itself is >> registered to the framework. An example is that a PLL may need to >> initialize itself to a specific VCO frequency, before registering to the >> framework. The parent rate needs to be known, for PLL multipliers and >> divisors to be configured properly. >> >> Introduce helper function of_clk_get_parent_rate, which can be used to >> obtain the parent rate of a clock, given a device node and index. > > I can't see how this patch helps you. First it's not guaranteed that > the parent is already registered, what do you do in this case? In the case when clock parent is not found, as you can see from the code, it simply returns zero, just like other clk get rate APIs. I thought the order of clock registration is based on order of the clock nodes in device tree. It makes sense to me to declare the parent clock before a child clock, so it's guaranteed that the parent is registered before the child. > Then the clock framework doesn't require that you initialize the PLL > before registering. That can be done in the clk ops later. Sure it's not mandatory. But what's wrong with me choosing to initialize the PLL clock to a known frequency before registering it to the framework? Ray -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/