<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <div class="moz-cite-prefix">On 8/1/2013 11:30 AM, Vipul Nayyar
      wrote:<br>
    </div>
    <blockquote
      cite="mid:1375374624.59740.YahooMailNeo@web194902.mail.sg3.yahoo.com"
      type="cite">
      <div style="color:#000; background-color:#fff; font-family:arial,
        helvetica, sans-serif;font-size:10pt">
        <div><span>Ok, I understand that including irq-extension.h in
            libchip drivers is not required.</span></div>
        <div style="color: rgb(0, 0, 0); font-size: 13px; font-family:
          arial, helvetica, sans-serif; background-color: transparent;
          font-style: normal;"><span><br>
          </span></div>
        <div style="color: rgb(0, 0, 0); font-size: 13px; font-family:
          arial, helvetica, sans-serif; background-color: transparent;
          font-style: normal;"><span>So, you agree that using ifdefs is
            the way to go. I should use IRQ API methods & </span><span
            style="font-size: 10pt;">rtems_interrupt_catch() for simple
            vectored architectures like it's done in ata.c.</span><span
            style="background-color: transparent;"> Right ?</span></div>
        <div style="color: rgb(0, 0, 0); font-size: 13px; font-family:
          arial, helvetica, clean, sans-serif; background-color:
          transparent; font-style: normal;"><br>
        </div>
      </div>
    </blockquote>
    Without looking at the code, Yes. :)<br>
    <br>
    Simple vectored architectures should use rtems_interrupt_catch().<br>
    <br>
    <br>
    <blockquote
      cite="mid:1375374624.59740.YahooMailNeo@web194902.mail.sg3.yahoo.com"
      type="cite">
      <div style="color:#000; background-color:#fff; font-family:arial,
        helvetica, sans-serif;font-size:10pt">
        <div style="color: rgb(0, 0, 0); font-size: 13px; font-family:
          arial, helvetica, clean, sans-serif; background-color:
          transparent; font-style: normal;">Regards</div>
        <div style="color: rgb(0, 0, 0); font-size: 13px; font-family:
          arial, helvetica, clean, sans-serif; background-color:
          transparent; font-style: normal;"><span
            style="background-color:transparent;line-height:1.22;">Vipul
            Nayyar </span></div>
        <div style="color: rgb(0, 0, 0); font-size:
          12.727272033691406px; font-family: arial, helvetica, clean,
          sans-serif; background-color: transparent; font-style:
          normal;"><br>
        </div>
        <div><br>
        </div>
        <div style="font-family: arial, helvetica, sans-serif;
          font-size: 10pt;">
          <div style="font-family: 'times new roman', 'new york', times,
            serif; font-size: 12pt;">
            <div dir="ltr">
              <hr size="1"> <font size="2" face="Arial"> <b><span
                    style="font-weight:bold;">From:</span></b> Sebastian
                Huber <a class="moz-txt-link-rfc2396E" href="mailto:sebastian.huber@embedded-brains.de"><sebastian.huber@embedded-brains.de></a><br>
                <b><span style="font-weight: bold;">To:</span></b> Vipul
                Nayyar <a class="moz-txt-link-rfc2396E" href="mailto:nayyar_vipul@yahoo.com"><nayyar_vipul@yahoo.com></a> <br>
                <b><span style="font-weight: bold;">Cc:</span></b> Joel
                Sherrill <a class="moz-txt-link-rfc2396E" href="mailto:joel.sherrill@oarcorp.com"><joel.sherrill@oarcorp.com></a>;
                <a class="moz-txt-link-rfc2396E" href="mailto:rtems-devel@rtems.org">"rtems-devel@rtems.org"</a> <a class="moz-txt-link-rfc2396E" href="mailto:rtems-devel@rtems.org"><rtems-devel@rtems.org></a> <br>
                <b><span style="font-weight: bold;">Sent:</span></b>
                Thursday, 1 August 2013 9:24 PM<br>
                <b><span style="font-weight: bold;">Subject:</span></b>
                Re: [PATCH] Updated legacy code in ata & dec21140<br>
              </font> </div>
            <div class="y_msg_container"><br>
              On 2013-08-01 17:41, Vipul Nayyar wrote:<br>
              ><br>
              > I added "#include <rtems/irq-extension.h>" in
              ata.c  so that the generic<br>
              > methods could work. Shouldn't including the
              extensions header solve the problem ?<br>
              <br>
              The BSPs providing the IRQ extensions API include this
              file via <bsp/irq.h> <br>
              indirectly and they should define
              BSP_FEATURE_IRQ_EXTENSION in <bsp.h>.<br>
              <br>
              -- <br>
              Sebastian Huber, embedded brains GmbH<br>
              <br>
              Address : Dornierstr. 4, D-82178 Puchheim, Germany<br>
              Phone  : +49 89 189 47 41-16<br>
              Fax    : +49 89 189 47 41-09<br>
              E-Mail  : <a moz-do-not-send="true"
                ymailto="mailto:sebastian.huber@embedded-brains.de"
                href="mailto:sebastian.huber@embedded-brains.de">sebastian.huber@embedded-brains.de</a><br>
              PGP    : Public key available on request.<br>
              <br>
              Diese Nachricht ist keine geschäftliche Mitteilung im
              Sinne des EHUG.<br>
              <br>
              <br>
            </div>
          </div>
        </div>
      </div>
    </blockquote>
    <br>
    <br>
    <pre class="moz-signature" cols="72">-- 
Joel Sherrill, Ph.D.              Director of Research & Development
<a class="moz-txt-link-abbreviated" href="mailto:joel.sherrill@OARcorp.com">joel.sherrill@OARcorp.com</a>         On-Line Applications Research 
Ask me about RTEMS: a free RTOS   Huntsville AL 35805
Support Available                 (256) 722-9985 </pre>
  </body>
</html>