Attention is currently required from: Ravi kumar, Martin Roth, Taniya Das, mturney mturney. Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55079 )
Change subject: sc7180: Add target specific GPIO pin definitions ......................................................................
Patch Set 15:
(1 comment)
Patchset:
PS15: This has been our usual workflow on these projects so I'd like to keep it working. All these different drivers are usually worked on at the same time (and often by different people in parallel), so we can't really just start with one part, clean it up all the way and only then begin working on the other.
Is there any other way we can address the builder performance issue? Maybe Jenkins could be changed to only verify the first X patches in each train or something? Would it help if we set the WIP flag when uploading and then only mark a patch as ready for review when it's ready to go in?
When pushing long patch trains, it is recommended to only push the full patch train once - the initial time, and only to rebase three or four patches at a time.
I'm not really sure how this is supposed to work, maybe I'm misunderstanding something. Rebasing any patch in the train automatically rebases the ones on top of it, right? Like I said we tend to work on many of these at once so it's not really possible to only work on the ones near the top.
-
Julius Werner (Code Review)