<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/coreboot/arm-trusted-firmware.git/tools/fiptool, branch topics/clang-format</title>
<subtitle>review.coreboot.org: arm-trusted-firmware.git
</subtitle>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/arm-trusted-firmware.git/'/>
<entry>
<title>feat(fiptool): handle FIP in a disk partition</title>
<updated>2023-01-23T10:45:53+00:00</updated>
<author>
<name>Antonio Borneo</name>
<email>antonio.borneo@foss.st.com</email>
</author>
<published>2022-09-22T10:15:27+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/arm-trusted-firmware.git/commit/?id=06e69f7c94637c693ea5eb26038096c196d10f07'/>
<id>06e69f7c94637c693ea5eb26038096c196d10f07</id>
<content type='text'>
When FIP is programmed in a disk partition, fiptool cannot be used
directly; this forces the user to temporarily copy the partition
to a file, apply fiptool and copy back the file. This is caused by
fstat() that returns zero file size on a block special file, thus
making fiptool commands info, update, unpack and remove to exit.

For either Linux host or Linux target, recover the partition size
with ioctl() and use it as FIP file size. E.g.:
	fiptool info /dev/disk/by-partlabel/fip-a
	fiptool info /dev/mtdblock4

While there, rework two identical error log messages to provide
more details about the failure and update the date in copyright.

Signed-off-by: Antonio Borneo &lt;antonio.borneo@foss.st.com&gt;
Change-Id: I7cab60e577422d94c24ba7e39458f58bcebc2336
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
When FIP is programmed in a disk partition, fiptool cannot be used
directly; this forces the user to temporarily copy the partition
to a file, apply fiptool and copy back the file. This is caused by
fstat() that returns zero file size on a block special file, thus
making fiptool commands info, update, unpack and remove to exit.

For either Linux host or Linux target, recover the partition size
with ioctl() and use it as FIP file size. E.g.:
	fiptool info /dev/disk/by-partlabel/fip-a
	fiptool info /dev/mtdblock4

While there, rework two identical error log messages to provide
more details about the failure and update the date in copyright.

Signed-off-by: Antonio Borneo &lt;antonio.borneo@foss.st.com&gt;
Change-Id: I7cab60e577422d94c24ba7e39458f58bcebc2336
</pre>
</div>
</content>
</entry>
<entry>
<title>refactor(security): add OpenSSL 1.x compatibility</title>
<updated>2022-11-11T18:33:42+00:00</updated>
<author>
<name>Juan Pablo Conde</name>
<email>juanpablo.conde@arm.com</email>
</author>
<published>2022-10-25T23:41:02+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/arm-trusted-firmware.git/commit/?id=cf2dd17ddda2f44f5dedddcaf48300d16358597a'/>
<id>cf2dd17ddda2f44f5dedddcaf48300d16358597a</id>
<content type='text'>
When updated to work with OpenSSL 3.0, the host tools lost their
compatibility with previous versions (1.x) of OpenSSL. This is
mainly due to the fact that 1.x APIs became deprecated in 3.0 and
therefore their use cause compiling errors. In addition, updating
for a newer version of OpenSSL meant improving the stability
against security threats. However, although version 1.1.1 is
now deprecated, it still receives security updates, so it would
not imply major security issues to keep compatibility with it too.

This patch adds backwards compatibility with OpenSSL 1.x versions
by adding back 1.x API code. It defines a macro USING_OPENSSL3,
which will select the appropriate OpenSSL API version depending on
the OpenSSL library path chosen (which is determined by the
already-existing OPENSSL_DIR variable).

In addition, cleanup items were packed in functions and moved to
the proper modules in order to make the code more maintainable and
legible.

Signed-off-by: Juan Pablo Conde &lt;juanpablo.conde@arm.com&gt;
Change-Id: I8deceb5e419edc73277792861882404790ccd33c
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
When updated to work with OpenSSL 3.0, the host tools lost their
compatibility with previous versions (1.x) of OpenSSL. This is
mainly due to the fact that 1.x APIs became deprecated in 3.0 and
therefore their use cause compiling errors. In addition, updating
for a newer version of OpenSSL meant improving the stability
against security threats. However, although version 1.1.1 is
now deprecated, it still receives security updates, so it would
not imply major security issues to keep compatibility with it too.

This patch adds backwards compatibility with OpenSSL 1.x versions
by adding back 1.x API code. It defines a macro USING_OPENSSL3,
which will select the appropriate OpenSSL API version depending on
the OpenSSL library path chosen (which is determined by the
already-existing OPENSSL_DIR variable).

In addition, cleanup items were packed in functions and moved to
the proper modules in order to make the code more maintainable and
legible.

Signed-off-by: Juan Pablo Conde &lt;juanpablo.conde@arm.com&gt;
Change-Id: I8deceb5e419edc73277792861882404790ccd33c
</pre>
</div>
</content>
</entry>
<entry>
<title>feat(fiptool): add cca, core_swd, plat cert in FIP</title>
<updated>2022-06-14T14:47:37+00:00</updated>
<author>
<name>laurenw-arm</name>
<email>lauren.wehrmeister@arm.com</email>
</author>
<published>2022-04-21T21:36:26+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/arm-trusted-firmware.git/commit/?id=147f52f3e81f7ccf1dae90bc5687ec137feeb46c'/>
<id>147f52f3e81f7ccf1dae90bc5687ec137feeb46c</id>
<content type='text'>
Added support for cca CoT in the fiptool by adding the cca,
core_swd, and plat key certificates.

Signed-off-by: Lauren Wehrmeister &lt;lauren.wehrmeister@arm.com&gt;
Change-Id: I1ba559e188ad8c33cb0e643d7a2fc6fb96736ab9
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Added support for cca CoT in the fiptool by adding the cca,
core_swd, and plat key certificates.

Signed-off-by: Lauren Wehrmeister &lt;lauren.wehrmeister@arm.com&gt;
Change-Id: I1ba559e188ad8c33cb0e643d7a2fc6fb96736ab9
</pre>
</div>
</content>
</entry>
<entry>
<title>refactor(security): upgrade tools to OpenSSL 3.0</title>
<updated>2022-05-16T17:01:59+00:00</updated>
<author>
<name>Juan Pablo Conde</name>
<email>juanpablo.conde@arm.com</email>
</author>
<published>2022-03-02T23:10:08+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/arm-trusted-firmware.git/commit/?id=9bc52d330fccb0e4df22006630350a42457d3306'/>
<id>9bc52d330fccb0e4df22006630350a42457d3306</id>
<content type='text'>
Host tools cert_tool and encrypt_fw refactored to be fully
compatible with OpenSSL v3.0.

Changes were made following the OpenSSL 3.0 migration guide:
https://www.openssl.org/docs/man3.0/man7/migration_guide.html
In some cases, those changes are straightforward and only
a small modification on the types or API calls was needed
(e.g.: replacing BN_pseudo_rand() with BN_rand(). Both identical
since v1.1.0).
The use of low level APIs is now deprecated. In some cases,
the new API provides a simplified solution for our goals and
therefore the code was simplified accordingly (e.g.: generating
RSA keys through EVP_RSA_gen() without the need of handling the
exponent). However, in some cases, a more
sophisticated approach was necessary, as the use of a context
object was required (e.g.: when retrieving the digest value from
an SHA file).

Signed-off-by: Juan Pablo Conde &lt;juanpablo.conde@arm.com&gt;
Change-Id: I978e8578fe7ab3e71307450ebe7e7812fbcaedb6
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Host tools cert_tool and encrypt_fw refactored to be fully
compatible with OpenSSL v3.0.

Changes were made following the OpenSSL 3.0 migration guide:
https://www.openssl.org/docs/man3.0/man7/migration_guide.html
In some cases, those changes are straightforward and only
a small modification on the types or API calls was needed
(e.g.: replacing BN_pseudo_rand() with BN_rand(). Both identical
since v1.1.0).
The use of low level APIs is now deprecated. In some cases,
the new API provides a simplified solution for our goals and
therefore the code was simplified accordingly (e.g.: generating
RSA keys through EVP_RSA_gen() without the need of handling the
exponent). However, in some cases, a more
sophisticated approach was necessary, as the use of a context
object was required (e.g.: when retrieving the digest value from
an SHA file).

Signed-off-by: Juan Pablo Conde &lt;juanpablo.conde@arm.com&gt;
Change-Id: I978e8578fe7ab3e71307450ebe7e7812fbcaedb6
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge "fix(fiptool): respect OPENSSL_DIR" into integration</title>
<updated>2021-12-22T21:24:44+00:00</updated>
<author>
<name>Madhukar Pappireddy</name>
<email>madhukar.pappireddy@arm.com</email>
</author>
<published>2021-12-22T21:24:44+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/arm-trusted-firmware.git/commit/?id=dd14d0f63f7d049cbcbfd3f8128f05e9a351bbe5'/>
<id>dd14d0f63f7d049cbcbfd3f8128f05e9a351bbe5</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>fix(fiptool): avoid packing the zero size images in the FIP</title>
<updated>2021-12-20T13:31:38+00:00</updated>
<author>
<name>Manish V Badarkhe</name>
<email>Manish.Badarkhe@arm.com</email>
</author>
<published>2021-12-18T11:26:25+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/arm-trusted-firmware.git/commit/?id=ab556c9c646f1b5f1b500449a5813a4eecdc0302'/>
<id>ab556c9c646f1b5f1b500449a5813a4eecdc0302</id>
<content type='text'>
Updated the fiptool to avoid packing the zero size images in
the FIP.
Also, updated the commitlint-json file to cover the fiptool
changes under a separate scope.

Change-Id: Id7ac3dcff0c7318546e49308d0f17b6cbd5eb24b
Signed-off-by: Manish V Badarkhe &lt;Manish.Badarkhe@arm.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Updated the fiptool to avoid packing the zero size images in
the FIP.
Also, updated the commitlint-json file to cover the fiptool
changes under a separate scope.

Change-Id: Id7ac3dcff0c7318546e49308d0f17b6cbd5eb24b
Signed-off-by: Manish V Badarkhe &lt;Manish.Badarkhe@arm.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>fix(fiptool): respect OPENSSL_DIR</title>
<updated>2021-12-08T12:01:58+00:00</updated>
<author>
<name>Ross Burton</name>
<email>ross.burton@arm.com</email>
</author>
<published>2021-10-25T11:27:59+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/arm-trusted-firmware.git/commit/?id=0a956f81805b46b1530f30dd79d16950dc491a7b'/>
<id>0a956f81805b46b1530f30dd79d16950dc491a7b</id>
<content type='text'>
fiptool links to libcrypto, so as with the other tools it should respect
OPENSSL_DIR for include/library paths.

Change-Id: Icd8c15fa5097db1da9a3a9222d9e267548c4c7e2
Signed-off-by: Ross Burton &lt;ross.burton@arm.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
fiptool links to libcrypto, so as with the other tools it should respect
OPENSSL_DIR for include/library paths.

Change-Id: Icd8c15fa5097db1da9a3a9222d9e267548c4c7e2
Signed-off-by: Ross Burton &lt;ross.burton@arm.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>feat(rme): add ENABLE_RME build option and support for RMM image</title>
<updated>2021-10-05T16:49:59+00:00</updated>
<author>
<name>Zelalem Aweke</name>
<email>zelalem.aweke@arm.com</email>
</author>
<published>2021-07-11T23:33:20+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/arm-trusted-firmware.git/commit/?id=5b18de09e80f87963df9a2e451c47e2321b8643a'/>
<id>5b18de09e80f87963df9a2e451c47e2321b8643a</id>
<content type='text'>
The changes include:

- A new build option (ENABLE_RME) to enable FEAT_RME

- New image called RMM. RMM is R-EL2 firmware that manages Realms.
  When building TF-A, a path to RMM image can be specified using
  the "RMM" build flag. If RMM image is not provided, TRP is built
  by default and used as RMM image.

- Support for RMM image in fiptool

Signed-off-by: Zelalem Aweke &lt;zelalem.aweke@arm.com&gt;
Change-Id: I017c23ef02e465a5198baafd665a60858ecd1b25
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The changes include:

- A new build option (ENABLE_RME) to enable FEAT_RME

- New image called RMM. RMM is R-EL2 firmware that manages Realms.
  When building TF-A, a path to RMM image can be specified using
  the "RMM" build flag. If RMM image is not provided, TRP is built
  by default and used as RMM image.

- Support for RMM image in fiptool

Signed-off-by: Zelalem Aweke &lt;zelalem.aweke@arm.com&gt;
Change-Id: I017c23ef02e465a5198baafd665a60858ecd1b25
</pre>
</div>
</content>
</entry>
<entry>
<title>fiptool: Do not call 'make clean' in 'all' target</title>
<updated>2021-04-07T17:36:20+00:00</updated>
<author>
<name>Pali Rohár</name>
<email>pali@kernel.org</email>
</author>
<published>2021-03-25T15:08:09+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/arm-trusted-firmware.git/commit/?id=dbeab0da963de2efd82ca748e6d3a112e54e49e8'/>
<id>dbeab0da963de2efd82ca748e6d3a112e54e49e8</id>
<content type='text'>
Calling 'make clean' in 'all' target is causing recompilation of binary
at every 'make' call, which is wrong.

Also building a new target via 'make TARGET' can cause infinite loop as
it is not defined as explicit make dependency. Dependent targets must be
specified after colon when defining target, which also prevents infinite
loops as make is able to detect these circular dependencies.

Moreover calling 'make clean' is supposed to be done by user when
configuration is changing.

So remove calling 'make clean' in 'all' target and define dependency for
'${PROJECT}' at correct place.

Signed-off-by: Pali Rohár &lt;pali@kernel.org&gt;
Change-Id: I70e7fd2b04b02f6a0650c82df91d58c9a4cb24d9
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Calling 'make clean' in 'all' target is causing recompilation of binary
at every 'make' call, which is wrong.

Also building a new target via 'make TARGET' can cause infinite loop as
it is not defined as explicit make dependency. Dependent targets must be
specified after colon when defining target, which also prevents infinite
loops as make is able to detect these circular dependencies.

Moreover calling 'make clean' is supposed to be done by user when
configuration is changing.

So remove calling 'make clean' in 'all' target and define dependency for
'${PROJECT}' at correct place.

Signed-off-by: Pali Rohár &lt;pali@kernel.org&gt;
Change-Id: I70e7fd2b04b02f6a0650c82df91d58c9a4cb24d9
</pre>
</div>
</content>
</entry>
<entry>
<title>tools: add mechanism to allow platform specific image UUID</title>
<updated>2021-03-24T04:19:31+00:00</updated>
<author>
<name>Pankaj Gupta</name>
<email>pankaj.gupta@nxp.com</email>
</author>
<published>2020-12-09T08:32:38+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/arm-trusted-firmware.git/commit/?id=3527d6d21db663b9f8f7a67c832cf10d55864d9a'/>
<id>3527d6d21db663b9f8f7a67c832cf10d55864d9a</id>
<content type='text'>
Generic framework is added to include platform defined UUID.

This framework is added for the following:
- All NXP SoC based platforms needed additional fip-fuse.bin
- NXP SoC lx2160a based platforms requires additional fip-ddr.bin

Signed-off-by: Pankaj Gupta &lt;pankaj.gupta@nxp.com&gt;
Change-Id: Ibe05d9c596256e34077287a490dfcd5b731ef2cf
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Generic framework is added to include platform defined UUID.

This framework is added for the following:
- All NXP SoC based platforms needed additional fip-fuse.bin
- NXP SoC lx2160a based platforms requires additional fip-ddr.bin

Signed-off-by: Pankaj Gupta &lt;pankaj.gupta@nxp.com&gt;
Change-Id: Ibe05d9c596256e34077287a490dfcd5b731ef2cf
</pre>
</div>
</content>
</entry>
</feed>
