* [U-Boot] [PATCH 0/2] ARM: atmel: sama5d3xek: enable NOR flash support @ 2014-06-20 8:15 Bo Shen 2014-06-20 8:16 ` [U-Boot] [PATCH 1/2] ARM: atmel: sama5d3xek: add nor flash init function Bo Shen 2014-06-20 8:16 ` [U-Boot] [PATCH 2/2] ARM: atmel: sama5d3xek: enable NOR flash support Bo Shen 0 siblings, 2 replies; 7+ messages in thread From: Bo Shen @ 2014-06-20 8:15 UTC (permalink / raw) To: u-boot Add NOR flash support on sama5d3xek board. Then, we can use NOR flash related command to access it. Bo Shen (2): ARM: atmel: sama5d3xek: add nor flash init function ARM: atmel: sama5d3xek: enable NOR flash support board/atmel/sama5d3xek/sama5d3xek.c | 58 +++++++++++++++++++++++++++++++++++++ include/configs/sama5d3xek.h | 12 +++++++- 2 files changed, 69 insertions(+), 1 deletion(-) -- 1.8.5.2 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH 1/2] ARM: atmel: sama5d3xek: add nor flash init function 2014-06-20 8:15 [U-Boot] [PATCH 0/2] ARM: atmel: sama5d3xek: enable NOR flash support Bo Shen @ 2014-06-20 8:16 ` Bo Shen 2014-06-20 8:16 ` [U-Boot] [PATCH 2/2] ARM: atmel: sama5d3xek: enable NOR flash support Bo Shen 1 sibling, 0 replies; 7+ messages in thread From: Bo Shen @ 2014-06-20 8:16 UTC (permalink / raw) To: u-boot Add NOR flash hardware init function, including SMC and PIO configuration. Signed-off-by: Bo Shen <voice.shen@atmel.com> --- board/atmel/sama5d3xek/sama5d3xek.c | 58 +++++++++++++++++++++++++++++++++++++ 1 file changed, 58 insertions(+) diff --git a/board/atmel/sama5d3xek/sama5d3xek.c b/board/atmel/sama5d3xek/sama5d3xek.c index c835c12..0e95609 100644 --- a/board/atmel/sama5d3xek/sama5d3xek.c +++ b/board/atmel/sama5d3xek/sama5d3xek.c @@ -67,6 +67,61 @@ void sama5d3xek_nand_hw_init(void) } #endif +#ifndef CONFIG_SYS_NO_FLASH +void sama5d3xek_nor_hw_init(void) +{ + struct at91_smc *smc = (struct at91_smc *)ATMEL_BASE_SMC; + + at91_periph_clk_enable(ATMEL_ID_SMC); + + /* Configure SMC CS0 for NOR flash */ + writel(AT91_SMC_SETUP_NWE(1) | AT91_SMC_SETUP_NCS_WR(0) | + AT91_SMC_SETUP_NRD(2) | AT91_SMC_SETUP_NCS_RD(0), + &smc->cs[0].setup); + writel(AT91_SMC_PULSE_NWE(10) | AT91_SMC_PULSE_NCS_WR(11) | + AT91_SMC_PULSE_NRD(10) | AT91_SMC_PULSE_NCS_RD(11), + &smc->cs[0].pulse); + writel(AT91_SMC_CYCLE_NWE(11) | AT91_SMC_CYCLE_NRD(14), + &smc->cs[0].cycle); + writel(AT91_SMC_TIMINGS_TCLR(0) | AT91_SMC_TIMINGS_TADL(0) | + AT91_SMC_TIMINGS_TAR(0) | AT91_SMC_TIMINGS_TRR(0) | + AT91_SMC_TIMINGS_TWB(0) | AT91_SMC_TIMINGS_RBNSEL(0)| + AT91_SMC_TIMINGS_NFSEL(0), &smc->cs[0].timings); + writel(AT91_SMC_MODE_RM_NRD | AT91_SMC_MODE_WM_NWE | + AT91_SMC_MODE_EXNW_DISABLE | + AT91_SMC_MODE_DBW_16 | + AT91_SMC_MODE_TDF_CYCLE(1), + &smc->cs[0].mode); + + /* Address pin (A1 ~ A23) configuration */ + at91_set_a_periph(AT91_PIO_PORTE, 1, 0); + at91_set_a_periph(AT91_PIO_PORTE, 2, 0); + at91_set_a_periph(AT91_PIO_PORTE, 3, 0); + at91_set_a_periph(AT91_PIO_PORTE, 4, 0); + at91_set_a_periph(AT91_PIO_PORTE, 5, 0); + at91_set_a_periph(AT91_PIO_PORTE, 6, 0); + at91_set_a_periph(AT91_PIO_PORTE, 7, 0); + at91_set_a_periph(AT91_PIO_PORTE, 8, 0); + at91_set_a_periph(AT91_PIO_PORTE, 9, 0); + at91_set_a_periph(AT91_PIO_PORTE, 10, 0); + at91_set_a_periph(AT91_PIO_PORTE, 11, 0); + at91_set_a_periph(AT91_PIO_PORTE, 12, 0); + at91_set_a_periph(AT91_PIO_PORTE, 13, 0); + at91_set_a_periph(AT91_PIO_PORTE, 14, 0); + at91_set_a_periph(AT91_PIO_PORTE, 15, 0); + at91_set_a_periph(AT91_PIO_PORTE, 16, 0); + at91_set_a_periph(AT91_PIO_PORTE, 17, 0); + at91_set_a_periph(AT91_PIO_PORTE, 18, 0); + at91_set_a_periph(AT91_PIO_PORTE, 19, 0); + at91_set_a_periph(AT91_PIO_PORTE, 20, 0); + at91_set_a_periph(AT91_PIO_PORTE, 21, 0); + at91_set_a_periph(AT91_PIO_PORTE, 22, 0); + at91_set_a_periph(AT91_PIO_PORTE, 23, 0); + /* CS0 pin configuration */ + at91_set_a_periph(AT91_PIO_PORTE, 26, 0); +} +#endif + #ifdef CONFIG_CMD_USB static void sama5d3xek_usb_hw_init(void) { @@ -181,6 +236,9 @@ int board_init(void) #ifdef CONFIG_NAND_ATMEL sama5d3xek_nand_hw_init(); #endif +#ifndef CONFIG_SYS_NO_FLASH + sama5d3xek_nor_hw_init(); +#endif #ifdef CONFIG_CMD_USB sama5d3xek_usb_hw_init(); #endif -- 1.8.5.2 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH 2/2] ARM: atmel: sama5d3xek: enable NOR flash support 2014-06-20 8:15 [U-Boot] [PATCH 0/2] ARM: atmel: sama5d3xek: enable NOR flash support Bo Shen 2014-06-20 8:16 ` [U-Boot] [PATCH 1/2] ARM: atmel: sama5d3xek: add nor flash init function Bo Shen @ 2014-06-20 8:16 ` Bo Shen 2014-06-30 9:55 ` Andy Pont 1 sibling, 1 reply; 7+ messages in thread From: Bo Shen @ 2014-06-20 8:16 UTC (permalink / raw) To: u-boot Signed-off-by: Bo Shen <voice.shen@atmel.com> --- include/configs/sama5d3xek.h | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/include/configs/sama5d3xek.h b/include/configs/sama5d3xek.h index da27180..63d6775 100644 --- a/include/configs/sama5d3xek.h +++ b/include/configs/sama5d3xek.h @@ -79,8 +79,18 @@ #define CONFIG_BOOTP_GATEWAY #define CONFIG_BOOTP_HOSTNAME -/* No NOR flash */ +/* NOR flash */ +#define CONFIG_CMD_FLASH + +#ifdef CONFIG_CMD_FLASH +#define CONFIG_SYS_FLASH_CFI +#define CONFIG_FLASH_CFI_DRIVER +#define CONFIG_SYS_FLASH_BASE 0x10000000 +#define CONFIG_SYS_MAX_FLASH_SECT 0x20000 +#define CONFIG_SYS_MAX_FLASH_BANKS 1 +#else #define CONFIG_SYS_NO_FLASH +#endif /* * Command line configuration. -- 1.8.5.2 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH 2/2] ARM: atmel: sama5d3xek: enable NOR flash support 2014-06-20 8:16 ` [U-Boot] [PATCH 2/2] ARM: atmel: sama5d3xek: enable NOR flash support Bo Shen @ 2014-06-30 9:55 ` Andy Pont 2014-06-30 10:01 ` Bo Shen 0 siblings, 1 reply; 7+ messages in thread From: Andy Pont @ 2014-06-30 9:55 UTC (permalink / raw) To: u-boot Hi Bo, > -/* No NOR flash */ > +/* NOR flash */ > +#define CONFIG_CMD_FLASH > + > +#ifdef CONFIG_CMD_FLASH > +#define CONFIG_SYS_FLASH_CFI > +#define CONFIG_FLASH_CFI_DRIVER > +#define CONFIG_SYS_FLASH_BASE 0x10000000 > +#define CONFIG_SYS_MAX_FLASH_SECT 0x20000 > +#define CONFIG_SYS_MAX_FLASH_BANKS 1 > +#else > #define CONFIG_SYS_NO_FLASH > +#endif The NOR flash device on the SAMA5D3x CPU modules supports hardware sector protection and according to the flash datasheet: "All blocks power-up in a locked state to protect array data from being altered during power transitions". In order to make your patch set work: #define CONFIG_SYS_FLASH_PROTECTION needs adding into the include/configs/sama5d3xek.h file. Andy. ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH 2/2] ARM: atmel: sama5d3xek: enable NOR flash support 2014-06-30 9:55 ` Andy Pont @ 2014-06-30 10:01 ` Bo Shen 2014-06-30 10:46 ` Andy Pont 0 siblings, 1 reply; 7+ messages in thread From: Bo Shen @ 2014-06-30 10:01 UTC (permalink / raw) To: u-boot Hi Andy, On 06/30/2014 05:55 PM, Andy Pont wrote: > Hi Bo, > >> -/* No NOR flash */ >> +/* NOR flash */ >> +#define CONFIG_CMD_FLASH >> + >> +#ifdef CONFIG_CMD_FLASH >> +#define CONFIG_SYS_FLASH_CFI >> +#define CONFIG_FLASH_CFI_DRIVER >> +#define CONFIG_SYS_FLASH_BASE 0x10000000 >> +#define CONFIG_SYS_MAX_FLASH_SECT 0x20000 >> +#define CONFIG_SYS_MAX_FLASH_BANKS 1 >> +#else >> #define CONFIG_SYS_NO_FLASH >> +#endif > > The NOR flash device on the SAMA5D3x CPU modules supports hardware sector > protection and according to the flash datasheet: "All blocks power-up in a > locked state to protect array data from being altered during power > transitions". In order to make your patch set work: > > #define CONFIG_SYS_FLASH_PROTECTION > > needs adding into the include/configs/sama5d3xek.h file. Thanks for your information. I will add it into next version. Thanks again. > Andy. > Best Regards, Bo Shen ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH 2/2] ARM: atmel: sama5d3xek: enable NOR flash support 2014-06-30 10:01 ` Bo Shen @ 2014-06-30 10:46 ` Andy Pont 2014-07-01 3:52 ` Bo Shen 0 siblings, 1 reply; 7+ messages in thread From: Andy Pont @ 2014-06-30 10:46 UTC (permalink / raw) To: u-boot Hi Bo, > >> +#ifdef CONFIG_CMD_FLASH > >> +#define CONFIG_SYS_FLASH_CFI > >> +#define CONFIG_FLASH_CFI_DRIVER > >> +#define CONFIG_SYS_FLASH_BASE 0x10000000 > >> +#define CONFIG_SYS_MAX_FLASH_SECT 0x20000 Is this the correct value for CONFIG_SYS_MAX_FLASH_SECT? Shouldn't it be 127 (if we ignore the top boot block) or 131 if they are included? Regards, Andy. ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH 2/2] ARM: atmel: sama5d3xek: enable NOR flash support 2014-06-30 10:46 ` Andy Pont @ 2014-07-01 3:52 ` Bo Shen 0 siblings, 0 replies; 7+ messages in thread From: Bo Shen @ 2014-07-01 3:52 UTC (permalink / raw) To: u-boot Hi Andy, On 06/30/2014 06:46 PM, Andy Pont wrote: > Hi Bo, > >>>> +#ifdef CONFIG_CMD_FLASH >>>> +#define CONFIG_SYS_FLASH_CFI >>>> +#define CONFIG_FLASH_CFI_DRIVER >>>> +#define CONFIG_SYS_FLASH_BASE 0x10000000 >>>> +#define CONFIG_SYS_MAX_FLASH_SECT 0x20000 > > Is this the correct value for CONFIG_SYS_MAX_FLASH_SECT? Shouldn't it be > 127 (if we ignore the top boot block) or 131 if they are included? oops, I fill this with the sector size. Thanks point it out. Here, I think we should fill with 131. > Regards, > > Andy. > Best Regards, Bo Shen ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2014-07-01 3:52 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-06-20 8:15 [U-Boot] [PATCH 0/2] ARM: atmel: sama5d3xek: enable NOR flash support Bo Shen 2014-06-20 8:16 ` [U-Boot] [PATCH 1/2] ARM: atmel: sama5d3xek: add nor flash init function Bo Shen 2014-06-20 8:16 ` [U-Boot] [PATCH 2/2] ARM: atmel: sama5d3xek: enable NOR flash support Bo Shen 2014-06-30 9:55 ` Andy Pont 2014-06-30 10:01 ` Bo Shen 2014-06-30 10:46 ` Andy Pont 2014-07-01 3:52 ` Bo Shen
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox