From: gregory.clement@free-electrons•com (Gregory CLEMENT)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH 4/9] clk: armada-370: Fix incorrect placement of __initconst
Date: Mon, 12 Aug 2013 12:30:42 +0200 [thread overview]
Message-ID: <5208B952.5030400@free-electrons.com> (raw)
In-Reply-To: <1376298847-21828-4-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/armada-370.c | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/clk/mvebu/armada-370.c b/drivers/clk/mvebu/armada-370.c
> index 079960e..fc777bd 100644
> --- a/drivers/clk/mvebu/armada-370.c
> +++ b/drivers/clk/mvebu/armada-370.c
> @@ -32,13 +32,13 @@
>
> enum { A370_CPU_TO_NBCLK, A370_CPU_TO_HCLK, A370_CPU_TO_DRAMCLK };
>
> -static const struct coreclk_ratio __initconst a370_coreclk_ratios[] = {
> +static const struct coreclk_ratio a370_coreclk_ratios[] __initconst = {
> { .id = A370_CPU_TO_NBCLK, .name = "nbclk" },
> { .id = A370_CPU_TO_HCLK, .name = "hclk" },
> { .id = A370_CPU_TO_DRAMCLK, .name = "dramclk" },
> };
>
> -static const u32 __initconst a370_tclk_freqs[] = {
> +static const u32 a370_tclk_freqs[] __initconst = {
> 16600000,
> 20000000,
> };
> @@ -52,7 +52,7 @@ static u32 __init a370_get_tclk_freq(void __iomem *sar)
> return a370_tclk_freqs[tclk_freq_select];
> }
>
> -static const u32 __initconst a370_cpu_freqs[] = {
> +static const u32 a370_cpu_freqs[] __initconst = {
> 400000000,
> 533000000,
> 667000000,
> @@ -78,7 +78,7 @@ static u32 __init a370_get_cpu_freq(void __iomem *sar)
> return cpu_freq;
> }
>
> -static const int __initconst a370_nbclk_ratios[32][2] = {
> +static const int a370_nbclk_ratios[32][2] __initconst = {
> {0, 1}, {1, 2}, {2, 2}, {2, 2},
> {1, 2}, {1, 2}, {1, 1}, {2, 3},
> {0, 1}, {1, 2}, {2, 4}, {0, 1},
> @@ -89,7 +89,7 @@ static const int __initconst a370_nbclk_ratios[32][2] = {
> {0, 1}, {0, 1}, {0, 1}, {0, 1},
> };
>
> -static const int __initconst a370_hclk_ratios[32][2] = {
> +static const int a370_hclk_ratios[32][2] __initconst = {
> {0, 1}, {1, 2}, {2, 6}, {2, 3},
> {1, 3}, {1, 4}, {1, 2}, {2, 6},
> {0, 1}, {1, 6}, {2, 10}, {0, 1},
> @@ -100,7 +100,7 @@ static const int __initconst a370_hclk_ratios[32][2] = {
> {0, 1}, {0, 1}, {0, 1}, {0, 1},
> };
>
> -static const int __initconst a370_dramclk_ratios[32][2] = {
> +static const int a370_dramclk_ratios[32][2] __initconst = {
> {0, 1}, {1, 2}, {2, 3}, {2, 3},
> {1, 3}, {1, 2}, {1, 2}, {2, 6},
> {0, 1}, {1, 3}, {2, 5}, {0, 1},
> @@ -152,7 +152,7 @@ CLK_OF_DECLARE(a370_core_clk, "marvell,armada-370-core-clock",
> * Clock Gating Control
> */
>
> -static const struct clk_gating_soc_desc __initconst a370_gating_desc[] = {
> +static const struct clk_gating_soc_desc a370_gating_desc[] __initconst = {
> { "audio", NULL, 0, 0 },
> { "pex0_en", NULL, 1, 0 },
> { "pex1_en", NULL, 2, 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:30 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 [this message]
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
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=5208B952.5030400@free-electrons.com \
--to=gregory.clement@free-electrons$(echo .)com \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox