commit f21bdc3020b24cce585c4df69b06913177c94182
author Edward O'Callaghan <eocallaghan@alterapraxis.com> Tue Oct 21 07:43:41 2014 +1100
committer Edward O'Callaghan <eocallaghan@alterapraxis.com> Mon Oct 27 12:51:00 2014 +0100
tree 622d01b8c806f0e69c098b0c87fc899f4ba1754d
parent 85836c2215498ff18746b3a7e85ed684cf2e119e

superio/*/superio.c: Don't hide pointer types behind typedefs

Hiding pointer types behind 'typedef's is usually not a great
idea at the best of times. Worse the typedef becomes an integer
at different stages in Coreboot. Let us refrain from doing this
at all.

Change-Id: Ia2ca8c98bb489daaa58f379433875864f6efabc8
Signed-off-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
Reviewed-on: http://review.coreboot.org/7136
Tested-by: build bot (Jenkins)
Reviewed-by: Nico Huber <nico.h@gmx.de>

src/superio/winbond/w83627hf/superio.c

diff --git a/src/superio/winbond/w83627hf/superio.c b/src/superio/winbond/w83627hf/superio.c
index 5445b47..2d69fad 100644
--- a/src/superio/winbond/w83627hf/superio.c
+++ b/src/superio/winbond/w83627hf/superio.c
@@ -44,7 +44,7 @@
 	return inb(port + 1);
 }
 
-static void enable_hwm_smbus(device_t dev)
+static void enable_hwm_smbus(struct device *dev)
 {
 	u8 reg8;
 
@@ -54,7 +54,7 @@
 	pnp_write_config(dev, 0x2b, reg8);
 }
 
-static void init_acpi(device_t dev)
+static void init_acpi(struct device *dev)
 {
 	u8 value = 0x20; /* FIXME: The 0x20 value here is never used? */
 	int power_on = 1;
@@ -99,7 +99,7 @@
 	}
 }
 
-static void w83627hf_init(device_t dev)
+static void w83627hf_init(struct device *dev)
 {
 	struct resource *res0;
 
@@ -121,7 +121,7 @@
 	}
 }
 
-static void w83627hf_pnp_enable_resources(device_t dev)
+static void w83627hf_pnp_enable_resources(struct device *dev)
 {
 	pnp_enable_resources(dev);