public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: Simon Glass <sjg@chromium.org>
Cc: "U-Boot Mailing List" <u-boot@lists.denx.de>,
	"Caleb Connolly" <caleb.connolly@linaro.org>,
	"Jan Kiszka" <jan.kiszka@siemens.com>,
	"Jerry Van Baren" <vanbaren@cideas.com>,
	"Marek Behún" <kabel@kernel.org>,
	"Marek Mojík" <marek.mojik@nic.cz>, "Marek Vasut" <marex@denx.de>,
	"Raymond Mao" <raymond.mao@linaro.org>,
	"Sumit Garg" <sumit.garg@linaro.org>,
	"This contributor prefers not to receive mails"
	<noreply@example.com>
Subject: Re: [PATCH v3 17/30] emulation: fdt: Relax condition for OF_HAS_PRIOR_STAGE
Date: Mon, 13 Jan 2025 09:44:51 -0600	[thread overview]
Message-ID: <20250113154451.GV3476@bill-the-cat> (raw)
In-Reply-To: <20250111000029.245022-18-sjg@chromium.org>

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

On Fri, Jan 10, 2025 at 05:00:16PM -0700, Simon Glass wrote:

> QEMU always gets its devicetree from the OF_BOARD mechanism so we should
> not depend on !BLOBLIST here.
> 
> It's not clear why we need to have any relationship with BLOBLIST so
> let's remove the entire condition.

This is fine.

> The logic of PRIOR_STAGE is quite a mess, unfortuantely. We should rely
> only standard passage to receive things from the prior stage. QEMU
> should implement standard passage to provide its devicetree to U-Boot
> However Linaro has blocked my patch to provide devicetree additions[1],
> so little breath should be held in respect of either change.
> 
> [1] https://lore.kernel.org/qemu-devel/20210926183410.256484-1-sjg@chromium.org/

Drop this portion.

Otherwise:

Reviewed-by: Tom Rini <trini@konsulko.com>

-- 
Tom

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

  reply	other threads:[~2025-01-13 15:45 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-01-10 23:59 [PATCH v3 00/30] upl: Prerequite patches for updated spec Simon Glass
2025-01-11  0:00 ` [PATCH v3 01/30] bloblist: Make BLOBLIST_ALLOC the default Simon Glass
2025-01-11  0:00 ` [PATCH v3 02/30] abuf: Provide a way to get the buffer address Simon Glass
2025-01-11  0:00 ` [PATCH v3 03/30] abuf: Allow use in host tools Simon Glass
2025-01-11  0:00 ` [PATCH v3 04/30] abuf: Provide a constant buffer Simon Glass
2025-01-11  0:00 ` [PATCH v3 05/30] cpu: Provide a way to get the physical-address size Simon Glass
2025-01-11  0:00 ` [PATCH v3 06/30] serial: Support info() method in ns16550 xPL with UPL Simon Glass
2025-01-11  0:00 ` [PATCH v3 07/30] mkimage: Update map_to_sysmem() to match its prototype Simon Glass
2025-01-11  0:00 ` [PATCH v3 08/30] x86: Enable meminfo command Simon Glass
2025-01-11  0:00 ` [PATCH v3 09/30] x86: Show the timestamp counter with bdinfo Simon Glass
2025-01-11  0:00 ` [PATCH v3 10/30] ofnode: Use 4K for a default tree-size Simon Glass
2025-01-11  0:00 ` [PATCH v3 11/30] ofnode: Indicate when out of space in a few places Simon Glass
2025-01-11  0:00 ` [PATCH v3 12/30] ofnode: Update of_add_subnode() to indicate name is alloced Simon Glass
2025-01-11  0:00 ` [PATCH v3 13/30] boot: Rename fit_image_get_data() Simon Glass
2025-01-11  0:00 ` [PATCH v3 14/30] boot: Rename fit_image_get_data_and_size() Simon Glass
2025-01-11  0:00 ` [PATCH v3 15/30] boot: Use fit_image_get_data() to get data Simon Glass
2025-01-11  0:00 ` [PATCH v3 16/30] test: Fix inpected typo in upl test Simon Glass
2025-01-11  0:00 ` [PATCH v3 17/30] emulation: fdt: Relax condition for OF_HAS_PRIOR_STAGE Simon Glass
2025-01-13 15:44   ` Tom Rini [this message]
2025-01-11  0:00 ` [PATCH v3 18/30] emulation: Use bloblist to hold tables Simon Glass
2025-01-11  0:00 ` [PATCH v3 19/30] x86: Create more space for SPL with qemu-x86_64 Simon Glass
2025-01-11  0:00 ` [PATCH v3 20/30] pci: video: Set up the pixel-format field Simon Glass
2025-01-11  0:00 ` [PATCH v3 21/30] x86: Show an error if video fails Simon Glass
2025-01-11  0:00 ` [PATCH v3 22/30] x86: Support jumping to a UPL image Simon Glass
2025-01-11  0:00 ` [PATCH v3 23/30] x86: Enable UPL handoff for SPL Simon Glass
2025-01-11  0:00 ` [PATCH v3 24/30] x86: Move tables to use SZ macros Simon Glass
2025-01-11  0:00 ` [PATCH v3 25/30] x86: Align the SMBIOS table to a 4K boundary Simon Glass
2025-01-11  0:00 ` [PATCH v3 26/30] efi_loader: Avoid mapping the ACPI tables twice Simon Glass
2025-01-11  0:00 ` [PATCH v3 27/30] x86: emulation: Enable bloblist Simon Glass
2025-01-11  0:00 ` [PATCH v3 28/30] dm: core: Clarify behaviour of ofnode_name_eq() Simon Glass
2025-01-11  0:00 ` [PATCH v3 29/30] dm: core: Provide ofnode_name_eq_unit() to accept a unit address Simon Glass
2025-01-11  0:00 ` [PATCH v3 30/30] dm: core: Provide ofnode_find_subnode_unit() Simon Glass
2025-01-23 14:13 ` [PATCH v3 00/30] upl: Prerequite patches for updated spec Tom Rini
2025-02-16  8:28   ` Patrick Rudolph
2025-02-16 12:56     ` Simon Glass
2025-03-06 14:34       ` Simon Glass

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=20250113154451.GV3476@bill-the-cat \
    --to=trini@konsulko.com \
    --cc=caleb.connolly@linaro.org \
    --cc=jan.kiszka@siemens.com \
    --cc=kabel@kernel.org \
    --cc=marek.mojik@nic.cz \
    --cc=marex@denx.de \
    --cc=noreply@example.com \
    --cc=raymond.mao@linaro.org \
    --cc=sjg@chromium.org \
    --cc=sumit.garg@linaro.org \
    --cc=u-boot@lists.denx.de \
    --cc=vanbaren@cideas.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