* [U-Boot] [PATCH v5 1/6] spl: display_options.o is required for SPI flash support in SPL
2011-12-05 10:58 [U-Boot] [PATCH v5 0/6] Add an SPL to boot the da850evm from SPI Christian Riesch
@ 2011-12-05 10:58 ` Christian Riesch
2011-12-05 10:58 ` [U-Boot] [PATCH v5 2/6] sf: Add spi_boot() to allow booting from SPI flash in an SPL Christian Riesch
` (5 subsequent siblings)
6 siblings, 0 replies; 21+ messages in thread
From: Christian Riesch @ 2011-12-05 10:58 UTC (permalink / raw)
To: u-boot
Signed-off-by: Christian Riesch <christian.riesch@omicron.at>
Cc: Heiko Schocher <hs@denx.de>
Cc: Mike Frysinger <vapier@gentoo.org>
---
lib/Makefile | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/lib/Makefile b/lib/Makefile
index 54708c2..35ba7ff 100644
--- a/lib/Makefile
+++ b/lib/Makefile
@@ -50,6 +50,8 @@ COBJS-$(CONFIG_SHA1) += sha1.o
COBJS-$(CONFIG_SHA256) += sha256.o
COBJS-y += strmhz.o
COBJS-$(CONFIG_RBTREE) += rbtree.o
+else
+COBJS-$(CONFIG_SPL_SPI_FLASH_SUPPORT) += display_options.o
endif
COBJS-y += ctype.o
--
1.7.0.4
^ permalink raw reply related [flat|nested] 21+ messages in thread* [U-Boot] [PATCH v5 2/6] sf: Add spi_boot() to allow booting from SPI flash in an SPL
2011-12-05 10:58 [U-Boot] [PATCH v5 0/6] Add an SPL to boot the da850evm from SPI Christian Riesch
2011-12-05 10:58 ` [U-Boot] [PATCH v5 1/6] spl: display_options.o is required for SPI flash support in SPL Christian Riesch
@ 2011-12-05 10:58 ` Christian Riesch
2011-12-05 19:33 ` Scott Wood
2011-12-05 10:58 ` [U-Boot] [PATCH v5 3/6] arm, davinci: Add SPL support for DA850 SoCs Christian Riesch
` (4 subsequent siblings)
6 siblings, 1 reply; 21+ messages in thread
From: Christian Riesch @ 2011-12-05 10:58 UTC (permalink / raw)
To: u-boot
Signed-off-by: Christian Riesch <christian.riesch@omicron.at>
Cc: Heiko Schocher <hs@denx.de>
Cc: Mike Frysinger <vapier@gentoo.org>
Cc: Scott Wood <scottwood@freescale.com>
Acked-by: Mike Frysinger <vapier@gentoo.org>
---
doc/README.SPL | 1 +
drivers/mtd/spi/Makefile | 4 +++
drivers/mtd/spi/spi_spl_load.c | 58 ++++++++++++++++++++++++++++++++++++++++
include/spi_flash.h | 3 ++
4 files changed, 66 insertions(+), 0 deletions(-)
create mode 100644 drivers/mtd/spi/spi_spl_load.c
diff --git a/doc/README.SPL b/doc/README.SPL
index 89d24a7..f01a8bd 100644
--- a/doc/README.SPL
+++ b/doc/README.SPL
@@ -65,3 +65,4 @@ CONFIG_SPL_NAND_SUPPORT (drivers/mtd/nand/libnand.o)
CONFIG_SPL_DMA_SUPPORT (drivers/dma/libdma.o)
CONFIG_SPL_POST_MEM_SUPPORT (post/drivers/memory.o)
CONFIG_SPL_NAND_LOAD (drivers/mtd/nand/nand_spl_load.o)
+CONFIG_SPL_SPI_LOAD (drivers/mtd/spi/spi_spl_load.o)
diff --git a/drivers/mtd/spi/Makefile b/drivers/mtd/spi/Makefile
index 57112af..90f8392 100644
--- a/drivers/mtd/spi/Makefile
+++ b/drivers/mtd/spi/Makefile
@@ -25,6 +25,10 @@ include $(TOPDIR)/config.mk
LIB := $(obj)libspi_flash.o
+ifdef CONFIG_SPL_BUILD
+COBJS-$(CONFIG_SPL_SPI_LOAD) += spi_spl_load.o
+endif
+
COBJS-$(CONFIG_SPI_FLASH) += spi_flash.o
COBJS-$(CONFIG_SPI_FLASH_ATMEL) += atmel.o
COBJS-$(CONFIG_SPI_FLASH_EON) += eon.o
diff --git a/drivers/mtd/spi/spi_spl_load.c b/drivers/mtd/spi/spi_spl_load.c
new file mode 100644
index 0000000..1aa30ac
--- /dev/null
+++ b/drivers/mtd/spi/spi_spl_load.c
@@ -0,0 +1,58 @@
+/*
+ * Copyright (C) 2011 OMICRON electronics GmbH
+ *
+ * based on drivers/mtd/nand/nand_spl_load.c
+ *
+ * Copyright (C) 2011
+ * Heiko Schocher, DENX Software Engineering, hs at denx.de.
+ *
+ * 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 <spi_flash.h>
+
+/*
+ * The main entry for SPI booting. It's necessary that SDRAM is already
+ * configured and available since this code loads the main U-Boot image
+ * from SPI into SDRAM and starts it from there.
+ */
+void spi_boot(void)
+{
+ struct spi_flash *flash;
+ void (*uboot)(void) __noreturn;
+
+ /*
+ * Load U-Boot image from SPI flash into RAM
+ */
+
+ flash = spi_flash_probe(CONFIG_SPL_SPI_BUS, CONFIG_SPL_SPI_CS,
+ CONFIG_SF_DEFAULT_SPEED, SPI_MODE_3);
+ if (!flash) {
+ puts("failed.\n");
+ hang();
+ }
+
+ spi_flash_read(flash, CONFIG_SYS_SPI_U_BOOT_OFFS,
+ CONFIG_SYS_SPI_U_BOOT_SIZE,
+ (void *) CONFIG_SYS_TEXT_BASE);
+
+ /*
+ * Jump to U-Boot image
+ */
+ uboot = (void *) CONFIG_SYS_TEXT_BASE;
+ (*uboot)();
+}
diff --git a/include/spi_flash.h b/include/spi_flash.h
index 2671ab5..9da9062 100644
--- a/include/spi_flash.h
+++ b/include/spi_flash.h
@@ -25,6 +25,7 @@
#include <spi.h>
#include <linux/types.h>
+#include <linux/compiler.h>
struct spi_flash {
struct spi_slave *spi;
@@ -68,4 +69,6 @@ static inline int spi_flash_erase(struct spi_flash *flash, u32 offset,
return flash->erase(flash, offset, len);
}
+void spi_boot(void) __noreturn;
+
#endif /* _SPI_FLASH_H_ */
--
1.7.0.4
^ permalink raw reply related [flat|nested] 21+ messages in thread* [U-Boot] [PATCH v5 2/6] sf: Add spi_boot() to allow booting from SPI flash in an SPL
2011-12-05 10:58 ` [U-Boot] [PATCH v5 2/6] sf: Add spi_boot() to allow booting from SPI flash in an SPL Christian Riesch
@ 2011-12-05 19:33 ` Scott Wood
2011-12-05 19:56 ` Mike Frysinger
0 siblings, 1 reply; 21+ messages in thread
From: Scott Wood @ 2011-12-05 19:33 UTC (permalink / raw)
To: u-boot
On 12/05/2011 04:58 AM, Christian Riesch wrote:
> Signed-off-by: Christian Riesch <christian.riesch@omicron.at>
> Cc: Heiko Schocher <hs@denx.de>
> Cc: Mike Frysinger <vapier@gentoo.org>
> Cc: Scott Wood <scottwood@freescale.com>
> Acked-by: Mike Frysinger <vapier@gentoo.org>
> ---
> doc/README.SPL | 1 +
> drivers/mtd/spi/Makefile | 4 +++
> drivers/mtd/spi/spi_spl_load.c | 58 ++++++++++++++++++++++++++++++++++++++++
> include/spi_flash.h | 3 ++
> 4 files changed, 66 insertions(+), 0 deletions(-)
> create mode 100644 drivers/mtd/spi/spi_spl_load.c
I'm not sure if you're the one assigning this to me in Patchwork (would
be nice if it had a history log), but I'm not the maintainer of SPI, nor
of SPL generically -- just NAND.
-Scott
^ permalink raw reply [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v5 2/6] sf: Add spi_boot() to allow booting from SPI flash in an SPL
2011-12-05 19:33 ` Scott Wood
@ 2011-12-05 19:56 ` Mike Frysinger
2011-12-05 20:03 ` Tom Rini
0 siblings, 1 reply; 21+ messages in thread
From: Mike Frysinger @ 2011-12-05 19:56 UTC (permalink / raw)
To: u-boot
On Monday 05 December 2011 14:33:40 Scott Wood wrote:
> On 12/05/2011 04:58 AM, Christian Riesch wrote:
> > Signed-off-by: Christian Riesch <christian.riesch@omicron.at>
> > Cc: Heiko Schocher <hs@denx.de>
> > Cc: Mike Frysinger <vapier@gentoo.org>
> > Cc: Scott Wood <scottwood@freescale.com>
> > Acked-by: Mike Frysinger <vapier@gentoo.org>
> > ---
> >
> > doc/README.SPL | 1 +
> > drivers/mtd/spi/Makefile | 4 +++
> > drivers/mtd/spi/spi_spl_load.c | 58
> > ++++++++++++++++++++++++++++++++++++++++ include/spi_flash.h
> > | 3 ++
> > 4 files changed, 66 insertions(+), 0 deletions(-)
> > create mode 100644 drivers/mtd/spi/spi_spl_load.c
>
> I'm not sure if you're the one assigning this to me in Patchwork (would
> be nice if it had a history log), but I'm not the maintainer of SPI, nor
> of SPL generically -- just NAND.
i don't plan on running these through my SPI branch ... seems like it should
go through the SPL with everything else here.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20111205/bc37c65f/attachment.pgp>
^ permalink raw reply [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v5 2/6] sf: Add spi_boot() to allow booting from SPI flash in an SPL
2011-12-05 19:56 ` Mike Frysinger
@ 2011-12-05 20:03 ` Tom Rini
2011-12-05 21:23 ` Mike Frysinger
0 siblings, 1 reply; 21+ messages in thread
From: Tom Rini @ 2011-12-05 20:03 UTC (permalink / raw)
To: u-boot
On Mon, Dec 5, 2011 at 12:56 PM, Mike Frysinger <vapier@gentoo.org> wrote:
> On Monday 05 December 2011 14:33:40 Scott Wood wrote:
>> On 12/05/2011 04:58 AM, Christian Riesch wrote:
>> > Signed-off-by: Christian Riesch <christian.riesch@omicron.at>
>> > Cc: Heiko Schocher <hs@denx.de>
>> > Cc: Mike Frysinger <vapier@gentoo.org>
>> > Cc: Scott Wood <scottwood@freescale.com>
>> > Acked-by: Mike Frysinger <vapier@gentoo.org>
>> > ---
>> >
>> > ?doc/README.SPL ? ? ? ? ? ? ? ? | ? ?1 +
>> > ?drivers/mtd/spi/Makefile ? ? ? | ? ?4 +++
>> > ?drivers/mtd/spi/spi_spl_load.c | ? 58
>> > ?++++++++++++++++++++++++++++++++++++++++ include/spi_flash.h
>> > ?| ? ?3 ++
>> > ?4 files changed, 66 insertions(+), 0 deletions(-)
>> > ?create mode 100644 drivers/mtd/spi/spi_spl_load.c
>>
>> I'm not sure if you're the one assigning this to me in Patchwork (would
>> be nice if it had a history log), but I'm not the maintainer of SPI, nor
>> of SPL generically -- just NAND.
>
> i don't plan on running these through my SPI branch ... seems like it should
> go through the SPL with everything else here.
How about this? I don't see SPL nor SPI on the Custodians page, but
there is TI and this is SPL for DaVinci (TI) stuff, so once everyone
is happy with the whole series I'll take this for u-boot-ti/master and
try for v2011.12 (since this series has been around for a while).
--
Tom
^ permalink raw reply [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v5 2/6] sf: Add spi_boot() to allow booting from SPI flash in an SPL
2011-12-05 20:03 ` Tom Rini
@ 2011-12-05 21:23 ` Mike Frysinger
0 siblings, 0 replies; 21+ messages in thread
From: Mike Frysinger @ 2011-12-05 21:23 UTC (permalink / raw)
To: u-boot
On Monday 05 December 2011 15:03:44 Tom Rini wrote:
> On Mon, Dec 5, 2011 at 12:56 PM, Mike Frysinger <vapier@gentoo.org> wrote:
> > On Monday 05 December 2011 14:33:40 Scott Wood wrote:
> >> On 12/05/2011 04:58 AM, Christian Riesch wrote:
> >> > Signed-off-by: Christian Riesch <christian.riesch@omicron.at>
> >> > Cc: Heiko Schocher <hs@denx.de>
> >> > Cc: Mike Frysinger <vapier@gentoo.org>
> >> > Cc: Scott Wood <scottwood@freescale.com>
> >> > Acked-by: Mike Frysinger <vapier@gentoo.org>
> >> > ---
> >> >
> >> > doc/README.SPL | 1 +
> >> > drivers/mtd/spi/Makefile | 4 +++
> >> > drivers/mtd/spi/spi_spl_load.c | 58
> >> > ++++++++++++++++++++++++++++++++++++++++ include/spi_flash.h
> >> > | 3 ++
> >> > 4 files changed, 66 insertions(+), 0 deletions(-)
> >> > create mode 100644 drivers/mtd/spi/spi_spl_load.c
> >>
> >> I'm not sure if you're the one assigning this to me in Patchwork (would
> >> be nice if it had a history log), but I'm not the maintainer of SPI, nor
> >> of SPL generically -- just NAND.
> >
> > i don't plan on running these through my SPI branch ... seems like it
> > should go through the SPL with everything else here.
>
> How about this? I don't see SPL nor SPI on the Custodians page, but
> there is TI and this is SPL for DaVinci (TI) stuff, so once everyone
> is happy with the whole series I'll take this for u-boot-ti/master and
> try for v2011.12 (since this series has been around for a while).
that's fine by me
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20111205/0e60b807/attachment.pgp>
^ permalink raw reply [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v5 3/6] arm, davinci: Add SPL support for DA850 SoCs
2011-12-05 10:58 [U-Boot] [PATCH v5 0/6] Add an SPL to boot the da850evm from SPI Christian Riesch
2011-12-05 10:58 ` [U-Boot] [PATCH v5 1/6] spl: display_options.o is required for SPI flash support in SPL Christian Riesch
2011-12-05 10:58 ` [U-Boot] [PATCH v5 2/6] sf: Add spi_boot() to allow booting from SPI flash in an SPL Christian Riesch
@ 2011-12-05 10:58 ` Christian Riesch
2011-12-05 10:58 ` [U-Boot] [PATCH v5 4/6] arm, da850evm: Add an SPL for SPI boot Christian Riesch
` (3 subsequent siblings)
6 siblings, 0 replies; 21+ messages in thread
From: Christian Riesch @ 2011-12-05 10:58 UTC (permalink / raw)
To: u-boot
This code adds an SPL for booting from SPI flash on DA850 SoCs.
Signed-off-by: Christian Riesch <christian.riesch@omicron.at>
Cc: Heiko Schocher <hs@denx.de>
Cc: Sandeep Paulraj <s-paulraj@ti.com>
Cc: Tom Rini <trini@ti.com>
Acked-by: Tom Rini <trini@ti.com>
---
arch/arm/cpu/arm926ejs/davinci/Makefile | 3 +-
arch/arm/cpu/arm926ejs/davinci/spl.c | 34 ++++++++++++++++++++++++++++++-
2 files changed, 35 insertions(+), 2 deletions(-)
diff --git a/arch/arm/cpu/arm926ejs/davinci/Makefile b/arch/arm/cpu/arm926ejs/davinci/Makefile
index 5ae89df..da7efac 100644
--- a/arch/arm/cpu/arm926ejs/davinci/Makefile
+++ b/arch/arm/cpu/arm926ejs/davinci/Makefile
@@ -38,7 +38,8 @@ COBJS-$(CONFIG_DRIVER_TI_EMAC) += lxt972.o dp83848.o et1011c.o ksz8873.o
ifdef CONFIG_SPL_BUILD
COBJS-y += spl.o
-COBJS-y += dm365_lowlevel.o
+COBJS-$(CONFIG_SOC_DM365) += dm365_lowlevel.o
+COBJS-$(CONFIG_SOC_DA8XX) += da850_lowlevel.o
endif
SOBJS = reset.o
diff --git a/arch/arm/cpu/arm926ejs/davinci/spl.c b/arch/arm/cpu/arm926ejs/davinci/spl.c
index d9b9398..20f798e 100644
--- a/arch/arm/cpu/arm926ejs/davinci/spl.c
+++ b/arch/arm/cpu/arm926ejs/davinci/spl.c
@@ -26,6 +26,16 @@
#include <nand.h>
#include <asm/arch/dm365_lowlevel.h>
#include <ns16550.h>
+#include <malloc.h>
+#include <spi_flash.h>
+
+DECLARE_GLOBAL_DATA_PTR;
+
+/* Define global data structure pointer to it*/
+static gd_t gdata __attribute__ ((section(".data")));
+static bd_t bdata __attribute__ ((section(".data")));
+
+#ifndef CONFIG_SPL_LIBCOMMON_SUPPORT
void puts(const char *str)
{
@@ -41,6 +51,8 @@ void putc(char c)
NS16550_putc((NS16550_t)(CONFIG_SYS_NS16550_COM1), c);
}
+#endif /* CONFIG_SPL_LIBCOMMON_SUPPORT */
+
inline void hang(void)
{
puts("### ERROR ### Please RESET the board ###\n");
@@ -50,14 +62,34 @@ inline void hang(void)
void board_init_f(ulong dummy)
{
+#ifdef CONFIG_SOC_DM365
dm36x_lowlevel_init(0);
+#endif
+#ifdef CONFIG_SOC_DA8XX
+ arch_cpu_init();
+#endif
relocate_code(CONFIG_SPL_STACK, NULL, CONFIG_SPL_TEXT_BASE);
}
void board_init_r(gd_t *id, ulong dummy)
{
-
+#ifdef CONFIG_SOC_DM365
nand_init();
puts("Nand boot...\n");
nand_boot();
+#endif
+#ifdef CONFIG_SOC_DA8XX
+ mem_malloc_init(CONFIG_SYS_TEXT_BASE - CONFIG_SYS_MALLOC_LEN,
+ CONFIG_SYS_MALLOC_LEN);
+
+ gd = &gdata;
+ gd->bd = &bdata;
+ gd->flags |= GD_FLG_RELOC;
+ gd->baudrate = CONFIG_BAUDRATE;
+ serial_init(); /* serial communications setup */
+ gd->have_console = 1;
+
+ puts("SPI boot...\n");
+ spi_boot();
+#endif
}
--
1.7.0.4
^ permalink raw reply related [flat|nested] 21+ messages in thread* [U-Boot] [PATCH v5 4/6] arm, da850evm: Add an SPL for SPI boot
2011-12-05 10:58 [U-Boot] [PATCH v5 0/6] Add an SPL to boot the da850evm from SPI Christian Riesch
` (2 preceding siblings ...)
2011-12-05 10:58 ` [U-Boot] [PATCH v5 3/6] arm, davinci: Add SPL support for DA850 SoCs Christian Riesch
@ 2011-12-05 10:58 ` Christian Riesch
2011-12-05 15:19 ` Tom Rini
2011-12-05 10:58 ` [U-Boot] [PATCH v5 5/6] mkimage: Fix variable length header support Christian Riesch
` (2 subsequent siblings)
6 siblings, 1 reply; 21+ messages in thread
From: Christian Riesch @ 2011-12-05 10:58 UTC (permalink / raw)
To: u-boot
Signed-off-by: Christian Riesch <christian.riesch@omicron.at>
Cc: Heiko Schocher <hs@denx.de>
Cc: Sandeep Paulraj <s-paulraj@ti.com>
Cc: Tom Rini <trini@ti.com>
Cc: Sudhakar Rajashekhara <sudhakar.raj@ti.com>
---
board/davinci/da8xxevm/da850evm.c | 4 +-
board/davinci/da8xxevm/u-boot-spl.lds | 73 +++++++++++++++++++++++++++
doc/README.davinci | 9 +++
include/configs/da850evm.h | 87 +++++++++++++++++++++++++++++++++
4 files changed, 172 insertions(+), 1 deletions(-)
create mode 100644 board/davinci/da8xxevm/u-boot-spl.lds
diff --git a/board/davinci/da8xxevm/da850evm.c b/board/davinci/da8xxevm/da850evm.c
index e827256..8e66c35 100644
--- a/board/davinci/da8xxevm/da850evm.c
+++ b/board/davinci/da8xxevm/da850evm.c
@@ -108,7 +108,7 @@ static const struct pinmux_config gpio_pins[] = {
#endif
};
-static const struct pinmux_resource pinmuxes[] = {
+const struct pinmux_resource pinmuxes[] = {
#ifdef CONFIG_DRIVER_TI_EMAC
PINMUX_ITEM(emac_pins_mdio),
#ifdef CONFIG_DRIVER_TI_EMAC_USE_RMII
@@ -135,6 +135,8 @@ static const struct pinmux_resource pinmuxes[] = {
PINMUX_ITEM(gpio_pins),
};
+const int pinmuxes_size = ARRAY_SIZE(pinmuxes);
+
static const struct lpsc_resource lpsc[] = {
{ DAVINCI_LPSC_AEMIF }, /* NAND, NOR */
{ DAVINCI_LPSC_SPI1 }, /* Serial Flash */
diff --git a/board/davinci/da8xxevm/u-boot-spl.lds b/board/davinci/da8xxevm/u-boot-spl.lds
new file mode 100644
index 0000000..6f6e065
--- /dev/null
+++ b/board/davinci/da8xxevm/u-boot-spl.lds
@@ -0,0 +1,73 @@
+/*
+ * (C) Copyright 2002
+ * Gary Jennejohn, DENX Software Engineering, <garyj@denx.de>
+ *
+ * (C) Copyright 2008
+ * Guennadi Liakhovetki, DENX Software Engineering, <lg@denx.de>
+ *
+ * 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
+ */
+
+MEMORY { .sram : ORIGIN = CONFIG_SPL_TEXT_BASE,\
+ LENGTH = CONFIG_SPL_MAX_SIZE }
+
+OUTPUT_FORMAT("elf32-littlearm", "elf32-littlearm", "elf32-littlearm")
+OUTPUT_ARCH(arm)
+ENTRY(_start)
+SECTIONS
+{
+ . = 0x00000000;
+
+ . = ALIGN(4);
+ .text :
+ {
+ __start = .;
+ arch/arm/cpu/arm926ejs/start.o (.text)
+ *(.text*)
+ } >.sram
+
+ . = ALIGN(4);
+ .rodata : { *(SORT_BY_ALIGNMENT(.rodata*)) } >.sram
+
+ . = ALIGN(4);
+ .data : { *(SORT_BY_ALIGNMENT(.data*)) } >.sram
+ . = ALIGN(4);
+ .rel.dyn : {
+ __rel_dyn_start = .;
+ *(.rel*)
+ __rel_dyn_end = .;
+ } >.sram
+
+ .dynsym : {
+ __dynsym_start = .;
+ *(.dynsym)
+ } >.sram
+
+ .bss :
+ {
+ . = ALIGN(4);
+ __bss_start = .;
+ *(.bss*)
+ . = ALIGN(4);
+ __bss_end__ = .;
+ } >.sram
+
+ __image_copy_end = .;
+ _end = .;
+}
diff --git a/doc/README.davinci b/doc/README.davinci
index 5f1bdc8..aa7c850 100644
--- a/doc/README.davinci
+++ b/doc/README.davinci
@@ -95,6 +95,15 @@ into the RAM.
The programmers and UBL are always released as part of any standard TI
software release associated with an SOC.
+Alternative boot method (DA850 EVM only):
+For the DA850 EVM an SPL (secondary program loader, see doc/README.SPL)
+is provided to load U-Boot directly from SPI flash. In this case, the
+SPL does the low level initialization that is otherwise done by the SPL.
+To build U-Boot with this SPL, do
+make da850evm_config
+make u-boot.ais
+and program the resulting u-boot.ais file to the SPI flash of the DA850 EVM.
+
Environment Variables
=====================
diff --git a/include/configs/da850evm.h b/include/configs/da850evm.h
index 2e2aa19..b30696a 100644
--- a/include/configs/da850evm.h
+++ b/include/configs/da850evm.h
@@ -65,6 +65,75 @@
#define CONFIG_NR_DRAM_BANKS 1 /* we have 1 bank of DRAM */
#define CONFIG_STACKSIZE (256*1024) /* regular stack */
+#define CONFIG_SYS_DA850_SYSCFG_SUSPSRC ( \
+ DAVINCI_SYSCFG_SUSPSRC_TIMER0 | \
+ DAVINCI_SYSCFG_SUSPSRC_SPI1 | \
+ DAVINCI_SYSCFG_SUSPSRC_UART2 | \
+ DAVINCI_SYSCFG_SUSPSRC_EMAC | \
+ DAVINCI_SYSCFG_SUSPSRC_I2C)
+
+/*
+ * PLL configuration
+ */
+#define CONFIG_SYS_DV_CLKMODE 0
+#define CONFIG_SYS_DA850_PLL0_POSTDIV 1
+#define CONFIG_SYS_DA850_PLL0_PLLDIV1 0x8000
+#define CONFIG_SYS_DA850_PLL0_PLLDIV2 0x8001
+#define CONFIG_SYS_DA850_PLL0_PLLDIV3 0x8002
+#define CONFIG_SYS_DA850_PLL0_PLLDIV4 0x8003
+#define CONFIG_SYS_DA850_PLL0_PLLDIV5 0x8002
+#define CONFIG_SYS_DA850_PLL0_PLLDIV6 CONFIG_SYS_DA850_PLL0_PLLDIV1
+#define CONFIG_SYS_DA850_PLL0_PLLDIV7 0x8005
+
+#define CONFIG_SYS_DA850_PLL1_POSTDIV 1
+#define CONFIG_SYS_DA850_PLL1_PLLDIV1 0x8000
+#define CONFIG_SYS_DA850_PLL1_PLLDIV2 0x8001
+#define CONFIG_SYS_DA850_PLL1_PLLDIV3 0x8002
+
+#define CONFIG_SYS_DA850_PLL0_PLLM 24
+#define CONFIG_SYS_DA850_PLL1_PLLM 21
+
+/*
+ * DDR2 memory configuration
+ */
+#define CONFIG_SYS_DA850_DDR2_DDRPHYCR (DV_DDR_PHY_PWRDNEN | \
+ DV_DDR_PHY_EXT_STRBEN | \
+ (0x4 << DV_DDR_PHY_RD_LATENCY_SHIFT))
+
+#define CONFIG_SYS_DA850_DDR2_SDBCR ( \
+ (1 << DV_DDR_SDCR_MSDRAMEN_SHIFT) | \
+ (1 << DV_DDR_SDCR_DDREN_SHIFT) | \
+ (1 << DV_DDR_SDCR_SDRAMEN_SHIFT) | \
+ (1 << DV_DDR_SDCR_BUS_WIDTH_SHIFT) | \
+ (0x3 << DV_DDR_SDCR_CL_SHIFT) | \
+ (0x2 << DV_DDR_SDCR_IBANK_SHIFT) | \
+ (0x2 << DV_DDR_SDCR_PAGESIZE_SHIFT))
+
+/* SDBCR2 is only used if IBANK_POS bit in SDBCR is set */
+#define CONFIG_SYS_DA850_DDR2_SDBCR2 0
+
+#define CONFIG_SYS_DA850_DDR2_SDTIMR ( \
+ (14 << DV_DDR_SDTMR1_RFC_SHIFT) | \
+ (2 << DV_DDR_SDTMR1_RP_SHIFT) | \
+ (2 << DV_DDR_SDTMR1_RCD_SHIFT) | \
+ (1 << DV_DDR_SDTMR1_WR_SHIFT) | \
+ (5 << DV_DDR_SDTMR1_RAS_SHIFT) | \
+ (8 << DV_DDR_SDTMR1_RC_SHIFT) | \
+ (1 << DV_DDR_SDTMR1_RRD_SHIFT) | \
+ (0 << DV_DDR_SDTMR1_WTR_SHIFT))
+
+#define CONFIG_SYS_DA850_DDR2_SDTIMR2 ( \
+ (7 << DV_DDR_SDTMR2_RASMAX_SHIFT) | \
+ (0 << DV_DDR_SDTMR2_XP_SHIFT) | \
+ (0 << DV_DDR_SDTMR2_ODT_SHIFT) | \
+ (17 << DV_DDR_SDTMR2_XSNR_SHIFT) | \
+ (199 << DV_DDR_SDTMR2_XSRD_SHIFT) | \
+ (0 << DV_DDR_SDTMR2_RTP_SHIFT) | \
+ (0 << DV_DDR_SDTMR2_CKE_SHIFT))
+
+#define CONFIG_SYS_DA850_DDR2_SDRCR 0x00000494
+#define CONFIG_SYS_DA850_DDR2_PBBPR 0x30
+
/*
* Serial Driver info
*/
@@ -76,6 +145,7 @@
#define CONFIG_CONS_INDEX 1 /* use UART0 for console */
#define CONFIG_BAUDRATE 115200 /* Default baud rate */
#define CONFIG_SYS_BAUDRATE_TABLE { 9600, 19200, 38400, 57600, 115200 }
+#define CONFIG_SYS_DA850_LPSC_UART DAVINCI_LPSC_UART2
#define CONFIG_SPI
#define CONFIG_SPI_FLASH
@@ -242,6 +312,23 @@
#undef CONFIG_CMD_ENV
#endif
+/* defines for SPL */
+#define CONFIG_SPL
+#define CONFIG_SPL_SPI_SUPPORT
+#define CONFIG_SPL_SPI_FLASH_SUPPORT
+#define CONFIG_SPL_SPI_LOAD
+#define CONFIG_SPL_SPI_BUS 0
+#define CONFIG_SPL_SPI_CS 0
+#define CONFIG_SPL_SERIAL_SUPPORT
+#define CONFIG_SPL_LIBCOMMON_SUPPORT
+#define CONFIG_SPL_LIBGENERIC_SUPPORT
+#define CONFIG_SPL_LDSCRIPT "$(BOARDDIR)/u-boot-spl.lds"
+#define CONFIG_SPL_STACK 0x8001ff00
+#define CONFIG_SPL_TEXT_BASE 0x80000000
+#define CONFIG_SPL_MAX_SIZE 32768
+#define CONFIG_SYS_SPI_U_BOOT_OFFS 0x8000
+#define CONFIG_SYS_SPI_U_BOOT_SIZE 0x30000
+
/* additions for new relocation code, must added to all boards */
#define CONFIG_SYS_SDRAM_BASE 0xc0000000
#define CONFIG_SYS_INIT_SP_ADDR (CONFIG_SYS_SDRAM_BASE + 0x1000 - /* Fix this */ \
--
1.7.0.4
^ permalink raw reply related [flat|nested] 21+ messages in thread* [U-Boot] [PATCH v5 4/6] arm, da850evm: Add an SPL for SPI boot
2011-12-05 10:58 ` [U-Boot] [PATCH v5 4/6] arm, da850evm: Add an SPL for SPI boot Christian Riesch
@ 2011-12-05 15:19 ` Tom Rini
0 siblings, 0 replies; 21+ messages in thread
From: Tom Rini @ 2011-12-05 15:19 UTC (permalink / raw)
To: u-boot
On 12/05/2011 03:58 AM, Christian Riesch wrote:
[snip]
> include/configs/da850evm.h | 87 +++++++++++++++++++++++++++++++++
[snip]
> +#define CONFIG_SYS_DA850_DDR2_SDTIMR2 ( \
> + (7 << DV_DDR_SDTMR2_RASMAX_SHIFT) | \
> + (0 << DV_DDR_SDTMR2_XP_SHIFT) | \
> + (0 << DV_DDR_SDTMR2_ODT_SHIFT) | \
> + (17 << DV_DDR_SDTMR2_XSNR_SHIFT) | \
> + (199 << DV_DDR_SDTMR2_XSRD_SHIFT) | \
> + (0 << DV_DDR_SDTMR2_RTP_SHIFT) | \
> + (0 << DV_DDR_SDTMR2_CKE_SHIFT))
Last time I guess I wasn't clear enough, sorry, there's 0 shifts other
places I disagree with two. I really think in some cases we're best off
just saying "Please look@the ... section of the TRM for this part for
an explanation of all bit fields." and not calculate or'ing in 0 here or
there.
--
Tom
^ permalink raw reply [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v5 5/6] mkimage: Fix variable length header support
2011-12-05 10:58 [U-Boot] [PATCH v5 0/6] Add an SPL to boot the da850evm from SPI Christian Riesch
` (3 preceding siblings ...)
2011-12-05 10:58 ` [U-Boot] [PATCH v5 4/6] arm, da850evm: Add an SPL for SPI boot Christian Riesch
@ 2011-12-05 10:58 ` Christian Riesch
2011-12-05 10:58 ` [U-Boot] [PATCH v5 6/6] arm, davinci: Add support for generating AIS images to the Makefile Christian Riesch
2011-12-05 20:05 ` [U-Boot] [PATCH v5 0/6] Add an SPL to boot the da850evm from SPI Tom Rini
6 siblings, 0 replies; 21+ messages in thread
From: Christian Riesch @ 2011-12-05 10:58 UTC (permalink / raw)
To: u-boot
Support for variable length images like AIS image was introduced
in commit f0662105b674a3874227316abf8536bebc9b5995. A parameter
"-s" was also introduced to prohibit copying of the image file
automatically in the main program. However, this parameter
was implemented incorrectly and the image file was copied
nevertheless.
Signed-off-by: Christian Riesch <christian.riesch@omicron.at>
Cc: Stefano Babic <sbabic@denx.de>
Cc: Heiko Schocher <hs@denx.de>
Acked-by: Stefano Babic <sbabic@denx.de>
---
tools/mkimage.c | 97 ++++++++++++++++++++++++++++---------------------------
1 files changed, 49 insertions(+), 48 deletions(-)
diff --git a/tools/mkimage.c b/tools/mkimage.c
index 36e28ec..eeb1b10 100644
--- a/tools/mkimage.c
+++ b/tools/mkimage.c
@@ -383,65 +383,66 @@ NXTARG: ;
exit (EXIT_FAILURE);
}
- if (!params.skipcpy &&
- (params.type == IH_TYPE_MULTI ||
- params.type == IH_TYPE_SCRIPT)) {
- char *file = params.datafile;
- uint32_t size;
-
- for (;;) {
- char *sep = NULL;
-
- if (file) {
- if ((sep = strchr(file, ':')) != NULL) {
- *sep = '\0';
+ if (!params.skipcpy) {
+ if (params.type == IH_TYPE_MULTI ||
+ params.type == IH_TYPE_SCRIPT) {
+ char *file = params.datafile;
+ uint32_t size;
+
+ for (;;) {
+ char *sep = NULL;
+
+ if (file) {
+ if ((sep = strchr(file, ':')) != NULL) {
+ *sep = '\0';
+ }
+
+ if (stat (file, &sbuf) < 0) {
+ fprintf (stderr, "%s: Can't stat %s: %s\n",
+ params.cmdname, file, strerror(errno));
+ exit (EXIT_FAILURE);
+ }
+ size = cpu_to_uimage (sbuf.st_size);
+ } else {
+ size = 0;
}
- if (stat (file, &sbuf) < 0) {
- fprintf (stderr, "%s: Can't stat %s: %s\n",
- params.cmdname, file, strerror(errno));
+ if (write(ifd, (char *)&size, sizeof(size)) != sizeof(size)) {
+ fprintf (stderr, "%s: Write error on %s: %s\n",
+ params.cmdname, params.imagefile,
+ strerror(errno));
exit (EXIT_FAILURE);
}
- size = cpu_to_uimage (sbuf.st_size);
- } else {
- size = 0;
- }
- if (write(ifd, (char *)&size, sizeof(size)) != sizeof(size)) {
- fprintf (stderr, "%s: Write error on %s: %s\n",
- params.cmdname, params.imagefile,
- strerror(errno));
- exit (EXIT_FAILURE);
- }
+ if (!file) {
+ break;
+ }
- if (!file) {
- break;
+ if (sep) {
+ *sep = ':';
+ file = sep + 1;
+ } else {
+ file = NULL;
+ }
}
- if (sep) {
- *sep = ':';
- file = sep + 1;
- } else {
- file = NULL;
- }
- }
+ file = params.datafile;
- file = params.datafile;
-
- for (;;) {
- char *sep = strchr(file, ':');
- if (sep) {
- *sep = '\0';
- copy_file (ifd, file, 1);
- *sep++ = ':';
- file = sep;
- } else {
- copy_file (ifd, file, 0);
- break;
+ for (;;) {
+ char *sep = strchr(file, ':');
+ if (sep) {
+ *sep = '\0';
+ copy_file (ifd, file, 1);
+ *sep++ = ':';
+ file = sep;
+ } else {
+ copy_file (ifd, file, 0);
+ break;
+ }
}
+ } else {
+ copy_file (ifd, params.datafile, 0);
}
- } else {
- copy_file (ifd, params.datafile, 0);
}
/* We're a bit of paranoid */
--
1.7.0.4
^ permalink raw reply related [flat|nested] 21+ messages in thread* [U-Boot] [PATCH v5 6/6] arm, davinci: Add support for generating AIS images to the Makefile
2011-12-05 10:58 [U-Boot] [PATCH v5 0/6] Add an SPL to boot the da850evm from SPI Christian Riesch
` (4 preceding siblings ...)
2011-12-05 10:58 ` [U-Boot] [PATCH v5 5/6] mkimage: Fix variable length header support Christian Riesch
@ 2011-12-05 10:58 ` Christian Riesch
2011-12-05 20:05 ` [U-Boot] [PATCH v5 0/6] Add an SPL to boot the da850evm from SPI Tom Rini
6 siblings, 0 replies; 21+ messages in thread
From: Christian Riesch @ 2011-12-05 10:58 UTC (permalink / raw)
To: u-boot
Signed-off-by: Christian Riesch <christian.riesch@omicron.at>
Cc: Stefano Babic <sbabic@denx.de>
Cc: Heiko Schocher <hs@denx.de>
Cc: Mike Frysinger <vapier@gentoo.org>
---
.gitignore | 1 +
Makefile | 13 +++++++++++++
2 files changed, 14 insertions(+), 0 deletions(-)
diff --git a/.gitignore b/.gitignore
index ff4bae0..e4e95e2 100644
--- a/.gitignore
+++ b/.gitignore
@@ -35,6 +35,7 @@
/u-boot.dis
/u-boot.lds
/u-boot.ubl
+/u-boot.ais
/u-boot.dtb
/u-boot.sb
diff --git a/Makefile b/Makefile
index fb658f4..5fe54c6 100644
--- a/Makefile
+++ b/Makefile
@@ -417,6 +417,18 @@ $(obj)u-boot.ubl: $(obj)spl/u-boot-spl.bin $(obj)u-boot.bin
rm $(obj)u-boot-ubl.bin
rm $(obj)spl/u-boot-spl-pad.bin
+$(obj)u-boot.ais: $(obj)spl/u-boot-spl.bin $(obj)u-boot.bin
+ $(obj)tools/mkimage -s -n /dev/null -T aisimage \
+ -e $(CONFIG_SPL_TEXT_BASE) \
+ -d $(obj)spl/u-boot-spl.bin \
+ $(obj)spl/u-boot-spl.ais
+ $(OBJCOPY) ${OBJCFLAGS} -I binary \
+ --pad-to=$(CONFIG_SPL_MAX_SIZE) -O binary \
+ $(obj)spl/u-boot-spl.ais $(obj)spl/u-boot-spl-pad.ais
+ cat $(obj)spl/u-boot-spl-pad.ais $(obj)u-boot.bin > \
+ $(obj)u-boot.ais
+ rm $(obj)spl/u-boot-spl{,-pad}.ais
+
$(obj)u-boot.sb: $(obj)u-boot.bin $(obj)spl/u-boot-spl.bin
elftosb -zdf imx28 -c $(TOPDIR)/board/$(BOARDDIR)/u-boot.bd \
-o $(obj)u-boot.sb
@@ -788,6 +800,7 @@ clobber: clean
@rm -f $(obj)u-boot.kwb
@rm -f $(obj)u-boot.imx
@rm -f $(obj)u-boot.ubl
+ @rm -f $(obj)u-boot.ais
@rm -f $(obj)u-boot.dtb
@rm -f $(obj)u-boot.sb
@rm -f $(obj)tools/{env/crc32.c,inca-swap-bytes}
--
1.7.0.4
^ permalink raw reply related [flat|nested] 21+ messages in thread* [U-Boot] [PATCH v5 0/6] Add an SPL to boot the da850evm from SPI
2011-12-05 10:58 [U-Boot] [PATCH v5 0/6] Add an SPL to boot the da850evm from SPI Christian Riesch
` (5 preceding siblings ...)
2011-12-05 10:58 ` [U-Boot] [PATCH v5 6/6] arm, davinci: Add support for generating AIS images to the Makefile Christian Riesch
@ 2011-12-05 20:05 ` Tom Rini
2011-12-06 16:46 ` Tom Rini
6 siblings, 1 reply; 21+ messages in thread
From: Tom Rini @ 2011-12-05 20:05 UTC (permalink / raw)
To: u-boot
On Mon, Dec 5, 2011 at 3:58 AM, Christian Riesch
<christian.riesch@omicron.at> wrote:
>
> Hi,
> this is v5 of the last part of my recent patchset
Following on to my last email in 2/6 of the series, if there's no
objections other than mine about commenting and not doing 0 bit
shifts, and that's an agreeable change, if there's no other objections
within 24h I'll put the series into u-boot-ti/master and send a pull
request to Albert. Thanks!
--
Tom
^ permalink raw reply [flat|nested] 21+ messages in thread* [U-Boot] [PATCH v5 0/6] Add an SPL to boot the da850evm from SPI
2011-12-05 20:05 ` [U-Boot] [PATCH v5 0/6] Add an SPL to boot the da850evm from SPI Tom Rini
@ 2011-12-06 16:46 ` Tom Rini
2011-12-06 17:08 ` Christian Riesch
0 siblings, 1 reply; 21+ messages in thread
From: Tom Rini @ 2011-12-06 16:46 UTC (permalink / raw)
To: u-boot
On Mon, Dec 5, 2011 at 1:05 PM, Tom Rini <tom.rini@gmail.com> wrote:
> On Mon, Dec 5, 2011 at 3:58 AM, Christian Riesch
> <christian.riesch@omicron.at> wrote:
>>
>> Hi,
>> this is v5 of the last part of my recent patchset
>
> Following on to my last email in 2/6 of the series, if there's no
> objections other than mine about commenting and not doing 0 bit
> shifts, and that's an agreeable change, if there's no other objections
> within 24h I'll put the series into u-boot-ti/master and send a pull
> request to Albert. ?Thanks!
OK, what toolchain are you using? For da850evm and gcc 4.5 I get:
arm-linux-gnueabi-ld: error: no memory region specified for loadable
section `.ARM.exidx'
make[1]: *** [/home/trini/work/u-boot/u-boot-ti/da850evm/spl/u-boot-spl] Error 1
make: *** [/home/trini/work/u-boot/u-boot-ti/da850evm/spl/u-boot-spl.bin]
Error 2
make: *** Waiting for unfinished jobs....
gcc-4.6 (Linaro, Ubuntu 'Lucid' packages) are fine. This isn't a
vanilla 4.5 however so if this works for others, I'm OK.
--
Tom
^ permalink raw reply [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v5 0/6] Add an SPL to boot the da850evm from SPI
2011-12-06 16:46 ` Tom Rini
@ 2011-12-06 17:08 ` Christian Riesch
2011-12-06 17:14 ` Tom Rini
0 siblings, 1 reply; 21+ messages in thread
From: Christian Riesch @ 2011-12-06 17:08 UTC (permalink / raw)
To: u-boot
Hi Tom,
On Tue, Dec 6, 2011 at 5:46 PM, Tom Rini <tom.rini@gmail.com> wrote:
> On Mon, Dec 5, 2011 at 1:05 PM, Tom Rini <tom.rini@gmail.com> wrote:
>> On Mon, Dec 5, 2011 at 3:58 AM, Christian Riesch
>> <christian.riesch@omicron.at> wrote:
>>>
>>> Hi,
>>> this is v5 of the last part of my recent patchset
>>
>> Following on to my last email in 2/6 of the series, if there's no
>> objections other than mine about commenting and not doing 0 bit
>> shifts, and that's an agreeable change, if there's no other objections
>> within 24h I'll put the series into u-boot-ti/master and send a pull
>> request to Albert. ?Thanks!
>
> OK, what toolchain are you using? ?For da850evm and gcc 4.5 I get:
I'm using
gcc version 4.5.2 (Sourcery G++ Lite 2011.03-41)
make da850evm_config && make -s -j3 u-boot.ais
builds without warnings.
Regards, Christian
> arm-linux-gnueabi-ld: error: no memory region specified for loadable
> section `.ARM.exidx'
> make[1]: *** [/home/trini/work/u-boot/u-boot-ti/da850evm/spl/u-boot-spl] Error 1
> make: *** [/home/trini/work/u-boot/u-boot-ti/da850evm/spl/u-boot-spl.bin]
> Error 2
> make: *** Waiting for unfinished jobs....
>
> gcc-4.6 (Linaro, Ubuntu 'Lucid' packages) are fine. ?This isn't a
> vanilla 4.5 however so if this works for others, I'm OK.
>
> --
> Tom
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
^ permalink raw reply [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v5 0/6] Add an SPL to boot the da850evm from SPI
2011-12-06 17:08 ` Christian Riesch
@ 2011-12-06 17:14 ` Tom Rini
2011-12-06 17:32 ` Christian Riesch
0 siblings, 1 reply; 21+ messages in thread
From: Tom Rini @ 2011-12-06 17:14 UTC (permalink / raw)
To: u-boot
On 12/06/2011 10:08 AM, Christian Riesch wrote:
> Hi Tom,
>
> On Tue, Dec 6, 2011 at 5:46 PM, Tom Rini <tom.rini@gmail.com> wrote:
>> On Mon, Dec 5, 2011 at 1:05 PM, Tom Rini <tom.rini@gmail.com> wrote:
>>> On Mon, Dec 5, 2011 at 3:58 AM, Christian Riesch
>>> <christian.riesch@omicron.at> wrote:
>>>>
>>>> Hi,
>>>> this is v5 of the last part of my recent patchset
>>>
>>> Following on to my last email in 2/6 of the series, if there's no
>>> objections other than mine about commenting and not doing 0 bit
>>> shifts, and that's an agreeable change, if there's no other objections
>>> within 24h I'll put the series into u-boot-ti/master and send a pull
>>> request to Albert. Thanks!
>>
>> OK, what toolchain are you using? For da850evm and gcc 4.5 I get:
>
> I'm using
> gcc version 4.5.2 (Sourcery G++ Lite 2011.03-41)
>
> make da850evm_config && make -s -j3 u-boot.ais
> builds without warnings.
How about 'all' (or da850evm) ?
--
Tom
^ permalink raw reply [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v5 0/6] Add an SPL to boot the da850evm from SPI
2011-12-06 17:14 ` Tom Rini
@ 2011-12-06 17:32 ` Christian Riesch
2011-12-06 17:34 ` Tom Rini
0 siblings, 1 reply; 21+ messages in thread
From: Christian Riesch @ 2011-12-06 17:32 UTC (permalink / raw)
To: u-boot
On Tue, Dec 6, 2011 at 6:14 PM, Tom Rini <trini@ti.com> wrote:
> On 12/06/2011 10:08 AM, Christian Riesch wrote:
>> Hi Tom,
>>
>> On Tue, Dec 6, 2011 at 5:46 PM, Tom Rini <tom.rini@gmail.com> wrote:
>>> On Mon, Dec 5, 2011 at 1:05 PM, Tom Rini <tom.rini@gmail.com> wrote:
>>>> On Mon, Dec 5, 2011 at 3:58 AM, Christian Riesch
>>>> <christian.riesch@omicron.at> wrote:
>>>>>
>>>>> Hi,
>>>>> this is v5 of the last part of my recent patchset
>>>>
>>>> Following on to my last email in 2/6 of the series, if there's no
>>>> objections other than mine about commenting and not doing 0 bit
>>>> shifts, and that's an agreeable change, if there's no other objections
>>>> within 24h I'll put the series into u-boot-ti/master and send a pull
>>>> request to Albert. ?Thanks!
>>>
>>> OK, what toolchain are you using? ?For da850evm and gcc 4.5 I get:
>>
>> I'm using
>> gcc version 4.5.2 (Sourcery G++ Lite 2011.03-41)
>>
>> make da850evm_config && make -s -j3 u-boot.ais
>> builds without warnings.
>
> How about 'all' (or da850evm) ?
$ cd u-boot
$ export CROSS_COMPILE=arm-none-linux-gnueabi-
$ make mrproper
$ make -s da850evm
Configuring for da850evm board...
Generating include/generated/asm-offsets.h
$
So this worked.
$ make mrproper
$ make da850evm_config
awk '(NF && $1 !~ /^#/) { print $1 ": " $1 "_config; $(MAKE)" }'
boards.cfg > .boards.depend
Configuring for da850evm board...
$ make -s all
Generating include/generated/asm-offsets.h
$
So this way it worked as well.
$ make -s u-boot.ais
Image Type: TI Davinci AIS Boot Image
AIS magic : 41504954
Image at : 0x80000000 size 0x000044ec
$
Ok.
This is the current ti master (commit
f50dda60868a9deeb70d220867704dec06f5b7b4) plus my patchset.
Regards, Christian
^ permalink raw reply [flat|nested] 21+ messages in thread* [U-Boot] [PATCH v5 0/6] Add an SPL to boot the da850evm from SPI
2011-12-06 17:32 ` Christian Riesch
@ 2011-12-06 17:34 ` Tom Rini
2011-12-06 22:17 ` Tom Rini
0 siblings, 1 reply; 21+ messages in thread
From: Tom Rini @ 2011-12-06 17:34 UTC (permalink / raw)
To: u-boot
On Tue, Dec 6, 2011 at 10:32 AM, Christian Riesch
<christian.riesch@omicron.at> wrote:
> On Tue, Dec 6, 2011 at 6:14 PM, Tom Rini <trini@ti.com> wrote:
>> On 12/06/2011 10:08 AM, Christian Riesch wrote:
>>> Hi Tom,
>>>
>>> On Tue, Dec 6, 2011 at 5:46 PM, Tom Rini <tom.rini@gmail.com> wrote:
>>>> On Mon, Dec 5, 2011 at 1:05 PM, Tom Rini <tom.rini@gmail.com> wrote:
>>>>> On Mon, Dec 5, 2011 at 3:58 AM, Christian Riesch
>>>>> <christian.riesch@omicron.at> wrote:
>>>>>>
>>>>>> Hi,
>>>>>> this is v5 of the last part of my recent patchset
>>>>>
>>>>> Following on to my last email in 2/6 of the series, if there's no
>>>>> objections other than mine about commenting and not doing 0 bit
>>>>> shifts, and that's an agreeable change, if there's no other objections
>>>>> within 24h I'll put the series into u-boot-ti/master and send a pull
>>>>> request to Albert. ?Thanks!
>>>>
>>>> OK, what toolchain are you using? ?For da850evm and gcc 4.5 I get:
>>>
>>> I'm using
>>> gcc version 4.5.2 (Sourcery G++ Lite 2011.03-41)
>>>
>>> make da850evm_config && make -s -j3 u-boot.ais
>>> builds without warnings.
>>
>> How about 'all' (or da850evm) ?
>
> $ cd u-boot
> $ export CROSS_COMPILE=arm-none-linux-gnueabi-
> $ make mrproper
> $ make -s da850evm
> Configuring for da850evm board...
> Generating include/generated/asm-offsets.h
> $
>
> So this worked.
>
> $ make mrproper
> $ make da850evm_config
> awk '(NF && $1 !~ /^#/) { print $1 ": " $1 "_config; $(MAKE)" }'
> boards.cfg > .boards.depend
> Configuring for da850evm board...
> $ make -s all
> Generating include/generated/asm-offsets.h
> $
>
> So this way it worked as well.
>
> $ make -s u-boot.ais
> Image Type: ? TI Davinci AIS Boot Image
> AIS magic : ? 41504954
> Image at ?: ? 0x80000000 size 0x000044ec
> $
>
> Ok.
>
> This is the current ti master (commit
> f50dda60868a9deeb70d220867704dec06f5b7b4) plus my patchset.
OK, thanks, I'll chalk it up to my toolchain then.
--
Tom
^ permalink raw reply [flat|nested] 21+ messages in thread* [U-Boot] [PATCH v5 0/6] Add an SPL to boot the da850evm from SPI
2011-12-06 17:34 ` Tom Rini
@ 2011-12-06 22:17 ` Tom Rini
2011-12-07 9:32 ` Christian Riesch
0 siblings, 1 reply; 21+ messages in thread
From: Tom Rini @ 2011-12-06 22:17 UTC (permalink / raw)
To: u-boot
On Tue, Dec 6, 2011 at 10:34 AM, Tom Rini <tom.rini@gmail.com> wrote:
> On Tue, Dec 6, 2011 at 10:32 AM, Christian Riesch
> <christian.riesch@omicron.at> wrote:
>> On Tue, Dec 6, 2011 at 6:14 PM, Tom Rini <trini@ti.com> wrote:
>>> On 12/06/2011 10:08 AM, Christian Riesch wrote:
>>>> Hi Tom,
>>>>
>>>> On Tue, Dec 6, 2011 at 5:46 PM, Tom Rini <tom.rini@gmail.com> wrote:
>>>>> On Mon, Dec 5, 2011 at 1:05 PM, Tom Rini <tom.rini@gmail.com> wrote:
>>>>>> On Mon, Dec 5, 2011 at 3:58 AM, Christian Riesch
>>>>>> <christian.riesch@omicron.at> wrote:
>>>>>>>
>>>>>>> Hi,
>>>>>>> this is v5 of the last part of my recent patchset
>>>>>>
>>>>>> Following on to my last email in 2/6 of the series, if there's no
>>>>>> objections other than mine about commenting and not doing 0 bit
>>>>>> shifts, and that's an agreeable change, if there's no other objections
>>>>>> within 24h I'll put the series into u-boot-ti/master and send a pull
>>>>>> request to Albert. ?Thanks!
>>>>>
>>>>> OK, what toolchain are you using? ?For da850evm and gcc 4.5 I get:
>>>>
>>>> I'm using
>>>> gcc version 4.5.2 (Sourcery G++ Lite 2011.03-41)
>>>>
>>>> make da850evm_config && make -s -j3 u-boot.ais
>>>> builds without warnings.
>>>
>>> How about 'all' (or da850evm) ?
>>
>> $ cd u-boot
>> $ export CROSS_COMPILE=arm-none-linux-gnueabi-
>> $ make mrproper
>> $ make -s da850evm
>> Configuring for da850evm board...
>> Generating include/generated/asm-offsets.h
>> $
>>
>> So this worked.
>>
>> $ make mrproper
>> $ make da850evm_config
>> awk '(NF && $1 !~ /^#/) { print $1 ": " $1 "_config; $(MAKE)" }'
>> boards.cfg > .boards.depend
>> Configuring for da850evm board...
>> $ make -s all
>> Generating include/generated/asm-offsets.h
>> $
>>
>> So this way it worked as well.
>>
>> $ make -s u-boot.ais
>> Image Type: ? TI Davinci AIS Boot Image
>> AIS magic : ? 41504954
>> Image at ?: ? 0x80000000 size 0x000044ec
>> $
>>
>> Ok.
>>
>> This is the current ti master (commit
>> f50dda60868a9deeb70d220867704dec06f5b7b4) plus my patchset.
>
> OK, thanks, I'll chalk it up to my toolchain then.
After talking with Wolfgang and Albert, we (OK, Wolfgang) found that
ELDK 4.2 toolchain also shows this issue, so please grab that and
figure out what's going on here. Thanks!
--
Tom
^ permalink raw reply [flat|nested] 21+ messages in thread* [U-Boot] [PATCH v5 0/6] Add an SPL to boot the da850evm from SPI
2011-12-06 22:17 ` Tom Rini
@ 2011-12-07 9:32 ` Christian Riesch
2011-12-09 10:36 ` Christian Riesch
0 siblings, 1 reply; 21+ messages in thread
From: Christian Riesch @ 2011-12-07 9:32 UTC (permalink / raw)
To: u-boot
Hi Tom,
On Tue, Dec 6, 2011 at 11:17 PM, Tom Rini <tom.rini@gmail.com> wrote:
> On Tue, Dec 6, 2011 at 10:34 AM, Tom Rini <tom.rini@gmail.com> wrote:
[...]
>> OK, thanks, I'll chalk it up to my toolchain then.
>
> After talking with Wolfgang and Albert, we (OK, Wolfgang) found that
> ELDK 4.2 toolchain also shows this issue, so please grab that and
> figure out what's going on here. Thanks!
With ELDK 4.2 I got the same error message:
arm-linux-ld: error: no memory region specified for loadable section
`.ARM.exidx'
In my linker script for the SPL (board/davinci/da8xxevm/u-boot-spl.lds) no
such section is defined. But somehow the linker in the Codesourcery
toolchain that I used before manages to place the .ARM.exidx section
between rodata and data automatically (The section is listed in the
corresponding .map file). However, with the ELDK 4.2 toolchain this section
(and the .got section) must be specified explicitly. With the patch below
(applied on top of the patchset) the SPL builds and links nicely with
both ELDK 4.2 and the Codesourcery toolchain.
Tom, could you please try if that works for your toolchain?
I don't know much about linker scripts, so my question is: Is this
the right way to solve this problem?
Thanks, Christian
---
board/davinci/da8xxevm/u-boot-spl.lds | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/board/davinci/da8xxevm/u-boot-spl.lds b/board/davinci/da8xxevm/u-boot-spl.lds
index 6f6e065..1ba0a59 100644
--- a/board/davinci/da8xxevm/u-boot-spl.lds
+++ b/board/davinci/da8xxevm/u-boot-spl.lds
@@ -44,6 +44,8 @@ SECTIONS
. = ALIGN(4);
.rodata : { *(SORT_BY_ALIGNMENT(.rodata*)) } >.sram
+ .ARM.exidx : { *(.ARM.exidx*) } > .sram
+ .got : { *(.got*) } > .sram
. = ALIGN(4);
.data : { *(SORT_BY_ALIGNMENT(.data*)) } >.sram
--
1.7.0.4
^ permalink raw reply related [flat|nested] 21+ messages in thread* [U-Boot] [PATCH v5 0/6] Add an SPL to boot the da850evm from SPI
2011-12-07 9:32 ` Christian Riesch
@ 2011-12-09 10:36 ` Christian Riesch
0 siblings, 0 replies; 21+ messages in thread
From: Christian Riesch @ 2011-12-09 10:36 UTC (permalink / raw)
To: u-boot
Hi again,
On Wed, Dec 7, 2011 at 10:32 AM, Christian Riesch <christian.riesch@omicron.at> wrote:
> Hi Tom,
>
> On Tue, Dec 6, 2011 at 11:17 PM, Tom Rini <tom.rini@gmail.com> wrote:
>> On Tue, Dec 6, 2011 at 10:34 AM, Tom Rini <tom.rini@gmail.com> wrote:
> [...]
>>> OK, thanks, I'll chalk it up to my toolchain then.
>>
>> After talking with Wolfgang and Albert, we (OK, Wolfgang) found that
>> ELDK 4.2 toolchain also shows this issue, so please grab that and
>> figure out what's going on here. Thanks!
>
> With ELDK 4.2 I got the same error message:
>
> arm-linux-ld: error: no memory region specified for loadable section
> `.ARM.exidx'
>
> In my linker script for the SPL (board/davinci/da8xxevm/u-boot-spl.lds) no
> such section is defined. But somehow the linker in the Codesourcery
> toolchain that I used before manages to place the .ARM.exidx section
> between rodata and data automatically (The section is listed in the
> corresponding .map file). However, with the ELDK 4.2 toolchain this section
> (and the .got section) must be specified explicitly. With the patch below
> (applied on top of the patchset) the SPL builds and links nicely with
> both ELDK 4.2 and the Codesourcery toolchain.
That's probably not the right way to solve the problem.
A run of ./MAKEALL -c arm926ejs with a modified version of MAKEALL that
copied the resulting u-boot.map file of each build into the log directory and
subsequently doing
for i in `ls *.map`; do grep exidx $i > /dev/null && echo $i; done
showed that all davinci boards have this .ARM.exidx section, so it must be
something in the davinci arch tree. After reading a lot of code and running
nm over the object files I found out that there is some connection between
the 64 bit division that are done in arch/arm/cpu/arm926ejs/davinci/timer.c
and these .exidx sections.
The relevant code in timer.c is
ulong get_timer(ulong base)
{
unsigned long long timer_diff;
timer_diff = get_ticks() - gd->timer_reset_value;
return (timer_diff / (gd->timer_rate_hz / CONFIG_SYS_HZ)) - base;
}
void __udelay(unsigned long usec)
{
unsigned long long endtime;
endtime = ((unsigned long long)usec * gd->timer_rate_hz) / 1000000UL;
endtime += get_ticks();
while (get_ticks() < endtime)
;
}
When I replace both divisions with the lldiv function from ldiv64.h (I have
seen that a lot of patches did such replacements in the u-boot code) the
exidx sections vanish (for all davinci based boards) and my patches compile
fine even with the ELDK 4.2 toolchain.
The patch below does the required changes in timer.c and also one replacement
in post/post.c.
Could you please comment on this solution?
@Heiko: The SPL of the cam_enc_4xx board suffered from the same linker
problem (.ARM.exidx section) and also after applying the patch I could
not build this board with ELDK 4.2 (although it works fine with
gcc version 4.5.2 (Sourcery G++ Lite 2011.03-41). Could you please
have a look@this? (arm-linux-ld: u-boot-spl: Not enough room for program
headers, try linking with -N)
Regards, Christian
---
arch/arm/cpu/arm926ejs/davinci/timer.c | 6 ++++--
post/post.c | 3 ++-
2 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/arch/arm/cpu/arm926ejs/davinci/timer.c b/arch/arm/cpu/arm926ejs/davinci/timer.c
index c7bf7a5..31f8633 100644
--- a/arch/arm/cpu/arm926ejs/davinci/timer.c
+++ b/arch/arm/cpu/arm926ejs/davinci/timer.c
@@ -40,6 +40,7 @@
#include <common.h>
#include <asm/io.h>
#include <asm/arch/timer_defs.h>
+#include <div64.h>
DECLARE_GLOBAL_DATA_PTR;
@@ -86,14 +87,15 @@ ulong get_timer(ulong base)
timer_diff = get_ticks() - gd->timer_reset_value;
- return (timer_diff / (gd->timer_rate_hz / CONFIG_SYS_HZ)) - base;
+ return lldiv(timer_diff, (gd->timer_rate_hz / CONFIG_SYS_HZ)) - base;
}
void __udelay(unsigned long usec)
{
unsigned long long endtime;
- endtime = ((unsigned long long)usec * gd->timer_rate_hz) / 1000000UL;
+ endtime = lldiv((unsigned long long)usec * gd->timer_rate_hz,
+ 1000000UL);
endtime += get_ticks();
while (get_ticks() < endtime)
diff --git a/post/post.c b/post/post.c
index 0e67ad7..745767a 100644
--- a/post/post.c
+++ b/post/post.c
@@ -24,6 +24,7 @@
#include <common.h>
#include <stdio_dev.h>
#include <watchdog.h>
+#include <div64.h>
#include <post.h>
#ifdef CONFIG_SYS_POST_HOTKEYS_GPIO
@@ -495,7 +496,7 @@ void post_reloc(void)
unsigned long post_time_ms(unsigned long base)
{
#if defined(CONFIG_PPC) || defined(CONFIG_ARM)
- return (unsigned long)(get_ticks() / (get_tbclk() / CONFIG_SYS_HZ))
+ return (unsigned long) lldiv(get_ticks(), (get_tbclk() / CONFIG_SYS_HZ))
- base;
#else
#warning "Not implemented yet"
--
1.7.0.4
^ permalink raw reply related [flat|nested] 21+ messages in thread