<p style="white-space: pre-wrap; word-wrap: break-word;">What doesn't work or needs to be implemented ?</p><p><a href="https://review.coreboot.org/22972">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/22972/2//COMMIT_MSG">Commit Message:</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/22972/2//COMMIT_MSG@7">Patch Set #2, Line 7:</a> <code style="font-family:monospace,monospace">mainboard/hp: Add Eliteboot Revolve 810 G1</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Elitebook</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/22972/2/src/mainboard/hp/revolve_810_g1/romstage.c">File src/mainboard/hp/revolve_810_g1/romstage.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/22972/2/src/mainboard/hp/revolve_810_g1/romstage.c@18">Patch Set #2, Line 18:</a> <code style="font-family:monospace,monospace">#include <stdint.h></code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please remove obsolete headers</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/22972/2/src/mainboard/hp/revolve_810_g1/romstage.c@91">Patch Set #2, Line 91:</a> <code style="font-family:monospace,monospace">  void *spd_file = cbfs_boot_map_with_leak("spd.bin", CBFS_TYPE_SPD,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please don't leak the file</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/22972">change 22972</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/22972"/><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: Iba9c361591697e6a2b3b7b485f7f1649c2a83524 </div>
<div style="display:none"> Gerrit-Change-Number: 22972 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Bill XIE <persmule@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Iru Cai <mytbk920423@gmail.com> </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-Comment-Date: Fri, 22 Dec 2017 08:46:35 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>