[coreboot-gerrit] New patch to review for coreboot: northbridge/via/vx900: transition away from device_t

[Antonello Dettori (dev@dettori.io)]

Antonello Dettori (dev at dettori.io) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17317

-gerrit

commit 9fa83f05a89bfa264ed44426d209c4d6ed8f50bc
Author: Antonello Dettori <dev at dettori.io>
Date:   Tue Nov 8 18:44:46 2016 +0100

    northbridge/via/vx900: transition away from device_t
    
    Replace the use of the old device_t definition inside
    northbridge/via/vx900.
    
    Change-Id: I04292a6b698a42a5c582eddcef7cf5a235e1a464
    Signed-off-by: Antonello Dettori <dev at dettori.io>
---
 src/northbridge/via/vx900/early_smbus.c  |  2 +-
 src/northbridge/via/vx900/raminit_ddr3.c |  2 +-
 src/northbridge/via/vx900/vx900.h        | 11 +++++++++++
 3 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/src/northbridge/via/vx900/early_smbus.c b/src/northbridge/via/vx900/early_smbus.c
index 93a9254..0cf7427 100644
--- a/src/northbridge/via/vx900/early_smbus.c
+++ b/src/northbridge/via/vx900/early_smbus.c
@@ -76,7 +76,7 @@ u8 smbus_read_byte(u32 smbus_dev, u8 addr, u8 offset)
 
 void enable_smbus(void)
 {
-	device_t dev;
+	pci_devfn_t dev;
 	u8 reg8;
 	u32 smbus_dev = (u32) SMBUS_IO_BASE;
 
diff --git a/src/northbridge/via/vx900/raminit_ddr3.c b/src/northbridge/via/vx900/raminit_ddr3.c
index 4878571..1e18296 100644
--- a/src/northbridge/via/vx900/raminit_ddr3.c
+++ b/src/northbridge/via/vx900/raminit_ddr3.c
@@ -1627,7 +1627,7 @@ void vx900_init_dram_ddr3(const dimm_layout * dimm_addr)
 	dimm_info dimm_prop;
 	ramctr_timing ctrl_prop;
 	rank_layout ranks;
-	device_t mcu;
+	pci_devfn_t mcu;
 
 	if (!ram_check_noprint_nodie(1 << 20, 1 << 20)) {
 		printram("RAM is already initialized. Skipping init\n");
diff --git a/src/northbridge/via/vx900/vx900.h b/src/northbridge/via/vx900/vx900.h
index 2590ab2..12e5733 100644
--- a/src/northbridge/via/vx900/vx900.h
+++ b/src/northbridge/via/vx900/vx900.h
@@ -41,6 +41,16 @@ uint64_t get_uma_memory_base(void);
 
 /* We use these throughout the code. They really belong in a generic part of
  * coreboot, but until bureaucracy gets them there, we still need them  */
+
+#ifdef __SIMPLE_DEVICE__
+void dump_pci_device(pci_devfn_t dev);
+void pci_mod_config8(pci_devfn_t dev, unsigned int where,
+		     uint8_t clr_mask, uint8_t set_mask);
+void pci_mod_config16(pci_devfn_t dev, unsigned int where,
+		      uint16_t clr_mask, uint16_t set_mask);
+void pci_mod_config32(pci_devfn_t dev, unsigned int where,
+		      uint32_t clr_mask, uint32_t set_mask);
+#else
 void dump_pci_device(device_t dev);
 void pci_mod_config8(device_t dev, unsigned int where,
 		     uint8_t clr_mask, uint8_t set_mask);
@@ -48,5 +58,6 @@ void pci_mod_config16(device_t dev, unsigned int where,
 		      uint16_t clr_mask, uint16_t set_mask);
 void pci_mod_config32(device_t dev, unsigned int where,
 		      uint32_t clr_mask, uint32_t set_mask);
+#endif
 
 #endif				/* __VX900_H */