From mboxrd@z Thu Jan 1 00:00:00 1970 From: Scott Wood Date: Fri, 5 Apr 2013 15:33:53 -0500 Subject: [U-Boot] [PATCH 1/5][v2] powerpc/mpc85xx:No NOR boot, do not compile IFC errata A003399 In-Reply-To: <1363936068-17107-1-git-send-email-prabhakar@freescale.com> (from prabhakar@freescale.com on Fri Mar 22 02:07:48 2013) References: <1363936068-17107-1-git-send-email-prabhakar@freescale.com> Message-ID: <1365194033.17535.9@snotra> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 03/22/2013 02:07:48 AM, Prabhakar Kushwaha wrote: > IFC errata A003399 is valid for IFC NOR boot i.e.if no on-board NOR > flash or > no NOR boot, do not compile its workaround. > > Signed-off-by: Prabhakar Kushwaha > --- > Based upon git://git.denx.de/u-boot.git branch master > Changes for v2: > - Add CONFIG_A003399_NOR_WORKAROUND for errata workaround > - Add CONFIG_A003399_NOR_WORKAROUND in README > > README | 4 ++++ > arch/powerpc/cpu/mpc85xx/cpu_init_early.c | 9 ++++++--- > arch/powerpc/cpu/mpc8xxx/fsl_ifc.c | 2 +- > 3 files changed, 11 insertions(+), 4 deletions(-) > > diff --git a/README b/README > index 7f2506a..c4ad69d 100644 > --- a/README > +++ b/README > @@ -414,6 +414,10 @@ The following options need to be configured: > See Freescale App Note 4493 for more information about > this erratum. > > + CONFIG_A003399_NOR_WORKAROUND > + Enables a workaround for IFC erratum A003399. It is only > + requred during NOR boot. > + > CONFIG_SYS_FSL_CORENET_SNOOPVEC_COREONLY > > This is the value to write into CCSR offset 0x18600 > diff --git a/arch/powerpc/cpu/mpc85xx/cpu_init_early.c > b/arch/powerpc/cpu/mpc85xx/cpu_init_early.c > index dacfdd1..3a4bfc5 100644 > --- a/arch/powerpc/cpu/mpc85xx/cpu_init_early.c > +++ b/arch/powerpc/cpu/mpc85xx/cpu_init_early.c > @@ -25,7 +25,8 @@ > > DECLARE_GLOBAL_DATA_PTR; > > -#if defined(CONFIG_SYS_FSL_ERRATUM_IFC_A003399) && > !defined(CONFIG_SYS_RAMBOOT) > +#if defined(CONFIG_SYS_FSL_ERRATUM_IFC_A003399)\ > + && defined(CONFIG_A003399_NOR_WORKAROUND) Why would CONFIG_A003399_NOR_WORKAROUND be enabled if CONFIG_SYS_FSL_ERRATUM_IFC_A003399 isn't? -Scott