diff options
| author | Tatenda Chipeperekwa <tatendac@codeaurora.org> | 2017-03-28 19:22:05 -0700 |
|---|---|---|
| committer | Gerrit - the friendly Code Review server <code-review@localhost> | 2017-04-13 18:20:18 -0700 |
| commit | 0325fec74c124ded524356fbb1edee6426980261 (patch) | |
| tree | dd422fe55a2efa05ce95d67870f3e2507849fcc4 /drivers/video/fbdev | |
| parent | 5921f2095dcff1b8516f8149c921b7f2477816db (diff) | |
msm: mdss: dp: fix handling of EDID extension blocks
Current implementation assumes that the first extension
block of the EDID data always has the CEA extension tag.
This is incorrect and can lead to incorrect parsing of
resolution data. Fix this by removing the hardcoding of
the extension tag for the first extension block.
CRs-Fixed: 2027108
Change-Id: I5ef4cdb186591e291b2217db7ccec6f942d79ca4
Signed-off-by: Tatenda Chipeperekwa <tatendac@codeaurora.org>
Diffstat (limited to 'drivers/video/fbdev')
| -rw-r--r-- | drivers/video/fbdev/msm/mdss_dp_aux.c | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/drivers/video/fbdev/msm/mdss_dp_aux.c b/drivers/video/fbdev/msm/mdss_dp_aux.c index 86facde66ba3..37209c161366 100644 --- a/drivers/video/fbdev/msm/mdss_dp_aux.c +++ b/drivers/video/fbdev/msm/mdss_dp_aux.c @@ -953,7 +953,6 @@ int mdss_dp_edid_read(struct mdss_dp_drv_pdata *dp) int rlen, ret = 0; int edid_blk = 0, blk_num = 0, retries = 10; bool edid_parsing_done = false; - const u8 cea_tag = 0x02; u32 const segment_addr = 0x30; u32 checksum = 0; bool phy_aux_update_requested = false; @@ -1037,15 +1036,6 @@ int mdss_dp_edid_read(struct mdss_dp_drv_pdata *dp) } else { edid_blk++; blk_num++; - - /* fix dongle byte shift issue */ - if (edid_blk == 1 && edid_buf[0] != cea_tag) { - u8 tmp[EDID_BLOCK_SIZE - 1]; - - memcpy(tmp, edid_buf, EDID_BLOCK_SIZE - 1); - edid_buf[0] = cea_tag; - memcpy(edid_buf + 1, tmp, EDID_BLOCK_SIZE - 1); - } } memcpy(dp->edid_buf + (edid_blk * EDID_BLOCK_SIZE), |