Module Name:    src
Committed By:   jmcneill
Date:           Thu Oct 15 09:06:04 UTC 2015

Modified Files:
        src/sys/arch/arm/nvidia: tegra_pcie.c tegra_pciereg.h

Log Message:
explicitly disable PCIe MSI as we dont support it yet


To generate a diff of this commit:
cvs rdiff -u -r1.3 -r1.4 src/sys/arch/arm/nvidia/tegra_pcie.c
cvs rdiff -u -r1.1 -r1.2 src/sys/arch/arm/nvidia/tegra_pciereg.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/arch/arm/nvidia/tegra_pcie.c
diff -u src/sys/arch/arm/nvidia/tegra_pcie.c:1.3 src/sys/arch/arm/nvidia/tegra_pcie.c:1.4
--- src/sys/arch/arm/nvidia/tegra_pcie.c:1.3	Fri Oct  2 05:22:50 2015
+++ src/sys/arch/arm/nvidia/tegra_pcie.c	Thu Oct 15 09:06:04 2015
@@ -1,4 +1,4 @@
-/* $NetBSD: tegra_pcie.c,v 1.3 2015/10/02 05:22:50 msaitoh Exp $ */
+/* $NetBSD: tegra_pcie.c,v 1.4 2015/10/15 09:06:04 jmcneill Exp $ */
 
 /*-
  * Copyright (c) 2015 Jared D. McNeill <jmcne...@invisible.ca>
@@ -29,7 +29,7 @@
 #include "locators.h"
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: tegra_pcie.c,v 1.3 2015/10/02 05:22:50 msaitoh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: tegra_pcie.c,v 1.4 2015/10/15 09:06:04 jmcneill Exp $");
 
 #include <sys/param.h>
 #include <sys/bus.h>
@@ -193,34 +193,51 @@ tegra_pcie_attach(device_t parent, devic
 }
 
 static int
-tegra_pcie_intr(void *priv)
+tegra_pcie_legacy_intr(struct tegra_pcie_softc *sc)
 {
-	struct tegra_pcie_softc *sc = priv;
+	const uint32_t msg = bus_space_read_4(sc->sc_bst, sc->sc_bsh_afi,
+	    AFI_MSG_REG);
 	struct tegra_pcie_ih *pcie_ih;
+	int rv = 0;
 
-	const uint32_t code = bus_space_read_4(sc->sc_bst, sc->sc_bsh_afi,
-	    AFI_INTR_CODE_REG);
-	const uint32_t sig = bus_space_read_4(sc->sc_bst, sc->sc_bsh_afi,
-	    AFI_INTR_SIGNATURE_REG);
-	bus_space_write_4(sc->sc_bst, sc->sc_bsh_afi, AFI_INTR_CODE_REG, 0);
-
-	switch (__SHIFTOUT(code, AFI_INTR_CODE_INT_CODE)) {
-	case AFI_INTR_CODE_SM_MSG:
+	if (msg & (AFI_MSG_INT0|AFI_MSG_INT1)) {
 		mutex_enter(&sc->sc_lock);
 		const u_int lastgen = sc->sc_intrgen;
 		TAILQ_FOREACH(pcie_ih, &sc->sc_intrs, ih_entry) {
 			int (*callback)(void *) = pcie_ih->ih_callback;
 			void *arg = pcie_ih->ih_arg;
 			mutex_exit(&sc->sc_lock);
-			const int rv = callback(arg);
-			if (rv)
-				return rv;
+			rv += callback(arg);
 			mutex_enter(&sc->sc_lock);
 			if (lastgen != sc->sc_intrgen)
 				break;
 		}
 		mutex_exit(&sc->sc_lock);
-		return 0;
+	} else if (msg & (AFI_MSG_PM_PME0|AFI_MSG_PM_PME1)) {
+		device_printf(sc->sc_dev, "PM PME message; AFI_MSG=%08x\n",
+		    msg);
+	} else {
+		bus_space_write_4(sc->sc_bst, sc->sc_bsh_afi, AFI_MSG_REG, msg);
+		rv = 1;
+	}
+
+	return rv;
+}
+
+static int
+tegra_pcie_intr(void *priv)
+{
+	struct tegra_pcie_softc *sc = priv;
+
+	const uint32_t code = bus_space_read_4(sc->sc_bst, sc->sc_bsh_afi,
+	    AFI_INTR_CODE_REG);
+	const uint32_t sig = bus_space_read_4(sc->sc_bst, sc->sc_bsh_afi,
+	    AFI_INTR_SIGNATURE_REG);
+	bus_space_write_4(sc->sc_bst, sc->sc_bsh_afi, AFI_INTR_CODE_REG, 0);
+
+	switch (__SHIFTOUT(code, AFI_INTR_CODE_INT_CODE)) {
+	case AFI_INTR_CODE_SM_MSG:
+		return tegra_pcie_legacy_intr(sc);
 	default:
 		device_printf(sc->sc_dev, "intr: code %#x sig %#x\n",
 		    code, sig);
@@ -231,6 +248,14 @@ tegra_pcie_intr(void *priv)
 static void
 tegra_pcie_enable(struct tegra_pcie_softc *sc)
 {
+	/* disable MSI */
+	bus_space_write_4(sc->sc_bst, sc->sc_bsh_afi,
+	    AFI_MSI_BAR_SZ_REG, 0);
+	bus_space_write_4(sc->sc_bst, sc->sc_bsh_afi,
+	    AFI_MSI_FPCI_BAR_ST_REG, 0);
+	bus_space_write_4(sc->sc_bst, sc->sc_bsh_afi,
+	    AFI_MSI_AXI_BAR_ST_REG, 0);
+
 	bus_space_write_4(sc->sc_bst, sc->sc_bsh_afi,
 	    AFI_SM_INTR_ENABLE_REG, 0xffffffff);
 	bus_space_write_4(sc->sc_bst, sc->sc_bsh_afi,

Index: src/sys/arch/arm/nvidia/tegra_pciereg.h
diff -u src/sys/arch/arm/nvidia/tegra_pciereg.h:1.1 src/sys/arch/arm/nvidia/tegra_pciereg.h:1.2
--- src/sys/arch/arm/nvidia/tegra_pciereg.h:1.1	Sun May  3 01:07:44 2015
+++ src/sys/arch/arm/nvidia/tegra_pciereg.h	Thu Oct 15 09:06:04 2015
@@ -1,4 +1,4 @@
-/* $NetBSD: tegra_pciereg.h,v 1.1 2015/05/03 01:07:44 jmcneill Exp $ */
+/* $NetBSD: tegra_pciereg.h,v 1.2 2015/10/15 09:06:04 jmcneill Exp $ */
 
 /*-
  * Copyright (c) 2015 Jared D. McNeill <jmcne...@invisible.ca>
@@ -30,11 +30,15 @@
 #define _ARM_TEGRA_PCIEREG_H
 
 /* AFI */
+#define AFI_MSI_BAR_SZ_REG	0x60
+#define AFI_MSI_FPCI_BAR_ST_REG	0x64
+#define AFI_MSI_AXI_BAR_ST_REG	0x68
 #define AFI_INTR_MASK_REG	0xb4
 #define AFI_INTR_CODE_REG	0xb8
 #define AFI_INTR_SIGNATURE_REG	0xbc
 #define AFI_SM_INTR_ENABLE_REG	0xc4
 #define AFI_AFI_INTR_ENABLE_REG	0xc8
+#define AFI_MSG_REG		0x190
 
 #define AFI_INTR_MASK_MSI	__BIT(8)
 #define AFI_INTR_MASK_INT	__BIT(0)
@@ -42,4 +46,9 @@
 #define AFI_INTR_CODE_INT_CODE	__BITS(4,0)
 #define AFI_INTR_CODE_SM_MSG	6
 
+#define AFI_MSG_INT1		__BITS(27,24)
+#define AFI_MSG_PM_PME1		__BIT(20)
+#define AFI_MSG_INT0		__BITS(11,8)
+#define AFI_MSG_PM_PME0		__BIT(4)
+
 #endif /* _ARM_TEGRA_PCIEREG_H */

Reply via email to