Fernando,
I think you are missing the CHNLSM_InterruptDSP() function changes from Felipe patch; ie. To wait less. Thank you, Best regards, Hari > -----Original Message----- > From: [email protected] [mailto:linux-omap- > [email protected]] On Behalf Of Guzman Lugo, Fernando > Sent: Wednesday, February 25, 2009 5:58 PM > To: [email protected] > Subject: [PATCH][OMAPZOOM] DSPBRIDGE: Remove unnecessary checks in > HW_MBOX_IsFull function > > > Hi, > > This patch removes unncessary checks in HW_MBOX_IsFull this patch is > base on the previous patch that Felipe Contreras sent. > > Regards, > Fernando. > > From: Fernando Guzman Lugo <[email protected]> > Date: Wed, 25 Feb 2009 17:48:01 -0600 > Subject: [PATCH] DSPBRIDGE: Remove unnecessary checks in HW_MBOX_IsFull > function > > This patch removes some unnecessary checks in HW_MBOX_IsFull > function > Signed-off-by: Guzman Lugo Fernando <[email protected]> > --- > drivers/dsp/bridge/hw/hw_mbox.c | 22 ++-------------------- > drivers/dsp/bridge/hw/hw_mbox.h | 6 ++---- > drivers/dsp/bridge/wmd/tiomap_sm.c | 6 +++--- > 3 files changed, 7 insertions(+), 27 deletions(-) > > diff --git a/drivers/dsp/bridge/hw/hw_mbox.c > b/drivers/dsp/bridge/hw/hw_mbox.c > index bc61d64..0f9e5a7 > --- a/drivers/dsp/bridge/hw/hw_mbox.c > +++ b/drivers/dsp/bridge/hw/hw_mbox.c > @@ -105,28 +105,10 @@ HW_STATUS HW_MBOX_MsgWrite(const u32 baseAddress, > const HW_MBOX_Id_t mailBoxId, > } > > /* Reads the full status register for mailbox. */ > -HW_STATUS HW_MBOX_IsFull(const u32 baseAddress, const HW_MBOX_Id_t > mailBoxId, > - u32 *const pIsFull) > +bool HW_MBOX_IsFull(const u32 baseAddress, const HW_MBOX_Id_t mailBoxId) > { > - HW_STATUS status = RET_OK; > - u32 fullStatus; > - > - /* Check input parameters */ > - CHECK_INPUT_PARAM(baseAddress, 0, RET_BAD_NULL_PARAM, RES_MBOX_BASE > + > - RES_INVALID_INPUT_PARAM); > - CHECK_INPUT_PARAM(pIsFull, NULL, RET_BAD_NULL_PARAM, RES_MBOX_BASE > + > - RES_INVALID_INPUT_PARAM); > - CHECK_INPUT_RANGE_MIN0(mailBoxId, HW_MBOX_ID_MAX, RET_INVALID_ID, > - RES_MBOX_BASE + RES_INVALID_INPUT_PARAM); > - > - /* read the is full status parameter for Mailbox */ > - fullStatus = > MLBMAILBOX_FIFOSTATUS___0_15FifoFullMBmRead32(baseAddress, > + return MLBMAILBOX_FIFOSTATUS___0_15FifoFullMBmRead32(baseAddress, > (u32)mailBoxId); > - > - /* fill in return parameter */ > - *pIsFull = (fullStatus & 0xFF); > - > - return status; > } > > /* Gets number of messages in a specified mailbox. */ > diff --git a/drivers/dsp/bridge/hw/hw_mbox.h > b/drivers/dsp/bridge/hw/hw_mbox.h > index 225fb40..e70305d > --- a/drivers/dsp/bridge/hw/hw_mbox.h > +++ b/drivers/dsp/bridge/hw/hw_mbox.h > @@ -158,11 +158,9 @@ extern HW_STATUS HW_MBOX_MsgWrite( > * > * PURPOSE: : this function reads the full status register for > mailbox. > */ > -extern HW_STATUS HW_MBOX_IsFull( > +extern bool HW_MBOX_IsFull( > const u32 baseAddress, > - const HW_MBOX_Id_t mailBoxId, > - u32 *const pIsFull > - ); > + const HW_MBOX_Id_t mailBoxId); > > /* > * FUNCTION : HW_MBOX_NumMsgGet > diff --git a/drivers/dsp/bridge/wmd/tiomap_sm.c > b/drivers/dsp/bridge/wmd/tiomap_sm.c > index edc3bcf..7acb4f8 > --- a/drivers/dsp/bridge/wmd/tiomap_sm.c > +++ b/drivers/dsp/bridge/wmd/tiomap_sm.c > @@ -178,7 +178,7 @@ DSP_STATUS CHNLSM_InterruptDSP(struct WMD_DEV_CONTEXT > *hDevContext) > #endif > #endif > HW_STATUS hwStatus; > - u32 mbxFull; > + bool mbxFull; > struct CFG_HOSTRES resources; > u16 cnt = 10; > u32 temp; > @@ -242,8 +242,8 @@ DSP_STATUS CHNLSM_InterruptDSP(struct WMD_DEV_CONTEXT > *hDevContext) > pDevContext->dwBrdState = BRD_RUNNING; > } > while (--cnt) { > - hwStatus = HW_MBOX_IsFull(resources.dwMboxBase, > - MBOX_ARM2DSP, &mbxFull); > + mbxFull = HW_MBOX_IsFull(resources.dwMboxBase, > + MBOX_ARM2DSP); > if (mbxFull) > UTIL_Wait(1000); /* wait for 1 ms) */ > else > -- > 1.5.6.4 > -- > 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 -- 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
