<p>Philipp Deppenwiese <strong>posted comments</strong> on this change.</p><p><a href="https://review.coreboot.org/16328">View Change</a></p><p>Patch set 21:</p><p style="white-space: pre-wrap; word-wrap: break-word;">@Nicola which bits 20:21 and where do we get them. Normally the bootguard enforcement policies are somewhere in the ME pci config space  but we don't know where exactly. If you have information about it please give me some input or make modifications yourself.</p><p>(5 comments)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://review.coreboot.org/#/c/16328/9/util/intelmetool/intelmetool.c">File util/intelmetool/intelmetool.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/16328/9/util/intelmetool/intelmetool.c@129">Patch Set #9, Line 129:</a> <code style="font-family:monospace,monospace"></code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">you have ME hardware on board but...</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li></ul></li><li><p><a href="https://review.coreboot.org/#/c/16328/9/util/intelmetool/msr.c">File util/intelmetool/msr.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/16328/9/util/intelmetool/msr.c@29">Patch Set #9, Line 29:</a> <code style="font-family:monospace,monospace">static uint64_t rdmsr(int addr)</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">static int ...</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li><li><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/16328/9/util/intelmetool/msr.c@60">Patch Set #9, Line 60:</a> <code style="font-family:monospace,monospace"></code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">include errno.h instead of hardcoding values</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li><li><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/16328/9/util/intelmetool/msr.c@73">Patch Set #9, Line 73:</a> <code style="font-family:monospace,monospace">#endif</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Return error code and pass msr value by argument</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li><li><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/16328/9/util/intelmetool/msr.c@85">Patch Set #9, Line 85:</a> <code style="font-family:monospace,monospace"></code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">O_RDONLY</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/16328">change 16328</a>. To unsubscribe, 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/16328"/><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: Ifeec8e20fa8efc35d7db4c6a84be1f118dccfc4a </div>
<div style="display:none"> Gerrit-Change-Number: 16328 </div>
<div style="display:none"> Gerrit-PatchSet: 21 </div>
<div style="display:none"> Gerrit-Owner: Philipp Deppenwiese <zaolin.daisuki@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Alexander Couzens <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-Reviewer: Damien Zammit <damien@zamaudio.com> </div>
<div style="display:none"> Gerrit-Reviewer: Felix Held <felix-coreboot@felixheld.de> </div>
<div style="display:none"> Gerrit-Reviewer: Martin Roth <martinroth@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Mathias Krause <minipli@googlemail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Nico Huber <nico.h@gmx.de> </div>
<div style="display:none"> Gerrit-Reviewer: Nicola Corna <nicola@corna.info> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> </div>
<div style="display:none"> Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net> </div>
<div style="display:none"> Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Philippe Mathieu-Daudé <f4bug@amsat.org> </div>
<div style="display:none"> Gerrit-Reviewer: Sumeet R Pawnikar <sumeet.r.pawnikar@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Werner Zeh <werner.zeh@siemens.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 03 Jul 2017 12:09:03 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>