Re: [U-Boot] Data abort in omap_gpio

2014-04-10 Thread Andreas Bießmann
Dear Tom Rini, On 04/09/2014 05:37 PM, Tom Rini wrote: > On Wed, Apr 09, 2014 at 05:22:57PM +0200, Andreas Bießmann wrote: >> (gdb) s >> 118 __raw_writel(l, reg); >> (gdb) print/s l >> $17 = 2048 >> (gdb) print/x l >> $18 = 0x800 >> (gdb) print/x reg >> $19 = 0x49050090 >> (gdb) s >> ^C

[U-Boot] [u-boot][coreboot-x86] Incremental Make fails

2014-04-10 Thread Hua Yanghao
Hi, When doing make for coreboot-x86, in arch/x86/lib/Makefile it re-generates libgcc.a from the original toolchain libgcc.a and did some patching for certain libgcc functions. And this seems having some issues when you try to do incremental compilation. Steps to reproduce the issue: 1. Setup tool

Re: [U-Boot] Data abort in omap_gpio

2014-04-10 Thread Andreas Bießmann
On 04/10/2014 09:22 AM, Andreas Bießmann wrote: > On 04/09/2014 05:37 PM, Tom Rini wrote: >> On Wed, Apr 09, 2014 at 05:22:57PM +0200, Andreas Bießmann wrote: > >>> (gdb) s >>> 118 __raw_writel(l, reg); >>> (gdb) print/s l >>> $17 = 2048 >>> (gdb) print/x l >>> $18 = 0x800 >>> (gdb) print

Re: [U-Boot] mx28: U-Boot fails to boot with VDD5V only power configuration

2014-04-10 Thread Ankur Patel
> Dear Friends, > > I'm bringing up a custom i.MX28 based board.  I'm having trouble bringing up > the u-boot (version v2013.04) loader.  The > > Powersupply has been design as > detailed in section 4.1.2.1 (VDD5V source only) of application note AN4199: > > http://www.freescale.com/files/32bit/d

Re: [U-Boot] [u-boot][coreboot-x86] Incremental Make fails

2014-04-10 Thread Masahiro Yamada
Hi Hua, As far as I tested, the problem you mentioned did not happen. On Thu, 10 Apr 2014 14:59:33 +0800 Hua Yanghao wrote: > Hi, > When doing make for coreboot-x86, in arch/x86/lib/Makefile it > re-generates libgcc.a from the original toolchain libgcc.a and did > some patching for certain lib

Re: [U-Boot] Modify go to allow chain loading a second u-boot

2014-04-10 Thread Helmut Raiger
This kind of option is clearly not what other u-boot commands use, I could not find a single command that supports '-' style options. If this had been widely used, I wouldn't have asked in the first place. It seems you did not look very carefully. Just for example: => help env env - environme

Re: [U-Boot] [PATCH] Enable big-endian processors to use EHCI interrupt transfers

2014-04-10 Thread Marek Vasut
On Wednesday, April 09, 2014 at 11:03:02 PM, Adrian Cox wrote: > Add endian support macros to interrupt transfers in the EHCI driver, to > allow operation on PowerPC. > > Signed-off-by: Adrian Cox Other than style issues, which need to be fixes (you can check for them with ./scripts/checkpatch.

Re: [U-Boot] [PATCH] Fix USB keyboard polling via control endpoint

2014-04-10 Thread Marek Vasut
On Sunday, April 06, 2014 at 01:17:46 PM, Wolfgang Denk wrote: [...] > @@ -131,7 +133,9 @@ void usb_kbd_generic_poll(void) > /* Submit a interrupt transfer request */ > maxp = usb_maxpacket(usb_kbd_dev, pipe); > usb_submit_int_msg(usb_kbd_dev, pipe, data->new, > -

Re: [U-Boot] [PATCH 2/2] dfu, nand: add medium specific polltimeout function

2014-04-10 Thread Marek Vasut
On Thursday, April 10, 2014 at 07:08:06 AM, Heiko Schocher wrote: > add a possibility to add a medium specific polltimeout > function. So it is possible to define different > poll timeouts. > > Used on nand medium, for setting the DFU_MANIFEST_POLL_TIMEOUT > only on nand ubi partitions, which is c

Re: [U-Boot] [PATCH 1/2] musb-new, dfu: first send request answer then call completions

2014-04-10 Thread Marek Vasut
On Thursday, April 10, 2014 at 07:08:05 AM, Heiko Schocher wrote: > comment in ep0_txstate() states: > > "report completions as soon as the fifo's loaded; there's no win > in waiting till this last packet gets acked". > > This is wrong for using dfu. In the dfu usecase we must send > a PollTimeo

Re: [U-Boot] [PATCH 2/2] dfu, nand: add medium specific polltimeout function

2014-04-10 Thread Pantelis Antoniou
Hi Marek, On Apr 10, 2014, at 10:54 AM, Marek Vasut wrote: > On Thursday, April 10, 2014 at 07:08:06 AM, Heiko Schocher wrote: >> add a possibility to add a medium specific polltimeout >> function. So it is possible to define different >> poll timeouts. >> >> Used on nand medium, for setting the

Re: [U-Boot] [u-boot][coreboot-x86] Incremental Make fails

2014-04-10 Thread Hua Yanghao
Hi Masahiro, > Strange. > "make coreboot-x86" is not supported anymore. > > Now, we must do "make coreboot-x86_config all" instead. We are still using U-boot 2014.01, which seems to be the latest stable release, isn't it? > >> 3. make # second make fails with message: >> make[1]: Entering direct

Re: [U-Boot] [PATCH] Fix USB keyboard polling via control endpoint

2014-04-10 Thread Adrian Cox
> From: "Marek Vasut" > Also, this USB_KBD_PDATA_SIZE should instead be called > "USB_KBD_LS_REPORT_SIZE" > [...] > What worries me a bit is that 64-byte high-speed report, but I never > saw a > device that would generate those. This section 5.6 is also the only > place that > mentions the high-s

Re: [U-Boot] [PATCH] Fix USB keyboard polling via control endpoint

2014-04-10 Thread Marek Vasut
On Thursday, April 10, 2014 at 10:49:56 AM, Adrian Cox wrote: > > From: "Marek Vasut" > > > > Also, this USB_KBD_PDATA_SIZE should instead be called > > "USB_KBD_LS_REPORT_SIZE" > > [...] > > What worries me a bit is that 64-byte high-speed report, but I never > > saw a > > device that would gene

[U-Boot] [PATCH v1 1/6] mtd: nand: omap_elm: remove #include omap_gpmc.h

2014-04-10 Thread Pekon Gupta
There is no dependency of omap_elm.c on omap_gpmc.h Signed-off-by: Pekon Gupta --- drivers/mtd/nand/omap_elm.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/mtd/nand/omap_elm.c b/drivers/mtd/nand/omap_elm.c index 47b1f1b..4c65f3b 100644 --- a/drivers/mtd/nand/omap_elm.c +++ b/driver

[U-Boot] [PATCH v1 0/6] mtd: nand: omap: clean up of omap_elm and omap_gpmc driver

2014-04-10 Thread Pekon Gupta
This patch series - removes un-wanted & redundant code in omap_elm and omap_gpmc drivers - refactors some private struct in omap_gpmc to make them generic across ecc-schemes - minor code fixes and clean-ups This series can be as-it-is applied on u-boot 'v2014.04-rc3'. *Testing Procedure* Boot

[U-Boot] [PATCH v1 2/6] mtd: nand: omap_elm: use bch_type instead of nibble count to differentiate between BCH4/BCH8/BCH16

2014-04-10 Thread Pekon Gupta
ELM hardware engine support ECC error detection for multiple ECC strengths like +--++ |Type | ECC syndrome length| +--++ |BCH4 | 6.5 bytes = 13 nibbles | |BCH8 | 13 byte = 26 nibbles | |BCH16 | 26 bytes = 52 nibbles | +--+

[U-Boot] [PATCH v1 4/6] mtd: nand: omap_gpmc: remove unused members of 'struct nand_bch_priv'

2014-04-10 Thread Pekon Gupta
This patch prepares to refactor 'struct nand_bch_priv' -> 'struct omap_nand_info' And thus performs following clean-ups: - remove nand_bch_priv.type: use nand_bch_priv.ecc_scheme instead - remove nand_bch_priv.mode: Signed-off-by: Pekon Gupta --- drivers/mtd/nand/omap_gpmc.c | 23 ++-

[U-Boot] [PATCH v1 6/6] mtd: nand: omap_gpmc: minor cleanup of omap_correct_data_bch

2014-04-10 Thread Pekon Gupta
This patch tries to avoid some local pointer dereferences, by using common local variables in omap_correct_data_bch() Signed-off-by: Pekon Gupta --- drivers/mtd/nand/omap_gpmc.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/mtd/nand/omap_gpmc.c b/driver

[U-Boot] [PATCH v1 5/6] mtd: nand: omap_gpmc: rename struct nand_bch_priv to struct omap_nand_info

2014-04-10 Thread Pekon Gupta
This patch renames 'struct nand_bch_priv' which currently holds private data only for BCH ECC schemes, into 'struct omap_nand_info' so that same can be used for all ECC schemes Signed-off-by: Pekon Gupta --- drivers/mtd/nand/omap_gpmc.c | 58 1 file

[U-Boot] [PATCH v1 3/6] mtd: nand: omap_elm: use macros for register definitions

2014-04-10 Thread Pekon Gupta
This patch adds macros for following parameters of ELM Hardware engine - ELM_MAX_CHANNELS: ELM can process 8 data streams simultaneously - ELM_MAX_ERRORS: ELM can detect upto 16 ECC error when using BCH16 scheme Signed-off-by: Pekon Gupta --- include/linux/mtd/omap_elm.h | 9 ++--- 1 file

[U-Boot] [PATCH] am33xx: add SSC enable macro

2014-04-10 Thread yegorslists
From: Yegor Yefremov Signed-off-by: Yegor Yefremov --- arch/arm/include/asm/arch-am33xx/clock.h |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/arch/arm/include/asm/arch-am33xx/clock.h b/arch/arm/include/asm/arch-am33xx/clock.h index 7637457..f00fad3 100644 --- a/arch

Re: [U-Boot] [PATCH 2/2] dfu, nand: add medium specific polltimeout function

2014-04-10 Thread Lukasz Majewski
Hi Pantelis, > Hi Marek, > > On Apr 10, 2014, at 10:54 AM, Marek Vasut wrote: > > > On Thursday, April 10, 2014 at 07:08:06 AM, Heiko Schocher wrote: > >> add a possibility to add a medium specific polltimeout > >> function. So it is possible to define different > >> poll timeouts. > >> > >> Us

[U-Boot] [RFC] am33xx: add spread spectrum clock (SSC) support

2014-04-10 Thread Yegor Yefremov
I've found following source code implementing SSC in kernel on TI's forum (http://e2e.ti.com/support/arm/sitara_arm/f/791/t/198104.aspx source file: http://e2e.ti.com/cfs-file.ashx/__key/telligent-evolution-components-attachments/00-791-01-00-01-12-33-83/AM335x_5F00_Spread_5F00_Spectrum.c). Now I'

Re: [U-Boot] [PATCH] Fix USB keyboard polling via control endpoint

2014-04-10 Thread Wolfgang Denk
Dear Marek, In message <201404101012.42527.ma...@denx.de> you wrote: > On Sunday, April 06, 2014 at 01:17:46 PM, Wolfgang Denk wrote: > [...] > > > @@ -131,7 +133,9 @@ void usb_kbd_generic_poll(void) > > /* Submit a interrupt transfer request */ > > maxp = usb_maxpacket(usb_kbd_dev, pipe);

Re: [U-Boot] [PATCH 2/2] dfu, nand: add medium specific polltimeout function

2014-04-10 Thread Heiko Schocher
Hello Lukasz, Am 10.04.2014 12:08, schrieb Lukasz Majewski: Hi Pantelis, Hi Marek, On Apr 10, 2014, at 10:54 AM, Marek Vasut wrote: On Thursday, April 10, 2014 at 07:08:06 AM, Heiko Schocher wrote: add a possibility to add a medium specific polltimeout function. So it is possible to define

[U-Boot] [PATCH 1/3] board:tricorder: always work with valid eeprom data

2014-04-10 Thread Andreas Bießmann
From: Andreas Bießmann Commit 890880583d84607e36b52a785a96b167728bbf73 introduced EEPROM parsing and board detection but faild to return a valid tricorder_eeprom struct for backup case. When pressing S200 while reading EEPROM we ignore the value. We returned falsely a tricorder_eeprom struct wit

[U-Boot] [PATCH 2/3] board:tricorder: enable omap_gpio clocks

2014-04-10 Thread Andreas Bießmann
From: Andreas Bießmann Signed-off-by: Andreas Bießmann Cc: Thomas Weber Signed-off-by: Andreas Bießmann --- include/configs/tricorder.h |3 +++ 1 file changed, 3 insertions(+) diff --git a/include/configs/tricorder.h b/include/configs/tricorder.h index 62f97d2..a609861 100644 --- a/incl

[U-Boot] [PATCH 0/3] board:tricorder: late fixups for 2014.04

2014-04-10 Thread Andreas Bießmann
These fixups are required for tricorder board to run with 2014.04. One big change is a different ecc layout which came in by Pekon Gupta's changes to the omap_gpmc driver. Another point was changed behaviour of omap_gpio driver, we need to switch the clocks now explicitely. Thomas, could you ple

[U-Boot] [PATCH 3/3] board:tricorder: fixup SPL OOB layout

2014-04-10 Thread Andreas Bießmann
From: Andreas Bießmann Commit d016dc42cedbf6102e100fa9ecb58462edfb14f8 changed the layout of BCH8 SW on omap3 boards. We need to adopt the ecc layout for the nand_spl_simle driver to avoid wrong ecc errors. Signed-off-by: Andreas Bießmann Cc: Thomas Weber Signed-off-by: Andreas Bießmann ---

Re: [U-Boot] [PATCH v1 1/6] mtd: nand: omap_elm: remove #include omap_gpmc.h

2014-04-10 Thread Stefan Roese
On 10.04.2014 11:35, Pekon Gupta wrote: There is no dependency of omap_elm.c on omap_gpmc.h Signed-off-by: Pekon Gupta Reviewed-by: Stefan Roese Thanks, Stefan ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-b

Re: [U-Boot] [PATCH v1 2/6] mtd: nand: omap_elm: use bch_type instead of nibble count to differentiate between BCH4/BCH8/BCH16

2014-04-10 Thread Stefan Roese
On 10.04.2014 11:35, Pekon Gupta wrote: ELM hardware engine support ECC error detection for multiple ECC strengths like +--++ |Type | ECC syndrome length| +--++ |BCH4 | 6.5 bytes = 13 nibbles | |BCH8 | 13 byte = 26 nibbles

Re: [U-Boot] [PATCH v1 3/6] mtd: nand: omap_elm: use macros for register definitions

2014-04-10 Thread Stefan Roese
On 10.04.2014 11:35, Pekon Gupta wrote: This patch adds macros for following parameters of ELM Hardware engine - ELM_MAX_CHANNELS: ELM can process 8 data streams simultaneously - ELM_MAX_ERRORS: ELM can detect upto 16 ECC error when using BCH16 scheme Signed-off-by: Pekon Gupta Reviewed-b

Re: [U-Boot] [PATCH v1 4/6] mtd: nand: omap_gpmc: remove unused members of 'struct nand_bch_priv'

2014-04-10 Thread Stefan Roese
On 10.04.2014 11:35, Pekon Gupta wrote: This patch prepares to refactor 'struct nand_bch_priv' -> 'struct omap_nand_info' And thus performs following clean-ups: - remove nand_bch_priv.type: use nand_bch_priv.ecc_scheme instead - remove nand_bch_priv.mode: Signed-off-by: Pekon Gupta Revi

Re: [U-Boot] [PATCH v1 5/6] mtd: nand: omap_gpmc: rename struct nand_bch_priv to struct omap_nand_info

2014-04-10 Thread Stefan Roese
On 10.04.2014 11:35, Pekon Gupta wrote: This patch renames 'struct nand_bch_priv' which currently holds private data only for BCH ECC schemes, into 'struct omap_nand_info' so that same can be used for all ECC schemes Signed-off-by: Pekon Gupta --- drivers/mtd/nand/omap_gpmc.c | 58 ++

Re: [U-Boot] [PATCH v1 2/6] mtd: nand: omap_elm: use bch_type instead of nibble count to differentiate between BCH4/BCH8/BCH16

2014-04-10 Thread Gupta, Pekon
Hi Stefan, >From: Stefan Roese [mailto:s...@denx.de] >>On 10.04.2014 11:35, Pekon Gupta wrote: >> ELM hardware engine support ECC error detection for multiple ECC strengths >> like >> +--++ >> |Type | ECC syndrome length| >> +--++

Re: [U-Boot] [PATCH v1 6/6] mtd: nand: omap_gpmc: minor cleanup of omap_correct_data_bch

2014-04-10 Thread Stefan Roese
On 10.04.2014 11:35, Pekon Gupta wrote: This patch tries to avoid some local pointer dereferences, by using common local variables in omap_correct_data_bch() Signed-off-by: Pekon Gupta --- drivers/mtd/nand/omap_gpmc.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff -

Re: [U-Boot] [PATCH 2/2] dfu, nand: add medium specific polltimeout function

2014-04-10 Thread Marek Vasut
On Thursday, April 10, 2014 at 12:08:44 PM, Lukasz Majewski wrote: [...] > Seems reasonable for me: +1 > > Some comment: > > Guys, please be consistent with CCing people. I didn't receive this > thread. Also this original reply from Pantelis was not CCed to Heiko. For mutt, I have the key 'a'

Re: [U-Boot] [PATCH] Fix USB keyboard polling via control endpoint

2014-04-10 Thread Marek Vasut
On Thursday, April 10, 2014 at 12:15:30 PM, Wolfgang Denk wrote: > Dear Marek, > > In message <201404101012.42527.ma...@denx.de> you wrote: > > On Sunday, April 06, 2014 at 01:17:46 PM, Wolfgang Denk wrote: > > [...] > > > > > @@ -131,7 +133,9 @@ void usb_kbd_generic_poll(void) > > > > > > /*

[U-Boot] [PATCH v2] Add endian support macros to interrupt transfers in the EHCI driver.

2014-04-10 Thread Adrian Cox
Update the EHCI driver to support interrupt transfers on PowerPC. Signed-off-by: Adrian Cox --- Changes in v2: - Whitespace and line length fixes for checkstyle drivers/usb/host/ehci-hcd.c | 56 --- 1 file changed, 31 insertions(+), 25 deletions(-)

[U-Boot] [PATCH v3] Fix USB keyboard polling via control endpoint

2014-04-10 Thread Adrian Cox
USB keyboard polling failed for some keyboards on PowerPC 5020. This was caused by requesting only 4 bytes of data from keyboards that produce an 8 byte HID report. Signed-off-by: Adrian Cox Signed-off-by: Wolfgang Denk Cc: Marek Vasut --- Changes in v3: - Fixed checkstyle warnings - Rena

Re: [U-Boot] [PATCH 07/11] MX6: use macro building for MX6Q/MX6DL iomux regs

2014-04-10 Thread Nikita Kiryanov
On 04/09/2014 06:46 PM, Tim Harvey wrote: On Wed, Apr 9, 2014 at 7:57 AM, Nikita Kiryanov wrote: Hi Tim, On 04/03/2014 09:01 AM, Tim Harvey wrote: This is an attempt at using a macro to allow mx6dl-ddr.h and mx6q-ddr.h registers to be used together which is needed for an SPL bootloader that

Re: [U-Boot] [PATCH v2] Add endian support macros to interrupt transfers in the EHCI driver.

2014-04-10 Thread Marek Vasut
On Thursday, April 10, 2014 at 02:29:45 PM, Adrian Cox wrote: > Update the EHCI driver to support interrupt transfers on PowerPC. > > Signed-off-by: Adrian Cox Applied for -next, thanks! Best regards, Marek Vasut ___ U-Boot mailing list U-Boot@lists.d

Re: [U-Boot] [PATCH v3] Fix USB keyboard polling via control endpoint

2014-04-10 Thread Marek Vasut
On Thursday, April 10, 2014 at 03:02:44 PM, Adrian Cox wrote: > USB keyboard polling failed for some keyboards on PowerPC 5020. > This was caused by requesting only 4 bytes of data from keyboards that > produce an 8 byte HID report. > > Signed-off-by: Adrian Cox > Signed-off-by: Wolfgang Denk A

Re: [U-Boot] [PATCH 2/2] dfu, nand: add medium specific polltimeout function

2014-04-10 Thread Lukasz Majewski
Hi Heiko, > Hello Lukasz, > > Am 10.04.2014 12:08, schrieb Lukasz Majewski: > > Hi Pantelis, > > > >> Hi Marek, > >> > >> On Apr 10, 2014, at 10:54 AM, Marek Vasut wrote: > >> > >>> On Thursday, April 10, 2014 at 07:08:06 AM, Heiko Schocher wrote: > add a possibility to add a medium specific

Re: [U-Boot] [PATCH 04/11] MX6: add common SPL configuration

2014-04-10 Thread Nikita Kiryanov
On 04/09/2014 06:32 PM, Tim Harvey wrote: On Wed, Apr 9, 2014 at 7:55 AM, Nikita Kiryanov wrote: Hi Tim, On 04/03/2014 09:01 AM, Tim Harvey wrote:> Add a common header which can hopefully be shared among imx6 SPL users Signed-off-by: Tim Harvey --- include/configs/imx6_spl.h | 64

Re: [U-Boot] [PATCH 09/11] IMX: add additional function for pinmux using an array

2014-04-10 Thread Nikita Kiryanov
On 04/09/2014 06:40 PM, Tim Harvey wrote: On Wed, Apr 9, 2014 at 7:56 AM, Nikita Kiryanov wrote: Hi Tim, On 04/03/2014 09:01 AM, Tim Harvey wrote: Add new function that can take an array of iomux configs, an index, and a stride to allow a multi-dimentional array of pinmux values to be used

Re: [U-Boot] [PATCH 07/11] MX6: use macro building for MX6Q/MX6DL iomux regs

2014-04-10 Thread Nikita Kiryanov
On 04/10/2014 05:08 PM, Nikita Kiryanov wrote: On 04/09/2014 06:46 PM, Tim Harvey wrote: On Wed, Apr 9, 2014 at 7:57 AM, Nikita Kiryanov wrote: Hi Tim, On 04/03/2014 09:01 AM, Tim Harvey wrote: This is an attempt at using a macro to allow mx6dl-ddr.h and mx6q-ddr.h registers to be used tog

[U-Boot] board/freescale/mx6slevk: Add support for SPI NOR on the Freescale MCIMX6SLEVK Board

2014-04-10 Thread Sam Catch
I hope this email is now in the right format. Hi All, I am trying to add support for the SPI NOR device on the Freescale MCIMX6SLEVK Board. I have created a patch that should enable support but the u-boot probe command still does not find the device. The process fails during the ECSPI transfer

Re: [U-Boot] board/freescale/mx6slevk: Add support for SPI NOR on the Freescale MCIMX6SLEVK Board

2014-04-10 Thread Fabio Estevam
On Thu, Apr 10, 2014 at 12:05 PM, Sam Catch wrote: > I hope this email is now in the right format. > > Hi All, > > I am trying to add support for the SPI NOR device on the Freescale > MCIMX6SLEVK Board. I have created a patch that should enable support but > the u-boot probe command still does n

Re: [U-Boot] How can I save U-Boot env vars to HUSH shell vars.

2014-04-10 Thread James Chargin
Dear Wolfgang, With your help, I have satisfied my need. On 04/09/2014 11:52 PM, Wolfgang Denk wrote: Dear James, In message <5345d493.4080...@gmail.com> you wrote: Heh. As soon as you have to change _any_ code, you can as well upgrade. I was hoping (in vain perhaps) to do my work modify

[U-Boot] [PATCH RESEND-WITH-JUSTIFICATION] spi: soft_spi: Support NULL din/dout buffers

2014-04-10 Thread Andrew Ruder
This mirrors the conventions used in other SPI drivers (kirkwood, davinci, atmel, et al) where the din/dout buffer can be NULL when the received/transmitted data isn't important. This reduces the need for allocating additional buffers when write-only/read-only functionality is needed. In the din

Re: [U-Boot] How can I save U-Boot env vars to HUSH shell vars.

2014-04-10 Thread Wolfgang Denk
Dear James, In message <5346cdc2.40...@gmail.com> you wrote: > > With your help, I have satisfied my need. I'm happy that I was able to help. Thanks for the summary and your actual code. > And, I appreciate you help very much. You are welcome. Best regards, Wolfgang Denk -- DENX Software

Re: [U-Boot] [PATCH RESEND-WITH-JUSTIFICATION] spi: soft_spi: Support NULL din/dout buffers

2014-04-10 Thread Andrew Ruder
I further justify this with a list of drivers that depend on half-duplex SPI operation: drivers/mmc/mmc_spi.c: spi_xfer(spi, 2 * 8, tok, NULL, 0); drivers/mtd/spi/eeprom_m95xxx.c:if (spi_xfer(slave, 8, buf, NULL, SPI_XFER_BEGIN | SPI_XFER_END)) drivers/mtd/spi/sf_ops.c:

Re: [U-Boot] [PATCH RESEND-WITH-JUSTIFICATION] spi: soft_spi: Support NULL din/dout buffers

2014-04-10 Thread Jagan Teki
On Thu, Apr 10, 2014 at 9:49 PM, Andrew Ruder wrote: > This mirrors the conventions used in other SPI drivers (kirkwood, > davinci, atmel, et al) where the din/dout buffer can be NULL when the > received/transmitted data isn't important. This reduces the need for > allocating additional buffers w

Re: [U-Boot] [PATCH RESEND-WITH-JUSTIFICATION] spi: soft_spi: Support NULL din/dout buffers

2014-04-10 Thread Andrew Ruder
On Fri, Apr 11, 2014 at 12:24:20AM +0530, Jagan Teki wrote: > At-least from zynq_spi.c case - I wouldn't find that obscure case as you > pointed > and even with dout NULL which is status poll from (sf_ops.c). zynq_spi is correct, soft_spi is not. zynq_spi.c: int spi_xfer(struct spi_slave *slave,

Re: [U-Boot] [PATCH RESEND-WITH-JUSTIFICATION] spi: soft_spi: Support NULL din/dout buffers

2014-04-10 Thread Jagan Teki
On Fri, Apr 11, 2014 at 12:27 AM, Andrew Ruder wrote: > On Fri, Apr 11, 2014 at 12:24:20AM +0530, Jagan Teki wrote: >> At-least from zynq_spi.c case - I wouldn't find that obscure case as you >> pointed >> and even with dout NULL which is status poll from (sf_ops.c). > > zynq_spi is correct, soft

Re: [U-Boot] [PATCH RESEND-WITH-JUSTIFICATION] spi: soft_spi: Support NULL din/dout buffers

2014-04-10 Thread Andrew Ruder
On Fri, Apr 11, 2014 at 12:33:45AM +0530, Jagan Teki wrote: > >> It would be great if you mentioned issue scenario for status poll case > >> drivers/mtd/spi/sf_ops.c: ret = spi_xfer(spi, 8, NULL, > >> &status, 0); > OK - means issue only with soft_spi.c is it? Yes, and a couple other

[U-Boot] [PATCH v3 2/5] usb: handle NULL table in usb_gadget_get_string

2014-04-10 Thread Rob Herring
From: Rob Herring Allow a NULL table to be passed to usb_gadget_get_string for cases when a string table may not be populated. Signed-off-by: Rob Herring --- drivers/usb/gadget/usbstring.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/usb/gadget/usbstring.c b/drivers/usb/gadge

[U-Boot] [PATCH 5/5] beagle: fastboot support

2014-04-10 Thread Rob Herring
From: Rob Herring Signed-off-by: Rob Herring --- include/configs/omap3_beagle.h | 5 + 1 file changed, 5 insertions(+) diff --git a/include/configs/omap3_beagle.h b/include/configs/omap3_beagle.h index 0b57421..e070760 100644 --- a/include/configs/omap3_beagle.h +++ b/include/configs/omap3

[U-Boot] [PATCH v3 0/5] Android Fastboot support

2014-04-10 Thread Rob Herring
From: Rob Herring I'm reviving the Android Fastboot support after 2+ years since the last posting[1]. The previous postings had some questions about licensing and source of some code. I believe I've traced the history sufficiently that the copyrights and source information are complete and cor

[U-Boot] [PATCH v3 1/5] common: introduce maximum load size

2014-04-10 Thread Rob Herring
From: Rob Herring Various commands that load images have no checks that a loaded image does not exceed the available RAM space and will happily continue overwriting u-boot or other RAM that should not be touched. Also, some commands such as USB DFU or fastboot need to know the maximum buffer size

[U-Boot] [PATCH v3 5/5] arm: beagle: enable Android fastboot support

2014-04-10 Thread Rob Herring
From: Rob Herring Enable Android Fastboot support on omap3_beagle board. Signed-off-by: Rob Herring --- include/configs/omap3_beagle.h | 5 + 1 file changed, 5 insertions(+) diff --git a/include/configs/omap3_beagle.h b/include/configs/omap3_beagle.h index 0b57421..e070760 100644 --- a/in

[U-Boot] [PATCH v3 3/5] image: add support for Android's boot image format

2014-04-10 Thread Rob Herring
From: Sebastian Siewior This patch adds support for the Android boot-image format. The header file is from the Android project and got slightly alterted so the struct + its defines are not generic but have something like a namespace. The header file is from bootloader/legacy/include/boot/bootimg.

[U-Boot] [PATCH v3 4/5] usb/gadget: add the fastboot gadget

2014-04-10 Thread Rob Herring
From: Sebastian Siewior This patch contains an implementation of the fastboot protocol on the device side and a little of documentation. The gadget expects the new-style gadget framework. The gadget implements the getvar, reboot, download and reboot commands. What is missing is the flash handling

Re: [U-Boot] [PATCH 5/5] beagle: fastboot support

2014-04-10 Thread Rob Herring
On Thu, Apr 10, 2014 at 2:18 PM, Rob Herring wrote: > From: Rob Herring > > Signed-off-by: Rob Herring Oops. Ignore this one... Rob ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] config: enable CONFIG_API in distro config

2014-04-10 Thread Rob Herring
From: Rob Herring CONFIG_API is needed for u-boot apps such as grub2, so enable it for distro config. Cc: Dennis Gilmore Signed-off-by: Rob Herring --- include/config_distro_defaults.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/config_distro_defaults.h b/include/config_distro

Re: [U-Boot] [PATCH 04/11] MX6: add common SPL configuration

2014-04-10 Thread Scott Wood
On Thu, 2014-04-10 at 17:37 +0300, Nikita Kiryanov wrote: > On 04/09/2014 06:32 PM, Tim Harvey wrote: > > This presents another challenge for SPL NAND as currently my SPL is > > ~70K. There is a lot of unnecessary code in the mtd nand layer that > > I'm including because that layer includes support

[U-Boot] [PATCH 1/2] ARM: highbank: use config_distro_defaults.h

2014-04-10 Thread Rob Herring
From: Rob Herring Adapt highbank to use config_distro_defaults.h and remove the redundant defines. Signed-off-by: Rob Herring --- include/configs/highbank.h | 25 ++--- 1 file changed, 2 insertions(+), 23 deletions(-) diff --git a/include/configs/highbank.h b/include/confi

[U-Boot] [PATCH 2/2] ARM: highbank: use default prompt

2014-04-10 Thread Rob Herring
From: Rob Herring Since highbank is actually shared between Highbank and Midway platforms, remove the Highbank name from the prompt and use the default. Signed-off-by: Rob Herring --- include/configs/highbank.h | 1 - 1 file changed, 1 deletion(-) diff --git a/include/configs/highbank.h b/inc

[U-Boot] [PATCH 0/14] Split out code from the enormous main.c

2014-04-10 Thread Simon Glass
A previous series created a way of using if () instead of #ifdef for controlling feature inclusion in U-Boot. The primary target of that series was common/main.c which is full of #ifdefs. That work was put on hold while the kbuild work was in progress. Since kbuild is now complete, it is time to t

[U-Boot] [PATCH 02/14] Rename hush to cli_hush

2014-04-10 Thread Simon Glass
Hush is a command-line interpreter, so rename it to make that clearer. Signed-off-by: Simon Glass --- board/keymile/common/common.c | 2 +- board/keymile/common/ivm.c | 2 +- common/Makefile| 2 +- common/{hush.c => cli_hush.c} | 4 ++-- common/main.c | 2

[U-Boot] [PATCH 07/14] Move command line API into cli.c

2014-04-10 Thread Simon Glass
We now have a single entry point to the CLI, whether simple or hush. Put this in its own file. Signed-off-by: Simon Glass --- common/Makefile | 1 + common/cli.c| 106 common/main.c | 93 --

[U-Boot] [PATCH 03/14] move CLI prototypes to cli.h and add comments

2014-04-10 Thread Simon Glass
Move the CLI prototypes from common.h to cli.h as part of an effort to reduce the size of common.h. Signed-off-by: Simon Glass --- board/ait/cam_enc_4xx/cam_enc_4xx.c | 1 + board/amcc/yucca/cmd_yucca.c| 1 + board/eltec/elppc/misc.c| 1 + board/eltec/mhpc/mhpc.c

[U-Boot] [PATCH 04/14] Split out simple parser and readline into separate files

2014-04-10 Thread Simon Glass
It doesn't make sense to have the simple parser and the readline code all in main. Split them out into separate files. Signed-off-by: Simon Glass --- common/Makefile |9 +- common/cli_readline.c | 670 common/cli_simple.c | 338 co

[U-Boot] [PATCH 12/14] main: Hide the hush/simple details inside cli.c

2014-04-10 Thread Simon Glass
Move these details from main (which doesn't care which parser is used) to cli.c where they belong. Signed-off-by: Simon Glass --- common/cli.c| 22 ++ common/cli_simple.c | 2 +- common/main.c | 16 ++-- include/cli.h | 13 - 4 fi

[U-Boot] [PATCH 14/14] main: Avoid unncessary strdup()/free()

2014-04-10 Thread Simon Glass
It doesn't seem necessary to use memory allocation in this code. The setenv() will make a copy anyway. Signed-off-by: Simon Glass --- common/main.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/common/main.c b/common/main.c index b0d3ea4..ab1f708 100644 --- a/common/

[U-Boot] [PATCH 01/14] Remove unnecessary use of hush header file

2014-04-10 Thread Simon Glass
Some files include hush.h but don't actually use it. Remove this where possible. Signed-off-by: Simon Glass --- arch/arm/cpu/arm926ejs/kirkwood/cpu.c | 1 - arch/arm/cpu/arm926ejs/orion5x/cpu.c | 1 - board/ait/cam_enc_4xx/cam_enc_4xx.c | 1 - board/mcc200/auto_update.c| 7 +-

[U-Boot] [PATCH 11/14] Simplify the main loop

2014-04-10 Thread Simon Glass
The main loop is easier to follow if the code is grouped into separate functions. Make this change, so that main_loop() is easier to read. Signed-off-by: Simon Glass --- common/main.c | 47 ++- 1 file changed, 26 insertions(+), 21 deletions(-) diff -

[U-Boot] [PATCH 10/14] m68k: powerpc: Clean up do_mdm_init

2014-04-10 Thread Simon Glass
This code seems unnecessarily complex. We really just need to check the global_data. Now that is it all in one place, and not arch-specific, this is pretty easy. Signed-off-by: Simon Glass --- arch/m68k/lib/board.c| 7 --- arch/powerpc/lib/board.c | 8 common/board_r.c

[U-Boot] [PATCH 09/14] Rename bootretry functions and remove #ifdefs

2014-04-10 Thread Simon Glass
Add a bootretry_ prefix to these two functions, and remove the need for the #ifdef around everything (it moves to the Makefile). Signed-off-by: Simon Glass --- board/hymod/hymod.c | 4 +--- board/hymod/input.c | 8 ++-- common/autoboot.c | 6 +- common/bootretry.c | 4 ++-- commo

[U-Boot] [PATCH 13/14] main: Make the execution path a little clearer in main.c

2014-04-10 Thread Simon Glass
bootdelay_process() never returns in some circumstances, whichs makes the control flow confusing. Change it so that the decision about how to execute the boot command is made in the main_loop() code, so it is easier to follow. Move CLI stuff to cli.c. Signed-off-by: Simon Glass --- common/autob

[U-Boot] [PATCH 06/14] Move autoboot code to autoboot.c

2014-04-10 Thread Simon Glass
The autoboot code is complex and long. It deserves its own file with a simple interface from main.c. Signed-off-by: Simon Glass --- common/Makefile| 5 + common/autoboot.c | 363 common/main.c | 366 +--

[U-Boot] [PATCH 05/14] Add cli_ prefix to readline functions

2014-04-10 Thread Simon Glass
This makes it clear where the code resides. Signed-off-by: Simon Glass --- board/ait/cam_enc_4xx/cam_enc_4xx.c | 2 +- board/amcc/yucca/cmd_yucca.c| 20 ++-- board/eltec/elppc/misc.c| 14 +++--- board/eltec/mhpc/mhpc.c | 12 ++--

[U-Boot] [PATCH 08/14] Move bootretry code into bootretry.c and clean up

2014-04-10 Thread Simon Glass
This code is only used by one board, so it seems a shame to clutter up the readline code with it. Move it into its own file. Signed-off-by: Simon Glass --- board/hymod/hymod.c | 1 + board/hymod/input.c | 1 + common/Makefile | 5 + common/autoboot.c | 1 + common/bootretr

[U-Boot] [PATCH] arm, da850: staticize funtions

2014-04-10 Thread Manish Badarkhe
Make funtions static which are locally used in file and remove the declaration from header file. Signed-off-by: Manish Badarkhe --- :100644 100644 a38... b91e948... M arch/arm/cpu/arm926ejs/davinci/da850_lowlevel.c :100644 100644 7f9c3f6... 45a325c... M arch/arm/include/asm/arch-davinci/d

Re: [U-Boot] [PATCH 10/10][v2] board/t104xrdb: Add support of NAND, SD, SPI boot for T104xRDB

2014-04-10 Thread Rommel G Custodio
Dear Prabhakar Kushwaha, Prabhakar Kushwaha freescale.com> writes: > > Add support of 2 stage NAND, SD, SPI boot loader using SPL framework. > here, PBL initialise the internal SRAM and copy SPL(160KB). This further > initialise DDR using SPD and environment and copy u-boot(768 KB) from NAND t

Re: [U-Boot] [PATCH 2/2] dfu, nand: add medium specific polltimeout function

2014-04-10 Thread Heiko Schocher
Hello Lukasz, Am 10.04.2014 16:31, schrieb Lukasz Majewski: Hi Heiko, Hello Lukasz, Am 10.04.2014 12:08, schrieb Lukasz Majewski: Hi Pantelis, Hi Marek, On Apr 10, 2014, at 10:54 AM, Marek Vasut wrote: On Thursday, April 10, 2014 at 07:08:06 AM, Heiko Schocher wrote: add a possibility

[U-Boot] [PATCH] powerpc/t1040rdb: added a break in switch case

2014-04-10 Thread shh.xie
From: Shaohui Xie There should be a break for case PHY_INTERFACE_MODE_SGMII, otherwise it will fall into case PHY_INTERFACE_MODE_RGMII. Signed-off-by: Shaohui Xie --- board/freescale/t104xrdb/eth.c | 1 + 1 file changed, 1 insertion(+) diff --git a/board/freescale/t104xrdb/eth.c b/board/frees

Re: [U-Boot] [PATCH 07/11] MX6: use macro building for MX6Q/MX6DL iomux regs

2014-04-10 Thread Tim Harvey
On Thu, Apr 10, 2014 at 7:51 AM, Nikita Kiryanov wrote: > On 04/10/2014 05:08 PM, Nikita Kiryanov wrote: > > Here I'm referring to the file arch/arm/include/asm/arch-mx6/mx6_ddr_regs.h > (which I'm guessing you may have already seen based on your comment on > patch 11...) > Nikita, Yes, I start

[U-Boot] [PATCHv3 00/13] mmc: exynos: code cleanup and support DDR mode

2014-04-10 Thread Jaehoon Chung
If card and host are supported DDR mode, then it can be used the DDR mode. This patch-set has dependency about beomho's patch-set. (Based-on u-boot-samsung repository) It's result for loading image. sdhci controller ->5260488 bytes read in 259 ms (19.4 MiB/s) dwmmc controller without DDR mode ->

[U-Boot] [PATCHv3 03/13] board: trats2: Enable device tree on Trats2

2014-04-10 Thread Jaehoon Chung
From: Beomho Seo This patch add dwmmc emmc controller node on exynos4 and exynos4412 device tree. Signed-off-by: Beomho Seo Signed-off-by: Jaehoon Chung Tested-by: Piotr Wilczek Cc: Lukasz Majewski Cc: Piotr Wilczek Cc: Minkyu Kang --- arch/arm/dts/exynos4.dtsi |8 a

[U-Boot] [PATCHv3 01/13] arm: exynos: pinmux: add sdmmc4 gpio configratuion

2014-04-10 Thread Jaehoon Chung
From: Beomho Seo For use dwmmc controller at exynos4, add SDMMC4 gpio configuration. In case SDMMC2, do not use 8 bit mode at exynos4. Signed-off-by: Beomho Seo Signed-off-by: Jaehoon Chung Tested-by: Piotr Wilczek Cc: Lukasz Majewski Cc: Piotr Wilczek Cc: Minkyu Kang --- arch/arm/cpu/arm

[U-Boot] [PATCHv3 04/13] ARM: exynos: board: change the mmc/sd init sequence

2014-04-10 Thread Jaehoon Chung
Exynos4 can be used the dwmmc controller for eMMC. Then it needs to check dwmmc_init() at first. Signed-off-by: Jaehoon Chung Reviewed-by: Lukasz Majewski Tested-by: Lukasz Majewski Acked-by: Lukasz Majewski --- board/samsung/common/board.c | 13 ++--- 1 file changed, 6 insertions(+

[U-Boot] [PATCHv3 05/13] ARM: exynos: clock: modify the set_mmc_clk for exynos4

2014-04-10 Thread Jaehoon Chung
Modified the mmc_set_clock for eynos4. The goal of this patch is that fsys-div register should be reset. And retore the div-value, not using the value of lowlevel_init. (For using SDMMC4, this patch is needs) Signed-off-by: Jaehoon Chung Tested-by: Lukasz Majewski Acked-by: Lukasz Majewski ---

[U-Boot] [PATCHv3 09/13] mmc: dw_mmc: support the DDR mode

2014-04-10 Thread Jaehoon Chung
Support the DDR mode at dw-mmc controller Signed-off-by: Jaehoon Chung Tested-by: Lukasz Majewski Acked-by: Lukasz Majewski --- drivers/mmc/dw_mmc.c | 12 ++-- include/dwmmc.h |3 +++ 2 files changed, 13 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/dw_mmc.c b/driv

[U-Boot] [PATCHv3 10/13] ARM: dts: exnyos: enable dw-mmc controller

2014-04-10 Thread Jaehoon Chung
Enabled the dw-mmc controller. Signed-off-by: Jaehoon Chung Tested-by: Lukasz Majewski Acked-by: Lukasz Majewski --- arch/arm/dts/exynos4412-trats2.dts |6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/arch/arm/dts/exynos4412-trats2.dts b/arch/arm/dts/exynos4412-trat

[U-Boot] [PATCHv3 06/13] mmc: exynos_dw_mmc: restore the property into host

2014-04-10 Thread Jaehoon Chung
Restore the platdata(property of dt) into host struct. Then data's information is maintained and reused anywhere. Signed-off-by: Jaehoon Chung Tested-by: Lukasz Majewski Acked-by: Lukasz Majewski --- drivers/mmc/exynos_dw_mmc.c | 208 +++ include/dwmmc.

[U-Boot] [PATCHv3 11/13] mmc: exynos_dw_mmc: enable the DDR mode

2014-04-10 Thread Jaehoon Chung
Set the ddr mode capability by default. Signed-off-by: Jaehoon Chung Tested-by: Lukasz Majewski Acked-by: Lukasz Majewski --- drivers/mmc/exynos_dw_mmc.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mmc/exynos_dw_mmc.c b/drivers/mmc/exynos_dw_mmc.c index 131ff72..96b5ab6 1006

[U-Boot] [PATCHv3 07/13] mmc: remove the unnecessary define and fix the wrong bit control

2014-04-10 Thread Jaehoon Chung
Signed-off-by: Jaehoon Chung Reviewed-by: Lukasz Majeski Tested-by: Lukasz Majewski Acked-by: Lukasz Majewski --- drivers/mmc/mmc.c |2 +- include/mmc.h | 18 ++ 2 files changed, 7 insertions(+), 13 deletions(-) diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c inde

  1   2   >