[U-Boot] [PATCH] panic: remove warning "'noreturn' function does return"

[Heiko Schocher]

since commit

commit d2e8b911c0a0661d395ccac72156040702ac842d
Author: Mike Frysinger 
Date:   Wed Jun 29 11:58:04 2011 +

panic: add noreturn attribute

I see the following warnings:

vsprintf.c: In function 'panic':
vsprintf.c:730: warning: 'noreturn' function does return

for nearly all boards. This patch fixes this warning.

Signed-off-by: Heiko Schocher 
cc: Mike Frysinger 
---
 lib/vsprintf.c |2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/lib/vsprintf.c b/lib/vsprintf.c
index 3b924ec..c029fbb 100644
--- a/lib/vsprintf.c
+++ b/lib/vsprintf.c
@@ -727,4 +727,6 @@ void panic(const char *fmt, ...)
udelay (10);/* allow messages to go out */
do_reset (NULL, 0, 0, NULL);
 #endif
+   while (1)
+   ;
 }
-- 
1.7.6

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH v2] post, memory test: add memory_post_test() to include file

[Heiko Schocher]

This include is needed, if this memory test is used "outside"
from post code, for example booting with nand_spl, and using
this memory test before copying u-boot code to RAM and jumping
to it.

Signed-off-by: Heiko Schocher 

---
changes for v2:
- make MAKEALL clean as Wolfgang Denk suggested
- rebased against 7130a579fdba5dd1bf99508fb0b1d13317542109


 include/post.h |1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/include/post.h b/include/post.h
index 3f259b7..3d23d22 100644
--- a/include/post.h
+++ b/include/post.h
@@ -147,6 +147,7 @@ unsigned long post_time_ms (unsigned long base);
 extern struct post_test post_list[];
 extern unsigned int post_list_size;
 extern int post_hotkeys_pressed(void);
+extern int memory_post_test(int flags);
 
 /*
  *  If GCC is configured to use a version of GAS that supports
-- 
1.7.6

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] problem with mpc837x start.S

[shawn Bai]

Hello, boys and girls, my name is shawn Bai. I am new here and hope not disturb 
you.
Although I have not posted before, what you do is very helpful to me to 
understand what is bsp, how it runs, and so on.
I don't know whether the question below is mentioned or not before.
If yes, do you mind telling me where I can find the answer.
Thanks in advance.

The question is about arch\powerpc\cpu\83xx\start.S.
the demo board I use is MPC837xERDB, there is 8MBytes Nor flash on it.
by default, uboot is booted from lower address, it means BMS field in HRCW is 
0, which will affect value of MSR[IP] later.
First instruction locates at _start label in uboot.
Before first instruction runs, hardware reset configuration word will be loaded 
to make some basic configuration.
Before branching to MONITOR_BASE+in_flash using blr instruction， size of LCS0 
is inflated to 4GBytes，
and eLBC access window 1 is configured with BAR MONITOR_BASE, and SIZE 8MBytes.
After this is done, blr to MONITOR_BASE+in_flash.
after branching to MONITOR_BASE+in_flash, LCS0 BAR is modified to MONITOR_BASE, 
and its size is configured with 8Mbytes again.
This is what I know.
But the problem which is confused to me is
after reset, the default value of LCS0 BAR is 0x_, and its size is 
8Mbytes.
In order to branch to MONITOR_BASE+in_flash, inlarge Nor flash to 4GB, and 
configure eLBC access window 1. 
But the actual size of Nor flash is 8MB , even after inlarging it, where 
(MONITOR_BASE+in_flash) points to? why?
why does branching to in_flash need absolute address? and make absolute address 
like this?
I rack my brains, but there is no result.
Do you mind showing me a way?
Thanks very much.
 
Shawn Bai

>
> --
>
> ___
> U-Boot mailing list
> U-Boot@lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>
>
> **
>   
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] arm, davinci: replace CONFIG_PRELOADER with CONFIG_SPL_BUILD

[Heiko Schocher]

Signed-off-by: Heiko Schocher 
---
 board/davinci/common/misc.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/board/davinci/common/misc.c b/board/davinci/common/misc.c
index 6103339..fed4900 100644
--- a/board/davinci/common/misc.c
+++ b/board/davinci/common/misc.c
@@ -33,7 +33,7 @@
 
 DECLARE_GLOBAL_DATA_PTR;
 
-#ifndef CONFIG_PRELOADER
+#ifndef CONFIG_SPL_BUILD
 int dram_init(void)
 {
/* dram_init must store complete ramsize in gd->ram_size */
-- 
1.7.6

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2 6/7] ARMV7: OMAP: I2C driver: Write more than 1 byte at a time in i2c_write

[Heiko Schocher]

Hello Michael,

Sorry for the long delay...

Michael Jones wrote:
> This allows the EEPROM layer to send a single i2c write command
> per page, and wait CONFIG_SYS_EEPROM_PAGE_WRITE_DELAY_MS between
> i2c write commands.
> 
> Signed-off-by: Michael Jones 
> ---
> Changes for v2:
>   - None. Resubmitting to include custodian in cc:
> 
>  drivers/i2c/omap24xx_i2c.c |  134 ++-
>  1 files changed, 56 insertions(+), 78 deletions(-)
> 
> diff --git a/drivers/i2c/omap24xx_i2c.c b/drivers/i2c/omap24xx_i2c.c
> index 966ffc4..4ae03bc 100644
> --- a/drivers/i2c/omap24xx_i2c.c
> +++ b/drivers/i2c/omap24xx_i2c.c
[...]
> @@ -372,26 +301,75 @@ int i2c_read (uchar chip, uint addr, int alen, uchar * 
> buffer, int len)
>  int i2c_write (uchar chip, uint addr, int alen, uchar * buffer, int len)
>  {
>   int i;
> + u16 status;
> + int i2c_error = 0;
>  
>   if (alen > 1) {
> - printf ("I2C read: addr len %d not supported\n", alen);
> + printf("I2C write: addr len %d not supported\n", alen);
>   return 1;
>   }
>  
>   if (addr + len > 256) {
> - printf ("I2C read: address out of range\n");
> + printf("I2C write: address 0x%x + 0x%x out of range\n");
>   return 1;
>   }
>  
> + /* wait until bus not busy */
> + wait_for_bb();
> +
> + /* start address phase - will write regoffset + len bytes data */
> + /* TODO consider case when !CONFIG_OMAP243X/34XX/44XX */

Do we have this usecase?

> + writew(alen+len, &i2c_base->cnt);

please change to "alen + len"

> + /* set slave address */
> + writew(chip, &i2c_base->sa);
> + /* stop bit needed here */
> + writew(I2C_CON_EN | I2C_CON_MST | I2C_CON_STT | I2C_CON_TRX |
> + I2C_CON_STP, &i2c_base->con);
> +
> + /* Send address byte */
> + status = wait_for_pin();
> +
> + if (status == 0 || status & I2C_STAT_NACK) {
> + i2c_error = 1;
> + printf("%s:%d error status=0x%x\n", __func__, __LINE__, status);

Can you change this printf to output some more info, instead __func__, __LINE__?

> + goto write_exit;
> + }
> +
> + if (status & I2C_STAT_XRDY) {
> + writeb(addr & 0xFF, &i2c_base->data);
> + writew(I2C_STAT_XRDY, &i2c_base->stat);
> + } else {
> + i2c_error = 1;
> + printf("%s:%d error status=0x%x\n", __func__, __LINE__, status);

here too.

> + goto write_exit;
> + }
> +
> + /* address phase is over, now write data */
>   for (i = 0; i < len; i++) {
> - if (i2c_write_byte (chip, addr + i, buffer[i])) {
> - printf ("I2C read: I/O error\n");
> - i2c_init (CONFIG_SYS_I2C_SPEED, CONFIG_SYS_I2C_SLAVE);
> - return 1;
> + status = wait_for_pin();
> +
> + if (status == 0 || status & I2C_STAT_NACK) {
> + i2c_error = 1;
> + printf("%s:%d error status=0x%x\n",
> + __func__, __LINE__, status);

also here ...

> + goto write_exit;
> + }
> +
> + if (status & I2C_STAT_XRDY) {
> + writeb(buffer[i], &i2c_base->data);
> + writew(I2C_STAT_XRDY, &i2c_base->stat);
> + } else {
> + i2c_error = 1;
> + printf("%s:%d i=%d error status=0x%x\n",
> + __func__, __LINE__, i, status);

and here.

> + goto write_exit;
>   }
>   }
>  
> - return 0;
> +write_exit:
> + flush_fifo();
> + writew(0x, &i2c_base->stat);
> + return i2c_error;
>  }
>  
>  static void wait_for_bb (void)

bye,
Heiko
-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] compiler warning associated with noreturn

[Kumar Gala]

With the following commit:

commit d2e8b911c0a0661d395ccac72156040702ac842d
Author: Mike Frysinger 
Date:   Wed Jun 29 11:58:04 2011 +

panic: add noreturn attribute

we now get warnings like the following if we build MPC8540ADS_config:

vsprintf.c: In function 'panic':
vsprintf.c:730:1: warning: 'noreturn' function does return

Seems like do_reset() needs a similar attribute?  Or some other change?

- k
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] powerpc/85xx: Cleanup handling of PVR detection for e500/e500mc/e5500

[Kumar Gala]

At some point we broke the detection of e500v1 class cores.  Fix that
and simply the code to just utilize PVR_VER() to have a single case
statement.

Signed-off-by: Kumar Gala 
---
 arch/powerpc/cpu/mpc85xx/cpu.c   |   32 +---
 arch/powerpc/include/asm/processor.h |4 
 2 files changed, 17 insertions(+), 19 deletions(-)

diff --git a/arch/powerpc/cpu/mpc85xx/cpu.c b/arch/powerpc/cpu/mpc85xx/cpu.c
index ce59c25..976e86e 100644
--- a/arch/powerpc/cpu/mpc85xx/cpu.c
+++ b/arch/powerpc/cpu/mpc85xx/cpu.c
@@ -46,7 +46,6 @@ int checkcpu (void)
 {
sys_info_t sysinfo;
uint pvr, svr;
-   uint fam;
uint ver;
uint major, minor;
struct cpu_type *cpu;
@@ -94,30 +93,25 @@ int checkcpu (void)
printf(", Version: %d.%d, (0x%08x)\n", major, minor, svr);
 
pvr = get_pvr();
-   fam = PVR_FAM(pvr);
ver = PVR_VER(pvr);
major = PVR_MAJ(pvr);
minor = PVR_MIN(pvr);
 
printf("Core:  ");
-   if (PVR_FAM(PVR_85xx)) {
-   switch(PVR_MEM(pvr)) {
-   case 0x1:
-   case 0x2:
-   puts("E500");
-   break;
-   case 0x3:
-   puts("E500MC");
-   break;
-   case 0x4:
-   puts("E5500");
-   break;
-   default:
-   puts("Unknown");
-   break;
-   }
-   } else {
+   switch(ver) {
+   case PVR_VER_E500_V1:
+   case PVR_VER_E500_V2:
+   puts("E500");
+   break;
+   case PVR_VER_E500MC:
+   puts("E500MC");
+   break;
+   case PVR_VER_E5500:
+   puts("E5500");
+   break;
+   default:
puts("Unknown");
+   break;
}
 
printf(", Version: %d.%d, (0x%08x)\n", major, minor, pvr);
diff --git a/arch/powerpc/include/asm/processor.h 
b/arch/powerpc/include/asm/processor.h
index 9c4651a..0c4cc25 100644
--- a/arch/powerpc/include/asm/processor.h
+++ b/arch/powerpc/include/asm/processor.h
@@ -938,6 +938,10 @@
 #define PVR_85xx   0x8020
 #define PVR_85xx_REV1  (PVR_85xx | 0x0010)
 #define PVR_85xx_REV2  (PVR_85xx | 0x0020)
+#define PVR_VER_E500_V10x8020
+#define PVR_VER_E500_V20x8021
+#define PVR_VER_E500MC 0x8023
+#define PVR_VER_E5500  0x8024
 
 #define PVR_86xx   0x8004
 
-- 
1.7.3.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] [v3] [upstream] powerpc/85xx: Adding configuration for DCSRCR to enable 32M access

[Kumar Gala]

On Jul 20, 2011, at 9:47 AM, Stephen George wrote:

> Configuring DCSRCR to define the DCSR space to be 1G instead
> of the default 4M. DCSRCR only allows selection of either 4M
> or 1G.
> Most DCSR registers are within 4M but the Nexus trace buffer
> is located at offset 16M within the DCSR.
> 
> Configuring the LAW to be 32M to allow access to the Nexus
> trace buffer. No TLB modification is required since accessing
> the Nexus trace buffer from within u-boot is not required.
> 
> Signed-off-by: Stephen George 
> ---
> arch/powerpc/cpu/mpc85xx/cpu_init.c   |   11 +++
> arch/powerpc/include/asm/immap_85xx.h |7 ++-
> board/freescale/corenet_ds/law.c  |3 ++-
> 3 files changed, 19 insertions(+), 2 deletions(-)

applied to 85xx

- k
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] powerpc/p2041rdb: Enable backside L2 cache support

[Kumar Gala]

On Jul 21, 2011, at 5:03 PM, Kumar Gala wrote:

> From: Mingkai Hu 
> 
> Signed-off-by: Mingkai Hu 
> Signed-off-by: Kumar Gala 
> ---
> include/configs/P2041RDB.h |2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)

applied to 85xx

- k
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] powerpc/85xx: Fix setting of EPAPR_MAGIC value

[Kumar Gala]

On Jul 21, 2011, at 4:50 PM, Kumar Gala wrote:

> Had a typo in the ifdef for 85xx, should be CONFIG_MPC85xx for it to get
> triggered.  Was pull in the non-BookE magic number.
> 
> Reported-by: John Cortell
> Signed-off-by: Kumar Gala 
> ---
> arch/powerpc/lib/bootm.c |2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)

applied to 85xx

- k
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 3/3] powerpc/85xx: Handle the lack of L2 cache on P2040/P2040E

[Kumar Gala]

On Jul 21, 2011, at 12:20 AM, Kumar Gala wrote:

> The P2040/P2040E have no L2 cache.  So we utilize the SVR to determine
> if we are one of these devices and skip the L2 init code in cpu_init.c
> and release.  For the device tree we skip the updating of the L2 cache
> properties but we still update the chain of caches so the CPC/L3 node
> can be properly updated.
> 
> Signed-off-by: Kumar Gala 
> ---
> arch/powerpc/cpu/mpc85xx/cpu_init.c |7 +++
> arch/powerpc/cpu/mpc85xx/fdt.c  |   23 +++
> arch/powerpc/cpu/mpc85xx/release.S  |   15 ++-
> 3 files changed, 36 insertions(+), 9 deletions(-)

applied to 85xx

- k
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/3] powerpc/85xx: Add support for P2041[e] XAUI in SERDES

[Kumar Gala]

On Jul 21, 2011, at 12:20 AM, Kumar Gala wrote:

> We add XAUI_FM1 into the SERDES tables for P2041[e] devices.  However
> for the P2040[e] devices that dont support XAUI we handle this at
> runtime via SVR checks.  If we are on a P2040[e] device the SERDES
> functions will behave as follows:
> 
> is_serdes_prtcl_valid() will always report invalid if prtcl passed in is
> XAUI_FM1.
> 
> serdes_get_prtcl() will report NONE if the prtcl in the table is set to
> XAUI_FM1.
> 
> Signed-off-by: Kumar Gala 
> ---
> arch/powerpc/cpu/mpc85xx/p2041_serdes.c |   26 +-
> 1 files changed, 21 insertions(+), 5 deletions(-)

applied to 85xx

- k
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 1/3] powerpc/85xx: Rename P2040 id & SERDES to P2041

[Kumar Gala]

On Jul 21, 2011, at 12:20 AM, Kumar Gala wrote:

> P2041 is the superset part that covers both P2040 & P2041.  The only
> difference between the two devices is that P2041 supports 10g/XAUI and
> has an L2 cache.
> 
> Signed-off-by: Kumar Gala 
> ---
> arch/powerpc/cpu/mpc85xx/Makefile   |6 +-
> arch/powerpc/cpu/mpc85xx/p2040_ids.c|  109 ---
> arch/powerpc/cpu/mpc85xx/p2040_serdes.c |   90 -
> arch/powerpc/cpu/mpc85xx/p2041_ids.c|  109 +++
> arch/powerpc/cpu/mpc85xx/p2041_serdes.c |   90 +
> 5 files changed, 202 insertions(+), 202 deletions(-)
> delete mode 100644 arch/powerpc/cpu/mpc85xx/p2040_ids.c
> delete mode 100644 arch/powerpc/cpu/mpc85xx/p2040_serdes.c
> create mode 100644 arch/powerpc/cpu/mpc85xx/p2041_ids.c
> create mode 100644 arch/powerpc/cpu/mpc85xx/p2041_serdes.c

applied to 85xx

- k
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/2] powerpc/85xx: Fix up clock_freq property in CAN node of dts

[Kumar Gala]

On Mar 6, 2011, at 10:17 PM, Kumar Gala wrote:

> From: Bhaskar Upadhaya 
> 
> Fix up the device tree property associated with the Flexcan clock
> frequency. This property is used to calculate the bit timing parameters
> for Flexcan.
> 
> Signed-off-by: Bhaskar Upadhaya 
> Signed-off-by: Kumar Gala 
> ---
> arch/powerpc/cpu/mpc85xx/fdt.c |3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)

applied to 85xx

- k
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] powerpc/p2041rdb: Enable SATA support

[Mingkai Hu]

Signed-off-by: Mingkai Hu 
---
 include/configs/P2041RDB.h |   20 
 1 files changed, 20 insertions(+), 0 deletions(-)

diff --git a/include/configs/P2041RDB.h b/include/configs/P2041RDB.h
index 63c8123..918f1b9 100644
--- a/include/configs/P2041RDB.h
+++ b/include/configs/P2041RDB.h
@@ -457,6 +457,26 @@
 #define CONFIG_DOS_PARTITION
 #endif /* CONFIG_PCI */
 
+/* SATA */
+#define CONFIG_FSL_SATA_V2
+#ifdef CONFIG_FSL_SATA_V2
+#define CONFIG_LIBATA
+#define CONFIG_FSL_SATA
+
+#define CONFIG_SYS_SATA_MAX_DEVICE 2
+#define CONFIG_SATA1
+#define CONFIG_SYS_SATA1   CONFIG_SYS_MPC85xx_SATA1_ADDR
+#define CONFIG_SYS_SATA1_FLAGS FLAGS_DMA
+#define CONFIG_SATA2
+#define CONFIG_SYS_SATA2   CONFIG_SYS_MPC85xx_SATA2_ADDR
+#define CONFIG_SYS_SATA2_FLAGS FLAGS_DMA
+
+#define CONFIG_LBA48
+#define CONFIG_CMD_SATA
+#define CONFIG_DOS_PARTITION
+#define CONFIG_CMD_EXT2
+#endif
+
 #ifdef CONFIG_FMAN_ENET
 #define CONFIG_SYS_FM1_DTSEC1_PHY_ADDR 0x2
 #define CONFIG_SYS_FM1_DTSEC2_PHY_ADDR 0x3
-- 
1.7.5.1


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH v2] powerpc/85xx: Add basic support for P1010RDB

[Kumar Gala]

From: Poonam Aggrwal 

Boot methods supported: NOR Flash, SPI Flash and SDCARD
This patch adds the following basic interfaces:
DDR3, eTSEC, DUART, I2C, SD/MMC, USB, SATA, PCIe, NOR Flash, SPI Flash.

P1010RDB Overview
-
1Gbyte DDR3 (on board DDR)
Local Bus (IFC):
32Mbyte 16bit NOR flash
32Mbyte SLC NAND Flash
64KB CPLD device(GPCM interface)
SPI Flash:
128 Mbit SPI Flash memory
SD/MMC:
connector to interface with the SD memory card
SATA:
1 internal SATA connect to 2.5. 160G SATA2 HDD
1 eSATA connector to rear panel
USB 2.0:
x1 USB 2.0 port: connected via a UTMI PHY to Mini-AB interface.
x1 USB 2.0 port: directly connected to Mini-AB interface Ethernet
eTSEC:
eTSEC1: Connected to RGMII PHY VSC8641XKO
eTSEC2: Connected to SGMII PHY VSC8221
eTSEC3: Connected to SGMII PHY VSC8221
eCAN:
Two DB-9 female connectors for Field bus interface
UART:
supports two UARTs up to 115200 bps for console
TDM:
2 FXS ports connected via an external SLIC to the TDM interface.
SLIC:
SPI SLIC
I2C:
Serial EEprom
Real time clock
256 Kbit M24256 I2C EEPROM
PCIe:
PCIe and mPCIe connectors.

Signed-off-by: Poonam Aggrwal 
Signed-off-by: Dipen Dudhat 
Signed-off-by: Prabhakar Kushwaha 
Signed-off-by: Ramneek Mehresh 
Signed-off-by: Bhaskar Upadhaya 
Signed-off-by: York Sun 
Signed-off-by: Kumar Gala 
---
v2:
* drop NAND support from this patch, will be follow on
* various format cleanups, etc

 board/freescale/p1010rdb/Makefile   |   52 +++
 board/freescale/p1010rdb/ddr.c  |  250 
 board/freescale/p1010rdb/law.c  |   35 ++
 board/freescale/p1010rdb/p1010rdb.c |  330 
 board/freescale/p1010rdb/tlb.c  |   98 +
 boards.cfg  |6 +
 include/configs/P1010RDB.h  |  714 +++
 7 files changed, 1485 insertions(+), 0 deletions(-)
 create mode 100644 board/freescale/p1010rdb/Makefile
 create mode 100644 board/freescale/p1010rdb/ddr.c
 create mode 100644 board/freescale/p1010rdb/law.c
 create mode 100644 board/freescale/p1010rdb/p1010rdb.c
 create mode 100644 board/freescale/p1010rdb/tlb.c
 create mode 100644 include/configs/P1010RDB.h

diff --git a/board/freescale/p1010rdb/Makefile 
b/board/freescale/p1010rdb/Makefile
new file mode 100644
index 000..1dcd490
--- /dev/null
+++ b/board/freescale/p1010rdb/Makefile
@@ -0,0 +1,52 @@
+#
+# Copyright 2010-2011 Freescale Semiconductor, Inc.
+#
+# 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 $(TOPDIR)/config.mk
+
+LIB= $(obj)lib$(BOARD).o
+
+COBJS-y+= $(BOARD).o
+COBJS-y+= ddr.o
+COBJS-y+= law.o
+COBJS-y+= tlb.o
+
+SRCS   := $(SOBJS:.o=.S) $(COBJS-y:.o=.c)
+OBJS   := $(addprefix $(obj),$(COBJS-y))
+SOBJS  := $(addprefix $(obj),$(SOBJS))
+
+$(LIB):$(obj).depend $(OBJS) $(SOBJS)
+   $(call cmd_link_o_target, $(OBJS))
+
+clean:
+   rm -f $(OBJS) $(SOBJS)
+
+distclean: clean
+   rm -f $(LIB) core *.bak .depend
+
+#
+
+# defines $(obj).depend target
+include $(SRCTREE)/rules.mk
+
+sinclude $(obj).depend
+
+#
diff --git a/board/freescale/p1010rdb/ddr.c b/board/freescale/p1010rdb/ddr.c
new file mode 100644
index 000..e5d8423
--- /dev/null
+++ b/board/freescale/p1010rdb/ddr.c
@@ -0,0 +1,250 @@
+/*
+ * Copyright 2010-2011 Freescale Semiconductor, Inc.
+ *
+ * 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 Gene

Re: [U-Boot] [PATCH] 85xx: enable FDT support for STX SSA board

[Kumar Gala]

On Jul 26, 2011, at 6:49 AM, Sergei Shtylyov wrote:

> Hello.
> 
> On 25-07-2011 17:15, Wolfgang Denk wrote:
> 
>> We also have to shift TEXT_BASE to accomodate for the additional
>> code size.
> 
>> Signed-off-by: Wolfgang Denk
>> Cc: Kumar Gala
> 
>> ---
>>  board/stx/stxssa/stxssa.c |8 
>>  include/configs/stxssa.h  |7 ++-
>>  2 files changed, 14 insertions(+), 1 deletions(-)
> 
>> diff --git a/board/stx/stxssa/stxssa.c b/board/stx/stxssa/stxssa.c
>> index 83ffcd2..c78538d 100644
>> --- a/board/stx/stxssa/stxssa.c
>> +++ b/board/stx/stxssa/stxssa.c
> [...]
>> @@ -247,6 +248,13 @@ reset_phy(void)
>>  #endif
>>  }
>> 
>> +#ifdef CONFIG_OF_BOARD_SETUP
>> +void ft_board_setup (void *blob, bd_t *bd)
>> +{
>> +ft_cpu_setup (blob, bd);
> 
>   I guess this won't pass checkpatch.pl (spaces before paren).
> 
> WBR, Sergei

applied to 85xx & fixed whitespace issue.

- k
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] mx53: use CONFIG_SYS_L2CACHE_OFF in config file

[Jason Liu]

Hi, Wolfgang,

2011/7/26 Wolfgang Denk :
> Dear Jason Liu,
>
> In message <1311668750-5471-1-git-send-email-jason@linaro.org> you wrote:
>> CONFIG_L2_OFF is obsolete after the following commit:
>>
>> e47f2db5371047eb9bcd115fee084e6a8a92a239
>> armv7: rename cache related CONFIG flags
>> Replace the cache related CONFIG flags with more meaningful
>> names. Following are the changes:
>> CONFIG_L2_OFF      -> CONFIG_SYS_L2CACHE_OFF
>>
>> But the above commit does not clean up all the mx53 board config file.
>> This patch does it by using CONFIG_SYS_L2CACHE_OFF instead of _L2_OFF
>>
>> Signed-off-by: Jason Liu 
>> Cc:Stefano Babic 
>
> why are you posting a second version of this patch just a few minutes
> after the first one, and witrhout any indication ifd there are any
> differences between the two versions, and what these differences might
> be?
>
> Please STOP doing this!

It's my fault to send out the second email when I thought the previous
email was not show up in mail-list.
will take care to avoid it later. Sorry for the noise.

Jason

>
>
> 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: w...@denx.de
> The faster I go, the behinder I get.                 -- Lewis Carroll
> ___
> U-Boot mailing list
> U-Boot@lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] Transfer Details

[WESTERN UNION OFFICE]

Dear Friend,

The first $5,000.00 was sent today. My working partner has helped me to send
the first $5,000.00 to you through western union. So contact our Western Union
claims Agent to pick up this $5,000 now: Contact person: Mr. Don C john, TEL:
+2347031309835. E-mail: (wumt_trans...@w.cn) Ask him to give you the
MTCN, Sender Name to pick the $5,000.00. I told him to keep sending you
$5,000.00 daily until the payment of $1.500,000.00 is completed. Again forward
him your Full Name, Telephone number and address so that he will be sure.

Thanks.

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] AT91: rework and fixes

[Hong Xu]

Hi Reinhard,

On 07/26/2011 08:59 PM, Reinhard Meyer wrote:
> Dear Hong Xu,
>  > Hi Albert,
>  >
>  > On 07/18/2011 02:39 PM, Albert ARIBAUD wrote:
>  > > Hi Eric,
>  > >
>  > > Le 18/07/2011 04:52, Hong Xu a écrit :
>  > > > Hi Reinhard,
>  > > >
>  > > > It's a pity to see that some of the AT91 boards are planed to be
> removed
>  > > > by Wolfgang.
>  > > >
>  > > > Several weeks ago, the patches for SAM9261/9G10 have got Acks
> from you.
>  > > > See http://www.mail-archive.com/u-boot@lists.denx.de/msg54829.html
>  > > > But I don't know why it's not merged upstream.
>
> Now, an actual at91sam9261_devices.c and at91sam9261.h are on top of
> u-boot-atmel/master.
> If your board patches for az91sam9261-ek still work with that, I can
> apply them again to master.

OK, I'll check and resend the board fix.

>
>  > > >
>  > > > For SAM9RL, the V3 patches have been sent long time ago and was
> pinged.
>  > > > But no response any longer.
>
> I have changes requested there.

You mean the change requests for V3 patches? I never saw them. Is it a 
general comments towards AT91 or specific to V3 patches?

But anyway, I'd like to send the patches again in the coming days.

>
>  > > >
>  > > > The similar stories are also applied for SAM9263 and SAM9M10G45.
>
> Changes requested as well.

Ditto.

[...]
>
> Anyway, for work reasons, there will be no extended vacation this
> summer :(

Sorry to hear this.

> Again, the work flow, as I want it followed, is as such:
>
> 1. convert/fix all SoC specific files for 9263 9g45 etc. to be in line
> and equivalent to the now existing files for 9260 and 9261.
> (Next should be for 9263, we should be looking at existing patches to
> fix that SoC).
> I will NOT accept patches that do not fully implement the name schemes
> laid out in 9260 and 9261.
>
> 2. Fix any boards to use those files AND to use relocation properly.
>
> 3. Make sure those build without warning or errors.
>
> 4. Make sure those boards still "boot" with that u-boot.
>
> 5. When a board has passed those tests (I cannot test point 4) the
> "removal" patch for that board will not be applied.

OK, the SoC fixes for 9263 and 9G45 will come firstly.

Thanks.

BR,
Eric

> Best Regards,
> Reinhard
>

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] Upgrade Your Mailbox

[Andrea . Spencer]

Your mailbox has exceeded the limit of 20 GB, which is as set by your manager, 
you are currently at 20.9GB, you will not be able to create new e-mail to send 
or receive again until you validate your mailbox.To re-validate your 
mailbox,you can Click 
Here



Thanks
System Administrator
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] fw_printenv help needed

[Glen VanMilligan]

I spent all day, but I think the answer to my question is that the kernel
needs to be constructed with the CONFIG_MTD_CMDLINE_PARTS set to '"Y".  Now
I can see two MTD partition in the /proc/mtd file.

I'm still having a problem with fw_printenv.  My output is:
# fw_printenv
MTD_open
MTD_ioctl
MTD_ioctl
CannoMTD_close
t read bad block mark: Invalid argument
#
Comments?
--Glen

On Tue, Jul 26, 2011 at 11:05 AM, Glen VanMilligan  wrote:

> I have an application that needs to access the environment setting of
> uboot.  I see that fw_printenv is available to do the job.  The problem for
> me is how do I set uboot and mtd for using this device?
>
> When I run the fw_printenv, I get the message "Can't open /dev/mtd0: No
> such device or address"
>
> This implies that an MTD device is setup.  I have the MTD turned on in the
> kernel, because in /dev I see four mtd0-mtd3 devices, four
> mtdblock0-mtdblock3 devices.  However, if I cat the /proc/mtd file, I get a
> blank listing.
>  # cat /proc/mtd
> dev:size   erasesize  name
> #
> I do have a fw_env.config file in the /etc directory that looks like this:
>
> # Configuration file for fw_(printenv/saveenv) utility.
> # Up to two entries are valid, ic this case the redundant
> #   environment sector is assumed present.
>
> #MTD Device name, Device offset, Env. size, Flash sector size
> /dev/mtd0 0x60x40x2
>
> So the question is, how do I mount and use this to make this device
> available?
>
> --Glen
>
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 1/3] lib: add uuid_str_to_bin for use with bootp and PXE uuid

[Jason Hobbs]

On Mon, Jul 25, 2011 at 11:37:34PM +0200, Wolfgang Denk wrote:
> > +void uuid_str_to_bin(const char *uuid, unsigned char *out)

I will add a separate function to verify the format of a UUID string.
It doesn't belong in this function, which otherwise only needs to read
the string once.  It will check for length, that '-' is placed
appropriately, and that proper hexadecimal numbers are used.

Thanks,
Jason
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 1/7] Add generic, reusable menu code

[Jason Hobbs]

Dear Wolfgang,

On Mon, Jul 25, 2011 at 11:18:15PM +0200, Wolfgang Denk wrote:
> I am happy that you provide documentation in doc/README.menu, but I
> really dislike that the code itself is basicly uncommented. It is a
> major pain to have to switch between the README and the source files
> when trying to understand the code.
> 
> Please add sufficient comments to the code to make it readable.

I don't want to duplicate the interface documentation in the README and
the source file.  Maybe I'll add shorter summary descriptions to the
README and move the detailed interface doc into the source file.  I'll
also add some comments for describing the behavior of the internals.

Thanks,
Jason
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 6/7] Add pxecfg command

[Jason Hobbs]

Dear Wolfgang,

On Mon, Jul 25, 2011 at 11:15:36PM +0200, Wolfgang Denk wrote:
> > Add pxecfg command, which is intended to mimic PXELINUX
> > functionality.
> 
> After thinking a long while about this, I came to the conclusion that
> my initial feeling that "pxecfg" is ugly was actually correct.  Please
> let's call this simply "pxe" as I already suggested in my initial
> feedback.

Ok. I'll also combine the dhcp PXE request options patches with this
series. Though they can be used independently, it will make this
publish/review cycle less work and make it easier for those who are
picking up the patches off the list.

> Another general comment:  you are adding tons of completely
> undocumented, uncommented code here.
> 
> This is not acceptable.  Please provide sufficient comments so that
> the average engineer has a chance to understand what the code is
> (supposed to be) doing without spending needless effords.

I'll add comments.
 
> > +static char *from_env(char *envvar)
> > +{
> > +   char *ret;
> > +
> > +   ret = getenv(envvar);
> > +
> > +   if (!ret)
> > +   printf("missing environment variable: %s\n", envvar);
> > +
> > +   return ret;
> > +}
> 
> I don't like this. It just blows up the code and shifts error handling
> from the place where it happens.  In the result, you will have to
> check return codes on several function call levels.  I recommend to
> drop this function.

I added this in response to your suggestion to make the print 'missing
environment variable' code common. From the caller's perspective,
from_env as with getenv, so I don't understand your concern about
handling return codes in several function call levels. Do you have
another suggestion that doesn't involve going back to scattering printf's
throughout the code?
 
> > +/*
> > + * Returns the ethaddr environment variable formated with -'s instead of 
> > :'s
> > + */
> > +static void format_mac_pxecfg(char **outbuf)
> 
> void?
> 
> > +   char *p, *ethaddr;
> > +
> > +   *outbuf = NULL;
> 
> This is redundant, please omit.
> 
> > +   ethaddr = from_env("ethaddr");
> > +
> > +   if (!ethaddr)
> > +   return;
> 
> It makes little sense to check for errors, to report errors, and then
> to continue as if nothing happened.
> 
> > +   *outbuf = strdup(ethaddr);
> 
> Can we please al;locate the buffer in the caller, and do without this?
> This is only good for memory leaks.

I'll change this to allocate in the caller, as you suggest.  We didn't
continue as if nothing happened, though. format_mac_pxecfg's caller can
checks the value of *outbuf for NULL and doesn't use it if it's NULL.
Anyhow, that will change as a result of the allocate in caller change.

> 
> > +/*
> > + * Returns the directory the file specified in the bootfile env variable is
> > + * in.
> > + */
> > +static char *get_bootfile_path(void)
> > +{
> > +   char *bootfile, *bootfile_path, *last_slash;
> > +   size_t path_len;
> > +
> > +   bootfile = from_env("bootfile");
> > +
> > +   if (!bootfile)
> > +   return NULL;
> > +
> > +   last_slash = strrchr(bootfile, '/');
> > +
> > +   if (last_slash == NULL)
> > +   return NULL;
> 
> This looks unnecessarily stringent to me.  Why can we not accept a
> plain file name [we can always use "./" if we need a path for the
> directory] ?

Yes, that's he behavior, as you've suggested.  I'll add comments to make
this clearer.  This function generates a prefix if it's required, and
NULL if it isn't or if bootfile isn't defined. The NULL prefix results
in the plain filename being used.  It's awkward to use a NULL, I thought
about using a zero length string, but that was awkward too.  I'll see if
I can improve this when I go after eliminating the dynamic allocation.

> 
> > +   if (path_len > MAX_TFTP_PATH_LEN) {
> > +   printf("Base path too long (%s%s)\n",
> > +   bootfile_path ? bootfile_path : "",
> > +   file_path);
> 
> Indentation is one level only.  Please fix globally.

Moving these printf args substantially to the right follows kernel
CodingStyle guidelines and is more readable than a single level of
indentation.  Is this a deviation from the kernel CodingStyle that
should go into the U-boot coding style wiki?

> 
> > +   if (bootfile_path)
> > +   free(bootfile_path);
> > +
> > +   return -ENAMETOOLONG;
> 
> All this dynamically allocated strings just blow up the code.  Can we
> try to do without?

I'll look for a way to get rid of these.

> > +   if (do_tftpb(NULL, 0, 3, tftp_argv)) {
> > +   printf("File not found\n");
> > +   return -ENOENT;
> 
> Does TFTP not already print an error message?

It does.  I'll drop this one.

> > +static int get_pxecfg_file(char *file_path, void *file_addr)
> > +{
> > +   unsigned long config_file_size;
> > +   int err;
> > +
> > +   err = get_relfile(file_path, file_addr);
> > +
> > +   if (err < 0)
> > +   return err;
> > +
> > 

[U-Boot] RFC [PATCH 4/5 v5] dreamplug: initial board support.

[Jason Cooper]

Copied wholeheartedly from board/Marvell/guruplug and modified to add support
for SPI NOR flash.

Also, the following features are used:
- Use the Marvell Integrated RTC.
- display Marvell CPU/L2/etc speeds if requested.

Note: this still uses MACH_TYPE_GURUPLUG until MACH_TYPE_DREAMPLUG is accepted
into mainline Linux.  The source from Globalscale did the same.

Signed-off-by: Jason Cooper 
---
 MAINTAINERS  |4 +
 MAKEALL  |1 +
 board/Marvell/dreamplug/Makefile |   54 +++
 board/Marvell/dreamplug/dreamplug.c  |  157 
 board/Marvell/dreamplug/dreamplug.h  |   42 +
 board/Marvell/dreamplug/kwbimage.cfg |  163 ++
 boards.cfg   |1 +
 include/configs/dreamplug.h  |  138 
 8 files changed, 560 insertions(+), 0 deletions(-)
 create mode 100644 board/Marvell/dreamplug/Makefile
 create mode 100644 board/Marvell/dreamplug/dreamplug.c
 create mode 100644 board/Marvell/dreamplug/dreamplug.h
 create mode 100644 board/Marvell/dreamplug/kwbimage.cfg
 create mode 100644 include/configs/dreamplug.h

diff --git a/MAINTAINERS b/MAINTAINERS
index 2bba7b4..9affbea 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -70,6 +70,10 @@ Conn Clark 
 
ESTEEM192E  MPC8xx
 
+Jason Cooper 
+
+   dreamplug   ARM926EJS (Kirkwood SoC)
+
 Joe D'Abbraccio 
 
MPC837xERDB MPC837x
diff --git a/MAKEALL b/MAKEALL
index 51db13e..55a122f 100755
--- a/MAKEALL
+++ b/MAKEALL
@@ -335,6 +335,7 @@ LIST_ARM9=" \
cp966   \
da830evm\
da850evm\
+   dreamplug   \
edb9301 \
edb9302 \
edb9302a\
diff --git a/board/Marvell/dreamplug/Makefile b/board/Marvell/dreamplug/Makefile
new file mode 100644
index 000..9ee5406
--- /dev/null
+++ b/board/Marvell/dreamplug/Makefile
@@ -0,0 +1,54 @@
+#
+# (C) Copyright 2011
+# Jason Cooper 
+#
+# Based on work by:
+# Marvell Semiconductor 
+# Written-by: Siddarth Gore 
+#
+# 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., 51 Franklin Street, Fifth Floor, Boston,
+# MA 02110-1301 USA
+#
+
+include $(TOPDIR)/config.mk
+
+LIB= $(obj)lib$(BOARD).o
+
+COBJS  := dreamplug.o
+
+SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)
+OBJS   := $(addprefix $(obj),$(COBJS))
+SOBJS  := $(addprefix $(obj),$(SOBJS))
+
+$(LIB):$(obj).depend $(OBJS) $(SOBJS)
+   $(call cmd_link_o_target, $(OBJS) $(SOBJS))
+
+clean:
+   rm -f $(SOBJS) $(OBJS)
+
+distclean: clean
+   rm -f $(LIB) core *.bak .depend
+
+#
+
+# defines $(obj).depend target
+include $(SRCTREE)/rules.mk
+
+sinclude $(obj).depend
+
+#
diff --git a/board/Marvell/dreamplug/dreamplug.c 
b/board/Marvell/dreamplug/dreamplug.c
new file mode 100644
index 000..cdc094a
--- /dev/null
+++ b/board/Marvell/dreamplug/dreamplug.c
@@ -0,0 +1,157 @@
+/*
+ * (C) Copyright 2011
+ * Jason Cooper 
+ *
+ * Based on work by:
+ * Marvell Semiconductor 
+ * Written-by: Siddarth Gore 
+ *
+ * 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., 51 Franklin Street, Fifth Floor, Boston,
+ * MA 02110-1301 USA
+ */
+
+#include 
+#include 
+#include 
+#include 
+#include "dreamplug.h"
+
+DECLARE_GLOBAL_DATA_PTR;
+
+int board_early_init_f(void)
+{
+   /*
+* default gpio con

[U-Boot] RFC [PATCH 2/5] arm/kirkwood: print speeds with cpu info.

[Jason Cooper]

Signed-off-by: Jason Cooper 
---
 arch/arm/cpu/arm926ejs/kirkwood/cpu.c|   46 ++
 arch/arm/include/asm/arch-kirkwood/cpu.h |1 +
 2 files changed, 47 insertions(+), 0 deletions(-)

diff --git a/arch/arm/cpu/arm926ejs/kirkwood/cpu.c 
b/arch/arm/cpu/arm926ejs/kirkwood/cpu.c
index b4a4c04..a69f9f2 100644
--- a/arch/arm/cpu/arm926ejs/kirkwood/cpu.c
+++ b/arch/arm/cpu/arm926ejs/kirkwood/cpu.c
@@ -270,11 +270,26 @@ static void kw_sysrst_check(void)
 }
 
 #if defined(CONFIG_DISPLAY_CPUINFO)
+#define MSAR_CPUCLCK_EXTRACT(X)(((X & 0x2) >> 1) | ((X & 0x40) >> 
21) | \
+   ((X & 0x18) >> 1))
+#define MSAR_L2CLCK_EXTRACT(X)  (((X & 0x600) >> 9) | ((X & 0x8) >> 17))
+#define MSAR_DDRCLCK_RTIO_MASK  (0xf << 5)
+
+#define MSAR_TCLCK_OFFS21
+#define MSAR_TCLCK_MASK(0x1 << MSAR_TCLCK_OFFS)
+#define MV_BOARD_TCLK_166MHZ   16667
+#define MV_BOARD_TCLK_200MHZ   2
+#define MSAR_TCLCK_167 (0x1 << MSAR_TCLCK_OFFS)
+#define MSAR_TCLCK_200 (0x0 << MSAR_TCLCK_OFFS)
+
 int print_cpuinfo(void)
 {
char *rev;
u16 devid = (readl(KW_REG_PCIE_DEVID) >> 16) & 0x;
u8 revid = readl(KW_REG_PCIE_REVID) & 0xff;
+   u32 cpu_clk, t_clk, tmp;
+   u32 sys_clk, l2_clk;
+   u32 l2_ratio, ddr_ratio;
 
if ((readl(KW_REG_DEVICE_ID) & 0x03) > 2) {
printf("Error.. %s:Unsupported Kirkwood SoC 88F%04x\n", 
__FUNCTION__, devid);
@@ -297,6 +312,37 @@ int print_cpuinfo(void)
}
 
printf("SoC:   Kirkwood 88F%04x_%s\n", devid, rev);
+
+   tmp = readl(MPP_SAMPLE_AT_RESET);
+   cpu_clk = MSAR_CPUCLCK_EXTRACT(tmp);
+   if (cpu_clk == 0x9)
+   cpu_clk = 1200;
+
+   l2_ratio = MSAR_L2CLCK_EXTRACT(tmp);
+   l2_clk = cpu_clk / l2_ratio;
+
+   ddr_ratio = tmp & MSAR_DDRCLCK_RTIO_MASK;
+   ddr_ratio = ddr_ratio >> 5;
+   if (ddr_ratio == 4)
+   sys_clk = 400;
+
+   switch (tmp & MSAR_TCLCK_MASK) {
+   case MSAR_TCLCK_167:
+   t_clk = MV_BOARD_TCLK_166MHZ;
+   break;
+   case MSAR_TCLCK_200:
+   t_clk = MV_BOARD_TCLK_200MHZ;
+   break;
+   default:
+   t_clk = MV_BOARD_TCLK_200MHZ;
+   break;
+   }
+
+   printf("CPU running @ %dMHz L2 running @ %dMHz\n",
+   cpu_clk, l2_clk);
+   printf("SysClock = %dMHz, TClock = %dMHz\n",
+   sys_clk, t_clk / 100);
+
return 0;
 }
 #endif /* CONFIG_DISPLAY_CPUINFO */
diff --git a/arch/arm/include/asm/arch-kirkwood/cpu.h 
b/arch/arm/include/asm/arch-kirkwood/cpu.h
index d28c51a..28ddd25 100644
--- a/arch/arm/include/asm/arch-kirkwood/cpu.h
+++ b/arch/arm/include/asm/arch-kirkwood/cpu.h
@@ -41,6 +41,7 @@
 #define KW_REG_SYSRST_CNT  (KW_MPP_BASE + 0x50)
 #define SYSRST_CNT_1SEC_VAL(25*100)
 #define KW_REG_MPP_OUT_DRV_REG (KW_MPP_BASE + 0xE0)
+#define MPP_SAMPLE_AT_RESET(KW_MPP_BASE + 0x30)
 
 enum memory_bank {
BANK0,
-- 
1.7.0.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] RFC [PATCH 0/5 v5] add dreamplug support

[Jason Cooper]

All,

This is a complete redo from v4.  I received a copy of the u-boot source
code GlobalScale used in the Dreamplug.  This series is a mix of my code
and theirs.

Patches 1 and 2 add features to boards using the kirkwood arch.  I think.
I have no way to test it on guruplug/sheevaplug/etc.  They work on
dreamplug, though.

Patch 3 increases the timeout for slow-to-respond EHCI chips, like that
found in the dreamplug.  It may help other places.  If it's non-harmful,
it can also be applied separately.

Patch 4 adds the dreamplug support.  It uses the spi flash, boots from it,
writes the environment to it, etc.  It also enables and uses patches 1 and
2.

Patch 5 is for once the dreamplug machid and board support are added to the
mailine Linux kernel.

Jason Cooper (5):
  drivers/rtc: add Marvell Integrated RTC.
  arm/kirkwood: print speeds with cpu info.
  usb: Some EHCI chipsets are slow to respond.
  dreamplug: initial board support.
  dreamplug: use MACH_TYPE_DREAMPLUG

 MAINTAINERS  |4 +
 MAKEALL  |1 +
 arch/arm/cpu/arm926ejs/kirkwood/cpu.c|   46 +
 arch/arm/include/asm/arch-kirkwood/cpu.h |1 +
 board/Marvell/dreamplug/Makefile |   54 ++
 board/Marvell/dreamplug/dreamplug.c  |  154 
 board/Marvell/dreamplug/dreamplug.h  |   42 
 board/Marvell/dreamplug/kwbimage.cfg |  163 ++
 boards.cfg   |1 +
 drivers/rtc/Makefile |1 +
 drivers/rtc/mvinteg_rtc.c|  151 +++
 drivers/rtc/mvinteg_rtc.h|   89 
 include/configs/dreamplug.h  |  138 +
 include/usb.h|2 +-
 14 files changed, 846 insertions(+), 1 deletions(-)
 create mode 100644 board/Marvell/dreamplug/Makefile
 create mode 100644 board/Marvell/dreamplug/dreamplug.c
 create mode 100644 board/Marvell/dreamplug/dreamplug.h
 create mode 100644 board/Marvell/dreamplug/kwbimage.cfg
 create mode 100644 drivers/rtc/mvinteg_rtc.c
 create mode 100644 drivers/rtc/mvinteg_rtc.h
 create mode 100644 include/configs/dreamplug.h

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] RFC [PATCH 5/5 v5] dreamplug: use MACH_TYPE_DREAMPLUG

[Jason Cooper]

Signed-off-by: Jason Cooper 
---
 board/Marvell/dreamplug/dreamplug.c |7 ++-
 include/configs/dreamplug.h |2 +-
 2 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/board/Marvell/dreamplug/dreamplug.c 
b/board/Marvell/dreamplug/dreamplug.c
index cdc094a..f848db7 100644
--- a/board/Marvell/dreamplug/dreamplug.c
+++ b/board/Marvell/dreamplug/dreamplug.c
@@ -104,11 +104,8 @@ int board_early_init_f(void)
 
 int board_init(void)
 {
-   /*
-* arch number of board
-* XXX: change to MACH_TYPE_DREAMPLUG once in Linux mainline.
-*/
-   gd->bd->bi_arch_number = MACH_TYPE_GURUPLUG;
+   /* arch number of board */
+   gd->bd->bi_arch_number = MACH_TYPE_DREAMPLUG;
 
/* adress of boot parameters */
gd->bd->bi_boot_params = kw_sdram_bar(0) + 0x100;
diff --git a/include/configs/dreamplug.h b/include/configs/dreamplug.h
index 2a908f4..4be2e59 100644
--- a/include/configs/dreamplug.h
+++ b/include/configs/dreamplug.h
@@ -39,7 +39,7 @@
 #define CONFIG_SHEEVA_88SV131  1   /* CPU Core subversion */
 #define CONFIG_KIRKWOOD1   /* SOC Family Name */
 #define CONFIG_KW88F6281   1   /* SOC Name */
-#define CONFIG_MACH_GURUPLUG   /* Machine type */
+#define CONFIG_MACH_DREAMPLUG  /* Machine type */
 #define CONFIG_SKIP_LOWLEVEL_INIT  /* disable board lowlevel_init */
 
 /*
-- 
1.7.0.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] RFC [PATCH 1/5] drivers/rtc: add Marvell Integrated RTC.

[Jason Cooper]

Signed-off-by: Jason Cooper 
---
 drivers/rtc/Makefile  |1 +
 drivers/rtc/mvinteg_rtc.c |  151 +
 drivers/rtc/mvinteg_rtc.h |   89 ++
 3 files changed, 241 insertions(+), 0 deletions(-)
 create mode 100644 drivers/rtc/mvinteg_rtc.c
 create mode 100644 drivers/rtc/mvinteg_rtc.h

diff --git a/drivers/rtc/Makefile b/drivers/rtc/Makefile
index e4be4a4..ed63e9c 100644
--- a/drivers/rtc/Makefile
+++ b/drivers/rtc/Makefile
@@ -55,6 +55,7 @@ COBJS-$(CONFIG_MCFRTC) += mcfrtc.o
 COBJS-$(CONFIG_RTC_MK48T59) += mk48t59.o
 COBJS-$(CONFIG_RTC_MPC5200) += mpc5xxx.o
 COBJS-$(CONFIG_RTC_MPC8xx) += mpc8xx.o
+COBJS-$(CONFIG_RTC_MVINTEG) += mvinteg_rtc.o
 COBJS-$(CONFIG_RTC_PCF8563) += pcf8563.o
 COBJS-$(CONFIG_RTC_PL031) += pl031.o
 COBJS-$(CONFIG_RTC_PT7C4338) += pt7c4338.o
diff --git a/drivers/rtc/mvinteg_rtc.c b/drivers/rtc/mvinteg_rtc.c
new file mode 100644
index 000..295e4f7
--- /dev/null
+++ b/drivers/rtc/mvinteg_rtc.c
@@ -0,0 +1,151 @@
+/*
+ * Copyright (C) 2011
+ * Jason Cooper 
+ *
+ * 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
+ */
+
+/*
+ * Date & Time support for Marvell Integrated RTC
+ */
+
+#include 
+#include 
+#include 
+#include "mvinteg_rtc.h"
+
+/* This RTC does not support century, so we assume 20 */
+#define CENTURY 20
+
+int rtc_get(struct rtc_time *t)
+{
+   u32 time;
+   u32 date;
+   u8  tens;
+   u8  single;
+
+   /* read the time register */
+   time = MV_REG_READ(MVINTEG_RTCTIME_REG);
+
+   /* read the date register */
+   date = MV_REG_READ(MVINTEG_RTCDATE_REG);
+
+   /* seconds */
+   tens   = ((time & MVINTEG_10SEC_MSK) >> MVINTEG_10SEC_SFT);
+   single = ((time & MVINTEG_SEC_MSK)   >> MVINTEG_SEC_SFT);
+   t->tm_sec = 10 * tens + single;
+
+   /* minutes */
+   tens   = ((time & MVINTEG_10MIN_MSK) >> MVINTEG_10MIN_SFT);
+   single = ((time & MVINTEG_MIN_MSK)   >> MVINTEG_MIN_SFT);
+   t->tm_min = 10 * tens + single;
+
+   /* hours */
+   tens   = ((time & MVINTEG_10HOUR_MSK) >> MVINTEG_10HOUR_SFT);
+   single = ((time & MVINTEG_HOUR_MSK)   >> MVINTEG_HOUR_SFT);
+   t->tm_hour = 10 * tens + single;
+
+   /* day */
+   t->tm_wday = ((time & MVINTEG_DAY_MSK) >> MVINTEG_DAY_SFT);
+   t->tm_wday--;
+
+   /* date */
+   tens   = ((date & MVINTEG_10DATE_MSK) >> MVINTEG_10DATE_SFT);
+   single = ((date & MVINTEG_DATE_MSK)   >> MVINTEG_DATE_SFT);
+   t->tm_mday = 10 * tens + single;
+
+   /* month */
+   tens   = ((date & MVINTEG_10MON_MSK) >> MVINTEG_10MON_SFT);
+   single = ((date & MVINTEG_MON_MSK)   >> MVINTEG_MON_SFT);
+   t->tm_mon = 10 * tens + single;
+
+   /* year */
+   tens   = ((date & MVINTEG_10YEAR_MSK) >> MVINTEG_10YEAR_SFT);
+   single = ((date & MVINTEG_YEAR_MSK)   >> MVINTEG_YEAR_SFT);
+   t->tm_year = (CENTURY * 100) + (10 * tens) + single;
+
+   /* not supported in this RTC */
+   t->tm_yday  = 0;
+   t->tm_isdst = 0;
+
+   return 0;
+}
+
+int rtc_set(struct rtc_time *t)
+{
+   u32 time = 0;
+   u32 date = 0;
+   u32 tens;
+   u32 single;
+
+   /* seconds */
+   tens   = t->tm_sec / 10;
+   single = t->tm_sec % 10;
+   time |= ((tens   << MVINTEG_10SEC_SFT) & MVINTEG_10SEC_MSK) |
+   ((single << MVINTEG_SEC_SFT)   & MVINTEG_SEC_MSK);
+
+   /* minutes */
+   tens   = t->tm_min / 10;
+   single = t->tm_min % 10;
+   time |= ((tens   << MVINTEG_10MIN_SFT) & MVINTEG_10MIN_MSK) |
+   ((single << MVINTEG_MIN_SFT)   & MVINTEG_MIN_MSK);
+
+   /* hours (24) */
+   tens   = t->tm_hour / 10;
+   single = t->tm_hour % 10;
+   time |= ((tens   << MVINTEG_10HOUR_SFT) & MVINTEG_10HOUR_MSK) |
+   ((single << MVINTEG_HOUR_SFT)   & MVINTEG_HOUR_MSK);
+
+   /* day */
+   single = t->tm_wday + 1;
+   time |= ((single << MVINTEG_DAY_SFT) & MVINTEG_DAY_MSK);
+
+   /* date */
+   tens   = t->tm_mday / 10;
+   single = t->tm_mday % 10;
+   date |= ((tens   << MVINTEG_10DATE_SFT) & MVINTEG_10DATE_MSK) |
+   ((single << MVINTEG_DATE_SFT)   & MVINTEG_DATE_MSK);
+
+   /* month */
+   

[U-Boot] RFC [PATCH 3/5] usb: Some EHCI chipsets are slow to respond.

[Jason Cooper]

This fixes 'EHCI timed out on TD...' on the dreamplug board.

Signed-off-by: Jason Cooper 
---
 include/usb.h |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/include/usb.h b/include/usb.h
index 53603a5..168e2b2 100644
--- a/include/usb.h
+++ b/include/usb.h
@@ -46,7 +46,7 @@
  * This is the timeout to allow for submitting an urb in ms. We allow more
  * time for a BULK device to react - some are slow.
  */
-#define USB_TIMEOUT_MS(pipe) (usb_pipebulk(pipe) ? 5000 : 100)
+#define USB_TIMEOUT_MS(pipe) (usb_pipebulk(pipe) ? 5000 : 500)
 
 /* device request (setup) */
 struct devrequest {
-- 
1.7.0.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] part_dos: fix crash with big sector size

[Sergei Shtylyov]

Hello.

I wrote:

> Apple iPod nanos have sector sizes of 2 or 4 KiB, which crashes U-Boot when it
> tries to read the MBR into 512-byte buffer situated on stack. Instead use the
> variable length arrays to be safe with any large sector size.

> Signed-off-by: Sergei Shtylyov 

> ---
> The same change is probably needed for disk/part_amiga.c but I'm not really
> sure if Amiga supports USB... :-)

Damn, the FAT code is also hardwired for 512-byte sectors. I wonder which 
century we are in? :-)

WBR, Sergei
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH RFC] drivers/rtc: add Marvell Integrated RTC.

[Jason]

On Sun, Jul 24, 2011 at 03:13:38PM -0400, Jason wrote:
> To write a u-boot.kwb test image to flash, here is what I do:
> 
> openocd$ sheevaplug_init
> openocd$ load_image u-boot # known good from Marvell
> openocd$ load_image u-boot.kwb 0x0080  # monster I created
> openocd$ resume 0x0060 # run the known good
> 
> then, in u-boot
> 
> u-boot$ sf probe 0
> u-boot$ sf erase 0x0 0x10
> u-boot$ sf write 0x0080 0x0 0x10
> 

Okay, here's neat problem.  In the globalscale u-boot code, they set the
environment address to 0x3 in the 2MB spi nor flash.  So, they have:

#ifdef CONFIG_SPI_FLASH
#define CONFIG_ENV_IS_IN_SPI_FLASH 1
#define CONFIG_ENV_SECT_SIZE   0x1 /* 64K */
#else
#define CONFIG_ENV_IS_NOWHERE  1   /* if env in SDRAM */
#endif

and...

#define CONFIG_ENV_SIZE0x1000  /* 4k */
#define CONFIG_ENV_ADDR0x3
#define CONFIG_ENV_OFFSET  0x3 /* env starts here */


If that's an absolute address, and I wrote my u-boot to 0x0 in the
flash, did 'saveenv' to remove the CRC message, then that's why it died
after reset, right?  I must have overwrote part of u-boot with the env.

In my meager understanding, I thought maybe u-boot structured itself so
the env was physically stored 0x3 into the code and it would
magically work.  Guess not.  :-P

So, what is a standard flash layout?  Do I put u-boot at 0x0 and just
throw the env out at 0x10 (1MB), or, put the env at 0x0 and u-boot
in the second erase block (0x1)?  Doesn't u-boot have to be at 0x0
so the cpu can find it?

Thanks for any advice.

btw - I have everything else tested and working.  RTC, spi flash, cpu
speed, etc.  This issue is the last holdout before I resubmit v5.

thx,

Jason.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 5/7] common: add run_command2 for running simple or hush commands

[Jason Hobbs]

On Mon, Jul 25, 2011 at 10:42:18PM +0200, Wolfgang Denk wrote:
> Dear "Jason Hobbs",
> > -# ifndef CONFIG_SYS_HUSH_PARSER -  run_command (p, 0); -#
> > else -  parse_string_outer(p, FLAG_PARSE_SEMICOLON | -
> > FLAG_EXIT_FROM_LOOP); -# endif +run_command2(p, 0);
> 
> Indentation seems wrong here - it should be one TAB more to the right?

Grr, yes. I will change and resubmit.

> 
> > +int run_command2(const char *cmd, int flag)
> > +{
> > +#ifndef CONFIG_SYS_HUSH_PARSER
> > +   if (run_command(cmd, flag) == -1)
> > +   return 1;
> > +#else
> > +   if (parse_string_outer(cmd,
> > +   FLAG_PARSE_SEMICOLON | FLAG_EXIT_FROM_LOOP) != 0)
> > +   return 1;
> > +#endif
> > +   return 0;
> > +}
> 
> Can we make this inline [in the normal (non-menu) case], please?

Yes, I will change and resubmit.

Thanks,
Jason
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH V4 3/5] nand spl: add NAND Library to new SPL

[Scott Wood]

On Tue, 26 Jul 2011 13:04:01 -0500
Scott Wood  wrote:

> On Tue, 26 Jul 2011 14:09:16 +0200
> Simon Schwarz  wrote:
> 
> > diff --git a/drivers/mtd/nand/Makefile b/drivers/mtd/nand/Makefile
> > index 8b598f6..cdc9a14 100644
> > --- a/drivers/mtd/nand/Makefile
> > +++ b/drivers/mtd/nand/Makefile
> > @@ -26,12 +26,16 @@ include $(TOPDIR)/config.mk
> >  LIB:= $(obj)libnand.o
> >  
> >  ifdef CONFIG_CMD_NAND
> > +ifdef CONFIG_SPL_BUILD
> > +COBJS-y += nand_spl.o
> > +else
> >  COBJS-y += nand.o
> >  COBJS-y += nand_base.o
> >  COBJS-y += nand_bbt.o
> > -COBJS-y += nand_ecc.o
> >  COBJS-y += nand_ids.o
> >  COBJS-y += nand_util.o
> > +endif
> > +COBJS-y += nand_ecc.o
> 
> You're assuming all NAND SPLs will want nand_ecc -- this will not fit in
> most current ones.

Well, nand_correct_data is used by some of the SPLs, but the ifdef
CONFIG_NAND_SPL would have to be updated so the rest doesn't get included.

And some SPLs (FSL drivers) don't use anything from that file.

-Scott

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH V4 2/5] omap-common: add nand spl support

[Scott Wood]

On Tue, 26 Jul 2011 14:09:15 +0200
Simon Schwarz  wrote:

> +#ifdef CONFIG_SPL_NAND_SUPPORT
> +static void nand_load_image(void)
> +{
> + gpmc_init();
> + nand_init();
> + nand_copy_image(CONFIG_SYS_NAND_U_BOOT_OFFS,
> + CONFIG_SYS_NAND_U_BOOT_SIZE,
> + (uchar *)CONFIG_SYS_NAND_U_BOOT_DST);
> +#ifdef CONFIG_NAND_ENV_DST
> + nand_copy_image(CONFIG_ENV_OFFSET, CONFIG_ENV_SIZE,
> + (uchar *)CONFIG_NAND_ENV_DST);
> +#ifdef CONFIG_ENV_OFFSET_REDUND
> + nand_copy_image(CONFIG_ENV_OFFSET_REDUND, CONFIG_ENV_SIZE,
> + (uchar *)CONFIG_NAND_ENV_DST + CONFIG_ENV_SIZE);
> +#endif
> +#endif
> + parse_image_header((struct image_header *)CONFIG_SYS_NAND_U_BOOT_DST);
> +}
> +#endif /* CONFIG_SPL_NAND_SUPPORT */

I'm not sure that "load" versus "copy" conveys the difference between this
function and the low-level nand_copy_image.

Where is nand_copy_image() defined?

> diff --git a/include/nand.h b/include/nand.h
> index 8d94b5c..e0f20f6 100644
> --- a/include/nand.h
> +++ b/include/nand.h
> @@ -132,6 +132,8 @@ int nand_lock( nand_info_t *meminfo, int tight );
>  int nand_unlock( nand_info_t *meminfo, ulong start, ulong length );
>  int nand_get_lock_status(nand_info_t *meminfo, loff_t offset);
>  
> +void nand_copy_image(unsigned int offs, unsigned int size, uchar *dst);

Use loff_t for offset.  Maybe call it "nand_spl_copy_image" or
"nand_spl_load_image".

-Scott

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH V4 3/5] nand spl: add NAND Library to new SPL

[Scott Wood]

On Tue, 26 Jul 2011 14:09:16 +0200
Simon Schwarz  wrote:

> diff --git a/drivers/mtd/nand/Makefile b/drivers/mtd/nand/Makefile
> index 8b598f6..cdc9a14 100644
> --- a/drivers/mtd/nand/Makefile
> +++ b/drivers/mtd/nand/Makefile
> @@ -26,12 +26,16 @@ include $(TOPDIR)/config.mk
>  LIB  := $(obj)libnand.o
>  
>  ifdef CONFIG_CMD_NAND
> +ifdef CONFIG_SPL_BUILD
> +COBJS-y += nand_spl.o
> +else
>  COBJS-y += nand.o
>  COBJS-y += nand_base.o
>  COBJS-y += nand_bbt.o
> -COBJS-y += nand_ecc.o
>  COBJS-y += nand_ids.o
>  COBJS-y += nand_util.o
> +endif
> +COBJS-y += nand_ecc.o

You're assuming all NAND SPLs will want nand_ecc -- this will not fit in
most current ones.

Where is nand_spl.c, and what is in it that is appropriate for all targets?

-Scott

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] post, memory test: add memory_post_test() to include file

[Heiko Schocher]

Hello Wolfgang,

Wolfgang Denk wrote:
> Dear Heiko,
> 
> In message <20110726082651.f2ce3138e...@gemini.denx.de> I wrote:
>> In message <1306909447-19603-3-git-send-email...@denx.de> you wrote:
>>> This include is needed, if this memory test is used "outside"
>>> from post code, for example booting with nand_spl, and using
>>> this memory test before copying u-boot code to RAM and jumping
>>> to it.
>>>
>>> Signed-off-by: Heiko Schocher 
>>> ---
>>>  include/post.h |1 +
>>>  1 files changed, 1 insertions(+), 0 deletions(-)
>> This got applied as commit f18714d, but obviously this patch has not
>> been tested before as it is breaking a ton of boards:
> ...
>> Please fix!!
> 
> To keep the number of non-compiling commits small and being able to
> apply more patches now I decided not to wait for your fix, but to
> revert the broken commit.
> 
> Please resubmit the (fixed) patch.

Argh, did a "MAKEALL arm" only ... Sorry for the inconvinience!
I look at this ASAP.

bye,
Heiko
-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] fw_printenv help needed

[Glen VanMilligan]

I have an application that needs to access the environment setting of
uboot.  I see that fw_printenv is available to do the job.  The problem for
me is how do I set uboot and mtd for using this device?

When I run the fw_printenv, I get the message "Can't open /dev/mtd0: No such
device or address"

This implies that an MTD device is setup.  I have the MTD turned on in the
kernel, because in /dev I see four mtd0-mtd3 devices, four
mtdblock0-mtdblock3 devices.  However, if I cat the /proc/mtd file, I get a
blank listing.
 # cat /proc/mtd
dev:size   erasesize  name
#
I do have a fw_env.config file in the /etc directory that looks like this:

# Configuration file for fw_(printenv/saveenv) utility.
# Up to two entries are valid, ic this case the redundant
#   environment sector is assumed present.

#MTD Device name, Device offset, Env. size, Flash sector size
/dev/mtd0 0x60x40x2

So the question is, how do I mount and use this to make this device
available?

--Glen
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [RFC PATCH] POST: add test before execution of post_run to avoid unneeded run

[Wolfgang Denk]

Dear Valentin Longchamp,

In message <1311602833-29320-1-git-send-email-valentin.longch...@keymile.com> 
you wrote:
> Some boards have the environment variables defined in a slow EEPROM. post_run
> accesses these environment variables to define which tests have to be run (in
> post_get_flags). This is very slow before the code relocation on some boards
> with a slow I2C EEPROM for environement variables.
> 
> This patch adds a check at the beginning of post_run to avoid to run
> post_get_flags (which may be slow on some boards for the above reason) for 
> some
> given POST flags if no tests are defined with these flags (for instance, no 
> need
> to run POST tests with POST_ROM if no tests for POST_ROM are defined).

I think I understand what you are tying to do, but I'm sorry, I cannot
understand your code.

> diff --git a/post/post.c b/post/post.c
> index 1b7f2aa..d97f330 100644
> --- a/post/post.c
> +++ b/post/post.c
> @@ -169,6 +169,18 @@ static void post_bootmode_test_off (void)
>   post_word_store (word);
>  }
>  
> +static int post_test_defined(int flags)
> +{
> + int i;
> + int run_flags = flags & ~(POST_RAM | POST_ROM);

Why are you masking out the POST_RAM and POST_ROM bits here?

Assuming your intention is to shortcut post_run(), then this should
be done always, independent of the mode.

> + for (i = 0; i < post_list_size; i++)
> + if (post_list[i].flags & run_flags)
> + return 1;

Please try if omitting run_flags and testing 

if (post_list[i].flags & flags)

instead still has the intended effect.  If so, then please resubmit a
cleaned up patch.

If not, then please explain in detail how your patch is supposed to
work.

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: w...@denx.de
Women are more easily and more deeply terrified ...  generating  more
sheer horror than the male of the species.
-- Spock, "Wolf in the Fold", stardate 3615.4
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH, AT91 rework] pm9261: compiles with the AT91 reworked scheme

[Asen Chavdarov Dimov]

Signed-off-by: Asen Chavdarov Dimov 
---
Note:
This patch was a part of patches, but it is better to be done step by step and
so this patch is a split from the patch series with message id:
<1307606409-29818-3-git-send-email-di...@ronetix.at>
and Patchwork:
http://patchwork.ozlabs.org/patch/99666/

Changes for v2:
- rename at91_serial_hw_init() to at91_seriald_hw_init()

checkpatch complains at "board/ronetix/pm9261/pm9261.c:148", but
checkpatch could be mistaken.

 board/ronetix/pm9261/led.c|9 -
 board/ronetix/pm9261/pm9261.c |   38 ++
 boards.cfg|2 +-
 include/configs/pm9261.h  |   18 ++
 4 files changed, 33 insertions(+), 34 deletions(-)

diff --git a/board/ronetix/pm9261/led.c b/board/ronetix/pm9261/led.c
index ff21ce6..f659870 100644
--- a/board/ronetix/pm9261/led.c
+++ b/board/ronetix/pm9261/led.c
@@ -24,17 +24,16 @@
  */
 
 #include 
-#include 
 #include 
-#include 
-#include 
+#include 
+#include 
 
 void coloured_LED_init(void)
 {
-   at91_pmc_t  *pmc= (at91_pmc_t *) AT91_PMC_BASE;
+   struct at91_pmc *pmc = (struct at91_pmc *)ATMEL_BASE_PMC;
 
/* Enable clock */
-   writel(1 << AT91SAM9261_ID_PIOC, &pmc->pcer);
+   writel(1 << ATMEL_ID_PIOC, &pmc->pcer);
 
at91_set_pio_output(CONFIG_RED_LED, 1);
at91_set_pio_output(CONFIG_GREEN_LED, 1);
diff --git a/board/ronetix/pm9261/pm9261.c b/board/ronetix/pm9261/pm9261.c
index 2c50fe8..871b94a 100644
--- a/board/ronetix/pm9261/pm9261.c
+++ b/board/ronetix/pm9261/pm9261.c
@@ -26,17 +26,15 @@
 
 #include 
 #include 
-#include 
+#include 
 #include 
 #include 
 #include 
 #include 
 #include 
-#include 
 #include 
-#include 
-#include 
-#include 
+#include 
+
 #include 
 #include 
 #include 
@@ -56,9 +54,9 @@ DECLARE_GLOBAL_DATA_PTR;
 static void pm9261_nand_hw_init(void)
 {
unsigned long csa;
-   at91_smc_t  *smc= (at91_smc_t *) AT91_SMC_BASE;
-   at91_matrix_t   *matrix = (at91_matrix_t *) AT91_MATRIX_BASE;
-   at91_pmc_t  *pmc= (at91_pmc_t *) AT91_PMC_BASE;
+   struct at91_smc *smc = (struct at91_smc *)ATMEL_BASE_SMC;
+   struct at91_matrix *matrix = (struct at91_matrix *)ATMEL_BASE_MATRIX;
+   struct at91_pmc *pmc = (struct at91_pmc *)ATMEL_BASE_PMC;
 
/* Enable CS3 */
csa = readl(&matrix->csa) | AT91_MATRIX_CSA_EBI_CS3A;
@@ -86,8 +84,8 @@ static void pm9261_nand_hw_init(void)
AT91_SMC_MODE_TDF_CYCLE(2),
&smc->cs[3].mode);
 
-   writel(1 << AT91SAM9261_ID_PIOA |
-   1 << AT91SAM9261_ID_PIOC,
+   writel(1 << ATMEL_ID_PIOA |
+   1 << ATMEL_ID_PIOC,
&pmc->pcer);
 
/* Configure RDY/BSY */
@@ -105,8 +103,8 @@ static void pm9261_nand_hw_init(void)
 #ifdef CONFIG_DRIVER_DM9000
 static void pm9261_dm9000_hw_init(void)
 {
-   at91_smc_t  *smc= (at91_smc_t *) AT91_SMC_BASE;
-   at91_pmc_t  *pmc= (at91_pmc_t *) AT91_PMC_BASE;
+   struct at91_smc *smc = (struct at91_smc *)ATMEL_BASE_SMC;
+   struct at91_pmc *pmc = (struct at91_pmc *)ATMEL_BASE_PMC;
 
/* Configure SMC CS2 for DM9000 */
writel(AT91_SMC_SETUP_NWE(2) | AT91_SMC_SETUP_NCS_WR(0) |
@@ -127,7 +125,7 @@ static void pm9261_dm9000_hw_init(void)
&smc->cs[2].mode);
 
/* Configure Interrupt pin as input, no pull-up */
-   writel(1 << AT91SAM9261_ID_PIOA, &pmc->pcer);
+   writel(1 << ATMEL_ID_PIOA, &pmc->pcer);
at91_set_pio_input(AT91_PIO_PORTA, 24, 0);
 }
 #endif
@@ -147,7 +145,7 @@ vidinfo_t panel_info = {
vl_vsync_len:   1,
vl_upper_margin:1,
vl_lower_margin:0,
-   mmio:   AT91SAM9261_LCDC_BASE,
+   mmio:   ATMEL_BASE_LCDC,
 };
 
 void lcd_enable(void)
@@ -162,7 +160,7 @@ void lcd_disable(void)
 
 static void pm9261_lcd_hw_init(void)
 {
-   at91_pmc_t  *pmc= (at91_pmc_t *) AT91_PMC_BASE;
+   struct at91_pmc *pmc = (struct at91_pmc *)ATMEL_BASE_PMC;
 
at91_set_a_periph(AT91_PIO_PORTB, 1, 0);/* LCDHSYNC */
at91_set_a_periph(AT91_PIO_PORTB, 2, 0);/* LCDDOTCK */
@@ -189,7 +187,7 @@ static void pm9261_lcd_hw_init(void)
 
writel(1 << 17, &pmc->scer); /* LCD controller Clock, AT91SAM9261 only 
*/
 
-   gd->fb_base = AT91SAM9261_SRAM_BASE;
+   gd->fb_base = ATMEL_BASE_SRAM;
 }
 
 #ifdef CONFIG_LCD_INFO
@@ -241,13 +239,13 @@ void lcd_show_board_info(void)
 
 int board_init(void)
 {
-   at91_pmc_t  *pmc= (at91_pmc_t *) AT91_PMC_BASE;
+   struct at91_pmc *pmc = (struct at91_pmc *)ATMEL_BASE_PMC;
 
/* Enable Ctrlc */
console_init_f();
 
-   writel(1 << AT91SAM9261_ID_PIOA |
-   1 << AT91SAM9261_ID_PIOC,
+   writel(1 << ATMEL_ID_PIOA |
+   1 << ATMEL_ID_PIOC,
&pmc->pcer);
 
/* arch number of PM9261-Board */
@@ -256,7 +254,7 @@ int board_init(vo

Re: [U-Boot] [PATCH v2 2/2] add command fitupd to run an update from a FIT image

[Wolfgang Denk]

Dear Andreas Pretzsch,

In message <1310831460-14450-3-git-send-email-...@cn-eng.de> you wrote:
> Command calls update_tftp() analogous to automatic update described
> in doc/README.update.
> 
> Usage:
> fitupd [addr]
> - run update from FIT image at addr
>   or from tftp 'updatefile'
> 
> Signed-off-by: Andreas Pretzsch 
> ---
> Changes for v2:
>- void update_tftp() => int update_tftp(): return success/fail
> ---
>  common/Makefile |1 +
>  common/cmd_fitupd.c |   36 
>  doc/README.update   |5 +
>  3 files changed, 42 insertions(+), 0 deletions(-)
>  create mode 100644 common/cmd_fitupd.c

Applied, 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: w...@denx.de
Harrisberger's Fourth Law of the Lab:
Experience is directly proportional to the
amount of equipment ruined.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2 1/2] automatic update from FIT image: add optional address parameter

[Wolfgang Denk]

Dear Andreas Pretzsch,

In message <1310831460-14450-2-git-send-email-...@cn-eng.de> you wrote:
> Current update_tftp() flow:
>   1.) fetch "updatefile" from defined TFTP server
>   2.) check if FIT format
>   3.) flash contained images
> 
> Add an address parameter to update_tftp(). If this address is non-zero,
> skip the TFTP transfer and use the image at this address.
> Also extend update_tftp() to return success/fail.
> 
> Signed-off-by: Andreas Pretzsch 
> ---
> Changes for v2:
>- void update_tftp() => int update_tftp(): return success/fail
> ---
>  common/main.c   |4 ++--
>  common/update.c |   20 ++--
>  2 files changed, 16 insertions(+), 8 deletions(-)

Applied, 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: w...@denx.de
In an infinite universe all things are possible, including the possi-
bility that the universe does not exist.
- Terry Pratchett, _The Dark Side of the Sun_
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] post: fix indendation/brace confusion

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1310667306-24948-1-git-send-email-vap...@gentoo.org> you wrote:
> From: James Kosin 
> 
> The post.c code is missing braces around the pass case, and as a
> result, the diagnostic function will post both fail and pass for
> a failed test.  The reason for this bug is probably the incorrect
> indentation used, so when reading the code it seems like there
> are proper braces.
> 
> Indent the code to the correct depth and put proper braces around
> the "else" branch of the "if" statement.
> 
> Signed-off-by: James Kosin 
> Signed-off-by: Mike Frysinger 
> ---
>  post/post.c |   24 
>  1 files changed, 12 insertions(+), 12 deletions(-)

Applied, 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: w...@denx.de
The speed of time is one second per second.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] fix unconfigured out-of-tree building of tools

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1309488980-16074-1-git-send-email-vap...@gentoo.org> you wrote:
> Now that the tools target requires the generated version header file, we
> need to make sure that the directory it writes to exists.  In a configured
> tree, this is taken care of for us.  But in an unconfigured one, the dir
> does not yet exist causing a build error like so:
> 
> /bin/sh: line 5: /u-boot_build/include/version_autogenerated.h.tmp: No 
> such file or directory
> 
> So create the dir for this file before we attempt to generate it.
> 
> Signed-off-by: Mike Frysinger 
> ---
>  Makefile |1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)

Applied, 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: w...@denx.de
Always leave room to add an explanation if it doesn't work out.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] panic: add noreturn attribute

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1309384684-26415-1-git-send-email-vap...@gentoo.org> you wrote:
> Since panic() never returns, we should add an appropriate attribute to
> let gcc improve optimization around it.
> 
> Signed-off-by: Mike Frysinger 
> ---
>  include/common.h |2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

Applied, 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: w...@denx.de
Single tasking: Just Say No.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] simplify clobber behavior with out-of-tree builds

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1308976675-28145-1-git-send-email-vap...@gentoo.org> you wrote:
> The targets/prerequisites are the same here; the rules only differ in
> the recipes.  So move the if logic protection to the recipe part so we
> can keep the rest the same.
> 
> Signed-off-by: Mike Frysinger 
> ---
>  Makefile |5 +
>  1 files changed, 1 insertions(+), 4 deletions(-)

Applied, 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: w...@denx.de
There are certain things men must do to remain men.
-- Kirk, "The Ultimate Computer", stardate 4929.4
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2] post: new nor flash test

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1305070540-29687-1-git-send-email-vap...@gentoo.org> you wrote:
> This adds a simple flash test to automatically verify erasing,
> writing, and reading of sectors.  The code is based on existing
> Blackfin tests but generalized for everyone to use.
> 
> Signed-off-by: Mike Frysinger 
> ---
> v2
>   - convert to malloc()
>   - rewrite changelog a bit
> 
>  include/post.h|1 +
>  post/drivers/Makefile |2 +-
>  post/drivers/flash.c  |  107 
> +
>  post/tests.c  |   15 ++-
>  4 files changed, 123 insertions(+), 2 deletions(-)
>  create mode 100644 post/drivers/flash.c

Applied, 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: w...@denx.de
The only way to learn a new programming language is by  writing  pro-
grams in it.- Brian Kernighan
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] post: use ARRAY_SIZE

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1305048515-15299-1-git-send-email-vap...@gentoo.org> you wrote:
> We've got a handy dandy macro already for calculating the number of
> elements in an array, so use it.
> 
> Signed-off-by: Mike Frysinger 
> ---
>  post/board/lwmon/sysmon.c |2 +-
>  post/cpu/mpc8xx/ether.c   |4 +---
>  post/cpu/mpc8xx/spr.c |3 +--
>  post/cpu/mpc8xx/uart.c|4 +---
>  post/cpu/ppc4xx/spr.c |3 +--
>  post/drivers/memory.c |2 +-
>  post/lib_powerpc/andi.c   |3 +--
>  post/lib_powerpc/cmp.c|3 +--
>  post/lib_powerpc/cmpi.c   |3 +--
>  post/lib_powerpc/cr.c |   12 
>  post/lib_powerpc/fpu/mul-subnormal-single-1.c |2 +-
>  post/lib_powerpc/load.c   |3 +--
>  post/lib_powerpc/multi.c  |2 +-
>  post/lib_powerpc/rlwimi.c |3 +--
>  post/lib_powerpc/rlwinm.c |3 +--
>  post/lib_powerpc/rlwnm.c  |3 +--
>  post/lib_powerpc/srawi.c  |3 +--
>  post/lib_powerpc/store.c  |3 +--
>  post/lib_powerpc/three.c  |3 +--
>  post/lib_powerpc/threei.c |3 +--
>  post/lib_powerpc/threex.c |3 +--
>  post/lib_powerpc/two.c|3 +--
>  post/lib_powerpc/twox.c   |3 +--
>  post/post.c   |2 +-
>  post/tests.c  |2 +-
>  25 files changed, 28 insertions(+), 52 deletions(-)

Applied, 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: w...@denx.de
Brain: an apparatus with which we think we think.- Ambrose Bierce
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] post: add gpio hotkey support

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1305046881-5818-1-git-send-email-vap...@gentoo.org> you wrote:
> Now that we have the generic GPIO layer, we can easily provide a common
> implementation for the post_hotkeys_pressed() function based on it.
> 
> Signed-off-by: Mike Frysinger 
> ---
>  post/post.c |   21 +
>  1 files changed, 21 insertions(+), 0 deletions(-)

Applied, 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: w...@denx.de
Knowledge, sir, should be free to all!
-- Harry Mudd, "I, Mudd", stardate 4513.3
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] post: fix up I/O helper usage

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1304996478-1536-1-git-send-email-vap...@gentoo.org> you wrote:
> The I/O API from Linux defaults to little endian accesses.  In order to
> do big endian accesses, there are a "be" variants.  The "le32" variants
> are arch-specific and not terribly common, so change it to the normal
> Linux API funcs.
> 
> Signed-off-by: Mike Frysinger 
> ---
>  include/post.h |4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/post.h b/include/post.h
> index 519cef1..c9ec2f4 100644
> --- a/include/post.h
> +++ b/include/post.h
> @@ -78,12 +78,12 @@
>  
>  static inline ulong post_word_load (void)
>  {
> - return in_le32((volatile void *)(_POST_WORD_ADDR));
> + return inl((volatile void *)(_POST_WORD_ADDR));
>  }

Is this supposed to fix any real problem, or just a change according
to your personal preferences?

My initial impression is that we lose a bit of documentation
(explicitely telling the reader that these are LE accesses has a
certain benefit over hoping the reader has this information present).

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: w...@denx.de
Heavier than air flying machines are impossible.
-- Lord Kelvin, President, Royal Society, c. 1895
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] serial: implement common uart post test

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1305392175-31018-1-git-send-email-vap...@gentoo.org> you wrote:
> The current arch/driver specific UART posts basically boil down to setting
> the UART to loop back mode, then reading and writing data.  If we ignore
> the loop back part, the rest can be built upon the existing common serial
> API.  So let's do just that.
> 
> First add a call back for serial drivers to implement loop back control.
> Then write a post test that walks all of the serial drivers, puts them
> into loop back mode, and verifies that reading/writing at all the diff
> baud rates is OK.
> 
> If a serial driver doesn't support loop back mode (either it can't or
> it hasn't done so yet), then skip it.  This should allow for people to
> easily migrate to the new post test with existing serial drivers.
> 
> I haven't touched the few already existing uart post tests as I don't
> the hardware or knowledge of converting them over.  So I've marked the
> new test as weak which will allow the existing tests to override the
> default until they are converted.
> 
> Signed-off-by: Mike Frysinger 
> ---
>  common/serial.c  |   90 
> ++
>  include/serial.h |5 +++
>  2 files changed, 95 insertions(+), 0 deletions(-)

Applied, 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: w...@denx.de
"The majority of the stupid is  invincible  and  guaranteed  for  all
time.  The  terror  of their tyranny, however, is alleviated by their
lack of consistency."   - Albert Einstein
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 3/3] serial: drop useless ctlr field

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1304136211-1316-3-git-send-email-vap...@gentoo.org> you wrote:
> The multi serial support has a "ctlr" field which almost no one uses,
> but everyone is forced to set to useless strings.  So punt it.
> 
> Funny enough, the only code that actually reads this field (the mpc8xx
> driver) has a typo where it meant to look for the SCC driver.  Fix it
> while converting the check to use the name field.
> 
> Signed-off-by: Mike Frysinger 
> CC: Heiko Schocher 
> CC: Anatolij Gustschin 
> CC: Tom Rix 
> CC: Minkyu Kang 
> CC: Craig Nauman 
> CC: Marek Vasut 
> CC: Prafulla Wadaskar 
> CC: Mahavir Jain 
> ---
> note: while i've compiled tested all of these, i obviously can't
>   runtime test them ... i have runtime tested the common code
>   though on a Blackfin part ...
> 
>  arch/powerpc/cpu/mpc512x/serial.c  |   11 +--
>  arch/powerpc/cpu/mpc5xxx/serial.c  |2 --
>  arch/powerpc/cpu/mpc8xx/serial.c   |8 +++-
>  board/logicpd/zoom2/zoom2_serial.h |2 --
>  drivers/serial/serial.c|   11 +--
>  drivers/serial/serial_pxa.c|3 ---
>  drivers/serial/serial_s3c24x0.c|9 -
>  drivers/serial/serial_s5p.c|   11 +--
>  include/serial.h   |2 --
>  9 files changed, 22 insertions(+), 37 deletions(-)

Applied, 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: w...@denx.de
1 1 was a race-horse, 2 2 was 1 2. When 1 1 1 1 race, 2 2 1 1 2.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/3] serial: drop serial_register return value

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1304136211-1316-2-git-send-email-vap...@gentoo.org> you wrote:
> The serial_register function never fails (always return 0), so change it
> to a void function to avoid wasting overhead on it.
> 
> Signed-off-by: Mike Frysinger 
> ---
>  common/serial.c |4 +---
>  1 files changed, 1 insertions(+), 3 deletions(-)

Applied, 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: w...@denx.de
A dog always bit deepest on the veterinary hand.
- Terry Pratchett, _Wyrd Sisters_
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 1/3] serial: push default_serial_console to drivers

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1304136211-1316-1-git-send-email-vap...@gentoo.org> you wrote:
> Rather than sticking arch/board/driver specific logic in the common
> serial code, push it all out to the respective drivers.  The serial
> drivers declare these funcs weak so that boards can still override
> things with their own definition.
> 
> Signed-off-by: Mike Frysinger 
> CC: Heiko Schocher 
> CC: Anatolij Gustschin 
> CC: Tom Rix 
> CC: Minkyu Kang 
> CC: Craig Nauman 
> CC: Prafulla Wadaskar 
> CC: Mahavir Jain 
> ---
> note: while i've compiled tested all of these, i obviously can't
>   runtime test them ... i have runtime tested the common code
>   though on a Blackfin part ...
> 
>  arch/powerpc/cpu/mpc512x/serial.c  |   12 ++
>  arch/powerpc/cpu/mpc5xxx/serial.c  |6 +++
>  arch/powerpc/cpu/mpc8xx/serial.c   |   10 +
>  board/logicpd/zoom2/zoom2_serial.c |5 +++
>  common/serial.c|   68 
> 
>  drivers/serial/serial.c|   17 +
>  drivers/serial/serial_s3c24x0.c|   14 +++
>  drivers/serial/serial_s5p.c|   16 
>  8 files changed, 80 insertions(+), 68 deletions(-)

Applied, 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: w...@denx.de
Lead me not into temptation... I can find it myself.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] disable security warning flags when possible

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1303754800-14317-1-git-send-email-vap...@gentoo.org> you wrote:
> Some toolchains enable security warning flags by default, but these don't
> really make sense in the u-boot world.  Such as forcing changes like:
>   -printf(foo);
>   +printf("%s", foo);
> 
> So disable the flags when the compiler supports them.  Linux has already
> merged a similar change in their build system.
> 
> Signed-off-by: Mike Frysinger 
> ---
>  config.mk |4 
>  1 files changed, 4 insertions(+), 0 deletions(-)

Applied, 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: w...@denx.de
The following statement is not true.  The previous statement is true.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] cmd_usage: constify

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1303638204-3262-1-git-send-email-vap...@gentoo.org> you wrote:
> The usage helper doesn't modify the command, so constify its input arg.
> 
> Signed-off-by: Mike Frysinger 
> ---
>  common/command.c  |2 +-
>  include/command.h |2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)

Applied, 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: w...@denx.de
In the beginning there was nothing.
And the Lord said "Let There Be Light!"
And still there was nothing, but at least now you could see it.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] constify default env

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1303638201-3221-1-git-send-email-vap...@gentoo.org> you wrote:
> I can't see any obvious needs for the default environment to be writable,
> so make it const.
> 
> Signed-off-by: Mike Frysinger 
> ---
>  common/env_common.c   |4 ++--
>  common/env_flash.c|2 +-
>  include/environment.h |2 +-
>  3 files changed, 4 insertions(+), 4 deletions(-)

Applied, 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: w...@denx.de
Uncontrolled power will turn even saints into savages. And we can all
be counted on to live down to our lowest impulses.
-- Parmen, "Plato's Stepchildren", stardate 5784.3
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2] env: allow people to force envcrc building

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1310157865-28441-1-git-send-email-vap...@gentoo.org> you wrote:
> For people who want to manually extract the embedded environment so that
> it can be manually packed into the final u-boot image, add a config opt
> to force building of the envcrc tool.
> 
> Signed-off-by: Mike Frysinger 
> ---
> v2
>   - updated to master after tools/ changes
> 
>  README|5 +
>  common/env_embedded.c |2 +-
>  include/common.h  |5 +
>  tools/envcrc.c|   10 +++---
>  4 files changed, 18 insertions(+), 4 deletions(-)

Applied, 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: w...@denx.de
"Where shall I begin, please your Majesty?" he asked. "Begin  at  the
beginning,"  the  King said, gravely, "and go on till you come to the
end: then stop."- Alice's Adventures in Wonderland, Lewis Carroll
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 6/4] sf: sst: support newer standardized flashes

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1303750794-8043-2-git-send-email-vap...@gentoo.org> you wrote:
> Newer SST flashes have dropped the Auto Address Increment (AAI) word
> programming (WP) modes in favor of the standard page programming mode
> that most flashes now support.  So add a flags field to the different
> flashes to support both modes with new and old styles.
> 
> Signed-off-by: Mike Frysinger 
> ---
>  drivers/mtd/spi/sst.c |   22 --
>  1 files changed, 20 insertions(+), 2 deletions(-)

Applied, 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: w...@denx.de
You are an excellent tactician, Captain. You let your second in  com-
mand attack while you sit and watch for weakness.
-- Khan Noonian Singh, "Space Seed", stardate 3141.9
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 5/4] sf: unify write disable commands

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1303750794-8043-1-git-send-email-vap...@gentoo.org> you wrote:
> Every spi flash uses the same write disnable command, so unify this in
> the common code.
> 
> Signed-off-by: Mike Frysinger 
> ---
>  drivers/mtd/spi/spi_flash_internal.h |9 +
>  drivers/mtd/spi/sst.c|2 +-
>  2 files changed, 10 insertions(+), 1 deletions(-)

Applied (after fixing the typo in the commit message), 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: w...@denx.de
Machines take me by surprise with great frequency.  - Alan Turing
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 4/4] sf: eon/stmicro: inline useless ID defines

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1303635958-10985-4-git-send-email-vap...@gentoo.org> you wrote:
> These defines are used in only one place, so just inline them.
> 
> Signed-off-by: Mike Frysinger 
> ---
>  drivers/mtd/spi/eon.c |4 +---
>  drivers/mtd/spi/stmicro.c |   25 -
>  2 files changed, 9 insertions(+), 20 deletions(-)

Applied, 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: w...@denx.de
"Love your country but never trust its government."
- from a hand-painted road sign in central Pennsylvania
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2] sf: kill off now-unused local state

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1310157219-27682-1-git-send-email-vap...@gentoo.org> you wrote:
> Now that the common spi_flash structure tracks all the info that these
> drivers need, kill off their local state indirection and use just what
> the common code provides.
> 
> Signed-off-by: Mike Frysinger 
> ---
> v2
>   - rebase onto latest master
> 
>  drivers/mtd/spi/eon.c  |   36 
>  drivers/mtd/spi/macronix.c |   36 
>  drivers/mtd/spi/spansion.c |   36 
>  drivers/mtd/spi/stmicro.c  |   37 -
>  drivers/mtd/spi/winbond.c  |   39 +--
>  5 files changed, 61 insertions(+), 123 deletions(-)

Applied, 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: w...@denx.de
... The prejudices people feel about each other disappear  when  then
get to know each other.
-- Kirk, "Elaan of Troyius", stardate 4372.5
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/4 v2] sf: unify write funcs

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1303750709-7976-1-git-send-email-vap...@gentoo.org> you wrote:
> Once we add a new page_size field for write lengths, we can unify the
> write methods for most of the spi flash drivers.
> 
> Signed-off-by: Mike Frysinger 
> ---
> v2
>   - push page program command into common code too since all spi
> flashes use the same one (0x02)
> 
>  drivers/mtd/spi/eon.c|   66 +--
>  drivers/mtd/spi/macronix.c   |   66 +--
>  drivers/mtd/spi/spansion.c   |   66 +--
>  drivers/mtd/spi/spi_flash.c  |   57 +++
>  drivers/mtd/spi/spi_flash_internal.h |8 
>  drivers/mtd/spi/stmicro.c|   66 +--
>  drivers/mtd/spi/winbond.c|   71 +
>  include/spi_flash.h  |5 ++-
>  8 files changed, 80 insertions(+), 325 deletions(-)

Applied, 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: w...@denx.de
The only person who always got his work done by Friday
 was Robinson Crusoe.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 1/4] sf: unify write enable commands

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1303635958-10985-1-git-send-email-vap...@gentoo.org> you wrote:
> Every spi flash uses the same write enable command, so unify this in
> the common code.
> 
> Signed-off-by: Mike Frysinger 
> ---
>  drivers/mtd/spi/eon.c|2 +-
>  drivers/mtd/spi/macronix.c   |2 +-
>  drivers/mtd/spi/ramtron.c|2 +-
>  drivers/mtd/spi/spansion.c   |2 +-
>  drivers/mtd/spi/spi_flash.c  |2 +-
>  drivers/mtd/spi/spi_flash_internal.h |8 
>  drivers/mtd/spi/sst.c|2 +-
>  drivers/mtd/spi/stmicro.c|2 +-
>  drivers/mtd/spi/winbond.c|2 +-
>  9 files changed, 16 insertions(+), 8 deletions(-)

Applied, 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: w...@denx.de
"An open mind has but one disadvantage: it collects dirt."
- a saying at RPI
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] MAKEALL: add -h/--help options

[Wolfgang Denk]

Dear Mike Frysinger,

In message <1303459303-28600-1-git-send-email-vap...@gentoo.org> you wrote:
> Convert all the comments at the top of the file into help text for people
> to easily get at with standard -h/--help options.
> 
> Signed-off-by: Mike Frysinger 
> ---
>  MAKEALL |  100 
> +++
>  1 files changed, 49 insertions(+), 51 deletions(-)

Applied, 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: w...@denx.de
I believe you find life such a problem because you  think  there  are
the  good  people  and the bad people. You're wrong, of course. There
are, always and only, the bad people, but some of them are  on  oppo-
site sides.  - Terry Pratchett, _Guards! Guards!_
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] Fix: watchdog timed out, if using md5 command

[Jens Scharsig]

Dear Wolfgang Denk:
> Dear Jens Scharsig,
>
> Arghhh.  Line wrapped, too.
>
> This is the last time I'm gonna fix your patches.  PLEASE fix your
> mailer!
>

Sorry about the trouble.
I have many times the word wrap will be deactivated. Our admin will
reset this option on each update.

On future patches i will watch this and use git to send mail.

Best regards

Jens Scharsig
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] Professional Web Design and Web Development Agency

[Deepak]

 

 

Hi,

 

 

As many as 90% websites do not deliver business to their potential because
they do not appeal to the target audience. And the reason most visitors
leave a website is because of a poor design and navigation.

 

So is your website really customer-ready? Not really, if we go by the
analysis of our experts. We are a New Delhi (India) based Web Designing
Company having more than 500 clients from: UK, USA, Australia, Canada.

 

With an experience of successfully developing more than 2500 websites, we
could help you generate more business from your website, if you are ready to
change the website.  Most of our clients have benefited from our expertise,
and we believe you could too, by getting an attractive website as follows.

 

http://osc4.template-help.com/wt_32369/index.html

http://osc4.template-help.com/wt_32420/index.html 

http://osc4.template-help.com/wt_32127/index.html 

http://osc4.template-help.com/wt_32155/index.html 

http://osc4.template-help.com/wt_31960/index.html

 

You could get a professional looking website as per your expectations

 

Getting started is easy. Just mail us beck and we will help you get started.


 

 

Regards,

 

Name: Deepak Chaudhary

 

 

Disclaimer: The CAN-SPAM Act of 2003 (Controlling the Assault of
Non-Solicited Pornography and Marketing Act) establishes requirements for
those who send commercial email, spells out penalties for spammers and
companies whose products are advertised in spam if they violate the law, and
gives consumers the right to ask e-mailers to stop spamming them. The above
mail  rrwebservi...@gmail.com and we ensure you will not receive any mails
in future

 

 

 

 

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/2] powerpc/85xx: provide 85xx flush_icache for cmd_cache

[Kumar Gala]

On May 23, 2011, at 1:38 PM, Matthew McClintock wrote:

> This provides a function that will override the weak function
> flush_icache to let 85xx boards to flush the icache
> 
> cc: Kumar Gala 
> Signed-off-by: Matthew McClintock 
> ---
> arch/powerpc/cpu/mpc85xx/start.S |2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)

applied to 85xx

- k

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] AT91: rework and fixes

[Andreas Bießmann]

Dear all,

Am 26.07.2011 14:59, schrieb Reinhard Meyer:
> Dear Hong Xu,
>> Hi Albert,



>> As a suggestion, how about we setup a git branch and collect AT91 
>> related patches during Reinhard's busy time and vocation time? If yes, 
>> can we ask for `pull request' from you?
> 
> I do not think such a "diagonal" approach is desired by anyone.
> 
> Anyway, for work reasons, there will be no extended vacation this
> summer :(

How sad!

> Again, the work flow, as I want it followed, is as such:



> 5. When a board has passed those tests (I cannot test point 4) the
> "removal" patch for that board will not be applied.

A removed board can always added again, if one cares.

regards

Andreas Bießmann
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 7/7]Timer: Remove reset_timer_masked()

[Wolfgang Denk]

Dear Graeme Russ,

In message <1310809080-3481-1-git-send-email-graeme.r...@gmail.com> you wrote:
> In some circumstances, reset_timer_masked() was called by timer_init() in
> order to perform architecture specific timer initialisation. In such
> cases, the required code in reset_timer_masked() has been moved into
> timer_init()
> 
> Signed-off-by: Graeme Russ 
> ---
> Changes since v2:
>  - Added Signed-off-by
>  - Fixed typo in description (be->by)
> 
> Changes since v1 ((WIP) [Timer]API Rewrite):
>  - Rebased to 68d4230c3ccce96a72c5b99e48399bf1796fe3c6
>  - Updated description
> 
>  arch/arm/cpu/arm1136/mx31/timer.c|7 ---
>  arch/arm/cpu/arm1136/omap24xx/timer.c|   14 +++---
>  arch/arm/cpu/arm1176/s3c64xx/timer.c |7 ---
>  arch/arm/cpu/arm720t/interrupts.c|7 ---
>  arch/arm/cpu/arm920t/at91/timer.c|8 
>  arch/arm/cpu/arm920t/at91rm9200/timer.c  |7 ---
>  arch/arm/cpu/arm920t/ep93xx/timer.c  |   10 +++---
>  arch/arm/cpu/arm920t/imx/timer.c |   10 +++---
>  arch/arm/cpu/arm920t/ks8695/timer.c  |   15 +--
>  arch/arm/cpu/arm920t/s3c24x0/timer.c |7 ---
>  arch/arm/cpu/arm925t/timer.c |   12 +++-
>  arch/arm/cpu/arm926ejs/armada100/timer.c |   10 ++
>  arch/arm/cpu/arm926ejs/kirkwood/timer.c  |   10 ++
>  arch/arm/cpu/arm926ejs/mb86r0x/timer.c   |   14 +++---
>  arch/arm/cpu/arm926ejs/mx25/timer.c  |9 -
>  arch/arm/cpu/arm926ejs/mx27/timer.c  |9 -
>  arch/arm/cpu/arm926ejs/orion5x/timer.c   |   10 ++
>  arch/arm/cpu/arm926ejs/pantheon/timer.c  |   10 ++
>  arch/arm/cpu/arm926ejs/spear/timer.c |   11 +++
>  arch/arm/cpu/armv7/mx5/timer.c   |   10 --
>  arch/arm/cpu/armv7/omap-common/timer.c   |   11 +++
>  arch/arm/cpu/armv7/tegra2/timer.c|7 ---
>  arch/arm/cpu/pxa/timer.c |8 +---
>  arch/arm/cpu/s3c44b0/timer.c |7 ---
>  arch/arm/cpu/sa1100/timer.c  |6 --
>  board/armltd/integrator/timer.c  |   13 -
>  26 files changed, 47 insertions(+), 202 deletions(-)

Applied, 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: w...@denx.de
No, I'm not going to explain it. If you  can't  figure  it  out,  you
didn't want to know anyway... :-)
   - Larry Wall in <1991aug7.180856.2...@netlabs.com>
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 6/7]Timer: Fix at91rm9200/spi.c timer usage

[Wolfgang Denk]

Dear Graeme Russ,

In message <1310809041-3442-1-git-send-email-graeme.r...@gmail.com> you wrote:
> Signed-off-by: Graeme Russ 
> ---
> Changes since v2:
>  - Added Signed-off-by
> 
> Changes since v1 ((WIP) [Timer]API Rewrite):
>  - Rebased to 68d4230c3ccce96a72c5b99e48399bf1796fe3c6
> 
>  arch/arm/cpu/arm920t/at91rm9200/spi.c |5 +++--
>  1 files changed, 3 insertions(+), 2 deletions(-)

Applied, 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: w...@denx.de
The following statement is not true.  The previous statement is true.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 5/7]Timer: Remove reset_timer() for non-Nios2 arches

[Wolfgang Denk]

Dear Graeme Russ,

In message <1310809000-3402-1-git-send-email-graeme.r...@gmail.com> you wrote:
> Signed-off-by: Graeme Russ 
> ---
> Changes since v2:
>  - Added Signed-off-by
> 
> Changes since v1 ((WIP) [Timer]API Rewrite):
>  - Rebased to 68d4230c3ccce96a72c5b99e48399bf1796fe3c6
> 
>  arch/arm/cpu/arm1136/mx31/timer.c|5 -
>  arch/arm/cpu/arm1136/mx35/timer.c|5 -
>  arch/arm/cpu/arm1136/omap24xx/timer.c|5 -
>  arch/arm/cpu/arm1176/s3c64xx/timer.c |5 -
>  arch/arm/cpu/arm1176/tnetv107x/timer.c   |9 -
>  arch/arm/cpu/arm720t/interrupts.c|5 -
>  arch/arm/cpu/arm920t/a320/timer.c|6 --
>  arch/arm/cpu/arm920t/at91/timer.c|6 --
>  arch/arm/cpu/arm920t/at91rm9200/timer.c  |6 --
>  arch/arm/cpu/arm920t/ep93xx/timer.c  |5 -
>  arch/arm/cpu/arm920t/imx/timer.c |6 --
>  arch/arm/cpu/arm920t/ks8695/timer.c  |7 +--
>  arch/arm/cpu/arm920t/s3c24x0/timer.c |6 --
>  arch/arm/cpu/arm925t/timer.c |6 --
>  arch/arm/cpu/arm926ejs/armada100/timer.c |5 -
>  arch/arm/cpu/arm926ejs/davinci/timer.c   |5 -
>  arch/arm/cpu/arm926ejs/kirkwood/timer.c  |5 -
>  arch/arm/cpu/arm926ejs/mb86r0x/timer.c   |5 -
>  arch/arm/cpu/arm926ejs/mx25/timer.c  |5 -
>  arch/arm/cpu/arm926ejs/mx27/timer.c  |5 -
>  arch/arm/cpu/arm926ejs/nomadik/timer.c   |   12 +---
>  arch/arm/cpu/arm926ejs/omap/timer.c  |6 --
>  arch/arm/cpu/arm926ejs/orion5x/timer.c   |5 -
>  arch/arm/cpu/arm926ejs/pantheon/timer.c  |5 -
>  arch/arm/cpu/arm926ejs/spear/timer.c |6 --
>  arch/arm/cpu/arm926ejs/versatile/timer.c |6 --
>  arch/arm/cpu/armv7/mx5/timer.c   |5 -
>  arch/arm/cpu/armv7/omap-common/timer.c   |5 -
>  arch/arm/cpu/armv7/s5p-common/timer.c|5 -
>  arch/arm/cpu/armv7/tegra2/timer.c|5 -
>  arch/arm/cpu/ixp/timer.c |5 -
>  arch/arm/cpu/lh7a40x/timer.c |6 --
>  arch/arm/cpu/pxa/timer.c |7 +--
>  arch/arm/cpu/s3c44b0/timer.c |6 --
>  arch/arm/cpu/sa1100/timer.c  |5 -
>  arch/avr32/cpu/interrupts.c  |7 ---
>  arch/blackfin/cpu/interrupts.c   |5 -
>  arch/m68k/cpu/mcf547x_8x/slicetimer.c|5 -
>  arch/m68k/lib/time.c |5 -
>  arch/microblaze/cpu/timer.c  |7 +--
>  arch/mips/cpu/mips32/time.c  |6 --
>  arch/powerpc/lib/interrupts.c|5 -
>  arch/sh/lib/time.c   |7 ---
>  arch/sh/lib/time_sh2.c   |8 
>  arch/sparc/lib/interrupts.c  |7 +--
>  arch/x86/lib/timer.c |5 -
>  board/armltd/integrator/timer.c  |5 -
>  board/armltd/vexpress/ca9x4_ct_vxp.c |5 -
>  board/nvidia/common/board.c  |1 -
>  49 files changed, 9 insertions(+), 270 deletions(-)

Applied, 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: w...@denx.de
Heavier than air flying machines are impossible.
-- Lord Kelvin, President, Royal Society, c. 1895
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 4/7]Timer: Allow reset_timer() only for systems with low resolution timers

[Wolfgang Denk]

Dear Graeme Russ,

In message <1310808956-3364-1-git-send-email-graeme.r...@gmail.com> you wrote:
> Signed-off-by: Graeme Russ 
> ---
> Changes since v2:
>  - Added Signed-off-by
> 
> Changes since v1 ((WIP) [Timer]API Rewrite):
>  - Rebased to 68d4230c3ccce96a72c5b99e48399bf1796fe3c6
>  - Introduce new #define CONFIG_SYS_LOW_RES_TIMER
> 
>  drivers/block/mg_disk.c |2 ++
>  drivers/mtd/cfi_flash.c |4 
>  include/configs/PCI5441.h   |1 +
>  include/configs/PK1C20.h|1 +
>  include/configs/nios2-generic.h |1 +
>  5 files changed, 9 insertions(+), 0 deletions(-)

Applied, 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: w...@denx.de
This cultural mystique surrounding the  biological  function  --  you
realize humans are overly preoccupied with the subject.
-- Kelinda the Kelvan, "By Any Other Name", stardate 4658.9
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 3/7]Timer: Remove set_timer completely

[Wolfgang Denk]

Dear Graeme Russ,

In message <1310808899-3325-1-git-send-email-graeme.r...@gmail.com> you wrote:
> Signed-off-by: Graeme Russ 
> ---
> Changes since v2:
>  - Added Signed-off-by
> 
> Changes since v1 ((WIP) [Timer]API Rewrite):
>  - Rebased to 68d4230c3ccce96a72c5b99e48399bf1796fe3c6
>  - Updated description
> 
>  arch/arm/cpu/arm1136/mx31/timer.c|5 -
>  arch/arm/cpu/arm1136/mx35/timer.c|4 
>  arch/arm/cpu/arm1136/omap24xx/timer.c|5 -
>  arch/arm/cpu/arm1176/s3c64xx/timer.c |5 -
>  arch/arm/cpu/arm1176/tnetv107x/timer.c   |5 -
>  arch/arm/cpu/arm720t/interrupts.c|5 -
>  arch/arm/cpu/arm920t/a320/timer.c|6 --
>  arch/arm/cpu/arm920t/at91/timer.c|5 -
>  arch/arm/cpu/arm920t/at91rm9200/timer.c  |5 -
>  arch/arm/cpu/arm920t/imx/timer.c |5 -
>  arch/arm/cpu/arm920t/ks8695/timer.c  |5 -
>  arch/arm/cpu/arm920t/s3c24x0/timer.c |5 -
>  arch/arm/cpu/arm925t/timer.c |5 -
>  arch/arm/cpu/arm926ejs/armada100/timer.c |5 -
>  arch/arm/cpu/arm926ejs/kirkwood/timer.c  |5 -
>  arch/arm/cpu/arm926ejs/mx25/timer.c  |5 -
>  arch/arm/cpu/arm926ejs/mx27/timer.c  |5 -
>  arch/arm/cpu/arm926ejs/omap/timer.c  |5 -
>  arch/arm/cpu/arm926ejs/orion5x/timer.c   |5 -
>  arch/arm/cpu/arm926ejs/pantheon/timer.c  |5 -
>  arch/arm/cpu/arm926ejs/spear/timer.c |5 -
>  arch/arm/cpu/arm926ejs/versatile/timer.c |5 -
>  arch/arm/cpu/armv7/mx5/timer.c   |5 -
>  arch/arm/cpu/armv7/omap-common/timer.c   |5 -
>  arch/arm/cpu/armv7/s5p-common/timer.c|5 -
>  arch/arm/cpu/armv7/tegra2/timer.c|5 -
>  arch/arm/cpu/armv7/u8500/timer.c |5 -
>  arch/arm/cpu/ixp/timer.c |5 -
>  arch/arm/cpu/lh7a40x/timer.c |5 -
>  arch/arm/cpu/pxa/timer.c |5 -
>  arch/arm/cpu/s3c44b0/timer.c |5 -
>  arch/arm/cpu/sa1100/timer.c  |5 -
>  arch/avr32/cpu/interrupts.c  |   16 
>  arch/m68k/cpu/mcf547x_8x/slicetimer.c|4 
>  arch/m68k/lib/board.c|2 --
>  arch/m68k/lib/time.c |   16 ++--
>  arch/microblaze/cpu/timer.c  |5 -
>  arch/mips/cpu/mips32/time.c  |6 --
>  arch/nios2/cpu/interrupts.c  |6 --
>  arch/powerpc/lib/board.c |2 --
>  arch/powerpc/lib/interrupts.c|5 -
>  arch/sh/lib/time.c   |7 +--
>  arch/sh/lib/time_sh2.c   |9 ++---
>  arch/sparc/lib/board.c   |2 --
>  arch/sparc/lib/interrupts.c  |5 -
>  arch/x86/lib/board.c |2 --
>  arch/x86/lib/timer.c |5 -
>  board/armltd/integrator/timer.c  |6 --
>  include/common.h |1 -
>  49 files changed, 5 insertions(+), 254 deletions(-)

Applied, 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: w...@denx.de
"What terrible way to die."
"There are no good ways."
-- Sulu and Kirk, "That Which Survives", stardate unknown
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 2/7]Timer: Remove calls to set_timer outside arch/

[Wolfgang Denk]

Dear Graeme Russ,

In message <1310808837-3286-1-git-send-email-graeme.r...@gmail.com> you wrote:
> There is no need to use set_timer(). Replace with appropriate use of
> get_timer()
> 
> Signed-off-by: Graeme Russ 
> ---
> Sorry Wolfgang, but this patch has been through a few versions before it
> got integrated into the larger Timer API hence the v1->v2->v1 debacle.
> 
> checkpatch complains about long lines and brace usage in the board specific
> flash.c files - They are deprecated and not worth fixing for style
> 
> Changes since v2:
>  - None - Revised to v3 with other patches in series
> 
> Changes since v1 ((WIP) [Timer]API Rewrite):
>  - Rebased to 68d4230c3ccce96a72c5b99e48399bf1796fe3c6
> 
> Changes since V1:
>  - Fix typo in commit message
>  - Add reset_timer() to Nios2 get_timer(0)
> 
> Changes since V2
>  - No longer remove reset_timer()
> ---
>  board/BuS/EB+MCF-EV123/flash.c |   10 ++
>  board/cobra5272/flash.c|   10 ++
>  board/idmr/flash.c |   10 ++
>  3 files changed, 18 insertions(+), 12 deletions(-)

Applied, 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: w...@denx.de
While most peoples' opinions change, the conviction of their correct-
ness never does.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] OMAP3 SPL: merge window question

[Wolfgang Denk]

Dear Simon Schwarz,

In message <20110718082935.GB3867@Zitronenbaum> you wrote:
> 
> I'am working on the OMAP3 SPL which is part of my bachelor thesis. Some of the
> code was already posted - but i staled my work to wait for the new SPL layout
> and the work on OMAP4 by Aneeshi V.
> 
> This weekend Annesh V. released the OMAP4 patch and I now start to merge my 
> OMAP3
> patches with the common parts of Aneesh's patch.
> 
> I think I can release a patch until the end of the week. Is it possible to get
> the patch into the upcoming release?
> 
> I finish my BA thesis in September - mainlining the code before would be 
> great.

I checked in the new SPL code right now.

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: w...@denx.de
Children begin by loving their parents. After a time they judge them.
Rarely, if ever, do they forgive them.  - Oscar Wilde
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 9/9] spl: add support for omap-common libraries

[Wolfgang Denk]

Dear Daniel Schwierzeck,

In message 
<1311005361-26241-10-git-send-email-daniel.schwierz...@googlemail.com> you 
wrote:
> Signed-off-by: Aneesh V 
> Signed-off-by: Daniel Schwierzeck 
> ---
> Changes since RFC v1:
> - none
> 
>  spl/Makefile |7 +++
>  1 files changed, 7 insertions(+), 0 deletions(-)

Applied, 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: w...@denx.de
Human beings were created by water to transport it uphill.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 8/9] spl: Add support for common libraries and drivers

[Wolfgang Denk]

Dear Daniel Schwierzeck,

In message 
<1311005361-26241-9-git-send-email-daniel.schwierz...@googlemail.com> you wrote:
> Signed-off-by: Aneesh V 
> Signed-off-by: Daniel Schwierzeck 
> ---
> Changes since RFC v1:
> - none
> 
>  spl/Makefile |   11 +++
>  1 files changed, 11 insertions(+), 0 deletions(-)

Applied, 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: w...@denx.de
Many aligators will be slain, but the swamp will remain.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 7/9] replace CONFIG_PRELOADER with CONFIG_SPL_BUILD

[Wolfgang Denk]

Dear Daniel Schwierzeck,

In message 
<1311005361-26241-8-git-send-email-daniel.schwierz...@googlemail.com> you wrote:
> From: Aneesh V 
> 
> replace all occurences of CONFIG_PRELOADER with CONFIG_SPL_BUILD
> 
> Signed-off-by: Aneesh V 
> Cc: Albert ARIBAUD 
> ---
> Changes since RFC v1:
> - none
> 
>  README|2 +-
>  arch/arm/cpu/arm1136/start.S  |   20 +-
>  arch/arm/cpu/arm1176/start.S  |4 +-
>  arch/arm/cpu/arm720t/start.S  |4 +-
>  arch/arm/cpu/arm920t/start.S  |4 +-
>  arch/arm/cpu/arm925t/start.S  |4 +-
>  arch/arm/cpu/arm926ejs/start.S|   18 
>  arch/arm/cpu/arm946es/start.S |4 +-
>  arch/arm/cpu/arm_intcm/start.S|4 +-
>  arch/arm/cpu/armv7/start.S|6 ++--
>  arch/arm/cpu/ixp/start.S  |4 +-
>  arch/arm/cpu/lh7a40x/start.S  |4 +-
>  arch/arm/cpu/pxa/start.S  |   30 ++--
>  arch/arm/cpu/s3c44b0/start.S  |4 +-
>  arch/arm/cpu/sa1100/start.S   |4 +-
>  arch/arm/lib/board.c  |2 +-
>  mmc_spl/board/samsung/smdkv310/Makefile   |2 +-
>  nand_spl/board/davinci/da8xxevm/Makefile  |4 +-
>  nand_spl/board/freescale/mx31pdk/Makefile |4 +-
>  nand_spl/board/karo/tx25/Makefile |4 +-
>  onenand_ipl/board/apollon/Makefile|4 +-
>  onenand_ipl/board/vpac270/Makefile|4 +-
>  22 files changed, 70 insertions(+), 70 deletions(-)

Applied, 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: w...@denx.de
I will also, for an appropriate fee, certify that  your  keyboard  is
object-oriented,  and  that  the bits on your hard disk are template-
compatible.- Jeffrey S. Haemer in <411akr$3...@cygnus.com>
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 6/9] scaled down version of generic libraries for SPL

[Wolfgang Denk]

Dear Daniel Schwierzeck,

In message 
<1311005361-26241-7-git-send-email-daniel.schwierz...@googlemail.com> you wrote:
> From: Aneesh V 
> 
> Signed-off-by: Aneesh V 
> Cc: Albert ARIBAUD 
> ---
> Changes since RFC v1:
> - none
> 
>  arch/arm/lib/Makefile   |2 ++
>  common/Makefile |9 ++---
>  drivers/serial/Makefile |3 +++
>  fs/fat/Makefile |6 +-
>  lib/Makefile|   11 +++
>  5 files changed, 23 insertions(+), 8 deletions(-)

Applied, 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: w...@denx.de
"How to make a million dollars:  First, get a million dollars."
- Steve Martin
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v4 5/9] arm: adjust PLATFORM_LIBS for SPL

[Wolfgang Denk]

Dear Daniel Schwierzeck,

In message 
<1311090701-11801-1-git-send-email-daniel.schwierz...@googlemail.com> you wrote:
> From: Aneesh V 
> 
> Signed-off-by: Aneesh V 
> Cc: Albert ARIBAUD 
> ---
> Changes since RFC v1:
> - none
> 
> Changes since RFC v2:
> - none
> 
> Changes since v3:
> - improved comment
> 
>  arch/arm/config.mk |   13 ++---
>  1 files changed, 10 insertions(+), 3 deletions(-)

Applied, 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: w...@denx.de
I distrust all systematisers, and avoid them. The will  to  a  system
shows a lack of honesty.
- Friedrich Wilhelm Nietzsche _Götzen-Dämmerung [The Twilight of  the
Idols]_ ``Maxims and Missiles'' no. 26
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 4/9] Hook SPL build-system into toplevel Makefile

[Wolfgang Denk]

Dear Daniel Schwierzeck,

In message 
<1311005361-26241-5-git-send-email-daniel.schwierz...@googlemail.com> you wrote:
> Signed-off-by: Aneesh V 
> Signed-off-by: Daniel Schwierzeck 
> ---
> Changes since RFC v1:
> - none
> 
>  Makefile |5 +
>  1 files changed, 5 insertions(+), 0 deletions(-)

Applied, 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: w...@denx.de
I still miss my ex-wife, but my aim is getting better.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 3/9] Extend build-system for SPL framework

[Wolfgang Denk]

Dear Daniel Schwierzeck,

In message 
<1311005361-26241-4-git-send-email-daniel.schwierz...@googlemail.com> you wrote:
> Signed-off-by: Aneesh V 
> Signed-off-by: Daniel Schwierzeck 
> ---
> Changes since RFC v1:
> - enable garbage collect of unused sections for SPL unconditionally
> 
> Changes since RFC v2:
> - none
> 
>  Makefile  |3 ++-
>  config.mk |   34 +-
>  2 files changed, 35 insertions(+), 2 deletions(-)

Applied, 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: w...@denx.de
Boykottiert Microsoft - Kauft Eure Fenster bei OBI!
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v4 2/9] spl: add initial support for a generic SPL framework

[Wolfgang Denk]

Dear Daniel Schwierzeck,

In message 
<1311011287-28223-1-git-send-email-daniel.schwierz...@googlemail.com> you wrote:
> Signed-off-by: Aneesh V 
> Signed-off-by: Daniel Schwierzeck 
> ---
> Changes since RFC v1:
> - added documentation for SPL
> 
> Changes since RFC v2:
> - renamed CONFIG_SYS_SPL_LDSCRIPT to CONFIG_SPL_LDSCRIPT
> - added missing documentation for CONFIG_SPL_LDSCRIPT
> 
> Changes since v3:
> - fixed documentation
> 
>  README |   39 ++
>  doc/README.SPL |   62 ++
>  spl/.gitignore |4 ++
>  spl/Makefile   |  100 
> 
>  4 files changed, 205 insertions(+), 0 deletions(-)
>  create mode 100644 doc/README.SPL
>  create mode 100644 spl/.gitignore
>  create mode 100644 spl/Makefile

Applied, 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: w...@denx.de
Far back in the mists of ancient time, in the great and glorious days
of the former Galactic Empire, life was wild, rich  and  largely  tax
free. - Douglas Adams, _The Hitchhiker's Guide to the Galaxy_
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 1/9] Use ALL-y style instead of ifeq blocks for better readability

[Wolfgang Denk]

Dear Daniel Schwierzeck,

In message 
<1311005361-26241-2-git-send-email-daniel.schwierz...@googlemail.com> you wrote:
> Signed-off-by: Daniel Schwierzeck 
> Cc: Mike Frysinger 
> Cc: Stefan Roese 
> Cc: Vipin Kumar 
> ---
> Changes since RFC v1:
> - none
> 
>  Makefile   |   20 ++--
>  arch/blackfin/config.mk|2 +-
>  board/pcs440ep/config.mk   |2 +-
>  board/spear/spear300/config.mk |2 +-
>  board/spear/spear310/config.mk |2 +-
>  board/spear/spear320/config.mk |2 +-
>  board/spear/spear600/config.mk |2 +-
>  7 files changed, 12 insertions(+), 20 deletions(-)

Applied, 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: w...@denx.de
You're dead, Jim.
-- McCoy, "Amok Time", stardate 3372.7
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] Fix: watchdog timed out, if using md5 command

[Wolfgang Denk]

Dear Jens Scharsig,

In message <4e24296f.6060...@bus-elektronik.de> you wrote:
> * Fix: if using md5 command watchdog timed out
> * change function call md5(..) to the watchdog-safe variant
>   md5_wd(..) to support watchdog reset
> 
> 
> Signed-off-by: Jens Scharsig 
> ---
>  common/cmd_md5sum.c |2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/common/cmd_md5sum.c b/common/cmd_md5sum.c
> index d6ebb80..629a74d 100644
> --- a/common/cmd_md5sum.c
> +++ b/common/cmd_md5sum.c
> @@ -37,7 +37,7 @@ static int do_md5sum(cmd_tbl_t *cmdtp, int flag, int
> argc, char * const argv[])


Arghhh.  Line wrapped, too.

This is the last time I'm gonna fix your patches.  PLEASE fix your
mailer!


Applied, 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: w...@denx.de
Good morning. This is the telephone company. Due  to  repairs,  we're
giving  you  advance notice that your service will be cut off indefi-
nitely at ten o'clock. That's two minutes from now.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH V3 5/5] devkit8000: Add nand-spl support for new SPL

[Simon Schwarz]

Dear Andreas,

On 07/26/2011 12:42 PM, Andreas Bießmann wrote:
> Dear Simon,
>



>> diff --git a/board/timll/devkit8000/devkit8000.c 
>> b/board/timll/devkit8000/devkit8000.c
>> index 95afaaa..9b53742 100644
>> --- a/board/timll/devkit8000/devkit8000.c
>> +++ b/board/timll/devkit8000/devkit8000.c
>> @@ -119,7 +119,7 @@ void set_muxconf_regs(void)
>>  MUX_DEVKIT8000();
>>   }
>>
>> -#ifdef CONFIG_DRIVER_DM9000
>> +#if defined(CONFIG_DRIVER_DM9000)&  !defined(CONFIG_SPL_BUILD)
>>   /*
>>* Routine: board_eth_init
>>* Description: Setting up the Ethernet hardware.
>> diff --git a/include/configs/devkit8000.h b/include/configs/devkit8000.h
>> index 125c690..22c9b8e 100644
>> --- a/include/configs/devkit8000.h
>> +++ b/include/configs/devkit8000.h
>> @@ -307,4 +307,50 @@
>>   
>> CONFIG_SYS_INIT_RAM_SIZE - \
>>   
>> GENERATED_GBL_DATA_SIZE)
>>
>> +/* SRAM config */
>> +#define CONFIG_SYS_SRAM_START  0x4020
>> +#define CONFIG_SYS_SRAM_SIZE   0x
>
> you mentioned this as 'Changed' in your reply to my last review. It is
> still unchanged (but comment about 64kB removed, so is the size not 64kB?).

Sorry, my mistake. NOW corrected in V4.

> regards
>
> Andreas Bießmann

Regards
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] Fix: watchdog timed out, if using sha1 command

[Wolfgang Denk]

Dear Jens Scharsig,

In message <4e24296b.9000...@bus-elektronik.de> you wrote:
> * Fix: if using sha1 command watchdog timed out
> * change function call sha1_csum(..) to the watchdog-safe variant
>   sha1_csum_wd(..) to support watchdog reset
> 
> 
> Signed-off-by: Jens Scharsig 
> ---
>  common/cmd_sha1sum.c |2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/common/cmd_sha1sum.c b/common/cmd_sha1sum.c
> index bb3cff0..2b2dd8b 100644
> --- a/common/cmd_sha1sum.c
> +++ b/common/cmd_sha1sum.c
> @@ -37,7 +37,7 @@ static int do_sha1sum(cmd_tbl_t *cmdtp, int flag, int
> argc, char * const argv[])

Please fix your mailer (or rather use "git send-email to submit
patches).  This patch is line-wrapped, I had to fix it manually.

Applied, 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: w...@denx.de
If the car industry behaved like the computer industry over the  last
30  years, a Rolls-Royce would cost $5, get 300 miles per gallon, and
blow up once a year killing all passengers inside.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH V3 0/5] OMAP3 and devkit8000 SPL support

[Simon Schwarz]

Dear Wolfgang Denk,

On 07/26/2011 01:06 PM, Wolfgang Denk wrote:
> Dear Simon Schwarz,
>
> In message<1311675739-10966-1-git-send-email-simonschwarz...@gmail.com>  you 
> wrote:
>> V1  Initial SPL support for OMAP3 was based on the old SPL
>> V2  Introduced major changes. It is based on the OMAP4-SPL patch by Aneesh V
>>  and the new SPL Framework by Daniel Schwierzeck and Aneesh V
>> V3  Some small bug fixes and correct placed SOB.
>>
>> This is based on the following patches:
>> - New SPL framework
>> - OMAP4 SPL
>> Both are already in u-boot-ti repo.
>
> Please run all your patches through checkpatch BEFORE submitting.
>
>
> For example, checkpatch reports a  number of "line over 80 characters"
> coding style violations distibuted all over your patches.
>
> Please fix these globally.

sorry - wrong tab width. Changed in V4.

> Best regards,
>
> Wolfgang Denk
>

Regards
Simon

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2 1/2] command/cmd_cache.c: Add optional flush arguments

[Wolfgang Denk]

Dear Matthew McClintock,

In message <1306267745-14531-1-git-send-email-...@freescale.com> you wrote:
> It might be desirable to have the ability to flush icache/dcache
> within u-boot, this patch gives each arch the ability to provide
> a flush_dcache/flush_icache function to let u-boot flush caches
> from the prompt
> 
> Signed-off-by: Matthew McClintock 
> ---
> v2: Use __weak instead of __attribute__((weak,alias))
> 
>  common/cmd_cache.c |   37 -
>  1 files changed, 28 insertions(+), 9 deletions(-)

Applied, 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: w...@denx.de
Q: How do you spell "onomatopoeia"?
A: The way it sounds.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] AT91: rework and fixes (was:Re: [PATCH 47/52] ARM: remove broken "at91sam9261ek" /"at91sam9g10ek" boards)

[Reinhard Meyer]

Dear Hong Xu,
> Hi Albert,
> 
> On 07/18/2011 02:39 PM, Albert ARIBAUD wrote:
> > Hi Eric,
> >
> > Le 18/07/2011 04:52, Hong Xu a écrit :
> >  > Hi Reinhard,
> >  >
> >  > It's a pity to see that some of the AT91 boards are planed to be removed
> >  > by Wolfgang.
> >  >
> >  > Several weeks ago, the patches for SAM9261/9G10 have got Acks from you.
> >  > See http://www.mail-archive.com/u-boot@lists.denx.de/msg54829.html
> >  > But I don't know why it's not merged upstream.

Now, an actual at91sam9261_devices.c and at91sam9261.h are on top of
u-boot-atmel/master.
If your board patches for az91sam9261-ek still work with that, I can
apply them again to master.

> >  >
> >  > For SAM9RL, the V3 patches have been sent long time ago and was pinged.
> >  > But no response any longer.

I have changes requested there.

> >  >
> >  > The similar stories are also applied for SAM9263 and SAM9M10G45.

Changes requested as well.

> >  >
> >  > We of course need the mainline support for all SAM9 boards in U-Boot.
> >  > But it's the time we need your plans/suggestions/NAKs/ACKs to move on.
> >  >
> >  > Thanks.
> >
> > Note: I'll naturally defer the decision of what to make with the AT91
> > boards to Reinhard, and likewise to other ARM custodians for boards the
> > architecture of which falls under their 'jurisdiction'.
> 
> Understand. But it looks like Reinhard has bunch of paid work at hand 
> and he has his vocation in the near future (He mentioned this in an 
> email towards Wolfgang's  [U-Boot] [PATCH 00/52] ARM: remove broken 
> boards). We noticed that there are several AT91 fixes in the mailing 
> list without any response.

This is not true. Most have been commented and changes requested.

> 
> As a suggestion, how about we setup a git branch and collect AT91 
> related patches during Reinhard's busy time and vocation time? If yes, 
> can we ask for `pull request' from you?

I do not think such a "diagonal" approach is desired by anyone.

Anyway, for work reasons, there will be no extended vacation this
summer :(

Again, the work flow, as I want it followed, is as such:

1. convert/fix all SoC specific files for 9263 9g45 etc. to be in line
and equivalent to the now existing files for 9260 and 9261.
(Next should be for 9263, we should be looking at existing patches to
fix that SoC).
I will NOT accept patches that do not fully implement the name schemes
laid out in 9260 and 9261.

2. Fix any boards to use those files AND to use relocation properly.

3. Make sure those build without warning or errors.

4. Make sure those boards still "boot" with that u-boot.

5. When a board has passed those tests (I cannot test point 4) the
"removal" patch for that board will not be applied.

Best Regards,
Reinhard

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] openrd_ultimate GigEth ports not working

[Alex Zeffertt]

git bisect says:

c2dd0d45540397704de9b13287417d21049d34c6 is the first bad commit
commit c2dd0d45540397704de9b13287417d21049d34c6
Author: Aneesh V 
Date:   Thu Jun 16 23:30:49 2011 +

armv7: integrate cache maintenance support

- Enable I-cache on bootup
- Enable MMU and D-cache immediately after relocation
- Do necessary initialization before enabling d-cache and MMU
- Changes to cleanup_before_linux()
- Make changes according to the new framework

Signed-off-by: Aneesh V 

:04 04 2e80477b0d05ad168feb89ef0e00d9bbd79b0cd0
5505c5f890b2af6e9b54bba915fdf3368abdfb16 M arch


Thanks,

Alex


On 26 July 2011 12:09, Wolfgang Denk  wrote:

> Dear Alex Zeffertt,
>
> In message <
> canniwjkvykei0qmrc8v5lvtar+q8czyetycpqdjzv34htcl...@mail.gmail.com> you
> wrote:
> >
> > I can confirm that the issue is the same with OpenRD Ultimate.  I.e.
> > ethernet is not working in HEAD but it is fine on tag v2011.06.
>
> And what says "git bisect" ?
>
> 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: w...@denx.de
> Superior ability breeds superior ambition.
>-- Spock, "Space Seed", stardate 3141.9
>
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2 3/3] common/cmd_ximg.c: add ifdef protection for gzip uncompression

[Wolfgang Denk]

Dear Matthew McClintock,

In message <1306252106-32358-1-git-send-email-...@freescale.com> you wrote:
> Print a message if we do not have the ability to uncompress a gzip
> image. Before, u-boot would just assume the routines were available
> 
> Signed-off-by: Matthew McClintock 
> ---
> v2: If gzip is not available block out case statement and let it fall through
> to the default case
> 
>  common/cmd_ximg.c |2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)

Applied, 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: w...@denx.de
"I go on working for the same reason a hen goes on laying eggs."
- H. L. Mencken
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH V4 5/5] devkit8000: Add nand-spl support for new SPL

[Simon Schwarz]

Add NAND SPL support to the devkit8000 config

Signed-off-by: Simon Schwarz 
---
V1 changes:
ADD devkit8000_nand to board.cfg
ADD nand_spl Makefile, llinker script, spl-devkit8000.c
ADD config ecc, SRAM, SPL to board config
ADD CONFIG_SYS_SRAM_START and _SIZE to board config
ADD CONFIG_SYS_SPL_TEXT_BASE, _MAX_SIZE and SPL_STACK to board config

V2 changes:
ADD CONFIG_SPL and LIBCOMMON, LIBDISK, I2C, LIBGENERIC, SERIAL, POWER, NAND and
CONFIG_SPL_LDSCRIPT to board config
CHG renamed CONFIG_SYS_SPL_* to CONFIG_SPL_*
ADD CONFIG_SYS_NAND_U_BOOT_START, _OFFS, _SIZE, _DST to board config: Where to
expect u-boot and where to load it.
ADD some barrier to not build board_eth_init in SPL
DEL no changes to board.cfg
DEL everything used the old nand_spl layout (Makefile, linker script,
spl-devkit8000.c)
CHG cosmetic

V3 changes:
CHG Deleted wrong comment

V4 changes:
CHG CONFIG_SYS_SRAM_SIZE NOW has the right value
CHG cosmetic - corrected style problems

Transition from V1 to V2 also includes that this patch is now based on
- the new SPL layout by Aneesh V and Daniel Schwierzeck
- the OMAP4 SPL patches by Aneesh V

This is the successor of "[U-Boot,5/5] devkit8000 nand_spl: add nand_spl
support"
(http://article.gmane.org/gmane.comp.boot-loaders.u-boot/102111)
---
 board/timll/devkit8000/devkit8000.c |2 +-
 include/configs/devkit8000.h|   46 +++
 2 files changed, 47 insertions(+), 1 deletions(-)

diff --git a/board/timll/devkit8000/devkit8000.c 
b/board/timll/devkit8000/devkit8000.c
index 95afaaa..9b53742 100644
--- a/board/timll/devkit8000/devkit8000.c
+++ b/board/timll/devkit8000/devkit8000.c
@@ -119,7 +119,7 @@ void set_muxconf_regs(void)
MUX_DEVKIT8000();
 }
 
-#ifdef CONFIG_DRIVER_DM9000
+#if defined(CONFIG_DRIVER_DM9000) & !defined(CONFIG_SPL_BUILD)
 /*
  * Routine: board_eth_init
  * Description: Setting up the Ethernet hardware.
diff --git a/include/configs/devkit8000.h b/include/configs/devkit8000.h
index 125c690..022069d 100644
--- a/include/configs/devkit8000.h
+++ b/include/configs/devkit8000.h
@@ -307,4 +307,50 @@
 
CONFIG_SYS_INIT_RAM_SIZE - \
 
GENERATED_GBL_DATA_SIZE)
 
+/* SRAM config */
+#define CONFIG_SYS_SRAM_START  0x4020
+#define CONFIG_SYS_SRAM_SIZE   0x1
+
+/* Defines for SPL */
+#define CONFIG_SPL
+
+#define CONFIG_SPL_LIBCOMMON_SUPPORT
+#define CONFIG_SPL_LIBDISK_SUPPORT
+#define CONFIG_SPL_I2C_SUPPORT
+#define CONFIG_SPL_LIBGENERIC_SUPPORT
+#define CONFIG_SPL_SERIAL_SUPPORT
+#define CONFIG_SPL_POWER_SUPPORT
+#define CONFIG_SPL_NAND_SUPPORT
+#define CONFIG_SPL_LDSCRIPT"$(CPUDIR)/omap-common/u-boot-spl.lds"
+
+#define CONFIG_SPL_TEXT_BASE   0x4020 /*CONFIG_SYS_SRAM_START*/
+#define CONFIG_SPL_MAX_SIZE0xB400  /* 45 K */
+#define CONFIG_SPL_STACK   LOW_LEVEL_SRAM_STACK
+
+#define CONFIG_SPL_BSS_START_ADDR  0x8000 /*CONFIG_SYS_SDRAM_BASE*/
+#define CONFIG_SPL_BSS_MAX_SIZE0x8
+
+/* NAND boot config */
+#define CONFIG_SYS_NAND_PAGE_COUNT 64
+#define CONFIG_SYS_NAND_PAGE_SIZE  2048
+#define CONFIG_SYS_NAND_OOBSIZE64
+#define CONFIG_SYS_NAND_BLOCK_SIZE (128*1024)
+#define CONFIG_SYS_NAND_BAD_BLOCK_POS  0
+#define CONFIG_SYS_NAND_ECCPOS {2, 3, 4, 5, 6, 7, 8, 9,\
+   10, 11, 12, 13}
+
+#define CONFIG_SYS_NAND_ECCSIZE512
+#define CONFIG_SYS_NAND_ECCBYTES   3
+
+#define CONFIG_SYS_NAND_ECCSTEPS   (CONFIG_SYS_NAND_PAGE_SIZE / \
+   CONFIG_SYS_NAND_ECCSIZE)
+#define CONFIG_SYS_NAND_ECCTOTAL   (CONFIG_SYS_NAND_ECCBYTES * \
+   CONFIG_SYS_NAND_ECCSTEPS)
+
+#define CONFIG_SYS_NAND_U_BOOT_START   CONFIG_SYS_NAND_U_BOOT_DST
+
+#define CONFIG_SYS_NAND_U_BOOT_OFFS0x8
+#define CONFIG_SYS_NAND_U_BOOT_SIZE0x20
+#define CONFIG_SYS_NAND_U_BOOT_DST CONFIG_SYS_TEXT_BASE
+
 #endif /* __CONFIG_H */
-- 
1.7.4.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH V4 4/5] omap3: new SPL structure support

[Simon Schwarz]

Support for the new spl structure. Using the interface defined by Aneesh V for
OMAP4

Signed-off-by: Simon Schwarz 
---
V1 changes:
ADD support for early console output in SPL

V2 changes:
ADD include omap_common.h in board.c
ADD implement new omap common interface omap_boot_device, omap_boot_mode and
omap_rev_string (very basic)
CHG cosmetic
CHG Don't add ecc switch command in SPL
ADD save_boot_params stump with warning to implement it

V3 changes:
none

V4 changes:
CHG cosmetic - corrected style problem

Transition from V1 to V2 also includes that this patch is now based on
- the new SPL layout by Aneesh V and Daniel Schwierzeck
- the OMAP4 SPL patches by Aneesh V

This is in some parts a anccesstor of "[U-Boot,2/5] devkit8000 nand_spl: omap3
support nand_spl boot"
(http://article.gmane.org/gmane.comp.boot-loaders.u-boot/102114) in V1
---
 arch/arm/cpu/armv7/omap3/board.c|   37 +-
 arch/arm/cpu/armv7/omap3/lowlevel_init.S|5 +++
 arch/arm/include/asm/arch-omap3/sys_proto.h |1 +
 3 files changed, 41 insertions(+), 2 deletions(-)

diff --git a/arch/arm/cpu/armv7/omap3/board.c b/arch/arm/cpu/armv7/omap3/board.c
index 4aaf97b..0070fc2 100644
--- a/arch/arm/cpu/armv7/omap3/board.c
+++ b/arch/arm/cpu/armv7/omap3/board.c
@@ -39,6 +39,7 @@
 #include 
 #include 
 #include 
+#include 
 
 /* Declarations */
 extern omap3_sysinfo sysinfo;
@@ -56,6 +57,28 @@ static const struct gpio_bank gpio_bank_34xx[6] = {
 
 const struct gpio_bank *const omap_gpio_bank = gpio_bank_34xx;
 
+#ifdef CONFIG_SPL_BUILD
+/*
+* We use static variables because global data is not ready yet.
+* Initialized data is available in SPL right from the beginning.
+* We would not typically need to save these parameters in regular
+* U-Boot. This is needed only in SPL at the moment.
+*/
+u32 omap3_boot_device = BOOT_DEVICE_NAND;
+u32 omap3_boot_mode = NAND_MODE_HW_ECC;
+
+u32 omap_boot_device(void)
+{
+   return omap3_boot_device;
+}
+
+u32 omap_boot_mode(void)
+{
+   return omap3_boot_mode;
+}
+#endif /* CONFIG_SPL_BUILD */
+
+
 /**
  * Routine: delay
  * Description: spinning delay to use before udelay works
@@ -197,6 +220,10 @@ void s_init(void)
 
per_clocks_enable();
 
+#ifdef CONFIG_SPL_BUILD
+   preloader_console_init();
+#endif
+
if (!in_sdram)
mem_init();
 }
@@ -245,7 +272,7 @@ void abort(void)
 {
 }
 
-#ifdef CONFIG_NAND_OMAP_GPMC
+#if defined(CONFIG_NAND_OMAP_GPMC) & !defined(CONFIG_SPL_BUILD)
 /**
  * OMAP3 specific command to switch between NAND HW and SW ecc
  */
@@ -273,7 +300,7 @@ U_BOOT_CMD(
"[hw/sw] - Switch between NAND hardware (hw) or software (sw) ecc 
algorithm"
 );
 
-#endif /* CONFIG_NAND_OMAP_GPMC */
+#endif /* CONFIG_NAND_OMAP_GPMC & !CONFIG_SPL_BUILD */
 
 #ifdef CONFIG_DISPLAY_BOARDINFO
 /**
@@ -402,3 +429,9 @@ void v7_outer_cache_disable(void)
omap3_update_aux_cr(0, 0x2);
 }
 #endif
+
+void omap_rev_string(char *omap_rev_string)
+{
+   sprintf(omap_rev_string, "OMAP3, sorry revision detection \
+   unimplemented");
+}
diff --git a/arch/arm/cpu/armv7/omap3/lowlevel_init.S 
b/arch/arm/cpu/armv7/omap3/lowlevel_init.S
index 67e8ceb..48a7ec6 100644
--- a/arch/arm/cpu/armv7/omap3/lowlevel_init.S
+++ b/arch/arm/cpu/armv7/omap3/lowlevel_init.S
@@ -35,6 +35,11 @@
 _TEXT_BASE:
.word   CONFIG_SYS_TEXT_BASE/* sdram load addr from config.mk */
 
+.global save_boot_params
+save_boot_params:
+   #warning "Please implement save_boot_params for OMAP3"
+   bx lr
+
 .global omap3_gp_romcode_call
 omap3_gp_romcode_call:
PUSH {r4-r12, lr} @ Save all registers from ROM code!
diff --git a/arch/arm/include/asm/arch-omap3/sys_proto.h 
b/arch/arm/include/asm/arch-omap3/sys_proto.h
index 995e7cb..7b60051 100644
--- a/arch/arm/include/asm/arch-omap3/sys_proto.h
+++ b/arch/arm/include/asm/arch-omap3/sys_proto.h
@@ -71,4 +71,5 @@ void power_init_r(void);
 void dieid_num_r(void);
 void do_omap3_emu_romcode_call(u32 service_id, u32 parameters);
 void omap3_gp_romcode_call(u32 service_id, u32 parameter);
+void omap_rev_string(char *omap_rev_string);
 #endif
-- 
1.7.4.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH V4 3/5] nand spl: add NAND Library to new SPL

[Simon Schwarz]

Insert some NAND driver sources into NAND SPL library.

Signed-off-by: Simon Schwarz 
---
V1 changes:
CHG Default to HW ecc in SPL build
ADD nand_read_buf16 function, read buffer
ADD omap_dev_ready function, indicte if chip is ready

V2 changes:
DEL GPMC_WAIT0_PIN_ACTIVE define
CHG omap_dev_ready() renamed to  omap_spl_dev_ready(), does not use the
GPMC_WAIT0_PIN_ACTIVE-define anymore
CHG ogpmc_read_buf16 renamed omap_spl_read_buf16
ADD omap_spl_read_buf, 8x buf read function
ADD CONFIG_SPL_POWER_SUPPORT and CONFIG_SPL_NAND_SUPPORT to SPL
CHG cosmetic
CHG nand_base and nand_bbt aren't needed for SPL anymore
CHG omap_nand_switch_ecc is not compiled for SPL
ADD entry for CONFIG_SPL_POWER_SUPPORT and CONFIG_SPL_NAND_SUPPORT to README.SPL

V3 changes:
DEL cosmetic (empty line)

Transition from V1 to V2 also includes that this patch is now based on
- the new SPL layout by Aneesh V and Daniel Schwierzeck
- the OMAP4 SPL patches by Aneesh V

This Patch is related to "[U-Boot,4/5] devkit8000 nand_spl: Add SPL NAND support
to omap_gpmc driver"
(http://article.gmane.org/gmane.comp.boot-loaders.u-boot/102115) in V1
---
 doc/README.SPL   |2 +
 drivers/mtd/nand/Makefile|6 +++-
 drivers/mtd/nand/omap_gpmc.c |   68 ++
 spl/Makefile |2 +
 4 files changed, 77 insertions(+), 1 deletions(-)

diff --git a/doc/README.SPL b/doc/README.SPL
index ce8e19f..2987f43 100644
--- a/doc/README.SPL
+++ b/doc/README.SPL
@@ -60,3 +60,5 @@ CONFIG_SPL_SPI_FLASH_SUPPORT (drivers/mtd/spi/libspi_flash.o)
 CONFIG_SPL_SPI_SUPPORT (drivers/spi/libspi.o)
 CONFIG_SPL_FAT_SUPPORT (fs/fat/libfat.o)
 CONFIG_SPL_LIBGENERIC_SUPPORT (lib/libgeneric.o)
+CONFIG_SPL_POWER_SUPPORT (drivers/power/libpower.o)
+CONFIG_SPL_NAND_SUPPORT (drivers/mtd/nand/libnand.o)
diff --git a/drivers/mtd/nand/Makefile b/drivers/mtd/nand/Makefile
index 8b598f6..cdc9a14 100644
--- a/drivers/mtd/nand/Makefile
+++ b/drivers/mtd/nand/Makefile
@@ -26,12 +26,16 @@ include $(TOPDIR)/config.mk
 LIB:= $(obj)libnand.o
 
 ifdef CONFIG_CMD_NAND
+ifdef CONFIG_SPL_BUILD
+COBJS-y += nand_spl.o
+else
 COBJS-y += nand.o
 COBJS-y += nand_base.o
 COBJS-y += nand_bbt.o
-COBJS-y += nand_ecc.o
 COBJS-y += nand_ids.o
 COBJS-y += nand_util.o
+endif
+COBJS-y += nand_ecc.o
 
 COBJS-$(CONFIG_NAND_ATMEL) += atmel_nand.o
 COBJS-$(CONFIG_DRIVER_NAND_BFIN) += bfin_nand.o
diff --git a/drivers/mtd/nand/omap_gpmc.c b/drivers/mtd/nand/omap_gpmc.c
index 99b9cef..61eac35 100644
--- a/drivers/mtd/nand/omap_gpmc.c
+++ b/drivers/mtd/nand/omap_gpmc.c
@@ -61,6 +61,55 @@ static void omap_nand_hwcontrol(struct mtd_info *mtd, 
int32_t cmd,
writeb(cmd, this->IO_ADDR_W);
 }
 
+#ifdef CONFIG_SPL_BUILD
+/* Check wait pin as dev ready indicator */
+int omap_spl_dev_ready(struct mtd_info *mtd)
+{
+   return gpmc_cfg->status & (1 << 8);
+}
+
+/*
+ * omap_spl_read_buf16 - [DEFAULT] read chip data into buffer
+ * @mtd:MTD device structure
+ * @buf:buffer to store date
+ * @len:number of bytes to read
+ *
+ * Default read function for 16bit buswith
+ *
+ * This function is based on nand_read_buf16 from nand_base.c. This version
+ * reads 32bit not 16bit although the bus only has 16bit.
+ */
+static void omap_spl_read_buf16(struct mtd_info *mtd, uint8_t *buf, int len)
+{
+   int i;
+   struct nand_chip *chip = mtd->priv;
+   u32 *p = (u32 *) buf;
+   len >>= 2;
+
+   for (i = 0; i < len; i++)
+   p[i] = readl(chip->IO_ADDR_R);
+}
+
+/*
+ * omap_spl_read_buf - [DEFAULT] read chip data into buffer
+ * @mtd:MTD device structure
+ * @buf:buffer to store date
+ * @len:number of bytes to read
+ *
+ * Default read function for 8bit buswith
+ *
+ * This is the same function as this from nand_base.c nand_read_buf
+ */
+static void omap_spl_read_buf(struct mtd_info *mtd, uint8_t *buf, int len)
+{
+   int i;
+   struct nand_chip *chip = mtd->priv;
+
+   for (i = 0; i < len; i++)
+   buf[i] = readb(chip->IO_ADDR_R);
+}
+#endif
+
 /*
  * omap_hwecc_init - Initialize the Hardware ECC for NAND flash in
  *   GPMC controller
@@ -224,6 +273,7 @@ static void omap_enable_hwecc(struct mtd_info *mtd, int32_t 
mode)
}
 }
 
+#ifndef CONFIG_SPL_BUILD
 /*
  * omap_nand_switch_ecc - switch the ECC operation b/w h/w ecc and s/w ecc.
  * The default is to come up on s/w ecc
@@ -280,6 +330,7 @@ void omap_nand_switch_ecc(int32_t hardware)
 
nand->options &= ~NAND_OWN_BUFFERS;
 }
+#endif /* CONFIG_SPL_BUILD */
 
 /*
  * Board-specific NAND initialization. The following members of the
@@ -338,7 +389,24 @@ int board_nand_init(struct nand_chip *nand)
 
nand->chip_delay = 100;
/* Default ECC mode */
+#ifndef CONFIG_SPL_BUILD
nand->ecc.mode = NAND_ECC_SOFT;
+#else
+   nand->ecc.mode = NAND_ECC_HW;
+   nand->ecc.layout = &hw_nand_oob;
+   nand->ecc.size = CONFIG_SYS_NAND_ECCSIZE;
+   nand->ec

[U-Boot] [PATCH V4 2/5] omap-common: add nand spl support

[Simon Schwarz]

Add NAND support for the new SPL structure.

Signed-off-by: Simon Schwarz 
---
This patch didn't exist before V2!

V2 changes:
ADD Some define-barriers for OMAP3 to only use NAND
ADD nand_load_image() - inits the OMAP gpmc, loads the images - parses the
header
CHG cosmetic
ADD do_reset() implementation for omap-common spl
ADD nand_copy_image to nand.h
ADD CPP barriers for mmc and nand support. The parts depending on library
support are only compiled if the respective library is included.

V3 changes:
ADD Comment why setup_clocks_for_console() isn't called for OMAP3
CHG cosmetic (deleted empty line)
CHG rename of NAND_MODE_HW to NAND_MODE_HW_ECC
DEL NAND_MODE_SW. Not used.

V4 changes:
CHG cosmetic - style problems

Transition from V1 to V2 also includes that this patch is now based on
- the new SPL layout by Aneesh V and Daniel Schwierzeck
- the OMAP4 SPL patches by Aneesh V
---
 arch/arm/cpu/armv7/omap-common/spl.c |   46 ++
 arch/arm/include/asm/omap_common.h   |1 +
 include/nand.h   |2 +
 3 files changed, 49 insertions(+), 0 deletions(-)

diff --git a/arch/arm/cpu/armv7/omap-common/spl.c 
b/arch/arm/cpu/armv7/omap-common/spl.c
index d177652..468699d 100644
--- a/arch/arm/cpu/armv7/omap-common/spl.c
+++ b/arch/arm/cpu/armv7/omap-common/spl.c
@@ -26,6 +26,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -107,6 +108,7 @@ static void parse_image_header(const struct image_header 
*header)
}
 }
 
+#ifdef CONFIG_SPL_MMC_SUPPORT
 static void mmc_load_image_raw(struct mmc *mmc)
 {
u32 image_size_sectors, err;
@@ -140,7 +142,9 @@ end:
hang();
}
 }
+#endif /* CONFIG_SPL_MMC_SUPPORT */
 
+#ifdef CONFIG_SPL_MMC_SUPPORT
 static void mmc_load_image_fat(struct mmc *mmc)
 {
s32 err;
@@ -173,7 +177,9 @@ end:
hang();
}
 }
+#endif /* CONFIG_SPL_MMC_SUPPORT */
 
+#ifdef CONFIG_SPL_MMC_SUPPORT
 static void mmc_load_image(void)
 {
struct mmc *mmc;
@@ -206,6 +212,27 @@ static void mmc_load_image(void)
hang();
}
 }
+#endif /* CONFIG_SPL_MMC_SUPPORT */
+
+#ifdef CONFIG_SPL_NAND_SUPPORT
+static void nand_load_image(void)
+{
+   gpmc_init();
+   nand_init();
+   nand_copy_image(CONFIG_SYS_NAND_U_BOOT_OFFS,
+   CONFIG_SYS_NAND_U_BOOT_SIZE,
+   (uchar *)CONFIG_SYS_NAND_U_BOOT_DST);
+#ifdef CONFIG_NAND_ENV_DST
+   nand_copy_image(CONFIG_ENV_OFFSET, CONFIG_ENV_SIZE,
+   (uchar *)CONFIG_NAND_ENV_DST);
+#ifdef CONFIG_ENV_OFFSET_REDUND
+   nand_copy_image(CONFIG_ENV_OFFSET_REDUND, CONFIG_ENV_SIZE,
+   (uchar *)CONFIG_NAND_ENV_DST + CONFIG_ENV_SIZE);
+#endif
+#endif
+   parse_image_header((struct image_header *)CONFIG_SYS_NAND_U_BOOT_DST);
+}
+#endif /* CONFIG_SPL_NAND_SUPPORT */
 
 void jump_to_image_no_args(void)
 {
@@ -228,10 +255,17 @@ void board_init_r(gd_t *id, ulong dummy)
boot_device = omap_boot_device();
debug("boot device - %d\n", boot_device);
switch (boot_device) {
+#ifdef CONFIG_SPL_MMC_SUPPORT
case BOOT_DEVICE_MMC1:
case BOOT_DEVICE_MMC2:
mmc_load_image();
break;
+#endif
+#ifdef CONFIG_SPL_NAND_SUPPORT
+   case BOOT_DEVICE_NAND:
+   nand_load_image();
+   break;
+#endif
default:
printf("SPL: Un-supported Boot Device - %d!!!\n", boot_device);
hang();
@@ -259,7 +293,11 @@ void preloader_console_init(void)
gd->flags |= GD_FLG_RELOC;
gd->baudrate = CONFIG_BAUDRATE;
 
+/* Console clock for OMAP3 is already initialized by per_clocks_enable()
+ * called in board.c by s_init() */
+#ifndef CONFIG_OMAP34XX
setup_clocks_for_console();
+#endif
serial_init();  /* serial communications setup */
 
/* Avoid a second "U-Boot" coming from this string */
@@ -270,3 +308,11 @@ void preloader_console_init(void)
omap_rev_string(rev_string_buffer);
printf("Texas Instruments %s\n", rev_string_buffer);
 }
+
+int do_reset(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
+{
+   debug("resetting cpu...");
+   reset_cpu(0);
+
+   return 0;
+}
diff --git a/arch/arm/include/asm/omap_common.h 
b/arch/arm/include/asm/omap_common.h
index d3cb857..13f6884 100644
--- a/arch/arm/include/asm/omap_common.h
+++ b/arch/arm/include/asm/omap_common.h
@@ -49,6 +49,7 @@ void preloader_console_init(void);
 #defineMMCSD_MODE_UNDEFINED0
 #define MMCSD_MODE_RAW 1
 #define MMCSD_MODE_FAT 2
+#define NAND_MODE_HW_ECC   3
 
 u32 omap_boot_device(void);
 u32 omap_boot_mode(void);
diff --git a/include/nand.h b/include/nand.h
index 8d94b5c..e0f20f6 100644
--- a/include/nand.h
+++ b/include/nand.h
@@ -132,6 +132,8 @@ int nand_lock( nand_info_t *meminfo, int tight );
 int nand_unlock( nand_info_t *meminfo, ulong start, ulong length

[U-Boot] [PATCH V4 1/5] omap3: Configure RAM bank 0 if in SPL

[Simon Schwarz]

OMAP3 relied on the memory config done by X-loader or Configuration Header. This
has to be reworked for the implementation of a SPL. This patch configures RAM
bank 0 if CONFIG_SPL_BUILD is set. Settings for Micron-RAM used by devkit8000
are added to mem.h

Signed-off-by: Simon Schwarz 
---
V1 changes:
ADD Settings for Micron RAM

V2 changes:
DEL spl_debug outputs if mem test fails/passes
CHG CONFIG_PRELOADER to CONFIG_SPL_BUILD

V3 changes:
none

Transition from V1 to V2 also includes that this patch is now based on
- the new SPL layout by Aneesh V and Daniel Schwierzeck
- the OMAP4 SPL patches by Aneesh V

This is the successor of "[U-Boot,3/5] devkit8000 nand_spl: Add RAM
configuration independent of x-loader or CH"
(http://article.gmane.org/gmane.comp.boot-loaders.u-boot/102114)
---
 arch/arm/cpu/armv7/omap3/sdrc.c   |   30 ++-
 arch/arm/include/asm/arch-omap3/mem.h |   36 +
 2 files changed, 65 insertions(+), 1 deletions(-)

diff --git a/arch/arm/cpu/armv7/omap3/sdrc.c b/arch/arm/cpu/armv7/omap3/sdrc.c
index 2a7970b..dac14d0 100644
--- a/arch/arm/cpu/armv7/omap3/sdrc.c
+++ b/arch/arm/cpu/armv7/omap3/sdrc.c
@@ -8,6 +8,9 @@
  * Copyright (C) 2004-2010
  * Texas Instruments Incorporated - http://www.ti.com/
  *
+ * Copyright (C) 2011
+ * Corscience GmbH & Co. KG - Simon Schwarz 
+ *
  * Author :
  * Vaibhav Hiremath 
  *
@@ -133,13 +136,38 @@ void do_sdrc_init(u32 cs, u32 early)
sdelay(0x2);
}
 
+#ifdef CONFIG_SPL_BUILD
+   /* If we use a SPL there is no x-loader nor config header so we have
+* to do the job ourselfs
+*/
+   if (cs == CS0) {
+   sdrc_actim_base0 = (struct sdrc_actim *)SDRC_ACTIM_CTRL0_BASE;
+
+   /* General SDRC config */
+   writel(V_MCFG, &sdrc_base->cs[cs].mcfg);
+   writel(V_RFR_CTRL, &sdrc_base->cs[cs].rfr_ctrl);
+
+   /* AC timings */
+   writel(V_ACTIMA_165, &sdrc_actim_base0->ctrla);
+   writel(V_ACTIMB_165, &sdrc_actim_base0->ctrlb);
+
+   /* Initialize */
+   writel(CMD_NOP, &sdrc_base->cs[cs].manual);
+   writel(CMD_PRECHARGE, &sdrc_base->cs[cs].manual);
+   writel(CMD_AUTOREFRESH, &sdrc_base->cs[cs].manual);
+   writel(CMD_AUTOREFRESH, &sdrc_base->cs[cs].manual);
+
+   writel(V_MR, &sdrc_base->cs[cs].mr);
+   }
+#endif /* CONFIG_SPL_BUILD */
+
/*
 * SDRC timings are set up by x-load or config header
 * We don't need to redo them here.
 * Older x-loads configure only CS0
 * configure CS1 to handle this ommission
 */
-   if (cs) {
+   if (cs == CS1) {
sdrc_actim_base0 = (struct sdrc_actim *)SDRC_ACTIM_CTRL0_BASE;
sdrc_actim_base1 = (struct sdrc_actim *)SDRC_ACTIM_CTRL1_BASE;
writel(readl(&sdrc_base->cs[CS0].mcfg),
diff --git a/arch/arm/include/asm/arch-omap3/mem.h 
b/arch/arm/include/asm/arch-omap3/mem.h
index f165949..8e28f77 100644
--- a/arch/arm/include/asm/arch-omap3/mem.h
+++ b/arch/arm/include/asm/arch-omap3/mem.h
@@ -128,6 +128,33 @@ enum {
(MICRON_XSR_165 << 0) | (MICRON_TXP_165 << 8) | \
(MICRON_TWTR_165 << 16))
 
+#define MICRON_RAMTYPE 0x1
+#define MICRON_DDRTYPE 0x0
+#define MICRON_DEEPPD  0x1
+#define MICRON_B32NOT160x1
+#define MICRON_BANKALLOCATION  0x2
+#define MICRON_RAMSIZE ((PHYS_SDRAM_1_SIZE/(1024*1024))/2)
+#define MICRON_ADDRMUXLEGACY   0x1
+#define MICRON_CASWIDTH0x5
+#define MICRON_RASWIDTH0x2
+#define MICRON_LOCKSTATUS  0x0
+#define MICRON_V_MCFG ((MICRON_LOCKSTATUS << 30) | (MICRON_RASWIDTH << 24) | \
+   (MICRON_CASWIDTH << 20) | (MICRON_ADDRMUXLEGACY << 19) | \
+   (MICRON_RAMSIZE << 8) | (MICRON_BANKALLOCATION << 6) | \
+   (MICRON_B32NOT16 << 4) | (MICRON_DEEPPD << 3) | \
+   (MICRON_DDRTYPE << 2) | (MICRON_RAMTYPE))
+
+#define MICRON_ARCV2030
+#define MICRON_ARE 0x1
+#define MICRON_V_RFR_CTRL ((MICRON_ARCV << 8) | (MICRON_ARE))
+
+#define MICRON_BL  0x2
+#define MICRON_SIL 0x0
+#define MICRON_CASL0x3
+#define MICRON_WBST0x0
+#define MICRON_V_MR ((MICRON_WBST << 9) | (MICRON_CASL << 4) | \
+   (MICRON_SIL << 3) | (MICRON_BL))
+
 /*
  * NUMONYX part of IGEP v2 (165MHz optimized) 6.06ns
  *   ACTIMA
@@ -171,10 +198,15 @@ enum {
 #define V_ACTIMA_165 INFINEON_V_ACTIMA_165
 #define V_ACTIMB_165 INFINEON_V_ACTIMB_165
 #endif
+
 #ifdef CONFIG_OMAP3_MICRON_DDR
 #define V_ACTIMA_165 MICRON_V_ACTIMA_165
 #define V_ACTIMB_165 MICRON_V_ACTIMB_165
+#define V_MCFG MICRON_V_MCFG
+#define V_RFR_CTRL

[U-Boot] [PATCH V4 0/5] OMAP3 and devkit8000 SPL support

[Simon Schwarz]

V1  Initial SPL support for OMAP3 was based on the old SPL
V2  Introduced major changes. It is based on the OMAP4-SPL patch by
Aneesh V and the new SPL Framework by Daniel Schwierzeck and Aneesh V
V3  Some small bug fixes and correct placed SOB.
V4  Corrected one bugfix and some style problems

This is based on the following patches:
- New SPL framework
- OMAP4 SPL
Both are already in u-boot-ti repo.

Simon Schwarz (5):
  omap3: Configure RAM bank 0 if in SPL
  omap-common: add nand spl support
  nand spl: add NAND Library to new SPL
  omap3: new SPL structure support
  devkit8000: Add nand-spl support for new SPL

 arch/arm/cpu/armv7/omap-common/spl.c|   46 ++
 arch/arm/cpu/armv7/omap3/board.c|   37 ++-
 arch/arm/cpu/armv7/omap3/lowlevel_init.S|5 ++
 arch/arm/cpu/armv7/omap3/sdrc.c |   30 +++-
 arch/arm/include/asm/arch-omap3/mem.h   |   36 ++
 arch/arm/include/asm/arch-omap3/sys_proto.h |1 +
 arch/arm/include/asm/omap_common.h  |1 +
 board/timll/devkit8000/devkit8000.c |2 +-
 doc/README.SPL  |2 +
 drivers/mtd/nand/Makefile   |6 ++-
 drivers/mtd/nand/omap_gpmc.c|   68 +++
 include/configs/devkit8000.h|   46 ++
 include/nand.h  |2 +
 spl/Makefile|2 +
 14 files changed, 279 insertions(+), 5 deletions(-)

-- 
1.7.4.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot