[U-Boot] [PATCH V2 8/9] configs: ti_omap3_common: Enable workaround for ARM errata 454179, 430973, 621766

[Igor Grinberg <grinberg@compulab.co.il>]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 02/25/15 13:19, Paul Kocialkowski wrote:
> Le mardi 24 f?vrier 2015 ? 16:57 -0600, Nishanth Menon a ?crit :
>> Enable the OMAP3 specific errata code for 454179, 430973, 621766
>> and while at it, remove legacy non-revision checked errata logic.
>>
>> Signed-off-by: Nishanth Menon <nm@ti.com>
>> ---
>>  arch/arm/cpu/armv7/omap3/board.c  |   16 ----------------
>>  include/configs/ti_omap3_common.h |    6 ++++++
>>  2 files changed, 6 insertions(+), 16 deletions(-)
>>
>> diff --git a/arch/arm/cpu/armv7/omap3/board.c b/arch/arm/cpu/armv7/omap3/board.c
>> index 7ce30949a6c6..cc3a43341335 100644
>> --- a/arch/arm/cpu/armv7/omap3/board.c
>> +++ b/arch/arm/cpu/armv7/omap3/board.c
>> @@ -35,7 +35,6 @@ DECLARE_GLOBAL_DATA_PTR;
>>  
>>  /* Declarations */
>>  extern omap3_sysinfo sysinfo;
>> -static void omap3_setup_aux_cr(void);
>>  #ifndef CONFIG_SYS_L2CACHE_OFF
>>  static void omap3_invalidate_l2_cache_secure(void);
>>  #endif
>> @@ -246,9 +245,6 @@ void s_init(void)
>>  
>>  	try_unlock_memory();
>>  
>> -	/* Errata workarounds */
>> -	omap3_setup_aux_cr();
>> -
>>  #ifndef CONFIG_SYS_L2CACHE_OFF
>>  	/* Invalidate L2-cache from secure mode */
>>  	omap3_invalidate_l2_cache_secure();
>> @@ -428,18 +424,6 @@ void omap3_update_aux_cr_secure(u32 set_bits, u32 clear_bits)
>>  	}
>>  }
>>  
>> -static void omap3_setup_aux_cr(void)
>> -{
>> -	/* Workaround for Cortex-A8 errata: #454179 #430973
>> -	 *	Set "IBE" bit
>> -	 *	Set "Disable Branch Size Mispredicts" bit
>> -	 * Workaround for erratum #621766
>> -	 *	Enable L1NEON bit
>> -	 * ACR |= (IBE | DBSM | L1NEON) => ACR |= 0xE0
>> -	 */
>> -	omap3_update_aux_cr_secure(0xE0, 0);
>> -}
>> -
>>  #ifndef CONFIG_SYS_L2CACHE_OFF
>>  static void omap3_update_aux_cr(u32 set_bits, u32 clear_bits)
>>  {
>> diff --git a/include/configs/ti_omap3_common.h b/include/configs/ti_omap3_common.h
>> index f909f6b94a18..db15c12ddf53 100644
>> --- a/include/configs/ti_omap3_common.h
>> +++ b/include/configs/ti_omap3_common.h
> 
> Only a couple of OMAP3 boards in U-Boot include ti_omap3_common.h. You
> can find a complete list of all the configs that need those options
> enabled from my first patch on that topic:
> http://lists.denx.de/pipermail/u-boot/2015-February/205809.html
> 
> I guess it's better to enable those on all the currently-supported omap3
> platforms: that's the way it was so far and it won't hurt in cases it's
> not needed thanks to the revision detection.

Or...
I think this patch might be a good enough argument to request people
switch to include ti_omap3_common.h instead of duplicating it in
private configs...

Don't you think?

> 
>> @@ -27,6 +27,12 @@
>>  # define CONFIG_SYS_MALLOC_F_LEN	(1 << 10)
>>  #endif
>>  
>> +/* Common ARM Erratas */
>> +#define CONFIG_ARM_ARCH_CP15_ERRATA
>> +#define CONFIG_ARM_ERRATA_454179
>> +#define CONFIG_ARM_ERRATA_430973
>> +#define CONFIG_ARM_ERRATA_621766
>> +
>>  /* The chip has SDRC controller */
>>  #define CONFIG_SDRC
>>  
> 
> 
> 
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
> 

- -- 
Regards,
Igor.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQIcBAEBAgAGBQJU7bKeAAoJEBDE8YO64Efa/FUP/03ShzfpDHCl4aaSSoY8zLgl
K/C3eeK2EFKxS/CDCQCV4KGk/lXdIbmx2BHH9ErKXwUmMp63QZUo6EzLjks9CeKl
HZ9yBVxr9e3X2ztvZUpBWc5C6YpFxRfO9f5Q3GeLsv+217dccLRiomTVYlKLXQmJ
VUHofOfKlF5b0sQxcAI+wONNkhsSX5rVHmcm6NziC4w8ZedJy5zer6rTfMY54WPx
Z3jDKkvWK5DWOXNEpu0OVRC1RlKB6kqgedyaQTqwDLpL6Hz7oF3KeVJTKxbQgOPy
8vh2PeLX0AERL1hz6fEaG6pFnAT/V/4uEmczBAgAnCTr6NiCpC2QtlC1PAWTkg/c
4NPz/z5HM4o2cTpqI9Yc6oWExqZzANQLBJYtm8bD2pkUMA/RbVuLugnOka9trVFw
MHRqOBlAtq2tchvLZIhzQrh+bb5KsKWNFdoIkCFXUwYdqasMhxoW2G/4fDsf0oeL
q1vQV0LJp/qysLzz8u5603cFR7IvFKWSr4wP95YvlEhx6BGsKPXu0QeTVfpsPHZX
L4tP1X/wM10XGN2HGenz2FimtaUUJZTxHzj18ucMfyLs0hAfT981auyvyMLR5Up2
bFlB6b93y1BpM0DxpmgcuA77gLQL9jNKTyH76W/AQysOikSQ8JrSZg52zS3X0m6Q
QJ4jdRF+EAY/Eh/fIni0
=q25V
-----END PGP SIGNATURE-----