Re: [Intel-gfx] [PATCH v2 4/4] drm: Add and handle new aspect ratios in DRM layer
On Mon, Oct 17, 2016 at 08:02:49AM +0200, Daniel Vetter wrote: > On Thu, Oct 13, 2016 at 10:28:14AM -0700, Jim Bride wrote: > > On Tue, Aug 09, 2016 at 08:25:50PM +0530, Shashank Sharma wrote: > > > HDMI 2.0/CEA-861-F introduces two new aspect ratios: > > > - 64:27 > > > - 256:135 > > > > > > This patch: > > > - Adds new DRM flags for to represent these new aspect ratios. > > > - Adds new cases to handle these aspect ratios while converting > > > from user->kernel mode or vise versa. > > > > > > V2: Rebase > > > > > > Signed-off-by: Shashank Sharma > > > Reviewed-by: Sean Paul > > > Cc: Daniel Vetter > > > Cc: Emil Velikov > > > --- > > > drivers/gpu/drm/drm_modes.c | 12 > > > include/uapi/drm/drm_mode.h | 6 ++ > > > 2 files changed, 18 insertions(+) > > > > > > diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c > > > index 9d8f00d..ed1b07b 100644 > > > --- a/drivers/gpu/drm/drm_modes.c > > > +++ b/drivers/gpu/drm/drm_modes.c > > > @@ -1481,6 +1481,12 @@ void drm_mode_convert_to_umode(struct > > > drm_mode_modeinfo *out, > > > case HDMI_PICTURE_ASPECT_16_9: > > > out->flags |= DRM_MODE_FLAG_PIC_AR_16_9; > > > break; > > > + case HDMI_PICTURE_ASPECT_64_27: > > > + out->flags |= DRM_MODE_FLAG_PIC_AR_64_27; > > > + break; > > > + case DRM_MODE_PICTURE_ASPECT_256_135: > > > + out->flags |= DRM_MODE_FLAG_PIC_AR_256_135; > > > + break; > > > case HDMI_PICTURE_ASPECT_RESERVED: > > > default: > > > out->flags |= DRM_MODE_FLAG_PIC_AR_NONE; > > > @@ -1542,6 +1548,12 @@ int drm_mode_convert_umode(struct drm_display_mode > > > *out, > > > case DRM_MODE_FLAG_PIC_AR_16_9: > > > out->picture_aspect_ratio |= HDMI_PICTURE_ASPECT_16_9; > > > break; > > > + case DRM_MODE_FLAG_PIC_AR_64_27: > > > + out->picture_aspect_ratio |= HDMI_PICTURE_ASPECT_64_27; > > > + break; > > > + case DRM_MODE_FLAG_PIC_AR_256_135: > > > + out->picture_aspect_ratio |= HDMI_PICTURE_ASPECT_256_135; > > > + break; > > > default: > > > out->picture_aspect_ratio = HDMI_PICTURE_ASPECT_NONE; > > > break; > > > diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h > > > index 77c869d6..4d3429b 100644 > > > --- a/include/uapi/drm/drm_mode.h > > > +++ b/include/uapi/drm/drm_mode.h > > > @@ -81,6 +81,8 @@ extern "C" { > > > #define DRM_MODE_PICTURE_ASPECT_NONE 0 > > > #define DRM_MODE_PICTURE_ASPECT_4_3 1 > > > #define DRM_MODE_PICTURE_ASPECT_16_9 2 > > > +#define DRM_MODE_PICTURE_ASPECT_64_273 > > > +#define DRM_MODE_PICTURE_ASPECT_256_135 4 > > > > Minor nit here, but in my tree the '4' above doesn't line up > > with the three previous definitions. I downloaded the series as > > a mbox from patchwork. > > r-b with that bikeshed addressed or not? Also any reason you didn't r-b > patch 3? I'm happy with both 3 and 4, so sure. They both had r-b already by Sean so I didn't bother tacking my own r-b on there. Jim > -Daniel > -- > Daniel Vetter > Software Engineer, Intel Corporation > http://blog.ffwll.ch ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH v2 4/4] drm: Add and handle new aspect ratios in DRM layer
> r-b with that bikeshed addressed or not? Hello Daniel, I have already acknowledged this comment, and I am publishing next set today with rebase, addressing this comment. So you can consider this one done. Regards Shashank -Original Message- From: Daniel Vetter [mailto:daniel.vet...@ffwll.ch] On Behalf Of Daniel Vetter Sent: Monday, October 17, 2016 11:33 AM To: Jim Bride Cc: Sharma, Shashank ; dri-de...@lists.freedesktop.org; seanp...@chromium.org; jose.ab...@synopsys.com; Daniel Vetter ; intel-gfx@lists.freedesktop.org; Vetter, Daniel Subject: Re: [Intel-gfx] [PATCH v2 4/4] drm: Add and handle new aspect ratios in DRM layer On Thu, Oct 13, 2016 at 10:28:14AM -0700, Jim Bride wrote: > On Tue, Aug 09, 2016 at 08:25:50PM +0530, Shashank Sharma wrote: > > HDMI 2.0/CEA-861-F introduces two new aspect ratios: > > - 64:27 > > - 256:135 > > > > This patch: > > - Adds new DRM flags for to represent these new aspect ratios. > > - Adds new cases to handle these aspect ratios while converting > > from user->kernel mode or vise versa. > > > > V2: Rebase > > > > Signed-off-by: Shashank Sharma > > Reviewed-by: Sean Paul > > Cc: Daniel Vetter > > Cc: Emil Velikov > > --- > > drivers/gpu/drm/drm_modes.c | 12 > > include/uapi/drm/drm_mode.h | 6 ++ > > 2 files changed, 18 insertions(+) > > > > diff --git a/drivers/gpu/drm/drm_modes.c > > b/drivers/gpu/drm/drm_modes.c index 9d8f00d..ed1b07b 100644 > > --- a/drivers/gpu/drm/drm_modes.c > > +++ b/drivers/gpu/drm/drm_modes.c > > @@ -1481,6 +1481,12 @@ void drm_mode_convert_to_umode(struct > > drm_mode_modeinfo *out, > > case HDMI_PICTURE_ASPECT_16_9: > > out->flags |= DRM_MODE_FLAG_PIC_AR_16_9; > > break; > > + case HDMI_PICTURE_ASPECT_64_27: > > + out->flags |= DRM_MODE_FLAG_PIC_AR_64_27; > > + break; > > + case DRM_MODE_PICTURE_ASPECT_256_135: > > + out->flags |= DRM_MODE_FLAG_PIC_AR_256_135; > > + break; > > case HDMI_PICTURE_ASPECT_RESERVED: > > default: > > out->flags |= DRM_MODE_FLAG_PIC_AR_NONE; @@ -1542,6 +1548,12 @@ > > int drm_mode_convert_umode(struct drm_display_mode *out, > > case DRM_MODE_FLAG_PIC_AR_16_9: > > out->picture_aspect_ratio |= HDMI_PICTURE_ASPECT_16_9; > > break; > > + case DRM_MODE_FLAG_PIC_AR_64_27: > > + out->picture_aspect_ratio |= HDMI_PICTURE_ASPECT_64_27; > > + break; > > + case DRM_MODE_FLAG_PIC_AR_256_135: > > + out->picture_aspect_ratio |= HDMI_PICTURE_ASPECT_256_135; > > + break; > > default: > > out->picture_aspect_ratio = HDMI_PICTURE_ASPECT_NONE; > > break; > > diff --git a/include/uapi/drm/drm_mode.h > > b/include/uapi/drm/drm_mode.h index 77c869d6..4d3429b 100644 > > --- a/include/uapi/drm/drm_mode.h > > +++ b/include/uapi/drm/drm_mode.h > > @@ -81,6 +81,8 @@ extern "C" { > > #define DRM_MODE_PICTURE_ASPECT_NONE 0 > > #define DRM_MODE_PICTURE_ASPECT_4_31 > > #define DRM_MODE_PICTURE_ASPECT_16_9 2 > > +#define DRM_MODE_PICTURE_ASPECT_64_27 3 > > +#define DRM_MODE_PICTURE_ASPECT_256_1354 > > Minor nit here, but in my tree the '4' above doesn't line up with the > three previous definitions. I downloaded the series as a mbox from > patchwork. r-b with that bikeshed addressed or not? Also any reason you didn't r-b patch 3? -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH v2 4/4] drm: Add and handle new aspect ratios in DRM layer
On Thu, Oct 13, 2016 at 10:28:14AM -0700, Jim Bride wrote: > On Tue, Aug 09, 2016 at 08:25:50PM +0530, Shashank Sharma wrote: > > HDMI 2.0/CEA-861-F introduces two new aspect ratios: > > - 64:27 > > - 256:135 > > > > This patch: > > - Adds new DRM flags for to represent these new aspect ratios. > > - Adds new cases to handle these aspect ratios while converting > > from user->kernel mode or vise versa. > > > > V2: Rebase > > > > Signed-off-by: Shashank Sharma > > Reviewed-by: Sean Paul > > Cc: Daniel Vetter > > Cc: Emil Velikov > > --- > > drivers/gpu/drm/drm_modes.c | 12 > > include/uapi/drm/drm_mode.h | 6 ++ > > 2 files changed, 18 insertions(+) > > > > diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c > > index 9d8f00d..ed1b07b 100644 > > --- a/drivers/gpu/drm/drm_modes.c > > +++ b/drivers/gpu/drm/drm_modes.c > > @@ -1481,6 +1481,12 @@ void drm_mode_convert_to_umode(struct > > drm_mode_modeinfo *out, > > case HDMI_PICTURE_ASPECT_16_9: > > out->flags |= DRM_MODE_FLAG_PIC_AR_16_9; > > break; > > + case HDMI_PICTURE_ASPECT_64_27: > > + out->flags |= DRM_MODE_FLAG_PIC_AR_64_27; > > + break; > > + case DRM_MODE_PICTURE_ASPECT_256_135: > > + out->flags |= DRM_MODE_FLAG_PIC_AR_256_135; > > + break; > > case HDMI_PICTURE_ASPECT_RESERVED: > > default: > > out->flags |= DRM_MODE_FLAG_PIC_AR_NONE; > > @@ -1542,6 +1548,12 @@ int drm_mode_convert_umode(struct drm_display_mode > > *out, > > case DRM_MODE_FLAG_PIC_AR_16_9: > > out->picture_aspect_ratio |= HDMI_PICTURE_ASPECT_16_9; > > break; > > + case DRM_MODE_FLAG_PIC_AR_64_27: > > + out->picture_aspect_ratio |= HDMI_PICTURE_ASPECT_64_27; > > + break; > > + case DRM_MODE_FLAG_PIC_AR_256_135: > > + out->picture_aspect_ratio |= HDMI_PICTURE_ASPECT_256_135; > > + break; > > default: > > out->picture_aspect_ratio = HDMI_PICTURE_ASPECT_NONE; > > break; > > diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h > > index 77c869d6..4d3429b 100644 > > --- a/include/uapi/drm/drm_mode.h > > +++ b/include/uapi/drm/drm_mode.h > > @@ -81,6 +81,8 @@ extern "C" { > > #define DRM_MODE_PICTURE_ASPECT_NONE 0 > > #define DRM_MODE_PICTURE_ASPECT_4_31 > > #define DRM_MODE_PICTURE_ASPECT_16_9 2 > > +#define DRM_MODE_PICTURE_ASPECT_64_27 3 > > +#define DRM_MODE_PICTURE_ASPECT_256_1354 > > Minor nit here, but in my tree the '4' above doesn't line up > with the three previous definitions. I downloaded the series as > a mbox from patchwork. r-b with that bikeshed addressed or not? Also any reason you didn't r-b patch 3? -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH v2 4/4] drm: Add and handle new aspect ratios in DRM layer
On Tue, Aug 09, 2016 at 08:25:50PM +0530, Shashank Sharma wrote: > HDMI 2.0/CEA-861-F introduces two new aspect ratios: > - 64:27 > - 256:135 > > This patch: > - Adds new DRM flags for to represent these new aspect ratios. > - Adds new cases to handle these aspect ratios while converting > from user->kernel mode or vise versa. > > V2: Rebase > > Signed-off-by: Shashank Sharma > Reviewed-by: Sean Paul > Cc: Daniel Vetter > Cc: Emil Velikov > --- > drivers/gpu/drm/drm_modes.c | 12 > include/uapi/drm/drm_mode.h | 6 ++ > 2 files changed, 18 insertions(+) > > diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c > index 9d8f00d..ed1b07b 100644 > --- a/drivers/gpu/drm/drm_modes.c > +++ b/drivers/gpu/drm/drm_modes.c > @@ -1481,6 +1481,12 @@ void drm_mode_convert_to_umode(struct > drm_mode_modeinfo *out, > case HDMI_PICTURE_ASPECT_16_9: > out->flags |= DRM_MODE_FLAG_PIC_AR_16_9; > break; > + case HDMI_PICTURE_ASPECT_64_27: > + out->flags |= DRM_MODE_FLAG_PIC_AR_64_27; > + break; > + case DRM_MODE_PICTURE_ASPECT_256_135: > + out->flags |= DRM_MODE_FLAG_PIC_AR_256_135; > + break; > case HDMI_PICTURE_ASPECT_RESERVED: > default: > out->flags |= DRM_MODE_FLAG_PIC_AR_NONE; > @@ -1542,6 +1548,12 @@ int drm_mode_convert_umode(struct drm_display_mode > *out, > case DRM_MODE_FLAG_PIC_AR_16_9: > out->picture_aspect_ratio |= HDMI_PICTURE_ASPECT_16_9; > break; > + case DRM_MODE_FLAG_PIC_AR_64_27: > + out->picture_aspect_ratio |= HDMI_PICTURE_ASPECT_64_27; > + break; > + case DRM_MODE_FLAG_PIC_AR_256_135: > + out->picture_aspect_ratio |= HDMI_PICTURE_ASPECT_256_135; > + break; > default: > out->picture_aspect_ratio = HDMI_PICTURE_ASPECT_NONE; > break; > diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h > index 77c869d6..4d3429b 100644 > --- a/include/uapi/drm/drm_mode.h > +++ b/include/uapi/drm/drm_mode.h > @@ -81,6 +81,8 @@ extern "C" { > #define DRM_MODE_PICTURE_ASPECT_NONE 0 > #define DRM_MODE_PICTURE_ASPECT_4_3 1 > #define DRM_MODE_PICTURE_ASPECT_16_9 2 > +#define DRM_MODE_PICTURE_ASPECT_64_273 > +#define DRM_MODE_PICTURE_ASPECT_256_135 4 Minor nit here, but in my tree the '4' above doesn't line up with the three previous definitions. I downloaded the series as a mbox from patchwork. Jim > > /* Aspect ratio flag bitmask (4 bits 22:19) */ > #define DRM_MODE_FLAG_PIC_AR_MASK(0x0F<<19) > @@ -90,6 +92,10 @@ extern "C" { > (DRM_MODE_PICTURE_ASPECT_4_3<<19) > #define DRM_MODE_FLAG_PIC_AR_16_9 \ > (DRM_MODE_PICTURE_ASPECT_16_9<<19) > +#define DRM_MODE_FLAG_PIC_AR_64_27 \ > + (DRM_MODE_PICTURE_ASPECT_64_27<<19) > +#define DRM_MODE_FLAG_PIC_AR_256_135 \ > + (DRM_MODE_PICTURE_ASPECT_256_135<<19) > > /* DPMS flags */ > /* bit compatible with the xorg definitions. */ > -- > 1.9.1 > > ___ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
[Intel-gfx] [PATCH v2 4/4] drm: Add and handle new aspect ratios in DRM layer
HDMI 2.0/CEA-861-F introduces two new aspect ratios: - 64:27 - 256:135 This patch: - Adds new DRM flags for to represent these new aspect ratios. - Adds new cases to handle these aspect ratios while converting from user->kernel mode or vise versa. V2: Rebase Signed-off-by: Shashank Sharma Reviewed-by: Sean Paul Cc: Daniel Vetter Cc: Emil Velikov --- drivers/gpu/drm/drm_modes.c | 12 include/uapi/drm/drm_mode.h | 6 ++ 2 files changed, 18 insertions(+) diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c index 9d8f00d..ed1b07b 100644 --- a/drivers/gpu/drm/drm_modes.c +++ b/drivers/gpu/drm/drm_modes.c @@ -1481,6 +1481,12 @@ void drm_mode_convert_to_umode(struct drm_mode_modeinfo *out, case HDMI_PICTURE_ASPECT_16_9: out->flags |= DRM_MODE_FLAG_PIC_AR_16_9; break; + case HDMI_PICTURE_ASPECT_64_27: + out->flags |= DRM_MODE_FLAG_PIC_AR_64_27; + break; + case DRM_MODE_PICTURE_ASPECT_256_135: + out->flags |= DRM_MODE_FLAG_PIC_AR_256_135; + break; case HDMI_PICTURE_ASPECT_RESERVED: default: out->flags |= DRM_MODE_FLAG_PIC_AR_NONE; @@ -1542,6 +1548,12 @@ int drm_mode_convert_umode(struct drm_display_mode *out, case DRM_MODE_FLAG_PIC_AR_16_9: out->picture_aspect_ratio |= HDMI_PICTURE_ASPECT_16_9; break; + case DRM_MODE_FLAG_PIC_AR_64_27: + out->picture_aspect_ratio |= HDMI_PICTURE_ASPECT_64_27; + break; + case DRM_MODE_FLAG_PIC_AR_256_135: + out->picture_aspect_ratio |= HDMI_PICTURE_ASPECT_256_135; + break; default: out->picture_aspect_ratio = HDMI_PICTURE_ASPECT_NONE; break; diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h index 77c869d6..4d3429b 100644 --- a/include/uapi/drm/drm_mode.h +++ b/include/uapi/drm/drm_mode.h @@ -81,6 +81,8 @@ extern "C" { #define DRM_MODE_PICTURE_ASPECT_NONE 0 #define DRM_MODE_PICTURE_ASPECT_4_31 #define DRM_MODE_PICTURE_ASPECT_16_9 2 +#define DRM_MODE_PICTURE_ASPECT_64_27 3 +#define DRM_MODE_PICTURE_ASPECT_256_1354 /* Aspect ratio flag bitmask (4 bits 22:19) */ #define DRM_MODE_FLAG_PIC_AR_MASK (0x0F<<19) @@ -90,6 +92,10 @@ extern "C" { (DRM_MODE_PICTURE_ASPECT_4_3<<19) #define DRM_MODE_FLAG_PIC_AR_16_9 \ (DRM_MODE_PICTURE_ASPECT_16_9<<19) +#define DRM_MODE_FLAG_PIC_AR_64_27 \ + (DRM_MODE_PICTURE_ASPECT_64_27<<19) +#define DRM_MODE_FLAG_PIC_AR_256_135 \ + (DRM_MODE_PICTURE_ASPECT_256_135<<19) /* DPMS flags */ /* bit compatible with the xorg definitions. */ -- 1.9.1 ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx