public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed
From: Anton Mikanovich <amikan@ilbers.de>
To: isar-users@googlegroups.com, Richa Bharti <Richa.Bharti@siemens.com>
Subject: Re: [PATCH] wic: support passing extraargs for EFI partition creation
Date: Thu, 18 Dec 2025 09:28:04 +0200	[thread overview]
Message-ID: <03fe0f50-b239-4d2f-8825-4de8cd6e249d@ilbers.de> (raw)
In-Reply-To: <20251217130253.947946-1-Richa.Bharti@siemens.com>

Hello Richa,

17/12/2025 15:02, 'Richa Bharti' via isar-users wrote:
> Some EFI firmwares require FAT filesystems to use specific geometry or
> sector parameters.
>
> Add support for an `extraargs` source parameter to allow passing
> additional mkdosfs options when creating the EFI System Partition (ESP).
> If no extra arguments are provided, mkdosfs defaults to "-S 512",
> preserving existing behaviour.
>
> Signed-off-by: Richa Bharti <Richa.Bharti@siemens.com>
> ---
>   meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py | 6 ++++--
>   1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py b/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
> index 446398d0..6862a288 100644
> --- a/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
> +++ b/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
> @@ -506,8 +506,10 @@ class BootimgEFIPlugin(SourcePlugin):
>   
>           label = part.label if part.label else "ESP"
>   
> -        dosfs_cmd = "mkdosfs -n %s -i %s -C %s %d" % \
> -                    (label, part.fsuuid, bootimg, blocks)
> +        extraargs = source_params['extraargs'] or "-S 512"
This will fail with KeyError: 'extraargs' if no extraargs declared.
> +
> +        dosfs_cmd = "mkdosfs -n %s -i %s %s -C %s %d" % \
> +                    (label, part.fsuuid, extraargs, bootimg, blocks)
>           exec_native_cmd(dosfs_cmd, native_sysroot)
>   
>           mcopy_cmd = "mcopy -i %s -s %s/* ::/" % (bootimg, hdddir)

Moreover, features like this should be covered by some target or have a 
test\
case in CI.

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/03fe0f50-b239-4d2f-8825-4de8cd6e249d%40ilbers.de.

  reply	other threads:[~2025-12-18  7:28 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-12-17 13:02 'Richa Bharti' via isar-users
2025-12-18  7:28 ` Anton Mikanovich [this message]
2025-12-18  8:22   ` [PATCH v2] " 'Richa Bharti' via isar-users
2025-12-18  8:29     ` RICHA BHARTI
2025-12-18  8:39     ` 'Bharti, Richa' via isar-users

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=03fe0f50-b239-4d2f-8825-4de8cd6e249d@ilbers.de \
    --to=amikan@ilbers.de \
    --cc=Richa.Bharti@siemens.com \
    --cc=isar-users@googlegroups.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox