<p>Angel Pons has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/28833">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Documentation/mainboard/gigabyte/ga-h61m-s2pv: Expand page<br><br>Uniformize the Yes/No in the tables, expand the internal programming<br>section and explain how to patch a defective flash descriptor.<br><br>Change-Id: I972bb8948c29ce0eba46daa92ce6b6052db7b063<br>Signed-off-by: Angel Pons <th3fanbus@gmail.com><br>---<br>M Documentation/mainboard/gigabyte/ga-h61m-s2pv.md<br>1 file changed, 35 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/33/28833/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/Documentation/mainboard/gigabyte/ga-h61m-s2pv.md b/Documentation/mainboard/gigabyte/ga-h61m-s2pv.md</span><br><span>index 54c647e..05c793b 100644</span><br><span>--- a/Documentation/mainboard/gigabyte/ga-h61m-s2pv.md</span><br><span>+++ b/Documentation/mainboard/gigabyte/ga-h61m-s2pv.md</span><br><span>@@ -9,13 +9,13 @@</span><br><span> +---------------------+------------+</span><br><span> | Type | Value |</span><br><span> +=====================+============+</span><br><span style="color: hsl(0, 100%, 40%);">-| Socketed flash | no |</span><br><span style="color: hsl(120, 100%, 40%);">+| Socketed flash | No |</span><br><span> +---------------------+------------+</span><br><span> | Model | MX25L3206E |</span><br><span> +---------------------+------------+</span><br><span> | Size | 4 MiB |</span><br><span> +---------------------+------------+</span><br><span style="color: hsl(0, 100%, 40%);">-| In circuit flashing | yes |</span><br><span style="color: hsl(120, 100%, 40%);">+| In circuit flashing | Yes |</span><br><span> +---------------------+------------+</span><br><span> | Package | SOIC-8 |</span><br><span> +---------------------+------------+</span><br><span>@@ -23,13 +23,43 @@</span><br><span> +---------------------+------------+</span><br><span> | Dual BIOS feature | Yes |</span><br><span> +---------------------+------------+</span><br><span style="color: hsl(0, 100%, 40%);">-| Internal flashing | yes |</span><br><span style="color: hsl(120, 100%, 40%);">+| Internal flashing | Yes |</span><br><span> +---------------------+------------+</span><br><span> ```</span><br><span> </span><br><span> ### Internal programming</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-The main SPI flash can be accessed using [flashrom].</span><br><span style="color: hsl(120, 100%, 40%);">+The main SPI flash can be accessed using [flashrom]. The DualBIOS backup flash</span><br><span style="color: hsl(120, 100%, 40%);">+chip is accessible as well using the `dualbiosindex` programmer parameter.</span><br><span style="color: hsl(120, 100%, 40%);">+Since the flash recovery mechanism works even with coreboot installed on the</span><br><span style="color: hsl(120, 100%, 40%);">+main flash chip (it still restores the vendor UEFI though), it is useful to</span><br><span style="color: hsl(120, 100%, 40%);">+leave the backup chip untouched.</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+### Notes about the original firmware</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+The original IFD defines the BIOS region as the whole flash chip. While this is</span><br><span style="color: hsl(120, 100%, 40%);">+not an issue if flashing a complete image, it confuses flashrom and trashes the</span><br><span style="color: hsl(120, 100%, 40%);">+flash chip's contents when using the--ifd option. However, this can be easily</span><br><span style="color: hsl(120, 100%, 40%);">+fixed by reading the IFD with flashrom, editing the correct values into it with</span><br><span style="color: hsl(120, 100%, 40%);">+ifdtool and then reflashing it.</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+Create a layout.txt with the following contents:</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+ 00000000:00000fff fd</span><br><span style="color: hsl(120, 100%, 40%);">+ 00180000:003fffff bios</span><br><span style="color: hsl(120, 100%, 40%);">+ 00001000:0017ffff me</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+After that, simply run:</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+```bash</span><br><span style="color: hsl(120, 100%, 40%);">+sudo flashrom -p internal --ifd -i fd -r ifd.rom</span><br><span style="color: hsl(120, 100%, 40%);">+ifdtool -n layout.txt ifd.rom</span><br><span style="color: hsl(120, 100%, 40%);">+sudo flashrom -p internal --ifd -i fd -w ifd.rom.new</span><br><span style="color: hsl(120, 100%, 40%);">+```</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+After flashing, power cycle the computer to ensure the new IFD is being used.</span><br><span style="color: hsl(120, 100%, 40%);">+If only a reboot is done, the old IFD layout is still seen by flashrom, even if</span><br><span style="color: hsl(120, 100%, 40%);">+the IFD on the flash chip is correctly defining the new region layout.</span><br><span> </span><br><span> ## Technology</span><br><span> </span><br><span>@@ -43,7 +73,7 @@</span><br><span> +------------------+--------------------------------------------------+</span><br><span> | SuperIO | ITE IT8728F |</span><br><span> +------------------+--------------------------------------------------+</span><br><span style="color: hsl(0, 100%, 40%);">-| EC | |</span><br><span style="color: hsl(120, 100%, 40%);">+| EC | None |</span><br><span> +------------------+--------------------------------------------------+</span><br><span> | Coprocessor | Intel ME |</span><br><span> +------------------+--------------------------------------------------+</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/28833">change 28833</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/28833"/><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-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I972bb8948c29ce0eba46daa92ce6b6052db7b063 </div>
<div style="display:none"> Gerrit-Change-Number: 28833 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Angel Pons <th3fanbus@gmail.com> </div>