From mboxrd@z Thu Jan 1 00:00:00 1970 From: York Sun Date: Tue, 14 Oct 2014 11:31:52 -0700 Subject: [U-Boot] [PATCH v3] powerpc/85xx: enable some P1/P2 boards mtdparts for nor flash In-Reply-To: References: <1408327396-19630-1-git-send-email-yangbo.lu@freescale.com> Message-ID: <543D6C18.5000602@freescale.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 08/17/2014 07:43 PM, York Sun wrote: > Yangbo, > > On 8/17/14 7:03 PM, "Yangbo Lu" wrote: > >> Enable these boards mtdparts for nor flash: >> p1020rdb-pd, p1021rdb-pc, p1022ds, p1025twr, and p2020rdb-pc. >> >> Signed-off-by: Xie Xiaobo >> Signed-off-by: Yangbo Lu >> Cc: York Sun >> Cc: Scott Wood >> --- >> Changes for v2: >> - changed commit message >> - fixed max line length problem in code >> Changes for v3: >> - added a Cc to York Sun >> --- >> include/configs/P1022DS.h | 12 ++++++++++++ >> include/configs/p1_p2_rdb_pc.h | 31 +++++++++++++++++++++++++++++++ >> include/configs/p1_twr.h | 12 ++++++++++++ >> 3 files changed, 55 insertions(+) >> >> diff --git a/include/configs/P1022DS.h b/include/configs/P1022DS.h >> index 139d4fe..4480270 100644 >> --- a/include/configs/P1022DS.h >> +++ b/include/configs/P1022DS.h >> @@ -620,6 +620,18 @@ >> #endif >> >> /* >> + * Dynamic MTD Partition support with mtdparts >> + */ >> +#define CONFIG_MTD_DEVICE >> +#define CONFIG_MTD_PARTITIONS >> +#define CONFIG_CMD_MTDPARTS >> +#define CONFIG_FLASH_CFI_MTD >> +#define MTDIDS_DEFAULT "nor0=e8000000.nor" >> +#define MTDPARTS_DEFAULT "mtdparts=e8000000.nor:48m(ramdisk)," \ >> + "14m(diagnostic),2m(dink),6m(kernel),58112k(fs)," \ >> + "512k(dtb),768k(u-boot)" >> + > > Don't you need to consider both 32- and 36-bit physical address for the > flash base here? The same comment applies to p1_p2_rdb_pc.h as well. > Please respond. York