Re: [PATCH] clk: tegra30: make tegra_clk_pll_params __ro_after_init
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> --- > drivers/clk/tegra/clk-tegra30.c | 16 > 1 file changed, 8 insertions(+), 8 deletions(-) Applied, thanks. Thierry signature.asc Description: PGP signature
Re: [PATCH] clk: tegra30: make tegra_clk_pll_params __ro_after_init
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 > --- > drivers/clk/tegra/clk-tegra30.c | 16 > 1 file changed, 8 insertions(+), 8 deletions(-) Applied, thanks. Thierry signature.asc Description: PGP signature
Re: [PATCH] clk: tegra30: make tegra_clk_pll_params __ro_after_init
Acked-By: Peter De SchrijverI 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 > --- > 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 = 200, > .input_max = 3100, > .cf_min = 100, > @@ -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 = 200, > .input_max = 3100, > .cf_min = 100, > @@ -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 = 200, > .input_max = 3100, > .cf_min = 100, > @@ -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 = 200, > .input_max = 4000, > .cf_min = 100, > @@ -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 = 200, > .input_max = 4000, > .cf_min = 100, > @@ -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 = 200, > .input_max = 4000, > .cf_min = 100, > @@ -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 = 200, > .input_max = 3100, > .cf_min = 100, > @@ -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 = 1200, > .input_max = 21600, > .cf_min = 1200, > -- > 1.9.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-clk" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] clk: tegra30: make tegra_clk_pll_params __ro_after_init
Acked-By: Peter De Schrijver 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 > --- > 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 = 200, > .input_max = 3100, > .cf_min = 100, > @@ -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 = 200, > .input_max = 3100, > .cf_min = 100, > @@ -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 = 200, > .input_max = 3100, > .cf_min = 100, > @@ -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 = 200, > .input_max = 4000, > .cf_min = 100, > @@ -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 = 200, > .input_max = 4000, > .cf_min = 100, > @@ -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 = 200, > .input_max = 4000, > .cf_min = 100, > @@ -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 = 200, > .input_max = 3100, > .cf_min = 100, > @@ -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 = 1200, > .input_max = 21600, > .cf_min = 1200, > -- > 1.9.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-clk" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH] clk: tegra30: make tegra_clk_pll_params __ro_after_init
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--- 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 = 200, .input_max = 3100, .cf_min = 100, @@ -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 = 200, .input_max = 3100, .cf_min = 100, @@ -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 = 200, .input_max = 3100, .cf_min = 100, @@ -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 = 200, .input_max = 4000, .cf_min = 100, @@ -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 = 200, .input_max = 4000, .cf_min = 100, @@ -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 = 200, .input_max = 4000, .cf_min = 100, @@ -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 = 200, .input_max = 3100, .cf_min = 100, @@ -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 = 1200, .input_max = 21600, .cf_min = 1200, -- 1.9.1
[PATCH] clk: tegra30: make tegra_clk_pll_params __ro_after_init
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 --- 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 = 200, .input_max = 3100, .cf_min = 100, @@ -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 = 200, .input_max = 3100, .cf_min = 100, @@ -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 = 200, .input_max = 3100, .cf_min = 100, @@ -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 = 200, .input_max = 4000, .cf_min = 100, @@ -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 = 200, .input_max = 4000, .cf_min = 100, @@ -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 = 200, .input_max = 4000, .cf_min = 100, @@ -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 = 200, .input_max = 3100, .cf_min = 100, @@ -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 = 1200, .input_max = 21600, .cf_min = 1200, -- 1.9.1