[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Ignore TMDS clock limit for DP++ when EDID override is set (rev2)

2018-01-09 Thread Patchwork
== Series Details == Series: drm/i915: Ignore TMDS clock limit for DP++ when EDID override is set (rev2) URL : https://patchwork.freedesktop.org/series/35338/ State : success == Summary == Series 35338v2 drm/i915: Ignore TMDS clock limit for DP++ when EDID override is set https://patchwork.f

[Intel-gfx] ✓ Fi.CI.IGT: success for igt/kms-crc-rotation: Fix flip subtests to follow kms guidelines.

2018-01-09 Thread Patchwork
== Series Details == Series: igt/kms-crc-rotation: Fix flip subtests to follow kms guidelines. URL : https://patchwork.freedesktop.org/series/36246/ State : success == Summary == Test kms_frontbuffer_tracking: Subgroup fbc-1p-offscren-pri-shrfb-draw-render: fail -

[Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [1/3] debugger: Convert to current intel_register_access_init()

2018-01-09 Thread Patchwork
== Series Details == Series: series starting with [1/3] debugger: Convert to current intel_register_access_init() URL : https://patchwork.freedesktop.org/series/36242/ State : failure == Summary == Test kms_frontbuffer_tracking: Subgroup fbc-1p-offscren-pri-shrfb-draw-render:

[Intel-gfx] ✓ Fi.CI.BAT: success for igt/kms-crc-rotation: Fix flip subtests to follow kms guidelines.

2018-01-09 Thread Patchwork
== Series Details == Series: igt/kms-crc-rotation: Fix flip subtests to follow kms guidelines. URL : https://patchwork.freedesktop.org/series/36246/ State : success == Summary == IGT patchset tested on top of latest successful build 834321a5d76a16783000441a02d7e79e72be9cc9 tools: Cannonlake po

[Intel-gfx] ✓ Fi.CI.IGT: success for overlay: Update .gitignore

2018-01-09 Thread Patchwork
== Series Details == Series: overlay: Update .gitignore URL : https://patchwork.freedesktop.org/series/36241/ State : success == Summary == Test kms_flip: Subgroup flip-vs-absolute-wf_vblank-interruptible: fail -> PASS (shard-hsw) fdo#100368 fdo#100368 http

[Intel-gfx] [PATCH i-g-t] igt/kms-crc-rotation: Fix flip subtests to follow kms guidelines.

2018-01-09 Thread Anusha Srivatsa
Fix flip subtest that used plaform names in igt_require() instead of testing if rotation property is supported on given combination of rotation/flips. Suggested-by: Daniel Vetter Cc: Daniel Vetter Signed-off-by: Anusha Srivatsa --- tests/kms_rotation_crc.c | 7 +-- 1 file changed, 1 inserti

[Intel-gfx] ✓ Fi.CI.IGT: success for pull in headers for major/minor/makedev funcs (rev2)

2018-01-09 Thread Patchwork
== Series Details == Series: pull in headers for major/minor/makedev funcs (rev2) URL : https://patchwork.freedesktop.org/series/9042/ State : success == Summary == Test gem_eio: Subgroup in-flight-contexts: notrun -> INCOMPLETE (shard-snb) fdo#104058 Test kms_front

Re: [Intel-gfx] [PATCH i-g-t] lib/igt_kms.c: Unconditionally include poll.h

2018-01-09 Thread Rhys Kidd
On 9 January 2018 at 05:11, Petri Latvala wrote: > Commit 98c64b33a793 ("lib/igt_kms: Drop all stale events on first > commit.") added a use of poll() to igt_kms.c, but that file only > includes poll.h when HAVE_UDEV is defined. Move the include outside > the UDEV conditional. > > Fixes: 98c64b33

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/3] debugger: Convert to current intel_register_access_init()

2018-01-09 Thread Patchwork
== Series Details == Series: series starting with [1/3] debugger: Convert to current intel_register_access_init() URL : https://patchwork.freedesktop.org/series/36242/ State : success == Summary == IGT patchset tested on top of latest successful build 834321a5d76a16783000441a02d7e79e72be9cc9

[Intel-gfx] ✗ Fi.CI.IGT: failure for include inttypes.h for PRI defines (rev2)

2018-01-09 Thread Patchwork
== Series Details == Series: include inttypes.h for PRI defines (rev2) URL : https://patchwork.freedesktop.org/series/9041/ State : failure == Summary == Test gem_shrink: Subgroup reclaim: pass -> DMESG-FAIL (shard-hsw) Test kms_frontbuffer_tracking: Subgr

[Intel-gfx] ✓ Fi.CI.BAT: success for overlay: Update .gitignore

2018-01-09 Thread Patchwork
== Series Details == Series: overlay: Update .gitignore URL : https://patchwork.freedesktop.org/series/36241/ State : success == Summary == IGT patchset tested on top of latest successful build 834321a5d76a16783000441a02d7e79e72be9cc9 tools: Cannonlake port clock programming with latest DRM-

Re: [Intel-gfx] [PATCH 2/3] drm/i915: Reinitialize sink scrambling/TMDS clock ratio on HPD

2018-01-09 Thread Sharma, Shashank
Regards Shashank On 1/9/2018 11:31 PM, Ville Syrjälä wrote: On Thu, Dec 28, 2017 at 08:32:05PM +0530, Sharma, Shashank wrote: On 12/22/2017 11:58 PM, Ville Syrjala wrote: From: Ville Syrjälä The LG 4k TV I have doesn't deassert HPD when I turn the TV off, but when I turn it back on it wil

[Intel-gfx] ✓ Fi.CI.BAT: success for pull in headers for major/minor/makedev funcs (rev2)

2018-01-09 Thread Patchwork
== Series Details == Series: pull in headers for major/minor/makedev funcs (rev2) URL : https://patchwork.freedesktop.org/series/9042/ State : success == Summary == IGT patchset tested on top of latest successful build 834321a5d76a16783000441a02d7e79e72be9cc9 tools: Cannonlake port clock prog

[Intel-gfx] [PATCH i-g-t 2/3] debugger: Clean up register access helper in debug_rdata

2018-01-09 Thread Rhys Kidd
Clean up the register access helper initialized with intel_register_access_init() by debug_rdata Signed-off-by: Rhys Kidd --- debugger/debug_rdata.c | 1 + 1 file changed, 1 insertion(+) diff --git a/debugger/debug_rdata.c b/debugger/debug_rdata.c index 45084b8e..4db7cbcd 100644 --- a/debugger/

[Intel-gfx] [PATCH i-g-t 1/3] debugger: Convert to current intel_register_access_init()

2018-01-09 Thread Rhys Kidd
Fixes: 301ad44c ("lib: Open debugfs files for the given DRM device") Signed-off-by: Rhys Kidd --- debugger/debug_rdata.c | 2 +- debugger/eudb.c| 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/debugger/debug_rdata.c b/debugger/debug_rdata.c index 61d82d9e..45084b8e 1

[Intel-gfx] [PATCH i-g-t 3/3] debugger: No longer rely on compatability define in intel_bufmgr.h

2018-01-09 Thread Rhys Kidd
Symbol rename from dri_* to drm_intel_* introduced a number of compatability defines within intel_bufmgr.h. Replace the old function with the new function, consistent with the balance of this file. Signed-off-by: Rhys Kidd --- debugger/eudb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-

[Intel-gfx] [PATCH i-g-t] overlay: Update .gitignore

2018-01-09 Thread Rhys Kidd
Fixes: 865a47ca ("overlay: parse tracepoints from sysfs to figure out fields' location") Signed-off-by: Rhys Kidd --- overlay/.gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/overlay/.gitignore b/overlay/.gitignore index bf91f08e..9ee5a356 100644 --- a/overlay/.gitignore +++ b/over

[Intel-gfx] ✓ Fi.CI.BAT: success for include inttypes.h for PRI defines (rev2)

2018-01-09 Thread Patchwork
== Series Details == Series: include inttypes.h for PRI defines (rev2) URL : https://patchwork.freedesktop.org/series/9041/ State : success == Summary == IGT patchset tested on top of latest successful build 834321a5d76a16783000441a02d7e79e72be9cc9 tools: Cannonlake port clock programming wi

[Intel-gfx] [PATCH i-g-t] include inttypes.h for PRI defines

2018-01-09 Thread Mike Frysinger
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96620 Signed-off-by: Mike Frysinger --- lib/igt_fb.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/igt_fb.c b/lib/igt_fb.c index dcae07dff021..ded639e833f1 100644 --- a/lib/igt_fb.c +++ b/lib/igt_fb.c @@ -27,6 +27,7 @@ #include

[Intel-gfx] [PATCH i-g-t] pull in headers for major/minor/makedev funcs

2018-01-09 Thread Mike Frysinger
Use the portable autoconf helper to figure out which header to include for these funcs. Linux C libs are moving to not implicitly include them via sys/types.h anymore, and other OS's have long required you to pull in the right header directly. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?i

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Use the engine name directly in the error_state file (rev2)

2018-01-09 Thread Patchwork
== Series Details == Series: drm/i915: Use the engine name directly in the error_state file (rev2) URL : https://patchwork.freedesktop.org/series/36215/ State : success == Summary == Test kms_flip: Subgroup flip-vs-absolute-wf_vblank-interruptible: fail -> PASS

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/bxt: Enable VBT based BL control for DP (rev4)

2018-01-09 Thread Patchwork
== Series Details == Series: drm/i915/bxt: Enable VBT based BL control for DP (rev4) URL : https://patchwork.freedesktop.org/series/25323/ State : failure == Summary == Applying: drm/i915/bxt: Enable VBT based BL control for DP (v2) error: Failed to merge in the changes. Using index info to re

[Intel-gfx] [PATCH] drm/i915/bxt: Enable VBT based BL control for DP (v2)

2018-01-09 Thread Mustamin B Mustaffa
Currently, BXT_PP is hardcoded with value '0'. It practically disabled eDP backlight on MRB (BXT) platform. This patch will tell which BXT_PP registers (there are two set of PP_CONTROL in the spec) to be used as defined in VBT (Video Bios Timing table) and this will enabled eDP backlight controll

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Use the engine name directly in the error_state file (rev2)

2018-01-09 Thread Patchwork
== Series Details == Series: drm/i915: Use the engine name directly in the error_state file (rev2) URL : https://patchwork.freedesktop.org/series/36215/ State : success == Summary == Series 36215v2 drm/i915: Use the engine name directly in the error_state file https://patchwork.freedesktop.org

[Intel-gfx] [PATCH v2] drm/i915: Use the engine name directly in the error_state file

2018-01-09 Thread Michel Thierry
Instead of using local string names that we will have to keep maintaining, use the engine->name directly. v2: Better invalid engine_id handling, capture_bo will not be able know the engine_id and end up with -1 (Michal). Suggested-by: Michal Wajdeczko Signed-off-by: Michel Thierry Cc: Michal Wa

Re: [Intel-gfx] [PATCH 02/27] drm/i915/icl: Add the ICL PCI IDs

2018-01-09 Thread Oscar Mateo
On 01/09/2018 05:02 PM, De Marchi, Lucas wrote: On Tue, 2018-01-09 at 16:09 -0800, Oscar Mateo wrote: On 01/09/2018 03:23 PM, Paulo Zanoni wrote: This is the current PCI ID list in our documentation. Let's leave the _gt#_ part out for now since our current documentation is not 100% clear and

Re: [Intel-gfx] [PATCH 02/27] drm/i915/icl: Add the ICL PCI IDs

2018-01-09 Thread De Marchi, Lucas
On Tue, 2018-01-09 at 16:09 -0800, Oscar Mateo wrote: > > On 01/09/2018 03:23 PM, Paulo Zanoni wrote: > > This is the current PCI ID list in our documentation. > > > > Let's leave the _gt#_ part out for now since our current documentation > > is not 100% clear and we don't need this info now anyw

Re: [Intel-gfx] [PATCH] drm/i915: add support for specifying DMC firmware override by module param

2018-01-09 Thread Srivatsa, Anusha
Hi, This patch could be really helpful Is there any issue with it? It's not merged yet. Regards, Anusha >-Original Message- >From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf Of >Jani Nikula >Sent: Wednesday, November 22, 2017 8:20 AM >To: Joonas Lahtine

Re: [Intel-gfx] [PATCH] drm/i915: Use the engine name directly in the error_state file

2018-01-09 Thread Michel Thierry
On 09/01/18 13:37, Michel Thierry wrote: On 09/01/18 12:46, Chris Wilson wrote: Quoting Michal Wajdeczko (2018-01-09 20:39:09) On Tue, 09 Jan 2018 20:33:55 +0100, Michel Thierry wrote: Instead of using local string names that we will have to keep maintaining, use the engine->name directly.

Re: [Intel-gfx] [PATCH 01/11] drm/i915/cnl: Add Cannonlake PCI IDs for another SKU.

2018-01-09 Thread Lucas De Marchi
On Tue, Jan 9, 2018 at 10:48 AM, Paulo Zanoni wrote: > Em Sex, 2017-12-22 às 15:18 -0800, Rodrigo Vivi escreveu: >> By the Spec all CNL skus are GT2. > > This is definitely not my understanding, some of the PCI IDs in our > driver are clearly marked as GT1 on the spec. > > But since we don't use t

Re: [Intel-gfx] [PATCH 02/27] drm/i915/icl: Add the ICL PCI IDs

2018-01-09 Thread Oscar Mateo
On 01/09/2018 03:23 PM, Paulo Zanoni wrote: This is the current PCI ID list in our documentation. Let's leave the _gt#_ part out for now since our current documentation is not 100% clear and we don't need this info now anyway. v2: Use the new ICL_11 naming (Kelvin Gardiner). v3: Latest IDs as

Re: [Intel-gfx] [PATCH 01/27] drm/i915/icl: Add initial Icelake definitions.

2018-01-09 Thread Oscar Mateo
On 01/09/2018 03:23 PM, Paulo Zanoni wrote: From: Rodrigo Vivi Icelake is a Intel® Processor containing Intel® HD Graphics. This is just an initial Icelake definition. PCI IDs, Icelake support and new features coming in following patches. v2: Add .ddb_size and .has_guc (Michal Wajdeczko). v

[Intel-gfx] [PATCH 27/27] drm/i915/gen11: add support for reading the timestamp frequency

2018-01-09 Thread Paulo Zanoni
The only thing that differs here is that the crystal clock freq now has four possible values. This patch gets rid of the "Unknown gen, unable to compute..." message at boot for gen11. Reviewed-by: Lionel Landwerlin Signed-off-by: Paulo Zanoni --- drivers/gpu/drm/i915/i915_reg.h | 6 +

[Intel-gfx] [PATCH 25/27] drm/i915/icl: Enable RC6 and RPS in Gen11

2018-01-09 Thread Paulo Zanoni
From: Oscar Mateo AFAICT, once the new interrupt is in place, the rest should behave the same as Gen10. v2: Update ring frequencies (Sagar) Cc: Daniele Ceraolo Spurio Cc: Sagar Arun Kamble Cc: Paulo Zanoni Signed-off-by: Oscar Mateo --- drivers/gpu/drm/i915/i915_debugfs.c | 10 +-

[Intel-gfx] [PATCH 23/27] drm/i915/icl: Split out the servicing of the Selector and Shared IIR registers

2018-01-09 Thread Paulo Zanoni
From: Oscar Mateo Both for clarity and so that we can reuse it later on. v2: - local_clock returns a u64 (Tvrtko) - Use the funky BIT(bit) version (Tvrtko) - wait_start not required (Tvrtko) - Use time_after64 (Oscar) Cc: Tvrtko Ursulin Cc: Daniele Ceraolo Spurio Signed-off-by: Oscar

[Intel-gfx] [PATCH 20/27] drm/i915/icl: Make use of the SW counter field in the new context descriptor

2018-01-09 Thread Paulo Zanoni
From: Oscar Mateo The new context descriptor format in Gen11 contains two assignable fields: the SW Context ID (technically 11 bits, but practically limited to 2032 entries due to some being reserved for future use by the GuC) and the SW Counter (6 bits). We don't want to limit ourselves too muc

[Intel-gfx] [PATCH 17/27] drm/i915/icl: Enable the extra video decode and enhancement boxes for Icelake 11

2018-01-09 Thread Paulo Zanoni
From: Oscar Mateo Icelake 11 has one vebox and two vdboxes (0 and 2). Bspec: 21140 v2: Split out in two (Daniele) Cc: Daniele Ceraolo Spurio Signed-off-by: Oscar Mateo --- drivers/gpu/drm/i915/i915_pci.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/i915_pci.c b/d

[Intel-gfx] [PATCH 21/27] drm/i915/icl: Add reset control register changes

2018-01-09 Thread Paulo Zanoni
From: Michel Thierry The bits used to reset the different engines/domains have changed in GEN11, this patch maps the reset engine mask bits with the new bits in the reset control register. v2: Use shift-left instead of BIT macro to match the file style (Paulo). v3: Reuse gen8_reset_engines (Dani

[Intel-gfx] [PATCH 19/27] drm/i915/icl: Added ICL 11 slice, subslice and EU fuse detection

2018-01-09 Thread Paulo Zanoni
From: Kelvin Gardiner This patch adds support to detect ICL, slice, subslice and EU fuse settings. Add addresses for ICL 11 slice, subslice and EU fuses registers. These register addresses are the same as previous platforms but the format and / or the meaning of the information is different. The

[Intel-gfx] [PATCH 24/27] drm/i915/icl: Handle RPS interrupts correctly for Gen11

2018-01-09 Thread Paulo Zanoni
From: Oscar Mateo Using the new hierarchical interrupt infrastructure. Cc: Tvrtko Ursulin Cc: Daniele Ceraolo Spurio Cc: Sagar Arun Kamble Cc: Paulo Zanoni Signed-off-by: Oscar Mateo --- drivers/gpu/drm/i915/i915_irq.c | 68 +--- drivers/gpu/drm/i915/in

[Intel-gfx] [PATCH 22/27] drm/i915/icl: Add configuring MOCS in new Icelake engines

2018-01-09 Thread Paulo Zanoni
From: Tomasz Lis In Icelake, there are more engines on which Memory Object Control States need to be configured. Besides adding Icelake under Skylake config, the patch makes sure MOCS register addresses for the new engines are properly defined. Additional patch might be need later, in case the s

[Intel-gfx] [PATCH 26/27] drm/i915/icl: allow the reg_read ioctl to read the RCS TIMESTAMP register

2018-01-09 Thread Paulo Zanoni
This enables the Mesa driver to advertise support for ARB_timer_query, and thus an OpenGL version higher than 3.2. Based on the CNL patch by Nanley Chery. v2: Rebase. Cc: Anuj Phogat Cc: Nanley Chery Cc: Rodrigo Vivi Requested-by: Anuj Phogat Tested-by: Anuj Phogat Signed-off-by: Paulo Zano

[Intel-gfx] [PATCH 18/27] drm/i915/icl: Update subslice define for ICL 11

2018-01-09 Thread Paulo Zanoni
From: Kelvin Gardiner ICL 11 has a greater number of maximum subslices. This patch updates the subslice max define to reflect this. Bspec: 21139 Reviewed-by: Oscar Mateo Reviewed-by: Daniele Ceraolo Spurio Signed-off-by: Kelvin Gardiner --- drivers/gpu/drm/i915/intel_ringbuffer.h | 2 +- 1

[Intel-gfx] [PATCH 13/27] drm/i915/icl: Gen11 forcewake support

2018-01-09 Thread Paulo Zanoni
From: Daniele Ceraolo Spurio The main difference with previous GENs is that starting from Gen11 each VCS and VECS engine has its own power well, which only exist if the related engine exists in the HW. The fallback forcewake request workaround is only needed on gen9 according to the HSDES WA entr

[Intel-gfx] [PATCH 15/27] drm/i915/icl: new context descriptor support

2018-01-09 Thread Paulo Zanoni
From: "Ceraolo Spurio, Daniele" Starting from Gen11 the context descriptor format has been updated in the HW. The hw_id field has been considerably reduced in size and engine class and instance fields have been added. There is a slight name clashing issue because the field that we call hw_id is

[Intel-gfx] [PATCH 11/27] drm/i915/icl: Gen11 render context size

2018-01-09 Thread Paulo Zanoni
From: Tvrtko Ursulin The current size may be bigger than the correct one, this needs to be revisited later. v2: Rebase. Acked-by: Ben Widawsky Signed-off-by: Tvrtko Ursulin Signed-off-by: Rodrigo Vivi --- drivers/gpu/drm/i915/intel_engine_cs.c | 2 ++ 1 file changed, 2 insertions(+) diff -

[Intel-gfx] [PATCH 10/27] drm/i915/icl: Enhanced execution list support

2018-01-09 Thread Paulo Zanoni
From: Thomas Daniel Supports two-element submission using the new enhanced execlist mechanism v2: Rebase. v3: Switch from !IS_GEN11 to GEN < 11 (Daniele Ceraolo Spurio). v4: Use the elsq registers instead of elsp. (Daniele Ceraolo Spurio) Signed-off-by: Thomas Daniel Signed-off-by: Rodrigo Viv

[Intel-gfx] [PATCH 14/27] drm/i915/icl: Set graphics mode register for gen11

2018-01-09 Thread Paulo Zanoni
From: kgardine This patch clears a single bit. The bit is 0 by default but expected not to be set. Explicitly clearing the bit in this patch is intended to indicate some thinking has occurred, and that we want this bit cleared and we are not just excepting the default value. v2 (from Paulo): fix

[Intel-gfx] [PATCH 16/27] drm/i915/icl: Check for fused-off VDBOX and VEBOX instances

2018-01-09 Thread Paulo Zanoni
From: Oscar Mateo In Gen11, the Video Decode engines (aka VDBOX, aka VCS, aka BSD) and the Video Enhancement engines (aka VEBOX, aka VECS) could be fused off. Also, each VDBOX and VEBOX has its own power well, which only exist if the related engine exists in the HW. Unfortunately, we have a Catc

[Intel-gfx] [PATCH 12/27] drm/i915/icl: Add Indirect Context Offset for Gen11

2018-01-09 Thread Paulo Zanoni
From: Michel Thierry v2: rebased to intel_lr_indirect_ctx_offset Signed-off-by: Michel Thierry Signed-off-by: Rodrigo Vivi Signed-off-by: Michal Wajdeczko --- drivers/gpu/drm/i915/intel_lrc.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu/drm/i915/intel_lrc.c b/drivers/

[Intel-gfx] [PATCH 06/27] drm/i915/icl: Prepare for more rings

2018-01-09 Thread Paulo Zanoni
From: Tvrtko Ursulin Gen11 will add more VCS and VECS rings so prepare the infrastructure to support that. Bspec: 7021 v2: Rebase. v3: Rebase. v4: Rebase. v5: Rebase. v6: - Update for POR changes. (Daniele Ceraolo Spurio) - Add provisional guc engine ids - to be checked and confirmed. v7:

[Intel-gfx] [PATCH 02/27] drm/i915/icl: Add the ICL PCI IDs

2018-01-09 Thread Paulo Zanoni
This is the current PCI ID list in our documentation. Let's leave the _gt#_ part out for now since our current documentation is not 100% clear and we don't need this info now anyway. v2: Use the new ICL_11 naming (Kelvin Gardiner). v3: Latest IDs as per BSpec (Oscar). v4: Make it compile (Paulo).

[Intel-gfx] [PATCH 03/27] drm/i915/icl: add icelake_init_clock_gating()

2018-01-09 Thread Paulo Zanoni
For now it does nothing, except for avoiding a MISSING_CASE. v2: Rebase. Signed-off-by: Paulo Zanoni --- drivers/gpu/drm/i915/intel_pm.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c index 1db79a860b

[Intel-gfx] [PATCH 01/27] drm/i915/icl: Add initial Icelake definitions.

2018-01-09 Thread Paulo Zanoni
From: Rodrigo Vivi Icelake is a Intel® Processor containing Intel® HD Graphics. This is just an initial Icelake definition. PCI IDs, Icelake support and new features coming in following patches. v2: Add .ddb_size and .has_guc (Michal Wajdeczko). v3: Add the ICL_FEATURES macro (Kelvin Gardiner).

[Intel-gfx] [PATCH 09/27] drm/i915/icl: Correctly initialize the Gen11 engines

2018-01-09 Thread Paulo Zanoni
From: Oscar Mateo Gen11 has up to 4 VCS and up to 2 VECS engines, this patch adds mmio base definitions for all of them. Bspec: 20944 Bspec: 7021 v2: Set the correct mmio_base in intel_engines_init_mmio; updating the base mmio values any later would cause incorrect reads in i915_gem_sanitize (M

[Intel-gfx] [PATCH 04/27] drm/i915/icl: Icelake interrupt register addresses and bits

2018-01-09 Thread Paulo Zanoni
From: Tvrtko Ursulin MMIO addresses and register definition for the new interrupt registers in Gen11. v2: Removed spelt out VCS and VECS bit definitions. (Daniel Vetter) v3: Adjust VCS and VECS. (Daniele Ceraolo Spurio) v4: Bikeshedding (Paulo). Cc: Ceraolo Spurio, Daniele Signed-off-by: Tvrtk

[Intel-gfx] [PATCH 07/27] drm/i915/icl: Interrupt handling

2018-01-09 Thread Paulo Zanoni
From: Tvrtko Ursulin v2: Rebase. v3: * Remove DPF, it has been removed from SKL+. * Fix -internal rebase wrt. execlists interrupt handling. v4: Rebase. v5: * Updated for POR changes. (Daniele Ceraolo Spurio) * Merged with irq handling fixes by Daniele Ceraolo Spurio: * Simplify t

[Intel-gfx] [PATCH 05/27] drm/i915/icl: Show interrupt registers in debugfs

2018-01-09 Thread Paulo Zanoni
From: Tvrtko Ursulin v2: Update for POR changes. (Daniele Ceraolo Spurio) Signed-off-by: Tvrtko Ursulin Signed-off-by: Rodrigo Vivi Cc: Ceraolo Spurio, Daniele --- drivers/gpu/drm/i915/i915_debugfs.c | 82 - 1 file changed, 81 insertions(+), 1 deletion(-)

[Intel-gfx] [PATCH 08/27] drm/i915/icl: Ringbuffer interrupt handling

2018-01-09 Thread Paulo Zanoni
From: Tvrtko Ursulin Since it is not possible to mask individual engine instances and they are all permanently unmasked we do not need to do anything for engine interrupt management. v2: Rebase. v3: Remove gen 11 extra check in logical_render_ring_init. v4: Rebase fixes. v5: Rebase/refactor. v6:

[Intel-gfx] [PATCH 00/27] ICL basic enabling + GEM

2018-01-09 Thread Paulo Zanoni
Hello This is the first series of patches for the Icelake platform. Unlike the other series that introduced new platforms, this one is very small and only contains patches for very basic enabling, interrupts and some GEM code. No patches for display or other subsystems yet and GEM is not complete

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: Use the engine name directly in the error_state file

2018-01-09 Thread Patchwork
== Series Details == Series: drm/i915: Use the engine name directly in the error_state file URL : https://patchwork.freedesktop.org/series/36215/ State : failure == Summary == Test gem_tiled_swapping: Subgroup non-threaded: pass -> DMESG-WARN (shard-hsw) fdo#10421

Re: [Intel-gfx] [PATCH i-g-t 2/2] tests/perf_pmu: Exercise busy stats and lite-restore

2018-01-09 Thread Chris Wilson
Quoting Tvrtko Ursulin (2018-01-09 16:16:21) > From: Tvrtko Ursulin > > While developing a fix for an accounting hole in busy stats we realized > lite-restore is a potential edge case which would be interesting to check > is properly handled. > > It is unfortnately quite timing sensitive to hit

Re: [Intel-gfx] [PATCH] drm/i915: Use the engine name directly in the error_state file

2018-01-09 Thread Michel Thierry
On 09/01/18 12:46, Chris Wilson wrote: Quoting Michal Wajdeczko (2018-01-09 20:39:09) On Tue, 09 Jan 2018 20:33:55 +0100, Michel Thierry wrote: Instead of using local string names that we will have to keep maintaining, use the engine->name directly. Suggested-by: Michal Wajdeczko Signed-off

Re: [Intel-gfx] [PATCH i-g-t 1/2] tests/perf_pmu: Verify busyness when PMU is enabled after engine got busy

2018-01-09 Thread Chris Wilson
Quoting Tvrtko Ursulin (2018-01-09 16:16:20) > From: Tvrtko Ursulin > > Make sure busyness is correctly reported when PMU is enabled after the > engine is already busy with a single long batch. > > Signed-off-by: Tvrtko Ursulin > --- > tests/perf_pmu.c | 42

Re: [Intel-gfx] [PATCH] drm/i915: Use the engine name directly in the error_state file

2018-01-09 Thread Chris Wilson
Quoting Michal Wajdeczko (2018-01-09 20:39:09) > On Tue, 09 Jan 2018 20:33:55 +0100, Michel Thierry > wrote: > > > Instead of using local string names that we will have to keep > > maintaining, use the engine->name directly. > > > > Suggested-by: Michal Wajdeczko > > Signed-off-by: Michel Thie

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Ignore TMDS clock limit for DP++ when EDID override is set (rev2)

2018-01-09 Thread Patchwork
== Series Details == Series: drm/i915: Ignore TMDS clock limit for DP++ when EDID override is set (rev2) URL : https://patchwork.freedesktop.org/series/35338/ State : success == Summary == Series 35338v2 drm/i915: Ignore TMDS clock limit for DP++ when EDID override is set https://patchwork.f

Re: [Intel-gfx] [PATCH] drm/i915: Use the engine name directly in the error_state file

2018-01-09 Thread Michal Wajdeczko
On Tue, 09 Jan 2018 20:33:55 +0100, Michel Thierry wrote: Instead of using local string names that we will have to keep maintaining, use the engine->name directly. Suggested-by: Michal Wajdeczko Signed-off-by: Michel Thierry Cc: Michal Wajdeczko --- drivers/gpu/drm/i915/i915_gpu_error.c

Re: [Intel-gfx] [PATCH] drm: i915: Fix audio issue on BXT

2018-01-09 Thread Runyan, Arthur J
Sorry, I've been out. I'm checking on this. -Original Message- From: Pandiyan, Dhinakaran Sent: Thursday, 4 January, 2018 2:00 PM To: Singh, Gaurav K Cc: intel-gfx@lists.freedesktop.org; Vivi, Rodrigo ; subransu.s.pru...@intel.com; Runyan, Arthur J Subject: Re: [Intel-gfx] [PATCH] drm

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Use the engine name directly in the error_state file

2018-01-09 Thread Patchwork
== Series Details == Series: drm/i915: Use the engine name directly in the error_state file URL : https://patchwork.freedesktop.org/series/36215/ State : success == Summary == Series 36215v1 drm/i915: Use the engine name directly in the error_state file https://patchwork.freedesktop.org/api/1.

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Don't allow HDCP on PORT E/F

2018-01-09 Thread Patchwork
== Series Details == Series: drm/i915: Don't allow HDCP on PORT E/F URL : https://patchwork.freedesktop.org/series/36214/ State : failure == Summary == Applying: drm/i915: Don't allow HDCP on PORT E/F Using index info to reconstruct a base tree... M drivers/gpu/drm/i915/intel_hdmi.c Fall

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Only disable HDCP when it's active

2018-01-09 Thread Patchwork
== Series Details == Series: drm/i915: Only disable HDCP when it's active URL : https://patchwork.freedesktop.org/series/36212/ State : failure == Summary == Applying: drm/i915: Only disable HDCP when it's active Using index info to reconstruct a base tree... M drivers/gpu/drm/i915/intel

[Intel-gfx] [PATCH] drm/i915: Use the engine name directly in the error_state file

2018-01-09 Thread Michel Thierry
Instead of using local string names that we will have to keep maintaining, use the engine->name directly. Suggested-by: Michal Wajdeczko Signed-off-by: Michel Thierry Cc: Michal Wajdeczko --- drivers/gpu/drm/i915/i915_gpu_error.c | 23 ++- 1 file changed, 10 insertions(+),

[Intel-gfx] ✗ Fi.CI.IGT: warning for series starting with [1/2] tests/perf_pmu: Verify busyness when PMU is enabled after engine got busy

2018-01-09 Thread Patchwork
== Series Details == Series: series starting with [1/2] tests/perf_pmu: Verify busyness when PMU is enabled after engine got busy URL : https://patchwork.freedesktop.org/series/36201/ State : warning == Summary == Test gem_eio: Subgroup in-flight: pass -> DMESG-W

[Intel-gfx] [PATCH] drm/i915: Don't allow HDCP on PORT E/F

2018-01-09 Thread Sean Paul
Port E doesn't have HDCP support, and Port F is disabled. Don't setup the hdcp shim on those. Reviewed-by: Daniel Vetter Signed-off-by: Sean Paul --- drivers/gpu/drm/i915/intel_hdmi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_hdmi.c b/driv

[Intel-gfx] [PATCH] drm/i915: Only disable HDCP when it's active

2018-01-09 Thread Sean Paul
Instead of always trying to disable HDCP. Only run hdcp_disable when the state is not UNDESIRED. This will catch cases where it's enabled and also cases where enable failed and the state is left in DESIRED mode. Note that things won't blow up if disable is attempted while already disabled, it's ju

Re: [Intel-gfx] [PATCH 01/11] drm/i915/cnl: Add Cannonlake PCI IDs for another SKU.

2018-01-09 Thread Paulo Zanoni
Em Sex, 2017-12-22 às 15:18 -0800, Rodrigo Vivi escreveu: > By the Spec all CNL skus are GT2. This is definitely not my understanding, some of the PCI IDs in our driver are clearly marked as GT1 on the spec. But since we don't use this GTX number anywhere for CNL for the Kernel driver, can't we j

Re: [Intel-gfx] [PATCH 2/3] drm/i915: Reinitialize sink scrambling/TMDS clock ratio on HPD

2018-01-09 Thread Ville Syrjälä
On Thu, Dec 28, 2017 at 08:32:05PM +0530, Sharma, Shashank wrote: > > > On 12/22/2017 11:58 PM, Ville Syrjala wrote: > > From: Ville Syrjälä > > > > The LG 4k TV I have doesn't deassert HPD when I turn the TV off, but > > when I turn it back on it will pulse the HPD line. By that time it has > >

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] tests/perf_pmu: Verify busyness when PMU is enabled after engine got busy

2018-01-09 Thread Patchwork
== Series Details == Series: series starting with [1/2] tests/perf_pmu: Verify busyness when PMU is enabled after engine got busy URL : https://patchwork.freedesktop.org/series/36201/ State : success == Summary == IGT patchset tested on top of latest successful build 834321a5d76a16783000441a0

[Intel-gfx] [PATCH i-g-t 1/2] tests/perf_pmu: Verify busyness when PMU is enabled after engine got busy

2018-01-09 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Make sure busyness is correctly reported when PMU is enabled after the engine is already busy with a single long batch. Signed-off-by: Tvrtko Ursulin --- tests/perf_pmu.c | 42 ++ 1 file changed, 42 insertions(+) diff --git a/tests/

[Intel-gfx] [PATCH i-g-t 2/2] tests/perf_pmu: Exercise busy stats and lite-restore

2018-01-09 Thread Tvrtko Ursulin
From: Tvrtko Ursulin While developing a fix for an accounting hole in busy stats we realized lite-restore is a potential edge case which would be interesting to check is properly handled. It is unfortnately quite timing sensitive to hit lite-restore in the fashion test needs, so downside of this

[Intel-gfx] ✗ Fi.CI.IGT: failure for lib/igt_kms: Fix build warning in igt_display_drop_events

2018-01-09 Thread Patchwork
== Series Details == Series: lib/igt_kms: Fix build warning in igt_display_drop_events URL : https://patchwork.freedesktop.org/series/36190/ State : failure == Summary == Test kms_cursor_crc: Subgroup cursor-64x64-suspend: pass -> SKIP (shard-hsw) fdo#103540

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: Fix using BIT_ULL() vs. BIT() for power domain masks

2018-01-09 Thread Patchwork
== Series Details == Series: drm/i915: Fix using BIT_ULL() vs. BIT() for power domain masks URL : https://patchwork.freedesktop.org/series/36188/ State : failure == Summary == Test kms_frontbuffer_tracking: Subgroup fbc-1p-primscrn-pri-indfb-draw-blt: fail -> PASS

Re: [Intel-gfx] [-next PATCH 3/4] treewide: Use DEVICE_ATTR_RO

2018-01-09 Thread Greg Kroah-Hartman
On Thu, Dec 21, 2017 at 11:34:10AM +0200, Sakari Ailus wrote: > Hi Joe, > > On Tue, Dec 19, 2017 at 10:15:08AM -0800, Joe Perches wrote: > > diff --git a/drivers/staging/media/atomisp/pci/atomisp2/hmm/hmm.c > > b/drivers/staging/media/atomisp/pci/atomisp2/hmm/hmm.c > > index a1c81c12718c..4338b8a

Re: [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Fix using BIT_ULL() vs. BIT() for power domain masks

2018-01-09 Thread Imre Deak
On Tue, Jan 09, 2018 at 01:33:48PM +, Patchwork wrote: > == Series Details == > > Series: drm/i915: Fix using BIT_ULL() vs. BIT() for power domain masks > URL : https://patchwork.freedesktop.org/series/36188/ > State : success Thanks for the reviews, pushed to -dinq. > > == Summary == >

Re: [Intel-gfx] [RFC v2 2/8] drm/ioctl: Remove trailing whitespace

2018-01-09 Thread Laurent Pinchart
Hi Noralf, Thank you for the patch. On Thursday, 4 January 2018 00:21:04 EET Noralf Trønnes wrote: > Remove a couple of trailing spaces. > > Signed-off-by: Noralf Trønnes Reviewed-by: Laurent Pinchart > --- > drivers/gpu/drm/drm_ioctl.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletio

Re: [Intel-gfx] [RFC v2 3/8] drm: Export some ioctl functions

2018-01-09 Thread Laurent Pinchart
Hi Noralf, Thank you for the patch. On Thursday, 4 January 2018 00:21:05 EET Noralf Trønnes wrote: > Export the following functions so in-kernel users can allocate > dumb buffers: > - drm_file_alloc > - drm_file_free > - drm_prime_handle_to_fd_ioctl > - drm_mode_addfb2 > - drm_mode_create_dumb_io

Re: [Intel-gfx] [PATCH v2] drm/i915/pmu: Only enumerate available counters in sysfs

2018-01-09 Thread Chris Wilson
Quoting Tvrtko Ursulin (2018-01-09 12:20:05) > From: Tvrtko Ursulin > > Switch over to dynamically creating device attributes, which are in turn > used by the perf core to expose available counters in sysfs. > > This way we do not expose counters which are not avaiable on the current > platform,

Re: [Intel-gfx] [PATCH] drm/i915: Fix using BIT_ULL() vs. BIT() for power domain masks

2018-01-09 Thread Joonas Lahtinen
On Tue, 2018-01-09 at 14:20 +0200, Imre Deak wrote: > The power domain masks are 64 bit wide, so we need BIT_ULL() when > setting bits in them, these ones were missed during converting from 32 > to 64 bit masks. All 3 enums are <32 atm, so this didn't cause a real > problem. > > Fixes: d8fc70b7367

Re: [Intel-gfx] [PATCH i-g-t 6/6] i-g-t: kms_plane_scaling: test for multi pipe with scaling

2018-01-09 Thread Maarten Lankhorst
Op 13-12-17 om 10:50 schreef Vidya Srinivas: > From: Jyoti Yadav > > Patch adds subtest to display primary and overlay planes on two > connected pipes and runs scaling test on both pipes > > Signed-off-by: Jyoti Yadav > Signed-off-by: Mahesh Kumar > Signed-off-by: Vidya Srinivas > --- > tests/

[Intel-gfx] ✓ Fi.CI.BAT: success for lib/igt_kms: Fix build warning in igt_display_drop_events

2018-01-09 Thread Patchwork
== Series Details == Series: lib/igt_kms: Fix build warning in igt_display_drop_events URL : https://patchwork.freedesktop.org/series/36190/ State : success == Summary == IGT patchset tested on top of latest successful build 834321a5d76a16783000441a02d7e79e72be9cc9 tools: Cannonlake port clock

Re: [Intel-gfx] [PATCH] drm/i915: Fix using BIT_ULL() vs. BIT() for power domain masks

2018-01-09 Thread Chris Wilson
Quoting Imre Deak (2018-01-09 12:20:40) > The power domain masks are 64 bit wide, so we need BIT_ULL() when > setting bits in them, these ones were missed during converting from 32 > to 64 bit masks. All 3 enums are <32 atm, so this didn't cause a real > problem. > > Fixes: d8fc70b7367b ("drm/i915

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Fix using BIT_ULL() vs. BIT() for power domain masks

2018-01-09 Thread Patchwork
== Series Details == Series: drm/i915: Fix using BIT_ULL() vs. BIT() for power domain masks URL : https://patchwork.freedesktop.org/series/36188/ State : success == Summary == Series 36188v1 drm/i915: Fix using BIT_ULL() vs. BIT() for power domain masks https://patchwork.freedesktop.org/api/1.

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/pmu: Only enumerate available counters in sysfs (rev2)

2018-01-09 Thread Patchwork
== Series Details == Series: drm/i915/pmu: Only enumerate available counters in sysfs (rev2) URL : https://patchwork.freedesktop.org/series/35689/ State : success == Summary == Series 35689v2 drm/i915/pmu: Only enumerate available counters in sysfs https://patchwork.freedesktop.org/api/1.0/ser

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: Ignore TMDS clock limit for DP++ when EDID override is set (rev2)

2018-01-09 Thread Patchwork
== Series Details == Series: drm/i915: Ignore TMDS clock limit for DP++ when EDID override is set (rev2) URL : https://patchwork.freedesktop.org/series/35338/ State : failure == Summary == Warning: bzip CI_DRM_3609/shard-glkb6/results32.json.bz2 wasn't in correct JSON format Test kms_cursor_

[Intel-gfx] [PATCH i-g-t] lib/igt_kms: Fix build warning in igt_display_drop_events

2018-01-09 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Check read(2) return for robustness and to silence gcc. Signed-off-by: Tvrtko Ursulin Cc: Maarten Lankhorst -- Compile tested only. --- lib/igt_kms.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/lib/igt_kms.c b/lib/igt_kms.c index ec3b716791a2.

Re: [Intel-gfx] [PATCH i-g-t 4/6] i-g-t kms_plane_scaling: test scaling with tiling rotation and pixel formats

2018-01-09 Thread Maarten Lankhorst
Op 14-12-17 om 11:55 schreef Daniel Vetter: > Just a quick comment at the bottom. > > On Wed, Dec 13, 2017 at 03:20:50PM +0530, Vidya Srinivas wrote: >> @@ -312,23 +480,40 @@ static void test_plane_scaling(data_t *d, enum pipe >> pipe) >> igt_require_f(valid_tests, "no valid crtc/connector co

Re: [Intel-gfx] [PATCH i-g-t 2/6] i-g-t: lib: Add plane pixel format support

2018-01-09 Thread Maarten Lankhorst
Op 09-01-18 om 13:10 schreef Maarten Lankhorst: > Op 13-12-17 om 10:50 schreef Vidya Srinivas: >> From: Mahesh Kumar >> >> This patch adds the following: >> - Query supported pixel formats from kernel for a given >> plane. >> - Get number of supported pixel formats for a plane >> - Check if format

Re: [Intel-gfx] [PATCH i-g-t 3/6] i-g-t: lib/igt_kms: Run kms_plane for all supported pixel formats

2018-01-09 Thread Maarten Lankhorst
Op 13-12-17 om 10:50 schreef Vidya Srinivas: > From: Mahesh Kumar > > This patch adds a subtest related to pixel format testing. The test > create framebuffer with all supported pixel formats for primary and > sprite planes which can be drawn using cairo and commits the same on display. > > Signed

[Intel-gfx] [PATCH] drm/i915: Fix using BIT_ULL() vs. BIT() for power domain masks

2018-01-09 Thread Imre Deak
The power domain masks are 64 bit wide, so we need BIT_ULL() when setting bits in them, these ones were missed during converting from 32 to 64 bit masks. All 3 enums are <32 atm, so this didn't cause a real problem. Fixes: d8fc70b7367b ("drm/i915: Make power domain masks 64 bit long") Cc: Joonas L

[Intel-gfx] [PATCH v2] drm/i915/pmu: Only enumerate available counters in sysfs

2018-01-09 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Switch over to dynamically creating device attributes, which are in turn used by the perf core to expose available counters in sysfs. This way we do not expose counters which are not avaiable on the current platform, and are so more consistent between what we reply to open a

  1   2   >