...

SET_TLB_ENTRY(1, CONFIG_SYS_NAND_BASE, CONFIG_SYS_NAND_BASE_PHYS,

•              MAS3_SW|MAS3_SR, MAS2_I|MAS2_G,
  

•              MAS3_SX|MAS3_SW|MAS3_SR, MAS2_I|MAS2_G,
                0, 5, BOOKE_PAGESZ_64K, 1),
  

No.

Without MAS3_SX, the board can't run in booting from NAND.

...

SET_TLB_ENTRY(1, CONFIG_SYS_PLATFORM_SRAM_BASE, @@ -61,7 +72,8 @@

struct fsl_e_tlb_entry tlb_table[] = { MAS3_SX|MAS3_SW|MAS3_SR, 0, 0, 7, BOOKE_PAGESZ_256K, 1),

-#ifdef CONFIG_SYS_RAMBOOT +#if defined(CONFIG_SYS_RAMBOOT) || (defined(CONFIG_SPL) \

•      && !defined(CONFIG_SPL_COMMON_INIT_DDR))
  

  SET_TLB_ENTRY(1, CONFIG_SYS_DDR_SDRAM_BASE, CONFIG_SYS_DDR_SDRAM_BASE, MAS3_SX|MAS3_SW|MAS3_SR, 0,

This will have the result of mapping DDR in the SPL where it's not used, but not in the TPL where it is.

For the TPL the ddr initial by initdram(), so don't need to mapping ddr here. I intend to change it to #if defined(CONFIG_SYS_RAMBOOT) || !defined(CONFIG_SPL_COMMON_INIT_DDR) Is it more clear?

-Scott

-Po Liu