Re: [U-Boot] [PATCH 4/6] travis-ci: use buildman -P everywhere

2016-10-26 Thread Heiko Schocher
Hello Stephen, Am 26.10.2016 um 19:05 schrieb Stephen Warren: From: Stephen Warren This places build results into a board-specific directory rather than a buildman-thread-specific directory. This is required so that we can access the directory from test.py, and there's no

Re: [U-Boot] [PATCH 3/6] travis-ci: centralize ~/.buildman editing

2016-10-26 Thread Heiko Schocher
Hello Stephen, Am 26.10.2016 um 19:05 schrieb Stephen Warren: From: Stephen Warren Any time an x86 toolchain is used, we need to edit ~/.buildman to reference it. Move the editing logic into a central place so that it doesn't have to be duplicated everywhere that uses the

Re: [U-Boot] [PATCH 2/6] travis-ci: use correct exit code on errors

2016-10-26 Thread Heiko Schocher
Hello Stephen, Am 26.10.2016 um 19:05 schrieb Stephen Warren: From: Stephen Warren The phrase "if [ $? -ne 0 ]; then exit $?; fi" doesn't work correctly; by the time the "exit" statement runs, $? has already been over-written by the result of the [ command. Fix this by

Re: [U-Boot] [PATCH 1/6] travis-ci: Use = not : when writing ~/.buildman

2016-10-26 Thread Heiko Schocher
Hello Stephen, Am 26.10.2016 um 19:05 schrieb Stephen Warren: From: Stephen Warren Travis CI seems to be confused when there's a colon in an echo command, and this is currently worked around using a variable that contains the text we want to echo. Use = syntax instead so

Re: [U-Boot] Unable to decompress files greater than 500 MB

2016-10-26 Thread Heiko Schocher
Hello Zakir Ahmed, Am 26.10.2016 um 12:22 schrieb Zakir Ahmed: Hi, We are using u-boot source code which has the support for most of the compressions like bz2, lzma, gz etc. We tried uncompressing bz2, lzma and gz images and most of them either fail or data abort or hang indefinitely for

Re: [U-Boot] [PATCH 3/3] power: regulator: Add limits checking while setting current

2016-10-26 Thread Keerthy
On Wednesday 26 October 2016 10:01 PM, Simon Glass wrote: Hi Keerthy, On 26 October 2016 at 01:12, Keerthy wrote: Currently the specific set ops functions are directly called without any check for min/max current limits for a regulator. Check for them and proceed.

Re: [U-Boot] [PATCH 0/6] Add ARMv8 PSCI framework

2016-10-26 Thread Hongbo Zhang
York, Yes I am going to send an update, one patch needs to be reordered as I've mentioned. I didn't get other feedback. But one thing I'm wondering is how to introduce the CONFIG_ options well, Tom's concerns make sense, and I had my explain too, we need a final solution before I send out a v2.

Re: [U-Boot] [PATCH 5/5] configs: add the CONFIG_BLK and CONFIG_DM_MMC/OPS for exynos4 series

2016-10-26 Thread Jaehoon Chung
On 10/27/2016 01:31 AM, Simon Glass wrote: > On 23 October 2016 at 23:08, Jaehoon Chung wrote: >> To use the CONFIG_BLK/CONFIG_DM_MMC/CONFIG_DM_MMC_OPS, add the >> configurations for exynos4 series.(by default) >> >> Signed-off-by: Jaehoon Chung >>

Re: [U-Boot] [PATCH 3/5] arm: dts: exynos4: change the compatible to meaning name

2016-10-26 Thread Jaehoon Chung
On 10/27/2016 01:31 AM, Simon Glass wrote: > On 23 October 2016 at 23:08, Jaehoon Chung wrote: >> Changed the compatible from generic thing to meaning compatible. >> "samsung,exynos-mmc" should be removed. >> >> Signed-off-by: Jaehoon Chung >> ---

Re: [U-Boot] [PATCH 4/5] arm: dts: exynos4: change to use their parent node

2016-10-26 Thread Jaehoon Chung
On 10/27/2016 01:31 AM, Simon Glass wrote: > On 23 October 2016 at 23:08, Jaehoon Chung wrote: >> Didn't overwrite the node. Just reuse the parent node. >> And add the Exynos Series specific properties. >> >> Signed-off-by: Jaehoon Chung >> --- >>

Re: [U-Boot] [PATCH 2/5] arm: dts: exynos4x12: remove the dupulicated node

2016-10-26 Thread Jaehoon Chung
On 10/27/2016 01:31 AM, Simon Glass wrote: > On 23 October 2016 at 23:08, Jaehoon Chung wrote: >> mshc_0 node should be overwriten the defined node in exynos4.dtsi. > > I think you mean 'The mshc_0 node overwrites the defined node...' > >> It's meaningless. So removed

Re: [U-Boot] [PATCH 1/5] arm: dts: exynos4x12: fix the minor things

2016-10-26 Thread Jaehoon Chung
On 10/25/2016 08:04 AM, Tom Rini wrote: > On Mon, Oct 24, 2016 at 03:08:53PM +0900, Jaehoon Chung wrote: > >> Fixed the minor things. >> 1. Removed '`' thing in comments >> 2. Fix the wrong interrupts value from 77 to 131 >> >> Signed-off-by: Jaehoon Chung >> --- >>

[U-Boot] [PATCH] nand: force BBT scan for controllers which need it

2016-10-26 Thread Stefan Agner
From: Stefan Agner Commit 35c204d8a9d0 ("nand: reinstate lazy bad block scanning") introduced lazy BBT scanning. However, some controller do parts of the initialization (mxs_nand.c) during scan_bbt, hence for those controllers the BBT must be scanned at initialization

Re: [U-Boot] [PATCH v3 09/10] sun5i: add defines used by the PSCI code

2016-10-26 Thread Maxime Ripard
Hi, On Wed, Oct 26, 2016 at 02:10:32PM +0200, Antoine Tenart wrote: > The sun5i SoCs can take advantage of the newly introduce PSCI suspend > function. Add defines used by the PSCI code. > > Signed-off-by: Antoine Tenart > --- > include/configs/sun5i.h | 3

Re: [U-Boot] [PATCH v3 08/10] sun5/7i: add an implementation of the psci suspend function

2016-10-26 Thread Maxime Ripard
Hi, On Wed, Oct 26, 2016 at 02:10:31PM +0200, Antoine Tenart wrote: > Add the suspend psci function for sun5i and sun7i. Thus function > switches the cpu clk source to osc24M or to losc depending on the > SoC family. > > Signed-off-by: Antoine Tenart > --- >

Re: [U-Boot] [PATCH 1/2] mkimage: Fix missing free() in show_valid_options()

2016-10-26 Thread Tom Rini
On Wed, Oct 26, 2016 at 01:19:11PM -0600, Simon Glass wrote: > The allocated memory should be freed. Fix it. > > Signed-off-by: Simon Glass > Reported-by: Coverity (CID: 150963) Reviewed-by: Tom Rini -- Tom signature.asc Description: Digital

Re: [U-Boot] [PATCH 2/2] image: Protect against overflow in unknown_msg()

2016-10-26 Thread Tom Rini
On Wed, Oct 26, 2016 at 01:19:12PM -0600, Simon Glass wrote: > Coverity complains that this can overflow. If we later increase the size > of one of the strings in the table, it could happen. > > Adjust the code to protect against this. > > Signed-off-by: Simon Glass >

Re: [U-Boot] [Resend RFC PATCH 1/2] armv8: Fix dcache disable function

2016-10-26 Thread york sun
On 10/26/2016 02:00 PM, Stephen Warren wrote: > On 10/26/2016 02:29 PM, york sun wrote: >> On 10/26/2016 01:12 PM, Stephen Warren wrote: >>> On 10/26/2016 01:54 PM, york sun wrote: On 10/26/2016 12:47 PM, Stephen Warren wrote: > > There are two data structures in ARM U-Boot that

Re: [U-Boot] [PATCH] powerpc/t2080: DDR controller erratum A-009942

2016-10-26 Thread york sun
On 10/26/2016 12:54 PM, Hamish Martin wrote: > > On 10/27/2016 05:33 AM, york sun wrote: >> On 10/21/2016 04:37 PM, york sun wrote: >>> On 10/18/2016 07:48 PM, Hamish Martin wrote: This erratum is already implemented for other ARM based QorIQ platforms with the Gen4 DDR controller. Port

Re: [U-Boot] [PATCH v2] powerpc/t2080: CPU erratum A-007907

2016-10-26 Thread york sun
On 10/24/2016 01:48 PM, Darwin Dingel wrote: > +#define CONFIG_SYS_FSL_ERRATUM_A007907 Sorry I cannot merge this patch now. This new macro is rejected by recent enforcement of using Kconfig. I will hold this in my queue and change it to Kconfig when I finish converting these macros. York

Re: [U-Boot] [PATCH v2] drivers/pci/Kconfig: Add PCI

2016-10-26 Thread Stephen Warren
On 10/26/2016 03:15 PM, Tom Rini wrote: Add 'PCI' as a menu option and migrate all existing users. Signed-off-by: Tom Rini --- Changes in v2: - Per Stephen's comments, only a handful of Tegra configs have PCI so move all of that logic to the config files directly rather

[U-Boot] [PATCH v2] drivers/pci/Kconfig: Add PCI

2016-10-26 Thread Tom Rini
Add 'PCI' as a menu option and migrate all existing users. Signed-off-by: Tom Rini --- Changes in v2: - Per Stephen's comments, only a handful of Tegra configs have PCI so move all of that logic to the config files directly rather than have it in

Re: [U-Boot] [Resend RFC PATCH 1/2] armv8: Fix dcache disable function

2016-10-26 Thread york sun
On 10/26/2016 12:54 PM, york@nxp.com wrote: > On 10/26/2016 12:47 PM, Stephen Warren wrote: >> >> There are two data structures in ARM U-Boot that describe memory layout: >> >> 1) A list of RAM memory regions. U-Boot uses these to know where to >> relocate itself to (it relocates itself to the

Re: [U-Boot] [Resend RFC PATCH 1/2] armv8: Fix dcache disable function

2016-10-26 Thread Stephen Warren
On 10/26/2016 02:29 PM, york sun wrote: On 10/26/2016 01:12 PM, Stephen Warren wrote: On 10/26/2016 01:54 PM, york sun wrote: On 10/26/2016 12:47 PM, Stephen Warren wrote: There are two data structures in ARM U-Boot that describe memory layout: 1) A list of RAM memory regions. U-Boot uses

Re: [U-Boot] [PATCH] drivers/pci/Kconfig: Add PCI

2016-10-26 Thread Tom Rini
On Wed, Oct 26, 2016 at 11:11:31AM -0600, Stephen Warren wrote: > On 10/24/2016 02:13 PM, Tom Rini wrote: > >Add 'PCI' as a menu option and migrate all existing users. > > > >Signed-off-by: Tom Rini > >--- > >Tegra is in a funny spot here. TEGRA_COMMON will select DM_PCI and

Re: [U-Boot] [PATCH] arm: dts: Pine64: add Ethernet alias

2016-10-26 Thread André Przywara
On 26/10/16 19:51, Jagan Teki wrote: Hi, > On Fri, Oct 21, 2016 at 5:41 AM, Andre Przywara > wrote: >> The sun8i-emac driver works fine with the A64 Ethernet IP, but we are >> missing an alias entry to trigger the driver instantiation by U-Boot. >> Add the line to point

Re: [U-Boot] [PATCH 0/6] Add ARMv8 PSCI framework

2016-10-26 Thread york sun
On 10/18/2016 12:18 AM, Hongbo Zhang wrote: > Ping all, > Some time ago I saw several people mentioned ARMv8 PSCI, are you > interested in leaving your review comments? > I know it is not so easy for reviewing assembly language patches, but > code structures and assembly instructions in this patch

Re: [U-Boot] [Resend RFC PATCH 1/2] armv8: Fix dcache disable function

2016-10-26 Thread york sun
On 10/26/2016 01:12 PM, Stephen Warren wrote: > On 10/26/2016 01:54 PM, york sun wrote: >> On 10/26/2016 12:47 PM, Stephen Warren wrote: >>> >>> There are two data structures in ARM U-Boot that describe memory layout: >>> >>> 1) A list of RAM memory regions. U-Boot uses these to know where to >>>

Re: [U-Boot] [Resend RFC PATCH 1/2] armv8: Fix dcache disable function

2016-10-26 Thread Stephen Warren
On 10/26/2016 01:54 PM, york sun wrote: On 10/26/2016 12:47 PM, Stephen Warren wrote: There are two data structures in ARM U-Boot that describe memory layout: 1) A list of RAM memory regions. U-Boot uses these to know where to relocate itself to (it relocates itself to the top of RAM at

Re: [U-Boot] [PATCH] powerpc/t2080: DDR controller erratum A-009942

2016-10-26 Thread Hamish Martin
On 10/27/2016 05:33 AM, york sun wrote: > On 10/21/2016 04:37 PM, york sun wrote: >> On 10/18/2016 07:48 PM, Hamish Martin wrote: >>> This erratum is already implemented for other ARM based QorIQ >>> platforms with the Gen4 DDR controller. Port the fix to the Gen3 >>> controller and enable it for

Re: [U-Boot] [Resend RFC PATCH 1/2] armv8: Fix dcache disable function

2016-10-26 Thread york sun
On 10/26/2016 12:47 PM, Stephen Warren wrote: > > There are two data structures in ARM U-Boot that describe memory layout: > > 1) A list of RAM memory regions. U-Boot uses these to know where to > relocate itself to (it relocates itself to the top of RAM at boot), and > to fill in the /memory node

Re: [U-Boot] [Resend RFC PATCH 1/2] armv8: Fix dcache disable function

2016-10-26 Thread Stephen Warren
On 10/24/2016 04:59 AM, Mark Rutland wrote: On Fri, Oct 21, 2016 at 07:31:52PM +, york sun wrote: On 10/20/2016 01:34 PM, Stephen Warren wrote: On 10/19/2016 11:06 PM, york sun wrote: I understand the data in dirty cache is not lost when the dcache is disabled. It is just not accessible.

Re: [U-Boot] [Resend RFC PATCH 1/2] armv8: Fix dcache disable function

2016-10-26 Thread Stephen Warren
On 10/24/2016 04:44 AM, Mark Rutland wrote: Hi, Sorry for joining this a bit late; apologies if the below re-treads ground already covered. On Wed, Oct 19, 2016 at 09:25:02AM -0600, Stephen Warren wrote: On 10/14/2016 02:17 PM, York Sun wrote: Current code turns off d-cache first, then flush

[U-Boot] [PATCH 2/2] image: Protect against overflow in unknown_msg()

2016-10-26 Thread Simon Glass
Coverity complains that this can overflow. If we later increase the size of one of the strings in the table, it could happen. Adjust the code to protect against this. Signed-off-by: Simon Glass Reported-by: Coverity (CID: 150964) --- common/image.c | 3 ++- 1 file changed,

[U-Boot] [PATCH 1/2] mkimage: Fix missing free() in show_valid_options()

2016-10-26 Thread Simon Glass
The allocated memory should be freed. Fix it. Signed-off-by: Simon Glass Reported-by: Coverity (CID: 150963) --- tools/mkimage.c | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/mkimage.c b/tools/mkimage.c index 3c594a0..521fa80 100644 --- a/tools/mkimage.c +++

Re: [U-Boot] [PATCH] cmd/fdt: Make fdt get value endian-safe for single-cell properties

2016-10-26 Thread Simon Glass
Hi Andreas, On 26 October 2016 at 09:02, Andreas Färber wrote: > On a Raspberry Pi 2 disagreements on cell endianness can be observed: > > U-Boot> fdt print /soc/gpio@7e20 phandle > phandle = <0x000d> > U-Boot> fdt get value myvar /soc/gpio@7e20 phandle;

Re: [U-Boot] [PATCH] MAINTAINERS: Fix syntax and update filename for FDT

2016-10-26 Thread Simon Glass
On 26 October 2016 at 09:14, Andreas Färber wrote: > Let get_maintainers.pl pick up the new cmd/fdt.c. > > Cc: Simon Glass > Signed-off-by: Andreas Färber > --- > MAINTAINERS | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [U-Boot] [PATCH] travis-ci: build Tegra boards

2016-10-26 Thread Tom Rini
On Wed, Oct 26, 2016 at 01:05:52PM -0600, Stephen Warren wrote: > From: Stephen Warren > > ARMv7 Tegra boards aren't currently covered by any other travis-ci jobs. > Add a new job to build them. > > Signed-off-by: Stephen Warren Reviewed-by: Tom Rini

[U-Boot] [PATCH] travis-ci: build Tegra boards

2016-10-26 Thread Stephen Warren
From: Stephen Warren ARMv7 Tegra boards aren't currently covered by any other travis-ci jobs. Add a new job to build them. Signed-off-by: Stephen Warren --- .travis.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.travis.yml b/.travis.yml

Re: [U-Boot] [PATCH 6/6] travis-ci: compile with buildman when running test/py

2016-10-26 Thread Stephen Warren
On 10/26/2016 12:59 PM, Tom Rini wrote: On Wed, Oct 26, 2016 at 11:05:36AM -0600, Stephen Warren wrote: From: Stephen Warren Use buildman to compile any U-Boot binary tested by test/py. This re-uses all the work done elsewhere to make buildman work within Travis-CI, in

Re: [U-Boot] [PATCH 6/6] travis-ci: compile with buildman when running test/py

2016-10-26 Thread Tom Rini
On Wed, Oct 26, 2016 at 11:05:36AM -0600, Stephen Warren wrote: > From: Stephen Warren > > Use buildman to compile any U-Boot binary tested by test/py. This > re-uses all the work done elsewhere to make buildman work within > Travis-CI, in particular related to toolchain

Re: [U-Boot] [PATCH 6/6] travis-ci: compile with buildman when running test/py

2016-10-26 Thread Stephen Warren
On 10/26/2016 11:05 AM, Stephen Warren wrote: From: Stephen Warren Use buildman to compile any U-Boot binary tested by test/py. This re-uses all the work done elsewhere to make buildman work within Travis-CI, in particular related to toolchain downloading and buildman

Re: [U-Boot] [PATCH] arm: dts: Pine64: add Ethernet alias

2016-10-26 Thread Jagan Teki
On Fri, Oct 21, 2016 at 5:41 AM, Andre Przywara wrote: > The sun8i-emac driver works fine with the A64 Ethernet IP, but we are > missing an alias entry to trigger the driver instantiation by U-Boot. > Add the line to point U-Boot to the Ethernet DT node. > This enables

Re: [U-Boot] [PATCH] arm: dts: Pine64: add Ethernet alias

2016-10-26 Thread Jagan Teki
On Wed, Oct 26, 2016 at 1:14 PM, Hans de Goede wrote: > Hi, > > > On 26-10-16 09:00, Jagan Teki wrote: >> >> On Tue, Oct 25, 2016 at 3:59 AM, André Przywara >> wrote: >>> >>> On 21/10/16 11:28, Hans de Goede wrote: Hi, On 21-10-16

Re: [U-Boot] [PATCH v8 05/23] net: Kconfig: Add FEC_MXC entry

2016-10-26 Thread Jagan Teki
On Wed, Oct 26, 2016 at 10:46 PM, Markus Niebel wrote: > Hello > > Am Montag, den 24.10.2016, 17:27 +0530 schrieb Jagan Teki: >> From: Jagan Teki >> >> Added kconfig for FEC_MXC driver. >> >> Cc: Joe Hershberger >> Cc:

Re: [U-Boot] [PATCH] fsl_qman: Implement device tree fixup for QBMan on ARM processors

2016-10-26 Thread Scott Wood
On Wed, 2016-10-26 at 10:01 -0400, Roy Pledge wrote: > Add support for device tree fixup for the DPAA1 QBMan nodes in ARM platforms > > Signed-off-by: Roy Pledge > --- >  arch/arm/cpu/armv8/fsl-layerscape/fdt.c|   89 > >  

Re: [U-Boot] [PATCH v8 05/23] net: Kconfig: Add FEC_MXC entry

2016-10-26 Thread Markus Niebel
Hello Am Montag, den 24.10.2016, 17:27 +0530 schrieb Jagan Teki: > From: Jagan Teki > > Added kconfig for FEC_MXC driver. > > Cc: Joe Hershberger > Cc: Simon Glass > Cc: Fabio Estevam > Cc:

Re: [U-Boot] [PATCH] spi: Add support for MX66U51235F, MX66L1G45G and MT25QU02G

2016-10-26 Thread Jagan Teki
On Wed, Oct 26, 2016 at 9:18 PM, Bacrau, Dumitru wrote: > Hello Teki, > > The device that I have used is MT25QU02GCBB8E12-0SIT, with the datasheet > available here: >

Re: [U-Boot] [PATCH v8 01/23] serial: Kconfig: Add MXC_UART entry

2016-10-26 Thread Jagan Teki
On Wed, Oct 26, 2016 at 10:44 PM, Markus Niebel wrote: > Hello > > Am Montag, den 24.10.2016, 17:27 +0530 schrieb Jagan Teki: >> From: Jagan Teki >> >> Added kconfig for MXC_UART driver. >> >> Cc: Simon Glass >> Cc: Fabio Estevam

Re: [U-Boot] [PATCH 2/2] armv8: ls1046aqds: add lpuart support

2016-10-26 Thread york sun
On 09/26/2016 11:36 PM, shh@gmail.com wrote: > From: Shaohui Xie > > LPUART0 is used by default, and it's using platform clock, if a different > clock is preferred, user can define CONFIG_LPUART_CLK. Shaohui, You may have to choose another name for macro

Re: [U-Boot] [PATCH v8 01/23] serial: Kconfig: Add MXC_UART entry

2016-10-26 Thread Markus Niebel
Hello Am Montag, den 24.10.2016, 17:27 +0530 schrieb Jagan Teki: > From: Jagan Teki > > Added kconfig for MXC_UART driver. > > Cc: Simon Glass > Cc: Fabio Estevam > Cc: Stefano Babic > Cc: Peng Fan

Re: [U-Boot] [PATCH] spi: Add support for MX66U51235F, MX66L1G45G and MT25QU02G

2016-10-26 Thread Bacrau, Dumitru
Hello Teki, The device that I have used is MT25QU02GCBB8E12-0SIT, with the datasheet available here: https://www.micron.com/~/media/documents/products/data-sheet/nor-flash/serial-nor/mt25q/die-rev-b/mt25q_qlkt_u_02g_cbb_0.pdf. Thanks a lot, Radu -Original Message- From: Jagan Teki

[U-Boot] [PATCH] armv8: fsl-layerscape: Add DPAA1 QBMan support

2016-10-26 Thread Roy Pledge
Add Queue Manager and Buffer Manager support for NXP Layerscape devices based on DPAA1. This includes LS1043A and LS1046A devices. Roy Pledge (1): fsl_qman: Implement device tree fixup for QBMan on ARM processors arch/arm/cpu/armv8/fsl-layerscape/fdt.c| 89

Re: [U-Boot] [PATCH v3 2/4] usb: dwc3: Add helper functions to enable snooping and burst settings

2016-10-26 Thread york sun
Rajat, On 08/04/2016 02:42 AM, Marek Vasut wrote: > On 08/04/2016 11:26 AM, Rajat Srivastava wrote: > > You can use clrsetbits_le32() or such if this is not direct port from Linux. > > > Parens around number are not necessary, please remove. > Are you going to update your patch? York

Re: [U-Boot] [PATCH] drivers/pci/Kconfig: Add PCI

2016-10-26 Thread Tom Rini
On Wed, Oct 26, 2016 at 11:11:31AM -0600, Stephen Warren wrote: > On 10/24/2016 02:13 PM, Tom Rini wrote: > >Add 'PCI' as a menu option and migrate all existing users. > > > >Signed-off-by: Tom Rini > >--- > >Tegra is in a funny spot here. TEGRA_COMMON will select DM_PCI and

Re: [U-Boot] [PATCH] drivers/pci/Kconfig: Add PCI

2016-10-26 Thread Stephen Warren
On 10/24/2016 02:13 PM, Tom Rini wrote: Add 'PCI' as a menu option and migrate all existing users. Signed-off-by: Tom Rini --- Tegra is in a funny spot here. TEGRA_COMMON will select DM_PCI and DM_PCICOMPAT. But adding PCI to the list here results in the following:

[U-Boot] [PATCH 6/6] travis-ci: compile with buildman when running test/py

2016-10-26 Thread Stephen Warren
From: Stephen Warren Use buildman to compile any U-Boot binary tested by test/py. This re-uses all the work done elsewhere to make buildman work within Travis-CI, in particular related to toolchain downloading and buildman config file creation. Signed-off-by: Stephen Warren

[U-Boot] [PATCH 2/6] travis-ci: use correct exit code on errors

2016-10-26 Thread Stephen Warren
From: Stephen Warren The phrase "if [ $? -ne 0 ]; then exit $?; fi" doesn't work correctly; by the time the "exit" statement runs, $? has already been over-written by the result of the [ command. Fix this by explicitly storing $? and then using that stored value in both the

[U-Boot] [PATCH 4/6] travis-ci: use buildman -P everywhere

2016-10-26 Thread Stephen Warren
From: Stephen Warren This places build results into a board-specific directory rather than a buildman-thread-specific directory. This is required so that we can access the directory from test.py, and there's no risk of a particular build's results being over-written by

[U-Boot] [PATCH 5/6] travis-ci: don't invoke exit on success

2016-10-26 Thread Stephen Warren
From: Stephen Warren Invoking exit prevents any subsequent build commands from running, and future patches will add extra commands. Signed-off-by: Stephen Warren --- .travis.yml | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git

[U-Boot] [PATCH 3/6] travis-ci: centralize ~/.buildman editing

2016-10-26 Thread Stephen Warren
From: Stephen Warren Any time an x86 toolchain is used, we need to edit ~/.buildman to reference it. Move the editing logic into a central place so that it doesn't have to be duplicated everywhere that uses the x86 toolchain; future patches will add additional cases where

[U-Boot] [PATCH 1/6] travis-ci: Use = not : when writing ~/.buildman

2016-10-26 Thread Stephen Warren
From: Stephen Warren Travis CI seems to be confused when there's a colon in an echo command, and this is currently worked around using a variable that contains the text we want to echo. Use = syntax instead so that we can remove the work-around; it's rather confusing until

Re: [U-Boot] [PATCH v2] powerpc/t2080: CPU erratum A-007907

2016-10-26 Thread york sun
On 10/24/2016 02:16 PM, york@nxp.com wrote: >> + >> +#ifdef CONFIG_SYS_FSL_ERRATUM_A007907 >> +flush_dcache(); >> +mtspr(L1CSR2, (mfspr(L1CSR2) & ~L1CSR2_DCSTASHID)); >> +sync(); >> +#endif >> + > > My erratum document shows a second step is to insert sync instruction > before each

Re: [U-Boot] [PATCH v7 1/2] armv8: Support loading 32-bit OS in AArch32 execution state

2016-10-26 Thread york sun
On 10/07/2016 11:56 PM, Alison Wang wrote: > To support loading a 32-bit OS, the execution state will change from > AArch64 to AArch32 when jumping to kernel. > > The architecture information will be got through checking FIT image, > then U-Boot will load 32-bit OS or 64-bit OS automatically. > >

Re: [U-Boot] [PATCH] armv8/ls1043a: Add the OCRAM initialization

2016-10-26 Thread york sun
On 10/24/2016 09:30 PM, Calvin Johnson wrote: I wonder why we don't see ECC errors before this patch. We have LS1043A boots on NAND, SD. >>> >>> OCRAM has a requirement of initializing before first time "read". >>> If user reads OCRAM before **initializing**; ECC error will come.

Re: [U-Boot] [PATCH v9 21/23] imx6: icorem6: Add NAND support

2016-10-26 Thread Jagan Teki
On Wed, Oct 26, 2016 at 8:38 PM, Stefano Babic wrote: > Hi Jagan, > > > On 25/10/2016 08:23, Jagan Teki wrote: >> From: Jagan Teki >> >> Add NAND support for Engicam i.CoreM6 qdl board. >> >> Boot Log: >> >> >> U-Boot SPL

Re: [U-Boot] [PATCH v4 1/2] armv8/ls1043a: fixup GIC offset according to SVR and SCFG_GIC400_ALIGN[GIC_ADDR_BIT]

2016-10-26 Thread york sun
On 10/26/2016 03:39 AM, Wenbin Song wrote: >>> + >>> +ENTRY(smp_kick_all_cpus) >>> + /* Kick secondary cpus up by SGI 0 interrupt */ >>> + mov x29, lr /* Save LR */ >>> + bl fix_gic_offset >>> + bl gic_kick_secondary_cpus >>> + mov lr, x29

Re: [U-Boot] [PATCH] powerpc/t2080: DDR controller erratum A-009942

2016-10-26 Thread york sun
On 10/21/2016 04:37 PM, york sun wrote: > On 10/18/2016 07:48 PM, Hamish Martin wrote: >> This erratum is already implemented for other ARM based QorIQ >> platforms with the Gen4 DDR controller. Port the fix to the Gen3 >> controller and enable it for T2080 and T2081. >> >> Reviewed-by: Chris

Re: [U-Boot] [PATCH] drivers/pci/Kconfig: Add PCI

2016-10-26 Thread Simon Glass
Hi Tom, On 24 October 2016 at 13:13, Tom Rini wrote: > Add 'PCI' as a menu option and migrate all existing users. > > Signed-off-by: Tom Rini > --- > Tegra is in a funny spot here. TEGRA_COMMON will select DM_PCI and > DM_PCICOMPAT. But adding PCI to

Re: [U-Boot] [PATCH 3/3] power: regulator: Add limits checking while setting current

2016-10-26 Thread Simon Glass
Hi Keerthy, On 26 October 2016 at 01:12, Keerthy wrote: > Currently the specific set ops functions are directly > called without any check for min/max current limits for a regulator. > Check for them and proceed. > > Signed-off-by: Keerthy > --- >

Re: [U-Boot] [PATCH 2/4] Fix spelling of "occur".

2016-10-26 Thread Simon Glass
On 23 October 2016 at 20:45, Vagrant Cascadian wrote: > Signed-off-by: Vagrant Cascadian > --- > > board/sysam/amcore/amcore.c | 2 +- > tools/mkimage.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) Reviewed-by: Simon Glass

Re: [U-Boot] [PATCH 2/3] power: regulator: Add limits checking while setting voltage

2016-10-26 Thread Simon Glass
On 26 October 2016 at 01:12, Keerthy wrote: > Currently the specific set ops functions are directly > called without any check for voltage limits for a regulator. > Check for them and proceed. > > Signed-off-by: Keerthy > --- >

Re: [U-Boot] [PATCH 1/3] power: regulator: Introduce regulator_set_value_force function

2016-10-26 Thread Simon Glass
On 26 October 2016 at 01:12, Keerthy wrote: > In case we want to force a particular value on a regulator > irrespective of the min/max constraints for testing purposes > one can call regulator_set_value_force function. > > Signed-off-by: Keerthy > --- >

Re: [U-Boot] [PATCH 5/5] configs: add the CONFIG_BLK and CONFIG_DM_MMC/OPS for exynos4 series

2016-10-26 Thread Simon Glass
On 23 October 2016 at 23:08, Jaehoon Chung wrote: > To use the CONFIG_BLK/CONFIG_DM_MMC/CONFIG_DM_MMC_OPS, add the > configurations for exynos4 series.(by default) > > Signed-off-by: Jaehoon Chung > --- > configs/odroid_defconfig | 3 +++ >

Re: [U-Boot] [PATCH 4/4] Fix spelling of "resetting".

2016-10-26 Thread Simon Glass
On 23 October 2016 at 20:45, Vagrant Cascadian wrote: > Cover-Letter: Fixes several spelling errors for the words "resetting", > "extended", "occur", and "multiple". > > Signed-off-by: Vagrant Cascadian > --- > > arch/arm/mach-exynos/dmc_init_ddr3.c

Re: [U-Boot] [PATCH 3/5] arm: dts: exynos4: change the compatible to meaning name

2016-10-26 Thread Simon Glass
On 23 October 2016 at 23:08, Jaehoon Chung wrote: > Changed the compatible from generic thing to meaning compatible. > "samsung,exynos-mmc" should be removed. > > Signed-off-by: Jaehoon Chung > --- > arch/arm/dts/exynos4.dtsi | 25

Re: [U-Boot] [PATCH 4/5] arm: dts: exynos4: change to use their parent node

2016-10-26 Thread Simon Glass
On 23 October 2016 at 23:08, Jaehoon Chung wrote: > Didn't overwrite the node. Just reuse the parent node. > And add the Exynos Series specific properties. > > Signed-off-by: Jaehoon Chung > --- > arch/arm/dts/exynos4210-origen.dts | 24

Re: [U-Boot] [PATCH 2/5] arm: dts: exynos4x12: remove the dupulicated node

2016-10-26 Thread Simon Glass
On 23 October 2016 at 23:08, Jaehoon Chung wrote: > mshc_0 node should be overwriten the defined node in exynos4.dtsi. I think you mean 'The mshc_0 node overwrites the defined node...' > It's meaningless. So removed this node. Instead, use the node in exynos4.dtsi. 'So

Re: [U-Boot] [PATCH 3/4] Fix spelling of "extended".

2016-10-26 Thread Simon Glass
On 23 October 2016 at 20:45, Vagrant Cascadian wrote: > Signed-off-by: Vagrant Cascadian > --- > > cmd/bootm.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Simon Glass

Re: [U-Boot] [PATCH 1/4] Fix spelling of "multiple".

2016-10-26 Thread Simon Glass
On 23 October 2016 at 20:45, Vagrant Cascadian wrote: > Signed-off-by: Vagrant Cascadian > --- > > fs/ubifs/ubifs.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Simon Glass

Re: [U-Boot] [PATCH v4 00/29] Kconfig: Move console options to Kconfig

2016-10-26 Thread Simon Glass
Hi Tom, On 23 October 2016 at 18:40, Tom Rini wrote: > On Mon, Oct 17, 2016 at 08:12:33PM -0600, Simon Glass wrote: > >> This series moves a number of console-related CONFIG options to Kconfig. >> Those that are not currently used are removed. >> >> A few unused video drivers

Re: [U-Boot] [PATCH 0/8] dm: Update on serial driver progress

2016-10-26 Thread Simon Glass
Hi, On 21 October 2016 at 01:10, Marek Vasut wrote: > On 10/21/2016 08:52 AM, Marcel Ziswiler wrote: >> Hi Simon >> >> On Thu, 2016-10-20 at 13:06 -0600, Simon Glass wrote: >>> Only three serial drivers remain to be converted. This series drops >>> two of those, since the boards

Re: [U-Boot] [PATCH v1 3/3] gpio: at91_gpio: Add the clock support

2016-10-26 Thread Simon Glass
On 20 October 2016 at 20:07, Wenyou Yang wrote: > Add the clock support. > > Signed-off-by: Wenyou Yang > --- > > drivers/gpio/at91_gpio.c | 13 + > 1 file changed, 13 insertions(+) Reviewed-by: Simon Glass

Re: [U-Boot] [PATCH v1 2/3] gpio: at91_gpio: Add the device tree support

2016-10-26 Thread Simon Glass
Hi Wenyou, On 20 October 2016 at 20:07, Wenyou Yang wrote: > Add the device tree support. > > Signed-off-by: Wenyou Yang > --- > > drivers/gpio/at91_gpio.c | 15 +++ > 1 file changed, 15 insertions(+) Reviewed-by: Simon Glass

Re: [U-Boot] [PATCH v1 1/3] gpio: Kconfig: Add CONFIG_AT91_GPIO option

2016-10-26 Thread Simon Glass
On 20 October 2016 at 20:07, Wenyou Yang wrote: > The CONFIG_AT91_GPIO option is used to select AT91 PIO GPIO driver. > > Signed-off-by: Wenyou Yang > --- > > drivers/gpio/Kconfig | 13 + > 1 file changed, 13 insertions(+) Reviewed-by:

Re: [U-Boot] Booting a proprietary OS

2016-10-26 Thread Simon Glass
Hi, On 19 October 2016 at 00:47, Paolo De Luca Bosso wrote: > Hi! > > Yes, i have a question about U-Boot. > > Can i use U-Boot as bootloader for a proprietary operating system? Yes. > > I want use bootm command and a bin file. In particular the Image kernel is >

Re: [U-Boot] [PATCH v1] arm: Add support for PDU001

2016-10-26 Thread Simon Glass
Hi Felix, On 19 October 2016 at 04:12, Felix Brack wrote: > This patch adds support for the PDU001 board. > > Signed-off-by: Felix Brack > --- > > arch/arm/Kconfig | 1 + > arch/arm/cpu/armv7/am33xx/Kconfig | 9 ++ > board/eets/pdu001/Kconfig

Re: [U-Boot] [PATCH v2 0/3] LS1046A secure boot target addition

2016-10-26 Thread Andreas Färber
Hi, Am 26.10.2016 um 22:17 schrieb Sumit Garg: > The patch-set does the following: > > 1. Add NOR secure boot target on ls1046aqds platform. > 2. Add QSPI secure boot target on ls1046ardb platform. Please double-check your clock, you are posting in the future. Regards, Andreas -- SUSE Linux

[U-Boot] [PATCH] MAINTAINERS: Fix syntax and update filename for FDT

2016-10-26 Thread Andreas Färber
Let get_maintainers.pl pick up the new cmd/fdt.c. Cc: Simon Glass Signed-off-by: Andreas Färber --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 8e67202..3d18f28 100644 --- a/MAINTAINERS +++

[U-Boot] [PATCH] cmd/fdt: Make fdt get value endian-safe for single-cell properties

2016-10-26 Thread Andreas Färber
On a Raspberry Pi 2 disagreements on cell endianness can be observed: U-Boot> fdt print /soc/gpio@7e20 phandle phandle = <0x000d> U-Boot> fdt get value myvar /soc/gpio@7e20 phandle; printenv myvar myvar=0x0D00 Fix this by always treating the pointer as __be32 and

[U-Boot] [PATCH] fsl_qman: Implement device tree fixup for QBMan on ARM processors

2016-10-26 Thread Roy Pledge
Add support for device tree fixup for the DPAA1 QBMan nodes in ARM platforms Signed-off-by: Roy Pledge --- arch/arm/cpu/armv8/fsl-layerscape/fdt.c| 89 .../arm/cpu/armv8/fsl-layerscape/fsl_lsch2_speed.c |3 +

Re: [U-Boot] [PATCH v3 01/10] arm: add atomic functions with return support

2016-10-26 Thread Mark Rutland
Hi, On Wed, Oct 26, 2016 at 02:10:24PM +0200, Antoine Tenart wrote: > Implement three atomic functions to allow making an atomic operation > that returns the value. Adds: atomic_add_return(), atomic_sub_return(), > atomic_inc_return() and atomic_dec_return(). > > Signed-off-by: Antoine Tenart

Re: [U-Boot] [PATCH v9 21/23] imx6: icorem6: Add NAND support

2016-10-26 Thread Stefano Babic
Hi Jagan, On 25/10/2016 08:23, Jagan Teki wrote: > From: Jagan Teki > > Add NAND support for Engicam i.CoreM6 qdl board. > > Boot Log: > > > U-Boot SPL 2016.09-rc2-30755-gd3dc581-dirty (Sep 28 2016 - 23:00:43) > Trying to boot from NAND > NAND : 512 MiB >

Re: [U-Boot] [PATCH v3 06/10] tegra: select ARM_GIC for Tegra TK1s

2016-10-26 Thread Stephen Warren
On 10/26/2016 08:59 AM, Antoine Tenart wrote: Hello, On Wed, Oct 26, 2016 at 08:55:27AM -0600, Stephen Warren wrote: On 10/26/2016 06:10 AM, Antoine Tenart wrote: Select the newly introduced ARM_GIC option to the relevant configuration which also have a psci implementation. This doesn't

Re: [U-Boot] [PATCH v3 06/10] tegra: select ARM_GIC for Tegra TK1s

2016-10-26 Thread Antoine Tenart
Hello, On Wed, Oct 26, 2016 at 08:55:27AM -0600, Stephen Warren wrote: > On 10/26/2016 06:10 AM, Antoine Tenart wrote: > > Select the newly introduced ARM_GIC option to the relevant configuration > > which also have a psci implementation. > > This doesn't look right; all Tegras have a GIC, so

Re: [U-Boot] [PATCH v3 06/10] tegra: select ARM_GIC for Tegra TK1s

2016-10-26 Thread Stephen Warren
On 10/26/2016 06:10 AM, Antoine Tenart wrote: Select the newly introduced ARM_GIC option to the relevant configuration which also have a psci implementation. This doesn't look right; all Tegras have a GIC, so it's not a board-specific option. Perhaps TEGRA_COMMON or TEGRA_ARMV[78]_COMMON

Re: [U-Boot] [PATCH V4 1/8] tools: imximage: add plugin support

2016-10-26 Thread Stefano Babic
On 25/10/2016 02:09, Peng Fan wrote: > Hi Stefano, > On Mon, Oct 24, 2016 at 04:05:53PM +0200, Stefano Babic wrote: >> Hi Peng, >> >> On 11/10/2016 08:29, Peng Fan wrote: >>> Add plugin support for imximage. >>> >>> Define CONFIG_USE_IMXIMG_PLUGIN in defconfig to enable using plugin. >>> >>>

Re: [U-Boot] [PATCH v1 1/4] sf: add ADDR_4B for 4byte address support

2016-10-26 Thread Yao Yuan
On Wed, Oct 26, 2016 at 3:39 PM, Jagan Teki wrote: > On Wed, Sep 28, 2016 at 4:39 PM, Yuan Yao wrote: > > From: Yuan Yao > > > > Some new flash don't support bar register but use 4bytes address to > > support exceed 16MB flash size. > > So add flash

[U-Boot] [PATCH v3 05/10] exynos: select ARM_GIC for TARGET_ARNDALE

2016-10-26 Thread Antoine Tenart
Select the newly introduced ARM_GIC option to the relevant configuration which also have a psci implementation. Signed-off-by: Antoine Tenart --- arch/arm/mach-exynos/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/mach-exynos/Kconfig

[U-Boot] [PATCH v3 09/10] sun5i: add defines used by the PSCI code

2016-10-26 Thread Antoine Tenart
The sun5i SoCs can take advantage of the newly introduce PSCI suspend function. Add defines used by the PSCI code. Signed-off-by: Antoine Tenart --- include/configs/sun5i.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/include/configs/sun5i.h

  1   2   >