From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jon Hunter Subject: Re: [PATCH 2/2] clk: tegra: Micro-optimize Tegra210 clock setup Date: Thu, 23 Jun 2016 13:26:29 +0100 Message-ID: <576BD575.8010808@nvidia.com> References: <20160623105231.24383-1-thierry.reding@gmail.com> <20160623105231.24383-2-thierry.reding@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20160623105231.24383-2-thierry.reding@gmail.com> Sender: linux-clk-owner@vger.kernel.org To: Thierry Reding Cc: Peter De Schrijver , Rhyland Klein , linux-tegra@vger.kernel.org, linux-clk@vger.kernel.org List-Id: linux-tegra@vger.kernel.org On 23/06/16 11:52, Thierry Reding wrote: > From: Thierry Reding > > sor_safe being the parent of the dpaux and dpaux1 clocks, it's not only > natural, but also slightly more efficient, to initialize it before its > children. This avoids orphaning the dpaux and dpaux1 clocks only to get > them reparented when the sor_safe clock is registered. > > Signed-off-by: Thierry Reding > --- > drivers/clk/tegra/clk-tegra210.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/clk/tegra/clk-tegra210.c b/drivers/clk/tegra/clk-tegra210.c > index fe295b4102ca..b4df5c46642f 100644 > --- a/drivers/clk/tegra/clk-tegra210.c > +++ b/drivers/clk/tegra/clk-tegra210.c > @@ -2466,6 +2466,10 @@ static __init void tegra210_periph_clk_init(void __iomem *clk_base, > 1, 2); > clks[TEGRA210_CLK_XUSB_SS_DIV2] = clk; > > + clk = tegra_clk_register_periph_fixed("sor_safe", "pll_p", 0, clk_base, > + 1, 17, 222); > + clks[TEGRA210_CLK_SOR_SAFE] = clk; > + > clk = tegra_clk_register_periph_fixed("dpaux", "sor_safe", 0, clk_base, > 1, 17, 181); > clks[TEGRA210_CLK_DPAUX] = clk; > @@ -2474,10 +2478,6 @@ static __init void tegra210_periph_clk_init(void __iomem *clk_base, > 1, 17, 207); > clks[TEGRA210_CLK_DPAUX1] = clk; > > - clk = tegra_clk_register_periph_fixed("sor_safe", "pll_p", 0, clk_base, > - 1, 17, 222); > - clks[TEGRA210_CLK_SOR_SAFE] = clk; > - > /* pll_d_dsi_out */ > clk = clk_register_gate(NULL, "pll_d_dsi_out", "pll_d_out0", 0, > clk_base + PLLD_MISC0, 21, 0, &pll_d_lock); > Acked-by: Jon Hunter Tested-by: Jon Hunter Cheers Jon -- nvpublic