From: Tony Lindgren <tony@atomide.com>
To: "Luís Cargnini" <lvcargnini@gmail.com>
Cc: OMAP <Linux-omap-open-source@linux.omap.com>
Subject: Re: [PATCH 01/01] arch/arm/plat-omap/mcbsp.c
Date: Sun, 21 Oct 2007 17:00:50 -0700 [thread overview]
Message-ID: <20071022000047.GC6984@atomide.com> (raw)
In-Reply-To: <ae36f8040709252025o3325b001k75d133411812f923@mail.gmail.com>
* Luís Cargnini <lvcargnini@gmail.com> [070925 20:26]:
> this patch modify the McBSP interrupt handling behavior turning the irq of
> McBSP ports shared and it is working, I'm deveolping a device driver and
> percepted that this is a change necessary for new device drivers could been
> implemented to tak advantage of McBSP and could use it, example is ADC
> communications throw McBSP interface for example, this modification do not
> harm the kernel and add more felxibility and functionality to the kernel.
>
> -- linux-omap-2.6/arch/arm/plat-omap/mcbsp.c 2007-03-12
> 16:44:03.501890844-0300
> +++ linux-omap-2.6-instramed/arch/arm/plat-omap/mcbsp.c 2007-09-25 23:58:
> 39.324154145 -0300
>
> static irqreturn_t omap_mcbsp_tx_irq_handler(int irq, void *dev_id)
> @@ -304,7 +320,7 @@ int omap_mcbsp_request(unsigned int id)
>
> if (mcbsp[id].io_type == OMAP_MCBSP_IRQ_IO) {
> /* We need to get IRQs here */
> - err = request_irq(mcbsp[id].tx_irq,
> omap_mcbsp_tx_irq_handler, 0,
> + err = request_irq(mcbsp[id].tx_irq,
> omap_mcbsp_tx_irq_handler, IRQF_SHARED,
> "McBSP",
> (void *) (&mcbsp[id]));
> if (err != 0) {
> @@ -315,8 +331,7 @@ int omap_mcbsp_request(unsigned int id)
>
> init_completion(&(mcbsp[id].tx_irq_completion));
>
> -
> - err = request_irq(mcbsp[id].rx_irq,
> omap_mcbsp_rx_irq_handler, 0,
> + err = request_irq(mcbsp[id].rx_irq,
> omap_mcbsp_rx_irq_handler, IRQF_SHARED,
> "McBSP",
> (void *) (&mcbsp[id]));
> if (err != 0) {
Let's consider this once there's something else actually something using
the interrupt.
Tony
next prev parent reply other threads:[~2007-10-22 0:00 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-09-26 3:25 [PATCH 01/01] arch/arm/plat-omap/mcbsp.c Luís Cargnini
2007-10-22 0:00 ` Tony Lindgren [this message]
[not found] ` <ae36f8040710212048v42f3fb96pc8b0e2db9496f93a@mail.gmail.com>
[not found] ` <20071022035841.GH6984@atomide.com>
[not found] ` <ae36f8040710220957m721ab992g898da6f2d12eac8@mail.gmail.com>
[not found] ` <20071022172101.GD15946@atomide.com>
[not found] ` <ae36f8040710221108h20b9ba91ked981b475a6f1199@mail.gmail.com>
[not found] ` <ae36f8040710221319o4ae1e14cn8a5040be4555235c@mail.gmail.com>
[not found] ` <20071022205648.GA17300@atomide.com>
2007-10-23 3:17 ` Luís Cargnini
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=20071022000047.GC6984@atomide.com \
--to=tony@atomide.com \
--cc=Linux-omap-open-source@linux.omap.com \
--cc=lvcargnini@gmail.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox