All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: Aswin Murugan <aswin.murugan@oss.qualcomm.com>
Cc: ilias.apalodimas@linaro.org, casey.connolly@linaro.org,
	quic_varada@quicinc.com, neil.armstrong@linaro.org,
	sumit.garg@kernel.org, u-boot@lists.u-boot-project.org,
	balaji.selvanathan@oss.qualcomm.com, Michael.Srba@seznam.cz,
	nathbappai@gmail.com, jbx6244@gmail.com,
	quentin.schulz@cherry.de, anshuld@ti.com,
	marcello.bauer@9elements.com, tan.siewert@9elements.com,
	u-boot-qcom@groups.io
Subject: Re: [PATCH RESEND v5 0/5] Add FIT multi-DTB selection for Qualcomm
Date: Mon, 12 Oct 2026 11:50:42 -0600	[thread overview]
Message-ID: <20261012175042.GD3722450@bill-the-cat> (raw)
In-Reply-To: <20261012170754.1606314-1-aswin.murugan@oss.qualcomm.com>

[-- Attachment #1: Type: text/plain, Size: 2502 bytes --]

On Mon, Oct 12, 2026 at 10:37:49PM +0530, Aswin Murugan wrote:
> This series adds dynamic device tree selection from FIT images for
> Qualcomm Snapdragon platforms, enabling U-Boot to select the
> appropriate DTB based on hardware parameters detected at boot.
> 
> Qualcomm FIT-based DTB format is documented in [1].
> 
> The FIT image contains only DTBs, while the kernel is provided
> separately as part of a UKI image.
> 
> The implementation consists of:
> 
> 1. Hardware detection infrastructure: Introduces the qcom,hwinfo
>    driver and qcom_hwdetect helper to collect hardware identification
>    data from SMEM, IMEM and TCSR into a single qcom_hw_params
>    structure for consumers such as FIT multi-DTB selection.
> 
> 2. FIT multi-DTB selection: Implements DTB selection based on the
>    aggregated hardware information. The code parses metadata DTBs,
>    matches FIT configurations against detected hardware attributes,
>    applies overlays, and loads the selected DTB for EFI boot.
> 
> [1] https://github.com/qualcomm-linux/qcom-dtb-metadata/blob/main/
>     Documentation.md
> 
> ---
> 
> Changes in v4:

... which was sent as v5, and this is a re-send of?

> 1. Drop the SMEM cache infrastructure.
> 
> 2. misc: qcom: add qcom,hwinfo driver to provide a common interface
>    for reading IMEM boot cookie and TCSR SOC version information.
> 
> 3. arm: snapdragon: add qcom_hwdetect helper to aggregate hardware
>    information from SMEM, IMEM and TCSR into a single
>    struct qcom_hw_params.
> 
> 4. Refactor FIT multi-DTB selection to consume qcom_hw_params instead
>    of accessing SMEM, IMEM and TCSR data directly.
> 
> 5. Separate hardware discovery from FIT parsing and DTB matching
>    logic, reducing complexity and improving code organization.
> 
> 6. Load FIT images directly from a raw GPT partition and remove the
>    FAT filesystem image based loading approach.
> 
> 7. Drop IH_TYPE_FATFS, mkimage FAT image generation support and
>    related documentation updates.
> 
> 8. arm: dts: enable qcom,hwinfo on qcs9100-ride-r3, qcs615-ride and
>    qcs6490-rb3gen2 by adding the required qcom,hwinfo DT nodes and
>    board-specific offsets.
> 
> Link to v3:
> https://lore.kernel.org/u-boot/20260417120951.3454249-1-aswin.murugan@oss.qualcomm.com/

The FAT comments in v3 were addressed. I'm not really sure the discussion about if
this approach makes sense as really was addressed however?

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

  parent reply	other threads:[~2026-10-12 17:50 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-12 17:07 [PATCH RESEND v5 0/5] Add FIT multi-DTB selection for Qualcomm Aswin Murugan
2026-10-12 17:07 ` [PATCH RESEND v5 1/5] misc: qcom: Add HW-Info aggregator driver Aswin Murugan
2026-10-12 17:07 ` [PATCH RESEND v5 2/5] mach-snapdragon: Add hardware detection helper Aswin Murugan
2026-10-12 17:07 ` [PATCH RESEND v5 3/5] mach-snapdragon: Add FIT multi-DTB selection support Aswin Murugan
2026-10-12 17:07 ` [PATCH RESEND v5 4/5] configs: qcom_defconfig: Enable QCOM_FIT_MULTIDTB Aswin Murugan
2026-10-12 17:07 ` [PATCH RESEND v5 5/5] arm: dts: Enable qcom, hwinfo on qcs9100-ride-r3, qcs615-ride and qcs6490-rb3gen2 Aswin Murugan
2026-10-12 17:50 ` Tom Rini [this message]
2026-10-13  7:20   ` [PATCH RESEND v5 0/5] Add FIT multi-DTB selection for Qualcomm Aswin Murugan
2026-10-13 15:03     ` Tom Rini
2026-10-16 12:15       ` Sumit Garg
2026-10-16 16:27         ` Tom Rini

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=20261012175042.GD3722450@bill-the-cat \
    --to=trini@konsulko.com \
    --cc=Michael.Srba@seznam.cz \
    --cc=anshuld@ti.com \
    --cc=aswin.murugan@oss.qualcomm.com \
    --cc=balaji.selvanathan@oss.qualcomm.com \
    --cc=casey.connolly@linaro.org \
    --cc=ilias.apalodimas@linaro.org \
    --cc=jbx6244@gmail.com \
    --cc=marcello.bauer@9elements.com \
    --cc=nathbappai@gmail.com \
    --cc=neil.armstrong@linaro.org \
    --cc=quentin.schulz@cherry.de \
    --cc=quic_varada@quicinc.com \
    --cc=sumit.garg@kernel.org \
    --cc=tan.siewert@9elements.com \
    --cc=u-boot-qcom@groups.io \
    --cc=u-boot@lists.u-boot-project.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.