Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756929AbbEETzw (ORCPT ); Tue, 5 May 2015 15:55:52 -0400 Received: from hqemgate15.nvidia.com ([216.228.121.64]:4069 "EHLO hqemgate15.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752520AbbEETzu (ORCPT ); Tue, 5 May 2015 15:55:50 -0400 X-PGP-Universal: processed; by hqnvupgp07.nvidia.com on Tue, 05 May 2015 12:54:23 -0700 Message-ID: <55492043.2010202@nvidia.com> Date: Tue, 5 May 2015 15:55:47 -0400 From: Rhyland Klein User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Benson Leung CC: Peter De Schrijver , Mike Turquette , Stephen Warren , Stephen Boyd , Thierry Reding , Alexandre Courbot , , , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v4 10/20] clk: tegra: pll: Add logic for out-of-table rates for T210 References: <1430757460-9478-1-git-send-email-rklein@nvidia.com> <1430757460-9478-11-git-send-email-rklein@nvidia.com> In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2329 Lines: 66 On 5/4/2015 7:34 PM, Benson Leung wrote: > On Mon, May 4, 2015 at 9:37 AM, Rhyland Klein wrote: >> For Tegra210, the logic to calculate out-of-table rates is different >> from previous generations. Add callbacks that can be overridden to >> allow for different ways of calculating rates. Default to >> _cal_rate when not specified. > > You should mention that you're adding a new flag and new members that > may override mdiv as well. Yes, will add that. > >> >> Based on original work by Aleksandr Frid >> >> Signed-off-by: Rhyland Klein >> --- >> diff --git a/drivers/clk/tegra/clk.h b/drivers/clk/tegra/clk.h >> index 3f92f1ad3961..b63ef31a2d7a 100644 >> --- a/drivers/clk/tegra/clk.h >> +++ b/drivers/clk/tegra/clk.h >> @@ -199,6 +199,8 @@ struct div_nmp { >> * base register. >> * TEGRA_PLL_BYPASS - PLL has bypass bit >> * TEGRA_PLL_HAS_LOCK_ENABLE - PLL has bit to enable lock monitoring >> + * TEGRA_MDIV_NEW - Switch to new method for calculating fixed mdiv >> + * it may be more accurate (especially if SDM present) >> */ >> struct tegra_clk_pll_params { >> unsigned long input_min; >> @@ -235,7 +237,13 @@ struct tegra_clk_pll_params { >> struct div_nmp *div_nmp; >> struct tegra_clk_pll_freq_table *freq_table; >> unsigned long fixed_rate; >> + bool vco_out; > > vco_out is unused and unrelated to this change? vco_out is used to mark which PLL's have a vco_out, and is going to be used in the new out-of-table rate calculation in tegra210, so I will move adding this member until that patch. > >> + u16 mdiv_default; >> + u32 (*round_p_to_pdiv)(u32 p, u32 *pdiv); >> void (*set_gain)(struct tegra_clk_pll_freq_table *cfg); >> + int (*calc_rate)(struct clk_hw *hw, >> + struct tegra_clk_pll_freq_table *cfg, >> + unsigned long rate, unsigned long parent_rate); > > Add kerneldoc for new members. Will do, thanks! -rhyland -- nvpublic -- 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/