Skip to contentSkip to conversation

ARM: shmobile: koelsch-reference: Annotate clk_names as __initconst

PATCH New Simon Horman 4 participants 4 messages Mar 17, 2014 – Mar 18, 2014

Subsystems (4)
Arm (32-bit) Arm sub-architectures SuperH Arm/RISC-V/Renesas architecture
lore
Share & tools
lore

Simon Horman

author
#0 lore raw

PATCH 1/1 ARM: shmobile: koelsch-reference: Annotate clk_names as __initconst

Cc Geert Uytterhoeven <[email protected]>
Signed-off-by Simon Horman <[email protected]>
---
 arch/arm/mach-shmobile/board-koelsch-reference.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Based on renesas-devel-v3.14-rc6-20140314
Requires "ARM: shmobile: koelsch-reference: Work around core clock issues"

arch/arm/mach-shmobile/board-koelsch-reference.c +1 -1
index a39114a..63117d52 100644
@@ -110,7 +110,7 @@ static const struct clk_name clk_names[] __initconst = {
 /*
  * This is a really crude hack to work around core platform clock issues
  */
-static const struct clk_name clk_enables[] = {
+static const struct clk_name clk_enables[] __initconst = {
 	{ "ether", NULL, "ee700000.ethernet" },
 	{ "i2c2", NULL, "e6530000.i2c" },
 	{ "msiof0", NULL, "e6e20000.spi" },
Signature · 1 line
1.8.5.2
#1 lore raw

PATCH 1/1 ARM: shmobile: koelsch-reference: Annotate clk_names as __initconst

Quoted 2 lines · Simon Horman
Cc Geert Uytterhoeven <[email protected]>
Signed-off-by Simon Horman <[email protected]>
Acked-by Geert Uytterhoeven <[email protected]>

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds

#2 lore raw

PATCH 1/1 ARM: shmobile: koelsch-reference: Annotate clk_names as __initconst

In reply to Simon Horman

Hello.

On 17-03-2014 6:19, Simon Horman wrote:

Apparently you meant clk_enables in the subject, not clk_names.

Quoted 2 lines
Cc Geert Uytterhoeven <[email protected]>
Signed-off-by Simon Horman <[email protected]>

[...]

arch/arm/mach-shmobile/board-koelsch-reference.c static const struct clk_name clk_names[] __initconst = { · line 113

Quoted 10 lines
arch/arm/mach-shmobile/board-koelsch-reference.c +1 -1
index a39114a..63117d52 100644
@@ -110,7 +110,7 @@ static const struct clk_name clk_names[] __initconst = {
  /*
   * This is a really crude hack to work around core platform clock issues
   */
-static const struct clk_name clk_enables[] = {
+static const struct clk_name clk_enables[] __initconst = {

WBR, Sergei

#3 lore raw

PATCH 1/1 ARM: shmobile: koelsch-reference: Annotate clk_names as __initconst

Quoted 5 lines · Sergei Shtylyov

Hello.

On 17-03-2014 6:19, Simon Horman wrote:

Apparently you meant clk_enables in the subject, not clk_names.

Thanks, I have queued up the following:

From: Simon Horman <[email protected]>

ARM: shmobile: koelsch-reference: Annotate clk_enables as __initconst

Signed-off-by Simon Horman <[email protected]>
Acked-by Geert Uytterhoeven <[email protected]>
---
 arch/arm/mach-shmobile/board-koelsch-reference.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
arch/arm/mach-shmobile/board-koelsch-reference.c +1 -1
index a39114a..63117d52 100644
@@ -110,7 +110,7 @@ static const struct clk_name clk_names[] __initconst = {
 /*
  * This is a really crude hack to work around core platform clock issues
  */
-static const struct clk_name clk_enables[] = {
+static const struct clk_name clk_enables[] __initconst = {
 	{ "ether", NULL, "ee700000.ethernet" },
 	{ "i2c2", NULL, "e6530000.i2c" },
 	{ "msiof0", NULL, "e6e20000.spi" },
Signature · 1 line
1.8.5.2