[PATCH v6 6/6] doc: dtbinding: Update documentation for Generic Firmware loader

[Simon Glass]

Hi Christian,

On 2026-04-09T13:32:56, Christian Marangi <ansuelsmth at gmail.com> wrote:
> doc: dtbinding: Update documentation for Generic Firmware loader
>
> Update documentation for Generic Firmware loader, generalize it from FS
> specific and add new property and example for FIP loader.
>
> Also add details on the usage of 'phandlepart', 'mtdpart' and 'ubivol'
> property.
>
> Signed-off-by: Christian Marangi <ansuelsmth at gmail.com>
>
> .../misc/{fs_loader.txt => fw_loader.txt}          | 29 +++++++++++++++++++---
>  1 file changed, 26 insertions(+), 3 deletions(-)

> diff --git a/doc/device-tree-bindings/misc/fs_loader.txt b/doc/device-tree-bindings/misc/fw_loader.txt
> @@ -1,17 +1,23 @@
> -* File system firmware loader
> +* Generic Firmware loader
>
>  Required properties:
>  --------------------
>
> -- compatible: should contain "u-boot,fs-loader"
> +- compatible: should contain "u-boot,fs-loader" or "u-boot,fip-loader"
> +- phandlepart: which block storage device and partition the image loading from,
> ...
> +- mtdpart: which partition of ubi the image loading from, this property is
> ...
> +- ubivol: which volume of ubi the image loading from, this property is required
> ...
> +- partoffset: valid ONLY for "u-boot,fip-loader". Offset of the partition to

All these properties except compatible are listed under 'Required
properties' but they are actually optional, right? Only one of
phandlepart or mtdpart+ubivol is needed, and partoffset is
FIP-specific. Please can you split this into 'Required properties'
(just compatible) and 'Optional properties' sections?

Regards,
Simon