diff options
author | Edward O'Callaghan <eocallaghan@alterapraxis.com> | 2014-10-21 07:43:41 +1100 |
---|---|---|
committer | Edward O'Callaghan <eocallaghan@alterapraxis.com> | 2014-10-27 12:51:00 +0100 |
commit | f21bdc3020b24cce585c4df69b06913177c94182 (patch) | |
tree | 622d01b8c806f0e69c098b0c87fc899f4ba1754d /src/superio/intel | |
parent | 85836c2215498ff18746b3a7e85ed684cf2e119e (diff) | |
download | coreboot-f21bdc3020b24cce585c4df69b06913177c94182.tar.xz |
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>
Diffstat (limited to 'src/superio/intel')
-rw-r--r-- | src/superio/intel/i3100/superio.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/superio/intel/i3100/superio.c b/src/superio/intel/i3100/superio.c index d0c506626d..50c9de12da 100644 --- a/src/superio/intel/i3100/superio.c +++ b/src/superio/intel/i3100/superio.c @@ -24,19 +24,19 @@ #include "i3100.h" #include <arch/io.h> -static void pnp_enter_ext_func_mode(device_t dev) +static void pnp_enter_ext_func_mode(struct device *dev) { outb(0x80, dev->path.pnp.port); outb(0x86, dev->path.pnp.port); } -static void pnp_exit_ext_func_mode(device_t dev) +static void pnp_exit_ext_func_mode(struct device *dev) { outb(0x68, dev->path.pnp.port); outb(0x08, dev->path.pnp.port); } -static void i3100_init(device_t dev) +static void i3100_init(struct device *dev) { if (!dev->enabled) return; |