evie/android_kernel_oneplus_msm8998
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

msm: hdmi: edid: always add timings parsed from DTD to reserved list

Browse source 
Resolutions listed in the Detailed Timing Descriptor (DTD) have
detailed information about all the timings. Current implementation
attempts to match the parsed resolution with the resolution database
based on just the Video Information Code (VIC). This can potentially
overlook mismatches in certain parameters such as refresh rates and
can cause unintended results. Fix this by always adding the resolutions
parsed from the DTDs to the reserved timing list.

CRs-Fixed: 1109812
Change-Id: I8b926f08ce9b44c4a16a32fed26fedd351d77ae8
Signed-off-by: Aravind Venkateswaran <aravindh@codeaurora.org>
This commit is contained in:
Aravind Venkateswaran 2017-01-15 15:22:46 -08:00
parent 1507849767
commit bc176f3364
1 changed files with 8 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
drivers/video/fbdev/msm/mdss_hdmi_edid.c
View file

@ -1607,9 +1607,14 @@ static void hdmi_edid_detail_desc(struct hdmi_edid_ctrl *edid_ctrl,
(timing.refresh_rate % 100) / 10,
timing.refresh_rate % 10);
rc = hdmi_get_video_id_code(&timing, NULL);
if (rc < 0)
rc = hdmi_set_resv_timing_info(&timing);
/*
* Always add resolutions parsed from DTD in the reserved
* timing info. This can avoid matching resolutions that have
* a non-integral fps denominators with corresponding
* resolutions that have an integral fps denominator.
* For example - 640x480p@59.94Hz --> 640x480p@60Hz
*/
rc = hdmi_set_resv_timing_info(&timing);
} else {
rc = -EINVAL;
}