[U-Boot] [PATCH 1/1 v5] arm: add support for the suen3 board from keymile

2010-02-18 Thread Heiko Schocher
Add support for the ARM part of the mgcoge2, named suen3.
This board is based on the Marvell Kirkwood (88F6281) SoC.
As there come more board variants, common code is stored in
board/keymile/km_arm/km_arm.c

Signed-off-by: Holger Brunck holger.bru...@keymile.com
Signed-off-by: Stefan Roese s...@denx.de
Signed-off-by: Heiko Schocher h...@denx.de
---
- changes since v1:
  added comments from Wolfgang Denk:
  get rid of flash_info_t define in board config
  (to get this working patch 1/2 is introduced/needed)

- changes since v2:
  added comments from Wolfgang Denk
  - rearranged if/else in do_spi_toggle()
  - added I/O accessor functions for bootcounter

- changes since v3:
  added comment Scott Wood
  - removed nand_init in do_spi_toggle()

  added comments from Prafulla Wadaskar
  - km-arm.h renamed to km_arm.h
  - reworked eeprom_write_enable() (deleted it)
(when reviewing this function, it cropped up, that
 this pin is connected through a gpio pin, not as
 in previous version, through the boco (a FPGA))
  - moved set_sda(), set_scl(), get_sda(), get_scl()
to km_arm.c
  - split patch in 4 patches (for each board an extra patch)
  - renamed sdramregs.txt in kwbimage.cfg, also license
info added.

- changes since v4
  added comments from Prafulla Wadaskar
  - add comments in board/keymile/km_arm/kwbimage.cfg
  - cleanup define CONFIG_KIRKWOOD_GPIO
  - corect wrong name at end of config file
  - add only support for the suen3 target, others
follow later

---
 MAINTAINERS   |2 +-
 MAKEALL   |1 +
 Makefile  |6 +
 board/keymile/common/common.c |6 +-
 board/keymile/km_arm/Makefile |   51 ++
 board/keymile/km_arm/config.mk|   28 
 board/keymile/km_arm/km_arm.c |  323 +
 board/keymile/km_arm/kwbimage.cfg |  176 
 include/configs/km_arm.h  |  191 ++
 include/configs/suen3.h   |  103 
 10 files changed, 884 insertions(+), 3 deletions(-)
 create mode 100644 board/keymile/km_arm/Makefile
 create mode 100644 board/keymile/km_arm/config.mk
 create mode 100644 board/keymile/km_arm/km_arm.c
 create mode 100644 board/keymile/km_arm/kwbimage.cfg
 create mode 100644 include/configs/km_arm.h
 create mode 100644 include/configs/suen3.h

diff --git a/MAINTAINERS b/MAINTAINERS
index fe63a9f..fe9494c 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -409,9 +409,9 @@ Heiko Schocher h...@denx.de
muas3001MPC8270
municse MPC5200
sc3 PPC405GP
+   suen3   ARM926EJS (Kirkwood SoC)
uc101   MPC5200

-
 Peter De Schrijver p...@mind.be

ML2 PPC4xx
diff --git a/MAKEALL b/MAKEALL
index 684db67..dfac5e4 100755
--- a/MAKEALL
+++ b/MAKEALL
@@ -579,6 +579,7 @@ LIST_ARM9= \
spear310\
spear320\
spear600\
+   suen3   \
trab\
VCMA9   \
versatile   \
diff --git a/Makefile b/Makefile
index e4431f7..3ea2b53 100644
--- a/Makefile
+++ b/Makefile
@@ -3047,6 +3050,9 @@ spear320_config : unconfig
 spear600_config :  unconfig
@$(MKCONFIG) -n $@ -t $(@:_config=) spear6xx arm arm926ejs 
$(@:_config=) spear spear

+suen3_config:  unconfig
+   @$(MKCONFIG) $(@:_config=) arm arm926ejs km_arm keymile kirkwood
+
 SX1_stdout_serial_config \
 SX1_config:unconfig
@mkdir -p $(obj)include
diff --git a/board/keymile/common/common.c b/board/keymile/common/common.c
index ec27bda..7b4eefd 100644
--- a/board/keymile/common/common.c
+++ b/board/keymile/common/common.c
@@ -35,6 +35,7 @@
 #include libfdt.h
 #endif

+#include ../common/common.h
 #if defined(CONFIG_HARD_I2C) || defined(CONFIG_SOFT_I2C)
 #include i2c.h

@@ -421,7 +422,6 @@ static int get_scl (void)

return ((val  SCL_BIT) == SCL_BIT);
 }
-
 #endif

 #if !defined(CONFIG_KMETER1)
@@ -500,7 +500,7 @@ void i2c_init_board(void)
out_8 (dev-cr, (I2C_CR_MEN));

 #else
-#if defined(CONFIG_HARD_I2C)
+#if defined(CONFIG_HARD_I2C)  !defined(CONFIG_MACH_SUEN3)
volatile immap_t *immap = (immap_t *)CONFIG_SYS_IMMR ;
volatile i2c8260_t *i2c = (i2c8260_t *)immap-im_i2c;

@@ -578,10 +578,12 @@ int fdt_get_node_and_value (void *blob,
 }
 #endif

+#if !defined(CONFIG_MACH_SUEN3)
 int ethernet_present (void)
 {
return (in_8((u8 *)CONFIG_SYS_PIGGY_BASE + CONFIG_SYS_SLOT_ID_OFF)  
0x80);
 }
+#endif

 int board_eth_init (bd_t *bis)
 {
diff --git a/board/keymile/km_arm/Makefile b/board/keymile/km_arm/Makefile
new file mode 100644
index 000..fffc6a3
--- /dev/null
+++ b/board/keymile/km_arm/Makefile
@@ -0,0 +1,51 @@
+#
+# (C) Copyright 2009
+# Marvell Semiconductor www.marvell.com
+# Prafulla Wadaskar prafu...@marvell.com
+#
+# See file CREDITS for list of 

Re: [U-Boot] [PATCH 1/1 v5] arm: add support for the suen3 board from keymile

2010-02-18 Thread Prafulla Wadaskar
 

 -Original Message-
 From: Heiko Schocher [mailto:h...@denx.de] 
 Sent: Thursday, February 18, 2010 1:54 PM
 To: U-Boot user list
 Cc: Prafulla Wadaskar; Wolfgang Denk; Scott Wood; Stefan Roese
 Subject: [PATCH 1/1 v5] arm: add support for the suen3 board 
 from keymile
 
 Add support for the ARM part of the mgcoge2, named suen3.
 This board is based on the Marvell Kirkwood (88F6281) SoC.
 As there come more board variants, common code is stored in
 board/keymile/km_arm/km_arm.c
 
 Signed-off-by: Holger Brunck holger.bru...@keymile.com
 Signed-off-by: Stefan Roese s...@denx.de
 Signed-off-by: Heiko Schocher h...@denx.de
 ---
 - changes since v1:
   added comments from Wolfgang Denk:
   get rid of flash_info_t define in board config
   (to get this working patch 1/2 is introduced/needed)
 
 - changes since v2:
   added comments from Wolfgang Denk
   - rearranged if/else in do_spi_toggle()
   - added I/O accessor functions for bootcounter
 
 - changes since v3:
   added comment Scott Wood
   - removed nand_init in do_spi_toggle()
 
   added comments from Prafulla Wadaskar
   - km-arm.h renamed to km_arm.h
   - reworked eeprom_write_enable() (deleted it)
 (when reviewing this function, it cropped up, that
  this pin is connected through a gpio pin, not as
  in previous version, through the boco (a FPGA))
   - moved set_sda(), set_scl(), get_sda(), get_scl()
 to km_arm.c
   - split patch in 4 patches (for each board an extra patch)
   - renamed sdramregs.txt in kwbimage.cfg, also license
 info added.
 
 - changes since v4
   added comments from Prafulla Wadaskar
   - add comments in board/keymile/km_arm/kwbimage.cfg
   - cleanup define CONFIG_KIRKWOOD_GPIO
   - corect wrong name at end of config file
   - add only support for the suen3 target, others
 follow later
 
 ---
  MAINTAINERS   |2 +-
  MAKEALL   |1 +
  Makefile  |6 +
  board/keymile/common/common.c |6 +-
  board/keymile/km_arm/Makefile |   51 ++
  board/keymile/km_arm/config.mk|   28 
  board/keymile/km_arm/km_arm.c |  323 
 +
  board/keymile/km_arm/kwbimage.cfg |  176 
  include/configs/km_arm.h  |  191 ++
  include/configs/suen3.h   |  103 
  10 files changed, 884 insertions(+), 3 deletions(-)
  create mode 100644 board/keymile/km_arm/Makefile
  create mode 100644 board/keymile/km_arm/config.mk
  create mode 100644 board/keymile/km_arm/km_arm.c
  create mode 100644 board/keymile/km_arm/kwbimage.cfg

  create mode 100644 include/configs/km_arm.h
  create mode 100644 include/configs/suen3.h

It would be good if you can merge these two files since single board board 
support
For additional board that you are planning, you can even add separate file or 
think of splitting it.
For better readability and to avoid complexity, I recommend to have one single 
file per board in include/config

...snip...
 diff --git a/board/keymile/km_arm/km_arm.c 
 b/board/keymile/km_arm/km_arm.c
 new file mode 100644
 index 000..31610be
 --- /dev/null
 +++ b/board/keymile/km_arm/km_arm.c
 @@ -0,0 +1,323 @@
 +/*
 + * (C) Copyright 2009
 + * Marvell Semiconductor www.marvell.com
 + * Prafulla Wadaskar prafu...@marvell.com
 + *
 + * (C) Copyright 2009
 + * Stefan Roese, DENX Software Engineering, s...@denx.de.
 + *
 + * (C) Copyright 2010
 + * Heiko Schocher, DENX Software Engineering, h...@denx.de.
 + *
 + * See file CREDITS for list of people who contributed to this
 + * project.
 + *
 + * This program is free software; you can redistribute it and/or
 + * modify it under the terms of the GNU General Public License as
 + * published by the Free Software Foundation; either version 2 of
 + * the License, or (at your option) any later version.
 + *
 + * This program is distributed in the hope that it will be useful,
 + * but WITHOUT ANY WARRANTY; without even the implied warranty of
 + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.   See the
 + * GNU General Public License for more details.
 + *
 + * You should have received a copy of the GNU General Public License
 + * along with this program; if not, write to the Free Software
 + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
 + * MA 02110-1301 USA
 + */
 +
 +#include common.h
 +#include i2c.h
 +#include nand.h
 +#include netdev.h
 +#include miiphy.h
 +#include asm/io.h
 +#include asm/arch/kirkwood.h
 +#include asm/arch/mpp.h
 +
 +#include ../common/common.h
 +
 +DECLARE_GLOBAL_DATA_PTR;
 +
 +static int   io_dev;
 +extern I2C_MUX_DEVICE *i2c_mux_ident_muxstring (uchar *buf);
 +
 +/* Multi-Purpose Pins Functionality configuration */
 +u32 kwmpp_config[] = {
 + MPP0_NF_IO2,
 + MPP1_NF_IO3,
 + MPP2_NF_IO4,
 + MPP3_NF_IO5,
 + MPP4_NF_IO6,
 + MPP5_NF_IO7,
 + MPP6_SYSRST_OUTn,
 + MPP7_PEX_RST_OUTn,
 +#if defined(CONFIG_SOFT_I2C)

Re: [U-Boot] [PATCH 1/1 v5] arm: add support for the suen3 board from keymile

2010-02-18 Thread Heiko Schocher
Hello Prafulla,

Prafulla Wadaskar wrote:
 -Original Message-
 From: Heiko Schocher [mailto:h...@denx.de] 
 Sent: Thursday, February 18, 2010 1:54 PM
 To: U-Boot user list
 Cc: Prafulla Wadaskar; Wolfgang Denk; Scott Wood; Stefan Roese
 Subject: [PATCH 1/1 v5] arm: add support for the suen3 board 
 from keymile

 Add support for the ARM part of the mgcoge2, named suen3.
 This board is based on the Marvell Kirkwood (88F6281) SoC.
 As there come more board variants, common code is stored in
 board/keymile/km_arm/km_arm.c

 Signed-off-by: Holger Brunck holger.bru...@keymile.com
 Signed-off-by: Stefan Roese s...@denx.de
 Signed-off-by: Heiko Schocher h...@denx.de
 ---
[...]
  MAINTAINERS   |2 +-
  MAKEALL   |1 +
  Makefile  |6 +
  board/keymile/common/common.c |6 +-
  board/keymile/km_arm/Makefile |   51 ++
  board/keymile/km_arm/config.mk|   28 
  board/keymile/km_arm/km_arm.c |  323 
 +
  board/keymile/km_arm/kwbimage.cfg |  176 
  include/configs/km_arm.h  |  191 ++
  include/configs/suen3.h   |  103 
  10 files changed, 884 insertions(+), 3 deletions(-)
  create mode 100644 board/keymile/km_arm/Makefile
  create mode 100644 board/keymile/km_arm/config.mk
  create mode 100644 board/keymile/km_arm/km_arm.c
  create mode 100644 board/keymile/km_arm/kwbimage.cfg
 
  create mode 100644 include/configs/km_arm.h
  create mode 100644 include/configs/suen3.h
 
 It would be good if you can merge these two files since single board board 
 support
 For additional board that you are planning, you can even add separate file or 
 think of splitting it.
 For better readability and to avoid complexity, I recommend to have one 
 single file per board in include/config

Hmm.. I think we tend to collect common config options in
one file, so board config files are not so big. Ok, actual
it is only one arm board from keymile here, but there
come more, and so it is sensible to collect common
options for this group of boards immediately ...

 ...snip...
 diff --git a/board/keymile/km_arm/km_arm.c 
 b/board/keymile/km_arm/km_arm.c
 new file mode 100644
 index 000..31610be
 --- /dev/null
 +++ b/board/keymile/km_arm/km_arm.c
 @@ -0,0 +1,323 @@
 +/*
 + * (C) Copyright 2009
 + * Marvell Semiconductor www.marvell.com
 + * Prafulla Wadaskar prafu...@marvell.com
 + *
 + * (C) Copyright 2009
 + * Stefan Roese, DENX Software Engineering, s...@denx.de.
 + *
 + * (C) Copyright 2010
 + * Heiko Schocher, DENX Software Engineering, h...@denx.de.
 + *
 + * See file CREDITS for list of people who contributed to this
 + * project.
 + *
 + * This program is free software; you can redistribute it and/or
 + * modify it under the terms of the GNU General Public License as
 + * published by the Free Software Foundation; either version 2 of
 + * the License, or (at your option) any later version.
 + *
 + * This program is distributed in the hope that it will be useful,
 + * but WITHOUT ANY WARRANTY; without even the implied warranty of
 + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 + * GNU General Public License for more details.
 + *
 + * You should have received a copy of the GNU General Public License
 + * along with this program; if not, write to the Free Software
 + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
 + * MA 02110-1301 USA
 + */
 +
 +#include common.h
 +#include i2c.h
 +#include nand.h
 +#include netdev.h
 +#include miiphy.h
 +#include asm/io.h
 +#include asm/arch/kirkwood.h
 +#include asm/arch/mpp.h
 +
 +#include ../common/common.h
 +
 +DECLARE_GLOBAL_DATA_PTR;
 +
 +static int  io_dev;
 +extern I2C_MUX_DEVICE *i2c_mux_ident_muxstring (uchar *buf);
 +
 +/* Multi-Purpose Pins Functionality configuration */
 +u32 kwmpp_config[] = {
 +MPP0_NF_IO2,
 +MPP1_NF_IO3,
 +MPP2_NF_IO4,
 +MPP3_NF_IO5,
 +MPP4_NF_IO6,
 +MPP5_NF_IO7,
 +MPP6_SYSRST_OUTn,
 +MPP7_PEX_RST_OUTn,
 +#if defined(CONFIG_SOFT_I2C)
 +MPP8_GPIO,  /* SDA */
 +MPP9_GPIO,  /* SCL */
 +#else
 
 you can remove else part and ifdef with CONFIG_HARD_I2C

But if this is in code, the user can fast switch to HARD_I2C,
if he wants it ... Ok, I delete it.

 +MPP8_TW_SDA,
 +MPP9_TW_SCK,
 +#endif
 
 ...snip...
 +int misc_init_r(void)
 +{
 +I2C_MUX_DEVICE  *i2cdev;
 +char *str;
 +int mach_type;
 +
 +/* add I2C Bus for I/O Expander */
 +i2cdev = i2c_mux_ident_muxstring((uchar *)pca9554a:70:a);
 +io_dev = i2cdev-busid;
 +puts(Piggy:);
 +if (ethernet_present() == 0)
 +puts ( not);
 +puts( present\n);
 +
 
 Please remove below stuff, this is risky and not needed

see comment for this from Stefan:

http://lists.denx.de/pipermail/u-boot/2010-February/067410.html

I got no comment from you, so I lthought it is Ok!

What should 

Re: [U-Boot] Atmel AT91 USB DFU and USB device port driver

2010-02-18 Thread Krishna Pattabiraman
2010/2/16 Remy Bohmer li...@bohmer.net

 Hi,

 2010/2/15 Krishna Pattabiraman krishna.li...@googlemail.com:
  Hi All,
 
  I am currently developing a USB DFU feature for at91sam9263 board. I have
 to
  develop the driver for USB device port.  I looked the uboot-usb git cdc
  branch it has a at91_udc.c driver it doesnt contain the udc_* api

 The only thing that is supported by this particular branch is to add
 CDC-ECM support to U-boot, thus make U-boot work as ethernet gadget
 over USB.


Hi Remy,

 I want to know if the cdc branch of the uboot-usb is a working one or still
in development. there were some compilation error when I enable the
CONFIG_USB_ETHER and GADGET_AT91. I am compiling it for at91sam9263-ek. But
also paralelly i brought it to the main tree as it is needed for me in my
work. and It compiles fine.

I disabled the CONFIG_MACB  in the header file and after that I dont know
what should I do to enable the usb0. Can you please give me some idea how to
bring up the usb ethernet interface in term of testing the functionality. in
my host machine there is no changes in dmesg and lsusb. Any help would be
great.

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


Re: [U-Boot] [PATCH] AT91: Added support for taskit Stamp9G20 and PortuxG20

2010-02-18 Thread Wolfgang Denk
Dear Achim Ehrlich,

In message 4b7c14ec.7050...@taskit.de you wrote:
 Signed-off-by: Achim Ehrlich aehrl...@taskit.de
 ---
...
 --- a/Makefile
 +++ b/Makefile
 @@ -2906,6 +2906,15 @@ TNY_A9260_config   :   unconfig
   @echo #define CONFIG_$(@:_config=) 1 $(obj)include/config.h
   @$(MKCONFIG) -a tny_a9260 arm arm926ejs tny_a9260 calao at91
  
 +portuxG20_config \
 +stamp9G20_config :   unconfig
 + @mkdir -p $(obj)include
 + @if [ $(findstring portux,$@) ] ; then \
 + echo #define CONFIG_PORTUXG20 1 $(obj)include/config.h; \
 + $(XECHO) ... PortuxG20;\
 + fi;

NAK. We don't accept such scripting in the Makefile any more. Please
use the available mkconfig features instead.


 diff --git a/board/taskit/stamp9G20/partition.c 
 b/board/taskit/stamp9G20/partition.c
 new file mode 100644
 index 000..2629c67
 --- /dev/null
 +++ b/board/taskit/stamp9G20/partition.c

So we are adding yet another copy of this file. We already have 12 of
these. Mostly identical.

Isn't there a more intelligent way to handle this?

...
 +int dram_init(void)
 +{
 + gd-bd-bi_dram[0].start = PHYS_SDRAM;
 + gd-bd-bi_dram[0].size = PHYS_SDRAM_SIZE;
 + return 0;
 +}

Please consider using get_ram_size() for memory auto-sizing and
testing.

 +#ifdef CONFIG_RESET_PHY_R
 +void reset_phy(void)
 +{
 +#ifdef CONFIG_MACB


CONFIG_MACB seems to be an undocumented variable. What is it supposed
to do?


 diff --git a/include/configs/stamp9G20.h b/include/configs/stamp9G20.h
 new file mode 100644
 index 000..5008554
 --- /dev/null
 +++ b/include/configs/stamp9G20.h
...
 +/* ARM asynchronous clock */
 +#define AT91_MAIN_CLOCK  18432000/* 18.432 MHz crystal */

This should be changed into a CONFIG_SYS_ (globally).


 +#define AT91_CPU_NAMEAT91SAM9G20

Ditto.

 +#define CONFIG_ATMEL_USART   1
 +#undef CONFIG_USART0
 +#undef CONFIG_USART1
 +#undef CONFIG_USART2
 +#define CONFIG_USART31   /* USART 3 is DBGU */

Do not undefine what is not defined anyway.

 +/* LED */
 +
 +#undef CONFIG_AT91_LED

Ditto.

 +/*
 + * Command line configuration.
 + */
 +#include config_cmd_default.h
 +#undef CONFIG_CMD_BDI
 +#undef CONFIG_CMD_FPGA
 +#undef CONFIG_CMD_IMI
 +#undef CONFIG_CMD_IMLS
 +#undef CONFIG_CMD_SOURCE

Is there any specific reason for disabling the bdi, imi, imls
and source commands? They are pretty useful, and you don't seem to
be especially short on resources...

...
 +/* USB */
 +#define CONFIG_USB_ATMEL
 +#define CONFIG_USB_OHCI_NEW  1
 +#define CONFIG_DOS_PARTITION 1
 +#define CONFIG_SYS_USB_OHCI_CPU_INIT 1
 +#define CONFIG_SYS_USB_OHCI_REGS_BASE0x0050  /* 
 AT91SAM9260_UHP_BASE */

Line too long. Please fix globally.

...
 + basicargs=console=ttyS0,115200 mem=64M\0  \
 + 
 +
 +
 +

Excessive white space. Drop 3 of the empty lines.

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
Do we define evil as the absence of goodness? It seems only  logical
that shit happens--we discover this by the process of elimination.
-- Larry Wall
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] AT91: Added support for taskit Stamp9G20 and PortuxG20

2010-02-18 Thread Detlev Zundel
Hi Achim,

 Signed-off-by: Achim Ehrlich aehrl...@taskit.de
 ---
  Makefile   |9 ++
  board/taskit/stamp9G20/Makefile|   56 ++
  board/taskit/stamp9G20/config.mk   |1 +
  board/taskit/stamp9G20/led.c   |   35 ++
  board/taskit/stamp9G20/partition.c |   40 +++
  board/taskit/stamp9G20/stamp9G20.c |  200 
 
  include/configs/stamp9G20.h|  197 +++
  7 files changed, 538 insertions(+), 0 deletions(-)
  create mode 100644 board/taskit/stamp9G20/Makefile
  create mode 100644 board/taskit/stamp9G20/config.mk
  create mode 100644 board/taskit/stamp9G20/led.c
  create mode 100644 board/taskit/stamp9G20/partition.c
  create mode 100644 board/taskit/stamp9G20/stamp9G20.c
  create mode 100644 include/configs/stamp9G20.h

 diff --git a/Makefile b/Makefile
 index 524b9da..ef1a7d4 100644
 --- a/Makefile
 +++ b/Makefile
 @@ -2906,6 +2906,15 @@ TNY_A9260_config   :   unconfig
   @echo #define CONFIG_$(@:_config=) 1 $(obj)include/config.h
   @$(MKCONFIG) -a tny_a9260 arm arm926ejs tny_a9260 calao at91
  
 +portuxG20_config \
 +stamp9G20_config :   unconfig
 + @mkdir -p $(obj)include
 + @if [ $(findstring portux,$@) ] ; then \
 + echo #define CONFIG_PORTUXG20 1 $(obj)include/config.h; \
 + $(XECHO) ... PortuxG20;\
 + fi;
 + @$(MKCONFIG) -a stamp9G20 arm arm926ejs stamp9G20 taskit at91
 +
  
  ## ARM Integrator boards - see doc/README-integrator for more info.
  integratorap_config  \
 diff --git a/board/taskit/stamp9G20/Makefile b/board/taskit/stamp9G20/Makefile
 new file mode 100644
 index 000..ef82428
 --- /dev/null
 +++ b/board/taskit/stamp9G20/Makefile
 @@ -0,0 +1,56 @@
 +#
 +# (C) Copyright 2003-2008
 +# Wolfgang Denk, DENX Software Engineering, w...@denx.de.
 +#
 +# (C) Copyright 2008
 +# Stelian Pop stelian@leadtechdesign.com
 +# Lead Tech Design www.leadtechdesign.com

Well actually I am not a fan of including all these copyrights in a file
which has so little content - so I'd vote to remove them and include
your own copyright.

But if for some reason the copyrights are not removed, then please at
least add yourself in.


[...]

 diff --git a/board/taskit/stamp9G20/led.c b/board/taskit/stamp9G20/led.c
 new file mode 100644
 index 000..dc6ac63
 --- /dev/null
 +++ b/board/taskit/stamp9G20/led.c
 @@ -0,0 +1,35 @@
 +/*
 + * (C) Copyright 2007-2008
 + * Stelian Pop stelian@leadtechdesign.com
 + * Lead Tech Design www.leadtechdesign.com

Dito.

 + *
 + * 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 common.h
 +#include asm/arch/at91sam9260.h
 +#include asm/arch/at91_pmc.h
 +#include asm/arch/gpio.h
 +#include asm/arch/io.h
 +
 +void coloured_LED_init(void)
 +{
 + /* Enable clock */
 + at91_sys_write(AT91_PMC_PCER, 1  AT91SAM9260_ID_PIOC);
 +}
 diff --git a/board/taskit/stamp9G20/partition.c 
 b/board/taskit/stamp9G20/partition.c
 new file mode 100644
 index 000..2629c67
 --- /dev/null
 +++ b/board/taskit/stamp9G20/partition.c
 @@ -0,0 +1,40 @@
 +/*
 + * (C) Copyright 2008
 + * Ulf Samuelsson u...@atmel.com

Dito.

 diff --git a/board/taskit/stamp9G20/stamp9G20.c 
 b/board/taskit/stamp9G20/stamp9G20.c
 new file mode 100644
 index 000..28cd8b1
 --- /dev/null
 +++ b/board/taskit/stamp9G20/stamp9G20.c
 @@ -0,0 +1,200 @@
 +/*
 + * (C) Copyright 2007-2008
 + * Stelian Pop stelian@leadtechdesign.com
 + * Lead Tech Design www.leadtechdesign.com
 + *
 + * Achim Ehrlich aehrl...@taskit.de
 + * taskit GmbH www.taskit.de

Ok, this time you added your name - although I would have expected a
year.

Cheers
  Detlev

-- 
Infolge sklavischer Kategoritis  werden die wissenschaftlichen unlogischen
und - wie wir sehen werden -  oft sinnlosen Fragen wie Wo wohnst du?, Was
bist du?, Welche Religion?, Welche Rasse?, Welche Nationalität? alle-
samt heute für logische Fragen gehalten.   -- Richard Buckminster Fuller
--
DENX Software Engineering GmbH,  MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: 

Re: [U-Boot] [PATCH 1/1 v5] arm: add support for the suen3 board from keymile

2010-02-18 Thread Wolfgang Denk
Dear Heiko Schocher,

In message 4b7d06dc.8020...@denx.de you wrote:
...
   create mode 100644 include/configs/km_arm.h
   create mode 100644 include/configs/suen3.h
  
  It would be good if you can merge these two files since single board board 
  support
  For additional board that you are planning, you can even add separate file 
  or think of splitting it.
  For better readability and to avoid complexity, I recommend to have one 
  single file per board in include/config
 
 Hmm.. I think we tend to collect common config options in
 one file, so board config files are not so big. Ok, actual
 it is only one arm board from keymile here, but there
 come more, and so it is sensible to collect common
 options for this group of boards immediately ...

ACK. I agree to do this split right from the beginning if it is clear
that more such boards will be added soon.

  +u32 kwmpp_config[] = {
  +  MPP0_NF_IO2,
  +  MPP1_NF_IO3,
  +  MPP2_NF_IO4,
  +  MPP3_NF_IO5,
  +  MPP4_NF_IO6,
  +  MPP5_NF_IO7,
  +  MPP6_SYSRST_OUTn,
  +  MPP7_PEX_RST_OUTn,
  +#if defined(CONFIG_SOFT_I2C)
  +  MPP8_GPIO,  /* SDA */
  +  MPP9_GPIO,  /* SCL */
  +#else
  
  you can remove else part and ifdef with CONFIG_HARD_I2C
 
 But if this is in code, the user can fast switch to HARD_I2C,
 if he wants it ... Ok, I delete it.

I understood that Prafulla NOT suggested delete the code, but to
change the

#else
into
#endif
#ifdef CONFIG_HARD_I2C

...
  Please remove below stuff, this is risky and not needed
 
 see comment for this from Stefan:
 
 http://lists.denx.de/pipermail/u-boot/2010-February/067410.html
 
 I got no comment from you, so I lthought it is Ok!
 
 What should I do?

Keep it. It really doen't hurt. This whole MACH_TYPE stuff is broken
by design anyway. So adding such workarounds are just collateral
damages and can be ignored for now. Eventually we can get rid of this
when we switch to using device trees.


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 list is only as strong as its weakest link.-- Don Knuth
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 1/1 v5] arm: add support for the suen3 board from keymile

2010-02-18 Thread Prafulla Wadaskar
 

 -Original Message-
 From: Wolfgang Denk [mailto:w...@denx.de] 
 Sent: Thursday, February 18, 2010 6:12 PM
 To: h...@denx.de
 Cc: Prafulla Wadaskar; U-Boot user list; Scott Wood; Stefan Roese
 Subject: Re: [PATCH 1/1 v5] arm: add support for the suen3 
 board from keymile
 
 Dear Heiko Schocher,
 
 In message 4b7d06dc.8020...@denx.de you wrote:
...snip...
   +MPP7_PEX_RST_OUTn,
   +#if defined(CONFIG_SOFT_I2C)
   +MPP8_GPIO,  /* SDA */
   +MPP9_GPIO,  /* SCL */
   +#else
   
   you can remove else part and ifdef with CONFIG_HARD_I2C
  
  But if this is in code, the user can fast switch to HARD_I2C,
  if he wants it ... Ok, I delete it.
 
 I understood that Prafulla NOT suggested delete the code, but to
 change the

Ack

 
   #else
 into
   #endif
   #ifdef CONFIG_HARD_I2C
 

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


Re: [U-Boot] Boot based on I2C EEPROM value

2010-02-18 Thread Detlev Zundel
Hi Srivatsan,

 The modem I am working on contains two software images in flash memory:

 1.Default image
 2.Backup image

  

 Typically, the Default image will get loaded. If the application
 software determines that the Default image is causing issues, it will
 set a value of 1 in an address in the I2C EEPROM and reboot the board.

  

 So, U-boot will need to read this EEPROM value and determine whether it
 should boot the Default or Backup image i.e.

  

 if EEPROM data == 0

 bootm $default_addr

 else 

 bootm $backup_addr

  

 How can I read and parse the EEPROM data from U-boot? 

  

 U-boot has the facility to read the I2C device from the command line as
 in (EEPROM dev num = 50, address = 2, count = 1):

 = i2c md 50 2 1

 0002: 01.

  

 How do I parse this from within a U-boot command that can be run at
 boot-up?

Ah, this is interesting.  I was about to tell you to use the standard
i2c commands to transfer data from the chip into memory and then use the
hush shell and the standard itest *addr = 1 facility.  Unfortunately,
the i2c subsystem does not implement such functionality.  So currently
I would advise to implement such read and write operations to
transfer to/from memory.

With this addition, you could even propose a patch to remove crc32
subcommand from i2c as this can then be done in two steps.  So maybe you
can even get away with this addition without adding anything to the
binary size of U-Boot ;)

Then itest would work like this:

= md.b 0x20 1
0020: 00.
= if itest *0x20 -eq 0 ; then echo zero;else echo not zero; fi
zero
= mw.b 0x20 1 1
= md.b 0x20 1 
0020: 01.
= if itest *0x20 -eq 0 ; then echo zero;else echo not zero; fi
not zero
= 

Cheers
  Detlev

-- 
An elephant is a mouse with an operating system.
-- Donald Knuth
--
DENX Software Engineering GmbH,  MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: d...@denx.de
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Boot based on I2C EEPROM value

2010-02-18 Thread Detlev Zundel
Hi again,

 Ah, this is interesting.  I was about to tell you to use the standard
 i2c commands to transfer data from the chip into memory and then use the
 hush shell and the standard itest *addr = 1 facility.  Unfortunately,
 the i2c subsystem does not implement such functionality.  So currently
 I would advise to implement such read and write operations to
 transfer to/from memory.
 
 With this addition, you could even propose a patch to remove crc32
 subcommand from i2c as this can then be done in two steps.  So maybe you
 can even get away with this addition without adding anything to the
 binary size of U-Boot ;)
 
In the meantime I found the eeprom command, which should allow you to
transfer the data into memory.  Still I believe the proposed
additions/removals to the i2c subsystem to be worthwhile.

Cheers
  Detlev

-- 
Microsoft gives you windows, Linux gives you the whole house.
--
DENX Software Engineering GmbH,  MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: d...@denx.de
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v6] arm: add support for the suen3 board from keymile

2010-02-18 Thread Heiko Schocher
Add support for the ARM part of the mgcoge2, named suen3.
This board is based on the Marvell Kirkwood (88F6281) SoC.
As there come more board variants, common config options
are collected in include/configs/km_arm.h. Also, this board
use common code for all keymile boards, which is stored in
board/keymile/common/common.c

Signed-off-by: Holger Brunck holger.bru...@keymile.com
Signed-off-by: Stefan Roese s...@denx.de
Signed-off-by: Heiko Schocher h...@denx.de

---
- changes since v1:
  added comments from Wolfgang Denk:
  get rid of flash_info_t define in board config
  (to get this working patch 1/2 is introduced/needed)

- changes since v2:
  added comments from Wolfgang Denk
  - rearranged if/else in do_spi_toggle()
  - added I/O accessor functions for bootcounter

- changes since v3:
  added comment Scott Wood
  - removed nand_init in do_spi_toggle()

  added comments from Prafulla Wadaskar
  - km-arm.h renamed to km_arm.h
  - reworked eeprom_write_enable() (deleted it)
(when reviewing this function, it cropped up, that
 this pin is connected through a gpio pin, not as
 in previous version, through the boco (a FPGA))
  - moved set_sda(), set_scl(), get_sda(), get_scl()
to km_arm.c
  - split patch in 4 patches (for each board an extra patch)
  - renamed sdramregs.txt in kwbimage.cfg, also license
info added.

- changes since v4
  added comments from Prafulla Wadaskar
  - add comments in board/keymile/km_arm/kwbimage.cfg
  - cleanup define CONFIG_KIRKWOOD_GPIO
  - corect wrong name at end of config file
  - add only support for the suen3 target, others
follow later

- changes since v5
  #ifdef CONFIG_HARD_I2C instead of #else

 MAINTAINERS   |2 +-
 MAKEALL   |1 +
 Makefile  |3 +
 board/keymile/common/common.c |6 +-
 board/keymile/km_arm/Makefile |   51 ++
 board/keymile/km_arm/config.mk|   28 
 board/keymile/km_arm/km_arm.c |  324 +
 board/keymile/km_arm/kwbimage.cfg |  176 
 include/configs/km_arm.h  |  191 ++
 include/configs/suen3.h   |  103 
 10 files changed, 882 insertions(+), 3 deletions(-)
 create mode 100644 board/keymile/km_arm/Makefile
 create mode 100644 board/keymile/km_arm/config.mk
 create mode 100644 board/keymile/km_arm/km_arm.c
 create mode 100644 board/keymile/km_arm/kwbimage.cfg
 create mode 100644 include/configs/km_arm.h
 create mode 100644 include/configs/suen3.h

diff --git a/MAINTAINERS b/MAINTAINERS
index fe63a9f..fe9494c 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -409,9 +409,9 @@ Heiko Schocher h...@denx.de
muas3001MPC8270
municse MPC5200
sc3 PPC405GP
+   suen3   ARM926EJS (Kirkwood SoC)
uc101   MPC5200

-
 Peter De Schrijver p...@mind.be

ML2 PPC4xx
diff --git a/MAKEALL b/MAKEALL
index 684db67..dfac5e4 100755
--- a/MAKEALL
+++ b/MAKEALL
@@ -579,6 +579,7 @@ LIST_ARM9= \
spear310\
spear320\
spear600\
+   suen3   \
trab\
VCMA9   \
versatile   \
diff --git a/Makefile b/Makefile
index e4431f7..dc609c0 100644
--- a/Makefile
+++ b/Makefile
@@ -3047,6 +3047,9 @@ spear320_config : unconfig
 spear600_config :  unconfig
@$(MKCONFIG) -n $@ -t $(@:_config=) spear6xx arm arm926ejs 
$(@:_config=) spear spear

+suen3_config:  unconfig
+   @$(MKCONFIG) $(@:_config=) arm arm926ejs km_arm keymile kirkwood
+
 SX1_stdout_serial_config \
 SX1_config:unconfig
@mkdir -p $(obj)include
diff --git a/board/keymile/common/common.c b/board/keymile/common/common.c
index ec27bda..7b4eefd 100644
--- a/board/keymile/common/common.c
+++ b/board/keymile/common/common.c
@@ -35,6 +35,7 @@
 #include libfdt.h
 #endif

+#include ../common/common.h
 #if defined(CONFIG_HARD_I2C) || defined(CONFIG_SOFT_I2C)
 #include i2c.h

@@ -421,7 +422,6 @@ static int get_scl (void)

return ((val  SCL_BIT) == SCL_BIT);
 }
-
 #endif

 #if !defined(CONFIG_KMETER1)
@@ -500,7 +500,7 @@ void i2c_init_board(void)
out_8 (dev-cr, (I2C_CR_MEN));

 #else
-#if defined(CONFIG_HARD_I2C)
+#if defined(CONFIG_HARD_I2C)  !defined(CONFIG_MACH_SUEN3)
volatile immap_t *immap = (immap_t *)CONFIG_SYS_IMMR ;
volatile i2c8260_t *i2c = (i2c8260_t *)immap-im_i2c;

@@ -578,10 +578,12 @@ int fdt_get_node_and_value (void *blob,
 }
 #endif

+#if !defined(CONFIG_MACH_SUEN3)
 int ethernet_present (void)
 {
return (in_8((u8 *)CONFIG_SYS_PIGGY_BASE + CONFIG_SYS_SLOT_ID_OFF)  
0x80);
 }
+#endif

 int board_eth_init (bd_t *bis)
 {
diff --git a/board/keymile/km_arm/Makefile b/board/keymile/km_arm/Makefile
new file mode 100644
index 000..fffc6a3
--- /dev/null
+++ 

Re: [U-Boot] Atmel AT91 USB DFU and USB device port driver

2010-02-18 Thread Remy Bohmer
Hi,

  I want to know if the cdc branch of the uboot-usb is a working one

It is a working one, I use it myself already a long time, but it needs
some rework to make it mainline ready/clean, I have not put much time
in it lately...

 or still in development. there were some compilation error when I enable the

What kind of error?

 CONFIG_USB_ETHER and GADGET_AT91. I am compiling it for at91sam9263-ek.

The tree is ready for the 9261-ek so the 9263-ek is a close match, but
it might lack some configuration code.

 But also paralelly i brought it to the main tree as it is needed for me in my
 work. and It compiles fine.
 I disabled the CONFIG_MACB  in the header file and after that I dont know
 what should I do to enable the usb0. Can you please give me some idea how to
 bring up the usb ethernet interface in term of testing the functionality. in
 my host machine there is no changes in dmesg and lsusb. Any help would be
 great.

It behaves like any other network adapter in U-boot. It will only be
brought up and thus visible to the host if some network action is
active, like e.g. a tftp get/put command.  Once that action is
completed, the connection is dropped again, and the pullup to the host
deactivated what makes the device invisible for the host. If there are
more network drivers available you must first make the usb0 the active
adapter.

Kind regards,

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


Re: [U-Boot] AmigaOneG3SE

2010-02-18 Thread Bill Toner
There's a small group forming to work on updating the MAI/AmigaOneG3SE board
build for u-boot.

http://amigaworld.net/modules/newbb/viewtopic.php?mode=viewtopictopic_id=30736forum=15start=0viewmode=flatorder=0

I'm so far unable to get a complete make run through. Strangely, I'm getting
an error that there is no .depend target for the _depend to trigger. but the
Makefile does include rules.mk same as the sbc8641d Makefile does, and that
one seems to build fine, so I'm using that for comparison. If I add a second
name to the .depend target
.depend $(obj).depend:
then it does give a message that two identical names for that target are
present, so I believe the rules.mk is being included. I'm struggling to
understand this issue.

We've also requested sources from the binary image distributor and are
waiting a response. There's an image in my board that works, so someone was
able to compile something at some point in the past... I'll try getting that
version from git as well and see if it builds that way while we wait for an
answer.

If there's any advice available for someone learning their way around u-boot
sources and makefiles, we'd appreciate it.



--

Hi all,
I just tried compiling u-boot for the AmigaOneG3SE during testing of a
code change and ran into a number of compiler warnings followed by a
compiler error.

CONFIG_AMIGAONEG3SE is also found in a lot of common files:
common/cmd_bootm.c:#ifdef CONFIG_AMIGAONEG3SE
common/cmd_fdc.c:#ifdef CONFIG_AMIGAONEG3SE
common/cmd_fdc.c:#ifndef CONFIG_AMIGAONEG3SE
common/cmd_fdc.c:#ifndef CONFIG_AMIGAONEG3SE
common/cmd_fdc.c:#ifdef CONFIG_AMIGAONEG3SE
common/cmd_fdc.c:#ifdef CONFIG_AMIGAONEG3SE
common/cmd_fdc.c:#ifdef CONFIG_AMIGAONEG3SE
common/cmd_fdc.c:#ifdef CONFIG_AMIGAONEG3SE
common/cmd_ide.c:#ifndef CONFIG_AMIGAONEG3SE
common/cmd_ide.c:#ifndef CONFIG_AMIGAONEG3SE
common/cmd_ide.c:#if defined(CONFIG_AMIGAONEG3SE) || defined(CONFIG_SC3)
common/cmd_ide.c:#ifdef CONFIG_AMIGAONEG3SE
common/cmd_ide.c:#ifndef CONFIG_AMIGAONEG3SE
common/cmd_ide.c:#if defined(CONFIG_AMIGAONEG3SE) || defined(CONFIG_SC3)
common/cmd_ide.c:#if defined(CONFIG_AMIGAONEG3SE) || defined(CONFIG_SC3)
common/cmd_ide.c:#ifdef CONFIG_AMIGAONEG3SE
common/cmd_ide.c:#ifdef CONFIG_AMIGAONEG3SE
common/cmd_ide.c:#ifdef CONFIG_AMIGAONEG3SE
common/cmd_ide.c:#ifdef CONFIG_AMIGAONEG3SE
common/cmd_ide.c:#ifdef CONFIG_AMIGAONEG3SE
common/cmd_ide.c:   !defined(CONFIG_AMIGAONEG3SE) \
common/cmd_ide.c:#ifdef CONFIG_AMIGAONEG3SE
common/cmd_ide.c:#ifdef CONFIG_AMIGAONEG3SE
common/cmd_ide.c:#ifdef CONFIG_AMIGAONEG3SE
common/cmd_ide.c:#ifdef CONFIG_AMIGAONEG3SE
common/cmd_ide.c:#endif /* CONFIG_AMIGAONEG3SE */
common/cmd_nvedit.c:#ifdef CONFIG_AMIGAONEG3SE
common/cmd_nvedit.c:#endif  /* CONFIG_AMIGAONEG3SE */
common/console.c:#ifdef CONFIG_AMIGAONEG3SE
common/env_common.c:#ifdef CONFIG_AMIGAONEG3SE
common/env_common.c:#ifdef CONFIG_AMIGAONEG3SE
common/env_common.c:#ifdef CONFIG_AMIGAONEG3SE
common/env_common.c:#ifdef CONFIG_AMIGAONEG3SE
common/env_nvram.c:#ifdef CONFIG_AMIGAONEG3SE
common/env_nvram.c:#ifdef CONFIG_AMIGAONEG3SE
common/env_nvram.c:#ifdef CONFIG_AMIGAONEG3SE
common/env_nvram.c:#ifdef CONFIG_AMIGAONEG3SE
common/env_nvram.c:#ifdef CONFIG_AMIGAONEG3SE
common/main.c:#ifdef CONFIG_AMIGAONEG3SE
cpu/74xx_7xx/cpu.c:#ifdef CONFIG_AMIGAONEG3SE
cpu/74xx_7xx/cpu.c:#if defined(CONFIG_AMIGAONEG3SE) ||
defined(CONFIG_SYS_CONFIG_BUS_CLK)
cpu/74xx_7xx/cpu.c:#else/* ! CONFIG_AMIGAONEG3SE and
!CONFIG_SYS_CONFIG_BUS_CLK*/
cpu/74xx_7xx/cpu.c:#endif   /* CONFIG_AMIGAONEG3SE or
CONFIG_SYS_CONFIG_BUS_CLK*/
cpu/74xx_7xx/interrupts.c:#if defined(DEBUG)  !defined(CONFIG_AMIGAONEG3SE)
cpu/74xx_7xx/speed.c:#ifdef CONFIG_AMIGAONEG3SE
cpu/74xx_7xx/start.S:#if defined(CONFIG_AMIGAONEG3SE) || \
cpu/74xx_7xx/traps.c:#ifdef CONFIG_AMIGAONEG3SE
cpu/74xx_7xx/traps.c:#ifdef CONFIG_AMIGAONEG3SE
drivers/rtc/mc146818.c:#ifdef CONFIG_AMIGAONEG3SE
drivers/rtc/mc146818.c:#ifdef CONFIG_AMIGAONEG3SE
drivers/rtc/mk48t59.c:#elif defined(CONFIG_AMIGAONEG3SE)
include/asm-ppc/global_data.h:#ifdef CONFIG_AMIGAONEG3SE
lib_ppc/board.c:#ifdef CONFIG_AMIGAONEG3SE

Anyway, just wanted to drop a line that the build appears to be broken,
and drop a hint that it would be great if someone (Thomas?) would clean
up the Amiga #ifdefs that currently are litter a number of common files.
Is the AmigaOne port being maintained?  Does anyone use it?  Can it be
removed?:)

It looks like someone else brought this up a while back:
http://lists.denx.de/pipermail/u-boot/2006-January/013030.html

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


[U-Boot] iMX51 - help on running u-boot out of SPI NOR flash

2010-02-18 Thread Carlo McKee
Hello,

I would like to know if someone knows if it is possible to run u-boot out of
SPI NOR flash on the iMX51. If so, how to do it?  I am pretty much a
beginner.

Thanks for the help

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


[U-Boot] Taskit Portux920T port - Some issues

2010-02-18 Thread Luca Lazzarin
Hi, we are Enrico  Luca.

We have tried to port u-boot to a Portux920T MiniPC.
It seems we are successful, in fact u-boot brings up the device with no
error and can boot a linux kernel.
Ethernet driver works, dhcp works and major problems are present.

The only thing we can't get right is flash programming. We are trying to use
CFI flash driver that shows evident timing issues, at least in our opinion.
When running saveenv more than once we get the typical Flash not erased
error, while running cp.b (for example) does not give us any problem.
Running a bootm kerneladdr initaddr command it frequently gives us a No
boot image - CRC error, but running it another time, without rebooting the
machine, results in a successful boot. We ended up thinking that is a CFI
driver bug, but we know we could easily be wrong, so we include our config
in this message.
Our flash chip is a AMD MIRROR  S29L128N 128Mbit - AMD (Spansion) (227e) -
16MB.

P.S.: we have tried to disable buffered writing (as suggested for example in
the voiceblue.h file, with no success)

/*
 * Ulf Samuelsson u...@atmel.com
 * Rick Bronson r...@efn.org
 *
 * Configuration settings for the AT91RM9200EK board.
 *
 * 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
 */

#ifndef __CONFIG_H
#define __CONFIG_H

/* ARM asynchronous clock */
/*
 * from 18.432 MHz crystal
 * (18432000 / 4 * 39)
 */
#define AT91C_MAIN_CLOCK 179712000
/*
 * peripheral clock
 * (AT91C_MASTER_CLOCK / 3)
 */
#define AT91C_MASTER_CLOCK 59904000

#define AT91_SLOW_CLOCK 32768 /* slow clock */

#define CONFIG_ARM920T 1 /* This is an ARM920T Core */
#define CONFIG_AT91RM9200 1 /* It's an Atmel AT91RM9200 SoC */
#define CONFIG_AT91RM9200EK 1 /* on an AT91RM9200EK Board */
#undef  CONFIG_USE_IRQ /* we don't need IRQ/FIQ stuff */
#define USE_920T_MMU 1
/*
  Those following defines are needed to support latest kernel - don't
  bother...
*/
#define CONFIG_CMDLINE_TAG 1 /* enable passing of ATAGs */
#define CONFIG_SETUP_MEMORY_TAGS 1
#define CONFIG_INITRD_TAG 1 /* and remember to mkimage initrd */

/*
 * LowLevel Init
 */
#ifndef CONFIG_SKIP_LOWLEVEL_INIT
#define CONFIG_SYS_USE_MAIN_OSCILLATOR 1
/* flash */
#define CONFIG_SYS_EBI_CFGR_VAL 0x
#define CONFIG_SYS_SMC_CSR0_VAL 0x3284 /* 16bit, 2 TDF, 4 WS */

/* clocks */
#define CONFIG_SYS_PLLAR_VAL 0x20263E04 /* 179.712000 MHz for PCK */
#define CONFIG_SYS_PLLBR_VAL 0x10483E0E /* 48.054857 MHz (divider by 2 for
USB) */
/* PCK/3 = MCK Master Clock = 59.904000MHz from PLLA */
#define CONFIG_SYS_MCKR_VAL 0x0202

/* sdram */
#define CONFIG_SYS_PIOC_ASR_VAL 0x /* Configure PIOC as peripheral
(D16/D31) */
#define CONFIG_SYS_PIOC_BSR_VAL 0x
#define CONFIG_SYS_PIOC_PDR_VAL 0x
#define CONFIG_SYS_EBI_CSA_VAL 0x0002 /* CS1=CONFIG_SYS_SDRAM */
#define CONFIG_SYS_SDRC_CR_VAL 0x2188c155 /* set up the CONFIG_SYS_SDRAM */
#define CONFIG_SYS_SDRAM 0x2000 /* address of the CONFIG_SYS_SDRAM */
#define CONFIG_SYS_SDRAM1 0x2080 /* address of the CONFIG_SYS_SDRAM */
#define CONFIG_SYS_SDRAM_VAL 0x /* value written to CONFIG_SYS_SDRAM
*/
#define CONFIG_SYS_SDRC_MR_VAL 0x0002 /* Precharge All */
#define CONFIG_SYS_SDRC_MR_VAL1 0x0004 /* refresh */
#define CONFIG_SYS_SDRC_MR_VAL2 0x0003 /* Load Mode Register */
#define CONFIG_SYS_SDRC_MR_VAL3 0x /* Normal Mode */
#define CONFIG_SYS_SDRC_TR_VAL 0x02E0 /* Write refresh rate */
#else
#define CONFIG_SKIP_RELOCATE_UBOOT
#endif /* CONFIG_SKIP_LOWLEVEL_INIT */

/* hardcode so no __divsi3 : AT91C_MASTER_CLOCK / baudrate / 16 */
#define CONFIG_SYS_AT91C_BRGR_DIVISOR 33

/*
 * Memory Configuration
 */
#define CONFIG_NR_DRAM_BANKS 1
#define PHYS_SDRAM 0x2000
#define PHYS_SDRAM_SIZE 0x0400 /* 64 megs */

#define CONFIG_SYS_MEMTEST_START PHYS_SDRAM
#define CONFIG_SYS_MEMTEST_END \
 (CONFIG_SYS_MEMTEST_START + PHYS_SDRAM_SIZE - 262144)

/*
 * Hardware drivers
 */

/*
 * UART Configuration
 *
 * define one of these to choose the DBGU,
 * USART0 or USART1 as console
 */
#define CONFIG_AT91RM9200_USART
#define CONFIG_DBGU
#undef CONFIG_USART0
#undef CONFIG_USART1
/* don't include RTS/CTS flow control support */
#undef CONFIG_HWFLOW
/* disable modem initialization stuff */
#undef CONFIG_MODEM_SUPPORT

#define 

Re: [U-Boot] iMX51 - help on running u-boot out of SPI NOR flash

2010-02-18 Thread Stefano Babic
Carlo McKee wrote:
 Hello,
 
 I would like to know if someone knows if it is possible to run u-boot out of
 SPI NOR flash on the iMX51. If so, how to do it?  I am pretty much a
 beginner.

Yes, it works. The patchset I sent to the ML for the mx51evk supports
u-boot loaded on SPI NOR or SD card. The processor can boot from any of
these devices.

Regards,
Stefano Babic

-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: off...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 1/1] ppc4xx: add support for alternate format for ndfc

2010-02-18 Thread fkan
From: Feng Kan f...@amcc.com

This is to lock down the ordering in the correction routine against
the calculate routine. Otherwise, incorrect define would cause ECC errors.

Signed-off-by: Feng Kan f...@amcc.com
Acked-by: Victor Gallardo vgalla...@amcc.com
---
 drivers/mtd/nand/ndfc.c |6 ++
 1 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/drivers/mtd/nand/ndfc.c b/drivers/mtd/nand/ndfc.c
index 0dd6789..88e341d 100644
--- a/drivers/mtd/nand/ndfc.c
+++ b/drivers/mtd/nand/ndfc.c
@@ -89,9 +89,15 @@ static int ndfc_calculate_ecc(struct mtd_info *mtdinfo,
 
/* The NDFC uses Smart Media (SMC) bytes order
 */
+#ifdef CONFIG_MTD_NAND_ECC_SMC
ecc_code[0] = p[1];
ecc_code[1] = p[2];
ecc_code[2] = p[3];
+#else
+   ecc_code[0] = p[2];
+   ecc_code[1] = p[1];
+   ecc_code[2] = p[3];
+#endif
 
return 0;
 }
-- 
1.5.5

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


Re: [U-Boot] Boot based on I2C EEPROM value

2010-02-18 Thread Canchivaram, Srivatsan
Hi Detlev,

Thanks for the response. I was able to use the 'eeprom' command to read
from the I2C device and copy to memory. 
With the itest.b command, the comparison always resulted in false even
when the condition itself was true i.e.

mw.b 0x20 1
if itest.b *0x20 -eq 1
This would result as false.

Through debug of the itest code, I found out the reason to be
endianness. My target is the PowerPC. Therefore, the itest.b's read of
address *20 always returned Byte 3 instead of Byte 0. 

I am able to workaround this issue by forcing the itest function to read
the correct byte. Do you know if there is any patch available for Big
Endian in any of the recent versions? Or is this something that needs to
be added to the source code? I am using version 1.3.3.

Thanks,
Srivatsan

-Original Message-
From: Detlev Zundel [mailto:d...@denx.de] 
Sent: Thursday, February 18, 2010 7:53 AM
To: Canchivaram, Srivatsan
Cc: u-boot@lists.denx.de
Subject: Re: [U-Boot] Boot based on I2C EEPROM value

Hi Srivatsan,

 The modem I am working on contains two software images in flash
memory:

 1.Default image
 2.Backup image

  

 Typically, the Default image will get loaded. If the application
 software determines that the Default image is causing issues, it will
 set a value of 1 in an address in the I2C EEPROM and reboot the board.

  

 So, U-boot will need to read this EEPROM value and determine whether
it
 should boot the Default or Backup image i.e.

  

 if EEPROM data == 0

 bootm $default_addr

 else 

 bootm $backup_addr

  

 How can I read and parse the EEPROM data from U-boot? 

  

 U-boot has the facility to read the I2C device from the command line
as
 in (EEPROM dev num = 50, address = 2, count = 1):

 = i2c md 50 2 1

 0002: 01.

  

 How do I parse this from within a U-boot command that can be run at
 boot-up?

Ah, this is interesting.  I was about to tell you to use the standard
i2c commands to transfer data from the chip into memory and then use the
hush shell and the standard itest *addr = 1 facility.  Unfortunately,
the i2c subsystem does not implement such functionality.  So currently
I would advise to implement such read and write operations to
transfer to/from memory.

With this addition, you could even propose a patch to remove crc32
subcommand from i2c as this can then be done in two steps.  So maybe you
can even get away with this addition without adding anything to the
binary size of U-Boot ;)

Then itest would work like this:

= md.b 0x20 1
0020: 00.
= if itest *0x20 -eq 0 ; then echo zero;else echo not zero; fi
zero
= mw.b 0x20 1 1
= md.b 0x20 1 
0020: 01.
= if itest *0x20 -eq 0 ; then echo zero;else echo not zero; fi
not zero
= 

Cheers
  Detlev

-- 
An elephant is a mouse with an operating system.
-- Donald Knuth
--
DENX Software Engineering GmbH,  MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: d...@denx.de
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 1/1] ppc4xx: add support for alternate format for ndfc

2010-02-18 Thread Wolfgang Denk
Dear f...@amcc.com,

In message 1266531913-20756-1-git-send-email-f...@amcc.com you wrote:
 From: Feng Kan f...@amcc.com
 
 This is to lock down the ordering in the correction routine against
 the calculate routine. Otherwise, incorrect define would cause ECC errors.
 
 Signed-off-by: Feng Kan f...@amcc.com
 Acked-by: Victor Gallardo vgalla...@amcc.com
 ---
  drivers/mtd/nand/ndfc.c |6 ++
  1 files changed, 6 insertions(+), 0 deletions(-)
 
 diff --git a/drivers/mtd/nand/ndfc.c b/drivers/mtd/nand/ndfc.c
 index 0dd6789..88e341d 100644
 --- a/drivers/mtd/nand/ndfc.c
 +++ b/drivers/mtd/nand/ndfc.c
 @@ -89,9 +89,15 @@ static int ndfc_calculate_ecc(struct mtd_info *mtdinfo,
  
   /* The NDFC uses Smart Media (SMC) bytes order
*/
 +#ifdef CONFIG_MTD_NAND_ECC_SMC
   ecc_code[0] = p[1];
   ecc_code[1] = p[2];
   ecc_code[2] = p[3];
 +#else
 + ecc_code[0] = p[2];
 + ecc_code[1] = p[1];
 + ecc_code[2] = p[3];
 +#endif

This patch seems wrong to me as CONFIG_MTD_NAND_ECC_SMC is nowhere
defined.  [Also, it's not documented anywhere.]

If this is fixing a bug, then please describe the exact problem, how
to reproduce it, and how this patch is supposed to fix this problem.

As is, this makes no sense to me.


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
Don't panic.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Boot based on I2C EEPROM value

2010-02-18 Thread Wolfgang Denk
Dear Canchivaram, Srivatsan,

In message e3b2c7c726b07c4fa70ffaed2c8a2e6abff...@owa.usmonolithics.com you 
wrote:
 
 Thanks for the response. I was able to use the 'eeprom' command to read
 from the I2C device and copy to memory. 
 With the itest.b command, the comparison always resulted in false even
 when the condition itself was true i.e.
 
 mw.b 0x20 1
 if itest.b *0x20 -eq 1
 This would result as false.
 
 Through debug of the itest code, I found out the reason to be
 endianness. My target is the PowerPC. Therefore, the itest.b's read of
 address *20 always returned Byte 3 instead of Byte 0. 

Argh...

 I am able to workaround this issue by forcing the itest function to read
 the correct byte. Do you know if there is any patch available for Big
 Endian in any of the recent versions? Or is this something that needs to
 be added to the source code? I am using version 1.3.3.

This is a bug still present in current code. evalexp() in
common/cmd_itest.c does not pay atention to any .b or .w
modifiers.

This needs to be fixed. Patches are welcome!

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
panic: can't find /
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] extracting u-boot version from Linux

2010-02-18 Thread Parks, Derek
Hello all,

 

Newbie to u-boot here!  I have read basic user and command manuals and
could not find a solution:

 

I have an c++ application that needs to display various versions of
system components; u-boot, kernel, O/S flavor, hardware and software
versions.  I'm struggling with getting the u-boot version and ask for
your help.  My ideal situation is to have U-boot report (somehow) to
Linux it's version number and have Linux store it in environment
variables.  Any thoughts?

 

Thanks!!

 

 

 

 

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


Re: [U-Boot] [PATCH 1/1] ppc4xx: add support for alternate format for ndfc

2010-02-18 Thread Feng Kan
Dear Wolfgang:

The problem goes back a bit. The ordering you see in the ndfc file has been 
changed a
few times, back and forth and cause quite a bit of problem. The define we speak 
of is
in the driver/mtd/nand/nand_ecc.c file. The nand_correct_data function uses two 
ways
of check ECC correctness. However the ndfc calculate only supports one 
ordering, although
both placement method in the patch would work. It also serves to nail down the 
ordering
depending on the define is used or not.

There is also the following in the code, should you agree, this will also need 
to be removed
as well.

/* The PPC4xx NDFC uses Smart Media (SMC) bytes order */
#ifdef CONFIG_NAND_NDFC
#define CONFIG_MTD_NAND_ECC_SMC
#endif


Feng Kan

On 02/18/2010 03:13 PM, Wolfgang Denk wrote:
 Dear f...@amcc.com,

 In message1266531913-20756-1-git-send-email-f...@amcc.com  you wrote:
 From: Feng Kanf...@amcc.com

 This is to lock down the ordering in the correction routine against
 the calculate routine. Otherwise, incorrect define would cause ECC errors.

 Signed-off-by: Feng Kanf...@amcc.com
 Acked-by: Victor Gallardovgalla...@amcc.com
 ---
   drivers/mtd/nand/ndfc.c |6 ++
   1 files changed, 6 insertions(+), 0 deletions(-)

 diff --git a/drivers/mtd/nand/ndfc.c b/drivers/mtd/nand/ndfc.c
 index 0dd6789..88e341d 100644
 --- a/drivers/mtd/nand/ndfc.c
 +++ b/drivers/mtd/nand/ndfc.c
 @@ -89,9 +89,15 @@ static int ndfc_calculate_ecc(struct mtd_info *mtdinfo,

  /* The NDFC uses Smart Media (SMC) bytes order
   */
 +#ifdef CONFIG_MTD_NAND_ECC_SMC
  ecc_code[0] = p[1];
  ecc_code[1] = p[2];
  ecc_code[2] = p[3];
 +#else
 +ecc_code[0] = p[2];
 +ecc_code[1] = p[1];
 +ecc_code[2] = p[3];
 +#endif

 This patch seems wrong to me as CONFIG_MTD_NAND_ECC_SMC is nowhere
 defined.  [Also, it's not documented anywhere.]

 If this is fixing a bug, then please describe the exact problem, how
 to reproduce it, and how this patch is supposed to fix this problem.

 As is, this makes no sense to me.


 Best regards,

 Wolfgang Denk


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


Re: [U-Boot] [PATCH] Bug: do_reset issued via netconsole does not reset mpc83xx cpu.

2010-02-18 Thread Kim Phillips
On Mon, 15 Feb 2010 18:18:34 +0100
Detlev Zundel d...@denx.de wrote:

 Michael Zaidman michael.zaid...@gmail.com writes:
 
  The do_reset routine in the cpu/mpc83xx/cpu.c file does not reset
  the mpc83xx cpu when issued via netconsole.
 
  Moving the console output resetting the board. to the beginning of
  the routine before disabling interrupts solved the problem.
 
  Signed-off-by: Michael Zaidman michael.zaid...@gmail.com
 
 Acked-by: Detlev Zundel d...@denx.de
  ---
   cpu/mpc83xx/cpu.c |8 +++-
   1 files changed, 3 insertions(+), 5 deletions(-)

applied to u-boot-mpc83xx.

Thanks everyone.

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


[U-Boot] [PATCH] mpc83xx: remove hardcoded network addresses from config files

2010-02-18 Thread Kim Phillips
and avoid e.g., two identical boards from causing random networking
conflicts when hooked up to the same network.

Signed-off-by: Kim Phillips kim.phill...@freescale.com
---
 include/configs/MPC8313ERDB.h |7 ---
 include/configs/MPC8315ERDB.h |2 --
 include/configs/MPC8323ERDB.h |6 --
 include/configs/MPC832XEMDS.h |2 --
 include/configs/MPC8349EMDS.h |8 
 include/configs/MPC8349ITX.h  |   12 
 include/configs/MPC8360EMDS.h |2 --
 include/configs/MPC8360ERDK.h |8 
 include/configs/MPC837XEMDS.h |2 --
 include/configs/MPC837XERDB.h |   12 
 include/configs/sbc8349.h |8 
 11 files changed, 0 insertions(+), 69 deletions(-)

diff --git a/include/configs/MPC8313ERDB.h b/include/configs/MPC8313ERDB.h
index 0a4ba29..1478ec8 100644
--- a/include/configs/MPC8313ERDB.h
+++ b/include/configs/MPC8313ERDB.h
@@ -580,13 +580,6 @@
  */
 #define CONFIG_ENV_OVERWRITE
 
-#define CONFIG_ETHADDR 00:E0:0C:00:95:01
-#define CONFIG_ETH1ADDR00:E0:0C:00:95:02
-
-#define CONFIG_IPADDR  10.0.0.2
-#define CONFIG_SERVERIP10.0.0.1
-#define CONFIG_GATEWAYIP   10.0.0.1
-#define CONFIG_NETMASK 255.0.0.0
 #define CONFIG_NETDEV  eth1
 
 #define CONFIG_HOSTNAMEmpc8313erdb
diff --git a/include/configs/MPC8315ERDB.h b/include/configs/MPC8315ERDB.h
index cfed4ca..a8570ce 100644
--- a/include/configs/MPC8315ERDB.h
+++ b/include/configs/MPC8315ERDB.h
@@ -616,9 +616,7 @@
 
 #if defined(CONFIG_TSEC_ENET)
 #define CONFIG_HAS_ETH0
-#define CONFIG_ETHADDR 04:00:00:00:00:0A
 #define CONFIG_HAS_ETH1
-#define CONFIG_ETH1ADDR04:00:00:00:00:0B
 #endif
 
 #define CONFIG_BAUDRATE 115200
diff --git a/include/configs/MPC8323ERDB.h b/include/configs/MPC8323ERDB.h
index 8e6f7ce..4046f80 100644
--- a/include/configs/MPC8323ERDB.h
+++ b/include/configs/MPC8323ERDB.h
@@ -525,17 +525,11 @@
 #define CONFIG_ENV_OVERWRITE
 
 #define CONFIG_HAS_ETH0/* add support for 
ethaddr */
-#define CONFIG_ETHADDR 00:04:9f:ef:03:01
 #define CONFIG_HAS_ETH1/* add support for 
eth1addr */
-#define CONFIG_ETH1ADDR00:04:9f:ef:03:02
 
 /* use mac_read_from_eeprom() to read ethaddr from I2C EEPROM (see 
CONFIG_SYS_I2C_EEPROM) */
 #define CONFIG_SYS_I2C_MAC_OFFSET  0x7f00  /* MAC address offset in I2C 
EEPROM */
 
-#define CONFIG_IPADDR  10.0.0.2
-#define CONFIG_SERVERIP10.0.0.1
-#define CONFIG_GATEWAYIP   10.0.0.1
-#define CONFIG_NETMASK 255.0.0.0
 #define CONFIG_NETDEV  eth1
 
 #define CONFIG_HOSTNAMEmpc8323erdb
diff --git a/include/configs/MPC832XEMDS.h b/include/configs/MPC832XEMDS.h
index 1adac64..2ad5f60 100644
--- a/include/configs/MPC832XEMDS.h
+++ b/include/configs/MPC832XEMDS.h
@@ -544,9 +544,7 @@
 
 #if defined(CONFIG_UEC_ETH)
 #define CONFIG_HAS_ETH0
-#define CONFIG_ETHADDR 00:04:9f:ef:03:01
 #define CONFIG_HAS_ETH1
-#define CONFIG_ETH1ADDR00:04:9f:ef:03:02
 #endif
 
 #define CONFIG_BAUDRATE115200
diff --git a/include/configs/MPC8349EMDS.h b/include/configs/MPC8349EMDS.h
index 6361c45..bf28d9e 100644
--- a/include/configs/MPC8349EMDS.h
+++ b/include/configs/MPC8349EMDS.h
@@ -685,22 +685,14 @@
 #define CONFIG_ENV_OVERWRITE
 
 #if defined(CONFIG_TSEC_ENET)
-#define CONFIG_ETHADDR 00:04:9f:ef:23:33
 #define CONFIG_HAS_ETH1
 #define CONFIG_HAS_ETH0
-#define CONFIG_ETH1ADDR00:E0:0C:00:7E:21
 #endif
 
-#define CONFIG_IPADDR  192.168.1.253
-
 #define CONFIG_HOSTNAMEmpc8349emds
 #define CONFIG_ROOTPATH/nfsroot/rootfs
 #define CONFIG_BOOTFILEuImage
 
-#define CONFIG_SERVERIP192.168.1.1
-#define CONFIG_GATEWAYIP   192.168.1.1
-#define CONFIG_NETMASK 255.255.255.0
-
 #define CONFIG_LOADADDR80  /* default location for tftp 
and bootm */
 
 #define CONFIG_BOOTDELAY   6   /* -1 disables auto-boot */
diff --git a/include/configs/MPC8349ITX.h b/include/configs/MPC8349ITX.h
index eaa59fd..52e2851 100644
--- a/include/configs/MPC8349ITX.h
+++ b/include/configs/MPC8349ITX.h
@@ -678,18 +678,6 @@ boards, we say we have two, but don't display a message if 
we find only one. */
  */
 #define CONFIG_ENV_OVERWRITE
 
-#ifdef CONFIG_HAS_ETH0
-#define CONFIG_ETHADDR 00:E0:0C:00:8C:01
-#endif
-
-#ifdef CONFIG_HAS_ETH1
-#define CONFIG_ETH1ADDR00:E0:0C:00:8C:02
-#endif
-
-#define CONFIG_IPADDR  192.168.1.253
-#define CONFIG_SERVERIP192.168.1.1
-#define CONFIG_GATEWAYIP   192.168.1.1
-#define CONFIG_NETMASK 255.255.252.0
 #define CONFIG_NETDEV  eth0
 
 #ifdef CONFIG_MPC8349ITX
diff --git a/include/configs/MPC8360EMDS.h b/include/configs/MPC8360EMDS.h
index 6fa7287..b9b5eab 100644
--- a/include/configs/MPC8360EMDS.h
+++ b/include/configs/MPC8360EMDS.h
@@ 

Re: [U-Boot] AmigaOneG3SE

2010-02-18 Thread Peter Tyser
Bill Toner wrote:
 There's a small group forming to work on updating the MAI/AmigaOneG3SE board
 build for u-boot.

 http://amigaworld.net/modules/newbb/viewtopic.php?mode=viewtopictopic_id=30736forum=15start=0viewmode=flatorder=0

 I'm so far unable to get a complete make run through. Strangely, I'm getting
 an error that there is no .depend target for the _depend to trigger. but the
 Makefile does include rules.mk same as the sbc8641d Makefile does, and that
 one seems to build fine, so I'm using that for comparison. If I add a second
 name to the .depend target
 .depend $(obj).depend:
 then it does give a message that two identical names for that target are
 present, so I believe the rules.mk is being included. I'm struggling to
 understand this issue.

 We've also requested sources from the binary image distributor and are
 waiting a response. There's an image in my board that works, so someone was
 able to compile something at some point in the past... I'll try getting that
 version from git as well and see if it builds that way while we wait for an
 answer.

 If there's any advice available for someone learning their way around u-boot
 sources and makefiles, we'd appreciate it.
   

Hi Bill,
In board/MAI/AmigaOneG3SE/Makefile some of the files listed in $(COBJS) 
don't exist, eg ../bios_emulator/x86interface.c.  $(SRCS) is created 
from $(COBJS), which in turn makes the .depend target in rules.mk fail.

It looks like some of the missing files were removed in this commit:

commit 221838cc7eb178370ff62aa05920a582e12ac322
Author: Jason Jin jason@freescale.com
Date:   Tue Jul 10 09:03:22 2007 +0800

Remove the bios emulator from MAI board.

The bios emulator in the MAI board can not pass compile
and have a lot of crap in it. remove it and will have a
clean and small bios emulator in the drivers directory
which can be uesed for every board.

Signed-off-by: Jason Jin jason@freescale.com

However, as Jason noted the AmigaOneG3SE wasn't compiling prior to his 
commit.

If I were in your shoes I would dig into the current U-Boot code and 
start getting familiar with it as it will likely take a bit of work to 
get the AmigaOneG3SE working again.  Perhaps if you go far enough back 
in U-Boot's history you can find a point where the AmigaOneG3SE will 
compile and run to use as a working reference.  If you know which 
version currently works for you that would be a good starting point.

Best,
Peter


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


[U-Boot] please unsubscribe me

2010-02-18 Thread Mike Norrix
This is way too much email, please unsubsubscribe me Thanks
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] please unsubscribe me

2010-02-18 Thread Daniel Gorsulowski
Mike Norrix wrote:
 This is way too much email, please unsubsubscribe me Thanks

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


[U-Boot] Problem with programming UPM

2010-02-18 Thread Lasse Skov
Hey

I had tried to program the UPM on my MPC8377 cpu.

When I run the upmconfig function in the /cpu/mpc83xx/cpu.c

And when I try to do the dummy write then the target is resetting.

*dummy = 0;  /* Write the value to memory and increment MAD */

Why this happens, anyone had an idea?

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


Re: [U-Boot] [PATCH 1/1] ppc4xx: add support for alternate format for ndfc

2010-02-18 Thread Stefan Roese
Hi Feng,

On Thursday 18 February 2010 23:25:13 f...@amcc.com wrote:
 From: Feng Kan f...@amcc.com
 
 This is to lock down the ordering in the correction routine against
 the calculate routine. Otherwise, incorrect define would cause ECC errors.

It was my impression that we (finally) had done this ordering correct. The 
last changes were upon your request:

68e74567cf317318df52dbcb2ac170ffc5e7758a:
ppc4xx: Fix ECC Correction bug with SMC ordering for NDFC driver

I don't see how this patch should fix a potential problem. Please explain 
which problem exactly is fixed with this change. As Wolfgang already 
mentioned, CONFIG_MTD_NAND_ECC_SMC will not be set in this file.

Cheers,
Stefan

--
DENX Software Engineering GmbH,  MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: off...@denx.de
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot