[coreboot] Squash or not squash current Haswell patches?

Stefan Reinauer reinauer at google.com
Wed Mar 13 01:03:13 CET 2013

Hi, Paul,

On Tue, Mar 12, 2013 at 3:01 PM, Paul Menzel <
paulepanter at users.sourceforge.net> wrote:

> first thanks to Google and Aaron for the Haswell patches. As most of
> them seem to be fix-up patches I would say to squash those into the
> Haswell commit and add comments into the code were appropriate.
> That would reduce the amount of patches and the one Haswell commit would
> work right away(?).
> Or what do I miss?

In the past a lot of the times new components have been published in one
big chunk without development history. Doing so is the easiest way for the
contributor, but it does not offer a very high degree of transparency about
how the code was developed, and why it looks the way it looks. It also
creates very large patches, which some in the coreboot community have
disliked in the past. The code is not finished yet. Expect more patches
even after the current merge is done.

Taking the feedback from previous contributions into regard, we decided to
send out more fine grained contributions, and send them out as early as we
possibly can, while making the review process easy on the community by not
removing our thought process from the contribution.

I realize that large contributions and development at an incredibly high
pace are a significant effort for the community to handle, and so I would
like to make sure that we contribute, and continue contributing, in the
best possible way. No matter how we form out the process, there will always
be room for improvement, and there will always be time pressure on getting
those contributions integrated.

Seeing both sides of the medal, being a corporate contributor as well as a
maintainer of the coreboot community project, I am very happy about
feedback on how to make life better and easier for everyone in the
community while maintaining a high quality code base.

Stefan Reinauer
Google Inc.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20130312/2b24945a/attachment-0001.html>

More information about the coreboot mailing list