* Hiroshi DOYU <[email protected]> [090922 08:12]:
> Hi Tony,
> 
> From: "ext C.A, Subramaniam" <[email protected]>
> Subject: [PATCH 8/10] omap mailbox: OMAP4-Mailbox - Adds code changes to 
> support OMAP4 mailbox.
> Date: Tue, 22 Sep 2009 16:33:32 +0200
> 
> > From 6e3100d338c37928724821cab460ced1437bfa59 Mon Sep 17 00:00:00 2001
> > From: C A Subramaniam <[email protected]>
> > Date: Tue, 8 Sep 2009 22:36:20 +0530
> > Subject: [PATCH 8/10] omap mailbox: OMAP4-Mailbox - Adds code changes to 
> > support OMAP4 mailbox.
> > 
> > This patch adds code changes in the mailbox driver module to
> > add support for OMAP4 mailbox.
> > 
> > Signed-off-by: Hari Kanigeri <[email protected]>
> > Signed-off-by: C A Subramaniam <[email protected]>
> > Signed-off-by: Ramesh Gupta G <[email protected]>
> > ---
> >  arch/arm/mach-omap2/mailbox.c |  120 
> > +++++++++++++++++++++++++++++++++++------
> >  arch/arm/plat-omap/mailbox.c  |   25 +++++++--
> >  2 files changed, 123 insertions(+), 22 deletions(-)
> > 
> > diff --git a/arch/arm/mach-omap2/mailbox.c b/arch/arm/mach-omap2/mailbox.c
> > index 6f71f37..3d85807 100644
> > --- a/arch/arm/mach-omap2/mailbox.c
> > +++ b/arch/arm/mach-omap2/mailbox.c
> 
> [...]
> 
> > @@ -70,10 +88,9 @@ static inline void mbox_write_reg(u32 val, size_t ofs)
> >  static int omap2_mbox_startup(struct omap_mbox *mbox)
> >  {
> >     unsigned int l;
> > -
> >     mbox_ick_handle = clk_get(NULL, "mailboxes_ick");
> >     if (IS_ERR(mbox_ick_handle)) {
> > -           printk("Could not get mailboxes_ick\n");
> > +           printk(KERN_ERR "Could not get mailboxes_ick\n");
> >             return -ENODEV;
> >     }
> >     clk_enable(mbox_ick_handle);
> > @@ -85,7 +102,6 @@ static int omap2_mbox_startup(struct omap_mbox *mbox)
> >     l = mbox_read_reg(MAILBOX_SYSCONFIG);
> >     l |= 0x00000011;
> >     mbox_write_reg(l, MAILBOX_SYSCONFIG);
> > -
> >     omap2_mbox_enable_irq(mbox, IRQ_RX);
> >  
> >     return 0;
> 
> The above may conflict with the following patch, but it won't be
> diffcult to solve the confliction.
> 
> http://patchwork.kernel.org/patch/44224/
> 
> Tony, can you get the above "softreset" patch in?

Added to omap-fixes.
 
> [...]
> 
> > @@ -156,6 +172,8 @@ static void omap2_mbox_ack_irq(struct omap_mbox *mbox,
> >     u32 bit = (irq == IRQ_TX) ? p->notfull_bit : p->newmsg_bit;
> >  
> >     mbox_write_reg(bit, p->irqstatus);
> > +   /* Flush post writing */
> > +    mbox_read_reg(p->irqstatus);
> >  }
> 
> This is the same as the following patch:
> 
> http://patchwork.kernel.org/patch/43740/
> 
> Tony, can you get the above "flush posted write" patch in?

This too.

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to