== Series Details ==
Series: Add automation support for DP Compliance (Rev 6)
URL : https://patchwork.freedesktop.org/series/18329/
State : success
== Summary ==
Series 18329v1 Add automation support for DP Compliance (Rev 6)
The intel_dp_autotest_video_pattern() function gets invoked through the
compliance test handler on a HPD short pulse if the test type is
set to DP_TEST_VIDEO_PATTERN. This performs the DPCD registers
reads to read the requested test pattern, video pattern resolution,
frame rate and bits per color
This patch adds support to handle automated DP compliance
link training test requests. This patch has been tested with
Unigraf DPR-120 DP Compliance device for testing Link
Training Compliance.
After we get a short pulse Compliance test request, test
request values are read and hotplug uevent is
v4:
* Remove redundant single bit defs (Jani Nikula)
v3:
* Fix the conventions in bit definitions (Jani Nikula)
v2:
* Add all the other DP Complianec TEST register defs (Jani Nikula)
Cc: dri-de...@lists.freedesktop.org
Cc: Jani Nikula
Cc: Daniel Vetter
This patch addresses a few issues from the original patch for
DP Compliance EDID test support submitted by
Todd Previte
Video Mode requested in the EDID test handler for the EDID Read
test (CTS 4.2.2.3) should be set to PREFERRED as per the CTS spec.
v2:
* Added read
This patch series addresses all the review comments from the previous series:
https://patchwork.freedesktop.org/series/18256/
On top of that it also adds a fix for CRC mismatches seen during
18bpp video pattern compliance testing. This is done by forcing dither
on the pipe to be disabled if it is
== Series Details ==
Series: drm/i915: reinstate call to trace_i915_vma_bind (rev2)
URL : https://patchwork.freedesktop.org/series/17929/
State : success
== Summary ==
Series 17929v2 drm/i915: reinstate call to trace_i915_vma_bind
== Series Details ==
Series: series starting with [1/2] drm/i915: Squelch smatch warning for
statement with no effect
URL : https://patchwork.freedesktop.org/series/18326/
State : success
== Summary ==
Series 18326v1 Series without cover letter
The call went away in:
commit 3b16525cc4c1a43e9053cfdc414356eea24bdfad
Author: Chris Wilson
Date: Thu Aug 4 16:32:25 2016 +0100
drm/i915: Split insertion/binding of an object into the VM
It is useful to have this trace as it pairs nicely with the vma_unbind
one
drivers/gpu/drm/i915/i915_cmd_parser.c:808:23: error: not an lvalue
drivers/gpu/drm/i915/i915_cmd_parser.c:811:23: error: not an lvalue
drivers/gpu/drm/i915/i915_cmd_parser.c:814:23: error: not an lvalue
drivers/gpu/drm/i915/i915_cmd_parser.c:808:23: error: not an lvalue
The sgt iterators cause an
drivers/gpu/drm/i915/i915_gpu_error.c:846 i915_error_object_create() warn:
statement has no effect 7
everywhere they are used. If we change the code slightly, we can achieve
the same increment without altering the output or raising a warning.
textdata bss
On Fri, Jan 20, 2017 at 01:35:21PM -0800, Daniele Ceraolo Spurio wrote:
>
>
> On 20/01/17 13:28, Chris Wilson wrote:
> >On Fri, Jan 20, 2017 at 01:16:13PM -0800, Daniele Ceraolo Spurio wrote:
> >>Ping. can anyone review/comment on this?
> >>
> >>Thanks,
> >>Daniele
> >>
> >>On 12/01/17 14:21,
On 20/01/17 13:28, Chris Wilson wrote:
On Fri, Jan 20, 2017 at 01:16:13PM -0800, Daniele Ceraolo Spurio wrote:
Ping. can anyone review/comment on this?
Thanks,
Daniele
On 12/01/17 14:21, daniele.ceraolospu...@intel.com wrote:
From: Daniele Ceraolo Spurio
On Fri, Jan 20, 2017 at 01:16:13PM -0800, Daniele Ceraolo Spurio wrote:
> Ping. can anyone review/comment on this?
>
> Thanks,
> Daniele
>
> On 12/01/17 14:21, daniele.ceraolospu...@intel.com wrote:
> >From: Daniele Ceraolo Spurio
> >
> >The call went away in:
>
Ping. can anyone review/comment on this?
Thanks,
Daniele
On 12/01/17 14:21, daniele.ceraolospu...@intel.com wrote:
From: Daniele Ceraolo Spurio
The call went away in:
commit 3b16525cc4c1a43e9053cfdc414356eea24bdfad
Author: Chris Wilson
On Fri, Jan 20, 2017 at 07:54:09PM -, Patchwork wrote:
> == Series Details ==
>
> Series: series starting with [CI,1/3] drm/i915/huc: Avoid attempting to
> authenticate non-existent fw
> URL : https://patchwork.freedesktop.org/series/18320/
> State : failure
>
> == Summary ==
>
> Series
On Tue, Dec 27, 2016 at 03:56:54PM +0100, Daniel Vetter wrote:
> On Thu, Dec 22, 2016 at 04:04:41PM +0200, ville.syrj...@linux.intel.com wrote:
> > From: Ville Syrjälä
> >
> > crtc->config is on its way out. Let's reduce our dependence on it a
> > little bit by
== Series Details ==
Series: series starting with [CI,1/3] drm/i915/huc: Avoid attempting to
authenticate non-existent fw
URL : https://patchwork.freedesktop.org/series/18320/
State : failure
== Summary ==
Series 18320v1 Series without cover letter
ville.syrj...@linux.intel.com writes:
> From: Ville Syrjälä
>
> Make the code selecting the RGB quantization range a little less magicy
> by wrapping it up in a small helper.
This series seems good. I won't have the ability to test it on vc4
within a reasonable
HuC firmware is mapped at GuC accessible range. Let's add an assert to
verify that.
Cc: Anusha Srivatsa
Cc: Arkadiusz Hiler
Cc: Chris Wilson
Reviewed-by: Michal Wajdeczko
Signed-off-by:
---
drivers/gpu/drm/i915/i915_params.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/i915_params.c
b/drivers/gpu/drm/i915/i915_params.c
index 0e280fb..90d838f 100644
--- a/drivers/gpu/drm/i915/i915_params.c
+++ b/drivers/gpu/drm/i915/i915_params.c
@@
HuC authentication is called even if HuC firmware is not present in the
system, leading to NULL ptr dereference on not allocated gem_object.
Let's avoid trying to authenticate HuC if its firmware is not loaded
successfully.
Fixes: dac84a388528 ("drm/i915/huc: Support HuC authentication")
v2:
On Fri, 20 Jan 2017 17:45:53 +0100,
Pierre-Louis Bossart wrote:
>
> On 1/20/17 5:15 AM, Takashi Iwai wrote:
> > On Fri, 20 Jan 2017 23:22:31 +0100,
> > Jerome Anand wrote:
> >>
> >> + had_ops_v1 = had_ops_v1;/* unused */
> >
> > Until now I didn't realize that the whole v1 stuff is never
A simple helper for getting the igt_output_t struct corresponding to the
given DRM connector id.
Cc: Tomeu Vizoso
Signed-off-by: Lyude
---
lib/igt_kms.c | 26 ++
lib/igt_kms.h | 2 ++
2 files changed, 28 insertions(+)
diff
For the purpose of testing things such as hotplugging and bad monitors,
the ChromeOS team ended up designing a neat little device known as the
Chamelium. More information on this can be found here:
https://www.chromium.org/chromium-os/testing/chamelium
This adds support for a couple of
Resending because I forgot to stage some files before sending this out…
This adds basic support for using the Chamelium in hotplugging tests, along
with checking video outputs.
So far there's still definitely some bugs with video capturing, however they
seem to mostly be the Chamelium's fault.
A simple helper that checks whether or not the given connector has the
"Broadcast RGB" prop, and if so sets it to the given mode. Required for
working with the Chamelium since the Chamelium EDIDs enable limited
color ranges by default on i915 and break frame dump comparisons/CRCs.
Cc: Tomeu
This adds some basic helpers for connecting to udev and watching for
sysfs hotplug events.
Cc: Tomeu Vizoso
Signed-off-by: Lyude
Changes since v1:
- Remove unused arg from documentation
Changes since v2:
- Make udev_monitor explicit so that we can use
The default autoresume delay is about 5 seconds. It's possible on a
system that's not very fast this might not be a long enough time, since
an asynchronous hotplug event we scheduled on the chamelium that was
intended to happen during suspend could happen before we actually manage
to suspend. So,
== Series Details ==
Series: drm/i915: Introduce intel_cdclk_state (rev3)
URL : https://patchwork.freedesktop.org/series/16994/
State : warning
== Summary ==
Series 16994v3 drm/i915: Introduce intel_cdclk_state
https://patchwork.freedesktop.org/api/1.0/series/16994/revisions/3/mbox/
Test
Jani,
This patch adds a fix for CRC mismatch errors as well.
The reason I have squashed this fix with this patch is because
these CRC failures are seen for a Video Pattern test.
It uses the approach that we discussed where a piep_config->dither_force_disable
is used to disable dithering for
From: Ville Syrjälä
Rather than passing all the different parameters (cdclk,vco so
far) sparately to the set_cdclk() functions, just pass the
entire cdclk state.
v2: Deal with churn
v3: Drop the usless .ref assignment (Ander)
Signed-off-by: Ville Syrjälä
From: Ville Syrjälä
Here's the latest version of my cdclk state work. I tried to accomodate
most of the review feedback, although there were some comments which
I think are better left for a followup work. Mainly about renaming some
functions and whatnot to make
From: Ville Syrjälä
Move the vlv_program_pfi_credits() into vlv_set_cdclk() and
chv_set_cdclk() so that we can neuter vlv_modeset_commit_cdclk().
Signed-off-by: Ville Syrjälä
Reviewed-by: Ander Conselvan de Oliveira
From: Ville Syrjälä
Let's clean up the mess we have in the if ladder that assigns the
.get_cdclk() hooks. The grouping of the platforms by the function
results in a thing that's not really legible, so let's do it the
other way around and order the if ladder by
From: Ville Syrjälä
The current dev_cdclk vs. cdclk vs. atomic_cdclk_freq is quite a mess.
So here I'm introducing the "actual" and "logical" naming for our
cdclk state. "actual" is what we'll bash into the hardware and "logical"
is what everyone should use for
From: Ville Syrjälä
ilk_max_pixel_rate() will now give the "correct" pixel rate for all
platforms, so let's rename it to intel_max_pixel_rate() and kill
off intel_mode_max_pixclk().
v2: Fix typo in commit message (Ander)
Signed-off-by: Ville Syrjälä
From: Ville Syrjälä
The hack to grab the pipe A power domain around VLV/CHV cdclk
programming has surely outlived its usefulness. We should be
holding sufficient power domains during any modeset, so let's
just nuke this hack.
v2: Fix typo in commit message (Ander)
From: Ville Syrjälä
With the cdclk state, all the .modeset_commit_cdclk() hooks are
now pointless wrappers. Let's replace them with just a .set_cdclk()
function pointer. However let's wrap that in a small helper that
does the state comparison and prints a unified
From: Ville Syrjälä
Clean up the dev vs. dev_priv straggles that are making things
look inconsistentt.
v2: Deal with churn
Signed-off-by: Ville Syrjälä
Reviewed-by: Ander Conselvan de Oliveira
---
From: Ville Syrjälä
Rather than recomptuing the pipe pixel rate on demand everwhere, let's
just stick the precomputed value into the crtc state.
v2: Rebase due to min_pixclk[] code movement
Document the new pixel_rate struct member (Ander)
Combine vlv/chv
From: Ville Syrjälä
Rather than compute the vco inside bxt_set_cdclk() let's precompute it
outside and pass it in. A small step towards a fully precomputed cdclk
state.
Signed-off-by: Ville Syrjälä
Reviewed-by: Ander Conselvan de
From: Ville Syrjälä
Rename the .get_display_clock_speed() hook to .get_cdclk().
.get_cdclk() is more specific (which clock) and it's much
shorter.
Signed-off-by: Ville Syrjälä
Reviewed-by: Ander Conselvan de Oliveira
From: Ville Syrjälä
Move ilk_pipe_pixel_rate() next to its only caller
(intel_crtc_compute_pixel_rate()).
Signed-off-by: Ville Syrjälä
Reviewed-by: Ander Conselvan de Oliveira
---
From: Ville Syrjälä
Introduce intel_cdclk state which for now will track the cdclk
frequency, the vco frequency and the reference frequency (not sure we
want the last one, but I put it there anyway). We'll also make the
.get_cdclk() function fill out this state
From: Ville Syrjälä
Let's try to shrink intel_display.c a bit by moving the cdclk/rawclk
stuff to a new file. It's all reasonably self contained so we don't
even have to add that many non-static symbols.
We'll also take the opportunity to shuffle around the
== Series Details ==
Series: drm/i915/dp: do not proceed with autotests if we don't ACK them (rev2)
URL : https://patchwork.freedesktop.org/series/18305/
State : success
== Summary ==
Series 18305v2 drm/i915/dp: do not proceed with autotests if we don't ACK them
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_plane_lowres.c | 27 +--
1 file changed, 21 insertions(+), 6 deletions(-)
diff --git a/tests/kms_plane_lowres.c
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_ccs.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/kms_ccs.c b/tests/kms_ccs.c
index 047a3e87..11acda89 100644
---
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_universal_plane.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/tests/kms_universal_plane.c
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_rmfb.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/kms_rmfb.c b/tests/kms_rmfb.c
index 17a3065a..5753d74c 100644
---
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_psr_sink_crc.c | 52 +---
1 file changed, 23 insertions(+), 29 deletions(-)
diff --git
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_plane_multiple.c | 209 -
1 file changed, 131 insertions(+), 78 deletions(-)
diff --git
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_sink_crc_basic.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/kms_sink_crc_basic.c b/tests/kms_sink_crc_basic.c
index
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_pwrite_crc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/kms_pwrite_crc.c b/tests/kms_pwrite_crc.c
index
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_rotation_crc.c | 63
1 file changed, 32 insertions(+), 31 deletions(-)
diff --git
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_vblank.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/kms_vblank.c b/tests/kms_vblank.c
index 122fbc8f..0d250653
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/prime_mmap_kms.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/prime_mmap_kms.c b/tests/prime_mmap_kms.c
index
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_properties.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/tests/kms_properties.c b/tests/kms_properties.c
index
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_plane_scaling.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/tests/kms_plane_scaling.c
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_fbc_crc.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/tests/kms_fbc_crc.c b/tests/kms_fbc_crc.c
index
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_cursor_crc.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c
index
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_busy.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/kms_busy.c b/tests/kms_busy.c
index 1ae5d7fb..e6276927 100644
---
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_panel_fitting.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/tests/kms_panel_fitting.c
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_legacy_colorkey.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/tests/kms_legacy_colorkey.c b/tests/kms_legacy_colorkey.c
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_cursor_legacy.c | 32 ++--
1 file changed, 18 insertions(+), 14 deletions(-)
diff --git a/tests/kms_cursor_legacy.c
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_atomic_transition.c | 27 +--
1 file changed, 13 insertions(+), 14 deletions(-)
diff --git a/tests/kms_atomic_transition.c
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_mmio_vs_cs_flip.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/tests/kms_mmio_vs_cs_flip.c b/tests/kms_mmio_vs_cs_flip.c
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_flip_event_leak.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/kms_flip_event_leak.c b/tests/kms_flip_event_leak.c
index
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_chv_cursor_fail.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/tests/kms_chv_cursor_fail.c b/tests/kms_chv_cursor_fail.c
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_crtc_background_color.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/kms_crtc_background_color.c
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_pipe_color.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/kms_pipe_color.c b/tests/kms_pipe_color.c
index
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_plane.c | 94 ++-
1 file changed, 52 insertions(+), 42 deletions(-)
diff --git
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_mmap_write_crc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/kms_mmap_write_crc.c b/tests/kms_mmap_write_crc.c
index
Add changes reflecting the new support for dynamic number of planes per pipe.
Signed-off-by: Robert Foss
---
tests/kms_fence_pin_leak.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/kms_fence_pin_leak.c b/tests/kms_fence_pin_leak.c
index
Rework kmstest_crtc and kmstest_plane structs and their usage
to not depend on a static plane count.
Signed-off-by: Robert Foss
---
lib/igt_kms.c | 47 ++-
lib/igt_kms.h | 4 ++--
2 files changed, 32 insertions(+), 19
In upcoming drm-misc-next changes, the number of planes per pipe has
been increased as more than one primary plane can be associated with
a pipe.
The simple fix for this would be to simply bump hardcoded value for
number of frames per pipe.
But a better solution would be to add support for
Add an index property which helps accessing the corresponding
igt_plane_t structure through the igt_*_get_plane() functions.
Signed-off-by: Robert Foss
---
lib/igt_kms.c| 12 ++--
lib/igt_kms.h| 1 +
tests/kms_plane_lowres.c | 2 +-
3
Rename these properties to have them use the same naming convention
as the igt_*_t structs.
Signed-off-by: Robert Foss
---
lib/igt_kms.c| 16 +---
lib/igt_kms.h| 4 ++--
tests/kms_plane_lowres.c | 10 +-
3 files changed, 16
This series implements dynamic plane count support in lib/igt_kms and modifies
all of the tests that rely on a static plane count.
Currently it has has gone through incomplete testing on vc4 (many tests are
intel only)
and intel (on intel-ci).
This series can be found in branch
== Series Details ==
Series: series starting with [v3,1/2] drm/i915/huc: Avoid attempting to
authenticate non-existent fw (rev3)
URL : https://patchwork.freedesktop.org/series/18301/
State : failure
== Summary ==
Series 18301v3 Series without cover letter
On 20/01/17 18:44, Jani Nikula wrote:
On Fri, 20 Jan 2017, Martin Peres wrote:
On 19/01/17 11:18, Jani Nikula wrote:
On Wed, 18 Jan 2017, Martin Peres wrote:
On 16/12/16 15:48, Daniel Vetter wrote:
On Fri, Dec 16, 2016 at
There is no point in setting intel_dp->compliance.test_type, and
proceeding with the autotests, if we're about to NAK the request. Some
drive-by cleanups while at it.
v2: look at the ACK bit, as the result may also contain
TEST_EDID_CHECKSUM_WRITE
Cc: Manasi Navare
On Fri, Jan 20, 2017 at 04:58:51PM +0100, Michał Winiarski wrote:
> On Fri, Jan 20, 2017 at 04:56:08PM +0100, Michał Winiarski wrote:
> > HuC firmware is mapped at GuC accessible range. Let's add an assert to
> > verify that.
> >
> > v2: Split from previous commit (Michał)
> > v3: Oops, hit send
== Series Details ==
Series: drm/i915/dp: do not proceed with autotests if we don't ACK them
URL : https://patchwork.freedesktop.org/series/18305/
State : success
== Summary ==
Series 18305v1 drm/i915/dp: do not proceed with autotests if we don't ACK them
On 1/20/17 5:15 AM, Takashi Iwai wrote:
On Fri, 20 Jan 2017 23:22:31 +0100,
Jerome Anand wrote:
+ had_ops_v1 = had_ops_v1;/* unused */
Until now I didn't realize that the whole v1 stuff is never used in
the current patchset. Will it be ever used in future? If not, can't
we
On Fri, 20 Jan 2017, Martin Peres wrote:
> On 19/01/17 11:18, Jani Nikula wrote:
>> On Wed, 18 Jan 2017, Martin Peres wrote:
>>> On 16/12/16 15:48, Daniel Vetter wrote:
On Fri, Dec 16, 2016 at 12:29:05PM +0200, Jani Nikula wrote:
On 19/01/17 11:18, Jani Nikula wrote:
On Wed, 18 Jan 2017, Martin Peres wrote:
On 16/12/16 15:48, Daniel Vetter wrote:
On Fri, Dec 16, 2016 at 12:29:05PM +0200, Jani Nikula wrote:
The two remaining patches from [1], rebased.
BR,
Jani.
[1]
On 19/01/17 13:34, Ville Syrjälä wrote:
On Wed, Jan 18, 2017 at 11:05:18PM +0200, Martin Peres wrote:
On 16/12/16 15:48, Daniel Vetter wrote:
On Fri, Dec 16, 2016 at 12:29:05PM +0200, Jani Nikula wrote:
The two remaining patches from [1], rebased.
BR,
Jani.
[1]
On Fri, 20 Jan 2017, Manasi Navare wrote:
> On Fri, Jan 20, 2017 at 05:33:51PM +0200, Jani Nikula wrote:
>> On Fri, 20 Jan 2017, Manasi Navare wrote:
>> > v3:
>> > * Fix the conventions in bit definitions (Jani Nikula)
>> > v2:
>> > * Add all
== Series Details ==
Series: drm/i915/huc: Avoid attempting to authenticate non-existent fw
URL : https://patchwork.freedesktop.org/series/18300/
State : warning
== Summary ==
Series 18300v1 drm/i915/huc: Avoid attempting to authenticate non-existent fw
On Thu, Jan 19, 2017 at 10:56 PM, Jani Nikula wrote:
> On Fri, 20 Jan 2017, Anusha Srivatsa wrote:
>> With GLK was introduced IS_LP.
>> With this, we can use IS_GEN9 and IS_LP to put Skylake
>> and Kabylake in the same bucket. The main intention
On Fri, Jan 20, 2017 at 05:33:51PM +0200, Jani Nikula wrote:
> On Fri, 20 Jan 2017, Manasi Navare wrote:
> > v3:
> > * Fix the conventions in bit definitions (Jani Nikula)
> > v2:
> > * Add all the other DP Complianec TEST register defs (Jani Nikula)
> > Cc:
HuC authentication is called even if HuC firmware is not present in the
system, leading to NULL ptr dereference on not allocated gem_object.
Let's avoid trying to authenticate HuC if its firmware is not loaded
successfully.
Fixes: dac84a388528 ("drm/i915/huc: Support HuC authentication")
v2:
From: Ville Syrjälä
We are missing the AUX_DIST registers for pipes A and B. Add them,
and move the pipe C AUX_DIST registers next to the AUX_OFFSET registers
for a bit more convenience.
Signed-off-by: Ville Syrjälä
---
On Thu, Jan 12, 2017 at 3:47 AM, Ander Conselvan de Oliveira
wrote:
> Apply workarounds to Geminilake, and annoatate those that are applied
> uncondionally when they apply to GLK based on the workaround database.
>
> Signed-off-by: Ander Conselvan de
On Fri, Jan 20, 2017 at 04:56:08PM +0100, Michał Winiarski wrote:
> HuC firmware is mapped at GuC accessible range. Let's add an assert to
> verify that.
>
> v2: Split from previous commit (Michał)
> v3: Oops, hit send before compiling, s/huc_fw/huc->fw
And now I've amended the wrong patch...
On Fri, Jan 20, 2017 at 1:32 AM, Ander Conselvan De Oliveira
wrote:
> On Thu, 2017-01-19 at 10:39 -0800, Rodrigo Vivi wrote:
>> According to wa_database this Wa persist on GLK as it was on BXT.
>>
>> Cc: Tim Gore
>> Cc: Mika Kuoppala
There is no point in setting intel_dp->compliance.test_type, and
proceeding with the autotests, if we're about to NAK the request. Some
drive-by cleanups while at it.
Cc: Manasi Navare
Signed-off-by: Jani Nikula
---
HuC firmware is mapped at GuC accessible range. Let's add an assert to
verify that.
v2: Split from previous commit (Michał)
v3: Oops, hit send before compiling, s/huc_fw/huc->fw
Cc: Anusha Srivatsa
Cc: Arkadiusz Hiler
Cc: Chris Wilson
1 - 100 of 160 matches
Mail list logo