Message ID | 1494898066-11825-1-git-send-email-qiang.zhao@nxp.com |
---|---|
State | Superseded |
Delegated to: | York Sun |
Headers | show |
On 05/15/2017 06:43 PM, Zhao Qiang wrote: > modify u_qe_init to upload QE firmware from SD card when it is SD > boot > > Signed-off-by: Zhao Qiang <qiang.zhao@nxp.com> > --- > Changes for v2: > - fix issue of memory leak > Changes for v3: > - add CONFIG_SYS_QE_FMAN_FW_IN_NOR to ls1021a > Changes for v4: > - rebase due to memory-mapping conflict > > drivers/qe/qe.c | 37 ++++++++++++++++++++++++++++++++++++- > include/configs/ls1021aqds.h | 1 + > include/configs/ls1021atwr.h | 1 + > include/configs/ls1043a_common.h | 2 ++ > include/configs/ls1043ardb.h | 4 +--- > 5 files changed, 41 insertions(+), 4 deletions(-) > > diff --git a/drivers/qe/qe.c b/drivers/qe/qe.c > index 4f0a278..52e4d7c 100644 > --- a/drivers/qe/qe.c > +++ b/drivers/qe/qe.c > @@ -8,6 +8,7 @@ > */ > > #include <common.h> > +#include <malloc.h> > #include <command.h> > #include <linux/errno.h> > #include <asm/io.h> > @@ -17,6 +18,10 @@ > #include <asm/arch/immap_ls102xa.h> > #endif > > +#ifdef CONFIG_SYS_QE_FMAN_FW_IN_MMC > +#include <mmc.h> > +#endif > + > #define MPC85xx_DEVDISR_QE_DISABLE 0x1 > > qe_map_t *qe_immr = NULL; > @@ -194,8 +199,38 @@ void u_qe_init(void) > { > qe_immr = (qe_map_t *)(CONFIG_SYS_IMMR + QE_IMMR_OFFSET); > > - u_qe_upload_firmware((const void *)CONFIG_SYS_QE_FW_ADDR); > +#if defined(CONFIG_SYS_QE_FMAN_FW_IN_NOR) > + void *addr = (void *)CONFIG_SYS_QE_FW_ADDR; > +#elif defined(CONFIG_SYS_QE_FMAN_FW_IN_MMC) > + int dev = CONFIG_SYS_MMC_ENV_DEV; > + void *addr = malloc(CONFIG_SYS_QE_FMAN_FW_LENGTH); > + u32 cnt = CONFIG_SYS_QE_FMAN_FW_LENGTH / 512; > + u32 blk = CONFIG_SYS_QE_FW_ADDR / 512; > + > + if (mmc_initialize(gd->bd)) { > + printf("%s: mmc_initialize() failed\n", __func__); > + return; > + } > + struct mmc *mmc = find_mmc_device(CONFIG_SYS_MMC_ENV_DEV); > + > + if (!mmc) { > + free(addr); > + printf("\nMMC cannot find device for ucode\n"); > + } else { > + printf("\nMMC read: dev # %u, block # %u, count %u ...\n", > + dev, blk, cnt); > + mmc_init(mmc); > + (void)mmc->block_dev.block_read(&mmc->block_dev, blk, cnt, > + addr); > + /* flush cache after read */ > + flush_cache((ulong)addr, cnt * 512); > + } > +#endif > + u_qe_upload_firmware(addr); > out_be32(&qe_immr->iram.iready, QE_IRAM_READY); You are using addr outside of #if section. This causes compiling error for T1024, T1040, T1042, etc. Please test on both ARM and PowerPC platforms before sending a new version. York
diff --git a/drivers/qe/qe.c b/drivers/qe/qe.c index 4f0a278..52e4d7c 100644 --- a/drivers/qe/qe.c +++ b/drivers/qe/qe.c @@ -8,6 +8,7 @@ */ #include <common.h> +#include <malloc.h> #include <command.h> #include <linux/errno.h> #include <asm/io.h> @@ -17,6 +18,10 @@ #include <asm/arch/immap_ls102xa.h> #endif +#ifdef CONFIG_SYS_QE_FMAN_FW_IN_MMC +#include <mmc.h> +#endif + #define MPC85xx_DEVDISR_QE_DISABLE 0x1 qe_map_t *qe_immr = NULL; @@ -194,8 +199,38 @@ void u_qe_init(void) { qe_immr = (qe_map_t *)(CONFIG_SYS_IMMR + QE_IMMR_OFFSET); - u_qe_upload_firmware((const void *)CONFIG_SYS_QE_FW_ADDR); +#if defined(CONFIG_SYS_QE_FMAN_FW_IN_NOR) + void *addr = (void *)CONFIG_SYS_QE_FW_ADDR; +#elif defined(CONFIG_SYS_QE_FMAN_FW_IN_MMC) + int dev = CONFIG_SYS_MMC_ENV_DEV; + void *addr = malloc(CONFIG_SYS_QE_FMAN_FW_LENGTH); + u32 cnt = CONFIG_SYS_QE_FMAN_FW_LENGTH / 512; + u32 blk = CONFIG_SYS_QE_FW_ADDR / 512; + + if (mmc_initialize(gd->bd)) { + printf("%s: mmc_initialize() failed\n", __func__); + return; + } + struct mmc *mmc = find_mmc_device(CONFIG_SYS_MMC_ENV_DEV); + + if (!mmc) { + free(addr); + printf("\nMMC cannot find device for ucode\n"); + } else { + printf("\nMMC read: dev # %u, block # %u, count %u ...\n", + dev, blk, cnt); + mmc_init(mmc); + (void)mmc->block_dev.block_read(&mmc->block_dev, blk, cnt, + addr); + /* flush cache after read */ + flush_cache((ulong)addr, cnt * 512); + } +#endif + u_qe_upload_firmware(addr); out_be32(&qe_immr->iram.iready, QE_IRAM_READY); +#ifdef CONFIG_SYS_QE_FMAN_FW_IN_MMC + free(addr); +#endif } #endif diff --git a/include/configs/ls1021aqds.h b/include/configs/ls1021aqds.h index 5fcaf85..fb077fb 100644 --- a/include/configs/ls1021aqds.h +++ b/include/configs/ls1021aqds.h @@ -125,6 +125,7 @@ unsigned long get_board_ddr_clk(void); #if !defined(CONFIG_SD_BOOT) && !defined(CONFIG_NAND_BOOT) && \ !defined(CONFIG_QSPI_BOOT) #define CONFIG_U_QE +#define CONFIG_SYS_QE_FMAN_FW_IN_NOR #endif /* diff --git a/include/configs/ls1021atwr.h b/include/configs/ls1021atwr.h index 2a20dec..240c2e2 100644 --- a/include/configs/ls1021atwr.h +++ b/include/configs/ls1021atwr.h @@ -150,6 +150,7 @@ #if !defined(CONFIG_SD_BOOT) && !defined(CONFIG_NAND_BOOT) && \ !defined(CONFIG_QSPI_BOOT) #define CONFIG_U_QE +#define CONFIG_SYS_QE_FMAN_FW_IN_NOR #endif /* diff --git a/include/configs/ls1043a_common.h b/include/configs/ls1043a_common.h index 155da92..56f5ee6 100644 --- a/include/configs/ls1043a_common.h +++ b/include/configs/ls1043a_common.h @@ -219,6 +219,7 @@ */ #define CONFIG_SYS_QE_FMAN_FW_IN_MMC #define CONFIG_SYS_FMAN_FW_ADDR (512 * 0x4800) +#define CONFIG_SYS_QE_FW_ADDR (512 * 0x4a08) #elif defined(CONFIG_QSPI_BOOT) #define CONFIG_SYS_QE_FW_IN_SPIFLASH #define CONFIG_SYS_FMAN_FW_ADDR 0x40900000 @@ -230,6 +231,7 @@ #define CONFIG_SYS_QE_FMAN_FW_IN_NOR /* FMan fireware Pre-load address */ #define CONFIG_SYS_FMAN_FW_ADDR 0x60900000 +#define CONFIG_SYS_QE_FW_ADDR 0x60940000 #endif #define CONFIG_SYS_QE_FMAN_FW_LENGTH 0x10000 #define CONFIG_SYS_FDT_PAD (0x3000 + CONFIG_SYS_QE_FMAN_FW_LENGTH) diff --git a/include/configs/ls1043ardb.h b/include/configs/ls1043ardb.h index a162378..2ce7ddd 100644 --- a/include/configs/ls1043ardb.h +++ b/include/configs/ls1043ardb.h @@ -277,11 +277,9 @@ /* QE */ #ifndef SPL_NO_QE -#if !defined(CONFIG_SD_BOOT) && !defined(CONFIG_NAND_BOOT) && \ - !defined(CONFIG_QSPI_BOOT) +#if !defined(CONFIG_NAND_BOOT) && !defined(CONFIG_QSPI_BOOT) #define CONFIG_U_QE #endif -#define CONFIG_SYS_QE_FW_ADDR 0x60940000 #endif /* USB */
modify u_qe_init to upload QE firmware from SD card when it is SD boot Signed-off-by: Zhao Qiang <qiang.zhao@nxp.com> --- Changes for v2: - fix issue of memory leak Changes for v3: - add CONFIG_SYS_QE_FMAN_FW_IN_NOR to ls1021a Changes for v4: - rebase due to memory-mapping conflict drivers/qe/qe.c | 37 ++++++++++++++++++++++++++++++++++++- include/configs/ls1021aqds.h | 1 + include/configs/ls1021atwr.h | 1 + include/configs/ls1043a_common.h | 2 ++ include/configs/ls1043ardb.h | 4 +--- 5 files changed, 41 insertions(+), 4 deletions(-)