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

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. > Change

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

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 Reviewed-

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

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 and

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

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 Reviewed-by:

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

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 sin

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

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 > pl

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

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: Man

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

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 10/12] actions: Move defconfig options to Kconfig

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 11/12] arm: add Cubieboard7 board support

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"). > > Reviewe

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

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 00/12] Actions S700 SoC support

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 prese

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

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 EFI

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

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 b/i

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

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: det

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

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 chan

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

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 >>

[PATCH] arm: meson: imply DM_RNG

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 --- a/ar

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

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. >

Gitlab error: execution took longer than 1h

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' ... https

Re: Gitlab error: execution took longer than 1h

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: '/tmp

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

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- > > > > From:

Re: Gitlab error: execution took longer than 1h

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 be

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

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: [PATCH v3 4/5] rockchip: Enable HDMI output on rk3399 board w/ HDMI

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 Sad

[PATCH 00/22] buildman: Improve summary output

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 th

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

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

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 2aaedf44ac7..8

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

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 tes

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

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 b/tools/build

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

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 06/22] buildman: Add a test helper for creating a line prefix

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 b/tools/buildman/test.

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

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| 57

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

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 inf

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

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 b/tools/buil

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

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 a/tools/bui

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

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 b/tools/buildman/builde

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

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 7a3b658b00e..6541fa8f41

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

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 terminal.Print()

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

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

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 insertions(+),

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

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 a/

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

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 insertion

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

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

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 sour

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

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 18/22] buildman: Update the 'theory of operation' a little

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 b/tools/buildma

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

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] mtd: nand: pxa3xx: fix raw read when last_chunk_size == 0

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() i

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

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. U

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

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 1/2] dm: dump.c: Fix segfault when entry->of_match is NULL

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

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 inde

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

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

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

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 of

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

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 b/.azure-p

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

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 large

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

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 alrea

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

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 'dm-pu

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

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. > > Sign

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

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 make

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

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 fixe

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

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) > > are

Re: Please pull u-boot-video

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 a

Re: Please pull u-boot-x86

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 brea

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

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 ++ arch/x86/cpu/coreboot/K

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

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 5/6] x86: coreboot: Allow building an SPL image

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

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()

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

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 --- board/coreboot/

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

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 95a89c072d5..212b4d5

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

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 simp

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

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: [RFC 5/7] gpio: mxc_gpio: add OF_PLATDATA support

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 c924e52f0

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

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 delet

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

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 10:05,

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

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: [RFC 1/7] mmc: fsl_esdhc_imx: add OF_PLATDATA support

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 b/drivers/mmc/fsl_esdhc_

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

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 b/tools/dtoc/dtb_platdata.p

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

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 10:1

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

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

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: Re: [PATCH v2 1/2] serial: ns16550: Revert "Move PCI access from ofdata_to_platdata() to probe()"

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: [PATCH v2 2/2] efi_loader: identify EFI system partition

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 +++ > lib/efi_loader/efi_

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

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 >> --- >> incl

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

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

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 > > >

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

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 arch/ar

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

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 4/5] arm: orion5x: finish moving SoC to mach-orion5x

The SPL linker script had been left in arch/$(ARCH)/cpu/$(CPU)/$(SOC) so move it to the already-established arch/$(ARCH)/mach-$(SOC) location. Signed-off-by: Trevor Woerner --- arch/arm/mach-orion5x/Kconfig | 2 +- arch/arm/{cpu/arm926ejs/orion5x => mach-orion5x

[PATCH 3/5] rename symbol: CONFIG_ORION5X -> CONFIG_ARCH_ORION5X

Have this symbol follow the pattern of all other such symbols. This patch also removes a TODO from the code. Signed-off-by: Trevor Woerner --- arch/arm/Kconfig| 2 +- arch/arm/Makefile | 3 +-- arch/arm/mach-orion5x/Kconfig | 4 ++-- configs/edminiv2_defconfig

[PATCH 5/5] rename symbol: CONFIG_KIRKWOOD -> CONFIG_ARCH_KIRKWOOD

Have this symbol follow the pattern of all other such symbols. This patch also removes a TODO from the code. Signed-off-by: Trevor Woerner --- Kconfig | 2 +- arch/arm/Kconfig | 2 +- arch/arm/Makefile| 3 +-- arch/arm/dts/M