Re: [U-Boot] [PATCH] saveenv: standardize enablement

2009-01-28 Thread Wolfgang Denk
Dear Mike, In message <1230623965-16546-1-git-send-email-vap...@gentoo.org> you wrote: > Rather than special casing each environment type for enabling the saveenv > command, have them all behave the same. This avoids bitrot as new env > sources are added/removed. > > Signed-off-by: Mike Frysinge

Re: [U-Boot] Pull request: nand flash

2009-01-28 Thread Wolfgang Denk
Dear Scott & Dave, In message <20090123170833.ga12...@ld0162-tx32.am.freescale.net> you wrote: > The following changes since commit 4d0b54685c5c656023b826089ef8cc0ea1c5cd9e: > Haavard Skinnemoen (1): > Merge branch 'fixes' > > are available in the git repository at: > > git://git.den

[U-Boot] [PATCH] mpc8536ds.c: include sata.h to for needed function prototypes

2009-01-28 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk --- board/freescale/mpc8536ds/mpc8536ds.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/board/freescale/mpc8536ds/mpc8536ds.c b/board/freescale/mpc8536ds/mpc8536ds.c index 1e2e4e6..eb80500 100644 --- a/board/freescale/mpc8536ds/mpc8536ds.c

Re: [U-Boot] [PATCH/RFC, 0/2] I2C rework -- what do you think?

2009-01-28 Thread Heiko Schocher
Hello ksi, k...@koi8.net schrieb: > On Tue, 27 Jan 2009, Heiko Schocher wrote: >> k...@koi8.net wrote: >>> Hi everyone, >>> >>> Here is the first draft of major I2C subsystem rework. This is not a >> patch [...] >>> the new code. It is a big job and I don't want to spend a lot of time >> doing >>>

Re: [U-Boot] Please pull u-boot-mpc85xx.git

2009-01-28 Thread Wolfgang Denk
Dear Andy & Haiying, in message <1232753424-2863-1-git-send-email-aflem...@freescale.com> you wrote: > are available in the git repository at: ... > Haiying Wang (2): > Add README file for MPC8572DS board > Add secondary CPUs processor frequency for e500 core ^^

Re: [U-Boot] Pull request: nand flash

2009-01-28 Thread Liu Dave
> > Dave Liu (2): > > mpc83xx: enable eLBC NAND support for MPC8315ERDB board > > NAND: Fix cache and memory inconsistency issue > > > This commit (commit c70564e6b1bd08f) broke the SIMPC8313_LP board: > > nand_boot_fsl_elbc.o:

Re: [U-Boot] [PATCH] saveenv: standardize enablement

2009-01-28 Thread Mike Frysinger
On Wednesday 28 January 2009 03:17:06 Wolfgang Denk wrote: > In message <1230623965-16546-1-git-send-email-vap...@gentoo.org> you wrote: > > Rather than special casing each environment type for enabling the saveenv > > command, have them all behave the same. This avoids bitrot as new env > > sourc

Re: [U-Boot] Please pull u-boot-mpc85xx.git

2009-01-28 Thread Wolfgang Denk
Dear Andy & Kumar, In message <1232753424-2863-1-git-send-email-aflem...@freescale.com> you wrote: > are available in the git repository at: ... > Kumar Gala (10): ... > 85xx: Add a 36-bit physical configuration for MPC8572DS This patch (commit 18af1c5f0f74) raises a couple of warnings, for

Re: [U-Boot] Pull request: nand flash

2009-01-28 Thread Wolfgang Denk
Dear Dave, In message you wrote: > > nand_boot_fsl_elbc.o: In function `nand_boot': > > nand_spl/board/sheldon/simpc8313/nand_boot_fsl_elbc.c:150:=20 > > undefined reference to `flush_cache' > > make[1]: *** [/home/wd/git/u-boot/work/nand_spl/u-boot-spl] Error 1 > > make: *** [nand_spl] Error 2

Re: [U-Boot] [PATCH] mpc83xx: fix undefined reference to `flush_cache' error in simpc8313 build

2009-01-28 Thread Wolfgang Denk
Dear Kim Phillips, In message <20090127160353.ec184554.kim.phill...@freescale.com> you wrote: > extend commit c70564e6b1bd08f3230182392238907f3531a87e > "NAND: Fix cache and memory inconsistency issue" to add the cache.o dependency > to the simpc8313 build and fix this: > > ...Large Page NAND...C

Re: [U-Boot] [PATCH] saveenv: standardize enablement

2009-01-28 Thread Wolfgang Denk
Dear Mike Frysinger, In message <200901280350.10310.vap...@gentoo.org> you wrote: > > > This patch breaks support for the > > looks like the actual board name was missed in your e-mail ? Oops. ... for the SPD823TS board. Thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH

Re: [U-Boot] Please pull u-boot-mpc85xx.git

2009-01-28 Thread Stefan Roese
Hi Wolfgang, On Wednesday 28 January 2009, Wolfgang Denk wrote: > This patch (commit 18af1c5f0f74) raises a couple of warnings, for > example: > > -> ./MAKEALL MPC8572DS_36BIT > ... enabling 36-bit physical addressing. > Configuring for MPC8572DS board... > cfi_flash.c: In function 'flash_init': >

[U-Boot] [PATCH 00/31] updates for keymile boards

2009-01-28 Thread Heiko Schocher
following Patch series makes a lot of updates for the existing keymile boards mgsuvd, mgcoge and kmeter1. Also they add a new board from this manufacturer based on a MPC852T. Heiko Schocher (31): powerpc: 8xx serial: add configurable SMC Rx buffer len environment, netconsole: reducing

[U-Boot] [PATCH 01/31] powerpc: 8xx serial: add configurable SMC Rx buffer len

2009-01-28 Thread Heiko Schocher
This patch adds the configuration option CONFIG_SYS_SMC_RXBUFLEN. With this option it is possible to allow the receive buffer for the SMC on 8xx to be greater then 1. In case CONFIG_SYS_SMC_RXBUFLEN == 1 or it is not defined this driver works as the old version. Signed-off-by: Heiko Schocher ---

[U-Boot] [PATCH 02/31] environment, netconsole: reducing packages when printing environment variables

2009-01-28 Thread Heiko Schocher
Actual, do_printenv () uses putc () to print the Environment- variables. If using netconsole, this will result in a lot of packages with single characters in it. With the new config option CONFIG_ENV_BUFFER_PRINT do_printenv () uses now puts () to print a complete Environmentvariable, so the number

[U-Boot] [PATCH 03/31] netloop: speed up NetLoop

2009-01-28 Thread Heiko Schocher
NetLoop polls every cycle with getenv some environment variables. This is horribly slow, especially when the environment is big. This patch reads only the environment variables in NetLoop, when they were changed. Signed-off-by: Heiko Schocher --- common/cmd_nvedit.c |6 +++ include/common.h

[U-Boot] [PATCH 06/31] mpc82xx, mgcoge: add CS4 on mgcoge

2009-01-28 Thread Heiko Schocher
This patch adds CS4 on mgcoge to enable the FPGA Signed-off-by: Heiko Schocher --- include/configs/mgcoge.h | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/include/configs/mgcoge.h b/include/configs/mgcoge.h index 12be62a..f772627 100644 --- a/include/configs

[U-Boot] [PATCH 07/31] mpc83xx kmeter1: change some register settings

2009-01-28 Thread Heiko Schocher
Signed-off-by: Heiko Schocher --- include/configs/kmeter1.h | 25 ++--- 1 files changed, 14 insertions(+), 11 deletions(-) diff --git a/include/configs/kmeter1.h b/include/configs/kmeter1.h index 7b78a82..f9c01f5 100644 --- a/include/configs/kmeter1.h +++ b/include/configs/

[U-Boot] [PATCH 08/31] powerpc, keymile boards: added cmdline feature

2009-01-28 Thread Heiko Schocher
added the cmdline features: - auto-completion - history - editing within the cmdline Signed-off-by: Thomas Herzmann Signed-off-by: Heiko Schocher --- include/configs/keymile-common.h |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/include/configs/keymile-common.h b/

[U-Boot] [PATCH 09/31] powerpc: mgcoge, mgsuvd: RxBuffer for serial input increased to 128 byte

2009-01-28 Thread Heiko Schocher
powerpc: mgcoge, mgsuvd: RxBuffer for serial input increased to 128 byte The size of the serial RXBuffer is increased from 16 to 128 bytes to enable comfortable copy-and-paste of commands with the mouse Signed-off-by: Thomas Herzmann Signed-off-by: Heiko Schocher --- include/configs/mgcoge.h |

[U-Boot] [PATCH 11/31] powerpc, mgcoge, mgsuvd: MTD feature activated, partitions set

2009-01-28 Thread Heiko Schocher
- activated the MTD and JFFS2 cmdline-features - set the default partition table to meet the needs of the keymile boot concept - increased the u-boot size for mgsuvd from 256k to 384k (will be needed soon) Signed-off-by: Thomas Herzmann Signed-off-by: Heiko Schocher --- include/configs/keym

[U-Boot] [PATCH 10/31] 8xx, mgsuvd: size and position of environment corrected

2009-01-28 Thread Heiko Schocher
adapted the position and size of the environment sector to match the defines in mgcoge, kmeter1 and make them dependant of the MONITOR_LEN instead of an absolute address Signed-off-by: Thomas Herzmann Signed-off-by: Heiko Schocher --- include/configs/mgsuvd.h |6 +++--- 1 files changed, 3 i

[U-Boot] [PATCH 12/31] 82xx, mgcoge: corrected wrong flash sizes

2009-01-28 Thread Heiko Schocher
flash size of mgcoge corrected boot to 4M (was 32) cfg to 32M (was 64) Signed-off-by: Thomas Herzmann Signed-off-by: Heiko Schocher --- include/configs/mgcoge.h |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/configs/mgcoge.h b/include/configs/mgcoge.h in

[U-Boot] [PATCH 13/31] powerpc: mgcoge, mgsuvd: move IVM feature config to keymile-common.h

2009-01-28 Thread Heiko Schocher
the IVM feature is enabled for all keymile boards therefore the configuration parameters are trans- ferred from the board specific config files to keymile-common.h. Signed-off-by: Thomas Herzmann Signed-off-by: Heiko Schocher --- include/configs/keymile-common.h | 20 inc

[U-Boot] [PATCH 14/31] powerpc: mgcoge, mgsuvd: make some definitions in keymile-common.h conditional

2009-01-28 Thread Heiko Schocher
in order to be able to "patch" some definitions of keymile's config files very easy, these definitions are surrounded by ifdefs Signed-off-by: Thomas Herzmann Signed-off-by: Heiko Schocher --- include/configs/keymile-common.h | 31 ++- include/configs/mgcoge.h

[U-Boot] [PATCH 16/31] dtt: call dtt_init () conditional in init_sequence

2009-01-28 Thread Heiko Schocher
If CONFIG_SYS_DTT_BOARD_INIT is defined, dtt_init () is not called in the init_sequence. Signed-off-by: Heiko Schocher --- include/dtt.h |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/include/dtt.h b/include/dtt.h index 399b64a..1c8eda7 100644 --- a/include/dtt.h +++ b

[U-Boot] [PATCH 17/31] mpc83xx, i2c: add mux support for fsl_i2c

2009-01-28 Thread Heiko Schocher
Signed-off-by: Heiko Schocher --- 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

[U-Boot] [PATCH 18/31] mpc83xx, kmeter1: add I2C, dtt, eeprom support

2009-01-28 Thread Heiko Schocher
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 --- board/keymile/common/common.c|4 +++- board/keym

[U-Boot] [PATCH 19/31] i2c, mpc83xx: add CONFIG_SYS_I2C_INIT_BOARD for fsl_i2c

2009-01-28 Thread Heiko Schocher
This patch adds the possibility to call a board specific i2c bus reset routine for the fsl_i2c bus driver, and adds this option for the keymile kmeter1 board. Signed-off-by: Heiko Schocher --- board/keymile/common/common.c | 14 ++ drivers/i2c/fsl_i2c.c |6 ++ inclu

[U-Boot] [PATCH 20/31] mpc83xx, kmeter1: extract common I2C options in keymile header

2009-01-28 Thread Heiko Schocher
Signed-off-by: Heiko Schocher --- include/configs/keymile-common.h |8 include/configs/kmeter1.h| 19 --- 2 files changed, 4 insertions(+), 23 deletions(-) diff --git a/include/configs/keymile-common.h b/include/configs/keymile-common.h index 99c3380..4ff6f

[U-Boot] [PATCH 22/31] mpc83xx, kmeter1, mtd: set the default partition table

2009-01-28 Thread Heiko Schocher
Signed-off-by: Heiko Schocher --- include/configs/kmeter1.h |9 + 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/include/configs/kmeter1.h b/include/configs/kmeter1.h index d66ba8b..feb6150 100644 --- a/include/configs/kmeter1.h +++ b/include/configs/kmeter1.h @@ -322

[U-Boot] [PATCH 21/31] mpc83xx, kmeter1: autodetect size of DDR II Ram.

2009-01-28 Thread Heiko Schocher
Signed-off-by: Heiko Schocher --- board/keymile/kmeter1/kmeter1.c | 31 +-- include/configs/kmeter1.h |4 ++-- 2 files changed, 23 insertions(+), 12 deletions(-) diff --git a/board/keymile/kmeter1/kmeter1.c b/board/keymile/kmeter1/kmeter1.c index 27f0998..

[U-Boot] [PATCH 23/31] powerpc: mgcoge, mgsuvd, kmeter1: activate pram data area

2009-01-28 Thread Heiko Schocher
This patch activates a protected area (pram) that will not be used by the linux kernel. This is done for all keymile boards Signed-off-by: Thomas Herzmann Signed-off-by: Heiko Schocher --- include/configs/keymile-common.h | 20 +--- include/configs/kmeter1.h|2 ++

[U-Boot] [PATCH 24/31] mgcoge, mgsuvd, kmeter1: removed double env params

2009-01-28 Thread Heiko Schocher
This patch removes double environment entries for all keymile boards (mgcoge, mgsuvd, kmeter1) These double entries (baudrate, bootcmd) otherwise cause errors. Signed-off-by: Thomas Herzmann Signed-off-by: Heiko Schocher --- include/configs/keymile-common.h |2 -- 1 files changed, 0 inserti

[U-Boot] [PATCH 25/31] powerpc: mgcoge, mgsuvd, kmeter1: board Id parameters

2009-01-28 Thread Heiko Schocher
This patch leads to correct use of the boardId parameters for the boards mgcoge, mgsuvd, kmeter1 when downloading a new esw script via tftp Signed-off-by: Thomas Herzmann Signed-off-by: Heiko Schocher --- include/configs/keymile-common.h |6 -- 1 files changed, 4 insertions(+), 2 deleti

[U-Boot] [PATCH 26/31] powerpc: mgcoge, mgsuvd, kmeter1: dont print errormsg if second IVM Block lacks..

2009-01-28 Thread Heiko Schocher
The second IVM inventory block for the keymile boards is optional only, so dont print an error message, if this block dont exists. Signed-off-by: Heiko Schocher --- board/keymile/common/common.c |7 --- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/board/keymile/common/c

[U-Boot] [PATCH 27/31] 8xx, mgsuvd: rename board to a more generic name

2009-01-28 Thread Heiko Schocher
renaming the "mgsuvd" board port into "km8xx", because there will come more similar boards from keymile. Compiling the mgsuvd board with "make mgsuvd_config" remains. Signed-off-by: Heiko Schocher --- Makefile |8 ++-- board/keymile/{mgsuvd => km

[U-Boot] [PATCH 29/31] 8xx, kmsupx4: add bootcount support

2009-01-28 Thread Heiko Schocher
Signed-off-by: Heiko Schocher --- include/configs/keymile-common.h |2 +- include/configs/km8xx.h |4 +--- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/include/configs/keymile-common.h b/include/configs/keymile-common.h index 20f7bab..3736cd8 100644 --- a/includ

[U-Boot] [PATCH 28/31] 8xx: add support for new keymile kmsupx4 board.

2009-01-28 Thread Heiko Schocher
This patch adds support for the kmsupx4 board from Keymile, based on a Freescale MPC852T CPU - serial console on SMC1 - 32 MB SDRAM - 32 MB NOR Flash - Ethernet over SCC3 - I2C Bitbang Signed-off-by: Heiko Schocher --- Makefile |9 +- board/keymile/common/common.

[U-Boot] [PATCH 30/31] 8xx, km8xx: update in dts node "flash", the reg value correct.

2009-01-28 Thread Heiko Schocher
Signed-off-by: Heiko Schocher --- board/keymile/km8xx/km8xx.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/board/keymile/km8xx/km8xx.c b/board/keymile/km8xx/km8xx.c index 3f0e65f..a117845 100644 --- a/board/keymile/km8xx/km8xx.c +++ b/board/keymile/km8xx/km8xx.c @@

[U-Boot] [PATCH 31/31] 83xx, kmeter1: added bootcount feature

2009-01-28 Thread Heiko Schocher
add CONFIG_BOUTCOUNT_LIMIT feature for 83xx CPU. activate this feature for the kmeter1 board. Signed-off-by: Heiko Schocher --- cpu/mpc83xx/cpu.c| 27 +++ include/configs/keymile-common.h |2 -- 2 files changed, 27 insertions(+), 2 deletions(-) diff

Re: [U-Boot] IP_t should be a "packed" struct

2009-01-28 Thread Luigi 'Comio' Mantellini
Hi ML, I noticed that also the others headers structs have the same potential problem. I attached the patch to fix the net.h header file. best regards, luigi On Tuesday 27 January 2009 19:32:10 Luigi 'Comio' Mantellini wrote: > Hi ML, > > I'm working on a mips target and I used qemu_mips targe

[U-Boot] [PATCH 15/31] powerpc: keymile boards: conditional definition of a private env part

2009-01-28 Thread Heiko Schocher
by defining CONFIG_KM_DEF_ENV_PRIVATE somewhere the content of CONFIG_KM_DEF_ENV_PRIVATE will be included into the default environment Signed-off-by: Thomas Herzmann Signed-off-by: Heiko Schocher --- include/configs/keymile-common.h |8 +++- 1 files changed, 7 insertions(+), 1 deletions

[U-Boot] [PATCH 05/31] powerpc: extract common config options for keymile boards.

2009-01-28 Thread Heiko Schocher
This patch extracts common config options for keymile boards to the keymile-common.h file. Additionally FLASH_PROTECTION is added. Signed-off-by: Heiko Schocher --- include/configs/keymile-common.h | 84 ++ include/configs/kmeter1.h| 58 ---

[U-Boot] [PATCH 04/31] powerpc: 82xx serial: add configurable SMC Rx buffer len

2009-01-28 Thread Heiko Schocher
This patch adds the configuration option CONFIG_SYS_SMC_RXBUFLEN. With this option it is possible to allow the receive buffer for the SMC on 82xx to be greater then 1. In case CONFIG_SYS_SMC_RXBUFLEN == 1 or it is not defined this driver works as the old version. Signed-off-by: Heiko Schocher ---

Re: [U-Boot] [PATCH 02/31] environment, netconsole: reducing packages when printing environment variables

2009-01-28 Thread Stefan Roese
Hi Heiko, On Wednesday 28 January 2009, Heiko Schocher wrote: > Actual, do_printenv () uses putc () to print the Environment- > variables. If using netconsole, this will result in a lot of > packages with single characters in it. With the new config > option CONFIG_ENV_BUFFER_PRINT do_printenv ()

Re: [U-Boot] [PATCH 19/31] i2c, mpc83xx: add CONFIG_SYS_I2C_INIT_BOARD for fsl_i2c

2009-01-28 Thread Heiko Schocher
Hello Joakim, Joakim Tjernlund wrote: >> This patch adds the possibility to call a board specific >> i2c bus reset routine for the fsl_i2c bus driver, and adds >> this option for the keymile kmeter1 board. >> [...] >> @@ -478,6 +480,17 @@ static int i2c_make_abort (void) >> */ >> void i2c_init_

Re: [U-Boot] [PATCH 02/31] environment, netconsole: reducing packages when printing environment variables

2009-01-28 Thread Heiko Schocher
Hello Stefan, Stefan Roese wrote: > On Wednesday 28 January 2009, Heiko Schocher wrote: >> Actual, do_printenv () uses putc () to print the Environment- >> variables. If using netconsole, this will result in a lot of >> packages with single characters in it. With the new config >> option CONFIG_EN

Re: [U-Boot] [PATCH 19/31] i2c, mpc83xx: add CONFIG_SYS_I2C_INIT_BOARD for fsl_i2c

2009-01-28 Thread Joakim Tjernlund
> > This patch adds the possibility to call a board specific > i2c bus reset routine for the fsl_i2c bus driver, and adds > this option for the keymile kmeter1 board. > > Signed-off-by: Heiko Schocher > --- > board/keymile/common/common.c | 14 ++ > drivers/i2c/fsl_i2c.c |

Re: [U-Boot] Please pull u-boot-mpc85xx.git

2009-01-28 Thread Wolfgang Denk
Dear Stefan Roese, In message <200901281007.47146...@denx.de> you wrote: > > This problem has nothing to do with the patch mentioned above. I addressed Git bisect says so... > this warning with my patch "jedec_flash: Fix compilation warning in > jedec_flash.c" which is still under discussion.

Re: [U-Boot] [PATCH 02/31] environment, netconsole: reducing packages when printing environment variables

2009-01-28 Thread Wolfgang Denk
Dear Heiko Schocher, In message <49803324.3000...@denx.de> you wrote: > > > Why not fold this "putc()" into the puts() above (replace it with printf)? > > I make it in a printf (Think there was no reason for not doing it so) Check if printf() is not printing a series of single characters again!

Re: [U-Boot] [PATCH 03/31] netloop: speed up NetLoop

2009-01-28 Thread Wolfgang Denk
Dear Heiko Schocher, In message <498027a2.8060...@denx.de> you wrote: > NetLoop polls every cycle with getenv some environment variables. > This is horribly slow, especially when the environment is big. > > This patch reads only the environment variables in NetLoop, > when they were changed. ...

Re: [U-Boot] [PATCH 03/31] netloop: speed up NetLoop

2009-01-28 Thread Heiko Schocher
Hello Wolfgang, Wolfgang Denk wrote: > In message <498027a2.8060...@denx.de> you wrote: >> NetLoop polls every cycle with getenv some environment variables. >> This is horribly slow, especially when the environment is big. >> >> This patch reads only the environment variables in NetLoop, >> when t

Re: [U-Boot] [PATCH 19/31] i2c, mpc83xx: add CONFIG_SYS_I2C_INIT_BOARD for fsl_i2c

2009-01-28 Thread Joakim Tjernlund
Heiko Schocher wrote on 28/01/2009 11:54:22: > Hello Joakim, > > Joakim Tjernlund wrote: > >> This patch adds the possibility to call a board specific > >> i2c bus reset routine for the fsl_i2c bus driver, and adds > >> this option for the keymile kmeter1 board. > >> > [...] > >> @@ -478,6 +480,

Re: [U-Boot] [PATCH 02/31] environment, netconsole: reducing packages when printing environment variables

2009-01-28 Thread Stefan Roese
On Wednesday 28 January 2009, Wolfgang Denk wrote: > > > Why not fold this "putc()" into the puts() above (replace it with > > > printf)? > > > > I make it in a printf (Think there was no reason for not doing it so) > > Check if printf() is not printing a series of single characters again! > Also,

Re: [U-Boot] [PATCH 04/31] powerpc: 82xx serial: add configurable SMC Rx buffer len

2009-01-28 Thread Wolfgang Denk
Dear Heiko Schocher, In message <498027a8.3010...@denx.de> you wrote: > This patch adds the configuration option CONFIG_SYS_SMC_RXBUFLEN. We discussed this before. I explained that I do not want to have this added level of complexity in the driver. Why should I reconsider? > With this option i

Re: [U-Boot] [PATCH 08/31] powerpc, keymile boards: added cmdline feature

2009-01-28 Thread Wolfgang Denk
Dear Heiko Schocher, In message <498027c5.9070...@denx.de> you wrote: > added the cmdline features: > - auto-completion > - history > - editing within the cmdline Umm... a commit message "added cmdline feature" is pretty much confusing. Please come up with a better summary. Best regards, Wol

Re: [U-Boot] [PATCH 12/31] 82xx, mgcoge: corrected wrong flash sizes

2009-01-28 Thread Wolfgang Denk
Dear Heiko Schocher, In message <498027d6.3070...@denx.de> you wrote: > flash size of mgcoge corrected > boot to 4M (was 32) > cfg to 32M (was 64) > > Signed-off-by: Thomas Herzmann > Signed-off-by: Heiko Schocher > --- > include/configs/mgcoge.h |4 ++-- > 1 files changed, 2 insertion

Re: [U-Boot] [PATCH 19/31] i2c, mpc83xx: add CONFIG_SYS_I2C_INIT_BOARD for fsl_i2c

2009-01-28 Thread Heiko Schocher
Hello Joakim, Joakim Tjernlund wrote: > Heiko Schocher wrote on 28/01/2009 11:54:22: >> Joakim Tjernlund wrote: This patch adds the possibility to call a board specific i2c bus reset routine for the fsl_i2c bus driver, and adds this option for the keymile kmeter1 board. >> [..

Re: [U-Boot] Please pull u-boot-mpc85xx.git

2009-01-28 Thread Stefan Roese
Hi Wolfgang, On Wednesday 28 January 2009, Wolfgang Denk wrote: > > This problem has nothing to do with the patch mentioned above. I > > addressed > > Git bisect says so... Strange. Are you sure? The code in question (line 219 in jedec_flash.c) was added with patch 94f9279f7bbdc01bbc7cf85aedf9b5

Re: [U-Boot] [PATCH 10/31] 8xx, mgsuvd: size and position of environment corrected

2009-01-28 Thread Wolfgang Denk
Dear Heiko Schocher, In message <498027ce.6080...@denx.de> you wrote: > adapted the position and size of the environment > sector to match the defines in mgcoge, kmeter1 > and make them dependant of the MONITOR_LEN > instead of an absolute address > > Signed-off-by: Thomas Herzmann > Signed-off-

Re: [U-Boot] [PATCH 16/31] dtt: call dtt_init () conditional in init_sequence

2009-01-28 Thread Wolfgang Denk
Dear Heiko Schocher, In message <498027ea.5070...@denx.de> you wrote: > If CONFIG_SYS_DTT_BOARD_INIT is defined, dtt_init () is > not called in the init_sequence. > > Signed-off-by: Heiko Schocher > --- > include/dtt.h |2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --gi

Re: [U-Boot] [PATCH 26/31] powerpc: mgcoge, mgsuvd, kmeter1: dont print errormsg if second IVM Block lacks..

2009-01-28 Thread Wolfgang Denk
Dear Heiko Schocher, In message <4980281a.1010...@denx.de> you wrote: > The second IVM inventory block for the keymile boards is optional > only, so dont print an error message, if this block dont exists. PLEASE provide somewhat usable commit messages. Keep in mind that the Subject line is used

Re: [U-Boot] [PATCH 02/31] environment, netconsole: reducing packages when printing environment variables

2009-01-28 Thread Wolfgang Denk
Dear Stefan Roese, In message <200901281256.14264...@denx.de> you wrote: > > > Check if printf() is not printing a series of single characters again! > > Also, we tend to prefer using puts() / putc() over printf() because of > > the huge performance overhead involved in printf() when only plain >

Re: [U-Boot] [PATCH 09/31] powerpc: mgcoge, mgsuvd: RxBuffer for serial input increased to 128 byte

2009-01-28 Thread Wolfgang Denk
Dear Heiko Schocher, In message <498027cb.8030...@denx.de> you wrote: > powerpc: mgcoge, mgsuvd: RxBuffer for serial input increased to 128 byte > > The size of the serial RXBuffer is increased from > 16 to 128 bytes to enable comfortable > copy-and-paste of commands with the mouse Why do you in

Re: [U-Boot] [PATCH 04/31] powerpc: 82xx serial: add configurable SMC Rx buffer len

2009-01-28 Thread Heiko Schocher
Hello Wolfgang, Wolfgang Denk wrote: > In message <498027a8.3010...@denx.de> you wrote: >> This patch adds the configuration option CONFIG_SYS_SMC_RXBUFLEN. > > We discussed this before. I explained that I do not want to have this > added level of complexity in the driver. > > Why should I recon

Re: [U-Boot] [PATCH 04/31] powerpc: 82xx serial: add configurable SMC Rx buffer len

2009-01-28 Thread Wolfgang Denk
Dear Heiko Schocher, In message <498050aa.6070...@denx.de> you wrote: > > > We discussed this before. I explained that I do not want to have this > > added level of complexity in the driver. > > > > Why should I reconsider? > > Hups, seems I missed something (Seems I stepped into this subject,

Re: [U-Boot] [PATCH 09/31] powerpc: mgcoge, mgsuvd: RxBuffer for serial input increased to 128 byte

2009-01-28 Thread Heiko Schocher
Hello Wolfgang, Wolfgang Denk wrote: > In message <498027cb.8030...@denx.de> you wrote: >> powerpc: mgcoge, mgsuvd: RxBuffer for serial input increased to 128 byte >> >> The size of the serial RXBuffer is increased from >> 16 to 128 bytes to enable comfortable >> copy-and-paste of commands with th

Re: [U-Boot] Watchdog timer and booting in Linux kernel

2009-01-28 Thread Sachin Rane
Hi, Thanks for the reply. >Why don't you use the existing code for example for 440EPx from our kernel tree? Are you pointing to the code which comes under 'CONFIG_BOOKE_WDT' in 'arch/powerpc/kernel/traps.c'? If not, can you give me the location of the 'example' code in kerenl code? Regards, S

Re: [U-Boot] [PATCH 12/31] 82xx, mgcoge: corrected wrong flash sizes

2009-01-28 Thread Heiko Schocher
Hello Wolfgang, Wolfgang Denk wrote: > In message <498027d6.3070...@denx.de> you wrote: >> flash size of mgcoge corrected >> boot to 4M (was 32) >> cfg to 32M (was 64) >> >> Signed-off-by: Thomas Herzmann >> Signed-off-by: Heiko Schocher >> --- >> include/configs/mgcoge.h |4 ++-- >> 1

Re: [U-Boot] [PATCH 10/31] 8xx, mgsuvd: size and position of environment corrected

2009-01-28 Thread Heiko Schocher
Hello Wolfgang, Wolfgang Denk wrote: > In message <498027ce.6080...@denx.de> you wrote: >> adapted the position and size of the environment >> sector to match the defines in mgcoge, kmeter1 >> and make them dependant of the MONITOR_LEN >> instead of an absolute address >> >> Signed-off-by: Thomas

Re: [U-Boot] [PATCH 19/31] i2c, mpc83xx: add CONFIG_SYS_I2C_INIT_BOARD for fsl_i2c

2009-01-28 Thread Joakim Tjernlund
> Hello Joakim, > > Joakim Tjernlund wrote: > > Heiko Schocher wrote on 28/01/2009 11:54:22: > >> Joakim Tjernlund wrote: > This patch adds the possibility to call a board specific > i2c bus reset routine for the fsl_i2c bus driver, and adds > this option for the keymile kmeter1 bo

Re: [U-Boot] [PATCH 09/31] powerpc: mgcoge, mgsuvd: RxBuffer for serial input increased to 128 byte

2009-01-28 Thread Wolfgang Denk
Dear Heiko Schocher, In message <49805339.8090...@denx.de> you wrote: > > > Why do you initialize these settins as 16 in patch N when you increase > > the value in patch N+1? This makes little sense to me. Why not using > > the right value right from the beginning? > > Because it shows a develop

Re: [U-Boot] Watchdog timer and booting in Linux kernel

2009-01-28 Thread Wolfgang Denk
Dear "Sachin Rane", In message <73fa539107892c4c8c7a5aca10c161c85be...@mx2.alphioncorp.local> you wrote: > > >Why don't you use the existing code for example for 440EPx from our > kernel tree? > > Are you pointing to the code which comes under 'CONFIG_BOOKE_WDT' in > 'arch/powerpc/kernel/traps

Re: [U-Boot] [PATCH 10/31] 8xx, mgsuvd: size and position of environment corrected

2009-01-28 Thread Wolfgang Denk
Dear Heiko Schocher, In message <498055f5.4010...@denx.de> you wrote: > > As you see in the Signed-off-by, this patch comes from the manufacturer > from the board, and he needs such a big space. I recom,mend you contact and ask him again, pointing out the performance impact he is going to suffer

Re: [U-Boot] [PATCH 03/11] drivers/pci: Create pci_map_bar function

2009-01-28 Thread Kumar Gala
On Dec 3, 2008, at 11:04 PM, Becky Bruce wrote: > It is no longer always true that the pci bus address can be > used as the virtual address for pci accesses. pci_map_bar() > is created to return the virtual address for a pci region. > > Signed-off-by: Becky Bruce > --- > drivers/pci/pci.c | 1

Re: [U-Boot] [PATCH 10/11] drivers/block/ahci: Fix pci mapping bug

2009-01-28 Thread Kumar Gala
On Dec 3, 2008, at 11:04 PM, Becky Bruce wrote: > The code assumes that the pci bus address and the virtual > address used to access a region are the same, but they might > not be. Fix this assumption. > > Signed-off-by: Becky Bruce > --- > drivers/block/ahci.c |7 ++- > 1 files changed,

Re: [U-Boot] Please pull u-boot-mpc85xx.git

2009-01-28 Thread Kumar Gala
On Jan 28, 2009, at 2:55 AM, Wolfgang Denk wrote: > Dear Andy & Kumar, > > In message <1232753424-2863-1-git-send-email-aflem...@freescale.com> > you wrote: >> are available in the git repository at: > ... >> Kumar Gala (10): > ... >> 85xx: Add a 36-bit physical configuration for MPC8572DS

[U-Boot] [PATCH] 85xx: Fix compile breakage with MPC8540EVAL

2009-01-28 Thread Kumar Gala
Configuring for MPC8540EVAL board... mpc8540eval.c: In function 'checkboard': mpc8540eval.c:53: error: invalid operands to binary / make[1]: *** [mpc8540eval.o] Error 1 Signed-off-by: Kumar Gala --- board/mpc8540eval/mpc8540eval.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff

Re: [U-Boot] Please pull u-boot-mpc85xx.git

2009-01-28 Thread Haiying Wang
On Wed, 2009-01-28 at 09:33 +0100, Wolfgang Denk wrote: > Dear Andy & Haiying, > > in message <1232753424-2863-1-git-send-email-aflem...@freescale.com> you > wrote: > > are available in the git repository at: > ... > > Haiying Wang (2): > > Add README file for MPC8572DS board > > Add

Re: [U-Boot] [PATCH] 85xx: Fix compile breakage with MPC8540EVAL

2009-01-28 Thread Haiying Wang
Acked-by: Haiying Wang On Wed, 2009-01-28 at 08:31 -0600, Kumar Gala wrote: > Configuring for MPC8540EVAL board... > mpc8540eval.c: In function 'checkboard': > mpc8540eval.c:53: error: invalid operands to binary / > make[1]: *** [mpc8540eval.o] Error 1 > > Signed-off-by: Kumar Gala > --- > bo

Re: [U-Boot] Please pull u-boot-mpc85xx.git

2009-01-28 Thread Kumar Gala
On Jan 28, 2009, at 8:36 AM, Haiying Wang wrote: > On Wed, 2009-01-28 at 09:33 +0100, Wolfgang Denk wrote: >> Dear Andy & Haiying, >> >> in message <1232753424-2863-1-git-send-email- >> aflem...@freescale.com> you wrote: >>> are available in the git repository at: >> ... >>> Haiying Wang (2): >>

Re: [U-Boot] [PATCH] MIPS: Fix compile warning in au1x00_eth.c

2009-01-28 Thread Shinya Kuribayashi
Stefan Roese wrote: > This patch fixes this compilation warning: > > ./MAKEALL dbau1000 > Configuring for dbau1x00 board... > au1x00_eth.c: In function 'au1x00_send': > au1x00_eth.c:158: warning: passing argument 1 of 'virt_to_phys' discards > qualifiers from pointer target type > ... > > S

[U-Boot] [PATCH] Add support for the digsy MTC board.

2009-01-28 Thread Grzegorz Bernacki
This is the InterControl custom device based on the MPC5200B chip. Signed-off-by: Grzegorz Bernacki diff --git a/MAKEALL b/MAKEALL index 530fe82..530651d 100755 --- a/MAKEALL +++ b/MAKEALL @@ -47,6 +47,7 @@ LIST_5xxx=" \ BC3450 \ cm5200 \ cpci5

Re: [U-Boot] [PATCH] ppc4xx: Fix initialization of SDRAM_CODT register in ppc_4xx_ddr2.c

2009-01-28 Thread carolyn.j.smith
Okay, I will resubmit my patch. Carolyn >-Original Message- >From: Adam Graham [mailto:agra...@amcc.com] >Sent: Tuesday, January 27, 2009 11:22 PM >To: Stefan Roese; u-boot@lists.denx.de >Cc: Smith, Carolyn J; Adam Graham; Victor Gallardo >Subject: RE: [U-Boot] [PATCH] ppc4xx: Fix initial

Re: [U-Boot] [PATCH] MIPS: Fix compile warning in au1x00_eth.c

2009-01-28 Thread Stefan Roese
On Wednesday 28 January 2009, Shinya Kuribayashi wrote: > Stefan Roese wrote: > > This patch fixes this compilation warning: > > > > ./MAKEALL dbau1000 > > Configuring for dbau1x00 board... > > au1x00_eth.c: In function 'au1x00_send': > > au1x00_eth.c:158: warning: passing argument 1 of 'virt_to_ph

Re: [U-Boot] Pull request: nand flash

2009-01-28 Thread Scott Wood
Wolfgang Denk wrote: > Dear Dave, > > In message > you > wrote: >>> nand_boot_fsl_elbc.o: In function `nand_boot': >>> nand_spl/board/sheldon/simpc8313/nand_boot_fsl_elbc.c:150:=20 >>> undefined reference to `flush_cache' >>> make[1]: *** [/home/wd/git/u-boot/work/nand_spl/u-boot-spl] Error 1 >

[U-Boot] [PATCH] ppc4xx: Fixt initialization of SDRAM_CODT register in ppc_4xx_ddr2.c

2009-01-28 Thread carolyn.j.smith
This fixes the initialization of the SDRAM_CODT register in the ppc4xx DDR2 initialization code. It also removes the use the SDRAM_CODT_FEEDBACK_RCV_SINGLE_END and SDRAM_CODT_FEEDBACK_DRV_SINGLE_END #define's since they are reserved bits. Signed-off-by: Carolyn Smith --- diff -uprN old/cpu/ppc

Re: [U-Boot] [PATCH] V2 NAND_SPL support for phycore imx31to Scott Wood

2009-01-28 Thread Scott Wood
On Wed, Jan 28, 2009 at 07:43:54AM +0300, Maxim Artamonov wrote: > In message on Friday 12 December 2008 04:55:49 Scott Wood wrote: > > On Fri, Dec 05, 2008 at 02:13:51PM +0300, Maxim Artamonov wrote: > > > +#ifdef CONFIG_NAND_SPL > > > +/* somewhat macro to reduce bin size for CONFIG_NAND_SPL*/ >

Re: [U-Boot] [PATCH v2 5/9] AVR32: macb - Disable 100mbps if clock is slow

2009-01-28 Thread Jean-Christophe PLAGNIOL-VILLARD
> >> + > >> +#ifdef CONFIG_MACB_FORCE10M > >> + printf("%s: 100Mbps is not supported on this board - forcing > >> 10Mbps.\n", > >> +netdev->name); > >> + > >> + adv &= ~ADVERTISE_100FULL; > >> + adv &= ~ADVERTISE_100HALF; > >> + adv &= ~ADVERTISE_100BASE4; > >> +#endif

Re: [U-Boot] U-Boot hangs after uncompressing kernel image on MPC5200B

2009-01-28 Thread Marco Antônio Possamai
Thank you for your reply Mr. Likely! I was indeed using the wrong image file. So I've tried the uImage generated by the kernel build, and it still hangs. It might be the addresses I'm using. I've read some FAQs sections that seemed to give me some hope, but they all were related to wrong RAM addre

Re: [U-Boot] IP_t should be a "packed" struct

2009-01-28 Thread Ben Warren
Luigi 'Comio' Mantellini wrote: > Hi ML, > > I'm working on a mips target and I used qemu_mips target to simulate my > target > (that I hope to have in the next week...) > > Following my activities I noticed that IP_t structure is no defined with > attribute "packed". I noticed this issue becaus

[U-Boot] Pull request u-boot-blackfin.git

2009-01-28 Thread Mike Frysinger
The following changes since commit 2b6fd5c77db9c6ed3cea9799c86ff922cf0107b2: Kim Phillips (1): mpc83xx: fix undefined reference to `flush_cache' error in simpc8313 build are available in the git repository at: git://www.denx.de/git/u-boot-blackfin.git master Mike Frysinger (39):

Re: [U-Boot] IP_t should be a "packed" struct

2009-01-28 Thread Jerry Van Baren
Ben Warren wrote: > Luigi 'Comio' Mantellini wrote: >> Hi ML, >> >> I'm working on a mips target and I used qemu_mips target to simulate my >> target >> (that I hope to have in the next week...) >> >> Following my activities I noticed that IP_t structure is no defined with >> attribute "packed".

Re: [U-Boot] [PATCH] mpc83xx: fix undefined reference to `flush_cache' error in simpc8313 build

2009-01-28 Thread Ron Madrid
--- On Wed, 1/28/09, Wolfgang Denk wrote: > From: Wolfgang Denk > Subject: Re: [U-Boot] [PATCH] mpc83xx: fix undefined reference to > `flush_cache' error in simpc8313 build > To: "Kim Phillips" > Cc: u-boot@lists.denx.de, "Ron Madrid" > Date: Wednesday, January 28, 2009, 1:01 AM > Dear Kim Ph

[U-Boot] [PATCH] SPD823TS: do not define CONFIG_CMD_ENV

2009-01-28 Thread Mike Frysinger
Since the SPD823TS board does not actually have any writable flash to save its environment, undefine CONFIG_CMD_ENV so the "saveenv" command is disabled. This fixes the build error: common/libcommon.a(cmd_nvedit.o): In function `do_saveenv': common/cmd_nvedit.c:557: undefined reference to `saveenv

Re: [U-Boot] [PATCH] SPD823TS: do not define CONFIG_CMD_ENV

2009-01-28 Thread Mike Frysinger
On Wednesday 28 January 2009 13:48:55 Mike Frysinger wrote: > Since the SPD823TS board does not actually have any writable flash to save > its environment, undefine CONFIG_CMD_ENV so the "saveenv" command is > disabled. btw, i find the "CONFIG_CMD_ENV -> saveenv" bindings a bit confusing. any qu

Re: [U-Boot] IP_t should be a "packed" struct

2009-01-28 Thread Ben Warren
Jerry Van Baren wrote: > Ben Warren wrote: >> Luigi 'Comio' Mantellini wrote: >>> Hi ML, >>> >>> I'm working on a mips target and I used qemu_mips target to simulate >>> my target (that I hope to have in the next week...) >>> >>> Following my activities I noticed that IP_t structure is no defined

Re: [U-Boot] [PATCH] ppc4xx: Fixt initialization of SDRAM_CODT register in ppc_4xx_ddr2.c

2009-01-28 Thread Wolfgang Denk
Dear Carolyn, In message <2ba6bdb4c941c94bbd8375d55e05ccf9124be50...@us-bv-m12.global.tektronix.net> you wrote: > This fixes the initialization of the SDRAM_CODT register in the ppc4xx DDR2 > initialization code. It also removes the use the > SDRAM_CODT_FEEDBACK_RCV_SINGLE_END and SDRAM_CODT_F

Re: [U-Boot] [PATCH] mpc83xx: fix undefined reference to `flush_cache' error in simpc8313 build

2009-01-28 Thread Wolfgang Denk
Dear Ron Madrid, In message <107578.37644...@web83507.mail.sp1.yahoo.com> you wrote: > > > I went ahead and applied this to fix the build issue. > > Please complain > > if it doesn't work. > > So I take it that the build worked for you? My build failed. I didn;t re-test yet. > ./MAKEALL SIMPC

  1   2   3   >