Skip to contentSkip to conversation

clk: tegra114: Fix incorrect placement of __initdata

PATCH New Sachin Kamat 3 participants 6 messages Aug 8, 2013 – Aug 9, 2013

Subsystems (3)
Common clk Tegra architecture support Tegra clock driver
lore
Share & tools
lore

Sachin Kamat

author
#0 lore raw

PATCH 1/3 clk: tegra114: Fix incorrect placement of __initdata

__initdata should be placed between the variable name and equal sign for the variable to be placed in the intended section.

Signed-off-by Sachin Kamat <[email protected]>
---
 drivers/clk/tegra/clk-tegra114.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/clk/tegra/clk-tegra114.c +1 -1
index f74ed19..66fa1d6 100644
@@ -2182,7 +2182,7 @@ static const struct of_device_id pmc_match[] __initconst = {
  * dfll_soc/dfll_ref apparently must be kept enabled, otherwise I2C5
  * breaks
  */
-static __initdata struct tegra_clk_init_table init_table[] = {
+static struct tegra_clk_init_table init_table[] __initdata = {
 	{uarta, pll_p, 408000000, 0},
 	{uartb, pll_p, 408000000, 0},
 	{uartc, pll_p, 408000000, 0},
Signature · 1 line
1.7.9.5
author
#1 lore raw

PATCH 2/3 clk: tegra20: Fix incorrect placement of __initdata

__initdata should be placed between the variable name and equal sign for the variable to be placed in the intended section.

Signed-off-by Sachin Kamat <[email protected]>
---
 drivers/clk/tegra/clk-tegra20.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/clk/tegra/clk-tegra20.c +1 -1
index 759ca47..ebe94ce 100644
@@ -1223,7 +1223,7 @@ static struct tegra_cpu_car_ops tegra20_cpu_car_ops = {
 #endif
 };
 
-static __initdata struct tegra_clk_init_table init_table[] = {
+static struct tegra_clk_init_table init_table[] __initdata = {
 	{pll_p, clk_max, 216000000, 1},
 	{pll_p_out1, clk_max, 28800000, 1},
 	{pll_p_out2, clk_max, 48000000, 1},
Signature · 1 line
1.7.9.5
author
#2 lore raw

PATCH 3/3 clk: tegra30: Fix incorrect placement of __initdata

In reply to Sachin Kamat

__initdata should be placed between the variable name and equal sign for the variable to be placed in the intended section.

Signed-off-by Sachin Kamat <[email protected]>
---
 drivers/clk/tegra/clk-tegra30.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/clk/tegra/clk-tegra30.c +1 -1
index e2c6ca0..ab35040 100644
@@ -1901,7 +1901,7 @@ static struct tegra_cpu_car_ops tegra30_cpu_car_ops = {
 #endif
 };
 
-static __initdata struct tegra_clk_init_table init_table[] = {
+static struct tegra_clk_init_table init_table[] __initdata = {
 	{uarta, pll_p, 408000000, 0},
 	{uartb, pll_p, 408000000, 0},
 	{uartc, pll_p, 408000000, 0},
Signature · 1 line
1.7.9.5
#4 lore raw

PATCH 1/3 clk: tegra114: Fix incorrect placement of __initdata

Quoting Stephen Warren (2013-08-08 11:16:55)

Quoted 11 lines · Sachin Kamat
Quoted 2 lines

__initdata should be placed between the variable name and equal sign for the variable to be placed in the intended section.

The series,

Acked-by Stephen Warren <[email protected]>

Patch 1,

Tested-by Stephen Warren <[email protected]>

Mike, I assume you'll take these through the clock tree.

Done.

Regards,
Mike