From: "Ilias Apalodimas" <ilias.apalodimas@linaro.org>
To: "Balaji Selvanathan" <balaji.selvanathan@oss.qualcomm.com>,
"Sumit Garg" <sumit.garg@kernel.org>, <u-boot-qcom@groups.io>,
<u-boot@lists.u-boot-project.org>
Cc: "Tom Rini" <trini@konsulko.com>,
"Ilias Apalodimas" <ilias.apalodimas@linaro.org>,
"Casey Connolly" <casey.connolly@linaro.org>,
"Neil Armstrong" <neil.armstrong@linaro.org>,
"Aswin Murugan" <aswin.murugan@oss.qualcomm.com>,
"Varadarajan Narayanan" <varadarajan.narayanan@oss.qualcomm.com>,
"Quentin Schulz" <quentin.schulz@cherry.de>,
"Biswapriyo Nath" <nathbappai@gmail.com>
Subject: Re: [PATCH 2/2] configs: qcom: ignore OsIndications for capsule-on-disk
Date: Fri, 28 Aug 2026 11:09:00 +0300 [thread overview]
Message-ID: <DL0F9J2C97EZ.27F7FR5QIOWAK@linaro.org> (raw)
In-Reply-To: <20260813-qcom-capsule-dynamic-fw-images-v1-2-e595ff79496f@oss.qualcomm.com>
Hi Balaji,
On Thu Aug 13, 2026 at 8:56 AM EEST, Balaji Selvanathan wrote:
> Qualcomm boards apply capsule-on-disk updates from U-Boot without the
> OS first setting the OsIndications EFI variable that the capsule
> runtime normally checks. EFI_IGNORE_OSINDICATIONS makes U-Boot skip
> that check so an on-disk capsule is applied at boot.
>
> Set it once in the shared qcom_defconfig so the boards that include it
> (qcm6490, qcs615, qcs8300, qcs9100) all pick it up.
>
> Signed-off-by: Balaji Selvanathan <balaji.selvanathan@oss.qualcomm.com>
> ---
> The option defaults to y only while EFI_RT_VOLATILE_STORE is off. This
> series depends on "configs: qcom_defconfig: Enable EFI variable runtime
> storage" [1], which turns EFI_RT_VOLATILE_STORE on, so the default no
> longer applies and it has to be set explicitly. Apply that patch first.
>
> [1] https://lore.kernel.org/u-boot/20260720-runtime-v1-1-7f904b4cb3ed@oss.qualcomm.com/
> ---
> configs/qcom_defconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/configs/qcom_defconfig b/configs/qcom_defconfig
> index c52f619850d..960c44b1f70 100644
> --- a/configs/qcom_defconfig
> +++ b/configs/qcom_defconfig
> @@ -9,6 +9,7 @@ CONFIG_SYS_LOAD_ADDR=0xA0000000
> # CONFIG_EFI_HAVE_RUNTIME_RESET is not set
> CONFIG_EFI_RUNTIME_UPDATE_CAPSULE=y
> CONFIG_EFI_CAPSULE_ON_DISK=y
> +CONFIG_EFI_IGNORE_OSINDICATIONS=y
You don't want to enable this. First of all that Kconfig option is set to 'y' by default.
Looking at our Kconfig again it should only be allowed if EFI_RT_VOLATILE_STORE is not enabled as
well since it breaks the EFI spec. That option was added years ago when we had no variable
runtime support. If your Kconfig enables EFI_RT_VOLATILE_STORE, you should writethat variable
on capsule updates.
Thanks
/Ilias
> CONFIG_EFI_CAPSULE_FIRMWARE_RAW=y
> CONFIG_BUTTON_CMD=y
> CONFIG_FIT=y
next prev parent reply other threads:[~2026-08-28 8:09 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-13 5:56 [PATCH 0/2] qcom: capsule: build fw_images[] dynamically from the partition table Balaji Selvanathan via U-Boot
2026-08-13 5:56 ` [PATCH 1/2] " Balaji Selvanathan via U-Boot
2026-08-13 5:56 ` [PATCH 2/2] configs: qcom: ignore OsIndications for capsule-on-disk Balaji Selvanathan via U-Boot
2026-08-28 8:09 ` Ilias Apalodimas [this message]
2026-08-26 9:16 ` [PATCH 0/2] qcom: capsule: build fw_images[] dynamically from the partition table Balaji Selvanathan
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=DL0F9J2C97EZ.27F7FR5QIOWAK@linaro.org \
--to=ilias.apalodimas@linaro.org \
--cc=aswin.murugan@oss.qualcomm.com \
--cc=balaji.selvanathan@oss.qualcomm.com \
--cc=casey.connolly@linaro.org \
--cc=nathbappai@gmail.com \
--cc=neil.armstrong@linaro.org \
--cc=quentin.schulz@cherry.de \
--cc=sumit.garg@kernel.org \
--cc=trini@konsulko.com \
--cc=u-boot-qcom@groups.io \
--cc=u-boot@lists.u-boot-project.org \
--cc=varadarajan.narayanan@oss.qualcomm.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 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.