<div dir="ltr"><div class="gmail_quote" style="font-size:small;line-height:20px">As Wim pointed out, it is possible, but not exactly clear as there was a big effort to clean up the buildopts and agesa wrappers. We have a slightly different implementation at Sage, but we are working up something that will work with the <a href="http://coreboot.org">coreboot.org</a> code. Here is what you need to know.</div><div class="gmail_quote" style="font-size:small;line-height:20px"><br></div><div class="gmail_quote" style="font-size:small;line-height:20px">The gizmosphere/gizmo2 is an example of how a board with soldered down memory has the spd added to the rom image, but there are some differences with the AGESA binary PI implementation.<br></div><div class="gmail_quote" style="font-size:small;line-height:20px"><div class="gmail_quote"><br></div><div class="gmail_quote">* Add a .spd.hex file to the mainboard folder.. Everything useful pretty much is defined in the first 30 bytes. Leave the CRC checksum bytes at locations 126/127 as zeroes so that the CRC will get calculated runtime.</div><div class="gmail_quote">* Update the mainboard Makefile.inc to include the .spd.hex</div><div class="gmail_quote">* Update the mainboard Kconfig to select SPD_CACHE</div><div class="gmail_quote">* Update the mainboard BiosCallOuts.c file for the function to retrieve the SPD data from {AGESA_READ_SPD, agesa_ReadSpd } to {AGESA_READ_SPD, agesa_ReadSpd_from_cbfs }</div><div class="gmail_quote">* Update  devicetree.cb file for turn off the SMBUS, dev 14.0. Remove the lines for the chip drivers and remove the [register "spdAddrLookup" = "...], since the SPD data is coming from CBFS.</div><div class="gmail_quote">* Add PSO_ENTRY ROMDATA PlatformMemoryConfiguration[]  with all settings into a a file under the mainboard and then add  it to agesawrapper_amdinitpost() : </div><div class="gmail_quote">PostParams->MemConfig.PlatformMemoryConfiguration = PlatformMemoryConfiguration;. </div><div class="gmail_quote"><br></div><div class="gmail_quote">Since agesawrapper.c was moved to the chipset directory we need to check for the table in the mainboard, but don't currently have a mechanism.  We have some thoughts on how to do it, but it isn't prepared yet.</div><div class="gmail_quote"><br></div><div class="gmail_quote">Regards,</div><div class="gmail_quote">Marc</div><div class="gmail_quote"><br></div></div></div><br><div class="gmail_quote">On Wed, May 13, 2015 at 3:28 AM Kyösti Mälkki <<a href="mailto:kyosti.malkki@gmail.com">kyosti.malkki@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On ke, 2015-05-13 at 10:18 +0200, Wim Vervoorn wrote:<br>
> Hello Kyosti,<br>
><br>
> I do agree with you that it is much easier and straight forward to change memory parameters when you are using the source agesa but it is definitely possible to do this for the binary agesa as well. You can provide the binary AGESA with external tables during run-time.<br>
><br>
<br>
Thanks Wim!<br>
<br>
I can see how the relevant table that used to be in buildOPts.c is now<br>
implemented for DB-FT3b-LC board in agesawrapper.c.<br>
<br>
So Wolfgang, you can pay for the development, which IMHO should not be<br>
more than 2 hours of engineering time, given the infrastructure is<br>
already there in binaryPI. Or request a free copy of the GPL'd<br>
DB-FT3b-LC coreboot OSP package from Sage (<a href="http://se-eng.com" target="_blank">se-eng.com</a>) to see for<br>
yourself where the table has to seat.<br>
<br>
Or you can wait until DB-FT3b-LC board support appears upstreamed at<br>
<a href="http://coreboot.org" target="_blank">coreboot.org</a>. I have this board here on my desk so it might not take<br>
that long.<br>
<br>
Regards,<br>
Kyösti<br>
<br>
<br>
<br>
--<br>
coreboot mailing list: <a href="mailto:coreboot@coreboot.org" target="_blank">coreboot@coreboot.org</a><br>
<a href="http://www.coreboot.org/mailman/listinfo/coreboot" target="_blank">http://www.coreboot.org/mailman/listinfo/coreboot</a></blockquote></div>