commit 98b72dadf07bf8694f78384d21a4add7ac76c1f2
author Patrick Rudolph <patrick.rudolph@9elements.com> Tue Dec 10 12:08:37 2019 +0100
committer Patrick Georgi <pgeorgi@google.com> Fri Dec 13 09:04:55 2019 +0000
tree c062709bc8a6565629bb08da8f0abe629216ba6a
parent 8a2204896a4bd784a34d3c4a0bba1b5415aecf52

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>

src/superio/smsc/lpc47n227/superio.c

diff --git a/src/superio/smsc/lpc47n227/superio.c b/src/superio/smsc/lpc47n227/superio.c
index 57297ab..911343d 100644
--- a/src/superio/smsc/lpc47n227/superio.c
+++ b/src/superio/smsc/lpc47n227/superio.c
@@ -48,6 +48,7 @@
 	.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 @@
 {
 	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_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(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);
 }
 
 /**