linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: Exynos4: Turn off clocks for NAND, OneNAND and TSI controllers
@ 2012-08-24 14:52 Chander Kashyap
  2012-08-28  8:15 ` Thomas Abraham
  0 siblings, 1 reply; 3+ messages in thread
From: Chander Kashyap @ 2012-08-24 14:52 UTC (permalink / raw)
  To: linux-arm-kernel

The clocks for NAND, OneNAND and Transport Stream Interface(TSI)
controllers could be either enabled or disabled at boot. To ensure
that these are turned off until used, add them to the list of clocks
to be turned off during boot.

Signed-off-by: Chander Kashyap <chander.kashyap@linaro.org>
---
 arch/arm/mach-exynos/clock-exynos4.c |   12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/arch/arm/mach-exynos/clock-exynos4.c b/arch/arm/mach-exynos/clock-exynos4.c
index 2f51293..7cc5491 100644
--- a/arch/arm/mach-exynos/clock-exynos4.c
+++ b/arch/arm/mach-exynos/clock-exynos4.c
@@ -501,6 +501,10 @@ static struct clk exynos4_init_clocks_off[] = {
 		.enable		= exynos4_clk_ip_cam_ctrl,
 		.ctrlbit	= (1 << 3),
 	}, {
+		.name		= "tsi",
+		.enable		= exynos4_clk_ip_fsys_ctrl,
+		.ctrlbit	= (1 << 4),
+	}, {
 		.name		= "hsmmc",
 		.devname	= "exynos4-sdhci.0",
 		.parent		= &exynos4_clk_aclk_133.clk,
@@ -530,6 +534,14 @@ static struct clk exynos4_init_clocks_off[] = {
 		.enable		= exynos4_clk_ip_fsys_ctrl,
 		.ctrlbit	= (1 << 9),
 	}, {
+		.name		= "onenand",
+		.enable		= exynos4_clk_ip_fsys_ctrl,
+		.ctrlbit	= (1 << 15),
+	}, {
+		.name		= "nfcon",
+		.enable		= exynos4_clk_ip_fsys_ctrl,
+		.ctrlbit	= (1 << 16),
+	}, {
 		.name		= "dac",
 		.devname	= "s5p-sdo",
 		.enable		= exynos4_clk_ip_tv_ctrl,
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH] ARM: Exynos4: Turn off clocks for NAND, OneNAND and TSI controllers
  2012-08-24 14:52 [PATCH] ARM: Exynos4: Turn off clocks for NAND, OneNAND and TSI controllers Chander Kashyap
@ 2012-08-28  8:15 ` Thomas Abraham
  2012-08-28 23:20   ` Kukjin Kim
  0 siblings, 1 reply; 3+ messages in thread
From: Thomas Abraham @ 2012-08-28  8:15 UTC (permalink / raw)
  To: linux-arm-kernel

On 24 August 2012 20:22, Chander Kashyap <chander.kashyap@linaro.org> wrote:
> The clocks for NAND, OneNAND and Transport Stream Interface(TSI)
> controllers could be either enabled or disabled at boot. To ensure
> that these are turned off until used, add them to the list of clocks
> to be turned off during boot.
>
> Signed-off-by: Chander Kashyap <chander.kashyap@linaro.org>
> ---
>  arch/arm/mach-exynos/clock-exynos4.c |   12 ++++++++++++
>  1 file changed, 12 insertions(+)
>
> diff --git a/arch/arm/mach-exynos/clock-exynos4.c b/arch/arm/mach-exynos/clock-exynos4.c
> index 2f51293..7cc5491 100644
> --- a/arch/arm/mach-exynos/clock-exynos4.c
> +++ b/arch/arm/mach-exynos/clock-exynos4.c
> @@ -501,6 +501,10 @@ static struct clk exynos4_init_clocks_off[] = {
>                 .enable         = exynos4_clk_ip_cam_ctrl,
>                 .ctrlbit        = (1 << 3),
>         }, {
> +               .name           = "tsi",
> +               .enable         = exynos4_clk_ip_fsys_ctrl,
> +               .ctrlbit        = (1 << 4),
> +       }, {
>                 .name           = "hsmmc",
>                 .devname        = "exynos4-sdhci.0",
>                 .parent         = &exynos4_clk_aclk_133.clk,
> @@ -530,6 +534,14 @@ static struct clk exynos4_init_clocks_off[] = {
>                 .enable         = exynos4_clk_ip_fsys_ctrl,
>                 .ctrlbit        = (1 << 9),
>         }, {
> +               .name           = "onenand",
> +               .enable         = exynos4_clk_ip_fsys_ctrl,
> +               .ctrlbit        = (1 << 15),
> +       }, {
> +               .name           = "nfcon",
> +               .enable         = exynos4_clk_ip_fsys_ctrl,
> +               .ctrlbit        = (1 << 16),
> +       }, {
>                 .name           = "dac",
>                 .devname        = "s5p-sdo",
>                 .enable         = exynos4_clk_ip_tv_ctrl,
> --
> 1.7.9.5

Reviewed-by: Thomas Abraham <thomas.abraham@linaro.org>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH] ARM: Exynos4: Turn off clocks for NAND, OneNAND and TSI controllers
  2012-08-28  8:15 ` Thomas Abraham
@ 2012-08-28 23:20   ` Kukjin Kim
  0 siblings, 0 replies; 3+ messages in thread
From: Kukjin Kim @ 2012-08-28 23:20 UTC (permalink / raw)
  To: linux-arm-kernel

On 08/28/12 01:15, Thomas Abraham wrote:
> On 24 August 2012 20:22, Chander Kashyap<chander.kashyap@linaro.org>  wrote:
>> The clocks for NAND, OneNAND and Transport Stream Interface(TSI)
>> controllers could be either enabled or disabled at boot. To ensure
>> that these are turned off until used, add them to the list of clocks
>> to be turned off during boot.
>>
>> Signed-off-by: Chander Kashyap<chander.kashyap@linaro.org>
>> ---
>>   arch/arm/mach-exynos/clock-exynos4.c |   12 ++++++++++++
>>   1 file changed, 12 insertions(+)
>>
>> diff --git a/arch/arm/mach-exynos/clock-exynos4.c b/arch/arm/mach-exynos/clock-exynos4.c
>> index 2f51293..7cc5491 100644
>> --- a/arch/arm/mach-exynos/clock-exynos4.c
>> +++ b/arch/arm/mach-exynos/clock-exynos4.c
>> @@ -501,6 +501,10 @@ static struct clk exynos4_init_clocks_off[] = {
>>                  .enable         = exynos4_clk_ip_cam_ctrl,
>>                  .ctrlbit        = (1<<  3),
>>          }, {
>> +               .name           = "tsi",
>> +               .enable         = exynos4_clk_ip_fsys_ctrl,
>> +               .ctrlbit        = (1<<  4),
>> +       }, {
>>                  .name           = "hsmmc",
>>                  .devname        = "exynos4-sdhci.0",
>>                  .parent         =&exynos4_clk_aclk_133.clk,
>> @@ -530,6 +534,14 @@ static struct clk exynos4_init_clocks_off[] = {
>>                  .enable         = exynos4_clk_ip_fsys_ctrl,
>>                  .ctrlbit        = (1<<  9),
>>          }, {
>> +               .name           = "onenand",
>> +               .enable         = exynos4_clk_ip_fsys_ctrl,
>> +               .ctrlbit        = (1<<  15),
>> +       }, {
>> +               .name           = "nfcon",
>> +               .enable         = exynos4_clk_ip_fsys_ctrl,
>> +               .ctrlbit        = (1<<  16),
>> +       }, {
>>                  .name           = "dac",
>>                  .devname        = "s5p-sdo",
>>                  .enable         = exynos4_clk_ip_tv_ctrl,
>> --
>> 1.7.9.5
>
> Reviewed-by: Thomas Abraham<thomas.abraham@linaro.org>

Applied, thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-08-28 23:20 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-08-24 14:52 [PATCH] ARM: Exynos4: Turn off clocks for NAND, OneNAND and TSI controllers Chander Kashyap
2012-08-28  8:15 ` Thomas Abraham
2012-08-28 23:20   ` Kukjin Kim

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).