September 2008 - U-Boot

[U-Boot] Run hello_world.c example code at u-boot prompt using ARM Board
by naveen yadav 06 Sep '08

06 Sep '08

Hi All, I want to run hello_world.c example with printf enable . I load tftp hello_world.bin using at 0x200000 >tftp 0x20000 hello_world.bin >go 0x200000 After this System get freeze. I am very thankfull if any body have any Idea. I am using ARM 11. Best Regards Naveen

3 2

[U-Boot] [PATCH 2/2] Update i386 code (sc520_cdp)
by Graeme Russ 06 Sep '08

06 Sep '08

<this post (un)intentionally left blank> This post was going to include /include/asm-i386/ic/ssi.h but I figured out how to get it into the diff file

1 0

[U-Boot] [PATCH 0/2] Update i386 code (sc520_cdp)
by Graeme Russ 06 Sep '08

06 Sep '08

This is the first time I have submitted to an open source project, so please bear with me... I have been trying to compile U-Boot for a custom AMD SC520 board for the last few weeks. I started on the 1.3.4 release but figured it would be for the betterment of U-Boot to get the i386 code base back in line with the master branch. As far as I can tell, the i386 / sc520 ports have not been actively developed for some time (maybe as far back as 2003/2004). If anyone else on this list has build for i386 / sc520 using a recent commit, can you please let me know. My build environment is: - Ubuntu 8.04 (i86_32) - gcc (GCC) 4.2.3 (Ubuntu 4.2.3-2ubuntu7) - GNU Make 3.81 - Eclipse Version: 3.4.0 In my efforts, I think I may have uncovered a few issues which go beyond the i386 port. These mainly relate to PCI and ATA as described below. As a result, I have had to temporarily disable support for SATA, PCI and VIDEO for the sc520_cdp board. I have added a #define GRUSS_TESTING in /inlude/configs/sc520_cdp.h which is used to conditionally undefine: CONFIG_CMD_SATA CFG_SATA_MAXBUS CFG_SATA_DEVS_PER_BUS CFG_SATA_MAX_DEVICE CONFIG_ATA_PIIX CONFIG_VIDEO CONFIG_PCI CONFIG_PCI_PNP CONFIG_PCI_SCAN_SHOW Some #ifdef CONFIG_PCI and #ifdef CONFIG_VIDEO have been added to /lib_i386/video_bios.c and /lib_i386/bios_setup.c to resolve several compiler and linker warnings and errors as a result of removing PCI and Video support The detailed list of errors "sidestepped" by these 'hacks' is listed below (apologies for the formatting - I probably should have included compiler output) file: /drivers/block/ata_piix.c line 313 code: printf ("ata%u is slow to respond,plz be patient\n", port); error: format '%u' expects type 'unsigned int', but argument 2 has type 'struct sata_port *' line 321 code: printf ("ata%u failed to respond : ", port); error: format '%u' expects type 'unsigned int', but argument 2 has type 'struct sata_port *' line 366 code: input_data (&port[num].ioaddr, (ulong *) iobuf, ATA_SECTORWORDS); error: dereferencing type-punned pointer will break strict-aliasing rules lines 392 to 397 code: sata_cpy (sata_dev_desc[devno].revision, iop->fw_rev, sizeof (sata_dev_desc[devno].revision)); sata_cpy (sata_dev_desc[devno].vendor, iop->model, sizeof (sata_dev_desc[devno].vendor)); sata_cpy (sata_dev_desc[devno].product, iop->serial_no, sizeof (sata_dev_desc[devno].product)); error: pointer targets in passing argument 1 of 'sata_cpy' differ in signedness file: /drivers/block/ata_piix.h lines 45 to 49 code: #define ata_id_u64(id,n) \ (((u64) (id)[(n) + 3] << 48) | \ ((u64) (id)[(n) + 2] << 32) | \ ((u64) (id)[(n) + 1] << 16) | \ ((u64) (id)[(n) + 0]) ) error: "ata_id_u64" redefined lines 53 to 67 code: static inline void ata_dump_id (u16 * id) { PRINTF ("49 = 0x%04x " "53 = 0x%04x " "63 = 0x%04x " "64 = 0x%04x " "75 = 0x%04x \n", id[49], id[53], id[63], id[64], id[75]); PRINTF ("80 = 0x%04x " "81 = 0x%04x " "82 = 0x%04x " "83 = 0x%04x " "84 = 0x%04x \n", id[80], id[81], id[82], id[83], id[84]); PRINTF ("88 = 0x%04x " "93 = 0x%04x\n", id[88], id[93]); } error: static declaration of 'ata_dump_id' follows non-static declaration file: /include/libata.h lines 451 - 455 code: #define ata_id_u64(id,n) \ ( ((u64) (id)[(n) + 3] << 48) | \ ((u64) (id)[(n) + 2] << 32) | \ ((u64) (id)[(n) + 1] << 16) | \ ((u64) (id)[(n) + 0]) ) error: this is the location of the previous definition line 666 code: void ata_dump_id(u16 *id); error: previous declaration of 'ata_dump_id' was here file: /lib_i386/video_bios.c lines 88 to 90 code: if ((devbusfn = pci_find_class(PCI_CLASS_VIDEO, PCI_CLASS_VIDEO_STD, PCI_CLASS_VIDEO_PROG_IF_VGA, 0)) != -1) { error: undefined reference to `pci_find_class'

1 0

[U-Boot] running u-boot from SDRAM
by Roman Mashak 06 Sep '08

06 Sep '08

Hello, I need to port U-Boot on the platform, consisting of master and daughter boards. Master boards runs WinCE; daughter board features SoC, based on ARM926EJ-S, on-chip SDRAM and ROM. Master launches daughter board by downloading binary image of bootcode in to the on-chip SDRAM, then does remap (i.e. 0x0 points at SDRAM) and boot loader starts off. Therefore I'm wondering -- will U-Boot be fine with such a boot mode, usually systems start from flash (NOR or NAND), but here it's in RAM. I believe it should be OK, but just want to make sure. Looking forward to receiving feedback from you. Thanks. -- Roman Mashak

2 2

Re: [U-Boot] [new uImage] Cleanup cmd_bootm.c
by Wolfgang Denk 06 Sep '08

06 Sep '08

Dear Marian, in commit 1ee1180b you added code to common/cmd_bootm.c which uses the env_get_char() function. By then, this went through unnoticed. I'm sorry, but env_get_char() is a function that is internal to the environment handling code. It is not supposed to be exported to arbitrary code. Please do not make any assumptions about the internals of how the environment is stored. Could you please provide a cleanup patch? Thanks in advance. 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 "There's only one way to have a happy marriage and as soon as I learn what it is I'll get married again." - Clint Eastwood

3 7

[U-Boot-Users] [PATCH] (resubmit) ppc4xx: ML507: Environment in flash and MTD Support
by Ricardo Ribalda Delgado 06 Sep '08

06 Sep '08

-Relocate the location of u-boot in the flash -Save the environment in one sector of the flash memory -MTD Support Signed-off-by: Ricardo Ribalda Delgado <ricardo.ribalda(a)uam.es> --- -Previus commit included files from other commit Makefile | 2 +- board/xilinx/ml507/xparameters.h | 2 +- include/configs/ml507.h | 15 +++++++++++---- 3 files changed, 13 insertions(+), 6 deletions(-) diff --git a/Makefile b/Makefile index 8f4fdd0..ea572cf 100644 --- a/Makefile +++ b/Makefile @@ -1352,7 +1352,7 @@ ml300_config: unconfig ml507_flash_config: unconfig @mkdir -p $(obj)include $(obj)board/xilinx/ml507 @cp $(obj)board/xilinx/ml507/u-boot-rom.lds $(obj)board/xilinx/ml507/u-boot.lds - @echo "TEXT_BASE = 0xFE3E0000" > $(obj)board/xilinx/ml507/config.tmp + @echo "TEXT_BASE = 0xFE360000" > $(obj)board/xilinx/ml507/config.tmp @$(MKCONFIG) $(@:_flash_config=) ppc ppc4xx ml507 xilinx ml507_config: unconfig diff --git a/board/xilinx/ml507/xparameters.h b/board/xilinx/ml507/xparameters.h index 6a8e183..77d2ddf 100644 --- a/board/xilinx/ml507/xparameters.h +++ b/board/xilinx/ml507/xparameters.h @@ -24,7 +24,7 @@ #define XPAR_DDR2_SDRAM_MEM_BASEADDR 0x00000000 #define XPAR_IIC_EEPROM_BASEADDR 0x81600000 #define XPAR_INTC_0_BASEADDR 0x81800000 -#define XPAR_LLTEMAC_0_BASEADDR 0x81c00000 +#define XPAR_LLTEMAC_0_BASEADDR 0x81C00000 #define XPAR_UARTLITE_0_BASEADDR 0x84000000 #define XPAR_FLASH_MEM0_BASEADDR 0xFE000000 #define XPAR_PLB_CLOCK_FREQ_HZ 100000000 diff --git a/include/configs/ml507.h b/include/configs/ml507.h index a79bc1e..c653a51 100644 --- a/include/configs/ml507.h +++ b/include/configs/ml507.h @@ -33,7 +33,7 @@ #define CFG_SDRAM_SIZE_MB 256 #define CFG_MONITOR_BASE TEXT_BASE #define CFG_MONITOR_LEN ( 192 * 1024 ) -#define CFG_MALLOC_LEN ( 128 * 1024 ) +#define CFG_MALLOC_LEN ( CFG_ENV_SIZE + 128 * 1024 ) /*Uart*/ #define CONFIG_XILINX_UARTLITE @@ -49,6 +49,8 @@ #define CONFIG_CMD_ELF #define CONFIG_CMD_IRQ #define CONFIG_CMD_REGINFO +#define CONFIG_CMD_JFFS2 +#define CONFIG_JFFS2_CMDLINE #undef CONFIG_CMD_I2C #undef CONFIG_CMD_DTT #undef CONFIG_CMD_NET @@ -58,9 +60,11 @@ #undef CONFIG_CMD_IMLS /*Env*/ -#define CFG_ENV_IS_NOWHERE -#define CFG_ENV_SIZE 0x200 -#define CFG_ENV_OFFSET 0x100 +#define CFG_ENV_IS_IN_FLASH +#define CFG_ENV_SIZE 0x20000 +#define CFG_ENV_SECT_SIZE 0x20000 +#define CFG_ENV_OFFSET 0x340000 +#define CFG_ENV_ADDR (XPAR_FLASH_MEM0_BASEADDR+CFG_ENV_OFFSET) /*Misc*/ #define CONFIG_BOOTDELAY 5 /* autoboot after 5 seconds */ @@ -111,5 +115,8 @@ #define CFG_MAX_FLASH_BANKS 1 #define CFG_MAX_FLASH_SECT 259 #define CFG_FLASH_PROTECTION +#define MTDIDS_DEFAULT "nor0=ml507-flash" +#define MTDPARTS_DEFAULT "mtdparts=ml507-flash:-(user)" + #endif /* __CONFIG_H */ -- 1.5.6.3

2 5

Re: [U-Boot] [U-Boot-Users] [PATCH] Optimize ext2fs read functions
by Wolfgang Denk 06 Sep '08

06 Sep '08

Dear Ryan, In message <00b001c901aa$56c5c700$30065e0a(a)SHZ.ST.COM> you wrote: > > I'm sorry that I can't send patch mail by git-send-mail because I'm in ST internal network. > I just post the whole output by git-format-patch tool. That's OK. > If it's buggy, please let me know. Not really buggy, but: > > >From 0a08bfe1f0fc864a9920501a825e6698d082f231 Mon Sep 17 00:00:00 2001 > From: Ryan Chen <ryan.chen(a)st.com> > Date: Tue, 19 Aug 2008 07:11:23 -0400 > Subject: [PATCH] Optimize ext2fs read functions > > --- Your Signed-off-by: line is missing here. And there were a few white space errors in your code: .dotest/patch:94: trailing whitespace. while(((inode->b.blocks.dir_blocks[fileblock + 1] - .dotest/patch:174: trailing whitespace. blknr = blknr << log2blocksize; .dotest/patch:175: trailing whitespace. .dotest/patch:181: trailing whitespace. .dotest/patch:186: trailing whitespace. warning: squelched 2 whitespace errors warning: 7 lines applied after fixing whitespace errors. I can confirm that the code indeed improves read performance; in my tests (reading a file of 1580 KiB from a PCMCIA attached harddisk ona MPC855P at 66 MHz) I measured an improvement of about 2.8, i. . the read time for this fil went down from 7.2 to 2.6 seconds. Please clean up and resubmit. Thanks in advance. 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 Quotation, n. The act of repeating erroneously the words of another. The words erroneously repeated. - Ambrose Bierce _The Devil's Dictionary_

1 0

Re: [U-Boot] usb/25
by u-boot＠bugs.denx.de 06 Sep '08

06 Sep '08

Synopsis: [PATCH] USB EHCI: reset root hub State-Changed-From-To: open->closed State-Changed-By: wd State-Changed-When: Sat, 06 Sep 2008 01:26:27 +0200 State-Changed-Why: Applied (slightly modified) as commit e90fb6afab2c0c074dfb67bacb4de179eb188a24

1 0

[U-Boot] unassigned-patches/25: [PATCH] USB EHCI: reset root hub
by u-boot＠bugs.denx.de 06 Sep '08

06 Sep '08

Some of multi-function USB controllers (e.g. ISP1562) allow root hub resetting only via EHCI registers. So, this patch adds the corresponding kind of reset to OHCI's hc_reset() if the newly introduced CONFIG_PCI_EHCI_DEVNO option is set (e.g. for Socrates board). Signed-off-by: Yuri Tikhonov <yur(a)emcraft.com> --- Added to GNATS database as unassigned-patches/25 >Responsible: patch-coord >Message-Id: <1218807733-26929-3-git-send-email-dzu(a)denx.de> >In-Reply-To: <1218807733-26929-2-git-send-email-dzu(a)denx.de> >References: <1218807733-26929-1-git-send-email-dzu(a)denx.de> <1218807733-26929-2-git-send-email-dzu(a)denx.de> >Patch-Date: Fri Aug 15 15:42:10 +0200 2008 --- drivers/usb/usb_ohci.c | 31 +++++++++++++++++++++++++++++++ drivers/usb/usb_ohci.h | 3 +++ include/configs/socrates.h | 1 + 3 files changed, 35 insertions(+), 0 deletions(-) diff --git a/drivers/usb/usb_ohci.c b/drivers/usb/usb_ohci.c index fd60edb..0f5bd06 100644 --- a/drivers/usb/usb_ohci.c +++ b/drivers/usb/usb_ohci.c @@ -1571,11 +1571,42 @@ int submit_int_msg(struct usb_device *dev, unsigned long pipe, void *buffer, static int hc_reset (ohci_t *ohci) { +#ifdef CONFIG_PCI_EHCI_DEVNO + pci_dev_t pdev; + struct pci_device_id ehci_pci_ids[] = { + {0x1131, 0x1562}, /* Philips 1562 PCI EHCI module ids */ + {0, 0} + }; +#endif int timeout = 30; int smm_timeout = 50; /* 0,5 sec */ dbg("%s\n", __FUNCTION__); +#ifdef CONFIG_PCI_EHCI_DEVNO + /* + * Some multi-function controllers (e.g. ISP1562) allow root hub + * resetting via EHCI registers only. + */ + pdev = pci_find_devices(ehci_pci_ids, CONFIG_PCI_EHCI_DEVNO); + if (pdev != -1) { + u32 base; + int timeout = 1000; + + pci_read_config_dword(pdev, PCI_BASE_ADDRESS_0, &base); + writel (readl(base + EHCI_USBCMD_OFF) | EHCI_USBCMD_HCRESET, + base + EHCI_USBCMD_OFF); + + while (readl(base + EHCI_USBCMD_OFF) & EHCI_USBCMD_HCRESET) { + if (timeout-- <= 0) { + printf("USB RootHub reset timed out!"); + break; + } + udelay(1); + } + } else + printf("No EHCI func at %d index!\n", CONFIG_PCI_EHCI_DEVNO); +#endif if (readl (&ohci->regs->control) & OHCI_CTRL_IR) { /* SMM owns the HC */ writel (OHCI_OCR, &ohci->regs->cmdstatus); /* request ownership */ info("USB HC TakeOver from SMM"); diff --git a/drivers/usb/usb_ohci.h b/drivers/usb/usb_ohci.h index 380cb4c..7a04bf5 100644 --- a/drivers/usb/usb_ohci.h +++ b/drivers/usb/usb_ohci.h @@ -195,6 +195,9 @@ struct ohci_regs { } roothub; } __attribute((aligned(32))); +/* Some EHCI controls */ +#define EHCI_USBCMD_OFF 0x20 +#define EHCI_USBCMD_HCRESET (1 << 1) /* OHCI CONTROL AND STATUS REGISTER MASKS */ diff --git a/include/configs/socrates.h b/include/configs/socrates.h index 8a64942..fdc1557 100644 --- a/include/configs/socrates.h +++ b/include/configs/socrates.h @@ -403,6 +403,7 @@ #define CONFIG_USB_OHCI_NEW 1 #define CONFIG_PCI_OHCI 1 #define CONFIG_PCI_OHCI_DEVNO 3 /* Number in PCI list */ +#define CONFIG_PCI_EHCI_DEVNO (CONFIG_PCI_OHCI_DEVNO / 2) #define CFG_USB_OHCI_MAX_ROOT_PORTS 15 #define CFG_USB_OHCI_SLOT_NAME "ohci_pci" #define CFG_OHCI_SWAP_REG_ACCESS 1 -- 1.5.6.1

4 6

Re: [U-Boot] common/23
by u-boot＠bugs.denx.de 06 Sep '08

06 Sep '08

Synopsis: [PATCH] RX 8025 RTC: analyze 12/24-hour mode flag in rtc_get(). State-Changed-From-To: open->closed State-Changed-By: wd State-Changed-When: Sat, 06 Sep 2008 01:23:16 +0200 State-Changed-Why: Applied as commit 5875d358f025c1b042d8a0f08384b756de7256c9

1 0