[edk2] [PATCH edk2-platforms 12/41] LS1043/BoardLib : Add support for LS1043 BoardLib.

Leif Lindholm leif.lindholm at linaro.org
Tue Dec 18 10:50:01 PST 2018


On Wed, Nov 28, 2018 at 08:31:26PM +0530, Meenakshi Aggarwal wrote:
> BoardLib will contain functions specific for LS1043aRdb board.
> 
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Meenakshi Aggarwal <meenakshi.aggarwal at nxp.com>
> Reviewed-by: Leif Lindholm <leif.lindholm at linaro.org>

Sorry, I do have a few more (trivial) comments.
But feel free to keep the reviewed-by if you address them.

> ---
>  .../NXP/LS1043aRdbPkg/Include/IfcBoardSpecific.h   | 109 +++++++++++++++++++++

Ifc->NxpIfc?

>  .../NXP/LS1043aRdbPkg/Library/BoardLib/BoardLib.c  |  69 +++++++++++++
>  .../LS1043aRdbPkg/Library/BoardLib/BoardLib.inf    |  31 ++++++
>  3 files changed, 209 insertions(+)
>  create mode 100644 Platform/NXP/LS1043aRdbPkg/Include/IfcBoardSpecific.h
>  create mode 100644 Platform/NXP/LS1043aRdbPkg/Library/BoardLib/BoardLib.c
>  create mode 100644 Platform/NXP/LS1043aRdbPkg/Library/BoardLib/BoardLib.inf
> 
> diff --git a/Platform/NXP/LS1043aRdbPkg/Include/IfcBoardSpecific.h b/Platform/NXP/LS1043aRdbPkg/Include/IfcBoardSpecific.h
> new file mode 100644
> index 0000000..261867a
> --- /dev/null
> +++ b/Platform/NXP/LS1043aRdbPkg/Include/IfcBoardSpecific.h
> @@ -0,0 +1,109 @@
> +/** IfcBoardSpecificLib.h
> +
> +  IFC Flash Board Specific Macros and structure
> +
> +  Copyright 2017 NXP
> +
> +  This program and the accompanying materials
> +  are licensed and made available under the terms and conditions of the BSD License
> +  which accompanies this distribution. The full text of the license may be found at
> +  http://opensource.org/licenses/bsd-license.php
> +
> +  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
> +  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
> +
> +**/
> +#ifndef __IFC__BOARD_SPECIFIC_H__
> +#define __IFC__BOARD_SPECIFIC_H__

NXP_ prefix?

> +
> +#include <Ifc.h>
> +
> +// On board flash support
> +#define IFC_NAND_BUF_BASE    0x7E800000
> +
> +// On board Inegrated flash Controller chip select configuration
> +#define IFC_NOR_CS    IFC_CS0
> +#define IFC_NAND_CS   IFC_CS1
> +#define IFC_FPGA_CS   IFC_CS2
> +
> +// board-specific NAND timing
> +#define NAND_FTIM0    (IFC_FTIM0_NAND_TCCST(0x7) | \
> +                      IFC_FTIM0_NAND_TWP(0x18)   | \
> +                      IFC_FTIM0_NAND_TWCHT(0x7) | \
> +                      IFC_FTIM0_NAND_TWH(0xa))
> +
> +#define NAND_FTIM1    (IFC_FTIM1_NAND_TADLE(0x32) | \
> +                      IFC_FTIM1_NAND_TWBE(0x39)  | \
> +                      IFC_FTIM1_NAND_TRR(0xe)   | \
> +                      IFC_FTIM1_NAND_TRP(0x18))
> +
> +#define NAND_FTIM2    (IFC_FTIM2_NAND_TRAD(0xf) | \
> +                      IFC_FTIM2_NAND_TREH(0xa) | \
> +                      IFC_FTIM2_NAND_TWHRE(0x1e))
> +
> +#define NAND_FTIM3    0x0
> +
> +#define NAND_CSPR   (IFC_CSPR_PHYS_ADDR(IFC_NAND_BUF_BASE) \
> +                            | IFC_CSPR_PORT_SIZE_8 \
> +                            | IFC_CSPR_MSEL_NAND \
> +                            | IFC_CSPR_V)
> +
> +#define NAND_CSPR_EXT   0x0
> +#define NAND_AMASK      0xFFFF0000
> +
> +#define NAND_CSOR     (IFC_CSOR_NAND_ECC_ENC_EN /* ECC on encode */ \
> +                      | IFC_CSOR_NAND_ECC_DEC_EN /* ECC on decode */ \
> +                      | IFC_CSOR_NAND_ECC_MODE_4 /* 4-bit ECC */ \
> +                      | IFC_CSOR_NAND_RAL_3       /* RAL = 3 Bytes */ \
> +                      | IFC_CSOR_NAND_PGS_2K      /* Page Size = 2K */ \
> +                      | IFC_CSOR_NAND_SPRZ_64     /* Spare size = 64 */ \
> +                      | IFC_CSOR_NAND_PB(6))     /* 2^6 Pages Per Block */
> +
> +// board-specific NOR timing
> +#define NOR_FTIM0     (IFC_FTIM0_NOR_TACSE(0x1) | \
> +                      IFC_FTIM0_NOR_TEADC(0x1) | \
> +                      IFC_FTIM0_NOR_TAVDS(0x0) | \
> +                      IFC_FTIM0_NOR_TEAHC(0xc))
> +#define NOR_FTIM1     (IFC_FTIM1_NOR_TACO(0x1c) | \
> +                      IFC_FTIM1_NOR_TRAD_NOR(0xb) |\
> +                      IFC_FTIM1_NOR_TSEQRAD_NOR(0x9))
> +#define NOR_FTIM2     (IFC_FTIM2_NOR_TCS(0x1) | \
> +                      IFC_FTIM2_NOR_TCH(0x4) | \
> +                      IFC_FTIM2_NOR_TWPH(0x8) | \
> +                      IFC_FTIM2_NOR_TWP(0x10))
> +#define NOR_FTIM3     0x0
> +
> +#define NOR_CSPR      (IFC_CSPR_PHYS_ADDR(FixedPcdGet64 (PcdIfcRegion1BaseAddr)) \
> +                      | IFC_CSPR_PORT_SIZE_16 \
> +                      | IFC_CSPR_MSEL_NOR        \
> +                      | IFC_CSPR_V)
> +
> +#define NOR_CSPR_EXT  0x0
> +#define NOR_AMASK     IFC_AMASK(128*1024*1024)
> +#define NOR_CSOR      (IFC_CSOR_NOR_ADM_SHIFT(4) | \
> +                      IFC_CSOR_NOR_TRHZ_80)
> +
> +// board-specific fpga timing
> +#define FPGA_BASE_PHYS  0x7fb00000
> +#define FPGA_CSPR_EXT   0x0
> +#define FPGA_CSPR       (IFC_CSPR_PHYS_ADDR(FPGA_BASE_PHYS) | \
> +                        IFC_CSPR_PORT_SIZE_8 | \
> +                        IFC_CSPR_MSEL_GPCM | \
> +                        IFC_CSPR_V)
> +
> +#define FPGA_AMASK      IFC_AMASK(64 * 1024)
> +#define FPGA_CSOR       (IFC_CSOR_NOR_ADM_SHIFT(4) | \
> +                        IFC_CSOR_NOR_NOR_MODE_AVD_NOR | \
> +                        IFC_CSOR_NOR_TRHZ_80)
> +
> +#define FPGA_FTIM0      (IFC_FTIM0_GPCM_TACSE(0xf) | \
> +                        IFC_FTIM0_GPCM_TEADC(0xf) | \
> +                        IFC_FTIM0_GPCM_TEAHC(0xf))
> +#define FPGA_FTIM1      (IFC_FTIM1_GPCM_TACO(0xff) | \
> +                        IFC_FTIM1_GPCM_TRAD(0x3f))
> +#define FPGA_FTIM2      (IFC_FTIM2_GPCM_TCS(0xf) | \
> +                        IFC_FTIM2_GPCM_TCH(0xf) | \
> +                        IFC_FTIM2_GPCM_TWP(0xff))
> +#define FPGA_FTIM3      0x0
> +
> +#endif //__IFC__BOARD_SPECIFIC_H__
> diff --git a/Platform/NXP/LS1043aRdbPkg/Library/BoardLib/BoardLib.c b/Platform/NXP/LS1043aRdbPkg/Library/BoardLib/BoardLib.c
> new file mode 100644
> index 0000000..a101a8d
> --- /dev/null
> +++ b/Platform/NXP/LS1043aRdbPkg/Library/BoardLib/BoardLib.c
> @@ -0,0 +1,69 @@
> +/** @file
> +
> +  Copyright 2018 NXP
> +
> +  This program and the accompanying materials
> +  are licensed and made available under the terms and conditions of the BSD License
> +  which accompanies this distribution.  The full text of the license may be found at
> +  http://opensource.org/licenses/bsd-license.php
> +
> +  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
> +  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
> +
> +**/
> +
> +#include <IfcBoardSpecific.h>
> +
> +VOID
> +GetIfcNorFlashTimings (
> +  IN IFC_TIMINGS * NorIfcTimings
> +  )
> +{
> +  NorIfcTimings->Ftim[0] = NOR_FTIM0;
> +  NorIfcTimings->Ftim[1] = NOR_FTIM1;
> +  NorIfcTimings->Ftim[2] = NOR_FTIM2;
> +  NorIfcTimings->Ftim[3] = NOR_FTIM3;
> +  NorIfcTimings->Cspr = NOR_CSPR;
> +  NorIfcTimings->CsprExt = NOR_CSPR_EXT;
> +  NorIfcTimings->Amask = NOR_AMASK;
> +  NorIfcTimings->Csor = NOR_CSOR;
> +  NorIfcTimings->CS = IFC_NOR_CS;
> +
> +  return ;

Please drop the space before ;.

/
    Leif

> +}
> +
> +VOID
> +GetIfcFpgaTimings (
> +  IN IFC_TIMINGS  *FpgaIfcTimings
> +  )
> +{
> +  FpgaIfcTimings->Ftim[0] = FPGA_FTIM0;
> +  FpgaIfcTimings->Ftim[1] = FPGA_FTIM1;
> +  FpgaIfcTimings->Ftim[2] = FPGA_FTIM2;
> +  FpgaIfcTimings->Ftim[3] = FPGA_FTIM3;
> +  FpgaIfcTimings->Cspr = FPGA_CSPR;
> +  FpgaIfcTimings->CsprExt = FPGA_CSPR_EXT;
> +  FpgaIfcTimings->Amask = FPGA_AMASK;
> +  FpgaIfcTimings->Csor = FPGA_CSOR;
> +  FpgaIfcTimings->CS = IFC_FPGA_CS;
> +
> +  return;
> +}
> +
> +VOID
> +GetIfcNandFlashTimings (
> +  IN IFC_TIMINGS * NandIfcTimings
> +  )
> +{
> +  NandIfcTimings->Ftim[0] = NAND_FTIM0;
> +  NandIfcTimings->Ftim[1] = NAND_FTIM1;
> +  NandIfcTimings->Ftim[2] = NAND_FTIM2;
> +  NandIfcTimings->Ftim[3] = NAND_FTIM3;
> +  NandIfcTimings->Cspr = NAND_CSPR;
> +  NandIfcTimings->CsprExt = NAND_CSPR_EXT;
> +  NandIfcTimings->Amask = NAND_AMASK;
> +  NandIfcTimings->Csor = NAND_CSOR;
> +  NandIfcTimings->CS = IFC_NAND_CS;
> +
> +  return;
> +}
> diff --git a/Platform/NXP/LS1043aRdbPkg/Library/BoardLib/BoardLib.inf b/Platform/NXP/LS1043aRdbPkg/Library/BoardLib/BoardLib.inf
> new file mode 100644
> index 0000000..7d2702b
> --- /dev/null
> +++ b/Platform/NXP/LS1043aRdbPkg/Library/BoardLib/BoardLib.inf
> @@ -0,0 +1,31 @@
> +#  @file
> +#
> +#  Copyright 2018 NXP
> +#
> +#  This program and the accompanying materials
> +#  are licensed and made available under the terms and conditions of the BSD License
> +#  which accompanies this distribution.  The full text of the license may be found at
> +#  http://opensource.org/licenses/bsd-license.php
> +#
> +#  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
> +#  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
> +#
> +
> +[Defines]
> +  INF_VERSION                    = 0x0001001A
> +  BASE_NAME                      = BoardLib
> +  FILE_GUID                      = 8ecefc8f-a2c4-4091-b80f-92da7c4ab37f
> +  MODULE_TYPE                    = BASE
> +  VERSION_STRING                 = 1.0
> +  LIBRARY_CLASS                  = BoardLib
> +
> +[Sources.common]
> +  BoardLib.c
> +
> +[Packages]
> +  MdePkg/MdePkg.dec
> +  Platform/NXP/LS1043aRdbPkg/LS1043aRdbPkg.dec
> +  Silicon/NXP/NxpQoriqLs.dec
> +
> +[FixedPcd]
> +  gNxpQoriqLsTokenSpaceGuid.PcdIfcRegion1BaseAddr
> -- 
> 1.9.1
> 


More information about the edk2-devel mailing list