Allow device hints to wire the unit numbers of devices.

- An "at" hint now reserves a device name.
- A new BUS_HINT_DEVICE_UNIT method is added to the bus interface.  When
  determining the unit number of a device, this method is invoked to
  let the bus driver specify the unit of a device given a specific
  devclass.  This is the only way a device can be given a name reserved
  via an "at" hint.
- Implement BUS_HINT_DEVICE_UNIT() for the acpi(4) and isa(4) bus drivers.
  Both of these busses implement this by comparing the resources for a
  given hint device with the resources enumerated by ACPI/PnPBIOS and
  wire a unit if the hint resources are a subset of the "real" resources.
- Use bus_hinted_children() for adding hinted devices on isa(4) busses
  now instead of doing it by hand.
- Remove the unit kludging from sio(4) as it is no longer necessary.

Prodding from:	peter, imp
OK'd by:	marcel
MFC after:	1 month

4 files changed, 149 insertions, 103 deletions

diff --git a/sys/isa/isa_common.c b/sys/isa/isa_common.c
index eb0618e..bcf7729 100644
--- a/sys/isa/isa_common.c
+++ b/sys/isa/isa_common.c
@@ -467,15 +467,41 @@ isa_assign_resources(device_t child)
 	return (0);
 }
 
+/*
+ * Claim any unallocated resources to keep other devices from using
+ * them.
+ */
+static void
+isa_claim_resources(device_t dev, device_t child)
+{
+	struct isa_device *idev = DEVTOISA(child);
+	struct resource_list *rl = &idev->id_resources;
+	struct resource_list_entry *rle;
+	int rid;
+
+	STAILQ_FOREACH(rle, rl, link) {
+		if (!rle->res) {
+			rid = rle->rid;
+			resource_list_alloc(rl, dev, child, rle->type, &rid,
+			    0ul, ~0ul, 1, 0);
+		}
+	}
+}
+
+/*
+ * Called after other devices have initialised to probe for isa devices.
+ */
 void
 isa_probe_children(device_t dev)
 {
-	device_t *children;
+	struct isa_device *idev;
+	device_t *children, child;
 	struct isa_config *cfg;
 	int nchildren, i;
 
 	/*
-	 * Create all the children by calling driver's identify methods.
+	 * Create all the non-hinted children by calling drivers'
+	 * identify methods.
 	 */
 	bus_generic_probe(dev);
 
@@ -496,8 +522,7 @@ isa_probe_children(device_t dev)
 	}
 
 	for (i = 0; i < nchildren; i++) {
-		device_t child = children[i];
-		struct isa_device *idev = DEVTOISA(child);
+		idev = DEVTOISA(children[i]);
 
 		bzero(cfg, sizeof(*cfg));
 		if (idev->id_config_cb)
@@ -507,16 +532,39 @@ isa_probe_children(device_t dev)
 	free(cfg, M_TEMP);
 
 	/*
-	 * Next probe all non-pnp devices so that they claim their
-	 * resources first.
+	 * Next, probe all the PnP BIOS devices so they can subsume any
+	 * hints.
 	 */
+	for (i = 0; i < nchildren; i++) {
+		child = children[i];
+		idev = DEVTOISA(child);
+
+		if (idev->id_order > ISA_ORDER_PNPBIOS)
+			continue;
+		if (!TAILQ_EMPTY(&idev->id_configs) &&
+		    !isa_assign_resources(child))
+			continue;
+
+		if (device_probe_and_attach(child) == 0)
+			isa_claim_resources(dev, child);
+	}
+	free(children, M_TEMP);
+
+	/*
+	 * Next, enumerate hinted devices and probe all non-pnp devices so
+	 * that they claim their resources first.
+	 */
+	bus_enumerate_hinted_children(dev);
+	if (device_get_children(dev, &children, &nchildren))
+		return;
 	if (bootverbose)
 		printf("isa_probe_children: probing non-PnP devices\n");
 	for (i = 0; i < nchildren; i++) {
-		device_t child = children[i];
-		struct isa_device *idev = DEVTOISA(child);
+		child = children[i];
+		idev = DEVTOISA(child);
 
-		if (TAILQ_FIRST(&idev->id_configs))
+		if (device_is_attached(child) ||
+		    !TAILQ_EMPTY(&idev->id_configs))
 			continue;
 
 		device_probe_and_attach(child);
@@ -528,31 +576,15 @@ isa_probe_children(device_t dev)
 	if (bootverbose)
 		printf("isa_probe_children: probing PnP devices\n");
 	for (i = 0; i < nchildren; i++) {
-		device_t child = children[i];
-		struct isa_device* idev = DEVTOISA(child);
+		child = children[i];
+		idev = DEVTOISA(child);
 
-		if (!TAILQ_FIRST(&idev->id_configs))
+		if (device_is_attached(child) || TAILQ_EMPTY(&idev->id_configs))
 			continue;
 
 		if (isa_assign_resources(child)) {
-			struct resource_list *rl = &idev->id_resources;
-			struct resource_list_entry *rle;
-
 			device_probe_and_attach(child);
-
-			/*
-			 * Claim any unallocated resources to keep other
-			 * devices from using them.
-			 */
-			STAILQ_FOREACH(rle, rl, link) {
-				if (!rle->res) {
-					int rid = rle->rid;
-					resource_list_alloc(rl, dev, child,
-							    rle->type,
-							    &rid,
-							    0, ~0, 1, 0);
-				}
-			}
+			isa_claim_resources(dev, child);
 		}
 	}
 
@@ -580,6 +612,7 @@ isa_add_child(device_t dev, int order, const char *name, int unit)
 
 	resource_list_init(&idev->id_resources);
 	TAILQ_INIT(&idev->id_configs);
+	idev->id_order = order;
 
 	device_set_ivars(child, idev);
 
@@ -833,23 +866,9 @@ static void
 isa_child_detached(device_t dev, device_t child)
 {
 	struct isa_device* idev = DEVTOISA(child);
-	struct resource_list *rl = &idev->id_resources;
-	struct resource_list_entry *rle;
 
-	if (TAILQ_FIRST(&idev->id_configs)) {
-		/*
-		 * Claim any unallocated resources to keep other
-		 * devices from using them.
-		 */
-		STAILQ_FOREACH(rle, rl, link) {
-			if (!rle->res) {
-				int rid = rle->rid;
-				resource_list_alloc(rl, dev, child,
-						    rle->type,
-						    &rid, 0, ~0, 1, 0);
-			}
-		}
-	}
+	if (TAILQ_FIRST(&idev->id_configs))
+		isa_claim_resources(dev, child);
 }
 
 static void
@@ -900,20 +919,8 @@ isa_driver_added(device_t dev, driver_t *driver)
 
 		device_probe_and_attach(child);
 
-		if (TAILQ_FIRST(&idev->id_configs)) {
-			/*
-			 * Claim any unallocated resources to keep other
-			 * devices from using them.
-			 */
-			STAILQ_FOREACH(rle, rl, link) {
-				if (!rle->res) {
-					int rid = rle->rid;
-					resource_list_alloc(rl, dev, child,
-							    rle->type,
-							    &rid, 0, ~0, 1, 0);
-				}
-			}
-		}
+		if (TAILQ_FIRST(&idev->id_configs))
+			isa_claim_resources(dev, child);
 	}
 
 	free(children, M_TEMP);
@@ -1077,6 +1084,8 @@ static device_method_t isa_methods[] = {
 	DEVMETHOD(bus_deactivate_resource, bus_generic_deactivate_resource),
 	DEVMETHOD(bus_child_pnpinfo_str, isa_child_pnpinfo_str),
 	DEVMETHOD(bus_child_location_str, isa_child_location_str),
+	DEVMETHOD(bus_hinted_child,	isa_hinted_child),
+	DEVMETHOD(bus_hint_device_unit,	isa_hint_device_unit),
 
 	/* ISA interface */
 	DEVMETHOD(isa_add_config,	isa_add_config),
@@ -1086,11 +1095,7 @@ static device_method_t isa_methods[] = {
 	{ 0, 0 }
 };
 
-driver_t isa_driver = {
-	"isa",
-	isa_methods,
-	1,			/* no softc */
-};
+DEFINE_CLASS_0(isa, isa_driver, isa_methods, 0);
 
 devclass_t isa_devclass;
 
diff --git a/sys/isa/isa_common.h b/sys/isa/isa_common.h
index c6d6e43..340ad17 100644
--- a/sys/isa/isa_common.h
+++ b/sys/isa/isa_common.h
@@ -59,6 +59,7 @@ struct isa_device {
 	int			id_pnpbios_handle; /* pnp handle, if any */
 	int			id_pnp_csn; /* pnp Card Number */
 	int			id_pnp_ldn; /* pnp Logical device on card */
+	int			id_order;
 };
 
 #define DEVTOISA(dev)	((struct isa_device *) device_get_ivars(dev))
diff --git a/sys/isa/isahint.c b/sys/isa/isahint.c
index 01f8823..e2ce6a4 100644
--- a/sys/isa/isahint.c
+++ b/sys/isa/isahint.c
@@ -33,10 +33,11 @@ __FBSDID("$FreeBSD$");
 #include <sys/bus.h>
 #include <sys/module.h>
 #include <isa/isavar.h>
+#include <isa/isa_common.h>
 #include <machine/resource.h>
 
-static void
-isahint_add_device(device_t parent, const char *name, int unit)
+void
+isa_hinted_child(device_t parent, const char *name, int unit)
 {
 	device_t	child;
 	int		sensitive, start, count;
@@ -82,43 +83,79 @@ isahint_add_device(device_t parent, const char *name, int unit)
 	isa_set_configattr(child, (isa_get_configattr(child)|ISACFGATTR_HINTS));
 }
 
-static void
-isahint_identify(driver_t *driver, device_t parent)
+static int
+isa_match_resource_hint(device_t dev, int type, long value)
 {
-	int i;
-	static char buf[] = "isaXXX";
-	const char *dname;
-	int dunit;
-
-	/*
-	 * Add all devices configured to be attached to parent.
-	 */
-	sprintf(buf, "isa%d", device_get_unit(parent));
-	i = 0;
-	while (resource_find_match(&i, &dname, &dunit, "at", buf) == 0)
-		isahint_add_device(parent, dname, dunit);
-
-	/*
-	 * and isa?
-	 */
-	i = 0;
-	while (resource_find_match(&i, &dname, &dunit, "at", "isa") == 0)
-		isahint_add_device(parent, dname, dunit);
+	struct isa_device* idev = DEVTOISA(dev);
+	struct resource_list *rl = &idev->id_resources;
+	struct resource_list_entry *rle;
+
+	STAILQ_FOREACH(rle, rl, link) {
+		if (rle->type != type)
+			continue;
+		if (rle->start <= value && rle->end >= value)
+			return (1);
+	}
+	return (0);
 }
 
-static device_method_t isahint_methods[] = {
-	/* Device interface */
-	DEVMETHOD(device_identify,	isahint_identify),
-
-	{ 0, 0 }
-};
-
-static driver_t isahint_driver = {
-	"hint",
-	isahint_methods,
-	1,			/* no softc */
-};
-
-static devclass_t hint_devclass;
-
-DRIVER_MODULE(isahint, isa, isahint_driver, hint_devclass, 0, 0);
+void
+isa_hint_device_unit(device_t bus, device_t child, const char *name, int *unitp)
+{
+	const char *s;
+	long value;
+	int line, matches, unit;
+
+	line = 0;
+	for (;;) {
+		if (resource_find_dev(&line, name, &unit, "at", NULL) != 0)
+			break;
+
+		/* Must have an "at" for isa. */
+		resource_string_value(name, unit, "at", &s);
+		if (!(strcmp(s, device_get_nameunit(bus)) == 0 ||
+		    strcmp(s, device_get_name(bus)) == 0))
+			continue;
+
+		/*
+		 * Check for matching resources.  We must have at least one,
+		 * and all resources specified have to match.
+		 *
+		 * XXX: We may want to revisit this to be more lenient and wire
+		 * as long as it gets one match.
+		 */
+		matches = 0;
+		if (resource_long_value(name, unit, "port", &value) == 0) {
+			if (isa_match_resource_hint(child, SYS_RES_IOPORT,
+			    value))
+				matches++;
+			else
+				continue;
+		}
+		if (resource_long_value(name, unit, "maddr", &value) == 0) {
+			if (isa_match_resource_hint(child, SYS_RES_MEMORY,
+			    value))
+				matches++;
+			else
+				continue;
+		}
+		if (resource_long_value(name, unit, "irq", &value) == 0) {
+			if (isa_match_resource_hint(child, SYS_RES_IRQ, value))
+				matches++;
+			else
+				continue;
+		}
+		if (resource_long_value(name, unit, "drq", &value) == 0) {
+			if (isa_match_resource_hint(child, SYS_RES_DRQ, value))
+				matches++;
+			else
+				continue;
+		}
+
+		if (matches > 0) {
+			/* We have a winner! */
+			*unitp = unit;
+			break;
+		}
+	}
+}
diff --git a/sys/isa/isavar.h b/sys/isa/isavar.h
index 59d3bc9..3b1a2e6 100644
--- a/sys/isa/isavar.h
+++ b/sys/isa/isavar.h
@@ -181,6 +181,9 @@ int	isa_dmatc(int chan);
 		    (int)(chan), (uintmax_t)(size)); \
 	} while (0) 
 
+void	isa_hinted_child(device_t parent, const char *name, int unit);
+void	isa_hint_device_unit(device_t bus, device_t child, const char *name,
+	    int *unitp);
 int	isab_attach(device_t dev);
 
 #ifdef PC98