All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sumit Garg <sumit.garg@kernel.org>
To: Balaji Selvanathan <balaji.selvanathan@oss.qualcomm.com>
Cc: u-boot@lists.denx.de, u-boot-qcom@groups.io,
	Tom Rini <trini@konsulko.com>,
	Quentin Schulz <quentin.schulz@cherry.de>,
	Ilias Apalodimas <ilias.apalodimas@linaro.org>,
	Rasmus Villemoes <ravi@prevas.dk>, Simon Glass <sjg@chromium.org>,
	Javier Tia <floss@jetm.me>,
	Mikhail Kshevetskiy <mikhail.kshevetskiy@iopsys.eu>,
	Varadarajan Narayanan <quic_varada@quicinc.com>,
	Javier Martinez Canillas <javierm@redhat.com>,
	Richard Genoud <richard.genoud@bootlin.com>,
	Jan Kiszka <jan.kiszka@siemens.com>,
	David Lechner <dlechner@baylibre.com>,
	Casey Connolly <casey.connolly@linaro.org>,
	Marek Vasut <marek.vasut+renesas@mailbox.org>,
	Christian Marangi <ansuelsmth@gmail.com>,
	Michael Walle <mwalle@kernel.org>,
	Sumit Garg <sumit.garg@oss.qualcomm.com>,
	Neil Armstrong <neil.armstrong@linaro.org>,
	Aswin Murugan <aswin.murugan@oss.qualcomm.com>,
	Jerome Forissier <jerome.forissier@arm.com>,
	Mattijs Korpershoek <mkorpershoek@kernel.org>
Subject: Re: [PATCH v4 7/8] configs: Enable partition type GUID for QCS9100/QCM6490/QCS615 boards
Date: Fri, 1 May 2026 13:32:51 +0530	[thread overview]
Message-ID: <afReKzicksgelbPI@sumit-xelite> (raw)
In-Reply-To: <20260428-type-v4-7-b7051dc45a88@oss.qualcomm.com>

On Tue, Apr 28, 2026 at 01:01:49PM +0530, Balaji Selvanathan wrote:
> Enable CONFIG_PARTITION_TYPE_GUID and configure SCSI environment
> partition type GUID for QCS9100, QCM6490 and QCS615 configurations.
> This allows these platforms to locate the environment partition
> using GPT type GUID instead of UUID.
> 
> Reviewed-by: Simon Glass <sjg@chromium.org>
> Signed-off-by: Balaji Selvanathan <balaji.selvanathan@oss.qualcomm.com>
> ---
> Changes in v4:
> - Add type guid configs to QCS9100 defconfig
> 
> Changes in v3:
> - No changes
> 
> Changes in v2:
> - Enable above new config in qcom_qcs615_defconfig and qcm6490_defconfig
> ---
>  configs/qcm6490_defconfig      | 4 ++++
>  configs/qcom_qcs615_defconfig  | 4 ++++
>  configs/qcom_qcs9100_defconfig | 3 ++-
>  3 files changed, 10 insertions(+), 1 deletion(-)
> 
> diff --git a/configs/qcm6490_defconfig b/configs/qcm6490_defconfig
> index b088367f86c..9560886a37a 100644
> --- a/configs/qcm6490_defconfig
> +++ b/configs/qcm6490_defconfig
> @@ -16,3 +16,7 @@ CONFIG_DEFAULT_DEVICE_TREE="qcom/qcs6490-rb3gen2"
>  
>  CONFIG_FASTBOOT_BUF_ADDR=0xd8800000
>  CONFIG_PHY_QCOM_QMP_COMBO=y
> +
> +CONFIG_ENV_IS_IN_SCSI=y
> +CONFIG_ENV_SCSI_PART_USE_TYPE_GUID=y
> +CONFIG_ENV_SCSI_PART_TYPE_GUID="bc0330eb-3410-4951-a617-03898dbe3372"

These config options are common among targets, can you rather add an env
related config fragment which can be included in all these defconfigs.

Also, please add the partition name here as a comment to easily locate the
partition.

-Sumit

> diff --git a/configs/qcom_qcs615_defconfig b/configs/qcom_qcs615_defconfig
> index 27666a8129d..d0ffa567619 100644
> --- a/configs/qcom_qcs615_defconfig
> +++ b/configs/qcom_qcs615_defconfig
> @@ -22,3 +22,7 @@ CONFIG_REMAKE_ELF=y
>  CONFIG_TEXT_BASE=0x9fc00000
>  
>  CONFIG_FASTBOOT_BUF_ADDR=0xa1600000
> +
> +CONFIG_ENV_IS_IN_SCSI=y
> +CONFIG_ENV_SCSI_PART_USE_TYPE_GUID=y
> +CONFIG_ENV_SCSI_PART_TYPE_GUID="bc0330eb-3410-4951-a617-03898dbe3372"
> diff --git a/configs/qcom_qcs9100_defconfig b/configs/qcom_qcs9100_defconfig
> index 082106157bb..fbaf21952f2 100644
> --- a/configs/qcom_qcs9100_defconfig
> +++ b/configs/qcom_qcs9100_defconfig
> @@ -11,6 +11,7 @@ CONFIG_REMAKE_ELF=y
>  CONFIG_FASTBOOT_BUF_ADDR=0xdb300000
>  CONFIG_DEFAULT_DEVICE_TREE="qcom/qcs9100-ride-r3"
>  CONFIG_ENV_IS_IN_SCSI=y
> -CONFIG_ENV_SCSI_PART_UUID="71cb9cd0-acf1-b6cb-ad91-be9572fe11a9"
> +CONFIG_ENV_SCSI_PART_USE_TYPE_GUID=y
> +CONFIG_ENV_SCSI_PART_TYPE_GUID="bc0330eb-3410-4951-a617-03898dbe3372"
>  # CONFIG_ENV_IS_DEFAULT is not set
>  # CONFIG_ENV_IS_NOWHERE is not set
> 
> -- 
> 2.34.1
> 

  reply	other threads:[~2026-05-01  8:03 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-28  7:31 [PATCH v4 0/8] Add partition type GUID support for environment Balaji Selvanathan
2026-04-28  7:31 ` [PATCH v4 1/8] disk: Add partition lookup by type GUID functionality Balaji Selvanathan
2026-05-07 13:42   ` Quentin Schulz
2026-04-28  7:31 ` [PATCH v4 2/8] scsi: Add partition lookup by type GUID for SCSI devices Balaji Selvanathan
2026-05-01  2:12   ` Simon Glass
2026-04-28  7:31 ` [PATCH v4 3/8] scsi: Optimize scsi_get_blk_by_uuid() loop iteration Balaji Selvanathan
2026-04-28  7:31 ` [PATCH v4 4/8] env: scsi: Fix ENV_SCSI_HW_PARTITION default value type Balaji Selvanathan
2026-04-28  7:31 ` [PATCH v4 5/8] env: scsi: Add partition type GUID support and choice-based selection Balaji Selvanathan
2026-04-28  7:31 ` [PATCH v4 6/8] env: scsi: Implement partition type GUID lookup Balaji Selvanathan
2026-04-28  7:31 ` [PATCH v4 7/8] configs: Enable partition type GUID for QCS9100/QCM6490/QCS615 boards Balaji Selvanathan
2026-05-01  8:02   ` Sumit Garg [this message]
2026-04-28  7:31 ` [PATCH v4 8/8] test: dm: Add partition type GUID lookup test 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=afReKzicksgelbPI@sumit-xelite \
    --to=sumit.garg@kernel.org \
    --cc=ansuelsmth@gmail.com \
    --cc=aswin.murugan@oss.qualcomm.com \
    --cc=balaji.selvanathan@oss.qualcomm.com \
    --cc=casey.connolly@linaro.org \
    --cc=dlechner@baylibre.com \
    --cc=floss@jetm.me \
    --cc=ilias.apalodimas@linaro.org \
    --cc=jan.kiszka@siemens.com \
    --cc=javierm@redhat.com \
    --cc=jerome.forissier@arm.com \
    --cc=marek.vasut+renesas@mailbox.org \
    --cc=mikhail.kshevetskiy@iopsys.eu \
    --cc=mkorpershoek@kernel.org \
    --cc=mwalle@kernel.org \
    --cc=neil.armstrong@linaro.org \
    --cc=quentin.schulz@cherry.de \
    --cc=quic_varada@quicinc.com \
    --cc=ravi@prevas.dk \
    --cc=richard.genoud@bootlin.com \
    --cc=sjg@chromium.org \
    --cc=sumit.garg@oss.qualcomm.com \
    --cc=trini@konsulko.com \
    --cc=u-boot-qcom@groups.io \
    --cc=u-boot@lists.denx.de \
    /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.