linux-efi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Adriana Nicolae <adriana@arista.com>
To: Rob Herring <robh@kernel.org>
Cc: krzk@kernel.org, jdelvare@suse.com, frowand.list@gmail.com,
	 linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
	vasilykh@arista.com,  arm.ebbr-discuss@arm.com,
	boot-architecture@lists.linaro.org,  linux-efi@vger.kernel.org,
	uefi-discuss@lists.uefi.org,
	 linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v2 0/2] DMI: Scan for DMI table from DTS info
Date: Thu, 23 Oct 2025 05:20:53 +0300	[thread overview]
Message-ID: <CAERbo5z6BzHqQxXdxPxmxE_eDR7GGGbt3A8kB0gQiWFBE-28Ug@mail.gmail.com> (raw)
In-Reply-To: <20251022201953.GA206947-robh@kernel.org>

On Wed, Oct 22, 2025 at 11:19 PM Rob Herring <robh@kernel.org> wrote:
>
> On Wed, Oct 22, 2025 at 04:45:25AM -0700, adriana wrote:
> > Some bootloaders like U-boot, particularly for the ARM architecture,
> > provide SMBIOS/DMI tables at a specific memory address. However, these
> > systems often do not boot using a full UEFI environment, which means the
> > kernel's standard EFI DMI scanner cannot find these tables.
>
> I thought u-boot is a pretty complete UEFI implementation now. If
> there's standard way for UEFI to provide this, then that's what we
> should be using. I know supporting this has been discussed in context of
> EBBR spec, but no one involved in that has been CC'ed here.

Regarding the use of UEFI, the non UEFI boot is used on Broadcom iProc which
boots initially into a Hardware Security Module which validates U-boot and then
loads it. This specific path does not utilize U-Boot's UEFI
implementation or the
standard UEFI boot services to pass tables like SMBIOS.

Because there's no UEFI configuration table available in this boot mode, we need
an alternative mechanism to pass the SMBIOS table address to the kernel. The
/chosen node seemed like the most straightforward way for the bootloader to
communicate this non-discoverable information.

I wasn't aware of the EBBR discussions covering this. I've added the
boot-architecture and arm.ebbr-discuss lists to the Cc. If there's a preferred
EBBR-compliant way to handle this for non-UEFI boots, I'm happy to adapt
the approach.

>
> > This series adds support for the kernel to find these tables by
> > reading properties from the Device Tree /chosen node. The bootloader
> > can specify the physical addresses using "linux,smbios-table" and
> > "linux,smbios3-table".
>
> /chosen node entries go in chosen.yaml schema in dtschema repository.
> But first, I need to see some agreement this is how we want to support
> this.
>
> Rob

Adriana

       reply	other threads:[~2025-10-23  2:21 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <0f006338-e69b-4b3f-b91f-0cc683544011@kernel.org>
     [not found] ` <20251022114527.618908-1-adriana@arista.com>
     [not found]   ` <20251022201953.GA206947-robh@kernel.org>
2025-10-23  2:20     ` Adriana Nicolae [this message]
2025-10-23  8:21       ` [PATCH v2 0/2] DMI: Scan for DMI table from DTS info Ard Biesheuvel
2025-10-23 13:34         ` Adriana Nicolae
2025-10-23 13:53           ` Ard Biesheuvel
2025-10-23 14:47             ` Adriana Nicolae
2025-10-24  9:49               ` Ard Biesheuvel
2025-10-24 11:07                 ` Ilias Apalodimas
2025-10-24 18:07                   ` Tom Rini
2025-10-31  8:51                     ` Adriana Nicolae
2025-10-31  8:40   ` [PATCH v3 " adriana
2025-10-31  8:41     ` [PATCH v3 1/2] dt-bindings: firmware: Add binding for SMBIOS /chosen properties adriana
2025-10-31  8:52       ` Ard Biesheuvel
2025-10-31  9:05         ` Ard Biesheuvel
2025-10-31  8:41     ` [PATCH v3 2/2] drivers: firmware: dmi_scan: Add support for reading SMBIOS from DT adriana
2025-10-31  9:05       ` Ard Biesheuvel
2025-10-31  9:31       ` Ilias Apalodimas
2025-10-31 10:10     ` [PATCH v4 0/2] DMI: Scan for DMI table from DTS info adriana
2025-10-31 10:10       ` [PATCH v4 1/2] dt-bindings: firmware: Add binding for SMBIOS /chosen properties adriana
2025-10-31 11:15         ` Rob Herring
2025-10-31 11:43           ` Rob Herring
2025-10-31 12:31             ` Adriana Nicolae
2025-10-31 10:10       ` [PATCH v4 2/2] drivers: firmware: dmi_scan: Add support for reading SMBIOS from DT adriana
2025-10-31 10:17       ` [PATCH v4 0/2] DMI: Scan for DMI table from DTS info Ard Biesheuvel
2025-10-31 11:03         ` Ilias Apalodimas
2025-10-31 11:59       ` [PATCH v5 0/1] " adriana
2025-10-31 11:59         ` [PATCH v5 1/1] drivers: firmware: dmi_scan: Add support for reading SMBIOS from DT adriana
2025-10-31 14:19         ` [PATCH v5 0/1] DMI: Scan for DMI table from DTS info Conor Dooley

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=CAERbo5z6BzHqQxXdxPxmxE_eDR7GGGbt3A8kB0gQiWFBE-28Ug@mail.gmail.com \
    --to=adriana@arista.com \
    --cc=arm.ebbr-discuss@arm.com \
    --cc=boot-architecture@lists.linaro.org \
    --cc=devicetree@vger.kernel.org \
    --cc=frowand.list@gmail.com \
    --cc=jdelvare@suse.com \
    --cc=krzk@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-efi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=robh@kernel.org \
    --cc=uefi-discuss@lists.uefi.org \
    --cc=vasilykh@arista.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;
as well as URLs for NNTP newsgroup(s).