<p><a href="https://review.coreboot.org/c/coreboot/+/29749">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/29749/9/src/mainboard/google/dragonegg/dsdt.asl">File src/mainboard/google/dragonegg/dsdt.asl:</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/29749/9/src/mainboard/google/dragonegg/dsdt.asl@22">Patch Set #9, Line 22:</a> <code style="font-family:monospace,monospace">0x05</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">the revision is 0x02</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Although there is no harm to declare this value as 0x5 :)<br>but still i will make this as 0x02 as you have asked.</p><p style="white-space: pre-wrap; word-wrap: break-word;">here is the recommendation from spec</p><p style="white-space: pre-wrap; word-wrap: break-word;">"A revision field value greater than or equal to 2 signifies that integers declared within the Definition Block are to be evaluated as 64 bit values"</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/29749/9/src/mainboard/google/dragonegg/romstage_fsp_params.c">File src/mainboard/google/dragonegg/romstage_fsp_params.c:</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/29749/9/src/mainboard/google/dragonegg/romstage_fsp_params.c@23">Patch Set #9, Line 23:</a> <code style="font-family:monospace,monospace">void mainboard_memory_init_params(FSPM_UPD *mupd)</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Can this file be removed, since it contains 1 empty function only?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">We can't remove this file because we might have some overrides as epatch which will apply over this file during chromium build.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/29749/9/src/mainboard/google/dragonegg/spd/empty.spd.hex">File src/mainboard/google/dragonegg/spd/empty.spd.hex:</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/29749/9/src/mainboard/google/dragonegg/spd/empty.spd.hex@1">Patch Set #9, Line 1:</a> <code style="font-family:monospace,monospace">00 </code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Why this empty spd file? Remove it if not used</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">make sense</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/c/coreboot/+/29749">change 29749</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/+/29749"/><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: I2cc269cb0050bf5b031f48cfe114485c55ab8fa9 </div>
<div style="display:none"> Gerrit-Change-Number: 29749 </div>
<div style="display:none"> Gerrit-PatchSet: 9 </div>
<div style="display:none"> Gerrit-Owner: Shelley Chen <shchen@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Aamir Bohra <aamir.bohra@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> </div>
<div style="display:none"> Gerrit-Reviewer: Shelley Chen <shchen@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Subrata Banik <subrata.banik@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-CC: Frans Hendriks <fhendriks@eltan.com> </div>
<div style="display:none"> Gerrit-CC: HAOUAS Elyes <ehaouas@noos.fr> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 22 Nov 2018 08:47:26 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: HAOUAS Elyes <ehaouas@noos.fr> </div>
<div style="display:none"> Comment-In-Reply-To: Frans Hendriks <fhendriks@eltan.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>