<p>Matt DeVillier has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/25326">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/broadwell: Add option to disable PCIe AER capability<br><br>The Advanced Error Reporting capability was hardcoded in the PCIe<br>extended capability list, but it might not always be possible.<br><br>The Librem 13v1 does not seem to have working AER and this option<br>was needed and tested on the Librem 13v1. Without it, the linux<br>console gets spammed with AER errrors.<br><br>Change-Id: If2e0ec42c93f1fee927eacdf0099004cf9302fbe<br>Signed-off-by: Youness Alaoui <kakaroto@kakaroto.homelinux.net><br>---<br>M src/soc/intel/broadwell/Kconfig<br>M src/soc/intel/broadwell/pcie.c<br>2 files changed, 10 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/26/25326/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/soc/intel/broadwell/Kconfig b/src/soc/intel/broadwell/Kconfig</span><br><span>index cfab489..5d8d602 100644</span><br><span>--- a/src/soc/intel/broadwell/Kconfig</span><br><span>+++ b/src/soc/intel/broadwell/Kconfig</span><br><span>@@ -47,6 +47,10 @@</span><br><span>   bool</span><br><span>         default y</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+config PCIEXP_AER</span><br><span style="color: hsl(120, 100%, 40%);">+      bool</span><br><span style="color: hsl(120, 100%, 40%);">+  default y</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> config PCIEXP_COMMON_CLOCK</span><br><span>    bool</span><br><span>         default y</span><br><span>diff --git a/src/soc/intel/broadwell/pcie.c b/src/soc/intel/broadwell/pcie.c</span><br><span>index cf25749..53a1eac 100644</span><br><span>--- a/src/soc/intel/broadwell/pcie.c</span><br><span>+++ b/src/soc/intel/broadwell/pcie.c</span><br><span>@@ -554,8 +554,12 @@</span><br><span>        pci_update_config8(dev, 0xf5, 0x0f, 0);</span><br><span> </span><br><span>  /* Set AER Extended Cap ID to 01h and Next Cap Pointer to 200h. */</span><br><span style="color: hsl(0, 100%, 40%);">-      pci_update_config32(dev, 0x100, ~(1 << 29) & ~0xfffff,</span><br><span style="color: hsl(0, 100%, 40%);">-                (1 << 29) | 0x10001);</span><br><span style="color: hsl(120, 100%, 40%);">+   if (IS_ENABLED(CONFIG_PCIEXP_AER))</span><br><span style="color: hsl(120, 100%, 40%);">+            pci_update_config32(dev, 0x100, ~(1 << 29) & ~0xfffff,</span><br><span style="color: hsl(120, 100%, 40%);">+                      (1 << 29) | 0x10001);</span><br><span style="color: hsl(120, 100%, 40%);">+   else</span><br><span style="color: hsl(120, 100%, 40%);">+          pci_update_config32(dev, 0x100, ~(1 << 29) & ~0xfffff,</span><br><span style="color: hsl(120, 100%, 40%);">+                      (1 << 29));</span><br><span> </span><br><span>        /* Set L1 Sub-State Cap ID to 1Eh and Next Cap Pointer to None. */</span><br><span>   if (IS_ENABLED(CONFIG_PCIEXP_L1_SUB_STATE))</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/25326">change 25326</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/25326"/><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: newchange </div>
<div style="display:none"> Gerrit-Change-Id: If2e0ec42c93f1fee927eacdf0099004cf9302fbe </div>
<div style="display:none"> Gerrit-Change-Number: 25326 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Matt DeVillier <matt.devillier@gmail.com> </div>