Re: [PATCH v10 9/9] riscv: sifive: unmatched: Switch to use binman to generate u-boot.itb

2021-05-19 Thread Bin Meng
Hi Green, On Wed, May 19, 2021 at 11:19 PM Green Wan wrote: > > Update to use binman instead. > > Signed-off-by: Green Wan > --- > arch/riscv/dts/hifive-unmatched-a00-u-boot.dtsi | 1 + > board/sifive/unmatched/Kconfig | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/

Re: [PATCH] efi_loader: capsule: add a debug message in case of no key

2021-05-19 Thread Heinrich Schuchardt
On 5/10/21 10:19 AM, AKASHI Takahiro wrote: It will probably be a common error case that a certificate (public key) is not provided by the system while capsule authentication is enabled. So add a debug message. Signed-off-by: AKASHI Takahiro --- lib/efi_loader/efi_capsule.c | 4 +++- 1 file

[PATCH 1/1] efi_loader: remove weak efi_get_public_key_data()

2021-05-19 Thread Heinrich Schuchardt
Configuring a system with CONFIG_EFI_CAPSULE_AUTHENTICATE=y but without a method to retrieve the public key data is a misconfiguration. We should get a build failure for it. Therefore remove the weak efi_get_public_key_data() implementation. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/

[PULL] u-boot-riscv/master

2021-05-19 Thread Leo Liang
Hi Tom, The following changes since commit 428bec7cf956c3558bbdfda4d2ba23beb73a68ba: Merge branch '2021-05-17-assorted-fixes' (2021-05-18 14:17:54 -0400) are available in the Git repository at: g...@source.denx.de:u-boot/custodians/u-boot-riscv.git for you to fetch changes up to a4691f363

Re: [PATCH] efi_loader: capsule: remove authentication data

2021-05-19 Thread Heinrich Schuchardt
On 5/10/21 10:20 AM, AKASHI Takahiro wrote: If capsule authentication is disabled and yet a capsule file is signed, its signature must be removed from image data to flush. Otherwise, the firmware will be corrupted after update. Fixes: 04be98bd6bcf ("efi: capsule: Add support for uefi capsule

Re: [PATCH 06/18] image: Drop IMAGE_ENABLE_SHA1

2021-05-19 Thread Alex G.
On 5/19/21 4:55 PM, Simon Glass wrote: Hi Alex, On Wed, 19 May 2021 at 11:44, Alex G wrote: On 5/19/21 11:36 AM, Simon Glass wrote: Hi Alexandru, On Mon, 17 May 2021 at 10:38, Alexandru Gagniuc wrote: From: Simon Glass We already have a host Kconfig for SHA1. Use CONFIG_IS_ENABLED

Re: [PATCH] efi_loader: move asm/setjmp.h to efi_loader.h

2021-05-19 Thread Heinrich Schuchardt
On 5/10/21 10:21 AM, AKASHI Takahiro wrote: In the commit c982874e930d ("efi_loader: refactor efi_setup_loaded_image()"), setjmp-related definitions were moved to efi_loaded_image_obj in efi_loader.h. So setjmp.h is no longer refererenced in efi_api.h. This also fixes some error when efi_api.h w

Re: [PATCH v10 9/9] riscv: sifive: unmatched: Switch to use binman to generate u-boot.itb

2021-05-19 Thread Green Wan
Hi Bin, Thanks for the comment. Originally, I wanted to keep the binman change tracked separately. I'll give it a try and create the new patch if it doesn't. Regards, Green On Thu, May 20, 2021 at 9:57 AM Bin Meng wrote: > Hi Green, > > On Wed, May 19, 2021 at 11:19 PM Green Wan wrote: > > >

Re: [PATCH] test/py: efi_capsule: aligned with efidebug syntax changes

2021-05-19 Thread Heinrich Schuchardt
On 5/10/21 10:21 AM, AKASHI Takahiro wrote: In the subject: %s/aligned/align/ After the commit c70f44817d46 ("efi_loader: simplify 'printenv -e'"), "-all" option is no longer necessary. Just remove them in the test script. 'env print -e -all Capsule' leads to an error CMD_RET_USAGE. Caps

Re: [PATCH v2 1/1] sandbox: don't refer to symbol _init

2021-05-19 Thread Rick Chen
> From: U-Boot On Behalf Of Heinrich Schuchardt > Sent: Wednesday, May 19, 2021 6:03 PM > To: Simon Glass > Cc: Ovidiu Panait ; Bin Meng > ; Stefan Roese ; Masahiro Yamada > ; u-boot@lists.denx.de; Heinrich Schuchardt > > Subject: [PATCH v2 1/1] sandbox: don't refer to symbol _init > > GCC pr

Re: [PATCH v10 9/9] riscv: sifive: unmatched: Switch to use binman to generate u-boot.itb

2021-05-19 Thread Bin Meng
Hi Green, On Thu, May 20, 2021 at 11:10 AM Green Wan wrote: > > Hi Bin, > > Thanks for the comment. Originally, I wanted to keep the binman change > tracked separately. I'll give it a try and create the new patch if it doesn't. > Yep, this situation is like previous FU740 PCIe driver. How we or

Re: [PATCH 1/2] net: luton: remove address translation after ofnode_read_resource

2021-05-19 Thread Ramon Fried
On Tue, May 18, 2021 at 4:40 PM Patrick Delaunay wrote: > > Removed call of ofnode_translate_address() after ofnode_read_resource > in luton_switch.c:luton_probe(); it is unnecessary since > the commit feb7ac457c20 ("dm: core: Add address translation in > fdt_get_resource"). > > Fixes: feb7ac457c2

<    1   2