* [PATCH v1] ref-manual: uboot-sign: Support firmware property in FIT config
@ 2025-05-20 9:54 Jamin Lin
2025-05-22 9:34 ` [docs] " Antonin Godard
0 siblings, 1 reply; 5+ messages in thread
From: Jamin Lin @ 2025-05-20 9:54 UTC (permalink / raw)
To: docs; +Cc: troy_lee, jamin_lin
Add documentation for the UBOOT_FIT_CONF_FIRMWARE variable, which allows
users to specify an image to be assigned to the "firmware" property of
the FIT configuration node. This explicitly defines the primary image
to boot, instead of relying on the first entry in the "loadables" list.
Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
---
documentation/ref-manual/classes.rst | 2 ++
documentation/ref-manual/variables.rst | 8 ++++++++
2 files changed, 10 insertions(+)
diff --git a/documentation/ref-manual/classes.rst b/documentation/ref-manual/classes.rst
index 5187033f7..54a98bf24 100644
--- a/documentation/ref-manual/classes.rst
+++ b/documentation/ref-manual/classes.rst
@@ -3426,6 +3426,8 @@ The variables used by this class are:
- :term:`UBOOT_FIT_USER_SETTINGS`: adds a user-specific snippet to the U-Boot
Image Tree Source (ITS). Users can include their custom U-Boot Image Tree
Source (ITS) snippet in this variable.
+- :term:`UBOOT_FIT_CONF_FIRMWARE`: adds one image to the ``firmware`` property
+ of the configuration node.
- :term:`UBOOT_FIT_CONF_USER_LOADABLES`: adds one or more user-defined images
to the ``loadables`` property of the configuration node.
diff --git a/documentation/ref-manual/variables.rst b/documentation/ref-manual/variables.rst
index 643a3e7ae..b430fa51f 100644
--- a/documentation/ref-manual/variables.rst
+++ b/documentation/ref-manual/variables.rst
@@ -10327,6 +10327,14 @@ system and gives an overview of their function and contents.
do_compile[depends] += "trusted-firmware-a:do_deploy"
+ :term:`UBOOT_FIT_CONF_FIRMWARE`
+ Adds one image to the ``firmware`` property of the configuration node
+ of the U-Boot Image Tree Source (ITS). Sets the firmware property to select
+ the image to boot first. If not set, the first entry in "loadables" is used
+ instead.::
+
+ UBOOT_FIT_CONF_FIRMWARE = "fwa"
+
:term:`UBOOT_FIT_CONF_USER_LOADABLES`
Adds one or more user-defined images to the ``loadables`` property of the
configuration node of the U-Boot Image Tree Source (ITS). This variable
--
2.43.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [docs] [PATCH v1] ref-manual: uboot-sign: Support firmware property in FIT config
2025-05-20 9:54 [PATCH v1] ref-manual: uboot-sign: Support firmware property in FIT config Jamin Lin
@ 2025-05-22 9:34 ` Antonin Godard
2025-05-23 1:40 ` Jamin Lin
0 siblings, 1 reply; 5+ messages in thread
From: Antonin Godard @ 2025-05-22 9:34 UTC (permalink / raw)
To: jamin_lin, docs; +Cc: troy_lee
Hi,
On Tue May 20, 2025 at 11:54 AM CEST, Jamin Lin via lists.yoctoproject.org wrote:
> Add documentation for the UBOOT_FIT_CONF_FIRMWARE variable, which allows
> users to specify an image to be assigned to the "firmware" property of
> the FIT configuration node. This explicitly defines the primary image
> to boot, instead of relying on the first entry in the "loadables" list.
>
> Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
> ---
> documentation/ref-manual/classes.rst | 2 ++
> documentation/ref-manual/variables.rst | 8 ++++++++
> 2 files changed, 10 insertions(+)
>
> diff --git a/documentation/ref-manual/classes.rst b/documentation/ref-manual/classes.rst
> index 5187033f7..54a98bf24 100644
> --- a/documentation/ref-manual/classes.rst
> +++ b/documentation/ref-manual/classes.rst
> @@ -3426,6 +3426,8 @@ The variables used by this class are:
> - :term:`UBOOT_FIT_USER_SETTINGS`: adds a user-specific snippet to the U-Boot
> Image Tree Source (ITS). Users can include their custom U-Boot Image Tree
> Source (ITS) snippet in this variable.
> +- :term:`UBOOT_FIT_CONF_FIRMWARE`: adds one image to the ``firmware`` property
> + of the configuration node.
> - :term:`UBOOT_FIT_CONF_USER_LOADABLES`: adds one or more user-defined images
> to the ``loadables`` property of the configuration node.
>
> diff --git a/documentation/ref-manual/variables.rst b/documentation/ref-manual/variables.rst
> index 643a3e7ae..b430fa51f 100644
> --- a/documentation/ref-manual/variables.rst
> +++ b/documentation/ref-manual/variables.rst
> @@ -10327,6 +10327,14 @@ system and gives an overview of their function and contents.
>
> do_compile[depends] += "trusted-firmware-a:do_deploy"
>
> + :term:`UBOOT_FIT_CONF_FIRMWARE`
> + Adds one image to the ``firmware`` property of the configuration node
> + of the U-Boot Image Tree Source (ITS). Sets the firmware property to select
> + the image to boot first. If not set, the first entry in "loadables" is used
> + instead.::
Extra "." here.
When you write:
"""
The first entry in "loadables" is used instead
"""
It's not really clear to me what it is used for.
Used to boot?
> +
> + UBOOT_FIT_CONF_FIRMWARE = "fwa"
> +
> :term:`UBOOT_FIT_CONF_USER_LOADABLES`
> Adds one or more user-defined images to the ``loadables`` property of the
> configuration node of the U-Boot Image Tree Source (ITS). This variable
Antonin
--
Antonin Godard, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [docs] [PATCH v1] ref-manual: uboot-sign: Support firmware property in FIT config
2025-05-22 9:34 ` [docs] " Antonin Godard
@ 2025-05-23 1:40 ` Jamin Lin
2025-05-23 7:33 ` Antonin Godard
0 siblings, 1 reply; 5+ messages in thread
From: Jamin Lin @ 2025-05-23 1:40 UTC (permalink / raw)
To: Antonin Godard, docs@lists.yoctoproject.org; +Cc: Troy Lee
Hi Antonin,
> Subject: Re: [docs] [PATCH v1] ref-manual: uboot-sign: Support firmware
> property in FIT config
>
> Hi,
>
> On Tue May 20, 2025 at 11:54 AM CEST, Jamin Lin via lists.yoctoproject.org
> wrote:
> > Add documentation for the UBOOT_FIT_CONF_FIRMWARE variable, which
> > allows users to specify an image to be assigned to the "firmware"
> > property of the FIT configuration node. This explicitly defines the
> > primary image to boot, instead of relying on the first entry in the "loadables"
> list.
> >
> > Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
> > ---
> > documentation/ref-manual/classes.rst | 2 ++
> > documentation/ref-manual/variables.rst | 8 ++++++++
> > 2 files changed, 10 insertions(+)
> >
> > diff --git a/documentation/ref-manual/classes.rst
> > b/documentation/ref-manual/classes.rst
> > index 5187033f7..54a98bf24 100644
> > --- a/documentation/ref-manual/classes.rst
> > +++ b/documentation/ref-manual/classes.rst
> > @@ -3426,6 +3426,8 @@ The variables used by this class are:
> > - :term:`UBOOT_FIT_USER_SETTINGS`: adds a user-specific snippet to the
> U-Boot
> > Image Tree Source (ITS). Users can include their custom U-Boot Image
> Tree
> > Source (ITS) snippet in this variable.
> > +- :term:`UBOOT_FIT_CONF_FIRMWARE`: adds one image to the
> ``firmware`` property
> > + of the configuration node.
> > - :term:`UBOOT_FIT_CONF_USER_LOADABLES`: adds one or more
> user-defined images
> > to the ``loadables`` property of the configuration node.
> >
> > diff --git a/documentation/ref-manual/variables.rst
> > b/documentation/ref-manual/variables.rst
> > index 643a3e7ae..b430fa51f 100644
> > --- a/documentation/ref-manual/variables.rst
> > +++ b/documentation/ref-manual/variables.rst
> > @@ -10327,6 +10327,14 @@ system and gives an overview of their function
> and contents.
> >
> > do_compile[depends] += "trusted-firmware-a:do_deploy"
> >
> > + :term:`UBOOT_FIT_CONF_FIRMWARE`
> > + Adds one image to the ``firmware`` property of the configuration
> node
> > + of the U-Boot Image Tree Source (ITS). Sets the firmware property to
> select
> > + the image to boot first. If not set, the first entry in "loadables" is
> used
> > + instead.::
>
> Extra "." here.
>
Will fix
> When you write:
>
> """
> The first entry in "loadables" is used instead """
>
> It's not really clear to me what it is used for.
> Used to boot?
>
Example:1
configurations {
default = "conf";
conf {
firmware = "fwA";
loadables = "fwB", "fwC", "fwD";
};
};
Boot fwA
Example:2
configurations {
default = "conf";
conf {
loadables = "fwB", "fwC", "fwD";
};
};
Boot fwB
Example:3
configurations {
default = "conf";
conf {
loadables = "fwA" "fwB", "fwC", "fwD";
};
};
Boot fwA
Thanks-Jamin
> > +
> > + UBOOT_FIT_CONF_FIRMWARE = "fwa"
> > +
> > :term:`UBOOT_FIT_CONF_USER_LOADABLES`
> > Adds one or more user-defined images to the ``loadables`` property
> of the
> > configuration node of the U-Boot Image Tree Source (ITS). This
> > variable
>
>
> Antonin
>
> --
> Antonin Godard, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [docs] [PATCH v1] ref-manual: uboot-sign: Support firmware property in FIT config
2025-05-23 1:40 ` Jamin Lin
@ 2025-05-23 7:33 ` Antonin Godard
2025-05-23 8:46 ` Jamin Lin
0 siblings, 1 reply; 5+ messages in thread
From: Antonin Godard @ 2025-05-23 7:33 UTC (permalink / raw)
To: Jamin Lin, docs@lists.yoctoproject.org; +Cc: Troy Lee
On Fri May 23, 2025 at 3:40 AM CEST, Jamin Lin wrote:
> Hi Antonin,
>
>> Subject: Re: [docs] [PATCH v1] ref-manual: uboot-sign: Support firmware
>> property in FIT config
>>
>> Hi,
>>
>> On Tue May 20, 2025 at 11:54 AM CEST, Jamin Lin via lists.yoctoproject.org
>> wrote:
>> > Add documentation for the UBOOT_FIT_CONF_FIRMWARE variable, which
>> > allows users to specify an image to be assigned to the "firmware"
>> > property of the FIT configuration node. This explicitly defines the
>> > primary image to boot, instead of relying on the first entry in the "loadables"
>> list.
>> >
>> > Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
>> > ---
>> > documentation/ref-manual/classes.rst | 2 ++
>> > documentation/ref-manual/variables.rst | 8 ++++++++
>> > 2 files changed, 10 insertions(+)
>> >
>> > diff --git a/documentation/ref-manual/classes.rst
>> > b/documentation/ref-manual/classes.rst
>> > index 5187033f7..54a98bf24 100644
>> > --- a/documentation/ref-manual/classes.rst
>> > +++ b/documentation/ref-manual/classes.rst
>> > @@ -3426,6 +3426,8 @@ The variables used by this class are:
>> > - :term:`UBOOT_FIT_USER_SETTINGS`: adds a user-specific snippet to the
>> U-Boot
>> > Image Tree Source (ITS). Users can include their custom U-Boot Image
>> Tree
>> > Source (ITS) snippet in this variable.
>> > +- :term:`UBOOT_FIT_CONF_FIRMWARE`: adds one image to the
>> ``firmware`` property
>> > + of the configuration node.
>> > - :term:`UBOOT_FIT_CONF_USER_LOADABLES`: adds one or more
>> user-defined images
>> > to the ``loadables`` property of the configuration node.
>> >
>> > diff --git a/documentation/ref-manual/variables.rst
>> > b/documentation/ref-manual/variables.rst
>> > index 643a3e7ae..b430fa51f 100644
>> > --- a/documentation/ref-manual/variables.rst
>> > +++ b/documentation/ref-manual/variables.rst
>> > @@ -10327,6 +10327,14 @@ system and gives an overview of their function
>> and contents.
>> >
>> > do_compile[depends] += "trusted-firmware-a:do_deploy"
>> >
>> > + :term:`UBOOT_FIT_CONF_FIRMWARE`
>> > + Adds one image to the ``firmware`` property of the configuration
>> node
>> > + of the U-Boot Image Tree Source (ITS). Sets the firmware property to
>> select
>> > + the image to boot first. If not set, the first entry in "loadables" is
>> used
>> > + instead.::
>>
>> Extra "." here.
>>
>
> Will fix
>
>> When you write:
>>
>> """
>> The first entry in "loadables" is used instead """
>>
>> It's not really clear to me what it is used for.
>> Used to boot?
>>
>
> Example:1
>
> configurations {
> default = "conf";
> conf {
> firmware = "fwA";
> loadables = "fwB", "fwC", "fwD";
> };
> };
>
> Boot fwA
>
> Example:2
>
> configurations {
> default = "conf";
> conf {
> loadables = "fwB", "fwC", "fwD";
> };
> };
>
> Boot fwB
>
> Example:3
>
> configurations {
> default = "conf";
> conf {
> loadables = "fwA" "fwB", "fwC", "fwD";
> };
> };
>
> Boot fwA
Thanks that clears things up! Can you modify the last sentence from:
"""
If not set, the first entry in "loadables" is used instead
"""
to
"""
If not set, the first entry in "loadables" is used to boot instead
"""
Thanks,
Antonin
--
Antonin Godard, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [docs] [PATCH v1] ref-manual: uboot-sign: Support firmware property in FIT config
2025-05-23 7:33 ` Antonin Godard
@ 2025-05-23 8:46 ` Jamin Lin
0 siblings, 0 replies; 5+ messages in thread
From: Jamin Lin @ 2025-05-23 8:46 UTC (permalink / raw)
To: Antonin Godard, docs@lists.yoctoproject.org; +Cc: Troy Lee
Hi Antonin
> Subject: Re: [docs] [PATCH v1] ref-manual: uboot-sign: Support firmware
> property in FIT config
>
> On Fri May 23, 2025 at 3:40 AM CEST, Jamin Lin wrote:
> > Hi Antonin,
> >
> >> Subject: Re: [docs] [PATCH v1] ref-manual: uboot-sign: Support
> >> firmware property in FIT config
> >>
> >> Hi,
> >>
> >> On Tue May 20, 2025 at 11:54 AM CEST, Jamin Lin via
> >> lists.yoctoproject.org
> >> wrote:
> >> > Add documentation for the UBOOT_FIT_CONF_FIRMWARE variable,
> which
> >> > allows users to specify an image to be assigned to the "firmware"
> >> > property of the FIT configuration node. This explicitly defines the
> >> > primary image to boot, instead of relying on the first entry in the
> "loadables"
> >> list.
> >> >
> >> > Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
> >> > ---
> >> > documentation/ref-manual/classes.rst | 2 ++
> >> > documentation/ref-manual/variables.rst | 8 ++++++++
> >> > 2 files changed, 10 insertions(+)
> >> >
> >> > diff --git a/documentation/ref-manual/classes.rst
> >> > b/documentation/ref-manual/classes.rst
> >> > index 5187033f7..54a98bf24 100644
> >> > --- a/documentation/ref-manual/classes.rst
> >> > +++ b/documentation/ref-manual/classes.rst
> >> > @@ -3426,6 +3426,8 @@ The variables used by this class are:
> >> > - :term:`UBOOT_FIT_USER_SETTINGS`: adds a user-specific snippet
> >> > to the
> >> U-Boot
> >> > Image Tree Source (ITS). Users can include their custom U-Boot
> >> > Image
> >> Tree
> >> > Source (ITS) snippet in this variable.
> >> > +- :term:`UBOOT_FIT_CONF_FIRMWARE`: adds one image to the
> >> ``firmware`` property
> >> > + of the configuration node.
> >> > - :term:`UBOOT_FIT_CONF_USER_LOADABLES`: adds one or more
> >> user-defined images
> >> > to the ``loadables`` property of the configuration node.
> >> >
> >> > diff --git a/documentation/ref-manual/variables.rst
> >> > b/documentation/ref-manual/variables.rst
> >> > index 643a3e7ae..b430fa51f 100644
> >> > --- a/documentation/ref-manual/variables.rst
> >> > +++ b/documentation/ref-manual/variables.rst
> >> > @@ -10327,6 +10327,14 @@ system and gives an overview of their
> >> > function
> >> and contents.
> >> >
> >> > do_compile[depends] += "trusted-firmware-a:do_deploy"
> >> >
> >> > + :term:`UBOOT_FIT_CONF_FIRMWARE`
> >> > + Adds one image to the ``firmware`` property of the
> >> > + configuration
> >> node
> >> > + of the U-Boot Image Tree Source (ITS). Sets the firmware
> >> > + property to
> >> select
> >> > + the image to boot first. If not set, the first entry in
> >> > + "loadables" is
> >> used
> >> > + instead.::
> >>
> >> Extra "." here.
> >>
> >
> > Will fix
> >
> >> When you write:
> >>
> >> """
> >> The first entry in "loadables" is used instead """
> >>
> >> It's not really clear to me what it is used for.
> >> Used to boot?
> >>
> >
> > Example:1
> >
> > configurations {
> > default = "conf";
> > conf {
> > firmware = "fwA";
> > loadables = "fwB", "fwC", "fwD";
> > };
> > };
> >
> > Boot fwA
> >
> > Example:2
> >
> > configurations {
> > default = "conf";
> > conf {
> > loadables = "fwB", "fwC", "fwD";
> > };
> > };
> >
> > Boot fwB
> >
> > Example:3
> >
> > configurations {
> > default = "conf";
> > conf {
> > loadables = "fwA" "fwB", "fwC", "fwD";
> > };
> > };
> >
> > Boot fwA
>
> Thanks that clears things up! Can you modify the last sentence from:
>
> """
> If not set, the first entry in "loadables" is used instead """
>
> to
>
> """
> If not set, the first entry in "loadables" is used to boot instead """
>
Thanks for the review and suggestion.
I resend v2 here, https://patchwork.yoctoproject.org/project/docs/patch/20250523084314.775497-1-jamin_lin@aspeedtech.com/
Thanks-Jamin
> Thanks,
> Antonin
>
> --
> Antonin Godard, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2025-05-23 8:46 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-05-20 9:54 [PATCH v1] ref-manual: uboot-sign: Support firmware property in FIT config Jamin Lin
2025-05-22 9:34 ` [docs] " Antonin Godard
2025-05-23 1:40 ` Jamin Lin
2025-05-23 7:33 ` Antonin Godard
2025-05-23 8:46 ` Jamin Lin
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.