Re: [U-Boot] [PATCH 1/3] ARM: i.MX6: exclude the ARM errata from i.MX6 UP system

2017-08-16 Thread Stefano Babic 
On 08/08/2017 07:34, Peng Fan wrote:
> The ARM errata 751472, 794072, 761320 only applied
> to the following configuration:
> 
> This erratum affects configurations with either:
> - One processor if the ACP is present
> - Two or more processors
> 
> i.MX6 family does not have the ACP and thus only the MPCore system
> will be impacted, which are the i.MX6DQ, i.MX6DL, and i.MX6QP.
> 
> Signed-off-by: Peng Fan 
> Cc: Stefano Babic 
> Cc: Fabio Estevam 
> ---
All 3 patches are assigned to me - the second patch with the
work-arouund is really in Albert's area of competence, but as the
patchset (the 3rd one activates it for MX6) is most for i.MX6, I hope it
is ok if I apply it to u-boot-imx (else raise a hand, I will revert them !).

Applied to u-boot-imx, -master, thanks !

Best regards,
Stefano Babic


-- 
=
DENX Software Engineering GmbH,  Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 1/3] ARM: i.MX6: exclude the ARM errata from i.MX6 UP system

2017-08-08 Thread Peng Fan 


> -Original Message-
> From: Stefano Babic [mailto:sba...@denx.de]
> Sent: Tuesday, August 08, 2017 4:50 PM
> To: Peng Fan ; sba...@denx.de
> Cc: van.free...@gmail.com; u-boot@lists.denx.de; Fabio Estevam
> 
> Subject: Re: [PATCH 1/3] ARM: i.MX6: exclude the ARM errata from i.MX6 UP
> system
> 
> Hi Peng,
> 
> On 08/08/2017 07:34, Peng Fan wrote:
> > The ARM errata 751472, 794072, 761320 only applied to the following
> > configuration:
> >
> > This erratum affects configurations with either:
> > - One processor if the ACP is present
> > - Two or more processors
> >
> 
> Apart that I agree that such as work-around should be applied just to the
> affected SOCs, which is the issues to apply them to the whole MX6 family ?
> 
> For example, 751472 requires to set CP15 bit 11 with a specific sequence. 
> Right,
> single core MX6 do not need it, but even if this is set in this way, is there 
> a side-
> effect ? Just curious...

I have no idea about there is side effect if applying them to single core mx6.
But we better apply them only affected SoCs.

Regards,
Peng.
> 
> > i.MX6 family does not have the ACP and thus only the MPCore system
> > will be impacted, which are the i.MX6DQ, i.MX6DL, and i.MX6QP.
> >
> > Signed-off-by: Peng Fan 
> > Cc: Stefano Babic 
> > Cc: Fabio Estevam 
> > ---
> >  arch/arm/mach-imx/mx6/Kconfig | 15 +++
> >  1 file changed, 11 insertions(+), 4 deletions(-)
> >
> > diff --git a/arch/arm/mach-imx/mx6/Kconfig
> > b/arch/arm/mach-imx/mx6/Kconfig index bb2ce33..2be67e1 100644
> > --- a/arch/arm/mach-imx/mx6/Kconfig
> > +++ b/arch/arm/mach-imx/mx6/Kconfig
> > @@ -1,24 +1,31 @@
> >  if ARCH_MX6
> >
> > +config MX6_SMP
> > +   select ARM_ERRATA_751472
> > +   select ARM_ERRATA_761320
> > +   select ARM_ERRATA_794072
> > +   bool
> > +
> >  config MX6
> > +   select ARM_ERRATA_743622 if !MX6UL
> > bool
> > default y
> > -   select ARM_ERRATA_743622 if !MX6UL
> > -   select ARM_ERRATA_751472 if !MX6UL
> > -   select ARM_ERRATA_761320 if !MX6UL
> > -   select ARM_ERRATA_794072 if !MX6UL
> > imply CMD_FUSE
> >
> >  config MX6D
> > +   select MX6_SMP
> > bool
> >
> >  config MX6DL
> > +   select MX6_SMP
> > bool
> >
> >  config MX6Q
> > +   select MX6_SMP
> > bool
> >
> >  config MX6QDL
> > +   select MX6_SMP
> > bool
> >
> >  config MX6S
> >
> 
> Best regards,
> Stefano
> 
> --
> 
> =
> DENX Software Engineering GmbH,  Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
> 
> =
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 1/3] ARM: i.MX6: exclude the ARM errata from i.MX6 UP system

2017-08-08 Thread Stefano Babic 
Hi Peng,

On 08/08/2017 07:34, Peng Fan wrote:
> The ARM errata 751472, 794072, 761320 only applied
> to the following configuration:
> 
> This erratum affects configurations with either:
> - One processor if the ACP is present
> - Two or more processors
> 

Apart that I agree that such as work-around should be applied just to
the affected SOCs, which is the issues to apply them to the whole MX6
family ?

For example, 751472 requires to set CP15 bit 11 with a specific
sequence. Right, single core MX6 do not need it, but even if this is set
in this way, is there a side-effect ? Just curious...

> i.MX6 family does not have the ACP and thus only the MPCore system
> will be impacted, which are the i.MX6DQ, i.MX6DL, and i.MX6QP.
> 
> Signed-off-by: Peng Fan 
> Cc: Stefano Babic 
> Cc: Fabio Estevam 
> ---
>  arch/arm/mach-imx/mx6/Kconfig | 15 +++
>  1 file changed, 11 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm/mach-imx/mx6/Kconfig b/arch/arm/mach-imx/mx6/Kconfig
> index bb2ce33..2be67e1 100644
> --- a/arch/arm/mach-imx/mx6/Kconfig
> +++ b/arch/arm/mach-imx/mx6/Kconfig
> @@ -1,24 +1,31 @@
>  if ARCH_MX6
>  
> +config MX6_SMP
> + select ARM_ERRATA_751472
> + select ARM_ERRATA_761320
> + select ARM_ERRATA_794072
> + bool
> +
>  config MX6
> + select ARM_ERRATA_743622 if !MX6UL
>   bool
>   default y
> - select ARM_ERRATA_743622 if !MX6UL
> - select ARM_ERRATA_751472 if !MX6UL
> - select ARM_ERRATA_761320 if !MX6UL
> - select ARM_ERRATA_794072 if !MX6UL
>   imply CMD_FUSE
>  
>  config MX6D
> + select MX6_SMP
>   bool
>  
>  config MX6DL
> + select MX6_SMP
>   bool
>  
>  config MX6Q
> + select MX6_SMP
>   bool
>  
>  config MX6QDL
> + select MX6_SMP
>   bool
>  
>  config MX6S
> 

Best regards,
Stefano

-- 
=
DENX Software Engineering GmbH,  Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot