Linux Samsung SOC development
 help / color / mirror / Atom feed
From: Markus Reichl <m.reichl@fivetechno.de>
To: linux-samsung-soc@vger.kernel.org
Subject: Re: PM / devfreq: exynos-bus: need for suspend OPP?
Date: Mon, 21 Nov 2016 19:10:57 +0100	[thread overview]
Message-ID: <3720389.YmyvQuOEbi@dell2.five-lan.de> (raw)
In-Reply-To: <58330FC8.5090606@math.uni-bielefeld.de>

Hi Tobias,

Am Montag, 21. November 2016, 16:16:24 schrieb Tobias Jakobi:
> Hello Markus,
> 
> 
> Markus Reichl wrote:
> > Hi Tobias,
> > 
> > Am 21.11.2016 um 14:33 schrieb Tobias Jakobi:
> >> Hello everyone,
> >>
> >> I was thinking about the following. At the moment we have a suspend OPP
> >> for cpufreq-dt in place for the Exynos4412 SoC (added in
> >> 1605b60ad064c7019db8ade07f0b7bdc8c197b93). The rationale behind is that
> >> the board using the SoC might not have some PMIC reset in place. In case
> >> the board goes into reboot with a low OPP (i.e. low frequency, but also
> >> low core voltage), this results in a hang when the first-stage
> >> bootloaders sets its default core frequency.
> >>
> >> So this is properly handled in the kernel just fine, except for some
> >> corner cases like emergency reboot.
> >>
> >> But some time ago devfreq support for the various busses in the
> >> Exynos4412 was added. On the ODROID boards e.g. this adjust MIF and INT
> >> voltage.
> >>
> >> Let us take the DMC bus. Operating frequency is 100~400MHz and voltage
> >> is 900~1050mV.
> >>
> >> Now let's look at the corresponding board file
> >> (http://git.denx.de/?p=u-boot.git;a=blob;f=board/samsung/odroid/odroid.c#l234)
> >> in upstream u-boot. If I read this correctly DMC is set to 400MHz there.
> >>
> >> Here's the question. Could this, similar to the cpufreq/core frequency
> >> issue, pose a problem when the kernel goes into reboot when DMC is on
> >> the lowest OPP state?
> >>
> >> I'm not saying that it does. This just came to my mind during some
> >> recent discussion.
> > 
> > Made a test with
> > # echo performance > /sys/class/devfreq/bus_leftbus/governor 
> > # echo performance > /sys/class/devfreq/bus_dmc/governor 
> > just before reboot.
> > 
> > 20 out of 20 reboots worked.
> > 
> > With devfreq simple_ondemand governor around 50% reboots hung.
> thanks for the tests. I assume by 'reboots hung' you mean that the boot
> process doesn't even reach u-boot?

Yes, in these cases u-boot does not appear on the U3.

--
Markus
> 
> - Tobias
> 
> 
> > This could support your thoughts above.
> > 
> > Servus,
> > --
> > Markus
> >>
> >>
> >> With best wishes,
> >> Tobias

      reply	other threads:[~2016-11-21 18:11 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-21 13:33 PM / devfreq: exynos-bus: need for suspend OPP? Tobias Jakobi
2016-11-21 14:48 ` Markus Reichl
2016-11-21 15:06   ` MyungJoo Ham
2016-11-21 15:23     ` Tobias Jakobi
2016-11-21 16:21       ` MyungJoo Ham
2016-11-21 22:15         ` Tobias Jakobi
2016-11-22 12:39           ` MyungJoo Ham
2016-11-23 13:55             ` Tobias Jakobi
2016-11-21 15:16   ` Tobias Jakobi
2016-11-21 18:10     ` Markus Reichl [this message]

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=3720389.YmyvQuOEbi@dell2.five-lan.de \
    --to=m.reichl@fivetechno.de \
    --cc=linux-samsung-soc@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox