From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Marcus Lorentzon <marcus.xm.lorentzon@stericsson.com>
Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>,
"dri-devel@lists.freedesktop.org"
<dri-devel@lists.freedesktop.org>
Subject: Re: [PATCH 4/5] drm/i915: add 'rotation' CRTC property
Date: Tue, 20 Mar 2012 20:49:13 +0200 [thread overview]
Message-ID: <20120320184913.GX4917@intel.com> (raw)
In-Reply-To: <4F68B4CE.8070407@stericsson.com>
On Tue, Mar 20, 2012 at 05:48:14PM +0100, Marcus Lorentzon wrote:
> Then it would also be nice to define some rules in KMS of what should be
> modeset params and what should be properties. Modeset params seems hard
> to add (struct change) and properties seems very loosely defined and
> device specific. Having something in between would be nice (standard and
> easy to add), to make it easier to add things like plane z-order
> (instead of driver specific ioctls or properties).
> Maybe a new "atomic" modeset with just a list of {object_id,
> property_id, value} making all params into properties and modeset more
> dynamic. It would also solve the atomic modset, even with device
> specific properties and future features.
Great minds think alike ;) Exactly what I've suggested a few times
before. Unfortunately I haven't had the time to do any prototyping
yet.
--
Ville Syrjälä
Intel OTC
next prev parent reply other threads:[~2012-03-20 18:41 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-20 14:48 [PATCH 1/5] drm: add drm_property_change_is_valid Paulo Zanoni
2012-03-20 14:48 ` [PATCH 2/5] drm: WARN() when drm_connector_attach_property fails Paulo Zanoni
2012-03-20 14:48 ` [PATCH 3/5] drm: add CRTC properties Paulo Zanoni
2012-03-20 15:00 ` Rob Clark
2012-03-20 15:09 ` Alex Deucher
2012-03-20 15:48 ` Daniel Vetter
2012-03-20 14:48 ` [PATCH 4/5] drm/i915: add 'rotation' CRTC property Paulo Zanoni
2012-03-20 16:48 ` Marcus Lorentzon
2012-03-20 18:49 ` Ville Syrjälä [this message]
2012-03-20 14:48 ` [PATCH 5/5] drm/i915: add overscan compensation CRTC properties Paulo Zanoni
2012-03-20 15:00 ` Alex Deucher
2012-03-20 15:37 ` [PATCH 1/5] drm: add drm_property_change_is_valid Ville Syrjälä
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=20120320184913.GX4917@intel.com \
--to=ville.syrjala@linux.intel.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=marcus.xm.lorentzon@stericsson.com \
--cc=paulo.r.zanoni@intel.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 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.