RE: [PATCH 1/2] ARM: McBSP: Fix request for irq in OMAP4

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----Original Message-----
> From: linux-omap-owner@xxxxxxxxxxxxxxx [mailto:linux-omap-owner@xxxxxxxxxxxxxxx] On Behalf Of Jarkko
> Nikula
> Sent: Wednesday, May 05, 2010 4:00 PM
> To: Candelaria Villarreal, Jorge
> Cc: linux-omap@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH 1/2] ARM: McBSP: Fix request for irq in OMAP4
> 
> On Tue,  4 May 2010 18:37:49 -0500
> Jorge Eduardo Candelaria <jorge.candelaria@xxxxxx> wrote:
> 
> > In OMAP4, there is only one irq line for TX and RX paths. Use
> > the correct irq line to avoid errors at runtime.
> >
> > Also, request irq line only once (instead of requesting for TX
> > and RX).
> >
> 
> > Signed-off-by: Jorge Eduardo Candelaria <jorge.candelaria@xxxxxx>
> > ---
> >  arch/arm/mach-omap2/mcbsp.c |   12 ++++--------
> >  arch/arm/plat-omap/mcbsp.c  |   20 ++++++++++++--------
> >  2 files changed, 16 insertions(+), 16 deletions(-)
> >
> > diff --git a/arch/arm/mach-omap2/mcbsp.c b/arch/arm/mach-omap2/mcbsp.c
> > index 2f3cad6..6b550ec 100644
> > --- a/arch/arm/mach-omap2/mcbsp.c
> > +++ b/arch/arm/mach-omap2/mcbsp.c
> > @@ -187,32 +187,28 @@ static struct omap_mcbsp_platform_data omap44xx_mcbsp_pdata[] = {
> >  		.phys_base      = OMAP44XX_MCBSP1_BASE,
> >  		.dma_rx_sync    = OMAP44XX_DMA_MCBSP1_RX,
> >  		.dma_tx_sync    = OMAP44XX_DMA_MCBSP1_TX,
> > -		.rx_irq         = INT_24XX_MCBSP1_IRQ_RX,
> > -		.tx_irq         = INT_24XX_MCBSP1_IRQ_TX,
> > +		.tx_irq         = INT_44XX_MCBSP1_IRQ,
> >  		.ops            = &omap2_mcbsp_ops,
> >  	},
> 
> These INT_44XX_MCBSPx appears to be undefined. Commit
> 5772ca7d7897722e7ed4c476bafee0b07bd9193e is removing them and I don't
> find are there somewhere a patch adding them to irqs-44xx.h.

Name isn't right here. Should be "OMAP44XX_IRQ_MCBSP1" defined
in " arch/arm/plat-omap/include/plat/irqs-44xx.h"
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux