[Intel-gfx] [PATCH v4 2/2] drm: Add HDMI 2.0 VIC support for AVI info-frames

[Sharma, Shashank]

Regards

Shashank


On 3/23/2017 5:28 PM, Ilia Mirkin wrote:
> On Thu, Mar 23, 2017 at 11:22 AM, Sharma, Shashank
> <shashank.sharma at intel.com> wrote:
>> On 3/23/2017 5:17 PM, Ilia Mirkin wrote:
>>> On Thu, Mar 23, 2017 at 11:14 AM, Shashank Sharma
>>> <shashank.sharma at 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.
> OK, so ... I have a bug in my filesystem, and I'm bisecting it and
> rebooting my box at each step. I happen to hit this commit in my
> bisect and my monitor doesn't turn on? Seems unpleasant, no?
Cant say no :-), will happily change the order if this makes you feel 
better.

Regards
Shashank