Re: [PATCH V2] clk: tegra: implement a reset driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Dec 03, 2013 at 09:28:49PM +0100, Stephen Warren wrote:
> From: Stephen Warren <swarren@xxxxxxxxxx>
> 
> The Tegra CAR module implements both a clock and reset controller. So
> far, the driver exposes the clock feature via the common clock API and
> the reset feature using a custom API. This patch adds an implementation
> of the common reset framework API (include/linux/reset*.h). The legacy
> reset implementation will be removed once all drivers have been
> converted.
> 
> Cc: pdeschrijver@xxxxxxxxxx
> Cc: linux-tegra@xxxxxxxxxxxxxxx
> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> Cc: Mike Turquette <mturquette@xxxxxxxxxx>
> Signed-off-by: Stephen Warren <swarren@xxxxxxxxxx>
> Reviewed-by: Thierry Reding <treding@xxxxxxxxxx>

Acked-By: Peter De Schrijver <pdeschrijver@xxxxxxxxxx>

> ---
> v2: Move the added lines in the file, to avoid the need for function
>     prototypes.
> 
> Peter, Mike,
> 
> This patch is part of a series with strong internal depdendencies. I'm
> looking for an ack so that I can take the entire series through the Tegra
> and arm-soc trees. The series will be part of a stable branch that can be
> merged into other subsystems if needed to avoid/resolve dependencies.
> ---
>  drivers/clk/tegra/clk-tegra114.c |  3 ++-
>  drivers/clk/tegra/clk-tegra124.c |  2 +-
>  drivers/clk/tegra/clk-tegra20.c  |  3 ++-
>  drivers/clk/tegra/clk-tegra30.c  |  3 ++-
>  drivers/clk/tegra/clk.c          | 50 +++++++++++++++++++++++++++++++++++++++-
>  drivers/clk/tegra/clk.h          |  2 +-
>  6 files changed, 57 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/clk/tegra/clk-tegra114.c b/drivers/clk/tegra/clk-tegra114.c
> index 70bfefebcca5..84d9003ae816 100644
> --- a/drivers/clk/tegra/clk-tegra114.c
> +++ b/drivers/clk/tegra/clk-tegra114.c
> @@ -1457,7 +1457,8 @@ static void __init tegra114_clock_init(struct device_node *np)
>  		return;
>  	}
>  
> -	clks = tegra_clk_init(TEGRA114_CLK_CLK_MAX, TEGRA114_CLK_PERIPH_BANKS);
> +	clks = tegra_clk_init(clk_base, TEGRA114_CLK_CLK_MAX,
> +				TEGRA114_CLK_PERIPH_BANKS);
>  	if (!clks)
>  		return;
>  
> diff --git a/drivers/clk/tegra/clk-tegra124.c b/drivers/clk/tegra/clk-tegra124.c
> index 623fbf35ee08..6f8b1d99e5b9 100644
> --- a/drivers/clk/tegra/clk-tegra124.c
> +++ b/drivers/clk/tegra/clk-tegra124.c
> @@ -1402,7 +1402,7 @@ static void __init tegra124_clock_init(struct device_node *np)
>  		return;
>  	}
>  
> -	clks = tegra_clk_init(TEGRA124_CLK_CLK_MAX, 6);
> +	clks = tegra_clk_init(clk_base, TEGRA124_CLK_CLK_MAX, 6);
>  	if (!clks)
>  		return;
>  
> diff --git a/drivers/clk/tegra/clk-tegra20.c b/drivers/clk/tegra/clk-tegra20.c
> index be5bdbab78a6..d438a089354c 100644
> --- a/drivers/clk/tegra/clk-tegra20.c
> +++ b/drivers/clk/tegra/clk-tegra20.c
> @@ -1108,7 +1108,8 @@ static void __init tegra20_clock_init(struct device_node *np)
>  		BUG();
>  	}
>  
> -	clks = tegra_clk_init(TEGRA20_CLK_CLK_MAX, TEGRA20_CLK_PERIPH_BANKS);
> +	clks = tegra_clk_init(clk_base, TEGRA20_CLK_CLK_MAX,
> +				TEGRA20_CLK_PERIPH_BANKS);
>  	if (!clks)
>  		return;
>  
> diff --git a/drivers/clk/tegra/clk-tegra30.c b/drivers/clk/tegra/clk-tegra30.c
> index 92c46f1fb5d1..49465529bb09 100644
> --- a/drivers/clk/tegra/clk-tegra30.c
> +++ b/drivers/clk/tegra/clk-tegra30.c
> @@ -1429,7 +1429,8 @@ static void __init tegra30_clock_init(struct device_node *np)
>  		BUG();
>  	}
>  
> -	clks = tegra_clk_init(TEGRA30_CLK_CLK_MAX, TEGRA30_CLK_PERIPH_BANKS);
> +	clks = tegra_clk_init(clk_base, TEGRA30_CLK_CLK_MAX,
> +				TEGRA30_CLK_PERIPH_BANKS);
>  	if (!clks)
>  		return;
>  
> diff --git a/drivers/clk/tegra/clk.c b/drivers/clk/tegra/clk.c
> index a12a5f5107ec..c0a7d7723510 100644
> --- a/drivers/clk/tegra/clk.c
> +++ b/drivers/clk/tegra/clk.c
> @@ -18,6 +18,8 @@
>  #include <linux/clk-provider.h>
>  #include <linux/of.h>
>  #include <linux/clk/tegra.h>
> +#include <linux/reset-controller.h>
> +#include <linux/tegra-soc.h>
>  
>  #include "clk.h"
>  
> @@ -121,6 +123,35 @@ static struct tegra_clk_periph_regs periph_regs[] = {
>  	},
>  };
>  
> +static void __iomem *clk_base;
> +
> +static int tegra_clk_rst_assert(struct reset_controller_dev *rcdev,
> +		unsigned long id)
> +{
> +	/*
> +	 * If peripheral is on the APB bus then we must read the APB bus to
> +	 * flush the write operation in apb bus. This will avoid peripheral
> +	 * access after disabling clock. Since the reset driver has no
> +	 * knowledge of which reset IDs represent which devices, simply do
> +	 * this all the time.
> +	 */
> +	tegra_read_chipid();
> +
> +	writel_relaxed(BIT(id % 32),
> +			clk_base + periph_regs[id / 32].rst_set_reg);
> +
> +	return 0;
> +}
> +
> +static int tegra_clk_rst_deassert(struct reset_controller_dev *rcdev,
> +		unsigned long id)
> +{
> +	writel_relaxed(BIT(id % 32),
> +			clk_base + periph_regs[id / 32].rst_clr_reg);
> +
> +	return 0;
> +}
> +
>  struct tegra_clk_periph_regs *get_reg_bank(int clkid)
>  {
>  	int reg_bank = clkid / 32;
> @@ -133,8 +164,10 @@ struct tegra_clk_periph_regs *get_reg_bank(int clkid)
>  	}
>  }
>  
> -struct clk ** __init tegra_clk_init(int num, int banks)
> +struct clk ** __init tegra_clk_init(void __iomem *regs, int num, int banks)
>  {
> +	clk_base = regs;
> +
>  	if (WARN_ON(banks > ARRAY_SIZE(periph_regs)))
>  		return NULL;
>  
> @@ -203,6 +236,17 @@ void __init tegra_init_from_table(struct tegra_clk_init_table *tbl,
>  	}
>  }
>  
> +static struct reset_control_ops rst_ops = {
> +	.assert = tegra_clk_rst_assert,
> +	.deassert = tegra_clk_rst_deassert,
> +};
> +
> +static struct reset_controller_dev rst_ctlr = {
> +	.ops = &rst_ops,
> +	.owner = THIS_MODULE,
> +	.of_reset_n_cells = 1,
> +};
> +
>  void __init tegra_add_of_provider(struct device_node *np)
>  {
>  	int i;
> @@ -220,6 +264,10 @@ void __init tegra_add_of_provider(struct device_node *np)
>  	clk_data.clks = clks;
>  	clk_data.clk_num = clk_num;
>  	of_clk_add_provider(np, of_clk_src_onecell_get, &clk_data);
> +
> +	rst_ctlr.of_node = np;
> +	rst_ctlr.nr_resets = clk_num * 32;
> +	reset_controller_register(&rst_ctlr);
>  }
>  
>  void __init tegra_register_devclks(struct tegra_devclk *dev_clks, int num)
> diff --git a/drivers/clk/tegra/clk.h b/drivers/clk/tegra/clk.h
> index 7f110acfe2a1..39f24959daf7 100644
> --- a/drivers/clk/tegra/clk.h
> +++ b/drivers/clk/tegra/clk.h
> @@ -594,7 +594,7 @@ void tegra_init_dup_clks(struct tegra_clk_duplicate *dup_list,
>  		struct clk *clks[], int clk_max);
>  
>  struct tegra_clk_periph_regs *get_reg_bank(int clkid);
> -struct clk **tegra_clk_init(int num, int periph_banks);
> +struct clk **tegra_clk_init(void __iomem *clk_base, int num, int periph_banks);
>  
>  struct clk **tegra_lookup_dt_id(int clk_id, struct tegra_clk *tegra_clk);
>  
> -- 
> 1.8.1.5
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux