Thanks Shashank.

When they get merged and propagated to nightly I merge the last one in
internal.

On Thu, 2016-03-24 at 12:57 +0000, Sharma, Shashank wrote:
> Ok, will do it for other patches too.
> 
> Regards
> Shashank
> -----Original Message-----
> From: Jani Nikula [mailto:[email protected]] 
> Sent: Thursday, March 24, 2016 6:26 PM
> To: Sharma, Shashank; Vivi, Rodrigo
> Cc: [email protected]
> Subject: Re: [Intel-gfx] [PATCH v2 1/5] drm: add picture aspect ratio
> flags
> 
> On Thu, 24 Mar 2016, Shashank Sharma <[email protected]>
> wrote:
> > This patch adds drm flag bits for aspect ratio information
> > 
> > Currently drm flag bits don't have field for mode's picture aspect 
> > ratio. This field will help the driver to pick mode with right
> > aspect 
> > ratio, and help in setting right VIC field in avi infoframes.
> > 
> > Signed-off-by: Shashank Sharma <[email protected]>
> > ---
> >  include/uapi/drm/drm_mode.h | 18 +++++++++++++-----
> 
> Please use scripts/get_maintainer.pl on the patch to figure out who
> it should be sent to. This file is not maintained by us, so we can't
> just merge it.
> 
> For example:
> 
> $ scripts/get_maintainer.pl 0001-drm-add-picture-aspect-ratio
> -flags.patch
> David Airlie <[email protected]> (maintainer:DRM DRIVERS) 
> [email protected] (open list:DRM DRIVERS) 
> [email protected] (open list)
> 
> Of course, since you ultimately aim for a change in drm/i915, you
> should also Cc: intel-gfx.
> 
> BR,
> Jani.
> 
> 
> >  1 file changed, 13 insertions(+), 5 deletions(-)
> > 
> > diff --git a/include/uapi/drm/drm_mode.h
> > b/include/uapi/drm/drm_mode.h 
> > index 50adb46..3389bd1 100644
> > --- a/include/uapi/drm/drm_mode.h
> > +++ b/include/uapi/drm/drm_mode.h
> > @@ -73,6 +73,19 @@
> >  #define  DRM_MODE_FLAG_3D_TOP_AND_BOTTOM   (7<<14)
> >  #define  DRM_MODE_FLAG_3D_SIDE_BY_SIDE_HALF        (8<<14)
> >  
> > +/* Picture aspect ratio options */
> > +#define DRM_MODE_PICTURE_ASPECT_NONE               0
> > +#define DRM_MODE_PICTURE_ASPECT_4_3                1
> > +#define DRM_MODE_PICTURE_ASPECT_16_9               2
> > +
> > +/* Aspect ratio flag bitmask (4 bits 21:19) */
> > +#define     DRM_MODE_FLAG_PARMASK          (0x0F<<19)
> > +#define  DRM_MODE_FLAG_PARNONE \
> > +           (DRM_MODE_PICTURE_ASPECT_NONE << 19) #define 
> >  DRM_MODE_FLAG_PAR4_3 
> > +\
> > +           (DRM_MODE_PICTURE_ASPECT_4_3 << 19) #define 
> >  DRM_MODE_FLAG_PAR16_9 
> > +\
> > +           (DRM_MODE_PICTURE_ASPECT_16_9 << 19)
> >  
> >  /* DPMS flags */
> >  /* bit compatible with the xorg definitions. */ @@ -88,11 +101,6
> > @@
> >  #define DRM_MODE_SCALE_CENTER              2 /* Centered, no
> > scaling */
> >  #define DRM_MODE_SCALE_ASPECT              3 /* Full screen,
> > preserve aspect */
> >  
> > -/* Picture aspect ratio options */
> > -#define DRM_MODE_PICTURE_ASPECT_NONE       0
> > -#define DRM_MODE_PICTURE_ASPECT_4_3        1
> > -#define DRM_MODE_PICTURE_ASPECT_16_9       2
> > -
> >  /* Dithering mode options */
> >  #define DRM_MODE_DITHERING_OFF     0
> >  #define DRM_MODE_DITHERING_ON      1
> 
> --
> Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to