<p>Evan Green has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/25643">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">cheza: Fix SPI clocking and pinmuxing<br><br>The original SDM845 SPI code in question clocked the SPI bus at 200MHz.<br>The Winbond part can only go up to 104MHz. This change clocks it at<br>100MHz, which still leaves a little bit of headroom for variations.<br>It also removes the extra data[2] and data[3] pinmuxing, as this board<br>does not use quad SPI.<br><br>BUG=b:74020650<br>TEST=Booting via SPI on Cheza now works<br><br>Change-Id: I0c1432d4e31fa07b192d914c987327ae4e2ed11f<br>---<br>M src/soc/qualcomm/sdm845/fs_spi.c<br>1 file changed, 2 insertions(+), 6 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/43/25643/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/soc/qualcomm/sdm845/fs_spi.c b/src/soc/qualcomm/sdm845/fs_spi.c</span><br><span>index 0199c5b..0197773 100644</span><br><span>--- a/src/soc/qualcomm/sdm845/fs_spi.c</span><br><span>+++ b/src/soc/qualcomm/sdm845/fs_spi.c</span><br><span>@@ -380,8 +380,8 @@</span><br><span> </span><br><span>    *(volatile uint32_t *)(gcc_qspi_base + AHB_CBCR) = 0x00000001;</span><br><span>       *(volatile uint32_t *)(gcc_qspi_base + SER_CBCR) = 0x00000001;</span><br><span style="color: hsl(0, 100%, 40%);">-  // 200MHZ</span><br><span style="color: hsl(0, 100%, 40%);">-       *(volatile uint32_t *)(gcc_qspi_base + SER_CFG_RCGR) = 0x00000105;</span><br><span style="color: hsl(120, 100%, 40%);">+    // 1200MHz / 12 = 100MHZ</span><br><span style="color: hsl(120, 100%, 40%);">+      *(volatile uint32_t *)(gcc_qspi_base + SER_CFG_RCGR) = 0x0000010B;</span><br><span>   // ROOT_EN and UPDATE</span><br><span>        *(volatile uint32_t *)(gcc_qspi_base + SER_CMD_RCGR) = 0x00000003;</span><br><span>   return true;</span><br><span>@@ -433,10 +433,6 @@</span><br><span>  *(volatile uint32_t *)(tlmm_base + 0x1000 * 91) = 0x0011;</span><br><span>    // qspi_data[1]  (2ma, fcn 4, PD)</span><br><span>    *(volatile uint32_t *)(tlmm_base + 0x1000 * 92) = 0x0011;</span><br><span style="color: hsl(0, 100%, 40%);">-       // qspi_data[2]  (2ma, fcn 4, PD)</span><br><span style="color: hsl(0, 100%, 40%);">-       *(volatile uint32_t *)(tlmm_base + 0x1000 * 93) = 0x0011;</span><br><span style="color: hsl(0, 100%, 40%);">-       // qspi_data[3]  (2ma, fcn 4, PD)</span><br><span style="color: hsl(0, 100%, 40%);">-       *(volatile uint32_t *)(tlmm_base + 0x1000 * 94) = 0x0011;</span><br><span>    return true;</span><br><span> }</span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/25643">change 25643</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/25643"/><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: I0c1432d4e31fa07b192d914c987327ae4e2ed11f </div>
<div style="display:none"> Gerrit-Change-Number: 25643 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Evan Green <evgreen@chromium.org> </div>