June 2013 - U-Boot

[U-Boot] [PATCH] cmd_bootm: Add command line arguments to Plan 9
by Steven Stallion 10 Jun '13

10 Jun '13

This patch introduces support for command line arguments to Plan 9. Plan 9 generally dedicates a small region of kernel memory (known as CONFADDR) for runtime configuration. A new environment variable named confaddr was introduced to indicate this location when copying arguments. Signed-off-by: Steven Stallion <sstallion(a)gmail.com> --- common/cmd_bootm.c | 19 +++++++++++++++++++ 1 files changed, 19 insertions(+), 0 deletions(-) diff --git a/common/cmd_bootm.c b/common/cmd_bootm.c index 05130b6..5c62271 100644 --- a/common/cmd_bootm.c +++ b/common/cmd_bootm.c @@ -1533,6 +1533,7 @@ static int do_bootm_plan9(int flag, int argc, char * const argv[], bootm_headers_t *images) { void (*entry_point)(void); + char *s; if ((flag != 0) && (flag != BOOTM_STATE_OS_GO)) return 1; @@ -1544,6 +1545,24 @@ static int do_bootm_plan9(int flag, int argc, char * const argv[], } #endif + if ((s = getenv("confaddr")) != NULL) { + char *confaddr = (char *)simple_strtoul(s, NULL, 16); + + if (argc > 2) { + int i; + + s = confaddr; + for (i = 2; i < argc; i++) { + if (i > 2) + *s++ = '\n'; + strcpy(s, argv[i]); + s += strlen(argv[i]); + } + } else if ((s = getenv("bootargs")) != NULL) { + strcpy(confaddr, s); + } + } + entry_point = (void (*)(void))images->ep; printf("## Transferring control to Plan 9 (at address %08lx) ...\n", -- 1.7.0.4

2 3

[U-Boot] usb, composite: after unregister gadget driver set composite to NULL
by Heiko Schocher 09 Jun '13

09 Jun '13

Without this, second usb_composite_register() call fails always with -EINVAL. Signed-off-by: Heiko Schocher <hs(a)denx.de> Cc: Lukasz Majewski <l.majewski(a)samsung.com> Cc: Kyungmin Park <kyungmin.park(a)samsung.com> Cc: Marek Vasut <marex(a)denx.de> --- drivers/usb/gadget/composite.c | 1 + 1 Datei geändert, 1 Zeile hinzugefügt(+) diff --git a/drivers/usb/gadget/composite.c b/drivers/usb/gadget/composite.c index 0716e52..f867793 100644 --- a/drivers/usb/gadget/composite.c +++ b/drivers/usb/gadget/composite.c @@ -1099,4 +1099,5 @@ void usb_composite_unregister(struct usb_composite_driver *driver) if (composite != driver) return; usb_gadget_unregister_driver(&composite_driver); + composite = NULL; } -- 1.7.11.7

3 2

Re: [U-Boot] [PATCH] cmd_bootm: Add command line arguments to Plan 9
by Wolfgang Denk 09 Jun '13

09 Jun '13

Dear Steven, please always keep the mailing list on Cc: - this is especially important for patch reviews, where such messages need to be picked up for patchwork, too. In message <CAGGHmKHmLAd_85SgHyC=cEUmHu8u4ENQyj3Wt3rqyVdzAtWoSw(a)mail.gmail.com> you wrote: > > > Please make this code configurable, so that people who never intend to > > use Plan 9 do not suffer from the increased code size. > > This is already done, if you look at the do_bootm_plan9 function, you'll > see it is guarded by CONFIG_BOOTM_PLAN9. These changes only affect users > that are booting Plan 9. I see. Hm... I wonder which version of U-Boot your patch is against? The line numbers in your patch are off by at least 126 lines, and common/cmd_bootm.c has not been touched for many months ? > > ERROR: do not use assignment in if condition > > I noticed the errors, however the style and format of my changes are the > same as those in other functions in cmd_bootm; do_bootm_netbsd probably > being the closest example. I did not watch to introduce style drift. But we should not add more bad style code either. Feel free to send a cleanup patch for the existing code parts. In any case, do not add more such stuff. > It's as unlimited as you have memory :-) That said, adjacent pages to > CONFADDR are zeroed out at boot, so there is no possibility of overflow > once you branch to the kernel. Adjacent pages being zeroed - that means that you _are_ actually limited to one page size? > > Why do you make this (completely undocumented!!!) distinction between > > "bootm" being used with one or more arguments? Why can you not > > simply _always_ use bootargs ? > > This is to support passing arguments via bootm. This behavior is consistent > with NetBSD. ...which I consider a really weird desiign that IMO should not be followed without need. If you decide to do so nevertheless, then please 1) document the behaviour 2) factor out the common code instead of copying it > > What if the user did not set the "confaddr" environment variable? > > Then the memory reagion there is left uninitialized? Does this not > > cause undefined behaviour when booting Plan 9? > > There are checks which account for uninitialized memory at boot. It's ugly, > but it's how the OS deals with configuration. I don't like it either. How does it detect if there are valid arguments (versus random crap) in the memory range starting at "confaddr"? I can see no checksum or similar? > And how does Plan 9 learn where to find this date? I cannot see how > > you pass this address on to Plan 9? > > Like most things in Plan 9, it is a compiled offset (defined in mem.h). > CONFADDR is fixed, so as long as the configuration is dumped to the right > location (which can change between kernels), it will work. But then makes no sense to use a "confaddr" environment variable for this - the user has no real choice of setting this variable: either it matches the fixed CONFADDR value, in which case it works, or it is different, in which case it will silently fail. This is bad. I think you should use a CONFIG_SYS_CONFADDR constant instead. > Even worse - this code is actually pretty dangerous: "confaddr" is > > neither a reserved name, nor is it in any way exotic enough to be sure > > nobody else would use this in his environment. But if somebody does, > > he will suddenly find that some (for him random) data is copied unex- > > pectedly to that memory range. This may cause nasty crashes or other > > undefined behaviour which is very difficult to debug. > > True, but it's no different than loadaddr. This only affects Plan 9 users, > and this behavior will be documented for the kernels which support U-Boot. > I don't think there is too much danger here, though documentation will be > important. It is not documented in U-Boot. But actually I think this is void - using an envrionment variable here makes no sense in the first place. 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 To be is to program.

2 4

[U-Boot] [PATCH] MIPS: asm/errno.h: switch to asm-generic/errno.h
by Daniel Schwierzeck 08 Jun '13

08 Jun '13

This fixes several warnings like In file included from ./u-boot/include/linux/mtd/mtd.h:13:0, from env_onenand.c:37: ./u-boot/build/vct_platinumavc_onenand_small/include2/asm/errno.h:52:0: warning: "ENOMSG" redefined [enabled by default] Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck(a)gmail.com> --- arch/mips/include/asm/errno.h | 144 +----------------------------------------- 1 file changed, 1 insertion(+), 143 deletions(-) diff --git a/arch/mips/include/asm/errno.h b/arch/mips/include/asm/errno.h index 1665a63..4c82b50 100644 --- a/arch/mips/include/asm/errno.h +++ b/arch/mips/include/asm/errno.h @@ -1,143 +1 @@ -/* - * This file is subject to the terms and conditions of the GNU General Public - * License. See the file "COPYING" in the main directory of this archive - * for more details. - * - * Copyright (C) 1995, 1999, 2001, 2002 by Ralf Baechle - */ -#ifndef _ASM_MIPS_ERRNO_H -#define _ASM_MIPS_ERRNO_H - -/* - * These first 34 error codes are from Linux 2.6, <asm-generic/errno-base.h> - */ -#define EPERM 1 /* Operation not permitted */ -#define ENOENT 2 /* No such file or directory */ -#define ESRCH 3 /* No such process */ -#define EINTR 4 /* Interrupted system call */ -#define EIO 5 /* I/O error */ -#define ENXIO 6 /* No such device or address */ -#define E2BIG 7 /* Argument list too long */ -#define ENOEXEC 8 /* Exec format error */ -#define EBADF 9 /* Bad file number */ -#define ECHILD 10 /* No child processes */ -#define EAGAIN 11 /* Try again */ -#define ENOMEM 12 /* Out of memory */ -#define EACCES 13 /* Permission denied */ -#define EFAULT 14 /* Bad address */ -#define ENOTBLK 15 /* Block device required */ -#define EBUSY 16 /* Device or resource busy */ -#define EEXIST 17 /* File exists */ -#define EXDEV 18 /* Cross-device link */ -#define ENODEV 19 /* No such device */ -#define ENOTDIR 20 /* Not a directory */ -#define EISDIR 21 /* Is a directory */ -#define EINVAL 22 /* Invalid argument */ -#define ENFILE 23 /* File table overflow */ -#define EMFILE 24 /* Too many open files */ -#define ENOTTY 25 /* Not a typewriter */ -#define ETXTBSY 26 /* Text file busy */ -#define EFBIG 27 /* File too large */ -#define ENOSPC 28 /* No space left on device */ -#define ESPIPE 29 /* Illegal seek */ -#define EROFS 30 /* Read-only file system */ -#define EMLINK 31 /* Too many links */ -#define EPIPE 32 /* Broken pipe */ -#define EDOM 33 /* Math argument out of domain of func */ -#define ERANGE 34 /* Math result not representable */ - -/* - * These error numbers are intended to be MIPS ABI compatible - */ -#define ENOMSG 35 /* No message of desired type */ -#define EIDRM 36 /* Identifier removed */ -#define ECHRNG 37 /* Channel number out of range */ -#define EL2NSYNC 38 /* Level 2 not synchronized */ -#define EL3HLT 39 /* Level 3 halted */ -#define EL3RST 40 /* Level 3 reset */ -#define ELNRNG 41 /* Link number out of range */ -#define EUNATCH 42 /* Protocol driver not attached */ -#define ENOCSI 43 /* No CSI structure available */ -#define EL2HLT 44 /* Level 2 halted */ -#define EDEADLK 45 /* Resource deadlock would occur */ -#define ENOLCK 46 /* No record locks available */ -#define EBADE 50 /* Invalid exchange */ -#define EBADR 51 /* Invalid request descriptor */ -#define EXFULL 52 /* Exchange full */ -#define ENOANO 53 /* No anode */ -#define EBADRQC 54 /* Invalid request code */ -#define EBADSLT 55 /* Invalid slot */ -#define EDEADLOCK 56 /* File locking deadlock error */ -#define EBFONT 59 /* Bad font file format */ -#define ENOSTR 60 /* Device not a stream */ -#define ENODATA 61 /* No data available */ -#define ETIME 62 /* Timer expired */ -#define ENOSR 63 /* Out of streams resources */ -#define ENONET 64 /* Machine is not on the network */ -#define ENOPKG 65 /* Package not installed */ -#define EREMOTE 66 /* Object is remote */ -#define ENOLINK 67 /* Link has been severed */ -#define EADV 68 /* Advertise error */ -#define ESRMNT 69 /* Srmount error */ -#define ECOMM 70 /* Communication error on send */ -#define EPROTO 71 /* Protocol error */ -#define EDOTDOT 73 /* RFS specific error */ -#define EMULTIHOP 74 /* Multihop attempted */ -#define EBADMSG 77 /* Not a data message */ -#define ENAMETOOLONG 78 /* File name too long */ -#define EOVERFLOW 79 /* Value too large for defined data type */ -#define ENOTUNIQ 80 /* Name not unique on network */ -#define EBADFD 81 /* File descriptor in bad state */ -#define EREMCHG 82 /* Remote address changed */ -#define ELIBACC 83 /* Can not access a needed shared library */ -#define ELIBBAD 84 /* Accessing a corrupted shared library */ -#define ELIBSCN 85 /* .lib section in a.out corrupted */ -#define ELIBMAX 86 /* Attempting to link in too many shared libraries */ -#define ELIBEXEC 87 /* Cannot exec a shared library directly */ -#define EILSEQ 88 /* Illegal byte sequence */ -#define ENOSYS 89 /* Function not implemented */ -#define ELOOP 90 /* Too many symbolic links encountered */ -#define ERESTART 91 /* Interrupted system call should be restarted */ -#define ESTRPIPE 92 /* Streams pipe error */ -#define ENOTEMPTY 93 /* Directory not empty */ -#define EUSERS 94 /* Too many users */ -#define ENOTSOCK 95 /* Socket operation on non-socket */ -#define EDESTADDRREQ 96 /* Destination address required */ -#define EMSGSIZE 97 /* Message too long */ -#define EPROTOTYPE 98 /* Protocol wrong type for socket */ -#define ENOPROTOOPT 99 /* Protocol not available */ -#define EPROTONOSUPPORT 120 /* Protocol not supported */ -#define ESOCKTNOSUPPORT 121 /* Socket type not supported */ -#define EOPNOTSUPP 122 /* Operation not supported on transport endpoint */ -#define EPFNOSUPPORT 123 /* Protocol family not supported */ -#define EAFNOSUPPORT 124 /* Address family not supported by protocol */ -#define EADDRINUSE 125 /* Address already in use */ -#define EADDRNOTAVAIL 126 /* Cannot assign requested address */ -#define ENETDOWN 127 /* Network is down */ -#define ENETUNREACH 128 /* Network is unreachable */ -#define ENETRESET 129 /* Network dropped connection because of reset */ -#define ECONNABORTED 130 /* Software caused connection abort */ -#define ECONNRESET 131 /* Connection reset by peer */ -#define ENOBUFS 132 /* No buffer space available */ -#define EISCONN 133 /* Transport endpoint is already connected */ -#define ENOTCONN 134 /* Transport endpoint is not connected */ -#define EUCLEAN 135 /* Structure needs cleaning */ -#define ENOTNAM 137 /* Not a XENIX named type file */ -#define ENAVAIL 138 /* No XENIX semaphores available */ -#define EISNAM 139 /* Is a named type file */ -#define EREMOTEIO 140 /* Remote I/O error */ -#define EINIT 141 /* Reserved */ -#define EREMDEV 142 /* Error 142 */ -#define ESHUTDOWN 143 /* Cannot send after transport endpoint shutdown */ -#define ETOOMANYREFS 144 /* Too many references: cannot splice */ -#define ETIMEDOUT 145 /* Connection timed out */ -#define ECONNREFUSED 146 /* Connection refused */ -#define EHOSTDOWN 147 /* Host is down */ -#define EHOSTUNREACH 148 /* No route to host */ -#define EWOULDBLOCK EAGAIN /* Operation would block */ -#define EALREADY 149 /* Operation already in progress */ -#define EINPROGRESS 150 /* Operation now in progress */ -#define ESTALE 151 /* Stale NFS file handle */ -#define ECANCELED 158 /* AIO operation canceled */ - -#endif /* _ASM_MIPS_ERRNO_H */ +#include <asm-generic/errno.h> -- 1.8.1.2

1 0

[U-Boot] [PATCH v2 14/16] sf: Use spi_flash_read_common() in write status poll
by Jagannadha Sutradharudu Teki 08 Jun '13

08 Jun '13

Instead of using spi_xfer for SPI_XFER_BEGIN and SPI_XFER_END separatley use common read call spi_flash_read_common() which does the same. Signed-off-by: Jagannadha Sutradharudu Teki <jaganna(a)xilinx.com> --- Changes for v2: - none drivers/mtd/spi/spi_flash.c | 17 +++++------------ 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/drivers/mtd/spi/spi_flash.c b/drivers/mtd/spi/spi_flash.c index 821aa2e..765d4bc 100644 --- a/drivers/mtd/spi/spi_flash.c +++ b/drivers/mtd/spi/spi_flash.c @@ -193,32 +193,25 @@ int spi_flash_cmd_read_fast(struct spi_flash *flash, u32 offset, int spi_flash_cmd_poll_bit(struct spi_flash *flash, unsigned long timeout, u8 cmd, u8 poll_bit) { - struct spi_slave *spi = flash->spi; unsigned long timebase; int ret; u8 status; - ret = spi_xfer(spi, 8, &cmd, NULL, SPI_XFER_BEGIN); - if (ret) { - debug("SF: Failed to send command %02x: %d\n", cmd, ret); - return ret; - } - timebase = get_timer(0); do { WATCHDOG_RESET(); - ret = spi_xfer(spi, 8, NULL, &status, 0); - if (ret) - return -1; + ret = spi_flash_read_common(flash, &cmd, 1, &status, 1); + if (ret < 0) { + debug("SF: fail to read read status register\n"); + return ret; + } if ((status & poll_bit) == 0) break; } while (get_timer(timebase) < timeout); - spi_xfer(spi, 0, NULL, NULL, SPI_XFER_END); - if ((status & poll_bit) == 0) return 0; -- 1.8.3

2 1

[U-Boot] [PULL] : Please pull u-boot-imx
by Stefano Babic 08 Jun '13

08 Jun '13

Hi Albert, please pull from u-boot-imx, thanks. The following changes since commit cac423a730d3506154744485af1bbc1cd3a1e6a8: Merge branch 'u-boot-ti/master' into 'u-boot-arm/master' (2013-05-11 22:24:28 +0200) are available in the git repository at: git://www.denx.de/git/u-boot-imx.git master for you to fetch changes up to 4a1c7b13ae104d4526d3176793b7f6b06694df15: vf610twr: Drop unneeded 'status' variable (2013-06-06 17:52:08 +0200) ---------------------------------------------------------------- Alison Wang (7): arm: vf610: Add IOMUX support for Vybrid VF610 arm: vf610: Add Vybrid VF610 CPU support net: fec_mxc: Add support for Vybrid VF610 arm: vf610: Add watchdog support for Vybrid VF610 arm: vf610: Add uart support for Vybrid VF610 arm: vf610: Add Vybrid VF610 to mxc_ocotp document arm: vf610: Add basic support for Vybrid VF610TWR board Andrew Gabbasov (1): mx6: mx6qsabrelite/nitrogen6x: Remove incorrect setting of gpio CS signal Benoît Thébaudeau (1): imx: spl: Merge libimx-common make rules Fabio Estevam (7): mx28evk: Add splash screen support mx23evk: Add splash screen support video: mxsfb: Add an entry for mx23evk/mx28vk video modes mx6slevk: Allow booting a device tree kernel wandboard: Enable HDMI splashscreen ARM: imx: Fix incorrect usage of CONFIG_SYS_MMC_ENV_PART vf610twr: Drop unneeded 'status' variable Luka Perkov (1): wandboard: fix typo in README Marek Vasut (1): arm: mxs: Fix vectoring table crafting Otavio Salvador (2): build: Use generic boot logo matching wandboard: Add Boot Splash image with Wandboard logo Renato Frias (2): mx6qsabreauto: Add i2c to mx6qsabreauto board mx6qsabreauto: Add Port Expander reset SARTRE Leo (1): Add support for Congatec Conga-QEVAl board MAINTAINERS | 8 + Makefile | 2 +- arch/arm/cpu/arm926ejs/mxs/mxs.c | 25 +- arch/arm/cpu/armv7/vf610/Makefile | 42 +++ arch/arm/cpu/armv7/vf610/generic.c | 324 +++++++++++++++++++ arch/arm/cpu/armv7/vf610/timer.c | 103 ++++++ arch/arm/imx-common/Makefile | 2 +- arch/arm/imx-common/iomux-v3.c | 6 + arch/arm/include/asm/arch-vf610/clock.h | 39 +++ arch/arm/include/asm/arch-vf610/crm_regs.h | 225 +++++++++++++ arch/arm/include/asm/arch-vf610/imx-regs.h | 419 +++++++++++++++++++++++++ arch/arm/include/asm/arch-vf610/iomux-vf610.h | 101 ++++++ arch/arm/include/asm/imx-common/iomux-v3.h | 18 ++ board/boundary/nitrogen6x/nitrogen6x.c | 1 - board/congatec/cgtqmx6eval/Makefile | 42 +++ board/congatec/cgtqmx6eval/README | 29 ++ board/congatec/cgtqmx6eval/cgtqmx6eval.c | 167 ++++++++++ board/freescale/mx23evk/mx23evk.c | 6 + board/freescale/mx23evk/spl_boot.c | 32 ++ board/freescale/mx28evk/iomux.c | 33 ++ board/freescale/mx28evk/mx28evk.c | 6 + board/freescale/mx6qsabreauto/mx6qsabreauto.c | 57 ++++ board/freescale/mx6qsabrelite/mx6qsabrelite.c | 1 - board/freescale/vf610twr/Makefile | 39 +++ board/freescale/vf610twr/imximage.cfg | 33 ++ board/freescale/vf610twr/vf610twr.c | 407 ++++++++++++++++++++++++ board/wandboard/README | 4 +- board/wandboard/wandboard.c | 98 ++++++ boards.cfg | 2 + doc/README.mxc_ocotp | 1 + doc/README.vf610 | 10 + drivers/net/fec_mxc.c | 4 +- drivers/serial/Makefile | 1 + drivers/serial/serial_lpuart.c | 132 ++++++++ drivers/video/mxsfb.c | 5 + drivers/watchdog/Makefile | 2 +- include/configs/cgtqmx6eval.h | 194 ++++++++++++ include/configs/mx23evk.h | 17 + include/configs/mx28evk.h | 17 + include/configs/mx53ard.h | 3 +- include/configs/mx6qsabre_common.h | 2 +- include/configs/mx6qsabreauto.h | 7 +- include/configs/mx6qsabresd.h | 1 - include/configs/mx6slevk.h | 2 +- include/configs/vf610twr.h | 140 +++++++++ include/configs/wandboard.h | 20 +- spl/Makefile | 6 +- tools/Makefile | 27 +- tools/logos/wandboard.bmp | Bin 0 -> 22390 bytes 49 files changed, 2818 insertions(+), 44 deletions(-) create mode 100644 arch/arm/cpu/armv7/vf610/Makefile create mode 100644 arch/arm/cpu/armv7/vf610/generic.c create mode 100644 arch/arm/cpu/armv7/vf610/timer.c create mode 100644 arch/arm/include/asm/arch-vf610/clock.h create mode 100644 arch/arm/include/asm/arch-vf610/crm_regs.h create mode 100644 arch/arm/include/asm/arch-vf610/imx-regs.h create mode 100644 arch/arm/include/asm/arch-vf610/iomux-vf610.h create mode 100644 board/congatec/cgtqmx6eval/Makefile create mode 100644 board/congatec/cgtqmx6eval/README create mode 100644 board/congatec/cgtqmx6eval/cgtqmx6eval.c create mode 100644 board/freescale/vf610twr/Makefile create mode 100644 board/freescale/vf610twr/imximage.cfg create mode 100644 board/freescale/vf610twr/vf610twr.c create mode 100644 doc/README.vf610 create mode 100644 drivers/serial/serial_lpuart.c create mode 100644 include/configs/cgtqmx6eval.h create mode 100644 include/configs/vf610twr.h create mode 100644 tools/logos/wandboard.bmp -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic(a)denx.de =====================================================================

2 1

[U-Boot] MIPS FLASH mapping
by Drassal, Allan 08 Jun '13

08 Jun '13

Hello, I have been working for a while trying to port U-Boot over to a MIPS AR7161 SoC chip. The actual product is a Buffalo WZR-HP-AG300H router. I have been making quite a bit of successful progress, bu the two FLASH chip archetecture is throwing me off a little. There are two FLASH chips, 16MiB W25Q128BV. The first FLASH chip is where U-Boot resides and is mapped to 0xBF000000 to 0xBFFFFFFF. The second FLASH chip is suppsed to map to 0xBE000000 to 0xBEFFFFFF. I have been successful in modifying the driver (with a few glitches that I am still fixing) to support the second chip. Since the FLASH is SPI based, the first or second chip is selected with a CS line, which I added support for in the driver. The two chips are detected properly at startup, and the size is correct, however, the mapping (KSEG1?) is mapping only the first chip, and I have been looking for how to get the second chip mapped. I think it has to do with the lowlevel init functions, but am difficulty understanding where the FLASH is mapped. I am fairly sure I have the low level setup correct, PLL, DDR, etc, which took a little work since I can't locate any techinical manuals for this chip, I had to go off reverse engineering the registers based on other ar71xx based boards. The original bootloader maps these correctly, and in openocd I can read data from the second FLASH chip. However, when I run the new bootloader I am working on the addresses are not mapped correctly for the second FLASH chip, even in openocd. Any assistance would be appreciated. When I get something working a little better I would like to contribute the results. Allan

1 0

Re: [U-Boot] [U-BOOT] mmc: sdhci: Enable 8-bit bus width only for 3.0 spec onwards
by Jagannadha Sutradharudu Teki 08 Jun '13

08 Jun '13

Request for an update on this. Thanks, Jagan. > -----Original Message----- > From: Jagannadha Sutradharudu Teki [mailto:jagannadha.sutradharudu- > teki(a)xilinx.com] > Sent: 21 May 2013 15:02 > To: u-boot(a)lists.denx.de > Cc: monstr(a)monstr.eu; Andy Fleming; Jagannadha Sutradharudu Teki > Subject: [U-BOOT] mmc: sdhci: Enable 8-bit bus width only for 3.0 spec onwards > > CAP register don't have any information for 8-bit buswidth support on 2.0 sdhci > spec, only from 3.0 onwards bit[18] got this information. > > Due to this misassignment in sdhci, mmc is setting 8-bit buswidth using > mmc_set_bus_width even if controller doesn't support. > Below change has code information. > "mmc: Properly determine maximum supported bus width" > (sha1: 7798f6dbd5e1a3030ed81a81da5dfb57c3307cac) > > Bug log: <mmc plus and emmc cards) > ------- > zynq-uboot> mmcinfo > Error detected in status(0x208100)! > Device: zynq_sdhci > Manufacturer ID: fe > ..... > > So enable 8-bit support only for 3.0 spec using CAP and for below 3.0 assign > mmc->host_caps = MMC_MODE_8BIT on respective platform driver if host have > a support. > > Signed-off-by: Jagannadha Sutradharudu Teki <jaganna(a)xilinx.com> > --- > drivers/mmc/sdhci.c | 6 ++++-- > 1 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c index 1eaea04..c5631bf > 100644 > --- a/drivers/mmc/sdhci.c > +++ b/drivers/mmc/sdhci.c > @@ -486,8 +486,10 @@ int add_sdhci(struct sdhci_host *host, u32 max_clk, > u32 min_clk) > mmc->voltages |= host->voltages; > > mmc->host_caps = MMC_MODE_HS | MMC_MODE_HS_52MHz | > MMC_MODE_4BIT; > - if (caps & SDHCI_CAN_DO_8BIT) > - mmc->host_caps |= MMC_MODE_8BIT; > + if ((host->version & SDHCI_SPEC_VER_MASK) >= SDHCI_SPEC_300) { > + if (caps & SDHCI_CAN_DO_8BIT) > + mmc->host_caps |= MMC_MODE_8BIT; > + } > if (host->host_caps) > mmc->host_caps |= host->host_caps; > > -- > 1.7.4 > This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.

3 2

[U-Boot] pull request: u-boot-arm/master
by Albert ARIBAUD 07 Jun '13

07 Jun '13

Hello Tom, The following changes since commit 3da0e5750b24a9491058df6126c7be577a276c09: arm: factorize relocate_code routine (2013-05-30 20:24:38 +0200) are available in the git repository at: git://git.denx.de/u-boot-arm master for you to fetch changes up to 4596dcc1d4ea5763e0f92cf5becd9fc7d4c6e674: am33xx/omap: Move save_omap_boot_params to omap-common/boot-common.c (2013-06-05 08:46:49 -0400) ---------------------------------------------------------------- Tom Rini (3): omap-common/hwinit-common.c: Mark omap_rev_string as static am33xx: Correct NON_SECURE_SRAM_START/END am33xx/omap: Move save_omap_boot_params to omap-common/boot-common.c arch/arm/cpu/armv7/omap-common/boot-common.c | 39 ++++++++++++++++++++++++ arch/arm/cpu/armv7/omap-common/hwinit-common.c | 38 +---------------------- arch/arm/include/asm/arch-am33xx/omap.h | 4 +-- arch/arm/include/asm/arch-am33xx/sys_proto.h | 1 + arch/arm/include/asm/arch-omap4/sys_proto.h | 1 + arch/arm/include/asm/arch-omap5/sys_proto.h | 1 + board/isee/igep0033/board.c | 9 ++++++ board/phytec/pcm051/board.c | 9 ++++++ board/ti/am335x/board.c | 9 ++++++ board/ti/ti814x/evm.c | 9 ++++++ include/configs/am335x_evm.h | 10 ++++-- include/configs/igep0033.h | 10 ++++-- include/configs/pcm051.h | 10 ++++-- 13 files changed, 105 insertions(+), 45 deletions(-) Amicalement, -- Albert.

2 1

[U-Boot] Pull request: u-boot-video/master
by Anatolij Gustschin 07 Jun '13

07 Jun '13

Hey Tom, The following changes since commit 5ed6f447af60aabd2669d913f673793c1ce48f47: P1022DS: Set CONFIG_SPL_MAX_SIZE directly (2013-05-03 09:19:43 -0400) are available in the git repository at: git://git.denx.de/u-boot-video.git master for you to fetch changes up to ea697ae7eb059d7467c6854ce562c31df510eb59: ARM: bcm2835: add simplefb DT node during bootz/m (2013-06-05 22:40:38 +0200) ---------------------------------------------------------------- Eric Nelson (1): cfb_console: flush FB cache at end of public functions Stephen Warren (2): lcd: add functions to set up simplefb device tree ARM: bcm2835: add simplefb DT node during bootz/m board/raspberrypi/rpi_b/rpi_b.c | 14 +++- common/lcd.c | 87 ++++++++++++++++++++ .../video/simple-framebuffer.txt | 25 ++++++ drivers/video/cfb_console.c | 15 ++-- include/configs/rpi_b.h | 2 + include/lcd.h | 3 + 6 files changed, 139 insertions(+), 7 deletions(-) create mode 100644 doc/device-tree-bindings/video/simple-framebuffer.txt Please pull. Thanks! Anatolij

2 1