Benoit, On Thu, Nov 18, 2010 at 5:28 PM, Cousson, Benoit <b-cousson@xxxxxx> wrote: > On 11/18/2010 8:15 PM, Hari Kanigeri wrote: >> >> disabling rx interrupt on omap4 is different than its pre-decessors. >> The bit in OMAP4_MAILBOX_IRQENABLE_CLR should be set to disable the >> interrupts instead of clearing the bit. >> >> Signed-off-by: Hari Kanigeri<h-kanigeri2@xxxxxx> >> --- >> arch/arm/mach-omap2/mailbox.c | 5 ++++- >> 1 files changed, 4 insertions(+), 1 deletions(-) >> >> diff --git a/arch/arm/mach-omap2/mailbox.c b/arch/arm/mach-omap2/mailbox.c >> index 42dbfa4..82b5ced 100644 >> --- a/arch/arm/mach-omap2/mailbox.c >> +++ b/arch/arm/mach-omap2/mailbox.c >> @@ -195,7 +195,10 @@ static void omap2_mbox_disable_irq(struct omap_mbox >> *mbox, >> struct omap_mbox2_priv *p = (struct omap_mbox2_priv *)mbox->priv; >> u32 l, bit = (irq == IRQ_TX) ? p->notfull_bit : p->newmsg_bit; >> l = mbox_read_reg(p->irqdisable); >> - l&= ~bit; >> + if (cpu_is_omap44xx()) > > Since it is not omap version specific but IP version specific, you should > not use cpu_is_ to do that. Moreover cpu_is calls should be used during init > only. > You can use the rev field in hwmod_class in order to detect the IP version. > Smartreflex series for 3630 is already using that kind of mechanism. > You will have to copy that revision information into pdata struct and then > use that here. I see your point, but since mailbox hwmod patches from Omar are still under review I didn't find any other option than to enable this This is critical functionality that I want to include in and not wait till the hwmod patches are accepted. Please let me know if there is any other way of approaching this problem ? Thank you, Best regards, Hari -- 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