* [PATCH 1/1] da830: macro rename DA8XX_LPSC0_DMAX to DA8XX_LPSC0_PRUSS.
@ 2011-02-11 14:18 Subhasish Ghosh
2011-02-23 15:24 ` Nori, Sekhar
0 siblings, 1 reply; 3+ messages in thread
From: Subhasish Ghosh @ 2011-02-11 14:18 UTC (permalink / raw)
To: davinci-linux-open-source
Cc: linux-arm-kernel, m-watkins, nsekhar, sachi, Subhasish Ghosh,
Kevin Hilman, Russell King, Cyril Chemparathy, Juha Kuikka,
open list
The PRUSS was named as DMAX on da830 platform.
This patch resolves the naming conflict by replacing the macro
DA8XX_LPSC0_DMAX with DA8XX_LPSC0_PRUSS.
Signed-off-by: Subhasish Ghosh <subhasish@mistralsolutions.com>
---
arch/arm/mach-davinci/da830.c | 2 +-
arch/arm/mach-davinci/include/mach/psc.h | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/mach-davinci/da830.c b/arch/arm/mach-davinci/da830.c
index ec23ab4..0577a5d 100644
--- a/arch/arm/mach-davinci/da830.c
+++ b/arch/arm/mach-davinci/da830.c
@@ -148,7 +148,7 @@ static struct clk scr2_ss_clk = {
static struct clk dmax_clk = {
.name = "dmax",
.parent = &pll0_sysclk2,
- .lpsc = DA8XX_LPSC0_DMAX,
+ .lpsc = DA8XX_LPSC0_PRUSS,
.flags = ALWAYS_ENABLED,
};
diff --git a/arch/arm/mach-davinci/include/mach/psc.h b/arch/arm/mach-davinci/include/mach/psc.h
index 62b0858..a47e6f2 100644
--- a/arch/arm/mach-davinci/include/mach/psc.h
+++ b/arch/arm/mach-davinci/include/mach/psc.h
@@ -150,7 +150,7 @@
#define DA8XX_LPSC0_SCR0_SS 10
#define DA8XX_LPSC0_SCR1_SS 11
#define DA8XX_LPSC0_SCR2_SS 12
-#define DA8XX_LPSC0_DMAX 13
+#define DA8XX_LPSC0_PRUSS 13
#define DA8XX_LPSC0_ARM 14
#define DA8XX_LPSC0_GEM 15
--
1.7.2.3
^ permalink raw reply [flat|nested] 3+ messages in thread* RE: [PATCH 1/1] da830: macro rename DA8XX_LPSC0_DMAX to DA8XX_LPSC0_PRUSS.
2011-02-11 14:18 [PATCH 1/1] da830: macro rename DA8XX_LPSC0_DMAX to DA8XX_LPSC0_PRUSS Subhasish Ghosh
@ 2011-02-23 15:24 ` Nori, Sekhar
2011-02-24 10:27 ` Subhasish Ghosh
0 siblings, 1 reply; 3+ messages in thread
From: Nori, Sekhar @ 2011-02-23 15:24 UTC (permalink / raw)
To: Subhasish Ghosh, davinci-linux-open-source
Cc: linux-arm-kernel, Watkins, Melissa, sachi, Kevin Hilman,
Russell King, Chemparathy, Cyril, Juha Kuikka, open list
Hi Subhasish,
Please add a "davinci: " prefix to the subject line of
mach-davinci patches. The patch is not really specific
to DA830.
On Fri, Feb 11, 2011 at 19:48:25, Subhasish Ghosh wrote:
> The PRUSS was named as DMAX on da830 platform.
> This patch resolves the naming conflict by replacing the macro
> DA8XX_LPSC0_DMAX with DA8XX_LPSC0_PRUSS.
This description is inaccurate. Firstly, there is no naming
conflict. DMAX is an internal name for the module which is
known as PRUSS in TI public documentation. This patch just
gets the code in sync with TI documentation.
Can you please update the subject and re-spin this patch?
Thanks,
Sekhar
>
> Signed-off-by: Subhasish Ghosh <subhasish@mistralsolutions.com>
> ---
> arch/arm/mach-davinci/da830.c | 2 +-
> arch/arm/mach-davinci/include/mach/psc.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/mach-davinci/da830.c b/arch/arm/mach-davinci/da830.c
> index ec23ab4..0577a5d 100644
> --- a/arch/arm/mach-davinci/da830.c
> +++ b/arch/arm/mach-davinci/da830.c
> @@ -148,7 +148,7 @@ static struct clk scr2_ss_clk = {
> static struct clk dmax_clk = {
> .name = "dmax",
> .parent = &pll0_sysclk2,
> - .lpsc = DA8XX_LPSC0_DMAX,
> + .lpsc = DA8XX_LPSC0_PRUSS,
> .flags = ALWAYS_ENABLED,
> };
>
> diff --git a/arch/arm/mach-davinci/include/mach/psc.h b/arch/arm/mach-davinci/include/mach/psc.h
> index 62b0858..a47e6f2 100644
> --- a/arch/arm/mach-davinci/include/mach/psc.h
> +++ b/arch/arm/mach-davinci/include/mach/psc.h
> @@ -150,7 +150,7 @@
> #define DA8XX_LPSC0_SCR0_SS 10
> #define DA8XX_LPSC0_SCR1_SS 11
> #define DA8XX_LPSC0_SCR2_SS 12
> -#define DA8XX_LPSC0_DMAX 13
> +#define DA8XX_LPSC0_PRUSS 13
> #define DA8XX_LPSC0_ARM 14
> #define DA8XX_LPSC0_GEM 15
>
> --
> 1.7.2.3
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH 1/1] da830: macro rename DA8XX_LPSC0_DMAX to DA8XX_LPSC0_PRUSS.
2011-02-23 15:24 ` Nori, Sekhar
@ 2011-02-24 10:27 ` Subhasish Ghosh
0 siblings, 0 replies; 3+ messages in thread
From: Subhasish Ghosh @ 2011-02-24 10:27 UTC (permalink / raw)
To: Nori, Sekhar, davinci-linux-open-source
Cc: linux-arm-kernel, Watkins, Melissa, sachi, Kevin Hilman,
Russell King, Chemparathy, Cyril, Juha Kuikka, open list
> Hi Subhasish,
>
> Please add a "davinci: " prefix to the subject line of
> mach-davinci patches. The patch is not really specific
> to DA830.
SG -- OK, Will do
>
> On Fri, Feb 11, 2011 at 19:48:25, Subhasish Ghosh wrote:
>> The PRUSS was named as DMAX on da830 platform.
>> This patch resolves the naming conflict by replacing the macro
>> DA8XX_LPSC0_DMAX with DA8XX_LPSC0_PRUSS.
>
> This description is inaccurate. Firstly, there is no naming
> conflict. DMAX is an internal name for the module which is
> known as PRUSS in TI public documentation. This patch just
> gets the code in sync with TI documentation.
>
> Can you please update the subject and re-spin this patch?
SG - OK, Will do.
>
> Thanks,
> Sekhar
>
>>
>> Signed-off-by: Subhasish Ghosh <subhasish@mistralsolutions.com>
>> ---
>> arch/arm/mach-davinci/da830.c | 2 +-
>> arch/arm/mach-davinci/include/mach/psc.h | 2 +-
>> 2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/mach-davinci/da830.c
>> b/arch/arm/mach-davinci/da830.c
>> index ec23ab4..0577a5d 100644
>> --- a/arch/arm/mach-davinci/da830.c
>> +++ b/arch/arm/mach-davinci/da830.c
>> @@ -148,7 +148,7 @@ static struct clk scr2_ss_clk = {
>> static struct clk dmax_clk = {
>> .name = "dmax",
>> .parent = &pll0_sysclk2,
>> - .lpsc = DA8XX_LPSC0_DMAX,
>> + .lpsc = DA8XX_LPSC0_PRUSS,
>> .flags = ALWAYS_ENABLED,
>> };
>>
>> diff --git a/arch/arm/mach-davinci/include/mach/psc.h
>> b/arch/arm/mach-davinci/include/mach/psc.h
>> index 62b0858..a47e6f2 100644
>> --- a/arch/arm/mach-davinci/include/mach/psc.h
>> +++ b/arch/arm/mach-davinci/include/mach/psc.h
>> @@ -150,7 +150,7 @@
>> #define DA8XX_LPSC0_SCR0_SS 10
>> #define DA8XX_LPSC0_SCR1_SS 11
>> #define DA8XX_LPSC0_SCR2_SS 12
>> -#define DA8XX_LPSC0_DMAX 13
>> +#define DA8XX_LPSC0_PRUSS 13
>> #define DA8XX_LPSC0_ARM 14
>> #define DA8XX_LPSC0_GEM 15
>>
>> --
>> 1.7.2.3
>>
>>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-02-24 10:26 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-02-11 14:18 [PATCH 1/1] da830: macro rename DA8XX_LPSC0_DMAX to DA8XX_LPSC0_PRUSS Subhasish Ghosh
2011-02-23 15:24 ` Nori, Sekhar
2011-02-24 10:27 ` Subhasish Ghosh
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®