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
next prev parent 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.