From: gregory.clement@free-electrons.com (Gregory CLEMENT)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 7/9] clk: kirkwood: Fix incorrect placement of __initconst
Date: Mon, 12 Aug 2013 12:32:00 +0200 [thread overview]
Message-ID: <5208B9A0.3040406@free-electrons.com> (raw)
In-Reply-To: <1376298847-21828-7-git-send-email-sachin.kamat@linaro.org>
On 12/08/2013 11:14, Sachin Kamat wrote:
> __initconst should be placed between the variable name and equal
> sign for the variable to be placed in the intended section.
>
Acked-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> Cc: Gregory Clement <gregory.clement@free-electrons.com>
> ---
> drivers/clk/mvebu/kirkwood.c | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/clk/mvebu/kirkwood.c b/drivers/clk/mvebu/kirkwood.c
> index 71d2461..2636a55 100644
> --- a/drivers/clk/mvebu/kirkwood.c
> +++ b/drivers/clk/mvebu/kirkwood.c
> @@ -78,7 +78,7 @@
>
> enum { KIRKWOOD_CPU_TO_L2, KIRKWOOD_CPU_TO_DDR };
>
> -static const struct coreclk_ratio __initconst kirkwood_coreclk_ratios[] = {
> +static const struct coreclk_ratio kirkwood_coreclk_ratios[] __initconst = {
> { .id = KIRKWOOD_CPU_TO_L2, .name = "l2clk", },
> { .id = KIRKWOOD_CPU_TO_DDR, .name = "ddrclk", }
> };
> @@ -90,7 +90,7 @@ static u32 __init kirkwood_get_tclk_freq(void __iomem *sar)
> return (opt) ? 166666667 : 200000000;
> }
>
> -static const u32 __initconst kirkwood_cpu_freqs[] = {
> +static const u32 kirkwood_cpu_freqs[] __initconst = {
> 0, 0, 0, 0,
> 600000000,
> 0,
> @@ -111,12 +111,12 @@ static u32 __init kirkwood_get_cpu_freq(void __iomem *sar)
> return kirkwood_cpu_freqs[opt];
> }
>
> -static const int __initconst kirkwood_cpu_l2_ratios[8][2] = {
> +static const int kirkwood_cpu_l2_ratios[8][2] __initconst = {
> { 0, 1 }, { 1, 2 }, { 0, 1 }, { 1, 3 },
> { 0, 1 }, { 1, 4 }, { 0, 1 }, { 0, 1 }
> };
>
> -static const int __initconst kirkwood_cpu_ddr_ratios[16][2] = {
> +static const int kirkwood_cpu_ddr_ratios[16][2] __initconst = {
> { 0, 1 }, { 0, 1 }, { 1, 2 }, { 0, 1 },
> { 1, 3 }, { 0, 1 }, { 1, 4 }, { 2, 9 },
> { 1, 5 }, { 1, 6 }, { 0, 1 }, { 0, 1 },
> @@ -145,7 +145,7 @@ static void __init kirkwood_get_clk_ratio(
> }
> }
>
> -static const u32 __initconst mv88f6180_cpu_freqs[] = {
> +static const u32 mv88f6180_cpu_freqs[] __initconst = {
> 0, 0, 0, 0, 0,
> 600000000,
> 800000000,
> @@ -158,7 +158,7 @@ static u32 __init mv88f6180_get_cpu_freq(void __iomem *sar)
> return mv88f6180_cpu_freqs[opt];
> }
>
> -static const int __initconst mv88f6180_cpu_ddr_ratios[8][2] = {
> +static const int mv88f6180_cpu_ddr_ratios[8][2] __initconst = {
> { 0, 1 }, { 0, 1 }, { 0, 1 }, { 0, 1 },
> { 0, 1 }, { 1, 3 }, { 1, 4 }, { 1, 5 }
> };
> @@ -219,7 +219,7 @@ CLK_OF_DECLARE(mv88f6180_core_clk, "marvell,mv88f6180-core-clock",
> * Clock Gating Control
> */
>
> -static const struct clk_gating_soc_desc __initconst kirkwood_gating_desc[] = {
> +static const struct clk_gating_soc_desc kirkwood_gating_desc[] __initconst = {
> { "ge0", NULL, 0, 0 },
> { "pex0", NULL, 2, 0 },
> { "usb0", NULL, 3, 0 },
>
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
next prev parent reply other threads:[~2013-08-12 10:32 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-12 9:13 [PATCH 1/9] clk: bcm2835: Fix incorrect placement of __initconst Sachin Kamat
2013-08-12 9:14 ` [PATCH 2/9] clk: nomadik: " Sachin Kamat
2013-08-13 6:14 ` Linus Walleij
2013-08-12 9:14 ` [PATCH 3/9] clk: u300: " Sachin Kamat
2013-08-16 15:10 ` Linus Walleij
2013-08-12 9:14 ` [PATCH 4/9] clk: armada-370: " Sachin Kamat
2013-08-12 10:30 ` Gregory CLEMENT
2013-08-12 9:14 ` [PATCH 5/9] clk: armada-xp: " Sachin Kamat
2013-08-12 10:31 ` Gregory CLEMENT
2013-08-12 9:14 ` [PATCH 6/9] clk: dove: " Sachin Kamat
2013-08-12 10:31 ` Gregory CLEMENT
2013-08-12 9:14 ` [PATCH 7/9] clk: kirkwood: " Sachin Kamat
2013-08-12 10:32 ` Gregory CLEMENT [this message]
2013-08-12 9:14 ` [PATCH 8/9] clk: sunxi: " Sachin Kamat
2013-08-28 1:44 ` Mike Turquette
2013-08-12 9:14 ` [PATCH 9/9] clk: s3c64xx: Fix incorrect placement of __initdata Sachin Kamat
2013-08-12 15:48 ` [PATCH 1/9] clk: bcm2835: Fix incorrect placement of __initconst Stephen Warren
2013-08-19 3:38 ` Sachin Kamat
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5208B9A0.3040406@free-electrons.com \
--to=gregory.clement@free-electrons.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.