Module Name:    src
Committed By:   cegger
Date:           Thu May  7 06:08:02 UTC 2009

Modified Files:
        src/sys/dev/pci: esm.c esmvar.h

Log Message:
device_t/softc split
patch presented on tech-kern@, no comments on it


To generate a diff of this commit:
cvs rdiff -u -r1.48 -r1.49 src/sys/dev/pci/esm.c
cvs rdiff -u -r1.16 -r1.17 src/sys/dev/pci/esmvar.h

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/dev/pci/esm.c
diff -u src/sys/dev/pci/esm.c:1.48 src/sys/dev/pci/esm.c:1.49
--- src/sys/dev/pci/esm.c:1.48	Tue May  5 21:47:49 2009
+++ src/sys/dev/pci/esm.c	Thu May  7 06:08:02 2009
@@ -1,4 +1,4 @@
-/*      $NetBSD: esm.c,v 1.48 2009/05/05 21:47:49 cegger Exp $      */
+/*      $NetBSD: esm.c,v 1.49 2009/05/07 06:08:02 cegger Exp $      */
 
 /*-
  * Copyright (c) 2002, 2003 Matt Fredette
@@ -66,7 +66,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: esm.c,v 1.48 2009/05/05 21:47:49 cegger Exp $");
+__KERNEL_RCSID(0, "$NetBSD: esm.c,v 1.49 2009/05/07 06:08:02 cegger Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -159,7 +159,7 @@
 			             struct esm_dma *);
 
 
-CFATTACH_DECL2(esm, sizeof(struct esm_softc),
+CFATTACH_DECL2_NEW(esm, sizeof(struct esm_softc),
     esm_match, esm_attach, esm_detach, NULL, NULL, esm_childdet);
 
 const struct audio_hw_if esm_hw_if = {
@@ -287,7 +287,7 @@
 {
 	int i;
 
-	printf("%s registers:", device_xname(&ess->sc_dev));
+	printf("%s registers:", device_xname(ess->sc_dev));
 	for (i = 0; i < (sizeof dump_regs / sizeof dump_regs[0]); i++) {
 		if (i % 5 == 0)
 			printf("\n");
@@ -336,7 +336,7 @@
 	}
 	if (t == 20)
 		printf("%s: esm_read_codec() PROGLESS timed out.\n",
-		    device_xname(&ess->sc_dev));
+		    device_xname(ess->sc_dev));
 
 	bus_space_write_1(ess->st, ess->sh, PORT_CODEC_CMD,
 	    CODEC_CMD_READ | regno);
@@ -352,7 +352,7 @@
 	if (t == 20)
 		/* Timed out, but perform dummy read. */
 		printf("%s: esm_read_codec() RW_DONE timed out.\n",
-		    device_xname(&ess->sc_dev));
+		    device_xname(ess->sc_dev));
 
 	*result = bus_space_read_2(ess->st, ess->sh, PORT_CODEC_REG);
 
@@ -376,7 +376,7 @@
 	if (t == 20) {
 		/* Timed out. Abort writing. */
 		printf("%s: esm_write_codec() PROGLESS timed out.\n",
-		    device_xname(&ess->sc_dev));
+		    device_xname(ess->sc_dev));
 		return -1;
 	}
 
@@ -431,7 +431,7 @@
 		bus_space_write_2(ess->st, ess->sh, PORT_DSP_DATA, reg);
 	}
 	if (t == 1000)
-		printf("%s: apu_setindex() timed out.\n", device_xname(&ess->sc_dev));
+		printf("%s: apu_setindex() timed out.\n", device_xname(ess->sc_dev));
 }
 
 static inline uint16_t
@@ -461,7 +461,7 @@
 		bus_space_write_2(ess->st, ess->sh, PORT_DSP_DATA, data);
 	}
 	if (t == 1000)
-		printf("%s: wp_wrapu() timed out.\n", device_xname(&ess->sc_dev));
+		printf("%s: wp_wrapu() timed out.\n", device_xname(ess->sc_dev));
 }
 
 static inline void
@@ -599,7 +599,7 @@
 		delay(21);
 
 		/* Try cold reset. */
-		printf("%s: will perform cold reset.\n", device_xname(&ess->sc_dev));
+		printf("%s: will perform cold reset.\n", device_xname(ess->sc_dev));
 		data = bus_space_read_2(ess->st, ess->sh, PORT_GPIO_DIR);
 		if (pci_conf_read(ess->pc, ess->tag, 0x58) & 1)
 			data |= 0x10;
@@ -697,14 +697,14 @@
 	p = &ess->sc_dma;
 	if ((char *)start != (char *)p->addr + MAESTRO_PLAYBUF_OFF) {
 		printf("%s: esm_init_output: bad addr %p\n",
-		    device_xname(&ess->sc_dev), start);
+		    device_xname(ess->sc_dev), start);
 		return EINVAL;
 	}
 
 	ess->pch.base = DMAADDR(p) + MAESTRO_PLAYBUF_OFF;
 
 	DPRINTF(ESM_DEBUG_DMA, ("%s: pch.base = 0x%x\n",
-		device_xname(&ess->sc_dev), ess->pch.base));
+		device_xname(ess->sc_dev), ess->pch.base));
 
 	return 0;
 }
@@ -719,7 +719,7 @@
 	p = &ess->sc_dma;
 	if ((char *)start != (char *)p->addr + MAESTRO_RECBUF_OFF) {
 		printf("%s: esm_init_input: bad addr %p\n",
-		    device_xname(&ess->sc_dev), start);
+		    device_xname(ess->sc_dev), start);
 		return EINVAL;
 	}
 
@@ -733,7 +733,7 @@
 	}
 
 	DPRINTF(ESM_DEBUG_DMA, ("%s: rch.base = 0x%x\n",
-		device_xname(&ess->sc_dev), ess->rch.base));
+		device_xname(ess->sc_dev), ess->rch.base));
 
 	return 0;
 }
@@ -764,7 +764,7 @@
 #ifdef DIAGNOSTIC
 	if (ess->pactive) {
 		printf("%s: esm_trigger_output: already running",
-		    device_xname(&ess->sc_dev));
+		    device_xname(ess->sc_dev));
 		return EINVAL;
 	}
 #endif
@@ -774,7 +774,7 @@
 	p = &ess->sc_dma;
 	if ((char *)start != (char *)p->addr + MAESTRO_PLAYBUF_OFF) {
 		printf("%s: esm_trigger_output: bad addr %p\n",
-		    device_xname(&ess->sc_dev), start);
+		    device_xname(ess->sc_dev), start);
 		return EINVAL;
 	}
 
@@ -880,7 +880,7 @@
 #ifdef DIAGNOSTIC
 	if (ess->ractive) {
 		printf("%s: esm_trigger_input: already running",
-		    device_xname(&ess->sc_dev));
+		    device_xname(ess->sc_dev));
 		return EINVAL;
 	}
 #endif
@@ -890,7 +890,7 @@
 	p = &ess->sc_dma;
 	if ((char *)start != (char *)p->addr + MAESTRO_RECBUF_OFF) {
 		printf("%s: esm_trigger_input: bad addr %p\n",
-		    device_xname(&ess->sc_dev), start);
+		    device_xname(ess->sc_dev), start);
 		return EINVAL;
 	}
 
@@ -1602,6 +1602,7 @@
 	int error;
 
 	ess = device_private(self);
+	ess->sc_dev = self;
 	pa = (struct pci_attach_args *)aux;
 	pc = pa->pa_pc;
 	tag = pa->pa_tag;
@@ -1619,7 +1620,7 @@
 	/* Map I/O register */
 	if (pci_mapreg_map(pa, PCI_CBIO, PCI_MAPREG_TYPE_IO, 0,
 	    &ess->st, &ess->sh, NULL, &ess->sz)) {
-		aprint_error_dev(&ess->sc_dev, "can't map i/o space\n");
+		aprint_error_dev(ess->sc_dev, "can't map i/o space\n");
 		return;
 	}
 
@@ -1633,24 +1634,24 @@
 
 	DPRINTF(ESM_DEBUG_PCI,
 	    ("%s: sub-system vendor 0x%4.4x, product 0x%4.4x\n",
-	    device_xname(&ess->sc_dev),
+	    device_xname(ess->sc_dev),
 	    PCI_VENDOR(ess->subid), PCI_PRODUCT(ess->subid)));
 
 	/* Map and establish the interrupt. */
 	if (pci_intr_map(pa, &ih)) {
-		aprint_error_dev(&ess->sc_dev, "can't map interrupt\n");
+		aprint_error_dev(ess->sc_dev, "can't map interrupt\n");
 		return;
 	}
 	intrstr = pci_intr_string(pc, ih);
 	ess->ih = pci_intr_establish(pc, ih, IPL_AUDIO, esm_intr, self);
 	if (ess->ih == NULL) {
-		aprint_error_dev(&ess->sc_dev, "can't establish interrupt");
+		aprint_error_dev(ess->sc_dev, "can't establish interrupt");
 		if (intrstr != NULL)
 			aprint_normal(" at %s", intrstr);
 		aprint_normal("\n");
 		return;
 	}
-	aprint_normal_dev(&ess->sc_dev, "interrupting at %s\n",
+	aprint_normal_dev(ess->sc_dev, "interrupting at %s\n",
 	    intrstr);
 
 	/*
@@ -1660,7 +1661,7 @@
 	/* power up chip */
 	if ((error = pci_activate(pa->pa_pc, pa->pa_tag, self,
 	    pci_activate_null)) && error != EOPNOTSUPP) {
-		aprint_error_dev(&ess->sc_dev, "cannot activate %d\n",
+		aprint_error_dev(ess->sc_dev, "cannot activate %d\n",
 		    error);
 		return;
 	}
@@ -1684,7 +1685,7 @@
 
 	esm_read_codec(ess, 0, &codec_data);
 	if (codec_data == 0x80) {
-		aprint_error_dev(&ess->sc_dev, "PT101 codec detected!\n");
+		aprint_error_dev(ess->sc_dev, "PT101 codec detected!\n");
 		return;
 	}
 
@@ -1713,7 +1714,7 @@
 	/* allocate our DMA region */
 	if (esm_allocmem(ess, MAESTRO_DMA_SZ, MAESTRO_DMA_ALIGN,
 		&ess->sc_dma)) {
-		aprint_error_dev(&ess->sc_dev, "couldn't allocate memory!\n");
+		aprint_error_dev(ess->sc_dev, "couldn't allocate memory!\n");
 		return;
 	}
 	ess->rings_alloced = 0;
@@ -1723,7 +1724,7 @@
 		wc_wrreg(ess, pcmbar,
 		    DMAADDR(&ess->sc_dma) >> WAVCACHE_BASEADDR_SHIFT);
 
-	audio_attach_mi(&esm_hw_if, self, &ess->sc_dev);
+	audio_attach_mi(&esm_hw_if, self, ess->sc_dev);
 
 	if (!pmf_device_register(self, esm_suspend, esm_resume))
 		aprint_error_dev(self, "couldn't establish power handler\n");
@@ -1806,7 +1807,7 @@
 #if 0
 	if (mixer_reinit(dev)) {
 		printf("%s: unable to reinitialize the mixer\n",
-		    device_xname(&ess->sc_dev));
+		    device_xname(ess->sc_dev));
 		return ENXIO;
 	}
 #endif

Index: src/sys/dev/pci/esmvar.h
diff -u src/sys/dev/pci/esmvar.h:1.16 src/sys/dev/pci/esmvar.h:1.17
--- src/sys/dev/pci/esmvar.h:1.16	Sat Feb 23 02:17:16 2008
+++ src/sys/dev/pci/esmvar.h	Thu May  7 06:08:02 2009
@@ -1,4 +1,4 @@
-/*	$NetBSD: esmvar.h,v 1.16 2008/02/23 02:17:16 dyoung Exp $	*/
+/*	$NetBSD: esmvar.h,v 1.17 2009/05/07 06:08:02 cegger Exp $	*/
 
 /*-
  * Copyright (c) 2002, 2003 Matt Fredette
@@ -150,7 +150,7 @@
 };
 
 struct esm_softc {
-	struct device		sc_dev;
+	device_t		sc_dev;
 
 	bus_space_tag_t		st;
 	bus_space_handle_t	sh;

Reply via email to