All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eduardo Valentin <eduardo.valentin@nokia.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: Linux-OMAP <linux-omap@vger.kernel.org>,
	ALSA-Devel <alsa-devel@vger.kernel.org>,
	Jarkko Nikula <jhnikula@gmail.com>,
	"Nurkkala Eero.An (EXT-Offcode/Oulu)"
	<ext-Eero.Nurkkala@nokia.com>,
	"Ujfalusi Peter (Nokia-D/Tampere)" <peter.ujfalusi@nokia.com>,
	Mark Brown <broonie@opensource.wolfsonmicro.com>,
	Eero Nurkkala <ext-eero.nurkkala@nokia.com>
Subject: [PATCHv5 13/20] OMAP: McBSP: Do not enable wakeups for no-idle mode
Date: Thu, 20 Aug 2009 16:18:19 +0300	[thread overview]
Message-ID: <1250774306-7581-13-git-send-email-eduardo.valentin@nokia.com> (raw)
In-Reply-To: <1250774306-7581-1-git-send-email-eduardo.valentin@nokia.com>

From: Eero Nurkkala <ext-eero.nurkkala@nokia.com>

When no-idle mode is taken, wakeups need not to be enabled.
Moreover, CLOCKACTIVITY bits are unnecessary with this mode
also.

Signed-off-by: Eero Nurkkala <ext-eero.nurkkala@nokia.com>
Acked-by: Eduardo Valentin <eduardo.valentin@nokia.com>
---
 arch/arm/plat-omap/mcbsp.c |   13 +++++++------
 1 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/arch/arm/plat-omap/mcbsp.c b/arch/arm/plat-omap/mcbsp.c
index 85176a5..56a5688 100644
--- a/arch/arm/plat-omap/mcbsp.c
+++ b/arch/arm/plat-omap/mcbsp.c
@@ -319,16 +319,17 @@ static inline void omap34xx_mcbsp_request(struct omap_mcbsp *mcbsp)
 		syscon &= ~(ENAWAKEUP | SIDLEMODE(0x03) | CLOCKACTIVITY(0x03));
 
 		spin_lock_irq(&mcbsp->lock);
-		if (mcbsp->dma_op_mode == MCBSP_DMA_MODE_THRESHOLD)
-			syscon |= SIDLEMODE(0x02);
-		else
+		if (mcbsp->dma_op_mode == MCBSP_DMA_MODE_THRESHOLD) {
+			syscon |= (ENAWAKEUP | SIDLEMODE(0x02) |
+					CLOCKACTIVITY(0x02));
+			OMAP_MCBSP_WRITE(mcbsp->io_base, WAKEUPEN,
+					XRDYEN | RRDYEN);
+		} else {
 			syscon |= SIDLEMODE(0x01);
+		}
 		spin_unlock_irq(&mcbsp->lock);
 
-		syscon |= (ENAWAKEUP | CLOCKACTIVITY(0x02));
 		OMAP_MCBSP_WRITE(mcbsp->io_base, SYSCON, syscon);
-
-		OMAP_MCBSP_WRITE(mcbsp->io_base, WAKEUPEN, XRDYEN | RRDYEN);
 	}
 }
 
-- 
1.6.2.GIT


  parent reply	other threads:[~2009-08-20 13:33 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-20 13:18 [PATCHv5 01/20] OMAP: McBSP: Provide functions for ASoC frame syncronization Eduardo Valentin
2009-08-20 13:18 ` [PATCHv5 02/20] OMAP: McBSP: Add IRQEN, IRQSTATUS, THRESHOLD2 and THRESHOLD1 registers Eduardo Valentin
2009-08-20 13:18 ` Eduardo Valentin
2009-08-20 13:18 ` [PATCHv5 03/20] OMAP: McBSP: Use appropriate value for startup delay Eduardo Valentin
2009-08-20 13:18 ` Eduardo Valentin
2009-08-20 13:18 ` [PATCHv5 04/20] OMAP: McBSP: Add transmit/receive threshold handler Eduardo Valentin
2009-08-20 13:18 ` Eduardo Valentin
2009-08-20 13:18 ` [PATCHv5 05/20] OMAP: McBSP: Create and export max_(r|t)x_thres property Eduardo Valentin
2009-08-20 13:18 ` Eduardo Valentin
2009-08-20 13:18 ` [PATCHv5 06/20] OMAP3: McBSP: Lower the maximum buffersize for McBSP1,3,4,5 Eduardo Valentin
2009-08-20 13:18 ` [PATCHv5 06/20] OMAP3: McBSP: Lower the maximum buffersize for McBSP1, 3, 4, 5 Eduardo Valentin
2009-08-20 13:18 ` [PATCHv5 07/20] OMAP: McBSP: Rename thres sysfs symbols Eduardo Valentin
2009-08-20 13:18 ` Eduardo Valentin
2009-08-20 13:18 ` [PATCHv5 08/20] OMAP: McBSP: Add link DMA mode selection Eduardo Valentin
2009-08-20 13:18 ` Eduardo Valentin
2009-08-20 13:18 ` [PATCHv5 09/20] OMAP: McBSP: Wakeups utilized Eduardo Valentin
2009-08-20 13:18 ` Eduardo Valentin
2009-08-20 13:18 ` [PATCHv5 10/20] OMAP: McBSP: Change wakeup signals Eduardo Valentin
2009-08-20 13:18 ` Eduardo Valentin
2009-08-20 13:18 ` [PATCHv5 11/20] OMAP: McBSP: Retain McBSP FCLK clockactivity Eduardo Valentin
2009-08-20 13:18 ` Eduardo Valentin
2009-08-20 13:18 ` [PATCHv5 12/20] OMAP: McBSP: Configure NO IDLE mode for DMA mode different of threshold Eduardo Valentin
2009-08-20 13:18 ` Eduardo Valentin
2009-08-20 13:18 ` [PATCHv5 13/20] OMAP: McBSP: Do not enable wakeups for no-idle mode Eduardo Valentin
2009-08-20 13:18 ` Eduardo Valentin [this message]
2009-08-20 13:18 ` [PATCHv5 14/20] OMAP: McBSP: Let element DMA mode hit retention also Eduardo Valentin
2009-08-20 13:18 ` Eduardo Valentin
2009-08-20 13:18 ` [PATCHv5 15/20] ASoC: OMAP: Enable DMA burst mode Eduardo Valentin
2009-08-20 13:18 ` Eduardo Valentin
2009-08-20 13:18 ` [PATCHv5 16/20] ASoC: OMAP: Make DMA 64 aligned Eduardo Valentin
2009-08-20 13:18 ` Eduardo Valentin
2009-08-20 13:18 ` [PATCHv5 17/20] ASoC: Add runtime check for RFIG and XFIG Eduardo Valentin
2009-08-20 13:18 ` Eduardo Valentin
2009-08-20 13:18 ` [PATCHv5 18/20] ASoC: Always syncronize audio transfers on frames Eduardo Valentin
2009-08-20 13:18 ` Eduardo Valentin
2009-08-20 13:18 ` [PATCHv5 19/20] ASoC: OMAP: Use McBSP threshold to playback and capture Eduardo Valentin
2009-08-20 13:18 ` Eduardo Valentin
2009-08-20 13:18 ` [PATCHv5 20/20] ASoC: OMAP: Use DMA operating mode of McBSP Eduardo Valentin
2009-08-20 13:18 ` Eduardo Valentin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1250774306-7581-13-git-send-email-eduardo.valentin@nokia.com \
    --to=eduardo.valentin@nokia.com \
    --cc=alsa-devel@vger.kernel.org \
    --cc=broonie@opensource.wolfsonmicro.com \
    --cc=ext-Eero.Nurkkala@nokia.com \
    --cc=jhnikula@gmail.com \
    --cc=linux-omap@vger.kernel.org \
    --cc=peter.ujfalusi@nokia.com \
    --cc=tiwai@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.