<p><a href="https://review.coreboot.org/27478">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/27478/1/payloads/libpayload/drivers/usb/xhci.c">File payloads/libpayload/drivers/usb/xhci.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/27478/1/payloads/libpayload/drivers/usb/xhci.c@411">Patch Set #1, Line 411:</a> <code style="font-family:monospace,monospace">                      fatal("noop command failed.\n");</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not sure die() is the right move here. This is just a debugging feature anyway, it shouldn't matter for production firmware. For debugging on early bring-up boards, I've seen plenty of cases where there was some kind of error here but the system was still usable afterwards (e.g. in case you didn't want to boot from USB, or if only one of several USB controllers was broken somehow). I'm fine with adding the condition code to the message if you want, but I don't think it should hang.</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/27478">change 27478</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/27478"/><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: Ifa0b2fb087f5f0a36ba017a774fc98b33ab035a4 </div>
<div style="display:none"> Gerrit-Change-Number: 27478 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Raul Rangel <rrangel@chromium.org> </div>
<div style="display:none"> Gerrit-Reviewer: Martin Roth <martinroth@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-CC: Julius Werner <jwerner@chromium.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 13 Jul 2018 22:32:12 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>