<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Feb 6, 2021, 2:09 PM  <<a href="mailto:chrisj@rtems.org">chrisj@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">From: Chris Johns <<a href="mailto:chrisj@rtems.org" target="_blank" rel="noreferrer">chrisj@rtems.org</a>><br>
<br>
- The call to enable the openpic irq for the ISA bridge falls<br>
  because the IRQ used is offset by the ISA bus signals and<br>
  the openpic call expects an IRA relative to it's signals.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Falls to fails</div><div dir="auto">What's IRA</div><div dir="auto">And its not it's </div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
- Add the MVME 2600/2700 to the list is an ISA bridge.<br>
<br>
Closes #4231<br>
---<br>
 bsps/powerpc/shared/irq/irq_init.c          | 2 +-<br>
 bsps/powerpc/shared/irq/openpic_i8259_irq.c | 2 +-<br>
 2 files changed, 2 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/bsps/powerpc/shared/irq/irq_init.c b/bsps/powerpc/shared/irq/irq_init.c<br>
index 1a44992a5b..1042c9d1a8 100644<br>
--- a/bsps/powerpc/shared/irq/irq_init.c<br>
+++ b/bsps/powerpc/shared/irq/irq_init.c<br>
@@ -310,7 +310,7 @@ void BSP_rtems_irq_mng_init(unsigned cpuId)<br>
 #endif<br>
     known_cpi_isa_bridge = 1;<br>
   }<br>
-  if ( currentBoard == MVME_2300 ) {<br>
+  if ( currentBoard == MVME_2300 || currentBoard == MVME_2600_2700_W_MVME761 ) {<br>
     /* nothing to do for W83C553 bridge */<br>
     known_cpi_isa_bridge = 1;<br>
   }<br>
diff --git a/bsps/powerpc/shared/irq/openpic_i8259_irq.c b/bsps/powerpc/shared/irq/openpic_i8259_irq.c<br>
index 4a9c393f7f..513b9ac3e0 100644<br>
--- a/bsps/powerpc/shared/irq/openpic_i8259_irq.c<br>
+++ b/bsps/powerpc/shared/irq/openpic_i8259_irq.c<br>
@@ -214,7 +214,7 @@ int BSP_setup_the_pic(rtems_irq_global_settings* config)<br>
        /*<br>
      * Must enable PCI/ISA bridge IRQ<br>
      */<br>
-       openpic_enable_irq (BSP_PCI_ISA_BRIDGE_IRQ);<br>
+    openpic_enable_irq (BSP_PCI_ISA_BRIDGE_IRQ - BSP_PCI_IRQ_LOWEST_OFFSET);<br>
 #endif<br>
 #endif<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I think the patch itself is ok to commit. It would be good to know that it doesn't negatively impact other BSPs with openpic and i8259. Which other BSPs use this code?</div><div dir="auto"><br></div><div dir="auto">And does this mean that no bsp with openpic and i8259 had working ISA interrupts?</div><div dir="auto"><br></div><div dir="auto">--joel</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
-- <br>
2.27.0<br>
<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div></div>