linux-omap.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: Steve Sakoman <sakoman@gmail.com>
Cc: Enric Balletbo Serra <eballetbo@gmail.com>,
	"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
	Tony Lindgren <tony@atomide.com>,
	Javier Martinez Canillas <javier@dowhile0.org>
Subject: Re: omap DSS fails with tft410 driver panel?
Date: Fri, 23 Nov 2012 10:46:34 +0200	[thread overview]
Message-ID: <50AF37EA.3080400@ti.com> (raw)
In-Reply-To: <CAGDS+nm2YVNyq+eAF-VSq785KAKWC2GM4v5U7N1vLGTNNDfa7Q@mail.gmail.com>

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

On 2012-11-23 08:14, Steve Sakoman wrote:
> On Thu, Nov 22, 2012 at 12:47 AM, Tomi Valkeinen <tomi.valkeinen@ti.com> wrote:
> 
>>> I'll add the printk's to omapfb_setup_plane() that were requested by
>>> Tomi and report back.
> 
> Today was a holiday, so family obligations didn't allow much time to
> look at this.
> 
> I did however do a quick build with some printk's in
> omapfb_setup_plane() so I could narrow down where to start looking.
> 
> I discovered that the error (omapdss OVERLAY error: check_overlay:
> paddr cannot be 0) is happening as a result of the
> ovl->set_overlay_info() call in the else clause of the code below:
> 
>     if (pi->enabled) {
>         r = omapfb_setup_overlay(fbi, ovl, pi->pos_x, pi->pos_y,
>             pi->out_width, pi->out_height);
>         if (r) {
>             printk("omapfb_setup_plane: pi->enabled &
> omapfb_setup_overlay()\n");
>             goto undo;
>         }
>     } else {
>         struct omap_overlay_info info;
> 
>         ovl->get_overlay_info(ovl, &info);
> 
>         info.pos_x = pi->pos_x;
>         info.pos_y = pi->pos_y;
>         info.out_width = pi->out_width;
>         info.out_height = pi->out_height;
> 
>         r = ovl->set_overlay_info(ovl, &info);
>         if (r) {
>             printk("omapfb_setup_plane: !pi->enabled &
> ovl->set_overlay_info failed\n");
>             goto undo;
>         }
>     }

Ok, so hmm... So the overlay is disabled, and the paddr is zero. Can you
put a printk at dss/apply.c:dss_olv_set_info, and print ovl->id and
info->paddr? And perhaps also to omapfb-main.c:omapfb_setup_overlay, at
the point where it sets info.paddr, as I think that's the only place
where omapfb sets paddr.

I don't know what's going on there, why the paddr is suddenly zero.
Looking at the log from Enric, SETUP_PLANE works fine just fine, and
then, for no reason, next SETUP_PLANE fails, and the log doesn't show
anything much happening between.

However, I think omapdss may have been more permissive in the past,
allowing paddr 0 if the overlay is disabled. I could add that back, but
I'd rather first understand what's happening here.

> I'll look at this more over as I get time over the coming days.
> 
>> How can I reproduce this? Is there a downloadable rootfs somewhere that
>> I could try?
> 
> The only downloadable rootfs I have is 3.5 based.  I'll try to get a
> 3.6 image posted in the next couple of days.
> 
>> Could you also copy full kernel boot log to pastebin or such?
> 
> OK, will do that with my next build.  I'll assume you want dss
> debugging enabled for that.

That's probably not needed. I mostly wanted to know if there's some dss
error/warning seen on the boot. But probably not if the problem happens
in the code above.

 Tomi



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

  reply	other threads:[~2012-11-23  8:46 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-11 15:58 omap DSS fails with tft410 driver panel? Enric Balletbo Serra
2012-10-12 10:56 ` Tomi Valkeinen
2012-10-15 16:08   ` Enric Balletbo Serra
2012-10-16  7:30     ` Tomi Valkeinen
2012-11-22  0:08 ` Steve Sakoman
2012-11-22  8:47   ` Tomi Valkeinen
2012-11-23  6:14     ` Steve Sakoman
2012-11-23  8:46       ` Tomi Valkeinen [this message]
2012-11-23 15:20         ` Steve Sakoman
2012-11-27 11:32           ` Tomi Valkeinen
2012-11-27 19:15             ` Steve Sakoman
2012-11-29 11:22               ` Tomi Valkeinen
2012-11-29 11:33                 ` Enric Balletbo Serra
2012-11-29 14:24                   ` Steve Sakoman
2012-11-29 14:30                     ` Enric Balletbo Serra
2012-11-29 14:18                 ` Steve Sakoman
2012-11-29 14:31                   ` Tomi Valkeinen
2012-11-29 14:53                     ` Steve Sakoman
2012-11-29 14:56                       ` Tomi Valkeinen
2012-12-18  8:31                         ` Javier Martinez Canillas
2012-12-18 11:23                           ` Javier Martinez Canillas
2012-12-18 13:45                             ` Javier Martinez Canillas
2012-12-19  8:33                               ` Tomi Valkeinen
2012-12-19 11:27                                 ` Javier Martinez Canillas
2012-12-19  8:24                           ` Tomi Valkeinen
2012-12-21  8:30   ` Laurent Pinchart
2012-12-21 10:19     ` Javier Martinez Canillas
2013-09-12 11:12       ` pawel cern

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=50AF37EA.3080400@ti.com \
    --to=tomi.valkeinen@ti.com \
    --cc=eballetbo@gmail.com \
    --cc=javier@dowhile0.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=sakoman@gmail.com \
    --cc=tony@atomide.com \
    /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).