<p>Lijian Zhao has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/22278">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/skylake: Update coding style for i2c<br><br>From comment from https://review.coreboot.org/#/c/22238/, the coding<br>style need to be update.<br><br>TEST=N/A<br><br>Change-Id: Id022648951c0f11216aa32f422b5095476f82f8c<br>Signed-off-by: Lijian Zhao <lijian.zhao@intel.com><br>---<br>M src/soc/intel/skylake/i2c.c<br>1 file changed, 24 insertions(+), 12 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/78/22278/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/intel/skylake/i2c.c b/src/soc/intel/skylake/i2c.c<br>index dce83e7..55e23e0 100644<br>--- a/src/soc/intel/skylake/i2c.c<br>+++ b/src/soc/intel/skylake/i2c.c<br>@@ -43,12 +43,18 @@<br> int i2c_soc_devfn_to_bus(unsigned int devfn)<br> {<br>   switch (devfn) {<br>-     case PCH_DEVFN_I2C0: return 0;<br>-       case PCH_DEVFN_I2C1: return 1;<br>-       case PCH_DEVFN_I2C2: return 2;<br>-       case PCH_DEVFN_I2C3: return 3;<br>-       case PCH_DEVFN_I2C4: return 4;<br>-       case PCH_DEVFN_I2C5: return 5;<br>+       case PCH_DEVFN_I2C0:<br>+         return 0;<br>+    case PCH_DEVFN_I2C1:<br>+         return 1;<br>+    case PCH_DEVFN_I2C2:<br>+         return 2;<br>+    case PCH_DEVFN_I2C3:<br>+         return 3;<br>+    case PCH_DEVFN_I2C4:<br>+         return 4;<br>+    case PCH_DEVFN_I2C5:<br>+         return 5;<br>     }<br>     return -1;<br> }<br>@@ -56,12 +62,18 @@<br> int i2c_soc_bus_to_devfn(unsigned int bus)<br> {<br>  switch (bus) {<br>-       case 0: return PCH_DEVFN_I2C0;<br>-       case 1: return PCH_DEVFN_I2C1;<br>-       case 2: return PCH_DEVFN_I2C2;<br>-       case 3: return PCH_DEVFN_I2C3;<br>-       case 4: return PCH_DEVFN_I2C4;<br>-       case 5: return PCH_DEVFN_I2C5;<br>+       case 0:<br>+              return PCH_DEVFN_I2C0;<br>+       case 1:<br>+              return PCH_DEVFN_I2C1;<br>+       case 2:<br>+              return PCH_DEVFN_I2C2;<br>+       case 3:<br>+              return PCH_DEVFN_I2C3;<br>+       case 4:<br>+              return PCH_DEVFN_I2C4;<br>+       case 5:<br>+              return PCH_DEVFN_I2C5;<br>        }<br>     return -1;<br> }<br></pre><p>To view, visit <a href="https://review.coreboot.org/22278">change 22278</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/22278"/><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: Id022648951c0f11216aa32f422b5095476f82f8c </div>
<div style="display:none"> Gerrit-Change-Number: 22278 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Lijian Zhao <lijian.zhao@intel.com> </div>