Re: [RESEND PATCH] kconfig: Proposed language extension for multiple builds

Hi-- On 3/10/23 18:37, Simon Glass wrote: > (I am sending this again to get more feedback) > > In the case of Linux, only one build is produced so there is only a > single configuration. For other projects, such as U-Boot and Zephyr, the > same code is used to produce multiple builds, each with

[RESEND PATCH] kconfig: Proposed language extension for multiple builds

(I am sending this again to get more feedback) In the case of Linux, only one build is produced so there is only a single configuration. For other projects, such as U-Boot and Zephyr, the same code is used to produce multiple builds, each with related (but different) options enabled. This can be

Re: [PATCH v2 3/5] binman: add tests for sign option

Hi Ivan, On Fri, 10 Mar 2023 at 17:47, Simon Glass wrote: > > Add the test which provides sequence of actions: > 1. create the image from binman dts > 2. create public and private keys > 3. add public key into dtb with fdt_add_pubkey > 4. 1. sign FIT container with new sign option with

Re: [PATCH v2 3/5] binman: add tests for sign option

Add the test which provides sequence of actions: 1. create the image from binman dts 2. create public and private keys 3. add public key into dtb with fdt_add_pubkey 4. 1. sign FIT container with new sign option with extracting from image 2. sign exact FIT container with

Re: [PATCH v3] binman: bintool: Add support for tool directories

Hi Neha, On Fri, 24 Feb 2023 at 03:51, Neha Malcom Francis wrote: > > Currently, bintool supports external compilable tools as single > executable files. Adding support for git repos that can be used to run > non-compilable scripting tools that cannot otherwise be present in > binman. > >

Re: [PATCH] x86: Revert "x86: minnowmax: Adjust CONFIG_TEXT_BASE"

Hi Bin, On Sun, 5 Feb 2023 at 19:35, Bin Meng wrote: > > Hi Simon, > > On Mon, Feb 6, 2023 at 8:54 AM Simon Glass wrote: > > > > This causes Minnowmax to stop booting. I am not sure why though. > > > > Drop the EFI_LOADER feature instead, since it is not needed. > > > > This reverts commit

Re: [PATCH v2 1/5] binman: add documentation for binman sign option

Add the documentation about binman sign option and providing an example. Signed-off-by: Ivan Mikhaylov --- tools/binman/binman.rst | 18 ++ 1 file changed, 18 insertions(+) Applied to u-boot-dm/next, thanks!

Re: [PATCH v2 2/5] binman: add sign option for binman

Hi Ivan, On Tue, 7 Mar 2023 at 14:13, Ivan Mikhaylov wrote: > > Introduce proof of concept for binman's new option which provides sign > and replace FIT containers in binary images. > > Usage as example: > > from: > mkimage -G privateky -r -o sha256,rsa4096 -F fit > binman replace -i flash.bin

Re: [PATCH v2 4/5] tools: add fdt_add_pubkey

From: Roman Kopytin Having to use the -K option to mkimage to populate U-Boot's .dtb with the public key while signing the kernel FIT image is often a little awkward. In particular, when using a meta-build system such as bitbake/Yocto, having the tasks of the kernel and U-Boot recipes

Re: [PATCH v2 5/5] test_vboot.py: include test of fdt_add_pubkey tool

Hi Ivan, On Tue, 7 Mar 2023 at 14:13, Ivan Mikhaylov wrote: > > From: Roman Kopytin > > Signed-off-by: Roman Kopytin > Cc: Rasmus Villemoes > --- > test/py/tests/test_vboot.py | 8 > 1 file changed, 8 insertions(+) > > diff --git a/test/py/tests/test_vboot.py

Re: [PATCH V5 08/12] iot2050: Add script for signing artifacts

Hi Jan, On Sun, 12 Feb 2023 at 22:33, Jan Kiszka wrote: > > On 13.02.23 05:26, Simon Glass wrote: > > Hi Jan, > > > > On Tue, 7 Feb 2023 at 11:39, Simon Glass wrote: > >> > >> Hi Jan, > >> > >> On Tue, 7 Feb 2023 at 09:45, Jan Kiszka wrote: > >>> > >>> On 07.02.23 16:28, Simon Glass wrote: >

Re: [PATCH v7 13/23] core: fdtaddr: add devfdt_get_addr_size_index_ptr function

On Fri, 10 Mar 2023 at 08:44, Johan Jonker wrote: > > Add devfdt_get_addr_size_index_ptr function with the same > functionality as devfdt_get_addr_size_index, but instead > a return pointer is given. > Use map_sysmem() function as cast for the return. > Make same fix for

Re: [PATCH v6 15/22] drivers: use dev_read_addr_index_ptr when cast to pointer

Hi Johan, On Mon, 6 Mar 2023 at 12:32, Johan Jonker wrote: > > > > On 3/6/23 19:20, Simon Glass wrote: > > Hi Johan, > > > > On Thu, 2 Mar 2023 at 17:15, Johan Jonker wrote: > >> > >> The fdt_addr_t and phys_addr_t size have been decoupled. A 32bit CPU > >> can expect 64-bit data from the

Re: [PATCH v6 12/22] core: fdtaddr: add devfdt_get_addr_size_index_ptr function

Hi Johan, On Mon, 6 Mar 2023 at 12:55, Johan Jonker wrote: > > > > On 3/6/23 18:53, Simon Glass wrote: > > Hi Johan, > > > > On Thu, 2 Mar 2023 at 17:15, Johan Jonker wrote: > >> > >> Add devfdt_get_addr_size_index_ptr function with the same > >> functionality as devfdt_get_addr_size_index, but

sandbox_spl build warning

Hi Marek, I see this in CI for -next: Building current source for 1 boards (0 threads, 64 jobs per thread) sandbox: w+ sandbox_spl + 1159 | fdt_test_header_get(uts, fdt, "magic", fdt_magic(fdt)); + | ^ +test/cmd/fdt.c:1159:9: note: referencing argument 2 of type ‘char

Re: [PATCH V7 08/15] tools: Add script for converting public key into device tree include

Hi Jan, On Tue, 28 Feb 2023 at 10:22, Jan Kiszka wrote: > > From: Jan Kiszka > > Allows to create a public key device tree dtsi for inclusion into U-Boot > SPL and proper during first build already. This can be achieved via > CONFIG_DEVICE_TREE_INCLUDES. > > Signed-off-by: Jan Kiszka > --- >

Re: [PATCH v2 2/5] binman: add sign option for binman

Hi Ivan, On Tue, 7 Mar 2023 at 14:13, Ivan Mikhaylov wrote: > > Introduce proof of concept for binman's new option which provides sign > and replace FIT containers in binary images. > > Usage as example: > > from: > mkimage -G privateky -r -o sha256,rsa4096 -F fit > binman replace -i flash.bin

Re: [PATCH 09/10] efi: Support showing tables

Hi Heinrich, On Mon, 6 Mar 2023 at 01:39, Heinrich Schuchardt wrote: > > On 2/26/23 02:33, Simon Glass wrote: > > Add a command to display the tables provided by EFI. > > > > Signed-off-by: Simon Glass > > --- > > > > cmd/efi.c | 40 +++- > >

Re: [PATCH v7 16/23] drivers: use dev_read_addr_index_ptr when cast to pointer

Hi Johan, On Fri, 10 Mar 2023 at 08:45, Johan Jonker wrote: > > The fdt_addr_t and phys_addr_t size have been decoupled. A 32bit CPU > can expect 64-bit data from the device tree parser, so use > dev_read_addr_index_ptr instead of the dev_read_addr_index function > in the various files in the

Re: [PATCH v7 22/23] arm: stm32mp: spl: fix function with fdt_addr_t input

On Fri, 10 Mar 2023 at 08:47, Johan Jonker wrote: > > The fdt_addr_t and phys_addr_t size have been decoupled. > A 32bit CPU can expect 64-bit data from the device tree parser, > so fix ofnode_get_addr_size function with fdt_addr_t input to > be able to handle both sizes for stm32mp SoC in spl.c

Re: [PATCH v7 11/23] core: remap: fix regmap_init_mem_plat() reg size handeling

Hi Johan, On Fri, 10 Mar 2023 at 08:42, Johan Jonker wrote: > > The fdt_addr_t and phys_addr_t size have been decoupled. > A 32bit CPU can expect 64-bit data from the device tree parser, Sorry if you already responded and I missed it. I don't understand this line. It looks like

Re: [PATCH 3/6] examples: Don't use LTO for hello_world

On Thu, 9 Mar 2023 at 08:22, Tom Rini wrote: > > If we're building U-Boot with LTO, we don't want to use that for > examples as it's more work than required. > > Signed-off-by: Tom Rini > --- > examples/standalone/Makefile | 4 > 1 file changed, 4 insertions(+) Reviewed-by: Simon Glass

Re: [PATCH v2] nand: brcmnand: add iproc support

On 03/08/2023 01:42 PM, Linus Walleij wrote: Add support for the iproc Broadcom NAND controller, used in Northstar SoCs for example. Based on the Linux driver. Cc: Philippe Reynes Cc: Dario Binacchi Reviewed-by: Michael Trimarchi Signed-off-by: Linus Walleij --- ChangeLog v1->v2: - Check

Re: [PATCH v2] mtd: rawnand: nand_base: Handle algorithm selection

On 03/08/2023 01:28 PM, Linus Walleij wrote: For BRCMNAND with 1-bit BCH ECC (BCH-1) such as used on the D-Link DIR-885L and DIR-890L routers, we need to explicitly select the ECC like this in the device tree: nand-ecc-algo = "bch"; nand-ecc-strength = <1>; nand-ecc-step-size = <512>;

[PATCH v2 2/2] x86: minnowmax: Adjust CONFIG_TEXT_BASE et at

At present U-Boot no longer builds as a complete rom for minnowmaxs since it is too big for the section. Adjust CONFIG_TEXT_BASE and CONFIG_SYS_MONITOR_LEN to allow more code space. Move the MRC cache out of the way too. Add documentation on how to make this change safely. Signed-off-by: Simon

[PATCH v2 1/2] x86: Revert "x86: minnowmax: Adjust CONFIG_TEXT_BASE"

This causes Minnowmax to stop booting. Revert it so we have a good example of how to do this fully. This reverts commit 66e2c665f3b60d726e1c26ad44ac97aa76ead6f5. Signed-off-by: Simon Glass --- (no changes since v1) configs/minnowmax_defconfig | 2 +- 1 file changed, 1 insertion(+), 1

Re: [PATCH v2 02/26] cmd: nvedit: check for ENV_SUPPORT

On Thu, Mar 09, 2023 at 12:45:14PM -0800, Troy Kisky wrote: > On Thu, Mar 9, 2023 at 11:36 AM Tom Rini wrote: > > > On Thu, Mar 09, 2023 at 11:20:33AM -0800, Troy Kisky wrote: > > > On Wed, Mar 1, 2023 at 7:33 AM Tom Rini wrote: > > > > > > > On Fri, Feb 24, 2023 at 10:10:23AM -0800, Troy Kisky

Re: Please pull u-boot-dm

On Fri, Mar 10, 2023 at 12:54:57PM -0800, Simon Glass wrote: > Hi Tom, > > https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/15515 > > Marek pointed out that these are bug fixes and tests so should > probably go into -master. > > The following changes since commit

[PULL] u-boot-usb/master

Two minimal Kconfig/Makefile fixes for USB below. The following changes since commit b08ffdffdce95e267e782366f4a77bf6b5537b28: Merge https://source.denx.de/u-boot/custodians/u-boot-marvell (2023-03-06 14:56:05 -0500) are available in the Git repository at:

[PULL] u-boot-sh/master

Assorted Renesas fixes below, namely MMC clocking breakage fix, clock Kconfig fix, pin control unused symbols removal, and sysinfo fix and enablement on now fixed platforms. The following changes since commit b08ffdffdce95e267e782366f4a77bf6b5537b28: Merge

[PATCH v2 12/12] efI: Allow packaging a kernel in the debugging script

Add an option to package a kernel into the debugging script used for EFI. The name of the kernel must be added to the script. By default it is assumed that the kernel is built in the /tmp/kernel directory. Signed-off-by: Simon Glass --- Changes in v2: - Fix typos in commit message

Please pull u-boot-dm

Hi Tom, https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/15515 Marek pointed out that these are bug fixes and tests so should probably go into -master. The following changes since commit b08ffdffdce95e267e782366f4a77bf6b5537b28: Merge

Re: [PATCH 1/2] CI: Allow job tag to be optionally set globally

On Fri, 10 Mar 2023 at 01:54, Peter Hoyes wrote: > > From: Peter Hoyes > > The default behavior of Gitlab runners is to only run jobs which match > the configured tag, although there is an option to run untagged jobs > [1]. > > To support running the CI in more complex environments where

Re: [PATCH v9 05/10] arm_ffa: introduce armffa command

Hi Abdellatif, On Fri, 10 Mar 2023 at 06:10, Abdellatif El Khlifi wrote: > > Provide armffa command showcasing the use of the FF-A driver > > armffa is a command showcasing how to use the FF-A driver and how to invoke > its operations. This provides a guidance to the client developers on how to

Re: [PATCH v9 09/10] arm_ffa: efi: introduce FF-A MM communication

Hi Abdellatif, On Fri, 10 Mar 2023 at 06:11, Abdellatif El Khlifi wrote: > > Add MM communication support using FF-A transport > > This feature allows accessing MM partitions services through > EFI MM communication protocol. MM partitions such as StandAlonneMM > or smm-gateway secure partitions

Re: [PATCH v9 00/10] introduce Arm FF-A support

Hi Abdellatif, On Fri, 10 Mar 2023 at 06:12, Abdellatif El Khlifi wrote: > > Adding support for Arm FF-A v1.0 (Arm Firmware Framework for Armv8-A) [A]. > > FF-A describes interfaces (ABIs) that standardize communication > between the Secure World and Normal World. These interfaces enable a pair

Re: [PATCH v9 02/10] lib: uuid: introduce uuid_str_to_le_bin function

On Fri, 10 Mar 2023 at 06:10, Abdellatif El Khlifi wrote: > > convert UUID string to little endian binary data > > Signed-off-by: Abdellatif El Khlifi > Cc: Tom Rini > Cc: Simon Glass > Cc: Ilias Apalodimas > Cc: Jens Wiklander > > --- > Changelog: > === > > v9: > > * add a full

Re: [PATCH v1 4/4] test: exit: fix run_commandf() warnings

On Fri, 10 Mar 2023 at 10:55, Evgeny Bachinin wrote: > > Fix warnings after adding printf-like attribute format for > run_commandf(): > warning: too many arguments for format [-Wformat-extra-args] > > Signed-off-by: Evgeny Bachinin > --- > test/cmd/exit.c | 18 +- > 1 file

Re: [PATCH v9 06/10] arm_ffa: introduce the FF-A Sandbox driver

Hi Abdellatif, On Fri, 10 Mar 2023 at 06:10, Abdellatif El Khlifi wrote: > > Provide a Sandbox driver to emulate the FF-A ABIs > > The emulated ABIs are those supported by the FF-A core driver > and according to FF-A specification v1.0. > > The Sandbox driver provides operations allowing the

Re: [PATCH v9 07/10] arm_ffa: introduce Sandbox test cases for UCLASS_FFA

Hi Abdellatif, On Fri, 10 Mar 2023 at 06:10, Abdellatif El Khlifi wrote: > > Add functional test cases for the FF-A core driver > > These tests rely on the FF-A Sandbox driver which helps in > inspecting the FF-A core driver. > > Signed-off-by: Abdellatif El Khlifi > Cc: Tom Rini > Cc: Simon

Re: [PATCH v1 3/4] test: fdt: fix run_commandf() warnings

On Fri, 10 Mar 2023 at 10:55, Evgeny Bachinin wrote: > > Fix warnings both for 32bit and 64bit architecture after adding > printf-like attribute format for run_commandf(): > warning: format ‘%x’ expects argument of type ‘unsigned int’, but > argument 2 has type ‘ulong {aka long unsigned int}’

Re: [PATCH v1 1/4] cli: run_commandf(): small fixups

Hi Evgeny, On Fri, 10 Mar 2023 at 10:55, Evgeny Bachinin wrote: > > * vsnprintf() can truncate cmd, hence it makes no sense to launch such > command (it's broken). Moreover, it's better to signalize to the caller > about such case (for facilitating debugging or bug hunting). > > * Fix kernel-doc

Re: [PATCH 2/2] CI: Allow a mirror to be specified for Docker Hub

On Fri, 10 Mar 2023 at 01:55, Peter Hoyes wrote: > > From: Peter Hoyes > > To conserve bandwidth and potentially avoid rate limits, allow a local > mirror of Docker Hub to be specified globally. The default value is > unchanged. > > Signed-off-by: Peter Hoyes > --- > .gitlab-ci.yml | 3 ++- >

Re: [PATCH v9 08/10] arm_ffa: introduce armffa command Sandbox test

Hi Abdellatif, On Fri, 10 Mar 2023 at 06:11, Abdellatif El Khlifi wrote: > > Add Sandbox test for the armffa command > > Signed-off-by: Abdellatif El Khlifi > Cc: Tom Rini > Cc: Simon Glass > Cc: Ilias Apalodimas > Cc: Jens Wiklander > > --- > Changelog: > === > > v9: align the

Re: [PATCH v1 2/4] unit-test: cover run_commandf() by test-cases

On Fri, 10 Mar 2023 at 10:55, Evgeny Bachinin wrote: > > As run_commandf() is variadic version of run_command() and just a wrapper, > hence apply similar run_command's test-cases. > > Let's avoid warning about empty string passing: > warning: zero-length gnu_printf format string

Re: [PATCH v2 2/3] drivers: pci: sandbox: Add stub sandbox PCI MPS support

On Fri, 10 Mar 2023 at 11:08, wrote: > > From: Stephen Carlson > > Reports the sandbox swapcase PCI Express device to support a 256 byte > Maximum Payload Size for MPS tuning tests. > > Signed-off-by: Stephen Carlson > --- > drivers/misc/swap_case.c | 3 +++ > 1 file changed, 3 insertions(+) >

Re: [PATCH 0/9] binman: Show missing blob message when building U-Boot

Hi Jonas, On Sun, 19 Feb 2023 at 14:02, Jonas Karlman wrote: > > binman currently support showing a helpful missing blob message, but > only when the --allow-missing flag is used. > > This changes so that binman is invoked with the --allow-missing flag > and the helpful message can be shown by

Re: [RFC PATCH] binman: bintool: etype: Add support for ti-secure entry

Hi, On Thu, 9 Mar 2023 at 21:35, Neha Malcom Francis wrote: > > Hi Andrew, Simon > > On 01/03/23 22:41, Andrew Davis wrote: > > On 2/28/23 9:10 AM, Simon Glass wrote: > >> Hi Neha, > >> > >> On Tue, 28 Feb 2023 at 02:50, Neha Malcom Francis > >> wrote: > >>> > >>> Hi Simon, > >>> > >>> On

Re: [PATCH] sysinfo: gpio: fix loop over DT "revisions" array

On Fri, 10 Mar 2023 at 02:58, Rasmus Villemoes wrote: > > There can certainly be a lot more elements in the "revisions" (and > "names") arrays than there are gpios used to form the trinary number > we're searching for; we simply don't know the array size up-front. > > Nor do we need to, because

[PATCH v2 11/12] efi: Support showing tables

Add a command (for the app and payload) to display the tables provided by EFI. Signed-off-by: Simon Glass --- Changes in v2: - Make use of common code cmd/Makefile | 2 +- cmd/efi.c | 33 - doc/usage/cmd/efi.rst | 22 ++

[PATCH v2 10/12] efi: Split out table-listing code into a new file

This code is used with EFI_LOADER but is also useful (with some modifications) for the EFI app and payload. Move it into a shared file. Show the address of the table so it can be examined if needed. Also show the table name as unknown if necessary. Our list of GUIDs is fairly small.

[PATCH v2 09/12] doc: Add help for the efi command

This command currently has no help. Add some. Signed-off-by: Simon Glass --- (no changes since v1) doc/usage/cmd/efi.rst | 197 ++ doc/usage/index.rst | 1 + 2 files changed, 198 insertions(+) create mode 100644 doc/usage/cmd/efi.rst diff --git

[PATCH v2 07/12] efi: Add another tranch of GUIDs

Provide information about the GUIDs supplied by QEMU, so far as it is known. These values are used in the 'efi table' command as well as the printf format string %sU Signed-off-by: Simon Glass --- Changes in v2: - Update commit message to explain why these are needed - Drop EFI_SMBIOS which

[PATCH v2 08/12] efi: Include GUID names with EFI app and payload

These are currently only available when running with EFI_LOADER. Expand this to include the app and payload, since it is useful to be able to decode things there. Signed-off-by: Simon Glass --- Changes in v2: - Add new patch to enable GUID names with EFI app and payload lib/uuid.c | 2 +- 1

[PATCH v2 05/12] x86: Exit EFI boot services before starting kernel

When running the EFI app, we need to exit boot services before jumping to Linux. At some point it may be possible to jump to Linux and pass on the boot services, so that the Linux efistub can be used. For now, this is not implemented. Signed-off-by: Simon Glass --- (no changes since v1)

[PATCH v2 06/12] x86: Support zboot and bootm in the EFI app

These have been disabled due to the rudimentary support available. It is a little better now, so enable these options. Signed-off-by: Simon Glass --- (no changes since v1) configs/efi-x86_app32_defconfig | 2 +- configs/efi-x86_app64_defconfig | 2 +- 2 files changed, 2 insertions(+), 2

[PATCH v2 04/12] x86: Support booting a 64-bit kernel from 64-bit U-Boot

Add the missing code to handle this. For a 64-bit kernel the entry address is 0x200 bytes after the normal entry. Rename the parameter to boot_linux_kernel() accordingly. Update the comments to indicate that these are addresses, not pointers. Signed-off-by: Simon Glass --- (no changes since

[PATCH v2 02/12] x86: Adjust bootparam.h to be more like linux

This likely came from Linux originally, so update it to match v6.2 more. This has no functional change. Signed-off-by: Simon Glass --- (no changes since v1) arch/x86/include/asm/bootparam.h | 70 +--- arch/x86/lib/zimage.c| 2 +- 2 files changed, 47

[PATCH v2 03/12] x86: Add return-value comment to cpu_jump_to_64bit()

This does not mention what it returns. Add the missing documentation. Signed-off-by: Simon Glass --- (no changes since v1) arch/x86/include/asm/cpu.h | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/x86/include/asm/cpu.h b/arch/x86/include/asm/cpu.h index 3346012d335..aa03ef598e1

[PATCH v2 01/12] efi: Set RUN_64BIT correctly for the EFI app

The U-Boot EFI app can run as a 64-bit program, so set the Kconfig correctly in that case. Make sure it doesn't build SPL, since there is no need to switch from 32 to 64 bit when running. Signed-off-by: Simon Glass --- (no changes since v1) arch/x86/Kconfig| 4 ++--

[PATCH v2 00/12] efi: Improvements to booting and debugging

This series provides support for booting a kernel from the EFI app. So far this is pretty basic, since it only supports the actual booting, not providing ACPI tables, etc. But it is a start. It also includes an enhancement to the 'efi' command and a few other clean-ups. Changes in v2: - Update

[PATCH 2/2] buildman: Fix CROSS_COMPILE output for sandbox

The previous attempt at fixing this broke the normal usage of the -A flag. At present, 'buildman -A sandbox' adds the path containing the toolchain. We can assume that this is in the path and we don't want to set CROSS_COMPILE=/bin/ Change this to align with what MakeEnvironment() does, but only

[PATCH 1/2] Revert "buildman: Correct CROSS_COMPILE output for sandbox"

This reverts commit bd0a548ad4a155fec29473d4cc8e135832926973. Signed-off-by: Simon Glass --- tools/buildman/toolchain.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/buildman/toolchain.py b/tools/buildman/toolchain.py index 8f9130bdcdf..688f2e26872 100644 ---

[PATCH v3 13/13] video: Add a note about the broken implementation

The cls command is broken. Previous discussion about this was at [1] and [2]. For now, add a note to the source code. [1] https://patchwork.ozlabs.org/project/uboot/patch/ 20221022092058.106052-1-heinrich.schucha...@canonical.com/ [2] https://patchwork.ozlabs.org/project/uboot/patch/

[PATCH v3 10/13] video: Remove duplicate cursor-positioning function

There are two functions for positioning the cursor on the console. Remove one of them. Signed-off-by: Simon Glass --- (no changes since v1) drivers/video/vidconsole-uclass.c | 44 +++ 1 file changed, 10 insertions(+), 34 deletions(-) diff --git

[PATCH v3 12/13] efi: Add dhrystone, dcache and scroll lines to app

Add these options to provide some performance measurement, see cache status and slightly speed up the appallingly slow console. Signed-off-by: Simon Glass --- (no changes since v1) configs/efi-x86_app64_defconfig | 3 +++ 1 file changed, 3 insertions(+) diff --git

[PATCH v3 08/13] bbinfo: Show the size of the copy framebuffer

If the copy framebuffer is enabled, show its size. Signed-off-by: Simon Glass --- (no changes since v2) Changes in v2: - Add new patch to show the size of the copy framebuffer with bdinfo cmd/bdinfo.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/cmd/bdinfo.c

[PATCH v3 11/13] video: Clear the vidconsole rather than the video

It is better to clear the console device rather than the video device, since the console has the text display. We also need to reset the cursor position with the console, but not with the video device. Add a new function to handle this and update the 'cls' command to use it. Signed-off-by: Simon

[PATCH v3 09/13] efi: Adjust script to show pre-relocation output on terminal

When running with video enabled, the pre-relocation output of U-Boot is currently lost. Add a -serial flag to show it on the terminal. Signed-off-by: Simon Glass --- (no changes since v1) scripts/build-efi.sh | 2 ++ 1 file changed, 2 insertions(+) diff --git a/scripts/build-efi.sh

[PATCH v3 07/13] video: Allow a copy framebuffer with pre-allocated fb

At present it is not possible for the video driver to use a pre-allocated frame buffer (such as is done with EFI) with the copy framebuffer. This can be useful to speed up the display. Adjust the implementation so that copy_size can be set to the required size, with this being allocated if the

[PATCH v3 06/13] efi: Support copy framebuffer

Add support for this to EFI in case it becomes useful. At present it just slows things down. You can enable CONFIG_VIDEO_COPY to turn it on. Signed-off-by: Simon Glass --- Changes in v3: - Add a comment about the fallback Changes in v2: - Obtain copy framebuffer size from EFI instead of using

[PATCH v3 05/13] efi: Use a fixed value for the timer clock

It is not yet clear how to read the timer via EFI. The current value seems much too high on a Framework laptop I tried. Adjust it to a lower hard-coded value for now. Signed-off-by: Simon Glass --- Changes in v3: - Add a comment about the value drivers/timer/tsc_timer.c | 9 + 1 file

[PATCH v3 04/13] x86: Add a few more items to bdinfo

Add the timer and vendor/model information. Signed-off-by: Simon Glass --- Changes in v3: - Show the CPU vendor name also arch/x86/lib/bdinfo.c | 6 ++ cmd/bdinfo.c | 5 + include/init.h| 3 +++ 3 files changed, 14 insertions(+) diff --git a/arch/x86/lib/bdinfo.c

[PATCH v3 03/13] efi: Support a 64-bit frame buffer address

The current vesa structure only provides a 32-bit value for the frame buffer. Many modern machines use an address outside the range. It is still useful to have this common struct, but add a separate frame-buffer address as well. Add a comment for vesa_setup_video_priv() while we are here.

[PATCH v3 02/13] efi: video: Return mode info for app also

The mode info is currently not initialised for the app. Fix this by returning it from the function. Signed-off-by: Simon Glass --- (no changes since v1) drivers/video/efi.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/video/efi.c b/drivers/video/efi.c

[PATCH v3 01/13] efi: video: Move payload code into a function

Put this into a function, as we have done for the app implementation. Comment both functions. FOr now the app still does not access it correctly. Signed-off-by: Simon Glass --- (no changes since v1) drivers/video/efi.c | 83 +++-- 1 file changed, 57

[PATCH v3 00/13] video: efi: Improve the EFI-app video console

This does not work on some 64-bit machines since only a 32-bit address for the framebuffer is supported in the VESA structure. This series corrects this and makes a few other minor, video-related improvements. Changes in v3: - Use Returns instead of @return - Show the CPU vendor name also - Add

[PATCH v2 2/3] drivers: pci: sandbox: Add stub sandbox PCI MPS support

From: Stephen Carlson Reports the sandbox swapcase PCI Express device to support a 256 byte Maximum Payload Size for MPS tuning tests. Signed-off-by: Stephen Carlson --- drivers/misc/swap_case.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/misc/swap_case.c

[PATCH v2 1/3] cmd: pci: Add command to set MPS of all PCIe devices

From: Stephen Carlson Enable tuning of the PCI Express MPS (Maximum Payload Size) of each device. The Maximum Read Request Size is not altered. The SAFE method uses the largest MPS value supported by all devices in the system for each device. This method is the same algorithm as used by Linux

[PATCH v2 3/3] test: Add test for new command pci_mps

From: Stephen Carlson Adds a test for the new pci_mps command to ensure that it can set the Maximum Payload Size (MPS) of all devices to 256 bytes in the sandbox environment. Enables the pci_mps command in the sandbox environment so that this test can be run. Signed-off-by: Stephen Carlson ---

Re: [RFC/PATCH] lib/Kconfig: Enable OF_LIBFDT_OVERLAY by default when FIT is enabled

On Sun, Jan 29, 2023 at 06:30:22PM +0200, Laurent Pinchart wrote: > FIT image support is commonly used to bundle a kernel image, a device > tree, and device tree overlays. Applying overlays requires the > OF_LIBFDT_OVERLAY config option to be set, which lots of boards fail to > select, most

Re: [GIT PULL] Please pull u-boot-mmc master

On Fri, Mar 10, 2023 at 05:28:40PM +0100, Marek Vasut wrote: > On 3/10/23 16:44, Tom Rini wrote: > > On Fri, Mar 10, 2023 at 01:22:15PM +0900, Jaehoon Chung wrote: > > > Dear Tom, > > > > > > > > > Please pull u-boot-mmc master into u-boot master branch. > > > If there is any problem, let me

Re: [PATCH RFC u-boot-mvebu 0/2] arm: mvebu: Fix eMMC boot

On Monday 06 March 2023 12:53:25 Pali Rohár wrote: > On Monday 06 March 2023 11:15:35 Martin Rowe wrote: > > On Sun, 5 Mar 2023 at 16:04, Pali Rohár wrote: > > > > > On Sunday 05 March 2023 12:46:34 Pali Rohár wrote: > > > > On Sunday 05 March 2023 02:24:27 Martin Rowe wrote: > > > > > On Sat, 4

Re: [PATCH v2 1/3] starfive: pci: Add StarFive JH7110 pcie driver

On Friday 10 March 2023 18:36:44 Minda Chen wrote: > On 2023/3/8 15:31, Pali Rohár wrote: > > Hello! See few comments below. > > > > On Wednesday 08 March 2023 13:48:31 Minda Chen wrote: > >> From: Mason Huo > >> > >> Add pcie driver for StarFive JH7110, the driver depends on > >> starfive

[RESEND PATCH v7 09/23] rockchip: pwm: rk_pwm: use reg variable with phys_addr_t size

The fdt_addr_t and phys_addr_t size have been decoupled. A 32bit CPU can expect 64-bit data from the device tree parser, so use a reg variable with phys_addr_t size in the rk_pwm.c file. Signed-off-by: Johan Jonker --- Changed V6: new patch --- drivers/pwm/rk_pwm.c | 2 +- 1 file changed, 1

[PATCH v7 23/23] include: fdtdec: decouple fdt_addr_t and phys_addr_t size

The DT specification supports CPUs with both 32-bit and 64-bit addressing capabilities. In U-boot the fdt_addr_t and phys_addr_t size are coupled by a typedef. The MTD NAND drivers for 32-bit CPU's can describe partitions with a 64-bit reg property. These partitions synced from Linux end up with

[PATCH v7 22/23] arm: stm32mp: spl: fix function with fdt_addr_t input

The fdt_addr_t and phys_addr_t size have been decoupled. A 32bit CPU can expect 64-bit data from the device tree parser, so fix ofnode_get_addr_size function with fdt_addr_t input to be able to handle both sizes for stm32mp SoC in spl.c file. Signed-off-by: Johan Jonker --- Changed V7: remove

[PATCH v7 21/23] drivers: fix debug string with fdt_addr_t input

The fdt_addr_t and phys_addr_t size have been decoupled. A 32bit CPU can expect 64-bit data from the device tree parser, so fix some debug strings with fdt_addr_t to be able to handle both sizes. Signed-off-by: Johan Jonker Reviewed-by: Simon Glass --- Changed V5: new patch ---

[PATCH v7 20/23] drivers: use devfdt_get_addr_ptr when cast to pointer

The fdt_addr_t and phys_addr_t size have been decoupled. A 32bit CPU can expect 64-bit data from the device tree parser, so use devfdt_get_addr_ptr instead of the devfdt_get_addr function in the various files in the drivers directory that cast to a pointer. Signed-off-by: Johan Jonker

[PATCH v7 19/23] drivers: use devfdt_get_addr_index_ptr when cast to pointer

The fdt_addr_t and phys_addr_t size have been decoupled. A 32bit CPU can expect 64-bit data from the device tree parser, so use devfdt_get_addr_index_ptr instead of the devfdt_get_addr_index function in the various files in the drivers directory that cast to a pointer. Signed-off-by: Johan Jonker

[PATCH v7 18/23] drivers: use devfdt_get_addr_size_index_ptr when cast to pointer

The fdt_addr_t and phys_addr_t size have been decoupled. A 32bit CPU can expect 64-bit data from the device tree parser, so use devfdt_get_addr_size_index_ptr instead of the devfdt_get_addr_size_index function in the various files in the drivers directory that cast to a pointer. Signed-off-by:

[PATCH v7 17/23] drivers: use dev_read_addr_ptr when cast to pointer

The fdt_addr_t and phys_addr_t size have been decoupled. A 32bit CPU can expect 64-bit data from the device tree parser, so use dev_read_addr_ptr instead of the dev_read_addr function in the various files in the drivers directory that cast to a pointer. Signed-off-by: Johan Jonker Reviewed-by:

[PATCH v7 16/23] drivers: use dev_read_addr_index_ptr when cast to pointer

The fdt_addr_t and phys_addr_t size have been decoupled. A 32bit CPU can expect 64-bit data from the device tree parser, so use dev_read_addr_index_ptr instead of the dev_read_addr_index function in the various files in the drivers directory that cast to a pointer. Signed-off-by: Johan Jonker

[PATCH v7 15/23] spi: spi-aspeed-smc: use devfdt_get_addr_index_ptr

The fdt_addr_t and phys_addr_t size have been decoupled. A 32bit CPU can expect 64-bit data from the device tree parser, so use devfdt_get_addr_index_ptr and devfdt_get_addr_size_index_ptr function in the spi-aspeed-smc.c file. Also fix dev_dbg to be able to handle both sizes. Signed-off-by:

[PATCH v7 14/23] core: read: add dev_read_addr_index_ptr function

Add dev_read_addr_index_ptr function with the same functionality as dev_read_addr_index, but instead a return pointer is given. Use map_sysmem() function as cast for the return. Make same fix for dev_read_addr_ptr() function. Signed-off-by: Johan Jonker Reviewed-by: Simon Glass --- Changed V6:

[PATCH v7 13/23] core: fdtaddr: add devfdt_get_addr_size_index_ptr function

Add devfdt_get_addr_size_index_ptr function with the same functionality as devfdt_get_addr_size_index, but instead a return pointer is given. Use map_sysmem() function as cast for the return. Make same fix for devfdt_get_addr_index_ptr() function. Suggested-by: Michael Nazzareno Trimarchi

[PATCH v7 12/23] rockchip: rk3288: syscon_rk3288: store syscon platdata in regmap

The Rockchip SoC rk3288 has 2 types of device trees floating around. A 64bit reg size when synced from Linux and a 32bit for U-boot. A pre-probe function in the syscon class driver assumes only 32bit. For other odd reg structures the regmap must be defined in the individual syscon driver. Store

[PATCH v7 11/23] core: remap: fix regmap_init_mem_plat() reg size handeling

The fdt_addr_t and phys_addr_t size have been decoupled. A 32bit CPU can expect 64-bit data from the device tree parser, so convert regmap_init_mem_plat() input to handel both. The syscon class driver also makes use of the regmap_init_mem_plat() function, but has no way of knowing the format of

Re: [PATCH v7 01/23] mtd: nand: raw: rockchip_nfc: use dev_read_addr_ptr

Hi John On Fri, Mar 10, 2023 at 5:40 PM Johan Jonker wrote: > > The fdt_addr_t and phys_addr_t size have been decoupled. > A 32bit CPU can expext 64-bit data from the device tree parser, > so use dev_read_addr_ptr in the rockchip_nfc.c file. > > Signed-off-by: Johan Jonker > Reviewed-by:

[PATCH v7 10/23] include: dm: ofnode: fix headers

When fdt_addr_t and phys_addr_t are split it turns out that the header don't match the functions, so fix the headers. Signed-off-by: Johan Jonker Reviewed-by: Simon Glass Reviewed-by: Kever Yang --- include/dm/ofnode.h | 16 1 file changed, 8 insertions(+), 8 deletions(-)

  1   2   >