apf27: add support of armadeus apf27
Subsystems (3)
v4 cover apf27: add support of armadeus apf27
Difference between v4 and v3:
- use standard start.S
- use part to know u-boot offset
- fix some comments format issue
Difference between v3 and v2:
- rebase on imx branch
- merge support and spl patch
- clean code of apf27_gpio_init
- clean code of start.S
Difference between v1 and v2:
- use spl instead of nand_spl
- use gpio api
- remove use of useless CONFIG_SYS
- remove use of config.mk
- remove dead code
Philippe Reynes (4):
mx27: add missing constant for mx27
apf27: add support for the armadeus APF27 board
apf27: add FPGA support for the apf27 board
Update u-boot-nand.bin to be used with spl
MAINTAINERS | 5 + Makefile | 6 + arch/arm/cpu/arm926ejs/mx27/asm-offsets.c | 5 + arch/arm/include/asm/arch-mx27/imx-regs.h | 2 +- board/armadeus/apf27/Makefile | 54 +++ board/armadeus/apf27/apf27.c | 374 ++++++++++++++++++++ board/armadeus/apf27/apf27.h | 450 ++++++++++++++++++++++++ board/armadeus/apf27/fpga.c | 250 ++++++++++++++ board/armadeus/apf27/fpga.h | 39 +++ board/armadeus/apf27/splboot.S | 528 +++++++++++++++++++++++++++++ boards.cfg | 1 + include/configs/apf27.h | 446 ++++++++++++++++++++++++ 12 files changed, 2159 insertions(+), 1 deletions(-) create mode 100644 board/armadeus/apf27/Makefile create mode 100644 board/armadeus/apf27/apf27.c create mode 100644 board/armadeus/apf27/apf27.h create mode 100644 board/armadeus/apf27/fpga.c create mode 100644 board/armadeus/apf27/fpga.h create mode 100644 board/armadeus/apf27/splboot.S create mode 100644 include/configs/apf27.h
Signature · 1 line
1.7.4.4
PATCH v4 1/4 mx27: add missing constant for mx27
Add some missing constant (chip select, ...)
--- arch/arm/cpu/arm926ejs/mx27/asm-offsets.c | 5 +++++ arch/arm/include/asm/arch-mx27/imx-regs.h | 2 +- 2 files changed, 6 insertions(+), 1 deletions(-)
arch/arm/cpu/arm926ejs/mx27/asm-offsets.c +5 -0
@@ -41,5 +41,10 @@ int main(void)DEFINE (ESDCFG1_ROF , offsetof(struct esdramc_regs , esdcfg1));DEFINE (ESDMISC_ROF , offsetof(struct esdramc_regs , esdmisc)); +DEFINE (GPCR ,IMX_SYSTEM_CTL_BASE + + offsetof(struct system_control_regs , gpcr)); +DEFINE (FMCR ,IMX_SYSTEM_CTL_BASE + + offsetof(struct system_control_regs , fmcr)); +return 0 ; }
arch/arm/include/asm/arch-mx27/imx-regs.h +1 -1
@@ -185,7 +185,7 @@ struct iim_regs {struct fuse_bank {u32 fuse_regs[0x20 ];u32 fuse_rsvd[0xe0 ]; - } bank[1 ]; + } bank[2 ]; };struct fuse_bank0_regs {
Signature · 1 line
1.7.4.4
PATCH v4 1/4 mx27: add missing constant for mx27
Quoted 30 lines · Philippe Reynes
Add some missing constant (chip select, ...)
Signed-off-by Philippe Reynes <[email protected]>Signed-off-by Eric Jarrige <[email protected]>--- arch/arm/cpu/arm926ejs/mx27/asm-offsets.c | 5 +++++ arch/arm/include/asm/arch-mx27/imx-regs.h | 2 +- 2 files changed, 6 insertions(+), 1 deletions(-)arch/arm/cpu/arm926ejs/mx27/asm-offsets.c +5
-0@@ -41,5 +41,10 @@ int main(void)DEFINE (ESDCFG1_ROF , offsetof(struct esdramc_regs , esdcfg1));DEFINE (ESDMISC_ROF , offsetof(struct esdramc_regs , esdmisc)); +DEFINE (GPCR ,IMX_SYSTEM_CTL_BASE + + offsetof(struct system_control_regs , gpcr)); +DEFINE (FMCR ,IMX_SYSTEM_CTL_BASE + + offsetof(struct system_control_regs , fmcr)); +return 0 ; }arch/arm/include/asm/arch-mx27/imx-regs.h +0
-0
arch/arm/include/asm/arch-mx27/imx-regs.h struct iim_regs { · line 191
Quoted 8 lines
arch/arm/include/asm/arch-mx27/imx-regs.h +1
-1@@ -185,7 +185,7 @@ struct iim_regs {struct fuse_bank {u32 fuse_regs[0x20 ];u32 fuse_rsvd[0xe0 ]; - } bank[1 ]; + } bank[2 ]; };struct fuse_bank0_regs {
Best regards,
Stefano Babic
Signature · 5 lines
===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de =====================================================================
PATCH v4 2/4 apf27: add support for the armadeus APF27 board
In reply to Philippe Reynes--- MAINTAINERS | 5 + board/armadeus/apf27/Makefile | 51 ++++ board/armadeus/apf27/apf27.c | 356 +++++++++++++++++++++++++++ board/armadeus/apf27/apf27.h | 450 ++++++++++++++++++++++++++++++++++ board/armadeus/apf27/splboot.S | 528 ++++++++++++++++++++++++++++++++++++++++ boards.cfg | 1 + include/configs/apf27.h | 446 +++++++++++++++++++++++++++++++++ 7 files changed, 1837 insertions(+), 0 deletions(-) create mode 100644 board/armadeus/apf27/Makefile create mode 100644 board/armadeus/apf27/apf27.c create mode 100644 board/armadeus/apf27/apf27.h create mode 100644 board/armadeus/apf27/splboot.S create mode 100644 include/configs/apf27.h
MAINTAINERS +5 -0
@@ -717,6 +717,11 @@ Ilko Iliev <[email protected]> PM9263 AT91SAM9263 PM9G45 ARM926EJS (AT91SAM9G45 SoC) +Eric Jarrige <[email protected]> +Philippe Reynes <[email protected]> + + apf27 ARM926EJS (i.MX27 SoC) + Michael Jones <[email protected]> omap3_mvblx ARM ARMV7 (OMAP3xx SoC)
board/armadeus/apf27/Makefile new file +51 -0
51 lines not on this page · raw message
board/armadeus/apf27/apf27.c new file +356 -0
356 lines not on this page · raw message
board/armadeus/apf27/apf27.h new file +450 -0
450 lines not on this page · raw message
board/armadeus/apf27/splboot.S new file +528 -0
528 lines not on this page · raw message
boards.cfg +1 -0
@@ -195,6 +195,7 @@ jadecpu arm arm926ejs jadecpu syteco mx25pdk arm arm926ejs mx25pdk freescale mx25 mx25pdk:IMX_CONFIG=board/freescale/mx25pdk/imximage.cfg tx25 arm arm926ejs tx25 karo mx25 zmx25 arm arm926ejs zmx25 syteco mx25 +apf27 arm arm926ejs apf27 armadeus mx27 imx27lite arm arm926ejs imx27lite logicpd mx27 magnesium arm arm926ejs imx27lite logicpd mx27 mx23_olinuxino arm arm926ejs mx23_olinuxino olimex mxs mx23_olinuxino
include/configs/apf27.h new file +446 -0
446 lines not on this page · raw message
Signature · 1 line
1.7.4.4
PATCH v4 2/4 apf27: add support for the armadeus APF27 board
Hi Philippe,
Quoted 10 lines · Philippe Reynes
board/armadeus/apf27/Makefile new file +10
-0+# +# See file CREDITS for list of people who contributed to this +# project. +# +# This program is free software; you can redistribute it and/or +# modify it under the terms of the GNU General Public License as +# published by the Free Software Foundation; either version 2 of +# the License, or (at your option) any later version. +# +# This program is distributed in the hope that it will be useful,
board/armadeus/apf27/Makefile line 25
Quoted 10 lines
board/armadeus/apf27/Makefile new file +10
-0+# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program; if not, write to the Free Software +# Foundation, Inc., 59 Temple Place, Suite 330, Boston, +# MA 02111-1307 USA +# +
commit eca3aeb352c964bdb28b8e191d6326370245e03f introduces a short way to add licence to a file. Instead of the full text, a simply identifier is added to the header.
Please replaces the whole text with the identifier, for GPL2+ you should add:
# SPDX-License-Identifier: GPL-2.0+
This issue must be fixed globally.
board/armadeus/apf27/Makefile line 37
Quoted 12 lines
board/armadeus/apf27/Makefile new file +12
-0+include $(TOPDIR)/config.mk + +LIB = $(obj)lib$(BOARD).o + +ifndef CONFIG_SPL_BUILD +COBJS := apf27.o +endif + +ifdef CONFIG_SPL_BUILD +SOBJS := splboot.o +endif +
Maybe you can use here a ifndef...else syntax else duplicating the test.
Quoted 38 lines
board/armadeus/apf27/apf27.c new file +32
-0@@ -0,0 +1,356 @@ +/* +* Copyright (C) 2007 Sascha Hauer, Pengutronix +* Copyright (C) 2008-2012 Eric Jarrige <[email protected]> +* +* This program is free software; you can redistribute it and/or +* modify it under the terms of the GNU General Public License as +* published by the Free Software Foundation; either version 2 of +* the License, or (at your option) any later version. +* +* This program is distributed in the hope that it will be useful, +* but WITHOUT ANY WARRANTY; without even the implied warranty of +* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +* GNU General Public License for more details. +* +* You should have received a copy of the GNU General Public License +* along with this program; if not, write to the Free Software +* Foundation, Inc., 59 Temple Place, Suite 330, Boston, +* MA 02111-1307 USA +* +*/ + +#include <common.h> +#include <environment.h> +#include <jffs2/jffs2.h> +#include <nand.h> +#include <netdev.h> +#include <asm/io.h> +#include <asm/arch/imx-regs.h> +#include <asm/arch/gpio.h> +#include <asm/errno.h> +#include "apf27.h" +#include "crc.h"
board/armadeus/apf27/apf27.c line 42
Quoted 10 lines
board/armadeus/apf27/apf27.c new file +10
-0+ +DECLARE_GLOBAL_DATA_PTR ; + +static inline void wait (void ) +{ +int i; + +for (i =0 ; i <0x10000 ; i++) + asmvolatile ("nop" ); +}
There is already a function doing this, rather accessible only to armv7, whose name is sdelay() - simple spin loop.
At least I suggest to use the same name. However, the function uses general ARM asm instructions and it could be used by i.mx27, too. Maybe can we move it into arch/arm/lib/ ? I have added Albert as ARM Custodian for his opinion on this point.
Quoted 34 lines
board/armadeus/apf27/apf27.c new file +34
-0+ +/* +* Fuse bank 1 row 8 is "reserved for future use" and therefore available for +* customer use. The APF27 board uses this fuse to store the board revision: +* 0: initial board revision +* 1: first revision - Presence of the second RAM chip on the board is blown in +* fuse bank 1 row 9 bit 0 - No hardware change +* N: to be defined +*/ +static u32 get_board_rev (void ) +{ +struct iim_regs *iim = (struct iim_regs *)IMX_IIM_BASE ; + +return readl (&iim->bank[1 ].fuse_regs[8 ]); +} + +/* +* Fuse bank 1 row 9 is "reserved for future use" and therefore available for +* customer use. The APF27 board revision 1 uses the bit 0 to permanently store +* the presence of the second RAM chip +* 0: AFP27 with 1 RAM of 64 MiB +* 1: AFP27 with 2 RAM chips of 64 MiB each (128MB) +*/ +static int get_num_ram_bank (void ) +{ +struct iim_regs *iim = (struct iim_regs *)IMX_IIM_BASE ; +int nr_dram_banks =1 ; + +if ((get_board_rev () >0 ) && (CONFIG_NR_DRAM_BANKS >1 )) + nr_dram_banks +=readl (&iim->bank[1 ].fuse_regs[9 ]) &0x01 ; +else + nr_dram_banks =CONFIG_NR_DRAM_POPULATED ; + +return nr_dram_banks;
board/armadeus/apf27/apf27.c line 86
Quoted 10 lines
board/armadeus/apf27/apf27.c new file +10
-0+} + +static void apf27_port_init (int port,u32 gpio_dr,u32 ocr1,u32 ocr2, +u32 iconfa1,u32 iconfa2,u32 iconfb1,u32 iconfb2, +u32 icr1,u32 icr2,u32 imr,u32 gpio_dir,u32 gpr, +u32 puen,u32 gius) +{ +struct gpio_port_regs *regs = (struct gpio_port_regs *)IMX_GPIO_BASE ; + +writel (gpio_dr, ®s->port[port].gpio_dr);
We have general gpio functions in u-boot to set/get gpios.
Quoted 61 lines
board/armadeus/apf27/apf27.c new file +61
-0+writel (ocr1, ®s->port[port].ocr1); +writel (ocr2, ®s->port[port].ocr2); +writel (iconfa1, ®s->port[port].iconfa1); +writel (iconfa2, ®s->port[port].iconfa2); +writel (iconfb1, ®s->port[port].iconfb1); +writel (iconfb2, ®s->port[port].iconfb2); +writel (icr1, ®s->port[port].icr1); +writel (icr2, ®s->port[port].icr2); +writel (imr, ®s->port[port].imr); +writel (gpio_dir, ®s->port[port].gpio_dir); +writel (gpr, ®s->port[port].gpr); +writel (puen, ®s->port[port].puen); +writel (gius, ®s->port[port].gius); +} + +#define APF27_PORT_INIT (n) apf27_port_init(PORT##n, ACFG_DR_##n##_VAL, \ + ACFG_OCR1_##n##_VAL, ACFG_OCR2_##n##_VAL, ACFG_ICFA1_##n##_VAL, \ + ACFG_ICFA2_##n##_VAL, ACFG_ICFB1_##n##_VAL, ACFG_ICFB2_##n##_VAL, \ + ACFG_ICR1_##n##_VAL, ACFG_ICR2_##n##_VAL, ACFG_IMR_##n##_VAL, \ + ACFG_DDIR_##n##_VAL, ACFG_GPR_##n##_VAL, ACFG_PUEN_##n##_VAL, \ + ACFG_GIUS_##n##_VAL) + +void apf27_gpio_init (void ) +{ +APF27_PORT_INIT (A ); +APF27_PORT_INIT (B ); +APF27_PORT_INIT (C ); +APF27_PORT_INIT (D ); +APF27_PORT_INIT (E ); +APF27_PORT_INIT (F ); +} + +static int apf27_devices_init (void ) +{ +struct gpio_port_regs *regs = (struct gpio_port_regs *)IMX_GPIO_BASE ; +int i; +unsigned int mode[] = { +PD0_AIN_FEC_TXD0 , +PD1_AIN_FEC_TXD1 , +PD2_AIN_FEC_TXD2 , +PD3_AIN_FEC_TXD3 , +PD4_AOUT_FEC_RX_ER , +PD5_AOUT_FEC_RXD1 , +PD6_AOUT_FEC_RXD2 , +PD7_AOUT_FEC_RXD3 , +PD8_AF_FEC_MDIO , +PD9_AIN_FEC_MDC |GPIO_PUEN , +PD10_AOUT_FEC_CRS , +PD11_AOUT_FEC_TX_CLK , +PD12_AOUT_FEC_RXD0 , +PD13_AOUT_FEC_RX_DV , +PD14_AOUT_FEC_CLR , +PD15_AOUT_FEC_COL , +PD16_AIN_FEC_TX_ER , +PF23_AIN_FEC_TX_EN , +PE12_PF_UART1_TXD , +PE13_PF_UART1_RXD , +PC5_PF_I2C2_DATA , +PC6_PF_I2C2_CLK , +PD17_PF_I2C_DATA , +PD18_PF_I2C_CLK ,
board/armadeus/apf27/apf27.c line 157
Quoted 10 lines
board/armadeus/apf27/apf27.c new file +10
-0+ }; + +for (i =0 ; i <ARRAY_SIZE (mode); i++) +imx_gpio_mode (mode[i]); + +#ifdef CONFIG_MXC_MMC +mx27_sd2_init_pins (); +imx_gpio_mode ((GPIO_PORTF |GPIO_OUT |GPIO_PUEN |GPIO_GPIO |16 )); +writel (readl (®s->port[PORTF ].gpio_dr) | (1 <<16 ), + ®s->port[PORTF ].gpio_dr);
Can you use the gpio accessors (gpio_set_value() in this case) ?
Quoted 114 lines
board/armadeus/apf27/apf27.c new file +114
-0+#endif +#if defined(CONFIG_I2C_MXC ) && defined(CONFIG_I2C_MULTI_BUS ) +bus_i2c_init (IMX_I2C1_BASE ,CONFIG_SYS_I2C_SPEED ,CONFIG_SYS_I2C_SLAVE , +NULL ,NULL ); +bus_i2c_init (IMX_I2C2_BASE ,CONFIG_SYS_I2C_SPEED ,CONFIG_SYS_I2C_SLAVE , +NULL ,NULL ); +#endif +return 0 ; +} + +static void board_setup_aipi (void ) +{ +struct aipi_regs *aipi1 = (struct aipi_regs *)IMX_AIPI1_BASE ; +struct aipi_regs *aipi2 = (struct aipi_regs *)IMX_AIPI2_BASE ; +struct system_control_regs *system = + (struct system_control_regs *)IMX_SYSTEM_CTL_BASE ; +u32 gpcr; + +/* setup AIPI1 and AIPI2 */ +writel (ACFG_AIPI1_PSR0_VAL , &aipi1->psr0); +writel (ACFG_AIPI1_PSR1_VAL , &aipi1->psr1); +writel (ACFG_AIPI2_PSR0_VAL , &aipi2->psr0); +writel (ACFG_AIPI2_PSR1_VAL , &aipi2->psr1); + +/* Change SDRAM signal strengh */ + gpcr =readl (&system->gpcr); + gpcr |=ACFG_GPCR_VAL ; +writel (gpcr, &system->gpcr); +} + +static void board_setup_csx (void ) +{ +struct weim_regs *weim = (struct weim_regs *)IMX_WEIM_BASE ; + +writel (ACFG_CS0U_VAL , &weim->cs0u); +writel (ACFG_CS0L_VAL , &weim->cs0l); +writel (ACFG_CS0A_VAL , &weim->cs0a); + +writel (ACFG_CS1U_VAL , &weim->cs1u); +writel (ACFG_CS1L_VAL , &weim->cs1l); +writel (ACFG_CS1A_VAL , &weim->cs1a); + +writel (ACFG_CS2U_VAL , &weim->cs2u); +writel (ACFG_CS2L_VAL , &weim->cs2l); +writel (ACFG_CS2A_VAL , &weim->cs2a); + +writel (ACFG_CS3U_VAL , &weim->cs3u); +writel (ACFG_CS3L_VAL , &weim->cs3l); +writel (ACFG_CS3A_VAL , &weim->cs3a); + +writel (ACFG_CS4U_VAL , &weim->cs4u); +writel (ACFG_CS4L_VAL , &weim->cs4l); +writel (ACFG_CS4A_VAL , &weim->cs4a); + +writel (ACFG_CS5U_VAL , &weim->cs5u); +writel (ACFG_CS5L_VAL , &weim->cs5l); +writel (ACFG_CS5A_VAL , &weim->cs5a); + +writel (ACFG_EIM_VAL , &weim->eim); +} + +static void board_setup_port (void ) +{ +struct system_control_regs *system = + (struct system_control_regs *)IMX_SYSTEM_CTL_BASE ; + +writel (ACFG_FMCR_VAL , &system->fmcr); +} + +static void board_setup_clock (void ) +{ +struct pll_regs *pll = (struct pll_regs *)IMX_PLL_BASE ; +u32 cscr; + +/* disable MPLL/SPLL first */ + cscr =readl (&pll->cscr); + cscr &= ~(CSCR_MPEN |CSCR_SPEN ); +writel (cscr, &pll->cscr); + +/* pll clock initialization */ +writel (ACFG_MPCTL0_VAL , &pll->mpctl0); +writel (ACFG_SPCTL0_VAL , &pll->spctl0); + + cscr =ACFG_CSCR_VAL |CSCR_MPLL_RESTART |CSCR_SPLL_RESTART ; +writel (cscr, &pll->cscr); + +/* add some delay here */ +wait (); + +/* peripheral clock divider */ +writel (ACFG_PCDR0_VAL , &pll->pcdr0); +writel (ACFG_PCDR1_VAL , &pll->pcdr1); + +/* Configure PCCR0 and PCCR1 */ +writel (ACFG_PCCR0_VAL , &pll->pccr0); +writel (ACFG_PCCR1_VAL , &pll->pccr1); +} + +int board_early_init_f (void ) +{ +board_setup_aipi (); +board_setup_csx (); +board_setup_port (); +board_setup_clock (); + +return 0 ; +} + +int board_init (void ) +{ + gd->bd->bi_boot_params =PHYS_SDRAM_1 +0x100 ; + +apf27_gpio_init (); +apf27_devices_init ();
board/armadeus/apf27/apf27.c line 281
Quoted 10 lines
board/armadeus/apf27/apf27.c new file +10
-0+ +return 0 ; +} + +int dram_init (void ) +{ +/* dram_init must store complete ramsize in gd->ram_size */ +if (get_num_ram_bank () >1 ) { + gd->ram_size =PHYS_SDRAM_2 -CONFIG_SYS_SDRAM_BASE + +get_ram_size ((void *)PHYS_SDRAM_2 ,PHYS_SDRAM_2_SIZE );
To my understanding: I know that the result is the same, but you mix here dynamic computed with static values. Why do you not call get_ram_size() twice, that is:
gd->ram_size = get_ram_size((void *)PHYS_SDRAM_1,
PHYS_SDRAM_1_SIZE);
if (get_num_ram_bank() > 1) {
gd->ram_size += get_ram_size((void *)PHYS_SDRAM_2, PHYS_SDRAM_2_SIZE);
}
board/armadeus/apf27/apf27.c line 317
Quoted 11 lines
board/armadeus/apf27/apf27.c new file +11
-0+/* +* Miscellaneous initialisation +*/ +int misc_init_r (void ) +{ +char *s; +u_char *firmware_buffer = (u_char *)(CONFIG_SYS_LOAD_ADDR + +CONFIG_SYS_MONITOR_LEN ); +size_t size =0 ; +size_t offset =-1 ; +#if defined(CONFIG_FPGA )
You add FPGA in a later patch, that then modify this new file. This code makes sense only when the FPGA patch is applied. But as you are adding a new board, it makes sense that you squash patch 3/4 and patch2/4.
Quoted 4 lines
board/armadeus/apf27/apf27.c new file +4
-0+char *autoload =getenv ("firmware_autoload" ); +#endif +u8 pnum; +struct mtd_device *dev;
board/armadeus/apf27/apf27.c line 331
Quoted 10 lines
board/armadeus/apf27/apf27.c new file +10
-0+struct part_info *part; + +/* detect compatibility issue of environment version */ + s =getenv ("env_version" ); +if ((NULL == s) || (0 !=strcmp (s,CONFIG_ENV_VERSION ))) { +char *const vars[] = {"flash_reset_env" }; +printf ("*** Warning - Environment version change suggests: " ); +printf ("run flash_reset_env; reset\n" ); +set_default_vars (1 , vars); + }
The code here depends on the environment - only to better understanding, have you considered to move these tests in a script, too ? I understand that you need some plausibility tests after u-boot has completed and before going into the main loop. What about using the PREBOOT variable ? You could then set preboot to do exactly what you want in a script while now you have the logic split between code (here) and environment.
board/armadeus/apf27/apf27.c line 343
Quoted 12 lines
board/armadeus/apf27/apf27.c new file +12
-0+ +/* Unlock whole flash but U-Boot */ +if (mtdparts_init () ==0 ) { +if (find_dev_and_part ("u-boot" , + &dev, &pnum, &part) ==0 ) { + offset = part->size; + } + } + +if (nand_unlock (&nand_info[0 ], offset, nand_info[0 ].size - offset,0 )) +printf ("NAND flash lock/unlocked failed\n" ); +
Also this could be part of a script, but you decide what is better for your board.
board/armadeus/apf27/apf27.c line 356
Quoted 9 lines
board/armadeus/apf27/apf27.c new file +9
-0+int checkboard (void ) +{ +printf ("Board: Armadeus APF27 revision %d\n" ,get_board_rev ()); +return 0 ; +} + +void lowlevel_init (void ) +{ +}
It looks like you had a missing function by linking. Maybe we can add a __weak function to general i.mx27 code, dropping here. An empty function makes no sense.
Quoted 68 lines
board/armadeus/apf27/apf27.h new file +62
-0@@ -0,0 +1,450 @@ +#ifndef __APF27_H +#define __APF27_H + +/* FPGA program pin configuration */ +#define ACFG_FPGA_PWR (GPIO_PORTF | 19)/* FPGA prog pin */ +#define ACFG_FPGA_PRG (GPIO_PORTF | 11)/* FPGA prog pin */ +#define ACFG_FPGA_CLK (GPIO_PORTF | 15)/* FPGA clk pin */ +#define ACFG_FPGA_RDATA 0xD6000000/* FPGA data addr */ +#define ACFG_FPGA_WDATA 0xD6000000/* FPGA data addr */ +#define ACFG_FPGA_INIT (GPIO_PORTF | 12)/* FPGA init pin */ +#define ACFG_FPGA_DONE (GPIO_PORTF | 9)/* FPGA done pin */ +#define ACFG_FPGA_RW (GPIO_PORTF | 21)/* FPGA done pin */ +#define ACFG_FPGA_CS (GPIO_PORTF | 22)/* FPGA done pin */ +#define ACFG_FPGA_SUSPEND (GPIO_PORTF | 10)/* FPGA done pin */ +#define ACFG_FPGA_RESET (GPIO_PORTF | 7)/* FPGA done pin */ + +/* +* MPU CLOCK source before PLL +* ACFG_CLK_FREQ (2/3 MPLL clock or ext 266 MHZ) +*/ +#define ACFG_MPCTL0_VAL 0x01EF15D5/* 399.000 MHz */ +#define ACFG_MPCTL1_VAL 0 +#define CONFIG_MPLL_FREQ 399 + +#define ACFG_CLK_FREQ (CONFIG_MPLL_FREQ*2/3)/* 266 MHz */ + +/* Serial clock source before PLL (should be named ACFG_SYSPLL_CLK_FREQ)*/ +#define ACFG_SPCTL0_VAL 0x0475206F/* 299.99937 MHz */ +#define ACFG_SPCTL1_VAL 0 +#define CONFIG_SPLL_FREQ 300/* MHz */ + +/* ARM bus frequency (have to be a CONFIG_MPLL_FREQ ratio) */ +#define CONFIG_ARM_FREQ 399/* up to 400 MHz */ + +/* external bus frequency (have to be a ACFG_CLK_FREQ ratio) */ +#define CONFIG_HCLK_FREQ 133/* (ACFG_CLK_FREQ/2) */ + +#define CONFIG_PERIF1_FREQ 16/* 16.625 MHz UART, GPT, PWM*/ +#define CONFIG_PERIF2_FREQ 33/* 33.25 MHz CSPI and SDHC */ +#define CONFIG_PERIF3_FREQ 33/* 33.25 MHz LCD*/ +#define CONFIG_PERIF4_FREQ 33/* 33.25 MHz CSI*/ +#define CONFIG_SSI1_FREQ 66/* 66.50 MHz SSI1*/ +#define CONFIG_SSI2_FREQ 66/* 66.50 MHz SSI2*/ +#define CONFIG_MSHC_FREQ 66/* 66.50 MHz MSHC*/ +#define CONFIG_H264_FREQ 66/* 66.50 MHz H264*/ +#define CONFIG_CLK0_DIV 3/* Divide CLK0 by 4 */ +#define CONFIG_CLK0_EN 1/* CLK0 enabled */ + +/* external bus frequency (have to be a CONFIG_HCLK_FREQ ratio) */ +#define CONFIG_NFC_FREQ 44/* NFC Clock up to 44 MHz wh 133MHz*/ + +/* external serial bus frequency (have to be a CONFIG_SPLL_FREQ ratio) */ +#define CONFIG_USB_FREQ 60/* 60 MHz */ + +/* +* SDRAM +*/ +#if (ACFG_SDRAM_MBYTE_SYZE ==64 )/* micron MT46H16M32LF -6 */ +/* micron 64MB */ +#define ACFG_SDRAM_NUM_COL 9/* 8, 9, 10 or 11 +column address bits */ +#define ACFG_SDRAM_NUM_ROW 13/* 11, 12 or 13
board/armadeus/apf27/apf27.h line 72
Quoted 10 lines
board/armadeus/apf27/apf27.h new file +10
-0+row address bits */ +#define ACFG_SDRAM_REFRESH 3/* 0=OFF 1=2048 +2=4096 3=8192 refresh */ +#define ACFG_SDRAM_EXIT_PWD 25/* ns exit power +down delay */ +#define ACFG_SDRAM_W2R_DELAY 1/* write to read +cycle delay > 0 */ +#define ACFG_SDRAM_ROW_PRECHARGE_DELAY 18/* ns */ +#define ACFG_SDRAM_TMRD_DELAY 2/* Load mode register +cycle delay 1..4 */
I think there are some code style issues due to multiline comments.
I run tools/checkpatch and I get 310 errors from this patch, most of them due to DOS line endigs. All checkpatch errors must be fixed.
Quoted 135 lines
board/armadeus/apf27/apf27.h new file +135
-0+/* CS0 configuration for 16 bit nor flash */ +#define ACFG_CS0U_VAL 0x0000CC03 +#define ACFG_CS0L_VAL 0xa0330D01 +#define ACFG_CS0A_VAL 0x00220800 + +#define ACFG_CS1U_VAL 0x00000f00 +#define ACFG_CS1L_VAL 0x00000D01 +#define ACFG_CS1A_VAL 0 + +#define ACFG_CS2U_VAL 0 +#define ACFG_CS2L_VAL 0 +#define ACFG_CS2A_VAL 0 + +#define ACFG_CS3U_VAL 0 +#define ACFG_CS3L_VAL 0 +#define ACFG_CS3A_VAL 0 + +#define ACFG_CS4U_VAL 0 +#define ACFG_CS4L_VAL 0 +#define ACFG_CS4A_VAL 0 + +/* FPGA 16 bit data bus */ +#define ACFG_CS5U_VAL 0x00000600 +#define ACFG_CS5L_VAL 0x00000D01 +#define ACFG_CS5A_VAL 0 + +#define ACFG_EIM_VAL 0x00002200 + + +/* +* FPGA specific settings +*/ + +/* CLKO */ +#define ACFG_CCSR_VAL 0x00000305 +/* drive strength CLKO set to 2*/ +#define ACFG_DSCR10_VAL 0x00020000 +/* drive strength A1..A12 set to 2*/ +#define ACFG_DSCR3_VAL 0x02AAAAA8 +/* drive strength ctrl*/ +#define ACFG_DSCR7_VAL 0x00020880 +/* drive strength data*/ +#define ACFG_DSCR2_VAL 0xAAAAAAAA + + +/* +* Default configuration for GPIOs and peripherals +*/ +#define ACFG_DDIR_A_VAL 0x00000000 +#define ACFG_OCR1_A_VAL 0x00000000 +#define ACFG_OCR2_A_VAL 0x00000000 +#define ACFG_ICFA1_A_VAL 0xFFFFFFFF +#define ACFG_ICFA2_A_VAL 0xFFFFFFFF +#define ACFG_ICFB1_A_VAL 0xFFFFFFFF +#define ACFG_ICFB2_A_VAL 0xFFFFFFFF +#define ACFG_DR_A_VAL 0x00000000 +#define ACFG_GIUS_A_VAL 0xFFFFFFFF +#define ACFG_ICR1_A_VAL 0x00000000 +#define ACFG_ICR2_A_VAL 0x00000000 +#define ACFG_IMR_A_VAL 0x00000000 +#define ACFG_GPR_A_VAL 0x00000000 +#define ACFG_PUEN_A_VAL 0xFFFFFFFF + +#define ACFG_DDIR_B_VAL 0x00000000 +#define ACFG_OCR1_B_VAL 0x00000000 +#define ACFG_OCR2_B_VAL 0x00000000 +#define ACFG_ICFA1_B_VAL 0xFFFFFFFF +#define ACFG_ICFA2_B_VAL 0xFFFFFFFF +#define ACFG_ICFB1_B_VAL 0xFFFFFFFF +#define ACFG_ICFB2_B_VAL 0xFFFFFFFF +#define ACFG_DR_B_VAL 0x00000000 +#define ACFG_GIUS_B_VAL 0xFF3FFFF0 +#define ACFG_ICR1_B_VAL 0x00000000 +#define ACFG_ICR2_B_VAL 0x00000000 +#define ACFG_IMR_B_VAL 0x00000000 +#define ACFG_GPR_B_VAL 0x00000000 +#define ACFG_PUEN_B_VAL 0xFFFFFFFF + +#define ACFG_DDIR_C_VAL 0x00000000 +#define ACFG_OCR1_C_VAL 0x00000000 +#define ACFG_OCR2_C_VAL 0x00000000 +#define ACFG_ICFA1_C_VAL 0xFFFFFFFF +#define ACFG_ICFA2_C_VAL 0xFFFFFFFF +#define ACFG_ICFB1_C_VAL 0xFFFFFFFF +#define ACFG_ICFB2_C_VAL 0xFFFFFFFF +#define ACFG_DR_C_VAL 0x00000000 +#define ACFG_GIUS_C_VAL 0xFFFFC07F +#define ACFG_ICR1_C_VAL 0x00000000 +#define ACFG_ICR2_C_VAL 0x00000000 +#define ACFG_IMR_C_VAL 0x00000000 +#define ACFG_GPR_C_VAL 0x00000000 +#define ACFG_PUEN_C_VAL 0xFFFFFF87 + +#define ACFG_DDIR_D_VAL 0x00000000 +#define ACFG_OCR1_D_VAL 0x00000000 +#define ACFG_OCR2_D_VAL 0x00000000 +#define ACFG_ICFA1_D_VAL 0xFFFFFFFF +#define ACFG_ICFA2_D_VAL 0xFFFFFFFF +#define ACFG_ICFB1_D_VAL 0xFFFFFFFF +#define ACFG_ICFB2_D_VAL 0xFFFFFFFF +#define ACFG_DR_D_VAL 0x00000000 +#define ACFG_GIUS_D_VAL 0xFFFFFFFF +#define ACFG_ICR1_D_VAL 0x00000000 +#define ACFG_ICR2_D_VAL 0x00000000 +#define ACFG_IMR_D_VAL 0x00000000 +#define ACFG_GPR_D_VAL 0x00000000 +#define ACFG_PUEN_D_VAL 0xFFFFFFFF + +#define ACFG_DDIR_E_VAL 0x00000000 +#define ACFG_OCR1_E_VAL 0x00000000 +#define ACFG_OCR2_E_VAL 0x00000000 +#define ACFG_ICFA1_E_VAL 0xFFFFFFFF +#define ACFG_ICFA2_E_VAL 0xFFFFFFFF +#define ACFG_ICFB1_E_VAL 0xFFFFFFFF +#define ACFG_ICFB2_E_VAL 0xFFFFFFFF +#define ACFG_DR_E_VAL 0x00000000 +#define ACFG_GIUS_E_VAL 0xFCFFCCF8 +#define ACFG_ICR1_E_VAL 0x00000000 +#define ACFG_ICR2_E_VAL 0x00000000 +#define ACFG_IMR_E_VAL 0x00000000 +#define ACFG_GPR_E_VAL 0x00000000 +#define ACFG_PUEN_E_VAL 0xFFFFFFFF + +#define ACFG_DDIR_F_VAL 0x00000000 +#define ACFG_OCR1_F_VAL 0x00000000 +#define ACFG_OCR2_F_VAL 0x00000000 +#define ACFG_ICFA1_F_VAL 0xFFFFFFFF +#define ACFG_ICFA2_F_VAL 0xFFFFFFFF +#define ACFG_ICFB1_F_VAL 0xFFFFFFFF +#define ACFG_ICFB2_F_VAL 0xFFFFFFFF +#define ACFG_DR_F_VAL 0x00000000 +#define ACFG_GIUS_F_VAL 0xFF7F8000 +#define ACFG_ICR1_F_VAL 0x00000000 +#define ACFG_ICR2_F_VAL 0x00000000 +#define ACFG_IMR_F_VAL 0x00000000
board/armadeus/apf27/apf27.h line 336
Quoted 10 lines
board/armadeus/apf27/apf27.h new file +10
-0+#define ACFG_GPR_F_VAL 0x00000000 +#define ACFG_PUEN_F_VAL 0xFFFFFFFF + +/* Enforce DDR signal strengh & enable USB/PP/DMA burst override bits */ +#define ACFG_GPCR_VAL 0x0003000F + +#define ACFG_ESDMISC_VAL ESDMISC_LHD+ESDMISC_MDDREN + +/* FMCR select num LPDDR RAMs and nand 16bits, 2KB pages */ +#if (CONFIG_NR_DRAM_BANKS ==1 )
But is the number of banks not detected at runtime ? And CONFIG_NR_DRAM_BANKS is always 2 in your yonfiguration file, that means there is no case with only one bank and you should drop this dead code.
Quoted 201 lines
board/armadeus/apf27/apf27.h new file +114
-0+#define ACFG_FMCR_VAL 0xFFFFFFF9 +#elif (CONFIG_NR_DRAM_BANKS ==2 ) +#define ACFG_FMCR_VAL 0xFFFFFFFB +#endif + +#define ACFG_AIPI1_PSR0_VAL 0x20040304 +#define ACFG_AIPI1_PSR1_VAL 0xDFFBFCFB +#define ACFG_AIPI2_PSR0_VAL 0x00000000 +#define ACFG_AIPI2_PSR1_VAL 0xFFFFFFFF + +/* PCCR enable DMA FEC I2C1 IIM SDHC1 */ +#define ACFG_PCCR0_VAL 0x05070410 +#define ACFG_PCCR1_VAL 0xA14A0608 + +/* +* From here, there should not be any user configuration. +* All Equations are automatic +*/ + +/* fixme none integer value (7.5ns) => 2*hclock = 15ns */ +#define ACFG_2XHCLK_LGTH (2000/CONFIG_HCLK_FREQ)/* ns */ + +/* USB 60 MHz ; ARM up to 400; HClK up to 133MHz*/ +#define CSCR_MASK 0x0300800D + +#define ACFG_CSCR_VAL \ + (CSCR_MASK \ + |((((CONFIG_SPLL_FREQ/CONFIG_USB_FREQ)-1)&0x07) << 28) \ + |((((CONFIG_MPLL_FREQ/CONFIG_ARM_FREQ)-1)&0x03) << 12) \ + |((((ACFG_CLK_FREQ/CONFIG_HCLK_FREQ)-1)&0x03) << 8)) + +/* SSIx CLKO NFC H264 MSHC */ +#define ACFG_PCDR0_VAL \ + (((((ACFG_CLK_FREQ/CONFIG_MSHC_FREQ)-1)&0x3F)<<0) \ + |((((CONFIG_HCLK_FREQ/CONFIG_NFC_FREQ)-1)&0x0F)<<6) \ + |(((((ACFG_CLK_FREQ/CONFIG_H264_FREQ)-2)*2)&0x3F)<<10)\ + |(((((ACFG_CLK_FREQ/CONFIG_SSI1_FREQ)-2)*2)&0x3F)<<16)\ + |(((CONFIG_CLK0_DIV)&0x07)<<22)\ + |(((CONFIG_CLK0_EN)&0x01)<<25)\ + |(((((ACFG_CLK_FREQ/CONFIG_SSI2_FREQ)-2)*2)&0x3F)<<26)) + +/* PERCLKx */ +#define ACFG_PCDR1_VAL \ + (((((ACFG_CLK_FREQ/CONFIG_PERIF1_FREQ)-1)&0x3F)<<0) \ + |((((ACFG_CLK_FREQ/CONFIG_PERIF2_FREQ)-1)&0x3F)<<8) \ + |((((ACFG_CLK_FREQ/CONFIG_PERIF3_FREQ)-1)&0x3F)<<16) \ + |((((ACFG_CLK_FREQ/CONFIG_PERIF4_FREQ)-1)&0x3F)<<24)) + +/* SDRAM controller programming Values */ +#if (((2 *ACFG_SDRAM_CLOCK_CYCLE_CL_1 ) > (3 *ACFG_2XHCLK_LGTH )) || \ + (ACFG_SDRAM_CLOCK_CYCLE_CL_1 <1 )) +#define REG_FIELD_SCL_VAL 3 +#define REG_FIELD_SCLIMX_VAL 0 +#else +#define REG_FIELD_SCL_VAL \ + ((2*ACFG_SDRAM_CLOCK_CYCLE_CL_1+ACFG_2XHCLK_LGTH-1)/ \ + ACFG_2XHCLK_LGTH) +#define REG_FIELD_SCLIMX_VAL REG_FIELD_SCL_VAL +#endif + +#if ((2 *ACFG_SDRAM_RC_DELAY ) > (16 *ACFG_2XHCLK_LGTH )) +#define REG_FIELD_SRC_VAL 0 +#else +#define REG_FIELD_SRC_VAL \ + ((2*ACFG_SDRAM_RC_DELAY+ACFG_2XHCLK_LGTH-1)/ \ + ACFG_2XHCLK_LGTH) +#endif + +/* TBD Power down timer ; PRCT Bit Field Encoding; burst length 8 ; FP = 0*/ +#define REG_ESDCTL_BASE_CONFIG (0x80020485\ + | (((ACFG_SDRAM_NUM_ROW-11)&0x7)<<24)\ + | (((ACFG_SDRAM_NUM_COL-8)&0x3)<<20)\ + | (((ACFG_SDRAM_REFRESH)&0x7)<<13)) + +#define ACFG_NORMAL_RW_CMD ((0x0<<28)+REG_ESDCTL_BASE_CONFIG) +#define ACFG_PRECHARGE_CMD ((0x1<<28)+REG_ESDCTL_BASE_CONFIG) +#define ACFG_AUTOREFRESH_CMD ((0x2<<28)+REG_ESDCTL_BASE_CONFIG) +#define ACFG_SET_MODE_REG_CMD ((0x3<<28)+REG_ESDCTL_BASE_CONFIG) + +/* ESDRAMC Configuration Registers : force CL=3 to lpddr */ +#define ACFG_SDRAM_ESDCFG_REGISTER_VAL (0x0\ + | (((((2*ACFG_SDRAM_EXIT_PWD+ACFG_2XHCLK_LGTH-1)/ \ + ACFG_2XHCLK_LGTH)-1)&0x3)<<21)\ + | (((ACFG_SDRAM_W2R_DELAY-1)&0x1)<<20)\ + | (((((2*ACFG_SDRAM_ROW_PRECHARGE_DELAY+ \ + ACFG_2XHCLK_LGTH-1)/ACFG_2XHCLK_LGTH)-1)&0x3)<<18) \ + | (((ACFG_SDRAM_TMRD_DELAY-1)&0x3)<<16)\ + | (((ACFG_SDRAM_TWR_DELAY)&0x1)<<15)\ + | (((((2*ACFG_SDRAM_RAS_DELAY+ACFG_2XHCLK_LGTH-1)/ \ + ACFG_2XHCLK_LGTH)-1)&0x7)<<12) \ + | (((((2*ACFG_SDRAM_RRD_DELAY+ACFG_2XHCLK_LGTH-1)/ \ + ACFG_2XHCLK_LGTH)-1)&0x3)<<10) \ + | (((REG_FIELD_SCLIMX_VAL)&0x3)<<8)\ + | (((((2*ACFG_SDRAM_RCD_DELAY+ACFG_2XHCLK_LGTH-1)/ \ + ACFG_2XHCLK_LGTH)-1)&0x7)<<4) \ + | (((REG_FIELD_SRC_VAL)&0x0F)<<0)) + +/* Issue Mode register Command to SDRAM*/ +#define ACFG_SDRAM_MODE_REGISTER_VAL\ + ((((ACFG_SDRAM_BURST_LENGTH)&0x7)<<(0))\ + | (((REG_FIELD_SCL_VAL)&0x7)<<(4))\ + | ((0)<<(3)) /* sequentiql access */ \ + /*| (((ACFG_SDRAM_SINGLE_ACCESS)&0x1)<<(1))*/) + +/* Issue Extended Mode register Command to SDRAM*/ +#define ACFG_SDRAM_EXT_MODE_REGISTER_VAL \ + ((ACFG_SDRAM_PARTIAL_ARRAY_SR<<0)\ + | (ACFG_SDRAM_DRIVE_STRENGH<<(5))\ + | (1<<(ACFG_SDRAM_NUM_COL+ACFG_SDRAM_NUM_ROW+1+2))) + +/* Issue Precharge all Command to SDRAM*/ +#define ACFG_SDRAM_PRECHARGE_ALL_VAL (1<<10) + +#endif /* __APF27_H */ board/armadeus/apf27/splboot.S new file +81
-0@@ -0,0 +1,528 @@ +/* + * IMX27 NAND Flash SPL (Secondary Program Loader) + * + * Copyright (c) 2008 Armadeus Project / eja + * + * Based on Freescale NAND SPL + * + * Copyright (C) 1998, 1999, 2000, 2001, 2002 Red Hat, Inc. + * Copyright (c) 2008-2012 Eric Jarrige <[email protected]> + * + * See file CREDITS for list of people who contributed to this + * project. + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License as + * published by the Free Software Foundation; either version 2 of + * the License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, + * MA 02111-1307 USA + */ + +#include <config.h> +#include <generated/asm-offsets.h> +#include <version.h> +#include <asm/macro.h> +#include <asm/arch/imx-regs.h> +/* #include <linux/linkage.h> */ +#include "apf27.h" + +/* + * Standard NAND flash commands + */ +#define NAND_CMD_READ0 0 + +/* Extended commands for large page devices */ +#define NAND_CMD_READSTART 0x30 +#define NAND_CMD_CACHEDPROG 0x15 + +/* Status bits */ +#define NAND_STATUS_FAIL 0x01 +#define NAND_STATUS_FAIL_N1 0x02 +#define NAND_STATUS_TRUE_READY 0x20 +#define NAND_STATUS_READY 0x40 +#define NAND_STATUS_WP 0x80 + +#define IMX_NFC_MAIN_AREA0 (0xD8000000) +#define IMX_NFC_MAIN_AREA1 (0xD8000200) +#define IMX_NFC_SPARE_AREA0 (0xD8000800) +#define IMX_NFC_REGS (0xD8000E00) + +/* + * NFC registers address offset + */ +#define NFC_OFFSET_BUF_ADDR (0x04) /* Buffer Number for Page Data + Transfer To/From Flash Mem */ +#define NFC_OFFSET_FLASH_ADDR (0x06) /* NAND Flash Address */ +#define NFC_OFFSET_FLASH_CMD (0x08) /* NAND Flash Command */ +#define NFC_OFFSET_CONFIG (0x0A) /* NFC Internal Buffer Lock + Control */ +#define NFC_OFFSET_ECC_STATUS_RESULT (0x0C) /* Controller Status/Result of + Flash Operation */ +#define NFC_OFFSET_CONFIG1 (0x1A) /* Nand Flash Operation + Configuration 1 */ +#define NFC_OFFSET_CONFIG2 (0x1C) /* Nand Flash Operation + Configuration 2 */ + +/* NFC_ECC_STATUS_RESULT Status Register Bit Fields */ +#define NFC_ECC_STAT_ERM_SHFT (2) /* ERM shift */ +#define NFC_ECC_STAT_ERROR2 (1<<1) /* non correctable error */ + +/* NFC_CONFIG Control Register Bit Fields */ +#define NFC_CONFIG_UNLOCKED (1<<1) /* unlocked */ +
board/armadeus/apf27/splboot.S line 91
Quoted 10 lines
board/armadeus/apf27/splboot.S new file +10
-0+/* NFC_CONFIG1 Control Register Bit Fields */ +#define NFC_CONFIG1_ECC_EN (1<<3) +#define NFC_CONFIG1_INT_MSK (1<<4) + +/* NFC_CONFIG2 Control Register Bit Fields */ +#define NFC_CONFIG2_INT (1<<15) /* Interrupt */ +#define NFC_CONFIG2_FDO_PAGE (1<<3) /* Flash data output */ +#define NFC_CONFIG2_FDI (1<<2) /* Flash data input */ +#define NFC_CONFIG2_FADD (1<<1) /* Flash address input */ +#define NFC_CONFIG2_FCMD (1<<0) /* Flash command input */
Using SPL, the storage driver (in your case, NAND) is not a special one, but the usual driver from u-boot is taken. In this case, code in mxc_nand, that supports mx27, should run. Instead of it, you embedded your own (ok, from Freescale code) driver in assembly. Is there any reason that forbid to take the general driver ?
board/armadeus/apf27/splboot.S line 95
Quoted 4 lines
board/armadeus/apf27/splboot.S new file +4
-0+ + .macro nand_boot + +#ifdef CONFIG_BOOT_TRACE_REG
According to README: each define whose name starts with CONFIG_ is an option - it can use generally in U-Boot code and must be documented. If your goal was to add something related to apf27 only, do not use a CONFIG_ name or add documentation for it.
Quoted 10 lines
board/armadeus/apf27/splboot.S new file +10
-0+/* + * If CONFIG_BOOT_TRACE_REG is a SDRAM address then be sure to use the following + * 2 command after SDRAM init + */ + +/* Backup state of previous boot to CONFIG_BOOT_TRACE_REG+4*/ +#define BACKUP_TRACE() \ + ldr r4, =CONFIG_BOOT_TRACE_REG; \ + ldr r3, [r4]; \ + str r3, [r4, #0x04];
board/armadeus/apf27/splboot.S line 115
Quoted 10 lines
board/armadeus/apf27/splboot.S new file +10
-0+ +/* Save a state of boot at CONFIG_BOOT_TRACE_REG */ +#define BOOT_TRACE(val) \ + ldr r4, =CONFIG_BOOT_TRACE_REG; \ + ldr r3, =val; \ + str r3, [r4]; +#else +#define BACKUP_TRACE() +#define BOOT_TRACE(val) +#endif
Everything seems to me only for debug purpose. But does not work CONFIG_SPL_CONSOLE and CONFIG_SPL_SERIAL_SUPPORT ?
Quoted 194 lines
board/armadeus/apf27/splboot.S new file +194
-0+ +nand_boot_setup: + + /* Copy SPL image from flash to SDRAM first */ + BOOT_TRACE(1) + ldr r0, =IMX_NFC_MAIN_AREA0 + add r2, r0, #(IMX_NFC_SPARE_AREA0-IMX_NFC_MAIN_AREA0) //2KB NFC Buff + ldr r1, =CONFIG_SYS_NAND_U_BOOT_DST + + BOOT_TRACE(2) +1: ldmia r0!, {r3-r10} + stmia r1!, {r3-r10} + cmp r0, r2 + blo 1b + + + + /* Jump to SDRAM */ + BOOT_TRACE(3) + ldr r1, =0x7FF + and r0, pc, r1 /* offset of pc */ + ldr r1, =CONFIG_SYS_NAND_U_BOOT_DST + add r1, r1, #0x10 + add pc, r0, r1 + nop + nop + nop + nop + +nand_copy_main: + BOOT_TRACE(4) + /* r0: nfc base. Reloaded after each page copying */ + ldr r0, =IMX_NFC_MAIN_AREA0 + + /* r1: starting flash addr to be copied. Updated constantly */ + /* bypass the first preloaded pages */ + ldr r1, =(IMX_NFC_SPARE_AREA0-IMX_NFC_MAIN_AREA0) + + /* r2: end of 1st RAM buf. Doesn't change */ + ldr r2, =IMX_NFC_MAIN_AREA1 + + /* r12: NFC register base. Doesn't change */ + ldr r12, =IMX_NFC_REGS + + ldr r11, =CONFIG_SYS_NAND_U_BOOT_DST + + /* r13: end of SDRAM address for copying. Doesn't change */ + add r13, r11, #CONFIG_SYS_NAND_U_BOOT_SIZE + + /* r11: starting SDRAM address for copying. Updated constantly */ + add r11, r11, r1 + + /* unlock internal buffer */ + ldr r3, =NFC_CONFIG_UNLOCKED + strh r3, [r12, #NFC_OFFSET_CONFIG] + + /* enable ECC and mask interrupts */ + ldr r3, =(NFC_CONFIG1_ECC_EN | NFC_CONFIG1_INT_MSK) + strh r3, [r12, #NFC_OFFSET_CONFIG1] + +nfc_read_page: + BOOT_TRACE(5) + /* send NAND_CMD_READ0 command */ + ldr r3, =NAND_CMD_READ0; + strh r3, [r12, #NFC_OFFSET_FLASH_CMD] + + ldr r3, =NFC_CONFIG2_FCMD + strh r3, [r12, #NFC_OFFSET_CONFIG2] + bl do_wait_op_done + + /* send NAND address to read. TODO small page support */ + BOOT_TRACE(6) + mov r3, r1, lsr #1 + bl do_addr_input /* 1st addr cycle */ + + mov r3, r1, lsr #9 + and r3, r3, #0x03 + bl do_addr_input /* 2nd addr cycle */ + + mov r3, r1, lsr #11 + bl do_addr_input /* 3rd addr cycle */ + + mov r3, r1, lsr #19 + bl do_addr_input /* 4th addr cycle */ + + /* Small NAND flashs (== 1Gb) support 5 addr cycles */ + mov r3, r1, lsr #27 + bl do_addr_input /* 5th addr cycle */ + + /* send NAND_CMD_READSTART command. TODO small page support */ + BOOT_TRACE(7) + mov r3, #NAND_CMD_READSTART; + strh r3, [r12, #NFC_OFFSET_FLASH_CMD] + mov r3, #NFC_CONFIG2_FCMD + strh r3, [r12, #NFC_OFFSET_CONFIG2] + bl do_wait_op_done + + /* read and copy buf 0 */ + BOOT_TRACE(8) + mov r3, #0 + strh r3, [r12, #NFC_OFFSET_BUF_ADDR] + + mov r3, #NFC_CONFIG2_FDO_PAGE + strh r3, [r12, #NFC_OFFSET_CONFIG2] + bl do_wait_op_done + + bl test_and_copy_buffer + + /* read and copy buf 1 */ + mov r3, #1 + strh r3, [r12, #NFC_OFFSET_BUF_ADDR] + + mov r3, #NFC_CONFIG2_FDO_PAGE + strh r3, [r12, #NFC_OFFSET_CONFIG2] + bl do_wait_op_done + + bl test_and_copy_buffer + + /* here we should test if 512B page flash and bypass next buffers */ + /* read and copy buf 2. TODO small page support */ + mov r3, #2 + strh r3, [r12, #NFC_OFFSET_BUF_ADDR] + + mov r3, #NFC_CONFIG2_FDO_PAGE + strh r3, [r12, #NFC_OFFSET_CONFIG2] + bl do_wait_op_done + + bl test_and_copy_buffer + + /* read and copy buf 3 */ + mov r3, #3 + strh r3, [r12, #NFC_OFFSET_BUF_ADDR] + + mov r3, #NFC_CONFIG2_FDO_PAGE + strh r3, [r12, #NFC_OFFSET_CONFIG2] + bl do_wait_op_done + + bl test_and_copy_buffer + + /* is the last page ? */ + BOOT_TRACE(12) + cmp r11, r13 + bge nand_copy_main_done + + /* r0: nfc base. Reloaded after each page copying */ + ldr r0, =IMX_NFC_MAIN_AREA0 + /* r2: end of 1st RAM buf. Doesn't change */ + ldr r2, =IMX_NFC_MAIN_AREA1 + b nfc_read_page + +nand_copy_main_done: + BOOT_TRACE(13) + .endm /* nand_boot */ + + .macro init_aipi + /* + * setup AIPI1 and AIPI2 + */ + write32 AIPI1_PSR0, ACFG_AIPI1_PSR0_VAL + write32 AIPI1_PSR1, ACFG_AIPI1_PSR1_VAL + write32 AIPI2_PSR0, ACFG_AIPI2_PSR0_VAL + write32 AIPI2_PSR1, ACFG_AIPI2_PSR1_VAL + + /* Change SDRAM signal strengh */ + ldr r0, =GPCR + ldr r1, =ACFG_GPCR_VAL + ldr r5, [r0] + orr r5, r5, r1 + str r5, [r0] + + .endm /* init_aipi */ + + .macro init_clock + ldr r0, =CSCR + /* disable MPLL/SPLL first */ + ldr r1, [r0] + bic r1, r1, #(CSCR_MPEN|CSCR_SPEN) + str r1, [r0] + + /* + * pll clock initialization predefined in apf27.h + */ + write32 MPCTL0, ACFG_MPCTL0_VAL + write32 SPCTL0, ACFG_SPCTL0_VAL + + write32 CSCR, ACFG_CSCR_VAL|CSCR_MPLL_RESTART|CSCR_SPLL_RESTART + + /* + * add some delay here + */ + mov r1, #0x1000 + 1: subs r1, r1, #0x1 + bne 1b +
board/armadeus/apf27/splboot.S line 319
Quoted 10 lines
board/armadeus/apf27/splboot.S new file +10
-0+ /* peripheral clock divider */ + write32 PCDR0, ACFG_PCDR0_VAL + write32 PCDR1, ACFG_PCDR1_VAL + + /* Configure PCCR0 and PCCR1 */ + write32 PCCR0, ACFG_PCCR0_VAL + write32 PCCR1, ACFG_PCCR1_VAL + + .endm /* init_clock */ +
Well, this code makes what the driver for NAND is supposed to do.
Quoted 11 lines
board/armadeus/apf27/splboot.S new file +11
-0+.globl board_init_f +board_init_f: + + /* + * invalidate I/D cache/TLB and drain write buffer + */ + mov r0, #0 + mcr p15, 0, r0, c7, c7, 0 /* invalidate I cache and D cache */ + mcr p15, 0, r0, c8, c7, 0 /* invalidate TLBs */ + mcr p15, 0, r0, c7, c10, 4 /* Drain the write buffer */ +
board/armadeus/apf27/splboot.S line 340
Quoted 10 lines
board/armadeus/apf27/splboot.S new file +10
-0+ /* + * disable MMU stuff and caches + */ + mrc p15, 0, r0, c1, c0, 0 + bic r0, r0, #0x00002300 /* clear bits 13, 9:8 (--V- --RS) */ + bic r0, r0, #0x00000087 /* clear bits 7, 2:0 (B--- -CAM) */ + orr r0, r0, #0x00000002 /* set bit 2 (A) Align */ + orr r0, r0, #0x00001000 /* set bit 12 (I) I-Cache */ + mcr p15, 0, r0, c1, c0, 0 +
Ok, this must be done in assembly - normally is part of lowelevel_init, while board_init_f() is already written in C.
Really I was waiting that general code is called, that is spl_nand_load_image() and then the driver function. You are pushing a parallel way that is, at the end, still the old way to boot (nand_spl).
Quoted 646 lines
board/armadeus/apf27/splboot.S new file +188
-0+init_aipi_start: + init_aipi + + /* check if sdram has been setup (running within sdram) */ + cmp pc, #0xa0000000 /* start of first sdram memory space */ + blo init_clock_start + cmp pc, #0xc0000000 /* end of second sdram memory space */ + blo regular_boot + + /* running from sdram with full code present -> regular_boot */ +init_clock_start: + init_clock + +init_sdram_start: + bl setup_sdram_ddr + + /* save state of previous boot (SDRAM is configured)*/ + BACKUP_TRACE() + + /* nand_boot BOOT_TRACE(1..13) */ + + nand_boot + + BOOT_TRACE(14) /* start regular U-Boot */ + +regular_boot: /* jump to start of next 2kiB block (U-Boot) */ + ldr r0, =0xfffff800 + and r0, r0, pc + add pc, r0, #0x800 + +do_wait_op_done: + 1: + ldrh r3, [r12, #NFC_OFFSET_CONFIG2] + ands r3, r3, #NFC_CONFIG2_INT + beq 1b + mov r3, #0x0 + strh r3, [r12, #NFC_OFFSET_CONFIG2] + mov pc, lr + +do_addr_input: + mov r9, lr + and r3, r3, #0xFF + strh r3, [r12, #NFC_OFFSET_FLASH_ADDR] + mov r3, #NFC_CONFIG2_FADD + strh r3, [r12, #NFC_OFFSET_CONFIG2] + bl do_wait_op_done + mov pc, r9 + +test_and_copy_buffer: + /* check for bad block (2 bits error in main or spare are)*/ + BOOT_TRACE(9) + ldrh r4, [r12, #NFC_OFFSET_ECC_STATUS_RESULT] + ands r4, r4, #(NFC_ECC_STAT_ERROR2| \ + (NFC_ECC_STAT_ERROR2<<NFC_ECC_STAT_ERM_SHFT)) + bne skip_bad_buffer + + /* check BI byte of the current spare buffer */ + ldr r4, =IMX_NFC_SPARE_AREA0 + ldrh r3, [r12, #NFC_OFFSET_BUF_ADDR] /* for the current buffer */ + orr r4, r3, lsl #0x04 + + /* at bi word offset 4. */ + /* Fixme position change betwwen 8 and 16 bits bus */ + ldrh r4, [r4, #0x04] + and r4, r4, #0x0FF00 /* has to be 0xFFxx */ + cmp r4, #0x0FF00 + bne skip_bad_buffer + +copy_good_buffer: + /* copying 512 bytes buffer */ + BOOT_TRACE(10) +1: ldmia r0!, {r3-r10} + stmia r11!, {r3-r10} + cmp r0, r2 + blo 1b + b end_of_copy + +skip_bad_buffer: + BOOT_TRACE(11) + /* bad pages do not contain valid data and have to be skip */ + add r0, r0, #0x200 + + /* rewind ram addr to start of buffer */ + ldr r3, =(~0x1FF) + and r11, r11, r3 + +end_of_copy: + add r2, r2, #0x200 + add r1, r1, #0x200 + + mov pc, lr + +setup_sdram_ddr: + + /* wait for SDRAM/LPDDR ready (SDRAMRDY) */ + ldr r0, =IMX_ESD_BASE + ldr r4, =ESDMISC_SDRAM_RDY +2: ldr r1, [r0, #ESDMISC_ROF] + ands r1, r1, r4 + bpl 2b + + /* LPDDR Soft Reset Mobile/Low Power DDR SDRAM. */ + ldr r0, =IMX_ESD_BASE + ldr r4, =ACFG_ESDMISC_VAL + orr r1, r4, #ESDMISC_MDDR_DL_RST + str r1, [r0, #ESDMISC_ROF] + + /* Hold for more than 200ns */ + ldr r1, =0x10000 +1: subs r1, r1, #0x1 + bne 1b + + str r4, [r0] + + ldr r0, =IMX_ESD_BASE + ldr r1, =ACFG_SDRAM_ESDCFG_REGISTER_VAL + str r1, [r0, #ESDCFG0_ROF] + + ldr r0, =IMX_ESD_BASE + ldr r1, =ACFG_PRECHARGE_CMD + str r1, [r0, #ESDCTL0_ROF] + + /* write8(0xA0001000, any value) */ + ldr r1, =PHYS_SDRAM_1+ACFG_SDRAM_PRECHARGE_ALL_VAL + strb r2, [r1] + + ldr r1, =ACFG_AUTOREFRESH_CMD + str r1, [r0, #ESDCTL0_ROF] + + ldr r4, =PHYS_SDRAM_1 /* CSD0 base address */ + + ldr r6,=0x7 /* load loop counter */ +1: str r5,[r4] /* run auto-refresh cycle to array 0 */ + subs r6,r6,#1 + bne 1b + + ldr r1, =ACFG_SET_MODE_REG_CMD + str r1, [r0, #ESDCTL0_ROF] + + /* set standard mode register */ + ldr r4, = PHYS_SDRAM_1+ACFG_SDRAM_MODE_REGISTER_VAL + strb r2, [r4] + + /* set extended mode register */ + ldr r4, =PHYS_SDRAM_1+ACFG_SDRAM_EXT_MODE_REGISTER_VAL + strb r5, [r4] + + ldr r1, =ACFG_NORMAL_RW_CMD + str r1, [r0, #ESDCTL0_ROF] + + /* 2nd sdram */ + ldr r0, =IMX_ESD_BASE + ldr r1, =ACFG_SDRAM_ESDCFG_REGISTER_VAL + str r1, [r0, #ESDCFG1_ROF] + + ldr r0, =IMX_ESD_BASE + ldr r1, =ACFG_PRECHARGE_CMD + str r1, [r0, #ESDCTL1_ROF] + + /* write8(0xB0001000, any value) */ + ldr r1, =PHYS_SDRAM_2+ACFG_SDRAM_PRECHARGE_ALL_VAL + strb r2, [r1] + + ldr r1, =ACFG_AUTOREFRESH_CMD + str r1, [r0, #ESDCTL1_ROF] + + ldr r4, =PHYS_SDRAM_2 /* CSD1 base address */ + + ldr r6,=0x7 /* load loop counter */ +1: str r5,[r4] /* run auto-refresh cycle to array 0 */ + subs r6,r6,#1 + bne 1b + + ldr r1, =ACFG_SET_MODE_REG_CMD + str r1, [r0, #ESDCTL1_ROF] + + /* set standard mode register */ + ldr r4, =PHYS_SDRAM_2+ACFG_SDRAM_MODE_REGISTER_VAL + strb r2, [r4] + + /* set extended mode register */ + ldr r4, =PHYS_SDRAM_2+ACFG_SDRAM_EXT_MODE_REGISTER_VAL + strb r2, [r4] + + ldr r1, =ACFG_NORMAL_RW_CMD + str r1, [r0, #ESDCTL1_ROF] + + mov pc, lrboards.cfg +1
-0@@ -195,6 +195,7 @@ jadecpu arm arm926ejs jadecpu syteco mx25pdk arm arm926ejs mx25pdk freescale mx25 mx25pdk:IMX_CONFIG=board/freescale/mx25pdk/imximage.cfg tx25 arm arm926ejs tx25 karo mx25 zmx25 arm arm926ejs zmx25 syteco mx25 +apf27 arm arm926ejs apf27 armadeus mx27 imx27lite arm arm926ejs imx27lite logicpd mx27 magnesium arm arm926ejs imx27lite logicpd mx27 mx23_olinuxino arm arm926ejs mx23_olinuxino olimex mxs mx23_olinuxinoinclude/configs/apf27.h new file +436
-0@@ -0,0 +1,446 @@ +/* +* +* Configuration settings for the Armadeus Project motherboard APF27 +* +* Copyright (C) 2008-2012 ej / Armadeus Project <[email protected]> +* +* This program is free software; you can redistribute it and/or +* modify it under the terms of the GNU General Public License as +* published by the Free Software Foundation; either version 2 of +* the License, or (at your option) any later version. +* +* This program is distributed in the hope that it will be useful, +* but WITHOUT ANY WARRANTY; without even the implied warranty of +* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +* GNU General Public License for more details. +* +* You should have received a copy of the GNU General Public License +* along with this program; if not, write to the Free Software +* Foundation, Inc., 59 Temple Place, Suite 330, Boston, +* MA 02111-1307 USA +*/ + +#ifndef __CONFIG_H +#define __CONFIG_H + +#define CONFIG_VERSION_VARIABLE +#define CONFIG_ENV_VERSION "3.3" +#define CONFIG_IDENT_STRING " apf27 patch 3.9" +#define CONFIG_BOARD_NAME apf27 + +/* +* SoC configurations +*/ +#define CONFIG_ARM926EJS /* this is an ARM926EJS CPU */ +#define CONFIG_MX27 /* in a Freescale i.MX27 Chip */ +#define CONFIG_MACH_TYPE 1698/* APF27 */ + +/* +* Enable the call to miscellaneous platform dependent initialization. +*/ +#define CONFIG_SYS_NO_FLASH /* to be define before <config_cmd_default.h> */ +#define CONFIG_MISC_INIT_R +#define CONFIG_BOARD_EARLY_INIT_F +#ifdef CONFIG_SPL_BUILD +#define CONFIG_SKIP_LOWLEVEL_INIT +#endif + +/* +* Board display option +*/ +#define CONFIG_DISPLAY_BOARDINFO +#define CONFIG_DISPLAY_CPUINFO + +/* +* SPL +*/ +/* Copy SPL+U-Boot here */ +#define CONFIG_SYS_NAND_U_BOOT_DST (CONFIG_SYS_TEXT_BASE - 0x800) +/* Size is the partion size */ +#define CONFIG_SYS_NAND_U_BOOT_SIZE CONFIG_SYS_MONITOR_LEN +/* Build image with spl and u-boot */ +#define CONFIG_NAND_U_BOOT +#define CONFIG_SPL_PAD_TO 0xa0000800 + +/* +* SPL +*/ +#define CONFIG_SPL +#define CONFIG_SPL_TEXT_BASE 0xa0000000 +#define CONFIG_SPL_STACK 0xffffff00 +#define CONFIG_SYS_NAND_U_BOOT_START 0xa0000800 +#define CONFIG_SYS_NAND_U_BOOT_OFFS 0x800 + +/* +* BOOTP options +*/ +#define CONFIG_BOOTP_SUBNETMASK +#define CONFIG_BOOTP_GATEWAY +#define CONFIG_BOOTP_HOSTNAME +#define CONFIG_BOOTP_BOOTPATH +#define CONFIG_BOOTP_BOOTFILESIZE +#define CONFIG_BOOTP_DNS +#define CONFIG_BOOTP_DNS2 + +#define CONFIG_HOSTNAME CONFIG_BOARD_NAME +#define CONFIG_ROOTPATH "/tftpboot/" __stringify(CONFIG_BOARD_NAME) "-root" + +/* +* U-Boot Commands +*/ +#include <config_cmd_default.h> + +#define CONFIG_CMD_ASKENV /* ask for env variable */ +#define CONFIG_CMD_BSP /* Board Specific functions */ +#define CONFIG_CMD_CACHE /* icache, dcache */ +#define CONFIG_CMD_DATE +#define CONFIG_CMD_DHCP /* DHCP Support */ +#define CONFIG_CMD_DNS +#define CONFIG_CMD_EEPROM +#define CONFIG_CMD_EXT2 +#define CONFIG_CMD_FAT /* FAT support */ +#define CONFIG_CMD_IMX_FUSE /* imx iim fuse */ +#define CONFIG_CMD_I2C +#define CONFIG_CMD_MII /* MII support */ +#define CONFIG_CMD_MMC +#define CONFIG_CMD_MTDPARTS /* MTD partition support */ +#define CONFIG_CMD_NAND /* NAND support */ +#define CONFIG_CMD_NAND_LOCK_UNLOCK +#define CONFIG_CMD_NAND_TRIMFFS +#define CONFIG_CMD_NFS /* NFS support */ +#define CONFIG_CMD_PING /* ping support */ +#define CONFIG_CMD_SETEXPR /* setexpr support */ +#define CONFIG_CMD_UBI +#define CONFIG_CMD_UBIFS + +/* +* Memory configurations +*/ +#define CONFIG_NR_DRAM_POPULATED 1 +#define CONFIG_NR_DRAM_BANKS 2 + +#define ACFG_SDRAM_MBYTE_SYZE 64 + +#define PHYS_SDRAM_1 0xA0000000 +#define PHYS_SDRAM_2 0xB0000000 +#define CONFIG_SYS_SDRAM_BASE PHYS_SDRAM_1 +#define CONFIG_SYS_MALLOC_LEN (CONFIG_ENV_SIZE + (512<<10)) +#define CONFIG_SYS_MEMTEST_START 0xA0000000/* memtest test area */ +#define CONFIG_SYS_MEMTEST_END 0xA0300000/* 3 MiB RAM test */ + +#define CONFIG_SYS_INIT_SP_ADDR (CONFIG_SYS_SDRAM_BASE \ + + PHYS_SDRAM_1_SIZE - 0x0100000) + +#define CONFIG_SYS_TEXT_BASE 0xA0000800 + +/* +* FLASH organization +*/ +#define ACFG_MONITOR_OFFSET 0x00000000 +#define CONFIG_SYS_MONITOR_LEN 0x00100000/* 1MiB */ +#define CONFIG_ENV_IS_IN_NAND +#define CONFIG_ENV_OVERWRITE +#define CONFIG_ENV_OFFSET 0x00100000/* NAND offset */ +#define CONFIG_ENV_SIZE 0x00020000/* 128kB */ +#define CONFIG_ENV_RANGE 0X00080000/* 512kB */ +#define CONFIG_ENV_OFFSET_REDUND \ + (CONFIG_ENV_OFFSET + CONFIG_ENV_RANGE)/* +512kB */ +#define CONFIG_ENV_SIZE_REDUND CONFIG_ENV_SIZE/* 512kB */ +#define CONFIG_FIRMWARE_OFFSET 0x00200000 +#define CONFIG_KERNEL_OFFSET 0x00300000 +#define CONFIG_ROOTFS_OFFSET 0x00800000 + +#define CONFIG_MTDMAP "mxc_nand.0" +#define MTDIDS_DEFAULT "nand0=" CONFIG_MTDMAP +#define MTDPARTS_DEFAULT "mtdparts=" CONFIG_MTDMAP \ + ":1M(u-boot)ro," \ + "512K(env)," \ + "512K(env2)," \ + "512K(firmware)," \ + "512K(dtb)," \ + "5M(kernel)," \ + "-(rootfs)" + +/* +* U-Boot general configurations +*/ +#define CONFIG_SYS_LONGHELP +#define CONFIG_SYS_PROMPT "BIOS> "/* prompt string */ +#define CONFIG_SYS_CBSIZE 2048/* console I/O buffer */ +#define CONFIG_SYS_PBSIZE \ + (CONFIG_SYS_CBSIZE+sizeof(CONFIG_SYS_PROMPT)+16) +/* Print buffer size */ +#define CONFIG_SYS_MAXARGS 16/* max command args */ +#define CONFIG_SYS_BARGSIZE CONFIG_SYS_CBSIZE +/* Boot argument buffer size */ +#define CONFIG_AUTO_COMPLETE +#define CONFIG_CMDLINE_EDITING +#define CONFIG_SYS_HUSH_PARSER /* enable the "hush" shell */ +#define CONFIG_SYS_PROMPT_HUSH_PS2 "> "/* secondary prompt string */ + +/* +* Boot Linux +*/ +#define CONFIG_CMDLINE_TAG /* send commandline to Kernel */ +#define CONFIG_SETUP_MEMORY_TAGS /* send memory definition to kernel */ +#define CONFIG_INITRD_TAG /* send initrd params */ + +#define CONFIG_OF_LIBFDT + +#define CONFIG_BOOTDELAY 5 +#define CONFIG_ZERO_BOOTDELAY_CHECK +#define CONFIG_BOOTFILE __stringify(CONFIG_BOARD_NAME) "-linux.bin" +#define CONFIG_BOOTARGS "console=" __stringify(ACFG_CONSOLE_DEV) "," \ + __stringify(CONFIG_BAUDRATE) " " MTDPARTS_DEFAULT \ + " ubi.mtd=rootfs root=ubi0:rootfs rootfstype=ubifs " + +#define ACFG_CONSOLE_DEV ttySMX0 +#define CONFIG_BOOTCOMMAND "run ubifsboot" +#define CONFIG_SYS_AUTOLOAD "no" +/* +* Default load address for user programs and kernel +*/ +#define CONFIG_LOADADDR 0xA0000000 +#define CONFIG_SYS_LOAD_ADDR CONFIG_LOADADDR + +/* +* Extra Environments +*/ +#define CONFIG_EXTRA_ENV_SETTINGS \ + "env_version=" CONFIG_ENV_VERSION "\0" \ + "consoledev=" __stringify(ACFG_CONSOLE_DEV) "\0" \ + "mtdparts=" MTDPARTS_DEFAULT "\0" \ + "partition=nand0,6\0" \ + "u-boot_addr=" __stringify(ACFG_MONITOR_OFFSET) "\0" \ + "env_addr=" __stringify(CONFIG_ENV_OFFSET) "\0" \ + "firmware_addr=" __stringify(CONFIG_FIRMWARE_OFFSET) "\0" \ + "kernel_addr=" __stringify(CONFIG_KERNEL_OFFSET) "\0" \ + "rootfs_addr=" __stringify(CONFIG_ROOTFS_OFFSET) "\0" \ + "board_name=" __stringify(CONFIG_BOARD_NAME) "\0" \ + "kernel_addr_r=A0000000\0" \ + "addnfsargs=setenv bootargs ${bootargs} " \ + "root=/dev/nfs rw nfsroot=${serverip}:${rootpath}\0" \ + "addubifsargs=setenv bootargs ${bootargs} " \ + "ubi.mtd=rootfs root=ubi0:rootfs rootfstype=ubifs\0" \ + "addmmcargs=setenv bootargs ${bootargs} " \ + "root=/dev/mmcblk0p1 rootfstype=ext2\0" \ + "addipargs=setenv bootargs ${bootargs} " \ + "ip=${ipaddr}:${serverip}:${gatewayip}:${netmask}:" \ + "${hostname}:eth0:off \0" \ + "nfsboot=setenv bootargs console=${consoledev},${baudrate} " \ + "${mtdparts} ${extrabootargs}; run addnfsargs addipargs;"\ + "nfs ${kernel_addr_r} " \ + "${serverip}:${rootpath}/boot/${board_name}-linux.bin;" \ + "if test -n ${fdt_addr_r} ; then " \ + "nand read ${fdt_addr_r} dtb; fi;" \ + "bootm ${kernel_addr_r} - ${fdt_addr_r}\0" \ + "ubifsboot=setenv bootargs console=${consoledev},${baudrate} " \ + "${mtdparts} ${extrabootargs};run addubifsargs addipargs;"\ + "if test -n ${fdt_addr_r} ; then " \ + "nand read ${fdt_addr_r} dtb; fi;" \ + "nboot ${kernel_addr_r} kernel;" \ + "bootm ${kernel_addr_r} - ${fdt_addr_r}\0" \ + "mmcboot=setenv bootargs console=${consoledev},${baudrate} " \ + "${mtdparts} ${extrabootargs}; run addmmcargs addipargs;"\ + "if test -n ${fdt_addr_r} ; then " \ + "nand read ${fdt_addr_r} dtb; fi;" \ + "mmc dev 0; ext2load mmc 0 ${kernel_addr_r}" \ + " /boot/${board_name}-linux.bin;" \ + "bootm ${kernel_addr_r} - ${fdt_addr_r}\0" \ + "boot_nfs_fallback_flash=if ping ${serverip};" \ + "then echo Boot over NFS; run nfsboot;" \ + "else echo Boot from the board; run ubifsboot;" \ + "fi\0" \ + "firmware_autoload=0\0" \ + "flash_uboot=nand unlock ${u-boot_addr} ;" \ + "nand erase.part u-boot;" \ + "if nand write.trimffs ${fileaddr} ${u-boot_addr} ${filesize};"\ + "then nand lock; nand unlock ${env_addr};" \ + "echo Flashing of uboot succeed;" \ + "else echo Flashing of uboot failed;" \ + "fi; \0" \ + "flash_firmware=nand erase.part firmware;" \ + "if nand write.trimffs ${fileaddr} ${firmware_addr} ${filesize};"\ + "then echo Flashing of Firmware succeed;" \ + "else echo Flashing of Firmware failed;" \ + "fi\0" \ + "flash_kernel=nand erase.part kernel;" \ + "if nand write.trimffs ${fileaddr} ${kernel_addr} ${filesize};"\ + "then echo Flashing of kernel succeed;" \ + "else echo Flashing of kernel failed;" \ + "fi\0" \ + "flash_rootfs=nand erase.part rootfs;" \ + "if nand write.trimffs ${fileaddr} ${rootfs_addr} ${filesize};"\ + "then echo Flashing of rootfs succeed;" \ + "else echo Flashing of rootfs failed;" \ + "fi\0" \ + "flash_dtb=nand erase.part dtb;" \ + "if nand write.trimffs ${fileaddr} dtb ${filesize};" \ + "then echo Flashing of rootfs succeed;" \ + "else echo Flashing of rootfs failed;" \ + "fi\0" \ + "flash_reset_env=env default -f -a; saveenv;" \ + "echo Flash environment variables erased!\0" \ + "download_uboot=tftpboot ${loadaddr} ${board_name}-u-boot-nand.bin\0" \ + "download_kernel=tftpboot ${loadaddr} ${board_name}-linux.bin\0" \ + "download_rootfs=tftpboot ${loadaddr} ${board_name}-rootfs.ubi\0" \ + "download_dtb=tftpboot ${loadaddr} imx27-${board_name}.dtb\0" \ + "update_uboot=run download_uboot flash_uboot\0" \ + "update_kernel=run download_kernel flash_kernel\0" \ + "update_rootfs=run download_rootfs flash_rootfs\0" \ + "update_dtb=run download_dtb flash_dtb\0" \ + "update_all=run download_kernel flash_kernel download_rootfs " \ + "flash_rootfs download_uboot flash_uboot\0" \ + "unlock_regs=mw 10000008 0; mw 10020008 0\0" \ + +/* +* Serial Driver +*/ +#define CONFIG_MXC_UART +#define CONFIG_CONS_INDEX 1 +#define CONFIG_BAUDRATE 115200 +#define CONFIG_MXC_UART_BASE UART1_BASE + +/* +* GPIO +*/ +#define CONFIG_MXC_GPIO + +/* +* NOR +*/ + +/* +* NAND +*/ +#define CONFIG_NAND_MXC + +#define CONFIG_MXC_NAND_REGS_BASE 0xD8000000 +#define CONFIG_SYS_NAND_BASE CONFIG_MXC_NAND_REGS_BASE +#define CONFIG_SYS_MAX_NAND_DEVICE 1 + +#define CONFIG_MXC_NAND_HWECC +#define CONFIG_SYS_NAND_LARGEPAGE +#define CONFIG_SYS_NAND_BUSWIDTH_16BIT +#define NAND_MAX_CHIPS 1 + + +#define CONFIG_FLASH_SHOW_PROGRESS 45 +#define CONFIG_SYS_NAND_QUIET 1 + +#define CONFIG_BOOT_TRACE_REG 0xAFFFFFF8/* Addr to store traces of SPL boot*/ + +/* +* Partitions & Filsystems +*/ +#define CONFIG_MTD_DEVICE +#define CONFIG_MTD_PARTITIONS +#define CONFIG_DOS_PARTITION +#define CONFIG_SUPPORT_VFAT + +/* +* UBIFS +*/ +#define CONFIG_RBTREE +#define CONFIG_LZO + +/* +* Ethernet (on SOC imx FEC) +*/ +#define CONFIG_FEC_MXC +#define CONFIG_FEC_MXC_PHYADDR 0x1f +#define CONFIG_MII /* MII PHY management */ + +/* +* FPGA +*/ +#define CONFIG_FPGA CONFIG_SYS_SPARTAN3 +#define CONFIG_FPGA_COUNT 1 +#define CONFIG_FPGA_XILINX +#define CONFIG_FPGA_SPARTAN3 +#define CONFIG_SYS_FPGA_WAIT 20000/* 20 ms */ +#define CONFIG_SYS_FPGA_PROG_FEEDBACK +#define CONFIG_SYS_FPGA_CHECK_CTRLC +#define CONFIG_SYS_FPGA_CHECK_ERROR + +/* +* Fuses - IIM +*/ +#ifdef CONFIG_CMD_IMX_FUSE +#define IIM_MAC_BANK 0 +#define IIM_MAC_ROW 5 +#define IIM0_SCC_KEY 11 +#define IIM1_SUID 1 +#endif + +/* +* I2C +*/ + +#ifdef CONFIG_CMD_I2C +#define CONFIG_HARD_I2C +#define CONFIG_I2C_MXC +#define CONFIG_I2C_MULTI_BUS +#define CONFIG_SYS_I2C_BASE IMX_I2C1_BASE +#define CONFIG_SYS_I2C_SPEED 100000/* 100 kHz */ +#define CONFIG_SYS_I2C_SLAVE 0x7F +#define CONFIG_SYS_I2C_NOPROBES { } + +#ifdef CONFIG_CMD_EEPROM +# define CONFIG_SYS_I2C_EEPROM_ADDR 0x50/* EEPROM 24LC02 */ +# define CONFIG_SYS_I2C_EEPROM_ADDR_LEN 1/* bytes of address */ +#define CONFIG_SYS_EEPROM_PAGE_WRITE_BITS 3 +#define CONFIG_SYS_EEPROM_PAGE_WRITE_DELAY_MS 10/* msec */ +#endif /* CONFIG_CMD_EEPROM */ +#endif /* CONFIG_CMD_I2C */ + +/* +* SD/MMC +*/ +#ifdef CONFIG_CMD_MMC +#define CONFIG_MMC +#define CONFIG_GENERIC_MMC +#define CONFIG_MXC_MMC +#define CONFIG_MXC_MCI_REGS_BASE 0x10014000 +#endif + +/* +* RTC +*/ +#ifdef CONFIG_CMD_DATE +#define CONFIG_RTC_DS1374 +#define CONFIG_SYS_RTC_BUS_NUM 0 +#endif /* CONFIG_CMD_DATE */ + +/* +* Clocks +*/ +#define CONFIG_SYS_HZ 1000/* Ticks per second */ + +/* +* PLL +* +* 31 | x |x| x x x x |x x x x x x x x x x |x x|x x x x|x x x x x x x x x x| 0 +* |CPLM|X|----PD---|--------MFD---------|XXX|--MFI--|-----MFN-----------| +*/ +#define CONFIG_MX27_CLK32 32768/* 32768 or 32000 Hz crystal */ + +#if (ACFG_SDRAM_MBYTE_SYZE ==64 )/* micron MT46H16M32LF -6 */ +/* micron 64MB */ +#define PHYS_SDRAM_1_SIZE 0x04000000/* 64 MB */ +#define PHYS_SDRAM_2_SIZE 0x04000000/* 64 MB */ +#endif + +#if (ACFG_SDRAM_MBYTE_SYZE ==128 ) +/* micron 128MB */ +#define PHYS_SDRAM_1_SIZE 0x08000000/* 128 MB */
include/configs/apf27.h line 446
Quoted 10 lines
include/configs/apf27.h new file +10
-0+#define PHYS_SDRAM_2_SIZE 0x08000000/* 128 MB */ +#endif + +#if (ACFG_SDRAM_MBYTE_SYZE ==256 ) +/* micron 256MB */ +#define PHYS_SDRAM_1_SIZE 0x10000000/* 256 MB */ +#define PHYS_SDRAM_2_SIZE 0x10000000/* 256 MB */ +#endif + +#endif /* __CONFIG_H */
Best regards,
Stefano Babic
Signature · 5 lines
===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de =====================================================================
PATCH v4 2/4 apf27: add support for the armadeus APF27 board
Hi Stefano,
Quoted 337 lines · Stefano Babic
Quoted 87 lines
board/armadeus/apf27/splboot.S new file +81
-0@@ -0,0 +1,528 @@ +/* + * IMX27 NAND Flash SPL (Secondary Program Loader) + * + * Copyright (c) 2008 Armadeus Project / eja + * + * Based on Freescale NAND SPL + * + * Copyright (C) 1998, 1999, 2000, 2001, 2002 Red Hat, Inc. + * Copyright (c) 2008-2012 Eric Jarrige<[email protected]> + * + * See file CREDITS for list of people who contributed to this + * project. + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License as + * published by the Free Software Foundation; either version 2 of + * the License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, + * MA 02111-1307 USA + */ + +#include<config.h> +#include<generated/asm-offsets.h> +#include<version.h> +#include<asm/macro.h> +#include<asm/arch/imx-regs.h> +/* #include<linux/linkage.h> */ +#include "apf27.h" + +/* + * Standard NAND flash commands + */ +#define NAND_CMD_READ0 0 + +/* Extended commands for large page devices */ +#define NAND_CMD_READSTART 0x30 +#define NAND_CMD_CACHEDPROG 0x15 + +/* Status bits */ +#define NAND_STATUS_FAIL 0x01 +#define NAND_STATUS_FAIL_N1 0x02 +#define NAND_STATUS_TRUE_READY 0x20 +#define NAND_STATUS_READY 0x40 +#define NAND_STATUS_WP 0x80 + +#define IMX_NFC_MAIN_AREA0 (0xD8000000) +#define IMX_NFC_MAIN_AREA1 (0xD8000200) +#define IMX_NFC_SPARE_AREA0 (0xD8000800) +#define IMX_NFC_REGS (0xD8000E00) + +/* + * NFC registers address offset + */ +#define NFC_OFFSET_BUF_ADDR (0x04) /* Buffer Number for Page Data + Transfer To/From Flash Mem */ +#define NFC_OFFSET_FLASH_ADDR (0x06) /* NAND Flash Address */ +#define NFC_OFFSET_FLASH_CMD (0x08) /* NAND Flash Command */ +#define NFC_OFFSET_CONFIG (0x0A) /* NFC Internal Buffer Lock + Control */ +#define NFC_OFFSET_ECC_STATUS_RESULT (0x0C) /* Controller Status/Result of + Flash Operation */ +#define NFC_OFFSET_CONFIG1 (0x1A) /* Nand Flash Operation + Configuration 1 */ +#define NFC_OFFSET_CONFIG2 (0x1C) /* Nand Flash Operation + Configuration 2 */ + +/* NFC_ECC_STATUS_RESULT Status Register Bit Fields */ +#define NFC_ECC_STAT_ERM_SHFT (2) /* ERM shift */ +#define NFC_ECC_STAT_ERROR2 (1<<1) /* non correctable error */ + +/* NFC_CONFIG Control Register Bit Fields */ +#define NFC_CONFIG_UNLOCKED (1<<1) /* unlocked */ +board/armadeus/apf27/splboot.S line 91
Quoted 10 lines
board/armadeus/apf27/splboot.S new file +10
-0+/* NFC_CONFIG1 Control Register Bit Fields */ +#define NFC_CONFIG1_ECC_EN (1<<3) +#define NFC_CONFIG1_INT_MSK (1<<4) + +/* NFC_CONFIG2 Control Register Bit Fields */ +#define NFC_CONFIG2_INT (1<<15) /* Interrupt */ +#define NFC_CONFIG2_FDO_PAGE (1<<3) /* Flash data output */ +#define NFC_CONFIG2_FDI (1<<2) /* Flash data input */ +#define NFC_CONFIG2_FADD (1<<1) /* Flash address input */ +#define NFC_CONFIG2_FCMD (1<<0) /* Flash command input */Using SPL, the storage driver (in your case, NAND) is not a special one, but the usual driver from u-boot is taken. In this case, code in
mxc_nand, that supports mx27, should run. Instead of it, you embedded your own (ok, from Freescale code) driver in assembly. Is there any reason that forbid to take the general driver ?board/armadeus/apf27/splboot.S line 95
Quoted 4 lines
board/armadeus/apf27/splboot.S new file +4
-0+ + .macro nand_boot + +#ifdef CONFIG_BOOT_TRACE_REGAccording to README: each define whose name starts with
CONFIG_is an option - it can use generally in U-Boot code and must be documented. If your goal was to add something related to apf27 only, do not use aCONFIG_name or add documentation for it.Quoted 10 lines
board/armadeus/apf27/splboot.S new file +10
-0+/* + * If CONFIG_BOOT_TRACE_REG is a SDRAM address then be sure to use the following + * 2 command after SDRAM init + */ + +/* Backup state of previous boot to CONFIG_BOOT_TRACE_REG+4*/ +#define BACKUP_TRACE() \ + ldr r4, =CONFIG_BOOT_TRACE_REG; \ + ldr r3, [r4]; \ + str r3, [r4, #0x04];board/armadeus/apf27/splboot.S line 115
Quoted 10 lines
board/armadeus/apf27/splboot.S new file +10
-0+ +/* Save a state of boot at CONFIG_BOOT_TRACE_REG */ +#define BOOT_TRACE(val) \ + ldr r4, =CONFIG_BOOT_TRACE_REG; \ + ldr r3, =val; \ + str r3, [r4]; +#else +#define BACKUP_TRACE() +#define BOOT_TRACE(val) +#endifEverything seems to me only for debug purpose. But does not work
CONFIG_SPL_CONSOLEandCONFIG_SPL_SERIAL_SUPPORT?Quoted 198 lines
board/armadeus/apf27/splboot.S new file +198
-0+ +nand_boot_setup: + + /* Copy SPL image from flash to SDRAM first */ + BOOT_TRACE(1) + ldr r0, =IMX_NFC_MAIN_AREA0 + add r2, r0, #(IMX_NFC_SPARE_AREA0-IMX_NFC_MAIN_AREA0) //2KB NFC Buff + ldr r1, =CONFIG_SYS_NAND_U_BOOT_DST + + BOOT_TRACE(2) +1: ldmia r0!, {r3-r10} + stmia r1!, {r3-r10} + cmp r0, r2 + blo 1b + + + + /* Jump to SDRAM */ + BOOT_TRACE(3) + ldr r1, =0x7FF + and r0, pc, r1 /* offset of pc */ + ldr r1, =CONFIG_SYS_NAND_U_BOOT_DST + add r1, r1, #0x10 + add pc, r0, r1 + nop + nop + nop + nop + +nand_copy_main: + BOOT_TRACE(4) + /* r0: nfc base. Reloaded after each page copying */ + ldr r0, =IMX_NFC_MAIN_AREA0 + + /* r1: starting flash addr to be copied. Updated constantly */ + /* bypass the first preloaded pages */ + ldr r1, =(IMX_NFC_SPARE_AREA0-IMX_NFC_MAIN_AREA0) + + /* r2: end of 1st RAM buf. Doesn't change */ + ldr r2, =IMX_NFC_MAIN_AREA1 + + /* r12: NFC register base. Doesn't change */ + ldr r12, =IMX_NFC_REGS + + ldr r11, =CONFIG_SYS_NAND_U_BOOT_DST + + /* r13: end of SDRAM address for copying. Doesn't change */ + add r13, r11, #CONFIG_SYS_NAND_U_BOOT_SIZE + + /* r11: starting SDRAM address for copying. Updated constantly */ + add r11, r11, r1 + + /* unlock internal buffer */ + ldr r3, =NFC_CONFIG_UNLOCKED + strh r3, [r12, #NFC_OFFSET_CONFIG] + + /* enable ECC and mask interrupts */ + ldr r3, =(NFC_CONFIG1_ECC_EN | NFC_CONFIG1_INT_MSK) + strh r3, [r12, #NFC_OFFSET_CONFIG1] + +nfc_read_page: + BOOT_TRACE(5) + /* send NAND_CMD_READ0 command */ + ldr r3, =NAND_CMD_READ0; + strh r3, [r12, #NFC_OFFSET_FLASH_CMD] + + ldr r3, =NFC_CONFIG2_FCMD + strh r3, [r12, #NFC_OFFSET_CONFIG2] + bl do_wait_op_done + + /* send NAND address to read. TODO small page support */ + BOOT_TRACE(6) + mov r3, r1, lsr #1 + bl do_addr_input /* 1st addr cycle */ + + mov r3, r1, lsr #9 + and r3, r3, #0x03 + bl do_addr_input /* 2nd addr cycle */ + + mov r3, r1, lsr #11 + bl do_addr_input /* 3rd addr cycle */ + + mov r3, r1, lsr #19 + bl do_addr_input /* 4th addr cycle */ + + /* Small NAND flashs (== 1Gb) support 5 addr cycles */ + mov r3, r1, lsr #27 + bl do_addr_input /* 5th addr cycle */ + + /* send NAND_CMD_READSTART command. TODO small page support */ + BOOT_TRACE(7) + mov r3, #NAND_CMD_READSTART; + strh r3, [r12, #NFC_OFFSET_FLASH_CMD] + mov r3, #NFC_CONFIG2_FCMD + strh r3, [r12, #NFC_OFFSET_CONFIG2] + bl do_wait_op_done + + /* read and copy buf 0 */ + BOOT_TRACE(8) + mov r3, #0 + strh r3, [r12, #NFC_OFFSET_BUF_ADDR] + + mov r3, #NFC_CONFIG2_FDO_PAGE + strh r3, [r12, #NFC_OFFSET_CONFIG2] + bl do_wait_op_done + + bl test_and_copy_buffer + + /* read and copy buf 1 */ + mov r3, #1 + strh r3, [r12, #NFC_OFFSET_BUF_ADDR] + + mov r3, #NFC_CONFIG2_FDO_PAGE + strh r3, [r12, #NFC_OFFSET_CONFIG2] + bl do_wait_op_done + + bl test_and_copy_buffer + + /* here we should test if 512B page flash and bypass next buffers */ + /* read and copy buf 2. TODO small page support */ + mov r3, #2 + strh r3, [r12, #NFC_OFFSET_BUF_ADDR] + + mov r3, #NFC_CONFIG2_FDO_PAGE + strh r3, [r12, #NFC_OFFSET_CONFIG2] + bl do_wait_op_done + + bl test_and_copy_buffer + + /* read and copy buf 3 */ + mov r3, #3 + strh r3, [r12, #NFC_OFFSET_BUF_ADDR] + + mov r3, #NFC_CONFIG2_FDO_PAGE + strh r3, [r12, #NFC_OFFSET_CONFIG2] + bl do_wait_op_done + + bl test_and_copy_buffer + + /* is the last page ? */ + BOOT_TRACE(12) + cmp r11, r13 + bge nand_copy_main_done + + /* r0: nfc base. Reloaded after each page copying */ + ldr r0, =IMX_NFC_MAIN_AREA0 + /* r2: end of 1st RAM buf. Doesn't change */ + ldr r2, =IMX_NFC_MAIN_AREA1 + b nfc_read_page + +nand_copy_main_done: + BOOT_TRACE(13) + .endm /* nand_boot */ + + .macro init_aipi + /* + * setup AIPI1 and AIPI2 + */ + write32 AIPI1_PSR0, ACFG_AIPI1_PSR0_VAL + write32 AIPI1_PSR1, ACFG_AIPI1_PSR1_VAL + write32 AIPI2_PSR0, ACFG_AIPI2_PSR0_VAL + write32 AIPI2_PSR1, ACFG_AIPI2_PSR1_VAL + + /* Change SDRAM signal strengh */ + ldr r0, =GPCR + ldr r1, =ACFG_GPCR_VAL + ldr r5, [r0] + orr r5, r5, r1 + str r5, [r0] + + .endm /* init_aipi */ + + .macro init_clock + ldr r0, =CSCR + /* disable MPLL/SPLL first */ + ldr r1, [r0] + bic r1, r1, #(CSCR_MPEN|CSCR_SPEN) + str r1, [r0] + + /* + * pll clock initialization predefined in apf27.h + */ + write32 MPCTL0, ACFG_MPCTL0_VAL + write32 SPCTL0, ACFG_SPCTL0_VAL + + write32 CSCR, ACFG_CSCR_VAL|CSCR_MPLL_RESTART|CSCR_SPLL_RESTART + + /* + * add some delay here + */ + mov r1, #0x1000 + 1: subs r1, r1, #0x1 + bne 1b + + /* peripheral clock divider */ + write32 PCDR0, ACFG_PCDR0_VAL + write32 PCDR1, ACFG_PCDR1_VAL +
Quoted 8 lines
board/armadeus/apf27/splboot.S line 319
Quoted 6 lines
board/armadeus/apf27/splboot.S new file +6
-0+ /* Configure PCCR0 and PCCR1 */ + write32 PCCR0, ACFG_PCCR0_VAL + write32 PCCR1, ACFG_PCCR1_VAL + + .endm /* init_clock */ +Well, this code makes what the driver for NAND is supposed to do.
Yes, this is a nand driver written in asm.
The goal is the have a spl smaller than 2 KB.
Quoted 26 lines
Quoted 12 lines
board/armadeus/apf27/splboot.S new file +11
-0+.globl board_init_f +board_init_f: + + /* + * invalidate I/D cache/TLB and drain write buffer + */ + mov r0, #0 + mcr p15, 0, r0, c7, c7, 0 /* invalidate I cache and D cache */ + mcr p15, 0, r0, c8, c7, 0 /* invalidate TLBs */ + mcr p15, 0, r0, c7, c10, 4 /* Drain the write buffer */ +board/armadeus/apf27/splboot.S line 340
Quoted 10 lines
board/armadeus/apf27/splboot.S new file +10
-0+ /* + * disable MMU stuff and caches + */ + mrc p15, 0, r0, c1, c0, 0 + bic r0, r0, #0x00002300 /* clear bits 13, 9:8 (--V- --RS) */ + bic r0, r0, #0x00000087 /* clear bits 7, 2:0 (B--- -CAM) */ + orr r0, r0, #0x00000002 /* set bit 2 (A) Align */ + orr r0, r0, #0x00001000 /* set bit 12 (I) I-Cache */ + mcr p15, 0, r0, c1, c0, 0 +Ok, this must be done in assembly - normally is part of
lowelevel_init, whileboard_init_f()is already written in C.
Quoted 3 lines
Really I was waiting that general code is called, that is
spl_nand_load_image()and then the driver function. You are pushing a parallel way that is, at the end, still the old way to boot (nand_spl).
This board boot from nand, so the SPL should be lower than 2KB. The SPL start from NFC (nand) buffer, and It has to :
- initialize the ram
- copy itself to another location (ram)
- jump to this another location
- copy u-boot from nand to ram
- jump to ram
We have tried to use SPL framework (with nand driver), but the result is a SPL bigger than 2 KB, so we can't use it.
To have a SPL smaller than 2 KB, we have written everything in assembler.
Are you agree with this solution for the SPL ?
Quoted 182 lines
Quoted 182 lines
board/armadeus/apf27/splboot.S new file +180
-0+init_aipi_start: + init_aipi + + /* check if sdram has been setup (running within sdram) */ + cmp pc, #0xa0000000 /* start of first sdram memory space */ + blo init_clock_start + cmp pc, #0xc0000000 /* end of second sdram memory space */ + blo regular_boot + + /* running from sdram with full code present -> regular_boot */ +init_clock_start: + init_clock + +init_sdram_start: + bl setup_sdram_ddr + + /* save state of previous boot (SDRAM is configured)*/ + BACKUP_TRACE() + + /* nand_boot BOOT_TRACE(1..13) */ + + nand_boot + + BOOT_TRACE(14) /* start regular U-Boot */ + +regular_boot: /* jump to start of next 2kiB block (U-Boot) */ + ldr r0, =0xfffff800 + and r0, r0, pc + add pc, r0, #0x800 + +do_wait_op_done: + 1: + ldrh r3, [r12, #NFC_OFFSET_CONFIG2] + ands r3, r3, #NFC_CONFIG2_INT + beq 1b + mov r3, #0x0 + strh r3, [r12, #NFC_OFFSET_CONFIG2] + mov pc, lr + +do_addr_input: + mov r9, lr + and r3, r3, #0xFF + strh r3, [r12, #NFC_OFFSET_FLASH_ADDR] + mov r3, #NFC_CONFIG2_FADD + strh r3, [r12, #NFC_OFFSET_CONFIG2] + bl do_wait_op_done + mov pc, r9 + +test_and_copy_buffer: + /* check for bad block (2 bits error in main or spare are)*/ + BOOT_TRACE(9) + ldrh r4, [r12, #NFC_OFFSET_ECC_STATUS_RESULT] + ands r4, r4, #(NFC_ECC_STAT_ERROR2| \ + (NFC_ECC_STAT_ERROR2<<NFC_ECC_STAT_ERM_SHFT)) + bne skip_bad_buffer + + /* check BI byte of the current spare buffer */ + ldr r4, =IMX_NFC_SPARE_AREA0 + ldrh r3, [r12, #NFC_OFFSET_BUF_ADDR] /* for the current buffer */ + orr r4, r3, lsl #0x04 + + /* at bi word offset 4. */ + /* Fixme position change betwwen 8 and 16 bits bus */ + ldrh r4, [r4, #0x04] + and r4, r4, #0x0FF00 /* has to be 0xFFxx */ + cmp r4, #0x0FF00 + bne skip_bad_buffer + +copy_good_buffer: + /* copying 512 bytes buffer */ + BOOT_TRACE(10) +1: ldmia r0!, {r3-r10} + stmia r11!, {r3-r10} + cmp r0, r2 + blo 1b + b end_of_copy + +skip_bad_buffer: + BOOT_TRACE(11) + /* bad pages do not contain valid data and have to be skip */ + add r0, r0, #0x200 + + /* rewind ram addr to start of buffer */ + ldr r3, =(~0x1FF) + and r11, r11, r3 + +end_of_copy: + add r2, r2, #0x200 + add r1, r1, #0x200 + + mov pc, lr + +setup_sdram_ddr: + + /* wait for SDRAM/LPDDR ready (SDRAMRDY) */ + ldr r0, =IMX_ESD_BASE + ldr r4, =ESDMISC_SDRAM_RDY +2: ldr r1, [r0, #ESDMISC_ROF] + ands r1, r1, r4 + bpl 2b + + /* LPDDR Soft Reset Mobile/Low Power DDR SDRAM. */ + ldr r0, =IMX_ESD_BASE + ldr r4, =ACFG_ESDMISC_VAL + orr r1, r4, #ESDMISC_MDDR_DL_RST + str r1, [r0, #ESDMISC_ROF] + + /* Hold for more than 200ns */ + ldr r1, =0x10000 +1: subs r1, r1, #0x1 + bne 1b + + str r4, [r0] + + ldr r0, =IMX_ESD_BASE + ldr r1, =ACFG_SDRAM_ESDCFG_REGISTER_VAL + str r1, [r0, #ESDCFG0_ROF] + + ldr r0, =IMX_ESD_BASE + ldr r1, =ACFG_PRECHARGE_CMD + str r1, [r0, #ESDCTL0_ROF] + + /* write8(0xA0001000, any value) */ + ldr r1, =PHYS_SDRAM_1+ACFG_SDRAM_PRECHARGE_ALL_VAL + strb r2, [r1] + + ldr r1, =ACFG_AUTOREFRESH_CMD + str r1, [r0, #ESDCTL0_ROF] + + ldr r4, =PHYS_SDRAM_1 /* CSD0 base address */ + + ldr r6,=0x7 /* load loop counter */ +1: str r5,[r4] /* run auto-refresh cycle to array 0 */ + subs r6,r6,#1 + bne 1b + + ldr r1, =ACFG_SET_MODE_REG_CMD + str r1, [r0, #ESDCTL0_ROF] + + /* set standard mode register */ + ldr r4, = PHYS_SDRAM_1+ACFG_SDRAM_MODE_REGISTER_VAL + strb r2, [r4] + + /* set extended mode register */ + ldr r4, =PHYS_SDRAM_1+ACFG_SDRAM_EXT_MODE_REGISTER_VAL + strb r5, [r4] + + ldr r1, =ACFG_NORMAL_RW_CMD + str r1, [r0, #ESDCTL0_ROF] + + /* 2nd sdram */ + ldr r0, =IMX_ESD_BASE + ldr r1, =ACFG_SDRAM_ESDCFG_REGISTER_VAL + str r1, [r0, #ESDCFG1_ROF] + + ldr r0, =IMX_ESD_BASE + ldr r1, =ACFG_PRECHARGE_CMD + str r1, [r0, #ESDCTL1_ROF] + + /* write8(0xB0001000, any value) */ + ldr r1, =PHYS_SDRAM_2+ACFG_SDRAM_PRECHARGE_ALL_VAL + strb r2, [r1] + + ldr r1, =ACFG_AUTOREFRESH_CMD + str r1, [r0, #ESDCTL1_ROF] + + ldr r4, =PHYS_SDRAM_2 /* CSD1 base address */ + + ldr r6,=0x7 /* load loop counter */ +1: str r5,[r4] /* run auto-refresh cycle to array 0 */ + subs r6,r6,#1 + bne 1b + + ldr r1, =ACFG_SET_MODE_REG_CMD + str r1, [r0, #ESDCTL1_ROF] + + /* set standard mode register */ + ldr r4, =PHYS_SDRAM_2+ACFG_SDRAM_MODE_REGISTER_VAL + strb r2, [r4] +
Quoted 8 lines
board/armadeus/apf27/splboot.S line 528
Quoted 8 lines
board/armadeus/apf27/splboot.S new file +8
-0+ /* set extended mode register */ + ldr r4, =PHYS_SDRAM_2+ACFG_SDRAM_EXT_MODE_REGISTER_VAL + strb r2, [r4] + + ldr r1, =ACFG_NORMAL_RW_CMD + str r1, [r0, #ESDCTL1_ROF] + + mov pc, lr
Best regards,
Philippe
PATCH v4 2/4 apf27: add support for the armadeus APF27 board
Hi Philippe,
Quoted 15 lines · trem
Quoted 6 lines
Ok, this must be done in assembly - normally is part of
lowelevel_init, whileboard_init_f()is already written in C.Really I was waiting that general code is called, that is
spl_nand_load_image()and then the driver function. You are pushing a parallel way that is, at the end, still the old way to boot (nand_spl).This board boot from nand, so the SPL should be lower than 2KB. The SPL start from NFC (nand) buffer, and It has to :
- initialize the ram
- copy itself to another location (ram)
- jump to this another location
- copy u-boot from nand to ram
- jump to ram
Quoted 4 lines
We have tried to use SPL framework (with nand driver), but the result is a SPL bigger than 2 KB, so we can't use it.
To have a SPL smaller than 2 KB, we have written everything in assembler.
Ok, sorry - I am now used with current i.MX where there is not this constraint. I reread the boot part of i.MX27 manual and I got the point. I understand there is no other way for the i.MX27.
I would like you add your useful explanation why you add an assembly NAND driver (so what you have written here in the lines above) to the commit message, so that is clear why you were forced to use this way instead of the SPL framework.
Quoted 3 lines
Are you agree with this solution for the SPL ?
I understand there is no other solution ;-)
Best regards,
Stefano
Signature · 5 lines
===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de =====================================================================
Fabio Estevam
reviewerPATCH v4 2/4 apf27: add support for the armadeus APF27 board
Quoted 10 lines · Stefano Babic
I would like you add your useful explanation why you add an assembly NAND driver (so what you have written here in the lines above) to the commit message, so that is clear why you were forced to use this way instead of the SPL framework.
Quoted 3 lines
Are you agree with this solution for the SPL ?
I understand there is no other solution ;-)
mx31 also has the same 2kB limitation and we don't use the NAND driver in assembly there, right?
PATCH v4 2/4 apf27: add support for the armadeus APF27 board
Hi Fabio,
Quoted 8 lines · Fabio Estevam
Quoted 6 lines · Stefano Babic
I would like you add your useful explanation why you add an assembly NAND driver (so what you have written here in the lines above) to the commit message, so that is clear why you were forced to use this way instead of the SPL framework.
Quoted 1 line
Quoted 8 lines
Quoted 4 lines
Quoted 2 lines
Are you agree with this solution for the SPL ?
I understand there is no other solution ;-)
mx31 also has the same 2kB limitation and we don't use the NAND driver in assembly there, right?
You're absolutely right, thanks for pointing out ! And it uses the generic framework.
Philippe, could you take a look at the mx31pdk implementation ? SPL is less thn 2KB (and you can set CONFIG_SPL_MAX_SIZE in your config file):
1756 0 0 1756 6dc ./spl/u-boot-spl
Best regards,
Stefano Babic
Signature · 5 lines
===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de =====================================================================
PATCH v4 3/4 apf27: add FPGA support for the apf27 board
In reply to Philippe Reynes--- board/armadeus/apf27/Makefile | 3 + board/armadeus/apf27/apf27.c | 18 +++ board/armadeus/apf27/fpga.c | 250 +++++++++++++++++++++++++++++++++++++++++ board/armadeus/apf27/fpga.h | 39 +++++++ 4 files changed, 310 insertions(+), 0 deletions(-) create mode 100644 board/armadeus/apf27/fpga.c create mode 100644 board/armadeus/apf27/fpga.h
board/armadeus/apf27/Makefile +3 -0
@@ -29,6 +29,9 @@ LIB = $(obj)lib$(BOARD).o ifndef CONFIG_SPL_BUILD COBJS := apf27.o +ifdef CONFIG_FPGA +COBJS += fpga.o +endif endif ifdef CONFIG_SPL_BUILD
board/armadeus/apf27/apf27.c +18 -0
@@ -30,6 +30,7 @@#include <asm/errno.h> #include "apf27.h" #include "crc.h" +#include "fpga.h" DECLARE_GLOBAL_DATA_PTR ;
@@ -321,6 +322,23 @@ int misc_init_r(void)struct mtd_device *dev;struct part_info *part; +#if defined(CONFIG_FPGA ) +/* init and download fpga */ +if ((autoload) && (0 ==strcmp (autoload,"1" ))) { +if (mtdparts_init () ==0 ) +if (find_dev_and_part ("firmware" , + &dev, &pnum, &part) ==0 ) { + size = part->size; +if (nand_read_skip_bad (&nand_info[0 ], + part->offset, &size, +NULL , part->size, + firmware_buffer)) + size =0 ; + } + } +APF27_init_fpga (firmware_buffer, size); +#endif +/* detect compatibility issue of environment version */ s =getenv ("env_version" );if ((NULL == s) || (0 !=strcmp (s,CONFIG_ENV_VERSION ))) {
board/armadeus/apf27/fpga.c new file +250 -0
250 lines not on this page · raw message
board/armadeus/apf27/fpga.h new file +39 -0
@@ -0,0 +1,39 @@ +/* +* (C) Copyright 2002-2012 +* Eric Jarrige <[email protected]> +* Rich Ireland, Enterasys Networks, rireland at enterasys.com. +* Keith Outwater, keith_outwater at mvis.com. +* +* See file CREDITS for list of people who contributed to this +* project. +* +* This program is free software; you can redistribute it and/or +* modify it under the terms of the GNU General Public License as +* published by the Free Software Foundation; either version 2 of +* the License, or (at your option) any later version. +* +* This program is distributed in the hope that it will be useful, +* but WITHOUT ANY WARRANTY; without even the implied warranty of +* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +* GNU General Public License for more details. +* +* You should have received a copy of the GNU General Public License +* along with this program; if not, write to the Free Software +* Foundation, Inc., 59 Temple Place, Suite 330, Boston, +* MA 02111-1307 USA +* +*/ +extern int APF27_init_fpga (u_char *buffer,size_t size); + +extern int fpga_pre_fn (int cookie); +extern int fpga_pgm_fn (int assert_pgm,int flush,int cookie); +extern int fpga_cs_fn (int assert_cs,int flush,int cookie); +extern int fpga_init_fn (int cookie); +extern int fpga_done_fn (int cookie); +extern int fpga_clk_fn (int assert_clk,int flush,int cookie); +extern int fpga_wr_fn (int assert_write,int flush,int cookie); +extern int fpga_rdata_fn (unsigned char *data,int cookie); +extern int fpga_wdata_fn (unsigned char data,int flush,int cookie); +extern int fpga_abort_fn (int cookie); +extern int fpga_post_fn (int cookie); +extern int fpga_busy_fn (int cookie);
Signature · 1 line
1.7.4.4
PATCH v4 3/4 apf27: add FPGA support for the apf27 board
Dear Philippe Reynes,
Quoted 17 lines · In message <[email protected]> you wrote:
board/armadeus/apf27/fpga.c new file +13
-0@@ -0,0 +1,250 @@ +/* +* (C) Copyright 2002-2012 +* Eric Jarrige <[email protected]> +* Rich Ireland, Enterasys Networks, rireland at enterasys.com. +* Keith Outwater, keith_outwater at mvis.com. +* +* See file CREDITS for list of people who contributed to this +* project. +* +* This program is free software; you can redistribute it and/or +* modify it under the terms of the GNU General Public License as +* published by the Free Software Foundation; either version 2 of +* the License, or (at your option) any later version.
board/armadeus/apf27/fpga.c line 23
Quoted 10 lines
board/armadeus/apf27/fpga.c new file +10
-0+* +* This program is distributed in the hope that it will be useful, +* but WITHOUT ANY WARRANTY; without even the implied warranty of +* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +* GNU General Public License for more details. +* +* You should have received a copy of the GNU General Public License +* along with this program; if not, write to the Free Software +* Foundation, Inc., 59 Temple Place, Suite 330, Boston, +* MA 02111-1307 USA
Please drop the full header and use a SPDX-License-Identifier instead. Please fix globally.
Thanks.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de No user-servicable parts inside. Refer to qualified service personnel.
PATCH v4 4/4 Update u-boot-nand.bin to be used with spl
In reply to Philippe Reynes--- Makefile | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-)
Makefile +6 -0
@@ -617,8 +617,14 @@ $(obj)u-boot.lds: $(LDSCRIPT) nand_spl: $(TIMESTAMP_FILE) $(VERSION_FILE) depend $(MAKE) -C nand_spl/board/$(BOARDDIR) all +ifeq ($(CONFIG_SPL),y) +$(obj)u-boot-nand.bin: $(obj)u-boot.bin $(obj)spl/u-boot-spl.bin + $(OBJCOPY) ${OBJCFLAGS} --pad-to=$(CONFIG_SPL_PAD_TO) -O binary $(obj)spl/u-boot-spl $(obj)spl/u-boot-spl-pad.bin && \ + cat $(obj)spl/u-boot-spl-pad.bin $(obj)u-boot.bin > $(obj)u-boot-nand.bin +else $(obj)u-boot-nand.bin: nand_spl $(obj)u-boot.bin cat $(obj)nand_spl/u-boot-spl-16k.bin $(obj)u-boot.bin > $(obj)u-boot-nand.bin +endif $(obj)spl/u-boot-spl.bin: $(SUBDIR_TOOLS) depend $(MAKE) -C spl all
Signature · 1 line
1.7.4.4
PATCH v4 4/4 Update u-boot-nand.bin to be used with spl
Quoted 24 lines · Philippe Reynes
Signed-off-by Philippe Reynes <[email protected]>Signed-off-by Eric Jarrige <[email protected]>--- Makefile | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-)Makefile +4
-1@@ -617,8 +617,14 @@ $(obj)u-boot.lds: $(LDSCRIPT) nand_spl: $(TIMESTAMP_FILE) $(VERSION_FILE) depend $(MAKE) -C nand_spl/board/$(BOARDDIR) all +ifeq ($(CONFIG_SPL),y) +$(obj)u-boot-nand.bin: $(obj)u-boot.bin $(obj)spl/u-boot-spl.bin + $(OBJCOPY) ${OBJCFLAGS} --pad-to=$(CONFIG_SPL_PAD_TO) -O binary $(obj)spl/u-boot-spl $(obj)spl/u-boot-spl-pad.bin && \ + cat $(obj)spl/u-boot-spl-pad.bin $(obj)u-boot.bin >$(obj)u-boot-nand.bin
Makefile +1
-0+else $(obj)u-boot-nand.bin: nand_spl $(obj)u-boot.bin cat $(obj)nand_spl/u-boot-spl-16k.bin$(obj)u-boot.bin > $(obj)u-boot-nand.bin
Makefile $(obj)u-boot.lds: $(LDSCRIPT) · line 630
Quoted 4 lines
Makefile +1
-0+endif $(obj)spl/u-boot-spl.bin: $(SUBDIR_TOOLS) depend $(MAKE) -C spl all
u-boot-nand.bin is from the legacy nand_spl subsystem. You should be using u-boot-with-spl.bin if you want this sort of image concatenation with the new SPL.
-Scott