* [PATCH] ARM: SAMSUNG: Modify the devname for pl330 clock
@ 2011-08-17 0:26 ` Boojin Kim
0 siblings, 0 replies; 8+ messages in thread
From: Boojin Kim @ 2011-08-17 0:26 UTC (permalink / raw)
To: linux-arm-kernel, linux-samsung-soc; +Cc: Kukjin Kim, Boojin Kim
This patch modified the devname for pl330 clock from 's3c-pl330' to 'dma-pl330'
to support new pl330 driver on dmaengine.
NOTE: This patch sould be added after merging new pl330 driver on dmaengine.
Signed-off-by: Boojin Kim <boojin.kim@samsung.com>
---
arch/arm/mach-exynos4/clock.c | 4 ++--
arch/arm/mach-s5p64x0/clock-s5p6440.c | 1 +
arch/arm/mach-s5p64x0/clock-s5p6450.c | 1 +
arch/arm/mach-s5pc100/clock.c | 4 ++--
arch/arm/mach-s5pv210/clock.c | 4 ++--
5 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/arch/arm/mach-exynos4/clock.c b/arch/arm/mach-exynos4/clock.c
index fee2dd8..04c997a 100644
--- a/arch/arm/mach-exynos4/clock.c
+++ b/arch/arm/mach-exynos4/clock.c
@@ -460,12 +460,12 @@ static struct clk init_clocks_off[] = {
.ctrlbit = (1 << 10),
}, {
.name = "dma",
- .devname = "s3c-pl330.0",
+ .devname = "dma-pl330.0",
.enable = exynos4_clk_ip_fsys_ctrl,
.ctrlbit = (1 << 0),
}, {
.name = "dma",
- .devname = "s3c-pl330.1",
+ .devname = "dma-pl330.1",
.enable = exynos4_clk_ip_fsys_ctrl,
.ctrlbit = (1 << 1),
}, {
diff --git a/arch/arm/mach-s5p64x0/clock-s5p6440.c b/arch/arm/mach-s5p64x0/clock-s5p6440.c
index c1f548f..c54c65d 100644
--- a/arch/arm/mach-s5p64x0/clock-s5p6440.c
+++ b/arch/arm/mach-s5p64x0/clock-s5p6440.c
@@ -147,6 +147,7 @@ static struct clk init_clocks_off[] = {
.ctrlbit = (1 << 8),
}, {
.name = "dma",
+ .devname = "dma-pl330",
.parent = &clk_hclk_low.clk,
.enable = s5p64x0_hclk0_ctrl,
.ctrlbit = (1 << 12),
diff --git a/arch/arm/mach-s5p64x0/clock-s5p6450.c b/arch/arm/mach-s5p64x0/clock-s5p6450.c
index 3d9b609..2d04abf 100644
--- a/arch/arm/mach-s5p64x0/clock-s5p6450.c
+++ b/arch/arm/mach-s5p64x0/clock-s5p6450.c
@@ -180,6 +180,7 @@ static struct clk init_clocks_off[] = {
.ctrlbit = (1 << 3),
}, {
.name = "dma",
+ .devname = "dma-pl330",
.parent = &clk_hclk_low.clk,
.enable = s5p64x0_hclk0_ctrl,
.ctrlbit = (1 << 12),
diff --git a/arch/arm/mach-s5pc100/clock.c b/arch/arm/mach-s5pc100/clock.c
index 6527c05..8d47709 100644
--- a/arch/arm/mach-s5pc100/clock.c
+++ b/arch/arm/mach-s5pc100/clock.c
@@ -460,13 +460,13 @@ static struct clk init_clocks_off[] = {
.ctrlbit = (1 << 2),
}, {
.name = "dma",
- .devname = "s3c-pl330.1",
+ .devname = "dma-pl330.1",
.parent = &clk_div_d1_bus.clk,
.enable = s5pc100_d1_0_ctrl,
.ctrlbit = (1 << 1),
}, {
.name = "dma",
- .devname = "s3c-pl330.0",
+ .devname = "dma-pl330.0",
.parent = &clk_div_d1_bus.clk,
.enable = s5pc100_d1_0_ctrl,
.ctrlbit = (1 << 0),
diff --git a/arch/arm/mach-s5pv210/clock.c b/arch/arm/mach-s5pv210/clock.c
index d35726a..239aaad 100644
--- a/arch/arm/mach-s5pv210/clock.c
+++ b/arch/arm/mach-s5pv210/clock.c
@@ -295,13 +295,13 @@ static struct clk_ops clk_fout_apll_ops = {
static struct clk init_clocks_off[] = {
{
.name = "dma",
- .devname = "s3c-pl330.0",
+ .devname = "dma-pl330.0",
.parent = &clk_hclk_psys.clk,
.enable = s5pv210_clk_ip0_ctrl,
.ctrlbit = (1 << 3),
}, {
.name = "dma",
- .devname = "s3c-pl330.1",
+ .devname = "dma-pl330.1",
.parent = &clk_hclk_psys.clk,
.enable = s5pv210_clk_ip0_ctrl,
.ctrlbit = (1 << 4),
--
1.7.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH] ARM: SAMSUNG: Modify the devname for pl330 clock
@ 2011-08-17 0:26 ` Boojin Kim
0 siblings, 0 replies; 8+ messages in thread
From: Boojin Kim @ 2011-08-17 0:26 UTC (permalink / raw)
To: linux-arm-kernel
This patch modified the devname for pl330 clock from 's3c-pl330' to 'dma-pl330'
to support new pl330 driver on dmaengine.
NOTE: This patch sould be added after merging new pl330 driver on dmaengine.
Signed-off-by: Boojin Kim <boojin.kim@samsung.com>
---
arch/arm/mach-exynos4/clock.c | 4 ++--
arch/arm/mach-s5p64x0/clock-s5p6440.c | 1 +
arch/arm/mach-s5p64x0/clock-s5p6450.c | 1 +
arch/arm/mach-s5pc100/clock.c | 4 ++--
arch/arm/mach-s5pv210/clock.c | 4 ++--
5 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/arch/arm/mach-exynos4/clock.c b/arch/arm/mach-exynos4/clock.c
index fee2dd8..04c997a 100644
--- a/arch/arm/mach-exynos4/clock.c
+++ b/arch/arm/mach-exynos4/clock.c
@@ -460,12 +460,12 @@ static struct clk init_clocks_off[] = {
.ctrlbit = (1 << 10),
}, {
.name = "dma",
- .devname = "s3c-pl330.0",
+ .devname = "dma-pl330.0",
.enable = exynos4_clk_ip_fsys_ctrl,
.ctrlbit = (1 << 0),
}, {
.name = "dma",
- .devname = "s3c-pl330.1",
+ .devname = "dma-pl330.1",
.enable = exynos4_clk_ip_fsys_ctrl,
.ctrlbit = (1 << 1),
}, {
diff --git a/arch/arm/mach-s5p64x0/clock-s5p6440.c b/arch/arm/mach-s5p64x0/clock-s5p6440.c
index c1f548f..c54c65d 100644
--- a/arch/arm/mach-s5p64x0/clock-s5p6440.c
+++ b/arch/arm/mach-s5p64x0/clock-s5p6440.c
@@ -147,6 +147,7 @@ static struct clk init_clocks_off[] = {
.ctrlbit = (1 << 8),
}, {
.name = "dma",
+ .devname = "dma-pl330",
.parent = &clk_hclk_low.clk,
.enable = s5p64x0_hclk0_ctrl,
.ctrlbit = (1 << 12),
diff --git a/arch/arm/mach-s5p64x0/clock-s5p6450.c b/arch/arm/mach-s5p64x0/clock-s5p6450.c
index 3d9b609..2d04abf 100644
--- a/arch/arm/mach-s5p64x0/clock-s5p6450.c
+++ b/arch/arm/mach-s5p64x0/clock-s5p6450.c
@@ -180,6 +180,7 @@ static struct clk init_clocks_off[] = {
.ctrlbit = (1 << 3),
}, {
.name = "dma",
+ .devname = "dma-pl330",
.parent = &clk_hclk_low.clk,
.enable = s5p64x0_hclk0_ctrl,
.ctrlbit = (1 << 12),
diff --git a/arch/arm/mach-s5pc100/clock.c b/arch/arm/mach-s5pc100/clock.c
index 6527c05..8d47709 100644
--- a/arch/arm/mach-s5pc100/clock.c
+++ b/arch/arm/mach-s5pc100/clock.c
@@ -460,13 +460,13 @@ static struct clk init_clocks_off[] = {
.ctrlbit = (1 << 2),
}, {
.name = "dma",
- .devname = "s3c-pl330.1",
+ .devname = "dma-pl330.1",
.parent = &clk_div_d1_bus.clk,
.enable = s5pc100_d1_0_ctrl,
.ctrlbit = (1 << 1),
}, {
.name = "dma",
- .devname = "s3c-pl330.0",
+ .devname = "dma-pl330.0",
.parent = &clk_div_d1_bus.clk,
.enable = s5pc100_d1_0_ctrl,
.ctrlbit = (1 << 0),
diff --git a/arch/arm/mach-s5pv210/clock.c b/arch/arm/mach-s5pv210/clock.c
index d35726a..239aaad 100644
--- a/arch/arm/mach-s5pv210/clock.c
+++ b/arch/arm/mach-s5pv210/clock.c
@@ -295,13 +295,13 @@ static struct clk_ops clk_fout_apll_ops = {
static struct clk init_clocks_off[] = {
{
.name = "dma",
- .devname = "s3c-pl330.0",
+ .devname = "dma-pl330.0",
.parent = &clk_hclk_psys.clk,
.enable = s5pv210_clk_ip0_ctrl,
.ctrlbit = (1 << 3),
}, {
.name = "dma",
- .devname = "s3c-pl330.1",
+ .devname = "dma-pl330.1",
.parent = &clk_hclk_psys.clk,
.enable = s5pv210_clk_ip0_ctrl,
.ctrlbit = (1 << 4),
--
1.7.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: SAMSUNG: Modify the devname for pl330 clock
2011-08-17 0:26 ` Boojin Kim
@ 2011-08-17 6:16 ` Vladimir Zapolskiy
-1 siblings, 0 replies; 8+ messages in thread
From: Vladimir Zapolskiy @ 2011-08-17 6:16 UTC (permalink / raw)
To: Boojin Kim; +Cc: linux-arm-kernel, linux-samsung-soc, Kukjin Kim
Hi,
looks like it contains my series as a subset.
--
With best wishes,
Vladimir
On 17.08.2011 03:26, Boojin Kim wrote:
> This patch modified the devname for pl330 clock from 's3c-pl330' to 'dma-pl330'
> to support new pl330 driver on dmaengine.
> NOTE: This patch sould be added after merging new pl330 driver on dmaengine.
>
> Signed-off-by: Boojin Kim<boojin.kim@samsung.com>
> ---
> arch/arm/mach-exynos4/clock.c | 4 ++--
> arch/arm/mach-s5p64x0/clock-s5p6440.c | 1 +
> arch/arm/mach-s5p64x0/clock-s5p6450.c | 1 +
> arch/arm/mach-s5pc100/clock.c | 4 ++--
> arch/arm/mach-s5pv210/clock.c | 4 ++--
> 5 files changed, 8 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm/mach-exynos4/clock.c b/arch/arm/mach-exynos4/clock.c
> index fee2dd8..04c997a 100644
> --- a/arch/arm/mach-exynos4/clock.c
> +++ b/arch/arm/mach-exynos4/clock.c
> @@ -460,12 +460,12 @@ static struct clk init_clocks_off[] = {
> .ctrlbit = (1<< 10),
> }, {
> .name = "dma",
> - .devname = "s3c-pl330.0",
> + .devname = "dma-pl330.0",
> .enable = exynos4_clk_ip_fsys_ctrl,
> .ctrlbit = (1<< 0),
> }, {
> .name = "dma",
> - .devname = "s3c-pl330.1",
> + .devname = "dma-pl330.1",
> .enable = exynos4_clk_ip_fsys_ctrl,
> .ctrlbit = (1<< 1),
> }, {
> diff --git a/arch/arm/mach-s5p64x0/clock-s5p6440.c b/arch/arm/mach-s5p64x0/clock-s5p6440.c
> index c1f548f..c54c65d 100644
> --- a/arch/arm/mach-s5p64x0/clock-s5p6440.c
> +++ b/arch/arm/mach-s5p64x0/clock-s5p6440.c
> @@ -147,6 +147,7 @@ static struct clk init_clocks_off[] = {
> .ctrlbit = (1<< 8),
> }, {
> .name = "dma",
> + .devname = "dma-pl330",
> .parent =&clk_hclk_low.clk,
> .enable = s5p64x0_hclk0_ctrl,
> .ctrlbit = (1<< 12),
> diff --git a/arch/arm/mach-s5p64x0/clock-s5p6450.c b/arch/arm/mach-s5p64x0/clock-s5p6450.c
> index 3d9b609..2d04abf 100644
> --- a/arch/arm/mach-s5p64x0/clock-s5p6450.c
> +++ b/arch/arm/mach-s5p64x0/clock-s5p6450.c
> @@ -180,6 +180,7 @@ static struct clk init_clocks_off[] = {
> .ctrlbit = (1<< 3),
> }, {
> .name = "dma",
> + .devname = "dma-pl330",
> .parent =&clk_hclk_low.clk,
> .enable = s5p64x0_hclk0_ctrl,
> .ctrlbit = (1<< 12),
> diff --git a/arch/arm/mach-s5pc100/clock.c b/arch/arm/mach-s5pc100/clock.c
> index 6527c05..8d47709 100644
> --- a/arch/arm/mach-s5pc100/clock.c
> +++ b/arch/arm/mach-s5pc100/clock.c
> @@ -460,13 +460,13 @@ static struct clk init_clocks_off[] = {
> .ctrlbit = (1<< 2),
> }, {
> .name = "dma",
> - .devname = "s3c-pl330.1",
> + .devname = "dma-pl330.1",
> .parent =&clk_div_d1_bus.clk,
> .enable = s5pc100_d1_0_ctrl,
> .ctrlbit = (1<< 1),
> }, {
> .name = "dma",
> - .devname = "s3c-pl330.0",
> + .devname = "dma-pl330.0",
> .parent =&clk_div_d1_bus.clk,
> .enable = s5pc100_d1_0_ctrl,
> .ctrlbit = (1<< 0),
> diff --git a/arch/arm/mach-s5pv210/clock.c b/arch/arm/mach-s5pv210/clock.c
> index d35726a..239aaad 100644
> --- a/arch/arm/mach-s5pv210/clock.c
> +++ b/arch/arm/mach-s5pv210/clock.c
> @@ -295,13 +295,13 @@ static struct clk_ops clk_fout_apll_ops = {
> static struct clk init_clocks_off[] = {
> {
> .name = "dma",
> - .devname = "s3c-pl330.0",
> + .devname = "dma-pl330.0",
> .parent =&clk_hclk_psys.clk,
> .enable = s5pv210_clk_ip0_ctrl,
> .ctrlbit = (1<< 3),
> }, {
> .name = "dma",
> - .devname = "s3c-pl330.1",
> + .devname = "dma-pl330.1",
> .parent =&clk_hclk_psys.clk,
> .enable = s5pv210_clk_ip0_ctrl,
> .ctrlbit = (1<< 4),
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] ARM: SAMSUNG: Modify the devname for pl330 clock
@ 2011-08-17 6:16 ` Vladimir Zapolskiy
0 siblings, 0 replies; 8+ messages in thread
From: Vladimir Zapolskiy @ 2011-08-17 6:16 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
looks like it contains my series as a subset.
--
With best wishes,
Vladimir
On 17.08.2011 03:26, Boojin Kim wrote:
> This patch modified the devname for pl330 clock from 's3c-pl330' to 'dma-pl330'
> to support new pl330 driver on dmaengine.
> NOTE: This patch sould be added after merging new pl330 driver on dmaengine.
>
> Signed-off-by: Boojin Kim<boojin.kim@samsung.com>
> ---
> arch/arm/mach-exynos4/clock.c | 4 ++--
> arch/arm/mach-s5p64x0/clock-s5p6440.c | 1 +
> arch/arm/mach-s5p64x0/clock-s5p6450.c | 1 +
> arch/arm/mach-s5pc100/clock.c | 4 ++--
> arch/arm/mach-s5pv210/clock.c | 4 ++--
> 5 files changed, 8 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm/mach-exynos4/clock.c b/arch/arm/mach-exynos4/clock.c
> index fee2dd8..04c997a 100644
> --- a/arch/arm/mach-exynos4/clock.c
> +++ b/arch/arm/mach-exynos4/clock.c
> @@ -460,12 +460,12 @@ static struct clk init_clocks_off[] = {
> .ctrlbit = (1<< 10),
> }, {
> .name = "dma",
> - .devname = "s3c-pl330.0",
> + .devname = "dma-pl330.0",
> .enable = exynos4_clk_ip_fsys_ctrl,
> .ctrlbit = (1<< 0),
> }, {
> .name = "dma",
> - .devname = "s3c-pl330.1",
> + .devname = "dma-pl330.1",
> .enable = exynos4_clk_ip_fsys_ctrl,
> .ctrlbit = (1<< 1),
> }, {
> diff --git a/arch/arm/mach-s5p64x0/clock-s5p6440.c b/arch/arm/mach-s5p64x0/clock-s5p6440.c
> index c1f548f..c54c65d 100644
> --- a/arch/arm/mach-s5p64x0/clock-s5p6440.c
> +++ b/arch/arm/mach-s5p64x0/clock-s5p6440.c
> @@ -147,6 +147,7 @@ static struct clk init_clocks_off[] = {
> .ctrlbit = (1<< 8),
> }, {
> .name = "dma",
> + .devname = "dma-pl330",
> .parent =&clk_hclk_low.clk,
> .enable = s5p64x0_hclk0_ctrl,
> .ctrlbit = (1<< 12),
> diff --git a/arch/arm/mach-s5p64x0/clock-s5p6450.c b/arch/arm/mach-s5p64x0/clock-s5p6450.c
> index 3d9b609..2d04abf 100644
> --- a/arch/arm/mach-s5p64x0/clock-s5p6450.c
> +++ b/arch/arm/mach-s5p64x0/clock-s5p6450.c
> @@ -180,6 +180,7 @@ static struct clk init_clocks_off[] = {
> .ctrlbit = (1<< 3),
> }, {
> .name = "dma",
> + .devname = "dma-pl330",
> .parent =&clk_hclk_low.clk,
> .enable = s5p64x0_hclk0_ctrl,
> .ctrlbit = (1<< 12),
> diff --git a/arch/arm/mach-s5pc100/clock.c b/arch/arm/mach-s5pc100/clock.c
> index 6527c05..8d47709 100644
> --- a/arch/arm/mach-s5pc100/clock.c
> +++ b/arch/arm/mach-s5pc100/clock.c
> @@ -460,13 +460,13 @@ static struct clk init_clocks_off[] = {
> .ctrlbit = (1<< 2),
> }, {
> .name = "dma",
> - .devname = "s3c-pl330.1",
> + .devname = "dma-pl330.1",
> .parent =&clk_div_d1_bus.clk,
> .enable = s5pc100_d1_0_ctrl,
> .ctrlbit = (1<< 1),
> }, {
> .name = "dma",
> - .devname = "s3c-pl330.0",
> + .devname = "dma-pl330.0",
> .parent =&clk_div_d1_bus.clk,
> .enable = s5pc100_d1_0_ctrl,
> .ctrlbit = (1<< 0),
> diff --git a/arch/arm/mach-s5pv210/clock.c b/arch/arm/mach-s5pv210/clock.c
> index d35726a..239aaad 100644
> --- a/arch/arm/mach-s5pv210/clock.c
> +++ b/arch/arm/mach-s5pv210/clock.c
> @@ -295,13 +295,13 @@ static struct clk_ops clk_fout_apll_ops = {
> static struct clk init_clocks_off[] = {
> {
> .name = "dma",
> - .devname = "s3c-pl330.0",
> + .devname = "dma-pl330.0",
> .parent =&clk_hclk_psys.clk,
> .enable = s5pv210_clk_ip0_ctrl,
> .ctrlbit = (1<< 3),
> }, {
> .name = "dma",
> - .devname = "s3c-pl330.1",
> + .devname = "dma-pl330.1",
> .parent =&clk_hclk_psys.clk,
> .enable = s5pv210_clk_ip0_ctrl,
> .ctrlbit = (1<< 4),
^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [PATCH] ARM: SAMSUNG: Modify the devname for pl330 clock
2011-08-17 6:16 ` Vladimir Zapolskiy
@ 2011-08-17 8:46 ` Boojin Kim
-1 siblings, 0 replies; 8+ messages in thread
From: Boojin Kim @ 2011-08-17 8:46 UTC (permalink / raw)
To: 'Vladimir Zapolskiy'
Cc: linux-arm-kernel, linux-samsung-soc, 'Kukjin Kim'
Vladimir Zapolskiy wrote:
> Hi,
>
> looks like it contains my series as a subset.
As I replied on your patches, yes, I know that. I sent it before see your
patch.
Do you add the missing about s5p64x0 in your patch?
If not, I will split up this patch to get missing about s5p64x0 in your
patch.
Thanks
Boojin Kim.
>
> --
> With best wishes,
> Vladimir
>
> On 17.08.2011 03:26, Boojin Kim wrote:
> > This patch modified the devname for pl330 clock from 's3c-pl330' to
> 'dma-pl330'
> > to support new pl330 driver on dmaengine.
> > NOTE: This patch sould be added after merging new pl330 driver on
> dmaengine.
> >
> > Signed-off-by: Boojin Kim<boojin.kim@samsung.com>
> > ---
> > arch/arm/mach-exynos4/clock.c | 4 ++--
> > arch/arm/mach-s5p64x0/clock-s5p6440.c | 1 +
> > arch/arm/mach-s5p64x0/clock-s5p6450.c | 1 +
> > arch/arm/mach-s5pc100/clock.c | 4 ++--
> > arch/arm/mach-s5pv210/clock.c | 4 ++--
> > 5 files changed, 8 insertions(+), 6 deletions(-)
> >
> > diff --git a/arch/arm/mach-exynos4/clock.c b/arch/arm/mach-
> exynos4/clock.c
> > index fee2dd8..04c997a 100644
> > --- a/arch/arm/mach-exynos4/clock.c
> > +++ b/arch/arm/mach-exynos4/clock.c
> > @@ -460,12 +460,12 @@ static struct clk init_clocks_off[] = {
> > .ctrlbit = (1<< 10),
> > }, {
> > .name = "dma",
> > - .devname = "s3c-pl330.0",
> > + .devname = "dma-pl330.0",
> > .enable = exynos4_clk_ip_fsys_ctrl,
> > .ctrlbit = (1<< 0),
> > }, {
> > .name = "dma",
> > - .devname = "s3c-pl330.1",
> > + .devname = "dma-pl330.1",
> > .enable = exynos4_clk_ip_fsys_ctrl,
> > .ctrlbit = (1<< 1),
> > }, {
> > diff --git a/arch/arm/mach-s5p64x0/clock-s5p6440.c b/arch/arm/mach-
> s5p64x0/clock-s5p6440.c
> > index c1f548f..c54c65d 100644
> > --- a/arch/arm/mach-s5p64x0/clock-s5p6440.c
> > +++ b/arch/arm/mach-s5p64x0/clock-s5p6440.c
> > @@ -147,6 +147,7 @@ static struct clk init_clocks_off[] = {
> > .ctrlbit = (1<< 8),
> > }, {
> > .name = "dma",
> > + .devname = "dma-pl330",
> > .parent =&clk_hclk_low.clk,
> > .enable = s5p64x0_hclk0_ctrl,
> > .ctrlbit = (1<< 12),
> > diff --git a/arch/arm/mach-s5p64x0/clock-s5p6450.c b/arch/arm/mach-
> s5p64x0/clock-s5p6450.c
> > index 3d9b609..2d04abf 100644
> > --- a/arch/arm/mach-s5p64x0/clock-s5p6450.c
> > +++ b/arch/arm/mach-s5p64x0/clock-s5p6450.c
> > @@ -180,6 +180,7 @@ static struct clk init_clocks_off[] = {
> > .ctrlbit = (1<< 3),
> > }, {
> > .name = "dma",
> > + .devname = "dma-pl330",
> > .parent =&clk_hclk_low.clk,
> > .enable = s5p64x0_hclk0_ctrl,
> > .ctrlbit = (1<< 12),
> > diff --git a/arch/arm/mach-s5pc100/clock.c b/arch/arm/mach-
> s5pc100/clock.c
> > index 6527c05..8d47709 100644
> > --- a/arch/arm/mach-s5pc100/clock.c
> > +++ b/arch/arm/mach-s5pc100/clock.c
> > @@ -460,13 +460,13 @@ static struct clk init_clocks_off[] = {
> > .ctrlbit = (1<< 2),
> > }, {
> > .name = "dma",
> > - .devname = "s3c-pl330.1",
> > + .devname = "dma-pl330.1",
> > .parent =&clk_div_d1_bus.clk,
> > .enable = s5pc100_d1_0_ctrl,
> > .ctrlbit = (1<< 1),
> > }, {
> > .name = "dma",
> > - .devname = "s3c-pl330.0",
> > + .devname = "dma-pl330.0",
> > .parent =&clk_div_d1_bus.clk,
> > .enable = s5pc100_d1_0_ctrl,
> > .ctrlbit = (1<< 0),
> > diff --git a/arch/arm/mach-s5pv210/clock.c b/arch/arm/mach-
> s5pv210/clock.c
> > index d35726a..239aaad 100644
> > --- a/arch/arm/mach-s5pv210/clock.c
> > +++ b/arch/arm/mach-s5pv210/clock.c
> > @@ -295,13 +295,13 @@ static struct clk_ops clk_fout_apll_ops = {
> > static struct clk init_clocks_off[] = {
> > {
> > .name = "dma",
> > - .devname = "s3c-pl330.0",
> > + .devname = "dma-pl330.0",
> > .parent =&clk_hclk_psys.clk,
> > .enable = s5pv210_clk_ip0_ctrl,
> > .ctrlbit = (1<< 3),
> > }, {
> > .name = "dma",
> > - .devname = "s3c-pl330.1",
> > + .devname = "dma-pl330.1",
> > .parent =&clk_hclk_psys.clk,
> > .enable = s5pv210_clk_ip0_ctrl,
> > .ctrlbit = (1<< 4),
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] ARM: SAMSUNG: Modify the devname for pl330 clock
@ 2011-08-17 8:46 ` Boojin Kim
0 siblings, 0 replies; 8+ messages in thread
From: Boojin Kim @ 2011-08-17 8:46 UTC (permalink / raw)
To: linux-arm-kernel
Vladimir Zapolskiy wrote:
> Hi,
>
> looks like it contains my series as a subset.
As I replied on your patches, yes, I know that. I sent it before see your
patch.
Do you add the missing about s5p64x0 in your patch?
If not, I will split up this patch to get missing about s5p64x0 in your
patch.
Thanks
Boojin Kim.
>
> --
> With best wishes,
> Vladimir
>
> On 17.08.2011 03:26, Boojin Kim wrote:
> > This patch modified the devname for pl330 clock from 's3c-pl330' to
> 'dma-pl330'
> > to support new pl330 driver on dmaengine.
> > NOTE: This patch sould be added after merging new pl330 driver on
> dmaengine.
> >
> > Signed-off-by: Boojin Kim<boojin.kim@samsung.com>
> > ---
> > arch/arm/mach-exynos4/clock.c | 4 ++--
> > arch/arm/mach-s5p64x0/clock-s5p6440.c | 1 +
> > arch/arm/mach-s5p64x0/clock-s5p6450.c | 1 +
> > arch/arm/mach-s5pc100/clock.c | 4 ++--
> > arch/arm/mach-s5pv210/clock.c | 4 ++--
> > 5 files changed, 8 insertions(+), 6 deletions(-)
> >
> > diff --git a/arch/arm/mach-exynos4/clock.c b/arch/arm/mach-
> exynos4/clock.c
> > index fee2dd8..04c997a 100644
> > --- a/arch/arm/mach-exynos4/clock.c
> > +++ b/arch/arm/mach-exynos4/clock.c
> > @@ -460,12 +460,12 @@ static struct clk init_clocks_off[] = {
> > .ctrlbit = (1<< 10),
> > }, {
> > .name = "dma",
> > - .devname = "s3c-pl330.0",
> > + .devname = "dma-pl330.0",
> > .enable = exynos4_clk_ip_fsys_ctrl,
> > .ctrlbit = (1<< 0),
> > }, {
> > .name = "dma",
> > - .devname = "s3c-pl330.1",
> > + .devname = "dma-pl330.1",
> > .enable = exynos4_clk_ip_fsys_ctrl,
> > .ctrlbit = (1<< 1),
> > }, {
> > diff --git a/arch/arm/mach-s5p64x0/clock-s5p6440.c b/arch/arm/mach-
> s5p64x0/clock-s5p6440.c
> > index c1f548f..c54c65d 100644
> > --- a/arch/arm/mach-s5p64x0/clock-s5p6440.c
> > +++ b/arch/arm/mach-s5p64x0/clock-s5p6440.c
> > @@ -147,6 +147,7 @@ static struct clk init_clocks_off[] = {
> > .ctrlbit = (1<< 8),
> > }, {
> > .name = "dma",
> > + .devname = "dma-pl330",
> > .parent =&clk_hclk_low.clk,
> > .enable = s5p64x0_hclk0_ctrl,
> > .ctrlbit = (1<< 12),
> > diff --git a/arch/arm/mach-s5p64x0/clock-s5p6450.c b/arch/arm/mach-
> s5p64x0/clock-s5p6450.c
> > index 3d9b609..2d04abf 100644
> > --- a/arch/arm/mach-s5p64x0/clock-s5p6450.c
> > +++ b/arch/arm/mach-s5p64x0/clock-s5p6450.c
> > @@ -180,6 +180,7 @@ static struct clk init_clocks_off[] = {
> > .ctrlbit = (1<< 3),
> > }, {
> > .name = "dma",
> > + .devname = "dma-pl330",
> > .parent =&clk_hclk_low.clk,
> > .enable = s5p64x0_hclk0_ctrl,
> > .ctrlbit = (1<< 12),
> > diff --git a/arch/arm/mach-s5pc100/clock.c b/arch/arm/mach-
> s5pc100/clock.c
> > index 6527c05..8d47709 100644
> > --- a/arch/arm/mach-s5pc100/clock.c
> > +++ b/arch/arm/mach-s5pc100/clock.c
> > @@ -460,13 +460,13 @@ static struct clk init_clocks_off[] = {
> > .ctrlbit = (1<< 2),
> > }, {
> > .name = "dma",
> > - .devname = "s3c-pl330.1",
> > + .devname = "dma-pl330.1",
> > .parent =&clk_div_d1_bus.clk,
> > .enable = s5pc100_d1_0_ctrl,
> > .ctrlbit = (1<< 1),
> > }, {
> > .name = "dma",
> > - .devname = "s3c-pl330.0",
> > + .devname = "dma-pl330.0",
> > .parent =&clk_div_d1_bus.clk,
> > .enable = s5pc100_d1_0_ctrl,
> > .ctrlbit = (1<< 0),
> > diff --git a/arch/arm/mach-s5pv210/clock.c b/arch/arm/mach-
> s5pv210/clock.c
> > index d35726a..239aaad 100644
> > --- a/arch/arm/mach-s5pv210/clock.c
> > +++ b/arch/arm/mach-s5pv210/clock.c
> > @@ -295,13 +295,13 @@ static struct clk_ops clk_fout_apll_ops = {
> > static struct clk init_clocks_off[] = {
> > {
> > .name = "dma",
> > - .devname = "s3c-pl330.0",
> > + .devname = "dma-pl330.0",
> > .parent =&clk_hclk_psys.clk,
> > .enable = s5pv210_clk_ip0_ctrl,
> > .ctrlbit = (1<< 3),
> > }, {
> > .name = "dma",
> > - .devname = "s3c-pl330.1",
> > + .devname = "dma-pl330.1",
> > .parent =&clk_hclk_psys.clk,
> > .enable = s5pv210_clk_ip0_ctrl,
> > .ctrlbit = (1<< 4),
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: SAMSUNG: Modify the devname for pl330 clock
2011-08-17 8:46 ` Boojin Kim
@ 2011-08-17 10:25 ` Vladimir Zapolskiy
-1 siblings, 0 replies; 8+ messages in thread
From: Vladimir Zapolskiy @ 2011-08-17 10:25 UTC (permalink / raw)
To: Boojin Kim; +Cc: linux-arm-kernel, linux-samsung-soc, Kukjin Kim
Hi,
On Wed, Aug 17, 2011 at 11:46 AM, Boojin Kim <boojin.kim@samsung.com> wrote:
> Vladimir Zapolskiy wrote:
>> Hi,
>>
>> looks like it contains my series as a subset.
>
> As I replied on your patches, yes, I know that. I sent it before see your
> patch.
> Do you add the missing about s5p64x0 in your patch?
> If not, I will split up this patch to get missing about s5p64x0 in your
> patch.
No, I haven't added that. In my opinion it would be nice, if you can
split the patch and fix more incorrespondent device names for s5p64x0,
or probably it might be done at once replacing my version of the
change for s5p64x0 only.
Thank you a lot!
--
With best wishes,
Vladimir
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] ARM: SAMSUNG: Modify the devname for pl330 clock
@ 2011-08-17 10:25 ` Vladimir Zapolskiy
0 siblings, 0 replies; 8+ messages in thread
From: Vladimir Zapolskiy @ 2011-08-17 10:25 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On Wed, Aug 17, 2011 at 11:46 AM, Boojin Kim <boojin.kim@samsung.com> wrote:
> Vladimir Zapolskiy wrote:
>> Hi,
>>
>> looks like it contains my series as a subset.
>
> As I replied on your patches, yes, I know that. I sent it before see your
> patch.
> Do you add the missing about s5p64x0 in your patch?
> If not, I will split up this patch to get missing about s5p64x0 in your
> patch.
No, I haven't added that. In my opinion it would be nice, if you can
split the patch and fix more incorrespondent device names for s5p64x0,
or probably it might be done at once replacing my version of the
change for s5p64x0 only.
Thank you a lot!
--
With best wishes,
Vladimir
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2011-08-17 10:25 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-08-17 0:26 [PATCH] ARM: SAMSUNG: Modify the devname for pl330 clock Boojin Kim
2011-08-17 0:26 ` Boojin Kim
2011-08-17 6:16 ` Vladimir Zapolskiy
2011-08-17 6:16 ` Vladimir Zapolskiy
2011-08-17 8:46 ` Boojin Kim
2011-08-17 8:46 ` Boojin Kim
2011-08-17 10:25 ` Vladimir Zapolskiy
2011-08-17 10:25 ` Vladimir Zapolskiy
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.