Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 14 Jul 2006 05:28:20 GMT
From:      Warner Losh <imp@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 101519 for review
Message-ID:  <200607140528.k6E5SKYH035477@repoman.freebsd.org>

next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=101519

Change 101519 by imp@imp_lighthouse on 2006/07/14 05:27:23

	Minor style nits
	Revert part of 1.132: better to put the card in reset than to
	write a 0 to EXCA_INTR.  Writing 0 seems to cause problems on
	TI bridges (it worked on my Sony with a Ricoh bridge).

Affected files ...

.. //depot/projects/arm/src/sys/dev/pccbb/pccbb.c#13 edit

Differences ...

==== //depot/projects/arm/src/sys/dev/pccbb/pccbb.c#13 (text+ko) ====

@@ -1165,13 +1165,11 @@
 	int err;
 
 	DPRINTF(("cbb_pcic_socket_enable:\n"));
-
 	/* power down/up the socket to reset */
 	err = cbb_do_power(brdev);
 	if (err)
 		return (err);
 	exca_reset(&sc->exca[0], child);
-
 	return (0);
 }
 
@@ -1181,15 +1179,12 @@
 	struct cbb_softc *sc = device_get_softc(brdev);
 
 	DPRINTF(("cbb_pcic_socket_disable\n"));
-
-	/* Turn off the card's interrupt and leave it in reset */
-	exca_putb(&sc->exca[0], EXCA_INTR, 0);
+	/* reset signal asserting... */
+	exca_clrb(&sc->exca[0], EXCA_INTR, EXCA_INTR_RESET);
 	tsleep(sc, PZERO, "cbbP1", hz / 100);
-
 	/* power down the socket */
 	cbb_power(brdev, CARD_OFF);
 	exca_putb(&sc->exca[0], EXCA_PWRCTL, 0);
-
 	/* wait 300ms until power fails (Tpf). */
 	tsleep(sc, PZERO, "cbbP1", hz * 300 / 1000);
 }
@@ -1213,6 +1208,7 @@
 cbb_power_disable_socket(device_t brdev, device_t child)
 {
 	struct cbb_softc *sc = device_get_softc(brdev);
+
 	if (sc->flags & CBB_16BIT_CARD)
 		cbb_pcic_power_disable_socket(brdev, child);
 	else
@@ -1224,6 +1220,7 @@
     struct resource *res)
 {
 	struct cbb_softc *sc = device_get_softc(brdev);
+
 	return (exca_activate_resource(&sc->exca[0], child, type, rid, res));
 }
 
@@ -1232,6 +1229,7 @@
     int rid, struct resource *res)
 {
 	struct cbb_softc *sc = device_get_softc(brdev);
+
 	return (exca_deactivate_resource(&sc->exca[0], child, type, rid, res));
 }
 



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