Re: [PATCH] Documentation/drm: Update rotation property with 90/270 and description

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 5/13/2015 9:57 AM, Jindal, Sonika wrote:


On 5/12/2015 6:20 PM, Ville Syrjälä wrote:
On Wed, Apr 15, 2015 at 04:05:19PM +0530, Sonika Jindal wrote:
Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx
Signed-off-by: Sonika Jindal <sonika.jindal@xxxxxxxxx>
---
  Documentation/DocBook/drm.tmpl |    7 +++++--
  1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/Documentation/DocBook/drm.tmpl
b/Documentation/DocBook/drm.tmpl
index f4976cd..266d50a 100644
--- a/Documentation/DocBook/drm.tmpl
+++ b/Documentation/DocBook/drm.tmpl
@@ -2853,9 +2853,12 @@ void intel_crt_init(struct drm_device *dev)
      <td rowspan="1" valign="top" >Plane</td>
      <td valign="top" >“rotation”</td>
      <td valign="top" >BITMASK</td>
-    <td valign="top" >{ 0, "rotate-0" }, { 2, "rotate-180" }</td>
+    <td valign="top" >{ 0, "rotate-0" }, { 1, "rotate-90" },
+                { 2, "rotate-180" }, { 3, "rotate-270" }</td>
      <td valign="top" >Plane</td>
-    <td valign="top" >TBD</td>
+    <td valign="top" >To set plane HW rotation. This rotation
property does
+        the plane rotation in counter clockwise direction which is
+        inline with the way XRandr works.</td>

I would suggest moving the thing to the generci props section since we
have omap and i915 both supporting it.
You mean in DRM properties section?
Right now, OMAP section also has rotation property. I will remove it
from OMAP section as well if you think drm is the better place.


As for the description, we should also document the reflect flags.

i915 doesn't support reflect flags. It only create rotation property with rotation flags.

For "generic" section, you mean moving to generic group of properties in i915 only right?
I might write it as something like this:
"rotate-0,rotate-90,rotate-180,rotate-270 rotate the image by the
specified amount in degrees in a counter clockwise direction.
reflect-x,reflect-y reflect the image along the specified axis,
prior to rotation."

      </tr>
      <tr>
      <td rowspan="17" valign="top" >SDVO-TV</td>
--
1.7.10.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx





[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux