All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Antonin Godard" <antonin.godard@bootlin.com>
To: <adrian.freihofer@siemens.com>, <docs@lists.yoctoproject.org>
Subject: Re: [docs] [PATCH 3/3] migration-5.3: replace kernel-fitimage by kernel-fit-image
Date: Tue, 10 Jun 2025 17:16:57 +0200	[thread overview]
Message-ID: <DAIYBB4O2ZKB.2DPEVG3GG5DGR@bootlin.com> (raw)
In-Reply-To: <20250606140647.80782-4-adrian.freihofer@siemens.com>

On Fri Jun 6, 2025 at 4:06 PM CEST, Adrian Freihofer via lists.yoctoproject.org wrote:
> From: Adrian Freihofer <adrian.freihofer@siemens.com>
>
> Add migration documentation for users moving from the removed
> kernel-fitimage.bbclass to the new kernel-fit-image.bbclass.
>
> Signed-off-by: Adrian Freihofer <adrian.freihofer@siemens.com>
> ---
>  .../migration-guides/migration-5.3.rst        | 35 +++++++++++++++++++
>  1 file changed, 35 insertions(+)
>
> diff --git a/documentation/migration-guides/migration-5.3.rst b/documentation/migration-guides/migration-5.3.rst
> index 22653fc911e..4e03d075b5e 100644
> --- a/documentation/migration-guides/migration-5.3.rst
> +++ b/documentation/migration-guides/migration-5.3.rst
> @@ -59,6 +59,41 @@ Removed classes
>  
>  The following classes have been removed in this release:
>  
> +-  ``kernel-fitimage.bbclass``: the class has been replaced by the
> +   :ref:`ref-classes-kernel-fit-image` class. The new implementation resolves
> +   a long-standing issue with the previous implementation (see
> +   `bug #12912 <https://bugzilla.yoctoproject.org/show_bug.cgi?id=12912>`__).

You can use:

  :yocto_bugs:`bug 12912</show_bug.cgi?id=12912>`

> +
> +   If you are using the kernel FIT image support, you will need to:
> +
> +   1. Make sure to include ``kernel-fit-extra-artifacts`` in your :term:`KERNEL_CLASSES`

You should use the "#." rst syntax here instead:

"""
#.  Make sure...

#.  Use the...
"""

> +      variable to ensure the required files are exposed to the :term:`DEPLOY_DIR_IMAGE`
> +      directory::
> +
> +         KERNEL_CLASSES += "kernel-fit-extra-artifacts"
> +
> +   2. Use the specific FIT image recipe rather than the base kernel recipe.
> +      For example, instead of::
> +
> +         bitbake linux-yocto
> +
> +      You now need to use::
> +
> +         bitbake linux-yocto-fitimage

Could we also suggest that the kernel-fit-image class should be inherited by the
custom kernel recipe or that an extra recipe should be created for the fitimage?
(I don't know what's the best option).

> +
> +   3. If you want to use the FIT image as a replacement for the kernel image in the
> +      root filesystem, add the following configuration to your machine configuration file::
> +
> +         # Do not install the kernel image package
> +         RRECOMMENDS:${KERNEL_PACKAGE_NAME}-base = ""
> +
> +         # Install the FIT image package
> +         MACHINE_EXTRA_RDEPENDS += "${PREFERRED_PROVIDER_virtual/kernel}-fitimage"
> +
> +         # Configure the image.bbclass to depend on the fitImage instead of only
> +         # the kernel to ensure the fitImage is built with the image
> +         KERNEL_DEPLOY_DEPEND = "${PREFERRED_PROVIDER_virtual/kernel}-fitimage:do_deploy"

I would suggest a last sentence:

"""
See the the :ref:`ref-classes-kernel-fit-image` section for more information.
"""

> +
>  Removed features
>  ~~~~~~~~~~~~~~~~
>  

Thanks!
Antonin

-- 
Antonin Godard, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com



      reply	other threads:[~2025-06-10 15:17 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-06-06 14:06 [PATCH 0/3] FIT image improvements documentation AdrianF
2025-06-06 14:06 ` [PATCH 1/3] ref-manual: replace kernel-fitimage by kernel-fit-image AdrianF
2025-06-10 15:16   ` [docs] " Antonin Godard
2025-06-06 14:06 ` [PATCH 2/3] release-notes: drop references to kernel-fitimage AdrianF
2025-06-06 14:06 ` [PATCH 3/3] migration-5.3: replace kernel-fitimage by kernel-fit-image AdrianF
2025-06-10 15:16   ` Antonin Godard [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=DAIYBB4O2ZKB.2DPEVG3GG5DGR@bootlin.com \
    --to=antonin.godard@bootlin.com \
    --cc=adrian.freihofer@siemens.com \
    --cc=docs@lists.yoctoproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.