June 2010 - U-Boot

Re: [U-Boot] U-boot for PPC405GP based custom Board
by Wolfgang Denk 28 Jun '10

28 Jun '10

Dear ravi.rao(a)rflelect.com, please post U-Boot related questions always on the U-Boot mailing list. In message <OF223063CB.29974FC8-ON8525774F.00694481-8525774F.00699FE3(a)slpower.com> you wrote: > > Thanks for the help. I deleted the flash.c and enabled the standard CFI > and it seems to have fixed my problem. As per your suggestion I downloaded > the latest current version of U-boot ( > u-boot-2010.06-rc3) and Iam getting the following error during > compilation. > > /opt/timesys/toolchains/ppc4xx-linux/bin/ppc4xx-linux-gcc -E -g -Os > -mrelocatable -fPIC -meabi -D__KERNEL__ -DTEXT_BASE=0xFFFC0000 > -I/mnt/homesA/ravi/g8k_26kernel/u-boot-2010.06-rc3/include -fno-builtin > -ffreestanding -nostdinc -isystem > /opt/timesys/toolchains/lib/gcc/powerpc-timesys-linux-gnu/4.2.1/include > -pipe -DCONFIG_PPC -D__powerpc__ -DCONFIG_4xx -ffixed-r2 -mstring > -msoft-float -Wa,-m405 -mcpu=405 -include > /mnt/homesA/ravi/g8k_26kernel/u-boot-2010.06-rc3/include/u-boot/u-boot.lds.h > -ansi -D__ASSEMBLY__ -P - > </mnt/homesA/ravi/g8k_26kernel/u-boot-2010.06-rc3/arch/powerpc/cpu/ppc4xx/u-boot.lds > >u-boot.lds > UNDEF_SYM=`/opt/timesys/toolchains/ppc4xx-linux/bin/ppc4xx-linux-objdump > -x board/amcc/taihu/libtaihu.a lib/libgeneric.a lib/lzma/liblzma.a You should not use the Taihu board consifuration, but instead create your own. ... > /opt/timesys/toolchains/bin/powerpc-timesys-linux-gnu-ld:/mnt/homesA/ravi/g8k_26kernel/u-boot-2010.06-rc3/arch/powerpc/cpu/ppc4xx/u-boot.lds:23: > ignoring invalid character `#' in expression > /opt/timesys/toolchains/bin/powerpc-timesys-linux-gnu-ld:/mnt/homesA/ravi/g8k_26kernel/u-boot-2010.06-rc3/arch/powerpc/cpu/ppc4xx/u-boot.lds:23: > syntax error > Can you please tell me what I am missing ?? This should not happen. Either your tool chain is broken, or there was some problem with your download. Is there any chance that a Windows system was involved, so you have corrupted line endings (\r\n instead of plain \n) ? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd(a)denx.de Humans do claim a great deal for that particular emotion (love). -- Spock, "The Lights of Zetar", stardate 5725.6

2 2

Re: [U-Boot] [PATCH V2] Allow PHY addresses on kirkwood egiga to be non continuous.
by Tor Krill 28 Jun '10

28 Jun '10

On Thu, 2010-06-24 at 14:26 +0200, Prafulla Wadaskar wrote: > > > -----Original Message----- > > From: Tor Krill [mailto:tor@excito.com] > > Sent: Thursday, June 24, 2010 5:05 PM > > To: Prafulla Wadaskar > > Cc: u-boot(a)lists.denx.de > > Subject: RE: [PATCH V2] Allow PHY addresses on kirkwood egiga > > to be non continuous. > > > > > > Hi Prafulla, > > > > On Thu, 2010-06-24 at 13:22 +0200, Prafulla Wadaskar wrote: > > > > > > > -----Original Message----- > > > > From: Tor Krill [mailto:tor@excito.com] > > > > Sent: Thursday, June 24, 2010 3:02 PM > > > > To: u-boot(a)lists.denx.de > > > > Cc: Prafulla Wadaskar; Tor Krill > > > > Subject: [PATCH V2] Allow PHY addresses on kirkwood egiga to > > > > be non continuous. > > > > > > > > Changing its configuration from PHY_BASE_ADR single value to use > > > > CONFIG_PHY_BASE_ADDRS as an array with adresses. > > > > > > > > Updated in version 2: > > > > > > > > Merged board updates to single patch. > > > > > > These two lines should go as changelog for v2 below (after ---) > > > > Sorry, there seems to be no end on my mistakes :( > > > > > > > > > > Signed-off-by: Tor Krill <tor(a)excito.com> > > > > --- > > > i.e. here.. > > > > > > > board/keymile/km_arm/km_arm.c | 3 ++- > > > > drivers/net/kirkwood_egiga.c | 3 ++- > > > > drivers/net/kirkwood_egiga.h | 8 ++++---- > > > > include/configs/guruplug.h | 2 +- > > > > include/configs/km_arm.h | 2 +- > > > > include/configs/openrd_base.h | 2 +- > > > > include/configs/rd6281a.h | 2 +- > > > > include/configs/sheevaplug.h | 2 +- > > > > 8 files changed, 13 insertions(+), 11 deletions(-) > > > > > > Hi Tom > > > I understood your problem. > > > But all of the Kirkwood boards so far have similar > > implementation which is standard practice. > > > How does your board design differed in this case? > > > > > > Instead of fixing it here for all other boards, > > > can't you simply align PHY address on your board? > > > I think this will be much easier. > > > > We use both ethernet interfaces by default in our design, with the phy > > addresses 0x08 and 0x18, and unfortunately we have already started the > > manufacturing of these units which makes it expensive and hard to > > reverse this design decision :( > > Do you think this will be corrected in next h/w version? > If yes, then you can keep this patch private to yourself. We will unfortunately not change this, what we know of now. If it had only been moving some resistors mounted or not it could perhaps have been fixable before mass production. But as it is now this would involve us changing the the design as well. > > > > But if we turn this around if the hardware allows this then > > why not let > > this be configurable in u-boot as well? Do this have any technical > > implications on the other boards? > > Overall > > 1. This patch increases u-boot binary size by 20 bytes. (may be > critical issue on low footprint boards like mv88f6281gtw_ge). Ok, i only get this to ~4 bytes but thats a valid point. > 2. For the boards using single port will also need to configure PHY address > for other port, which is meaningless, unnecessary and confusing. Sure you had to put something there to be on the safe side. But is that really that confusing, the SOC really have two ports, them being used or not? This is the exact same mechanism that is used for CONFIG_KIRKWOOD_EGIGA_PORTS which is implemented as an array telling the driver if it should initialize the port or not. > Technically no implications, but we need to keep code simple, small and standard. And again, i think the flexibility is worth the "small" prize. I know that fx the TK71 devkit have non consecutive PHY addresses (Even though it not being present in mainline u-boot) If you are able to set the addresses "freely", why limit this in software? The alternative, as i see it, would be to have defines for each port and then ifdef each init. Would that be an alternative? Regards, /Tor

3 2

[U-Boot] [PATCH] EHCI: zero out QH transfer overlay in ehci_submit_async()
by Sergei Shtylyov 28 Jun '10

28 Jun '10

ehci_submit_async() doesn't really zero out the QH transfer overlay (as the EHCI specification suggests) which leads to the controller seeing the 'token' field as the previous call has left it, i.e.: - if a timeout occured on the previous call (Active bit left as 1), controller incorrectly tries to complete a previous transaction on a newly programmed endpoint; - if a halt occured on the previous call (Halted bit set to 1), controller just ignores the newly programmed TD(s) and the function then keeps returning error ad infinitum. This turned out to be caused by the wrong orger of the arguments to the memset() call in ehci_alloc(), so the allocated TDs weren't cleared either. While at it, stop needlessly initializing the "next" pointers in the QH transfer overlay... Signed-off-by: Sergei Shtylyov <sshtylyov(a)mvista.com> --- This is quite serious error, so would be good to have the patch in v2010.06... drivers/usb/host/ehci-hcd.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) Index: u-boot/drivers/usb/host/ehci-hcd.c =================================================================== --- u-boot.orig/drivers/usb/host/ehci-hcd.c +++ u-boot/drivers/usb/host/ehci-hcd.c @@ -275,7 +275,7 @@ static void *ehci_alloc(size_t sz, size_ return NULL; } - memset(p, sz, 0); + memset(p, 0, sz); return p; } @@ -349,8 +349,6 @@ ehci_submit_async(struct usb_device *dev (dev->portnr << 23) | (dev->parent->devnum << 16) | (0 << 8) | (0 << 0); qh->qh_endpt2 = cpu_to_hc32(endpt); - qh->qh_overlay.qt_next = cpu_to_hc32(QT_NEXT_TERMINATE); - qh->qh_overlay.qt_altnext = cpu_to_hc32(QT_NEXT_TERMINATE); td = NULL; tdp = &qh->qh_overlay.qt_next;

3 2

Re: [U-Boot] U-boot for PPC405GP based custom Board
by ravi.rao＠rflelect.com 28 Jun '10

28 Jun '10

Sending the same message without confidentiality crap... ravi rao/RFL/SLI 06/28/2010 09:48 AM To Wolfgang Denk <wd(a)denx.de> cc u-boot(a)lists.denx.de Subject Re: [U-Boot] U-boot for PPC405GP based custom Board Hi, Looks like the syntax for .lds file has changed. It does not like any defines. I was trying to get the new syntax in .lds files to include file or define a constan but could not get a definitive answere. Any help is appreciated. By the way I have my own file for the custom board. Since the standard one also exhibited the problem I choose to send the error from taihu board. Thanks in advance, Ravishankar Govindarao RFL Electronics Inc. E-mail : Ravi.Rao(a)rflelect.com Voice: 973.334.3100 Ext. 233 Fax: 973.334.3863 Wolfgang Denk <wd(a)denx.de> 06/27/2010 05:18 PM To ravi.rao(a)rflelect.com cc u-boot(a)lists.denx.de Subject Re: [U-Boot] U-boot for PPC405GP based custom Board Dear ravi.rao(a)rflelect.com, please post U-Boot related questions always on the U-Boot mailing list. In message <OF223063CB.29974FC8-ON8525774F.00694481-8525774F.00699FE3(a)slpower.com> you wrote: > > Thanks for the help. I deleted the flash.c and enabled the standard CFI > and it seems to have fixed my problem. As per your suggestion I downloaded > the latest current version of U-boot ( > u-boot-2010.06-rc3) and Iam getting the following error during > compilation. > > /opt/timesys/toolchains/ppc4xx-linux/bin/ppc4xx-linux-gcc -E -g -Os > -mrelocatable -fPIC -meabi -D__KERNEL__ -DTEXT_BASE=0xFFFC0000 > -I/mnt/homesA/ravi/g8k_26kernel/u-boot-2010.06-rc3/include -fno-builtin > -ffreestanding -nostdinc -isystem > /opt/timesys/toolchains/lib/gcc/powerpc-timesys-linux-gnu/4.2.1/include > -pipe -DCONFIG_PPC -D__powerpc__ -DCONFIG_4xx -ffixed-r2 -mstring > -msoft-float -Wa,-m405 -mcpu=405 -include > /mnt/homesA/ravi/g8k_26kernel/u-boot-2010.06-rc3/include/u-boot/u-boot.lds.h > -ansi -D__ASSEMBLY__ -P - > </mnt/homesA/ravi/g8k_26kernel/u-boot-2010.06-rc3/arch/powerpc/cpu/ppc4xx/u-boot.lds > >u-boot.lds > UNDEF_SYM=`/opt/timesys/toolchains/ppc4xx-linux/bin/ppc4xx-linux-objdump > -x board/amcc/taihu/libtaihu.a lib/libgeneric.a lib/lzma/liblzma.a You should not use the Taihu board consifuration, but instead create your own. ... > /opt/timesys/toolchains/bin/powerpc-timesys-linux-gnu-ld:/mnt/homesA/ravi/g8k_26kernel/u-boot-2010.06-rc3/arch/powerpc/cpu/ppc4xx/u-boot.lds:23: > ignoring invalid character `#' in expression > /opt/timesys/toolchains/bin/powerpc-timesys-linux-gnu-ld:/mnt/homesA/ravi/g8k_26kernel/u-boot-2010.06-rc3/arch/powerpc/cpu/ppc4xx/u-boot.lds:23: > syntax error > Can you please tell me what I am missing ?? This should not happen. Either your tool chain is broken, or there was some problem with your download. Is there any chance that a Windows system was involved, so you have corrupted line endings (\r\n instead of plain \n) ? Best regards, Wolfgang Denk -- Humans do claim a great deal for that particular emotion (love). -- Spock, "The Lights of Zetar", stardate 5725.6

1 0

[U-Boot] [PATCH] net: Add option to disable fibre on M88E1111 PHY for PPC4xx
by Stefan Roese 28 Jun '10

28 Jun '10

By defining CONFIG_M88E1111_DISABLE_FIBRE boards can configure the M88E1111 PYH to disable fibre. This is needed for an upcoming PPC460GT based board, which has fibre/copper auto-selection enabled by default. This doesn't seem to work. So we disable fibre in the PHY register. Signed-off-by: Stefan Roese <sr(a)denx.de> Cc: Ben Warren <biggerbadderben(a)gmail.com> --- drivers/net/4xx_enet.c | 5 +++++ 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/drivers/net/4xx_enet.c b/drivers/net/4xx_enet.c index 2fac641..2958903 100644 --- a/drivers/net/4xx_enet.c +++ b/drivers/net/4xx_enet.c @@ -1095,6 +1095,11 @@ static int ppc_4xx_eth_init (struct eth_device *dev, bd_t * bis) miiphy_write (dev->name, reg, 0x18, 0x4101); miiphy_write (dev->name, reg, 0x09, 0x0e00); miiphy_write (dev->name, reg, 0x04, 0x01e1); +#if defined(CONFIG_M88E1111_DISABLE_FIBRE) + miiphy_read(dev->name, reg, 0x1b, &reg_short); + reg_short |= 0x8000; + miiphy_write(dev->name, reg, 0x1b, reg_short); +#endif #endif #if defined(CONFIG_M88E1112_PHY) if (bis->bi_phymode[devnum] == BI_PHYMODE_SGMII) { -- 1.7.1

2 2

[U-Boot] Updating YAFFS2 RootFS
by David Andrey 28 Jun '10

28 Jun '10

Hi, I'm looking to update the linux yaffs2 rootfs from U-Boot (2009-06). The whole update will be: * Take the rootfs.tgz from the yaffs2 "image" partition (ok, with yrdm / ywrm) for update or restore purposes. * Unpack it (maybe gunzip) * Update the Linux YAFFS2 RootFS (how ?) After a quick search (mail list, code) seems that: * No native support for this operation in u-boot. * Some "user" patch files exists (out of date ?) So, my questions: * Is the situation really like that ? * Which is the actual patch file ? * What about the future for YAFFS2 in U-Boot ? Is there any update planned ? Regards David

2 5

[U-Boot] Thanks
by Rajeesh.R 28 Jun '10

28 Jun '10

Thanks... With Thanks Rajeesh.R

1 0

[U-Boot] u-boot-2010.06-rc3 to AT91rm9200dk
by Rajeesh.R 28 Jun '10

28 Jun '10

Hi, I have compiled Uboot version u-boot-2010.06-rc3 and I got bin file . I tried it to download bin file to at91rm9200dk board it stops after some time witth error . I think this is due to insufficient of SRAM capability ie 16KB SRAM uboot size is 107KB How to download this uboot directly to flash ...location any other configurtion in uboot for this . . Wherther Romboot is required ..? With Thanks Rajeesh.R |

3 2

[U-Boot] [PATCH V2] Allow PHY addresses on kirkwood egiga to be non continuous.
by Tor Krill 28 Jun '10

28 Jun '10

Changing its configuration from PHY_BASE_ADR single value to use CONFIG_PHY_BASE_ADDRS as an array with adresses. Updated in version 2: Merged board updates to single patch. Signed-off-by: Tor Krill <tor(a)excito.com> --- board/keymile/km_arm/km_arm.c | 3 ++- drivers/net/kirkwood_egiga.c | 3 ++- drivers/net/kirkwood_egiga.h | 8 ++++---- include/configs/guruplug.h | 2 +- include/configs/km_arm.h | 2 +- include/configs/openrd_base.h | 2 +- include/configs/rd6281a.h | 2 +- include/configs/sheevaplug.h | 2 +- 8 files changed, 13 insertions(+), 11 deletions(-) diff --git a/board/keymile/km_arm/km_arm.c b/board/keymile/km_arm/km_arm.c index 53cf474..16da353 100644 --- a/board/keymile/km_arm/km_arm.c +++ b/board/keymile/km_arm/km_arm.c @@ -244,12 +244,13 @@ int dram_init(void) void reset_phy(void) { char *name = "egiga0"; + u8 port_addr[] = CONFIG_PHY_ADDRS; if (miiphy_set_current_dev(name)) return; /* reset the phy */ - miiphy_reset(name, CONFIG_PHY_BASE_ADR); + miiphy_reset(name, port_addr[0]); } #if defined(CONFIG_HUSH_INIT_VAR) diff --git a/drivers/net/kirkwood_egiga.c b/drivers/net/kirkwood_egiga.c index 932792e..7993e51 100644 --- a/drivers/net/kirkwood_egiga.c +++ b/drivers/net/kirkwood_egiga.c @@ -629,6 +629,7 @@ int kirkwood_egiga_initialize(bd_t * bis) int devnum; char *s; u8 used_ports[MAX_KWGBE_DEVS] = CONFIG_KIRKWOOD_EGIGA_PORTS; + u8 port_addr[MAX_KWGBE_DEVS] = PHY_ADDRS; for (devnum = 0; devnum < MAX_KWGBE_DEVS; devnum++) { /*skip if port is configured not to use */ @@ -712,7 +713,7 @@ int kirkwood_egiga_initialize(bd_t * bis) miiphy_register(dev->name, smi_reg_read, smi_reg_write); /* Set phy address of the port */ miiphy_write(dev->name, KIRKWOOD_PHY_ADR_REQUEST, - KIRKWOOD_PHY_ADR_REQUEST, PHY_BASE_ADR + devnum); + KIRKWOOD_PHY_ADR_REQUEST, port_addr[devnum]); #endif } return 0; diff --git a/drivers/net/kirkwood_egiga.h b/drivers/net/kirkwood_egiga.h index 30c773c..6ff3bb7 100644 --- a/drivers/net/kirkwood_egiga.h +++ b/drivers/net/kirkwood_egiga.h @@ -30,11 +30,11 @@ #define MAX_KWGBE_DEVS 2 /*controller has two ports */ -/* PHY_BASE_ADR is board specific and can be configured */ -#if defined (CONFIG_PHY_BASE_ADR) -#define PHY_BASE_ADR CONFIG_PHY_BASE_ADR +/* PHY addresses is board specific and can be configured */ +#if defined (CONFIG_PHY_ADDRS) +#define PHY_ADDRS CONFIG_PHY_ADDRS #else -#define PHY_BASE_ADR 0x08 /* default phy base addr */ +#define PHY_ADDRS {0x08,0x09} /* default phy base addr */ #endif /* Constants */ diff --git a/include/configs/guruplug.h b/include/configs/guruplug.h index 2fbc6ad..028eee7 100644 --- a/include/configs/guruplug.h +++ b/include/configs/guruplug.h @@ -175,7 +175,7 @@ #define CONFIG_KIRKWOOD_EGIGA /* Enable kirkwood Gbe Controller Driver */ #define CONFIG_SYS_FAULT_ECHO_LINK_DOWN /* detect link using phy */ #define CONFIG_KIRKWOOD_EGIGA_PORTS {1,1} /* enable both ports */ -#define CONFIG_PHY_BASE_ADR 0 +#define CONFIG_PHY_ADDRS {0,1} #define CONFIG_ENV_OVERWRITE /* ethaddr can be reprogrammed */ #define CONFIG_RESET_PHY_R /* use reset_phy() to init mv88e1121 PHY */ #endif /* CONFIG_CMD_NET */ diff --git a/include/configs/km_arm.h b/include/configs/km_arm.h index a928c2c..1c3fa0d 100644 --- a/include/configs/km_arm.h +++ b/include/configs/km_arm.h @@ -130,7 +130,7 @@ #define CONFIG_KIRKWOOD_EGIGA /* Enable kirkwood Gbe Controller Driver */ #define CONFIG_SYS_FAULT_ECHO_LINK_DOWN /* detect link using phy */ #define CONFIG_KIRKWOOD_EGIGA_PORTS {1,0} /* enable port 0 only */ -#define CONFIG_PHY_BASE_ADR 0 +#define CONFIG_PHY_ADDRS {0,1} #define CONFIG_ENV_OVERWRITE /* ethaddr can be reprogrammed */ #define CONFIG_RESET_PHY_R /* use reset_phy() to init 88E1118 PHY */ diff --git a/include/configs/openrd_base.h b/include/configs/openrd_base.h index d2f4502..37574ac 100644 --- a/include/configs/openrd_base.h +++ b/include/configs/openrd_base.h @@ -186,7 +186,7 @@ #define CONFIG_KIRKWOOD_EGIGA /* Enable kirkwood Gbe Controller Driver */ #define CONFIG_SYS_FAULT_ECHO_LINK_DOWN /* detect link using phy */ #define CONFIG_KIRKWOOD_EGIGA_PORTS {1,0} /* enable port 0 only */ -#define CONFIG_PHY_BASE_ADR 0x8 +#define CONFIG_PHY_ADDRS {0x8,0x9} #define CONFIG_ENV_OVERWRITE /* ethaddr can be reprogrammed */ #define CONFIG_RESET_PHY_R /* use reset_phy() to init mv8831116 PHY */ #endif /* CONFIG_CMD_NET */ diff --git a/include/configs/rd6281a.h b/include/configs/rd6281a.h index 3d8e25c..01550d1 100644 --- a/include/configs/rd6281a.h +++ b/include/configs/rd6281a.h @@ -176,7 +176,7 @@ #define CONFIG_MV88E61XX_MULTICHIP_ADRMODE #define CONFIG_DIS_AUTO_NEG_SPEED_GMII /*Disable Auto speed negociation */ #define CONFIG_PHY_SPEED _1000BASET /*Force PHYspeed to 1GBPs */ -#define CONFIG_PHY_BASE_ADR 0x0A +#define CONFIG_PHY_ADDRS {0x0A,0x0B} #define CONFIG_ENV_OVERWRITE /* ethaddr can be reprogrammed */ #define CONFIG_RESET_PHY_R /* use reset_phy() to init switch and PHY */ #define CONFIG_MV88E61XX_SWITCH /* Enable MV88E61XX switch driver */ diff --git a/include/configs/sheevaplug.h b/include/configs/sheevaplug.h index e9edc44..9fdfd7e 100644 --- a/include/configs/sheevaplug.h +++ b/include/configs/sheevaplug.h @@ -177,7 +177,7 @@ #define CONFIG_KIRKWOOD_EGIGA /* Enable kirkwood Gbe Controller Driver */ #define CONFIG_SYS_FAULT_ECHO_LINK_DOWN /* detect link using phy */ #define CONFIG_KIRKWOOD_EGIGA_PORTS {1,0} /* enable port 0 only */ -#define CONFIG_PHY_BASE_ADR 0 +#define CONFIG_PHY_ADDRS {0,1} #define CONFIG_ENV_OVERWRITE /* ethaddr can be reprogrammed */ #define CONFIG_RESET_PHY_R /* use reset_phy() to init mv8831116 PHY */ #endif /* CONFIG_CMD_NET */ -- 1.7.0.4

4 6

[U-Boot] At91rm9200dk ...Uboot ...How to download this uboot directly to flash
by Rajeesh.R 28 Jun '10

28 Jun '10

Hi, I have tried to down load uboot bin via xmodem I am using serial flash AT45DB642D . My aim is to boot from this flash . regards Rajeesh Dear Rajeesh.R, Am 28.06.2010 06:51, schrieb Rajeesh.R: > Hi, > > I have compiled Uboot version u-boot-2010.06-rc3 and I got bin file . > > I tried it to download bin file to at91rm9200dk board it stops after > some time witth error . How did you try to load the file to your device? > I think this is due to insufficient of SRAM capability ie 16KB SRAM > uboot size is 107KB I think you have chosen the BOOTROM based upload via xmodem cause you talk about SRAM size which does not matter if you boot from NOR flash on CS0.

2 1