From: Mike Turquette <mturquette@linaro.org>
To: Bintian Wang <bintian.wang@huawei.com>,
haojian.zhuang@linaro.org, zhangfei.gao@linaro.org,
arnd@arndb.de, jh80.chung@samsung.com, li.xin@linaro.org,
linux-kernel@vger.kernel.org, huxinwei@huawei.com
Cc: bintian.wang@huawei.com
Subject: Re: [PATCH] clk: hi3620: Move const initdata into correct code section
Date: Wed, 19 Nov 2014 16:57:24 -0800 [thread overview]
Message-ID: <20141120005724.25314.39047@quantum> (raw)
In-Reply-To: <1415624311-18980-1-git-send-email-bintian.wang@huawei.com>
Quoting Bintian Wang (2014-11-10 04:58:31)
> Use __initconst instead of __initdata for constant init data.
>
> Signed-off-by: Bintian Wang <bintian.wang@huawei.com>
Applied.
Thanks,
Mike
> ---
> drivers/clk/hisilicon/clk-hi3620.c | 70 +++++++++++++++++++-------------------
> 1 file changed, 35 insertions(+), 35 deletions(-)
>
> diff --git a/drivers/clk/hisilicon/clk-hi3620.c b/drivers/clk/hisilicon/clk-hi3620.c
> index 339945d..640ea33 100644
> --- a/drivers/clk/hisilicon/clk-hi3620.c
> +++ b/drivers/clk/hisilicon/clk-hi3620.c
> @@ -38,44 +38,44 @@
> #include "clk.h"
>
> /* clock parent list */
> -static const char *timer0_mux_p[] __initdata = { "osc32k", "timerclk01", };
> -static const char *timer1_mux_p[] __initdata = { "osc32k", "timerclk01", };
> -static const char *timer2_mux_p[] __initdata = { "osc32k", "timerclk23", };
> -static const char *timer3_mux_p[] __initdata = { "osc32k", "timerclk23", };
> -static const char *timer4_mux_p[] __initdata = { "osc32k", "timerclk45", };
> -static const char *timer5_mux_p[] __initdata = { "osc32k", "timerclk45", };
> -static const char *timer6_mux_p[] __initdata = { "osc32k", "timerclk67", };
> -static const char *timer7_mux_p[] __initdata = { "osc32k", "timerclk67", };
> -static const char *timer8_mux_p[] __initdata = { "osc32k", "timerclk89", };
> -static const char *timer9_mux_p[] __initdata = { "osc32k", "timerclk89", };
> -static const char *uart0_mux_p[] __initdata = { "osc26m", "pclk", };
> -static const char *uart1_mux_p[] __initdata = { "osc26m", "pclk", };
> -static const char *uart2_mux_p[] __initdata = { "osc26m", "pclk", };
> -static const char *uart3_mux_p[] __initdata = { "osc26m", "pclk", };
> -static const char *uart4_mux_p[] __initdata = { "osc26m", "pclk", };
> -static const char *spi0_mux_p[] __initdata = { "osc26m", "rclk_cfgaxi", };
> -static const char *spi1_mux_p[] __initdata = { "osc26m", "rclk_cfgaxi", };
> -static const char *spi2_mux_p[] __initdata = { "osc26m", "rclk_cfgaxi", };
> +static const char *timer0_mux_p[] __initconst = { "osc32k", "timerclk01", };
> +static const char *timer1_mux_p[] __initconst = { "osc32k", "timerclk01", };
> +static const char *timer2_mux_p[] __initconst = { "osc32k", "timerclk23", };
> +static const char *timer3_mux_p[] __initconst = { "osc32k", "timerclk23", };
> +static const char *timer4_mux_p[] __initconst = { "osc32k", "timerclk45", };
> +static const char *timer5_mux_p[] __initconst = { "osc32k", "timerclk45", };
> +static const char *timer6_mux_p[] __initconst = { "osc32k", "timerclk67", };
> +static const char *timer7_mux_p[] __initconst = { "osc32k", "timerclk67", };
> +static const char *timer8_mux_p[] __initconst = { "osc32k", "timerclk89", };
> +static const char *timer9_mux_p[] __initconst = { "osc32k", "timerclk89", };
> +static const char *uart0_mux_p[] __initconst = { "osc26m", "pclk", };
> +static const char *uart1_mux_p[] __initconst = { "osc26m", "pclk", };
> +static const char *uart2_mux_p[] __initconst = { "osc26m", "pclk", };
> +static const char *uart3_mux_p[] __initconst = { "osc26m", "pclk", };
> +static const char *uart4_mux_p[] __initconst = { "osc26m", "pclk", };
> +static const char *spi0_mux_p[] __initconst = { "osc26m", "rclk_cfgaxi", };
> +static const char *spi1_mux_p[] __initconst = { "osc26m", "rclk_cfgaxi", };
> +static const char *spi2_mux_p[] __initconst = { "osc26m", "rclk_cfgaxi", };
> /* share axi parent */
> -static const char *saxi_mux_p[] __initdata = { "armpll3", "armpll2", };
> -static const char *pwm0_mux_p[] __initdata = { "osc32k", "osc26m", };
> -static const char *pwm1_mux_p[] __initdata = { "osc32k", "osc26m", };
> -static const char *sd_mux_p[] __initdata = { "armpll2", "armpll3", };
> -static const char *mmc1_mux_p[] __initdata = { "armpll2", "armpll3", };
> -static const char *mmc1_mux2_p[] __initdata = { "osc26m", "mmc1_div", };
> -static const char *g2d_mux_p[] __initdata = { "armpll2", "armpll3", };
> -static const char *venc_mux_p[] __initdata = { "armpll2", "armpll3", };
> -static const char *vdec_mux_p[] __initdata = { "armpll2", "armpll3", };
> -static const char *vpp_mux_p[] __initdata = { "armpll2", "armpll3", };
> -static const char *edc0_mux_p[] __initdata = { "armpll2", "armpll3", };
> -static const char *ldi0_mux_p[] __initdata = { "armpll2", "armpll4",
> +static const char *saxi_mux_p[] __initconst = { "armpll3", "armpll2", };
> +static const char *pwm0_mux_p[] __initconst = { "osc32k", "osc26m", };
> +static const char *pwm1_mux_p[] __initconst = { "osc32k", "osc26m", };
> +static const char *sd_mux_p[] __initconst = { "armpll2", "armpll3", };
> +static const char *mmc1_mux_p[] __initconst = { "armpll2", "armpll3", };
> +static const char *mmc1_mux2_p[] __initconst = { "osc26m", "mmc1_div", };
> +static const char *g2d_mux_p[] __initconst = { "armpll2", "armpll3", };
> +static const char *venc_mux_p[] __initconst = { "armpll2", "armpll3", };
> +static const char *vdec_mux_p[] __initconst = { "armpll2", "armpll3", };
> +static const char *vpp_mux_p[] __initconst = { "armpll2", "armpll3", };
> +static const char *edc0_mux_p[] __initconst = { "armpll2", "armpll3", };
> +static const char *ldi0_mux_p[] __initconst = { "armpll2", "armpll4",
> "armpll3", "armpll5", };
> -static const char *edc1_mux_p[] __initdata = { "armpll2", "armpll3", };
> -static const char *ldi1_mux_p[] __initdata = { "armpll2", "armpll4",
> +static const char *edc1_mux_p[] __initconst = { "armpll2", "armpll3", };
> +static const char *ldi1_mux_p[] __initconst = { "armpll2", "armpll4",
> "armpll3", "armpll5", };
> -static const char *rclk_hsic_p[] __initdata = { "armpll3", "armpll2", };
> -static const char *mmc2_mux_p[] __initdata = { "armpll2", "armpll3", };
> -static const char *mmc3_mux_p[] __initdata = { "armpll2", "armpll3", };
> +static const char *rclk_hsic_p[] __initconst = { "armpll3", "armpll2", };
> +static const char *mmc2_mux_p[] __initconst = { "armpll2", "armpll3", };
> +static const char *mmc3_mux_p[] __initconst = { "armpll2", "armpll3", };
>
>
> /* fixed rate clocks */
> --
> 1.9.1
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
prev parent reply other threads:[~2014-11-20 0:57 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-10 12:58 [PATCH] clk: hi3620: Move const initdata into correct code section Bintian Wang
2014-11-11 13:55 ` Haojian Zhuang
2014-11-20 0:57 ` Mike Turquette [this message]
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=20141120005724.25314.39047@quantum \
--to=mturquette@linaro.org \
--cc=arnd@arndb.de \
--cc=bintian.wang@huawei.com \
--cc=haojian.zhuang@linaro.org \
--cc=huxinwei@huawei.com \
--cc=jh80.chung@samsung.com \
--cc=li.xin@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=zhangfei.gao@linaro.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox