Hi Deepak, On Thu, 2010-02-11 at 18:33 +0100, ext Deepak Chitriki wrote: > Removed NTFY_Notify() in WMD_MSG_Get() to avoid locking contention > as NTFY_Notify() is already invoked in InputMsg(). > > Cc: Ameya Palande <ameya.palande@xxxxxxxxx> > Cc: Omar Ramirez Luna <omar.ramirez@xxxxxx> > Cc: Nishanth Menon <nm@xxxxxx> > > Signed-off-by: Deepak Chitriki <deepak.chitriki@xxxxxx> > > --- > Ref: > v0:http://marc.info/?l=linux-omap&m=126584779011493&w=2 > v1:http://marc.info/?l=linux-omap&m=126586224627725&w=2 > v2:Comment fixes from v1 > > drivers/dsp/bridge/wmd/msg_sm.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/dsp/bridge/wmd/msg_sm.c b/drivers/dsp/bridge/wmd/msg_sm.c > index d778cbc..7414113 100644 > --- a/drivers/dsp/bridge/wmd/msg_sm.c > +++ b/drivers/dsp/bridge/wmd/msg_sm.c > @@ -295,7 +295,7 @@ DSP_STATUS WMD_MSG_Get(struct MSG_QUEUE *hMsgQueue, > (struct list_head *)pMsgFrame); > if (LST_IsEmpty(hMsgQueue->msgUsedList)) > SYNC_ResetEvent(hMsgQueue->hSyncEvent); > - else > + else > SYNC_SetEvent(hMsgQueue->hSyncEvent); > > fGotMsg = true; > @@ -344,7 +344,7 @@ DSP_STATUS WMD_MSG_Get(struct MSG_QUEUE *hMsgQueue, > } > hMsgQueue->refCount--; > /* Reset the event if there are still queued messages */ > - if (!LST_IsEmpty(hMsgQueue->msgUsedList)) > + if (!LST_IsEmpty(hMsgQueue->msgUsedList)) > SYNC_SetEvent(hMsgQueue->hSyncEvent); > > /* Exit critical section */ Is this the correct change? Cheers, Ameya. -- 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
