<p>Martin Roth <strong>posted comments</strong> on this change.</p><p><a href="https://review.coreboot.org/22455">View Change</a></p><p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +2</span></p><p>(1 comment)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://review.coreboot.org/#/c/22455/1/src/soc/amd/stoneyridge/acpi/soc_fch.asl">File src/soc/amd/stoneyridge/acpi/soc_fch.asl:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/22455/1/src/soc/amd/stoneyridge/acpi/soc_fch.asl@4">Patch Set #1, Line 4:</a> <code style="font-family:monospace,monospace">6</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I thought I was required to update the copyright of any file I worked, incl</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">No problem.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The copyright is on the contents of the file.  Technically, you can copyright any change you make, but really it's only for "creative" changes.  Changing whitespace, for example, legally qualifies for a copyright change, but if I saw someone updating the copyright for whitespace fixes, I'd probably object.</p><p style="white-space: pre-wrap; word-wrap: break-word;">In this case, since the contents of the file didn't change, the copyright shouldn't change.</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/22455">change 22455</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/22455"/><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: I5de2020411794bfcd3730789f62af9c9834a018b </div>
<div style="display:none"> Gerrit-Change-Number: 22455 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Richard Spiegel <richard.spiegel@silverbackltd.com> </div>
<div style="display:none"> Gerrit-Reviewer: Martin Roth <martinroth@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Richard Spiegel <richard.spiegel@silverbackltd.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 13 Nov 2017 21:28:08 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>