Re: [RFC 1/1] efi_loader: rename 'efidebug boot' 'bootefi'

2022-01-16 Thread Ilias Apalodimas
On Mon, Jan 17, 2022 at 10:16:51AM +0900, AKASHI Takahiro wrote: > On Sat, Jan 15, 2022 at 01:49:07AM +0100, Heinrich Schuchardt wrote: > > The efidebug command was conceived for testing purposes. > > Well, I initially implemented the command as an alternative of > "EFI shell" as the shell was

[PATCH v2] binman: Skip node generation for images read from files

2022-01-16 Thread Jan Kiszka
From: Jan Kiszka We can and should run the node generator only when creating a new image. When we read it back, there is no need to generate nodes - they already exits, and binman does not dive that deep into the image - and there is no way to provide the required fdt-list. So forward the mode

Re: [PATCH v2 01/20] remoteproc: k3_system_controller: Support optional boot_notification channel

2022-01-16 Thread Aswath Govindraju
Hi Tom, On 17/01/22 11:01 am, Aswath Govindraju wrote: > Hi Tom, > > On 13/01/22 7:42 pm, Tom Rini wrote: >> On Tue, Jan 11, 2022 at 01:25:26PM +0530, Aswath Govindraju wrote: >> >>> From: Nishanth Menon >>> >>> If there is an optional boot notification channel that an SoC uses >>> separate

Re: [PATCH v2 11/20] board: ti: j721s2: Add board support for J721S2

2022-01-16 Thread Aswath Govindraju
Hi Tom, On 13/01/22 7:46 pm, Tom Rini wrote: > On Tue, Jan 11, 2022 at 01:25:36PM +0530, Aswath Govindraju wrote: > >> From: David Huang >> >> Add board support for J721S2 SoC. >> >> Signed-off-by: David Huang >> Signed-off-by: Aswath Govindraju > [snip] >> diff --git

Re: [PATCH v2 09/20] ram: k3-ddrss: Add support for J721S2 SoC

2022-01-16 Thread Aswath Govindraju
Hi Tom, On 13/01/22 7:45 pm, Tom Rini wrote: > On Tue, Jan 11, 2022 at 01:25:34PM +0530, Aswath Govindraju wrote: > >> From: David Huang >> >> Add support for DDR subsystem in J721S2 SoC. >> >> Signed-off-by: David Huang >> Signed-off-by: Aswath Govindraju >> --- >> drivers/ram/Kconfig

Re: [PATCH v2 12/20] dt-bindings: ti-serdes-mux: Add defines for J721S2 SoC

2022-01-16 Thread Aswath Govindraju
Hi Tom, On 13/01/22 7:47 pm, Tom Rini wrote: > On Tue, Jan 11, 2022 at 01:25:37PM +0530, Aswath Govindraju wrote: > >> There are 4 lanes in the single instance of J721S2 SERDES. Each SERDES >> lane mux can select upto 4 different IPs. Define all the possible >> functions. >> >> Signed-off-by:

Re: [PATCH v2 01/20] remoteproc: k3_system_controller: Support optional boot_notification channel

2022-01-16 Thread Aswath Govindraju
Hi Tom, On 13/01/22 7:42 pm, Tom Rini wrote: > On Tue, Jan 11, 2022 at 01:25:26PM +0530, Aswath Govindraju wrote: > >> From: Nishanth Menon >> >> If there is an optional boot notification channel that an SoC uses >> separate from the rx path, use the same. >> >> Signed-off-by: Nishanth Menon

[PATCH v1] serial: npcm: Add support for Nuvoton NPCM SoCs

2022-01-16 Thread Stanley Chu
Add Nuvoton BMC NPCM7xx/NPCM8xx uart driver Signed-off-by: Stanley Chu --- drivers/serial/Kconfig | 7 ++ drivers/serial/Makefile | 1 + drivers/serial/serial_npcm.c | 151 +++ 3 files changed, 159 insertions(+) create mode 100644

Re: [PATCH] binman: Skip node generation for images read from files

2022-01-16 Thread Simon Glass
Hi Jan, On Sun, 16 Jan 2022 at 08:51, Jan Kiszka wrote: > > From: Jan Kiszka > > This unbreaks all read-backs of images that contain generator nodes in > their fdtmap. This issue is subtle enough that I think it could use a few lines of explanation. > > Signed-off-by: Jan Kiszka > --- > > I

Re: [PATCH v8 07/12] tools: mkeficapsule: allow for specifying GUID explicitly

2022-01-16 Thread AKASHI Takahiro
On Sat, Jan 01, 2022 at 11:42:45PM +0100, Heinrich Schuchardt wrote: > On 12/20/21 06:02, AKASHI Takahiro wrote: > > The existing options, "--fit" and "--raw," are only used to put a proper > > GUID in a capsule header, where GUID identifies a particular FMP (Firmware > > Management Protocol)

Re: [PATCH v8 06/12] test/py: efi_capsule: add image authentication test

2022-01-16 Thread AKASHI Takahiro
On Sat, Jan 01, 2022 at 11:18:47PM +0100, Heinrich Schuchardt wrote: > On 12/20/21 06:02, AKASHI Takahiro wrote: > > Add a couple of test cases against capsule image authentication > > for capsule-on-disk, where only a signed capsule file with the verified > > signature will be applied to the

Re: [PATCH v8 12/12] (RFC) efi_loader, dts: add public keys for capsules to device tree

2022-01-16 Thread AKASHI Takahiro
On Wed, Jan 12, 2022 at 01:03:54PM -0700, Simon Glass wrote: > Hi Heinrich, > > On Sat, 1 Jan 2022 at 15:53, Heinrich Schuchardt wrote: > > > > On 12/20/21 06:02, AKASHI Takahiro wrote: > > > By specifying CONFIG_EFI_CAPSULE_KEY_PATH, the build process will > > > automatically insert the given

Re: [PATCH] sunxi: call fdt_fixup_ethernet again to set macaddr for more aliases

2022-01-16 Thread Andre Przywara
On Sat, 11 Sep 2021 19:39:16 +0200 Jernej Skrabec wrote: > From: Icenowy Zheng > > Sometimes some ethernet aliases do not exist in U-Boot DT but they > exist in the DT used to boot the system (for example, modified via DT > overlays). In this situation setup_environment is called again in >

Re: [PATCH 1/1] i2c: mvtwsi: Add compatible string for allwinner, sun4i-a10-i2c

2022-01-16 Thread Andre Przywara
On Fri, 7 Jan 2022 11:52:54 -0600 Chris Morgan wrote: Hi Chris, > From: Chris Morgan > > This adds a compatible string for the Allwinner Sun4i-A10 I2C > controller. Without this, boards based on the R8 and A13 (at a > minimum) fail to boot. > > Signed-off-by: Chris Morgan Applied to

Re: [RFC 1/1] efi_loader: rename 'efidebug boot' 'bootefi'

2022-01-16 Thread AKASHI Takahiro
On Sat, Jan 15, 2022 at 01:49:07AM +0100, Heinrich Schuchardt wrote: > The efidebug command was conceived for testing purposes. Well, I initially implemented the command as an alternative of "EFI shell" as the shell was not able to run on EFI U-Boot at that time. > The manipulation of boot

Re: [PATCH] efidebug: avoid 'dfu_alt_info not defined' message

2022-01-16 Thread AKASHI Takahiro
On Sat, Jan 15, 2022 at 02:18:21AM +0100, Heinrich Schuchardt wrote: > If variable dfu_alt_info is not defined duplicate messages are displayed. > > => efidebug boot dump > Scanning disk mmc2.blk... > Scanning disk mmc1.blk... > Scanning disk mmc0.blk... > Found 3 disks >

Re: [PATCH v4 03/13] mmc: fsl: Use brackets around if()

2022-01-16 Thread Jaehoon Chung
On 1/17/22 5:19 AM, Simon Glass wrote: > At present the IS_ENABLED() macro has extra brackets, making it possible > to write: > >if IS_ENABLED(CONFIG_XXX) > > but it is a bit confusing. Add the missing brackets. > > Signed-off-by: Simon Glass Reviewed-by: Jaehoon Chung Best Regards,

[GIT PULL] Please pull u-boot-pmic master

2022-01-16 Thread Jaehoon Chung
Dear Tom, Please pull u-boot-pmic master into u-boot master branch. If there is a problem, let me know, plz Best Regards, Jaehoon Chung CI: https://source.denx.de/u-boot/custodians/u-boot-pmic/-/pipelines/10653 The following changes since commit d71dbe657c3c22f50ecb3df133e5cdd884daae7d:

Re: [PATCH] mkimage: fix segfault on MacOS arm64

2022-01-16 Thread Sergey V. Lobanov
Thanks a lot for your review, I’ve sent PATCH v2 with the changes related to your comments https://lists.denx.de/pipermail/u-boot/2022-January/472133.html > On 11 Jan 2022, at 22:42, Jessica Clarke wrote: > > On 2 Dec 2021, at 22:16, Sergey V. Lobanov wrote: >> >> mkimage segfaults due

Re: [BUG] sandbox: NO_SDL=1 gcc: error: arch/sandbox/cpu/sdl.o: No such file or directory

2022-01-16 Thread Milan P . Stanić
On Sun, 2022-01-16 at 23:41, Heinrich Schuchardt wrote: > On 1/16/22 22:14, Milan P. Stanić wrote: > > wrote: > > > On 1/13/22 14:41, Simon Glass wrote: > > > > Hi Heinrich, > > > > > > > > On Mon, 10 Jan 2022 at 16:22, Heinrich Schuchardt > > > > wrote: > > > > > Hello Simon, > > > > > > > >

[PATCH V2] mkimage: fix segfault on MacOS arm64

2022-01-16 Thread Sergey V. Lobanov
mkimage segfaults due ASLR mechasim on MacOS arm64 It is required to use _dyld_get_image_vmaddr_slide() to prevent segfault on MacOS arm64 This patch ased on the discussion https://github.com/u-boot/u-boot/commit/3b142045e8a7f0ab17b6099e9226296af45967d0 Thanks to Ronny Kotzschmar and ptpt52

Re: [BUG] sandbox: NO_SDL=1 gcc: error: arch/sandbox/cpu/sdl.o: No such file or directory

2022-01-16 Thread Heinrich Schuchardt
On 1/16/22 22:14, Milan P. Stanić wrote: wrote: On 1/13/22 14:41, Simon Glass wrote: Hi Heinrich, On Mon, 10 Jan 2022 at 16:22, Heinrich Schuchardt wrote: Hello Simon, compiling with SDL fails on Alpine Linux: https://gitlab.alpinelinux.org/alpine/aports/-/issues/13411 So I tried NO_SDL:

Re: [BUG] sandbox: NO_SDL=1 gcc: error: arch/sandbox/cpu/sdl.o: No such file or directory

2022-01-16 Thread Milan P . Stanić
On Sat, 2022-01-15 at 19:47, Heinrich Schuchardt wrote: > On 1/13/22 14:41, Simon Glass wrote: > > Hi Heinrich, > > > > On Mon, 10 Jan 2022 at 16:22, Heinrich Schuchardt > > wrote: > > > > > > Hello Simon, > > > > > > compiling with SDL fails on Alpine Linux: > > >

[PATCH v4 01/13] imx: Don't define __ASSEMBLY__ in C files

2022-01-16 Thread Simon Glass
This is supposed to be a build-system flag. Move it there so we can define it before linux/kconfig.h is included. Signed-off-by: Simon Glass --- (no changes since v1) arch/arm/mach-imx/Makefile| 2 +- arch/arm/mach-imx/imx8m/imximage-8mm-lpddr4.cfg |

[PATCH v4 00/13] kconfig: Add some new macros

2022-01-16 Thread Simon Glass
This series adds a few more macros to kconfig, along with a bit of refactoring of what is there. It also adds some basic tests for the kconfig macros. Finally, to make the tests work, it enhances buildman to support changing CONFIG options on the fly when building. Changes in v4: - Update stale

[PATCH 1/1] drivers: octeon: get rid of Unicode in code

2022-01-16 Thread Heinrich Schuchardt
Placing Unicode control codes in the middle of a comment does not make much sense. Let's get rid of all Unicode in drivers/ram/octeon/octeon3_lmc.c. Signed-off-by: Heinrich Schuchardt --- drivers/ram/octeon/octeon3_lmc.c | 34 1 file changed, 17 insertions(+),

RE: [PATCH v2] imx8mq_evk: configs: add/cleanup variables for distro boot

2022-01-16 Thread ZHIZHIKIN Andrey
Hello Stefano, > -Original Message- > From: U-Boot On Behalf Of Andrey Zhizhikin > Sent: Tuesday, January 11, 2022 11:50 AM > To: u-boot@lists.denx.de > Cc: sba...@denx.de; alice@nxp.com; an...@akkea.ca; feste...@gmail.com; > h...@denx.de; jh80.ch...@samsung.com; peng@nxp.com;

[PATCH v2] imx8mq_evk: configs: add/cleanup variables for distro boot

2022-01-16 Thread Andrey Zhizhikin
Add fdt_addr_r fdtfile which used by distro boot, and cleanup legacy environment variables. Signed-off-by: Andrey Zhizhikin --- Changes in V2: - Correct config variable name, it should be SYS_LOAD_ADDR include/configs/imx8mq_evk.h | 8 1 file changed, 4 insertions(+), 4 deletions(-)

Re: [PATCH v4 03/13] mmc: fsl: Use brackets around if()

2022-01-16 Thread Sean Anderson
On 1/16/22 3:19 PM, Simon Glass wrote: At present the IS_ENABLED() macro has extra brackets, making it possible to write: if IS_ENABLED(CONFIG_XXX) but it is a bit confusing. Add the missing brackets. Signed-off-by: Simon Glass --- (no changes since v1) drivers/mmc/fsl_esdhc_imx.c |

[PATCH v4 10/13] buildman: Add helper functions for updating .config files

2022-01-16 Thread Simon Glass
At present the only straightforward way to write tests that need a slightly different configuration is to create a new board with its own configuration. This is cumbersome. It would be useful if buildman could adjust the configuration of a build on the fly. In preparation for this, add a utility

[PATCH v4 13/13] test: Add some tests for kconfig.h

2022-01-16 Thread Simon Glass
The macros in this file are a little confusing and we currently have no tests to check that they work as expected. Add some tests which check the macros in C code. Add a few tests which check that the build errors are generated correctly too, using buildman's -a option. Signed-off-by: Simon

[PATCH v4 12/13] buildman: Allow adjusting board config on the fly

2022-01-16 Thread Simon Glass
Add a -a option to specify changes to the config before the build commences. For example buildman -a ~CONFIG_CMDLINE disables CONFIG_CMDLINE before doing the build. This makes it easier to try things out as well as to write tests without creating a new board or manually manging the .config

[PATCH v4 07/13] patman: Update test_util to run doc tests

2022-01-16 Thread Simon Glass
At present this function does not run the doctests. Allow the caller to pass these modules in as strings. Update patman to use this. Signed-off-by: Simon Glass --- (no changes since v1) tools/dtoc/main.py| 2 +- tools/patman/main.py | 20

[PATCH v4 11/13] buildman: Provide a hint on how to debug thread crashes

2022-01-16 Thread Simon Glass
If a thread crashes it is helpful to try the operation again with threading disabled. Add a hint about that. Signed-off-by: Simon Glass --- (no changes since v1) tools/buildman/builderthread.py | 2 +- tools/buildman/func_test.py | 4 +++- 2 files changed, 4 insertions(+), 2 deletions(-)

[PATCH v4 08/13] buildman: Add a flag to control the traceback

2022-01-16 Thread Simon Glass
At present the full horror of the Python traceback is shown by default. It is normally only useful for debugging. Turn it off by default and add a --debug flag to enable it. Signed-off-by: Simon Glass --- (no changes since v1) tools/buildman/cmdline.py | 2 ++ tools/buildman/main.py| 3

[PATCH v4 09/13] buildman: Make use of test_util

2022-01-16 Thread Simon Glass
Use test_util to run the tests, with the ability to select a single test to run, if desired. Signed-off-by: Simon Glass --- (no changes since v1) tools/buildman/main.py | 26 +++--- 1 file changed, 11 insertions(+), 15 deletions(-) diff --git a/tools/buildman/main.py

[PATCH v4 05/13] kconfig: Add support for conditional values

2022-01-16 Thread Simon Glass
At present if an optional Kconfig value needs to be used it must be bracketed by #ifdef. For example, with this Kconfig setup: config WIBBLE bool "Support wibbles, the world needs more wibbles" config WIBBLE_ADDR hex "Address of the wibble" depends on WIBBLE then the

[PATCH v4 06/13] bloblist: Update to use conditional value

2022-01-16 Thread Simon Glass
Use the new IF_ENABLED_INT() feature to avoid needing our own inline function to handle this case. Tidy up the logic to ensure that the value is only used when present. Update the 'expected' comment also. Signed-off-by: Simon Glass --- (no changes since v3) Changes in v3: - Add new patch to

[PATCH v4 04/13] kconfig: Update IS_ENABLED() internals

2022-01-16 Thread Simon Glass
The config_enabled() macro currently uses 0 as the default value. Update it to allow any value, so we can pass it something else, such as a non-existent function, to produce a build error if it is not defined. Also tidy up the code style for IS_ENABLED() and drop the unnecessary brackets (the

[PATCH v4 03/13] mmc: fsl: Use brackets around if()

2022-01-16 Thread Simon Glass
At present the IS_ENABLED() macro has extra brackets, making it possible to write: if IS_ENABLED(CONFIG_XXX) but it is a bit confusing. Add the missing brackets. Signed-off-by: Simon Glass --- (no changes since v1) drivers/mmc/fsl_esdhc_imx.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH v4 02/13] mips: Avoid using config_enabled() directly

2022-01-16 Thread Simon Glass
Use IS_ENABLED() instead, which is the correct macro for checking a CONFIG option. Signed-off-by: Simon Glass --- (no changes since v1) arch/mips/lib/cache.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/mips/lib/cache.c b/arch/mips/lib/cache.c index

[PATCH v3 4/4] rockchip: sdhci: Add HS400 Enhanced Strobe support for RK3568

2022-01-16 Thread Alper Nebi Yasak
On RK3568, a register bit must be set to enable Enhanced Strobe. However, it appears that the address of this register may differ from vendor to vendor and should be read from the underlying MMC IP. Let the Rockchip SDHCI driver read this address and set the relevant bit when Enhanced Strobe

[PATCH v3 3/4] rockchip: sdhci: Add HS400 Enhanced Strobe support for RK3399

2022-01-16 Thread Alper Nebi Yasak
On RK3399, a register bit must be set to enable Enhanced Strobe. Let the Rockchip SDHCI driver set it when Enhanced Strobe configuration is requested. However, having it set makes the lower-speed modes stop working and makes reinitialization fail, so let it be unset as needed in set_control_reg().

[PATCH v3 2/4] rockchip: sdhci: Fix RK3399 eMMC PHY power cycling

2022-01-16 Thread Alper Nebi Yasak
The Rockchip RK3399 eMMC PHY has to be power-cycled while changing its clock speed to some higher speeds. This is dependent on the desired SDHCI clock speed, and it looks like the PHY should be powered off while setting the SDHCI clock in these cases. Commit ac804143cfd1 ("mmc: rockchip_sdhci:

[PATCH v3 1/4] mmc: sdhci: Add HS400 Enhanced Strobe support

2022-01-16 Thread Alper Nebi Yasak
Delegate setting the Enhanced Strobe configuration to individual drivers if they set a function for it. Return -ENOTSUPP if they do not, like what the MMC uclass does. Signed-off-by: Alper Nebi Yasak Reviewed-by: Jaehoon Chung --- (no changes since v2) Changes in v2: - Add tag: "Reviewed-by:

[PATCH v3 0/4] rockchip: sdhci: Fix reinit and add HS400 Enhanced Strobe support

2022-01-16 Thread Alper Nebi Yasak
My rk3399-gru-kevin has some problems with the eMMC. The board can boot to U-Boot proper with the eMMC working at a low speed, but trying to reinitialize it with "mmc dev 0" or "mmc rescan" makes it unusable. If the HS400 mode is enabled, it times out while executing tuning and doesn't even start

[PATCH 1/1] efi_loader: printing TCG2 protocol GUID

2022-01-16 Thread Heinrich Schuchardt
We support the TCG2 protocol. Allow command efidebug to print it. Signed-off-by: Heinrich Schuchardt --- This is a follow-up to [PATCH 0/8] efi_loader: simplify printing GUIDs https://lists.denx.de/pipermail/u-boot/2022-January/472092.html --- include/efi_api.h | 4 include/efi_tcg2.h | 4

Re: [PATCH] spl: add support for custom boot method names

2022-01-16 Thread Michael Walle
Am 2022-01-13 16:22, schrieb Heiko Thiery: Currently the names MMC1, MMC2 and MMC2_2 are output in the SPL. To achieve more userbility here the name of the boot source can be returned. E.g. for "MMC1" -> "eMMC" or "MMC2" -> "SD card". Signed-off-by: Heiko Thiery Tested-by: Michael Walle

Re: [RFC] nvme: Split out PCI support

2022-01-16 Thread Mark Kettenis
> Date: Tue, 11 Jan 2022 11:16:54 -0500 > From: Tom Rini > > On Fri, Dec 31, 2021 at 09:00:20PM +0100, Mark Kettenis wrote: > > > Apple SoCs have an integrated NVMe controller that isn't connected > > over a PCIe bus. In preparation for adding support for this NVMe > > controller, split out the

[PATCH 2/2] input: apple: Add support for Apple SPI keyboard

2022-01-16 Thread Mark Kettenis
This driver adds support for the keyboard on Apple Silicon laptops. The controller for this keyboard sits on an SPI bus and uses an Apple-specific HID over SPI protocol. The packets sent by this controller for key presses and key releases are fairly simple and are decoded directly by the code in

[PATCH 1/2] spi: apple: Add driver for Apple SPI controller

2022-01-16 Thread Mark Kettenis
Add a driver for the SPI controller integrated on Apple SoCs. This is necessary to support the keyboard on Apple Silicon laopts since their keyboard uses an Apple-specific HID over SPI protocol. Signed-off-by: Mark Kettenis --- arch/arm/Kconfig| 2 + drivers/spi/Kconfig | 7 +

[PATCH 0/2] Apple M1 laptop keyboard support

2022-01-16 Thread Mark Kettenis
This short series implements support for the laptop keyboards on the Apple Silicon Mac laptops. These keyboards are connected to an SPI bus, so this adds a driver for the SPI controller integrated on the SoC, as well as the driver for the keyboard itself. This makes it possible to interact with

[PATCH] binman: Skip node generation for images read from files

2022-01-16 Thread Jan Kiszka
From: Jan Kiszka This unbreaks all read-backs of images that contain generator nodes in their fdtmap. Signed-off-by: Jan Kiszka --- I tried to write some test case as well, but the testsuite is too fragile and too non-intuitive to me to extend it. E.g., just adding a fdtmap to

[PATCH 7/8] efi_loader: user %pUs for printing GUIDs

2022-01-16 Thread Heinrich Schuchardt
For printing GUIDs with macro EFI_ENTRY use %pUs instead of %pUl to provide readable debug output. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_boottime.c | 26 +- lib/efi_loader/efi_capsule.c | 6 +++--- lib/efi_loader/efi_esrt.c | 6

[PATCH 8/8] cmd: printenv: simplify printing GUIDs

2022-01-16 Thread Heinrich Schuchardt
Use "%pS" to print text representations of GUIDs. Signed-off-by: Heinrich Schuchardt --- cmd/nvedit_efi.c | 39 ++- lib/uuid.c | 24 2 files changed, 26 insertions(+), 37 deletions(-) diff --git a/cmd/nvedit_efi.c

[PATCH 5/8] test: add test for %pUs

2022-01-16 Thread Heinrich Schuchardt
Add a unit test for the %pUs printf code. Use ut_asserteq_str() for checking string results. Signed-off-by: Heinrich Schuchardt --- test/print_ut.c | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/test/print_ut.c b/test/print_ut.c index

[PATCH 6/8] cmd: efidebug: simplify printing GUIDs

2022-01-16 Thread Heinrich Schuchardt
Use "%pS" to print text representations of GUIDs. Signed-off-by: Heinrich Schuchardt --- cmd/efidebug.c | 160 ++--- include/efi_api.h | 8 +++ include/efi_dt_fixup.h | 4 -- include/efi_rng.h | 4 -- lib/uuid.c | 116

[PATCH 3/8] disk: simplify part_print_efi()

2022-01-16 Thread Heinrich Schuchardt
Use printf code %pUs to print the text representation of the partition type GUID. Signed-off-by: Heinrich Schuchardt --- disk/part_efi.c | 21 - 1 file changed, 8 insertions(+), 13 deletions(-) diff --git a/disk/part_efi.c b/disk/part_efi.c index 3809333078..94e2930200

[PATCH 4/8] sandbox: imply PARTITION_TYPE_GUID

2022-01-16 Thread Heinrich Schuchardt
CONFIG_PARTITION_TYPE_GUID=y is needed for testing some GPT related functionality. Signed-off-by: Heinrich Schuchardt --- arch/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/Kconfig b/arch/Kconfig index ee32e8366e..ffb8f5c8ec 100644 --- a/arch/Kconfig +++ b/arch/Kconfig @@

[PATCH 2/8] lib: printf code %pUs for GUID text representation

2022-01-16 Thread Heinrich Schuchardt
In different places text representations are used for GUIDs, e.g. * command efidebug * command part list for GPT partitions To allow reducing code duplication introduce a new printf code %pUs. It will call uuid_guid_get_str() to get a text representation. If none is found it will fallback to

[PATCH 1/8] lib: compile uuid_guid_get_str if CONFIG_LIB_UUID=y

2022-01-16 Thread Heinrich Schuchardt
Currently uuid_guid_get_str() is only built if CONFIG_PARTITION_TYPE_GUID=y. To make it usable for other GUIDs compile it if CONFIG_LIB_UUID=y. The linker will take care of removing it if it is unused. Signed-off-by: Heinrich Schuchardt --- lib/uuid.c | 4 ++-- 1 file changed, 2 insertions(+),

[PATCH 0/8] efi_loader: simplify printing GUIDs

2022-01-16 Thread Heinrich Schuchardt
In different places text representations are used for GUIDs, e.g. * command efidebug * command printenv -e * command part list for GPT partitions Introduce a new printf code %pUs and use it to deduplicate the coding. Heinrich Schuchardt (8): lib: compile uuid_guid_get_str if

[PATCH 1/1] efi_selftest: implement printing GUIDs

2022-01-16 Thread Heinrich Schuchardt
The ESRT test may try to print a GUID if an error occurs. Implement the %pU print code. Correct the ESRT test to use %pU instead of %pUl to avoid the output of character 'l'. Signed-off-by: Heinrich Schuchardt --- lib/efi_selftest/efi_selftest_console.c | 25 +

[BUG] U-Boot selftest for ESRT table: %pUl

2022-01-16 Thread Heinrich Schuchardt
Hello Jose, you wrote the following lines: lib/efi_selftest/efi_selftest_esrt.c:124: efi_st_error("ESRT field mismatch for entry with fw_class=%pUl\n", lib/efi_selftest/efi_selftest_esrt.c:131: efi_st_error("ESRT field mismatch for entry with fw_class=%pUl\n",