summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorPatrick Rudolph <patrick.rudolph@9elements.com>2019-12-10 12:08:37 +0100
committerPatrick Georgi <pgeorgi@google.com>2019-12-13 09:04:55 +0000
commit98b72dadf07bf8694f78384d21a4add7ac76c1f2 (patch)
treec062709bc8a6565629bb08da8f0abe629216ba6a /src
parent8a2204896a4bd784a34d3c4a0bba1b5415aecf52 (diff)
downloadcoreboot-98b72dadf07bf8694f78384d21a4add7ac76c1f2.tar.xz
superio/*: Don't use conf_mode directly
Use the functions defined in device/pnp.h instead of using the conf_mode directly. This will make future refactoring easier. Change-Id: Ibb94d86b3ee861f44cded469ff58b545dd7311fd Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/37638 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
Diffstat (limited to 'src')
-rw-r--r--src/superio/nuvoton/nct5572d/superio.c12
-rw-r--r--src/superio/smsc/lpc47n227/superio.c13
-rw-r--r--src/superio/winbond/w83667hg-a/superio.c12
3 files changed, 19 insertions, 18 deletions
diff --git a/src/superio/nuvoton/nct5572d/superio.c b/src/superio/nuvoton/nct5572d/superio.c
index 30846872a0..76c983a0c7 100644
--- a/src/superio/nuvoton/nct5572d/superio.c
+++ b/src/superio/nuvoton/nct5572d/superio.c
@@ -43,29 +43,29 @@ static void nct5572d_init(struct device *dev)
/* TODO: Might potentially need code for HWM or FDC etc. */
case NCT5572D_KBC:
/* Enable mouse controller */
- pnp_enter_conf_mode_8787(dev);
+ pnp_enter_conf_mode(dev);
byte = pnp_read_config(dev, 0x2a);
byte &= ~(0x1 << 1);
pnp_write_config(dev, 0x2a, byte);
- pnp_exit_conf_mode_aa(dev);
+ pnp_exit_conf_mode(dev);
mouse_detected = pc_keyboard_init(PROBE_AUX_DEVICE);
if (!mouse_detected) {
printk(BIOS_INFO, "%s: Disable mouse controller.",
__func__);
- pnp_enter_conf_mode_8787(dev);
+ pnp_enter_conf_mode(dev);
byte = pnp_read_config(dev, 0x2a);
byte |= 0x1 << 1;
pnp_write_config(dev, 0x2a, byte);
- pnp_exit_conf_mode_aa(dev);
+ pnp_exit_conf_mode(dev);
}
break;
case NCT5572D_ACPI:
/* Set power state after power fail */
power_status = CONFIG_MAINBOARD_POWER_FAILURE_STATE;
get_option(&power_status, "power_on_after_fail");
- pnp_enter_conf_mode_8787(dev);
+ pnp_enter_conf_mode(dev);
pnp_set_logical_device(dev);
byte = pnp_read_config(dev, 0xe4);
byte &= ~0x60;
@@ -74,7 +74,7 @@ static void nct5572d_init(struct device *dev)
else if (power_status == MAINBOARD_POWER_KEEP)
byte |= (0x2 << 5);
pnp_write_config(dev, 0xe4, byte);
- pnp_exit_conf_mode_aa(dev);
+ pnp_exit_conf_mode(dev);
printk(BIOS_INFO, "set power %s after power fail\n", power_status ? "on" : "off");
break;
}
diff --git a/src/superio/smsc/lpc47n227/superio.c b/src/superio/smsc/lpc47n227/superio.c
index 57297aba5b..911343d890 100644
--- a/src/superio/smsc/lpc47n227/superio.c
+++ b/src/superio/smsc/lpc47n227/superio.c
@@ -48,6 +48,7 @@ static struct device_operations ops = {
.enable_resources = lpc47n227_pnp_enable_resources,
.enable = lpc47n227_pnp_enable,
.init = lpc47n227_init,
+ .ops_pnp_mode = &pnp_conf_mode_55_aa,
};
static struct pnp_info pnp_dev_info[] = {
@@ -81,10 +82,10 @@ void lpc47n227_pnp_set_resources(struct device *dev)
{
struct resource *res;
- pnp_enter_conf_mode_55(dev);
+ pnp_enter_conf_mode(dev);
for (res = dev->resource_list; res; res = res->next)
lpc47n227_pnp_set_resource(dev, res);
- pnp_exit_conf_mode_aa(dev);
+ pnp_exit_conf_mode(dev);
}
/*
@@ -93,9 +94,9 @@ void lpc47n227_pnp_set_resources(struct device *dev)
*/
void lpc47n227_pnp_enable_resources(struct device *dev)
{
- pnp_enter_conf_mode_55(dev);
+ pnp_enter_conf_mode(dev);
lpc47n227_pnp_set_enable(dev, 1);
- pnp_exit_conf_mode_aa(dev);
+ pnp_exit_conf_mode(dev);
}
/*
@@ -104,9 +105,9 @@ void lpc47n227_pnp_enable_resources(struct device *dev)
*/
void lpc47n227_pnp_enable(struct device *dev)
{
- pnp_enter_conf_mode_55(dev);
+ pnp_enter_conf_mode(dev);
lpc47n227_pnp_set_enable(dev, !!dev->enabled);
- pnp_exit_conf_mode_aa(dev);
+ pnp_exit_conf_mode(dev);
}
/**
diff --git a/src/superio/winbond/w83667hg-a/superio.c b/src/superio/winbond/w83667hg-a/superio.c
index 4a995d6b5d..69ab91bf98 100644
--- a/src/superio/winbond/w83667hg-a/superio.c
+++ b/src/superio/winbond/w83667hg-a/superio.c
@@ -43,29 +43,29 @@ static void w83667hg_a_init(struct device *dev)
/* TODO: Might potentially need code for HWM or FDC etc. */
case W83667HG_A_KBC:
/* Enable mouse controller */
- pnp_enter_conf_mode_8787(dev);
+ pnp_enter_conf_mode(dev);
byte = pnp_read_config(dev, 0x2a);
byte &= ~(0x1 << 1);
pnp_write_config(dev, 0x2a, byte);
- pnp_exit_conf_mode_aa(dev);
+ pnp_exit_conf_mode(dev);
mouse_detected = pc_keyboard_init(PROBE_AUX_DEVICE);
if (!mouse_detected && !acpi_is_wakeup_s3()) {
printk(BIOS_INFO, "%s: Disable mouse controller.",
__func__);
- pnp_enter_conf_mode_8787(dev);
+ pnp_enter_conf_mode(dev);
byte = pnp_read_config(dev, 0x2a);
byte |= 0x1 << 1;
pnp_write_config(dev, 0x2a, byte);
- pnp_exit_conf_mode_aa(dev);
+ pnp_exit_conf_mode(dev);
}
break;
case W83667HG_A_ACPI:
/* Set power state after power fail */
power_status = CONFIG_MAINBOARD_POWER_FAILURE_STATE;
get_option(&power_status, "power_on_after_fail");
- pnp_enter_conf_mode_8787(dev);
+ pnp_enter_conf_mode(dev);
pnp_set_logical_device(dev);
byte = pnp_read_config(dev, 0xe4);
byte &= ~0x60;
@@ -74,7 +74,7 @@ static void w83667hg_a_init(struct device *dev)
else if (power_status == MAINBOARD_POWER_KEEP)
byte |= (0x2 << 5);
pnp_write_config(dev, 0xe4, byte);
- pnp_exit_conf_mode_aa(dev);
+ pnp_exit_conf_mode(dev);
printk(BIOS_INFO, "set power %s after power fail\n", power_status ? "on" : "off");
break;
}