linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: l.stach@pengutronix.de (Lucas Stach)
To: linux-arm-kernel@lists.infradead.org
Subject: I.MX6 HDMI support in v4.2
Date: Tue, 08 Sep 2015 14:43:53 +0200	[thread overview]
Message-ID: <1441716233.13536.21.camel@pengutronix.de> (raw)
In-Reply-To: <20150908112909.GX21084@n2100.arm.linux.org.uk>

Am Dienstag, den 08.09.2015, 12:29 +0100 schrieb Russell King - ARM
Linux:
> On Tue, Sep 08, 2015 at 01:07:47PM +0200, Lucas Stach wrote:
> > Am Dienstag, den 08.09.2015, 12:01 +0100 schrieb Russell King - ARM
> > Linux:
> > > On Tue, Sep 08, 2015 at 12:56:18PM +0200, Lucas Stach wrote:
> > > > This is expected, it's the overlay adapter which doesn't really work
> > > > with imx-drm yet.
> > > 
> > > No, it's imx-drm which isn't working.  As I explained in my previous
> > > email, overlay planes are expected to do scaling.  imx-drm errors out
> > > attempts for that.  This is a kernel bug, not an Xorg driver bug.
> > > 
> > I would argue that this is a bug of the interface between kernel and
> > userspace.
> 
> Yes, only in so far as knowing beforehand whether scaling is possible.
> The only time that you get to know is when the call to display the plane
> fails.  That's a really poor interface.
> 
> > Scaling isn't something that can be expected to be usable on every
> > hardware (and in fact the IPU isn't able to do arbitrary scaling with
> > its 1024 in/out pixel constraints), but there is no clear way to
> > communicate this to userspace other than flat out rejecting the plane
> > update. Atomic may provide some better ways, but we are not there yet
> > for imx-drm.
> 
> I think that depends on your point of view - I suspect x86 people would
> be surprised by that comment. :)
> 
I think some of the recent Intel chips had the same problem, in that
they lost the ability to scale planes and ended up not exposing them due
to the poor interface.

> There was talk a while back when the overlay plane support went in that
> it was possible to do >1024 pixels, but it was complex, but the impression
> I was left with was one day it would work - and I'm still waiting. 

Don't wait for that. We can't do >1024 without doing a copy in memory,
which would be a big stretch of the plane definition and benefit.

In addition the IPU scaler has pitch and alignment constraints that make
the needed tiling extremely complex to get right. We fiddled with this
extensively to get it to the point that it is usable as a V4L2 mem2mem
device. Moving this to the display path is not going to happen, ever.

As all relevant resolutions already require a copy you are much better
of doing it with the GPU and get all the testing effort focused on that
path.

> I
> suspect that the iMX6 is going to be obsolete before we have a decent
> working video playback story on this hardware.
> 
MX6Plus is just around the corner and will probably keep us busy for a
while. ;)

Regards,
Lucas

-- 
Pengutronix e.K.             | Lucas Stach                 |
Industrial Linux Solutions   | http://www.pengutronix.de/  |

  reply	other threads:[~2015-09-08 12:43 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-07 10:55 I.MX6 HDMI support in v4.2 Krzysztof Hałasa
2015-09-07 11:25 ` Russell King - ARM Linux
2015-09-07 14:04   ` Krzysztof Hałasa
2015-09-08  9:16     ` Russell King - ARM Linux
2015-09-08 11:01       ` Krzysztof Hałasa
2015-09-08 12:57         ` Russell King - ARM Linux
2015-09-08 14:59           ` Krzysztof Hałasa
2015-09-10 10:25           ` Krzysztof Hałasa
2015-09-10 10:49             ` Russell King - ARM Linux
2015-09-10 11:29               ` Krzysztof Hałasa
2015-09-17  7:21               ` Philipp Zabel
2015-09-17  8:38                 ` Krzysztof Hałasa
2015-09-17  9:23                 ` Russell King - ARM Linux
2015-09-08 10:45   ` Krzysztof Hałasa
2015-09-08 10:56     ` Lucas Stach
2015-09-08 11:01       ` Russell King - ARM Linux
2015-09-08 11:07         ` Lucas Stach
2015-09-08 11:29           ` Russell King - ARM Linux
2015-09-08 12:43             ` Lucas Stach [this message]
2015-09-08 13:40               ` Russell King - ARM Linux
2015-09-08 14:17               ` Robert Nelson
2015-09-08 14:45                 ` Krzysztof Hałasa
2015-09-08 14:48                 ` Lucas Stach
2015-09-08 15:55                 ` Russell King - ARM Linux
2015-09-08 17:07                   ` Jon Nettleton
2015-09-08 11:06       ` Krzysztof Hałasa
2015-09-14  8:39       ` Krzysztof Hałasa
2015-09-15  8:24         ` Krzysztof Hałasa
2015-09-15 10:12           ` Russell King - ARM Linux
2015-09-15 11:01             ` Krzysztof Hałasa
2015-09-15 14:29               ` Russell King - ARM Linux
2015-09-15 16:53                 ` Krzysztof Hałasa
2015-09-15 15:53         ` Lucas Stach
2015-09-15 16:36           ` Russell King - ARM Linux
2015-09-15 16:53             ` Lucas Stach
2015-09-15 17:04               ` Russell King - ARM Linux
2015-09-15 19:01                 ` Lucas Stach
2015-09-28 14:48                 ` xf86-video-armada + etnaviv (Was: Re: I.MX6 HDMI support in v4.2) Lucas Stach
2015-09-28 15:24                   ` Russell King - ARM Linux
2015-09-28 15:40                     ` Lucas Stach
2015-09-28 16:50                       ` Russell King - ARM Linux
2015-09-29  8:28                         ` Lucas Stach
2015-09-29  8:41                           ` Russell King - ARM Linux
2015-09-29  9:01                             ` Lucas Stach
2015-09-15 16:57           ` I.MX6 HDMI support in v4.2 Krzysztof Hałasa
2015-09-16  7:57           ` Krzysztof Hałasa
2015-09-16 15:52             ` Lucas Stach

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=1441716233.13536.21.camel@pengutronix.de \
    --to=l.stach@pengutronix.de \
    --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 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).