| Message ID | 1498041253-16426-3-git-send-email-shashank.sharma@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On 06/21/2017 12:34 PM, Shashank Sharma wrote: > This patch adds a bool variable (ycbcr_420_allowed) in the drm connector > structure. While handling the EDID from HDMI 2.0 sinks, its important to > know if the source is capable of handling YCBCR 420 outputs or not, so that > a lot of work can be done/bypassed based on this information. One such > example is adding YCBCR420 only modes. > > If the driver knows that this source is not HDMI 2.0 capable, it will > not add YCBCR420-only modes while adding EDID modes, and will prevent > any runtime modeset failures. > > This variable will be initialized from I915 driver in the next patch > and will be used in the EDID handling for HDMI 2.0 specific features, > in this same series. > > V3: introduced the new variable > V4: changed variable name from is_hdmi2_src to ycbcr_420_allowed (Ville) > > Signed-off-by: Shashank Sharma <shashank.sharma@intel.com> > --- > include/drm/drm_connector.h | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h > index d8bb25f..7493fd3 100644 > --- a/include/drm/drm_connector.h > +++ b/include/drm/drm_connector.h > @@ -728,6 +728,15 @@ struct drm_connector { > bool interlace_allowed; > bool doublescan_allowed; > bool stereo_allowed; > + > + /** > + * @ycbcr_420_allowed : This bool indicates if this connector is > + * capable of handling YCBCR 420 output. While parsing the EDID > + * blocks, its very helpful to know, if the source is capable of > + * handling YCBCR 420 outputs. > + */ > + bool ycbcr_420_allowed; > + > /** > * @registered: Is this connector exposed (registered) with userspace? > * Protected by @mutex. > Reviewed-by: Neil Armstrong <narmstrong@baylibre.com>
diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h index d8bb25f..7493fd3 100644 --- a/include/drm/drm_connector.h +++ b/include/drm/drm_connector.h @@ -728,6 +728,15 @@ struct drm_connector { bool interlace_allowed; bool doublescan_allowed; bool stereo_allowed; + + /** + * @ycbcr_420_allowed : This bool indicates if this connector is + * capable of handling YCBCR 420 output. While parsing the EDID + * blocks, its very helpful to know, if the source is capable of + * handling YCBCR 420 outputs. + */ + bool ycbcr_420_allowed; + /** * @registered: Is this connector exposed (registered) with userspace? * Protected by @mutex.
This patch adds a bool variable (ycbcr_420_allowed) in the drm connector structure. While handling the EDID from HDMI 2.0 sinks, its important to know if the source is capable of handling YCBCR 420 outputs or not, so that a lot of work can be done/bypassed based on this information. One such example is adding YCBCR420 only modes. If the driver knows that this source is not HDMI 2.0 capable, it will not add YCBCR420-only modes while adding EDID modes, and will prevent any runtime modeset failures. This variable will be initialized from I915 driver in the next patch and will be used in the EDID handling for HDMI 2.0 specific features, in this same series. V3: introduced the new variable V4: changed variable name from is_hdmi2_src to ycbcr_420_allowed (Ville) Signed-off-by: Shashank Sharma <shashank.sharma@intel.com> --- include/drm/drm_connector.h | 9 +++++++++ 1 file changed, 9 insertions(+)