linux-kernel - RE: [PATCH 1/1] da830: macro rename DA8XX_LPSC0_DMAX to DA8XX_LPSC0

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 23 Feb 2011 20:54:52 +0530
From:	"Nori, Sekhar" <nsekhar@...com>
To:	Subhasish Ghosh <subhasish@...tralsolutions.com>,
	"davinci-linux-open-source@...ux.davincidsp.com" 
	<davinci-linux-open-source@...ux.davincidsp.com>
CC:	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>,
	"Watkins, Melissa" <m-watkins@...com>,
	"sachi@...tralsolutions.com" <sachi@...tralsolutions.com>,
	Kevin Hilman <khilman@...prootsystems.com>,
	Russell King <linux@....linux.org.uk>,
	"Chemparathy, Cyril" <cyril@...com>,
	Juha Kuikka <juha.kuikka@...ktrobit.com>,
	open list <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH 1/1] da830: macro rename DA8XX_LPSC0_DMAX to
 DA8XX_LPSC0_PRUSS.

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@...tralsolutions.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
> 
> 

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