All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tiago Dall'Agnol <tdallagnol@parks.com.br>
To: Dan Malek <dan@embeddededge.com>
Cc: linuxppc-embedded@ozlabs.org
Subject: Re: request_8xxirq
Date: Mon, 01 Aug 2005 09:55:58 -0300	[thread overview]
Message-ID: <42EE1BDE.6050009@parks.com.br> (raw)
In-Reply-To: <b50c9dbfb13d8e2f5a82c2a299e46240@embeddededge.com>

Hi Dan

Thanks for your attention. I believe that I'm doing some confusion about 
what is "request_8xxirq". I have this function (or some alias?) in my 
system, and I guess that there isn't any problem with it. But the 
problem I had was a pretty strange, and I decided to take a look at that 
function. And I hadn't find its implementation. I tried to grep 
request_8xxirq in the whole code, but I just found a 
EXPORT_SYMBOL(request_8xxirq).

Anyway, my problem was related to the SEC Lite module. In my Linux, what 
I had in source files originally was the use of SEC Lite interruption as 
a LEVEL4 (number 9). In some boards (I mean, the same models, just other 
boards), the interrupt handler was not called when it should be, i.e., 
when the SEC Lite module ends its processing and signals that the packed 
is ready for me. That's the way my system works, with interruption to 
signal that the packet was already processed.

So, after identifying that problem, I tried to change interrupt to 
LEVEL3 (number 7), and for my surprise... now it works, for all boards. 
But why?

Isn't it strange?

Best Regards,
Tiago

Dan Malek wrote:

>
> On Jul 29, 2005, at 1:01 PM, Tiago Dall'Agnol wrote:
>
>> Am I wrong? I didn't understand when that function request_irq is used.
>
>
> At some point I don't remember and against all of my wishes, the code
> was changed from using request_8xxirq to using request_irq using
> an ugly hack to add some offset to the 8xx interrupt vector number.
> I didn't see any bug that needed fixing, but people with the power to
> ignore my maintainer status decided to change it anyway.  So, now you
> have to modify your drivers to call request_irq() with some nonsensical
> interrupt number.  No value added, just confusion.
>
> Thanks.
>
>     -- Dan
>

  reply	other threads:[~2005-08-01 12:54 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-07-29  1:33 request_8xxirq Manish Joshi
2005-07-29 13:07 ` request_8xxirq Tiago Dall'Agnol
2005-07-29 13:40   ` request_8xxirq Alex Zeffertt
2005-07-29 13:52     ` request_8xxirq Tiago Dall'Agnol
2005-07-29 14:43       ` request_8xxirq Alex Zeffertt
2005-07-29 17:01         ` request_8xxirq Tiago Dall'Agnol
2005-07-29 17:43           ` request_8xxirq Dan Malek
2005-08-01 12:55             ` Tiago Dall'Agnol [this message]
2005-08-01 13:04           ` request_8xxirq Alex Zeffertt
2005-08-01 13:25             ` request_8xxirq Tiago Dall'Agnol

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=42EE1BDE.6050009@parks.com.br \
    --to=tdallagnol@parks.com.br \
    --cc=dan@embeddededge.com \
    --cc=linuxppc-embedded@ozlabs.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.