| Message ID | 1506975128-16514-1-git-send-email-bhumirks@gmail.com (mailing list archive) |
|---|---|
| State | Awaiting Upstream, archived |
| Headers | show |
Acked-By: Peter De Schrijver <pdeschrijver@nvidia.com> I would assume this also applies to many PLL structs for other Tegra SoCs? Peter. On Tue, Oct 03, 2017 at 01:42:08AM +0530, Bhumika Goyal wrote: > These structures are only passed to the functions tegra_clk_register_pll, > tegra_clk_register_pll{e/u} or tegra_periph_clk_init during the init > phase. These functions modify the structures only during the init phase > and after that the structures are never modified. Therefore, make them > __ro_after_init. > > Signed-off-by: Bhumika Goyal <bhumirks@gmail.com> > --- > drivers/clk/tegra/clk-tegra30.c | 16 ++++++++-------- > 1 file changed, 8 insertions(+), 8 deletions(-) > > diff --git a/drivers/clk/tegra/clk-tegra30.c b/drivers/clk/tegra/clk-tegra30.c > index a2d163f..1ce7d76 100644 > --- a/drivers/clk/tegra/clk-tegra30.c > +++ b/drivers/clk/tegra/clk-tegra30.c > @@ -359,7 +359,7 @@ > }; > > /* PLL parameters */ > -static struct tegra_clk_pll_params pll_c_params = { > +static struct tegra_clk_pll_params pll_c_params __ro_after_init = { > .input_min = 2000000, > .input_max = 31000000, > .cf_min = 1000000, > @@ -388,7 +388,7 @@ > .override_divp_shift = 15, > }; > > -static struct tegra_clk_pll_params pll_m_params = { > +static struct tegra_clk_pll_params pll_m_params __ro_after_init = { > .input_min = 2000000, > .input_max = 31000000, > .cf_min = 1000000, > @@ -409,7 +409,7 @@ > TEGRA_PLL_HAS_LOCK_ENABLE | TEGRA_PLL_FIXED, > }; > > -static struct tegra_clk_pll_params pll_p_params = { > +static struct tegra_clk_pll_params pll_p_params __ro_after_init = { > .input_min = 2000000, > .input_max = 31000000, > .cf_min = 1000000, > @@ -444,7 +444,7 @@ > TEGRA_PLL_HAS_LOCK_ENABLE, > }; > > -static struct tegra_clk_pll_params pll_d_params = { > +static struct tegra_clk_pll_params pll_d_params __ro_after_init = { > .input_min = 2000000, > .input_max = 40000000, > .cf_min = 1000000, > @@ -461,7 +461,7 @@ > TEGRA_PLL_USE_LOCK | TEGRA_PLL_HAS_LOCK_ENABLE, > }; > > -static struct tegra_clk_pll_params pll_d2_params = { > +static struct tegra_clk_pll_params pll_d2_params __ro_after_init = { > .input_min = 2000000, > .input_max = 40000000, > .cf_min = 1000000, > @@ -478,7 +478,7 @@ > TEGRA_PLL_USE_LOCK | TEGRA_PLL_HAS_LOCK_ENABLE, > }; > > -static struct tegra_clk_pll_params pll_u_params = { > +static struct tegra_clk_pll_params pll_u_params __ro_after_init = { > .input_min = 2000000, > .input_max = 40000000, > .cf_min = 1000000, > @@ -496,7 +496,7 @@ > TEGRA_PLL_HAS_LOCK_ENABLE, > }; > > -static struct tegra_clk_pll_params pll_x_params = { > +static struct tegra_clk_pll_params pll_x_params __ro_after_init = { > .input_min = 2000000, > .input_max = 31000000, > .cf_min = 1000000, > @@ -513,7 +513,7 @@ > TEGRA_PLL_USE_LOCK | TEGRA_PLL_HAS_LOCK_ENABLE, > }; > > -static struct tegra_clk_pll_params pll_e_params = { > +static struct tegra_clk_pll_params pll_e_params __ro_after_init = { > .input_min = 12000000, > .input_max = 216000000, > .cf_min = 12000000, > -- > 1.9.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-clk" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-clk" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tue, Oct 03, 2017 at 01:42:08AM +0530, Bhumika Goyal wrote: > These structures are only passed to the functions tegra_clk_register_pll, > tegra_clk_register_pll{e/u} or tegra_periph_clk_init during the init > phase. These functions modify the structures only during the init phase > and after that the structures are never modified. Therefore, make them > __ro_after_init. > > Signed-off-by: Bhumika Goyal <bhumirks@gmail.com> > --- > drivers/clk/tegra/clk-tegra30.c | 16 ++++++++-------- > 1 file changed, 8 insertions(+), 8 deletions(-) Applied, thanks. Thierry
diff --git a/drivers/clk/tegra/clk-tegra30.c b/drivers/clk/tegra/clk-tegra30.c index a2d163f..1ce7d76 100644 --- a/drivers/clk/tegra/clk-tegra30.c +++ b/drivers/clk/tegra/clk-tegra30.c @@ -359,7 +359,7 @@ }; /* PLL parameters */ -static struct tegra_clk_pll_params pll_c_params = { +static struct tegra_clk_pll_params pll_c_params __ro_after_init = { .input_min = 2000000, .input_max = 31000000, .cf_min = 1000000, @@ -388,7 +388,7 @@ .override_divp_shift = 15, }; -static struct tegra_clk_pll_params pll_m_params = { +static struct tegra_clk_pll_params pll_m_params __ro_after_init = { .input_min = 2000000, .input_max = 31000000, .cf_min = 1000000, @@ -409,7 +409,7 @@ TEGRA_PLL_HAS_LOCK_ENABLE | TEGRA_PLL_FIXED, }; -static struct tegra_clk_pll_params pll_p_params = { +static struct tegra_clk_pll_params pll_p_params __ro_after_init = { .input_min = 2000000, .input_max = 31000000, .cf_min = 1000000, @@ -444,7 +444,7 @@ TEGRA_PLL_HAS_LOCK_ENABLE, }; -static struct tegra_clk_pll_params pll_d_params = { +static struct tegra_clk_pll_params pll_d_params __ro_after_init = { .input_min = 2000000, .input_max = 40000000, .cf_min = 1000000, @@ -461,7 +461,7 @@ TEGRA_PLL_USE_LOCK | TEGRA_PLL_HAS_LOCK_ENABLE, }; -static struct tegra_clk_pll_params pll_d2_params = { +static struct tegra_clk_pll_params pll_d2_params __ro_after_init = { .input_min = 2000000, .input_max = 40000000, .cf_min = 1000000, @@ -478,7 +478,7 @@ TEGRA_PLL_USE_LOCK | TEGRA_PLL_HAS_LOCK_ENABLE, }; -static struct tegra_clk_pll_params pll_u_params = { +static struct tegra_clk_pll_params pll_u_params __ro_after_init = { .input_min = 2000000, .input_max = 40000000, .cf_min = 1000000, @@ -496,7 +496,7 @@ TEGRA_PLL_HAS_LOCK_ENABLE, }; -static struct tegra_clk_pll_params pll_x_params = { +static struct tegra_clk_pll_params pll_x_params __ro_after_init = { .input_min = 2000000, .input_max = 31000000, .cf_min = 1000000, @@ -513,7 +513,7 @@ TEGRA_PLL_USE_LOCK | TEGRA_PLL_HAS_LOCK_ENABLE, }; -static struct tegra_clk_pll_params pll_e_params = { +static struct tegra_clk_pll_params pll_e_params __ro_after_init = { .input_min = 12000000, .input_max = 216000000, .cf_min = 12000000,
These structures are only passed to the functions tegra_clk_register_pll, tegra_clk_register_pll{e/u} or tegra_periph_clk_init during the init phase. These functions modify the structures only during the init phase and after that the structures are never modified. Therefore, make them __ro_after_init. Signed-off-by: Bhumika Goyal <bhumirks@gmail.com> --- drivers/clk/tegra/clk-tegra30.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-)