<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">On 2018年01月31日 14:09, Kai-Heng Feng
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:20180131060944.30357-1-kai.heng.feng@canonical.com">
<pre wrap="">BugLink: <a class="moz-txt-link-freetext" href="https://bugs.launchpad.net/bugs/1746411">https://bugs.launchpad.net/bugs/1746411</a>
[Impact]
i915 crashed when probing for VBT pin. No graphics output after that.
[Test Case]
Enable Legacy ROM boot option in BIOS. Screen goes blank.
Cherrypick fixes can solve the issue.
[Fix]
Handle out of bounds pin value more appropriately.
Patches from Jani are most refactoring, but the fix we want depends on
them.
Patches from Rogrigo are the fixes we want. These patches are also
picked base on his suggestion,
<a class="moz-txt-link-freetext" href="https://bugs.freedesktop.org/show_bug.cgi?id=104139#c7">https://bugs.freedesktop.org/show_bug.cgi?id=104139#c7</a>
[Regression Potential]
Low. CNL and CNP are still quite new. These commits are limited to them.</pre>
</blockquote>
Looks fine to me.<br>
<br>
<pre wrap="">Acked-by: Hui Wang <a class="moz-txt-link-rfc2396E" href="mailto:hui.wang@canonical.com"><hui.wang@canonical.com></a></pre>
<blockquote type="cite"
cite="mid:20180131060944.30357-1-kai.heng.feng@canonical.com">
<pre wrap="">
Jani Nikula (12):
drm/i914/bios: amend child device config parameters
drm/i915/bios: document BDB versions of child device config fields
drm/i915/bios: remove the raw version of child device config
drm/i915/bios: add legacy contents to common child device config
drm/i915/bios: throw away high level child device union
drm/i915/bios: throw away struct old_child_dev_config
drm/i915/bios: document child device config dvo_port values a bit
better
drm/i914/bios: group device type definitions together
drm/i915/bios: throw away unused DVO_* macros
drm/i915/bios: drop the rest of the p_ prefixes from pointers
drm/i915/bios: split up iboost to hdmi and dp bitfields
drm/i915/bios: add DP max link rate to VBT child device struct
Rodrigo Vivi (4):
drm/i915/cnl: Don't trust VBT's alternate pin for port D for now.
drm/i915/cnl: Map VBT DDC Pin to BSpec DDC Pin.
drm/i915/cnp: Ignore VBT request for know invalid DDC pin.
drm/i915/cnp: Properly handle VBT ddc pin out of bounds.
drivers/gpu/drm/i915/i915_drv.h | 2 +-
drivers/gpu/drm/i915/intel_bios.c | 226 ++++++++++++++------------
drivers/gpu/drm/i915/intel_vbt_defs.h | 295 +++++++++++++++++++---------------
3 files changed, 290 insertions(+), 233 deletions(-)
</pre>
</blockquote>
<p><br>
</p>
</body>
</html>