Re: [U-Boot] [PATCH v2 7/8] arm/km: change default settings for egiga on mgcoge3un

2011-05-04 Thread Prafulla Wadaskar
> -Original Message- > From: Valentin Longchamp [mailto:valentin.longch...@keymile.com] > Sent: Tuesday, May 03, 2011 7:43 PM > To: u-boot@lists.denx.de > Cc: Prafulla Wadaskar; holger.bru...@keymile.com; Valentin Longchamp; > Wolfgang Denk; Detlev Zundel > Subject: [PATCH v2 7/8] arm/km:

Re: [U-Boot] [PATCH v2 6/6] TFTP: add tftpsrv command

2011-05-04 Thread Mike Frysinger
On Mon, Apr 18, 2011 at 12:19, Luca Ceresoli wrote: > +int do_tftpsrv(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) static -mike ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH v3 2/3] arm: omap: innovator: Prepare for mach-types.h changes

2011-05-04 Thread Igor Grinberg
This board used machine_is_* macros for identifying the arch number. Fix this by using CONFIG_MACH_TYPE configuration variable. Signed-off-by: Igor Grinberg --- v2: remove the ifdeferry by introducing config variable v3: do not introduce yet another CONFIG_, use existing CONFIG_MA

[U-Boot] Documenting the CONFIG_MACH_TYPE variable

2011-05-04 Thread Igor Grinberg
Hi Wolfgang, While looking at how to document the CONFIG_MACH_TYPE variable, I caught myself wondering, why don't we provide an option to set this variable in board config file and set the bi_arch_number field in a more generic location (e.g. arch/arm/lib/board.c: board_init_f()), then it is

Re: [U-Boot] [PATCH v2 6/8] arm/km: rename mgcoge2un to mgcoge3un

2011-05-04 Thread Prafulla Wadaskar
> -Original Message- > From: Valentin Longchamp [mailto:valentin.longch...@keymile.com] > Sent: Tuesday, May 03, 2011 7:43 PM > To: u-boot@lists.denx.de > Cc: Prafulla Wadaskar; holger.bru...@keymile.com; Valentin Longchamp; > Wolfgang Denk; Detlev Zundel > Subject: [PATCH v2 6/8] arm/km:

Re: [U-Boot] [PATCH v2 3/8] arm/km: add BootROM config file for memphis SDRAM

2011-05-04 Thread Prafulla Wadaskar
> -Original Message- > From: Valentin Longchamp [mailto:valentin.longch...@keymile.com] > Sent: Tuesday, May 03, 2011 7:43 PM > To: u-boot@lists.denx.de > Cc: Prafulla Wadaskar; holger.bru...@keymile.com; Valentin Longchamp; > Wolfgang Denk; Detlev Zundel > Subject: [PATCH v2 3/8] arm/km:

Re: [U-Boot] [PATCH v2 4/8] arm/km: disable ls (through jffs2 support)

2011-05-04 Thread Prafulla Wadaskar
> -Original Message- > From: Valentin Longchamp [mailto:valentin.longch...@keymile.com] > Sent: Tuesday, May 03, 2011 7:43 PM > To: u-boot@lists.denx.de > Cc: Prafulla Wadaskar; holger.bru...@keymile.com; Valentin Longchamp; > Wolfgang Denk; Detlev Zundel > Subject: [PATCH v2 4/8] arm/km:

Re: [U-Boot] [PATCH v2 2/8] arm/km: remove last_stage_init and unneeded printouts

2011-05-04 Thread Prafulla Wadaskar
> -Original Message- > From: Valentin Longchamp [mailto:valentin.longch...@keymile.com] > Sent: Tuesday, May 03, 2011 7:43 PM > To: u-boot@lists.denx.de > Cc: Prafulla Wadaskar; holger.bru...@keymile.com; Valentin Longchamp; > Wolfgang Denk; Detlev Zundel > Subject: [PATCH v2 2/8] arm/km:

Re: [U-Boot] [PATCH v2 2/8] arm/km: remove last_stage_init and unneeded printouts

2011-05-04 Thread Valentin Longchamp
Prafulla Wadaskar wrote: > >> -Original Message- >> From: Valentin Longchamp [mailto:valentin.longch...@keymile.com] >> Sent: Tuesday, May 03, 2011 7:43 PM >> To: u-boot@lists.denx.de >> Cc: Prafulla Wadaskar; holger.bru...@keymile.com; Valentin Longchamp; >> Wolfgang Denk; Detlev Zundel >

Re: [U-Boot] [PATCH v2 3/6] TFTP: rename "server" to "remote"

2011-05-04 Thread Luca Ceresoli
Detlev Zundel wrote: > > I'll start a new thread to discuss this. Hopefully we then come up with > a policy to stick into that wiki page. > Now that the checkpatch policy is much more clear, I started to clean up the networking stuff, on top of which the TFTP server sits. net/net.c alone counts

Re: [U-Boot] [PATCH v1] PPC405EX CHIP_21 erratum

2011-05-04 Thread Eibach, Dirk
Thanks, I can confirm this fixes the issue for me (405EX Rev. D with security). Still checkpatch finds some whitespace issues: .dotest/patch:13: space before tab in indent. defined(CONFIG_405EX_CHIP21_PVR_REV_D) && \ .dotest/patch:27: trailing whitespace. * warning: 2 lines add

Re: [U-Boot] [PATCH RESEND 2/2] mx5: Fix CONFIG_OF_LIBFDT redefined warning

2011-05-04 Thread Stefano Babic
On 05/04/2011 08:13 AM, Shawn Guo wrote: > With the following commit, CONFIG_OF_LIBFDT is redefined. > > 2fa8ca98c37d5b1bb0328b19ddb7e9d162cd9683 > Add CONFIG_OF_LIBFDT to more boards. > > Remove the duplicated definition to fix CONFIG_OF_LIBFDT redefined > warning. > > Signed-off-by: Shawn

Re: [U-Boot] [PATCH v2 3/8] arm/km: add BootROM config file for memphis SDRAM

2011-05-04 Thread Valentin Longchamp
Prafulla Wadaskar wrote: > >> -Original Message- >> From: Valentin Longchamp [mailto:valentin.longch...@keymile.com] >> Sent: Tuesday, May 03, 2011 7:43 PM >> To: u-boot@lists.denx.de >> Cc: Prafulla Wadaskar; holger.bru...@keymile.com; Valentin Longchamp; >> Wolfgang Denk; Detlev Zundel >

Re: [U-Boot] [PATCH v2 7/8] arm/km: change default settings for egiga on mgcoge3un

2011-05-04 Thread Valentin Longchamp
Prafulla Wadaskar wrote: > >> -Original Message- >> From: Valentin Longchamp [mailto:valentin.longch...@keymile.com] >> Sent: Tuesday, May 03, 2011 7:43 PM >> To: u-boot@lists.denx.de >> Cc: Prafulla Wadaskar; holger.bru...@keymile.com; Valentin Longchamp; >> Wolfgang Denk; Detlev Zundel >

Re: [U-Boot] [PATCH v2 6/8] arm/km: rename mgcoge2un to mgcoge3un

2011-05-04 Thread Valentin Longchamp
Prafulla Wadaskar wrote: > >> -Original Message- >> From: Valentin Longchamp [mailto:valentin.longch...@keymile.com] >> Sent: Tuesday, May 03, 2011 7:43 PM >> To: u-boot@lists.denx.de >> Cc: Prafulla Wadaskar; holger.bru...@keymile.com; Valentin Longchamp; >> Wolfgang Denk; Detlev Zundel >

Re: [U-Boot] [PATCH v1] PPC405EX CHIP_21 erratum

2011-05-04 Thread Stefan Roese
Hi Steve, On Tuesday 03 May 2011 18:59:00 Steven A. Falco wrote: > APM errata CHIP_21 for the 405EX/EXr (from the rev 1.09 document dated > 4/27/11) states that rev D processors may wake up with the wrong feature > set. I've personally seen that happen. This patch implements the > APM-proposed w

Re: [U-Boot] [PATCH RESEND 2/2] mx5: Fix CONFIG_OF_LIBFDT redefined warning

2011-05-04 Thread Shawn Guo
On Wed, May 04, 2011 at 10:04:42AM +0200, Stefano Babic wrote: > On 05/04/2011 08:13 AM, Shawn Guo wrote: > > With the following commit, CONFIG_OF_LIBFDT is redefined. > > > > 2fa8ca98c37d5b1bb0328b19ddb7e9d162cd9683 > > Add CONFIG_OF_LIBFDT to more boards. > > > > Remove the duplicated defin

Re: [U-Boot] [PATCH RESEND 2/2] mx5: Fix CONFIG_OF_LIBFDT redefined warning

2011-05-04 Thread Stefano Babic
On 05/04/2011 10:44 AM, Shawn Guo wrote: > As you can see from the commit below, > > 2fa8ca98c37d5b1bb0328b19ddb7e9d162cd9683 > Add CONFIG_OF_LIBFDT to more boards. > > CONFIG_OF_LIBFDT was defined with '1' in all those files. > > --- > include/configs/ca9x4_ct_vxp.h |2 ++ > include

[U-Boot] u-boot on x86 reloaded

2011-05-04 Thread Rudolf Marek
Hello, I was asked to post it here too. I would like to announce that I was able to boot Linux using u-boot on AMD64 motherboard Asrock 939A785G. The development was done on QEMU. How does it work? Coreboot does the complete HW init including memory setup PCI etc. When finishes it loads u-bo

[U-Boot] [PATCH] nand_spl: nand_boot.c: Remove CONFIG_SYS_NAND_READ_DELAY

2011-05-04 Thread Stefan Roese
There are multiple reasons why this define should be removed: First it saves some space and therefore fixes a problem we have on the canyonlands_nand and glacier_nand targets right now. Second, the define was hackish and would most likely not work on all board using nand_boot.c. Boards not provid

[U-Boot] [PATCH] nand_spl: nand_boot.c: Init nand_chip.options to 0

2011-05-04 Thread Stefan Roese
Patch 65a9db7b [nand_spl: Fix large page nand_command()] broke nand booting on canyonlands. "options" has to be initialized to 0. If not, boards might have the NAND_BUSWIDTH_16 bit set, resulting in wrong offset calculation. Signed-off-by: Stefan Roese Cc: Scott Wood Cc: Alex Waterman --- nand

Re: [U-Boot] [U-Boot-Users] U-Boot WinCE booting support?

2011-05-04 Thread Detlev Zundel
Hi, > On Tuesday, May 03, 2011 08:43:39 AM Balaji P wrote: >> Hi >> I am working in Arm based board which can support both linux and >> wince. We need unified bootloader to boot both. Please provide me some >> help. > > CCing uboot mailing list ... I can't help you more We do not have suppo

Re: [U-Boot] [PATCH v2 3/6] TFTP: rename "server" to "remote"

2011-05-04 Thread Detlev Zundel
Hi Luca, > Detlev Zundel wrote: >> >> I'll start a new thread to discuss this. Hopefully we then come up with >> a policy to stick into that wiki page. >> > > Now that the checkpatch policy is much more clear, I started to clean up > the networking stuff, on top of which the TFTP server sits. Th

Re: [U-Boot] u-boot on x86 reloaded

2011-05-04 Thread Graeme Russ
On 04/05/11 18:54, Rudolf Marek wrote: > Hello, > > I was asked to post it here too. > > I would like to announce that I was able to boot Linux using u-boot on AMD64 > motherboard Asrock 939A785G. The development was done on QEMU. Excellent - U-Boot on a new x86 board > > How does it work? >

[U-Boot] [PATCH v2 0/8] km/common mainlining

2011-05-04 Thread Holger Brunck
This series is the second effort of merging the Keymile boards support back into mainline. This serie is a smaller serie based on the first patch serie: http://lists.denx.de/pipermail/u-boot/2011-April/090013.html The first large serie was split up to three independent series, which is easier to re

[U-Boot] [PATCH v2 2/8] common/hush: make get_local_var visible for other users

2011-05-04 Thread Holger Brunck
We can actually only set from c-code local hush shell variables, with this patch we can also read them from c-code. Signed-off-by: Holger Brunck Signed-off-by: Valentin Longchamp Acked-by: Heiko Schocher cc: Wolfgang Denk cc: Detlev Zundel --- Changes for v2: - split up first large patch s

[U-Boot] [PATCH v2 3/8] km/common: implement setboardid command

2011-05-04 Thread Holger Brunck
Read out board id and HW key from the IVM eeprom and set these values as an environment variable. Signed-off-by: Holger Brunck Acked-by: Heiko Schocher cc: Wolfgang Denk cc: Detlev Zundel cc: Valentin Longchamp --- Changes for v2: - split up first large patch serie to three independent sma

[U-Boot] [PATCH v2 1/8] km/common: fix initial_boot_bank for bootpackages

2011-05-04 Thread Holger Brunck
From: Thomas Herzmann The initial_boot_bank can be set when more than one application is used in a bootpackage. But a value n <> 0 never led to booting from bank n. Instead, bank 0 was booted. This patch fixes this. Signed-off-by: Thomas Herzmann Signed-off-by: Holger Brunck Signed-off-by: Val

[U-Boot] [PATCH v2 4/8] km/common: implement boardId HWkey checks as u-boot cmd

2011-05-04 Thread Holger Brunck
From: Thomas Herzmann BoardId and HWKey are used to identify the HW class of a given board. The correct values are stored in the inventory eeprom. During creation time of a boot package the boardId and HWkey for the SW is stored in the default environment and burned into the flash. During boottim

[U-Boot] [PATCH v2 5/8] km/common: fix coding style issues in generic header

2011-05-04 Thread Holger Brunck
Signed-off-by: Holger Brunck Signed-off-by: Valentin Longchamp Acked-by: Heiko Schocher cc: Wolfgang Denk cc: Detlev Zundel --- Changes for v2: - split up first large patch serie to three independent smaller patch series - no change in the content of this patch include/configs/key

[U-Boot] [PATCH v2 6/8] keymile boards: move keymile specific header in subdir

2011-05-04 Thread Holger Brunck
From: Valentin Longchamp Collect all keymile specific common headers in include/configs/km. Signed-off-by: Heiko Schocher Signed-off-by: Valentin Longchamp cc: Wolfgang Denk cc: Detlev Zundel cc: Prafulla Wadaskar cc: Kim Phillips cc: Holger Brunck --- Changes for v2: - split up first

[U-Boot] [PATCH v2 7/8] km/common: simplify default environment

2011-05-04 Thread Holger Brunck
This is a first step to simplify the default environment. Move all the environment variables which are only needed for debugging purpose to textfiles in the scripts directory. In case of debugging these files can be loaded via tftp into RAM and set via the env import command. Other variables are id

[U-Boot] [PATCH v2 8/8] km/common: add pnvramsize to default environment

2011-05-04 Thread Holger Brunck
The pnvram size was used later from start scripts in linux. Therefore it was added to the default environment. Signed-off-by: Holger Brunck Signed-off-by: Valentin Longchamp Acked-by: Heiko Schocher cc: Wolfgang Denk cc: Detlev Zundel --- Changes for v2: - split up first large patch serie

Re: [U-Boot] u-boot on x86 reloaded

2011-05-04 Thread Rudolf Marek
Hello all, > Excellent - U-Boot on a new x86 board Yes and in general any supported coreboot board. > This is one of the ways I was thinking of getting U-Boot running on > mainstream PC boards. The other is to create U-Boot as a GRUB payload. > Either way, it's all about getting the SDRAM initia

[U-Boot] [PATCH 0/8] net/net.c: clenups for checkpatch compliance

2011-05-04 Thread Luca Ceresoli
Hi, net/net.c currently raises a lot of checkpatch issues: 76 errors, 197 warnings. This patchset fixes most of them, leaving 0 errors and 25 warnings. Since the changes are quite massive I split them in separate patches, one per each category of errors/warning. The 25 warnings still present are

[U-Boot] [PATCH 2/8] net/net.c: cosmetic: variable initializations

2011-05-04 Thread Luca Ceresoli
This removes the following checkpatch errors: - ERROR: do not initialise globals to 0 or NULL - ERROR: spaces required around that '=' (ctx:VxV) - ERROR: that open brace { should be on the previous line Signed-off-by: Luca Ceresoli Cc: Wolfgang Denk Cc: Ben Warren --- net/net.c | 42 +

[U-Boot] [PATCH 1/8] net/net.c: cosmetic: fix lines over 80 characters

2011-05-04 Thread Luca Ceresoli
This removes the following checkpatch warning: - WARNING: line over 80 characters There is still one such warning that is hard to fix with cosmetic-only changes without compromising code readability, so I', leaving it as it is for now: WARNING: line over 80 characters #1537: FILE: net.c:1537:

[U-Boot] [PATCH 3/8] net/net.c: cosmetic: fix whitespace issues

2011-05-04 Thread Luca Ceresoli
This removes the following checkpatch issues: - ERROR: space prohibited after that open parenthesis '(' - ERROR: space prohibited before that close parenthesis ')' - ERROR: space prohibited after that open square bracket '[' - ERROR: space prohibited after that '&' (ctx:WxW) - ERROR: spaces re

[U-Boot] [PATCH 4/8] net/net.c: cosmetic: fix brace issues

2011-05-04 Thread Luca Ceresoli
This removes the following checkpatch issues: - WARNING: braces {} are not necessary for single statement blocks - WARNING: braces {} are not necessary for any arm of this statement Signed-off-by: Luca Ceresoli Cc: Wolfgang Denk Cc: Ben Warren --- net/net.c | 38 +--

[U-Boot] [PATCH 5/8] net/net.c: cosmetic: fix pointer syntax issues

2011-05-04 Thread Luca Ceresoli
This removes the following checkpatch issues: - ERROR: "foo * bar" should be "foo *bar" - ERROR: "(foo*)" should be "(foo *)" Signed-off-by: Luca Ceresoli Cc: Wolfgang Denk Cc: Ben Warren --- net/net.c | 36 ++-- 1 files changed, 18 insertions(+), 18 deletio

[U-Boot] [PATCH 7/8] net/net.c: cosmetic: fix indentation

2011-05-04 Thread Luca Ceresoli
This removes the following checkpatch issues: - ERROR: switch and case should be at the same indent - WARNING: suspect code indent for conditional statements - WARNING: labels should not be indented Signed-off-by: Luca Ceresoli Cc: Wolfgang Denk Cc: Ben Warren --- net/net.c | 85 +

[U-Boot] [PATCH 8/8] net/net.c: cosmetic: do not use assignment in if condition

2011-05-04 Thread Luca Ceresoli
This removes the following checkpatch issue: - ERROR: do not use assignment in if condition Signed-off-by: Luca Ceresoli Cc: Wolfgang Denk Cc: Ben Warren --- net/net.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/net/net.c b/net/net.c index 13efa67..c17d071 10064

[U-Boot] [PATCH 6/8] net/net.c: cosmetic: parentheses not required for return

2011-05-04 Thread Luca Ceresoli
This removes the following checkpatch issue: - ERROR: return is not a function, parentheses are not required Signed-off-by: Luca Ceresoli Cc: Wolfgang Denk Cc: Ben Warren --- net/net.c | 30 +++--- 1 files changed, 15 insertions(+), 15 deletions(-) diff --git a/net/

[U-Boot] [PATCH] Decreases code size of the nand_spl

2011-05-04 Thread Alex Waterman
>From b59f1e5b0bc3684615756c12fd5c5f9fcaa4c812 Mon Sep 17 00:00:00 2001 From: Alex Waterman Date: Tue, 3 May 2011 15:00:23 -0400 Subject: [PATCH] Decreases code size of the nand_spl The canyonland boards nand_spl size is just under the maximum 4KByte size. This patch decreases the size of the nan

Re: [U-Boot] [PATCH 2/8] net/net.c: cosmetic: variable initializations

2011-05-04 Thread Mike Frysinger
On Wed, May 4, 2011 at 08:40, Luca Ceresoli wrote: >  - ERROR: that open brace { should be on the previous line >... > -uchar NetCDPAddr[6] = > - { 0x01, 0x00, 0x0c, 0xcc, 0xcc, 0xcc }; > +uchar NetCDPAddr[6] = { > + 0x01, 0x00, 0x0c, 0x

Re: [U-Boot] [PATCH 2/2] powerpc/85xx: verify the device tree before booting Linux

2011-05-04 Thread Kumar Gala
On May 3, 2011, at 5:20 PM, Scott Wood wrote: > On Tue, 3 May 2011 10:10:18 -0500 > Kumar Gala wrote: > >> >> On May 3, 2011, at 9:41 AM, Timur Tabi wrote: >> >>> On May 3, 2011, at 9:21 AM, Kumar Gala wrote: >>> > > + > +ft_verify_fdt(*of_flat_tree); Do we n

Re: [U-Boot] [PATCH 2/2] powerpc/85xx: verify the device tree before booting Linux

2011-05-04 Thread Timur Tabi
On May 4, 2011, at 8:40 AM, Kumar Gala wrote: > >> >> This seems to have a higher potential for false positives than a checksum. > > My feeling is the feature should not have false positives, the goal should be > to be as good as a checksum. I believe my v2 version of this patch satisfies th

Re: [U-Boot] [PATCH] Decreases code size of the nand_spl

2011-05-04 Thread Stefan Roese
Hi Alex, On Wednesday 04 May 2011 15:10:15 Alex Waterman wrote: > From b59f1e5b0bc3684615756c12fd5c5f9fcaa4c812 Mon Sep 17 00:00:00 2001 > From: Alex Waterman > Date: Tue, 3 May 2011 15:00:23 -0400 > Subject: [PATCH] Decreases code size of the nand_spl > > The canyonland boards nand_spl size is

[U-Boot] Add nand_spl support for TI OMAP Beagle PATCH status?

2011-05-04 Thread Simon Schwarz
hi list, there was a patch series in december last year with the subject "Add nand_spl support for TI OMAP Beagle". There was some discussion but then I haven't heard anything further. Is someone still working on it? Thanks! Simon ___ U-Boot mailing l

Re: [U-Boot] [PATCH] Decreases code size of the nand_spl

2011-05-04 Thread Alex Waterman
Stefan, > The 2nd patch fixes the size problem as well. So no need for your patch any > more. Ahh, I saw the first but missed the second I guess. > BTW: What platform/SoC are you using? I am using a custom board based on the AMCC sequoia board. At some point I plan to try and mainline our po

Re: [U-Boot] [PATCH v1] PPC405EX CHIP_21 erratum

2011-05-04 Thread Steven A. Falco
On 05/04/2011 04:36 AM, Stefan Roese wrote: > Hi Steve, > > On Tuesday 03 May 2011 18:59:00 Steven A. Falco wrote: >> APM errata CHIP_21 for the 405EX/EXr (from the rev 1.09 document dated >> 4/27/11) states that rev D processors may wake up with the wrong feature >> set. I've personally seen tha

[U-Boot] [PATCH v2] PPC405EX CHIP_21 erratum

2011-05-04 Thread Steven A. Falco
v2: Correct checkpatch errors/warnings re: whitespace, comment style, etc. Move PVR logic out of board config file. Simplify ifdef structure. APM errata CHIP_21 for the 405EX/EXr (from the rev 1.09 document dated 4/27/11) states that rev D processors may wake up with the wrong feature set. I've p

[U-Boot] [PATCH v3 0/8] keymile arm boards update, part 2

2011-05-04 Thread Valentin Longchamp
This series is the second effort of merging the Keymile boards support back into mainline. This series is a subset of the first patch series: http://lists.denx.de/pipermail/u-boot/2011-April/090013.html The first large series was split up to three independent series, which are easier to review and

[U-Boot] [PATCH v3 5/8] arm/km: add second serial interface for kirkwood

2011-05-04 Thread Valentin Longchamp
From: Holger Brunck Signed-off-by: Holger Brunck Signed-off-by: Valentin Longchamp Acked-by: Heiko Schocher cc: Wolfgang Denk cc: Detlev Zundel cc: Prafulla Wadaskar --- Changes for v2: - split up first large patch series to three independent smaller patch series include/configs/k

[U-Boot] [PATCH v3 1/8] arm/km: remove CONFIG_SYS_KWD_CONFIG from keymile-common.h

2011-05-04 Thread Valentin Longchamp
This define is marvell specific, so it should be present in km_arm. It is however not needed there either, since we set it to the default value that is already set in include/asm/arch-kirkwood/config.h Signed-off-by: Valentin Longchamp cc: Holger Brunck cc: Prafulla Wadaskar cc: Heiko Schocher

[U-Boot] [PATCH v3 8/8] arm/km: update mgcoge3un board support

2011-05-04 Thread Valentin Longchamp
From: Holger Brunck We change default settings for egiga on mgcoge3un. The reason we need this is that we have the gig port on mgcoge3un connected using a back-to-back pair of PHYs. There are no magnetics and because of that the port has to be run with a fixd configuration and auto-negotiation mu

[U-Boot] [PATCH v3 2/8] arm/km: move CONFIG_EXTRA_ENV_SETTINGS from board to km_arm file

2011-05-04 Thread Valentin Longchamp
Since all the boards define the same env settings, this simplifies the board files. Signed-off-by: Valentin Longchamp cc: Holger Brunck cc: Heiko Schocher cc: Wolfgang Denk cc: Detlev Zundel cc: Prafulla Wadaskar --- changes for v3: - this patch is new in the series include/configs

[U-Boot] [PATCH v3 3/8] arm/km: introduce bootcount env variable and clean km_arm

2011-05-04 Thread Valentin Longchamp
This environment variable is used to set the bootcount address for the kernel. last_stage_init is not available for arm platforms. So the calls to set_km_var and set_bootcount_addr are done in misc_init_r. Additionally some unneeded printouts were removed. Signed-off-by: Valentin Longchamp Sign

[U-Boot] [PATCH v3 4/8] arm/km: disable ls (through jffs2 support)

2011-05-04 Thread Valentin Longchamp
This is not supported on our km-arm boards since we have defined CONFIG_SYS_NO_FLASH for our NAND Flash chip. With CONFIG_CMD_JFFS2, the ls command is present and works very badly on our km-arm boards. Signed-off-by: Valentin Longchamp Signed-off-by: Holger Brunck Acked-by: Heiko Schocher Acke

[U-Boot] [PATCH v3 7/8] mvgbe: enable configurability of PORT_SERIAL_CONTROL_VALUE

2011-05-04 Thread Valentin Longchamp
This allows this configuration to be defined differently for some boards that request it. Signed-off-by: Valentin Longchamp cc: Holger Brunck cc: Heiko Schocher cc: Wolfgang Denk cc: Detlev Zundel cc: Prafulla Wadaskar --- Changes for v2: - split up first large patch series to three indep

[U-Boot] [PATCH v3 6/8] arm/km: rename mgcoge2un to mgcoge3un

2011-05-04 Thread Valentin Longchamp
The mgcoge2un target was only an intermediate step to mgcoge3un. For this reason the mgcoge2un support was moved to mgcoge3un, because it isn't needed to support both targets. We add the BootROM init file for the mgcoge3un memphis RAM. We also move the suen3 and suen8 boards into the correct cate

Re: [U-Boot] [PATCH 2/8] net/net.c: cosmetic: variable initializations

2011-05-04 Thread Luca Ceresoli
Mike Frysinger wrote: > On Wed, May 4, 2011 at 08:40, Luca Ceresoli wrote: >> - ERROR: that open brace { should be on the previous line >> ... >> -uchar NetCDPAddr[6] = >> - { 0x01, 0x00, 0x0c, 0xcc, 0xcc, 0xcc }; >> +uchar NetCDPAddr[6] = { >> +

Re: [U-Boot] [PATCH] nand_spl: nand_boot.c: Init nand_chip.options to 0

2011-05-04 Thread Scott Wood
On Wed, 4 May 2011 11:44:44 +0200 Stefan Roese wrote: > Patch 65a9db7b [nand_spl: Fix large page nand_command()] broke > nand booting on canyonlands. "options" has to be initialized to > 0. If not, boards might have the NAND_BUSWIDTH_16 bit set, > resulting in wrong offset calculation. > > Signe

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

2011-05-04 Thread Haiying Wang
On Wed, 2011-05-04 at 22:53 +0530, Poonam Aggrwal wrote: > +sinclude $(obj).depend > + > +# > diff --git a/nand_spl/board/freescale/p1010rdb/nand_boot.c > b/nand_spl/board/freescale/p1010rdb/nand_boot.c > new file mode 100644

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

2011-05-04 Thread Kumar Gala
On May 4, 2011, at 12:31 PM, Haiying Wang wrote: > On Wed, 2011-05-04 at 22:53 +0530, Poonam Aggrwal wrote: >> +sinclude $(obj).depend >> + >> +# >> diff --git a/nand_spl/board/freescale/p1010rdb/nand_boot.c >> b/nand_spl/bo

Re: [U-Boot] [PATCH] Decreases code size of the nand_spl

2011-05-04 Thread Scott Wood
On Wed, 4 May 2011 15:47:27 +0200 Stefan Roese wrote: > Hi Alex, > > On Wednesday 04 May 2011 15:10:15 Alex Waterman wrote: > > From b59f1e5b0bc3684615756c12fd5c5f9fcaa4c812 Mon Sep 17 00:00:00 2001 > > From: Alex Waterman > > Date: Tue, 3 May 2011 15:00:23 -0400 > > Subject: [PATCH] Decreases

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

2011-05-04 Thread Haiying Wang
On Wed, 2011-05-04 at 12:34 -0500, Kumar Gala wrote: > >> + > >> +#define udelay(x) {int i, j; for (i = 0; i < x; i++) for (j = 0; j < > >> 1; j++); } > > There were many comments on this udelay before, we should not use this > > define, but use the udelay() which u-boot provides. > > > > Is

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

2011-05-04 Thread Scott Wood
On Wed, 4 May 2011 12:34:20 -0500 Kumar Gala wrote: > > On May 4, 2011, at 12:31 PM, Haiying Wang wrote: > > > On Wed, 2011-05-04 at 22:53 +0530, Poonam Aggrwal wrote: > >> +sinclude $(obj).depend > >> + > >> +# > >> diff -

Re: [U-Boot] [PATCH] Decreases code size of the nand_spl

2011-05-04 Thread Alex Waterman
Scott, > Or we could apply both and save even more space, delaying the next time we > run into trouble. :-) Since the spl is so limited in space, would it be worth making every function in the spl use a local function pointer instead of lots of dereferences? - Alex

Re: [U-Boot] [PATCH] Decreases code size of the nand_spl

2011-05-04 Thread Scott Wood
On Wed, 4 May 2011 14:24:15 -0400 Alex Waterman wrote: > > Scott, > > > Or we could apply both and save even more space, delaying the next time we > > run into trouble. :-) > > Since the spl is so limited in space, would it be worth making every function > in the spl use a local function point

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

2011-05-04 Thread Andy Fleming
>> >> + >> >> +#define udelay(x) {int i, j; for (i = 0; i < x; i++) for (j = 0; j < >> >> 1; j++); } >> > There were many comments on this udelay before, we should not use this >> > define, but use the udelay() which u-boot provides. >> > >> >> Is there a udelay that is defined for the nand_sp

Re: [U-Boot] [PATCH v2] Decreases code size of the nand_spl

2011-05-04 Thread Alex Waterman
This patch decreases the code size of the nand_spl by turning multiple function pointer dereferences in a single function into a single local function pointer. Signed-off-by: Alex Waterman Cc: Scott Wood Cc: Stefan Roese --- This works on my local board but I haven't tested it on anything else

Re: [U-Boot] [PATCH] ppc, mgcoge: add DIP switch detection

2011-05-04 Thread Wolfgang Denk
Dear Holger Brunck, In message <1295951175-3635-1-git-send-email-holger.bru...@keymile.com> you wrote: > From: Andreas Huber > > This reads the DIP switch on mgcoge. The DIP switch is connected to > the BFTICU (0x4089) FPGA. If the DIP switch is set the environment > variable 'actual_bank'

[U-Boot] [PATCH] cmd_nvedit.c: make error message more helpful

2011-05-04 Thread Wolfgang Denk
When calling getenv_f() with a too small buffer, it would print an error message like this: env_buf too small [32] This is not really helpful as it does not give any indication which of the calls might have failed. Change this into: env_buf [32 bytes] too small for value of "hwc

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

2011-05-04 Thread Scott Wood
On Wed, 4 May 2011 14:15:58 -0500 Andy Fleming wrote: > >> >> + > >> >> +#define udelay(x) {int i, j; for (i = 0; i < x; i++) for (j = 0; j < > >> >> 1; j++); } > >> > There were many comments on this udelay before, we should not use this > >> > define, but use the udelay() which u-boot prov

Re: [U-Boot] [PATCH V3 3/6] mpc85xx boards: initdram() cleanup/bugfix

2011-05-04 Thread Wolfgang Denk
Dear Becky Bruce, In message <1292627879-9027-4-git-send-email-bec...@kernel.crashing.org> you wrote: > Correct initdram to use phys_size_t to represent the size of > dram; instead of changing this all over the place, and correcting > all the other random errors I've noticed, create a > common in

[U-Boot] [PATCH] Fix incorrect use of getenv() before relocation

2011-05-04 Thread Wolfgang Denk
A large number of boards incorrectly used getenv() in their board init code running before relocation. In some cases this caused U-Boot to hang when certain environment variables grew too long. Fix the code to use getenv_r(). Signed-off-by: Wolfgang Denk Cc: Stefan Roese Cc: The LEOX team Cc:

Re: [U-Boot] [PATCH v2 4/8] km/common: implement boardId HWkey checks as u-boot cmd

2011-05-04 Thread Wolfgang Denk
Dear Holger Brunck, In message <02d1e3f265123ff0296b1c38b227f6d30393ee77.1304508448.git.holger.bru...@keymile.com> you wrote: > From: Thomas Herzmann > > BoardId and HWKey are used to identify the HW class of a given board. > The correct values are stored in the inventory eeprom. During creati

Re: [U-Boot] Documenting the CONFIG_MACH_TYPE variable

2011-05-04 Thread Wolfgang Denk
Dear Igor Grinberg, In message <4dc0ffb0.5080...@compulab.co.il> you wrote: > > While looking at how to document the CONFIG_MACH_TYPE variable, > I caught myself wondering, why don't we provide an option to set > this variable in board config file and set the bi_arch_number field in > a more gene

Re: [U-Boot] [PATCH v2 7/8] arm/km: change default settings for egiga on mgcoge3un

2011-05-04 Thread Wolfgang Denk
Dear Prafulla Wadaskar, In message you wrote: > > It will be good if you split this patch, the mvgbe.h change would be generi= > c, independent and can be merged faster. > > mgcoge3un.h changes would be part of your new board config. When requesting changes like here, could you (and all other

Re: [U-Boot] [PATCH v2 6/8] arm/km: rename mgcoge2un to mgcoge3un

2011-05-04 Thread Wolfgang Denk
Dear Valentin Longchamp, In message <4dc10fec.1050...@keymile.com> you wrote: > > The current differences between our kirkwood boards are not many, but > since we have new boards coming where the difference number is going to > grow, it is going to be hard to keep many (even 2) boards in one sin

Re: [U-Boot] [PATCH v2 3/8] arm/km: add BootROM config file for memphis SDRAM

2011-05-04 Thread Wolfgang Denk
Dear Valentin Longchamp, In message <4dc1092e.4080...@keymile.com> you wrote: > > > You are just adding file here, whereas it is not being used. Is it like > > dead code to me. > > Yes, it is not used with the u-boot build system, we have a small script > that wraps around it. As this "small s

Re: [U-Boot] [PATCH 1/2] NS16550: buffer reads

2011-05-04 Thread Simon Glass
On Wed, Apr 6, 2011 at 2:28 PM, Scott Wood wrote: > On Wed, 6 Apr 2011 22:42:12 +0200 > Wolfgang Denk wrote: > > > Dear Scott Wood, > > > > In message <20110406203012.ga30...@schlenkerla.am.freescale.net> you > wrote: > > > This improves the performance of U-Boot when accepting rapid input, > >

Re: [U-Boot] [PATCH v3 1/4] Add support for SMSC95XX USB 2.0 10/100MBit Ethernet Adapter

2011-05-04 Thread Simon Glass
On Sat, Apr 30, 2011 at 10:50 AM, Rabin Vincent wrote: > On Thu, Apr 14, 2011 at 04:03, Simon Glass wrote: > > On Wed, Apr 13, 2011 at 12:16 PM, Andy Fleming > wrote: > >>> + read_buf &= ~HW_CFG_RXDOFF_; > >>> + > >>> +#ifdef CONFIG_TEGRA2 > >>> + /* Tegra2 requires NET_IP_ALIGN = 0

[U-Boot] usbtty corruption on omap3

2011-05-04 Thread Charles Manning
Hi All I recently moved from a really old uboot to git master of a few days back to be able to get the benefit of dcache/icache Unfortunately I'm now seeing data corruption on the serial stream using usbtty. Anyone else seeing this problem? Thanks Charles

Re: [U-Boot] [PATCH v2 00/10] armv7: cache maintenance operations

2011-05-04 Thread Simon Glass
> > > On Tue, Mar 8, 2011 at 5:07 AM, Aneesh V wrote: > >> With D-cache and MMU enabled for ARM in u-boot it becomes imperative to >> support a minimal set of cache maintenance operations and necessary >> initializations before enabling MMU. >> >> This series of patches attempt to do the following

Re: [U-Boot] usbtty corruption on omap3

2011-05-04 Thread Charles Manning
On Thursday 05 May 2011 13:44:01 Charles Manning wrote: > Hi All > > I recently moved from a really old uboot to git master of a few days back > to be able to get the benefit of dcache/icache > > Unfortunately I'm now seeing data corruption on the serial stream using > usbtty. > > Anyone else seein

Re: [U-Boot] Documenting the CONFIG_MACH_TYPE variable

2011-05-04 Thread Igor Grinberg
Hi Wolfgang, On 05/05/11 01:14, Wolfgang Denk wrote: > Dear Igor Grinberg, > > In message <4dc0ffb0.5080...@compulab.co.il> you wrote: >> While looking at how to document the CONFIG_MACH_TYPE variable, >> I caught myself wondering, why don't we provide an option to set >> this variable in board c

Re: [U-Boot] boot-up time optimization. Where to start?

2011-05-04 Thread Wolfgang Denk
Dear Alexander Stein, In message <201105030848.17576.alexander.st...@systec-electronic.com> you wrote: > > This specific version was selected due to relocation problems on ARM. But I > expect the dcache doesn't have that big influence on the named code part as > the environment is already in RA

Re: [U-Boot] [PATCH v2 6/8] arm/km: rename mgcoge2un to mgcoge3un

2011-05-04 Thread Valentin Longchamp
Dear Wolfgang Denk, Wolfgang Denk wrote: > Dear Valentin Longchamp, > > In message <4dc10fec.1050...@keymile.com> you wrote: >> The current differences between our kirkwood boards are not many, but >> since we have new boards coming where the difference number is going to >> grow, it is going t

Re: [U-Boot] [PATCH v2 3/8] arm/km: add BootROM config file for memphis SDRAM

2011-05-04 Thread Valentin Longchamp
Wolfgang Denk wrote: > Dear Valentin Longchamp, > > In message <4dc1092e.4080...@keymile.com> you wrote: >>> You are just adding file here, whereas it is not being used. Is it like >>> dead code to me. >> Yes, it is not used with the u-boot build system, we have a small script >> that wraps arou