All of lore.kernel.org
 help / color / mirror / Atom feed
From: sylvain.rochet@finsecur.com (Sylvain Rochet)
To: linux-arm-kernel@lists.infradead.org
Subject: AT91 slow clock mode regression/fixes, improvement proposal
Date: Mon, 22 Dec 2014 11:03:08 +0100	[thread overview]
Message-ID: <20141222100308.GB30666@gradator.net> (raw)
In-Reply-To: <B256D81BAE5131468A838E5D7A243641BFCB6CA0@penmbx01>

Hello Wenyou,

On Mon, Dec 22, 2014 at 08:32:15AM +0000, Yang, Wenyou wrote:
> > On Fri, Dec 19, 2014 at 02:50:04AM +0000, Yang, Wenyou wrote:
> > 
> > > > Furthermore, it looks like MCKRDY_TIMEOUT set to 1000 is not enough,
> > > > my board crashes in about 1 wake up to 10 with this value and works
> > > > perfectly fine with 4000.
> > >
> > > I also encountered this issue.
> > 
> > Fixed in proposed patch, since we don't timeout anymore.
> 
> I am verifying this patch. I still insist on remain the timeout.

Master Clock Ready is a special case, you need a timeout value which is 
fine at slow clock and master clock, that is, a timeout value which is 
fine despite the 10^4 difference in order of magnitude. I just don't 
know what to do, a fine timeout value for master clock is an almost 
infinite timeout on slow clock.

By the way, my board suspended and resumed every 2 seconds this 
week-end, which is about 110k suspend+wake up cycles and didn't reset on 
watchdog, which is quite a good news :-)

Regards,
Sylvain

  reply	other threads:[~2014-12-22 10:03 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-18 20:39 AT91 slow clock mode regression/fixes, improvement proposal Sylvain Rochet
2014-12-18 21:23 ` Sylvain Rochet
2014-12-19  2:50 ` Yang, Wenyou
2014-12-19 15:04   ` [PATCH] pm: at91: pm_slowclock: improve reliability of suspend/resume Sylvain Rochet
2014-12-19 15:26   ` AT91 slow clock mode regression/fixes, improvement proposal Sylvain Rochet
2014-12-22  8:32     ` Yang, Wenyou
2014-12-22 10:03       ` Sylvain Rochet [this message]
2015-01-05  3:32       ` Yang, Wenyou
2015-01-06 14:15         ` [PATCH] pm: at91: pm_slowclock: improve reliability of suspend/resume Sylvain Rochet
2015-01-06 14:16         ` Sylvain Rochet
2015-01-06 14:25         ` AT91 slow clock mode regression/fixes, improvement proposal Sylvain Rochet
2015-01-07  1:17           ` Yang, Wenyou
2015-01-07  9:14           ` Alexandre Belloni

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=20141222100308.GB30666@gradator.net \
    --to=sylvain.rochet@finsecur.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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.