Skip to contentSkip to conversation

openpic_hookup_cascade in linuxppc_2_5

PATCH In review Adrian Cox 2 participants 3 messages Mar 16, 2004

lore
Share & tools
lore

Adrian Cox

author
#0 lore raw

PATCH 1/1 openpic_hookup_cascade in linuxppc_2_5

This patch ports the openpic_hookup_cascade() function from 2_4_devel, as it is necessary for the Sandpoint X2. In doing this I have to change almost every user of openpic_init.

pmac needs no changes.
chrp, lopec, mcpn765, pplus, prep, and sandpoint need simple changes.

But some of the platforms don't use the new, single argument, form of openpic_init(). Thus, the following are already broken in 2.5: gemini, mvme5100, powerpmc250, prmpm750, and prpmc800. I don't have any of these platforms, so my patch doesn't touch them.

- Adrian Cox

arch/ppc/platforms/chrp_setup.c +1 -0
diff -Nru a/arch/ppc/platforms/chrp_setup.c b/arch/ppc/platforms/chrp_setup.c
@@ -407,6 +407,7 @@
 	OpenPIC_NumInitSenses = NR_IRQS - NUM_8259_INTERRUPTS;
 
 	openpic_init(NUM_8259_INTERRUPTS);
+        openpic_hookup_cascade(NUM_8259_INTERRUPTS, "82c59 cascade", &i8259_irq);
 
 	for (i = 0; i < NUM_8259_INTERRUPTS; i++)
 		irq_desc[i].handler = &i8259_pic;
arch/ppc/platforms/lopec_setup.c +1 -0
diff -Nru a/arch/ppc/platforms/lopec_setup.c b/arch/ppc/platforms/lopec_setup.c
@@ -208,6 +208,7 @@
 	openpic_set_sources(19, 1, OpenPIC_Addr + 0x110C0);
 
 	openpic_init(NUM_8259_INTERRUPTS);
+        openpic_hookup_cascade(NUM_8259_INTERRUPTS, "82c59 cascade", &i8259_irq);
 
 	/* Map i8259 interrupts */
 	for(i = 0; i < NUM_8259_INTERRUPTS; i++)
arch/ppc/platforms/mcpn765_setup.c +1 -0
diff -Nru a/arch/ppc/platforms/mcpn765_setup.c b/arch/ppc/platforms/mcpn765_setup.c
@@ -247,6 +247,7 @@
 		ppc_md.progress("init_irq: enter", 0);
 
 	openpic_init(NUM_8259_INTERRUPTS);
+        openpic_hookup_cascade(NUM_8259_INTERRUPTS, "82c59 cascade", &i8259_irq);
 
 	for(i=0; i < NUM_8259_INTERRUPTS; i++)
 		irq_desc[i].handler = &i8259_pic;
arch/ppc/platforms/pplus.c +2 -0
diff -Nru a/arch/ppc/platforms/pplus.c b/arch/ppc/platforms/pplus.c
@@ -672,6 +672,8 @@
 
 		openpic_set_sources(0, 16, OpenPIC_Addr + 0x10000);
 		openpic_init(NUM_8259_INTERRUPTS);
+                openpic_hookup_cascade(NUM_8259_INTERRUPTS, "82c59 cascade",
+                                       &i8259_irq);
 		ppc_md.get_irq = openpic_get_irq;
 	}
 
arch/ppc/platforms/prep_setup.c +4 -1
diff -Nru a/arch/ppc/platforms/prep_setup.c b/arch/ppc/platforms/prep_setup.c
@@ -854,8 +854,11 @@
 	int i;
 	unsigned int pci_viddid, pci_did;
 
-	if (OpenPIC_Addr != NULL)
+	if (OpenPIC_Addr != NULL) {
 		openpic_init(NUM_8259_INTERRUPTS);
+                openpic_hookup_cascade(NUM_8259_INTERRUPTS, "82c59 cascade",
+                                       &i8259_irq);
+        }
 	for ( i = 0 ; i < NUM_8259_INTERRUPTS ; i++ )
 		irq_desc[i].handler = &i8259_pic;
 	/* If we have a Raven PCI bridge or a Hawk PCI bridge / Memory
arch/ppc/platforms/sandpoint.c +1 -0
diff -Nru a/arch/ppc/platforms/sandpoint.c b/arch/ppc/platforms/sandpoint.c
@@ -444,6 +444,7 @@
 	openpic_set_sources(0, 16, OpenPIC_Addr + 0x10200);
 
 	openpic_init(NUM_8259_INTERRUPTS);
+        openpic_hookup_cascade(NUM_8259_INTERRUPTS, "82c59 cascade", &i8259_irq);
 
 	/*
 	 * openpic_init() has set up irq_desc[16-31] to be openpic
arch/ppc/syslib/open_pic.c +26 -12
diff -Nru a/arch/ppc/syslib/open_pic.c b/arch/ppc/syslib/open_pic.c
@@ -48,6 +48,8 @@
 static u_int NumSources;
 static int open_pic_irq_offset;
 static volatile OpenPIC_Source *ISR[NR_IRQS];
+static int openpic_cascade_irq = -1;
+static int (*openpic_cascade_fn)(struct pt_regs *);
 
 /* Global Operations */
 static void openpic_disable_8259_pass_through(void);
@@ -417,12 +419,6 @@
 	if (ppc_md.progress) ppc_md.progress("openpic: spurious",0x3bd);
 	openpic_set_spurious(OPENPIC_VEC_SPURIOUS+offset);
 
-	/* Initialize the cascade */
-	if (offset) {
-		if (request_irq(offset, no_action, SA_INTERRUPT,
-				"82c59 cascade", NULL))
-			printk("Unable to get OpenPIC IRQ 0 for cascade\n");
-	}
 	openpic_disable_8259_pass_through();
 #ifdef CONFIG_EPIC_SERIAL_MODE
 	openpic_eicr_set_clk(7);	/* Slowest value until we know better */
@@ -682,6 +678,19 @@
  *
  */
 
+/*
+ * Hookup a cascade to the OpenPIC.
+ */
+void __init
+openpic_hookup_cascade(u_int irq, char *name,
+	int (*cascade_fn)(struct pt_regs *))
+{
+	openpic_cascade_irq = irq;
+	openpic_cascade_fn = cascade_fn;
+	if (request_irq(irq, no_action, SA_INTERRUPT, name, NULL))
+		printk("Unable to get OpenPIC IRQ %d for cascade\n",
+				irq - open_pic_irq_offset);
+}
 
 /*
  *  Enable/disable an external interrupt source
@@ -841,13 +850,18 @@
 	int irq = openpic_irq();
 
 	/*
-	 * This needs to be cleaned up.  We don't necessarily have
-	 * an i8259 cascaded or even a cascade.
+	 * Check for the cascade interrupt and call the cascaded
+	 * interrupt controller function (usually i8259_irq) if so.
+	 * This should move to irq.c eventually.  -- paulus
 	 */
-	if (open_pic_irq_offset && irq == open_pic_irq_offset) {
-		/* Get the IRQ from the cascade. */
-		irq = i8259_irq(regs);
-		openpic_eoi();
+	if (irq == openpic_cascade_irq && openpic_cascade_fn != NULL) {
+		int cirq = openpic_cascade_fn(regs);
+
+		/* Allow for the cascade being shared with other devices */
+		if (cirq != -1) {
+			irq = cirq;
+			openpic_eoi();
+		}
 	} else if (irq == OPENPIC_VEC_SPURIOUS + open_pic_irq_offset)
 		irq = -1;
 	return irq;
include/asm-ppc/open_pic.h +2 -0
diff -Nru a/include/asm-ppc/open_pic.h b/include/asm-ppc/open_pic.h
@@ -40,6 +40,8 @@
 /* Exported functions */
 extern void openpic_set_sources(int first_irq, int num_irqs, void *isr);
 extern void openpic_init(int linux_irq_offset);
+extern void openpic_hookup_cascade(u_int irq, char *name,
+                                   int (*cascade_fn)(struct pt_regs *));
 extern void openpic_init_nmi_irq(u_int irq);
 extern u_int openpic_irq(void);
 extern void openpic_eoi(void);

** Sent via the linuxppc-embedded mail list. See http://lists.linuxppc.org/

author
#1 lore raw

PATCH 1/1 sandpoint X2 in linuxppc_2_5

Quoted 3 lines · Adrian Cox

This patch ports the openpic_hookup_cascade() function from 2_4_devel, as it is necessary for the Sandpoint X2. In doing this I have to change almost every user of openpic_init.

And here is the X2 patch. My earlier problems with serial interrupts were solved by putting those i8259 irqs into edge mode.

- Adrian Cox

===== arch/ppc/Kconfig 1.63 vs edited =====

arch/ppc/Kconfig +14 -1
@@ -622,9 +622,22 @@
 	depends on EV64260
 	default y
 
+config SANDPOINT_X2
+        bool "Run on X2 Sandpoint"
+        depends on SANDPOINT
+        default n
+        ---help---
+          This switches the Sandpoint platform to support the older X2
+          revision. The X2 has only two usable PCI slots and a more
+          restricted interrupt layout.
+
+          If you select this option the kernel will not run on later
+          revisions of the Sandpoint.
+
+
 config EPIC_SERIAL_MODE
 	bool
-	depends on 6xx && (LOPEC || SANDPOINT)
+	depends on 6xx && (LOPEC || SANDPOINT) && ! SANDPOINT_X2
 	default y
 
 config MPC10X_STORE_GATHERING

===== arch/ppc/platforms/sandpoint.c 1.23 vs edited =====

arch/ppc/platforms/sandpoint.c +50 -7

121 lines not on this page · raw message

===== arch/ppc/platforms/sandpoint.h 1.7 vs edited =====

arch/ppc/platforms/sandpoint.h +10 -0
@@ -35,6 +35,16 @@
 #define SANDPOINT_IDE_INT1		15	/* 8259 Test */
 #endif
 
+/* The Sandpoint X3 runs the epic in serial mode, with the i8259 cascade
+ * on IRQ 0. The X2 runs in parallel mode with the cascade on IRQ 1, shared
+ * with a PCI slot. */
+
+#ifdef CONFIG_SANDPOINT_X2
+#define SANDPOINT_CASCADE 17
+#else
+#define SANDPOINT_CASCADE 16
+#endif
+
 /*
  * The sandpoint boards have processor modules that either have an 8240 or
  * an MPC107 host bridge on them.  These bridges have an IDSEL line that allows

** Sent via the linuxppc-embedded mail list. See http://lists.linuxppc.org/

#2 lore raw
In reply to Adrian Cox
Quoted 5 lines · Adrian Cox

This patch ports the openpic_hookup_cascade() function from 2_4_devel, as it is necessary for the Sandpoint X2. In doing this I have to change almost every user of openpic_init.

Quoted 7 lines

pmac needs no changes.
chrp, lopec, mcpn765, pplus, prep, and sandpoint need simple changes.

But some of the platforms don't use the new, single argument, form of openpic_init(). Thus, the following are already broken in 2.5: gemini, mvme5100, powerpmc250, prmpm750, and prpmc800. I don't have any of these platforms, so my patch doesn't touch them.

As an aside, either your mailer or editor (or c&p'ing :)) is giving funky spacing on your changes, please fix.

As for openpic_hookup_cascade, when I introduced this way back when, Paul didn't think it was the best fix for (at the time) 2.5. Paul, what do you think about this change ?

Quoted 148 lines

diff -Nru a/arch/ppc/platforms/chrp_setup.c b/arch/ppc/platforms/chrp_setup.c

arch/ppc/platforms/chrp_setup.c +1 -0
@@ -407,6 +407,7 @@
 	OpenPIC_NumInitSenses = NR_IRQS - NUM_8259_INTERRUPTS;
 
 	openpic_init(NUM_8259_INTERRUPTS);
+        openpic_hookup_cascade(NUM_8259_INTERRUPTS, "82c59 cascade", &i8259_irq);
 
 	for (i = 0; i < NUM_8259_INTERRUPTS; i++)
 		irq_desc[i].handler = &i8259_pic;

diff -Nru a/arch/ppc/platforms/lopec_setup.c b/arch/ppc/platforms/lopec_setup.c

arch/ppc/platforms/lopec_setup.c +1 -0
@@ -208,6 +208,7 @@
 	openpic_set_sources(19, 1, OpenPIC_Addr + 0x110C0);
 
 	openpic_init(NUM_8259_INTERRUPTS);
+        openpic_hookup_cascade(NUM_8259_INTERRUPTS, "82c59 cascade", &i8259_irq);
 
 	/* Map i8259 interrupts */
 	for(i = 0; i < NUM_8259_INTERRUPTS; i++)

diff -Nru a/arch/ppc/platforms/mcpn765_setup.c b/arch/ppc/platforms/mcpn765_setup.c

arch/ppc/platforms/mcpn765_setup.c +1 -0
@@ -247,6 +247,7 @@
 		ppc_md.progress("init_irq: enter", 0);
 
 	openpic_init(NUM_8259_INTERRUPTS);
+        openpic_hookup_cascade(NUM_8259_INTERRUPTS, "82c59 cascade", &i8259_irq);
 
 	for(i=0; i < NUM_8259_INTERRUPTS; i++)
 		irq_desc[i].handler = &i8259_pic;

diff -Nru a/arch/ppc/platforms/pplus.c b/arch/ppc/platforms/pplus.c

arch/ppc/platforms/pplus.c +2 -0
@@ -672,6 +672,8 @@
 
 		openpic_set_sources(0, 16, OpenPIC_Addr + 0x10000);
 		openpic_init(NUM_8259_INTERRUPTS);
+                openpic_hookup_cascade(NUM_8259_INTERRUPTS, "82c59 cascade",
+                                       &i8259_irq);
 		ppc_md.get_irq = openpic_get_irq;
 	}
 

diff -Nru a/arch/ppc/platforms/prep_setup.c b/arch/ppc/platforms/prep_setup.c

arch/ppc/platforms/prep_setup.c +4 -1
@@ -854,8 +854,11 @@
 	int i;
 	unsigned int pci_viddid, pci_did;
 
-	if (OpenPIC_Addr != NULL)
+	if (OpenPIC_Addr != NULL) {
 		openpic_init(NUM_8259_INTERRUPTS);
+                openpic_hookup_cascade(NUM_8259_INTERRUPTS, "82c59 cascade",
+                                       &i8259_irq);
+        }
 	for ( i = 0 ; i < NUM_8259_INTERRUPTS ; i++ )
 		irq_desc[i].handler = &i8259_pic;
 	/* If we have a Raven PCI bridge or a Hawk PCI bridge / Memory

diff -Nru a/arch/ppc/platforms/sandpoint.c b/arch/ppc/platforms/sandpoint.c

arch/ppc/platforms/sandpoint.c +1 -0
@@ -444,6 +444,7 @@
 	openpic_set_sources(0, 16, OpenPIC_Addr + 0x10200);
 
 	openpic_init(NUM_8259_INTERRUPTS);
+        openpic_hookup_cascade(NUM_8259_INTERRUPTS, "82c59 cascade", &i8259_irq);
 
 	/*
 	 * openpic_init() has set up irq_desc[16-31] to be openpic

diff -Nru a/arch/ppc/syslib/open_pic.c b/arch/ppc/syslib/open_pic.c

arch/ppc/syslib/open_pic.c +26 -12
@@ -48,6 +48,8 @@
 static u_int NumSources;
 static int open_pic_irq_offset;
 static volatile OpenPIC_Source *ISR[NR_IRQS];
+static int openpic_cascade_irq = -1;
+static int (*openpic_cascade_fn)(struct pt_regs *);
 
 /* Global Operations */
 static void openpic_disable_8259_pass_through(void);
@@ -417,12 +419,6 @@
 	if (ppc_md.progress) ppc_md.progress("openpic: spurious",0x3bd);
 	openpic_set_spurious(OPENPIC_VEC_SPURIOUS+offset);
 
-	/* Initialize the cascade */
-	if (offset) {
-		if (request_irq(offset, no_action, SA_INTERRUPT,
-				"82c59 cascade", NULL))
-			printk("Unable to get OpenPIC IRQ 0 for cascade\n");
-	}
 	openpic_disable_8259_pass_through();
 #ifdef CONFIG_EPIC_SERIAL_MODE
 	openpic_eicr_set_clk(7);	/* Slowest value until we know better */
@@ -682,6 +678,19 @@
  *
  */
 
+/*
+ * Hookup a cascade to the OpenPIC.
+ */
+void __init
+openpic_hookup_cascade(u_int irq, char *name,
+	int (*cascade_fn)(struct pt_regs *))
+{
+	openpic_cascade_irq = irq;
+	openpic_cascade_fn = cascade_fn;
+	if (request_irq(irq, no_action, SA_INTERRUPT, name, NULL))
+		printk("Unable to get OpenPIC IRQ %d for cascade\n",
+				irq - open_pic_irq_offset);
+}
 
 /*
  *  Enable/disable an external interrupt source
@@ -841,13 +850,18 @@
 	int irq = openpic_irq();
 
 	/*
-	 * This needs to be cleaned up.  We don't necessarily have
-	 * an i8259 cascaded or even a cascade.
+	 * Check for the cascade interrupt and call the cascaded
+	 * interrupt controller function (usually i8259_irq) if so.
+	 * This should move to irq.c eventually.  -- paulus
 	 */
-	if (open_pic_irq_offset && irq == open_pic_irq_offset) {
-		/* Get the IRQ from the cascade. */
-		irq = i8259_irq(regs);
-		openpic_eoi();
+	if (irq == openpic_cascade_irq && openpic_cascade_fn != NULL) {
+		int cirq = openpic_cascade_fn(regs);
+
+		/* Allow for the cascade being shared with other devices */
+		if (cirq != -1) {
+			irq = cirq;
+			openpic_eoi();
+		}
 	} else if (irq == OPENPIC_VEC_SPURIOUS + open_pic_irq_offset)
 		irq = -1;
 	return irq;

diff -Nru a/include/asm-ppc/open_pic.h b/include/asm-ppc/open_pic.h

include/asm-ppc/open_pic.h line 47

Quoted 8 lines
include/asm-ppc/open_pic.h +2 -0
@@ -40,6 +40,8 @@
 /* Exported functions */
 extern void openpic_set_sources(int first_irq, int num_irqs, void *isr);
 extern void openpic_init(int linux_irq_offset);
+extern void openpic_hookup_cascade(u_int irq, char *name,
+                                   int (*cascade_fn)(struct pt_regs *));
 extern void openpic_init_nmi_irq(u_int irq);
 extern u_int openpic_irq(void);
 extern void openpic_eoi(void);

--
Tom Rini
http://gate.crashing.org/~trini/

** Sent via the linuxppc-embedded mail list. See http://lists.linuxppc.org/