Re: [U-Boot] ARM: OMAP3/4: proposal: Cleanup MUX

2010-11-09 Thread Premi, Sanjeev
-Original Message- From: u-boot-boun...@lists.denx.de [mailto:u-boot-boun...@lists.denx.de] On Behalf Of Nishanth Menon Sent: Tuesday, November 09, 2010 6:09 AM To: Peter Barada Cc: u-boot@lists.denx.de Subject: Re: [U-Boot] ARM: OMAP3/4: proposal: Cleanup MUX Peter Barada

Re: [U-Boot] [PATCH] Switch from archive libraries to partial linking

2010-11-09 Thread Andre Schwarz
Sebastian, drivers/qe already has an object called qe.o ... renaming qe.a to qe.o doesn't work. I'd suggest to build libqe.o and also adapt Toplevel Makefile accordingly. I also don't understand why ftd.c is not depending on CONFIG_QE. That's another question ... but leads to error having your

Re: [U-Boot] [PATCH 4/4] tools/env: use host build flags

2010-11-09 Thread Daniel Hobi
Hi Steve, On 09.11.2010 01:01, Steve Sakoman wrote: On Mon, Nov 8, 2010 at 3:15 PM, Wolfgang Denk w...@denx.de wrote: In message m2k4knd6lx@ohwell.denx.de you wrote: It just occurred to me that this will no longer cross compile the fw_env tool - effectively rendering the tool useless for

Re: [U-Boot] [PATCH] Switch from archive libraries to partial linking

2010-11-09 Thread Graeme Russ
On 09/11/10 09:04, Sebastien Carlier wrote: NOTE: This does not include the actual patch as it is too large for the mailing list (418 kB). Signed-off-by: Sebastien Carlier sebastien.carl...@gmail.com Patch applies with zero errors or warnings to my x86 branch, builds clean, runs and, as a

Re: [U-Boot] nand spl build with wrong CONFIG_SYS_TEXT_BASE

2010-11-09 Thread Haiying Wang
Dear Wolfgang, On Mon, 2010-11-08 at 16:36 -0700, Wolfgang Denk wrote: Dear Haiying Wang, Why it looks broken? I do need CONFIG_NAND defined for 8536DS nand build. For building nand uboot, 2 images are needed, one is 4K nand_spl Yes. But there is also MPC8536DS and MPC8536DS_36BIT and

Re: [U-Boot] [PATCH] [NEXT] da850evm: Add RMII support for EMAC

2010-11-09 Thread Detlev Zundel
Hi Ben, From: Sudhakar Rajashekhara sudhakar@ti.com This patch is a port of the work by Sudhakar Rajeshekhara in commit ab3effbcad8851cc65dc5241a01c064d2030a3b2 of git://arago-project.org/git/people/sandeep/u-boot-davinci.git. The da850 UI board has on it an RMII PHY which can be used

Re: [U-Boot] [WIP] tools/env: cleanup host build flags

2010-11-09 Thread Detlev Zundel
Hello Daniel, This patch makes tools/env/Makefile more similar to tools/imls: - define HOSTSRCS and HOSTCPPFLAGS, so that .depend generation works. - include U-Boot headers using -idirafter to prevent picking up u-boot/include/errno.h. - use HOSTCFLAGS_NOPED (fw_env.c does not conform to

Re: [U-Boot] [PATCH] Switch from archive libraries to partial linking

2010-11-09 Thread Stefano Babic
On 11/08/2010 11:04 PM, Sebastien Carlier wrote: NOTE: This does not include the actual patch as it is too large for the mailing list (418 kB). Your patch applies fine to i.MX branch - boards compile clean, tested on a MX.51 board. Tested-by: Stefano Babic sba...@denx.de Best regards,

Re: [U-Boot] [PATCH] [NEXT] da850evm: Add RMII support for EMAC

2010-11-09 Thread Mike Frysinger
On Friday, November 05, 2010 17:11:13 Ben Gardiner wrote: CC: Sandeep Paulraj s-paul...@ti.com spurious quote in this tag -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot mailing list U-Boot@lists.denx.de

Re: [U-Boot] ARM: OMAP3/4: proposal: Cleanup MUX

2010-11-09 Thread Nishanth Menon
Premi, Sanjeev wrote, on 11/09/2010 02:48 AM: Peter Barada wrote, on 11/05/2010 11:59 PM: Personally I'd like to see the kernel and u-boot disconnect on the Looks like we are getting to a consensus - we all seem to dislike the dependency of u-boot and kernel for mux as of today - can we

Re: [U-Boot] Why cmd_tbl_t is referenced as extern in defining __u_boot_cmd_start?

2010-11-09 Thread Mike Frysinger
On Tuesday, November 02, 2010 11:56:31 legendbb wrote: line 64 and 65 in include/command.h extern cmd_tbl_t __u_boot_cmd_start; extern cmd_tbl_t __u_boot_cmd_end; definition of cmd_tbl_t is just one line above these. Why extern is used? basic C question ? if you dont have the extern,

Re: [U-Boot] [PATCH] env_mmc: fix compile warning

2010-11-09 Thread Mike Frysinger
On Monday, November 08, 2010 01:59:48 Lei Wen wrote: Signed-off-by: Lei Wen lei...@marvell.com --- common/env_mmc.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/common/env_mmc.c b/common/env_mmc.c index 3d7fceb..7c9392c 100644 --- a/common/env_mmc.c +++

Re: [U-Boot] [PATCH 1/2] use MAC from environment and add support for new NET_MULTI api in ethernet driver of KS8695

2010-11-09 Thread Mike Frysinger
On Thursday, November 04, 2010 06:13:00 Bernstein, Michael wrote: diff --unified --recursive --new-file u-boot-2009.11/drivers/net/ks8695eth.c u-boot-2009.11,new/drivers/net/ks8695eth.c --- u-boot-2009.11/drivers/net/ks8695eth.c this is a pretty old tree. you should be using latest mainline

Re: [U-Boot] [RFC] Add 'led' command

2010-11-09 Thread Mike Frysinger
On Friday, November 05, 2010 01:50:36 Jason Kridner wrote: +int do_led ( cmd_tbl_t *cmdtp, int flag, int argc, char *argv[] ) much of the style of this code is broken. and i cant imagine this code compiling warning free with current master. no spaces around the paren, and the argv has been

Re: [U-Boot] [PATCH 2/2] use MAC from environment and add support for new NET_MULTI api in ethernet driver of KS8695

2010-11-09 Thread Mike Frysinger
On Thursday, November 04, 2010 06:15:09 Bernstein, Michael wrote: /* + * some defines to allow switching between NET_MULTI and old API + */ +#if defined(CONFIG_NET_MULTI) +#define KS8695_ETH_RESET ks8695_eth_reset +#define KS8695_ETH_HALT ks8695_eth_halt +#define KS8695_ETH_INIT

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-09 Thread Albert ARIBAUD
Le 04/11/2010 23:27, Albert ARIBAUD a écrit : Le 04/11/2010 23:22, Albert Aribaud a écrit : older ld emitted all ELF relocations in input sections named ..rel.dyn, whereas newer ld uses names of the form .rel*. The linker script only collected .rel.dyn input sections. Rewrite to collect all

Re: [U-Boot] [WIP] tools/env: cleanup host build flags

2010-11-09 Thread Steve Sakoman
On Tue, Nov 9, 2010 at 7:03 AM, Detlev Zundel d...@denx.de wrote: Hello Daniel, This patch makes tools/env/Makefile more similar to tools/imls: - define HOSTSRCS and HOSTCPPFLAGS, so that .depend generation works. - include U-Boot headers using -idirafter to prevent picking up  

[U-Boot] silly error caused by /usr/local/include/config.h - STD_END_SIZE build failure

2010-11-09 Thread Luke Kenneth Casson Leighton
http://www.mail-archive.com/u-boot-us...@lists.sourceforge.net/msg07045.html i've tracked down what the problem is, and the fix to avoid the problem occurring in the future is either: a) do as the person says - #include ../config.h not #include config.h b) add -I. to the gcc compile options, to

[U-Boot] [PATCH V2] [NEXT] da850: Add RMII support for EMAC

2010-11-09 Thread Ben Gardiner
From: Sudhakar Rajashekhara sudhakar@ti.com From: Sudhakar Rajashekhara sudhakar@ti.com This patch is a port of the work by Sudhakar Rajeshekhara in commit ab3effbcad8851cc65dc5241a01c064d2030a3b2 of git://arago-project.org/git/people/sandeep/u-boot-davinci.git. The da850 UI board has

Re: [U-Boot] nand spl build with wrong CONFIG_SYS_TEXT_BASE

2010-11-09 Thread Haiying Wang
On Tue, 2010-11-09 at 09:23 -0700, Wolfgang Denk wrote: Dear Haiying Wang, Is this all the error messages you get? Normally tehre should be another message, for example: [filename]:[lineno]:[column]: warning: this is the location of the previous definition or, if there was a

Re: [U-Boot] [PATCH] [NEXT] da850evm: Add RMII support for EMAC

2010-11-09 Thread Ben Gardiner
On Tue, Nov 9, 2010 at 8:43 AM, Mike Frysinger vap...@gentoo.org wrote: On Friday, November 05, 2010 17:11:13 Ben Gardiner wrote: CC: Sandeep Paulraj s-paul...@ti.com spurious quote in this tag -mike Oops. Thanks for catching that, Mike. On Tue, Nov 9, 2010 at 9:42 AM, Detlev Zundel

Re: [U-Boot] nand spl build with wrong CONFIG_SYS_TEXT_BASE

2010-11-09 Thread Scott Wood
On Tue, 9 Nov 2010 17:23:43 +0100 Wolfgang Denk w...@denx.de wrote: Yes, now the problem is only one define of CONFIG_SYS_TEXT_BASE is allowed for one board config,i.e CONFIG_NAND, but two images need to be generated for nand boot, one is u-boot-spl.bin, the other is u-boot.bin. They need

Re: [U-Boot] nand spl build with wrong CONFIG_SYS_TEXT_BASE

2010-11-09 Thread Wolfgang Denk
Dear Haiying Wang, In message 1289314359.2168.20.ca...@haiying-laptop you wrote: You did not answer my question. The error message shows _where_ the previous definition came from. Please check this. OK, the error message is: In file included from

Re: [U-Boot] nand spl build with wrong CONFIG_SYS_TEXT_BASE

2010-11-09 Thread Wolfgang Denk
Dear Scott Wood, In message 20101109102643.45d7f...@udp111988uds.am.freescale.net you wrote: Maybe they should use different variables then, say CONFIG_SYS_TEXT_BASE and CONFIG_SYS_TEXT_BASE_SPL ? Why? It's the same variable, just for a different build. Are we going to have _SPL and

Re: [U-Boot] nand spl build with wrong CONFIG_SYS_TEXT_BASE

2010-11-09 Thread Wolfgang Denk
Dear Haiying Wang, In message 1289320865.2168.92.ca...@haiying-laptop you wrote: It is in one of the error messages, there are others for the same error. Here is the total message when building tlb.c for nand_spl: ... powerpc-none-linux-gnuspe-gcc -g -Os -mrelocatable -fPIC -meabi

Re: [U-Boot] silly error caused by /usr/local/include/config.h - STD_END_SIZE build failure

2010-11-09 Thread Luke Kenneth Casson Leighton
geez, this really _isn't_ good - i've found _yet another_ identically-caused error, because i happen to have linux header files installed locally (/usr/include/linux) and the default HOST-CC setup compilation has that taking precedence over ./include/linux in the u-boot sources - so

Re: [U-Boot] [PATCH v2] Switch from archive libraries to partial linking

2010-11-09 Thread Sebastien Carlier
Dear André, On 2010-11-09 10:15:31, Andre Schwarz wrote: Date: Tue, 09 Nov 2010 10:15:31 +0100 From: Andre Schwarz andre.schw...@matrix-vision.de To: Sebastien Carlier sebastien.carl...@gmail.com CC: u-boot@lists.denx.de, Timur Tabi ti...@freescale.com, Kim Phillips

Re: [U-Boot] silly error caused by /usr/local/include/config.h - STD_END_SIZE build failure

2010-11-09 Thread Luke Kenneth Casson Leighton
On Tue, Nov 9, 2010 at 5:38 PM, Luke Kenneth Casson Leighton luke.leigh...@gmail.com wrote: geez, this really _isn't_ good - i've found _yet another_ identically-caused error, because i happen to have linux header files installed locally (/usr/include/linux) and the default HOST-CC setup

Re: [U-Boot] nand spl build with wrong CONFIG_SYS_TEXT_BASE

2010-11-09 Thread Scott Wood
On Tue, 9 Nov 2010 18:18:53 +0100 Wolfgang Denk w...@denx.de wrote: Dear Scott Wood, In message 20101109102643.45d7f...@udp111988uds.am.freescale.net you wrote: Maybe they should use different variables then, say CONFIG_SYS_TEXT_BASE and CONFIG_SYS_TEXT_BASE_SPL ? Why? It's the

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-09 Thread Albert ARIBAUD
Le 09/11/2010 19:24, Daniel Hobi a écrit : Hi Albert, On 04.11.2010 23:22, Albert Aribaud wrote: older ld emitted all ELF relocations in input sections named .rel.dyn, whereas newer ld uses names of the form .rel*. The linker script only collected .rel.dyn input sections. Rewrite to collect

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-09 Thread Albert ARIBAUD
Le 09/11/2010 19:47, Albert ARIBAUD a écrit : I think this V4 of my patchset could be now committed to u-boot-arm/master, and if possible even on u-boot-arm for the december release of u-boot, as it is a bugfix. ... even on u-boot, not u-boot-arm, of course. Amicalement, -- Albert.

Re: [U-Boot] nand spl build with wrong CONFIG_SYS_TEXT_BASE

2010-11-09 Thread Wolfgang Denk
Dear Scott Wood, In message 20101109122218.13f16...@udp111988uds.am.freescale.net you wrote: This is no problem as long as these variables doen't change their values on the fly. It's not on the fly, it's just that each image needs its own autoconf.mk. For me it is on the fly when

Re: [U-Boot] nand spl build with wrong CONFIG_SYS_TEXT_BASE

2010-11-09 Thread Scott Wood
On Tue, 9 Nov 2010 19:59:47 +0100 Wolfgang Denk w...@denx.de wrote: Dear Scott Wood, In message 20101109122218.13f16...@udp111988uds.am.freescale.net you wrote: This is no problem as long as these variables doen't change their values on the fly. It's not on the fly, it's just

Re: [U-Boot] [RFC] arm926ejs: fix jump to RAM nand_boot

2010-11-09 Thread Scott Wood
On Tue, 2 Nov 2010 10:17:23 +0100 Stefan Roese s...@denx.de wrote: Hi Heiko, On Tuesday 02 November 2010 09:55:46 Heiko Schocher wrote: - preloader copies first page of nand (nand_spl code) to 0xbb00 (some cpu internal mem) and jumps to this address - nand_spl does

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-09 Thread Andreas Bießmann
Dear Daniel Hobi, Am 09.11.2010 um 19:24 schrieb Daniel Hobi: And on many ARM platforms (including Kirkwood), the timer implementation is still accessing BSS variables before relocation. Is someone working on this? Candidates are: $ git grep static ulong timestamp

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-09 Thread Andreas Bießmann
Dear Albert Ardibaud, Am 09.11.2010 um 19:47 schrieb Albert ARIBAUD: Thank you. This patch is required to get Kirkwood-based boards working again when using the CodeSourcery 2009q3 toolchain. (can't find the 2010q3 Lite toolchain on CodeSourcery's site, latest is 2010q1 apparently... Can

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-09 Thread Albert ARIBAUD
Le 09/11/2010 20:27, Andreas Bießmann a écrit : BTW couldn't the stuff in #ifdef CONFIG_AT91FAMILY in global_data generalized in some way to be used for all arm boards? Don't know for others, but regarding the timer, there is no need for timestamp before relocation, so it needs not join gd

Re: [U-Boot] NAND on eLBC

2010-11-09 Thread Scott Wood
On Fri, 5 Nov 2010 10:21:53 +0100 Andre Schwarz andre.schw...@matrix-vision.de wrote: Scott, looks like I made a mistake on current MPC8377 NAND wiring. Nand-Flash write enable is wired to LFWE1 ... I connected each WE to its corresponding CS, i.e. : - CS0/WE0 - Nor Flash - CS1/WE1 -

Re: [U-Boot] nand spl build with wrong CONFIG_SYS_TEXT_BASE

2010-11-09 Thread Wolfgang Denk
Dear Scott Wood, In message 20101109130427.5e918...@udp111988uds.am.freescale.net you wrote: For me it is on the fly when running a single make all sees these values changing. In any case, the right answer is a separate autoconf.mk per image. NAK. If different images need to be linked

Re: [U-Boot] nand spl build with wrong CONFIG_SYS_TEXT_BASE

2010-11-09 Thread Scott Wood
On Tue, 9 Nov 2010 21:13:00 +0100 Wolfgang Denk w...@denx.de wrote: Dear Scott Wood, In message 20101109130427.5e918...@udp111988uds.am.freescale.net you wrote: For me it is on the fly when running a single make all sees these values changing. In any case, the right answer is a

Re: [U-Boot] nand spl build with wrong CONFIG_SYS_TEXT_BASE

2010-11-09 Thread Wolfgang Denk
Dear Scott Wood, In message 20101109143747.685f9...@udp111988uds.am.freescale.net you wrote: I don't see how #ifdef NAND_SPL #define CONFIG_SYS_TEXT_BASE xxx #else #define CONFIG_SYS_TEXT_BASE yyy #endif is more of a maintenance problem than #define CONFIG_SYS_TEXT_BASE_SPL xxx

Re: [U-Boot] nand spl build with wrong CONFIG_SYS_TEXT_BASE

2010-11-09 Thread Haiying Wang
On Tue, 2010-11-09 at 14:11 -0700, Wolfgang Denk wrote: Dear Scott Wood, In message 20101109143747.685f9...@udp111988uds.am.freescale.net you wrote: I don't see how #ifdef NAND_SPL #define CONFIG_SYS_TEXT_BASE xxx #else #define CONFIG_SYS_TEXT_BASE yyy #endif is more of a

Re: [U-Boot] nand spl build with wrong CONFIG_SYS_TEXT_BASE

2010-11-09 Thread Wolfgang Denk
Dear Haiying Wang, In message 1289337782.2168.148.ca...@haiying-laptop you wrote: Assuming that your code really needs to know the start address of the image, it should probably do something like this instead: #ifdef NAND_SPL #define CONFIG_SYS_TEXT_BASE xxx #define

Re: [U-Boot] nand spl build with wrong CONFIG_SYS_TEXT_BASE

2010-11-09 Thread Scott Wood
On Tue, 9 Nov 2010 22:11:41 +0100 Wolfgang Denk w...@denx.de wrote: #ifdef NAND_SPL #define CONFIG_SYS_TEXT_BASE_SPL xxx #define CONFIG_SYS_MONITOR_BASE CONFIG_SYS_TEXT_BASE_SPL #else #define CONFIG_SYS_TEXT_BASE yyy #define CONFIG_SYS_MONITOR_BASE CONFIG_SYS_TEXT_BASE #endif followed

Re: [U-Boot] nand spl build with wrong CONFIG_SYS_TEXT_BASE

2010-11-09 Thread Wolfgang Denk
Dear Scott Wood, In message 20101109155225.183a3...@udp111988uds.am.freescale.net you wrote: What, semantically, is the difference between the CONFIG_SYS_TEXT_BASE and CONFIG_SYS_MONITOR_BASE? Just that one is used from linker Nothing. scripts/makefiles and the other from C/assembly code

Re: [U-Boot] [PATCH] env_mmc: fix compile warning

2010-11-09 Thread Mike Frysinger
On Tuesday, November 09, 2010 10:18:42 Lei Wen wrote: Hi Mike, please dont top post I am sorry for that... Here is the warning: env_mmc.c: In function 'saveenv': env_mmc.c:109: warning: implicit declaration of function 'hexport' please resend your patch with an updated changelog -mike

Re: [U-Boot] RFC: Aligning arch initialisation sequences

2010-11-09 Thread Mike Frysinger
On Sunday, November 07, 2010 05:06:26 Graeme Russ wrote: Should all architectures strive to look as much like one another as possible? Should we accept that maybe this particular issue be thrown in the too hard basket? imo, we should strive to have these things in one common place and not just

Re: [U-Boot] [PATCH] env_mmc: fix compile warning

2010-11-09 Thread Lei Wen
Hi Mike, Please review this refined patch. From a7c694f84bec6b0db26ece4c77d06585be5a6755 Mon Sep 17 00:00:00 2001 From: Lei Wen lei...@marvell.com Date: Wed, 10 Nov 2010 07:44:03 +0800 Subject: [PATCH] env_mmc: fix compile warning hexport would complain implicit declaration, if we don't add the

Re: [U-Boot] [PATCH] env_mmc: fix compile warning

2010-11-09 Thread Mike Frysinger
On Tuesday, November 09, 2010 18:39:23 Lei Wen wrote: Hi Mike, Please review this refined patch. thanks, this looks much better not that ive ever used env_mmc, so i cant comment on correctness of it -mike signature.asc Description: This is a digitally signed message part.

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-09 Thread Eric Cooper
I have been periodically rebasing my patches for the Seagate DockStar on master. But ever since the elf_reloc changes were merged, I have been unable to get a working build. First I saw symptoms similar to what Alexander Holler reported (failing during NAND initialization due to incorrect BSS

Re: [U-Boot] Commit ecee9... (Program net device MAC addresses after initializing) breakage

2010-11-09 Thread Mike Frysinger
On Saturday, November 06, 2010 18:13:08 Graeme Russ wrote: I finally got a few spare hours to do some U-Boot hacking and to my dismay found the build for my board (eNET) had broken. It builds OK, but crashed during Ethernet initialisation. I tried winding back to a last known good commit

Re: [U-Boot] [PATCH] Switch from library archives to partial linking

2010-11-09 Thread Mike Frysinger
On Sunday, November 07, 2010 10:11:14 Sebastien Carlier wrote: On 11/07/2010 03:16 PM, Peter Tyser wrote: I'm guessing lots of boards will have this same issue. I imagine its due to include/config_cmd_defaults.h, so maybe if you fix the issue in that one place all the compile issues will

Re: [U-Boot] [PATCH] Switch from archive libraries to partial linking

2010-11-09 Thread Mike Frysinger
On Monday, November 08, 2010 17:04:32 Sebastien Carlier wrote: This commit changes all Makefiles to use partial linking (ld -r) instead of creating library archives, which forces all symbols to participate in linking, allowing non-weak symbols to override weak symbols as intended. This

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-09 Thread Albert ARIBAUD
Le 10/11/2010 00:43, Eric Cooper a écrit : I have been periodically rebasing my patches for the Seagate DockStar on master. But ever since the elf_reloc changes were merged, I have been unable to get a working build. First I saw symptoms similar to what Alexander Holler reported (failing