<p>Elyes HAOUAS has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/26256">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">sb/intel/i82801ix: Get rid of device_t<br><br>Use of device_t has been abandoned in ramstage.<br><br>Change-Id: Iefef4e72f1012c8a6edbb9e5c94bdc162bed93d0<br>Signed-off-by: Elyes HAOUAS <ehaouas@noos.fr><br>---<br>M src/southbridge/intel/i82801ix/hdaudio.c<br>M src/southbridge/intel/i82801ix/i82801ix.c<br>M src/southbridge/intel/i82801ix/lpc.c<br>M src/southbridge/intel/i82801ix/pci.c<br>M src/southbridge/intel/i82801ix/pcie.c<br>M src/southbridge/intel/i82801ix/sata.c<br>M src/southbridge/intel/i82801ix/smbus.c<br>M src/southbridge/intel/i82801ix/thermal.c<br>M src/southbridge/intel/i82801ix/usb_ehci.c<br>9 files changed, 29 insertions(+), 29 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/56/26256/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/southbridge/intel/i82801ix/hdaudio.c b/src/southbridge/intel/i82801ix/hdaudio.c</span><br><span>index c3602c4..1f26ba9 100644</span><br><span>--- a/src/southbridge/intel/i82801ix/hdaudio.c</span><br><span>+++ b/src/southbridge/intel/i82801ix/hdaudio.c</span><br><span>@@ -287,7 +287,7 @@</span><br><span>        }</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void azalia_set_subsystem(device_t dev, unsigned vendor, unsigned device)</span><br><span style="color: hsl(120, 100%, 40%);">+static void azalia_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</span><br><span> {</span><br><span>      if (!vendor || !device) {</span><br><span>            pci_write_config32(dev, PCI_SUBSYSTEM_VENDOR_ID,</span><br><span>diff --git a/src/southbridge/intel/i82801ix/i82801ix.c b/src/southbridge/intel/i82801ix/i82801ix.c</span><br><span>index 7d44fba..46650bf 100644</span><br><span>--- a/src/southbridge/intel/i82801ix/i82801ix.c</span><br><span>+++ b/src/southbridge/intel/i82801ix/i82801ix.c</span><br><span>@@ -25,7 +25,7 @@</span><br><span> </span><br><span> typedef struct southbridge_intel_i82801ix_config config_t;</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void i82801ix_enable_device(device_t dev)</span><br><span style="color: hsl(120, 100%, 40%);">+static void i82801ix_enable_device(struct device *dev)</span><br><span> {</span><br><span>     u32 reg32;</span><br><span> </span><br><span>@@ -51,7 +51,7 @@</span><br><span> </span><br><span> static void i82801ix_pcie_init(const config_t *const info)</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-       device_t pciePort[6];</span><br><span style="color: hsl(120, 100%, 40%);">+ struct device *pciePort[6];</span><br><span>  int i, slot_number = 1; /* Reserve slot number 0 for nb's PEG. */</span><br><span>        u32 reg32;</span><br><span> </span><br><span>@@ -86,7 +86,7 @@</span><br><span> </span><br><span>       /* Set slot implemented, slot number and slot power limits. */</span><br><span>       for (i = 0; i < 6; ++i) {</span><br><span style="color: hsl(0, 100%, 40%);">-            const device_t dev = pciePort[i];</span><br><span style="color: hsl(120, 100%, 40%);">+             const struct device *dev = pciePort[i];</span><br><span>              u32 xcap = pci_read_config32(dev, D28Fx_XCAP);</span><br><span>               if (info->pcie_slot_implemented & (1 << i))</span><br><span>                     xcap |=  PCI_EXP_FLAGS_SLOT;</span><br><span>@@ -115,10 +115,10 @@</span><br><span> </span><br><span> static void i82801ix_ehci_init(void)</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-   const device_t pciEHCI1 = dev_find_slot(0, PCI_DEVFN(0x1d, 7));</span><br><span style="color: hsl(120, 100%, 40%);">+       const struct device *pciEHCI1 = dev_find_slot(0, PCI_DEVFN(0x1d, 7));</span><br><span>        if (!pciEHCI1)</span><br><span>               die("EHCI controller (00:1d.7) not listed in devicetree.\n");</span><br><span style="color: hsl(0, 100%, 40%);">- const device_t pciEHCI2 = dev_find_slot(0, PCI_DEVFN(0x1a, 7));</span><br><span style="color: hsl(120, 100%, 40%);">+       const struct device *pciEHCI2 = dev_find_slot(0, PCI_DEVFN(0x1a, 7));</span><br><span>        if (!pciEHCI2)</span><br><span>               die("EHCI controller (00:1a.7) not listed in devicetree.\n");</span><br><span> </span><br><span>diff --git a/src/southbridge/intel/i82801ix/lpc.c b/src/southbridge/intel/i82801ix/lpc.c</span><br><span>index abc8667..76f16e7 100644</span><br><span>--- a/src/southbridge/intel/i82801ix/lpc.c</span><br><span>+++ b/src/southbridge/intel/i82801ix/lpc.c</span><br><span>@@ -90,9 +90,9 @@</span><br><span>  * 0x80 - The PIRQ is not routed.</span><br><span>  */</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void i82801ix_pirq_init(device_t dev)</span><br><span style="color: hsl(120, 100%, 40%);">+static void i82801ix_pirq_init(struct device *dev)</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-       device_t irq_dev;</span><br><span style="color: hsl(120, 100%, 40%);">+     struct device *irq_dev;</span><br><span>      /* Get the chip configuration */</span><br><span>     config_t *config = dev->chip_info;</span><br><span> </span><br><span>@@ -132,7 +132,7 @@</span><br><span>      }</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void i82801ix_gpi_routing(device_t dev)</span><br><span style="color: hsl(120, 100%, 40%);">+static void i82801ix_gpi_routing(struct device *dev)</span><br><span> {</span><br><span>  /* Get the chip configuration */</span><br><span>     config_t *config = dev->chip_info;</span><br><span>@@ -161,7 +161,7 @@</span><br><span>  pci_write_config32(dev, D31F0_GPIO_ROUT, reg32);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void i82801ix_power_options(device_t dev)</span><br><span style="color: hsl(120, 100%, 40%);">+static void i82801ix_power_options(struct device *dev)</span><br><span> {</span><br><span>       u8 reg8;</span><br><span>     u16 reg16, pmbase;</span><br><span>@@ -293,7 +293,7 @@</span><br><span>     outl(reg32, pmbase + 0x10);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void i82801ix_configure_cstates(device_t dev)</span><br><span style="color: hsl(120, 100%, 40%);">+static void i82801ix_configure_cstates(struct device *dev)</span><br><span> {</span><br><span>    u8 reg8;</span><br><span> </span><br><span>@@ -469,7 +469,7 @@</span><br><span> #endif</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void i82801ix_lpc_read_resources(device_t dev)</span><br><span style="color: hsl(120, 100%, 40%);">+static void i82801ix_lpc_read_resources(struct device *dev)</span><br><span> {</span><br><span>         /*</span><br><span>    *             I/O Resources</span><br><span>@@ -525,7 +525,7 @@</span><br><span>   res->flags = IORESOURCE_MEM | IORESOURCE_ASSIGNED | IORESOURCE_FIXED;</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void set_subsystem(device_t dev, unsigned vendor, unsigned device)</span><br><span style="color: hsl(120, 100%, 40%);">+static void set_subsystem(struct device *dev, unsigned vendor, unsigned device)</span><br><span> {</span><br><span>     if (!vendor || !device) {</span><br><span>            pci_write_config32(dev, PCI_SUBSYSTEM_VENDOR_ID,</span><br><span>@@ -536,7 +536,7 @@</span><br><span>       }</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void southbridge_inject_dsdt(device_t dev)</span><br><span style="color: hsl(120, 100%, 40%);">+static void southbridge_inject_dsdt(struct device *dev)</span><br><span> {</span><br><span>    global_nvs_t *gnvs = cbmem_add (CBMEM_ID_ACPI_GNVS, sizeof(*gnvs));</span><br><span> </span><br><span>@@ -558,9 +558,9 @@</span><br><span>        }</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void southbridge_fill_ssdt(device_t device)</span><br><span style="color: hsl(120, 100%, 40%);">+static void southbridge_fill_ssdt(struct device *device)</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-     device_t dev = dev_find_slot(0, PCI_DEVFN(0x1f,0));</span><br><span style="color: hsl(120, 100%, 40%);">+   struct device *dev = dev_find_slot(0, PCI_DEVFN(0x1f,0));</span><br><span>    config_t *chip = dev->chip_info;</span><br><span> </span><br><span>      intel_acpi_pcie_hotplug_generator(chip->pcie_hotplug_map, 8);</span><br><span>diff --git a/src/southbridge/intel/i82801ix/pci.c b/src/southbridge/intel/i82801ix/pci.c</span><br><span>index 5da2e49..13f92f0 100644</span><br><span>--- a/src/southbridge/intel/i82801ix/pci.c</span><br><span>+++ b/src/southbridge/intel/i82801ix/pci.c</span><br><span>@@ -43,7 +43,7 @@</span><br><span>    pci_write_config16(dev, PCI_SEC_STATUS, reg16);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void set_subsystem(device_t dev, unsigned vendor, unsigned device)</span><br><span style="color: hsl(120, 100%, 40%);">+static void set_subsystem(struct device *dev, unsigned vendor, unsigned device)</span><br><span> {</span><br><span>      /* NOTE: 0x54 is not the default position! */</span><br><span>        if (!vendor || !device) {</span><br><span>diff --git a/src/southbridge/intel/i82801ix/pcie.c b/src/southbridge/intel/i82801ix/pcie.c</span><br><span>index 502e765..12fd4e4 100644</span><br><span>--- a/src/southbridge/intel/i82801ix/pcie.c</span><br><span>+++ b/src/southbridge/intel/i82801ix/pcie.c</span><br><span>@@ -94,7 +94,7 @@</span><br><span>       }</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void pcie_set_subsystem(device_t dev, unsigned vendor, unsigned device)</span><br><span style="color: hsl(120, 100%, 40%);">+static void pcie_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</span><br><span> {</span><br><span>  /* NOTE: 0x94 is not the default position! */</span><br><span>        if (!vendor || !device) {</span><br><span>@@ -106,7 +106,7 @@</span><br><span>      }</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void pch_pciexp_scan_bridge(device_t dev)</span><br><span style="color: hsl(120, 100%, 40%);">+static void pch_pciexp_scan_bridge(struct device *dev)</span><br><span> {</span><br><span>      struct southbridge_intel_i82801ix_config *config = dev->chip_info;</span><br><span> </span><br><span>diff --git a/src/southbridge/intel/i82801ix/sata.c b/src/southbridge/intel/i82801ix/sata.c</span><br><span>index 8bfc476..c241532 100644</span><br><span>--- a/src/southbridge/intel/i82801ix/sata.c</span><br><span>+++ b/src/southbridge/intel/i82801ix/sata.c</span><br><span>@@ -208,7 +208,7 @@</span><br><span>     pci_write_config32(dev, 0x94, sclkcg);</span><br><span> </span><br><span>   if (is_mobile && config->sata_traffic_monitor) {</span><br><span style="color: hsl(0, 100%, 40%);">-             const device_t lpc_dev = dev_find_slot(0, PCI_DEVFN(0x1f, 0));</span><br><span style="color: hsl(120, 100%, 40%);">+                const struct device *lpc_dev = dev_find_slot(0, PCI_DEVFN(0x1f, 0));</span><br><span>                 if (((pci_read_config8(lpc_dev, D31F0_CxSTATE_CNF)</span><br><span>                                                   >> 3) & 3) == 3) {</span><br><span>                         u8 reg8 = pci_read_config8(dev, 0x9c);</span><br><span>@@ -224,7 +224,7 @@</span><br><span>         sata_program_indexed(dev, is_mobile);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void sata_enable(device_t dev)</span><br><span style="color: hsl(120, 100%, 40%);">+static void sata_enable(struct device *dev)</span><br><span> {</span><br><span>        /* Get the chip configuration */</span><br><span>     const config_t *const config = dev->chip_info;</span><br><span>@@ -251,7 +251,7 @@</span><br><span>      pci_write_config16(dev, 0x90, map);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void sata_set_subsystem(device_t dev, unsigned vendor, unsigned device)</span><br><span style="color: hsl(120, 100%, 40%);">+static void sata_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</span><br><span> {</span><br><span>        if (!vendor || !device) {</span><br><span>            pci_write_config32(dev, PCI_SUBSYSTEM_VENDOR_ID,</span><br><span>diff --git a/src/southbridge/intel/i82801ix/smbus.c b/src/southbridge/intel/i82801ix/smbus.c</span><br><span>index ccfcddb..2dcb4cb 100644</span><br><span>--- a/src/southbridge/intel/i82801ix/smbus.c</span><br><span>+++ b/src/southbridge/intel/i82801ix/smbus.c</span><br><span>@@ -25,7 +25,7 @@</span><br><span> #include <southbridge/intel/common/smbus.h></span><br><span> #include "i82801ix.h"</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void pch_smbus_init(device_t dev)</span><br><span style="color: hsl(120, 100%, 40%);">+static void pch_smbus_init(struct device *dev)</span><br><span> {</span><br><span>   u16 reg16;</span><br><span> </span><br><span>@@ -35,7 +35,7 @@</span><br><span>   pci_write_config16(dev, 0x80, reg16);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static int lsmbus_read_byte(device_t dev, u8 address)</span><br><span style="color: hsl(120, 100%, 40%);">+static int lsmbus_read_byte(struct device *dev, u8 address)</span><br><span> {</span><br><span>        u16 device;</span><br><span>  struct resource *res;</span><br><span>@@ -48,7 +48,7 @@</span><br><span>    return do_smbus_read_byte(res->base, device, address);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static int lsmbus_write_byte(device_t dev, u8 address, u8 val)</span><br><span style="color: hsl(120, 100%, 40%);">+static int lsmbus_write_byte(struct device *dev, u8 address, u8 val)</span><br><span> {</span><br><span>  u16 device;</span><br><span>  struct resource *res;</span><br><span>@@ -66,7 +66,7 @@</span><br><span>    .write_byte     = lsmbus_write_byte,</span><br><span> };</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void smbus_set_subsystem(device_t dev, unsigned vendor, unsigned device)</span><br><span style="color: hsl(120, 100%, 40%);">+static void smbus_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</span><br><span> {</span><br><span>    if (!vendor || !device) {</span><br><span>            pci_write_config32(dev, PCI_SUBSYSTEM_VENDOR_ID,</span><br><span>@@ -81,7 +81,7 @@</span><br><span>         .set_subsystem    = smbus_set_subsystem,</span><br><span> };</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void smbus_read_resources(device_t dev)</span><br><span style="color: hsl(120, 100%, 40%);">+static void smbus_read_resources(struct device *dev)</span><br><span> {</span><br><span>  struct resource *res = new_resource(dev, PCI_BASE_ADDRESS_4);</span><br><span>        res->base = SMBUS_IO_BASE;</span><br><span>diff --git a/src/southbridge/intel/i82801ix/thermal.c b/src/southbridge/intel/i82801ix/thermal.c</span><br><span>index 12cf898..b1ba943 100644</span><br><span>--- a/src/southbridge/intel/i82801ix/thermal.c</span><br><span>+++ b/src/southbridge/intel/i82801ix/thermal.c</span><br><span>@@ -50,7 +50,7 @@</span><br><span>       pci_write_config32(dev, 0x10, 0);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void thermal_set_subsystem(device_t dev, unsigned vendor, unsigned device)</span><br><span style="color: hsl(120, 100%, 40%);">+static void thermal_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</span><br><span> {</span><br><span>    if (!vendor || !device) {</span><br><span>            pci_write_config32(dev, PCI_SUBSYSTEM_VENDOR_ID,</span><br><span>diff --git a/src/southbridge/intel/i82801ix/usb_ehci.c b/src/southbridge/intel/i82801ix/usb_ehci.c</span><br><span>index 7719f16..bd73118 100644</span><br><span>--- a/src/southbridge/intel/i82801ix/usb_ehci.c</span><br><span>+++ b/src/southbridge/intel/i82801ix/usb_ehci.c</span><br><span>@@ -33,7 +33,7 @@</span><br><span>        printk(BIOS_DEBUG, "done.\n");</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void usb_ehci_set_subsystem(device_t dev, unsigned vendor, unsigned device)</span><br><span style="color: hsl(120, 100%, 40%);">+static void usb_ehci_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</span><br><span> {</span><br><span>   u8 access_cntl;</span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/26256">change 26256</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/26256"/><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: Iefef4e72f1012c8a6edbb9e5c94bdc162bed93d0 </div>
<div style="display:none"> Gerrit-Change-Number: 26256 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Elyes HAOUAS <ehaouas@noos.fr> </div>