On Thu, Dec 08, 2011 at 07:25:53PM +0100, Colin Cross wrote:
> On Thu, Dec 8, 2011 at 4:43 AM, Peter De Schrijver
> <pdeschrij...@nvidia.com<mailto:pdeschrij...@nvidia.com>> wrote:
> Rework the tegra20 clock code to support multiple tegra variants :
>
> * remove tegra2_periph_reset_assert/tegra2_periph_reset_deassert. This
> functionality should be in clock.c.
> * compile tegra_sdmmc_tap_delay only on tegra20 as this feature will not
> be available in future variants.
> * don't export clk_measure_input_freq as its functionality is also available
> using clk_get_rate().
>
> Signed-off-by: Peter De Schrijver
> <pdeschrij...@nvidia.com<mailto:pdeschrij...@nvidia.com>>
> ---
> arch/arm/mach-tegra/clock.c | 12 +++++++-----
> arch/arm/mach-tegra/clock.h | 8 ++++----
> arch/arm/mach-tegra/tegra2_clocks.c | 14 +-------------
> arch/arm/mach-tegra/timer.c | 12 ++++++++----
> 4 files changed, 20 insertions(+), 26 deletions(-)
>
> diff --git a/arch/arm/mach-tegra/clock.c b/arch/arm/mach-tegra/clock.c
> index f8d41ff..f27bdcc 100644
> --- a/arch/arm/mach-tegra/clock.c
> +++ b/arch/arm/mach-tegra/clock.c
> @@ -387,13 +387,15 @@ EXPORT_SYMBOL(tegra_clk_init_from_table);
>
> void tegra_periph_reset_deassert(struct clk *c)
> {
> - tegra2_periph_reset_deassert(c);
> + BUG_ON(!c->ops->reset);
> + c->ops->reset(c, false);
> }
> EXPORT_SYMBOL(tegra_periph_reset_deassert);
>
> void tegra_periph_reset_assert(struct clk *c)
> {
> - tegra2_periph_reset_assert(c);
> + BUG_ON(!c->ops->reset);
> + c->ops->reset(c, true);
> }
> EXPORT_SYMBOL(tegra_periph_reset_assert);
>
> @@ -403,9 +405,9 @@ void __init tegra_init_clock(void)
> }
>
> /*
> - * The SDMMC controllers have extra bits in the clock source register that
> - * adjust the delay between the clock and data to compenstate for delays
> - * on the PCB.
> + * The SDMMC controllers on tegra20 have extra bits in the clock source
> + * register that adjust the delay between the clock and data to compenstate
> + * for delays on the PCB.
> */
> void tegra_sdmmc_tap_delay(struct clk *c, int delay)
> {
>
> I created this wrapper around tegra2_sdmmc_tap_delay because I guessed the
> same requirement would be present in tegra3. If you don't expect this to be
> required on anything but tegra2, you could drop the wrapper and just have
> callers use tegra2_sdmmc_tap_delay.
>
I will ask around if this is needed for tegra30.
> diff --git a/arch/arm/mach-tegra/clock.h b/arch/arm/mach-tegra/clock.h
> index 688316a..135bb5f 100644
> --- a/arch/arm/mach-tegra/clock.h
> +++ b/arch/arm/mach-tegra/clock.h
> @@ -146,15 +146,15 @@ struct tegra_clk_init_table {
> };
>
> void tegra2_init_clocks(void);
> -void tegra2_periph_reset_deassert(struct clk *c);
> -void tegra2_periph_reset_assert(struct clk *c);
> void clk_init(struct clk *clk);
> struct clk *tegra_get_clock_by_name(const char *name);
> -unsigned long clk_measure_input_freq(void);
> int clk_reparent(struct clk *c, struct clk *parent);
> void tegra_clk_init_from_table(struct tegra_clk_init_table *table);
> unsigned long clk_get_rate_locked(struct clk *c);
> int clk_set_rate_locked(struct clk *c, unsigned long rate);
> +#ifdef CONFIG_ARCH_TEGRA_2x_SOC
> void tegra2_sdmmc_tap_delay(struct clk *c, int delay);
> -
> +#else
> +#define tegra2_sdmmc_tap_delay(c, d) do {} while(0);
>
> It's more standard to use an empty static inline function here.
>
clock.h is included in several board files. So making this a static inline
function results in a number of 'static function declared but not used'
warnings.
Cheers,
Peter.
_______________________________________________
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss
