[PATCH v3] drm/edid: Complete CEA modedb(VIC 1-107)

[Sharma, Shashank]

Regards

Shashank


On 1/30/2017 1:30 PM, Andrzej Hajda wrote:
> On 29.01.2017 06:41, Shashank Sharma wrote:
>> CEA-861-F specs defines new 4k video modes to be used with
>> HDMI 2.0 EDIDs. These modes start at VIC=93 and go all the
>> way till VIC=107.
>>
>> Our existing CEA modedb contains only 64 modes (VIC=1 to VIC=64). Now
>> to be able to parse 4k modes using the existing techniques, we have
>> to complete the modedb (VIC=65 onwards).
>>
>> This patch adds:
>> - Timings for existing CEA video modes (from VIC=65 till VIC=92)
>> - Newly added 4k modes (from VIC=93 to VIC=107).
>>
>> Cc: Jose Abreu <Jose.Abreu at synopsys.com>
>> Cc: Alex Deucher <alexander.deucher at amd.com>
>> Cc: Andrzej Hajda <a.hajda at samsung.com>
>>
>> V2: Addressed review comments from Jose:
>> - fix the timings for VIC 83, 90 and 91
>> - fix formatting for VIC 93-107
>>
>> V3: Rebase on drm-tip
>>
>> Signed-off-by: Shashank Sharma <shashank.sharma at intel.com>
>> Signed-off-by: Sonika Jindal <sonika.jindal at intel.com>
>> Reviewed-by: Jose Abreu <Jose.Abreu at synopsys.com>
>> Reviewed-by: Alex Deucher <alexander.deucher at amd.com>
>> ---
> Have you addressed in some other patch my concerns regarding polluting
> infoframes generated for HDMI 1.4 devices (CEA-861-E) with VICs specific
> only for HDMI 2.0 (CEA-861-F) ? It will happen
> for all UHD modes, see [1][2] for previous discussion.
Hello Andrzej

As you already know, the aspect ratio is indexed with the VIC, which 
comes from EDID.
Now, why would a HDMI 1.4 monitor, contain a HDMI 2.0 VIC ? If it does, 
its not compliant.

- Shashank
> [1]: https://www.spinics.net/lists/intel-gfx/msg110524.html
> [2]: https://www.spinics.net/lists/intel-gfx/msg110529.html
>
> Regards
> Andrzej
>