<p>Arthur Heymans <strong>uploaded patch set #7</strong> to this change.</p><p><a href="https://review.coreboot.org/c/coreboot/+/30242">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mb/foxconn/g41s-k: Add g41m variant<br><br>Was tested with the following:<br>- 2 DIMM slots<br>- USB<br>- Ethernet NIC<br>- automatic fan control<br>- Libgfxinit with VGA, DVI (HDMI slot unpopulated)<br>- PS2 Keyboard<br>- SATA<br>- PEG<br>- S3 resume<br><br>What does not work:<br>- Using the second DIMM slot on a channel<br>  G41 can only handle 2 ranks per channel and on this mainboard 1 rank<br>  per DIMM slot. Supporting this would require too much raminit rework<br>  and is not worth it (at least for me)<br><br>Change-Id: I67784038ef929f561b82365f00db70a69c024321<br>Signed-off-by: Arthur Heymans <arthur@aheymans.xyz><br>---<br>M src/mainboard/foxconn/g41s-k/Kconfig<br>M src/mainboard/foxconn/g41s-k/Kconfig.name<br>M src/mainboard/foxconn/g41s-k/Makefile.inc<br>M src/mainboard/foxconn/g41s-k/acpi/superio.asl<br>M src/mainboard/foxconn/g41s-k/hda_verb.c<br>M src/mainboard/foxconn/g41s-k/romstage.c<br>A src/mainboard/foxconn/g41s-k/variants/g41m/acpi/ich7_pci_irqs.asl<br>A src/mainboard/foxconn/g41s-k/variants/g41m/data.vbt<br>A src/mainboard/foxconn/g41s-k/variants/g41m/devicetree.cb<br>A src/mainboard/foxconn/g41s-k/variants/g41m/gma-mainboard.ads<br>R src/mainboard/foxconn/g41s-k/variants/g41s-k/acpi/ich7_pci_irqs.asl<br>R src/mainboard/foxconn/g41s-k/variants/g41s-k/data.vbt<br>R src/mainboard/foxconn/g41s-k/variants/g41s-k/devicetree.cb<br>R src/mainboard/foxconn/g41s-k/variants/g41s-k/gma-mainboard.ads<br>14 files changed, 293 insertions(+), 5 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/42/30242/7</pre><p>To view, visit <a href="https://review.coreboot.org/c/coreboot/+/30242">change 30242</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/c/coreboot/+/30242"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I67784038ef929f561b82365f00db70a69c024321 </div>
<div style="display:none"> Gerrit-Change-Number: 30242 </div>
<div style="display:none"> Gerrit-PatchSet: 7 </div>
<div style="display:none"> Gerrit-Owner: Arthur Heymans <arthur@aheymans.xyz> </div>
<div style="display:none"> Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz> </div>
<div style="display:none"> Gerrit-Reviewer: Martin Roth <martinroth@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>