On Mon, 24 Jun 2024 at 20:08, Bastian Germann bage@debian.org wrote:

Before 9d0750064e (doc: Move external FIT docs into the main body), the FIT property data-size was not a mandatory property and still it is not expected to be set alongside the data property.

Move the data-size property to the "Conditionally mandatory property" section, where it actually belongs.

Signed-off-by: Bastian Germann bage@debian.org

doc/usage/fit/source_file_format.rst | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)

Reviewed-by: Simon Glass sjg@chromium.org

Can you please add a PR here?

https://github.com/open-source-firmware/flat-image-tree

diff --git a/doc/usage/fit/source_file_format.rst b/doc/usage/fit/source_file_format.rst index 7727ab77c5..15990e3ff5 100644 --- a/doc/usage/fit/source_file_format.rst +++ b/doc/usage/fit/source_file_format.rst @@ -254,9 +254,6 @@ compression zstd zstd compressed ==================== ==================

-data-size

• size of the data in bytes

Conditionally mandatory property

@@ -276,6 +273,9 @@ data-position
    not relative to the loading of the FIT. This is mandatory if external data
    used with a fixed address.

+data-size
+    Size of the data in bytes. This is mandatory if external data is used.
+
os
    OS name, mandatory for types "kernel". Valid OS names are:

--
2.45.2

Regards, Simon