Re: [RFC 2/5] drm: uapi: Add HDMI 2.0 aspect ratio flags and HDMI 2.0+ mode flag
Hi Ville, On 23-03-2017 19:00, Ville Syrjälä wrote: > On Thu, Mar 23, 2017 at 06:54:52PM +, Jose Abreu wrote: >> Hi Ville, >> >> >> On 23-03-2017 15:16, Ville Syrjälä wrote: >>> On Wed, Mar 22, 2017 at 05:35:58PM +, Jose Abreu wrote: Add the HDMI 2.0 aspect ratio flags (64:27 and 256:135) and a new flag which will signal userspace that this is a HDMI 2.0+ mode. It is expected that these new flags will not be exported to userspace unless client asks to. >>> W.r.t. the aspect ratio userspace story, there was some discussion >>> on #dri-devel recently about extending the xrandr protocol for 3d stereo >>> support. If that is going to happen then it might make sense to try >>> and get the aspect ratio stuff also included at the same time so >>> that we would only have to deal with one protocol bump. >> Hmm, are you saying to just use one knob for 3d and the aspect >> ratios? > No. I'm saying that extending xrandr with either means a protocol bump. > But if we do both at the same time we just need one bump. Ok. Anything in this series that you think can start helping contribute for that? Best regards, Jose Miguel Abreu > >> Aren't there already userspace consumers of the 3d knob? > Not that I know of. And there's definitely nothing in xrandr for this > stuff, so any consumer would have to be something non-x11 based. > >> Best regards, >> Jose Miguel Abreu >> Signed-off-by: Jose AbreuCc: Carlos Palminha Cc: dri-devel@lists.freedesktop.org --- include/uapi/drm/drm_mode.h | 9 + 1 file changed, 9 insertions(+) diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h index 8c67fc0..62e679c 100644 --- a/include/uapi/drm/drm_mode.h +++ b/include/uapi/drm/drm_mode.h @@ -89,6 +89,8 @@ #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_27 3 +#define DRM_MODE_PICTURE_ASPECT_256_135 4 /* Aspect ratio flag bitmask (4 bits 22:19) */ #define DRM_MODE_FLAG_PIC_AR_MASK (0x0F<<19) @@ -98,6 +100,13 @@ (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) + +/* HDMI 2.0+ mode flag: will only be set if client supports it */ +#define DRM_MODE_FLAG_HDMI2 (1<<23) /* DPMS flags */ /* bit compatible with the xorg definitions. */ -- 1.9.1 ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.freedesktop.org_mailman_listinfo_dri-2Ddevel=DwIDAw=DPL6_X_6JkXFx7AXWqB0tg=WHDsc6kcWAl4i96Vm5hJ_19IJiuxx_p_Rzo2g-uHDKw=BBAO8Rra0sW4KcCLB5lMOfONxg0v9pfUSaDNr5WDh2w=p6tAeWYwiSDL7BonKeijv2zmweBF9eW7xV7OX5e5gG4= ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [RFC 2/5] drm: uapi: Add HDMI 2.0 aspect ratio flags and HDMI 2.0+ mode flag
Hi Ville, On 23-03-2017 15:16, Ville Syrjälä wrote: > On Wed, Mar 22, 2017 at 05:35:58PM +, Jose Abreu wrote: >> Add the HDMI 2.0 aspect ratio flags (64:27 and 256:135) and a new >> flag which will signal userspace that this is a HDMI 2.0+ mode. It >> is expected that these new flags will not be exported to userspace >> unless client asks to. > W.r.t. the aspect ratio userspace story, there was some discussion > on #dri-devel recently about extending the xrandr protocol for 3d stereo > support. If that is going to happen then it might make sense to try > and get the aspect ratio stuff also included at the same time so > that we would only have to deal with one protocol bump. Hmm, are you saying to just use one knob for 3d and the aspect ratios? Aren't there already userspace consumers of the 3d knob? Best regards, Jose Miguel Abreu > >> Signed-off-by: Jose Abreu>> Cc: Carlos Palminha >> Cc: dri-devel@lists.freedesktop.org >> --- >> include/uapi/drm/drm_mode.h | 9 + >> 1 file changed, 9 insertions(+) >> >> diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h >> index 8c67fc0..62e679c 100644 >> --- a/include/uapi/drm/drm_mode.h >> +++ b/include/uapi/drm/drm_mode.h >> @@ -89,6 +89,8 @@ >> #define DRM_MODE_PICTURE_ASPECT_NONE0 >> #define DRM_MODE_PICTURE_ASPECT_4_3 1 >> #define DRM_MODE_PICTURE_ASPECT_16_92 >> +#define DRM_MODE_PICTURE_ASPECT_64_27 3 >> +#define DRM_MODE_PICTURE_ASPECT_256_135 4 >> >> /* Aspect ratio flag bitmask (4 bits 22:19) */ >> #define DRM_MODE_FLAG_PIC_AR_MASK (0x0F<<19) >> @@ -98,6 +100,13 @@ >> (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) >> + >> +/* HDMI 2.0+ mode flag: will only be set if client supports it */ >> +#define DRM_MODE_FLAG_HDMI2 (1<<23) >> >> /* DPMS flags */ >> /* bit compatible with the xorg definitions. */ >> -- >> 1.9.1 >> >> >> ___ >> dri-devel mailing list >> dri-devel@lists.freedesktop.org >> https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.freedesktop.org_mailman_listinfo_dri-2Ddevel=DwIDAw=DPL6_X_6JkXFx7AXWqB0tg=WHDsc6kcWAl4i96Vm5hJ_19IJiuxx_p_Rzo2g-uHDKw=BBAO8Rra0sW4KcCLB5lMOfONxg0v9pfUSaDNr5WDh2w=p6tAeWYwiSDL7BonKeijv2zmweBF9eW7xV7OX5e5gG4= >> ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [RFC 2/5] drm: uapi: Add HDMI 2.0 aspect ratio flags and HDMI 2.0+ mode flag
On Thu, Mar 23, 2017 at 06:54:52PM +, Jose Abreu wrote: > Hi Ville, > > > On 23-03-2017 15:16, Ville Syrjälä wrote: > > On Wed, Mar 22, 2017 at 05:35:58PM +, Jose Abreu wrote: > >> Add the HDMI 2.0 aspect ratio flags (64:27 and 256:135) and a new > >> flag which will signal userspace that this is a HDMI 2.0+ mode. It > >> is expected that these new flags will not be exported to userspace > >> unless client asks to. > > W.r.t. the aspect ratio userspace story, there was some discussion > > on #dri-devel recently about extending the xrandr protocol for 3d stereo > > support. If that is going to happen then it might make sense to try > > and get the aspect ratio stuff also included at the same time so > > that we would only have to deal with one protocol bump. > > Hmm, are you saying to just use one knob for 3d and the aspect > ratios? No. I'm saying that extending xrandr with either means a protocol bump. But if we do both at the same time we just need one bump. > Aren't there already userspace consumers of the 3d knob? Not that I know of. And there's definitely nothing in xrandr for this stuff, so any consumer would have to be something non-x11 based. > > Best regards, > Jose Miguel Abreu > > > > >> Signed-off-by: Jose Abreu> >> Cc: Carlos Palminha > >> Cc: dri-devel@lists.freedesktop.org > >> --- > >> include/uapi/drm/drm_mode.h | 9 + > >> 1 file changed, 9 insertions(+) > >> > >> diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h > >> index 8c67fc0..62e679c 100644 > >> --- a/include/uapi/drm/drm_mode.h > >> +++ b/include/uapi/drm/drm_mode.h > >> @@ -89,6 +89,8 @@ > >> #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_27 3 > >> +#define DRM_MODE_PICTURE_ASPECT_256_135 4 > >> > >> /* Aspect ratio flag bitmask (4 bits 22:19) */ > >> #define DRM_MODE_FLAG_PIC_AR_MASK (0x0F<<19) > >> @@ -98,6 +100,13 @@ > >>(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) > >> + > >> +/* HDMI 2.0+ mode flag: will only be set if client supports it */ > >> +#define DRM_MODE_FLAG_HDMI2 (1<<23) > >> > >> /* DPMS flags */ > >> /* bit compatible with the xorg definitions. */ > >> -- > >> 1.9.1 > >> > >> > >> ___ > >> dri-devel mailing list > >> dri-devel@lists.freedesktop.org > >> https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.freedesktop.org_mailman_listinfo_dri-2Ddevel=DwIDAw=DPL6_X_6JkXFx7AXWqB0tg=WHDsc6kcWAl4i96Vm5hJ_19IJiuxx_p_Rzo2g-uHDKw=BBAO8Rra0sW4KcCLB5lMOfONxg0v9pfUSaDNr5WDh2w=p6tAeWYwiSDL7BonKeijv2zmweBF9eW7xV7OX5e5gG4= > >> -- Ville Syrjälä Intel OTC ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [RFC 2/5] drm: uapi: Add HDMI 2.0 aspect ratio flags and HDMI 2.0+ mode flag
On Wed, Mar 22, 2017 at 05:35:58PM +, Jose Abreu wrote: > Add the HDMI 2.0 aspect ratio flags (64:27 and 256:135) and a new > flag which will signal userspace that this is a HDMI 2.0+ mode. It > is expected that these new flags will not be exported to userspace > unless client asks to. W.r.t. the aspect ratio userspace story, there was some discussion on #dri-devel recently about extending the xrandr protocol for 3d stereo support. If that is going to happen then it might make sense to try and get the aspect ratio stuff also included at the same time so that we would only have to deal with one protocol bump. > > Signed-off-by: Jose Abreu> Cc: Carlos Palminha > Cc: dri-devel@lists.freedesktop.org > --- > include/uapi/drm/drm_mode.h | 9 + > 1 file changed, 9 insertions(+) > > diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h > index 8c67fc0..62e679c 100644 > --- a/include/uapi/drm/drm_mode.h > +++ b/include/uapi/drm/drm_mode.h > @@ -89,6 +89,8 @@ > #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 > > /* Aspect ratio flag bitmask (4 bits 22:19) */ > #define DRM_MODE_FLAG_PIC_AR_MASK(0x0F<<19) > @@ -98,6 +100,13 @@ > (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) > + > +/* HDMI 2.0+ mode flag: will only be set if client supports it */ > +#define DRM_MODE_FLAG_HDMI2 (1<<23) > > /* DPMS flags */ > /* bit compatible with the xorg definitions. */ > -- > 1.9.1 > > > ___ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Ville Syrjälä Intel OTC ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
[RFC 2/5] drm: uapi: Add HDMI 2.0 aspect ratio flags and HDMI 2.0+ mode flag
Add the HDMI 2.0 aspect ratio flags (64:27 and 256:135) and a new flag which will signal userspace that this is a HDMI 2.0+ mode. It is expected that these new flags will not be exported to userspace unless client asks to. Signed-off-by: Jose AbreuCc: Carlos Palminha Cc: dri-devel@lists.freedesktop.org --- include/uapi/drm/drm_mode.h | 9 + 1 file changed, 9 insertions(+) diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h index 8c67fc0..62e679c 100644 --- a/include/uapi/drm/drm_mode.h +++ b/include/uapi/drm/drm_mode.h @@ -89,6 +89,8 @@ #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) @@ -98,6 +100,13 @@ (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) + +/* HDMI 2.0+ mode flag: will only be set if client supports it */ +#define DRM_MODE_FLAG_HDMI2(1<<23) /* DPMS flags */ /* bit compatible with the xorg definitions. */ -- 1.9.1 ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel