[PATCH RFC] dts: imx8m{m,n,p,q}: Make optee packaging optional

2025-07-31 Thread Yannic Moog
TEE=/path/to/tee.bin Signed-off-by: Yannic Moog --- Due to a bug in binman, we had to guard binman tee.bin with OPTEE config as builds would error when tee.bin was not present in path; Even though optee_os was marked as optional in the binman tree. Since the bug has been resolved in d4f61eae2a

Re: [PATCH v1 5/5] test: Add test case for FIT image load address overlap detection

2025-07-31 Thread Yannic Moog
Am Dienstag, dem 29.07.2025 um 12:48 + schrieb Aristo Chen: > Add a new test case to verify that mkimage properly detects and reports > memory region overlaps in FIT image configurations. > > The test creates a FIT image with kernel and FDT components that have > the same load address (0x4

Re: [PATCH v1 4/5] tools: mkimage: detect overlapping load regions in FIT configurations

2025-07-31 Thread Yannic Moog
Am Dienstag, dem 29.07.2025 um 12:48 + schrieb Aristo Chen: > This patch adds a validation step in mkimage to detect memory region > overlaps between images specified in the same configuration of a > FIT image. If any overlaps are found, the tool prints an error and > aborts the build. > > Thi

Re: [PATCH v1 2/5] binman: Fix FIT image overlap issues for testFitSplitElf

2025-07-31 Thread Yannic Moog
; in split-elf operations. > > An upcoming commit will validate if the memory region is overlapped > > Signed-off-by: Aristo Chen Reviewed-by: Yannic Moog > --- >  tools/binman/ftest.py  |  2 +- >  tools/binman/test/Makefile |  6 - >

Re: [PATCH v1 1/5] binman: Fix FIT image overlap issues

2025-07-31 Thread Yannic Moog
Am Dienstag, dem 29.07.2025 um 12:48 + schrieb Aristo Chen: > Fix three binman tests that has memory region overlap issue, the test > cases needed to be updated to use non-overlapping memory layouts. > > * Tests fixed: >   - testFitFirmwareLoadables >   - testFitSignSimple >   - testFitSignNoS

Re: [PATCH 3/3] Add imx8mp-libra-fpsc board

2025-07-25 Thread Yannic Moog
Hi Benjamin, please add documentation for this board. What do you think about putting all the standard boot variables at the beginning of RAM and LOAD ADDR after that to avoid potential collisions? See phycore-imx8mm and tauri for reference. Am Freitag, dem 25.07.2025 um 12:35 +0200 schrieb Benj

[PATCH] board: phytec: migrate imx8mm boards to standard boot

2025-07-22 Thread Yannic Moog
variables reserved space. Signed-off-by: Yannic Moog --- phyGATE-Tauri and phyBOARD-Polis both have boot logic in the U-Boot environment. Remove that logic and let the respective distro handle the boot logic with standard boot. both boards can boot from mmc devices, ethernet and USB. With the

[PATCH v3] binman: Fix typing for python >= 3.7

2025-07-14 Thread Yannic Moog
To get the [] annotation working with python 3.7 and 3.8, import annotations. Reported-by: Tim Harvey Fixes: 21bc3433a43d ("binman: rework dropping absent entries from packaged image") Reviewed-by: Bryan Brattlof Signed-off-by: Yannic Moog --- Changes in v3: - drop the deprecated a

Re: [PATCH v2] binman: Fix typing for python >= 3.5

2025-07-14 Thread Yannic Moog
Am Montag, dem 14.07.2025 um 11:13 -0600 schrieb Tom Rini: > On Mon, Jul 14, 2025 at 08:29:20AM -0600, Tom Rini wrote: > > On Mon, Jul 14, 2025 at 08:56:57AM -0500, Bryan Brattlof wrote: > > > On July 14, 2025 thus sayeth Yannic Moog: > > > > Am Montag, dem 14.07.202

Re: [PATCH v2] binman: Fix typing for python >= 3.5

2025-07-14 Thread Yannic Moog
Am Montag, dem 14.07.2025 um 07:02 -0500 schrieb Bryan Brattlof: > On July 14, 2025 thus sayeth Yannic Moog: > > To make it work with python 3.5 and 3.6, need type hints from the > > typing module. > > > > Reported-by: Tim Harvey > > Fixes: 21bc3433a43d ("

[PATCH v2] binman: Fix typing for python >= 3.5

2025-07-13 Thread Yannic Moog
To make it work with python 3.5 and 3.6, need type hints from the typing module. Reported-by: Tim Harvey Fixes: 21bc3433a43d ("binman: rework dropping absent entries from packaged image") Signed-off-by: Yannic Moog --- Changes in v2: - drop newline changes - Link to

[PATCH] binman: Fix typing for python >= 3.5

2025-07-11 Thread Yannic Moog
To make it work with python 3.5 and 3.6, need type hints from the typing module. Reported-by: Tim Harvey Fixes: 21bc3433a43d ("binman: rework dropping absent entries from packaged image") Signed-off-by: Yannic Moog --- tools/binman/etype/cbfs.py| 3 ++- tools/binman/etype/mkim

Re: [PATCH v3 3/6] binman: rework dropping absent entries from packaged image

2025-07-09 Thread Yannic Moog
Hi Tim Am Mittwoch, dem 09.07.2025 um 10:50 -0700 schrieb Tim Harvey: > On Fri, Jun 13, 2025 at 5:04 AM Yannic Moog wrote: > > > > When blobs are absent and are marked as optional, they can be safely > > dropped from the binman tree. Use the drop_absent function for that. &

[PATCH v3] binman: migrate from pkg_resources to importlib

2025-06-30 Thread Yannic Moog
transition from pkg_resources to importlib_resources using __name__ results in TypeErrors [2]. [1] https://importlib-resources.readthedocs.io/en/latest/migration.html [2] https://github.com/python/importlib_resources/issues/60 Reviewed-by: Bryan Brattlof Signed-off-by: Yannic Moog --- Changes in

Re: [PATCH v2] binman: migrate from pkg_resources to importlib

2025-06-30 Thread Yannic Moog
Am Freitag, dem 27.06.2025 um 14:00 -0600 schrieb Tom Rini: > On Fri, Jun 27, 2025 at 09:07:36AM +0200, Yannic Moog wrote: > > > pkg_resources is deprecated, use migration guide in [1] to migrate to > > importlib.resources. > > Keep the Python 3.6 backwards compatibility.

[PATCH v2] binman: migrate from pkg_resources to importlib

2025-06-27 Thread Yannic Moog
/en/latest/migration.html Reviewed-by: Bryan Brattlof Signed-off-by: Yannic Moog --- Changes in v2: - rebase on to next - Link to v1: https://lore.kernel.org/r/20250610-pkg_resoures-removal-v1-1-97a269446...@phytec.de --- tools/binman/control.py | 12 ++-- 1 file changed, 6 insertions

[PATCH v3 5/6] binman: ftest: pass allow_fake_blob to _DoReadFileDtb

2025-06-13 Thread Yannic Moog
d-off-by: Yannic Moog --- tools/binman/ftest.py | 26 +++--- 1 file changed, 15 insertions(+), 11 deletions(-) diff --git a/tools/binman/ftest.py b/tools/binman/ftest.py index d707e2aa555b39316d206a4c170ebb83aa6daf72..32e1e0a14c03c9ebf1d9ce1f8278841959d7e5d8 100644 --- a/

[PATCH v3 0/6] Fix handling of optional blobs in binman

2025-06-13 Thread Yannic Moog
ore of a concept - Link to v1: https://lore.kernel.org/r/20250129-binman_faked_optional-v1-0-a4534bc67...@phytec.de --- Yannic Moog (6): binman: drop "faked" return value from check_fake_fname binman: mark optional missing blobs as absent binman: rework dr

[PATCH v3 3/6] binman: rework dropping absent entries from packaged image

2025-06-13 Thread Yannic Moog
of the created binman image. Signed-off-by: Yannic Moog --- tools/binman/control.py | 5 ++--- tools/binman/entry.py | 6 +- tools/binman/etype/cbfs.py| 3 ++- tools/binman/etype/mkimage.py | 2 +- tools/binman/etype/section.py | 16 tools/binman

[PATCH v3 6/6] binman: test: assert optional blobs don't cause non-functionality

2025-06-13 Thread Yannic Moog
When external blobs are marked optional, they should not cause a build to fail. Extend the test cases for FitTeeOsOptional and ExtblobOptional. Reviewed-by: Simon Glass Signed-off-by: Yannic Moog --- tools/binman/ftest.py | 16 1 file changed, 16 insertions(+) diff --git a

[PATCH v3 4/6] binman: add faked optional entry case in CheckForProblems

2025-06-13 Thread Yannic Moog
:1135: .binman_stamp] Error 103 To solve this contradictory messaging, when checking the faked blob list, remove entries that are allowed to be missing. Instead add an info message for faked optional blobs. Also reduce verbosity of the optional image warning to an info message. Signed-off-by: Yan

[PATCH v3 1/6] binman: drop "faked" return value from check_fake_fname

2025-06-13 Thread Yannic Moog
check_fake_fname sets the faked member of the entry. Use that member to get the faked status instead of a returned value indicating the same. Add type annotations to the modified functions while at it. Signed-off-by: Yannic Moog --- tools/binman/entry.py | 11 +-- tools

[PATCH v3 2/6] binman: mark optional missing blobs as absent

2025-06-13 Thread Yannic Moog
st case to highlight that we are testing not only an optional image but the image is missing as well. The behaviour for these is different where the latter will not be packaged into the image. Reported-by: Simon Glass Signed-off-by: Yannic Moog --- tools/binman/binman.rst| 7 +++ tools/bi

Re: [PATCH RFC v2 7/8] tools: binman: ftest: fix tests that require non-faked ext blobs

2025-06-13 Thread Yannic Moog
Am Freitag, dem 30.05.2025 um 12:18 +0100 schrieb Simon Glass: > Hi Yannic, > > On Tue, 27 May 2025 at 14:24, Yannic Moog wrote: > > > > By default, external blobs are faked. Some tests care only about more > > basic functionality. In those cases no external blob

Re: [PATCH RFC v2 5/8] binman: rework dropping absent entries from packaged image

2025-06-13 Thread Yannic Moog
Am Freitag, dem 30.05.2025 um 12:18 +0100 schrieb Simon Glass: > Hi Yannic, > > On Tue, 27 May 2025 at 14:24, Yannic Moog wrote: > > > > When blobs are absent and are marked as optional, they can be safely > > dropped from the binman tree. Use the drop_absent fu

Re: [PATCH RFC v2 4/8] binman: replace _entries with GetEntries()

2025-06-13 Thread Yannic Moog
Am Freitag, dem 30.05.2025 um 12:18 +0100 schrieb Simon Glass: > Hi Yannic, > > On Tue, 27 May 2025 at 14:24, Yannic Moog wrote: > > > > _entries, as highlighted by leading _, is a private member and thus > > should not be accessed directly. As a result use the GetE

Re: [PATCH RFC v2 0/8] Fix handling of optional blobs in binman

2025-06-13 Thread Yannic Moog
Hi Simon, Am Freitag, dem 30.05.2025 um 12:18 +0100 schrieb Simon Glass: > Hi Yannic, > > On Tue, 27 May 2025 at 14:24, Yannic Moog wrote: > > > > This series solves a contradiction regarding ext blobs packaged in > > binman. When they are marked as optional, by

[PATCH] binman: migrate from pkg_resources to importlib

2025-06-10 Thread Yannic Moog
/en/latest/migration.html Signed-off-by: Yannic Moog --- tools/binman/control.py | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tools/binman/control.py b/tools/binman/control.py index 81f61e3e152a9eab558cfc9667131a38082b61a1

Re: [PATCH RFC v2 3/8] tools: binman: mark optional missing blobs as absent

2025-06-06 Thread Yannic Moog
Hi Simon, Am Freitag, dem 30.05.2025 um 12:18 +0100 schrieb Simon Glass: > Hi Yannic, > > On Tue, 27 May 2025 at 14:24, Yannic Moog wrote: > > > > Optional blobs should mark themselves as absent to avoid being packed > > into an image. > > Extend the document

[PATCH RFC v2 7/8] tools: binman: ftest: fix tests that require non-faked ext blobs

2025-05-27 Thread Yannic Moog
: Yannic Moog --- tools/binman/ftest.py | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/tools/binman/ftest.py b/tools/binman/ftest.py index 3fccd4c8ef5f8757b54f3b3fe5fe7898fd166dcb..d7deb3e68ce69c9c1a132f8f29cb71638646277a 100644 --- a/tools/binman

[PATCH RFC v2 1/8] tools: binman: ftest: pass allow_fake_blob to _DoReadFileDtb

2025-05-27 Thread Yannic Moog
ff-by: Yannic Moog --- tools/binman/ftest.py | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/tools/binman/ftest.py b/tools/binman/ftest.py index fa17490001488a297090a65c0c099ecc9a40e0f8..52f0a729bdab0a0bd00f798bbc07d678dd4513de 100644 --- a/tools/binman/ftest.py +++ b/

[PATCH RFC v2 6/8] tools: binman: add faked optional entry case in CheckForProblems

2025-05-27 Thread Yannic Moog
:1135: .binman_stamp] Error 103 To solve this contradictory messaging, when checking the faked blob list, remove entries that are allowed to be missing. This patch does not change the behaviour as blobs remove themselves before CheckForProblems is called. Reviewed-by: Simon Glass Signed-off-by: Yan

[PATCH RFC v2 5/8] binman: rework dropping absent entries from packaged image

2025-05-27 Thread Yannic Moog
data structure (image) built with binman. This also allows us to get all error and warning messages that we want to report while avoiding putting missing optional entries in the final image. Signed-off-by: Yannic Moog --- tools/binman/control.py | 8 tools/binman/entry.py

[PATCH RFC v2 2/8] tools: binman: drop "faked" return value from check_fake_fname

2025-05-27 Thread Yannic Moog
check_fake_fname sets the faked member of the entry. Use that member to get the faked status instead of a returned value indicating the same. Add type annotations to the modified functions while at it. Signed-off-by: Yannic Moog --- tools/binman/entry.py | 11 +-- tools

[PATCH RFC v2 4/8] binman: replace _entries with GetEntries()

2025-05-27 Thread Yannic Moog
_entries, as highlighted by leading _, is a private member and thus should not be accessed directly. As a result use the GetEntries function instead. GetEntries() returns _entries. No functional change introduced in this commit. Signed-off-by: Yannic Moog --- tools/binman/etype/atf_fip.py

[PATCH RFC v2 8/8] binman: test: assert optional blobs don't cause non-functionality

2025-05-27 Thread Yannic Moog
When external blobs are marked optional, they should not cause a build to fail. Extend the test cases for FitTeeOsOptional and ExtblobOptional. Reviewed-by: Simon Glass Signed-off-by: Yannic Moog --- tools/binman/ftest.py | 16 1 file changed, 16 insertions(+) diff --git a

[PATCH RFC v2 3/8] tools: binman: mark optional missing blobs as absent

2025-05-27 Thread Yannic Moog
re removed before CheckForProblems is called. This, as well, is intended behaviour. Reported-by: Simon Glass Signed-off-by: Yannic Moog --- tools/binman/binman.rst| 7 +++ tools/binman/etype/blob.py | 2 ++ 2 files changed, 9 insertions(+) diff --git a/tools/binman/binman.rst b/to

[PATCH RFC v2 0/8] Fix handling of optional blobs in binman

2025-05-27 Thread Yannic Moog
ark as RFC due to failing tests. In current state, series is more of a concept - Link to v1: https://lore.kernel.org/r/20250129-binman_faked_optional-v1-0-a4534bc67...@phytec.de --- Yannic Moog (8): tools: binman: ftest: pass allow_fake_blob to _DoReadFileDtb tools: binman: drop

Re: [PATCH RFC 6/6] binman: doc: update Optional entries

2025-05-27 Thread Yannic Moog
Hi Simon, Am Dienstag, dem 27.05.2025 um 08:34 +0100 schrieb Simon Glass: > Hi Yannic, > > On Wed, 19 Feb 2025 at 00:01, Simon Glass wrote: > > > > Hi Yannic, > > > > On Tue, 18 Feb 2025 at 06:15, Yannic Moog wrote: > > > > > > O

Re: [PATCH RFC 6/6] binman: doc: update Optional entries

2025-02-18 Thread Yannic Moog
On Mon, 2025-02-17 at 06:13 -0700, Simon Glass wrote: > Hi Yannic, > > On Mon, 17 Feb 2025 at 00:21, Yannic Moog wrote: > > > > Hi Simon, > > > > On Fri, 2025-02-14 at 06:48 -0700, Simon Glass wrote: > > > Hi Yannic, > > > > >

Re: [PATCH RFC 6/6] binman: doc: update Optional entries

2025-02-16 Thread Yannic Moog
Hi Simon, On Fri, 2025-02-14 at 06:48 -0700, Simon Glass wrote: > Hi Yannic, > > On Fri, 14 Feb 2025 at 00:05, Yannic Moog wrote: > > > > On Thu, 2025-02-13 at 07:01 -0700, Simon Glass wrote: > > > Hi Yannic, > > > > > > On Thu, 13 Feb 2025 a

Re: [PATCH RFC 1/6] tools: binman: ftest.py: fake ext blobs per default

2025-02-16 Thread Yannic Moog
On Fri, 2025-02-14 at 06:48 -0700, Simon Glass wrote: > Hi Yannic, > > On Fri, 14 Feb 2025 at 00:18, Yannic Moog wrote: > > > > Hi Simon, > > > > On Thu, 2025-02-13 at 07:01 -0700, Simon Glass wrote: > > > Hi Yannic, > > > >

Re: [PATCH RFC 1/6] tools: binman: ftest.py: fake ext blobs per default

2025-02-13 Thread Yannic Moog
Hi Simon, On Thu, 2025-02-13 at 07:01 -0700, Simon Glass wrote: > Hi Yannic, > > On Thu, 13 Feb 2025 at 00:15, Yannic Moog wrote: > > > > Hi Simon, > > > > On Mon, 2025-02-10 at 06:06 -0700, Simon Glass wrote: > > > Hi Yannic, > > > >

Re: [PATCH RFC 6/6] binman: doc: update Optional entries

2025-02-13 Thread Yannic Moog
On Thu, 2025-02-13 at 07:01 -0700, Simon Glass wrote: > Hi Yannic, > > On Thu, 13 Feb 2025 at 00:21, Yannic Moog wrote: > > > > On Mon, 2025-02-10 at 06:08 -0700, Simon Glass wrote: > > > On Wed, 29 Jan 2025 at 03:30, Yannic Moog wrote: > > > > &g

Re: [PATCH RFC 6/6] binman: doc: update Optional entries

2025-02-12 Thread Yannic Moog
On Mon, 2025-02-10 at 06:08 -0700, Simon Glass wrote: > On Wed, 29 Jan 2025 at 03:30, Yannic Moog wrote: > > > > The binman documentation of Optional entries is not accurate in the > > sense that it does not cover blobs entry type. As this is also the most > > w

Re: [PATCH RFC 3/6] tools: binman: ftest: fix tests that require non-faked ext blobs

2025-02-12 Thread Yannic Moog
On Mon, 2025-02-10 at 06:09 -0700, Simon Glass wrote: > Hi, > > On Wed, 29 Jan 2025 at 03:30, Yannic Moog wrote: > > > > By default, external blobs are faked. Some tests care only about more > > basic functionality. In those cases no external blobs should be fake

Re: [PATCH RFC 1/6] tools: binman: ftest.py: fake ext blobs per default

2025-02-12 Thread Yannic Moog
Hi Simon, On Mon, 2025-02-10 at 06:06 -0700, Simon Glass wrote: > Hi Yannic, > > On Wed, 29 Jan 2025 at 03:30, Yannic Moog wrote: > > > > The top level Makefile calls binman with fake-ext-blobs. > > The test setup should reflect this to spot potential bug

[PATCH RFC 1/6] tools: binman: ftest.py: fake ext blobs per default

2025-01-29 Thread Yannic Moog
The top level Makefile calls binman with fake-ext-blobs. The test setup should reflect this to spot potential bugs before reaching users. Signed-off-by: Yannic Moog --- tools/binman/ftest.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/binman/ftest.py b/tools/binman

[PATCH RFC 5/6] binman: test: assert optional blobs don't cause non-functionality

2025-01-29 Thread Yannic Moog
When external blobs are marked optional, they should not cause a build to fail. Extend the test cases for FitTeeOsOptional and ExtblobOptional. Signed-off-by: Yannic Moog --- tools/binman/ftest.py | 16 1 file changed, 16 insertions(+) diff --git a/tools/binman/ftest.py b

[PATCH RFC 0/6] Fix handling of optional blobs in binman

2025-01-29 Thread Yannic Moog
e and extend the documentation. --- Yannic Moog (6): tools: binman: ftest.py: fake ext blobs per default tools: binman: ftest: pass allow_fake_blob to _DoReadFileDtb tools: binman: ftest: fix tests that require non-faked ext blobs binman: fix faked optional entry handling

[PATCH RFC 6/6] binman: doc: update Optional entries

2025-01-29 Thread Yannic Moog
The binman documentation of Optional entries is not accurate in the sense that it does not cover blobs entry type. As this is also the most widely used type to have the optional entry, document the interaction with faking blobs. Signed-off-by: Yannic Moog --- tools/binman/binman.rst | 9

[PATCH RFC 4/6] binman: fix faked optional entry handling

2025-01-29 Thread Yannic Moog
an FIT that use fit,firmware -- AssertionError: 0 != 103 This test failed due to the exposed bug when testing with faked external blobs. Signed-off-by: Yannic Moog --- tools/binman/control.py | 19 --- 1 file ch

[PATCH RFC 2/6] tools: binman: ftest: pass allow_fake_blob to _DoReadFileDtb

2025-01-29 Thread Yannic Moog
Some test cases don't use _DoTestFile directly which accepts allow_fake_blobs. However, they specifically test functionality that requires external blobs not to be faked. Extend the _DoReadFileDtb signature to allow passing that option to _DoTestFile. Signed-off-by: Yannic Moog --- tools/b

[PATCH RFC 3/6] tools: binman: ftest: fix tests that require non-faked ext blobs

2025-01-29 Thread Yannic Moog
: Yannic Moog --- tools/binman/ftest.py | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/tools/binman/ftest.py b/tools/binman/ftest.py index 82f839e33b0..66d1cc240d0 100644 --- a/tools/binman/ftest.py +++ b/tools/binman/ftest.py @@ -5188,13 +5188,15 @@ fdt

Re: [Upstream] [PATCH v2 1/2] board: phytec: common: Add product information to FTD

2025-01-28 Thread Yannic Moog
inter to data where neither the pointer nor the data must be modified. Therefore they should be const. Same for the documentation. I am fine with you moving the existing code and not adding documentation . I would like brief documentation for the newly created functions. Yannic > > On 16.01.25 0

[PATCH] arm: dts: imx8m*-u-boot: remove optional from tee-os entry

2025-01-21 Thread Yannic Moog
tee-os node is guarded by CONFIG_OPTEE. Since OPTEE adds driver support for OP-TEE, the binary should then be packaged in the bootable image. Remove the optional property to enforce this requirement. Signed-off-by: Yannic Moog --- arch/arm/dts/imx8mm-u-boot.dtsi | 1 - arch/arm/dts/imx8mn-u

Re: [PATCH v2] imx8m: Guard binman nodes with CONFIG_OPTEE

2025-01-21 Thread Yannic Moog
On Tue, 2025-01-21 at 11:09 -0300, Fabio Estevam wrote: > Hi Yannic, > > On Fri, Jan 17, 2025 at 5:24 AM Yannic Moog wrote: > > > I have a question: > > Since tee.bin is marked as optional, we still get the same behaviour when > > OPTEE is set but no > >

Re: [PATCH v2] imx8m: Guard binman nodes with CONFIG_OPTEE

2025-01-17 Thread Yannic Moog
ted Execution Environment (OP-TEE) and build with TEE=/path/to/tee.bin > > Some images are invalid > make: *** [Makefile:1135: .binman_stamp] Error 103 > > While at it, only pass tee to the loadables lines when CONFIG_OPTEE is > defined. > > Reported-by: Tim Harvey >

Re: [PATCH v3 1/6] arm: imx8m: add OP-TEE node

2025-01-16 Thread Yannic Moog
Hi Tim, Fabio, On Thu, 2025-01-16 at 14:45 -0800, Tim Harvey wrote: > On Fri, Jan 10, 2025 at 4:35 AM Yannic Moog wrote: > > > > Add tee node in SoC u-boot device trees. Use a kconfig entry to specify > > load and entry addresses for the op-tee image in the respective >

Re: [Upstream] [PATCH v2 1/2] board: phytec: common: Add product information to FTD

2025-01-16 Thread Yannic Moog
Hello Daniel, On Wed, 2025-01-15 at 02:35 -0800, Daniel Schultz wrote: > ft_board_setup inside the board code allows to alter > device-tree during the boot process. > > Introduce a new function for the PHYTEC SOM detection > to read the product name and part number from the EEPROM > content and i

[PATCH v3 5/6] doc: phytec: imx8mp: add OP-TEE documentation

2025-01-10 Thread Yannic Moog
Add documentation for the phyBOARD-Pollux i.MX 8M Plus on OP-TEE integration. Also add missing '-' to TF-A build instruction while at it. Signed-off-by: Yannic Moog --- doc/board/phytec/phycore-imx8mp.rst | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) di

[PATCH v3 6/6] doc: phytec: imx8mm: add OP-TEE documentation

2025-01-10 Thread Yannic Moog
Add instructions on how to build and package OP-TEE for the phycore-imx8mm based boards. The build instructions are identical for phyGATE-Tauri-L and phyBOARD-Polis. Also fix missig '-' for TF-A build instructions. Signed-off-by: Yannic Moog --- board/phytec/phycore_imx8mm/MAINTAINER

[PATCH v3 2/6] phycore-imx8mp_defconfig: add optee load address

2025-01-10 Thread Yannic Moog
The phyBOARD-Pollux expects 0x5600 address to load optee. Signed-off-by: Yannic Moog --- configs/phycore-imx8mp_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/phycore-imx8mp_defconfig b/configs/phycore-imx8mp_defconfig index 23981826109..ba716a6875b 100644 --- a

[PATCH v3 1/6] arm: imx8m: add OP-TEE node

2025-01-10 Thread Yannic Moog
]. [1] https://github.com/ARM-software/arm-trusted-firmware/tree/master/plat/imx/imx8m Signed-off-by: Yannic Moog --- arch/arm/dts/imx8mm-u-boot.dtsi | 17 - arch/arm/dts/imx8mn-u-boot.dtsi | 17 - arch/arm/dts/imx8mp-u-boot.dtsi | 17 - arch/arm

[PATCH v3 4/6] phycore-imx8mm_defconfig: add optee load address

2025-01-10 Thread Yannic Moog
The phyCORE i.MX 8M Mini expects 0x5600 address to load optee. Signed-off-by: Yannic Moog --- configs/phycore-imx8mm_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/phycore-imx8mm_defconfig b/configs/phycore-imx8mm_defconfig index 3f4336837e9..5a3e8162799 100644 --- a

[PATCH v3 3/6] imx8mm-phygate-tauri-l_defconfig: add optee load address

2025-01-10 Thread Yannic Moog
The phyGATE-Tauri-L expects 0x5600 address to load optee. Signed-off-by: Yannic Moog --- configs/imx8mm-phygate-tauri-l_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/imx8mm-phygate-tauri-l_defconfig b/configs/imx8mm-phygate-tauri-l_defconfig index 18fffa7405e

[PATCH v3 0/6] imx8(m): add optee node to binman FIT image

2025-01-10 Thread Yannic Moog
approach. - Add load addresses in defconfigs instead of -u-boot dts. --- Yannic Moog (6): arm: imx8m: add OP-TEE node phycore-imx8mp_defconfig: add optee load address imx8mm-phygate-tauri-l_defconfig: add optee load address phycore-imx8mm_defconfig: add optee load address

Re: [Upstream] [PATCH v2 1/6] arm: imx8m: add OP-TEE node

2024-12-06 Thread Yannic Moog
Hi Adam, On Wed, 2024-11-13 at 07:06 +, Yannic Moog wrote: > On Mon, 2024-11-11 at 04:49 -0600, Adam Ford wrote: > > On Mon, Nov 11, 2024 at 1:48 AM Yannic Moog wrote: > > > > > > Hi, > > > > > > On Fri, 2024-11-08 at 10:05 -0800, Tim Harvey

Re: [PATCH 3/3] board: phytec: phycore_imx8mm: Add RAUC boot logic to environment

2024-11-25 Thread Yannic Moog
On Fri, 2024-11-22 at 11:28 +0100, Yunus Bas wrote: > Add RAUC boot logic to the environment. > > Signed-off-by: Yunus Bas Reviewed-by: Yannic Moog > --- >  board/phytec/phycore_imx8mm/phycore_imx8mm.env | 7 ++- >  1 file changed, 6 insertions(+), 1 deletion(-) >

Re: [PATCH 2/3] phycore_imx8mm: Move default bootcmd to board env

2024-11-25 Thread Yannic Moog
On Fri, 2024-11-22 at 11:28 +0100, Yunus Bas wrote: > Move the default bootcmd from the defconfig to the board environment. > > Signed-off-by: Yunus Bas Reviewed-by: Yannic Moog > --- >  board/phytec/phycore_imx8mm/phycore_imx8mm.env | 9 + >  configs/phycor

Re: [PATCH 1/3] phycore_imx8mm: Switch to using env text files

2024-11-25 Thread Yannic Moog
Hi Yunus, please use get_maintainer script to get a list of people to send (a copy of) the patches to. Usually Maintainers are put in To: field and reviewers Cc: Additionally, I suggest using a cover letter to summarize the changes made in this patch series. On Fri, 2024-11-22 at 11:28 +0100,

Re: [PATCH v3 3/3] board: phytec: imx93: Add phyCORE-i.MX 93 support for all SOM variants

2024-11-20 Thread Yannic Moog
cope. > > Signed-off-by: Christoph Stoidner > Reviewed-by: Wadim Egorov Reviewed-by: Yannic Moog > --- > Cc: Mathieu Othacehe > Cc: Christoph Stoidner > Cc: Stefano Babic > Cc: Fabio Estevam > Cc: "NXP i.MX U-Boot Team" > Cc: Tom Rini > Cc: Yann

Re: [PATCH v3 2/3] board: phytec: imx93: Add eeprom-based hardware introspection

2024-11-20 Thread Yannic Moog
7;s eeprom contains all information about the existing variant > options. Add evaluation of the eeprom data to the spl/u-boot to > enable/disable HS400 and to select the appropriate ram configuration at > startup. > > Signed-off-by: Christoph Stoidner > Reviewed-by: Wadim Egoro

Re: [PATCH v2 2/3] board: phytec: imx93: Add eeprom-based hardware introspection

2024-11-20 Thread Yannic Moog
On Tue, 2024-11-19 at 17:45 +0100, Christoph Stoidner wrote: > > > On Do, 2024-11-14 at 07:30 +0100, Yannic Moog wrote: > > Hi Christoph, > > > > On Wed, 2024-11-13 at 17:00 +0100, Christoph Stoidner wrote: > > > The phyCORE-i.MX 93 is available in vari

Re: [PATCH v2] Makefile: Fake external blob with BINMAN_ALLOW_MISSING=1

2024-11-18 Thread Yannic Moog
On Fri, 2024-11-15 at 07:20 -0700, Simon Glass wrote: > Hi Yannic, > > On Thu, 14 Nov 2024 at 00:51, Yannic Moog wrote: > > > > Hi Simon, > > > > On Wed, 2024-11-06 at 08:35 -0700, Simon Glass wrote: > > > This flag was lost by a previous change and ha

Re: [PATCH v2 3/3] board: phytec: imx93: Add phyCORE-i.MX 93 support for all SOM variants

2024-11-14 Thread Yannic Moog
On Thu, 2024-11-14 at 12:48 +0100, Christoph Stoidner wrote: > > > On Do, 2024-11-14 at 07:37 +0100, Yannic Moog wrote: > > On Wed, 2024-11-13 at 17:00 +0100, Christoph Stoidner wrote: > > > The phyCORE-i.MX 93 is available in various variants (e.g. > > > diff

Re: [PATCH v2] Makefile: Fake external blob with BINMAN_ALLOW_MISSING=1

2024-11-13 Thread Yannic Moog
Hi Simon, On Wed, 2024-11-06 at 08:35 -0700, Simon Glass wrote: > This flag was lost by a previous change and has never been restored. > Without it, binman cannot fully handle missing blobs which are > themselves inputs to mkimage. > > Discussion on this at [1] indicated that this was necessary b

Re: [PATCH v2 3/3] board: phytec: imx93: Add phyCORE-i.MX 93 support for all SOM variants

2024-11-13 Thread Yannic Moog
cope. > > Signed-off-by: Christoph Stoidner > --- > Cc: Mathieu Othacehe > Cc: Christoph Stoidner > Cc: Stefano Babic > Cc: Fabio Estevam > Cc: "NXP i.MX U-Boot Team" > Cc: Tom Rini > Cc: Yannic Moog > Cc: Primoz Fiser > Cc: Andrej Picej >

Re: [PATCH v2 2/3] board: phytec: imx93: Add eeprom-based hardware introspection

2024-11-13 Thread Yannic Moog
ieu Othacehe > Cc: Christoph Stoidner > Cc: Stefano Babic > Cc: Fabio Estevam > Cc: "NXP i.MX U-Boot Team" > Cc: Tom Rini > Cc: Yannic Moog > Cc: Primoz Fiser > Cc: Andrej Picej > Cc: Wadim Egorov > --- > Changes in v2: > -

Re: [PATCH v2 1/6] arm: imx8m: add OP-TEE node

2024-11-13 Thread Yannic Moog
On Mon, 2024-11-11 at 04:49 -0600, Adam Ford wrote: > On Mon, Nov 11, 2024 at 1:48 AM Yannic Moog wrote: > > > > Hi, > > > > On Fri, 2024-11-08 at 10:05 -0800, Tim Harvey wrote: > > > On Fri, Nov 8, 2024 at 5:49 AM Adam Ford wrote: > > > > &g

Re: [PATCH v2 1/6] arm: imx8m: add OP-TEE node

2024-11-10 Thread Yannic Moog
Hi, On Fri, 2024-11-08 at 10:05 -0800, Tim Harvey wrote: > On Fri, Nov 8, 2024 at 5:49 AM Adam Ford wrote: > > > > On Thu, Nov 7, 2024 at 2:42 AM Yannic Moog wrote: > > > > > > Add tee node in SoC u-boot device trees. Use a kconfig entry to specify > >

Re: [Upstream] [PATCH v2 5/6] doc: phytec: imx8mp: add OP-TEE documentation

2024-11-08 Thread Yannic Moog
On Thu, 2024-11-07 at 10:07 +0100, Wadim Egorov wrote: > > > Am 07.11.24 um 09:23 schrieb Yannic Moog: > > Add documentation for the phyBOARD-Pollux i.MX 8M Plus on OP-TEE > > integration. > > Also add missing '-' to TF-A build instruction while at it. >

Re: [Upstream] [PATCH v2 2/6] phycore-imx8mp_defconfig: add optee load address

2024-11-08 Thread Yannic Moog
Hi Wadim, On Thu, 2024-11-07 at 09:52 +0100, Wadim Egorov wrote: > Hi Yannic, > > Am 07.11.24 um 09:23 schrieb Yannic Moog: > > The phyBOARD-Pollux expects 0x5600 address to load optee. > > You are already defining a default for IMX8M_OPTEE_LOAD_ADDR in the > mach

Re: [PATCH v2 1/6] arm: imx8m: add OP-TEE node

2024-11-08 Thread Yannic Moog
Hi Tim, On Thu, 2024-11-07 at 09:42 -0800, Tim Harvey wrote: > On Thu, Nov 7, 2024 at 12:24 AM Yannic Moog wrote: > > > > Add tee node in SoC u-boot device trees. Use a kconfig entry to specify > > load and entry addresses for the op-tee image in the respective > >

[PATCH v2 6/6] doc: phytec: imx8mm: add OP-TEE integration instructions

2024-11-07 Thread Yannic Moog
Add instructions on how to build and package OP-TEE for the phycore-imx8mm based boards. The build instructions are identical for phyGATE-Tauri-L and phyBOARD-Polis. Also fix missig '-' for TF-A build instructions. Signed-off-by: Yannic Moog --- doc/board/phytec/imx8mm-phygate-tauri-

[PATCH v2 4/6] phycore-imx8mm_defconfig: add optee load address

2024-11-07 Thread Yannic Moog
The phyCORE i.MX 8M Mini expects 0x5600 address to load optee. Signed-off-by: Yannic Moog --- configs/phycore-imx8mm_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/phycore-imx8mm_defconfig b/configs/phycore-imx8mm_defconfig index 48a0c0b8b4e..240b5f75a98 100644 --- a

[PATCH v2 5/6] doc: phytec: imx8mp: add OP-TEE documentation

2024-11-07 Thread Yannic Moog
Add documentation for the phyBOARD-Pollux i.MX 8M Plus on OP-TEE integration. Also add missing '-' to TF-A build instruction while at it. Signed-off-by: Yannic Moog --- doc/board/phytec/phycore-imx8mp.rst | 26 +- 1 file changed, 25 insertions(+), 1 deletio

[PATCH v2 3/6] imx8mm-phygate-tauri-l_defconfig: add optee load address

2024-11-07 Thread Yannic Moog
The phyGATE-Tauri-L expects 0x5600 address to load optee. Signed-off-by: Yannic Moog --- configs/imx8mm-phygate-tauri-l_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/imx8mm-phygate-tauri-l_defconfig b/configs/imx8mm-phygate-tauri-l_defconfig index c69fe50ec81

[PATCH v2 2/6] phycore-imx8mp_defconfig: add optee load address

2024-11-07 Thread Yannic Moog
The phyBOARD-Pollux expects 0x5600 address to load optee. Signed-off-by: Yannic Moog --- configs/phycore-imx8mp_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/phycore-imx8mp_defconfig b/configs/phycore-imx8mp_defconfig index 1240c7fcbd0..910109afd80 100644 --- a

[PATCH v2 1/6] arm: imx8m: add OP-TEE node

2024-11-07 Thread Yannic Moog
Add tee node in SoC u-boot device trees. Use a kconfig entry to specify load and entry addresses for the op-tee image in the respective defconfig. Signed-off-by: Yannic Moog --- arch/arm/dts/imx8mm-u-boot.dtsi | 17 - arch/arm/dts/imx8mn-u-boot.dtsi | 17 - arch

[PATCH v2 0/6] imx8(m): add optee node to binman FIT image

2024-11-07 Thread Yannic Moog
://patchwork.ozlabs.org/project/uboot/patch/zehdvr-bzm935...@mecka.net/ --- Changes in v2 (from RFC): - Use kconfig instead of trying pure dt approach. - Add load addresses in defconfigs instead of -u-boot dts. --- Yannic Moog (6): arm: imx8m: add OP-TEE node phycore-imx8mp_defconfig: add optee load

Re: [PATCH RFC 0/6] imx8(m): add optee node to binman FIT image

2024-10-25 Thread Yannic Moog
Hey Tim On Thu, 2024-10-24 at 08:37 -0700, Tim Harvey wrote: > On Thu, Oct 24, 2024 at 12:04 AM Yannic Moog > wrote: > > > > There have been attempts to get op-tee node integrated upstream in > > the > > past [1][2]. The challenge is on how to handle the load and

Re: [PATCH RFC 1/6] arm: dts: imx8m: add fit optee node

2024-10-25 Thread Yannic Moog
On Thu, 2024-10-24 at 08:32 -0700, Tim Harvey wrote: > On Thu, Oct 24, 2024 at 12:04 AM Yannic Moog > wrote: > > > > Add tee node in SoC u-boot device trees. Each board adds their > > specific > > load and entry addresses for the op-tee image in the respect

[PATCH RFC 1/6] arm: dts: imx8m: add fit optee node

2024-10-24 Thread Yannic Moog
Add tee node in SoC u-boot device trees. Each board adds their specific load and entry addresses for the op-tee image in the respective board-u-boot.dtsi file. Signed-off-by: Yannic Moog --- arch/arm/dts/imx8mm-u-boot.dtsi | 15 ++- arch/arm/dts/imx8mn-u-boot.dtsi | 15

[PATCH RFC 5/6] doc: phytec: imx8mp: add OP-TEE documentation

2024-10-24 Thread Yannic Moog
Add documentation for the phyBOARD-Pollux i.MX 8M Plus on OP-TEE integration. Also add missing '-' to TF-A build instruction while at it. Signed-off-by: Yannic Moog --- doc/board/phytec/phycore-imx8mp.rst | 26 +- 1 file changed, 25 insertions(+), 1 deletio

[PATCH RFC 4/6] arm: dts: imx8mm-phyboard-polis: add optee load address

2024-10-24 Thread Yannic Moog
The phyBOARD-Polis i.MX 8M Mini expects 0x5600 address to load optee. Signed-off-by: Yannic Moog --- arch/arm/dts/imx8mm-phyboard-polis-rdk-u-boot.dtsi | 5 + 1 file changed, 5 insertions(+) diff --git a/arch/arm/dts/imx8mm-phyboard-polis-rdk-u-boot.dtsi b/arch/arm/dts/imx8mm-phyboard

[PATCH RFC 2/6] arm: dts: imx8mp-phyboard-pollux: add optee load address

2024-10-24 Thread Yannic Moog
The phyBOARD-Pollux expects 0x5600 address to load optee. Signed-off-by: Yannic Moog --- arch/arm/dts/imx8mp-phyboard-pollux-rdk-u-boot.dtsi | 5 + 1 file changed, 5 insertions(+) diff --git a/arch/arm/dts/imx8mp-phyboard-pollux-rdk-u-boot.dtsi b/arch/arm/dts/imx8mp-phyboard-pollux

[PATCH RFC 6/6] doc: phytec: imx8mm: add OP-TEE integration instructions

2024-10-24 Thread Yannic Moog
Add instructions on how to build and package OP-TEE for the phycore-imx8mm based boards. The build instructions are identical for phyGATE-Tauri-L and phyBOARD-Polis. Also fix missig '-' for TF-A build instructions. Signed-off-by: Yannic Moog --- doc/board/phytec/imx8mm-phygate-tauri-

[PATCH RFC 3/6] arm: dts: imx8mm-phygate-tauri-l: add optee load address

2024-10-24 Thread Yannic Moog
The phyGATE-Tauri-L expects 0x5600 address to load optee. Signed-off-by: Yannic Moog --- arch/arm/dts/imx8mm-phygate-tauri-l-u-boot.dtsi | 5 + 1 file changed, 5 insertions(+) diff --git a/arch/arm/dts/imx8mm-phygate-tauri-l-u-boot.dtsi b/arch/arm/dts/imx8mm-phygate-tauri-l-u

[PATCH RFC 0/6] imx8(m): add optee node to binman FIT image

2024-10-24 Thread Yannic Moog
bs.org/project/uboot/patch/zehdvr-bzm935...@mecka.net/ [3] https://docs.u-boot.org/en/latest/develop/package/binman.html#optional-entries [4] https://docs.u-boot.org/en/latest/develop/package/binman.html#image-description-format see optional: description in [4] --- Yannic Moog (6): arm: dts:

  1   2   >