[PATCH 2/2] arm: layerscape: Disable erratum A009007 on LS1021A, LS1043A, and LS1046A

[Sean Anderson]

On 2/22/22 1:38 PM, Sean Anderson wrote:
> This erratum is reported to cause problems on these processors [1-3].
> The problem is usually with the clocking, which is supposed to be
> configured by the RCW [4]. However, if it is not set, or if the default
> clocking is not correct, then this erratum will cause an SError.
> However, according to Ran Wang in [1]:
> 
>> ... this erratum is used to pass USB compliance test only, you could
>> disable this workaround on your board if you don't any USB issue on
>> normal use case, I think it's fine.
> 
> So just disable this erratum by default for these processors.
> 
> [1] https://lore.kernel.org/all/761ddd61-05c1-d9b8-ac90-b8f425afde6c@denx.de/
> [2] https://community.nxp.com/t5/Layerscape/LS1046A-U-BOOT-HALT-AT-ERRATUM-A0090078/m-p/742993
> [3] https://community.nxp.com/t5/QorIQ/Why-does-the-LS1043A-U-Boot-hang-at-code-that-fixes-erratum/m-p/644412
> [4] https://source.codeaurora.org/external/qoriq/qoriq-components/rcw/tree/ls1046ardb/usb_phy_freq.rcw
> 
> Signed-off-by: Sean Anderson <sean.anderson at seco.com>
> ---
> 
>  arch/arm/cpu/armv7/ls102xa/Kconfig        | 1 -
>  arch/arm/cpu/armv8/fsl-layerscape/Kconfig | 2 --
>  2 files changed, 3 deletions(-)
> 
> diff --git a/arch/arm/cpu/armv7/ls102xa/Kconfig b/arch/arm/cpu/armv7/ls102xa/Kconfig
> index 6a948d7ba7..cec93a27db 100644
> --- a/arch/arm/cpu/armv7/ls102xa/Kconfig
> +++ b/arch/arm/cpu/armv7/ls102xa/Kconfig
> @@ -7,7 +7,6 @@ config ARCH_LS1021A
>  	select SYS_FSL_ERRATUM_A008407
>  	select SYS_FSL_ERRATUM_A008850
>  	select SYS_FSL_ERRATUM_A008997 if USB
> -	select SYS_FSL_ERRATUM_A009007 if USB
>  	select SYS_FSL_ERRATUM_A009008 if USB
>  	select SYS_FSL_ERRATUM_A009663
>  	select SYS_FSL_ERRATUM_A009798 if USB
> diff --git a/arch/arm/cpu/armv8/fsl-layerscape/Kconfig b/arch/arm/cpu/armv8/fsl-layerscape/Kconfig
> index 9bb870dcd8..f5a18053e6 100644
> --- a/arch/arm/cpu/armv8/fsl-layerscape/Kconfig
> +++ b/arch/arm/cpu/armv8/fsl-layerscape/Kconfig
> @@ -74,7 +74,6 @@ config ARCH_LS1043A
>  	select SYS_FSL_DDR_VER_50
>  	select SYS_FSL_ERRATUM_A008850 if !TFABOOT
>  	select SYS_FSL_ERRATUM_A008997
> -	select SYS_FSL_ERRATUM_A009007
>  	select SYS_FSL_ERRATUM_A009008
>  	select SYS_FSL_ERRATUM_A009660 if !TFABOOT
>  	select SYS_FSL_ERRATUM_A009663 if !TFABOOT
> @@ -112,7 +111,6 @@ config ARCH_LS1046A
>  	select SYS_FSL_ERRATUM_A008511 if !TFABOOT
>  	select SYS_FSL_ERRATUM_A008850 if !TFABOOT
>  	select SYS_FSL_ERRATUM_A008997
> -	select SYS_FSL_ERRATUM_A009007
>  	select SYS_FSL_ERRATUM_A009008
>  	select SYS_FSL_ERRATUM_A009798
>  	select SYS_FSL_ERRATUM_A009801
> 

ping? Can someone from NXP comment on this (either for or against)?

--Sean