[U-Boot] [PATCH] nxp/ls2088ardb: Add QSPI-boot support
Priyanka Jain
priyanka.jain at nxp.com
Thu Apr 13 07:29:05 UTC 2017
> -----Original Message-----
> From: U-Boot [mailto:u-boot-bounces at lists.denx.de] On Behalf Of Priyanka
> Jain
> Sent: Wednesday, April 12, 2017 1:53 PM
> To: york sun <york.sun at nxp.com>; u-boot at lists.denx.de
> Cc: Suresh Gupta <suresh.gupta at nxp.com>
> Subject: Re: [U-Boot] [PATCH] nxp/ls2088ardb: Add QSPI-boot support
>
>
>
> > -----Original Message-----
> > From: York Sun [mailto:york.sun at nxp.com]
> > Sent: Monday, April 10, 2017 10:30 PM
> > To: Priyanka Jain <priyanka.jain at nxp.com>; u-boot at lists.denx.de
> > Cc: Suresh Gupta <suresh.gupta at nxp.com>
> > Subject: Re: [PATCH] nxp/ls2088ardb: Add QSPI-boot support
> >
> > On 04/04/2017 11:01 PM, Priyanka Jain wrote:
> > > Signed-off-by: Priyanka Jain <priyanka.jain at nxp.com>
> > > Signed-off-by: Suresh Gupta <suresh.gupta at nxp.com>
> > > ---
> > > arch/arm/cpu/armv8/fsl-layerscape/Kconfig | 1 +
> > > arch/arm/dts/fsl-ls2080a-rdb.dts | 22 +++++++++++
> > > board/freescale/ls2080ardb/ls2080ardb.c | 23 +++++++++++-
> > > configs/ls2088ardb_qspi_defconfig | 41 ++++++++++++++++++++
> > > include/configs/ls2080a_common.h | 13 ++++++
> > > include/configs/ls2080aqds.h | 14 +------
> > > include/configs/ls2080ardb.h | 58
> ++++++++++++++++++++++++++++-
> > > 7 files changed, 157 insertions(+), 15 deletions(-) create mode
> > > 100644 configs/ls2088ardb_qspi_defconfig
> > >
> >
> > This patch needs more detail. There isn't a LS2088ARDB board. It is
> > still LS2080ARDB board with LS2088A SoC on it. How does it support
> > QSPI boot? The detail should be added in the README file.
> >
> I will update README for QSPI boot
> > > diff --git a/arch/arm/cpu/armv8/fsl-layerscape/Kconfig
> > > b/arch/arm/cpu/armv8/fsl-layerscape/Kconfig
> > > index b24462b..9bff57f 100644
> > > --- a/arch/arm/cpu/armv8/fsl-layerscape/Kconfig
> > > +++ b/arch/arm/cpu/armv8/fsl-layerscape/Kconfig
> > > @@ -160,6 +160,7 @@ endchoice
> > > config SYS_LS_PPA_FW_ADDR
> > > hex "Address of PPA firmware loading from"
> > > depends on FSL_LS_PPA
> > > + default 0x20a00000 if SYS_LS_PPA_FW_IN_XIP && QSPI_BOOT &&
> > > +ARCH_LS2080A
> > > default 0x40500000 if SYS_LS_PPA_FW_IN_XIP && QSPI_BOOT
> > > default 0x580a00000 if SYS_LS_PPA_FW_IN_XIP && ARCH_LS2080A
> > > default 0x60500000 if SYS_LS_PPA_FW_IN_XIP diff --git
> > > a/arch/arm/dts/fsl-ls2080a-rdb.dts
> > > b/arch/arm/dts/fsl-ls2080a-rdb.dts
> > > index 1a1813b..96dfafd 100644
> > > --- a/arch/arm/dts/fsl-ls2080a-rdb.dts
> > > +++ b/arch/arm/dts/fsl-ls2080a-rdb.dts
> > > @@ -15,6 +15,7 @@
> > > compatible = "fsl,ls2080a-rdb", "fsl,ls2080a";
> > >
> > > aliases {
> > > + spi0 = &qspi;
> > > spi1 = &dspi;
> > > };
> > > };
> > > @@ -33,3 +34,24 @@
> > > reg = <0>;
> > > };
> > > };
> > > +
> > > +&qspi {
> > > + bus-num = <0>;
> > > + status = "okay";
> > > +
> > > + qflash0: s25fs512s at 0 {
> > > + #address-cells = <1>;
> > > + #size-cells = <1>;
> > > + compatible = "spi-flash";
> > > + spi-max-frequency = <50000000>;
> > > + reg = <0>;
> > > + };
> > > +
> > > + qflash1: s25fs512s at 1 {
> > > + #address-cells = <1>;
> > > + #size-cells = <1>;
> > > + compatible = "spi-flash";
> > > + spi-max-frequency = <50000000>;
> > > + reg = <1>;
> > > + };
> > > +};
> > > diff --git a/board/freescale/ls2080ardb/ls2080ardb.c
> > > b/board/freescale/ls2080ardb/ls2080ardb.c
> > > index ea05ec6..ae0f44f 100644
> > > --- a/board/freescale/ls2080ardb/ls2080ardb.c
> > > +++ b/board/freescale/ls2080ardb/ls2080ardb.c
> > > @@ -57,12 +57,22 @@ unsigned long long get_qixis_addr(void)
> > >
> > > int checkboard(void)
> > > {
> > > +#ifndef CONFIG_FSL_QSPI
> > > u8 sw;
> > > +#endif
> > > char buf[15];
> > >
> > > cpu_name(buf);
> > > printf("Board: %s-RDB, ", buf);
> > >
> > > +/*
> > > + * If QSPI is enabled, CPLD needs to be accessed via I2C,
> > > + * but LS2080A/LS2088ARDB boards has limitation that it does
> > > + * not support CPLD access via I2C
> > > + */
> > > +#ifdef CONFIG_FSL_QSPI
> > > + printf("boot from QSPI\n");
> > > +#else
> > > sw = QIXIS_READ(arch);
> > > printf("Board Arch: V%d, ", sw >> 4);
> > > printf("Board version: %c, boot from ", (sw & 0xf) + 'A'); @@
> > > -74,11
> > > +84,13 @@ int checkboard(void)
> > > printf("vBank: %d\n", sw);
> > > else if (sw == 0x9)
> > > puts("NAND\n");
> > > + else if (sw == 0xf)
> > > + puts("QSPI\n");
> >
> > It is in the #else clause. It will never run, will it?
> >
> > <snip>
> I will remove this
> >
> > > diff --git a/include/configs/ls2080ardb.h
> > > b/include/configs/ls2080ardb.h index 2155a89..728c143 100644
> > > --- a/include/configs/ls2080ardb.h
> > > +++ b/include/configs/ls2080ardb.h
> > > @@ -32,7 +32,11 @@
> > > unsigned long get_board_sys_clk(void); #endif
> > >
> > > +#ifdef CONFIG_FSL_QSPI
> > > +#define CONFIG_SYS_CLK_FREQ 100000000
> > > +#else
> > > #define CONFIG_SYS_CLK_FREQ get_board_sys_clk()
> > > +#endif
> > > #define CONFIG_DDR_CLK_FREQ 133333333
> > > #define COUNTER_FREQUENCY_REAL
> (CONFIG_SYS_CLK_FREQ/4)
> > >
> > > @@ -116,7 +120,9 @@ unsigned long get_board_sys_clk(void);
> > > CONFIG_SYS_FLASH_BASE +
> > 0x40000000} #endif
> > >
> > > +#ifndef CONFIG_FSL_QSPI
> > > #define CONFIG_NAND_FSL_IFC
> > > +#endif
> > > #define CONFIG_SYS_NAND_MAX_ECCPOS 256
> > > #define CONFIG_SYS_NAND_MAX_OOBFREE 2
> > >
> > > @@ -154,7 +160,9 @@ unsigned long get_board_sys_clk(void);
> > > #define CONFIG_SYS_NAND_BASE_LIST { CONFIG_SYS_NAND_BASE }
> > > #define CONFIG_SYS_MAX_NAND_DEVICE 1
> > > #define CONFIG_MTD_NAND_VERIFY_WRITE
> > > +#ifndef CONFIG_FSL_QSPI
> > > #define CONFIG_CMD_NAND
> > > +#endif
> > >
> > > #define CONFIG_SYS_NAND_BLOCK_SIZE (512 * 1024)
> > >
> > > @@ -165,12 +173,14 @@ unsigned long get_board_sys_clk(void);
> > > #define QIXIS_LBMAP_DFLTBANK 0x00
> > > #define QIXIS_LBMAP_ALTBANK 0x04
> > > #define QIXIS_LBMAP_NAND 0x09
> > > +#define QIXIS_LBMAP_QSPI 0x0f
> > > #define QIXIS_RST_CTL_RESET 0x31
> > > #define QIXIS_RST_CTL_RESET_EN 0x30
> > > #define QIXIS_RCFG_CTL_RECONFIG_IDLE 0x20
> > > #define QIXIS_RCFG_CTL_RECONFIG_START 0x21
> > > #define QIXIS_RCFG_CTL_WATCHDOG_ENBLE 0x08
> > > #define QIXIS_RCW_SRC_NAND 0x119
> > > +#define QIXIS_RCW_SRC_QSPI 0x62
> > > #define QIXIS_RST_FORCE_MEM 0x01
> > >
> > > #define CONFIG_SYS_CSPR3_EXT (0x0)
> > > @@ -241,11 +251,13 @@ unsigned long get_board_sys_clk(void);
> > > #define CONFIG_SYS_CS2_FTIM2 CONFIG_SYS_NAND_FTIM2
> > > #define CONFIG_SYS_CS2_FTIM3 CONFIG_SYS_NAND_FTIM3
> > >
> > > +#ifndef CONFIG_QSPI_BOOT
> > > #define CONFIG_ENV_IS_IN_FLASH
> > > #define CONFIG_ENV_ADDR (CONFIG_SYS_FLASH_BASE +
> > 0x200000)
> > > #define CONFIG_ENV_SECT_SIZE 0x20000
> > > #define CONFIG_ENV_SIZE 0x2000
> > > #endif
> > > +#endif
> > >
> > > /* Debug Server firmware */
> > > #define CONFIG_SYS_DEBUG_SERVER_FW_IN_NOR @@ -256,6 +268,7 @@
> > > unsigned long get_board_sys_clk(void);
> > > /*
> > > * I2C
> > > */
> > > +#define CONFIG_SYS_I2C_FPGA_ADDR 0x66
LS2080ARDB current u-boot code is closely tied with qixis code at many places.
And if try to decouple QIXIS code with u-boot code, it will take a lot of code changes.
So, I just bypassed the final QIXIS call keeping other code changes for Qixis access via I2C.
For customized boards, which uses RDB code as reference, having proper way of CPLD access via I2C will be useful.
Also the board issue of CPLD access via I2C is getting fixed in next similar board LS2081ARDB.
Please suggest how to proceed on this.
Priyanka
> >
> > What do you need this for? You said earlier CPLD cannot be accessed
> > from I2C bus.
> >
> Not needed will remove this
> > > #define I2C_MUX_PCA_ADDR 0x75
> > > #define I2C_MUX_PCA_ADDR_PRI 0x75 /* Primary Mux*/
> > >
> > > @@ -263,11 +276,18 @@ unsigned long get_board_sys_clk(void);
> > > #define I2C_MUX_CH_DEFAULT 0x8
> > >
> > > /* SPI */
> > > -#ifdef CONFIG_FSL_DSPI
> > > +#if defined(CONFIG_FSL_QSPI) || defined(CONFIG_FSL_DSPI)
> > > #define CONFIG_SPI_FLASH
> > > #define CONFIG_SPI_FLASH_BAR
> > > +#ifdef CONFIG_FSL_DSPI
> > > #define CONFIG_SPI_FLASH_STMICRO
> > > #endif
> > > +#ifdef CONFIG_FSL_QSPI
> > > +#define CONFIG_SPI_FLASH_SPANSION
> > > +#define FSL_QSPI_FLASH_SIZE (1 << 26) /* 64MB */
> > > +#define FSL_QSPI_FLASH_NUM 2
> > > +#endif
> > > +#endif
> > >
> > > /*
> > > * RTC configuration
> > > @@ -347,6 +367,29 @@ unsigned long get_board_sys_clk(void);
> > > " 0x580800000 \0" \
> > > BOOTENV
> > > #else
> > > +#ifdef CONFIG_QSPI_BOOT
> > > +#define CONFIG_EXTRA_ENV_SETTINGS \
> > > + "hwconfig=fsl_ddr:bank_intlv=auto\0" \
> > > + "scriptaddr=0x80800000\0" \
> > > + "kernel_addr_r=0x81000000\0" \
> > > + "pxefile_addr_r=0x81000000\0" \
> > > + "fdt_addr_r=0x88000000\0" \
> > > + "ramdisk_addr_r=0x89000000\0" \
> > > + "loadaddr=0x80100000\0" \
> > > + "kernel_addr=0x100000\0" \
> > > + "ramdisk_addr=0x800000\0" \
> > > + "ramdisk_size=0x2000000\0" \
> > > + "fdt_high=0xa0000000\0" \
> > > + "initrd_high=0xffffffffffffffff\0" \
> > > + "kernel_start=0x21100000\0" \
> > > + "kernel_load=0xa0000000\0" \
> > > + "kernel_size=0x2800000\0" \
> > > + "mcmemsize=0x40000000\0" \
> > > + "fdtfile=fsl-ls2080a-rdb.dtb\0" \
> > > + "mcinitcmd=fsl_mc start mc 0x20300000" \
> > > + " 0x20800000 \0" \
> > > + BOOTENV
> > > +#else
> > > #define CONFIG_EXTRA_ENV_SETTINGS \
> > > "hwconfig=fsl_ddr:bank_intlv=auto\0" \
> > > "scriptaddr=0x80800000\0" \
> > > @@ -369,6 +412,7 @@ unsigned long get_board_sys_clk(void);
> > > " 0x580800000 \0" \
> > > BOOTENV
> > > #endif
> > > +#endif
> > >
> > >
> > > #undef CONFIG_BOOTARGS
> > > @@ -378,11 +422,19 @@ unsigned long get_board_sys_clk(void);
> > > " hugepagesz=2m hugepages=256"
> > >
> > > #undef CONFIG_BOOTCOMMAND
> > > +#ifdef CONFIG_QSPI_BOOT
> > > +/* Try to boot an on-QSPI kernel first, then do normal distro boot
> > > +*/ #define CONFIG_BOOTCOMMAND "run mcinitcmd && fsl_mc lazyapply
> > > +dpl
> > 0x20700000" \
> > > + " && cp.b $kernel_start $kernel_load $kernel_size" \
> > > + " && bootm $kernel_load" \
> > > + " || run distro_bootcmd"
> > > +#else
> >
> > QSPI is XIP device. I don't want to see these copying any more. Please
> > run bootm command directly.
> >
> I will update this command
> Priyanka
> > York
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot
More information about the U-Boot
mailing list