[U-Boot] [u-boot-release] [PATCH] fsl_sfp : Modify macros as per changes in SFP v3.4

[Sumit Garg]

> -----Original Message-----
> From: york sun
> Sent: Wednesday, August 31, 2016 9:04 PM
> To: Sumit Garg <sumit.garg at nxp.com>; u-boot at lists.denx.de
> Cc: Prabhakar Kushwaha <prabhakar.kushwaha at nxp.com>; Ruchika Gupta
> <ruchika.gupta at nxp.com>
> Subject: Re: [u-boot-release] [PATCH] fsl_sfp : Modify macros as per changes
> in SFP v3.4
> 
> On 08/30/2016 11:34 PM, Sumit Garg wrote:
> > In SFP v3.4 support for 8 keys in SRK table has been added which leads
> 
> I guess you mean "SFP v3.4 supports 8 keys ...".
>
I will correct this. 
 
> > to corresponding changes in OSPR key revocation field. So modify
> > OSPR_KEY_REVOC_XXX macros accordingly.
> >
> > Signed-off-by: Sumit Garg <sumit.garg at nxp.com>
> > ---
> >  include/fsl_sfp.h | 23 +++++++++++++++--------
> >  1 file changed, 15 insertions(+), 8 deletions(-)
> >
> > diff --git a/include/fsl_sfp.h b/include/fsl_sfp.h index
> > 2976a2c..42ff942 100644
> > --- a/include/fsl_sfp.h
> > +++ b/include/fsl_sfp.h
> > @@ -32,7 +32,7 @@
> >  /* Number of SRKH registers */
> >  #define NUM_SRKH_REGS	8
> >
> > -#if	defined(CONFIG_SYS_FSL_SFP_VER_3_2) ||	\
> > +#if defined(CONFIG_SYS_FSL_SFP_VER_3_2) ||	\
> >  	defined(CONFIG_SYS_FSL_SFP_VER_3_4)
> >  struct ccsr_sfp_regs {
> >  	u32 ospr;		/* 0x200 */
> > @@ -57,7 +57,7 @@ struct ccsr_sfp_regs {
> >  	u32 srk_hash[NUM_SRKH_REGS];	/* 0x23c Super Root Key Hash
> */
> >  	u32 oem_uid;		/* 0x9c OEM Unique ID */
> >  	u8 reserved2[0x04];
> > -	u32 ovpr;			/* 0xA4  Intent To Secure */
> > +	u32 ovpr;		/* 0xA4  Intent To Secure */
> 
> Please don't mix whitespace change in this patch.
> 
> York

Should I split this patch  and make a separate patch for the cleanup?

-Sumit