Re: [U-Boot] [PATCH v6 16/20] patman: buildman: Correct spelling of 'colour'

2014-08-09 Thread York Sun
On 8/9/14 3:34 PM, "Simon Glass" wrote: >Hi Fabio, > >On 9 August 2014 16:29, Fabio Estevam wrote: >> On Sat, Aug 9, 2014 at 7:17 PM, Simon Glass wrote: >>> Hi, >>> >>> On 9 August 2014 16:01, Fabio Estevam wrote: Hi Simon, On Sat, Aug 9, 2014 at 6:33 PM, Simon Glass wrote: >>

Re: [U-Boot] [PATCH 1/3] config: introduce a generic $bootcmd

2014-08-09 Thread Stephen Warren
On 08/09/2014 04:43 PM, Jeroen Hofstee wrote: > On 09-08-14 17:02, Hans de Goede wrote: >> On 08/08/2014 06:00 PM, Stephen Warren wrote: >>> On 08/07/2014 06:17 PM, Simon Glass wrote: ... Acked-by: Simon Glass >>> >>> For the list archive's record: Simon also replied to patch 2 with the >>> s

Re: [U-Boot] [PATCH 1/3] config: introduce a generic $bootcmd

2014-08-09 Thread Jeroen Hofstee
Hi Hans, On 09-08-14 17:02, Hans de Goede wrote: Hi, On 08/08/2014 06:00 PM, Stephen Warren wrote: On 08/07/2014 06:17 PM, Simon Glass wrote: Acked-by: Simon Glass For the list archive's record: Simon also replied to patch 2 with the same ack, but somehow the CC list got dropped to only my

Re: [U-Boot] [PATCH v6 16/20] patman: buildman: Correct spelling of 'colour'

2014-08-09 Thread Simon Glass
Hi Fabio, On 9 August 2014 16:29, Fabio Estevam wrote: > On Sat, Aug 9, 2014 at 7:17 PM, Simon Glass wrote: >> Hi, >> >> On 9 August 2014 16:01, Fabio Estevam wrote: >>> Hi Simon, >>> >>> On Sat, Aug 9, 2014 at 6:33 PM, Simon Glass wrote: This code came from another project. Correct the s

[U-Boot] [PATCH 2/2] config_distro_defaults.h: add CONFIG_API

2014-08-09 Thread Jeroen Hofstee
Grub, FreeBSD ubldr, vxworks etc depend on the API Signed-off-by: Jeroen Hofstee --- include/config_distro_defaults.h | 4 1 file changed, 4 insertions(+) diff --git a/include/config_distro_defaults.h b/include/config_distro_defaults.h index 5d18a4b..b4f6f3c 100644 --- a/include/config_dis

[U-Boot] [PATCH 1/2] api: fix build without CMD_NET support

2014-08-09 Thread Jeroen Hofstee
Provide stubs in case that no NET interface is supported. Signed-off-by: Jeroen Hofstee --- api/api_net.c | 30 ++ 1 file changed, 30 insertions(+) diff --git a/api/api_net.c b/api/api_net.c index 3f52d71..7b3805e 100644 --- a/api/api_net.c +++ b/api/api_net.c @@ -25

[U-Boot] Add CONFIG_API to config_distribution

2014-08-09 Thread Jeroen Hofstee
Since the goal of the common include is to boot many distros, also include the api to boot distro's relying on the api, like GRUB / ubldr / vxWorks. Jeroen Hofstee (2): api: fix build without CMD_NET support config_distro_defaults.h: add CONFIG_API api/api_net.c| 30 +

Re: [U-Boot] [PATCH v6 16/20] patman: buildman: Correct spelling of 'colour'

2014-08-09 Thread Fabio Estevam
On Sat, Aug 9, 2014 at 7:17 PM, Simon Glass wrote: > Hi, > > On 9 August 2014 16:01, Fabio Estevam wrote: >> Hi Simon, >> >> On Sat, Aug 9, 2014 at 6:33 PM, Simon Glass wrote: >>> This code came from another project. Correct the spelling now that it is >>> in U-Boot. >> >> >> >>> -c

Re: [U-Boot] [PATCH v6 16/20] patman: buildman: Correct spelling of 'colour'

2014-08-09 Thread Simon Glass
Hi, On 9 August 2014 16:01, Fabio Estevam wrote: > Hi Simon, > > On Sat, Aug 9, 2014 at 6:33 PM, Simon Glass wrote: >> This code came from another project. Correct the spelling now that it is >> in U-Boot. > > > >> -col: terminal.Color() object >> +col: terminal.Colour() obj

Re: [U-Boot] [PATCH v6 16/20] patman: buildman: Correct spelling of 'colour'

2014-08-09 Thread Fabio Estevam
Hi Simon, On Sat, Aug 9, 2014 at 6:33 PM, Simon Glass wrote: > This code came from another project. Correct the spelling now that it is > in U-Boot. > -col: terminal.Color() object > +col: terminal.Colour() object I thought that both forms were correct and that "color" is

Re: [U-Boot] [PATCH 1/3] config: introduce a generic $bootcmd

2014-08-09 Thread Tom Rini
On Sat, Aug 09, 2014 at 05:02:46PM +0200, Hans de Goede wrote: > Hi, > > On 08/08/2014 06:00 PM, Stephen Warren wrote: > > On 08/07/2014 06:17 PM, Simon Glass wrote: > >> Acked-by: Simon Glass > > > > For the list archive's record: Simon also replied to patch 2 with the same > > ack, but someho

[U-Boot] [PATCH v6 02/20] buildman: Add some notes about moving from MAKEALL

2014-08-09 Thread Simon Glass
For those used to MAKEALL, buildman seems strange. Add some notes to ease the transition. Signed-off-by: Simon Glass --- Changes in v6: - Add new patch to remove patman's -a option Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: None tools/buildman/README | 92 +

[U-Boot] [PATCH v6 12/20] buildman: Introduce an 'and' operator for board selection

2014-08-09 Thread Simon Glass
Currently buildman allows a list of boards to build to be specified on the command line. The list can include specific board names, architecture, SOC and so on. At present the list of boards is dealt with in an 'OR' fashion, and there is no way to specify something like 'arm & freescale', meaning

[U-Boot] [PATCH v6 14/20] patman: Move the 'git log' command into a function

2014-08-09 Thread Simon Glass
Move the code that builds a 'git log' command into a function so we can more easily adjust it. Signed-off-by: Simon Glass --- Changes in v6: None Changes in v5: - Update test for verbose option Changes in v4: None Changes in v3: - Add new patch to add a verbose option Changes in v2: None too

[U-Boot] [PATCH v6 13/20] buildman: Allow selection of the number of commits to build

2014-08-09 Thread Simon Glass
It is useful to be able to build only some of the commits in a branch. Add support for the -c option to allow this. It was previously parsed by buildman but not implemented. Suggested-by: York Sun Signed-off-by: Simon Glass --- Changes in v6: None Changes in v5: None Changes in v4: - Add new pa

[U-Boot] [PATCH v6 08/20] buildman: Remove unused non-incremental build method code

2014-08-09 Thread Simon Glass
The non-incremental build method is no longer used, so remove it. Signed-off-by: Simon Glass --- Changes in v6: - Add new patch to implement -c option Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: None tools/buildman/builder.py | 34 ---

[U-Boot] [PATCH v6 06/20] buildman: Refactor output options

2014-08-09 Thread Simon Glass
We need the output options to be available in several places. It's a pain to pass them into each function. Make them properties of the builder and add a single function to set them up. At the same time, add a function which produces summary output using these options. Signed-off-by: Simon Glass -

[U-Boot] [PATCH v6 09/20] buildman: Add an option to specify the buildman config file

2014-08-09 Thread Simon Glass
Add a new --config-file option (-G) to specify a different configuration file from the default ~/.buildman. Reported-by: Tom Rini Signed-off-by: Simon Glass --- Changes in v6: None Changes in v5: - Add new patch to add an 'and' operator for board selection Changes in v4: None Changes in v3: No

[U-Boot] [PATCH v6 01/20] buildman: Fix a few typos

2014-08-09 Thread Simon Glass
There are several typos in the README - fix them. Signed-off-by: Simon Glass --- Changes in v6: None Changes in v5: - Drop patch to search for *cc instead of *gcc for the compiler Changes in v4: None Changes in v3: None Changes in v2: None tools/buildman/README | 12 ++-- 1 file chang

[U-Boot] [PATCH v6 16/20] patman: buildman: Correct spelling of 'colour'

2014-08-09 Thread Simon Glass
This code came from another project. Correct the spelling now that it is in U-Boot. Signed-off-by: Simon Glass --- Changes in v6: None Changes in v5: None Changes in v4: - Fix typo in commit message Changes in v3: - Add new patch to sort command line options Changes in v2: None tools/buildma

[U-Boot] [PATCH v6 20/20] RFC: Deprecate MAKEALL

2014-08-09 Thread Simon Glass
Since buildman now includes most of the features of MAKEALL it is probably time to talk about deprecating MAKEALL. Comments welcome. Signed-off-by: Simon Glass --- Changes in v6: None Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: - Add new patch to fix existing typo

[U-Boot] [PATCH v6 15/20] patman: Only use git's --no-decorate when available

2014-08-09 Thread Simon Glass
Older versions of git (e.g. Ubuntu 10.04) do not support this flag. By default they do not decorate. So only enable this flag when supported. Suggested-by: Tom Rini Signed-off-by: Simon Glass --- Changes in v6: None Changes in v5: - Update test for output options Changes in v4: None Changes in

[U-Boot] [PATCH v6 19/20] patman: Remove the -a option

2014-08-09 Thread Simon Glass
It seems that this is no longer needed, since checkpatch.pl will catch whitespace problems in patches. Also the option is not widely used, so it seems safe to just remove it. Suggested-by: Masahiro Yamada Signed-off-by: Simon Glass --- Changes in v6: None Changes in v5: None Changes in v4: None

[U-Boot] [PATCH v6 03/20] buildman: Allow building of current source tree

2014-08-09 Thread Simon Glass
Originally buildman had some support for building the current source tree. However this was dropped before it was submitted, as part of the effort to make it faster when building entire branches. Reinstate this support. If no -b option is given, buildman will build the current source tree. Report

[U-Boot] [PATCH v6 04/20] buildman: Move BuilderThread code to its own file

2014-08-09 Thread Simon Glass
The builder.py file is getting too long, so split out some code. Signed-off-by: Simon Glass --- Changes in v6: - Add new patch to fix indentation in teminal.py Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: None tools/buildman/builder.py | 439 +--

[U-Boot] [PATCH v6 07/20] buildman: Add verbose option to display errors as they happen

2014-08-09 Thread Simon Glass
Normally buildman operates in two passes - one to do the build and another to summarise the errors. Add a verbose option (-v) to display build problems as they happen. With -e also given, this will display errors too. When building the current source tree (rather than a list of commits in a branch

[U-Boot] [PATCH v6 17/20] patman: Fix indentation in terminal.py

2014-08-09 Thread Simon Glass
This code came from a different project with 2-character indentation. Fix it for U-Boot. Signed-off-by: Simon Glass --- Changes in v6: None Changes in v5: None Changes in v4: - Fix missing import in builder.py Changes in v3: - Add new patch to move BuilderThread code to its own file Changes in

[U-Boot] [PATCH v6 11/20] buildman: Add a few more toolchain examples to the README

2014-08-09 Thread Simon Glass
The current README is a bit sparse in this area, so add a few more examples. Suggested-by: Tom Rini Signed-off-by: Simon Glass --- Changes in v6: None Changes in v5: None Changes in v4: - Add new patch to show a message when there are no errors Changes in v3: None Changes in v2: None tools/b

[U-Boot] [PATCH v6 10/20] buildman: Add a message indicating there are no errors

2014-08-09 Thread Simon Glass
If buildman finds no problems it prints nothing. This can be a bit confusing, so add a message that all is well. Signed-off-by: Simon Glass --- Changes in v6: None Changes in v5: None Changes in v4: - Add new patch to add a few more toolchain examples to the README Changes in v3: None Changes i

[U-Boot] [PATCH v6 05/20] buildman: Sort command line options

2014-08-09 Thread Simon Glass
These options have got slightly out of order. Fix them. Signed-off-by: Simon Glass --- Changes in v6: - Add new patch to fix the spelling of 'colour' Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: None tools/buildman/buildman.py | 12 ++-- 1 file changed, 6

[U-Boot] [PATCH v6 18/20] patman: Correct unit tests to run correctly

2014-08-09 Thread Simon Glass
It seems that doctest behaves differently now, and some of the unit tests do not run. Adjust the tests to work correctly. ./tools/patman/patman --test Signed-off-by: Simon Glass --- Changes in v6: None Changes in v5: None Changes in v4: None Changes in v3: - Add new patch to build current sou

[U-Boot] [PATCH v6 0/20] Add some missing buildman features and deprecate MAKEALL

2014-08-09 Thread Simon Glass
Buildman has been around for a little over a year and is used by a fair number of U-Boot developers. However quite a few people still use MAKEALL. Buildman was intended to replace MAKEALL, so perhaps now is a good time to start that process. The reasons to deprecate MAKEALL are: - We don't want t

Re: [U-Boot] [PATCH 0/25] Introduce driver model support for SPI, SPI flash, cros_ec

2014-08-09 Thread Simon Glass
Hi, On 14 July 2014 18:56, Simon Glass wrote: > Up until now driver model has not been used for any type of bus. Buses > have some unique properties and needs, so we cannot claim that driver > model can cover all the common cases unless we have converted a bus over > to driver model. > > SPI is a

Re: [U-Boot] [PATCH v3 2/9] arm: Set up global data before board_init_f()

2014-08-09 Thread Simon Glass
Applied to u-boot-dm/master. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v3 5/9] arm: Support pre-relocation malloc()

2014-08-09 Thread Simon Glass
Applied to u-boot-dm/master. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] pci: mx6: fix occasional link failures

2014-08-09 Thread Fabio Estevam
On Sat, Aug 9, 2014 at 12:37 PM, Fabio Estevam wrote: > Ok, so with the debug patch applied I let it running overnight and it > ran 39k+ times without PCI linkup failures. > > I agree with your comments and now I am doing as you suggested: > > Running top of tree u-boot-imx with only Tim's patch

Re: [U-Boot] [PATCH v4 4/5] usb/gadget: fastboot: add sparse image definitions

2014-08-09 Thread Steve Rae
( Google agreed to submit a patch with BSD3 license stay tuned!!! ) On 14-08-09 05:48 AM, Marek Vasut wrote: On Friday, August 08, 2014 at 12:17:38 AM, Wolfgang Denk wrote: Dear Marek, In message <201408072343.58591.ma...@denx.de> you wrote: OK, so we need a GPLv2-compatible header file

Re: [U-Boot] Please pull u-boot-sunxi.git/master

2014-08-09 Thread Albert ARIBAUD
Hi Hans, On Thu, 31 Jul 2014 16:30:22 +0200, Hans de Goede wrote: > Hi Albert > > Note: > > 1) This superseeds my previous pull-req as that conflicted with the Kconfig > changes > which have landed in u-boot/master, this pull-req replaces all the boards.cfg > changes > from the previous seri

[U-Boot] [PATCH] mkimage: fix compilation issues on OpenBSD

2014-08-09 Thread Luka Perkov
Signed-off-by: Luka Perkov --- tools/mkimage.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/mkimage.c b/tools/mkimage.c index 123d0c7..c70408c9 100644 --- a/tools/mkimage.c +++ b/tools/mkimage.c @@ -458,6 +458,7 @@ NXTARG: ; #if defined(_POSIX_SYNCHRONIZED_IO) && \

Re: [U-Boot] [PATCH] pci: mx6: fix occasional link failures

2014-08-09 Thread Fabio Estevam
Hi Marek, On Sat, Aug 9, 2014 at 8:51 AM, Marek Vasut wrote: > Well you do realize that this addition changes the timing of the code and also > generates writes on the AXI bus, right? I would be much fonder of your testing > if you did it with a pristine code and monitored the number of successf

Re: [U-Boot] [PATCH 1/3] config: introduce a generic $bootcmd

2014-08-09 Thread Hans de Goede
Hi, On 08/08/2014 06:00 PM, Stephen Warren wrote: > On 08/07/2014 06:17 PM, Simon Glass wrote: >> Acked-by: Simon Glass > > For the list archive's record: Simon also replied to patch 2 with the same > ack, but somehow the CC list got dropped to only myself and TomW. I've a bunch of patches rel

[U-Boot] mx6: spl: Rename ncs as ranks and move it to mx6_ddr3_cfg

2014-08-09 Thread Nikolay Dimitrov
Hi guys, I'm working on adding SO-DIMM SPL support on a custom imx6 board, so I'm thinking on the idea of which DDR3 settings belong to the controller and which belong to the DDR3 memory module/chips. My proposal is to rename the struct member "ncs" to "ranks" (as per JEDEC) and to move it a

Re: [U-Boot] Please pull u-boot-sunxi.git/master

2014-08-09 Thread Albert ARIBAUD
Hi Masahiro, On my way to applying this. On Fri, 01 Aug 2014 10:54:28 +0900, Masahiro Yamada wrote: > Hi Hans, > > > On Thu, 31 Jul 2014 16:30:22 +0200 > Hans de Goede wrote: > > > Hi Albert > > > > Note: > > > > 1) This superseeds my previous pull-req as that conflicted with the Kconfig

Re: [U-Boot] Python subprocess module missing in multiconfig.py

2014-08-09 Thread Marek Vasut
On Saturday, August 09, 2014 at 12:16:01 AM, Jeroen Hofstee wrote: > Hello Marek, > > On 07-08-14 23:52, Marek Vasut wrote: > > > > Thanks for CCing me. > > > > I sent a patch to OE-core to ship the python modules. This is a problem > > with the Yocto Qt SDK toolchain in general and you can trac

Re: [U-Boot] [PATCH] pci: mx6: fix occasional link failures

2014-08-09 Thread Marek Vasut
On Friday, August 08, 2014 at 05:35:23 PM, Fabio Estevam wrote: > On Fri, Aug 8, 2014 at 10:33 AM, Marek Vasut wrote: > > Can you guys test it on FSL hardware ? > > Sure, I am running u-boot-imx with Tim's patch applied and this > additional debug patch that shows the number of reboots and PCI >

Re: [U-Boot] [PATCH v4 4/5] usb/gadget: fastboot: add sparse image definitions

2014-08-09 Thread Marek Vasut
On Friday, August 08, 2014 at 12:17:38 AM, Wolfgang Denk wrote: > Dear Marek, > > In message <201408072343.58591.ma...@denx.de> you wrote: > > OK, so we need a GPLv2-compatible header file. That's the conclusion. > > Well, GPLv2 compatible might be acceptable, but GPLv2+ compatible > woould be be

Re: [U-Boot] [PATCH v2 0/15] arm: kconfig: move board select menus and other settings to SoC dir

2014-08-09 Thread Masahiro YAMADA
Hi Tom, 2014-08-06 12:17 GMT+09:00 Masahiro Yamada : > There are lots of boards supported in ARM architecture. > The board select menu in arch/arm/Kconfig is already too fat. > > This series introduces platform select between arch select and > board select. > > ARCH select (ARM) >|-- Platfo