Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 23 Mar 2015 06:52:54 GMT
From:      John-Mark Gurney <jmg@FreeBSD.org>
To:        Perforce Change Reviews <perforce@FreeBSD.org>
Subject:   PERFORCE change 1207530 for review
Message-ID:  <201503230652.t2N6qscu060980@skunkworks.freebsd.org>

next in thread | raw e-mail | index | archive | help

http://p4web.freebsd.org/@@1207530?ac=10

Change 1207530 by jmg@jmg_pciehp on 2015/03/23 06:51:53

	simplify how resume is handled, ones we resume, NULL out, and
	then resume the rest.. this keeps the list of important ones in
	a single place...
	
	Sponsored by:	FreeBSD Foundation

Affected files ...

.. //depot/projects/pciehotplug/sys/dev/pci/pci.c#8 edit

Differences ...

==== //depot/projects/pciehotplug/sys/dev/pci/pci.c#8 (text+ko) ====

@@ -3829,20 +3829,14 @@
 		case PCIC_BRIDGE:
 		case PCIC_BASEPERIPH:
 			BUS_RESUME_CHILD(dev, child);
+			devlist[i] = NULL;
 			break;
 		}
 	}
 	for (i = 0; i < numdevs; i++) {
 		child = devlist[i];
-		switch (pci_get_class(child)) {
-		case PCIC_DISPLAY:
-		case PCIC_MEMORY:
-		case PCIC_BRIDGE:
-		case PCIC_BASEPERIPH:
-			break;
-		default:
+		if (child != NULL)
 			BUS_RESUME_CHILD(dev, child);
-		}
 	}
 	free(devlist, M_TEMP);
 	return (0);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201503230652.t2N6qscu060980>