<div dir="ltr">On Thu, Jun 14, 2018 at 8:28 PM, Merlin Büge <<a href="mailto:toni@bluenox07.de">toni@bluenox07.de</a>> wrote:<br>><br>><br>> On Thu, 14 Jun 2018 09:45:49 +0300<br>> Kyösti Mälkki <<a href="mailto:kyosti.malkki@gmail.com">kyosti.malkki@gmail.com</a>> wrote:<br>><br>>><br>>> I hear you and weigh your opinion according to the number of commits I<br>>> can recognize you have authored on the repo.<br>><br>> I just want to mention:<br>> Generally, helping out with documentation and (especially) code review<br>> and even ordinary users who report bugs are also a value-add to the<br>> coreboot project.<br>><br><br><div>Agreed and Talidan should not get offended by previous comment. I get way more rude and personal in some of my reviews towards commercial vendors. Apologies for that.</div><div><br></div><div>Just recently, cases of RELOCATABLE_RAMSTAGE=n started to fail to boot
certain payload builds. The accumulated developers time it took to
bisect, discuss and come up with a sub-standard solution for that; should I say 8 hours across 5 active developers. That time is better spent elsewhere, I consider the discussion of why
we deprecate older boards from master just not productive at all.<br><br></div><div>Notice that even the little details Matthias provided earlier already moves us forwards.<br></div><br><div>The criteria I listed was my personal wishlist as flipping to RELOCATABLE_RAMSTAGE=y has proven to be quite easy and the remaining platforms are:</div><div><br></div><div><span style="font-family:monospace,monospace"> northbridge/amd/amdfam10</span></div><span style="font-family:monospace,monospace"> northbridge/amd/lx <- waiting for EARLY_CBMEM_INIT<br> northbridge/intel/i440bx <- already tested<br> northbridge/via/vx900 <- I know competent person to do this<br> soc/intel/fsp_baytrail <- I have hardware and know person to double-check <br></span><div><span style="font-family:monospace,monospace"> soc/intel/fsp_broadwell_de <- I know competent person to do this<br></span></div><div><br></div><br>Kyösti</div>