Re: [U-Boot] [PATCH 1/1] net: Add Xilinx LL Temac driver version2

2009-02-23 Thread Michal Simek
OK. I'll look at it this week.

Thanks,
Michal

 Hi Michal,
 
 You submitted this so long ago that this driver should be in the tree 
 right now.  Unfortunately, I didn't give a thorough enough review the 
 first time and I think it would be wrong to pull it in right now without 
 addressing some more things.  If you take care of this, I promise to put 
 it in the front of the queue for the next release:
 
 mon...@monstr.eu wrote:
 From: Michal Simek mon...@monstr.eu

 ---
   
 This '---' line ensures that nothing below it (including your SOB) goes 
 in the commit.  Please remove it, add a meaningful commit message and 
 move the next few lines below the next '---' line.  Also, the version 
 2 in the title should be in square brackets so it get stripped from the 
 commit title.
 v2: Coding style cleanup, use debug() instead of printf
 in debug messages

 v1: Initial version

 Signed-off-by: Michal Simek mon...@monstr.eu
 ---
  drivers/net/Makefile  |1 +
  drivers/net/xilinx_ll_temac.c |  540 
 +
  2 files changed, 541 insertions(+), 0 deletions(-)
  create mode 100644 drivers/net/xilinx_ll_temac.c

 diff --git a/drivers/net/Makefile b/drivers/net/Makefile
 index 631336a..1e80698 100644
 --- a/drivers/net/Makefile
 +++ b/drivers/net/Makefile
 @@ -68,6 +68,7 @@ COBJS-$(CONFIG_ULI526X) += uli526x.o
  COBJS-$(CONFIG_VSC7385_ENET) += vsc7385.o
  COBJS-$(CONFIG_XILINX_EMAC) += xilinx_emac.o
  COBJS-$(CONFIG_XILINX_EMACLITE) += xilinx_emaclite.o
 +COBJS-$(CONFIG_XILINX_LL_TEMAC) += xilinx_ll_temac.o
  COBJS-$(CONFIG_SH_ETHER) += sh_eth.o
  
   
 This doesn't apply any more.  Not your fault :)
 
 
 I get the following whitespace errors:
 
 Applying net: Add Xilinx LL Temac driver
 .dotest/patch:415: trailing whitespace.

 .dotest/patch:419: trailing whitespace.

 .dotest/patch:424: trailing whitespace.
 
 
 All of the above mentioned issues are ones that I could easily deal 
 with, but one thing that really does need to change is that you need to 
 use the CONFIG_NET_MULTI API.  In other words, your driver should have a 
 single initialize() function (prototyped in include/netdev.h), and an 
 eth_device struct that gets registered.  All your access functions 
 (eth_init, eth_send, eth_recv etc.) will be static and pointed to by the 
 eth_device struct.  Most drivers are already this way.
 
 Other than that, it's fine.  Sorry again for taking so very long.
 
 regards,
 Ben
 ___
 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


Re: [U-Boot] saveenv command erases flash u-boot

2009-02-23 Thread Michael Lawnick
rudresh said the following:
 i'm using u-boot -1.3.0 in MPC8313 evaluation board, it having a NAND , NOR
 Flash , u-boot stored in NOR flash. when i do the saveenv , reset command
 the u-boot is erased. i'm not able to fix  this issue.  help me to fix this
 issue

Sounds U-Boot and environment share a flash sector.
You'll have to give more info for correct diagnostic:
Flash size, sector size, u-boot start address and size, position of
environment data and if applicable of redundancy data.

-- 
Regards,
Michael

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


Re: [U-Boot] U-Boot Digest, Vol 9, Issue 275

2009-02-23 Thread Peter Griffin


 i'm using u-boot -1.3.0 in MPC8313 evaluation board, it having a NAND , NOR
 Flash , u-boot stored in NOR flash. when i do the saveenv , reset command
 the u-boot is erased. i'm not able to fix  this issue.  help me to fix this
 issue
 
On my board this was due to CONFIG_SYS_MONITOR_LEN not being big enough.

Peter


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


Re: [U-Boot] [PATCH 1/1 V3] cmd_bdinfo: move implementation to arch instead of common

2009-02-23 Thread Haavard Skinnemoen
Jean-Christophe PLAGNIOL-VILLARD wrote:
 introduce two new weak functions board_bdinfo and cpu_bdinfo to allow
 board and cpu to print more information
 
 Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD plagn...@jcrosoft.com
 Cc: Haavard Skinnemoen haavard.skinnem...@atmel.com
 Cc: Mike Frysinger vap...@gentoo.org
 ---
 rebase for-next
 Precedent version
 Ack-by: Mike Frysinger vap...@gentoo.org
 Ack-by: Haavard Skinnemoen haavard.skinnem...@atmel.com

Aren't the Acked-by lines supposed to go above the '---' line?

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


Re: [U-Boot] Status open patches: 8xxx

2009-02-23 Thread Pieter
Dear Wolfgang

Wolfgang Denk wrote:
 Dear Kim, Andy  Jon,

 I have the following patches still marked as open in my list. Could
 you please have a look... 

 8xxx:
 11/06 Anton Vorontsov[U-Boot] [PATCH] mpc83xx: increase BOOTMAPSZ to 16MB
 12/11 Paul Gortmaker [U-Boot] [PATCH] tsec: report when there is no 
 vendor specific PHY support.
 12/30 Poonam_Aggrwal-b1  [U-Boot] [PATCH][u-boot-85xx]The 32bit errata fix 
 for 8572 for DDR2 (fwd)
 01/15 hamoshame Lamotek  [U-Boot] MPC8572DS Registered DDR2 support.
 01/22 Pieter [U-Boot] Custom MPC8548 boot using FDT problem
Snip

The problem addressed in  01/22 Pieter   [U-Boot] Custom MPC8548 boot
using FDT problem   has been resolved and was a combination on defines
in the board.h file and definition in the device tree source. is there
something I need to do to resolve the open patch status

regards Pieter



smime.p7s
Description: S/MIME Cryptographic Signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Status open patches: AVR32

2009-02-23 Thread Haavard Skinnemoen
Wolfgang Denk wrote:
 I have the following patches still marked as open in my list. Could
 you please have a look... 

Sure. Would those patches be acceptable now or should I hold them off
until the next merge window?

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


Re: [U-Boot] saveenv command erases flash u-boot

2009-02-23 Thread Wolfgang Denk
Dear rudresh,

In message 22156928.p...@talk.nabble.com you wrote:
 
 i'm using u-boot -1.3.0 in MPC8313 evaluation board, it having a NAND , NOR
 Flash , u-boot stored in NOR flash. when i do the saveenv , reset command
 the u-boot is erased. i'm not able to fix  this issue.  help me to fix this
 issue

1.3.0 is extremely old. Please use current code instead
(v2009.03-rc1).  Chances are good that the problem has long been
fixed.

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 is wanted is not the will to believe,  but the will to find out,
which is the exact opposite.
-- Bertrand Russell, Skeptical Essays, 1928
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] U-Boot Digest, Vol 9, Issue 275

2009-02-23 Thread Wolfgang Denk
Dear Peter Griffin,

In message 1235312940.26727.2.ca...@quercus-bis you wrote:
 
 
  i'm using u-boot -1.3.0 in MPC8313 evaluation board, it having a NAND , NOR
  Flash , u-boot stored in NOR flash. when i do the saveenv , reset command
  the u-boot is erased. i'm not able to fix  this issue.  help me to fix this
  issue
  
 On my board this was due to CONFIG_SYS_MONITOR_LEN not being big enough.

when replying, please:

* Keep the original poster's subject
* Make sure to includ etheread information

In general, you probably want to read
http://www.netmeister.org/news/learn2quote.html

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
Copy from one, it's plagiarism; copy from two, it's research.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 2/9] AVR32: Make cacheflush cpu-dependent

2009-02-23 Thread Haavard Skinnemoen
Gunnar Rangoy wrote:
 From: Olav Morken olav...@gmail.com
 
 The AT32UC3A series of processors doesn't contain any cache, and issuing
 cache control instructions on those will cause an exception. This commit
 makes cacheflush.h arch-dependent in preparation for the AT32UC3A-support.
 
 Signed-off-by: Gunnar Rangoy gun...@rangoy.com
 Signed-off-by: Paul Driveklepp pauldrivekl...@gmail.com
 Signed-off-by: Olav Morken olav...@gmail.com

Applied to evk1100-prep and merged into next. I'll send it upstream as
soon as it's fine with Wolfgang.

Btw, I had to rebuild my next branch since it's become a bit stale.
Since all the non-merge commit IDs are the same, I hope it won't cause
any problems -- please let me know if you see any weird merge issues.

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


Re: [U-Boot] [PATCH v2 3/9] AVR32: Move addrspace.h to arch-directory, and move some functions from io.h to addrspace.h

2009-02-23 Thread Haavard Skinnemoen
Gunnar Rangoy wrote:
 From: Olav Morken olav...@gmail.com
 
 The AVR32A architecture (which AT32UC3A-series is based on) has a
 different memory layout than the AVR32B-architecture. This patch moves
 addrspace.h to an arch-dependent directory in preparation for
 AT32UC3A-support. It also moves some address-space manipulation
 functions from io.h to addrspace.h.
 
 Signed-off-by: Gunnar Rangoy gun...@rangoy.com
 Signed-off-by: Paul Driveklepp pauldrivekl...@gmail.com
 Signed-off-by: Olav Morken olav...@gmail.com

Applied to evk1100-prep, thanks.

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


Re: [U-Boot] [PATCH v2 4/9] AVR32: Make GPIO implmentation cpu dependent

2009-02-23 Thread Haavard Skinnemoen
Gunnar Rangoy wrote:
 There are some differences in the implementation of GPIO in the
 at32uc chip compared to the ap700x series.
 
 Signed-off-by: Gunnar Rangoy gun...@rangoy.com
 Signed-off-by: Paul Driveklepp pauldrivekl...@gmail.com
 Signed-off-by: Olav Morken olav...@gmail.com

Applied to evk1100-prep, thanks.

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


Re: [U-Boot] [PATCH v2 7/9] AVR32: Must add NOPs after disabling interrupts for AT32UC3A0512ES

2009-02-23 Thread Haavard Skinnemoen
Gunnar Rangoy wrote:
 From: Olav Morken olav...@gmail.com
 
 The AT32UC3A0512ES chip has a bug when disabling interrupts. As a
 workaround, two NOPs can be inserted.
 
 Signed-off-by: Gunnar Rangoy gun...@rangoy.com
 Signed-off-by: Paul Driveklepp pauldrivekl...@gmail.com
 Signed-off-by: Olav Morken olav...@gmail.com

Applied to evk1100-prep, thanks.

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



Re: [U-Boot] [PATCH 1/1 v2] Setup extra MIMC200 chip selects

2009-02-23 Thread Haavard Skinnemoen
Mark Jackson wrote:
 Added code to setup the extra Flash and FRAM chip selects as used on the 
 MIMC200 board.
 
 V2 moves the init code from the common cpu.c file into the board specific 
 setup file.
 
 Signed-off-by: Mark Jackson m...@mimc.co.uk

Applied to mimc200 after fixing up some whitespace damage, thanks.

 ---
   board/mimc/mimc200/mimc200.c |   14 ++
   1 files changed, 14 insertions(+), 0 deletions(-)
 
 diff --git a/board/mimc/mimc200/mimc200.c b/board/mimc/mimc200/mimc200.c
 index 8516dcb..423238b 100644
 --- a/board/mimc/mimc200/mimc200.c
 +++ b/board/mimc/mimc200/mimc200.c
 @@ -29,6 +29,8 @@
   #include asm/arch/hmatrix.h
   #include lcd.h
 
 +#include ../../../cpu/at32ap/hsmc3.h

it would probably be better if this file was moved somewhere under
include/asm-avr32, however.

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


Re: [U-Boot] [PATCH v3] Add 16bpp BMP support

2009-02-23 Thread Haavard Skinnemoen
Mark Jackson wrote:
 Haavard Skinnemoen wrote:
  Mark Jackson wrote:  
  We do NOT want to do everything that is possible, but only what is
  reasonable.
  Exactly ... otherwise where do you stop ?  JPG, GIF, TIFF, PNG, etc ? 
  We're *only* meant to be showing a simply boot up image (not view lots 
  of different sized photos or movies !!), in a very controlled 
  environment (i.e. no user options ... just what the designers want / 
  require).  
  
  Why not do it even simpler? Drop BMP and generate an image matching the
  native format of the LCD controller at compile time :-)  
 
 Not sure if you're serious, but that'd reduce some of the functionality I was 
 expecting to use.

Well, it was just a thought that struck me, so I'm not going to claim I
considered all the pros and cons.

 My splashimage is stored in a particular, separate flash partition, and I'm 
 intending to allow end-users to change the boot logo (via a userspace app ?) 
 to customise / personalise the unit as they require (e.g. their own company 
 logo).

You can still do this if the userspace app knows how to generate an
image in the correct format -- I'm not arguing against storing the
image in a separate flash partition or anything like that. I just think
it might be possible to reduce the run-time size and complexity of
u-boot by being more strict about the image format.

You could also add support for PNG, JPEG and any format you want to the
userspace app -- this will probably be much easier than adding similar
support to u-boot itself.

 Hard-coding the image would render this impossible.

Of course. But hard-coding the image _format_ isn't the same thing. In
a way, we're already using a hard-coded image format, but it's one that
is easy to generate for the host, not one that's easy to display by the
target.

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


Re: [U-Boot] [PATCH] i.MX31: fir imx31_phycore to work after Linux reboot

2009-02-23 Thread Sascha Hauer
Hi,

On Sat, Feb 21, 2009 at 10:32:33PM +0100, Wolfgang Denk wrote:
 Dear Guennadi Liakhovetski,
 
 In message pine.lnx.4.64.0902061353280.7...@axis700.grange you wrote:
  
   In message pine.lnx.4.64.0902061002160.4...@axis700.grange you wrote:
Upon power on i.MX31 enables most peripheral clocks, Linux disables the 
ones
   
   Why does U-Boot do that?
  
  i.MX31 does that - the CPU, not U-Boot. I.e., this is the default power-on 
  mode.
 
 Yea, but U-Boot performs h/w initialization. So why does it not - like
 Linux - set sane defaults (with only the necessary clocks enabled) ?

I simply forgot it. The clock registers do not get resetted on reset,
only on power up.

Sascha

-- 
Pengutronix e.K.   | |
Industrial Linux Solutions | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686   | Fax:   +49-5121-206917- |
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 2/9 v3] i.MX31: add a simple gpio driver

2009-02-23 Thread Anatolij Gustschin
Jean-Christophe PLAGNIOL-VILLARD wrote:

 On 10:37 Fri 06 Feb , Guennadi Liakhovetski wrote:
 This is a minimal driver, so far only managing output. It will
 be used by the mxc_spi.c driver.

 Signed-off-by: Guennadi Liakhovetski l...@denx.de
 ---

 diff --git a/drivers/gpio/mx31_gpio.c b/drivers/gpio/mx31_gpio.c
 new file mode 100644
 index 000..166deaf
 --- /dev/null
 +++ b/drivers/gpio/mx31_gpio.c
 @@ -0,0 +1,73 @@
 +/*
 + * Copyright (C) 2009
 + * Guennadi Liakhovetski, DENX Software Engineering, l...@denx.de
 + *
 + * See file CREDITS for list of people who contributed to this
 + * project.
 + *
 + * This program is free software; you can redistribute it and/or
 + * modify it under the terms of the GNU General Public License as
 + * published by the Free Software Foundation; either version 2 of
 + * the License, or (at your option) any later version.
 + *
 + * This program is distributed in the hope that it will be useful,
 + * but WITHOUT ANY WARRANTY; without even the implied warranty of
 + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 + * GNU General Public License for more details.
 + *
 + * You should have received a copy of the GNU General Public License
 + * along with this program; if not, write to the Free Software
 + * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
 + * MA 02111-1307 USA
 + */
 +#include common.h
 +#include asm/arch/mx31.h
 +#include asm/arch/mx31-regs.h
 +
 +/* GPIO port description */
 +static unsigned long gpio_ports[] = {
 +[0] = 0x53fcc000,
 +[1] = 0x53fd,
 +[2] = 0x53fa4000,
 +};
 I'll add qong patch before yours
 please fix it

there is v4 of this patch:

http://lists.denx.de/pipermail/u-boot/2009-February/047097.html

Jean-Christophe, do you ACK it now?

Best regards,
Anatolij

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


Re: [U-Boot] [PATCH 1/9 v4] i.MX31: fix SPI driver for shorter than 32 bit

2009-02-23 Thread Anatolij Gustschin
Jean-Christophe PLAGNIOL-VILLARD wrote:

 On 00:09 Sat 07 Feb , Anatolij Gustschin wrote:
 From: Guennadi Liakhovetski l...@denx.de

 Fix setting the SPI Control register, 8 and 16-bit transfers
 and a wrong pointer in the free routine in the mxc_spi driver.

 Signed-off-by: Guennadi Liakhovetski l...@denx.de
 Signed-off-by: Anatolij Gustschin ag...@denx.de
 ---

 Changes since v1: chose a simpler fix
 Changes since v2: fix the simpler fix: v2 worked as long as I just 
 resetted the board, powering the board down showed, that v2 wasn't 
 sufficient, it has lost the SPI Control register fix from v1.
 Changes since v3: add empty line in spi_free_slave()

 I'm going to push it via video tree as Jean-Christophe
 mean it is more video related patch and have no objection.
 please not I've not ack yet patch 1,2,3,5,9
 
 please apply only after I've add it this 5 patchs
 

Jean-Christophe, do you ACK v4 of this patch?

Best regards,
Anatolij

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


Re: [U-Boot] [PATCH] i.MX31: fir imx31_phycore to work after Linux reboot

2009-02-23 Thread Guennadi Liakhovetski
On Mon, 23 Feb 2009, Sascha Hauer wrote:

 Hi,
 
 On Sat, Feb 21, 2009 at 10:32:33PM +0100, Wolfgang Denk wrote:
  Dear Guennadi Liakhovetski,
  
  In message pine.lnx.4.64.0902061353280.7...@axis700.grange you wrote:
   
In message pine.lnx.4.64.0902061002160.4...@axis700.grange you wrote:
 Upon power on i.MX31 enables most peripheral clocks, Linux disables 
 the ones

Why does U-Boot do that?
   
   i.MX31 does that - the CPU, not U-Boot. I.e., this is the default 
   power-on 
   mode.
  
  Yea, but U-Boot performs h/w initialization. So why does it not - like
  Linux - set sane defaults (with only the necessary clocks enabled) ?
 
 I simply forgot it. The clock registers do not get resetted on reset,
 only on power up.

Well, it depends. On U-Boot reset they do get resetted, but not on Linux 
reboot. Reason - U-Boot uses the watchdog for reset, Linux just jumps to 
address 0. I tried using watchdog under Linux for reset - it just hung the 
system. You might want to investigate it further...

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.

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


Re: [U-Boot] saveenv command erases flash u-boot

2009-02-23 Thread rudresh

Details are as follows .. the flash size 16Mbyte Nor flash.. NAND flash
256MB. 
/*
 * Environment
 */

#if defined(CONFIG_NAND_U_BOOT)
#define CFG_ENV_IS_IN_NAND  1
#define CFG_ENV_SIZECFG_NAND_BLOCK_SIZE
#define CFG_ENV_OFFSET  ((102410) - (CFG_NAND_BLOCK_SIZE1))
#elif !defined(CFG_RAMBOOT)
#define CFG_ENV_IS_IN_FLASH 1
#define CFG_ENV_ADDR(CFG_MONITOR_BASE + 0x4)
//Daniel Change
#define CFG_ENV_SECT_SIZE   0x2 /* 128K(one sector) for env */
//Daniel Change
#define CFG_ENV_SIZE0x2000


/*
 * FLASH on the Local Bus
 */
#define CFG_FLASH_CFI   /* use the Common Flash 
Interface */
#define CFG_FLASH_CFI_DRIVER/* use the CFI driver */
#define CFG_FLASH_BASE  0xFE00  /* start of FLASH   */
#define CFG_FLASH_SIZE  16  /* flash size in MB */
#define CFG_FLASH_EMPTY_INFO/* display empty sectors */
#define CFG_FLASH_USE_BUFFER_WRITE  /* buffer up multiple bytes */

#define CFG_FLASH_BR_PRELIM (CFG_FLASH_BASE |/* flash Base address */ \
(2BR_PS_SHIFT) | /* 16 bit port size */ \
BR_V)/* valid */
#define CFG_FLASH_OR_PRELIM (0xFF00 /* 16 MByte */ \
| OR_GPCM_XACS \
| OR_GPCM_SCY_9 \
| OR_GPCM_EHTR \
| OR_GPCM_EAD)
/* 0xFF006FF7   TODO SLOW 16 MB flash size
*/
#define CFG_LBLAWBAR0_PRELIMCFG_FLASH_BASE  /* window base at flash base */
#define CFG_LBLAWAR0_PRELIM 0x8017  /* 16 MB window size */

#define CFG_MAX_FLASH_BANKS 1   /* number of banks */
#define CFG_MAX_FLASH_SECT  135 /* sectors per device */

#define CFG_FLASH_ERASE_TOUT6   /* Flash Erase Timeout (ms) */
#define CFG_FLASH_WRITE_TOUT500 /* Flash Write Timeout (ms) */

#define CFG_MONITOR_BASETEXT_BASE   /* start of monitor */


/* Address and size of Redundant Environment Sector */

#else
#define CFG_ENV_IS_NOWHERE  1   /* Store ENV in memory only */
#define CFG_ENV_ADDR(CFG_MONITOR_BASE - 0x1000)
#define CFG_ENV_SIZE0x2000
#endif

#define CONFIG_LOADS_ECHO   1   /* echo on for serial download */
#define CFG_LOADS_BAUD_CHANGE   1   /* allow baudrate change */

---


Michael Lawnick wrote:
 
 rudresh said the following:
 i'm using u-boot -1.3.0 in MPC8313 evaluation board, it having a NAND ,
 NOR
 Flash , u-boot stored in NOR flash. when i do the saveenv , reset command
 the u-boot is erased. i'm not able to fix  this issue.  help me to fix
 this
 issue
 
 Sounds U-Boot and environment share a flash sector.
 You'll have to give more info for correct diagnostic:
 Flash size, sector size, u-boot start address and size, position of
 environment data and if applicable of redundancy data.
 
 -- 
 Regards,
 Michael
 
 ___
 U-Boot mailing list
 U-Boot@lists.denx.de
 http://lists.denx.de/mailman/listinfo/u-boot
 
 

-- 
View this message in context: 
http://www.nabble.com/saveenv-command-erases-flash-u-boot-tp22156928p22159964.html
Sent from the Uboot - Users mailing list archive at Nabble.com.

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


Re: [U-Boot] saveenv command erases flash u-boot

2009-02-23 Thread Wolfgang Denk
Dear rudresh,

In message 22160018.p...@talk.nabble.com you wrote:
 
 same u-boot-1.3.0 working in other customize MPC8313 board with some

v1.3.0 is old and you will probably find very few people here who are
willing to spend (or actually waste) their time on problems that most
probably have been fixed long ago.

Please try current code.

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
Why should we subsidize intellectual curiosity? - Ronald Reagan
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 9/9 v3] ARM: add an eet variant of the imx31_phycore board

2009-02-23 Thread Anatolij Gustschin
Hi Guennadi,

On Friday 06 February 2009 10:38:01 am Guennadi Liakhovetski wrote:
 The eet variant of the imx31_phycore board has an OLED display, using a
 s6e63d6 display controller on the first SPI interface, using GPIO57 as a
 chip-select for it. With this configuration you can display 256 colour BMP
 images in 16-bit RGB (RGB565) LCD mode.

 Signed-off-by: Guennadi Liakhovetski l...@denx.de
 ---

 Changes since v1: adjusted to reflect changes in earlier patches: add .id
 field initialisation, configuration parameter renamed to CONFIG_S6E63D6.
 Changes since v2: fixed the left-over CONFIG_DISPLAY_S6E63D6 parameter.

can you please address comments from Jean-Christophe to this 9/9 v3 patch?
I'm steel waiting for ACK from Jean-Christophe for 1/9 v4,  2/9 v4 and 9/9 v3
patches from this series.

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


Re: [U-Boot] saveenv command erases flash u-boot

2009-02-23 Thread Michael Lawnick
rudresh said the following:
[quoting repaired]
 Michael Lawnick wrote:
 
 rudresh said the following:
 i'm using u-boot -1.3.0 in MPC8313 evaluation board, it having a NAND ,
 NOR
 Flash , u-boot stored in NOR flash. when i do the saveenv , reset command
 the u-boot is erased. i'm not able to fix  this issue.  help me to fix
 this
 issue
 
 Sounds U-Boot and environment share a flash sector.
 You'll have to give more info for correct diagnostic:
 Flash size, sector size, u-boot start address and size, position of
 environment data and if applicable of redundancy data.
 Details are as follows .. the flash size 16Mbyte Nor flash.. NAND flash
 256MB. 
 /*
  * Environment
  */
 
 #if defined(CONFIG_NAND_U_BOOT)
...
 #elif !defined(CFG_RAMBOOT)
...
 #else
...
 #endif

And now you expect me to guess your boot setting?

Assuming neither CONFIG_NAND_U_BOOT nor CFG_RAMBOOT are set: check
u-boot image size against 256k, as the offset of environment seems to be
set to CFG_MONITOR_BASE + 0x4

Or is ONFIG_NAND_U_BOOT set (would be the explanation why you mention
NAND at all ...)? With u-boot in NOR this indicates a general
code/configuration problem in your image.

HTH
-- 

Michael

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


Re: [U-Boot] [PATCH 6/7 v2] OMAP3: Add OMAP3 core changes for MUSB

2009-02-23 Thread Nishanth Menon
Grazvydas Ignotas said the following on 02/23/2009 02:13 PM:
 +#ifdef CONFIG_MUSB
 +/* Enable the MUSB interface clock */
 +sr32(prcm_base-iclken1_core, 4, 1, 0x1);
 +#endif
 
 the design of u-boot is to enagle the IP only when he use it
 so please do not enable the clock every time just when you want to use
 (in the USB driver init)
   
 This is already answered in

 http://lists.denx.de/pipermail/u-boot/2009-February/047452.html
 

 He probably wants to say that clocks should be enabled only when usb
 start is issued, as you might have u-boot compiled with USB defines
 set, but never actually use USB.
   
Comparing having all clock initialization at a central point (clock.c) -
which seems simple vs having get-put kind of clock apis in U-boot : I
might go for the simple solution of all clocks in a single place..
u-boot is supposed to keep it simple, enable/disable clocks on a need
basis is elegant, though could end up getting extended to i2c and other
peripherals too(if I were to stretch is pretty hard ;) ).. makes more
sense in kernel(which is already there) than here - my 2 cents..

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


[U-Boot] [PATCH 9/9 v4] ARM: add an eet variant of the imx31_phycore board

2009-02-23 Thread Guennadi Liakhovetski
The eet variant of the imx31_phycore board has an OLED display, using a
s6e63d6 display controller on the first SPI interface, using GPIO57 as a
chip-select for it. With this configuration you can display 256 colour BMP
images in 16-bit RGB (RGB565) LCD mode.

Signed-off-by: Guennadi Liakhovetski l...@denx.de
---
On Mon, 23 Feb 2009, Anatolij Gustschin wrote:

 can you please address comments from Jean-Christophe to this 9/9 v3 patch?
 I'm steel waiting for ACK from Jean-Christophe for 1/9 v4,  2/9 v4 and 9/9 v3
 patches from this series.

Oops, sorry, I was sure I had sent it already.

Changes since v1: adjusted to reflect changes in earlier patches: add .id 
field initialisation, configuration parameter renamed to CONFIG_S6E63D6.
Changes since v2: fixed the left-over CONFIG_DISPLAY_S6E63D6 parameter.
Changes since v3: added a comment to clarify the SPI chipselect value.

 Makefile  |6 +++-
 board/imx31_phycore/imx31_phycore.c   |   57 +
 include/asm-arm/arch-mx31/mx31-regs.h |   16 +
 include/configs/imx31_phycore.h   |   23 +
 4 files changed, 101 insertions(+), 1 deletions(-)

diff --git a/Makefile b/Makefile
index 787c5f2..6151e2c 100644
--- a/Makefile
+++ b/Makefile
@@ -3025,8 +3025,12 @@ apollon_config   : unconfig
 imx31_litekit_config   : unconfig
@$(MKCONFIG) $(@:_config=) arm arm1136 imx31_litekit NULL mx31
 
+imx31_phycore_eet_config \
 imx31_phycore_config   : unconfig
-   @$(MKCONFIG) $(@:_config=) arm arm1136 imx31_phycore NULL mx31
+   @if [ -n $(findstring imx31_phycore_eet_config,$@) ]; then
\
+   echo #define CONFIG_IMX31_PHYCORE_EET  
$(obj)include/config.h;  \
+   fi
+   @$(MKCONFIG) -a imx31_phycore arm arm1136 imx31_phycore NULL mx31
 
 mx31ads_config : unconfig
@$(MKCONFIG) $(@:_config=) arm arm1136 mx31ads freescale mx31
diff --git a/board/imx31_phycore/imx31_phycore.c 
b/board/imx31_phycore/imx31_phycore.c
index 4c64cb9..0cfa172 100644
--- a/board/imx31_phycore/imx31_phycore.c
+++ b/board/imx31_phycore/imx31_phycore.c
@@ -23,6 +23,7 @@
 
 
 #include common.h
+#include s6e63d6.h
 #include asm/arch/mx31.h
 #include asm/arch/mx31-regs.h
 
@@ -66,6 +67,62 @@ int board_init (void)
return 0;
 }
 
+#ifdef BOARD_LATE_INIT
+int board_late_init(void)
+{
+#ifdef CONFIG_S6E63D6
+   struct s6e63d6 data = {
+   /*
+* See comment in mxc_spi.c::decode_cs() for .cs field format.
+* We use GPIO 57 as a chipselect for the S6E63D6 and chipselect
+* 2 of the SPI controller #1, since it is unused.
+*/
+   .cs = 2 | (57  8),
+   .bus = 0,
+   .id = 0,
+   };
+   int ret;
+
+   /* SPI1 */
+   mx31_gpio_mux(MUX_CSPI1_SCLK__CSPI1_CLK);
+   mx31_gpio_mux(MUX_CSPI1_SPI_RDY__CSPI1_DATAREADY_B);
+   mx31_gpio_mux(MUX_CSPI1_MOSI__CSPI1_MOSI);
+   mx31_gpio_mux(MUX_CSPI1_MISO__CSPI1_MISO);
+   mx31_gpio_mux(MUX_CSPI1_SS0__CSPI1_SS0_B);
+   mx31_gpio_mux(MUX_CSPI1_SS1__CSPI1_SS1_B);
+   mx31_gpio_mux(MUX_CSPI1_SS2__CSPI1_SS2_B);
+
+   /* start SPI1 clock */
+   __REG(CCM_CGR2) = __REG(CCM_CGR2) | (3  2);
+
+   /* GPIO 57 */
+   /* sw_mux_ctl_key_col4_key_col5_key_col6_key_col7 */
+   mx31_gpio_mux(IOMUX_MODE(0x63, MUX_CTL_GPIO));
+
+   /* SPI1 CS2 is free */
+   ret = s6e63d6_init(data);
+   if (ret)
+   return ret;
+
+   /*
+* This is a magic sequence to initialise a C0240QGLA / C0283QGLC
+* OLED display connected to a S6E63D6 SPI display controller in the
+* 18 bit RGB mode
+*/
+   s6e63d6_index(data, 2);
+   s6e63d6_param(data, 0x0182);
+   s6e63d6_index(data, 3);
+   s6e63d6_param(data, 0x8130);
+   s6e63d6_index(data, 0x10);
+   s6e63d6_param(data, 0x);
+   s6e63d6_index(data, 5);
+   s6e63d6_param(data, 0x0001);
+   s6e63d6_index(data, 0x22);
+#endif
+   return 0;
+}
+#endif
+
 int checkboard (void)
 {
printf(Board: Phytec phyCore i.MX31\n);
diff --git a/include/asm-arm/arch-mx31/mx31-regs.h 
b/include/asm-arm/arch-mx31/mx31-regs.h
index e736052..dcc0805 100644
--- a/include/asm-arm/arch-mx31/mx31-regs.h
+++ b/include/asm-arm/arch-mx31/mx31-regs.h
@@ -124,7 +124,14 @@
 #define MUX_CTL_CSPI2_SS0  0x85
 #define MUX_CTL_CSPI2_SS1  0x86
 #define MUX_CTL_CSPI2_SS2  0x87
+#define MUX_CTL_CSPI1_SS2  0x88
+#define MUX_CTL_CSPI1_SCLK 0x89
+#define MUX_CTL_CSPI1_SPI_RDY  0x8a
 #define MUX_CTL_CSPI2_MOSI 0x8b
+#define MUX_CTL_CSPI1_MOSI 0x8c
+#define MUX_CTL_CSPI1_MISO 0x8d
+#define MUX_CTL_CSPI1_SS0  0x8e
+#define MUX_CTL_CSPI1_SS1  0x8f
 
 /*
  * Helper macros for the MUX_[contact name]__[pin function] macros
@@ -150,6 +157,15 @@
IOMUX_MODE(MUX_CTL_CSPI2_SPI_RDY, MUX_CTL_FUNC)
 #define 

Re: [U-Boot] [PATCH 4/9 v3] A driver for the S6E63D6 SPI display controller from Samsung

2009-02-23 Thread Anatolij Gustschin
Dear Wolfgang,

Wolfgang Denk wrote:
 Dear Anatolij,
 
 In message 498c596b.2030...@denx.de you wrote:
 Guennadi Liakhovetski wrote:
 This is a driver for the S6E63D6 SPI OLED display controller from Samsung.
 It only provides access to controller's registers so the client can freely
 configure it.

 Signed-off-by: Guennadi Liakhovetski l...@denx.de
 ---
 snip
  drivers/video/Makefile  |1 +
  drivers/video/s6e63d6.c |   76 
 +++
  include/s6e63d6.h   |   37 +++
  3 files changed, 114 insertions(+), 0 deletions(-)
  create mode 100644 drivers/video/s6e63d6.c
  create mode 100644 include/s6e63d6.h
 Applied to u-boot-video/master.
 
 
 Do you plan to send a pull request soon?  Please do - I would like to
 have a v2009.03-rc1 out soon.  Thanks.

I'm still waiting for ACK for patches 1/9 v4, 2/9 v4, 9/9 v4 from this
series. I'm going to send a pull request after Jean-Christophe ACK them.
Patch 3/9 v4 was already ACKed, but I cannot apply it as it applies on
top of 1/9 v4.

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


Re: [U-Boot] [PATCH 1/1] net: Add Xilinx LL Temac driver version2

2009-02-23 Thread Michal Simek
Hi Ben,

 Hi Michal,
 
 You submitted this so long ago that this driver should be in the tree
 right now.  Unfortunately, I didn't give a thorough enough review the
 first time and I think it would be wrong to pull it in right now without
 addressing some more things.  If you take care of this, I promise to put
 it in the front of the queue for the next release:
 
 mon...@monstr.eu wrote:
 From: Michal Simek mon...@monstr.eu

 ---
   
 This '---' line ensures that nothing below it (including your SOB) goes
 in the commit.  Please remove it, add a meaningful commit message and
 move the next few lines below the next '---' line.  Also, the version
 2 in the title should be in square brackets so it get stripped from the
 commit title.
 v2: Coding style cleanup, use debug() instead of printf
 in debug messages

 v1: Initial version

 Signed-off-by: Michal Simek mon...@monstr.eu
 ---
  drivers/net/Makefile  |1 +
  drivers/net/xilinx_ll_temac.c |  540
 +
  2 files changed, 541 insertions(+), 0 deletions(-)
  create mode 100644 drivers/net/xilinx_ll_temac.c

 diff --git a/drivers/net/Makefile b/drivers/net/Makefile
 index 631336a..1e80698 100644
 --- a/drivers/net/Makefile
 +++ b/drivers/net/Makefile
 @@ -68,6 +68,7 @@ COBJS-$(CONFIG_ULI526X) += uli526x.o
  COBJS-$(CONFIG_VSC7385_ENET) += vsc7385.o
  COBJS-$(CONFIG_XILINX_EMAC) += xilinx_emac.o
  COBJS-$(CONFIG_XILINX_EMACLITE) += xilinx_emaclite.o
 +COBJS-$(CONFIG_XILINX_LL_TEMAC) += xilinx_ll_temac.o
  COBJS-$(CONFIG_SH_ETHER) += sh_eth.o
  
   
 This doesn't apply any more.  Not your fault :)
 
 
 I get the following whitespace errors:
 
 Applying net: Add Xilinx LL Temac driver
 .dotest/patch:415: trailing whitespace.
   .dotest/patch:419: trailing whitespace.
   .dotest/patch:424: trailing whitespace.

yes, There were three tabs.

 
 All of the above mentioned issues are ones that I could easily deal
 with, but one thing that really does need to change is that you need to
 use the CONFIG_NET_MULTI API.  In other words, your driver should have a
 single initialize() function (prototyped in include/netdev.h), and an
 eth_device struct that gets registered.  All your access functions
 (eth_init, eth_send, eth_recv etc.) will be static and pointed to by the
 eth_device struct.  Most drivers are already this way.

I look at it and I did some change and the main problem is in Microblaze GCC.
We use GCC 3.4.1 and CONFIG_NET_MULTI use weak function and board_eth_init is
never called. We are working on GCC 4.1.2 but I don't know when I get it.

I have two more Xilinx eth drivers in U-BOOT and I think that will be the best
to do all changes together. What do you think?

Thanks,
Michal

 
 Other than that, it's fine.  Sorry again for taking so very long.
 
 regards,
 Ben


-- 
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 1/1 V3] cmd_bdinfo: move implementation to arch instead of common

2009-02-23 Thread Mike Frysinger
On Monday 23 February 2009 03:22:55 Haavard Skinnemoen wrote:
 Jean-Christophe PLAGNIOL-VILLARD wrote:
  introduce two new weak functions board_bdinfo and cpu_bdinfo to allow
  board and cpu to print more information
 
  Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD plagn...@jcrosoft.com
  Cc: Haavard Skinnemoen haavard.skinnem...@atmel.com
  Cc: Mike Frysinger vap...@gentoo.org
  ---
  rebase for-next
  Precedent version
  Ack-by: Mike Frysinger vap...@gentoo.org
  Ack-by: Haavard Skinnemoen haavard.skinnem...@atmel.com

 Aren't the Acked-by lines supposed to go above the '---' line?

yes, and they replace the CC: lines
-mike


signature.asc
Description: This is a digitally signed message part.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Status open patches: PPC4xx

2009-02-23 Thread Stefan Roese
Hi Wolfgang,

On Sunday 22 February 2009, Wolfgang Denk wrote:
 I have the following patches still marked as open in my list. Could
 you please have a look...

 PPC4xx:
 02/20 matthias.fu...@es  [U-Boot] [PATCH 4/6] ppc4xx: Cleanup linker
 scripts of esd 4xx boards

All those patches are better placed in next. I'll put them into a 
ppc4xx/next branch in a short while.

Best regards,
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


[U-Boot] [PATCH v3] smc911x: split out useful defines/functions into local header

2009-02-23 Thread Mike Frysinger
The smc911x driver has a lot of useful defines/functions which can be used
by pieces of code (such as example eeprom programmers).  Rather than
forcing each place to duplicate these defines/functions, split them out
of the smdc911x driver into a local header.

Signed-off-by: Mike Frysinger vap...@gentoo.org
CC: Sascha Hauer s.ha...@pengutronix.de
CC: Guennadi Liakhovetski l...@denx.de
CC: Magnus Lilja lilja.mag...@gmail.com
CC: Ben Warren biggerbadder...@gmail.com
---
v3
- rebased against master

this applies against master and next of u-boot.git

 drivers/net/smc911x.c |  457 +-
 drivers/net/smc911x.h |  494 +
 2 files changed, 496 insertions(+), 455 deletions(-)
 create mode 100644 drivers/net/smc911x.h

diff --git a/drivers/net/smc911x.c b/drivers/net/smc911x.c
index 9cc4fce..1ded8f0 100644
--- a/drivers/net/smc911x.c
+++ b/drivers/net/smc911x.c
@@ -27,38 +27,7 @@
 #include net.h
 #include miiphy.h
 
-#if defined (CONFIG_DRIVER_SMC911X_32_BIT)  \
-   defined (CONFIG_DRIVER_SMC911X_16_BIT)
-#error SMC911X: Only one of CONFIG_DRIVER_SMC911X_32_BIT and \
-   CONFIG_DRIVER_SMC911X_16_BIT shall be set
-#endif
-
-#if defined (CONFIG_DRIVER_SMC911X_32_BIT)
-static inline u32 __smc911x_reg_read(u32 addr)
-{
-   return *(volatile u32*)addr;
-}
-u32 smc911x_reg_read(u32 addr) __attribute__((weak, 
alias(__smc911x_reg_read)));
-
-static inline void __smc911x_reg_write(u32 addr, u32 val)
-{
-   *(volatile u32*)addr = val;
-}
-void smc911x_reg_write(u32 addr, u32 val) __attribute__((weak, 
alias(__smc911x_reg_write)));
-#elif defined (CONFIG_DRIVER_SMC911X_16_BIT)
-static inline u32 smc911x_reg_read(u32 addr)
-{
-   volatile u16 *addr_16 = (u16 *)addr;
-   return ((*addr_16  0x) | (*(addr_16 + 1)  16));
-}
-static inline void smc911x_reg_write(u32 addr, u32 val)
-{
-   *(volatile u16*)addr = (u16)val;
-   *(volatile u16*)(addr + 2) = (u16)(val  16);
-}
-#else
-#error SMC911X: undefined bus width
-#endif /* CONFIG_DRIVER_SMC911X_16_BIT */
+#include smc911x.h
 
 u32 pkt_data_pull(u32 addr) \
__attribute__ ((weak, alias (smc911x_reg_read)));
@@ -67,370 +36,6 @@ void pkt_data_push(u32 addr, u32 val) \
 
 #define mdelay(n)   udelay((n)*1000)
 
-/* Below are the register offsets and bit definitions
- * of the Lan911x memory space
- */
-#define RX_DATA_FIFO(CONFIG_DRIVER_SMC911X_BASE + 0x00)
-
-#define TX_DATA_FIFO(CONFIG_DRIVER_SMC911X_BASE + 0x20)
-#defineTX_CMD_A_INT_ON_COMP0x8000
-#defineTX_CMD_A_INT_BUF_END_ALGN   0x0300
-#defineTX_CMD_A_INT_4_BYTE_ALGN0x
-#defineTX_CMD_A_INT_16_BYTE_ALGN   0x0100
-#defineTX_CMD_A_INT_32_BYTE_ALGN   0x0200
-#defineTX_CMD_A_INT_DATA_OFFSET0x001F
-#defineTX_CMD_A_INT_FIRST_SEG  0x2000
-#defineTX_CMD_A_INT_LAST_SEG   0x1000
-#defineTX_CMD_A_BUF_SIZE   0x07FF
-#defineTX_CMD_B_PKT_TAG0x
-#defineTX_CMD_B_ADD_CRC_DISABLE0x2000
-#defineTX_CMD_B_DISABLE_PADDING0x1000
-#defineTX_CMD_B_PKT_BYTE_LENGTH0x07FF
-
-#define RX_STATUS_FIFO (CONFIG_DRIVER_SMC911X_BASE + 0x40)
-#defineRX_STS_PKT_LEN  0x3FFF
-#defineRX_STS_ES   0x8000
-#defineRX_STS_BCST 0x2000
-#defineRX_STS_LEN_ERR  0x1000
-#defineRX_STS_RUNT_ERR 0x0800
-#defineRX_STS_MCAST0x0400
-#defineRX_STS_TOO_LONG 0x0080
-#defineRX_STS_COLL 0x0040
-#defineRX_STS_ETH_TYPE 0x0020
-#defineRX_STS_WDOG_TMT 0x0010
-#defineRX_STS_MII_ERR  0x0008
-#defineRX_STS_DRIBBLING0x0004
-#defineRX_STS_CRC_ERR  0x0002
-#define RX_STATUS_FIFO_PEEK(CONFIG_DRIVER_SMC911X_BASE + 0x44)
-#define TX_STATUS_FIFO (CONFIG_DRIVER_SMC911X_BASE + 0x48)
-#defineTX_STS_TAG  0x
-#defineTX_STS_ES   0x8000
-#defineTX_STS_LOC  0x0800
-#defineTX_STS_NO_CARR  0x0400
-#defineTX_STS_LATE_COLL0x0200
-#defineTX_STS_MANY_COLL0x0100
-#defineTX_STS_COLL_CNT 0x0078
-#define

Re: [U-Boot] [PATCH v3] smc911x: split out useful defines/functions into local header

2009-02-23 Thread Guennadi Liakhovetski
On Mon, 23 Feb 2009, Mike Frysinger wrote:

 The smc911x driver has a lot of useful defines/functions which can be used
 by pieces of code (such as example eeprom programmers).  Rather than
 forcing each place to duplicate these defines/functions, split them out
 of the smdc911x driver into a local header.
 
 Signed-off-by: Mike Frysinger vap...@gentoo.org
 CC: Sascha Hauer s.ha...@pengutronix.de
 CC: Guennadi Liakhovetski l...@denx.de
 CC: Magnus Lilja lilja.mag...@gmail.com
 CC: Ben Warren biggerbadder...@gmail.com

Hm, sorry, have any of those other pieces of code been posted to the 
list?  I'd like to have a look at them to see what they need all those 
register definitions for.

 ---
 v3
   - rebased against master
 
 this applies against master and next of u-boot.git
 
  drivers/net/smc911x.c |  457 +-
  drivers/net/smc911x.h |  494 
 +
  2 files changed, 496 insertions(+), 455 deletions(-)
  create mode 100644 drivers/net/smc911x.h

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.

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/next branch] powerpc: Fix compilation warning in board.c

2009-02-23 Thread Stefan Roese
Fix this warning:

board.c: In function 'board_init_r':
board.c:653: warning: unused variable 'i'
board.c:651: warning: unused variable 'e'

Signed-off-by: Stefan Roese s...@denx.de
---
 lib_ppc/board.c |3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/lib_ppc/board.c b/lib_ppc/board.c
index 165cd1f..f1b62f5 100644
--- a/lib_ppc/board.c
+++ b/lib_ppc/board.c
@@ -648,9 +648,8 @@ void board_init_f (ulong bootflag)
 void board_init_r (gd_t *id, ulong dest_addr)
 {
cmd_tbl_t *cmdtp;
-   char *s, *e;
+   char *s;
bd_t *bd;
-   int i;
extern void malloc_bin_reloc (void);
 #ifndef CONFIG_ENV_IS_NOWHERE
extern char * env_name_spec;
-- 
1.6.1.3

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


[U-Boot] [PATCH/next branch] powerpc: Only use eth_getenv_enetaddr() if networking is enabled

2009-02-23 Thread Stefan Roese
Signed-off-by: Stefan Roese s...@denx.de
---
 lib_ppc/board.c |2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/lib_ppc/board.c b/lib_ppc/board.c
index f1b62f5..fc3cef6 100644
--- a/lib_ppc/board.c
+++ b/lib_ppc/board.c
@@ -888,8 +888,10 @@ void board_init_r (gd_t *id, ulong dest_addr)
bd-bi_ethspeed = 0x;
 #endif
 
+#ifdef CONFIG_CMD_NET
/* kept around for legacy kernels only ... ignore the next section */
eth_getenv_enetaddr(ethaddr, bd-bi_enetaddr);
+#endif
 #ifdef CONFIG_HAS_ETH1
eth_getenv_enetaddr(eth1addr, bd-bi_enet1addr);
 #endif
-- 
1.6.1.3

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


[U-Boot] [PATCH/next branch] ppc4xx: Don't write the MAC address into the internal SoC registers

2009-02-23 Thread Stefan Roese
Remove this code. It's not needed. The 4xx EMAC driver stores the MAC
addresses into the SoC registers instead.

Signed-off-by: Stefan Roese s...@denx.de
---
 cpu/ppc4xx/cpu_init.c |   29 +
 1 files changed, 1 insertions(+), 28 deletions(-)

diff --git a/cpu/ppc4xx/cpu_init.c b/cpu/ppc4xx/cpu_init.c
index a8f589a..577d33f 100644
--- a/cpu/ppc4xx/cpu_init.c
+++ b/cpu/ppc4xx/cpu_init.c
@@ -321,35 +321,9 @@ cpu_init_f (void)
  */
 int cpu_init_r (void)
 {
-#if defined(CONFIG_405GP)  || defined(CONFIG_405EP)
-   bd_t *bd = gd-bd;
-   unsigned long reg;
-   uchar enetaddr[6];
 #if defined(CONFIG_405GP)
uint pvr = get_pvr();
-#endif
-
-   /*
-* Write Ethernetaddress into on-chip register
-*/
-   reg = 0x;
-   eth_getenv_enetaddr(ethaddr, enetaddr);
-   reg |= enetaddr[0];   /* set high address */
-   reg = reg  8;
-   reg |= enetaddr[1];
-   out32 (EMAC_IAH, reg);
-
-   reg = 0x;
-   reg |= enetaddr[2];   /* set low address  */
-   reg = reg  8;
-   reg |= enetaddr[3];
-   reg = reg  8;
-   reg |= enetaddr[4];
-   reg = reg  8;
-   reg |= enetaddr[5];
-   out32 (EMAC_IAL, reg);
 
-#if defined(CONFIG_405GP)
/*
 * Set edge conditioning circuitry on PPC405GPr
 * for compatibility to existing PPC405GP designs.
@@ -358,7 +332,6 @@ int cpu_init_r (void)
mtdcr(ecr, 0x60606000);
}
 #endif  /* defined(CONFIG_405GP) */
-#endif  /* defined(CONFIG_405GP) || defined(CONFIG_405EP) */
 
-   return (0);
+   return 0;
 }
-- 
1.6.1.3

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


Re: [U-Boot] [PATCH v3] smc911x: split out useful defines/functions into local header

2009-02-23 Thread Guennadi Liakhovetski
On Mon, 23 Feb 2009, Mike Frysinger wrote:

 On Monday 23 February 2009 10:44:04 Guennadi Liakhovetski wrote:
  On Mon, 23 Feb 2009, Mike Frysinger wrote:
   The smc911x driver has a lot of useful defines/functions which can be
   used by pieces of code (such as example eeprom programmers).  Rather than
   forcing each place to duplicate these defines/functions, split them out
   of the smdc911x driver into a local header.
  
   Signed-off-by: Mike Frysinger vap...@gentoo.org
   CC: Sascha Hauer s.ha...@pengutronix.de
   CC: Guennadi Liakhovetski l...@denx.de
   CC: Magnus Lilja lilja.mag...@gmail.com
   CC: Ben Warren biggerbadder...@gmail.com
 
  Hm, sorry, have any of those other pieces of code been posted to the
  list?  I'd like to have a look at them to see what they need all those
  register definitions for.
 
 it's already been posted in this thread (and Ben has merged it).  but to 
 answer your question, external code doesnt generally need most of those 
 register definitions.  they're moved because they belong in a header file and 
 because it doesnt make sense to move just a few.

Ok, you mean these patches

[PATCH] smc911x_eeprom: new example app for managing newer SMC parts

Yep, agree, as long as some register definitions are used, it is better to 
move all to a header.

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.

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


Re: [U-Boot] Status open patches: 8xxx

2009-02-23 Thread Jon Loeliger
On Sun, 2009-02-22 at 01:00 +0100, Wolfgang Denk wrote:
 Dear Kim, Andy  Jon,
 
 I have the following patches still marked as open in my list. Could
 you please have a look... 
 
 8xxx:

 02/04 Jon Loeliger   Re: [U-Boot] [PATCH 4/8] mpc86xx: Add support to 
 populate addr map based on BATs
 02/04 Jon Loeliger   Re: [U-Boot] [PATCH 8/8] mpc8641hpcn: Change PCI MEM 
 pci bus address
 02/04 Jon Loeliger   Re: [U-Boot] [PATCH 1/8] mpc8641hpcn: Set up 
 outbound pci windows before inbound
 02/04 Jon Loeliger   Re: [U-Boot] [PATCH 5/8] mpc8641hpcn: Clean up PCI 
 mapping concepts

 02/12 Becky Bruce[U-Boot] [PATCH] mpc8641hpcn: Indicate 36-bit addr 
 map in boot messages
 02/12 Becky Bruce[U-Boot] [PATCH] MPC86xx: set CONFIG_MAX_MEM_MAPPED 
 to 2G by default
 02/12 Becky BruceRe: [U-Boot] [PATCH] MPC86xx: set 
 CONFIG_MAX_MEM_MAPPED to 2G by default
 02/12 Jon Loeliger   Re: [U-Boot] [PATCH] MPC86xx: set 
 CONFIG_MAX_MEM_MAPPED to 2G by default

 Best regards,
 
 Wolfgang Denk

Wolfgang,

It was my belief that all outstanding 86xx patches had
either been Acked-by me and should be picked up by either
Andy or Wolfgang, or had been subsumed into other already
applied patches.

Becky (or Kumar?),

Could you verify that the 36-bit and address-map related patches
are correct in WD's current master head and that you are happy
with them?  I thought we got them all there, but I could be wrong.
Or maybe we need to wait for a set to be picked up with the 85xx
tree from Andy still?

Thanks,
jdl


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


Re: [U-Boot] [PATCH 1/1] net: Add Xilinx LL Temac driver version2

2009-02-23 Thread Ben Warren
Hi Michal,

Michal Simek wrote:

snip
 All of the above mentioned issues are ones that I could easily deal
 with, but one thing that really does need to change is that you need to
 use the CONFIG_NET_MULTI API.  In other words, your driver should have a
 single initialize() function (prototyped in include/netdev.h), and an
 eth_device struct that gets registered.  All your access functions
 (eth_init, eth_send, eth_recv etc.) will be static and pointed to by the
 eth_device struct.  Most drivers are already this way.
 

 I look at it and I did some change and the main problem is in Microblaze GCC.
 We use GCC 3.4.1 and CONFIG_NET_MULTI use weak function and board_eth_init is
 never called. We are working on GCC 4.1.2 but I don't know when I get it.

   
According to the documentation I could find, weak symbols were present 
in gcc 3.4.1.  Are you sure you're using them properly?  Due to the way 
linking is performed in U-boot, any weak symbol overrides need to be in 
source files that have strongly linked symbols.  You'll see that all 
implementations of cpu_eth_init() and board_eth_init() are in files that 
already contain stuff that is sure to be linked.
 I have two more Xilinx eth drivers in U-BOOT and I think that will be the best
 to do all changes together. What do you think?

 Thanks,
 Michal

   
If you're really unable to make this change due to your tools, I'm not 
opposed to pulling this driver in.  I just want to know that you intend 
to port all the Xilinx drivers over some time in the near future.

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


[U-Boot] [PATCH] [85xx] Add eTSEC 1/2 IO override control definition

2009-02-23 Thread ksi
This adds missing tsec12ioovcr to include/asm-ppc/immap_85xx.h. It was
named res14 (for reserved) while tsec34ioovcr was properly there.

Signed-off-by: Sergey Kubushyn k...@koi8.net
---
diff -purN u-boot.orig/include/asm-ppc/immap_85xx.h 
u-boot/include/asm-ppc/immap_85xx.h
--- u-boot.orig/include/asm-ppc/immap_85xx.h2009-02-19 13:39:31.0 
-0800
+++ u-boot/include/asm-ppc/immap_85xx.h 2009-02-23 10:32:48.0 -0800
@@ -1667,7 +1667,11 @@ typedef struct ccsr_gur {
uintlbiuiplldcr0;   /* 0xe0f1c -- LBIU PLL Debug Reg 0 */
uintlbiuiplldcr1;   /* 0xe0f20 -- LBIU PLL Debug Reg 1 */
uintddrioovcr;  /* 0xe0f24 - DDR IO Override Control */
+#ifdef CONFIG_MPC8548
+   uinttsec12ioovcr;   /* 0xe0f28 - eTSEC 1/2 IO override control */
+#else
uintres14;  /* 0xe0f28 */
+#endif
uinttsec34ioovcr;   /* 0xe0f2c - eTSEC 3/4 IO override control */
charres15[61648];   /* 0xe0f30 to 0xef */
 } ccsr_gur_t;
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] [85xx] Add eTSEC 1/2 IO override control definition

2009-02-23 Thread Kumar Gala

On Feb 23, 2009, at 12:42 PM, k...@koi8.net wrote:

 This adds missing tsec12ioovcr to include/asm-ppc/immap_85xx.h. It was
 named res14 (for reserved) while tsec34ioovcr was properly there.

 Signed-off-by: Sergey Kubushyn k...@koi8.net
 ---
 diff -purN u-boot.orig/include/asm-ppc/immap_85xx.h u-boot/include/ 
 asm-ppc/immap_85xx.h
 --- u-boot.orig/include/asm-ppc/immap_85xx.h  2009-02-19  
 13:39:31.0 -0800
 +++ u-boot/include/asm-ppc/immap_85xx.h   2009-02-23  
 10:32:48.0 -0800
 @@ -1667,7 +1667,11 @@ typedef struct ccsr_gur {
   uintlbiuiplldcr0;   /* 0xe0f1c -- LBIU PLL Debug Reg 0 */
   uintlbiuiplldcr1;   /* 0xe0f20 -- LBIU PLL Debug Reg 1 */
   uintddrioovcr;  /* 0xe0f24 - DDR IO Override Control */
 +#ifdef CONFIG_MPC8548
 + uinttsec12ioovcr;   /* 0xe0f28 - eTSEC 1/2 IO override control */
 +#else
   uintres14;  /* 0xe0f28 */
 +#endif

don't bother with the ifdef if the field was previously reserved.


   uinttsec34ioovcr;   /* 0xe0f2c - eTSEC 3/4 IO override control */
   charres15[61648];   /* 0xe0f30 to 0xef */
 } ccsr_gur_t;

- k

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


Re: [U-Boot] Status open patches: 8xxx

2009-02-23 Thread Becky Bruce

On Feb 23, 2009, at 12:14 PM, Jon Loeliger wrote:

 On Sun, 2009-02-22 at 01:00 +0100, Wolfgang Denk wrote:
 Dear Kim, Andy  Jon,

 I have the following patches still marked as open in my list. Could
 you please have a look...

 8xxx:

 02/04 Jon Loeliger   Re: [U-Boot] [PATCH 4/8] mpc86xx: Add  
 support to populate addr map based on BATs
 02/04 Jon Loeliger   Re: [U-Boot] [PATCH 8/8] mpc8641hpcn:  
 Change PCI MEM pci bus address
 02/04 Jon Loeliger   Re: [U-Boot] [PATCH 1/8] mpc8641hpcn: Set  
 up outbound pci windows before inbound
 02/04 Jon Loeliger   Re: [U-Boot] [PATCH 5/8] mpc8641hpcn:  
 Clean up PCI mapping concepts

 02/12 Becky Bruce[U-Boot] [PATCH] mpc8641hpcn: Indicate 36- 
 bit addr map in boot messages
 02/12 Becky Bruce[U-Boot] [PATCH] MPC86xx: set  
 CONFIG_MAX_MEM_MAPPED to 2G by default
 02/12 Becky BruceRe: [U-Boot] [PATCH] MPC86xx: set  
 CONFIG_MAX_MEM_MAPPED to 2G by default
 02/12 Jon Loeliger   Re: [U-Boot] [PATCH] MPC86xx: set  
 CONFIG_MAX_MEM_MAPPED to 2G by default

 Best regards,

 Wolfgang Denk

 Wolfgang,

 It was my belief that all outstanding 86xx patches had
 either been Acked-by me and should be picked up by either
 Andy or Wolfgang, or had been subsumed into other already
 applied patches.

 Becky (or Kumar?),

 Could you verify that the 36-bit and address-map related patches
 are correct in WD's current master head and that you are happy
 with them?  I thought we got them all there, but I could be wrong.
 Or maybe we need to wait for a set to be picked up with the 85xx
 tree from Andy still?

So, there are only 2 patches of mine outstanding (the third message  
above is just a note about a dependency).

This one should still apply and needs to be picked up, WD can pick it  
up directly because it's very small:

 02/12 Becky Bruce[U-Boot] [PATCH] mpc8641hpcn: Indicate 36- 
 bit addr map in boot messages

The second one looks like it won't apply anymore due to a change from  
Stefan; I will submit a new one that WD can pick up directly as long  
as it's OK with you.

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


[U-Boot] [PATCH] [85xx] Add eTSEC 1/2 IO override control (corrected)

2009-02-23 Thread ksi
This adds tsec12ioovcr to include/asm-ppc/immap_85xx.h (was reserved.)

Signed-off-by: Sergey Kubushyn k...@koi8.net
---
diff -purN u-boot.orig/include/asm-ppc/immap_85xx.h 
u-boot/include/asm-ppc/immap_85xx.h
--- u-boot.orig/include/asm-ppc/immap_85xx.h2009-02-19 13:39:31.0 
-0800
+++ u-boot/include/asm-ppc/immap_85xx.h 2009-02-23 10:49:09.0 -0800
@@ -1667,7 +1667,7 @@ typedef struct ccsr_gur {
uintlbiuiplldcr0;   /* 0xe0f1c -- LBIU PLL Debug Reg 0 */
uintlbiuiplldcr1;   /* 0xe0f20 -- LBIU PLL Debug Reg 1 */
uintddrioovcr;  /* 0xe0f24 - DDR IO Override Control */
-   uintres14;  /* 0xe0f28 */
+   uinttsec12ioovcr;   /* 0xe0f28 - eTSEC 1/2 IO override control */
uinttsec34ioovcr;   /* 0xe0f2c - eTSEC 3/4 IO override control */
charres15[61648];   /* 0xe0f30 to 0xef */
 } ccsr_gur_t;

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


Re: [U-Boot] [PATCH] [85xx] Add eTSEC 1/2 IO override control definition

2009-02-23 Thread ksi
On Mon, 23 Feb 2009, Kumar Gala wrote:

 
 On Feb 23, 2009, at 12:42 PM, k...@koi8.net wrote:
 
  This adds missing tsec12ioovcr to include/asm-ppc/immap_85xx.h. It was
  named res14 (for reserved) while tsec34ioovcr was properly there.
  
  Signed-off-by: Sergey Kubushyn k...@koi8.net
  ---
  diff -purN u-boot.orig/include/asm-ppc/immap_85xx.h
  u-boot/include/asm-ppc/immap_85xx.h
  --- u-boot.orig/include/asm-ppc/immap_85xx.h2009-02-19 
  13:39:31.0
  -0800
  +++ u-boot/include/asm-ppc/immap_85xx.h 2009-02-23 10:32:48.0
  -0800
  @@ -1667,7 +1667,11 @@ typedef struct ccsr_gur {
  uintlbiuiplldcr0;   /* 0xe0f1c -- LBIU PLL Debug Reg 0 */
  uintlbiuiplldcr1;   /* 0xe0f20 -- LBIU PLL Debug Reg 1 */
  uintddrioovcr;  /* 0xe0f24 - DDR IO Override Control */
  +#ifdef CONFIG_MPC8548
  +   uinttsec12ioovcr;   /* 0xe0f28 - eTSEC 1/2 IO override control */
  +#else
  uintres14;  /* 0xe0f28 */
  +#endif
 
 don't bother with the ifdef if the field was previously reserved.

Corrected patch posted.

---
**
*  k...@homeKOI8 Net The impossible we do immediately.  *
*  Las Vegas   NV, USA  Miracles require 24-hour notice.   *
**
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 6/7 v2] OMAP3: Add OMAP3 core changes for MUSB

2009-02-23 Thread Dirk Behme
Nishanth Menon wrote:
 Grazvydas Ignotas said the following on 02/23/2009 02:13 PM:
 +#ifdef CONFIG_MUSB
 +/* Enable the MUSB interface clock */
 +sr32(prcm_base-iclken1_core, 4, 1, 0x1);
 +#endif
 
 the design of u-boot is to enagle the IP only when he use it
 so please do not enable the clock every time just when you want to use
 (in the USB driver init)
   
 This is already answered in

 http://lists.denx.de/pipermail/u-boot/2009-February/047452.html
 
 He probably wants to say that clocks should be enabled only when usb
 start is issued, as you might have u-boot compiled with USB defines
 set, but never actually use USB.
   
 Comparing having all clock initialization at a central point (clock.c) -
 which seems simple vs having get-put kind of clock apis in U-boot : I
 might go for the simple solution of all clocks in a single place..
 u-boot is supposed to keep it simple, enable/disable clocks on a need
 basis is elegant, though could end up getting extended to i2c and other
 peripherals too(if I were to stretch is pretty hard ;) ).. makes more
 sense in kernel(which is already there) than here - my 2 cents..

Yes, I agree. Let us keep it simple. If CONFIG_MUSB is enabled by 
additional patch, this clock will be necessary.

Thanks

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


[U-Boot] [PATCH] MPC86xx: set CONFIG_MAX_MEM_MAPPED to 2G by default

2009-02-23 Thread Becky Bruce
Currently, we get 256MB as the default, but since all the 86xx
board configs define a 2G BAT mapping for RAM, raise default
to 2G.

Signed-off-by: Becky Bruce bec...@kernel.crashing.org
---
 include/asm-ppc/config.h |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/include/asm-ppc/config.h b/include/asm-ppc/config.h
index 275a7c8..0900f65 100644
--- a/include/asm-ppc/config.h
+++ b/include/asm-ppc/config.h
@@ -22,7 +22,7 @@
 #define _ASM_CONFIG_H_
 
 #ifndef CONFIG_MAX_MEM_MAPPED
-#if defined(CONFIG_4xx) || defined(CONFIG_E500)
+#if defined(CONFIG_4xx) || defined(CONFIG_E500) || defined(CONFIG_MPC86xx)
 #define CONFIG_MAX_MEM_MAPPED  ((phys_size_t)2  30)
 #else
 #define CONFIG_MAX_MEM_MAPPED  (256  20)
-- 
1.5.6.6

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


Re: [U-Boot] Status open patches: 8xxx

2009-02-23 Thread Becky Bruce
OK, I have resubmitted the MAX_MEM_MAPPED patch to apply to the  
current master u-boot tree.

-Becky

On Feb 23, 2009, at 12:53 PM, Becky Bruce wrote:


 On Feb 23, 2009, at 12:14 PM, Jon Loeliger wrote:

 On Sun, 2009-02-22 at 01:00 +0100, Wolfgang Denk wrote:
 Dear Kim, Andy  Jon,

 I have the following patches still marked as open in my list. Could
 you please have a look...

 8xxx:

 02/04 Jon Loeliger   Re: [U-Boot] [PATCH 4/8] mpc86xx: Add
 support to populate addr map based on BATs
 02/04 Jon Loeliger   Re: [U-Boot] [PATCH 8/8] mpc8641hpcn:
 Change PCI MEM pci bus address
 02/04 Jon Loeliger   Re: [U-Boot] [PATCH 1/8] mpc8641hpcn: Set
 up outbound pci windows before inbound
 02/04 Jon Loeliger   Re: [U-Boot] [PATCH 5/8] mpc8641hpcn:
 Clean up PCI mapping concepts

 02/12 Becky Bruce[U-Boot] [PATCH] mpc8641hpcn: Indicate 36-
 bit addr map in boot messages
 02/12 Becky Bruce[U-Boot] [PATCH] MPC86xx: set
 CONFIG_MAX_MEM_MAPPED to 2G by default
 02/12 Becky BruceRe: [U-Boot] [PATCH] MPC86xx: set
 CONFIG_MAX_MEM_MAPPED to 2G by default
 02/12 Jon Loeliger   Re: [U-Boot] [PATCH] MPC86xx: set
 CONFIG_MAX_MEM_MAPPED to 2G by default

 Best regards,

 Wolfgang Denk

 Wolfgang,

 It was my belief that all outstanding 86xx patches had
 either been Acked-by me and should be picked up by either
 Andy or Wolfgang, or had been subsumed into other already
 applied patches.

 Becky (or Kumar?),

 Could you verify that the 36-bit and address-map related patches
 are correct in WD's current master head and that you are happy
 with them?  I thought we got them all there, but I could be wrong.
 Or maybe we need to wait for a set to be picked up with the 85xx
 tree from Andy still?

 So, there are only 2 patches of mine outstanding (the third message
 above is just a note about a dependency).

 This one should still apply and needs to be picked up, WD can pick it
 up directly because it's very small:

 02/12 Becky Bruce[U-Boot] [PATCH] mpc8641hpcn: Indicate 36-
 bit addr map in boot messages

 The second one looks like it won't apply anymore due to a change from
 Stefan; I will submit a new one that WD can pick up directly as long
 as it's OK with you.

 Cheers,
 Becky
 ___
 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] PXA build failure

2009-02-23 Thread Sergey Lapin
Hi, all!

I am used to build at91 board using Angstrom EABI toolchain (gcc-4.2.4)
without any problems. Now I try to build PXA board and have the
following error on final link stage: (I build master branch)

cd /home/slapin/u-boot  
~/openedhand/tmp/cross/bin/arm-angstrom-linux-gnueabi-ld -Bstatic -T 
/home/slapin/u-boot/board/pxa255_idp/u-boot.lds  -Ttext 0xa308 $UNDEF_SYM 
cpu/pxa/start.o \
--start-group lib_generic/libgeneric.a 
lib_generic/lzma/liblzma.a cpu/pxa/libpxa.a lib_arm/libarm.a 
fs/cramfs/libcramfs.a fs/fat/libfat.a fs/fdos/libfdos.a fs/jffs2/libjffs2.a 
fs/reiserfs/libreiserfs.a fs/ext2/libext2fs.a fs/yaffs2/libyaffs2.a 
net/libnet.a disk/libdisk.a drivers/bios_emulator/libatibiosemu.a 
drivers/block/libblock.a drivers/dma/libdma.a drivers/fpga/libfpga.a 
drivers/gpio/libgpio.a drivers/hwmon/libhwmon.a drivers/i2c/libi2c.a 
drivers/input/libinput.a drivers/misc/libmisc.a drivers/mmc/libmmc.a 
drivers/mtd/libmtd.a drivers/mtd/nand/libnand.a 
drivers/mtd/nand_legacy/libnand_legacy.a drivers/mtd/onenand/libonenand.a 
drivers/mtd/ubi/libubi.a drivers/mtd/spi/libspi_flash.a drivers/net/libnet.a 
drivers/net/phy/libphy.a drivers/net/sk98lin/libsk98lin.a drivers/pci/libpci.a 
drivers/pcmcia/libpcmcia.a drivers/spi/libspi.a drivers/rtc/librtc.a 
drivers/serial/libserial.a drivers/usb/libusb.a drivers/video/libvideo.a 
common/libcommon.a libfdt/libfdt.a api/libapi.a post/libpost.a 
board/pxa255_idp/libpxa255_idp.a --end-group -L 
/home/slapin/openedhand/tmp/cross/lib/gcc/arm-angstrom-linux-gnueabi/4.2.4 
-lgcc \
-Map u-boot.map -o u-boot
/home/slapin/openedhand/tmp/cross/lib/gcc/arm-angstrom-linux-gnueabi/4.2.4/libgcc.a(_udivsi3.o):
 In function `__aeabi_uidiv':
/home/slapin/openedhand/tmp/work/armv5te-angstrom-linux-gnueabi/gcc-cross-4.2.4-r3/gcc-4.2.4/gcc/config/arm/lib1funcs.asm:730:
 multiple definition of `__udivsi3'
lib_arm/libarm.a(_udivsi3.o):/home/slapin/u-boot/lib_arm/_udivsi3.S:17: first 
defined here
/home/slapin/openedhand/tmp/cross/bin/arm-angstrom-linux-gnueabi-ld: ERROR: 
Source object 
/home/slapin/openedhand/tmp/cross/lib/gcc/arm-angstrom-linux-gnueabi/4.2.4/libgcc.a(_udivdi3.o)
 has EABI version 4, but target u-boot has EABI version 0
/home/slapin/openedhand/tmp/cross/bin/arm-angstrom-linux-gnueabi-ld: failed to 
merge target specific data of file 
/home/slapin/openedhand/tmp/cross/lib/gcc/arm-angstrom-linux-gnueabi/4.2.4/libgcc.a(_udivdi3.o)
/home/slapin/openedhand/tmp/cross/bin/arm-angstrom-linux-gnueabi-ld: ERROR: 
Source object 
/home/slapin/openedhand/tmp/cross/lib/gcc/arm-angstrom-linux-gnueabi/4.2.4/libgcc.a(_umoddi3.o)
 has EABI version 4, but target u-boot has EABI version 0
/home/slapin/openedhand/tmp/cross/bin/arm-angstrom-linux-gnueabi-ld: failed to 
merge target specific data of file 
/home/slapin/openedhand/tmp/cross/lib/gcc/arm-angstrom-linux-gnueabi/4.2.4/libgcc.a(_umoddi3.o)
/home/slapin/openedhand/tmp/cross/bin/arm-angstrom-linux-gnueabi-ld: ERROR: 
Source object 
/home/slapin/openedhand/tmp/cross/lib/gcc/arm-angstrom-linux-gnueabi/4.2.4/libgcc.a(_udivsi3.o)
 has EABI version 4, but target u-boot has EABI version 0
/home/slapin/openedhand/tmp/cross/bin/arm-angstrom-linux-gnueabi-ld: failed to 
merge target specific data of file 
/home/slapin/openedhand/tmp/cross/lib/gcc/arm-angstrom-linux-gnueabi/4.2.4/libgcc.a(_udivsi3.o)
/home/slapin/openedhand/tmp/cross/bin/arm-angstrom-linux-gnueabi-ld: ERROR: 
Source object 
/home/slapin/openedhand/tmp/cross/lib/gcc/arm-angstrom-linux-gnueabi/4.2.4/libgcc.a(_clz.o)
 has EABI version 4, but target u-boot has EABI version 0
/home/slapin/openedhand/tmp/cross/bin/arm-angstrom-linux-gnueabi-ld: failed to 
merge target specific data of file 
/home/slapin/openedhand/tmp/cross/lib/gcc/arm-angstrom-linux-gnueabi/4.2.4/libgcc.a(_clz.o)
/home/slapin/openedhand/tmp/cross/lib/gcc/arm-angstrom-linux-gnueabi/4.2.4/libgcc.a(_udivdi3.o):(.ARM.exidx+0x0):
 undefined reference to `__aeabi_unwind_cpp_pr0'
/home/slapin/openedhand/tmp/cross/lib/gcc/arm-angstrom-linux-gnueabi/4.2.4/libgcc.a(_umoddi3.o):(.ARM.exidx+0x0):
 undefined reference to `__aeabi_unwind_cpp_pr0'

Board name is pxa255_idp. It seems that only PXA does have these problems,
I have tried some other boards and had no problems, again.

Any ideas?
S.

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


[U-Boot] Doctor Database

2009-02-23 Thread
Practicing Physicians in the USA 

Lots of Physicians in specialties like Orthopedics, Surgery, Radiology, 
Dermatology, Neurology, General Practice etc..

you can sort by many different fields like city, state or zip

Dramatic cost reduction: $399


{}{}{} Take all 4 items below for F REE when you order ***

** Optometrists

 Visiting Nurses  RN's

 Massage Therapists

-- Acupuncturists

send email to - : s...@accudoclist.com
  
only during this week 
-   Forward email 
to r...@accudoclist.com to purge you from our records
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Status open patches: 8xxx

2009-02-23 Thread Jon Loeliger
Becky Bruce wrote:

 This one should still apply and needs to be picked up, WD can pick it up 
 directly because it's very small:
 
 02/12 Becky Bruce[U-Boot] [PATCH] mpc8641hpcn: Indicate 
 36-bit addr map in boot messages

OK, sounds fine by me.  Thanks.


 The second one looks like it won't apply anymore due to a change from 
 Stefan; I will submit a new one that WD can pick up directly as long as 
 it's OK with you.

Yep, consider it pre-Ack'ed. :-)

Thanks,
jdl



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


Re: [U-Boot] [PATCH] MPC86xx: set CONFIG_MAX_MEM_MAPPED to 2G by default

2009-02-23 Thread Jon Loeliger
Becky Bruce wrote:
 Currently, we get 256MB as the default, but since all the 86xx
 board configs define a 2G BAT mapping for RAM, raise default
 to 2G.
 
 Signed-off-by: Becky Bruce bec...@kernel.crashing.org
 ---

Oh, OK, consider it...

Acked-by: Jon Loeliger j...@freescale.com

for real.

Thanks,
jdl

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


[U-Boot] bootvx on MPC8572

2009-02-23 Thread Robert Rzepiela
Hello

I am running u-boot.2009.01 on DS-8572 (whitefin). I use tftpboot to 
download
an elf image (from vxWorks release for the eval) and try to run it with 
bootx.
After messages about loading text etc etc. the board hangs right after 
printing
Starting vxWorks at  I am using default bootline .

Is there something obvious I am missing here or I need to get ready for 
JTAG ?

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


Re: [U-Boot] Pull request - net

2009-02-23 Thread Wolfgang Denk
Dear Ben Warren,

In message 49a255af.8030...@gmail.com you wrote:
 Wolfgang,
 
 The following changes since commit 32688e572ff96715b41420e9a7f280db6c399b65:
 
   Wolfgang Denk (1):
 
 Update CHANGELOG;  Prepare 2009.03-rc1
 
 are available in the git repository at:
 
   git://git.denx.de/u-boot-net.git master
 
 Heiko Schocher (1):
 
   netloop: speed up NetLoop
 
 Mike Frysinger (1):
 
   smc911x_eeprom: new example app for managing newer SMC parts
 
 Pieter Henning (1):
 
   Added Vitesse VSC8211 definitions to TSEC driver
 
  common/cmd_nvedit.c   |6 +
 
  drivers/net/tsec.c|   30 
 
  examples/.gitignore   |1 +
 
  examples/Makefile |7 +-
 
  examples/smc911x_eeprom.c |  381 
 +
 
  include/common.h  |1 +
 
  net/eth.c |   11 +-
 
  net/net.c |  134 +---
 
  8 files changed, 507 insertions(+), 64 deletions(-)
 
  create mode 100644 examples/smc911x_eeprom.c

Applied, thanks.

[Ummm... where are all these empty lines coming from in your
message?]

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 is mind?  No matter.  What is matter?  Never mind.
  -- Thomas Hewitt Key, 1799-1875
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Status open patches: 8xxx

2009-02-23 Thread Kim Phillips
On Sun, 22 Feb 2009 01:00:14 +0100
Wolfgang Denk w...@denx.de wrote:

 I have the following patches still marked as open in my list. Could
 you please have a look... 
 
 8xxx:
 11/06 Anton Vorontsov[U-Boot] [PATCH] mpc83xx: increase BOOTMAPSZ to 16MB

this is apparently pending resubmission from its author, Anton:

http://lists.denx.de/pipermail/u-boot/2008-November/043348.html:

Please don't queue this patch yet. I'll send an updated version
soon, that will also define CFG_BOOTM_LEN (so that bootm command
will able to gunzip 16MB images).

(Anton cc'd)

 01/22 Scott Wood Re: [U-Boot] mpc8313erdb: crash in image_print_type()

this isn't a patch; it's a thread that concludes that manual relocation
is needed.

 02/05 Valeriy Glushkov   [U-Boot] [PATCH] mpc8349itx: allow SATA boot from 
 the onboard SIL1334

this is commit c9e34fe2e86f7b6cc8260f4b24cbdc7dd81e04c5 in the
u-boot-mpc83xx tree.

 02/16 Kim Phillips   Re: [U-Boot] [PATCH] mpc8349itx: allow SATA boot 
 from the onboard SIL1334

I believe this is me saying I applied it to u-boot-mpc83xx, just as I
did above :).

 02/17 Heiko Schocher [U-Boot] [PATCH 03/12 v2] 83xx, kmeter1: add I2C, 
 dtt, eeprom support
 02/17 Heiko Schocher [U-Boot] [PATCH 08/12 v2] 83xx, kmeter: QE_ENET10 
 errata for Silicon Revision 2.1
 02/17 Heiko Schocher [U-Boot] [PATCH 09/12 v2] 8360, kmeter1: added 
 bootcount feature.

As he already mentioned, Heiko has already resubmitted a 3rd version of
these...

 02/19 Anton Vorontsov[U-Boot] [PATCH 0/7] Some PCIe patches for MPC83xx
 02/19 Anton Vorontsov[U-Boot] [PATCH 1/7] mpc83xx: MPC837XEMDS: 
 Initialize SerDes before negating PCIE reset signal
 02/19 Anton Vorontsov[U-Boot] [PATCH 2/7] PCI: Add pci_last_busno() helper
 02/19 Anton Vorontsov[U-Boot] [PATCH 3/7] mpc83xx: PCIe: Don't start bus 
 enumeration at 0
 02/19 Anton Vorontsov[U-Boot] [PATCH 4/7] mpc83xx: PCIe: Fix 
 CONFIG_PCI_SCAN_SHOW reporting bogus values
 02/19 Anton Vorontsov[U-Boot] [PATCH 5/7] mpc83xx: PCI: Fix bus-range fdt 
 fixups for PCI1 controllers
 02/19 Anton Vorontsov[U-Boot] [PATCH 6/7] mpc83xx: PCI: Fix hard-coded 
 first_busno value
 02/19 Anton Vorontsov[U-Boot] [PATCH 7/7] mpc83xx: MPC837XERDB: Add PCIe 
 support
 02/19 Heiko Schocher [U-Boot] [PATCH 3/9 v3] 83xx, kmeter1: add I2C, dtt, 
 eeprom support
 02/19 Heiko Schocher [U-Boot] [PATCH 5/9 v3] 83xx, kmeter1: autodetect 
 size of DDR II RAM
 02/19 Heiko Schocher [U-Boot] [PATCH 4/9 v3] 83xx, i2c: add mux support 
 for fsl_i2c
 02/19 Heiko Schocher [U-Boot] [PATCH 6/9 v3] 83xx: add missing 
 TIMING_CFG1_CASLAT_* defines
 02/19 Heiko Schocher [U-Boot] [PATCH 7/9 v3] 83xx, kmeter1: updates for 
 2009.03
 02/19 Heiko Schocher [U-Boot] [PATCH 8/9 v3] 83xx, kmeter: QE_ENET10 
 errata for Silicon Revision 2.1
 02/19 Heiko Schocher [U-Boot] [PATCH 9/9 v3] 8360, kmeter1: added 
 bootcount feature.

...here.  And Anton's are the latest submitted to the list and I
intend on picking them up and sending you a pull request soon.

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


Re: [U-Boot] Status open patches: AVR32

2009-02-23 Thread Wolfgang Denk
Dear Haavard Skinnemoen,

In message 20090223100951.62e26...@hskinnemoen-d830 you wrote:

  I have the following patches still marked as open in my list. Could
  you please have a look... 
 
 Sure. Would those patches be acceptable now or should I hold them off
 until the next merge window?

Some of them look like bug fixes to me, so they can (or even should?)
probably go in now. But it's up for you to decide.

If they are intended for next you can also send me a pull request so
I pullt hese into the next branch 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
Worlds may change, galaxies disintegrate, but a woman always  remains
a woman.
-- Kirk, The Conscience of the King, stardate 2818.9
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 2/6] Add FSL Can use framework

2009-02-23 Thread Kim Phillips
On Thu, 19 Feb 2009 20:56:50 +0100
Wolfgang Denk w...@denx.de wrote:

 Dear Anton Vorontsov,
 
 In message 20090219154545.gb26...@oksana.dev.rtsoft.ru you wrote:
  So far it's used for specifying whether we want to use FSL DR USB or
  FSL eSDHC devices on MPC837X processors.
  
  There are two more candidates for future use:
  1. USB ULPI PHY vs. TSEC1 on MPC8315E-RDB boards;
  2. Marvell vs. Vitesse PHYs on MPC8313E-RDB boards.

Isn't the marvell PHY detectable by s/w?

  +   const char *usb_or_esdhc = getenv(usb_or_esdhc);
 
 Please make it extendable, use a more generic name for one (and only
 one) environment variable. It makes littel sense to pollyte the
 envrionment with N additional variables. For example, call it
 hwconfig. Allow that this variable holds a list of config settings,
 separated for example by comma or colon or ...

agreed - assuming this truly is indetectable by s/w.  Does setenv
dip_sw_1_4 on work more universally?

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


Re: [U-Boot] [PATCH 6/7 v2] OMAP3: Add OMAP3 core changes for MUSB

2009-02-23 Thread Wolfgang Denk
Dear Nishanth Menon,

In message 49a296f0.4000...@gmail.com you wrote:

  He probably wants to say that clocks should be enabled only when usb
  start is issued, as you might have u-boot compiled with USB defines
  set, but never actually use USB.

Correct.

 Comparing having all clock initialization at a central point (clock.c) -
 which seems simple vs having get-put kind of clock apis in U-boot : I

But the explicit rule is only to enable interfaces (and this includes
clocks) when they are actually being used. Enabling all clocks even if
not needed may for example add significantly to the power consumption
and to EMC problems.

 might go for the simple solution of all clocks in a single place..

Please don't.

 u-boot is supposed to keep it simple, enable/disable clocks on a need
 basis is elegant, though could end up getting extended to i2c and other

Yes, indeed - if you followed the recent discussion about I2C rework
this is indeed under consideration.

 peripherals too(if I were to stretch is pretty hard ;) ).. makes more
 sense in kernel(which is already there) than here - my 2 cents..

Well, we're trying to improve...

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
Status quo. Latin for the mess we're in.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 4/9 v3] A driver for the S6E63D6 SPI display controller from Samsung

2009-02-23 Thread Wolfgang Denk
Dear Jean-Christophe,

In message 49a2a105.6070...@denx.de Anatolij Gustschin wrote:
 
  Do you plan to send a pull request soon?  Please do - I would like to
  have a v2009.03-rc1 out soon.  Thanks.
 
 I'm still waiting for ACK for patches 1/9 v4, 2/9 v4, 9/9 v4 from this
 series. I'm going to send a pull request after Jean-Christophe ACK them.
 Patch 3/9 v4 was already ACKed, but I cannot apply it as it applies on
 top of 1/9 v4.

Could you please handle these patches ASAP?

I would like to see some progress here, but we're blocked waiting for
your ACKs (or NAKs).

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
It's when they say 2 + 2 = 5 that I begin to argue.- Eric Pepke
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Status open patches: PPC4xx

2009-02-23 Thread Wolfgang Denk
Dear Stefan Roese,

In message 200902231551.40483...@denx.de you wrote:
 
 All those patches are better placed in next. I'll put them into a=20
 ppc4xx/next branch in a short while.

Understood, thanks. Please feel free to send a pull request for the
next branch any time you think appropriate.

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 1/6] ppc4xx: Abort autoboot only by space key on CPCI405 boards

2009-02-23 Thread Wolfgang Denk
Dear Stefan Roese,

In message 200902231653.23766...@denx.de you wrote:
 On Friday 20 February 2009, matthias.fu...@esd.eu wrote:
  From: Matthias Fuchs matthias.fu...@esd-electronics.com
 
  Signed-off-by: Matthias Fuchs matthias.fu...@esd-electronics.com
 
 All patches applied to ppc4xx/next. Thanks.

Feel free to send a pull request for the next branch, so they don;t
get out of sync...

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
Ill-chosen abstraction is particularly evident in the design  of  the
ADA  runtime  system.  The  interface to the ADA runtime system is so
opaque that it is impossible to model  or  predict  its  performance,
making it effectively useless for real-time systems.
  - Marc D.  Donner and David H. Jameson.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] mpc8641hpcn: Indicate 36-bit addr map in boot messages

2009-02-23 Thread Wolfgang Denk
Dear Becky Bruce,

In message 1234457012-31301-1-git-send-email-bec...@kernel.crashing.org you 
wrote:
 If 36-bit addressing is enabled, print a message on the console
 when we boot.
 
 Signed-off-by: Becky Bruce bec...@kernel.crashing.org
 ---
  board/freescale/mpc8641hpcn/mpc8641hpcn.c |3 +++
  1 files changed, 3 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
Bus error -- please leave by the rear door.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] MPC86xx: set CONFIG_MAX_MEM_MAPPED to 2G by default

2009-02-23 Thread Wolfgang Denk
Dear Becky Bruce,

In message 1235419011-22472-1-git-send-email-bec...@kernel.crashing.org you 
wrote:
 Currently, we get 256MB as the default, but since all the 86xx
 board configs define a 2G BAT mapping for RAM, raise default
 to 2G.
 
 Signed-off-by: Becky Bruce bec...@kernel.crashing.org
 ---
  include/asm-ppc/config.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
Killing is stupid; useless!
-- McCoy, A Private Little War, stardate 4211.8
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] MPC86xx: set CONFIG_MAX_MEM_MAPPED to 2G by default

2009-02-23 Thread Wolfgang Denk
Dear Jon Loeliger,

In message 49a30b1c@freescale.com you wrote:
 Becky Bruce wrote:
  Currently, we get 256MB as the default, but since all the 86xx
  board configs define a 2G BAT mapping for RAM, raise default
  to 2G.
  
  Signed-off-by: Becky Bruce bec...@kernel.crashing.org
  ---
 
 Oh, OK, consider it...
 
 Acked-by: Jon Loeliger j...@freescale.com
 
 for real.

Thanks, Jon!

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
Bus error -- driver executed.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] PXA build failure

2009-02-23 Thread Wolfgang Denk
Dear Sergey Lapin,

In message 20090223195848.ga23...@build.ossfans.org you wrote:
 Hi, all!
 
 I am used to build at91 board using Angstrom EABI toolchain (gcc-4.2.4)
 without any problems. Now I try to build PXA board and have the
 following error on final link stage: (I build master branch)

This ius a known issue - all PXA boards don;t build currently with an
EABI conformant tool chain.

 Board name is pxa255_idp. It seems that only PXA does have these problems,
 I have tried some other boards and had no problems, again.
 
 Any ideas?

Jean-Cristophe said he had some patches queued or planned, which we
are waiting for.

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 years of peak mental activity are undoubtedly between the ages of
four and eighteen. At four we know all the questions, at eighteen all
the answers.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Status open patches: 8xxx

2009-02-23 Thread Wolfgang Denk
Dear Kim,

In message 20090223154208.d3712e54.kim.phill...@freescale.com you wrote:
...
 ...here.  And Anton's are the latest submitted to the list and I
 intend on picking them up and sending you a pull request soon.

Thanks for the detailed report.

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
Lispers are among  the  best  grads  of  the  Sweep-It-Under-Someone-
Else's-Carpet  School of Simulated Simplicity. [Was that sufficiently
incendiary? :-)]  - Larry Wall in 1992jan10.201804.11...@netlabs.com
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Pull request - net

2009-02-23 Thread Ben Warren
Wolfgang Denk wrote:
 Dear Ben Warren,

 In message 49a255af.8030...@gmail.com you wrote:
   
 Wolfgang,

 The following changes since commit 32688e572ff96715b41420e9a7f280db6c399b65:

   Wolfgang Denk (1):

 Update CHANGELOG;  Prepare 2009.03-rc1

 are available in the git repository at:

   git://git.denx.de/u-boot-net.git master

 Heiko Schocher (1):

   netloop: speed up NetLoop

 Mike Frysinger (1):

   smc911x_eeprom: new example app for managing newer SMC parts

 Pieter Henning (1):

   Added Vitesse VSC8211 definitions to TSEC driver

  common/cmd_nvedit.c   |6 +

  drivers/net/tsec.c|   30 

  examples/.gitignore   |1 +

  examples/Makefile |7 +-

  examples/smc911x_eeprom.c |  381 
 +

  include/common.h  |1 +

  net/eth.c |   11 +-

  net/net.c |  134 +---

  8 files changed, 507 insertions(+), 64 deletions(-)

  create mode 100644 examples/smc911x_eeprom.c
 

 Applied, thanks.

 [Ummm... where are all these empty lines coming from in your
 message?]

   
I cutpaste from terminal to Thunderbird, then changed it to 
'preformat', which does who knows what.  It was done quickly...
 Best regards,

 Wolfgang Denk

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


Re: [U-Boot] [PATCH 3/9 v3] 83xx, kmeter1: add I2C, dtt, eeprom support

2009-02-23 Thread Kim Phillips
On Thu, 19 Feb 2009 17:24:05 +0100
Heiko Schocher h...@denx.de wrote:

 This patch adds I2C support for the Keymile kmeter1 board.
 It uses the First I2C Controller from the CPU, for
 accessing 4 temperature sensors, an eeprom with IVM data
 and the booteeprom over a pca9547 mux.
 
 Signed-off-by: Heiko Schocher h...@denx.de
 ---
 I let the dtt_bus env variable in this patch, and remove this,
 when I send a patch for moving dtt_init after relocation is
 done. Then this is no longer needed.

These things usually work the other way around; please fix this so
dtt_bus isn't needed.

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


Re: [U-Boot] [PATCH 4/9 v3] 83xx, i2c: add mux support for fsl_i2c

2009-02-23 Thread Kim Phillips
On Thu, 19 Feb 2009 17:24:09 +0100
Heiko Schocher h...@denx.de wrote:

 @@ -369,6 +370,23 @@ i2c_probe(uchar chip)
 
  int i2c_set_bus_num(unsigned int bus)
  {
 +#if defined(CONFIG_I2C_MUX)
 + if (bus  CONFIG_SYS_MAX_I2C_BUS) {
 + i2c_bus_num = bus;
 + } else {

[1]

 + int ret;
 +
 + ret = i2x_mux_select_mux(bus);
 + if (ret == 0) {
 + /* with CONFIG_I2C_MUX only I2C Controller 1
 +  * is usable
 +  */
 + i2c_bus_num = 0;
 + i2c_bus_num_mux = bus;
 + } else
 + return ret;
 + }

how about

ret = i2x_...
if (ret)
return ret;
/* with...

 +#else
  #ifdef CONFIG_SYS_I2C2_OFFSET
   if (bus  1) {
  #else
 @@ -378,7 +396,7 @@ int i2c_set_bus_num(unsigned int bus)
   }
 
   i2c_bus_num = bus;
 -
 +#endif

reuse code and moved the ifdef up, then [1] would just be:

if (bus = CONFIG_SYS_MAX_I2C_BUS) {

   return 0;
  }
 
 @@ -396,7 +414,11 @@ int i2c_set_bus_speed(unsigned int speed)
 
  unsigned int i2c_get_bus_num(void)
  {
 +#if defined(CONFIG_I2C_MUX)
 + return i2c_bus_num_mux;
 +#else
   return i2c_bus_num;
 +#endif
  }

I don't get this mux variant - why aren't we reusing i2c_bus_num in the
mux case?

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


Re: [U-Boot] [PATCH 4/9 v3] 83xx, i2c: add mux support for fsl_i2c

2009-02-23 Thread Kim Phillips
On Thu, 19 Feb 2009 17:24:09 +0100
Heiko Schocher h...@denx.de wrote:

 Signed-off-by: Heiko Schocher h...@denx.de
 ---
  drivers/i2c/fsl_i2c.c |   24 +++-
  1 files changed, 23 insertions(+), 1 deletions(-)
 
 diff --git a/drivers/i2c/fsl_i2c.c b/drivers/i2c/fsl_i2c.c
 index ce646fd..5242884 100644
 --- a/drivers/i2c/fsl_i2c.c
 +++ b/drivers/i2c/fsl_i2c.c
 @@ -42,6 +42,7 @@ DECLARE_GLOBAL_DATA_PTR;
  #define CONFIG_SYS_SPD_BUS_NUM 0
  #endif
  static unsigned int i2c_bus_num __attribute__ ((section (.data))) = 
 CONFIG_SYS_SPD_BUS_NUM;
 +static unsigned int i2c_bus_num_mux __attribute__ ((section (data))) = 0;

also, this line gets this from the compiler:

fsl_i2c.c:45: warning: 'i2c_bus_num_mux' defined but not used

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


Re: [U-Boot] [PATCH 6/9 v3] 83xx: add missing TIMING_CFG1_CASLAT_* defines

2009-02-23 Thread Kim Phillips
On Thu, 19 Feb 2009 17:24:17 +0100
Heiko Schocher h...@denx.de wrote:

 Signed-off-by: Heiko Schocher h...@denx.de
 ---
 this patch is in u-boot-mpc83xx.git, but still again posted,
 because it is not yet gone in mainline, and I need this patch
 to get the following patch 7/9 compiled

so rebase the patchseries on the tree you're targeting so you don't have
to continue resending the same patch to the list.

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


[U-Boot] Please pull u-boot-mpc83xx.git

2009-02-23 Thread Kim Phillips
Wolfgang,

please pull what is primarily PCIe support for the 837xRDB:

The following changes since commit 32688e572ff96715b41420e9a7f280db6c399b65:
  Wolfgang Denk (1):
Update CHANGELOG;  Prepare 2009.03-rc1

are available in the git repository at:

  git://git.denx.de/u-boot-mpc83xx.git master

Anton Vorontsov (6):
  PCI: Add pci_last_busno() helper
  mpc83xx: PCIe: Don't start bus enumeration at 0
  mpc83xx: PCIe: Fix CONFIG_PCI_SCAN_SHOW reporting bogus values
  mpc83xx: PCI: Fix bus-range fdt fixups for PCI1 controllers
  mpc83xx: PCI: Fix hard-coded first_busno value
  mpc83xx: MPC837XERDB: Add PCIe support

Heiko Schocher (1):
  83xx: add missing TIMING_CFG1_CASLAT_* defines

Valeriy Glushkov (1):
  mpc8349itx: allow SATA boot from the onboard SIL1334

 board/freescale/mpc837xerdb/pci.c |   57 +
 cpu/mpc83xx/pci.c |6 ++--
 cpu/mpc83xx/pcie.c|   13 +---
 drivers/pci/pci.c |   13 
 include/configs/MPC8349ITX.h  |   23 +-
 include/configs/MPC837XERDB.h |   21 +
 include/mpc83xx.h |2 +
 include/pci.h |2 +
 8 files changed, 127 insertions(+), 10 deletions(-)

Thanks,

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


[U-Boot] Clock handling (wasRe: [PATCH 6/7 v2] OMAP3: Add OMAP3 core changes for MUSB)

2009-02-23 Thread Nishanth Menon
Wolfgang Denk said the following on 02/23/2009 11:36 PM:

 Comparing having all clock initialization at a central point (clock.c) -
 which seems simple vs having get-put kind of clock apis in U-boot : I
 

 But the explicit rule is only to enable interfaces (and this includes
 clocks) when they are actually being used. Enabling all clocks even if
 not needed may for example add significantly to the power consumption
 and to EMC problems.
   
I agree power consumption in u-boot is higher due to lack of power
management support - but just having clocks is just one factor of it..
if power management is our concern, we probably need a unified strategy
for it. But inherently is'nt it an overkill at u-boot level (normal boot
time is less than few seconds)..

if we look at clock handling - there are multiple levels of clock handling:
a) enable all required clocks at a go - current omap3 code does that
b) proposed change - clock enable/disable on need basis.
c) clock tree architecture - OMAP3 as an example has a complex clock
tree, cascading clock dependencies etc. as an example: sys_clk OR clk32
could be functional clock source of gptimer module. to access gptimer
you need interface clock - but once you program it, unless you need to
access the regs, you can essentially shut the interface clock off.. till
lets say interrupt occurs..  we could even think of cascading clock
divisors here.. just to make things a little more complex..

How about voltage handling etc.. are we thinking of a complete power
architecture here? what kind of gains do we expect by having just (b)?
My feel is that it would make things a lot more complex than the need
is.. For the modules one needs (they are not too many at u-boot level),
enable the clocks at one place. if we go to (b) we might as well go to
(c).. mebbe even pull in clocksource like u-boot-v2 has, while we are it..

But in anycase, do we have a generic clock architecture(clock register,
unregister, get, put apis) we are to adhere to? or are we holding off
each patch and rejecting unless they call thier own omap3_musb_clk_get,
omap3_musb_clk_put? that does not sound to be a good idea :(.. maybe I
am missing the conversation here..

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


Re: [U-Boot] [PATCH 4/9 v3] 83xx, i2c: add mux support for fsl_i2c

2009-02-23 Thread Timur Tabi
On Thu, Feb 19, 2009 at 10:24 AM, Heiko Schocher h...@denx.de wrote:
 Signed-off-by: Heiko Schocher h...@denx.de
 ---

Could you add a description that says what I2C mux support is?  That
would make it easier to review this patch.

-- 
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 2/9 v3] i.MX31: add a simple gpio driver

2009-02-23 Thread Jean-Christophe PLAGNIOL-VILLARD
On 11:58 Mon 23 Feb , Anatolij Gustschin wrote:
 Jean-Christophe PLAGNIOL-VILLARD wrote:
 
  On 10:37 Fri 06 Feb , Guennadi Liakhovetski wrote:
  This is a minimal driver, so far only managing output. It will
  be used by the mxc_spi.c driver.
 
  Signed-off-by: Guennadi Liakhovetski l...@denx.de
  ---
 
  diff --git a/drivers/gpio/mx31_gpio.c b/drivers/gpio/mx31_gpio.c
  new file mode 100644
  index 000..166deaf
  --- /dev/null
  +++ b/drivers/gpio/mx31_gpio.c
  @@ -0,0 +1,73 @@
  +/*
  + * Copyright (C) 2009
  + * Guennadi Liakhovetski, DENX Software Engineering, l...@denx.de
  + *
  + * See file CREDITS for list of people who contributed to this
  + * project.
  + *
  + * This program is free software; you can redistribute it and/or
  + * modify it under the terms of the GNU General Public License as
  + * published by the Free Software Foundation; either version 2 of
  + * the License, or (at your option) any later version.
  + *
  + * This program is distributed in the hope that it will be useful,
  + * but WITHOUT ANY WARRANTY; without even the implied warranty of
  + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  + * GNU General Public License for more details.
  + *
  + * You should have received a copy of the GNU General Public License
  + * along with this program; if not, write to the Free Software
  + * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
  + * MA 02111-1307 USA
  + */
  +#include common.h
  +#include asm/arch/mx31.h
  +#include asm/arch/mx31-regs.h
  +
  +/* GPIO port description */
  +static unsigned long gpio_ports[] = {
  +  [0] = 0x53fcc000,
  +  [1] = 0x53fd,
  +  [2] = 0x53fa4000,
  +};
  I'll add qong patch before yours
  please fix it
 
 there is v4 of this patch:
 
 http://lists.denx.de/pipermail/u-boot/2009-February/047097.html
 
 Jean-Christophe, do you ACK it now?
you ahte it this V4

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


Re: [U-Boot] [PATCH 1/9 v4] i.MX31: fix SPI driver for shorter than 32 bit

2009-02-23 Thread Jean-Christophe PLAGNIOL-VILLARD
On 12:05 Mon 23 Feb , Anatolij Gustschin wrote:
 Jean-Christophe PLAGNIOL-VILLARD wrote:
 
  On 00:09 Sat 07 Feb , Anatolij Gustschin wrote:
  From: Guennadi Liakhovetski l...@denx.de
 
  Fix setting the SPI Control register, 8 and 16-bit transfers
  and a wrong pointer in the free routine in the mxc_spi driver.
 
  Signed-off-by: Guennadi Liakhovetski l...@denx.de
  Signed-off-by: Anatolij Gustschin ag...@denx.de
  ---
 
  Changes since v1: chose a simpler fix
  Changes since v2: fix the simpler fix: v2 worked as long as I just 
  resetted the board, powering the board down showed, that v2 wasn't 
  sufficient, it has lost the SPI Control register fix from v1.
  Changes since v3: add empty line in spi_free_slave()
 
  I'm going to push it via video tree as Jean-Christophe
  mean it is more video related patch and have no objection.
  please not I've not ack yet patch 1,2,3,5,9
  
  please apply only after I've add it this 5 patchs
  
 
 Jean-Christophe, do you ACK v4 of this patch?
You've got

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


Re: [U-Boot] [PATCH 2/9 v3] i.MX31: add a simple gpio driver

2009-02-23 Thread Jean-Christophe PLAGNIOL-VILLARD
  +static unsigned long gpio_ports[] = {
  +  [0] = 0x53fcc000,
  +  [1] = 0x53fd,
  +  [2] = 0x53fa4000,
  +};
  I'll add qong patch before yours
  please fix it
 
 there is v4 of this patch:
 
 http://lists.denx.de/pipermail/u-boot/2009-February/047097.html
 
 Jean-Christophe, do you ACK it now?
ok Ack for the V4

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


Re: [U-Boot] [PATCH 9/9 v4] ARM: add an eet variant of the imx31_phycore board

2009-02-23 Thread Jean-Christophe PLAGNIOL-VILLARD
On 13:34 Mon 23 Feb , Guennadi Liakhovetski wrote:
 The eet variant of the imx31_phycore board has an OLED display, using a
 s6e63d6 display controller on the first SPI interface, using GPIO57 as a
 chip-select for it. With this configuration you can display 256 colour BMP
 images in 16-bit RGB (RGB565) LCD mode.
 
 Signed-off-by: Guennadi Liakhovetski l...@denx.de
 ---
 On Mon, 23 Feb 2009, Anatolij Gustschin wrote:
 
  can you please address comments from Jean-Christophe to this 9/9 v3 patch?
  I'm steel waiting for ACK from Jean-Christophe for 1/9 v4,  2/9 v4 and 9/9 
  v3
  patches from this series.
 
 Oops, sorry, I was sure I had sent it already.
 
 Changes since v1: adjusted to reflect changes in earlier patches: add .id 
 field initialisation, configuration parameter renamed to CONFIG_S6E63D6.
 Changes since v2: fixed the left-over CONFIG_DISPLAY_S6E63D6 parameter.
 Changes since v3: added a comment to clarify the SPI chipselect value.
 
  Makefile  |6 +++-
  board/imx31_phycore/imx31_phycore.c   |   57 
 +
  include/asm-arm/arch-mx31/mx31-regs.h |   16 +
  include/configs/imx31_phycore.h   |   23 +
  4 files changed, 101 insertions(+), 1 deletions(-)
 
 diff --git a/Makefile b/Makefile
 index 787c5f2..6151e2c 100644
 --- a/Makefile
 +++ b/Makefile
 @@ -3025,8 +3025,12 @@ apollon_config : unconfig
  imx31_litekit_config : unconfig
   @$(MKCONFIG) $(@:_config=) arm arm1136 imx31_litekit NULL mx31
  
 +imx31_phycore_eet_config \
  imx31_phycore_config : unconfig
 - @$(MKCONFIG) $(@:_config=) arm arm1136 imx31_phycore NULL mx31
 + @if [ -n $(findstring imx31_phycore_eet_config,$@) ]; then
 \
please do replace with
@if [ -n $(findstring _eet_,$@) ]; then   \

And Ack

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


[U-Boot] [PATCH 1/1] PXA: timer use do_div and simplify it

2009-02-23 Thread Jean-Christophe PLAGNIOL-VILLARD
Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD plagn...@jcrosoft.com
---
 cpu/pxa/interrupts.c |   46 --
 1 files changed, 24 insertions(+), 22 deletions(-)

diff --git a/cpu/pxa/interrupts.c b/cpu/pxa/interrupts.c
index 40d8bf2..2bc5c50 100644
--- a/cpu/pxa/interrupts.c
+++ b/cpu/pxa/interrupts.c
@@ -28,6 +28,7 @@
 
 #include common.h
 #include asm/arch/pxa-regs.h
+#include div64.h
 
 #ifdef CONFIG_USE_IRQ
 #error: interrupts not implemented yet
@@ -41,6 +42,20 @@
 #error Timer frequency unknown - please config PXA CPU type
 #endif
 
+static inline unsigned long long tick_to_time(unsigned long long tick)
+{
+   tick *= CONFIG_SYS_HZ;
+   do_div(tick, TIMER_FREQ_HZ);
+   return tick;
+}
+
+static inline unsigned long long us_to_tick(unsigned long long us)
+{
+   us = us * TIMER_FREQ_HZ + 99;
+   do_div(us, 100);
+   return us;
+}
+
 int interrupt_init (void)
 {
/* nothing happens here - we don't setup any IRQs */
@@ -75,33 +90,20 @@ void reset_timer_masked (void)
 
 ulong get_timer_masked (void)
 {
-   unsigned long long ticks = get_ticks();
-
-   return (((ticks / TIMER_FREQ_HZ) * 1000) +
-   ((ticks % TIMER_FREQ_HZ) * 1000) / TIMER_FREQ_HZ);
+   return tick_to_time(get_ticks());
 }
 
 void udelay_masked (unsigned long usec)
 {
+   unsigned long long tmp;
ulong tmo;
-   ulong endtime;
-   signed long diff;
-
-   if (usec = 1000) {
-   tmo = usec / 1000;
-   tmo *= TIMER_FREQ_HZ;
-   tmo /= 1000;
-   } else {
-   tmo = usec * TIMER_FREQ_HZ;
-   tmo /= (1000*1000);
-   }
-
-   endtime = get_ticks() + tmo;
-
-   do {
-   ulong now = get_ticks();
-   diff = endtime - now;
-   } while (diff = 0);
+
+   tmo = us_to_tick(usec);
+   tmp = get_ticks() + tmo;/* get current timestamp */
+
+   while (get_ticks()  tmp)   /* loop till event */
+/*NOP*/;
+
 }
 
 /*
-- 
1.5.6.5

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


Re: [U-Boot] [PATCH 1/1 V3] cmd_bdinfo: move implementation to arch instead of common

2009-02-23 Thread Jean-Christophe PLAGNIOL-VILLARD
On 09:22 Mon 23 Feb , Haavard Skinnemoen wrote:
 Jean-Christophe PLAGNIOL-VILLARD wrote:
  introduce two new weak functions board_bdinfo and cpu_bdinfo to allow
  board and cpu to print more information
  
  Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD plagn...@jcrosoft.com
  Cc: Haavard Skinnemoen haavard.skinnem...@atmel.com
  Cc: Mike Frysinger vap...@gentoo.org
  ---
  rebase for-next
  Precedent version
  Ack-by: Mike Frysinger vap...@gentoo.org
  Ack-by: Haavard Skinnemoen haavard.skinnem...@atmel.com
 
 Aren't the Acked-by lines supposed to go above the '---' line?
due to a lost of change on this part I've just put you two on Cc

but If you are ok I've put your Ack-by

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


[U-Boot] [PATCH 2/2] bdinfo: coding style cleanup

2009-02-23 Thread Jean-Christophe PLAGNIOL-VILLARD
Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD plagn...@jcrosoft.com
---
 common/cmd_bdinfo.c |6 -
 lib_arm/bdinfo.c|   10 
 lib_avr32/bdinfo.c  |   16 +++---
 lib_i386/bdinfo.c   |   16 +++---
 lib_m68k/bdinfo.c   |   32 ++--
 lib_microblaze/bdinfo.c |   18 +++---
 lib_mips/bdinfo.c   |   16 +++---
 lib_nios/bdinfo.c   |   14 ++--
 lib_nios2/bdinfo.c  |   18 +++---
 lib_ppc/bdinfo.c|   54 +++---
 lib_sh/bdinfo.c |   16 +++---
 11 files changed, 110 insertions(+), 106 deletions(-)

diff --git a/common/cmd_bdinfo.c b/common/cmd_bdinfo.c
index 2d83b93..db25330 100644
--- a/common/cmd_bdinfo.c
+++ b/common/cmd_bdinfo.c
@@ -60,14 +60,18 @@ int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char 
*argv[])
 
 void print_eth(int idx)
 {
-   char name[10], *val;
+   char name[10];
+   char *val;
+
if (idx)
sprintf(name, eth%iaddr, idx);
else
strcpy(name, ethaddr);
+
val = getenv(name);
if (!val)
val = (not set);
+
printf(%-12s= %s\n, name, val);
 }
 
diff --git a/lib_arm/bdinfo.c b/lib_arm/bdinfo.c
index 0e5084b..d5b5a4b 100644
--- a/lib_arm/bdinfo.c
+++ b/lib_arm/bdinfo.c
@@ -35,9 +35,9 @@ void arch_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char 
*argv[])
int i;
bd_t *bd = gd-bd;
 
-   print_num (arch_number,   bd-bi_arch_number);
-   print_num (env_t, (ulong)bd-bi_env);
-   print_num (boot_params,   (ulong)bd-bi_boot_params);
+   print_num(arch_number,bd-bi_arch_number);
+   print_num(env_t,  (ulong)bd-bi_env);
+   print_num(boot_params,(ulong)bd-bi_boot_params);
 
for (i = 0; i  CONFIG_NR_DRAM_BANKS; ++i) {
print_num(DRAM bank,  i);
@@ -47,7 +47,7 @@ void arch_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char 
*argv[])
 
 #if defined(CONFIG_CMD_NET)
print_eth(0);
-   printf (ip_addr = %pI4\n, bd-bi_ip_addr);
+   printf(ip_addr = %pI4\n, bd-bi_ip_addr);
 #endif
-   printf (baudrate= %d bps\n, bd-bi_baudrate);
+   printf(baudrate= %d bps\n, bd-bi_baudrate);
 }
diff --git a/lib_avr32/bdinfo.c b/lib_avr32/bdinfo.c
index 616481d..a040c7a 100644
--- a/lib_avr32/bdinfo.c
+++ b/lib_avr32/bdinfo.c
@@ -34,14 +34,14 @@ void arch_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char 
*argv[])
 {
bd_t *bd = gd-bd;
 
-   print_num (boot_params,   (ulong)bd-bi_boot_params);
-   print_num (memstart,  (ulong)bd-bi_memstart);
-   print_lnum (memsize,  (u64)bd-bi_memsize);
-   print_num (flashstart,(ulong)bd-bi_flashstart);
-   print_num (flashsize, (ulong)bd-bi_flashsize);
-   print_num (flashoffset,   (ulong)bd-bi_flashoffset);
+   print_num(boot_params,(ulong)bd-bi_boot_params);
+   print_num(memstart,   (ulong)bd-bi_memstart);
+   print_lnum(memsize,   (u64)bd-bi_memsize);
+   print_num(flashstart, (ulong)bd-bi_flashstart);
+   print_num(flashsize,  (ulong)bd-bi_flashsize);
+   print_num(flashoffset,(ulong)bd-bi_flashoffset);
 
print_eth(0);
-   printf (ip_addr = %pI4\n, bd-bi_ip_addr);
-   printf (baudrate= %d bps\n, bd-bi_baudrate);
+   printf(ip_addr = %pI4\n, bd-bi_ip_addr);
+   printf(baudrate= %d bps\n, bd-bi_baudrate);
 }
diff --git a/lib_i386/bdinfo.c b/lib_i386/bdinfo.c
index 616481d..a040c7a 100644
--- a/lib_i386/bdinfo.c
+++ b/lib_i386/bdinfo.c
@@ -34,14 +34,14 @@ void arch_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char 
*argv[])
 {
bd_t *bd = gd-bd;
 
-   print_num (boot_params,   (ulong)bd-bi_boot_params);
-   print_num (memstart,  (ulong)bd-bi_memstart);
-   print_lnum (memsize,  (u64)bd-bi_memsize);
-   print_num (flashstart,(ulong)bd-bi_flashstart);
-   print_num (flashsize, (ulong)bd-bi_flashsize);
-   print_num (flashoffset,   (ulong)bd-bi_flashoffset);
+   print_num(boot_params,(ulong)bd-bi_boot_params);
+   print_num(memstart,   (ulong)bd-bi_memstart);
+   print_lnum(memsize,   (u64)bd-bi_memsize);
+   print_num(flashstart, (ulong)bd-bi_flashstart);
+   print_num(flashsize,  (ulong)bd-bi_flashsize);
+   print_num(flashoffset,(ulong)bd-bi_flashoffset);
 
print_eth(0);
-   printf (ip_addr = %pI4\n, bd-bi_ip_addr);
-   printf (baudrate= %d bps\n, bd-bi_baudrate);
+   printf(ip_addr = %pI4\n, bd-bi_ip_addr);
+   printf(baudrate= %d bps\n, bd-bi_baudrate);
 }
diff --git a/lib_m68k/bdinfo.c b/lib_m68k/bdinfo.c
index e77a1cf..e271145 100644
--- a/lib_m68k/bdinfo.c
+++ b/lib_m68k/bdinfo.c
@@ -35,27 +35,27 @@ void 

[U-Boot] [PATCH 1/2 V4] cmd_bdinfo: move implementation to arch instead of common

2009-02-23 Thread Jean-Christophe PLAGNIOL-VILLARD
introduce 3 new weak functions board_bdinfo, cpu_bdinfo and soc_bdinfo to allow
board, cpu and soc to print more information

Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD plagn...@jcrosoft.com
Acked-by: Mike Frysinger vap...@gentoo.org
Ack-by: Haavard Skinnemoen haavard.skinnem...@atmel.com
---
Change from V3
SOC weak function
and Mike Suggestion

Best Regards,
J.
 common/cmd_bdinfo.c |  462 ---
 include/bdinfo.h|   49 +
 lib_arm/Makefile|1 +
 lib_arm/bdinfo.c|   53 ++
 lib_avr32/Makefile  |1 +
 lib_avr32/bdinfo.c  |   47 +
 lib_blackfin/Makefile   |1 +
 lib_blackfin/bdinfo.c   |   55 ++
 lib_i386/Makefile   |1 +
 lib_i386/bdinfo.c   |   47 +
 lib_m68k/Makefile   |1 +
 lib_m68k/bdinfo.c   |   75 
 lib_microblaze/Makefile |1 +
 lib_microblaze/bdinfo.c |   51 +
 lib_mips/Makefile   |1 +
 lib_mips/bdinfo.c   |   47 +
 lib_nios/Makefile   |1 +
 lib_nios/bdinfo.c   |   46 +
 lib_nios2/Makefile  |1 +
 lib_nios2/bdinfo.c  |   54 ++
 lib_ppc/Makefile|1 +
 lib_ppc/bdinfo.c|  107 +++
 lib_sh/Makefile |1 +
 lib_sh/bdinfo.c |   47 +
 lib_sparc/Makefile  |   13 +-
 lib_sparc/bdinfo.c  |   61 ++
 26 files changed, 837 insertions(+), 388 deletions(-)
 rewrite common/cmd_bdinfo.c (88%)
 create mode 100644 include/bdinfo.h
 create mode 100644 lib_arm/bdinfo.c
 create mode 100644 lib_avr32/bdinfo.c
 create mode 100644 lib_blackfin/bdinfo.c
 create mode 100644 lib_i386/bdinfo.c
 create mode 100644 lib_m68k/bdinfo.c
 create mode 100644 lib_microblaze/bdinfo.c
 create mode 100644 lib_mips/bdinfo.c
 create mode 100644 lib_nios/bdinfo.c
 create mode 100644 lib_nios2/bdinfo.c
 create mode 100644 lib_ppc/bdinfo.c
 create mode 100644 lib_sh/bdinfo.c
 create mode 100644 lib_sparc/bdinfo.c

diff --git a/common/cmd_bdinfo.c b/common/cmd_bdinfo.c
dissimilarity index 88%
index 700314b..2d83b93 100644
--- a/common/cmd_bdinfo.c
+++ b/common/cmd_bdinfo.c
@@ -1,384 +1,78 @@
-/*
- * (C) Copyright 2003
- * Wolfgang Denk, DENX Software Engineering, w...@denx.de.
- *
- * See file CREDITS for list of people who contributed to this
- * project.
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation; either version 2 of
- * the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
- * MA 02111-1307 USA
- */
-
-/*
- * Boot support
- */
-#include common.h
-#include command.h
-
-DECLARE_GLOBAL_DATA_PTR;
-
-static void print_num(const char *, ulong);
-
-static void print_eth(int idx);
-
-#ifndef CONFIG_ARM /* PowerPC and other */
-static void print_lnum(const char *, u64);
-
-#ifdef CONFIG_PPC
-static void print_str(const char *, const char *);
-
-int do_bdinfo ( cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
-{
-   bd_t *bd = gd-bd;
-   char buf[32];
-
-#ifdef DEBUG
-   print_num (bd address,(ulong)bd   );
-#endif
-   print_num (memstart,  bd-bi_memstart );
-   print_lnum (memsize,  bd-bi_memsize  );
-   print_num (flashstart,bd-bi_flashstart   );
-   print_num (flashsize, bd-bi_flashsize);
-   print_num (flashoffset,   bd-bi_flashoffset  );
-   print_num (sramstart, bd-bi_sramstart);
-   print_num (sramsize,  bd-bi_sramsize );
-#if defined(CONFIG_5xx)  || defined(CONFIG_8xx) || \
-defined(CONFIG_8260) || defined(CONFIG_E500)
-   print_num (immr_base, bd-bi_immr_base);
-#endif
-   print_num (bootflags, bd-bi_bootflags);
-#if defined(CONFIG_405GP) || defined(CONFIG_405CR) || \
-defined(CONFIG_405EP) || defined(CONFIG_XILINX_405) || \
-defined(CONFIG_440EP) || defined(CONFIG_440GR) || \
-defined(CONFIG_440EPX) || defined(CONFIG_440GRX) ||\
-defined(CONFIG_440SP) || defined(CONFIG_440SPE)
-   print_str (procfreq,  strmhz(buf, bd-bi_procfreq));
-   print_str (plb_busfreq,   strmhz(buf, bd-bi_plb_busfreq));
-#if defined(CONFIG_405GP) || defined(CONFIG_405EP) || 
defined(CONFIG_XILINX_405) || \
-defined(CONFIG_440EP) || defined(CONFIG_440GR) || defined(CONFIG_440SPE) 
|| \
-defined(CONFIG_440EPX) || defined(CONFIG_440GRX)
-   print_str (pci_busfreq,   strmhz(buf, bd-bi_pci_busfreq));
-#endif
-#else  /* ! CONFIG_405GP, CONFIG_405CR, CONFIG_405EP, 

Re: [U-Boot] (no subject)

2009-02-23 Thread md ks
Hi,
Thanks for  your Reply. I am using u-boot-1.3.3 and linux kernel
2.6.29-rc2.
we are  able to display customised logo in U-boot level , kenel level
and file system level.
But  while switching from u-boot to Kernel , Screen(Display) is showing
  Blank screen (4 to 5 seconds).
Is it possible to avoid this blank screen. I want to implement clean
bootup process(No flicker/ Blank-ness from one level to other)

 You said, Shared FB, i don't have any idea about shared FB, Please
provide some help in this area and at least refference

 even though  at U-boot level we alloctes some FB , in Kernel
intialisation stage that FB is erased and allocates its own FB,
I am thinking like that , is it true.

 Is it possible to use same FB both in u-boot level and Kernel level.


Please help me.
mdks


2009/2/19 Nishanth Menon menon.nisha...@gmail.com

 md ks said the following on 02/19/2009 03:30 PM:
  Hi,
  I am omap-3530, u-boot 1.3 version , while switching from u-boot
  level to kernel level , i am getting flicker on LCD(display)., How to
  remove that flicker.
 What platform is this? did you mention u-boot 1.3? the current u-boots
 are numbered 2009-01 etc.. Where is the source code from? Further, could
 you see if u-boot from the mainline [1] resolves the issue?

 Regards,
 Nishanth Menon

 Ref:
 [1] http://git.denx.de/?p=u-boot.git;a=summary

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


[U-Boot] Some question about uboot

2009-02-23 Thread loody
Dear all:
While porting uboot, I find there are too many things I don't need at
all, like net, i2c etc.
How could I remove them from my uboot binary to minimize the size of
uboot, my flash limitation is 6k?
I try to edit Makefile to remove them by marking some library setting,
but it compile error after I doing so.
Is there any easier way to meet my requirement?
Appreciate your help,
miloody
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 3/9 v3] 83xx, kmeter1: add I2C, dtt, eeprom support

2009-02-23 Thread Heiko Schocher
Hello Kim,

Kim Phillips wrote:
 On Thu, 19 Feb 2009 17:24:05 +0100
 Heiko Schocher h...@denx.de wrote:
 
 This patch adds I2C support for the Keymile kmeter1 board.
 It uses the First I2C Controller from the CPU, for
 accessing 4 temperature sensors, an eeprom with IVM data
 and the booteeprom over a pca9547 mux.

 Signed-off-by: Heiko Schocher h...@denx.de
 ---
 I let the dtt_bus env variable in this patch, and remove this,
 when I send a patch for moving dtt_init after relocation is
 done. Then this is no longer needed.
 
 These things usually work the other way around; please fix this so
 dtt_bus isn't needed.

Ok

thanks
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


Re: [U-Boot] Some question about uboot

2009-02-23 Thread Aggrwal Poonam-B10812
 

 -Original Message-
 From: u-boot-boun...@lists.denx.de 
 [mailto:u-boot-boun...@lists.denx.de] On Behalf Of loody
 Sent: Tuesday, February 24, 2009 12:19 PM
 To: u-boot@lists.denx.de
 Subject: [U-Boot] Some question about uboot
 
 Dear all:
 While porting uboot, I find there are too many things I don't 
 need at all, like net, i2c etc.
 How could I remove them from my uboot binary to minimize the 
 size of uboot, my flash limitation is 6k?
 I try to edit Makefile to remove them by marking some library 
 setting, but it compile error after I doing so.
 Is there any easier way to meet my requirement?

Justr trim your board specific file in include/configs/ folder.
Remove the CONFIGS you  do not want.
 Appreciate your help,
 miloody
 ___
 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


Re: [U-Boot] [PATCH 4/9 v3] 83xx, i2c: add mux support for fsl_i2c

2009-02-23 Thread Heiko Schocher
Hello Kim,

Kim Phillips wrote:
 On Thu, 19 Feb 2009 17:24:09 +0100
 Heiko Schocher h...@denx.de wrote:
 
 @@ -369,6 +370,23 @@ i2c_probe(uchar chip)

  int i2c_set_bus_num(unsigned int bus)
  {
 +#if defined(CONFIG_I2C_MUX)
 +if (bus  CONFIG_SYS_MAX_I2C_BUS) {
 +i2c_bus_num = bus;
 +} else {
 
 [1]
 
 +int ret;
 +
 +ret = i2x_mux_select_mux(bus);
 +if (ret == 0) {
 +/* with CONFIG_I2C_MUX only I2C Controller 1
 + * is usable
 + */
 +i2c_bus_num = 0;
 +i2c_bus_num_mux = bus;
 +} else
 +return ret;
 +}
 
 how about
 
 ret = i2x_...
 if (ret)
   return ret;
 /* with...
 
 +#else
  #ifdef CONFIG_SYS_I2C2_OFFSET
  if (bus  1) {
  #else
 @@ -378,7 +396,7 @@ int i2c_set_bus_num(unsigned int bus)
  }

  i2c_bus_num = bus;
 -
 +#endif
 
 reuse code and moved the ifdef up, then [1] would just be:

Ok.

 if (bus = CONFIG_SYS_MAX_I2C_BUS) {
 
  return 0;
  }

 @@ -396,7 +414,11 @@ int i2c_set_bus_speed(unsigned int speed)

  unsigned int i2c_get_bus_num(void)
  {
 +#if defined(CONFIG_I2C_MUX)
 +return i2c_bus_num_mux;
 +#else
  return i2c_bus_num;
 +#endif
  }
 
 I don't get this mux variant - why aren't we reusing i2c_bus_num in the
 mux case?

Good question, have to think about it.

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


Re: [U-Boot] [PATCH] AT91RM9200 BGA port D defines

2009-02-23 Thread Jens Scharsig
Hello,

we are still working on a new U-Boot port for our customer and an own board. 
I think the release is in the second of 2009.

Best Regards,

Jens Scharsig


Jean-Christophe PLAGNIOL-VILLARD schrieb:
 On 08:17 Mon 23 Feb , Jens Scharsig wrote:
 This patch add's the defines for port d I/Os on AT91RM9200 (BGA package)

 Signed-off-by: Jens Scharsig e...@bus-elektronik.de
 ---
 
 applied
 
 will you send a board that use it soon?
 
 Best Regards,
 J.

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


[U-Boot] [PATCH] sh: Add support sh_eth to cpu of sh4

2009-02-23 Thread Nobuhiro Iwamatsu
Signed-off-by: Nobuhiro Iwamatsu iwamatsu.nobuh...@renesas.com
---
 cpu/sh4/cpu.c |1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/cpu/sh4/cpu.c b/cpu/sh4/cpu.c
index 52b6cfd..be410ab 100644
--- a/cpu/sh4/cpu.c
+++ b/cpu/sh4/cpu.c
@@ -23,6 +23,7 @@

 #include common.h
 #include command.h
+#include netdev.h
 #include asm/processor.h
 #include asm/cache.h

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


Re: [U-Boot] [PATCH] sh: Add support sh_eth to cpu of sh4

2009-02-23 Thread Mike Frysinger
On Tuesday 24 February 2009 02:20:40 Nobuhiro Iwamatsu wrote:
 Signed-off-by: Nobuhiro Iwamatsu iwamatsu.nobuh...@renesas.com
 ---
  cpu/sh4/cpu.c |1 +
  1 files changed, 1 insertions(+), 0 deletions(-)

 diff --git a/cpu/sh4/cpu.c b/cpu/sh4/cpu.c
 index 52b6cfd..be410ab 100644
 --- a/cpu/sh4/cpu.c
 +++ b/cpu/sh4/cpu.c
 @@ -23,6 +23,7 @@

  #include common.h
  #include command.h
 +#include netdev.h
  #include asm/processor.h
  #include asm/cache.h

this change doesnt really seem to jive with the changelog.  adding a header 
implies fixing of warning/build breakage, not Add support sh_eth to cpu of 
sh4 ...
-mike


signature.asc
Description: This is a digitally signed message part.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] sh: Add support sh_eth to cpu of sh4

2009-02-23 Thread Nobuhiro Iwamatsu
Mike Frysinger wrote:
 On Tuesday 24 February 2009 02:20:40 Nobuhiro Iwamatsu wrote:
 Signed-off-by: Nobuhiro Iwamatsu iwamatsu.nobuh...@renesas.com
 ---
  cpu/sh4/cpu.c |1 +
  1 files changed, 1 insertions(+), 0 deletions(-)

 diff --git a/cpu/sh4/cpu.c b/cpu/sh4/cpu.c
 index 52b6cfd..be410ab 100644
 --- a/cpu/sh4/cpu.c
 +++ b/cpu/sh4/cpu.c
 @@ -23,6 +23,7 @@

  #include common.h
  #include command.h
 +#include netdev.h
  #include asm/processor.h
  #include asm/cache.h
 
 this change doesnt really seem to jive with the changelog.  adding a header 
 implies fixing of warning/build breakage, not Add support sh_eth to cpu of 
 sh4 ...
 -mike
Thanks,
I will change changelog and resend.

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


Re: [U-Boot] [PATCH 4/9 v3] 83xx, i2c: add mux support for fsl_i2c

2009-02-23 Thread Heiko Schocher
Hello Kim,

Kim Phillips wrote:
 On Thu, 19 Feb 2009 17:24:09 +0100
 Heiko Schocher h...@denx.de wrote:
[...]
  unsigned int i2c_get_bus_num(void)
  {
 +#if defined(CONFIG_I2C_MUX)
 +return i2c_bus_num_mux;
 +#else
  return i2c_bus_num;
 +#endif
  }
 
 I don't get this mux variant - why aren't we reusing i2c_bus_num in the
 mux case?

Because i2c_bus_num is used as an index which hardware i2c controller
is used (0 or 1). In CONFIG_I2C_MUX case, you have more than 2 i2c
busses - i2c_bus_num would be greater than 1, so you must have a
variable, where you store which hardware adapter you use, and one
which stores on which i2c bus you are.

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