All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nicolas Ferre <nicolas.ferre@atmel.com>
To: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: alsa-devel@alsa-project.org,
	"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
	lrg@ti.com, linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/5] ASoC: wm8731: rework power management
Date: Fri, 01 Jul 2011 11:31:45 +0200	[thread overview]
Message-ID: <4E0D9401.1070300@atmel.com> (raw)
In-Reply-To: <20110630152203.GB3249@opensource.wolfsonmicro.com>

Le 30/06/2011 17:22, Mark Brown :
> On Thu, Jun 30, 2011 at 03:57:12PM +0200, Nicolas Ferre wrote:
> 
> Please fix your mailer to line wrap within paragraphs.
> 
>> First of all, I experienced issues while not having OSC enabled during
>> suspend/resume cycle. Am I right supposing that, if using oscillator
>> to clock the codec, I have to keep it running during a suspend/resume
>> cycle?
> 
> No, not at all.  What makes you believe that you would you need to do
> that?

Ok, I re-tested and for sure I was totally wrong! So we forget this part
of the patch.
I guess that this belief may come from my first implementation on 2.6.35
where OSC was not part of the DAPM... Sory for the noise.

I repost the "active bit" part real-soon-now.

Thanks for your help, best regards,
-- 
Nicolas Ferre

WARNING: multiple messages have this Message-ID (diff)
From: nicolas.ferre@atmel.com (Nicolas Ferre)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/5] ASoC: wm8731: rework power management
Date: Fri, 01 Jul 2011 11:31:45 +0200	[thread overview]
Message-ID: <4E0D9401.1070300@atmel.com> (raw)
In-Reply-To: <20110630152203.GB3249@opensource.wolfsonmicro.com>

Le 30/06/2011 17:22, Mark Brown :
> On Thu, Jun 30, 2011 at 03:57:12PM +0200, Nicolas Ferre wrote:
> 
> Please fix your mailer to line wrap within paragraphs.
> 
>> First of all, I experienced issues while not having OSC enabled during
>> suspend/resume cycle. Am I right supposing that, if using oscillator
>> to clock the codec, I have to keep it running during a suspend/resume
>> cycle?
> 
> No, not at all.  What makes you believe that you would you need to do
> that?

Ok, I re-tested and for sure I was totally wrong! So we forget this part
of the patch.
I guess that this belief may come from my first implementation on 2.6.35
where OSC was not part of the DAPM... Sory for the noise.

I repost the "active bit" part real-soon-now.

Thanks for your help, best regards,
-- 
Nicolas Ferre

WARNING: multiple messages have this Message-ID (diff)
From: Nicolas Ferre <nicolas.ferre@atmel.com>
To: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: alsa-devel@alsa-project.org, lrg@ti.com,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org,
	"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
Subject: Re: [PATCH 1/5] ASoC: wm8731: rework power management
Date: Fri, 01 Jul 2011 11:31:45 +0200	[thread overview]
Message-ID: <4E0D9401.1070300@atmel.com> (raw)
In-Reply-To: <20110630152203.GB3249@opensource.wolfsonmicro.com>

Le 30/06/2011 17:22, Mark Brown :
> On Thu, Jun 30, 2011 at 03:57:12PM +0200, Nicolas Ferre wrote:
> 
> Please fix your mailer to line wrap within paragraphs.
> 
>> First of all, I experienced issues while not having OSC enabled during
>> suspend/resume cycle. Am I right supposing that, if using oscillator
>> to clock the codec, I have to keep it running during a suspend/resume
>> cycle?
> 
> No, not at all.  What makes you believe that you would you need to do
> that?

Ok, I re-tested and for sure I was totally wrong! So we forget this part
of the patch.
I guess that this belief may come from my first implementation on 2.6.35
where OSC was not part of the DAPM... Sory for the noise.

I repost the "active bit" part real-soon-now.

Thanks for your help, best regards,
-- 
Nicolas Ferre


  reply	other threads:[~2011-07-01  9:31 UTC|newest]

Thread overview: 62+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-29 18:00 use dmaengine for atmel ssc/pcm dai drivers Nicolas Ferre
2011-06-29 18:00 ` Nicolas Ferre
2011-06-29 18:00 ` Nicolas Ferre
2011-06-29 18:00 ` [PATCH 1/5] ASoC: wm8731: rework power management Nicolas Ferre
2011-06-29 18:00   ` Nicolas Ferre
2011-06-29 18:00   ` Nicolas Ferre
2011-06-29 17:11   ` Mark Brown
2011-06-29 17:11     ` Mark Brown
2011-06-29 17:11     ` Mark Brown
2011-06-30 13:57     ` Nicolas Ferre
2011-06-30 13:57       ` Nicolas Ferre
2011-06-30 13:57       ` Nicolas Ferre
2011-06-30 15:04       ` [alsa-devel] " Nicolas Ferre
2011-06-30 15:04         ` Nicolas Ferre
2011-06-30 15:04         ` Nicolas Ferre
2011-06-30 15:22       ` Mark Brown
2011-06-30 15:22         ` Mark Brown
2011-06-30 15:22         ` Mark Brown
2011-07-01  9:31         ` Nicolas Ferre [this message]
2011-07-01  9:31           ` Nicolas Ferre
2011-07-01  9:31           ` Nicolas Ferre
2011-06-29 18:00   ` [PATCH 2/5] ASoC: atmel-ssc: add phybase in device structure Nicolas Ferre
2011-06-29 18:00     ` Nicolas Ferre
2011-06-29 17:48     ` Mark Brown
2011-06-29 17:48       ` Mark Brown
2011-07-01 14:46       ` [PATCH V2 " Nicolas Ferre
2011-07-01 14:46         ` Nicolas Ferre
2011-07-01 14:46         ` Nicolas Ferre
2011-06-29 18:00   ` [PATCH 3/5] ASoC: atmel-ssc: dmaengine usage switch depending on cpu Nicolas Ferre
2011-06-29 18:00     ` Nicolas Ferre
2011-06-29 18:00   ` [PATCH 4/5] ASoC: atmel_ssc_dai/atmel-pcm: adapt to dmaengine usage Nicolas Ferre
2011-06-29 18:00     ` Nicolas Ferre
2011-06-29 17:59     ` Mark Brown
2011-06-29 17:59       ` Mark Brown
2011-06-29 17:59       ` Mark Brown
2011-06-30  0:54     ` Koul, Vinod
2011-06-30  0:54       ` Koul, Vinod
2011-06-30  0:54       ` Koul, Vinod
2011-06-29 18:00   ` [PATCH 5/5] ASoC: atmel_ssc_dai: PM: actually stopping clock on suspend/resume Nicolas Ferre
2011-06-29 18:00     ` Nicolas Ferre
2011-07-01 10:43   ` [PATCH] ASoC: wm8731: set the ACTIVE bit if bias ON entered Nicolas Ferre
2011-07-01 10:43     ` Nicolas Ferre
2011-07-01 10:43     ` Nicolas Ferre
2011-07-01 14:18     ` Liam Girdwood
2011-07-01 14:18       ` Liam Girdwood
2011-07-01 14:18       ` Liam Girdwood
2011-07-01 16:15     ` Mark Brown
2011-07-01 16:15       ` Mark Brown
2011-07-04  9:46       ` Nicolas Ferre
2011-07-04  9:46         ` Nicolas Ferre
2011-07-04  9:46         ` Nicolas Ferre
2011-07-04 17:29         ` Mark Brown
2011-07-04 17:29           ` Mark Brown
2011-07-04 17:29           ` Mark Brown
2011-07-04 17:34           ` Liam Girdwood
2011-07-04 17:34             ` Liam Girdwood
2011-07-04 17:34             ` Liam Girdwood
2011-07-05  7:58           ` Nicolas Ferre
2011-07-05  7:58             ` Nicolas Ferre
2011-07-05  7:58             ` Nicolas Ferre
2011-07-05 20:27             ` [alsa-devel] " Mark Brown
2011-07-05 20:27               ` Mark Brown

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=4E0D9401.1070300@atmel.com \
    --to=nicolas.ferre@atmel.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@opensource.wolfsonmicro.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lrg@ti.com \
    --cc=u.kleine-koenig@pengutronix.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.