[PATCH 2/5] arch/arm/Makefile: sort machine names

2020-04-05 Thread Trevor Woerner
The comment hoped that people would keep the CONFIG_* macro names sorted, but this drifted over time. Put the list back in alphanumerical order. Signed-off-by: Trevor Woerner --- arch/arm/Makefile | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/arch/arm/Makefile

[PATCH 1/5] arm: lpc32xx: move SoC to mach-lpc32xx

2020-04-05 Thread Trevor Woerner
Following the example of most other SoCs in arch/$(ARCH)/cpu/$(CPU)/$(SOC) move the lpc32xx code from arch/arm/cpu/arm926ejs/lpc32xx to arch/arm/mach-lpc32xx. Following the checklist from commit 01f14456306c84f9bcd3945a10c98ae50e30542a: [1] move files from arch/arm/cpu/arm926ejs/lpc32xx to

Re: [PATCH v2 2/2] efi_loader: identify EFI system partition

2020-04-05 Thread AKASHI Takahiro
On Mon, Apr 06, 2020 at 07:12:56AM +0200, Heinrich Schuchardt wrote: > On 4/6/20 6:21 AM, AKASHI Takahiro wrote: > > Heinrich, > > > > On Sun, Apr 05, 2020 at 11:28:18AM +0200, Heinrich Schuchardt wrote: > >> For capsule updates we need to identify the EFI system partition. > > > > Right, but > >

Re: [PATCH 0/6] x86: efi: Add a 64-bit coreboot payload

2020-04-05 Thread Heinrich Schuchardt
On 4/6/20 1:22 AM, Simon Glass wrote: > This series creates a new 64-bit 'coreboot64' build which can be launched > from coreboot. It uses SPL to effect the jump to 64-bit mode. > > This was done in an attempt to get the Ubuntu FirmwareTestSuite[1] > working. Unfortunately the latest version shows

Re: [PATCH v2 2/2] efi_loader: identify EFI system partition

2020-04-05 Thread Heinrich Schuchardt
On 4/6/20 6:21 AM, AKASHI Takahiro wrote: > Heinrich, > > On Sun, Apr 05, 2020 at 11:28:18AM +0200, Heinrich Schuchardt wrote: >> For capsule updates we need to identify the EFI system partition. > > Right, but > >> Signed-off-by: Heinrich Schuchardt >> --- >> v2: >> no change >> --- >>

Re: [PATCH v2 2/2] efi_loader: identify EFI system partition

2020-04-05 Thread AKASHI Takahiro
Heinrich, On Sun, Apr 05, 2020 at 11:28:18AM +0200, Heinrich Schuchardt wrote: > For capsule updates we need to identify the EFI system partition. Right, but > Signed-off-by: Heinrich Schuchardt > --- > v2: > no change > --- > include/efi_loader.h | 7 +++ >

Re: Re: [PATCH v2 1/2] serial: ns16550: Revert "Move PCI access from ofdata_to_platdata() to probe()"

2020-04-05 Thread Bin Meng
Hi Simon, On Mon, Apr 6, 2020 at 11:43 AM Simon Glass wrote: > > Hi Bin, > > On Fri, 3 Apr 2020 at 02:35, Bin Meng wrote: > > > > Hi Wolfgang, > > > > On Fri, Apr 3, 2020 at 4:26 PM Wolfgang Wallner > > wrote: > > > > > > > > > Hi Andy, Bin, > > > > > > > -"Andy Shevchenko" schrieb: -

Re: [RFC] dm: uclass: add functions to get device by platdata

2020-04-05 Thread Simon Glass
Hi Walter, On Mon, 9 Mar 2020 at 12:27, Walter Lozano wrote: > > Hi Simon > > On 6/3/20 17:32, Simon Glass wrote: > > Hi Walter, > > > > On Fri, 6 Mar 2020 at 09:10, Walter Lozano > > wrote: > >> Hi Simon, > >> > >> Thanks again for taking the time to check my comments. > >> > >> On 6/3/20

Re: Re: [PATCH v2 1/2] serial: ns16550: Revert "Move PCI access from ofdata_to_platdata() to probe()"

2020-04-05 Thread Simon Glass
Hi Bin, On Fri, 3 Apr 2020 at 02:35, Bin Meng wrote: > > Hi Wolfgang, > > On Fri, Apr 3, 2020 at 4:26 PM Wolfgang Wallner > wrote: > > > > > > Hi Andy, Bin, > > > > > -"Andy Shevchenko" schrieb: - > > > On Thu, Apr 2, 2020 at 7:55 AM Bin Meng wrote: > > > > On Thu, Apr 2, 2020 at 1:55

Re: [RFC 7/7] mx6cuboxi: enable OF_PLATDATA

2020-04-05 Thread Simon Glass
On Sun, 29 Mar 2020 at 21:32, Walter Lozano wrote: > > Signed-off-by: Walter Lozano > --- > configs/mx6cuboxi_defconfig | 2 ++ > 1 file changed, 2 insertions(+) > Reviewed-by: SImon Glass (with commit msg)

Re: [RFC 1/7] mmc: fsl_esdhc_imx: add OF_PLATDATA support

2020-04-05 Thread Simon Glass
Hi Walter, On Sun, 29 Mar 2020 at 21:32, Walter Lozano wrote: > > Signed-off-by: Walter Lozano > --- > drivers/mmc/fsl_esdhc_imx.c | 46 + > 1 file changed, 42 insertions(+), 4 deletions(-) > > diff --git a/drivers/mmc/fsl_esdhc_imx.c

Re: [RFC 3/7] dtoc: update dtb_platdata to support cd-gpio

2020-04-05 Thread Simon Glass
On Sun, 29 Mar 2020 at 21:32, Walter Lozano wrote: > > Signed-off-by: Walter Lozano > --- > tools/dtoc/dtb_platdata.py | 9 ++--- > 1 file changed, 6 insertions(+), 3 deletions(-) This looks OK, but please add a test. > > diff --git a/tools/dtoc/dtb_platdata.py

Re: [RFC 5/7] gpio: mxc_gpio: add OF_PLATDATA support

2020-04-05 Thread Simon Glass
Hi Walter, On Sun, 29 Mar 2020 at 21:32, Walter Lozano wrote: > > Signed-off-by: Walter Lozano > --- > drivers/gpio/mxc_gpio.c | 27 ++- > 1 file changed, 26 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpio/mxc_gpio.c b/drivers/gpio/mxc_gpio.c > index

Re: [PATCH 2/2] dm: dump.c: Refactor dm_dump_drivers prints

2020-04-05 Thread Simon Glass
On Sun, 5 Apr 2020 at 10:47, Ovidiu Panait wrote: > > Refactor the printing sequence in dm_dump_drivers to make it more clear. > > Signed-off-by: Ovidiu Panait > Cc: Sean Anderson > Cc: Simon Glass > --- > drivers/core/dump.c | 18 +++--- > 1 file changed, 11 insertions(+), 7

Re: [PATCH 3/3] dm: core: refactor functions reading an u32 from dt

2020-04-05 Thread Simon Glass
Hi Dario, On Sat, 4 Apr 2020 at 06:49, wrote: > > > Il 2 aprile 2020 alle 20.54 Simon Glass ha scritto: > > > > > > Hi Dario, > > > > On Wed, 1 Apr 2020 at 13:34, wrote: > > > > > > > > > > Il 31 marzo 2020 alle 1.57 Simon Glass ha scritto: > > > > > > > > > > > > On Sun, 29 Mar 2020 at

Re: [RFC 2/7] mmc: fsl_esdhc_imx: add ofdata_to_platdata support

2020-04-05 Thread Simon Glass
Hi Walter On Sun, 29 Mar 2020 at 21:32, Walter Lozano wrote: > All of these commits need a commit message please. > Signed-off-by: Walter Lozano > --- > drivers/mmc/fsl_esdhc_imx.c | 71 ++--- > 1 file changed, 42 insertions(+), 29 deletions(-) > Reviewed-by:

Re: [PATCH 1/2] dm: dump.c: Fix segfault when entry->of_match is NULL

2020-04-05 Thread Simon Glass
Hi, On Sun, 5 Apr 2020 at 11:08, Ovidiu Panait wrote: > > On 05.04.2020 19:53, Sean Anderson wrote: > > > On 4/5/20 12:47 PM, Ovidiu Panait wrote: > >> Currently, dm drivers command produces a segfault: > >> => dm drivers > >> DriverCompatible > >>

Re: [PATCH V2 1/5] common: bouncebuf: Permit passing custom alignment check function

2020-04-05 Thread Jaehoon Chung
On 4/4/20 7:45 PM, Marek Vasut wrote: > Add extended version of the bounce_buffer_start(), which permits passing in > a custom alignment checker function for the buffer. This is useful e.g. on > systems with various DMA restrictions and where the checker function might > be more complex than a

[PATCH 3/6] x86: Update SPL for coreboot

2020-04-05 Thread Simon Glass
At present SPL only works on bare-metal builds. With a few tweaks it can be used for coreboot also. Signed-off-by: Simon Glass --- arch/x86/lib/spl.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/arch/x86/lib/spl.c b/arch/x86/lib/spl.c index

[PATCH 5/6] x86: coreboot: Allow building an SPL image

2020-04-05 Thread Simon Glass
Make a few adjustments to allow us to build an SPL image for coreboot. Signed-off-by: Simon Glass --- arch/x86/cpu/Makefile| 4 +++- arch/x86/cpu/coreboot/Makefile | 8 +++- arch/x86/cpu/coreboot/coreboot.c | 3 ++- arch/x86/cpu/coreboot/coreboot_spl.c | 12

[PATCH 4/6] x86: cbfs: Drop unwanted declaration

2020-04-05 Thread Simon Glass
The intention here is add a forward declaration, not actually declare a variable. Fix it. Signed-off-by: Simon Glass --- include/cbfs.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/cbfs.h b/include/cbfs.h index f3bc8ca24af..d915f9426d8 100644 --- a/include/cbfs.h

[PATCH 2/6] x86: Move work-around out of cpu_jump_to_64bit_uboot()

2020-04-05 Thread Simon Glass
At present this function copies U-Boot from the last 1MB of ROM. This is not the right way to do it. Instead, the binman symbol should provide the location. But in any case the code should live in the caller, spl_board_load_image(), so that the 64-bit jump function can be used elsewhere. Move it.

[PATCH 6/6] x86: Add a 64-bit coreboot build

2020-04-05 Thread Simon Glass
Add a build for running 64-bit U-Boot from coreboot (which is 32-bit). This uses binman to create an image with a 32-bit SPL and a 64-bit U-Boot. Coreboot boots into SPL and then SPL boots into U-Boot. This allows running 64-bit EFI images on x86. Signed-off-by: Simon Glass ---

[PATCH 0/6] x86: efi: Add a 64-bit coreboot payload

2020-04-05 Thread Simon Glass
This series creates a new 64-bit 'coreboot64' build which can be launched from coreboot. It uses SPL to effect the jump to 64-bit mode. This was done in an attempt to get the Ubuntu FirmwareTestSuite[1] working. Unfortunately the latest version shows a splash screen and hangs, perhaps due to some

[PATCH 1/6] x86: Allow building an SPL image for coreboot

2020-04-05 Thread Simon Glass
Coreboot runs in 32-bit mode and cannot run a 64-bit U-Boot. To get around this we can build a combined image with 32-bit SPL and 64-bit U-Boot. Add a build rule and binman definition for this. Signed-off-by: Simon Glass --- Makefile | 6 ++

Re: [PULL] u-boot-sh/master

2020-04-05 Thread Tom Rini
On Sat, Apr 04, 2020 at 10:14:06PM +0200, Marek Vasut wrote: > This is a small fix for current release (2020.04) > > The following changes since commit e0718b3ab754860bd47677e6b4fc5b70da42c4ab: > > Merge tag 'dm-pull-1apr20' of git://git.denx.de/u-boot-dm (2020-04-01 > 14:29:21 -0400) > >

Re: [PATCH v2] arch: Add explicit linker script for u-boot-elf

2020-04-05 Thread Tom Rini
On Mon, Mar 30, 2020 at 01:58:59PM +0200, Michal Simek wrote: > Commit f4dc714aaa2d ("arm64: Turn u-boot.bin back into an ELF file after > relocate-rela") > introduce REMAKE_ELF option to recreate u-boot.elf from u-boot -> > u-boot.bin + DT -> u-boot.elf. > > The best is to ilustrate it from

Re: Pull request, u-boot-tegra/master

2020-04-05 Thread Tom Rini
On Fri, Apr 03, 2020 at 10:23:03AM -0700, Tom Warren wrote: > Tom, > > Please pull u-boot-tegra/master into U-Boot/master. Thanks. > > All Tegra builds are OK on my system, and Stephen's test frame reports that > all tests pass. > This adds support for Jetson Nano, plus miscellaneous other

Re: Please pull u-boot-video

2020-04-05 Thread Tom Rini
On Thu, Apr 02, 2020 at 06:13:50PM +0200, Anatolij Gustschin wrote: > Hi Tom, > > The following changes since commit e0718b3ab754860bd47677e6b4fc5b70da42c4ab: > > Merge tag 'dm-pull-1apr20' of git://git.denx.de/u-boot-dm (2020-04-01 > 14:29:21 -0400) > > are available in the Git repository

Re: Please pull u-boot-x86

2020-04-05 Thread Tom Rini
On Sun, Apr 05, 2020 at 07:12:50AM +0800, Bin Meng wrote: > Hi Tom, > > This PR includes the following changes for v2020.04 release: > > - verdin-imx8mm board reST documentation update > - Intel Edison board ACPI table I2C/USB minor updates > - Fix a regression of ns16550 serial driver that

Re: [PATCH] mach-snapdragon: Fix overwriting last digit of serial number

2020-04-05 Thread Tom Rini
On Mon, Mar 16, 2020 at 05:51:51PM +0100, Jan-Christoph Tebbe wrote: > When generating the MAC address based on the boards serial number > the last digit was overwritten with the null termination. That way > boards with serial numbers close to each other would use the same > MAC address. > >

Re: [PATCH v2] Revert "common: spl_fit: Default to IH_OS_U_BOOT if FIT_IMAGE_TINY enabled"

2020-04-05 Thread Tom Rini
On Wed, Apr 01, 2020 at 07:42:04AM -0700, Ye Li wrote: > Commit cf8dcc5d02c3 ("common: spl_fit: Default to IH_OS_U_BOOT if > FIT_IMAGE_TINY enabled") is not correct, it will append fdt to each loadable > image. Actually when using TINY FIT, the first loadable image is thought as > u-boot and

Re: Pull request: u-boot-spi/master

2020-04-05 Thread Tom Rini
On Fri, Apr 03, 2020 at 08:00:24PM +0530, Jagan Teki wrote: > Hi Tom, > > Please pull this PR for the release. > > Summary: > - fix for MMIO window size (Tudor Ambarus) > > thanks, > Jagan. > > The following changes since commit e0718b3ab754860bd47677e6b4fc5b70da42c4ab: > > Merge tag

[PATCH] dm: core: Read parent ofdata before children

2020-04-05 Thread Simon Glass
At present a device can read its ofdata before its parent has done the same. This can cause problems in the case where the parent has a 'ranges' property, thus affecting the operation of dev_read_addr(), for example. We already probe parent devices before children so it does not seem to be a

[PATCH] travis/gitlab/azure: Run cppcheck in parallel

2020-04-05 Thread Simon Glass
This takes ages to run single-threaded. Adjust it to use all available processors. Signed-off-by: Simon Glass --- .azure-pipelines.yml | 2 +- .gitlab-ci.yml | 2 +- .travis.yml | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/.azure-pipelines.yml

Re: [PATCH 1/2] dm: dump.c: Fix segfault when entry->of_match is NULL

2020-04-05 Thread Ovidiu Panait
On 05.04.2020 19:53, Sean Anderson wrote: On 4/5/20 12:47 PM, Ovidiu Panait wrote: Currently, dm drivers command produces a segfault: => dm drivers DriverCompatible Segmentation fault (core dumped) This is caused by a NULL pointer dereference

Re: [PATCH 1/2] dm: dump.c: Fix segfault when entry->of_match is NULL

2020-04-05 Thread Sean Anderson
On 4/5/20 12:47 PM, Ovidiu Panait wrote: > Currently, dm drivers command produces a segfault: > => dm drivers > DriverCompatible > > Segmentation fault (core dumped) > > This is caused by a NULL pointer dereference of entry->of_match. > Add a check

[PATCH 1/2] dm: dump.c: Fix segfault when entry->of_match is NULL

2020-04-05 Thread Ovidiu Panait
Currently, dm drivers command produces a segfault: => dm drivers DriverCompatible Segmentation fault (core dumped) This is caused by a NULL pointer dereference of entry->of_match. Add a check to prevent this. Signed-off-by: Ovidiu Panait Cc: Sean

[PATCH 2/2] dm: dump.c: Refactor dm_dump_drivers prints

2020-04-05 Thread Ovidiu Panait
Refactor the printing sequence in dm_dump_drivers to make it more clear. Signed-off-by: Ovidiu Panait Cc: Sean Anderson Cc: Simon Glass --- drivers/core/dump.c | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/drivers/core/dump.c b/drivers/core/dump.c

[PATCH] ARM: rmobile: Unify Gen3 Salvator-X(S) and ULCB defconfigs

2020-04-05 Thread Marek Vasut
The r8a779{5,6,65}_salvator-x and r8a779{5,6,65}_ulcb_defconfig were building the same target, except for the default DT. The default DT is however only a detail, as the actual DT to be used to configure U-Boot is detected automatically based on the CPU ID, hence the default DT is not meaningful.

[PATCH] ARM: dts: rmobile: Scrub unused DT nodes

2020-04-05 Thread Marek Vasut
Remove DT nodes which are not used by U-Boot, like audio and video in/out nodes. This saves about 35 kiB on the resulting U-Boot binary without any impact on functionality. Signed-off-by: Marek Vasut Cc: Nobuhiro Iwamatsu --- arch/arm/dts/r8a77950-u-boot.dtsi | 77

[PATCH] mtd: nand: pxa3xx: fix raw read when last_chunk_size == 0

2020-04-05 Thread Baruch Siach
Commit 6293b0361d9 ("mtd: nand: pxa3xx: add raw read support") added the local data_len variable in handle_data_pio() to track read size, but forgot to update the condition of drain_fifo() call. That happens to work when the layout last_chunk_size != 0. But when last_chunk_size == 0, drain_fifo()

[PATCH 20/22] buildman: Update workflow documentation with more detail

2020-04-05 Thread Simon Glass
Make a few additions and change some wording in the workflow documentation. Signed-off-by: Simon Glass --- tools/buildman/README | 53 +-- 1 file changed, 36 insertions(+), 17 deletions(-) diff --git a/tools/buildman/README b/tools/buildman/README index

[PATCH 18/22] buildman: Update the 'theory of operation' a little

2020-04-05 Thread Simon Glass
Make a few updates to this important section of the documentation, to make things clearer. Signed-off-by: Simon Glass --- tools/buildman/README | 69 --- 1 file changed, 39 insertions(+), 30 deletions(-) diff --git a/tools/buildman/README

[PATCH 22/22] buildman: Add an option to ignore device-tree warnings

2020-04-05 Thread Simon Glass
Unfortunately the plague of device-tree warnings has not lifted. These warnings infiltrate almost every build, adding noise and confusion. Add a buildman option to ignore them. This option works only with the summary option (-s). It does not affect the build process. Signed-off-by: Simon Glass

[PATCH 21/22] buildman: Make -I the default

2020-04-05 Thread Simon Glass
At present buildman defaults to running 'mrproper' on every thread before it starts building commits for each board. This can add a delay of about 5 seconds to the start of the process, since the tools and other invariants must be rebuilt. In particular, a build without '-b', to build current

[PATCH 19/22] buildman: Add the abbreviation for --boards

2020-04-05 Thread Simon Glass
This option may be frequency used, so mention that it can be abbreviated to --bo Signed-off-by: Simon Glass --- tools/buildman/README | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tools/buildman/README b/tools/buildman/README index 86f5dfe501f..9bf4383f9f7 100644

[PATCH 15/22] buildman: Drop unused output code

2020-04-05 Thread Simon Glass
The commit counter is a hangover from when buildman processed each board for a commit. Now buildman processes each commit for a board, so this output is never triggered. Delete it. Signed-off-by: Simon Glass --- tools/buildman/builder.py | 5 - 1 file changed, 5 deletions(-) diff --git

[PATCH 14/22] buildman: Show a message when fetching a repo

2020-04-05 Thread Simon Glass
Fetching updated versions of a repo can take time. At present buildman gives no indication that it is doing this. Add a message to explain the delay. Tidy up a few other messages while we are here. Signed-off-by: Simon Glass --- tools/buildman/builder.py | 5 - 1 file changed, 4

[PATCH 17/22] buildman: Show a summary of the build result

2020-04-05 Thread Simon Glass
When buildman finishes it leaves the last summary line visible, which shows the number of successful builds, builds with warnings and builds with errors. It is useful also to see how many builds were done in total along with the time taken. Show these on a separate line before buildman finishes.

[PATCH 16/22] buildman: Show the number of builds remaining

2020-04-05 Thread Simon Glass
It is nice to see the actual number of builds remaining to complete. Add this in the progress message, using a different colour. Drop the unnecessary 'name' variable while we are here. Signed-off-by: Simon Glass --- tools/buildman/builder.py | 13 + 1 file changed, 9

[PATCH 12/22] patman: Support erasing a previously unfinished text line

2020-04-05 Thread Simon Glass
When printing progress it is useful to print a message and leave the cursor at the end of the line until the operation is finished. When it is finished, the line needs to be erased so a new line can start in its place. Add a function to handle clearing a line previously written by

[PATCH 13/22] buildman: Drop the line-clearing code in Builder

2020-04-05 Thread Simon Glass
The new feature in terminal can be used by buildman. Update the Builder class accordingly. Signed-off-by: Simon Glass --- tools/buildman/builder.py | 28 +++- 1 file changed, 3 insertions(+), 25 deletions(-) diff --git a/tools/buildman/builder.py

[PATCH 11/22] patman: Update flushing Print() for Python 3

2020-04-05 Thread Simon Glass
This does not seem to work on Python 3. Update the code to use the built-in support. Signed-off-by: Simon Glass --- tools/patman/terminal.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/tools/patman/terminal.py b/tools/patman/terminal.py index

[PATCH 10/22] buildman: Show the list of boards in magenta

2020-04-05 Thread Simon Glass
It is quite hard to see the list of board for each error line since the colour is the same as the actual error line. Show the board list in magenta so that it is easier to distinguish them. Signed-off-by: Simon Glass --- tools/buildman/builder.py | 15 --- tools/buildman/test.py|

[PATCH 09/22] buildman: Use an object to hold error lines

2020-04-05 Thread Simon Glass
At present the string for each error line is created in _CalcErrorDelta() and used to create the summary output. This is inflexible since all the information (error/warning character, error line, list of boards with that error line) is munged together in a string. Create an object to hold this

[PATCH 08/22] buildman: Use yellow consistently for warning lines

2020-04-05 Thread Simon Glass
At present warnings are shown in yellow in the summary (-s) but magenta in the detail listing (-e). Use yellow in both. Signed-off-by: Simon Glass --- tools/buildman/builder.py | 2 +- tools/buildman/test.py| 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH 06/22] buildman: Add a test helper for creating a line prefix

2020-04-05 Thread Simon Glass
The split/join code is repeated in a lot of places. Add a function to handle this. Signed-off-by: Simon Glass --- tools/buildman/test.py | 51 +- 1 file changed, 31 insertions(+), 20 deletions(-) diff --git a/tools/buildman/test.py

[PATCH 07/22] buildman: Test the output with --list-error-boards

2020-04-05 Thread Simon Glass
Add a test to cover this flag, which adds the name of each board to each error/warning line. Signed-off-by: Simon Glass --- tools/buildman/test.py | 46 +- 1 file changed, 32 insertions(+), 14 deletions(-) diff --git a/tools/buildman/test.py

[PATCH 03/22] buildman: Use an iterator to check test output

2020-04-05 Thread Simon Glass
Rather than using the absolute array index, use an interator to work through the expected output lines. This is easier to follow. Signed-off-by: Simon Glass --- tools/buildman/test.py | 94 +++--- 1 file changed, 52 insertions(+), 42 deletions(-) diff --git

[PATCH 01/22] buildman: Refactor error-line output int a function

2020-04-05 Thread Simon Glass
Reduce the amount of repeated code by creating an _OutputErrLines() function to hold this code. Signed-off-by: Simon Glass --- tools/buildman/builder.py | 32 +++- 1 file changed, 19 insertions(+), 13 deletions(-) diff --git a/tools/buildman/builder.py

[PATCH 05/22] buildman: Split out testOutput() into separate functions

2020-04-05 Thread Simon Glass
We want to add a few more tests similar to testOutput(). Split its logic into a function which runs buildman to get the output and another which checks the output. This will make it easier to reuse the code. Signed-off-by: Simon Glass --- tools/buildman/test.py | 34

[PATCH 02/22] buildman: Add test coverage for error/warning colour

2020-04-05 Thread Simon Glass
Buildman should output the right colours for each error/warning line. Some of these checks are missing. Add them. Signed-off-by: Simon Glass --- tools/buildman/test.py | 7 +++ 1 file changed, 7 insertions(+) diff --git a/tools/buildman/test.py b/tools/buildman/test.py index

[PATCH 04/22] buildman: Create temp directory in test setup

2020-04-05 Thread Simon Glass
Rather than having a few tests handle this themselves, create the temporary directory in the setUp() method and remove it in tearDown(). This will make it easier to add more tests. Only testOutput and testGit() actually need it, but it doesn't add to the test time noticeably to do this for all

[PATCH 00/22] buildman: Improve summary output

2020-04-05 Thread Simon Glass
This series makes the output produced by buildman a little more useful and easier to read: - when using -l to see which errors relate to which board, it shows the boards in a different colour so they can be distinguished from the errors - uses yellow consistently for warning lines - shows

Re: [PATCH v3 4/5] rockchip: Enable HDMI output on rk3399 board w/ HDMI

2020-04-05 Thread Manivannan Sadhasivam
On Thu, Apr 02, 2020 at 05:11:24PM +0530, Jagan Teki wrote: > Enable config options and console setting to respective > rk3399 board for HDMI output. > > Boards supported and tested on this patch are: > - NanoPc T4 > - NanoPi M4 > - NanoPi Neo4 > - ROC-RK3399-PC > - Rock960 > > Cc: Manivannan

Re: [PATCH] ARM: dts: stm32: Temporarily drop cd-gpios from AV96 DT

2020-04-05 Thread Manivannan Sadhasivam
On Thu, Apr 02, 2020 at 06:53:41PM +0200, Marek Vasut wrote: > The card-detect GPIO and any other GPIO access currently doesn't work in > U-Boot SPL on any STM32 platform and crashes the SPL. To work around this > problem on AV96 right before release, remove the cd-gpios from DT. This > patch must

Re: Gitlab error: execution took longer than 1h

2020-04-05 Thread Tom Rini
On Sun, Apr 05, 2020 at 07:02:44PM +0800, Bin Meng wrote: > Hi Heinrich, > > On Sun, Apr 5, 2020 at 6:56 PM Heinrich Schuchardt wrote: > > > > Hello Tom, > > > > I saw a GitLab error: build all PowerPC platforms: ERROR: Job failed: > > execution took longer than 1h0m0s seconds. Should this job

Re: [U-Boot] [PATCH v2 5/5] sifive: fu540: Enable spi-nor flash support

2020-04-05 Thread Bin Meng
Hi Jagan, On Sun, Apr 5, 2020 at 2:44 AM Jagan Teki wrote: > > Hi Bin, > > On Mon, Nov 18, 2019 at 8:19 AM Bin Meng wrote: > > > > Hi Segar Kadam, > > > > On Mon, Nov 18, 2019 at 4:59 AM Sagar Kadam wrote: > > > > > > > > > Hello Jagan/Bin, > > > > > > > -Original Message- > > > >

Re: Gitlab error: execution took longer than 1h

2020-04-05 Thread Bin Meng
Hi Heinrich, On Sun, Apr 5, 2020 at 6:56 PM Heinrich Schuchardt wrote: > > Hello Tom, > > I saw a GitLab error: build all PowerPC platforms: ERROR: Job failed: > execution took longer than 1h0m0s seconds. Should this job be split? > > This output was also strange: > No space left on device:

Gitlab error: execution took longer than 1h

2020-04-05 Thread Heinrich Schuchardt
Hello Tom, I saw a GitLab error: build all PowerPC platforms: ERROR: Job failed: execution took longer than 1h0m0s seconds. Should this job be split? This output was also strange: No space left on device: '/tmp/current/T1023RDB' No space left on device: '/tmp/current/MPC8349EMDS_SDRAM' ...

Re: [PATCH v9 03/12] serial: actions: add compatible string

2020-04-05 Thread Manivannan Sadhasivam
On Sun, Apr 05, 2020 at 11:08:21AM +0100, André Przywara wrote: > On 05/04/2020 07:59, Manivannan Sadhasivam wrote: > > On Wed, Apr 01, 2020 at 12:49:26PM +0530, Amit Singh Tomar wrote: > >> This patch adds "actions,owl-uart" string to the owl uart driver. It > >> is also defined in Linux kernel.

[PATCH] arm: meson: imply DM_RNG

2020-04-05 Thread Heinrich Schuchardt
All ARCH_MESON boards have a hardware random number generator. So we should enable building the RNG driver. Signed-off-by: Heinrich Schuchardt --- arch/arm/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index bbb1e2738b..eee4538c7e 100644 ---

Re: [PATCH v9 03/12] serial: actions: add compatible string

2020-04-05 Thread André Przywara
On 05/04/2020 07:59, Manivannan Sadhasivam wrote: > On Wed, Apr 01, 2020 at 12:49:26PM +0530, Amit Singh Tomar wrote: >> This patch adds "actions,owl-uart" string to the owl uart driver. It >> is also defined in Linux kernel. >> >> Reviewed-by: Andre Przywara >> Signed-off-by: Amit Singh Tomar

Re: [PATCH] efi_loader: enable RNG if DM_RNG is enabled

2020-04-05 Thread Heinrich Schuchardt
On 4/1/20 12:15 PM, Peter Robinson wrote: > Enable EFI_RNG_PROTOCOL by default if DM_RNG is set. > > Signed-off-by: Peter Robinson > --- > > I think it makes sense to enable EFI_RNG_PROTOCOL by default if a config > has enabled both DM_RNG and UEFI. > > lib/efi_loader/Kconfig | 1 + > 1 file

[PATCH v2 0/2] efi_loader: identify EFI system partition

2020-04-05 Thread Heinrich Schuchardt
For the implementation of capsule updates we need to know where the EFI system partition is located. With the patches the first available EFI system partition is determined both for MBR and GPT partition tables. v2: used BIT() macro to define bit mask Heinrich Schuchardt (2): part:

[PATCH v2 1/2] part: detect EFI system partition

2020-04-05 Thread Heinrich Schuchardt
Up to now for MBR and GPT partitions the info field 'bootable' was set to 1 if either the partition was an EFI system partition or the bootable flag was set. Turn info field 'bootable' into a bit mask with separate bits for bootable and EFI system partition. This will allow us to identify the

[PATCH v2 2/2] efi_loader: identify EFI system partition

2020-04-05 Thread Heinrich Schuchardt
For capsule updates we need to identify the EFI system partition. Signed-off-by: Heinrich Schuchardt --- v2: no change --- include/efi_loader.h | 7 +++ lib/efi_loader/efi_disk.c | 20 2 files changed, 27 insertions(+) diff --git a/include/efi_loader.h

Re: [PATCH v9 00/12] Actions S700 SoC support

2020-04-05 Thread Manivannan Sadhasivam
Hi Amit, On Wed, Apr 01, 2020 at 12:49:23PM +0530, Amit Singh Tomar wrote: > This adds Cubieboard7[1] support based on Action Semi's S700 SoC[2], It's > Quad-core ARMv8 SoC > with Cortex-A53 cores. Peripheral like UART seems to be compatible with S900 > SoC(basic support > for it is alreay

Re: [PATCH v9 12/12] doc: boards: add Cubieboard7 documentation

2020-04-05 Thread Manivannan Sadhasivam
On Wed, Apr 01, 2020 at 12:49:35PM +0530, Amit Singh Tomar wrote: > This adds build and flash steps for Actions S700 > based Cubieboard7 board. > > Signed-off-by: Amit Singh Tomar > --- > Changes since v8: > * No changes. > Changes since v7: > * No changes. > Changes since v6: >

Re: [PATCH v9 11/12] arm: add Cubieboard7 board support

2020-04-05 Thread Manivannan Sadhasivam
On Wed, Apr 01, 2020 at 12:49:34PM +0530, Amit Singh Tomar wrote: > The Cubieboard is a single board computer containing a > Actions S700 SoC(with 4 ARMv8 Cortex-A53 cores). > > This patch adds respective defconfig alongwith .dts(copied > from Linux v5.5-rc6 with hash "b3a987b0264d"). > >

Re: [PATCH v9 10/12] actions: Move defconfig options to Kconfig

2020-04-05 Thread Manivannan Sadhasivam
On Wed, Apr 01, 2020 at 12:49:33PM +0530, Amit Singh Tomar wrote: > This patch moves some of the config options from bubblegum_96_defconfig > to platform specific Kconfig file. > > Reviewed-by: Andre Przywara > Signed-off-by: Amit Singh Tomar Reviewed-by: Manivannan Sadhasivam Thanks, Mani

Re: [PATCH v9 08/12] arm: dts: actions: s700: add u-boot specific dtsi file

2020-04-05 Thread Manivannan Sadhasivam
On Wed, Apr 01, 2020 at 12:49:31PM +0530, Amit Singh Tomar wrote: > Devices like uart and clk are needed to be enabled before relocation. > this patch adds u-boot.dtsi file that mark these device as dm-pre-reloc. > > Reviewed-by: Andre Przywara > Signed-off-by: Amit Singh Tomar Reviewed-by:

Re: [PATCH v9 09/12] arm: add support Actions Semi S700

2020-04-05 Thread Manivannan Sadhasivam
On Wed, Apr 01, 2020 at 12:49:32PM +0530, Amit Singh Tomar wrote: > This patch adds basic support for Actions Semi based S700 > SoC, which is driven by common owl framework. > > Reviewed-by: Andre Przywara > Signed-off-by: Amit Singh Tomar Reviewed-by: Manivannan Sadhasivam Thanks, Mani >

Re: [PATCH v9 07/12] arm: actions: add S700 SoC device tree

2020-04-05 Thread Manivannan Sadhasivam
On Wed, Apr 01, 2020 at 12:49:30PM +0530, Amit Singh Tomar wrote: > This patch adds .dtsi file(sync with Linux 5.5-rc6 with hash "b3a987b0264d") > and required binding for S700 SoC that is a 64-bit Quad-core ARM > Cortex-A53 cores. > > It also provisions dts file to be built based on selected >

Re: [PATCH v9 06/12] clk: actions: Add common clock driver

2020-04-05 Thread Manivannan Sadhasivam
On Wed, Apr 01, 2020 at 12:49:29PM +0530, Amit Singh Tomar wrote: > This patch converts S900 clock driver to something common that can > be used for other SoCs, for instance S700(few of clk registers are same). > > Reviewed-by: Andre Przywara > Signed-off-by: Amit Singh Tomar > --- > Changes

Re: [PATCH v9 02/12] arm: actions: rename sysmap-s900 to sysmap-owl

2020-04-05 Thread Manivannan Sadhasivam
On Wed, Apr 01, 2020 at 12:49:25PM +0530, Amit Singh Tomar wrote: > Now that memory maps(for both S700 and S900 SoCs) can be managed using > a common file, rename sysmap-s900 to sysmap-owl to reflect the same. > > Reviewed-by: Manivannan Sadhasivam > Signed-off-by: Amit Singh Tomar

Re: [PATCH v9 05/12] arm: dts: actions: s900: add u-boot specific dtsi file

2020-04-05 Thread Manivannan Sadhasivam
On Wed, Apr 01, 2020 at 12:49:28PM +0530, Amit Singh Tomar wrote: > Devices like uart and clk are needed to be enabled before relocation. > This patch adds u-boot.dtsi file that mark these device as dm-pre-reloc. > > Reviewed-by: Manivannan Sadhasivam > Signed-off-by: Amit Singh Tomar

Re: [PATCH v9 04/12] arm: dts: sync dts for Action Semi S900

2020-04-05 Thread Manivannan Sadhasivam
On Wed, Apr 01, 2020 at 12:49:27PM +0530, Amit Singh Tomar wrote: > Synchronize device tree bindings with v5.5-rc6 tag with commit id > "b3a987b0264d". > > Also, it removes older clock binding defined for S900 along with undocumented > compatible string "actions,s900-serial" from serial driver

Re: [PATCH v9 03/12] serial: actions: add compatible string

2020-04-05 Thread Manivannan Sadhasivam
On Wed, Apr 01, 2020 at 12:49:26PM +0530, Amit Singh Tomar wrote: > This patch adds "actions,owl-uart" string to the owl uart driver. It > is also defined in Linux kernel. > > Reviewed-by: Andre Przywara > Signed-off-by: Amit Singh Tomar > --- > Changes since v8: > * No changes. >

Re: [PATCH v9 01/12] arm: actions: Add common framework for Actions Owl Semi SoCs

2020-04-05 Thread Manivannan Sadhasivam
On Wed, Apr 01, 2020 at 12:49:24PM +0530, Amit Singh Tomar wrote: > This commit adds common arch support for Actions Semi Owl > series SoCs and removes the Bubblegum96 board files. > > Reviewed-by: Andre Przywara > Signed-off-by: Amit Singh Tomar Reviewed-by: Manivannan Sadhasivam Thanks,