[U-Boot] [PATCH v1] PPC405EX CHIP_21 erratum

Eibach, Dirk Eibach at gdsys.de
Wed May 4 10:03:54 CEST 2011


Thanks, I can confirm this fixes the issue for me (405EX Rev. D with
security).

Still checkpatch finds some whitespace issues:
.dotest/patch:13: space before tab in indent.
       	defined(CONFIG_405EX_CHIP21_PVR_REV_D) && \
.dotest/patch:27: trailing whitespace.
	 * 
warning: 2 lines add whitespace errors.

> diff --git a/include/configs/kilauea.h 
> b/include/configs/kilauea.h index 031f8fb..2d3efba 100644
> --- a/include/configs/kilauea.h
> +++ b/include/configs/kilauea.h
> @@ -44,6 +44,17 @@
>  #endif
>  
>  /*
> + * CHIP_21 errata
> + */
> +//#define CONFIG_405EX_CHIP21_PVR_REV_C		
> 0x1291147f /* EX with security */
> +//#define CONFIG_405EX_CHIP21_PVR_REV_D		
> 0x12911475 /* EX with security */
> +//#define CONFIG_405EX_CHIP21_ECID3_REV_D		0x0	
>    /* EX with security */
> +
> +#define CONFIG_405EX_CHIP21_PVR_REV_C		
> 0x1291147d /* EX without security */
> +#define CONFIG_405EX_CHIP21_PVR_REV_D		
> 0x12911473 /* EX without security */
> +#define CONFIG_405EX_CHIP21_ECID3_REV_D		0x1	
>    /* EX without security */

I don't like defining expected PVR values in the board config file.
Maybe something like CONFIG_405EX_CHIP21_SECURITY and
CONFIG_405EX_CHIP21_NO_SECURITY would be more pleasant.

Then you could place

#ifdef CONFIG_405EX_CHIP21_SECURITY
#define CONFIG_405EX_CHIP21_PVR_REV_C		0x1291147f /* EX with
security */
#define CONFIG_405EX_CHIP21_PVR_REV_D		0x12911475 /* EX with
security */
#define CONFIG_405EX_CHIP21_ECID3_REV_D		0x0	   /* EX with
security */
#endif

#ifdef CONFIG_405EX_CHIP21_NO_SECURITY
#define CONFIG_405EX_CHIP21_PVR_REV_C		0x1291147d /* EX without
security */
#define CONFIG_405EX_CHIP21_PVR_REV_D		0x12911473 /* EX without
security */
#define CONFIG_405EX_CHIP21_ECID3_REV_D		0x1	   /* EX without
security */
#endif

in a more appropriate place.

Cheers
Dirk




More information about the U-Boot mailing list