linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 01/12] drm: Move DRM_ROTATE bits out of omapdrm into drm_crtc.h
       [not found] <1399622493-31701-1-git-send-email-sonika.jindal@intel.com>
@ 2014-05-09  8:01 ` sonika.jindal
  2014-05-09  9:21   ` Tomi Valkeinen
  2014-05-13  7:46   ` Thierry Reding
  0 siblings, 2 replies; 4+ messages in thread
From: sonika.jindal @ 2014-05-09  8:01 UTC (permalink / raw)
  To: sonika.jindal
  Cc: David Airlie, Tomi Valkeinen, Dave Airlie, Rob Clark,
	Daniel Vetter, Archit Taneja, dri-devel, linux-kernel,
	Ville Syrjälä

From: Ville Syrjälä <ville.syrjala at linux.intel.com>

The rotation property stuff should be standardized among all drivers.
Move the bits to drm_crtc.h from omap_drv.h.

Cc: David Airlie <airlied@linux.ie>
Cc: Tomi Valkeinen <tomi.valkeinen@ti.com>
Cc: Dave Airlie <airlied@redhat.com>
Cc: Rob Clark <robdclark@gmail.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Archit Taneja <archit@ti.com>
Cc: dri-devel@lists.freedesktop.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Reviewed-by: Sagar Kamble <sagar.a.kamble@intel.com>
Tested-by: Sagar Kamble <sagar.a.kamble@intel.com>
---
 drivers/gpu/drm/omapdrm/omap_drv.h |    7 -------
 include/drm/drm_crtc.h             |    8 ++++++++
 2 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/drivers/gpu/drm/omapdrm/omap_drv.h b/drivers/gpu/drm/omapdrm/omap_drv.h
index 428b2981..aac8e10 100644
--- a/drivers/gpu/drm/omapdrm/omap_drv.h
+++ b/drivers/gpu/drm/omapdrm/omap_drv.h
@@ -119,13 +119,6 @@ struct omap_drm_private {
 	struct omap_drm_irq error_handler;
 };
 
-/* this should probably be in drm-core to standardize amongst drivers */
-#define DRM_ROTATE_0	0
-#define DRM_ROTATE_90	1
-#define DRM_ROTATE_180	2
-#define DRM_ROTATE_270	3
-#define DRM_REFLECT_X	4
-#define DRM_REFLECT_Y	5
 
 #ifdef CONFIG_DEBUG_FS
 int omap_debugfs_init(struct drm_minor *minor);
diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
index 27f828c..4775008 100644
--- a/include/drm/drm_crtc.h
+++ b/include/drm/drm_crtc.h
@@ -64,6 +64,14 @@ struct drm_object_properties {
 	uint64_t values[DRM_OBJECT_MAX_PROPERTY];
 };
 
+/* rotation property bits */
+#define DRM_ROTATE_0	0
+#define DRM_ROTATE_90	1
+#define DRM_ROTATE_180	2
+#define DRM_ROTATE_270	3
+#define DRM_REFLECT_X	4
+#define DRM_REFLECT_Y	5
+
 enum drm_connector_force {
 	DRM_FORCE_UNSPECIFIED,
 	DRM_FORCE_OFF,
-- 
1.7.10.4


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH 01/12] drm: Move DRM_ROTATE bits out of omapdrm into drm_crtc.h
  2014-05-09  8:01 ` [PATCH 01/12] drm: Move DRM_ROTATE bits out of omapdrm into drm_crtc.h sonika.jindal
@ 2014-05-09  9:21   ` Tomi Valkeinen
  2014-05-13  7:46   ` Thierry Reding
  1 sibling, 0 replies; 4+ messages in thread
From: Tomi Valkeinen @ 2014-05-09  9:21 UTC (permalink / raw)
  To: sonika.jindal
  Cc: David Airlie, Dave Airlie, Rob Clark, Daniel Vetter,
	Archit Taneja, dri-devel, linux-kernel, Ville Syrjälä

[-- Attachment #1: Type: text/plain, Size: 972 bytes --]

On 09/05/14 11:01, sonika.jindal@intel.com wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> 
> The rotation property stuff should be standardized among all drivers.
> Move the bits to drm_crtc.h from omap_drv.h.
> 
> Cc: David Airlie <airlied@linux.ie>
> Cc: Tomi Valkeinen <tomi.valkeinen@ti.com>
> Cc: Dave Airlie <airlied@redhat.com>
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> Cc: Archit Taneja <archit@ti.com>
> Cc: dri-devel@lists.freedesktop.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Reviewed-by: Sagar Kamble <sagar.a.kamble@intel.com>
> Tested-by: Sagar Kamble <sagar.a.kamble@intel.com>
> ---
>  drivers/gpu/drm/omapdrm/omap_drv.h |    7 -------
>  include/drm/drm_crtc.h             |    8 ++++++++
>  2 files changed, 8 insertions(+), 7 deletions(-)

Acked-by: Tomi Valkeinen <tomi.valkeinen@ti.com>

 Tomi



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH 01/12] drm: Move DRM_ROTATE bits out of omapdrm into drm_crtc.h
  2014-05-09  8:01 ` [PATCH 01/12] drm: Move DRM_ROTATE bits out of omapdrm into drm_crtc.h sonika.jindal
  2014-05-09  9:21   ` Tomi Valkeinen
@ 2014-05-13  7:46   ` Thierry Reding
  2014-05-13  8:11     ` Jindal, Sonika
  1 sibling, 1 reply; 4+ messages in thread
From: Thierry Reding @ 2014-05-13  7:46 UTC (permalink / raw)
  To: sonika.jindal
  Cc: David Airlie, Tomi Valkeinen, Dave Airlie, Rob Clark,
	Daniel Vetter, Archit Taneja, dri-devel, linux-kernel,
	Ville Syrjälä

[-- Attachment #1: Type: text/plain, Size: 888 bytes --]

On Fri, May 09, 2014 at 01:31:22PM +0530, sonika.jindal@intel.com wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> 
> The rotation property stuff should be standardized among all drivers.
> Move the bits to drm_crtc.h from omap_drv.h.
> 
> Cc: David Airlie <airlied@linux.ie>
> Cc: Tomi Valkeinen <tomi.valkeinen@ti.com>
> Cc: Dave Airlie <airlied@redhat.com>
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> Cc: Archit Taneja <archit@ti.com>
> Cc: dri-devel@lists.freedesktop.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Reviewed-by: Sagar Kamble <sagar.a.kamble@intel.com>
> Tested-by: Sagar Kamble <sagar.a.kamble@intel.com>

This is missing your Signed-off-by line. Also the patch subject says
01/12, where are the other eleven patches?

Thierry

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH 01/12] drm: Move DRM_ROTATE bits out of omapdrm into drm_crtc.h
  2014-05-13  7:46   ` Thierry Reding
@ 2014-05-13  8:11     ` Jindal, Sonika
  0 siblings, 0 replies; 4+ messages in thread
From: Jindal, Sonika @ 2014-05-13  8:11 UTC (permalink / raw)
  To: Thierry Reding
  Cc: David Airlie, Tomi Valkeinen, Dave Airlie, Rob Clark,
	Daniel Vetter, Archit Taneja, dri-devel, linux-kernel,
	Ville Syrjälä

Hi Thierry,

This patch was sent out by mistake.
I am sorry for the confusion.
I am working on further patchset based on these patches.

Regards,
Sonika

On 5/13/2014 1:16 PM, Thierry Reding wrote:
> On Fri, May 09, 2014 at 01:31:22PM +0530, sonika.jindal@intel.com wrote:
>> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
>>
>> The rotation property stuff should be standardized among all drivers.
>> Move the bits to drm_crtc.h from omap_drv.h.
>>
>> Cc: David Airlie <airlied@linux.ie>
>> Cc: Tomi Valkeinen <tomi.valkeinen@ti.com>
>> Cc: Dave Airlie <airlied@redhat.com>
>> Cc: Rob Clark <robdclark@gmail.com>
>> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
>> Cc: Archit Taneja <archit@ti.com>
>> Cc: dri-devel@lists.freedesktop.org
>> Cc: linux-kernel@vger.kernel.org
>> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
>> Reviewed-by: Sagar Kamble <sagar.a.kamble@intel.com>
>> Tested-by: Sagar Kamble <sagar.a.kamble@intel.com>
>
> This is missing your Signed-off-by line. Also the patch subject says
> 01/12, where are the other eleven patches?
>
> Thierry
>

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-05-13  8:11 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <1399622493-31701-1-git-send-email-sonika.jindal@intel.com>
2014-05-09  8:01 ` [PATCH 01/12] drm: Move DRM_ROTATE bits out of omapdrm into drm_crtc.h sonika.jindal
2014-05-09  9:21   ` Tomi Valkeinen
2014-05-13  7:46   ` Thierry Reding
2014-05-13  8:11     ` Jindal, Sonika

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).