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 <[email protected]>
> ---
>  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
> 
> 

_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to