From: "Sharma, Shashank" <shashank.sharma@intel.com>
To: Ilia Mirkin <imirkin@alum.mit.edu>
Cc: Jose Abreu <jose.abreu@synopsys.com>,
Alex Deucher <alexander.deucher@amd.com>,
Intel Graphics Development <intel-gfx@lists.freedesktop.org>,
"dri-devel@lists.freedesktop.org"
<dri-devel@lists.freedesktop.org>,
Daniel Vetter <daniel.vetter@intel.com>
Subject: Re: [PATCH v4 2/2] drm: Add HDMI 2.0 VIC support for AVI info-frames
Date: Thu, 23 Mar 2017 17:22:54 +0200 [thread overview]
Message-ID: <3eed384c-f584-991d-87fa-db7483e67f6f@intel.com> (raw)
In-Reply-To: <CAKb7Uvi7oHuXE4srzCze7SJn=hUYekC+QMq-rojHFOyvcYEAKg@mail.gmail.com>
Regards
Shashank
On 3/23/2017 5:17 PM, Ilia Mirkin wrote:
> On Thu, Mar 23, 2017 at 11:14 AM, Shashank Sharma
> <shashank.sharma@intel.com> wrote:
>> HDMI 1.4b support the CEA video modes as per range of CEA-861-D (VIC 1-64).
>> For any other mode, the VIC filed in AVI infoframes should be 0.
>> HDMI 2.0 sinks, support video modes range as per CEA-861-F spec, which is
>> extended to (VIC 1-107).
>>
>> This patch adds a bool input variable, which indicates if the connected
>> sink is a HDMI 2.0 sink or not. This will make sure that we don't pass a
>> HDMI 2.0 VIC to a HDMI 1.4 sink.
> Should this patch come before the patch which recognizes VICs 65+?
> Otherwise if only the first patch is applied but not this one, you
> could end up with the bad scenario. (As can happen in bisections, for
> example.)
I kindof agree, this could be the case, but also, for the correct
functionality, you should have the whole series
together.
>> This patch touches all drm drivers, who are callers of this function
>> drm_hdmi_avi_infoframe_from_display_mode but to make sure there is
>> no change in current behavior, is_hdmi2 is kept as false.
>>
>> In case of I915 driver, this patch checks the connector->display_info
>> to check if the connected display is HDMI 2.0.
>>
>> Cc: Ville Syrjala <ville.syrjala@linux.intel.com>
>> Cc: Jose Abreu <jose.abreu@synopsys.com>
>> Cc: Andrzej Hajda <a.hajda@samsung.com>
>> Cc: Alex Deucher <alexander.deucher@amd.com>
>> Cc: Daniel Vetter <daniel.vetter@intel.com>
>>
>> PS: This patch touches a few lines in few files, which were
>> already above 80 char, so checkpatch gives 80 char warning again.
>> - gpu/drm/omapdrm/omap_encoder.c
>> - gpu/drm/i915/intel_sdvo.c
>>
>> Signed-off-by: Shashank Sharma <shashank.sharma@intel.com>
>> ---
>> drivers/gpu/drm/amd/amdgpu/dce_v10_0.c | 2 +-
>> drivers/gpu/drm/amd/amdgpu/dce_v11_0.c | 2 +-
>> drivers/gpu/drm/amd/amdgpu/dce_v8_0.c | 2 +-
>> drivers/gpu/drm/bridge/analogix-anx78xx.c | 3 ++-
>> drivers/gpu/drm/bridge/sii902x.c | 2 +-
>> drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 2 +-
>> drivers/gpu/drm/drm_edid.c | 12 +++++++++++-
>> drivers/gpu/drm/exynos/exynos_hdmi.c | 2 +-
>> drivers/gpu/drm/i2c/tda998x_drv.c | 2 +-
>> drivers/gpu/drm/i915/intel_hdmi.c | 5 ++++-
>> drivers/gpu/drm/i915/intel_sdvo.c | 3 ++-
>> drivers/gpu/drm/mediatek/mtk_hdmi.c | 2 +-
>> drivers/gpu/drm/omapdrm/omap_encoder.c | 3 ++-
>> drivers/gpu/drm/radeon/radeon_audio.c | 2 +-
>> drivers/gpu/drm/rockchip/inno_hdmi.c | 2 +-
>> drivers/gpu/drm/sti/sti_hdmi.c | 2 +-
>> drivers/gpu/drm/tegra/hdmi.c | 2 +-
>> drivers/gpu/drm/tegra/sor.c | 2 +-
>> drivers/gpu/drm/vc4/vc4_hdmi.c | 2 +-
>> drivers/gpu/drm/zte/zx_hdmi.c | 2 +-
>> include/drm/drm_edid.h | 3 ++-
>> 21 files changed, 38 insertions(+), 21 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/amd/amdgpu/dce_v10_0.c b/drivers/gpu/drm/amd/amdgpu/dce_v10_0.c
>> index d4452d8..ab7a399 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/dce_v10_0.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/dce_v10_0.c
>> @@ -1877,7 +1877,7 @@ static void dce_v10_0_afmt_setmode(struct drm_encoder *encoder,
>> dce_v10_0_audio_write_sad_regs(encoder);
>> dce_v10_0_audio_write_latency_fields(encoder, mode);
>>
>> - err = drm_hdmi_avi_infoframe_from_display_mode(&frame, mode);
>> + err = drm_hdmi_avi_infoframe_from_display_mode(&frame, mode, false);
>> if (err < 0) {
>> DRM_ERROR("failed to setup AVI infoframe: %zd\n", err);
>> return;
>> diff --git a/drivers/gpu/drm/amd/amdgpu/dce_v11_0.c b/drivers/gpu/drm/amd/amdgpu/dce_v11_0.c
>> index 5b24e89..3c5fd83 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/dce_v11_0.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/dce_v11_0.c
>> @@ -1861,7 +1861,7 @@ static void dce_v11_0_afmt_setmode(struct drm_encoder *encoder,
>> dce_v11_0_audio_write_sad_regs(encoder);
>> dce_v11_0_audio_write_latency_fields(encoder, mode);
>>
>> - err = drm_hdmi_avi_infoframe_from_display_mode(&frame, mode);
>> + err = drm_hdmi_avi_infoframe_from_display_mode(&frame, mode, false);
>> if (err < 0) {
>> DRM_ERROR("failed to setup AVI infoframe: %zd\n", err);
>> return;
>> diff --git a/drivers/gpu/drm/amd/amdgpu/dce_v8_0.c b/drivers/gpu/drm/amd/amdgpu/dce_v8_0.c
>> index d2590d7..c564dca 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/dce_v8_0.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/dce_v8_0.c
>> @@ -1760,7 +1760,7 @@ static void dce_v8_0_afmt_setmode(struct drm_encoder *encoder,
>> dce_v8_0_audio_write_sad_regs(encoder);
>> dce_v8_0_audio_write_latency_fields(encoder, mode);
>>
>> - err = drm_hdmi_avi_infoframe_from_display_mode(&frame, mode);
>> + err = drm_hdmi_avi_infoframe_from_display_mode(&frame, mode, false);
>> if (err < 0) {
>> DRM_ERROR("failed to setup AVI infoframe: %zd\n", err);
>> return;
>> diff --git a/drivers/gpu/drm/bridge/analogix-anx78xx.c b/drivers/gpu/drm/bridge/analogix-anx78xx.c
>> index a2a8236..f9b77b8 100644
>> --- a/drivers/gpu/drm/bridge/analogix-anx78xx.c
>> +++ b/drivers/gpu/drm/bridge/analogix-anx78xx.c
>> @@ -1097,7 +1097,8 @@ static void anx78xx_bridge_mode_set(struct drm_bridge *bridge,
>>
>> mutex_lock(&anx78xx->lock);
>>
>> - err = drm_hdmi_avi_infoframe_from_display_mode(&frame, adjusted_mode);
>> + err = drm_hdmi_avi_infoframe_from_display_mode(&frame, adjusted_mode,
>> + false);
>> if (err) {
>> DRM_ERROR("Failed to setup AVI infoframe: %d\n", err);
>> goto unlock;
>> diff --git a/drivers/gpu/drm/bridge/sii902x.c b/drivers/gpu/drm/bridge/sii902x.c
>> index 9126d03..dcf15d7 100644
>> --- a/drivers/gpu/drm/bridge/sii902x.c
>> +++ b/drivers/gpu/drm/bridge/sii902x.c
>> @@ -269,7 +269,7 @@ static void sii902x_bridge_mode_set(struct drm_bridge *bridge,
>> if (ret)
>> return;
>>
>> - ret = drm_hdmi_avi_infoframe_from_display_mode(&frame, adj);
>> + ret = drm_hdmi_avi_infoframe_from_display_mode(&frame, adj, false);
>> if (ret < 0) {
>> DRM_ERROR("couldn't fill AVI infoframe\n");
>> return;
>> diff --git a/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c b/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c
>> index af93f7a..5ff2886 100644
>> --- a/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c
>> +++ b/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c
>> @@ -1146,7 +1146,7 @@ static void hdmi_config_AVI(struct dw_hdmi *hdmi, struct drm_display_mode *mode)
>> u8 val;
>>
>> /* Initialise info frame from DRM mode */
>> - drm_hdmi_avi_infoframe_from_display_mode(&frame, mode);
>> + drm_hdmi_avi_infoframe_from_display_mode(&frame, mode, false);
>>
>> if (hdmi->hdmi_data.enc_out_format == YCBCR444)
>> frame.colorspace = HDMI_COLORSPACE_YUV444;
>> diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
>> index 3b494c2..f9a4b08 100644
>> --- a/drivers/gpu/drm/drm_edid.c
>> +++ b/drivers/gpu/drm/drm_edid.c
>> @@ -4664,12 +4664,14 @@ EXPORT_SYMBOL(drm_set_preferred_mode);
>> * data from a DRM display mode
>> * @frame: HDMI AVI infoframe
>> * @mode: DRM display mode
>> + * @is_hdmi2: Sink is HDMI 2.0 compliant
>> *
>> * Return: 0 on success or a negative error code on failure.
>> */
>> int
>> drm_hdmi_avi_infoframe_from_display_mode(struct hdmi_avi_infoframe *frame,
>> - const struct drm_display_mode *mode)
>> + const struct drm_display_mode *mode,
>> + bool is_hdmi2)
>> {
>> int err;
>>
>> @@ -4685,6 +4687,14 @@ drm_hdmi_avi_infoframe_from_display_mode(struct hdmi_avi_infoframe *frame,
>>
>> frame->video_code = drm_match_cea_mode(mode);
>>
>> + /*
>> + * HDMI 1.4 VIC range: 1 <= VIC <= 64 (CEA-861-D) but
>> + * HDMI 2.0 VIC range: 1 <= VIC <= 107 (CEA-861-F). So we
>> + * have to make sure we dont break HDMI 1.4 sinks.
>> + */
>> + if (!is_hdmi2 && frame->video_code > 64)
>> + frame->video_code = 0;
>> +
>> frame->picture_aspect = HDMI_PICTURE_ASPECT_NONE;
>>
>> /*
>> diff --git a/drivers/gpu/drm/exynos/exynos_hdmi.c b/drivers/gpu/drm/exynos/exynos_hdmi.c
>> index 5243840..5cb44d4 100644
>> --- a/drivers/gpu/drm/exynos/exynos_hdmi.c
>> +++ b/drivers/gpu/drm/exynos/exynos_hdmi.c
>> @@ -781,7 +781,7 @@ static void hdmi_reg_infoframes(struct hdmi_context *hdata)
>> }
>>
>> ret = drm_hdmi_avi_infoframe_from_display_mode(&frm.avi,
>> - &hdata->current_mode);
>> + &hdata->current_mode, false);
>> if (!ret)
>> ret = hdmi_avi_infoframe_pack(&frm.avi, buf, sizeof(buf));
>> if (ret > 0) {
>> diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c
>> index 86f47e1..d1e7ac5 100644
>> --- a/drivers/gpu/drm/i2c/tda998x_drv.c
>> +++ b/drivers/gpu/drm/i2c/tda998x_drv.c
>> @@ -712,7 +712,7 @@ tda998x_write_avi(struct tda998x_priv *priv, struct drm_display_mode *mode)
>> {
>> union hdmi_infoframe frame;
>>
>> - drm_hdmi_avi_infoframe_from_display_mode(&frame.avi, mode);
>> + drm_hdmi_avi_infoframe_from_display_mode(&frame.avi, mode, false);
>> frame.avi.quantization_range = HDMI_QUANTIZATION_RANGE_FULL;
>>
>> tda998x_write_if(priv, DIP_IF_FLAGS_IF2, REG_IF2_HB0, &frame);
>> diff --git a/drivers/gpu/drm/i915/intel_hdmi.c b/drivers/gpu/drm/i915/intel_hdmi.c
>> index 3eec74c..96644f3 100644
>> --- a/drivers/gpu/drm/i915/intel_hdmi.c
>> +++ b/drivers/gpu/drm/i915/intel_hdmi.c
>> @@ -458,11 +458,14 @@ static void intel_hdmi_set_avi_infoframe(struct drm_encoder *encoder,
>> struct intel_hdmi *intel_hdmi = enc_to_intel_hdmi(encoder);
>> const struct drm_display_mode *adjusted_mode =
>> &crtc_state->base.adjusted_mode;
>> + struct drm_connector *connector = &intel_hdmi->attached_connector->base;
>> + bool is_hdmi2 = connector->display_info.hdmi.scdc.supported;
>> union hdmi_infoframe frame;
>> int ret;
>>
>> ret = drm_hdmi_avi_infoframe_from_display_mode(&frame.avi,
>> - adjusted_mode);
>> + adjusted_mode,
>> + is_hdmi2);
>> if (ret < 0) {
>> DRM_ERROR("couldn't fill AVI infoframe\n");
>> return;
>> diff --git a/drivers/gpu/drm/i915/intel_sdvo.c b/drivers/gpu/drm/i915/intel_sdvo.c
>> index 816a6f5..694f626 100644
>> --- a/drivers/gpu/drm/i915/intel_sdvo.c
>> +++ b/drivers/gpu/drm/i915/intel_sdvo.c
>> @@ -1011,7 +1011,8 @@ static bool intel_sdvo_set_avi_infoframe(struct intel_sdvo *intel_sdvo,
>> ssize_t len;
>>
>> ret = drm_hdmi_avi_infoframe_from_display_mode(&frame.avi,
>> - &pipe_config->base.adjusted_mode);
>> + &pipe_config->base.adjusted_mode,
>> + false);
>> if (ret < 0) {
>> DRM_ERROR("couldn't fill AVI infoframe\n");
>> return false;
>> diff --git a/drivers/gpu/drm/mediatek/mtk_hdmi.c b/drivers/gpu/drm/mediatek/mtk_hdmi.c
>> index c262512..80c36af 100644
>> --- a/drivers/gpu/drm/mediatek/mtk_hdmi.c
>> +++ b/drivers/gpu/drm/mediatek/mtk_hdmi.c
>> @@ -975,7 +975,7 @@ static int mtk_hdmi_setup_avi_infoframe(struct mtk_hdmi *hdmi,
>> u8 buffer[17];
>> ssize_t err;
>>
>> - err = drm_hdmi_avi_infoframe_from_display_mode(&frame, mode);
>> + err = drm_hdmi_avi_infoframe_from_display_mode(&frame, mode, false);
>> if (err < 0) {
>> dev_err(hdmi->dev,
>> "Failed to get AVI infoframe from mode: %zd\n", err);
>> diff --git a/drivers/gpu/drm/omapdrm/omap_encoder.c b/drivers/gpu/drm/omapdrm/omap_encoder.c
>> index 86c977b..624f5b5 100644
>> --- a/drivers/gpu/drm/omapdrm/omap_encoder.c
>> +++ b/drivers/gpu/drm/omapdrm/omap_encoder.c
>> @@ -85,7 +85,8 @@ static void omap_encoder_mode_set(struct drm_encoder *encoder,
>> if (hdmi_mode && dssdev->driver->set_hdmi_infoframe) {
>> struct hdmi_avi_infoframe avi;
>>
>> - r = drm_hdmi_avi_infoframe_from_display_mode(&avi, adjusted_mode);
>> + r = drm_hdmi_avi_infoframe_from_display_mode(&avi, adjusted_mode,
>> + false);
>> if (r == 0)
>> dssdev->driver->set_hdmi_infoframe(dssdev, &avi);
>> }
>> diff --git a/drivers/gpu/drm/radeon/radeon_audio.c b/drivers/gpu/drm/radeon/radeon_audio.c
>> index b214663..49b42bf 100644
>> --- a/drivers/gpu/drm/radeon/radeon_audio.c
>> +++ b/drivers/gpu/drm/radeon/radeon_audio.c
>> @@ -516,7 +516,7 @@ static int radeon_audio_set_avi_packet(struct drm_encoder *encoder,
>> if (!connector)
>> return -EINVAL;
>>
>> - err = drm_hdmi_avi_infoframe_from_display_mode(&frame, mode);
>> + err = drm_hdmi_avi_infoframe_from_display_mode(&frame, mode, false);
>> if (err < 0) {
>> DRM_ERROR("failed to setup AVI infoframe: %d\n", err);
>> return err;
>> diff --git a/drivers/gpu/drm/rockchip/inno_hdmi.c b/drivers/gpu/drm/rockchip/inno_hdmi.c
>> index 006260d..b105f1c 100644
>> --- a/drivers/gpu/drm/rockchip/inno_hdmi.c
>> +++ b/drivers/gpu/drm/rockchip/inno_hdmi.c
>> @@ -294,7 +294,7 @@ static int inno_hdmi_config_video_avi(struct inno_hdmi *hdmi,
>> union hdmi_infoframe frame;
>> int rc;
>>
>> - rc = drm_hdmi_avi_infoframe_from_display_mode(&frame.avi, mode);
>> + rc = drm_hdmi_avi_infoframe_from_display_mode(&frame.avi, mode, false);
>>
>> if (hdmi->hdmi_data.enc_out_format == HDMI_COLORSPACE_YUV444)
>> frame.avi.colorspace = HDMI_COLORSPACE_YUV444;
>> diff --git a/drivers/gpu/drm/sti/sti_hdmi.c b/drivers/gpu/drm/sti/sti_hdmi.c
>> index ce2dcba..2128b8c 100644
>> --- a/drivers/gpu/drm/sti/sti_hdmi.c
>> +++ b/drivers/gpu/drm/sti/sti_hdmi.c
>> @@ -434,7 +434,7 @@ static int hdmi_avi_infoframe_config(struct sti_hdmi *hdmi)
>>
>> DRM_DEBUG_DRIVER("\n");
>>
>> - ret = drm_hdmi_avi_infoframe_from_display_mode(&infoframe, mode);
>> + ret = drm_hdmi_avi_infoframe_from_display_mode(&infoframe, mode, false);
>> if (ret < 0) {
>> DRM_ERROR("failed to setup AVI infoframe: %d\n", ret);
>> return ret;
>> diff --git a/drivers/gpu/drm/tegra/hdmi.c b/drivers/gpu/drm/tegra/hdmi.c
>> index cda0491..718d8db 100644
>> --- a/drivers/gpu/drm/tegra/hdmi.c
>> +++ b/drivers/gpu/drm/tegra/hdmi.c
>> @@ -734,7 +734,7 @@ static void tegra_hdmi_setup_avi_infoframe(struct tegra_hdmi *hdmi,
>> u8 buffer[17];
>> ssize_t err;
>>
>> - err = drm_hdmi_avi_infoframe_from_display_mode(&frame, mode);
>> + err = drm_hdmi_avi_infoframe_from_display_mode(&frame, mode, false);
>> if (err < 0) {
>> dev_err(hdmi->dev, "failed to setup AVI infoframe: %zd\n", err);
>> return;
>> diff --git a/drivers/gpu/drm/tegra/sor.c b/drivers/gpu/drm/tegra/sor.c
>> index a8f5289..fb2709c 100644
>> --- a/drivers/gpu/drm/tegra/sor.c
>> +++ b/drivers/gpu/drm/tegra/sor.c
>> @@ -1904,7 +1904,7 @@ tegra_sor_hdmi_setup_avi_infoframe(struct tegra_sor *sor,
>> value &= ~INFOFRAME_CTRL_ENABLE;
>> tegra_sor_writel(sor, value, SOR_HDMI_AVI_INFOFRAME_CTRL);
>>
>> - err = drm_hdmi_avi_infoframe_from_display_mode(&frame, mode);
>> + err = drm_hdmi_avi_infoframe_from_display_mode(&frame, mode, false);
>> if (err < 0) {
>> dev_err(sor->dev, "failed to setup AVI infoframe: %d\n", err);
>> return err;
>> diff --git a/drivers/gpu/drm/vc4/vc4_hdmi.c b/drivers/gpu/drm/vc4/vc4_hdmi.c
>> index e9cbe26..5d81301 100644
>> --- a/drivers/gpu/drm/vc4/vc4_hdmi.c
>> +++ b/drivers/gpu/drm/vc4/vc4_hdmi.c
>> @@ -394,7 +394,7 @@ static void vc4_hdmi_set_avi_infoframe(struct drm_encoder *encoder)
>> union hdmi_infoframe frame;
>> int ret;
>>
>> - ret = drm_hdmi_avi_infoframe_from_display_mode(&frame.avi, mode);
>> + ret = drm_hdmi_avi_infoframe_from_display_mode(&frame.avi, mode, false);
>> if (ret < 0) {
>> DRM_ERROR("couldn't fill AVI infoframe\n");
>> return;
>> diff --git a/drivers/gpu/drm/zte/zx_hdmi.c b/drivers/gpu/drm/zte/zx_hdmi.c
>> index c47b9cb..3e267dd 100644
>> --- a/drivers/gpu/drm/zte/zx_hdmi.c
>> +++ b/drivers/gpu/drm/zte/zx_hdmi.c
>> @@ -125,7 +125,7 @@ static int zx_hdmi_config_video_avi(struct zx_hdmi *hdmi,
>> union hdmi_infoframe frame;
>> int ret;
>>
>> - ret = drm_hdmi_avi_infoframe_from_display_mode(&frame.avi, mode);
>> + ret = drm_hdmi_avi_infoframe_from_display_mode(&frame.avi, mode, false);
>> if (ret) {
>> DRM_DEV_ERROR(hdmi->dev, "failed to get avi infoframe: %d\n",
>> ret);
>> diff --git a/include/drm/drm_edid.h b/include/drm/drm_edid.h
>> index a21d494..a4d174e7 100644
>> --- a/include/drm/drm_edid.h
>> +++ b/include/drm/drm_edid.h
>> @@ -348,7 +348,8 @@ drm_load_edid_firmware(struct drm_connector *connector)
>>
>> int
>> drm_hdmi_avi_infoframe_from_display_mode(struct hdmi_avi_infoframe *frame,
>> - const struct drm_display_mode *mode);
>> + const struct drm_display_mode *mode,
>> + bool is_hdmi2);
>> int
>> drm_hdmi_vendor_infoframe_from_display_mode(struct hdmi_vendor_infoframe *frame,
>> const struct drm_display_mode *mode);
>> --
>> 2.7.4
>>
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2017-03-23 15:22 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-23 15:14 [PATCH v4 1/2] drm/edid: Complete CEA modedb(VIC 1-107) Shashank Sharma
2017-03-23 15:14 ` [PATCH v4 2/2] drm: Add HDMI 2.0 VIC support for AVI info-frames Shashank Sharma
2017-03-23 15:17 ` Ilia Mirkin
2017-03-23 15:22 ` Sharma, Shashank [this message]
2017-03-23 15:28 ` Ilia Mirkin
2017-03-23 16:33 ` Sharma, Shashank
2017-03-23 15:28 ` Jose Abreu
2017-03-23 15:45 ` Ville Syrjälä
2017-03-23 15:57 ` Jose Abreu
2017-03-23 16:08 ` Sharma, Shashank
2017-03-23 16:33 ` Jose Abreu
2017-03-23 16:43 ` Sharma, Shashank
2017-03-23 17:11 ` Jose Abreu
2017-03-23 17:42 ` Ville Syrjälä
2017-03-23 17:53 ` Jose Abreu
2017-03-23 18:14 ` Ville Syrjälä
2017-03-23 18:35 ` Jose Abreu
2017-03-23 15:36 ` Ville Syrjälä
2017-03-23 15:52 ` Jose Abreu
2017-03-23 16:31 ` Sharma, Shashank
2017-03-23 17:01 ` Ville Syrjälä
2017-03-23 16:01 ` ✓ Fi.CI.BAT: success for series starting with [v4,1/2] drm/edid: Complete CEA modedb(VIC 1-107) Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=3eed384c-f584-991d-87fa-db7483e67f6f@intel.com \
--to=shashank.sharma@intel.com \
--cc=alexander.deucher@amd.com \
--cc=daniel.vetter@intel.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=imirkin@alum.mit.edu \
--cc=intel-gfx@lists.freedesktop.org \
--cc=jose.abreu@synopsys.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.