linux-omap.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Tony Lindgren <tony@atomide.com>
To: Aaro Koskinen <aaro.koskinen@iki.fi>
Cc: linux-omap@vger.kernel.org, "Sebastian Reichel" <sre@kernel.org>,
	"Pali Rohár" <pali.rohar@gmail.com>,
	"Pavel Machek" <pavel@ucw.cz>
Subject: Re: Nokia N900 FB regression?
Date: Wed, 23 Jul 2014 00:37:06 -0700	[thread overview]
Message-ID: <20140723073706.GH18374@atomide.com> (raw)
In-Reply-To: <20140722212318.GC2615@drone.musicnaut.iki.fi>

* Aaro Koskinen <aaro.koskinen@iki.fi> [140722 14:25]:
> Hi,
> 
> Somewhere between 3.16-rc2 and rc6 (I was on holidays...) I noticed
> the framebuffer stopped working on N900 (nothing on screen).
> 
> I bisected this to 9188883fd66e9809e93e06d5bbd49cf99a6cdbee (ARM: dts:
> Enable twl4030 off-idle configuration for selected omaps).
> With this commit reverted, I can see the Tux again with 3.16-rc6.
> 
> Any ideas?

Hmm, OK yeah that enables the deeper idle states. I have tested
this on n900, but only have remote access to it so could not see
the framebuffer.

Sounds like there's some twl regulator that we cannot idle
by default on n900. Here are the steps to narrow it down:

1. Try ti,twl4030-power-idle in omap3-n900.dts instead of
   ti,twl4030-power-idle-osc-off. This probably won't help
   unless you're enabling off-idle though, so if it does not
   help, just keep on using ti,twl4030-power-idle-osc-off.

2. Try commenting out one or more of the TWL_REMAP_SLEEP or
   TWL_REMAP_OFF lines in omap3_idle_rconfig in file
   drivers/mfd/twl4030-power.c. That's the recommended init
   sequence like the comments in the code mention. Chances
   are we're missing a regulator_get somewhere but finding
   out which one in omap3_idle_rconfig will help narrow it
   down :)

3. If it looks like a more complicated fix, we can just use
   compatible = "ti,twl4030-power-n900" to disable the twl
   features for v3.16 and re-enable it for v3.17.

Regards,

Tony

  parent reply	other threads:[~2014-07-23  7:38 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-22 21:23 Nokia N900 FB regression? Aaro Koskinen
2014-07-22 22:02 ` Nick Krause
2014-07-22 22:16   ` Nick Krause
2014-07-23  7:37 ` Tony Lindgren [this message]
2014-07-23 15:22   ` Nick Krause
2014-07-24 20:01   ` Aaro Koskinen
2014-07-25 11:13     ` Tony Lindgren

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=20140723073706.GH18374@atomide.com \
    --to=tony@atomide.com \
    --cc=aaro.koskinen@iki.fi \
    --cc=linux-omap@vger.kernel.org \
    --cc=pali.rohar@gmail.com \
    --cc=pavel@ucw.cz \
    --cc=sre@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;
as well as URLs for NNTP newsgroup(s).