On 30.01.2018 00:12, Rodrigo Vivi wrote:
> On Mon, Jan 29, 2018 at 05:42:53AM +0000, Kai Heng Feng wrote:
>>
>>> On 26 Jan 2018, at 6:25 AM, Rodrigo Vivi <rodrigo.v...@intel.com> wrote:
>>>
>>> If the table result is out of bounds on the array map
>>> there is something really wrong with VBT pin so we don't
>>> return that vbt_pin, but only return 0 instead.
>>>
>>> This basically reverts commit 'a8e6f3888b05 ("drm/i915/cnp:
>>> Ignore VBT request for know invalid DDC pin.")'
>>>
>>> Also this properly fixes commit 9c3b2689d01f ("drm/i915/cnl:
>>> Map VBT DDC Pin to BSpec DDC Pin.")
>>>
>>> v2: Do in a way that we don't break other platforms. (Jani)
>>>
>>> v3: Keep debug message (Jani)
>>>
>>> v4: Don't mess with 0 mapping was noticed by Jani and
>>> addressed with a simple solution suggested by Lucas
>>> that makes this even simpler.
>>>
>>> Fixes: a8e6f3888b05 ("drm/i915/cnp: Ignore VBT request for know invalid DDC
>>> pin.")
>>> Fixes: 9c3b2689d01f ("drm/i915/cnl: Map VBT DDC Pin to BSpec DDC Pin.")
>>> Cc: Radhakrishna Sripada <radhakrishna.srip...@intel.com>
>>> Cc: Jani Nikula <jani.nik...@intel.com>
>>> Cc: Kai Heng Feng <kai.heng.f...@canonical.com>
>>> Cc: Lucas De Marchi <lucas.demar...@intel.com>
>>> Suggested-by: Lucas De Marchi <lucas.demar...@intel.com>
>>> Signed-off-by: Rodrigo Vivi <rodrigo.v...@intel.com>
>>
>> Tested-by: Kai-Heng Feng <kai.heng.f...@canonical.com>
>
> merged. thanks for suggestions, reviews, tests and patience ;)
Shouldn't this and
drm/i915/cnp: Ignore VBT request for know invalid DDC pin.
be cc:stable? Though they aren't even in 4.16 yet.
--
t
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
