<p>Andrey Korolyov has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21005">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">drivers/ics954309: conditionalize clockgen setup<br><br>The Z61m board need not to set four upper bits on the clockgen<br>address due to platform-specific issue effectively preventing<br>writes to this area.<br><br>Change-Id: Ib17a67a31658256040d63cd45b354c1dfced952a<br>Signed-off-by: Andrey Korolyov <andrey@xdel.ru><br>---<br>M src/drivers/ics/954309/ics954309.c<br>1 file changed, 9 insertions(+), 3 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/05/21005/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/drivers/ics/954309/ics954309.c b/src/drivers/ics/954309/ics954309.c<br>index a3d1d81..dc334cd 100644<br>--- a/src/drivers/ics/954309/ics954309.c<br>+++ b/src/drivers/ics/954309/ics954309.c<br>@@ -27,7 +27,12 @@<br> static void ics954309_init(struct device *dev)<br> {<br>     struct drivers_ics_954309_config *config;<br>-    u8 initdata[12];<br>+#ifdef CONFIG_BOARD_LENOVO_Z61M<br>+   int p = 8;<br>+#else<br>+   int p = 12;<br>+#endif<br>+ u8 initdata[p];<br> <br>    if (!dev->enabled || dev->path.type != DEVICE_PATH_I2C)<br>                 return;<br>@@ -42,12 +47,13 @@<br>  initdata[5] = config->reg5;<br>        initdata[6] = config->reg6;<br>        initdata[7] = config->reg7;<br>+#ifndef CONFIG_BOARD_LENOVO_Z61M<br>     initdata[8] = config->reg8;<br>        initdata[9] = config->reg9;<br>        initdata[10] = config->reg10;<br>      initdata[11] = config->reg11;<br>-<br>-  smbus_block_write(dev, 0, 12, initdata);<br>+#endif<br>+    smbus_block_write(dev, 0, p, initdata);<br> }<br> <br> static struct device_operations ics954309_operations = {<br></pre><p>To view, visit <a href="https://review.coreboot.org/21005">change 21005</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/21005"/><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: Ib17a67a31658256040d63cd45b354c1dfced952a </div>
<div style="display:none"> Gerrit-Change-Number: 21005 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Andrey Korolyov <andrey@xdel.ru> </div>