Re: [PATCH] binman: Fill in most of the imx8mimage and imx8mcst tests

2025-03-11 Thread Fabio Estevam
Hi Marek, On Fri, Mar 7, 2025 at 2:42 AM Marek Vasut wrote: > So what should we do about this particular patch ? > > Should this be applied ? Rebased ? Updated somehow ? This patch causes a CI error: https://source.denx.de/u-boot/custodians/u-boot-imx/-/jobs/1057151

Re: [PATCH] binman: Fill in most of the imx8mimage and imx8mcst tests

2025-03-06 Thread Simon Glass
Hi Marek, On Thu, 6 Mar 2025 at 17:19, Marek Vasut wrote: > > On 3/7/25 12:30 AM, Simon Glass wrote: > > Hi, > > >>> One of the main goals of binman is to reduce the work needed to create > >>> a working image. If people have to go spelunking around the Internet, > >>> build it themselves, etc. t

Re: [PATCH] binman: Fill in most of the imx8mimage and imx8mcst tests

2025-03-06 Thread Marek Vasut
On 3/7/25 3:18 AM, Simon Glass wrote: Hi, One of the main goals of binman is to reduce the work needed to create a working image. If people have to go spelunking around the Internet, build it themselves, etc. that is not good. What you are asking me for is not the tool itself, but key material

Re: [PATCH] binman: Fill in most of the imx8mimage and imx8mcst tests

2025-03-06 Thread Marek Vasut
On 3/7/25 12:30 AM, Simon Glass wrote: Hi, One of the main goals of binman is to reduce the work needed to create a working image. If people have to go spelunking around the Internet, build it themselves, etc. that is not good. What you are asking me for is not the tool itself, but key materia

Re: [PATCH] binman: Fill in most of the imx8mimage and imx8mcst tests

2025-03-06 Thread Simon Glass
Hi Marek, On Sat, 15 Feb 2025 at 08:30, Marek Vasut wrote: > > On 2/15/25 2:38 PM, Simon Glass wrote: > > Hi Marek, > > > > On Sat, 8 Feb 2025 at 14:27, Marek Vasut wrote: > >> > >> On 2/7/25 1:49 AM, Simon Glass wrote: > >>> Hi Marek, > >>> > >>> On Thu, 6 Feb 2025 at 13:52, Marek Vasut wrote:

Re: [PATCH] binman: Fill in most of the imx8mimage and imx8mcst tests

2025-02-15 Thread Marek Vasut
On 2/15/25 2:38 PM, Simon Glass wrote: Hi Marek, On Sat, 8 Feb 2025 at 14:27, Marek Vasut wrote: On 2/7/25 1:49 AM, Simon Glass wrote: Hi Marek, On Thu, 6 Feb 2025 at 13:52, Marek Vasut wrote: On 2/6/25 1:42 PM, Simon Glass wrote: Hi, FAILED (errors=1) So where is the SRK_1_2_3_4_tab

Re: [PATCH] binman: Fill in most of the imx8mimage and imx8mcst tests

2025-02-15 Thread Simon Glass
Hi Marek, On Sat, 8 Feb 2025 at 14:27, Marek Vasut wrote: > > On 2/7/25 1:49 AM, Simon Glass wrote: > > Hi Marek, > > > > On Thu, 6 Feb 2025 at 13:52, Marek Vasut wrote: > >> > >> On 2/6/25 1:42 PM, Simon Glass wrote: > >> > >> Hi, > >> > > FAILED (errors=1) > > > > So where is the S

Re: [PATCH] binman: Fill in most of the imx8mimage and imx8mcst tests

2025-02-08 Thread Marek Vasut
On 2/7/25 1:49 AM, Simon Glass wrote: Hi Marek, On Thu, 6 Feb 2025 at 13:52, Marek Vasut wrote: On 2/6/25 1:42 PM, Simon Glass wrote: Hi, FAILED (errors=1) So where is the SRK_1_2_3_4_table.bin file? See the tool documentation: https://gitlab.apertis.org/pkg/imx-code-signing-tool/-/blob

Re: [PATCH] binman: Fill in most of the imx8mimage and imx8mcst tests

2025-02-06 Thread Simon Glass
Hi Marek, On Thu, 6 Feb 2025 at 13:52, Marek Vasut wrote: > > On 2/6/25 1:42 PM, Simon Glass wrote: > > Hi, > > >>> FAILED (errors=1) > >>> > >>> So where is the SRK_1_2_3_4_table.bin file? > >> See the tool documentation: > >> > >> https://gitlab.apertis.org/pkg/imx-code-signing-tool/-/blob/debi

Re: [PATCH] binman: Fill in most of the imx8mimage and imx8mcst tests

2025-02-06 Thread Marek Vasut
On 2/6/25 1:42 PM, Simon Glass wrote: Hi, FAILED (errors=1) So where is the SRK_1_2_3_4_table.bin file? See the tool documentation: https://gitlab.apertis.org/pkg/imx-code-signing-tool/-/blob/debian/3.4.0+dfsg-2/docs/CST_UG.pdf?ref_type=tags 3.1.2 Running the hab4_pki_tree script Example 3.

Re: [PATCH] binman: Fill in most of the imx8mimage and imx8mcst tests

2025-02-06 Thread Simon Glass
Hi, On Wed, 5 Feb 2025 at 03:20, Marek Vasut wrote: > > On 1/9/25 3:10 PM, Simon Glass wrote: > > Hi Marek, > > > > On Tue, 7 Jan 2025 at 04:24, Marek Vasut wrote: > >> > >> On 1/7/25 4:50 AM, Simon Glass wrote: > >>> Hi Marek, > >>> > >>> On Mon, 6 Jan 2025 at 13:51, Marek Vasut wrote: >

Re: [PATCH] binman: Fill in most of the imx8mimage and imx8mcst tests

2025-02-05 Thread Marek Vasut
On 1/9/25 3:10 PM, Simon Glass wrote: Hi Marek, On Tue, 7 Jan 2025 at 04:24, Marek Vasut wrote: On 1/7/25 4:50 AM, Simon Glass wrote: Hi Marek, On Mon, 6 Jan 2025 at 13:51, Marek Vasut wrote: On 1/6/25 3:38 PM, Simon Glass wrote: [...] I wonder if the cst needs to be updated (binman t

Re: [PATCH] binman: Fill in most of the imx8mimage and imx8mcst tests

2025-01-10 Thread Fabio Estevam
Hi Simon, On Thu, Jan 9, 2025 at 11:21 AM Simon Glass wrote: > I was able to build it from source code without docker. Is there a > public source-code repo somewhere, so Binman could build it? Yes, there is: https://gitlab.apertis.org/pkg/imx-code-signing-tool/-/tree/debian/3.4.0+dfsg-2?ref_ty

Re: [PATCH] binman: Fill in most of the imx8mimage and imx8mcst tests

2025-01-09 Thread Simon Glass
Hi Marek, On Tue, 7 Jan 2025 at 04:24, Marek Vasut wrote: > > On 1/7/25 4:50 AM, Simon Glass wrote: > > Hi Marek, > > > > On Mon, 6 Jan 2025 at 13:51, Marek Vasut wrote: > >> > >> On 1/6/25 3:38 PM, Simon Glass wrote: > >> > >> [...] > >> > >>> I wonder if the cst needs to be updated (binman too

Re: [PATCH] binman: Fill in most of the imx8mimage and imx8mcst tests

2025-01-07 Thread Marek Vasut
On 1/7/25 4:50 AM, Simon Glass wrote: Hi Marek, On Mon, 6 Jan 2025 at 13:51, Marek Vasut wrote: On 1/6/25 3:38 PM, Simon Glass wrote: [...] I wonder if the cst needs to be updated (binman tool -f cst)? At present it is using apt-get and has: $ dpkg -l imx-code-signing-tool Desired=Unknown

Re: [PATCH] binman: Fill in most of the imx8mimage and imx8mcst tests

2025-01-06 Thread Simon Glass
Hi Marek, On Mon, 6 Jan 2025 at 13:51, Marek Vasut wrote: > > On 1/6/25 3:38 PM, Simon Glass wrote: > > [...] > > > I wonder if the cst needs to be updated (binman tool -f cst)? At > > present it is using apt-get and has: > > > > $ dpkg -l imx-code-signing-tool > > Desired=Unknown/Install/Remove/

Re: [PATCH] binman: Fill in most of the imx8mimage and imx8mcst tests

2025-01-06 Thread Marek Vasut
On 1/6/25 3:38 PM, Simon Glass wrote: [...] I wonder if the cst needs to be updated (binman tool -f cst)? At present it is using apt-get and has: $ dpkg -l imx-code-signing-tool Desired=Unknown/Install/Remove/Purge/Hold | Status=Not/Inst/Conf-files/Unpacked/halF-conf/Half-inst/trig-aWait/Trig-

Re: [PATCH] binman: Fill in most of the imx8mimage and imx8mcst tests

2025-01-06 Thread Simon Glass
Hi Marek, On Sun, 5 Jan 2025 at 13:37, Marek Vasut wrote: > > Rename the test .dts and tests and include the M suffix to indicate > those tests are for i.MX8M, not for i.MX8, which is a different SoC. > > Update 339_nxp_imx8m.dts, add testNxpImx8MImageMkimageMissing and > update nxp_imx8mimage.py