<p>Elyes HAOUAS has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/23536">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mainboard/msi/ms9185: Fix coding style<br><br>Change-Id: I3cca4adbf04edfd88a9b8ae52cf4d62d429e6c45<br>Signed-off-by: Elyes HAOUAS <ehaouas@noos.fr><br>---<br>M src/mainboard/msi/ms9185/mptable.c<br>M src/mainboard/msi/ms9185/romstage.c<br>2 files changed, 11 insertions(+), 19 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/36/23536/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/mainboard/msi/ms9185/mptable.c b/src/mainboard/msi/ms9185/mptable.c</span><br><span>index e93602b..e90fd54 100644</span><br><span>--- a/src/mainboard/msi/ms9185/mptable.c</span><br><span>+++ b/src/mainboard/msi/ms9185/mptable.c</span><br><span>@@ -54,7 +54,7 @@</span><br><span>      {</span><br><span>            device_t dev = 0;</span><br><span>            struct resource *res;</span><br><span style="color: hsl(0, 100%, 40%);">-           for(i = 0; i < 3; i++) {</span><br><span style="color: hsl(120, 100%, 40%);">+           for (i = 0; i < 3; i++) {</span><br><span>                         dev = dev_find_device(0x1166, 0x0235, dev);</span><br><span>                  if (dev) {</span><br><span>                           res = find_resource(dev, PCI_BASE_ADDRESS_0);</span><br><span>@@ -80,9 +80,8 @@</span><br><span> </span><br><span> //USB</span><br><span>       outb(0x01, 0xc00); outb(0x0a, 0xc01);</span><br><span style="color: hsl(0, 100%, 40%);">-   for(i = 0; i < 3; i++) {</span><br><span style="color: hsl(120, 100%, 40%);">+   for (i = 0; i < 3; i++)</span><br><span>           smp_write_intsrc(mc, mp_INT, MP_IRQ_TRIGGER_LEVEL|MP_IRQ_POLARITY_LOW, m->bus_bcm5785_0, ((2+sysconf.sbdn)<<2)|i, m->apicid_bcm5785[0], 0xa); //</span><br><span style="color: hsl(0, 100%, 40%);">-    }</span><br><span> </span><br><span> </span><br><span> </span><br><span>@@ -91,7 +90,7 @@</span><br><span>    {</span><br><span>            device_t dev;</span><br><span>                dev = dev_find_device(0x1166, 0x0205, 0);</span><br><span style="color: hsl(0, 100%, 40%);">-               if(dev) {</span><br><span style="color: hsl(120, 100%, 40%);">+             if (dev) {</span><br><span>                   uint32_t dword;</span><br><span>                      dword = pci_read_config32(dev, 0x6c);</span><br><span>                        dword |= (1 << 4); // enable interrupts</span><br><span>@@ -101,46 +100,39 @@</span><br><span> </span><br><span> //First pci-x slot (on bcm5785) under bus_bcm5785_1:d.0</span><br><span>         // AIC 8130 Galileo Technology...</span><br><span style="color: hsl(0, 100%, 40%);">-       for(i = 0; i < 4; i++) {</span><br><span style="color: hsl(120, 100%, 40%);">+   for (i = 0; i < 4; i++)</span><br><span>           smp_write_intsrc(mc, mp_INT, MP_IRQ_TRIGGER_LEVEL|MP_IRQ_POLARITY_LOW, m->bus_bcm5785_1_1, (6 << 2)|i, m->apicid_bcm5785[1], 2 + (1+i)%4); //</span><br><span style="color: hsl(0, 100%, 40%);">-       }</span><br><span> </span><br><span> </span><br><span> //pci slot (on bcm5785)</span><br><span style="color: hsl(0, 100%, 40%);">-    for(i = 0; i < 4; i++) {</span><br><span style="color: hsl(120, 100%, 40%);">+   for (i = 0; i < 4; i++)</span><br><span>           smp_write_intsrc(mc, mp_INT, MP_IRQ_TRIGGER_LEVEL|MP_IRQ_POLARITY_LOW, m->bus_bcm5785_0, (5 << 2)|i, m->apicid_bcm5785[1], 8+i%4); //</span><br><span style="color: hsl(0, 100%, 40%);">-       }</span><br><span> </span><br><span> </span><br><span> //onboard ati</span><br><span>   smp_write_intsrc(mc, mp_INT, MP_IRQ_TRIGGER_LEVEL|MP_IRQ_POLARITY_LOW, m->bus_bcm5785_0, (4 << 2)|0, m->apicid_bcm5785[1], 0x1);</span><br><span> </span><br><span> //PCI-X on bcm5780</span><br><span style="color: hsl(0, 100%, 40%);">-  for(i = 0; i < 4; i++) {</span><br><span style="color: hsl(120, 100%, 40%);">+   for (i = 0; i < 4; i++)</span><br><span>           smp_write_intsrc(mc, mp_INT, MP_IRQ_TRIGGER_LEVEL|MP_IRQ_POLARITY_LOW, m->bus_bcm5780[1], (4 << 2)|i, m->apicid_bcm5785[1], 2 + (0+i)%4); //</span><br><span style="color: hsl(0, 100%, 40%);">-        }</span><br><span> </span><br><span> //onboard Broadcom</span><br><span style="color: hsl(0, 100%, 40%);">-     for(i = 0; i < 2; i++) {</span><br><span style="color: hsl(120, 100%, 40%);">+   for (i = 0; i < 2; i++)</span><br><span>           smp_write_intsrc(mc, mp_INT, MP_IRQ_TRIGGER_LEVEL|MP_IRQ_POLARITY_LOW, m->bus_bcm5780[2], (4 << 2)|i, m->apicid_bcm5785[1], 0xa + (0+i)%4); //</span><br><span style="color: hsl(0, 100%, 40%);">-      }</span><br><span> </span><br><span> </span><br><span> // First PCI-E x8</span><br><span style="color: hsl(0, 100%, 40%);">-  for(i = 0; i < 4; i++) {</span><br><span style="color: hsl(120, 100%, 40%);">+   for (i = 0; i < 4; i++)</span><br><span>           smp_write_intsrc(mc, mp_INT, MP_IRQ_TRIGGER_LEVEL|MP_IRQ_POLARITY_LOW, m->bus_bcm5780[5], (0 << 2)|i, m->apicid_bcm5785[1], 0xe); //</span><br><span style="color: hsl(0, 100%, 40%);">-        }</span><br><span> </span><br><span> </span><br><span> // Second PCI-E x8</span><br><span style="color: hsl(0, 100%, 40%);">- for(i = 0; i < 4; i++) {</span><br><span style="color: hsl(120, 100%, 40%);">+   for (i = 0; i < 4; i++)</span><br><span>           smp_write_intsrc(mc, mp_INT, MP_IRQ_TRIGGER_LEVEL|MP_IRQ_POLARITY_LOW, m->bus_bcm5780[3], (0 << 2)|i, m->apicid_bcm5785[1], 0xc); //</span><br><span style="color: hsl(0, 100%, 40%);">-        }</span><br><span> </span><br><span> // Third PCI-E x1</span><br><span style="color: hsl(0, 100%, 40%);">-      for(i = 0; i < 4; i++) {</span><br><span style="color: hsl(120, 100%, 40%);">+   for (i = 0; i < 4; i++)</span><br><span>           smp_write_intsrc(mc, mp_INT, MP_IRQ_TRIGGER_LEVEL|MP_IRQ_POLARITY_LOW, m->bus_bcm5780[4], (0 << 2)|i, m->apicid_bcm5785[1], 0xd); //</span><br><span style="color: hsl(0, 100%, 40%);">-        }</span><br><span> </span><br><span> /*Local Ints:  Type    Polarity    Trigger     Bus ID   IRQ    APIC ID PIN#*/</span><br><span>       mptable_lintsrc(mc, bus_isa);</span><br><span>diff --git a/src/mainboard/msi/ms9185/romstage.c b/src/mainboard/msi/ms9185/romstage.c</span><br><span>index a86b57a..e509529 100644</span><br><span>--- a/src/mainboard/msi/ms9185/romstage.c</span><br><span>+++ b/src/mainboard/msi/ms9185/romstage.c</span><br><span>@@ -181,7 +181,7 @@</span><br><span> </span><br><span> #if 0</span><br><span>    int i;</span><br><span style="color: hsl(0, 100%, 40%);">-  for(i = 0; i < 2; i++) {</span><br><span style="color: hsl(120, 100%, 40%);">+   for (i = 0; i < 2; i++) {</span><br><span>                 activate_spd_rom(sysinfo->ctrl+i);</span><br><span>                dump_smbus_registers();</span><br><span>      }</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/23536">change 23536</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/23536"/><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: I3cca4adbf04edfd88a9b8ae52cf4d62d429e6c45 </div>
<div style="display:none"> Gerrit-Change-Number: 23536 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Elyes HAOUAS <ehaouas@noos.fr> </div>