From: Tom Rini <trini@konsulko.com>
To: Simon Glass <sjg@chromium.org>
Cc: "U-Boot Mailing List" <u-boot@lists.denx.de>,
"Ahelenia Ziemiańska" <nabijaczleweli@nabijaczleweli.xyz>,
"Alexander Dahl" <ada@thorsis.com>,
"Alexander Graf" <agraf@csgraf.de>,
"Alper Nebi Yasak" <alpernebiyasak@gmail.com>,
"Anand Moon" <linux.amoon@gmail.com>,
"Anatolij Gustschin" <agust@denx.de>,
"Caleb Connolly" <caleb.connolly@linaro.org>,
"Chen-Yu Tsai" <wens@csie.org>,
"Dmitry Gerasimov" <di.gerasimov@gmail.com>,
"Heiko Schocher" <hs@denx.de>,
"Heinrich Schuchardt" <xypron.glpk@gmx.de>,
"Hou Zhiqiang" <Zhiqiang.Hou@nxp.com>,
"Hugo Cornelis" <hugo.cornelis@essensium.com>,
"Igor Opaniuk" <igor.opaniuk@gmail.com>,
"Ilias Apalodimas" <ilias.apalodimas@linaro.org>,
"Jagan Teki" <jagan@amarulasolutions.com>,
"Janne Grunau" <j@jannau.net>,
"Jerry Van Baren" <vanbaren@cideas.com>,
"Jiaxun Yang" <jiaxun.yang@flygoat.com>,
"Jonas Karlman" <jonas@kwiboo.se>,
"Leo Yu-Chi Liang" <ycliang@andestech.com>,
"Lukas Funke" <lukas.funke@weidmueller.com>,
"Marek Mojík" <marek.mojik@nic.cz>,
"Masahisa Kojima" <kojima.masahisa@socionext.com>,
"Mattijs Korpershoek" <mkorpershoek@baylibre.com>,
"Maxim Moskalets" <maximmosk4@gmail.com>,
"Michal Simek" <michal.simek@amd.com>,
"Miquel Raynal" <miquel.raynal@bootlin.com>,
"Neil Armstrong" <neil.armstrong@linaro.org>,
"Nikhil M Jain" <n-jain1@ti.com>,
"Patrick Delaunay" <patrick.delaunay@foss.st.com>,
"Pavel Herrmann" <morpheus.ibis@gmail.com>,
"Peter Robinson" <pbrobinson@gmail.com>,
"Philip Oberfichtner" <pro@denx.de>,
"Rasmus Villemoes" <rasmus.villemoes@prevas.dk>,
"Raymond Mao" <raymond.mao@linaro.org>,
"Sean Anderson" <seanga2@gmail.com>,
"Stefano Babic" <sbabic@denx.de>
Subject: Re: [PATCH 00/67] upl: Align with the updated spec
Date: Fri, 3 Jan 2025 08:26:51 -0600 [thread overview]
Message-ID: <20250103142651.GU3476@bill-the-cat> (raw)
In-Reply-To: <CAFLszTiSeBykLOSPGEFG6MV74KmC_e1F0PTkAfkR1DKxYstmeg@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1907 bytes --]
On Fri, Jan 03, 2025 at 12:45:36PM +1300, Simon Glass wrote:
> Hi Tom,
>
> On Fri, 3 Jan 2025 at 03:44, Tom Rini <trini@konsulko.com> wrote:
> >
> > On Thu, Jan 02, 2025 at 11:08:46AM +1300, Simon Glass wrote:
> >
> > > The current UPL spec[1] has been tidied up and improved over the last
> > > year, since U-Boot's original UPL support was written.
> > >
> > > This series addresses various issues, with a goal of having U-Boot boot
> > > EDK2.
> > >
> > > There is still more work to do, but this series gets to the point (on
> > > QEMU) where the ACPI tables are required. Further work will be needed to
> > > relocate the tables out of the QEMU firmware-filesystem.
> > >
> > > [1] git@github.com:UniversalPayload/spec.git commit 3f1450d
> >
> > Since this is on top of your fork, and not mainline, and you aren't even
> > noting that in your cover letter it's probably not worth the time of
> > everyone you sent this to, to look at.
>
> It applies to -next except for the QEMU script which you rejected. I
> can't do anything about that. So I think you may have the
> wrong end of the stick here.
>
> The only thing I can think of is that the UPL series depends on
> Raymond's bloblist patch [1]. But I was assuming that would be applied
> at some point and I didn't want to resend someone else's patch. I
> don't have it in the sjg tree either yet, as I was hoping he would
> write a test. In fact I would very-much like people to write tests
> (and docs if needed) without needing to be asked.
>
> Regards,
> Simon
>
> [1] https://lore.kernel.org/u-boot/CAFLszTjSGqmk8yYYbXFEaFZWFhF8uhj_rNxZz7ship1-Fb8p3Q@mail.gmail.com/
A 60 part series that includes changes to something that was rejected,
and depends on a series from someone else that was changes requested,
and is again, 60+ patches, is not at all helpful. Please stop doing
this.
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
next prev parent reply other threads:[~2025-01-03 14:27 UTC|newest]
Thread overview: 76+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-01-01 22:08 [PATCH 00/67] upl: Align with the updated spec Simon Glass
2025-01-01 22:08 ` [PATCH 01/67] bloblist: Make BLOBLIST_ALLOC the default Simon Glass
2025-01-01 22:08 ` [PATCH 02/67] abuf: Provide a way to get the buffer address Simon Glass
2025-01-01 22:08 ` [PATCH 03/67] abuf: Allow use in host tools Simon Glass
2025-01-01 22:08 ` [PATCH 04/67] abuf: Provide a constant buffer Simon Glass
2025-01-01 22:08 ` [PATCH 05/67] cpu: Provide a way to get the physical-address size Simon Glass
2025-01-01 22:08 ` [PATCH 06/67] serial: Support info() method in ns16550 xPL with UPL Simon Glass
2025-01-01 22:08 ` [PATCH 07/67] mkimage: Update map_to_sysmem() to match its prototype Simon Glass
2025-01-01 22:08 ` [PATCH 08/67] x86: Emable meminfo command Simon Glass
2025-01-03 14:51 ` Heinrich Schuchardt
2025-01-04 19:30 ` Simon Glass
2025-01-01 22:08 ` [PATCH 09/67] x86: Show the timestamp counter with bdinfo Simon Glass
2025-01-01 22:08 ` [PATCH 10/67] ofnode: Use 4K for a default tree-size Simon Glass
2025-01-01 22:08 ` [PATCH 11/67] ofnode: Indicate when out of space in a few places Simon Glass
2025-01-01 22:08 ` [PATCH 12/67] ofnode: Update of_add_subnode() to indicate name is alloced Simon Glass
2025-01-01 22:08 ` [PATCH 13/67] boot: Rename fit_image_get_data() Simon Glass
2025-01-03 14:55 ` Heinrich Schuchardt
2025-01-01 22:09 ` [PATCH 14/67] boot: Rename fit_image_get_data_and_size() Simon Glass
2025-01-01 22:09 ` [PATCH 15/67] boot: Update fit_image_get_emb_data to use abuf Simon Glass
2025-01-01 22:09 ` [PATCH 16/67] boot: Use fit_image_get_data() to get data Simon Glass
2025-01-01 22:09 ` [PATCH 17/67] boot: Update fit_image_get_data() to use abuf Simon Glass
2025-01-01 22:09 ` [PATCH 18/67] test: Fix inpected typo in upl test Simon Glass
2025-01-01 22:09 ` [PATCH 19/67] emulation: fdt: Relax condition for OF_HAS_PRIOR_STAGE Simon Glass
2025-01-01 22:09 ` [PATCH 20/67] emulation: Use bloblist to hold tables Simon Glass
2025-01-01 22:09 ` [PATCH 21/67] x86: Create more space for SPL with qemu-x86_64 Simon Glass
2025-01-01 22:09 ` [PATCH 22/67] upl: Create a function to create a memory node Simon Glass
2025-01-01 22:09 ` [PATCH 23/67] upl: Update add_upl_memmap() to use write_mem_node() Simon Glass
2025-01-01 22:09 ` [PATCH 24/67] upl: Update add_upl_memres() " Simon Glass
2025-01-01 22:09 ` [PATCH 25/67] upl: Support writing a UPL only for testing Simon Glass
2025-01-01 22:09 ` [PATCH 26/67] upl: Drop enum upl_serial_access_type Simon Glass
2025-01-01 22:09 ` [PATCH 27/67] upl: Init serial and graphics alists in upl_init() Simon Glass
2025-01-01 22:09 ` [PATCH 28/67] upl: Move serial and graphics addition-code to upl_common Simon Glass
2025-01-01 22:09 ` [PATCH 29/67] pci: video: Set up the pixel-format field Simon Glass
2025-01-01 22:09 ` [PATCH 30/67] x86: Show an error if video fails Simon Glass
2025-01-01 22:09 ` [PATCH 31/67] x86: Support jumping to a UPL image Simon Glass
2025-01-01 22:09 ` [PATCH 32/67] x86: Enable UPL handoff for SPL Simon Glass
2025-01-01 22:09 ` [PATCH 33/67] x86: Align the SMBIOS table to a 4K boundary Simon Glass
2025-01-03 15:10 ` Heinrich Schuchardt
2025-01-01 22:09 ` [PATCH 34/67] upl: Require OFNODE_MULTI_TREE when writing a UPL handoff Simon Glass
2025-01-01 22:09 ` [PATCH 35/67] upl: Drop the argument to spl_write_upl_handoff() Simon Glass
2025-01-01 22:09 ` [PATCH 36/67] x86: emulation: Enable UPL Simon Glass
2025-01-01 22:09 ` [PATCH 37/67] upl: Correct comment and condition in add_upl_memres() Simon Glass
2025-01-01 22:09 ` [PATCH 38/67] upl: Add addr/size tags in the reserved-memory node Simon Glass
2025-01-01 22:09 ` [PATCH 39/67] upl: Update upl_add_graphics() to return framebuffer Simon Glass
2025-01-01 22:09 ` [PATCH 40/67] upl: Add a function to write a UPL handoff to an abuf Simon Glass
2025-01-01 22:09 ` [PATCH 41/67] upl: Use a 64-bit value for a memregion base-address Simon Glass
2025-01-01 22:09 ` [PATCH 42/67] upl: Set bit 32 of the address when using ISA Simon Glass
2025-01-01 22:09 ` [PATCH 43/67] upl: Add RAM to the memory region Simon Glass
2025-01-01 22:09 ` [PATCH 44/67] upl: Add a compatible string for the upl-params node Simon Glass
2025-01-01 22:09 ` [PATCH 45/67] upl: Move buffer_addr_size() higher and rename Simon Glass
2025-01-01 22:09 ` [PATCH 46/67] upl: Factor out part of encode_reg() to new function Simon Glass
2025-01-01 22:09 ` [PATCH 47/67] upl: Move decode_addr_size() higher and rename Simon Glass
2025-01-01 22:09 ` [PATCH 48/67] upl: Factor out part of decode_reg() to new function Simon Glass
2025-01-01 22:09 ` [PATCH 49/67] upl: Correct name of upl-images Simon Glass
2025-01-01 22:09 ` [PATCH 50/67] dm: core: Clarify behaviour of ofnode_name_eq() Simon Glass
2025-01-01 22:09 ` [PATCH 51/67] dm: core: Allow ofnode_name_eq() to accept a unit address Simon Glass
2025-01-01 22:09 ` [PATCH 52/67] dm: core: Rewrite ofnode_find_subnode() to use ofnode API Simon Glass
2025-01-01 22:09 ` [PATCH 53/67] upl: test: Show the handoff FDT when debugging Simon Glass
2025-01-01 22:09 ` [PATCH 54/67] upl: Use a reg property to specify the FIT address and size Simon Glass
2025-01-01 22:09 ` [PATCH 55/67] upl: Update and tidy writing of images Simon Glass
2025-01-01 22:09 ` [PATCH 56/67] upl: Make use of alist_for_each() Simon Glass
2025-01-01 22:09 ` [PATCH 57/67] upl: Correct use of sizeof() Simon Glass
2025-01-01 22:09 ` [PATCH 58/67] upl: Drop acpi_nvs_size Simon Glass
2025-01-01 22:09 ` [PATCH 59/67] upl: Improve uniqueness of log_msg_ret() strings Simon Glass
2025-01-01 22:09 ` [PATCH 60/67] upl: Add PCI information Simon Glass
2025-01-01 22:09 ` [PATCH 61/67] upl: Handle serial on an ISA bus with /chosen node Simon Glass
2025-01-01 22:09 ` [PATCH 62/67] upl: Add missing word in comment for upl_write_handoff() Simon Glass
2025-01-01 22:09 ` [PATCH 63/67] upl: Add a command to execute a UPL payload Simon Glass
2025-01-01 22:09 ` [PATCH 64/67] scripts: Update qemu script to use 64-bit on x86 always Simon Glass
2025-01-01 22:09 ` [PATCH 65/67] scripts: Update qemu script to support specifying a UPL Simon Glass
2025-01-01 22:09 ` [PATCH 66/67] upl: Add reserved memory for the ACPI and SMBIOS tables Simon Glass
2025-01-01 22:09 ` [PATCH 67/67] upl: Add an 'addr' property for Tianocore Simon Glass
2025-01-02 14:44 ` [PATCH 00/67] upl: Align with the updated spec Tom Rini
2025-01-02 23:45 ` Simon Glass
2025-01-03 14:26 ` Tom Rini [this message]
2025-01-04 19:32 ` 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=20250103142651.GU3476@bill-the-cat \
--to=trini@konsulko.com \
--cc=Zhiqiang.Hou@nxp.com \
--cc=ada@thorsis.com \
--cc=agraf@csgraf.de \
--cc=agust@denx.de \
--cc=alpernebiyasak@gmail.com \
--cc=caleb.connolly@linaro.org \
--cc=di.gerasimov@gmail.com \
--cc=hs@denx.de \
--cc=hugo.cornelis@essensium.com \
--cc=igor.opaniuk@gmail.com \
--cc=ilias.apalodimas@linaro.org \
--cc=j@jannau.net \
--cc=jagan@amarulasolutions.com \
--cc=jiaxun.yang@flygoat.com \
--cc=jonas@kwiboo.se \
--cc=kojima.masahisa@socionext.com \
--cc=linux.amoon@gmail.com \
--cc=lukas.funke@weidmueller.com \
--cc=marek.mojik@nic.cz \
--cc=maximmosk4@gmail.com \
--cc=michal.simek@amd.com \
--cc=miquel.raynal@bootlin.com \
--cc=mkorpershoek@baylibre.com \
--cc=morpheus.ibis@gmail.com \
--cc=n-jain1@ti.com \
--cc=nabijaczleweli@nabijaczleweli.xyz \
--cc=neil.armstrong@linaro.org \
--cc=patrick.delaunay@foss.st.com \
--cc=pbrobinson@gmail.com \
--cc=pro@denx.de \
--cc=rasmus.villemoes@prevas.dk \
--cc=raymond.mao@linaro.org \
--cc=sbabic@denx.de \
--cc=seanga2@gmail.com \
--cc=sjg@chromium.org \
--cc=u-boot@lists.denx.de \
--cc=vanbaren@cideas.com \
--cc=wens@csie.org \
--cc=xypron.glpk@gmx.de \
--cc=ycliang@andestech.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