From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Chris Wilson <chris@chris-wilson.co.uk>,
Damien Lespiau <damien.lespiau@intel.com>,
sonika.jindal@intel.com, intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH 00/11] Support for 180 degree HW rotation
Date: Wed, 18 Jun 2014 14:57:19 +0300 [thread overview]
Message-ID: <20140618115719.GO27580@intel.com> (raw)
In-Reply-To: <20140618112139.GE14442@nuc-i3427.alporthouse.com>
On Wed, Jun 18, 2014 at 12:21:39PM +0100, Chris Wilson wrote:
> On Wed, Jun 18, 2014 at 12:12:31PM +0100, Damien Lespiau wrote:
> > On Wed, Jun 18, 2014 at 12:07:17PM +0100, Chris Wilson wrote:
> > > > Now that we have universal planes, I guess the last blocker for this
> > > > series is gone. Thanks for keeping at it.
> > >
> > > Hah. We were last arguing over the property names.
> >
> > But only because the property was on the CRTC and "rotation" (or
> > "primary-rotation") ended up in limbo? The property is now on the
> > (primary) plane object.
>
> Ville also had plans for a superproperty called rotation on the CRTC
> that would adjust all planes. Not quite sure what his final plans were.
I just liked it because it was so easy to hook in at the kernel level,
and it made it possible to test rotation without having support for it
in the ddx. But I don't mind if we just drop it for now. We can revisit
later if there's a real use case for it.
--
Ville Syrjälä
Intel OTC
next prev parent reply other threads:[~2014-06-18 11:57 UTC|newest]
Thread overview: 68+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-18 8:57 [PATCH 00/11] Support for 180 degree HW rotation sonika.jindal
2014-06-18 8:57 ` [PATCH 01/11] drm: Move DRM_ROTATE bits out of omapdrm into drm_crtc.h sonika.jindal
2014-06-18 8:57 ` [PATCH 02/11] drm: Add support_bits parameter to drm_property_create_bitmask() sonika.jindal
2014-06-18 8:57 ` [PATCH 03/11] drm: Add drm_mode_create_rotation_property() sonika.jindal
2014-06-18 8:57 ` [PATCH 04/11] drm/omap: Switch omapdrm over to drm_mode_create_rotation_property() sonika.jindal
2014-06-18 8:57 ` [PATCH 05/11] drm: Add drm_rect rotation functions sonika.jindal
2014-06-18 8:57 ` [PATCH 06/11] drm: Add drm_rotation_simplify() sonika.jindal
2014-06-18 8:57 ` [PATCH 07/11] drm/i915: Add 180 degree sprite rotation support sonika.jindal
2014-06-18 8:57 ` [PATCH 08/11] drm/i915: Make intel_plane_restore() return an error sonika.jindal
2014-06-18 8:57 ` [PATCH 09/11] drm/i915: Add rotation property for sprites sonika.jindal
2014-06-18 11:12 ` Damien Lespiau
2014-06-18 11:54 ` Jindal, Sonika
2014-06-18 13:09 ` Damien Lespiau
2014-06-18 12:01 ` Ville Syrjälä
2014-06-18 8:57 ` [PATCH 10/11] drm/i915: Add 180 degree primary plane rotation support sonika.jindal
2014-06-18 17:02 ` Damien Lespiau
2014-06-19 6:43 ` Jindal, Sonika
2014-06-19 7:07 ` Daniel Vetter
2014-06-19 7:52 ` Jindal, Sonika
2014-06-19 7:55 ` Daniel Vetter
2014-06-19 8:09 ` Jindal, Sonika
2014-06-19 8:21 ` Daniel Vetter
2014-06-19 12:39 ` [PATCH] drm: Resetting rotation property sonika.jindal
2014-06-23 5:35 ` [PATCH 0/3] Moving rotation_property to drm_plane sonika.jindal
2014-06-23 5:35 ` [PATCH 1/3] drm/i915: Add rotation property for sprites sonika.jindal
2014-06-23 5:36 ` [PATCH 2/3] drm/i915: Add 180 degree primary plane rotation support sonika.jindal
2014-06-24 10:14 ` Damien Lespiau
2014-06-24 10:26 ` Jindal, Sonika
2014-06-24 10:29 ` Damien Lespiau
2014-06-24 10:34 ` Jindal, Sonika
2014-06-23 5:36 ` [PATCH 3/3] drm: Resetting rotation property sonika.jindal
2014-06-24 10:27 ` Damien Lespiau
2014-06-24 12:08 ` [PATCH 0/3] Moving rotation_property to drm_plane sonika.jindal
2014-06-24 12:08 ` [PATCH 1/3] drm/i915: Add rotation property for sprites sonika.jindal
2014-06-24 12:08 ` [PATCH 2/3] drm/i915: Add 180 degree primary plane rotation support sonika.jindal
2014-06-30 6:12 ` [PATCH] " sonika.jindal
2014-06-24 12:08 ` [PATCH 3/3] drm: Resetting rotation property sonika.jindal
2014-07-02 8:51 ` [PATCH 0/3] Moving rotation_property to drm_plane Jindal, Sonika
2014-07-02 13:17 ` Damien Lespiau
2014-07-07 14:34 ` Daniel Vetter
2014-06-19 10:07 ` [PATCH 10/11] drm/i915: Add 180 degree primary plane rotation support Damien Lespiau
2014-06-19 10:38 ` Daniel Vetter
2014-06-27 10:34 ` Tvrtko Ursulin
2014-06-27 10:49 ` Jindal, Sonika
2014-06-27 11:12 ` Tvrtko Ursulin
2014-06-27 11:14 ` Jindal, Sonika
2014-06-27 10:38 ` Tvrtko Ursulin
2014-06-27 11:15 ` Jindal, Sonika
2014-06-18 8:57 ` [PATCH 11/11] tests/kms_rotation_crc: IGT for 180 degree HW rotation sonika.jindal
2014-06-18 11:32 ` Damien Lespiau
2014-06-18 11:39 ` Chris Wilson
2014-06-25 5:54 ` Jindal, Sonika
2014-06-25 5:57 ` Chris Wilson
2014-06-25 6:00 ` Jindal, Sonika
2014-06-18 12:00 ` Jindal, Sonika
2014-06-18 11:00 ` [PATCH 00/11] Support " Damien Lespiau
2014-06-18 11:07 ` Chris Wilson
2014-06-18 11:12 ` Damien Lespiau
2014-06-18 11:21 ` Chris Wilson
2014-06-18 11:37 ` Damien Lespiau
2014-06-18 11:57 ` Ville Syrjälä [this message]
2014-06-18 11:51 ` Jindal, Sonika
2014-06-19 6:11 ` [PATCH 1/1] Documentation: drm: describing rotation property for i915 sonika.jindal
2014-06-24 10:05 ` Damien Lespiau
2014-06-25 5:38 ` [PATCH 1/2] Documentation: drm: Removing placeholders for generic drm properties description sonika.jindal
2014-06-25 5:38 ` [PATCH 2/2] Documentation: drm: describing rotation property for i915 sonika.jindal
2014-07-02 11:01 ` Damien Lespiau
2014-07-02 11:00 ` [PATCH 1/2] Documentation: drm: Removing placeholders for generic drm properties description Damien Lespiau
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=20140618115719.GO27580@intel.com \
--to=ville.syrjala@linux.intel.com \
--cc=chris@chris-wilson.co.uk \
--cc=damien.lespiau@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=sonika.jindal@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox