Re: [Intel-gfx] [PATCH] drm: BIT(DRM_ROTATE_?) -> DRM_ROTATE_?

2016-07-28 Thread Joonas Lahtinen
This can be disregarded. -- Joonas Lahtinen Open Source Technology Center Intel Corporation ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Intel-gfx] [PATCH] drm: BIT(DRM_ROTATE_?) -> DRM_ROTATE_?

2016-07-28 Thread Joonas Lahtinen
Only property creation uses the rotation as an index, so convert the to figure the index when needed. v2: Use the new defines to build the _MASK defines (Sean) Cc: intel-gfx@lists.freedesktop.org Cc: linux-arm-...@vger.kernel.org Cc: freedr...@lists.freedesktop.org Cc: mal...@foss.arm.com Cc:

Re: [Intel-gfx] [PATCH] drm: BIT(DRM_ROTATE_?) -> DRM_ROTATE_?

2016-07-28 Thread Sean Paul
On Thu, Jul 28, 2016 at 2:44 AM, Joonas Lahtinen wrote: > On ti, 2016-07-26 at 10:49 -0400, Sean Paul wrote: >> On Mon, Jul 25, 2016 at 3:00 AM, Joonas Lahtinen >> wrote: >> > diff --git a/include/drm/drm_crtc.h

Re: [Intel-gfx] [PATCH] drm: BIT(DRM_ROTATE_?) -> DRM_ROTATE_?

2016-07-28 Thread Joonas Lahtinen
On ti, 2016-07-26 at 10:49 -0400, Sean Paul wrote: > On Mon, Jul 25, 2016 at 3:00 AM, Joonas Lahtinen > wrote: > > diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h > > index 3edeaf8..57bbc61 100644 > > --- a/include/drm/drm_crtc.h > > +++

Re: [Intel-gfx] [PATCH] drm: BIT(DRM_ROTATE_?) -> DRM_ROTATE_?

2016-07-26 Thread Sean Paul
On Mon, Jul 25, 2016 at 3:00 AM, Joonas Lahtinen wrote: > Only property creation uses the rotation as an index, so convert the > #define to the more used BIT(DRM_ROTATE_?) form and use __builtin_ffs > to figure the index when needed. > > Cc:

Re: [Intel-gfx] [PATCH] drm: BIT(DRM_ROTATE_?) -> DRM_ROTATE_?

2016-07-26 Thread Ville Syrjälä
On Mon, Jul 25, 2016 at 10:00:25AM +0300, Joonas Lahtinen wrote: > Only property creation uses the rotation as an index, so convert the > #define to the more used BIT(DRM_ROTATE_?) form and use __builtin_ffs > to figure the index when needed. > > Cc: intel-gfx@lists.freedesktop.org > Cc:

Re: [Intel-gfx] [PATCH] drm: BIT(DRM_ROTATE_?) -> DRM_ROTATE_?

2016-07-26 Thread Joonas Lahtinen
On ti, 2016-07-26 at 10:05 +0100, Liviu Dudau wrote: > > --- a/drivers/gpu/drm/arm/malidp_planes.c > > +++ b/drivers/gpu/drm/arm/malidp_planes.c > > @@ -108,7 +108,7 @@ static int malidp_de_plane_check(struct drm_plane > > *plane, > >   return -EINVAL; > >   > >   /* packed RGB888 /

Re: [Intel-gfx] [PATCH] drm: BIT(DRM_ROTATE_?) -> DRM_ROTATE_?

2016-07-26 Thread Liviu Dudau
Hi Joonas, On Mon, Jul 25, 2016 at 10:00:25AM +0300, Joonas Lahtinen wrote: > Only property creation uses the rotation as an index, so convert the > #define to the more used BIT(DRM_ROTATE_?) form and use __builtin_ffs > to figure the index when needed. > > Cc: intel-gfx@lists.freedesktop.org >

[Intel-gfx] [PATCH] drm: BIT(DRM_ROTATE_?) -> DRM_ROTATE_?

2016-07-25 Thread Joonas Lahtinen
Only property creation uses the rotation as an index, so convert the #define to the more used BIT(DRM_ROTATE_?) form and use __builtin_ffs to figure the index when needed. Cc: intel-gfx@lists.freedesktop.org Cc: linux-arm-...@vger.kernel.org Cc: freedr...@lists.freedesktop.org Cc: David Airlie