> -----Original Message----- > From: York Sun [mailto:york...@freescale.com] > Sent: Tuesday, November 24, 2015 11:14 AM > To: Kushwaha Prabhakar-B32579 <prabha...@freescale.com>; u- > b...@lists.denx.de > Cc: Srivastava Pratiyush-B46174 <pratiyush.srivast...@freescale.com> > Subject: Re: [PATCH 1/2][v2] armv8: LS2080A: Rename LS2085A to reflect > LS2080A > > > > On 11/23/2015 08:25 PM, Kushwaha Prabhakar-B32579 wrote: > > > >> -----Original Message----- > >> From: York Sun [mailto:york...@freescale.com] > >> Sent: Tuesday, November 24, 2015 12:49 AM > >> To: Kushwaha Prabhakar-B32579 <prabha...@freescale.com>; u- > >> b...@lists.denx.de > >> Cc: Srivastava Pratiyush-B46174 <pratiyush.srivast...@freescale.com> > >> Subject: Re: [PATCH 1/2][v2] armv8: LS2080A: Rename LS2085A to > >> reflect LS2080A > >> > >> > >> > >> On 11/09/2015 03:12 AM, Prabhakar Kushwaha wrote: > >>> LS2080A is a prime personality of Freescale’s LS2085A. It is a > >>> non-AIOP personality without support of DP-DDR, L2 switch, 1588, > >>> PCIe > >> endpoint etc. > >>> So renaming existing LS2085A code base to reflect LS2080A (Prime > >>> personality) > >>> > >>> Signed-off-by: Pratiyush Mohan Srivastava > >>> <pratiyush.srivast...@freescale.com> > >>> Signed-off-by: Prabhakar Kushwaha <prabha...@freescale.com> > >>> --- > >>> Changes for v2: Rebased on LS1043 patch set > >> > >> <snip> > >> > >>> diff --git a/arch/arm/include/asm/arch-fsl-layerscape/cpu.h > >>> b/arch/arm/include/asm/arch-fsl-layerscape/cpu.h > >>> index 2903996..5a91dcb 100644 > >>> --- a/arch/arm/include/asm/arch-fsl-layerscape/cpu.h > >>> +++ b/arch/arm/include/asm/arch-fsl-layerscape/cpu.h > >>> @@ -8,8 +8,10 @@ > >>> #define _FSL_LAYERSCAPE_CPU_H > >>> > >>> static struct cpu_type cpu_type_list[] = { > >>> - CPU_TYPE_ENTRY(LS2085, LS2085, 8), > >>> +#ifdef CONFIG_LS2080A > >>> CPU_TYPE_ENTRY(LS2080, LS2080, 8), > >>> +#endif > >>> + CPU_TYPE_ENTRY(LS2085, LS2085, 8), > >>> CPU_TYPE_ENTRY(LS2045, LS2045, 4), > >>> CPU_TYPE_ENTRY(LS1043, LS1043, 4), }; > >> > >> Why using ifdef here? > > > > I am trying to reduce size of table. > > Instead of adding all SoC in table only defined the required one at a time. > > I thought the plan was to support both LS2080 and LS2085 in one image built > for LS2080. We have always used a complete table for cpu list. If you are > flexible, I will drop the ifdef when merging this patch. >
Yes, we can drop the ifdef. --prabhakar _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot