[PATCH v3 6/6] configs: Select OF_LIBFDT_OVERLAY to hand over DTO via bloblist

2025-07-18 Thread Raymond Mao
Select OF_LIBFDT_OVERLAY for qemu_arm64 and vexpress_fvp_bloblist to hand over DTO via bloblist. Signed-off-by: Raymond Mao --- Changes in v3: - Initial patch. configs/qemu_arm64_defconfig| 1 + configs/vexpress_fvp_bloblist_defconfig | 1 + 2 files changed, 2 insertions(+) diff

[PATCH v3 5/6] fdtdec: apply DT overlays from bloblist

2025-07-18 Thread Raymond Mao
. [1] Add Transfer Entry for Devicetree Overlay https://github.com/FirmwareHandoff/firmware_handoff/pull/74 Signed-off-by: Raymond Mao --- Changes in v2: - Add more details into the commit message. - Check and compare the top-level compatible string of DT/DTO. - Check the reserved space in the base

[PATCH v3 4/6] bloblist: add API for applying blobs with specified tag

2025-07-18 Thread Raymond Mao
. [1] Firmware Handoff spec: https://github.com/FirmwareHandoff/firmware_handoff Signed-off-by: Raymond Mao --- Changes in v2: - Add inline header into the blob payload. - Add arg 'size' to 'bloblist_apply_blobs()' for more general purpose. Changes in v3: - Add including errno.h.

[PATCH v3 3/6] bloblist: fix a potential negative size for memmove

2025-07-18 Thread Raymond Mao
It causes a panic when blob is shrunk and 'new_alloced' is less than 'next_ofs'. The data area that needs to be moved should end up at 'hdr->used_size'. Fixes: 1fe59375498f ("bloblist: Support resizing a blob") Signed-off-by: Raymond Mao --- Changes in

[PATCH v3 2/6] bloblist: add helper functions

2025-07-18 Thread Raymond Mao
Add two helper functions for: 1. marking a blob void 2. getting blob record from a given blob data pointer. Signed-off-by: Raymond Mao --- Changes in v2: - None. Changes in v3: - None. common/bloblist.c | 13 + 1 file changed, 13 insertions(+) diff --git a/common/bloblist.c b

[PATCH v3 1/6] bloblist: add blob type for DT overlay

2025-07-18 Thread Raymond Mao
Add blob type for DT overlay according to the update of Firmware Handoff spec[1]. Add an inline header to represent the 'subtype' in a DT overlay blob payload. [1] Add Transfer Entry for Devicetree Overlay https://github.com/FirmwareHandoff/firmware_handoff/pull/74 Signed-off-by: R

[PATCH v3 0/6] Add support for DT overlays handoff

2025-07-18 Thread Raymond Mao
Devicetree Overlay https://github.com/FirmwareHandoff/firmware_handoff/pull/74 [2] https://github.com/raymo200915/optee_manifest/blob/dt_overlay_handoff/qemu_v8.xml Raymond Mao (6): bloblist: add blob type for DT overlay bloblist: add helper functions bloblist: fix a potential negative size f

Re: [PATCH v2 0/6] Add support for DT overlays handoff

2025-07-11 Thread Raymond Mao
Hi Tom, On Fri, 11 Jul 2025 at 11:45, Tom Rini wrote: > > On Fri, Jul 04, 2025 at 06:41:59AM -0700, Raymond Mao wrote: > > The series include refactoring on bloblist and fdtdec to support handoff > > of multiple DT overlays and applying them into the DT base during setup. &

Re: [PATCH v2 6/6] fdtdec: apply DT overlays from bloblist

2025-07-11 Thread Raymond Mao
Hi Tom, On Fri, 11 Jul 2025 at 11:45, Tom Rini wrote: > > On Fri, Jul 04, 2025 at 06:42:05AM -0700, Raymond Mao wrote: > > > During FDT setup, apply all existing DT overlays from the bloblist > > to the base FDT if bloblist is being used for handoff from previous > > b

[PATCH v2 6/6] fdtdec: apply DT overlays from bloblist

2025-07-04 Thread Raymond Mao
/firmware_handoff/pull/74 Signed-off-by: Raymond Mao --- Changes in v2: - Add more details into the commit message. - Check and compare the top-level compatible string of DT/DTO. - Check the reserved space in the base DT region, and skip resizing if the space is sufficient. - Move the DT space

[PATCH v2 5/6] bloblist: add API for applying blobs with specified tag

2025-07-04 Thread Raymond Mao
. [1] Firmware Handoff spec: https://github.com/FirmwareHandoff/firmware_handoff Signed-off-by: Raymond Mao --- Changes in v2: - Add inline header into the blob payload. - Add arg 'size' to 'bloblist_apply_blobs()' for more general purpose. com

[PATCH v2 4/6] bloblist: expose 'expand_by' as an output argument

2025-07-04 Thread Raymond Mao
following blobs of the FDT, its address will be shifted after FDT blob resizing. Thus, returning 'expand_by' helps the caller to get the shifted data pointer when it comes to get the overlay for applying. Signed-off-by: Raymond Mao --- Changes in v2: - None. common/bloblist.c

[PATCH v2 3/6] bloblist: fix a potential negative size for memmove

2025-07-04 Thread Raymond Mao
It causes a panic when blob is shrunk and 'new_alloced' is less than 'next_ofs'. The data area that needs to be moved should end up at 'hdr->used_size'. Fixes: 1fe59375498f ("bloblist: Support resizing a blob") Signed-off-by: Raymond Mao --- Changes i

[PATCH v2 2/6] bloblist: add helper functions

2025-07-04 Thread Raymond Mao
Add two helper functions for: 1. marking a blob void 2. getting blob record from a given blob data pointer. Signed-off-by: Raymond Mao --- Changes in v2: - None. common/bloblist.c | 13 + 1 file changed, 13 insertions(+) diff --git a/common/bloblist.c b/common/bloblist.c index

[PATCH v2 1/6] bloblist: add blob type for DT overlay

2025-07-04 Thread Raymond Mao
Add blob type for DT overlay according to the update of Firmware Handoff spec[1]. Add an inline header to represent the 'subtype' in a DT overlay blob payload. [1] Add Transfer Entry for Devicetree Overlay https://github.com/FirmwareHandoff/firmware_handoff/pull/74 Signed-off-by: R

[PATCH v2 0/6] Add support for DT overlays handoff

2025-07-04 Thread Raymond Mao
Devicetree Overlay https://github.com/FirmwareHandoff/firmware_handoff/pull/74 [2] https://github.com/raymo200915/optee_manifest/blob/dt_overlay_handoff/qemu_v8.xml Raymond Mao (6): bloblist: add blob type for DT overlay bloblist: add helper functions bloblist: fix a potential negative size f

Re: [PATCH 1/3] mbedtls: Introduce subset of compiler-rt from LLVM

2025-06-20 Thread Raymond Mao
Hi Tom, On Fri, 20 Jun 2025 at 15:00, Tom Rini wrote: > > On Fri, Jun 20, 2025 at 02:00:09PM -0400, Raymond Mao wrote: > > Hi Tom, > > > > On Fri, 20 Jun 2025 at 13:20, Tom Rini wrote: > > > > > > On Fri, Jun 20, 2025 at 08:10:08AM -0700, Raymond Mao w

Re: [PATCH 1/3] mbedtls: Introduce subset of compiler-rt from LLVM

2025-06-20 Thread Raymond Mao
Hi Tom, On Fri, 20 Jun 2025 at 13:20, Tom Rini wrote: > > On Fri, Jun 20, 2025 at 08:10:08AM -0700, Raymond Mao wrote: > > Introduce the implementation of udivti3 and udivmodti4 from LLVM > > compiler-rt builtins for building MbedTLS bignum module via Clang. > > > &

[PATCH 3/3] MAINTAINERS: Add myself into the list for MbedTLS

2025-06-20 Thread Raymond Mao
Add myself into the list for MbedTLS. Signed-off-by: Raymond Mao --- MAINTAINERS | 5 + 1 file changed, 5 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index d62dd35a385..d52624342f3 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1282,6 +1282,11 @@ T: git git://github.com/ARM

[PATCH 2/3] mbedtls: Add compiler-rt subset into the MbedTLS build

2025-06-20 Thread Raymond Mao
Add compiler-rt subset into the MbedTLS build for Clang to solve the link error due to missing '__udivti3' when RSA_PUBLIC_KEY_PARSER_MBEDTLS is selected. Fixes: 13de8483388c ("mbedtls: add mbedtls into the build system") Signed-off-by: Raymond Mao ---

[PATCH 1/3] mbedtls: Introduce subset of compiler-rt from LLVM

2025-06-20 Thread Raymond Mao
Introduce the implementation of udivti3 and udivmodti4 from LLVM compiler-rt builtins for building MbedTLS bignum module via Clang. Signed-off-by: Raymond Mao --- lib/mbedtls/port/compiler-rt/LICENSE.TXT | 311 ++ lib/mbedtls/port/compiler-rt/README.txt | 11

Re: [PATCH 6/6] fdtdec: apply DT overlays from bloblist

2025-06-20 Thread Raymond Mao
Hi Michal, On Fri, 20 Jun 2025 at 02:26, Michal Simek wrote: > > Hi, > > On 6/19/25 18:22, Raymond Mao wrote: > > Hi Michal, > > > > On Thu, 19 Jun 2025 at 07:39, Michal Simek wrote: > >> > >> > >> > >> On 6/18/25 16:59, Raymond

Re: [PATCH 6/6] fdtdec: apply DT overlays from bloblist

2025-06-19 Thread Raymond Mao
Hi Michal, On Thu, 19 Jun 2025 at 07:39, Michal Simek wrote: > > > > On 6/18/25 16:59, Raymond Mao wrote: > > During FDT setup, apply all existing DT overlays from the bloblist > > to the base FDT if bloblist is being used for handoff from previous > > boot sta

Re: [PATCH 1/6] bloblist: add blob type for DT overlay

2025-06-19 Thread Raymond Mao
Hi Tom, On Wed, 18 Jun 2025 at 18:04, Tom Rini wrote: > > On Wed, Jun 18, 2025 at 03:53:15PM -0400, Raymond Mao wrote: > > Hi Tom, > > > > On Wed, 18 Jun 2025 at 13:37, Tom Rini wrote: > > > > > > On Wed, Jun 18, 2025 at 07:59:23AM -0700, Raymond Mao

Re: [PATCH 1/6] bloblist: add blob type for DT overlay

2025-06-18 Thread Raymond Mao
Hi Tom, On Wed, 18 Jun 2025 at 13:37, Tom Rini wrote: > > On Wed, Jun 18, 2025 at 07:59:23AM -0700, Raymond Mao wrote: > > > Add blob type for DT overlay according to the update of Firmware > > Handoff spec[1]. > > > > [1] https://github.com/Firmwar

Re: [PATCH 4/6] bloblist: expose 'expand_by' as an output argument

2025-06-18 Thread Raymond Mao
Hi Tom, On Wed, 18 Jun 2025 at 13:40, Tom Rini wrote: > > On Wed, Jun 18, 2025 at 07:59:26AM -0700, Raymond Mao wrote: > > > Some advanced usecases of bloblist needs to know the actual bytes > > expanded or shrunk after blob resizing. > > For example, to support applyi

[PATCH 5/6] bloblist: add API for applying blobs with specified tag

2025-06-18 Thread Raymond Mao
Add an API to search for the blobs with specified tag and use the hook function to apply the blob data. Signed-off-by: Raymond Mao --- common/bloblist.c | 36 include/bloblist.h | 19 +++ 2 files changed, 55 insertions(+) diff --git a

[PATCH 6/6] fdtdec: apply DT overlays from bloblist

2025-06-18 Thread Raymond Mao
During FDT setup, apply all existing DT overlays from the bloblist to the base FDT if bloblist is being used for handoff from previous boot stage. Signed-off-by: Raymond Mao --- lib/fdtdec.c | 56 1 file changed, 56 insertions(+) diff --git

[PATCH 4/6] bloblist: expose 'expand_by' as an output argument

2025-06-18 Thread Raymond Mao
following blobs of the FDT, its address will be shifted after FDT blob resizing. Thus, returning 'expand_by' helps the caller to get the shifted data pointer when it comes to get the overlay for applying. Signed-off-by: Raymond Mao --- common/bloblist.c | 16 ---

[PATCH 2/6] bloblist: add helper functions

2025-06-18 Thread Raymond Mao
Add two helper functions for: 1. marking a blob void 2. getting blob record from a given blob data pointer. Signed-off-by: Raymond Mao --- common/bloblist.c | 13 + 1 file changed, 13 insertions(+) diff --git a/common/bloblist.c b/common/bloblist.c index 1c690f58b56..488908f605e

[PATCH 3/6] bloblist: fix a potential negative size for memmove

2025-06-18 Thread Raymond Mao
It causes a panic when blob is shrunk and 'new_alloced' is less than 'next_ofs'. The data area that needs to be moved should end up at 'hdr->used_size'. Fixes: 1fe59375498f ("bloblist: Support resizing a blob") Signed-off-by: Raymond Mao --- common/blo

[PATCH 0/6] Add support for DT overlays handoff

2025-06-18 Thread Raymond Mao
$ make toolchains $ make ARM_FIRMWARE_HANDOFF=y all Copy and rename your DT overlay blob as 'qemu_v8.dtb' into out/bin $ make ARM_FIRMWARE_HANDOFF=y run-only [1] https://github.com/raymo200915/optee_manifest/blob/dt_overlay_handoff/qemu_v8.xml Raymond Mao (6): bloblist: add blob t

[PATCH 1/6] bloblist: add blob type for DT overlay

2025-06-18 Thread Raymond Mao
Add blob type for DT overlay according to the update of Firmware Handoff spec[1]. [1] https://github.com/FirmwareHandoff/firmware_handoff/pull/74 Signed-off-by: Raymond Mao --- common/bloblist.c | 1 + include/bloblist.h | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a

Re: [PATCH v2] makefile: fix Clang link error when RSA_PUBLIC_KEY_PARSER is selected

2025-06-06 Thread Raymond Mao
Hi Jerome, On Fri, 6 Jun 2025 at 02:58, Jerome Forissier wrote: > > > > On 6/6/25 00:10, Raymond Mao wrote: > > Hi Tom, > > > > On Thu, 5 Jun 2025 at 17:46, Tom Rini wrote: > >> > >> On Thu, Jun 05, 2025 at 04:52:20PM -0400, Raymond Mao wrote: &g

Re: [PATCH v2] makefile: fix Clang link error when RSA_PUBLIC_KEY_PARSER is selected

2025-06-05 Thread Raymond Mao
Hi Tom, On Thu, 5 Jun 2025 at 17:46, Tom Rini wrote: > > On Thu, Jun 05, 2025 at 04:52:20PM -0400, Raymond Mao wrote: > > Hi Tom, > > > > On Thu, 5 Jun 2025 at 16:01, Tom Rini wrote: > > > > > > On Thu, Jun 05, 2025 at 03:45:00PM -0400, Raymond Mao wro

Re: [PATCH v2] makefile: fix Clang link error when RSA_PUBLIC_KEY_PARSER is selected

2025-06-05 Thread Raymond Mao
Hi Tom, On Thu, 5 Jun 2025 at 16:01, Tom Rini wrote: > > On Thu, Jun 05, 2025 at 03:45:00PM -0400, Raymond Mao wrote: > > Hi Tom, > > > > On Thu, 5 Jun 2025 at 13:15, Tom Rini wrote: > > > > > > On Tue, Jun 03, 2025 at 07:42:55AM -0700, Raymond Mao

Re: [PATCH v2] makefile: fix Clang link error when RSA_PUBLIC_KEY_PARSER is selected

2025-06-05 Thread Raymond Mao
Hi Tom, On Thu, 5 Jun 2025 at 13:15, Tom Rini wrote: > > On Tue, Jun 03, 2025 at 07:42:55AM -0700, Raymond Mao wrote: > > > MbedTLS bignum module needs '__udivti3' which is a 128-bit division > > function provided by the compiler runtime, typically libgcc for G

[PATCH v2] sandbox: solve undefined reference to pthread_kill symbol

2025-06-04 Thread Raymond Mao
o.0: error adding symbols: DSO missing from command line collect2: error: ld returned 1 exit status [...] Fixes: b989f9ed9fe1 ("test: lib: add initjmp() test") Signed-off-by: Raymond Mao --- Changes in v2: - update fixes tag in commit message. arch/sandbox/config.mk | 2 +- 1 file change

Re: [PATCH] sandbox: solve undefined reference to pthread_kill symbol

2025-06-04 Thread Raymond Mao
Hi Tom, On Wed, 4 Jun 2025 at 09:56, Tom Rini wrote: > > On Wed, Jun 04, 2025 at 03:33:31PM +0530, Anshul Dalal wrote: > > Hi Raymond, > > > > On Wed Jun 4, 2025 at 2:30 AM IST, Raymond Mao wrote: > > > Hi Tom, > > > > > > On Tue, 3 Jun 2025 a

Re: [PATCH] sandbox: solve undefined reference to pthread_kill symbol

2025-06-03 Thread Raymond Mao
Hi Tom, On Tue, 3 Jun 2025 at 15:23, Tom Rini wrote: > > On Tue, Jun 03, 2025 at 02:00:23PM -0400, Raymond Mao wrote: > > Hi Tom, > > > > On Tue, 3 Jun 2025 at 11:47, Tom Rini wrote: > > > > > > On Tue, Jun 03, 2025 at 08:13:58AM -0700, Raymond Mao w

Re: [PATCH] sandbox: solve undefined reference to pthread_kill symbol

2025-06-03 Thread Raymond Mao
Hi Tom, On Tue, 3 Jun 2025 at 11:47, Tom Rini wrote: > > On Tue, Jun 03, 2025 at 08:13:58AM -0700, Raymond Mao wrote: > > > This patch is to solve the sandbox building error: > > $ make O=build-sandbox -s sandbox_defconfig > > $ make O=build-sandbox -s -j2 &

[PATCH] sandbox: solve undefined reference to pthread_kill symbol

2025-06-03 Thread Raymond Mao
o.0: error adding symbols: DSO missing from command line collect2: error: ld returned 1 exit status [...] Signed-off-by: Raymond Mao --- arch/sandbox/config.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/sandbox/config.mk b/arch/sandbox/config.mk index dd9b7473fa9..9

[PATCH v2] makefile: fix Clang link error when RSA_PUBLIC_KEY_PARSER is selected

2025-06-03 Thread Raymond Mao
DCOMPILER_RT_BUILD_LIBFUZZER=OFF \ -DCOMPILER_RT_BUILD_PROFILE=OFF \ -DCMAKE_INSTALL_PREFIX= \ -DLLVM_ENABLE_PER_TARGET_RUNTIME_DIR=OFF ninja builtins sudo ninja install Fixes: 13de8483388c ("mbedtls: add mbedtls into the build system") Signed-off-by: Raymond Mao --- Changes in V2: - Add suppor

Re: [PATCH] makefile: fix Clang link error when RSA_PUBLIC_KEY_PARSER is selected

2025-05-29 Thread Raymond Mao
Hi Tom, Jerome, On Wed, 28 May 2025 at 11:46, Raymond Mao wrote: > > Hi Jerome, > > On Wed, 28 May 2025 at 11:37, Jerome Forissier > wrote: > > > > > > > > On 5/28/25 17:22, Raymond Mao wrote: > > > Hi Jerome, > > > > >

Re: [PATCH] makefile: fix Clang link error when RSA_PUBLIC_KEY_PARSER is selected

2025-05-28 Thread Raymond Mao
Hi Jerome, On Wed, 28 May 2025 at 11:37, Jerome Forissier wrote: > > > > On 5/28/25 17:22, Raymond Mao wrote: > > Hi Jerome, > > > > On Wed, 28 May 2025 at 04:25, Jerome Forissier > > wrote: > >> > >> Hi Rayond, > >> > >>

Re: [PATCH] makefile: fix Clang link error when RSA_PUBLIC_KEY_PARSER is selected

2025-05-28 Thread Raymond Mao
Hi Jerome, On Wed, 28 May 2025 at 04:25, Jerome Forissier wrote: > > Hi Rayond, > > On 5/27/25 23:05, Raymond Mao wrote: > > MbedTLS bignum module needs '__udivti3' which is a 128-bit division > > function provided by the compiler runtime, typically libgcc fo

[PATCH] armv8: fix Clang warning on writing 32-bit variable to a 64-bit register

2025-05-27 Thread Raymond Mao
Clang is stricter than GCC when it comes to inline assembly and expects the register to be written with explicitly same type of variable. Fixes: c0e1775a867c ("armv8: Add arch-specific sysinfo platform driver") Signed-off-by: Raymond Mao --- arch/arm/cpu/armv8/sysinfo.c | 2 +- 1 fi

[PATCH] makefile: fix Clang link error when RSA_PUBLIC_KEY_PARSER is selected

2025-05-27 Thread Raymond Mao
dd mbedtls into the build system") Signed-off-by: Raymond Mao --- Makefile | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/Makefile b/Makefile index f88684947ee..16506dcaec6 100644 --- a/Makefile +++ b/Makefile @@ -37,7 +37,6 @@ else ifeq ("riscv32", $(MK

Re: [PATCH v2] smbios: add support for type 9 system slot

2025-05-08 Thread Raymond Mao
Hi Heinrich, On Thu, 8 May 2025 at 10:27, Heinrich Schuchardt wrote: > > Am 8. Mai 2025 16:10:33 MESZ schrieb Raymond Mao : > >Implement SMBIOS Type 9 (System Slot). > >SMBIOS library will look for a subnode "system-slot" under > >"smbios" to constr

[PATCH v2] smbios: add support for type 9 system slot

2025-05-08 Thread Raymond Mao
device-function-number = <0x40>; data-bus-width = <0>; peer-grouping-count = <0>; slot-information = <0>; slot-physical-width = <0>; slot-pitch = <0>; slot-height = <0>; }

Re: [PATCH v3 00/22] passage: Define a standard for firmware data flow

2025-04-29 Thread Raymond Mao
Hi Simon, On Tue, 29 Apr 2025 at 10:33, Simon Glass wrote: > > Hi Raymond, > > On Mon, 28 Apr 2025 at 08:41, Raymond Mao wrote: > > > > Hi Simon, > > > > On Thu, 17 Apr 2025 at 14:16, Simon Glass wrote: > > > > > > > > > This seri

Re: [PATCH v3 00/22] passage: Define a standard for firmware data flow

2025-04-28 Thread Raymond Mao
Hi Simon, On Thu, 17 Apr 2025 at 14:16, Simon Glass wrote: > > > This series adds a standard way of passing information between different > firmware phases. This already exists in U-Boot at a very basic level, in > the form of a bloblist containing an spl_handoff structure, but the intent > here

Re: [PATCH v2 1/2] bloblist: fix the overriding of fdt from bloblist

2025-04-28 Thread Raymond Mao
/OP-TEE/optee_os/pull/7352 Please feel free to let me know if there are any concerns. Regards, Raymond On Mon, 31 Mar 2025 at 18:40, Raymond Mao wrote: > > When a bloblist is valid and contains fdt, it explicitly means > a previous boot stage is passing transfer list compliant with &

Re: [PATCH 0/4] bloblist: fdt: Clean up the code

2025-04-14 Thread Raymond Mao
Hi Simon, On Mon, 14 Apr 2025 at 16:47, Simon Glass wrote: > > Hi again Raymond, > > On Mon, 14 Apr 2025 at 14:41, Simon Glass wrote: > > > > Hi Raymond, > > > > On Mon, 14 Apr 2025 at 14:16, Raymond Mao wrote: > > > > > > Hi Simon, > &

Re: [PATCH 0/4] bloblist: fdt: Clean up the code

2025-04-14 Thread Raymond Mao
Hi Simon, On Mon, 14 Apr 2025 at 16:06, Simon Glass wrote: > > (trimming cc) > > Hi Raymond, > > On Mon, 14 Apr 2025 at 13:43, Raymond Mao wrote: > > > > Hi Simon, > > > > On Mon, 14 Apr 2025 at 15:35, Simon Glass wrote: > > > > > > H

Re: [PATCH 0/4] bloblist: fdt: Clean up the code

2025-04-14 Thread Raymond Mao
Hi Simon, On Mon, 14 Apr 2025 at 15:35, Simon Glass wrote: > > Hi Raymond, > > On Mon, 14 Apr 2025 at 07:08, Raymond Mao wrote: > > > > Hi Simon, > > > > On Sun, 13 Apr 2025 at 17:13, Simon Glass wrote: > > > > > > Hi Raymond, >

Re: [PATCH 0/4] bloblist: fdt: Clean up the code

2025-04-14 Thread Raymond Mao
Hi Simon, On Sun, 13 Apr 2025 at 17:13, Simon Glass wrote: > > Hi Raymond, > > On Mon, 7 Apr 2025 at 08:08, Raymond Mao wrote: > > > > Hi Simon, > > > > On Sun, 6 Apr 2025 at 18:06, Simon Glass wrote: > > > > > > Hi Raymond, >

Re: Community meeting April 8th 2025 (was: Re: [ANN] U-Boot v2025.04 released)

2025-04-10 Thread Raymond Mao
Hi all, I didn't have a chance to join the meeting due to a different time zone but I have a few comments on bloblist/fdt as below. On Wed, 9 Apr 2025 at 09:46, Caleb Connolly wrote: > > Hi Everyone, > > On 4/8/25 00:00, Tom Rini wrote: > > Hey all, > > > > It's release day and here's v2025.04.

Re: [PATCH 0/4] bloblist: fdt: Clean up the code

2025-04-07 Thread Raymond Mao
Hi Simon, On Sun, 6 Apr 2025 at 18:06, Simon Glass wrote: > > Hi Raymond, > > On Sat, 5 Apr 2025 at 07:09, Raymond Mao wrote: > > > > Hi Simon, > > > > On Fri, 4 Apr 2025 at 13:40, Simon Glass wrote: > > > > > > Hi Raymond, >

Re: [PATCH 0/4] bloblist: fdt: Clean up the code

2025-04-05 Thread Raymond Mao
Hi Simon, On Fri, 28 Mar 2025 at 11:44, Simon Glass wrote: > > The bloblist code took what I consider to be a wrong turn a year or so > ago. As discussed with Tom, this series proposes a way to arrange things > so that it is simpler to understand and manage. > > - Unwind some of the nesting in bl

[PATCH v2 2/2] env: point fdt address to the fdt in a bloblist

2025-04-05 Thread Raymond Mao
Point fdt_addr to the fdt embedded in the bloblist since fdt_addr is a default address for bootefi, bootm and booti to look for the device tree when launching the kernel. Signed-off-by: Raymond Mao --- Changes in v2: - Update the fdt from bloblist when importing the env. env/common.c | 21

Re: [PATCH 2/2] env: point fdt address to the fdt in a bloblist

2025-04-05 Thread Raymond Mao
Hi Tom and Simon, On Fri, 28 Mar 2025 at 20:03, Tom Rini wrote: > > On Fri, Mar 28, 2025 at 11:39:13PM +, Simon Glass wrote: > > Hi Raymond, > > > > On Fri, 28 Mar 2025 at 09:06, Raymond Mao wrote: > > > > > > Hi Simon, > > > >

Re: [PATCH 0/4] bloblist: fdt: Clean up the code

2025-04-05 Thread Raymond Mao
Hi Simon, On Thu, 3 Apr 2025 at 13:57, Simon Glass wrote: > > Hi Raymond, > > On Fri, 4 Apr 2025 at 03:09, Raymond Mao wrote: > > > > Hi Simon, > > > > On Fri, 28 Mar 2025 at 11:44, Simon Glass wrote: > > > > > > The bloblist code took wha

Re: [PATCH 0/4] bloblist: fdt: Clean up the code

2025-04-04 Thread Raymond Mao
Hi Simon, On Fri, 4 Apr 2025 at 13:40, Simon Glass wrote: > > Hi Raymond, > > On Sat, 5 Apr 2025 at 03:49, Raymond Mao wrote: > > > > Hi Tom and Simon, > > > > On Fri, 28 Mar 2025 at 21:00, Raymond Mao wrote: > > > > > > Hi Tom and Sim

Re: [PATCH 0/4] bloblist: fdt: Clean up the code

2025-04-04 Thread Raymond Mao
Hi Tom and Simon, On Fri, 4 Apr 2025 at 10:48, Raymond Mao wrote: > > Hi Tom and Simon, > > On Fri, 28 Mar 2025 at 21:00, Raymond Mao wrote: > > > > Hi Tom and Simon, > > > > On Fri, 28 Mar 2025 at 20:02, Tom Rini wrote: > > > > > > On Fr

[PATCH] mbedtls: remove incorrect attribute type checker

2025-04-04 Thread Raymond Mao
the EDK2 GenerateCapsule tool. S/MIME Capabilities are not common attributes in an EFI capsule, thus this failure cannot be reproduced with the capsules generated via mkeficapsule. Fixes: 7de0d155cce7 ("mbedtls: add PKCS7 parser porting layer") Reported-by: Ilias Apalodimas Signed-off-b

Re: [PATCH 0/4] bloblist: fdt: Clean up the code

2025-04-04 Thread Raymond Mao
Hi Tom and Simon, On Fri, 28 Mar 2025 at 21:00, Raymond Mao wrote: > > Hi Tom and Simon, > > On Fri, 28 Mar 2025 at 20:02, Tom Rini wrote: > > > > On Fri, Mar 28, 2025 at 11:38:14PM +, Simon Glass wrote: > > > Hi Tom, > > > > >

Re: [PATCH 0/4] bloblist: fdt: Clean up the code

2025-04-03 Thread Raymond Mao
Hi Simon, On Thu, 3 Apr 2025 at 14:18, Simon Glass wrote: > > Hi Raymond, > > On Fri, 4 Apr 2025 at 07:13, Raymond Mao wrote: > > > > Hi Simon, > > > > On Thu, 3 Apr 2025 at 13:57, Simon Glass wrote: > > > > > > Hi Raymond, >

Re: [PATCH 2/4] fdt: Introduce OF_BLOBLIST

2025-04-03 Thread Raymond Mao
Hi Simon, On Fri, 28 Mar 2025 at 11:44, Simon Glass wrote: > > Add an option which indicates that the devicetree comes from the > bloblist. > > After discussions with Tom, it seems we are comfortable with introducing > this option, ensuring of course that the transfer list is properly > supported

Re: [PATCH 1/2] efi_loader: Move public cert for capsules to .rodata

2025-04-01 Thread Raymond Mao
ile) FORCE > -ifeq ($(CONFIG_EFI_CAPSULE_CRT_FILE),"") > - $(error "CONFIG_EFI_CAPSULE_CRT_FILE is empty, EFI capsule > authentication \ > - public key must be specified when CONFIG_EFI_CAPSULE_AUTHENTICATE is > enabled") > -else > - $(call cmd,capsule_esl_gen) > -endif > - > -quiet_cmd_capsule_dtsi_gen = CAPSULE_DTSI_GEN $@ > -cmd_capsule_dtsi_gen = \ > - $(shell sed "s:ESL_BIN_FILE:$(abspath $<):" $(capsule_esl_input_file) > > $@) > - > -$(obj)/$(capsule_esl_dtsi): $(obj)/capsule_esl_file FORCE > - $(call cmd,capsule_dtsi_gen) > - > dtsi_include_list_deps := $(addprefix $(u_boot_dtsi_loc),$(subst > $(quote),,$(dtsi_include_list))) > > -ifdef CONFIG_EFI_CAPSULE_AUTHENTICATE > -dtsi_include_list += $(capsule_esl_dtsi) > -dtsi_include_list_deps += $(obj)/$(capsule_esl_dtsi) > -endif > - > ifneq ($(CHECK_DTBS),) > DT_CHECKER ?= dt-validate > DT_CHECKER_FLAGS ?= $(if $(DT_SCHEMA_FILES),-l $(DT_SCHEMA_FILES),-m) > -- > 2.49.0 > Tested-by: Raymond Mao Regards, Raymond

[PATCH v2 1/2] bloblist: fix the overriding of fdt from bloblist

2025-03-31 Thread Raymond Mao
w the devicetree to come from a bloblist") Signed-off-by: Raymond Mao Reviewed-by: Caleb Connolly Reviewed-by: Ilias Apalodimas Reviewed-by: Simon Glass --- Changes in v2: - None. lib/fdtdec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/fdtdec.c b/lib/fdtdec.c index f

Re: [PATCH 2/2] env: point fdt address to the fdt in a bloblist

2025-03-31 Thread Raymond Mao
Hi Tom, On Fri, 28 Mar 2025 at 19:51, Tom Rini wrote: > > On Thu, Mar 27, 2025 at 04:13:17PM -0700, Raymond Mao wrote: > > > Point fdt_addr to the fdt embedded in the bloblist since fdt_addr > > is a default address for bootefi, bootm and booti to look for the > > dev

Re: [PATCH 1/2] bloblist: fix the overriding of fdt from bloblist

2025-03-28 Thread Raymond Mao
Hi Simon, On Fri, 28 Mar 2025 at 11:13, Simon Glass wrote: > > Hi Raymond, > > On Fri, 28 Mar 2025 at 08:38, Raymond Mao wrote: > > > > Hi Simon, > > > > On Fri, 28 Mar 2025 at 10:26, Simon Glass wrote: > > > > > > Hi Raymond, > &

Re: [PATCH 0/4] bloblist: fdt: Clean up the code

2025-03-28 Thread Raymond Mao
Hi Tom and Simon, On Fri, 28 Mar 2025 at 20:02, Tom Rini wrote: > > On Fri, Mar 28, 2025 at 11:38:14PM +, Simon Glass wrote: > > Hi Tom, > > > > On Fri, 28 Mar 2025 at 10:18, Tom Rini wrote: > > > > > > On Fri, Mar 28, 2025 at 09:43:54AM -0600, Simon Glass wrote: > > > > > > > The bloblist c

Re: [PATCH v3 29/31] RFC: Revert "bloblist: Load the bloblist from the previous loader"

2025-03-28 Thread Raymond Mao
Hi Simon, On Fri, 28 Mar 2025 at 11:36, Simon Glass wrote: > > The logic of this has become too confusing. > > The primary issue with the patch is that U-Boot needs to set up a > bloblist in the first phase where BLOBLIST is enabled. Subsequent > phases can then use that bloblist. > > But the fi

Re: [PATCH 2/2] env: point fdt address to the fdt in a bloblist

2025-03-28 Thread Raymond Mao
Hi Simon, On Fri, 28 Mar 2025 at 07:35, Simon Glass wrote: > > Hi Raymond, > > On Thu, 27 Mar 2025 at 17:13, Raymond Mao wrote: > > > > Point fdt_addr to the fdt embedded in the bloblist since fdt_addr > > is a default address for bootefi, bootm and booti to loo

Re: [PATCH 1/2] bloblist: fix the overriding of fdt from bloblist

2025-03-28 Thread Raymond Mao
Hi Simon, On Fri, 28 Mar 2025 at 10:26, Simon Glass wrote: > > Hi Raymond, > > On Fri, 28 Mar 2025 at 08:17, Raymond Mao wrote: > > > > Hi Simon, > > > > On Fri, 28 Mar 2025 at 07:34, Simon Glass wrote: > > > > > > Hi Raymond, > &

Re: [PATCH 1/2] bloblist: fix the overriding of fdt from bloblist

2025-03-28 Thread Raymond Mao
Hi Caleb, On Fri, 28 Mar 2025 at 08:27, Caleb Connolly wrote: > > > > On 3/28/25 11:44, Caleb Connolly wrote: > > Hi Raymond, > > > > On 3/28/25 00:13, Raymond Mao wrote: > >> When a bloblist is valid and contains fdt, it explicitly means > >>

Re: [PATCH 1/2] bloblist: fix the overriding of fdt from bloblist

2025-03-28 Thread Raymond Mao
Hi Simon, On Fri, 28 Mar 2025 at 07:34, Simon Glass wrote: > > Hi Raymond, > > On Thu, 27 Mar 2025 at 17:13, Raymond Mao wrote: > > > > When a bloblist is valid and contains fdt, it explicitly means > > a previous boot stage is passing transfer list compl

[PATCH 2/2] env: point fdt address to the fdt in a bloblist

2025-03-27 Thread Raymond Mao
Point fdt_addr to the fdt embedded in the bloblist since fdt_addr is a default address for bootefi, bootm and booti to look for the device tree when launching the kernel. Signed-off-by: Raymond Mao --- env/common.c | 17 + 1 file changed, 17 insertions(+) diff --git a/env

[PATCH 1/2] bloblist: fix the overriding of fdt from bloblist

2025-03-27 Thread Raymond Mao
w the devicetree to come from a bloblist") Signed-off-by: Raymond Mao --- lib/fdtdec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/fdtdec.c b/lib/fdtdec.c index f09c9926a7a..c38738b48c7 100644 --- a/lib/fdtdec.c +++ b/lib/fdtdec.c @@ -1708,7 +1708,7 @@ int fdtdec_

[PATCH] tools: add pkg-config for preload_check_sign

2025-03-27 Thread Raymond Mao
The cflags and ldflags of preload_check_sign depend on the openssl package thus pkg-config is needed to get the location where openssl is installed. This fix a potential build failure when openssl is not from the distro and installed in a varied place. Signed-off-by: Raymond Mao --- tools

[PATCH] smbios: add support for type 9 system slot

2025-02-28 Thread Raymond Mao
device-function-number = <0x40>; data-bus-width = <0>; peer-grouping-count = <0>; slot-information = <0>; slot-physical-width = <0>; slot-pitch = <0>; slot-height = <0>;

Re: [PATCH v2 1/3] mbedtls: fix incorrect kconfig dependencies on mbedtls

2025-02-26 Thread Raymond Mao
Hi Tom, On Wed, 26 Feb 2025 at 10:39, Tom Rini wrote: > > On Wed, Feb 26, 2025 at 10:25:22AM -0500, Raymond Mao wrote: > > > Hi Tom, > > > > Is it able to merge this series for the upcoming release? > > This series includes the hotfix for the building failures

Re: [PATCH 1/3] mbedtls: fix incorrect kconfig dependencies on mbedtls

2025-02-26 Thread Raymond Mao
Sorry, please ignore my previous email. We should merge [v2]. On Wed, 26 Feb 2025 at 10:23, Raymond Mao wrote: > > Hi Tom, > > Is it able to merge this series for the upcoming release? > This series includes the hotfix for the building failures when > WGET_HTTPS,NET_LWIP a

Re: [PATCH v2 1/3] mbedtls: fix incorrect kconfig dependencies on mbedtls

2025-02-26 Thread Raymond Mao
re.kernel.org/u-boot/20250126184333.4058848-2-...@chromium.org/ Thanks regards, Raymond On Tue, 4 Feb 2025 at 03:49, Ilias Apalodimas wrote: > > On Tue, 4 Feb 2025 at 00:08, Raymond Mao wrote: > > > > Fixed the building failures when WGET_HTTPS,NET_LWIP and MBEDTLS_LIB > &

Re: [PATCH 1/3] mbedtls: fix incorrect kconfig dependencies on mbedtls

2025-02-26 Thread Raymond Mao
re.kernel.org/u-boot/20250126184333.4058848-2-...@chromium.org/ Thanks regards, Raymond On Fri, 31 Jan 2025 at 09:38, Raymond Mao wrote: > > Hi Ilias, > > On Fri, 31 Jan 2025 at 08:40, Ilias Apalodimas > wrote: > > > > Hi Raymond > > > > On Mon, 27 Jan 20

[PATCH] tools: add HOSTCFLAGS from openssl pkg-config

2025-02-26 Thread Raymond Mao
HOSTCFLAGS of some tools components (image-host, rsa-sign and ecdsa-libcrypto) depend on the directory where openssl is installed. Add them via pkg-config. This fixes a potential build failure in tools when openssl in installed in varied directories. Signed-off-by: Raymond Mao --- tools

[PATCH v2 2/2] bloblist: kconfig for mandatory incoming standard passage

2025-02-19 Thread Raymond Mao
, incoming standard passage is mandatory and U-Boot will report an error when a valid incoming transfer list is missing. Signed-off-by: Raymond Mao --- Changes in v2: - None. common/Kconfig | 14 +- common/bloblist.c | 7 +++ configs

[PATCH v2 1/2] bloblist: refactor xferlist and bloblist

2025-02-19 Thread Raymond Mao
. Signed-off-by: Raymond Mao Suggested-by: Ilias Apalodimas --- Changes in v2: - Call bloblist_check_reg_conv() inside bloblist_check() to check arg1 and arg2 first. arch/arm/lib/xferlist.c | 12 +++ common/bloblist.c | 73 +++-- common/board_f.c

[PATCH 1/2] bloblist: refactor xferlist and bloblist

2025-02-13 Thread Raymond Mao
. Signed-off-by: Raymond Mao --- arch/arm/lib/xferlist.c | 8 +++--- common/bloblist.c | 60 + common/board_f.c| 24 ++--- include/bloblist.h | 8 +++--- 4 files changed, 54 insertions(+), 46 deletions(-) diff --git a/arch/arm

[PATCH 2/2] bloblist: kconfig for mandatory incoming standard passage

2025-02-13 Thread Raymond Mao
, incoming standard passage is mandatory and U-Boot will report an error when a valid incoming transfer list is missing. Signed-off-by: Raymond Mao --- common/Kconfig | 14 +- common/bloblist.c | 7 +++ configs

Re: New Defects reported by Coverity Scan for Das U-Boot

2025-02-11 Thread Raymond Mao
Hi Tom, On Mon, 10 Feb 2025 at 17:26, Tom Rini wrote: > > Here's the latest report. > > -- Forwarded message - > From: > Date: Mon, Feb 10, 2025 at 4:12 PM > Subject: New Defects reported by Coverity Scan for Das U-Boot > To: > > > Hi, > > Please find the latest report on new de

Re: [PATCH 28/29] Revert "bloblist: Load the bloblist from the previous loader"

2025-02-07 Thread Raymond Mao
Hi Simon, On Fri, 7 Feb 2025 at 12:51, Simon Glass wrote: > > Hi Raymond, > > On Fri, 7 Feb 2025 at 08:10, Raymond Mao wrote: > > > > Hi Simon, > > > > On Thu, 6 Feb 2025 at 19:09, Simon Glass wrote: > > > > > > Hi Raymond, >

Re: [PATCH 01/19] mbedtls: Add SHA symbols for VPL

2025-02-07 Thread Raymond Mao
Hi Tom, On Thu, 30 Jan 2025 at 19:09, Tom Rini wrote: > > On Thu, Jan 30, 2025 at 04:49:36PM -0500, Raymond Mao wrote: > > Hi Tom, > > > > On Thu, 30 Jan 2025 at 14:55, Tom Rini wrote: > > > > > > On Mon, Jan 27, 2025 at 10:23:12AM

Re: [PATCH 28/29] Revert "bloblist: Load the bloblist from the previous loader"

2025-02-07 Thread Raymond Mao
Hi Simon, On Thu, 6 Feb 2025 at 19:09, Simon Glass wrote: > > Hi Raymond, > > On Thu, 6 Feb 2025 at 15:35, Raymond Mao wrote: > > > > Hi Simon, > > > > On Thu, 6 Feb 2025 at 10:41, Simon Glass wrote: > > > > > > Hi Raymond, >

Re: [PATCH 28/29] Revert "bloblist: Load the bloblist from the previous loader"

2025-02-06 Thread Raymond Mao
Hi Simon, On Thu, 6 Feb 2025 at 10:41, Simon Glass wrote: > > Hi Raymond, > > On Wed, 5 Feb 2025 at 08:25, Raymond Mao wrote: > > > > +CC Ilias, > > > > Hi Simon, > > > > On Tue, 4 Feb 2025 at 20:57, Simon Glass wrote: > > > > >

Re: [PATCH 29/29] bloblist: Allow relocation at the end of the SPL phase

2025-02-06 Thread Raymond Mao
Hi Simon, On Thu, 6 Feb 2025 at 16:28, Simon Glass wrote: > > Hi Raymond, > > On Wed, 5 Feb 2025 at 08:35, Raymond Mao wrote: > > > > Hi Simon, > > > > On Tue, 4 Feb 2025 at 20:57, Simon Glass wrote: > > > > > > Where the blobl

Re: [PATCH 29/29] bloblist: Allow relocation at the end of the SPL phase

2025-02-05 Thread Raymond Mao
Hi Simon, On Tue, 4 Feb 2025 at 20:57, Simon Glass wrote: > > Where the bloblist is located in internal memory and TF-A's BL31 blob > removes access to this memory, the best option seems to be to relocate > the bloblist just before running TF-A. > > Add an option to select this behaviour and prov

Re: [PATCH 28/29] Revert "bloblist: Load the bloblist from the previous loader"

2025-02-05 Thread Raymond Mao
+CC Ilias, Hi Simon, On Tue, 4 Feb 2025 at 20:57, Simon Glass wrote: > > The logic of this has become too confusing. > > The primary issue with the patch is that U-Boot needs to set up a > bloblist in the first phase where BLOBLIST is enabled. Subsequent > phases can then use that bloblist. > >

  1   2   3   4   5   6   7   8   9   >