 @@ -1,14 +1,14 @@
-/*	$NetBSD: pci.h,v 1.1.2.13 2013/07/24 03:24:03 riastradh Exp $	*/
+/*	$NetBSD: pci.h,v 1.1.2.14 2013/07/24 03:32:19 riastradh Exp $	*/
 /*-
  * Copyright (c) 2013 The NetBSD Foundation, Inc.
  * All rights reserved.
+ *
  * This code is derived from software contributed to The NetBSD Foundation
  * by Taylor R. Campbell.
+ *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
 @@ -64,54 +64,61 @@ struct pci_device_id {
 CTASSERT(PCI_CLASS_BRIDGE_ISA == 0x0601);
 #define	PCI_VENDOR_ID_INTEL	PCI_VENDOR_INTEL
 #define	PCI_DEVFN(DEV, FN)						\
 	(__SHIFTIN((DEV), __BITS(3, 7)) | __SHIFTIN((FN), __BITS(0, 2)))
 #define	PCI_SLOT(DEVFN)		__SHIFTOUT((DEVFN), __BITS(3, 7))
 #define	PCI_FUNC(DEVFN)		__SHIFTOUT((DEVFN), __BITS(0, 2))
 #define	PCI_CAP_ID_AGP	PCI_CAP_AGP
 struct pci_dev {
 	struct pci_attach_args	pd_pa;
-	bool			pd_kludged;	/* XXX pci_kludgey_find_dev */
+	int			pd_kludges;	/* Gotta lose 'em...  */
 #define	NBPCI_KLUDGE_GET_MUMBLE	0x01
 #define	NBPCI_KLUDGE_MAP_ROM	0x02
 	bus_space_tag_t		pd_rom_bst;
 	bus_space_handle_t	pd_rom_bsh;
 	bus_size_t		pd_rom_size;
 	void			*pd_rom_vaddr;
 	device_t		pd_dev;
 	struct pci_bus		*bus;
 	uint32_t		devfn;
 	uint16_t		vendor;
 	uint16_t		device;
 	uint16_t		subsystem_vendor;
 	uint16_t		subsystem_device;
 	uint8_t			revision;
 	uint32_t		class;
 	bool 			msi_enabled;
 };
 static inline device_t
 pci_dev_dev(struct pci_dev *pdev)
+{
 	return pdev->pd_dev;
+}
 static inline void
 linux_pci_dev_init(struct pci_dev *pdev, device_t dev,
-    const struct pci_attach_args *pa, bool kludged)
+    const struct pci_attach_args *pa, int kludges)
+{
 	const uint32_t subsystem_id = pci_conf_read(pa->pa_pc, pa->pa_tag,
 	    PCI_SUBSYS_ID_REG);
 	pdev->pd_pa = *pa;
-	pdev->pd_kludged = kludged;
+	pdev->pd_kludges = kludges;
 	pdev->pd_rom_vaddr = NULL;
 	pdev->pd_dev = dev;
 	pdev->bus = NULL;	/* XXX struct pci_dev::bus */
 	pdev->devfn = PCI_DEVFN(pa->pa_device, pa->pa_function);
 	pdev->vendor = PCI_VENDOR(pa->pa_id);
 	pdev->device = PCI_PRODUCT(pa->pa_id);
 	pdev->subsystem_vendor = PCI_SUBSYS_VENDOR(subsystem_id);
 	pdev->subsystem_device = PCI_SUBSYS_ID(subsystem_id);
 	pdev->revision = PCI_REVISION(pa->pa_class);
 	pdev->class = __SHIFTOUT(pa->pa_class, 0xffffff00UL); /* ? */
 	pdev->msi_enabled = false;
+}
 static inline int
 @@ -276,27 +283,27 @@ pci_kludgey_match_bus0_dev0_func0(const
 static inline struct pci_dev *
 pci_get_bus_and_slot(int bus, int slot)
+{
 	struct pci_attach_args pa;
 	KASSERT(bus == 0);
 	KASSERT(slot == PCI_DEVFN(0, 0));
 	if (!pci_find_device(&pa, &pci_kludgey_match_bus0_dev0_func0))
 		return NULL;
 	struct pci_dev *const pdev = kmem_zalloc(sizeof(*pdev), KM_SLEEP);
-	linux_pci_dev_init(pdev, NULL, &pa, true);
+	linux_pci_dev_init(pdev, NULL, &pa, NBPCI_KLUDGE_GET_MUMBLE);
 	return pdev;
+}
 static inline int		/* XXX inline?  */
 pci_kludgey_match_isa_bridge(const struct pci_attach_args *pa)
+{
 	if (PCI_CLASS(pa->pa_class) != PCI_CLASS_BRIDGE)
 		return 0;
 	if (PCI_SUBCLASS(pa->pa_class) != PCI_SUBCLASS_BRIDGE_ISA)
 		return 0;
 @@ -306,27 +313,71 @@ pci_kludgey_match_isa_bridge(const struc
 static inline struct pci_dev *
 pci_get_class(uint32_t class_subclass_shifted __unused,
     struct pci_dev *from __unused)
+{
 	struct pci_attach_args pa;
 	KASSERT(class_subclass_shifted == (PCI_CLASS_BRIDGE_ISA << 8));
 	KASSERT(from == NULL);
 	if (!pci_find_device(&pa, &pci_kludgey_match_isa_bridge))
 		return NULL;
 	struct pci_dev *const pdev = kmem_zalloc(sizeof(*pdev), KM_SLEEP);
-	linux_pci_dev_init(pdev, NULL, &pa, true);
+	linux_pci_dev_init(pdev, NULL, &pa, NBPCI_KLUDGE_GET_MUMBLE);
 	return pdev;
+}
 static inline void
 pci_dev_put(struct pci_dev *pdev)
+{
-	KASSERT(pdev->pd_kludged);
+	KASSERT(ISSET(pdev->pd_kludges, NBPCI_KLUDGE_GET_MUMBLE));
 	kmem_free(pdev, sizeof(*pdev));
+}
 #define	__pci_rom_iomem
 static inline void
 pci_unmap_rom(struct pci_dev *pdev, void __pci_rom_iomem *vaddr __unused)
+{
 	KASSERT(ISSET(pdev->pd_kludges, NBPCI_KLUDGE_MAP_ROM));
 	KASSERT(vaddr == pdev->pd_rom_vaddr);
 	bus_space_unmap(pdev->pd_rom_bst, pdev->pd_rom_bsh, pdev->pd_rom_size);
 	pdev->pd_kludges &= ~NBPCI_KLUDGE_MAP_ROM;
 	pdev->pd_rom_vaddr = NULL;
+}
 static inline void __pci_rom_iomem *
 pci_map_rom(struct pci_dev *pdev, size_t *sizep)
+{
 	bus_space_handle_t bsh;
 	bus_size_t size;
 	KASSERT(!ISSET(pdev->pd_kludges, NBPCI_KLUDGE_MAP_ROM));
 	if (pci_mapreg_map(&pdev->pd_pa, PCI_MAPREG_ROM, PCI_MAPREG_TYPE_ROM,
 		(BUS_SPACE_MAP_PREFETCHABLE | BUS_SPACE_MAP_LINEAR),
 		&pdev->pd_rom_bst, &pdev->pd_rom_bsh, NULL, &pdev->pd_rom_size)
 	    != 0) {
 		aprint_error_dev(pdev->pd_dev, "unable to map ROM\n");
 		return NULL;
+	}
 	pdev->pd_kludges |= NBPCI_KLUDGE_MAP_ROM;
 	/* XXX This type is obviously wrong in general...  */
 	if (pci_find_rom(&pdev->pd_pa, pdev->pd_rom_bst, pdev->pd_rom_bsh,
 		PCI_ROM_CODE_TYPE_X86, &bsh, &size)) {
 		aprint_error_dev(pdev->pd_dev, "unable to find ROM\n");
 		pci_unmap_rom(pdev, NULL);
 		return NULL;
+	}
 	KASSERT(size <= SIZE_T_MAX);
 	*sizep = size;
 	pdev->pd_rom_vaddr = bus_space_vaddr(pdev->pd_rom_bst, bsh);
 	return pdev->pd_rom_vaddr;
+}
 #endif  /* _LINUX_PCI_H_ */

 @@ -1,14 +1,14 @@
-/*	$NetBSD: drm_pci.c,v 1.1.2.2 2013/07/24 03:24:03 riastradh Exp $	*/
+/*	$NetBSD: drm_pci.c,v 1.1.2.3 2013/07/24 03:32:19 riastradh Exp $	*/
 /*-
  * Copyright (c) 2013 The NetBSD Foundation, Inc.
  * All rights reserved.
+ *
  * This code is derived from software contributed to The NetBSD Foundation
  * by Taylor R. Campbell.
+ *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
 @@ -20,27 +20,27 @@
  * ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED
  * TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
  * PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL THE FOUNDATION OR CONTRIBUTORS
  * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
  * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
  * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
  * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
  * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
  * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
  * POSSIBILITY OF SUCH DAMAGE.
  */
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: drm_pci.c,v 1.1.2.2 2013/07/24 03:24:03 riastradh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: drm_pci.c,v 1.1.2.3 2013/07/24 03:32:19 riastradh Exp $");
 #include <sys/types.h>
 #include <sys/errno.h>
 #include <sys/systm.h>
 #include <dev/pci/pcivar.h>
 #include <drm/drmP.h>
 static int	drm_pci_get_irq(struct drm_device *);
 static int	drm_pci_irq_install(struct drm_device *,
 		    irqreturn_t (*)(void *), int, const char *, void *,
 		    struct drm_bus_irq_cookie **);
 @@ -68,27 +68,27 @@ const struct drm_bus drm_pci_bus = {
 };
 static const struct pci_attach_args *
 drm_pci_attach_args(struct drm_device *dev)
+{
 	return &dev->pdev->pd_pa;
+}
 void
 drm_pci_attach(device_t self, const struct pci_attach_args *pa,
     struct pci_dev *pdev, struct drm_device *dev)
+{
-	linux_pci_dev_init(pdev, self, pa, false);
+	linux_pci_dev_init(pdev, self, pa, 0);
 	dev->pdev = pdev;
 	dev->pci_vendor = pdev->vendor;
 	dev->pci_device = pdev->device;
 	/* XXX Set the power state to D0?  */
+}
 int
 drm_pci_detach(struct drm_device *dev __unused, int flags __unused)
+{
 	/* XXX Disestablish irqs or anything?  */