Re: [U-Boot] [PATCH V2] usb: ci_udc: Allocate the qTD list directly

2014-07-01 Thread Jörg Krause
On 07/01/2014 02:04 AM, Marek Vasut wrote: Instead of weird allocation of ci_drv-items_mem and then even weirder distribution of offsets in this memory area into ci_drv-items array, just allocate ci_drv-items as a big slab of aligned memory (replaces ci_drv-items_mem) and let ci_get_qtd() do

[U-Boot] [PATCH] powerpc/t1040: fix i2c2 address on t1040qds and t104xrdb

2014-07-01 Thread Shengzhou Liu
The base address of I2C2 is 0x118100 instead of 0x119000. Signed-off-by: Shengzhou Liu shengzhou@freescale.com --- include/configs/T1040QDS.h | 2 +- include/configs/T104xRDB.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/configs/T1040QDS.h

Re: [U-Boot] [PATCH] powerpc/t1040: fix i2c2 address on t1040qds and t104xrdb

2014-07-01 Thread Priyanka Jain
Hello Shengzhou, T1040 has two dual I2C controller. First Dual I2C Controller : 0x118 (first i2c bus), 0x1181000(second I2C bus) Second Dual I2C Controller : 0x119 (third i2c bus), 0x1191000(fourth I2C bus) My understanding is CONFIG_SYS_FSL_I2C_OFFSET is offset for first I2C

[U-Boot] [PATCH RFC 3/3] arm:vf610-twr:Add fsl-spi-interface config and board info

2014-07-01 Thread Chao Fu
From: Chao Fu b44...@freescale.com Add spi device info for vf610-twr board. Enable fsl-spi-interface for compatibility of fsl-dspi and fsl-qspi. Signed-off-by: Chao Fu b44...@freescale.com --- board/freescale/vf610twr/vf610twr.c | 24 include/configs/vf610twr.h

[U-Boot] [PATCH RFC 1/3] spi:fsl_spi:Add compatibility for fsl-qspi and fsl-dspi modules driver

2014-07-01 Thread Chao Fu
From: Chao Fu b44...@freescale.com Freescale has some series of chips(e.g. vf610) contain two kinds of SPI modules, DSPI and QSPI. U-boot spi current code can't compile and enable the two modules at same time. So add fsl-spi-interface make two spi driver code work together. Usage:(e.g.)

[U-Boot] [PATCH RFC 2/3] arm:vf610:Add SPI bus num in SOC level for SPI compatibility

2014-07-01 Thread Chao Fu
From: Chao Fu b44...@freescale.com Add DSPI and QSPI bus definition in SOC level. Sf probe command parameter bus will decide which module will work. Add register base definition. Signed-off-by: Chao Fu b44...@freescale.com --- arch/arm/include/asm/arch-vf610/imx-regs.h | 2 ++ 1 file changed,

Re: [U-Boot] [PATCH V2] usb: ci_udc: Allocate the qTD list directly

2014-07-01 Thread Marek Vasut
On Tuesday, July 01, 2014 at 08:51:45 AM, Jörg Krause wrote: On 07/01/2014 02:04 AM, Marek Vasut wrote: Instead of weird allocation of ci_drv-items_mem and then even weirder distribution of offsets in this memory area into ci_drv-items array, just allocate ci_drv-items as a big slab of

Re: [U-Boot] [PATCH v4 0/3] mtd, ubi, ubifs: resync with Linux-3.14

2014-07-01 Thread Wolfgang Denk
Dear Scott, In message 1404169793.2435.204.ca...@snotra.buserror.net you wrote: (git) diff needs a reference to diff against. Maybe I missed some earlier comments about that, but how exactly should this be done here? When the code is synced, the corresponding Linux SHA1 or tag should be

Re: [U-Boot] [PATCH V2] usb: ci_udc: Allocate the qTD list directly

2014-07-01 Thread Jörg Krause
On 07/01/2014 12:17 PM, Marek Vasut wrote: On Tuesday, July 01, 2014 at 08:51:45 AM, Jörg Krause wrote: On 07/01/2014 02:04 AM, Marek Vasut wrote: Instead of weird allocation of ci_drv-items_mem and then even weirder distribution of offsets in this memory area into ci_drv-items array, just

Re: [U-Boot] [PATCH V2] usb: ci_udc: Allocate the qTD list directly

2014-07-01 Thread Marek Vasut
On Tuesday, July 01, 2014 at 01:03:25 PM, Jörg Krause wrote: On 07/01/2014 12:17 PM, Marek Vasut wrote: On Tuesday, July 01, 2014 at 08:51:45 AM, Jörg Krause wrote: On 07/01/2014 02:04 AM, Marek Vasut wrote: Instead of weird allocation of ci_drv-items_mem and then even weirder

Re: [U-Boot] [PATCH V2] usb: ci_udc: Allocate the qTD list directly

2014-07-01 Thread Jörg Krause
On 07/01/2014 01:19 PM, Marek Vasut wrote: [snip] Can you edit arch/arm/cpu/arm926ejs/cache.c and change the debug() to printf() , then re-test please ? I suspect this might trap something still. Ah, and please test on u-boot-usb/master with this patch. No additional output on the console.

Re: [U-Boot] [PATCH V2] usb: ci_udc: Allocate the qTD list directly

2014-07-01 Thread Marek Vasut
On Tuesday, July 01, 2014 at 01:22:41 PM, Jörg Krause wrote: On 07/01/2014 01:19 PM, Marek Vasut wrote: [snip] Can you edit arch/arm/cpu/arm926ejs/cache.c and change the debug() to printf() , then re-test please ? I suspect this might trap something still. Ah, and please test on

[U-Boot] [PATCH] README: document CONFIG_ENV_IS_IN_SPI_FLASH

2014-07-01 Thread Josh Wu
The option can be used to save the environment in spi flash. Implementation code is already exist in command/env_sf.c. But the documentation is missing. This patch add the details for this option to the README file. Signed-off-by: Josh Wu josh...@atmel.com --- README | 37

Re: [U-Boot] [PATCH V2] usb: ci_udc: Allocate the qTD list directly

2014-07-01 Thread Jörg Krause
On 07/01/2014 01:35 PM, Marek Vasut wrote: On Tuesday, July 01, 2014 at 01:22:41 PM, Jörg Krause wrote: On 07/01/2014 01:19 PM, Marek Vasut wrote: [snip] Can you edit arch/arm/cpu/arm926ejs/cache.c and change the debug() to printf() , then re-test please ? I suspect this might trap

[U-Boot] SPL broken on i.mx31 platforms

2014-07-01 Thread Helmut Raiger
Hi, the commit 41623c91 breaks the SPL on i.mx31 platforms. The original startup code (start.S) was position independent to allow relocation in board_init_f. This is necessary as the internal RAM used by the IPL to load the first 2kB from NAND is also used by the NAND controller to buffer

Re: [U-Boot] Please pull u-boot-ti/master

2014-07-01 Thread Albert ARIBAUD
Hi Tom, On Thu, 19 Jun 2014 18:03:25 -0400, Tom Rini tr...@ti.com wrote: Hey, The following changes since commit 0a26e1d6c394aacbf1153977b7348d1dff85db3f: arm: fix a double-definition error of _start symbol (2014-06-09 10:36:40 +0200) are available in the git repository at:

Re: [U-Boot] SPL broken on i.mx31 platforms

2014-07-01 Thread Fabio Estevam
Hi Helmut, On Tue, Jul 1, 2014 at 10:33 AM, Helmut Raiger helmut.rai...@hale.at wrote: Hi, the commit 41623c91 breaks the SPL on i.mx31 platforms. The original startup code (start.S) was position independent to allow relocation in board_init_f. This is necessary as the internal RAM used

Re: [U-Boot] [PATCH v3 06/11] arm:reset: call the reset_misc() before the cpu reset

2014-07-01 Thread Przemyslaw Marczak
Hello Jeroen, On 06/30/2014 08:30 PM, Jeroen Hofstee wrote: Hello Przemyslaw. [...] #include common.h +void __reset_misc(void) {} + +void reset_misc(void) +__attribute((weak, alias(__reset_misc))); + can you please use __weak here and provide a prototype, wherever it ends up in the

Re: [U-Boot] [PATCH v3 06/11] arm:reset: call the reset_misc() before the cpu reset

2014-07-01 Thread Przemyslaw Marczak
Hello Minkyu, On 06/27/2014 01:34 PM, Przemyslaw Marczak wrote: On 06/27/2014 11:40 AM, Minkyu Kang wrote: Dear Przemyslaw Marczak, On 26/06/14 23:15, Przemyslaw Marczak wrote: On an Odroid U3 board, the SOC is unable to reset the eMMC card in the DWMMC mode by the cpu software reset. Manual

Re: [U-Boot] [PATCH V2] usb: ci_udc: Allocate the qTD list directly

2014-07-01 Thread Stephen Warren
On 06/30/2014 06:04 PM, Marek Vasut wrote: Instead of weird allocation of ci_drv-items_mem and then even weirder distribution of offsets in this memory area into ci_drv-items array, just allocate ci_drv-items as a big slab of aligned memory (replaces ci_drv-items_mem) and let ci_get_qtd() do

Re: [U-Boot] [PATCH V2] usb: ci_udc: Allocate the qTD list directly

2014-07-01 Thread Marek Vasut
On Tuesday, July 01, 2014 at 05:03:17 PM, Stephen Warren wrote: On 06/30/2014 06:04 PM, Marek Vasut wrote: Instead of weird allocation of ci_drv-items_mem and then even weirder distribution of offsets in this memory area into ci_drv-items array, just allocate ci_drv-items as a big slab of

Re: [U-Boot] [PATCH V2] usb: ci_udc: Allocate the qTD list directly

2014-07-01 Thread Stephen Warren
On 07/01/2014 09:13 AM, Marek Vasut wrote: On Tuesday, July 01, 2014 at 05:03:17 PM, Stephen Warren wrote: On 06/30/2014 06:04 PM, Marek Vasut wrote: Instead of weird allocation of ci_drv-items_mem and then even weirder distribution of offsets in this memory area into ci_drv-items array, just

Re: [U-Boot] [PATCH V2] usb: ci_udc: Allocate the qTD list directly

2014-07-01 Thread Marek Vasut
On Tuesday, July 01, 2014 at 05:16:26 PM, Stephen Warren wrote: On 07/01/2014 09:13 AM, Marek Vasut wrote: On Tuesday, July 01, 2014 at 05:03:17 PM, Stephen Warren wrote: On 06/30/2014 06:04 PM, Marek Vasut wrote: Instead of weird allocation of ci_drv-items_mem and then even weirder

Re: [U-Boot] pull request for u-boot-tegra/master into ARM/master

2014-07-01 Thread Albert ARIBAUD
Hi Tom, On Thu, 19 Jun 2014 10:53:11 -0700, Tom Warren twarren.nvi...@gmail.com wrote: Albert, Please pull u-boot-tegra/master into ARM/master. Thanks! ./MAKEALL -s tegra AOK, checkpatch.pl is OK, and ./MAKEALL -a arm only shows failures that were already present in ARM/master. The

[U-Boot] [PATCH 6/6] usb: ci_udc: don't memalign() struct ci_req allocations

2014-07-01 Thread Stephen Warren
From: Stephen Warren swar...@nvidia.com struct ci_req is a purely software structure, and needs no specific memory alignment. Hence, allocate it with calloc() rather than memalign(). The use of memalign() was left-over from when struct ci_req was going to hold the aligned bounce buffer, but this

[U-Boot] [PATCH 5/6] usb: ci_udc: remove controller.items array

2014-07-01 Thread Stephen Warren
From: Stephen Warren swar...@nvidia.com There's no need to store an array of QTD pointers in the controller. Since the calculation is so simple, just have ci_get_qtd() perform it at run-time, rather than pre-calculating everything. Signed-off-by: Stephen Warren swar...@nvidia.com ---

[U-Boot] [PATCH 0/6] usb: ci_udc: fixes and cleanups

2014-07-01 Thread Stephen Warren
From: Stephen Warren swar...@nvidia.com This is a series of small fixes and cleanups either required by those fixes, or enabled now that the fixes are made. I hope that either patch 1 or 4 might fix the issues Jörg is seeing, but I'm not sure that will happen. The other patches shouldn't change

[U-Boot] [PATCH 2/6] usb: ci_udc: don't assume QTDs are adjacent when transmitting ZLPs

2014-07-01 Thread Stephen Warren
From: Stephen Warren swar...@nvidia.com Fix ci_ep_submit_next_request()'s ZLP transmission code to explicitly call ci_get_qtd() to find the address of the other QTD to use. This will allow us to correctly align each QTD individually in the future, which may involve leaving a gap between the QTDs.

[U-Boot] [PATCH 4/6] usb: ci_udc: fix items array size/stride calculation

2014-07-01 Thread Stephen Warren
From: Stephen Warren swar...@nvidia.com 2 QTDs are allocated for each EP. The current allocation scheme aligns the first QTD in each pair, but simply adds the struct size to calculate the second QTD's address. This will result in a non-cache-aligned addresss IF the system's ARCH_DMA_MINALIGN is

[U-Boot] [PATCH 1/6] usb: ci_udc: fix ci_flush_{qh, qtd} calls in ci_udc_probe()

2014-07-01 Thread Stephen Warren
From: Stephen Warren swar...@nvidia.com ci_udc_probe() initializes a pair of QHs and QTDs for each EP. After each pair has been initialized, the pair is cache-flushed. The conversion from QH/QTD index [0..2*NUM_END_POINTS) to EP index [0..NUM_ENDPOINTS] is incorrect; it simply subtracts 1 (which

[U-Boot] [PATCH 3/6] usb: ci_udc: lift ilist size calculations to global scope

2014-07-01 Thread Stephen Warren
From: Stephen Warren swar...@nvidia.com This will allow functions other than ci_udc_probe() to make use of the constants in a future change. This in turn requires converting the const int variables to #defines, since the initialization of one global const int can't depend on the value of another

Re: [U-Boot] [PATCH] mmc: sdhci: Fixed timeout for sdhci_send_command()

2014-07-01 Thread Andy Fleming
On Fri, Jun 27, 2014 at 4:37 AM, Pantelis Antoniou pantelis.anton...@gmail.com wrote: Hi Eli, On Jun 12, 2014, at 12:41 PM, Eli Billauer wrote: The current wait loop just reads the status 1 times, which makes the actual timeout period platform-dependent. The udelay() call within the

Re: [U-Boot] [PATCH] ARM: tegra: Add support for norrin board

2014-07-01 Thread Allen Martin
On Mon, Jun 30, 2014 at 02:32:20PM -0700, Stephen Warren wrote: On 06/30/2014 02:53 PM, Allen Martin wrote: Norrin (PM370) is a Tegra124 clamshell board that is very similar to venice2, but it has a different panel, the sdcard cd sense is flipped, and it has a different revision of the

[U-Boot] [PATCH 3/3] dfu: fix readback buffer overflow test

2014-07-01 Thread Stephen Warren
From: Stephen Warren swar...@nvidia.com The buffer is too small if it's size to read, not if it's = the size. This fixes the 1MB test case on Tegra, which has a 1MB buffer. Signed-off-by: Stephen Warren swar...@nvidia.com --- drivers/dfu/dfu.c | 2 +- 1 file changed, 1 insertion(+), 1

[U-Boot] [PATCH 2/3] test: dfu: add some more test cases

2014-07-01 Thread Stephen Warren
From: Stephen Warren swar...@nvidia.com On Tegra, the DFU buffer size is 1M. Consequently, the 8M test always fails. Add tests for the 1M size, and one byte less as a corner case, so that some large tests are executed and expected to pass. Signed-off-by: Stephen Warren swar...@nvidia.com ---

[U-Boot] [PATCH 1/3] test: dfu: cleanup before execution

2014-07-01 Thread Stephen Warren
From: Stephen Warren swar...@nvidia.com Call cleanup() before running tests too. If a previous test was CTRL-C'd some stale files may have been left around. dfu-util refuses to receive a file to a filename that already exists, which results in false test failures if the files aren't cleaned up

Re: [U-Boot] please pull u-boot-samsung master

2014-07-01 Thread Albert ARIBAUD
Hi Minkyu, On Mon, 30 Jun 2014 17:42:38 +0900, Minkyu Kang mk7.k...@samsung.com wrote: Dear Albert, The following changes since commit 0a26e1d6c394aacbf1153977b7348d1dff85db3f: arm: fix a double-definition error of _start symbol (2014-06-09 10:36:40 +0200) are available in the git

Re: [U-Boot] SPL broken on i.mx31 platforms

2014-07-01 Thread Albert ARIBAUD
Hi Fabio, On Tue, 1 Jul 2014 10:53:57 -0300, Fabio Estevam feste...@gmail.com wrote: Hi Helmut, On Tue, Jul 1, 2014 at 10:33 AM, Helmut Raiger helmut.rai...@hale.at wrote: Hi, the commit 41623c91 breaks the SPL on i.mx31 platforms. The original startup code (start.S) was position

Re: [U-Boot] [PATCH 1/4] kirkwood: Rename dns325 to dnskw

2014-07-01 Thread Albert ARIBAUD
Hi Bastien, On Sun, 8 Jun 2014 18:30:25 +0200, Bastien ROUCARIÈS roucaries.bast...@gmail.com wrote: From: Jamie Lentin j...@lentin.co.uk So we can re-use DNS-325 configuration for the DNS-320 without things getting confusing, rename all common parts from dns325 to dnskw, and use a config

Re: [U-Boot] [PATCH 1/4] kirkwood: Rename dns325 to dnskw

2014-07-01 Thread Bastien ROUCARIES
Le 1 juil. 2014 22:03, Albert ARIBAUD albert.u.b...@aribaud.net a écrit : Hi Bastien, On Sun, 8 Jun 2014 18:30:25 +0200, Bastien ROUCARIÈS roucaries.bast...@gmail.com wrote: From: Jamie Lentin j...@lentin.co.uk So we can re-use DNS-325 configuration for the DNS-320 without things

[U-Boot] [PATCH] usb: ci_udc: use var name ep/ci_ep consistently

2014-07-01 Thread Stephen Warren
From: Stephen Warren swar...@nvidia.com Almost all of ci_udc.c uses variable name ep for a struct usb_ep and ci_ep for a struct ci_ep. This is nice and consistent, and helps people know what type a variable is without searching for the declaration. handle_ep_complete() doesn't do this, so fix it

Re: [U-Boot] [PATCH 0/6] usb: ci_udc: fixes and cleanups

2014-07-01 Thread Jörg Krause
On 07/01/2014 07:41 PM, Stephen Warren wrote: From: Stephen Warren swar...@nvidia.com This is a series of small fixes and cleanups either required by those fixes, or enabled now that the fixes are made. I hope that either patch 1 or 4 might fix the issues Jörg is seeing, but I'm not sure that

Re: [U-Boot] [PATCH 0/6] usb: ci_udc: fixes and cleanups

2014-07-01 Thread Stephen Warren
On 07/01/2014 03:25 PM, Jörg Krause wrote: On 07/01/2014 07:41 PM, Stephen Warren wrote: From: Stephen Warren swar...@nvidia.com This is a series of small fixes and cleanups either required by those fixes, or enabled now that the fixes are made. I hope that either patch 1 or 4 might fix

Re: [U-Boot] [PATCH 0/6] usb: ci_udc: fixes and cleanups

2014-07-01 Thread Stephen Warren
On 07/01/2014 03:31 PM, Stephen Warren wrote: On 07/01/2014 03:25 PM, Jörg Krause wrote: On 07/01/2014 07:41 PM, Stephen Warren wrote: From: Stephen Warren swar...@nvidia.com This is a series of small fixes and cleanups either required by those fixes, or enabled now that the fixes are made.

Re: [U-Boot] [PATCH 0/6] usb: ci_udc: fixes and cleanups

2014-07-01 Thread Jörg Krause
On 07/01/2014 11:31 PM, Stephen Warren wrote: On 07/01/2014 03:25 PM, Jörg Krause wrote: On 07/01/2014 07:41 PM, Stephen Warren wrote: From: Stephen Warren swar...@nvidia.com This is a series of small fixes and cleanups either required by those fixes, or enabled now that the fixes are made.

Re: [U-Boot] [PATCH V2] usb: ci_udc: Allocate the qTD list directly

2014-07-01 Thread Jörg Krause
On 07/01/2014 01:22 PM, Jörg Krause wrote: On 07/01/2014 01:19 PM, Marek Vasut wrote: [snip] Can you edit arch/arm/cpu/arm926ejs/cache.c and change the debug() to printf() , then re-test please ? I suspect this might trap something still. Ah, and please test on u-boot-usb/master with this

Re: [U-Boot] [PATCH V2] usb: ci_udc: Allocate the qTD list directly

2014-07-01 Thread Stephen Warren
On 07/01/2014 04:34 PM, Jörg Krause wrote: On 07/01/2014 01:22 PM, Jörg Krause wrote: On 07/01/2014 01:19 PM, Marek Vasut wrote: [snip] Can you edit arch/arm/cpu/arm926ejs/cache.c and change the debug() to printf() , then re-test please ? I suspect this might trap something still. Ah, and

Re: [U-Boot] [PATCH 0/6] usb: ci_udc: fixes and cleanups

2014-07-01 Thread Jörg Krause
On 07/01/2014 11:36 PM, Stephen Warren wrote: [snip] Can you please try one more thing: Go back to u-boot-usb/master plus just your board support patches. Apply the following and test that: diff --git a/drivers/usb/gadget/ci_udc.c b/drivers/usb/gadget/ci_udc.c index

Re: [U-Boot] [PATCH V2] usb: ci_udc: Allocate the qTD list directly

2014-07-01 Thread Jörg Krause
On 07/02/2014 12:36 AM, Stephen Warren wrote: On 07/01/2014 04:34 PM, Jörg Krause wrote: On 07/01/2014 01:22 PM, Jörg Krause wrote: On 07/01/2014 01:19 PM, Marek Vasut wrote: [snip] Can you edit arch/arm/cpu/arm926ejs/cache.c and change the debug() to printf() , then re-test please ? I

Re: [U-Boot] [PATCH 0/6] usb: ci_udc: fixes and cleanups

2014-07-01 Thread Marek Vasut
On Wednesday, July 02, 2014 at 12:42:40 AM, Jörg Krause wrote: On 07/01/2014 11:36 PM, Stephen Warren wrote: [snip] Can you please try one more thing: Go back to u-boot-usb/master plus just your board support patches. Apply the following and test that: diff --git

Re: [U-Boot] [PATCH V2] usb: ci_udc: Allocate the qTD list directly

2014-07-01 Thread Stephen Warren
On 07/01/2014 04:47 PM, Jörg Krause wrote: On 07/02/2014 12:36 AM, Stephen Warren wrote: On 07/01/2014 04:34 PM, Jörg Krause wrote: On 07/01/2014 01:22 PM, Jörg Krause wrote: On 07/01/2014 01:19 PM, Marek Vasut wrote: [snip] Can you edit arch/arm/cpu/arm926ejs/cache.c and change the

Re: [U-Boot] [PATCH 0/6] usb: ci_udc: fixes and cleanups

2014-07-01 Thread Stephen Warren
On 07/01/2014 04:42 PM, Jörg Krause wrote: On 07/01/2014 11:36 PM, Stephen Warren wrote: [snip] Can you please try one more thing: Go back to u-boot-usb/master plus just your board support patches. Apply the following and test that: diff --git a/drivers/usb/gadget/ci_udc.c

Re: [U-Boot] [PATCH V2] usb: ci_udc: Allocate the qTD list directly

2014-07-01 Thread Jörg Krause
On 07/02/2014 12:51 AM, Stephen Warren wrote: [...] Loading: ## 4.3 MiB/s done Bytes transferred = 18003 (4653 hex) CACHE: Misaligned operation at range [40008000, 4000c653] OK, that particular error happens well after the network transfer phase of the tftp

[U-Boot] [PATCH] USB: gadget: atmel: zero out allocated requests

2014-07-01 Thread Stephen Warren
From: Stephen Warren swar...@nvidia.com A UDC's alloc_request method should zero out the newly allocated request. Ensure the Atmel driver does so. This issue was found by code inspection, following the investigation of an intermittent issue with ci_udc, which was tracked down to failing to zero

Re: [U-Boot] [PATCH V2] usb: ci_udc: Allocate the qTD list directly

2014-07-01 Thread Stephen Warren
On 07/01/2014 04:57 PM, Jörg Krause wrote: On 07/02/2014 12:51 AM, Stephen Warren wrote: [...] Loading: ## 4.3 MiB/s done Bytes transferred = 18003 (4653 hex) CACHE: Misaligned operation at range [40008000, 4000c653] OK, that particular error happens well

Re: [U-Boot] [PATCH v1 20/25] arm: kirkwood: Remove some dead code from cpu.c

2014-07-01 Thread Luka Perkov
On Fri, Jun 27, 2014 at 11:55:06AM +0200, Stefan Roese wrote: All those functions removed with this patch are not accessed at all. So lets remove them. Signed-off-by: Stefan Roese s...@denx.de --- arch/arm/cpu/arm926ejs/kirkwood/cpu.c | 55 --- 1 file

Re: [U-Boot] [PATCH] powerpc/t1040: fix i2c2 address on t1040qds and t104xrdb

2014-07-01 Thread York Sun
Shengzhou, If u-boot uses 2nd I2C controller, it is good to have some comments. Please complete with four I2C controllers' offset. York On 07/01/2014 12:37 AM, Jain Priyanka-B32167 wrote: Hello Shengzhou, T1040 has two dual I2C controller. First Dual I2C Controller : 0x118 (first i2c

[U-Boot] [PATCH] kirkwood: cosmetic: style fixes in kwbimage.cfg files

2014-07-01 Thread Luka Perkov
When diffing through the various kwbimage.cfg files only show relevant changes. Signed-off-by: Luka Perkov l...@openwrt.org CC: Prafulla Wadaskar prafu...@marvell.com CC: Stefan Roese s...@denx.de --- board/iomega/iconnect/kwbimage.cfg | 4 ++-- board/raidsonic/ib62x0/kwbimage.cfg | 22

[U-Boot] [PATCH] ARM: kirkwood: fix cpu info for 6282 device id

2014-07-01 Thread Luka Perkov
Signed-off-by: Luka Perkov l...@openwrt.org CC: Prafulla Wadaskar prafu...@marvell.com CC: Stefan Roese s...@denx.de --- arch/arm/cpu/arm926ejs/kirkwood/cpu.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/arch/arm/cpu/arm926ejs/kirkwood/cpu.c

[U-Boot] [PATCH] kirkwood: define empty CONFIG_MVGBE_PORTS by default

2014-07-01 Thread Luka Perkov
Each board with defines it's own set of values. If we do not define CONFIG_MVGBE_PORTS we will hit following error: mvgbe.c: In function 'mvgbe_initialize': mvgbe.c:700:34: error: 'CONFIG_MVGBE_PORTS' undeclared (first use in this function) u8 used_ports[MAX_MVGBE_DEVS] = CONFIG_MVGBE_PORTS;

[U-Boot] [PATCH] kirkwood: ib62x0: add CONFIG_SYS_GENERIC_BOARD define

2014-07-01 Thread Luka Perkov
Signed-off-by: Luka Perkov l...@openwrt.org CC: Prafulla Wadaskar prafu...@marvell.com CC: Stefan Roese s...@denx.de --- include/configs/ib62x0.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/configs/ib62x0.h b/include/configs/ib62x0.h index 186fd35..cd38577 100644 ---

Re: [U-Boot] [PATCH] USB: gadget: atmel: zero out allocated requests

2014-07-01 Thread Bo Shen
Hi Stephen Warren, On 07/02/2014 06:59 AM, Stephen Warren wrote: From: Stephen Warren swar...@nvidia.com A UDC's alloc_request method should zero out the newly allocated request. Ensure the Atmel driver does so. This issue was found by code inspection, following the investigation of an

Re: [U-Boot] [PATCH v3] ARM: tegra: Disable VPR

2014-07-01 Thread Alexandre Courbot
Tom, is this patch ok for you? If yes, do you plan to merge it? On Sat, Jun 28, 2014 at 1:21 AM, Thierry Reding thierry.red...@gmail.com wrote: On Tue, Jun 24, 2014 at 11:45:29AM +0900, Alexandre Courbot wrote: From: Bryan Wu pe...@nvidia.com On Tegra114 and Tegra124 platforms, certain