Applying patches to mainline (Was: Re: [PATCH v2 0/7] efi_loader: Add support for logging to a buffer)

Tom Rini trini at konsulko.com
Wed Dec 4 02:29:07 CET 2024


On Mon, Dec 02, 2024 at 05:21:05PM -0700, Simon Glass wrote:

> From my side I'd like to change the conversation a little, to how to
> land code, rather than why we should bother. "Code needs to land"
> should be the motto. If someone has taken the time to create
> something, our bias should be towards getting it in, with sufficient
> changes to make it fit the project. There are cases where something is
> just a bad idea, or should be done another way, but for people working
> on major features or changes, biasing towards not landing the code is
> just going to make them go elsewhere.

This is the wrong approach I believe. The goal has always been and
continues to be to have reviewed (whenever possible, our developer
community is small) incremental change over time. Just because something
has been posted a number of times does not mean it's ready to be merged.

Your challenge today is that on patchwork you have over 150 patches
covering a wide variety of topics and of which many series have
technically-merited feedback that needs to be addressed in a technical
manner.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20241203/6290c341/attachment.sig>


More information about the U-Boot mailing list