Re: [PATCH v3 00/19] binman: Simple templating feature and mkimage conversion

2023-07-09 Thread Jan Kiszka
On 10.07.23 04:40, Simon Glass wrote: > This series converts the mkimage entry type to be a section, i.e. based on > the entry_Section class. This makes it more consistent in its behaviour, > e.g. allowing symbol writing and expanded entries. > > A simple templating feature is also introduced, to

[PATCH v3 19/19] binman: Support writing symbols inside a mkimage image

2023-07-09 Thread Simon Glass
Add support for writing symbols and determining the assumed position of binaries inside a mkimage image. This is useful as an example for other entry types which might want to do the same thing. Signed-off-by: Simon Glass --- Changes in v3: - Add new patch to support writing symbols inside a

[PATCH v3 18/19] binman: Support templates at any level

2023-07-09 Thread Simon Glass
Allow templates to be used inside a section, not just in the top-level /binman node. Signed-off-by: Simon Glass --- (no changes since v1) tools/binman/control.py| 5 ++- tools/binman/ftest.py | 8 tools/binman/test/289_template_section.dts | 52

[PATCH v3 17/19] binman: Add a test for templating in a FIT

2023-07-09 Thread Simon Glass
Add this as a separate test case. Signed-off-by: Simon Glass --- Changes in v3: - Add new test case for templating in a FIT tools/binman/ftest.py | 7 + tools/binman/test/288_template_fit.dts | 37 ++ 2 files changed, 44 insertions(+) create mode

[PATCH v3 16/19] binman: Support templating with multiple images

2023-07-09 Thread Simon Glass
Allow a template to appear in the top level description when using multiple images. Signed-off-by: Simon Glass --- Changes in v3: - Drop duplicate dts-v1 header tools/binman/control.py | 5 +++-- tools/binman/ftest.py| 12 +++

[PATCH v3 15/19] binman: Support simple templates

2023-07-09 Thread Simon Glass
Collections can used to collect the contents of other entries into a single entry, but they result in a single entry, with the original entries 'left behind' in their old place. It is useful to be able to specific a set of entries ones and have it used in multiple images, or parts of an image.

[PATCH v3 14/19] dtoc: Allow inserting a list of nodes into another

2023-07-09 Thread Simon Glass
Provide a way to specify a phandle list of nodes which are to be inserted into an existing node. Signed-off-by: Simon Glass --- Changes in v3: - Adjust to use the new example file tools/dtoc/fdt.py | 19 +++ tools/dtoc/test/dtoc_test_copy.dts | 13

[PATCH v3 13/19] dtoc: Support copying the contents of a node into another

2023-07-09 Thread Simon Glass
This permits implementation of a simple templating system, where a node can be reused as a base for others. For now this adds new subnodes after any existing ones. Signed-off-by: Simon Glass --- Changes in v3: - Add a new devicetree file especially for node copying - Correct logic for merging

[PATCH v3 12/19] binman: Correct handling of zero bss size

2023-07-09 Thread Simon Glass
Fix the check for the __bss_size symbol, since it may be 0. Unfortunately there was no test coverage for this. Signed-off-by: Simon Glass --- (no changes since v1) tools/binman/elf_test.py | 5 + tools/binman/etype/u_boot_spl_bss_pad.py | 2 +-

[PATCH v3 11/19] binman: Drop __bss_size variable in bss_data.c

2023-07-09 Thread Simon Glass
This is not needed since the linker script sets it up. Drop the variable to avoid confusion. Fix the prototype for main() while we are here. Signed-off-by: Simon Glass --- (no changes since v1) tools/binman/test/bss_data.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH v3 09/19] binman: Convert mkimage to Entry_section

2023-07-09 Thread Simon Glass
From: Marek Vasut This is needed to handle mkimage with inner section located itself in a section. Signed-off-by: Marek Vasut Use BuildSectionData() instead of ObtainContents(), add tests and a few other minor fixes: Signed-off-by: Simon Glass --- Changes in v3: - Fix up some tests which now

[PATCH v3 10/19] binman: Provide a way to specify the fdt-list directly

2023-07-09 Thread Simon Glass
Sometimes multiple boards are built with binman and it is useful to specify a different FDT list for each. At present this is not possible without providing multiple values of the of-list entryarg (which is not supported in the U-Boot build system). Allow a fit,fdt-list-val string-list property

[PATCH v3 07/19] binman: Update elf to return number of written symbols

2023-07-09 Thread Simon Glass
Update the LookupAndWriteSymbols() function to return the number of symbols written. Also add some logging for when debugging is not enabled. Signed-off-by: Simon Glass --- Changes in v3: - Add new patch for elf to return number of written symbols tools/binman/elf.py | 13 +++--

[PATCH v3 08/19] binman: Add more detail on how ObtainContents() works

2023-07-09 Thread Simon Glass
This area of binman can be a bit confusing. Add some more comments to help. Signed-off-by: Simon Glass --- Changes in v3: - Add new patch with more detail on how ObtainContents() works tools/binman/entry.py | 3 +++ tools/binman/etype/section.py | 32 +++-

[PATCH v3 06/19] stm32mp15: Avoid writing symbols in SPL

2023-07-09 Thread Simon Glass
These boards use SPL in a mkimage entry and apparently access the symbol containing the image position of U-Boot, but put U-Boot in another image. This means that binman is unable to fill in the symbol correctly in the SPL binary. This doesn't matter at present since mkimage doesn't support

[PATCH v3 05/19] binman: Allow disabling symbol writing

2023-07-09 Thread Simon Glass
Some boards don't use symbol writing but do access the symbols in SPL. Provide an option to work around this. Signed-off-by: Simon Glass --- (no changes since v1) tools/binman/binman.rst | 7 ++ tools/binman/entry.py | 4 +++-

[PATCH v3 03/19] binman: Use GetEntries() to obtain section contents

2023-07-09 Thread Simon Glass
Some section types don't have a simple _entries list. Use the GetEntries() method in GetEntryContents() and other places to handle this. This makes the behaviour more consistent. Signed-off-by: Simon Glass --- (no changes since v1) tools/binman/etype/section.py | 18 +- 1

[PATCH v3 04/19] binman: Read _multiple_data_files in the correct place

2023-07-09 Thread Simon Glass
Move this to the ReadEntries() function where it belongs. Signed-off-by: Simon Glass --- (no changes since v1) tools/binman/etype/mkimage.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/binman/etype/mkimage.py b/tools/binman/etype/mkimage.py index

[PATCH v3 01/19] binman: Correct coverage gap in control

2023-07-09 Thread Simon Glass
Add a pragma to deal with the code-coverage gap which drops binman down to 90% coverage. Fixes: de65b122a25 (tools: Fall back to importlib_resources on Python 3.6) Signed-off-by: Simon Glass --- (no changes since v1) tools/binman/control.py | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH v3 02/19] binman: Init align_default in entry_Section

2023-07-09 Thread Simon Glass
This should be set up in the init function, to avoid a warning about a property not set up there. Fix it. Signed-off-by: Simon Glass --- (no changes since v1) tools/binman/etype/section.py | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/binman/etype/section.py

[PATCH v3 00/19] binman: Simple templating feature and mkimage conversion

2023-07-09 Thread Simon Glass
This series converts the mkimage entry type to be a section, i.e. based on the entry_Section class. This makes it more consistent in its behaviour, e.g. allowing symbol writing and expanded entries. A simple templating feature is also introduced, to reduce duplication when a set of entries must

Re: [PATCH 0/4] introduce EFI_RAM_DISK_PROTOCOL

2023-07-09 Thread AKASHI Takahiro
On Mon, Jul 10, 2023 at 11:13:12AM +0900, Masahisa Kojima wrote: > On Fri, 7 Jul 2023 at 18:12, AKASHI Takahiro > wrote: > > > > On Fri, Jul 07, 2023 at 05:19:33PM +0900, Masahisa Kojima wrote: > > > Hi Akashi-san, > > > > > > On Fri, 7 Jul 2023 at 16:16, AKASHI Takahiro > > > wrote: > > > > >

Re: [PATCH 0/4] introduce EFI_RAM_DISK_PROTOCOL

2023-07-09 Thread Masahisa Kojima
On Fri, 7 Jul 2023 at 18:12, AKASHI Takahiro wrote: > > On Fri, Jul 07, 2023 at 05:19:33PM +0900, Masahisa Kojima wrote: > > Hi Akashi-san, > > > > On Fri, 7 Jul 2023 at 16:16, AKASHI Takahiro > > wrote: > > > > > > On Fri, Jul 07, 2023 at 08:29:12AM +0200, Heinrich Schuchardt wrote: > > > > On

Re: [PATCH 07/10] test: dm: add SCMI base protocol test

2023-07-09 Thread AKASHI Takahiro
On Fri, Jul 07, 2023 at 11:35:49AM -0600, Simon Glass wrote: > Hi, > > On Tue, 4 Jul 2023 at 03:35, AKASHI Takahiro > wrote: > > > > Hi Simon, > > > > On Mon, Jul 03, 2023 at 02:30:57PM +0100, Simon Glass wrote: > > > Hi, > > > > > > On Mon, 3 Jul 2023 at 01:57, AKASHI Takahiro > > > wrote: >

[GIT PULL] Please pull fsl-qoirq-2023-7-6 for next

2023-07-09 Thread Peng Fan
Hi Tom, Please pull fsl-qoriq-2023-7-6 for next - Enable DM Serial for ls1043ardb and ls1046ardb/afrwy Fixed secure boot on LS-CH2 platforms -

Re: [PATCH 08/10] cmd: add scmi command for SCMI firmware

2023-07-09 Thread AKASHI Takahiro
On Fri, Jul 07, 2023 at 11:35:52AM -0600, Simon Glass wrote: > Hi, > > On Tue, 4 Jul 2023 at 02:26, AKASHI Takahiro > wrote: > > > > On Mon, Jul 03, 2023 at 02:30:54PM +0100, Simon Glass wrote: > > > Hi, > > > > > > On Mon, 3 Jul 2023 at 01:55, AKASHI Takahiro > > > wrote: > > > > > > > > On

[BUG] fdt_pack_reg in common/fdt_support.c can cause crash from unaligned access

2023-07-09 Thread David Virag
Hi, I'm trying to port U-Boot to a new board (Samsung JACKPOTLTE, ARMv8, Exynos7885) but when CONFIG_ARCH_FIXUP_FDT_MEMORY is enabled, the bootm command leads to an unaligned memory access, which results in a synchronous abort. After a long debugging session, I concluded that fdt_pack_reg in

[PATCH] common: Kconfig: Fix CMD_BMP/BMP dependency

2023-07-09 Thread Samuel Dionne-Riel
Using `default y` will not select BMP when CMD_BMP has been enabled, if it was already configured. By using `select`, if `CMD_BMP` is turned on, it will force the presence of `BMP`. Fixes: 072b0e16c482114d242580dd7a3197db5966705f Signed-off-by: Samuel Dionne-Riel --- cmd/Kconfig| 1 +

[PATCH v2 6/6] arm: mvebu: Remove unused alias from RC AC5X dts

2023-07-09 Thread Chris Packham
The sar-reg0 alias was left over from an earlier iteration of the patches adding support for this board. Remove the unused alias. Fixes: 6cc8b5db40 ("arm: mvebu: Add RD-AC5X board") Signed-off-by: Chris Packham --- arch/arm/dts/ac5-98dx35xx-rd.dts | 1 - 1 file changed, 1 deletion(-) diff

[PATCH v2 5/6] arm: mvebu: Add Allied Telesis x240 board

2023-07-09 Thread Chris Packham
The x240 and SE240 are a series of L2+ switches from Allied Telesis. There are a number of them in the range but as far as U-Boot is concerned all the CPU block components are the same so there's only one board defined. Signed-off-by: Chris Packham --- Notes: Changes in v2: - drop

[PATCH v2 4/6] mtd: nand: pxa3xx: Enable devbus/nand arbiter on Armada 8K

2023-07-09 Thread Chris Packham
The CN9130 SoC (an ARMADA 8K type) has both a NAND Flash Controller and a generic local bus controller (Device Bus Controller) that share common pins. With a board design that incorporates both a NAND flash and uses the Device Bus (in our case for an SRAM) accessing the Device Bus device fails

[PATCH v2 3/6] mtd: nand: pxa3xx: Add support for the Marvell AC5 SoC

2023-07-09 Thread Chris Packham
The NAND flash controller (NFC) on the AC5/AC5X SoC is the same as the NFC used on other Marvell SoCs. It does have the additional restriction of only supporting SDR timing modes up to 3. Signed-off-by: Chris Packham --- drivers/mtd/nand/raw/pxa3xx_nand.c | 17 ++--- 1 file changed,

[PATCH v2 2/6] arm: mvebu: ac5: Define mvebu_get_nand_clock()

2023-07-09 Thread Chris Packham
The NF_CLK for the AC5 SoC runs at 400MHz. There's no strapping or gating require so just add a mvebu_get_nand_clock() that returns this value. Signed-off-by: Chris Packham --- arch/arm/mach-mvebu/alleycat5/soc.c | 6 ++ 1 file changed, 6 insertions(+) diff --git

[PATCH v2 1/6] arm: mvebu: ac5: Add nand-controller node

2023-07-09 Thread Chris Packham
The AC5/AC5X SoC has a NAND flash controller. Add this to the SoC device tree. Signed-off-by: Chris Packham --- arch/arm/dts/ac5-98dx25xx.dtsi | 9 + 1 file changed, 9 insertions(+) diff --git a/arch/arm/dts/ac5-98dx25xx.dtsi b/arch/arm/dts/ac5-98dx25xx.dtsi index

[PATCH v2 0/6] Support for AC5X NAND and AT-x240 board

2023-07-09 Thread Chris Packham
This series adds support for the NAND flash controller on the AC5X SoC and adds support for the Allied Telesis x240 board. I've also included 2 unrelated changes. "arm: mvebu: Remove unused alias from RC AC5X dts" removes an unused alias from the dts. This was in the neighborhood of the x240 so I

Re: [bug report] sunxi: H6: no ethernet on Orange Pi One Plus

2023-07-09 Thread Anne Macedo
On Sun, Jul 09, 2023 at 04:32:22PM +, Anne Macedo wrote: > On Tue, Jul 04, 2023 at 11:22:29PM +, Anne Macedo wrote: > > Hey! > > > > I'm trying to bake Linux images for the Orange Pi One Plus using Yocto. > > Everything works fine, except for Ethernet. > > > > On the u-boot prompt: > >

Re: [bug report] sunxi: H6: no ethernet on Orange Pi One Plus

2023-07-09 Thread Anne Macedo
On Tue, Jul 04, 2023 at 11:22:29PM +, Anne Macedo wrote: > Hey! > > I'm trying to bake Linux images for the Orange Pi One Plus using Yocto. > Everything works fine, except for Ethernet. > > On the u-boot prompt: > > => dhcp > No ethernet found. > > After adding: > > CONFIG_SPL_SPI_SUNXI=y

Re: Pull request efi-2023-07-rc7

2023-07-09 Thread Tom Rini
On Sun, Jul 09, 2023 at 12:38:46PM +0200, Heinrich Schuchardt wrote: > Dear Tom, > > The following changes since commit 0beb649053b86b2cfd5cf55a0fc68bc2fe91a430: > > MAINTAINERS: correct at91 tree link (2023-07-07 11:37:09 -0400) > > are available in the Git repository at: > >

RE: [PATCH 1/2] cmd: thordown: Add proper dependency for CMD_THOR_DOWNLOAD

2023-07-09 Thread Soma, Ashok Reddy
Hi Heinrich, > -Original Message- > From: Heinrich Schuchardt > Sent: Sunday, July 9, 2023 7:09 PM > To: Soma, Ashok Reddy ; u- > b...@lists.denx.de > Cc: s...@chromium.org; ilias.apalodi...@linaro.org; rfried@gmail.com; > seanedm...@microsoft.com; tob...@waldekranz.com;

Re: [PATCH v3 02/11] capsule: authenticate: Add capsule public key in platform's dtb

2023-07-09 Thread Heinrich Schuchardt
Am 9. Juli 2023 15:33:17 MESZ schrieb Sughosh Ganu : >The EFI capsule authentication logic in u-boot expects the public key >in the form of an EFI Signature List(ESL) to be provided as part of >the platform's dtb. Currently, the embedding of the ESL file into the >dtb needs to be done manually.

Re: [PATCH 1/2] cmd: thordown: Add proper dependency for CMD_THOR_DOWNLOAD

2023-07-09 Thread Heinrich Schuchardt
Am 9. Juli 2023 15:09:57 MESZ schrieb Ashok Reddy Soma : >When CONFIG_CMD_USB and CONFIG_USB are disabled some compilation errors >are seen as below. Thanks for your patch. Currently we have no documentation for the thordown command. We should create a man page in /docs/usage/cmd/. Do you

[PATCH v3 11/11] sandbox: capsule: Generate capsule related files through binman

2023-07-09 Thread Sughosh Ganu
The EFI capsule files can now be generated as part of u-boot build. This is done through binman. Add capsule entry nodes in the u-boot.dtsi for the sandbox architecture for generating the capsules. Remove the corresponding generation of capsules from the capsule update conftest file. The capsules

[PATCH v3 10/11] sandbox: capsule: Add a config file for generating capsules

2023-07-09 Thread Sughosh Ganu
Support has been added to the mkeficapsule tool to generate capsules by parsing the capsule parameters through a config file. Add a config file for generating capsules. These capsules will be used for testing the capsule update feature on sandbox platform. Enable generation of capsules through

[PATCH v3 09/11] test: capsule: Remove public key embed logic from capsule update test

2023-07-09 Thread Sughosh Ganu
The embedding of the public key EFI Signature List(ESL) file into the platform's DTB is now done at the time of u-boot build. Remove this logic from the capsule update test' configuration. Include the public key for the sandbox and sandbox_flattree variant as part of the build. Signed-off-by:

[PATCH v3 08/11] test: py: Setup capsule files for testing

2023-07-09 Thread Sughosh Ganu
Support has being added through earlier commits to build capsules and embed the public key needed for capsule authentication as part of u-boot build. >From the testing point-of-view, this means the input files needed for the above have to be setup before invoking the build. Set this up in the

[PATCH v3 07/11] CI: capsule: Setup the files needed for capsule update testing

2023-07-09 Thread Sughosh Ganu
Support has being added through earlier commits to build capsules and embed the public key needed for capsule authentication as part of u-boot build. >From the testing point-of-view, this means the input files needed for generating the above have to be setup before invoking the build. Set this up

[PATCH v3 06/11] binman: capsule: Add support for generating capsules

2023-07-09 Thread Sughosh Ganu
Add support in binman for generating capsules. The capsule parameters can be specified either through a config file or through the capsule binman entry. Signed-off-by: Sughosh Ganu --- Changes since V2: * New patch which generates capsules through binman replacing the earlier make target.

[PATCH v3 05/11] doc: Add documentation to describe capsule config file format

2023-07-09 Thread Sughosh Ganu
The UEFI capsule can be generated either through command-line parameters, or, by specifying those in a config file. Add documentation to describe the format of the config file. Signed-off-by: Sughosh Ganu --- Changes since V2: None doc/develop/uefi/uefi.rst | 64

[PATCH v3 04/11] tools: mkeficapsule: Add support for parsing capsule params from config file

2023-07-09 Thread Sughosh Ganu
Add support for specifying the parameters needed for capsule generation through a config file, instead of passing them through command-line. Parameters for more than a single capsule file can be specified, resulting in generation of multiple capsules through a single invocation of the command.

[PATCH v3 02/11] capsule: authenticate: Add capsule public key in platform's dtb

2023-07-09 Thread Sughosh Ganu
The EFI capsule authentication logic in u-boot expects the public key in the form of an EFI Signature List(ESL) to be provided as part of the platform's dtb. Currently, the embedding of the ESL file into the dtb needs to be done manually. Add a signature node in the u-boot dtsi file and include

[PATCH v3 03/11] doc: capsule: Document the new mechanism to embed ESL file into dtb

2023-07-09 Thread Sughosh Ganu
Update the document to specify how the EFI Signature List(ESL) file can be embedded into the platform's dtb as part of the u-boot build. Signed-off-by: Sughosh Ganu --- Changes since V2: * Highlight the need to use the u-boot.dtsi file for embedding the public key ESL into the DTB.

[PATCH v3 01/11] nuvoton: npcm845-evb: Add a newline at the end of file

2023-07-09 Thread Sughosh Ganu
Add a newline at the end of the dts, without which the build fails when including the u-boot.dtsi file. Signed-off-by: Sughosh Ganu --- Changes since V2: * New patch arch/arm/dts/nuvoton-npcm845-evb.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v3 00/11] Integrate EFI capsule tasks into u-boot's build flow

2023-07-09 Thread Sughosh Ganu
This patchset aims to bring two capsule related tasks under the u-boot build flow. One is the embedding of the public key into the platform's dtb. The public key is in the form of an EFI Signature List(ESL) file and is used for capsule authentication. This is being achieved by adding the

[PATCH 1/2] cmd: thordown: Add proper dependency for CMD_THOR_DOWNLOAD

2023-07-09 Thread Ashok Reddy Soma
When CONFIG_CMD_USB and CONFIG_USB are disabled some compilation errors are seen as below. cmd/thordown.o: in function `usb_gadget_initialize': include/linux/usb/gadget.h:981: undefined reference to `board_usb_init' cmd/thordown.o: in function `do_thor_down': cmd/thordown.c:68: undefined

[PATCH 2/2] zynqmp: config: Add proper dependencies for USB

2023-07-09 Thread Ashok Reddy Soma
When CONFIG_CMD_USB and CONFIG_USB are disabled, still some compilation errors are seen as below. In file included from include/configs/xilinx_zynqmp.h:173, from include/config.h:3, from include/common.h:16, from env/common.c:10:

[PATCH 0/2] Fix dependencies of USB Kconfig options

2023-07-09 Thread Ashok Reddy Soma
When USB device driver CONFIG_USB and CONFIG_CMD_USB are disabled, some compilation issues are seen. Also CMD_THOR_DOWNLOAD should depend on CONFIG_CMD_USB. Add dependencies to resolve those issues and compile properly. Also remove unused config CONFIG_ZYNQMP_USB. Ashok Reddy Soma (2): cmd:

Re: [PATCH v3 u-boot] mmc: spl: Make partition choice in default_spl_mmc_emmc_boot_partition() more explicit

2023-07-09 Thread Pali Rohár
On Saturday 08 July 2023 11:28:34 Tom Rini wrote: > On Sat, Jul 08, 2023 at 10:30:47AM +0200, Pali Rohár wrote: > > On Friday 07 July 2023 19:43:34 Tom Rini wrote: > > > On Sat, Jul 08, 2023 at 12:54:38AM +0200, Pali Rohár wrote: > > > > > > > To make eMMC partition choosing in

Re: imx8mp: issues getting dtb with pcie enabled to boot

2023-07-09 Thread Fabio Estevam
On Fri, Jul 7, 2023 at 8:48 PM Sahaj Sarup wrote: > For now the fix seems to be `clocks = <_blk_ctrl 0>;` Please submit the fix to Linux. > this gets pcie working under linux but not under u-boot, i'm guessing pcie > for imx8 is not yet implemented? That's correct.

[PATCH v2] x86: Update docs link in bootparam header

2023-07-09 Thread Paul Barker
After Linux commit ff61f0791ce9, x86 documentation was moved to arch/x86 and the link in bootparam.h was broken. Signed-off-by: Paul Barker --- arch/x86/include/asm/bootparam.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/include/asm/bootparam.h

Re: [PATCH] x86: Update docs link in bootparam header

2023-07-09 Thread Paul Barker
On 09/07/2023 02:47, Heinrich Schuchardt wrote: On 7/7/23 17:41, Tom Rini wrote: On Fri, Jul 07, 2023 at 07:51:42AM +0100, Paul Barker wrote: After Linux commit ff61f0791ce9, x86 documentation was moved to arch/x86 and the link in bootparam.h was broken. Signed-off-by: Paul Barker ---  

Pull request efi-2023-07-rc7

2023-07-09 Thread Heinrich Schuchardt
Dear Tom, The following changes since commit 0beb649053b86b2cfd5cf55a0fc68bc2fe91a430: MAINTAINERS: correct at91 tree link (2023-07-07 11:37:09 -0400) are available in the Git repository at: https://source.denx.de/u-boot/custodians/u-boot-efi.git tags/efi-2023-07-rc7 for you to fetch

Re: [PATCH v2] x86: Update docs link in bootparam header

2023-07-09 Thread Heinrich Schuchardt
On 7/9/23 11:44, Paul Barker wrote: After Linux commit ff61f0791ce9, x86 documentation was moved to arch/x86 and the link in bootparam.h was broken. Signed-off-by: Paul Barker Reviewed-by: Heinrich Schuchardt --- arch/x86/include/asm/bootparam.h | 2 +- 1 file changed, 1 insertion(+),

Re: [PATCH] efi_loader: make efi_delete_handle() follow the EFI spec

2023-07-09 Thread Heinrich Schuchardt
On 6/26/23 12:04, Ilias Apalodimas wrote: The EFI doesn't allow removal of handles, unless all hosted protocols are cleanly removed. Our efi_delete_handle() is a bit intrusive. Although it does try to delete protocols before removing a handle, it doesn't care if that fails. Instead it only

Re: [PATCH 2/4 v2] efi_loader: check the status of disconnected drivers

2023-07-09 Thread Heinrich Schuchardt
On 6/20/23 08:19, Ilias Apalodimas wrote: efi_uninstall_protocol() calls efi_disconnect_all_drivers() but never checks the return value. Instead it tries to identify protocols that are still open after closing the ones that were opened with EFI_OPEN_PROTOCOL_BY_HANDLE_PROTOCOL,

Re: [PATCH 1/4 v2] efi_loader: reconnect drivers on failure

2023-07-09 Thread Heinrich Schuchardt
On 6/20/23 08:19, Ilias Apalodimas wrote: efi_disconnect_controller() doesn't reconnect drivers in case of failure. Reconnect the disconnected drivers properly Signed-off-by: Ilias Apalodimas Reviewed-by: Heinrich Schuchardt

Re: [PATCH v10 3/4] Boot var automatic management for removable medias

2023-07-09 Thread Heinrich Schuchardt
On 6/19/23 23:23, Raymond Mao wrote: Changes for complying to EFI spec §3.5.1.1 'Removable Media Boot Behavior'. Boot variables can be automatically generated during a removable media is probed. At the same time, unused boot variables will be detected and removed. Please note that currently the

Re: [PATCH v1] mkeficapsule: fix efi_firmware_management_capsule_header data type

2023-07-09 Thread Heinrich Schuchardt
On 6/16/23 10:28, Stefan Herbrechtsmeier wrote: From: Malte Schmidt The data type of item_offset_list shall be UINT64 according to the UEFI [1] specifications. In include/efi_api.h the correct data type is used. The bug was probably never noticed because of little endianness. [1]