<p><a href="https://review.coreboot.org/28258">View Change</a></p><p>8 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/28258/2/Documentation/mainboard/intel/dg43gt.md">File Documentation/mainboard/intel/dg43gt.md:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28258/2/Documentation/mainboard/intel/dg43gt.md@17">Patch Set #2, Line 17:</a> <code style="font-family:monospace,monospace"> NO!</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Any reason for this to be so intense? In any case, please harmonize the "yes" and "no" in the table (Yes/yes).</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28258/2/Documentation/mainboard/intel/dg43gt.md@32">Patch Set #2, Line 32:</a> <code style="font-family:monospace,monospace">to written</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">to *be* written</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28258/2/Documentation/mainboard/intel/dg43gt.md@34">Patch Set #2, Line 34:</a> <code style="font-family:monospace,monospace">--noverify</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Is there any reason for this? If it is not necessary, I would omit it.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28258/2/Documentation/mainboard/intel/dg43gt.md@45">Patch Set #2, Line 45:</a> <code style="font-family:monospace,monospace">reprogram </code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">maybe "reprogram it"?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">With the sentence as-is, I understand you can both reprogram and resolder the chip using a soldering iron.</p><p style="white-space: pre-wrap; word-wrap: break-word;">IMHO, I would say: "Having removed the flash chip, you can reprogram it externally then resolder it using a soldering iron.".</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28258/2/Documentation/mainboard/intel/dg43gt.md@51">Patch Set #2, Line 51:</a> <code style="font-family:monospace,monospace">this</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">*T*his</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28258/2/Documentation/mainboard/intel/dg43gt.md@53">Patch Set #2, Line 53:</a> <code style="font-family:monospace,monospace">3V</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">3.3V, maybe?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28258/2/Documentation/mainboard/intel/dg43gt.md@88">Patch Set #2, Line 88:</a> <code style="font-family:monospace,monospace">Windond</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Win*b*ond</p><p style="white-space: pre-wrap; word-wrap: break-word;">Should the model be in uppercase? (W83627DHG)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28258/2/Documentation/mainboard/intel/dg43gt.md@90">Patch Set #2, Line 90:</a> <code style="font-family:monospace,monospace">optionnaly</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">optionally</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/28258">change 28258</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/28258"/><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: comment </div>
<div style="display:none"> Gerrit-Change-Id: I4e9dc67e66f719d440679b11332e2c8a764024f4 </div>
<div style="display:none"> Gerrit-Change-Number: 28258 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Arthur Heymans <arthur@aheymans.xyz> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-CC: Angel Pons <th3fanbus@gmail.com> </div>
<div style="display:none"> Gerrit-CC: Felix Held <felix-coreboot@felixheld.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 28 Aug 2018 14:23:55 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>