RE: [PATCH 1/1] da830: macro rename DA8XX_LPSC0_DMAX toDA8XX_LPSC0_PRUSS.

From: Nori, Sekhar
Date: Wed Feb 23 2011 - 10:25:26 EST


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@xxxxxxxxxxxxxxxxxxxx>
> ---
> 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
>
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/