All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: Grazvydas Ignotas <notasas@gmail.com>
Cc: Aaro Koskinen <aaro.koskinen@iki.fi>,
	Igor Grinberg <grinberg@compulab.co.il>,
	Thomas Weber <weber@corscience.de>,
	Mike Rapoport <mike@compulab.co.il>,
	Steve Sakoman <steve@sakoman.com>,
	linux-omap <linux-omap@vger.kernel.org>
Subject: Re: Request for OMAPDSS testing
Date: Wed, 12 Jun 2013 09:01:49 +0300	[thread overview]
Message-ID: <51B80ECD.3070607@ti.com> (raw)
In-Reply-To: <CANOLnONCuV_mXJAx8fahUraypeewug7RjNGyhXqVm8FYRAAUcg@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1471 bytes --]

On 09/06/13 17:28, Grazvydas Ignotas wrote:
> Hello,
> 
> On Tue, Jun 4, 2013 at 10:40 AM, Tomi Valkeinen <tomi.valkeinen@ti.com> wrote:
>> I've made some big changes on the omapdss device model, which involves
>> converting all the panel drivers. I've got only a bunch of boards, so I
>> hope some of you can perhaps do some minimal tests on some other boards.
> 
> Doesn't work on pandora (legacy boardfile boot):
> 
> [    1.418823] OMAP DSS rev 2.0
> [    1.447448] omapfb omapfb: no displays
> [    1.454101] omapfb omapfb: failed to setup omapfb
> [    1.459106] platform omapfb: Driver omapfb requests probe deferral
> ...
> [    2.156860] panel-tpo-td043mtea1 spi1.1: failed to get LCD VCC regulator
> [    2.164093] spi spi1.1: Driver panel-tpo-td043mtea1 requests probe deferral
> ...
> 
> Needs this small patch:
> 
> --- a/arch/arm/mach-omap2/board-omap3pandora.c
> +++ b/arch/arm/mach-omap2/board-omap3pandora.c
> @@ -335,7 +338,7 @@ static struct regulator_consumer_supply
> pandora_vdds_supplies[] = {
>  };
> 
>  static struct regulator_consumer_supply pandora_vcc_lcd_supply[] = {
> -       REGULATOR_SUPPLY("vcc", "display0"),
> +       REGULATOR_SUPPLY("vcc", "spi1.1"),
>  };
> 
>  static struct regulator_consumer_supply pandora_usb_phy_supply[] = {
> 
> With that it shows Tux and fb console fine, so with above hunk:
> Tested-by: Grazvydas Ignotas <notasas@gmail.com>

Thanks, I've added the fix.

 Tomi



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 901 bytes --]

  reply	other threads:[~2013-06-12  6:02 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-04  7:40 Request for OMAPDSS testing Tomi Valkeinen
2013-06-06 11:30 ` Igor Grinberg
2013-06-06 20:43 ` Aaro Koskinen
2013-06-07  8:39   ` Tomi Valkeinen
2013-06-09 14:28 ` Grazvydas Ignotas
2013-06-12  6:01   ` Tomi Valkeinen [this message]
2013-06-13 15:51 ` Igor Grinberg
2013-06-13 16:01   ` Tomi Valkeinen
2013-06-16 12:28     ` Igor Grinberg
2013-06-17  7:08       ` Tomi Valkeinen
2013-06-17  8:40         ` Igor Grinberg
2013-06-27  6:41           ` Tomi Valkeinen

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=51B80ECD.3070607@ti.com \
    --to=tomi.valkeinen@ti.com \
    --cc=aaro.koskinen@iki.fi \
    --cc=grinberg@compulab.co.il \
    --cc=linux-omap@vger.kernel.org \
    --cc=mike@compulab.co.il \
    --cc=notasas@gmail.com \
    --cc=steve@sakoman.com \
    --cc=weber@corscience.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.