All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thierry Reding <thierry.reding@gmail.com>
To: Dave Airlie <airlied@gmail.com>
Cc: "X.Org Devel List" <xorg-devel@lists.freedesktop.org>,
	dri-devel <dri-devel@lists.freedesktop.org>
Subject: Re: tile property contents
Date: Wed, 15 Oct 2014 10:29:22 +0200	[thread overview]
Message-ID: <20141015082921.GD12807@ulmo> (raw)
In-Reply-To: <CAPM=9txxogkDSo1CcuULBY5fECAcnxJPAku1PSr8zWvo33Tf4g@mail.gmail.com>


[-- Attachment #1.1: Type: text/plain, Size: 4558 bytes --]

On Wed, Oct 15, 2014 at 06:35:52AM +1000, Dave Airlie wrote:
> On 14 October 2014 21:40, Thierry Reding <thierry.reding@gmail.com> wrote:
> > On Tue, Oct 14, 2014 at 01:23:22PM +1000, Dave Airlie wrote:
> >> Hi,
> >>
> >> So I've been hacking on mutter and the gnome pieces for tiling, and
> >> I've at least fixed mutter locally so maximise windows works and the
> >> heads are in the right order.
> >>
> >> Now I've strung all the pieces together using a single KMS property
> >> that X.org propogates, and mutter picks up and propagates over dbus as
> >> well,
> >>
> >> Currently I've ascii encoded the property into a blob,
> >>
> >> <ver>:<tileid>:<flags>:<maxhtiles>:<maxvtiles>:<h_tile_loc>:<v_tile_loc>:<tile_w>:<tile_h>
> >>
> >> I'm thinking of dropping the version field and just exposing TILE2
> >> property if we need it later to add more values,
> >>
> >> The other fields:
> >> tileid: a group id assigned by the kernel to all tiles in the same
> >> group - unique per group
> >> flags: bit 0 : single monitor enclosure
> >> maxhtiles: total number of horiz tiles
> >> maxvtiles: total number of vert tiles
> >> h_tile_loc: horiz location of this output in tile group
> >> v_tile_loc: vert location of this output in tile group
> >> tile_w: width of this tile
> >> tile_h: height of this tile.
> >>
> >> Now we extract all of these from the DisplayID v1.3 block, and I'm
> >> wondering if maybe I shouldn't just export the whole DisplayID tiling
> >> info block instead, it however encodes a few other pieces of
> >> information, including bezel info, and some flags specifying behaviour
> >> in some cases.
> >>
> >> The former could be more suitable for cases where DisplayID isn't
> >> available (Dual DSI panels?) but I'm worried abuot exposing too little
> >> at this point making TILE useless when the next monitor comes out.
> >
> > I don't think this is a good fit to describe dual DSI panels in the
> > first place. While one of the modes (left-right split) could probably be
> > described using the above, the other mode (odd-even split) is more
> > difficult. In the latter mode, one controller will provide the odd lines
> > and the other controller will provide the even lines.
> 
> Okay I'm happy with dual-DSI panels that you can hide those in the
> kernel, they don't seem hotpluggable,
> 
> so if you have one in your device-tree, you can probably just never
> expose the second crtc/encoder in the mode groups,
> and keep them for the kernel to use as slaves for the panel.

For reference: on Tegra a single CRTC sends pixel data to both DSI
outputs. The DSI outputs can then be programmed to take only the pixels
that they need to display. In the driver that I've submitted for review
a couple of days ago this is done by enslaving the second DSI output so
that it doesn't expose a regular DSI connector. The first DSI output is
then "augmented" to support a maximum of 8 data lanes instead of only 4
data lanes.

It'd be interesting to see if some of that can be extracted into common
code, but for now I've opted to handle it all in the Tegra driver since
it is the only implementation of this mode at this time.

> > One other thing that worries me about this is that we defer handling of
> > these complex configurations to userspace. I suppose this is fine, and
> > in fact the only way, if there is no knowledge about the tile layout in
> > kernel space. But if we know precisely how these various tiles are
> > connected, wouldn't we be better off abstracting this away within the
> > kernel and expose a single connector that is the union of all the tiles?
> > After all that's what the kernel is, an abstraction between hardware and
> > userspace.
> >
> 
> We can't do that for the MST panels, because the abstractions is too
> leaky, stealing crtcs dynamically at runtime, is screwed up and
> getting pageflipping across crtcs without userspace knowing is also a
> large pit of fail.

I see. That sounds like it can indeed not be reasonably hidden in the
kernel. Or at least I wouldn't know how, so dealing with it in userspace
seems like a better option.

Just for my understanding, is it typical for each of these panels to be
standalone (own housing, ...) or are there monitors that actually take
two connectors and each of them drives a different part of the same
panel? A quick search on the internet indicates that the former is more
common (I haven't actually been able to find an example of the latter).

Thierry

[-- Attachment #1.2: Type: application/pgp-signature, Size: 819 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2014-10-15  8:29 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-14  3:23 tile property contents Dave Airlie
     [not found] ` <CAPM=9tzCLM2eV2Gre_52AUcdSa0z_qqYFppfGvz75e5o9eUwFg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-10-14 11:40   ` Thierry Reding
2014-10-14 20:35     ` Dave Airlie
2014-10-15  8:29       ` Thierry Reding [this message]
2014-10-15 10:04         ` Daniel Stone
2014-10-15 11:07           ` Thierry Reding
2014-10-22  6:34   ` Andy Ritger
2014-10-22 21:20     ` Daniel Vetter
     [not found]       ` <CAKMK7uHE=cCfpZkSaCMmePF+M2uUGXVnY4wQ4qeEN43d8n2DGA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-10-22 23:03         ` Andy Ritger
2014-10-23  7:58           ` Daniel Vetter
     [not found]             ` <20141023075840.GZ26941-dv86pmgwkMBes7Z6vYuT8azUEOm+Xw19@public.gmane.org>
2014-10-24  7:25               ` Dave Airlie
2014-10-24  7:41                 ` Daniel Vetter
2014-12-03  0:01   ` Aaron Plattner
     [not found]     ` <547E52BC.6090905-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2014-12-03  3:04       ` Dave Airlie
     [not found]         ` <CAPM=9txM15Fiz75UMUkK=XvoGAU4+Ya=t6P3msOzstMxF8bM2g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-12-03 15:41           ` Aaron Plattner

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=20141015082921.GD12807@ulmo \
    --to=thierry.reding@gmail.com \
    --cc=airlied@gmail.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=xorg-devel@lists.freedesktop.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 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.