March 2008 - U-Boot

[U-Boot-Users] Author: Yuri Tikhonov <yur@emcraft.com>
by Wolfgang Denk 22 Mar '08

22 Mar '08

Add test for DPIC_SYS_ERROR_REG to be zero in the LWMON5 dsPIC POST. Signed-off-by: Yuri Tikhonov <yur(a)emcraft.com> --- --- diff --git a/post/board/lwmon5/dspic.c b/post/board/lwmon5/dspic.c index 65b96bc..dbaa074 100644 --- a/post/board/lwmon5/dspic.c +++ b/post/board/lwmon5/dspic.c @@ -97,8 +97,9 @@ int dspic_post_test(int flags) if (data == -1) { post_log("dsPIC : failed read system error\n"); ret = 1; - } else { + } else if (data != 0) { post_log("dsPIC SYS-ERROR code: 0x%04X\n", data); + ret = 1; } return ret;

1 0

[U-Boot-Users] Using u-boot to launch vxworks
by Zvika Vered 22 Mar '08

22 Mar '08

Hello, I ran u-boot on Xilinx ML405 board. It runs from flash and relocates to RAM. I want to use u-boot to launch a vxworks ELF image. The vxworks image is Ok. If I download it using JTAG and run it vxworks runs. But when I try to run it using U-boot I have problems. I tried the following sequnece: 1. tftpboot - download the ELF image via TFTP. The image is downloaded OK 2. elfboot - I tried also using vxboot. I get a u-boot crash. Vxworks is not loaded. What am I doing wrong ? Thanks, Zvika Vered IAI \ ELTA

1 0

[U-Boot-Users] Your new life is a click away
by Helena Craig 22 Mar '08

22 Mar '08

The world�s leading Mens Enlargement Supplement available at massive discounts. http://www.Filadelphio.com/ Go where no man has ventured

1 0

[U-Boot-Users] RFC: Remove CONFIG_OF_HAS_UBOOT_ENV and CONFIG_OF_HAS_BD_T
by Jerry Van Baren 22 Mar '08

22 Mar '08

Hi Dan, There are two configuration switches which are used only by the stxxtc board, of which you are listed as the maintainer. CONFIG_OF_HAS_UBOOT_ENV CONFIG_OF_HAS_BD_T These add the u-boot env variables and the bd_t struct into the fdt blob before booting linux. The value of doing this was debated a while back and the consensus was that it is Not A Good Thing[tm]. <http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/33643/focus=33653> I don't believe either is actually used in linux. I didn't find "u-boot-env" at all and the "bd_t" matches did not look like FDT stuff (struct bd_t stuff). Per the discussion on the u-boot list, these config switches were declared deprecated. I propose to remove them at this time. Best regards, gvb

1 0

[U-Boot-Users] [PATCH] ppc: Add CONFIG_TOP_HIDE option to hide memory area that doesn't get "touched"
by Stefan Roese 22 Mar '08

22 Mar '08

If CONFIG_TOP_HIDE is defined in the board config header, this specified memory area will get subtracted from the top (end) of ram and won't get "touched" at all by U-Boot. By fixing up gd->ram_size the Linux kernel should gets passed the now "corrected" memory size and won't touch it either. This should work for arch/ppc and arch/powerpc. Only Linux board ports in arch/powerpc with bootwrapper support, that recalculate the memory size from the SDRAM controller setup will have to get fixed. This patch enables this config option on some PPC440EPx boards as a workaround for the CHIP 11 errata. Here the description from the AMCC documentation: CHIP_11: End of memory range area restricted access. Category: 3 Overview: The 440EPx DDR controller does not acknowledge any transaction which is determined to be crossing over the end-of-memory-range boundary, even if the starting address is within valid memory space. Any such transaction from any PLB4 master will result in a PLB time-out on PLB4 bus. Impact: In case of such misaligned bursts, PLB4 masters will not retrieve any data at all, just the available data up to the end of memory, especially the 440 CPU. For example, if a CPU instruction required an operand located in memory within the last 7 words of memory, the DCU master would burst read 8 words to update the data cache and cross over the end-of-memory-range boundary. Such a DCU read would not be answered by the DDR controller, resulting in a PLB4 time-out and ultimately in a Machine Check interrupt. The data would be inaccessible to the CPU. Workaround: Forbid any application to access the last 256 bytes of DDR memory. For example, make your operating system believe that the last 256 bytes of DDR memory are absent. AMCC has a patch that does this, available for Linux. This patch sets CONFIG_TOP_HIDE for the following 440EPx boards: lwmon5, korat, sequoia The other remaining 440EPx board were intentionally not included since it is not clear to me, if they use the end of ram for some other purpose. This is unclear, since these boards have CONFIG_PRAM defined and even comments like this: PMC440.h: /* esd expects pram at end of physical memory. * So no logbuffer at the moment. */ It is strongly recommendet to not use the last 256 bytes on those boards too. Patches from the board maintainers are welcome. Signed-off-by: Stefan Roese <sr(a)denx.de> --- include/configs/korat.h | 2 ++ include/configs/lwmon5.h | 2 ++ include/configs/sequoia.h | 4 +++- lib_ppc/board.c | 17 +++++++++++++++++ 4 files changed, 24 insertions(+), 1 deletions(-) diff --git a/include/configs/korat.h b/include/configs/korat.h index dcec9b0..c215c63 100644 --- a/include/configs/korat.h +++ b/include/configs/korat.h @@ -144,6 +144,8 @@ #define SPD_EEPROM_ADDRESS {0x50} #define CONFIG_PROG_SDRAM_TLB #define CFG_DRAM_TEST +#define CONFIG_MEM_TOP_HIDE 256 /* don't use last 256 bytes */ + /* 440EPx errata CHIP 11 */ /* * I2C diff --git a/include/configs/lwmon5.h b/include/configs/lwmon5.h index c3f10c7..f573730 100644 --- a/include/configs/lwmon5.h +++ b/include/configs/lwmon5.h @@ -85,6 +85,8 @@ #define CFG_INIT_SP_OFFSET CFG_GBL_DATA_OFFSET #define CFG_POST_ALT_WORD_ADDR (CFG_PERIPHERAL_BASE + GPT0_COMP6) /* unused GPT0 COMP reg */ +#define CONFIG_MEM_TOP_HIDE 256 /* don't use last 256 bytes */ + /* 440EPx errata CHIP 11 */ /*----------------------------------------------------------------------- * Serial Port diff --git a/include/configs/sequoia.h b/include/configs/sequoia.h index dfa8779..0188234 100644 --- a/include/configs/sequoia.h +++ b/include/configs/sequoia.h @@ -221,6 +221,8 @@ #if !defined(CONFIG_NAND_U_BOOT) && !defined(CONFIG_NAND_SPL) #define CONFIG_DDR_DATA_EYE /* use DDR2 optimization */ #endif +#define CONFIG_MEM_TOP_HIDE 256 /* don't use last 256 bytes */ + /* 440EPx errata CHIP 11 */ /* * I2C @@ -275,7 +277,7 @@ "ip=${ipaddr}:${serverip}:${gatewayip}:${netmask}" \ ":${hostname}:${netdev}:off panic=1\0" \ "addtty=setenv bootargs ${bootargs} console=ttyS0,${baudrate}\0"\ - "addmisc=setenv bootargs ${bootargs} mem=${mem}\0" \ + "addmisc=setenv bootargs ${bootargs}\0" \ "flash_nfs=run nfsargs addip addtty addmisc;" \ "bootm ${kernel_addr}\0" \ "flash_self=run ramargs addip addtty addmisc;" \ diff --git a/lib_ppc/board.c b/lib_ppc/board.c index fbf1c5d..13b6f54 100644 --- a/lib_ppc/board.c +++ b/lib_ppc/board.c @@ -117,6 +117,10 @@ DECLARE_GLOBAL_DATA_PTR; #define TOTAL_MALLOC_LEN CFG_MALLOC_LEN #endif +#if !defined(CONFIG_MEM_TOP_HIDE) +#define CONFIG_MEM_TOP_HIDE 0 +#endif + extern ulong __init_end; extern ulong _end; ulong monitor_flash_len; @@ -425,6 +429,7 @@ void board_init_f (ulong bootflag) * relocate the code and continue running from DRAM. * * Reserve memory at end of RAM for (top down in that order): + * - area that won't get touched by U-Boot and Linux (optional) * - kernel log buffer * - protected RAM * - LCD framebuffer @@ -433,6 +438,18 @@ void board_init_f (ulong bootflag) */ len = (ulong)&_end - CFG_MONITOR_BASE; + /* + * Subtract specified amount of memory to hide so that it won't + * get "touched" at all by U-Boot. By fixing up gd->ram_size + * the Linux kernel should now get passed the now "corrected" + * memory size and won't touch it either. This should work + * for arch/ppc and arch/powerpc. Only Linux board ports in + * arch/powerpc with bootwrapper support, that recalculate the + * memory size from the SDRAM controller setup will have to + * get fixed. + */ + gd->ram_size -= CONFIG_MEM_TOP_HIDE; + addr = CFG_SDRAM_BASE + get_effective_memsize(); #ifdef CONFIG_LOGBUFFER -- 1.5.4.4

4 5

Re: [U-Boot-Users] [RFC][PATCH 1/1] Add board_eth_init() function
by Ben Warren 22 Mar '08

22 Mar '08

Gaaah! Sorry, forgot to CC the list. On Sat, Mar 22, 2008 at 8:05 AM, Ben Warren <biggerbadderben(a)gmail.com> wrote: > > On Sat, Mar 22, 2008 at 1:03 AM, Shinya Kuribayashi > <skuribay(a)ruby.dti.ne.jp> wrote: > > Hi Ben, > > > > with a quick glance, find the comments below. > > > > > > > > Ben Warren wrote: > > > diff --git a/include/netdev.h b/include/netdev.h > > > new file mode 100644 > > > index 0000000..19195fa > > > --- /dev/null > > > +++ b/include/netdev.h > > > @@ -0,0 +1,35 @@ > > > +/* > > > + * (C) Copyright 2008 > > > + * Benjamin Warren, biggerbadderben(a)gmail.com > > > + * > > > + * See file CREDITS for list of people who contributed to this > > > + * project. > > > + * > > > + * This program is free software; you can redistribute it and/or > > > + * modify it under the terms of the GNU General Public License as > > > + * published by the Free Software Foundation; either version 2 of > > > + * the License, or (at your option) any later version. > > > + * > > > + * This program is distributed in the hope that it will be useful, > > > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > > > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > > > + * GNU General Public License for more details. > > > + * > > > + * You should have received a copy of the GNU General Public License > > > + * along with this program; if not, write to the Free Software > > > + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, > > > + * MA 02111-1307 USA > > > + */ > > > + > > > +/* > > > + * netdev.h - definitions an prototypes for network devices > > > + */ > > > + > > > +#ifndef _NETDEV_H_ > > > +#define _NETDEV_H_ > > > + > > > +int tsec_initialize(bd_t * bis, int index, char *devname); > > > + > > > +#endif /* _NETDEV_H_ */ > > > + > > > + > > > > Do we need such controller dependent declarations in netdev.h? > > FWIW board_eth_init() is reasonable for me. > > > > This was done to prevent the board code from complaining at compile > time. I have much bigger plans than initialize() prototypes for this > file, but this is all it needs for now. Another option I considered > here was to #include netdev from within common.h, but I've never been > a fan of nesting header files. > > > > And I know we need to go on a step-by-step basis, do you have any > > plan to have register/unregister interfaces? > > > > Ah, remove the last 2 lines :-) > > > > > > > diff --git a/net/eth.c b/net/eth.c > > > index 16a6dcb..3d02272 100644 > > > --- a/net/eth.c > > > +++ b/net/eth.c > > > @@ -28,6 +28,13 @@ > > > > > > #if defined(CONFIG_CMD_NET) && defined(CONFIG_NET_MULTI) > > > > > > +int __board_eth_init(bd_t *bis) > > > +{ > > > + return 0; > > > +} > > > + > > > +int board_eth_init(bd_t *bis) __attribute((weak, > > > alias("__board_eth_init"))); > > > + > > > #ifdef CFG_GT_6426x > > > extern int gt6426x_eth_initialize(bd_t *bis); > > > #endif > > > > [This comment is not for Ben, but for everyone.] > > > > Do we need such alias to empty function? Only __attribute((weak)) is > > enough for me. If we have some default behavior which isn't empty, I > > can see the weak & alias. Thought? > > > > I plead ignorance on this one. I've never used weak functions before, > and every example I found within U-boot, GCC documentation and Google > had the alias. I'll try to get rid of it. > > > Shinya > > > > > Thanks for the feedback! > > regards, > Ben >

1 0

[U-Boot-Users] [RFC][PATCH 0/1] New Ethernet initialization
by Ben Warren 22 Mar '08

22 Mar '08

Hello, By any measure, net/eth.c is messy and only gets worse as each new Ethernet controller gets added. This patch moves initialization to board code, providing the following benefits: 1. Board developers have more control over the ordering of Ethernet interfaces on a board, and can easily instantiate several different interfaces. This will make a PHY library easier to implement. 2. Ethernet controllers are decoupled from the net library. 3. /net/eth.c will become much smaller and easier to read. I've initially implemented only on TSEC, because I have a couple of different boards with this controller (MPC8349EMDS and MPC8555CDS) and am able to test it. Note that in this patch the TSEC driver itself isn't touched - only where it's initialized. If feedback is positive, I'll port all other in-tree controllers and boards and will put in a branch of the net tree. Hopefully volunteers will line up to sanity test. Please have a look and let me know your thoughts. regards, Ben

2 1

[U-Boot-Users] [MIPS] Cleanup CP0 Status initialization
by Shinya Kuribayashi 22 Mar '08

22 Mar '08

Add setup_c0_status from Linux. For the moment we disable interrupts, set CU0, mark the kernel mode, and clear ERL and EXL. This is good enough for reset-time configuration and will work well across most processors. Signed-off-by: Shinya Kuribayashi <skuribay(a)ruby.dti.ne.jp> --- cpu/mips/start.S | 34 +++++++++++++++++++++++++--------- 1 files changed, 25 insertions(+), 9 deletions(-) diff --git a/cpu/mips/start.S b/cpu/mips/start.S index 0ecdd83..baac2ce 100644 --- a/cpu/mips/start.S +++ b/cpu/mips/start.S @@ -27,6 +27,30 @@ #include <asm/regdef.h> #include <asm/mipsregs.h> + /* + * For the moment disable interrupts, mark the kernel mode and + * set ST0_KX so that the CPU does not spit fire when using + * 64-bit addresses. + */ + .macro setup_c0_status set clr + .set push + mfc0 t0, CP0_STATUS + or t0, ST0_CU0 | \set | 0x1f | \clr + xor t0, 0x1f | \clr + mtc0 t0, CP0_STATUS + .set noreorder + sll zero, 3 # ehb + .set pop + .endm + + .macro setup_c0_status_reset +#ifdef CONFIG_64BIT + setup_c0_status ST0_KX 0 +#else + setup_c0_status 0 0 +#endif + .endm + #define RVECENT(f,n) \ b f; nop #define XVECENT(f,bev) \ @@ -214,15 +238,7 @@ reset: /* WP(Watch Pending), SW0/1 should be cleared. */ mtc0 zero, CP0_CAUSE - /* STATUS register */ -#ifdef CONFIG_TB0229 - li k0, ST0_CU0 -#else - mfc0 k0, CP0_STATUS -#endif - li k1, ~ST0_IE - and k0, k1 - mtc0 k0, CP0_STATUS + setup_c0_status_reset /* Init Timer */ mtc0 zero, CP0_COUNT

1 0

[U-Boot-Users] [MIPS] Initialize CP0 Cause before setting up CP0 Status register
by Shinya Kuribayashi 22 Mar '08

22 Mar '08

Without this change, we'll be suffering from deffered WATCH exception once Status.EXL is cleared. Make sure Cause.WP is cleared. Signed-off-by: Shinya Kuribayashi <skuribay(a)ruby.dti.ne.jp> --- cpu/mips/start.S | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/cpu/mips/start.S b/cpu/mips/start.S index fde2944..0ecdd83 100644 --- a/cpu/mips/start.S +++ b/cpu/mips/start.S @@ -211,6 +211,9 @@ reset: mtc0 zero, CP0_WATCHLO mtc0 zero, CP0_WATCHHI + /* WP(Watch Pending), SW0/1 should be cleared. */ + mtc0 zero, CP0_CAUSE + /* STATUS register */ #ifdef CONFIG_TB0229 li k0, ST0_CU0 @@ -221,9 +224,6 @@ reset: and k0, k1 mtc0 k0, CP0_STATUS - /* CAUSE register */ - mtc0 zero, CP0_CAUSE - /* Init Timer */ mtc0 zero, CP0_COUNT mtc0 zero, CP0_COMPARE

1 0

[U-Boot-Users] She said I thrust harder than him
by otra ahrens 22 Mar '08

22 Mar '08

Take her in 100 different positions, everything is possible when you are 9 inches long. http://www.ellemonc.com/ Desperate for 9 inches in her

1 0