[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915/icl: Assign genlock CRTC pointer in all slave CRTC states for tiled displays

2019-03-21 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915/icl: Assign genlock CRTC pointer in all slave CRTC states for tiled displays URL : https://patchwork.freedesktop.org/series/58393/ State : success == Summary == CI Bug Log - changes from CI_DRM_5792 -> Patchwork_12567

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/2] drm/i915/icl: Assign genlock CRTC pointer in all slave CRTC states for tiled displays

2019-03-21 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915/icl: Assign genlock CRTC pointer in all slave CRTC states for tiled displays URL : https://patchwork.freedesktop.org/series/58393/ State : warning == Summary == $ dim checkpatch origin/drm-tip d15c48b8741b drm/i915/icl: Assign

[Intel-gfx] ✓ Fi.CI.BAT: success for HDCP2.2 Phase II (rev4)

2019-03-21 Thread Patchwork
== Series Details == Series: HDCP2.2 Phase II (rev4) URL : https://patchwork.freedesktop.org/series/57232/ State : success == Summary == CI Bug Log - changes from CI_DRM_5792 -> Patchwork_12566 Summary --- **SUCCESS** No

[Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [1/3] drm/i915: Split Pineview device info into desktop and mobile (rev2)

2019-03-21 Thread Patchwork
== Series Details == Series: series starting with [1/3] drm/i915: Split Pineview device info into desktop and mobile (rev2) URL : https://patchwork.freedesktop.org/series/58123/ State : failure == Summary == CI Bug Log - changes from CI_DRM_5787_full -> Patchwork_12547_full

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for HDCP2.2 Phase II (rev4)

2019-03-21 Thread Patchwork
== Series Details == Series: HDCP2.2 Phase II (rev4) URL : https://patchwork.freedesktop.org/series/57232/ State : warning == Summary == $ dim sparse origin/drm-tip Sparse version: v0.5.2 Commit: drm/i915: debugfs: HDCP2.2 capability read Okay! Commit: drm: Add Content protection type

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for HDCP2.2 Phase II (rev4)

2019-03-21 Thread Patchwork
== Series Details == Series: HDCP2.2 Phase II (rev4) URL : https://patchwork.freedesktop.org/series/57232/ State : warning == Summary == $ dim checkpatch origin/drm-tip e5c335d05858 drm/i915: debugfs: HDCP2.2 capability read 42027b987c97 drm: Add Content protection type property -:60:

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Add not fenceable reason to not enable FBC

2019-03-21 Thread Patchwork
== Series Details == Series: drm/i915: Add not fenceable reason to not enable FBC URL : https://patchwork.freedesktop.org/series/58390/ State : success == Summary == CI Bug Log - changes from CI_DRM_5792 -> Patchwork_12565 Summary ---

Re: [Intel-gfx] [PATCH] drm/i915/ehl: All EHL ports are combo phys (v2)

2019-03-21 Thread kbuild test robot
: https://github.com/0day-ci/linux/commits/Bob-Paauwe/drm-i915-ehl-All-EHL-ports-are-combo-phys-v2/20190321-145619 base: git://anongit.freedesktop.org/drm-intel for-linux-next config: i386-randconfig-x073-201911 (attached as .config) compiler: gcc-7 (Debian 7.3.0-1) 7.3.0 reproduce: # save

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: stop storing the media fuse

2019-03-21 Thread Patchwork
== Series Details == Series: drm/i915: stop storing the media fuse URL : https://patchwork.freedesktop.org/series/58387/ State : success == Summary == CI Bug Log - changes from CI_DRM_5791 -> Patchwork_12564 Summary --- **SUCCESS**

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915: stop storing the media fuse

2019-03-21 Thread Patchwork
== Series Details == Series: drm/i915: stop storing the media fuse URL : https://patchwork.freedesktop.org/series/58387/ State : warning == Summary == $ dim sparse origin/drm-tip Sparse version: v0.5.2 Commit: drm/i915: stop storing the media fuse

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: stop storing the media fuse

2019-03-21 Thread Patchwork
== Series Details == Series: drm/i915: stop storing the media fuse URL : https://patchwork.freedesktop.org/series/58387/ State : warning == Summary == $ dim checkpatch origin/drm-tip de5dfcc93c76 drm/i915: stop storing the media fuse -:23: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'first' -

[Intel-gfx] ✓ Fi.CI.BAT: success for Do not re-read dpll registers

2019-03-21 Thread Patchwork
== Series Details == Series: Do not re-read dpll registers URL : https://patchwork.freedesktop.org/series/58382/ State : success == Summary == CI Bug Log - changes from CI_DRM_5791 -> Patchwork_12562 Summary --- **SUCCESS** No

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with drm/i915/ehl: Add EHL platform info and PCI IDs (rev2)

2019-03-21 Thread Patchwork
== Series Details == Series: series starting with drm/i915/ehl: Add EHL platform info and PCI IDs (rev2) URL : https://patchwork.freedesktop.org/series/58386/ State : failure == Summary == Applying: drm/i915/ehl: Add EHL platform info and PCI IDs Applying: drm/i915/ehl: Add ElkhartLake

[Intel-gfx] [PATCH 2/2] drm/i915/icl: Enable TRANSCODER PORT SYNC for tiled displays across separate ports

2019-03-21 Thread Manasi Navare
In case of tiled displays where different tiles are displayed across different ports, we need to synchronize the transcoders involved. This patch implements the transcoder port sync feature for synchronizing one master transcoder with one or more slave transcoders. This is only enbaled in slave

[Intel-gfx] [PATCH 1/2] drm/i915/icl: Assign genlock CRTC pointer in all slave CRTC states for tiled displays

2019-03-21 Thread Manasi Navare
In case of tiled displays when the two tiles are sent across two CRTCs over two separate DP SST connectors, we need a mechanism to synchronize the two CRTCs and their corresponding transcoders. So use the master-slave mode where there is one master corresponding to last horizontal and vertical

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Do not re-read dpll registers

2019-03-21 Thread Patchwork
== Series Details == Series: Do not re-read dpll registers URL : https://patchwork.freedesktop.org/series/58382/ State : warning == Summary == $ dim checkpatch origin/drm-tip b8b9a3a6bb07 drm/i915/skl: use previous pll hw readout 8b862f77d44e drm/i915/cnl: use previous pll hw readout -:41:

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: set vdbox/vebox enable masks on all gens

2019-03-21 Thread Patchwork
== Series Details == Series: drm/i915: set vdbox/vebox enable masks on all gens URL : https://patchwork.freedesktop.org/series/58381/ State : success == Summary == CI Bug Log - changes from CI_DRM_5791 -> Patchwork_12561 Summary ---

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Skip object locking around a no-op set-domain ioctl (rev4)

2019-03-21 Thread Patchwork
== Series Details == Series: drm/i915: Skip object locking around a no-op set-domain ioctl (rev4) URL : https://patchwork.freedesktop.org/series/58325/ State : success == Summary == CI Bug Log - changes from CI_DRM_5787_full -> Patchwork_12546_full

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/icl: Fix clockgating issue when using scalars (rev3)

2019-03-21 Thread Patchwork
== Series Details == Series: drm/i915/icl: Fix clockgating issue when using scalars (rev3) URL : https://patchwork.freedesktop.org/series/58081/ State : success == Summary == CI Bug Log - changes from CI_DRM_5791 -> Patchwork_12560 Summary

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Pass intel_context to i915_request_create()

2019-03-21 Thread Patchwork
== Series Details == Series: drm/i915: Pass intel_context to i915_request_create() URL : https://patchwork.freedesktop.org/series/58380/ State : success == Summary == CI Bug Log - changes from CI_DRM_5791 -> Patchwork_12559 Summary ---

Re: [Intel-gfx] [PATCH] drm/i915: Add not fenceable reason to not enable FBC

2019-03-21 Thread Souza, Jose
After sent I notice that the commit descriptions feels strange to read, maybe changing to "drm/i915: Add 'not fenceable' reason to not enable FBC" makes it better, any tips? On Thu, 2019-03-21 at 17:42 -0700, José Roberto de Souza wrote: > There is some kms_frontbuffer_tracking failures due FBC

[Intel-gfx] [PATCH v3 03/10] drm/i915: Attach content type property

2019-03-21 Thread Ramalingam C
Attaches the content type property for HDCP2.2 capable connectors. Implements the update of content type from property and apply the restriction on HDCP version selection. v2: s/cp_content_type/content_protection_type [daniel] disable at hdcp_atomic_check to avoid check at

[Intel-gfx] [PATCH v3 07/10] drm/i915: Populate downstream info for HDCP1.4

2019-03-21 Thread Ramalingam C
Implements drm blob property content_protection_downstream_info property on HDCP capable connectors. Downstream topology info is gathered across authentication stages and stored in intel_hdcp. When HDCP authentication is complete, new blob with latest downstream topology information is updated to

[Intel-gfx] [PATCH v3 06/10] drm: Add CP downstream_info property

2019-03-21 Thread Ramalingam C
This patch adds a optional CP downstream info blob property to the connectors. This enables the Userspace to read the information of HDCP authenticated downstream topology. Driver will updated this blob with all downstream information at the end of the authentication. In case userspace

[Intel-gfx] [PATCH v3 01/10] drm/i915: debugfs: HDCP2.2 capability read

2019-03-21 Thread Ramalingam C
Adding the HDCP2.2 capability of HDCP src and sink info into debugfs entry "i915_hdcp_sink_capability" This helps the userspace tests to skip the HDCP2.2 test on non HDCP2.2 sinks. Signed-off-by: Ramalingam C --- drivers/gpu/drm/i915/i915_debugfs.c | 13 +++--

[Intel-gfx] [PATCH v3 08/10] drm/i915: Populate downstream info for HDCP2.2

2019-03-21 Thread Ramalingam C
Populates the downstream info for HDCP2.2 encryption also. On success of encryption Blob is updated. Additional two variable are added to downstream info blob. Such as ver_in_force and content type. v2: s/cp_downstream/content_protection_downstream [daniel] v3:

[Intel-gfx] [PATCH v3 02/10] drm: Add Content protection type property

2019-03-21 Thread Ramalingam C
This patch adds a DRM ENUM property to the selected connectors. This property is used for mentioning the protected content's type from userspace to kernel HDCP authentication. Type of the stream is decided by the protected content providers. Type 0 content can be rendered on any HDCP protected

[Intel-gfx] [PATCH v3 09/10] drm: uevent for connector status change

2019-03-21 Thread Ramalingam C
DRM API for generating uevent for a status changes of connector's property. Signed-off-by: Ramalingam C --- drivers/gpu/drm/drm_sysfs.c | 28 include/drm/drm_sysfs.h | 5 - 2 files changed, 32 insertions(+), 1 deletion(-) diff --git

[Intel-gfx] [PATCH v3 10/10] drm/i915: uevent for HDCP status change

2019-03-21 Thread Ramalingam C
Invoking the uevent generator for the content protection property state change of a connector. This helps the userspace to detect the new state change without polling the property val. Signed-off-by: Ramalingam C --- drivers/gpu/drm/i915/intel_hdcp.c | 11 +++ 1 file changed, 11

[Intel-gfx] [PATCH v3 04/10] drm/i915: HDCP SRM parsing and revocation check

2019-03-21 Thread Ramalingam C
Implements the SRM table parsing for HDCP 1.4 and 2.2. And also revocation check is added at authentication of HDCP1.4 and 2.2 Signed-off-by: Ramalingam C --- drivers/gpu/drm/i915/i915_drv.c | 1 + drivers/gpu/drm/i915/i915_drv.h | 6 + drivers/gpu/drm/i915/intel_drv.h | 2 +

[Intel-gfx] [PATCH] drm/i915: Add not fenceable reason to not enable FBC

2019-03-21 Thread José Roberto de Souza
There is some kms_frontbuffer_tracking failures due FBC being disabled with the reason "framebuffer not tiled or fenced". Although the test is setting up everything correctly to have FBC enabled sporadically it is failing, due the alignment and size restrictions in

[Intel-gfx] [PATCH v3 05/10] drm/i915/sysfs: Node for hdcp srm

2019-03-21 Thread Ramalingam C
Binary Sysfs entry is created to pass the HDCP SRM table into kerel for the HDCP authentication purpose. Signed-off-by: Ramalingam C --- drivers/gpu/drm/i915/i915_sysfs.c | 32 +++ 1 file changed, 32 insertions(+) diff --git a/drivers/gpu/drm/i915/i915_sysfs.c

[Intel-gfx] [PATCH v3 00/10] HDCP2.2 Phase II

2019-03-21 Thread Ramalingam C
HDCP2.2 phase-II introduces below features: Addition of three connector properties HDCP Content Type HDCP Topology Addition of binary sysfs "hdcp_srm" parsing for HDCP1.4 and 2.2 SRM table Once HDCP1.4/2.2 authentication is completed

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/guc: GuC suspend path cleanup (rev2)

2019-03-21 Thread Patchwork
== Series Details == Series: drm/i915/guc: GuC suspend path cleanup (rev2) URL : https://patchwork.freedesktop.org/series/58370/ State : success == Summary == CI Bug Log - changes from CI_DRM_5791 -> Patchwork_12558 Summary ---

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/selftests: fix NULL vs IS_ERR() check in mock_context_barrier() (rev2)

2019-03-21 Thread Patchwork
== Series Details == Series: drm/i915/selftests: fix NULL vs IS_ERR() check in mock_context_barrier() (rev2) URL : https://patchwork.freedesktop.org/series/58337/ State : success == Summary == CI Bug Log - changes from CI_DRM_5787_full -> Patchwork_12545_full

[Intel-gfx] [PATCH] drm/i915: stop storing the media fuse

2019-03-21 Thread Daniele Ceraolo Spurio
We're already updating the engine_mask to reflect what's in the HW, so we can just get the info from there. A couple of macros have been added to facilitate this. Suggested-by: Chris Wilson Signed-off-by: Daniele Ceraolo Spurio Cc: Chris Wilson Cc: Michal Wajdeczko ---

Re: [Intel-gfx] linux-next: manual merge of the drm-intel tree with the drm-intel-fixes tree

2019-03-21 Thread Rodrigo Vivi
Hi Stephen, On Fri, Mar 22, 2019 at 10:57:28AM +1100, Stephen Rothwell wrote: > Hi all, > > Today's linux-next merge of the drm-intel tree got a conflict in: > > drivers/gpu/drm/i915/gvt/mmio_context.c > > between commit: > > 1e8b15a1988e ("drm/i915/gvt: Add in context mmio 0x20D8 to gen9

[Intel-gfx] [PATCH] drm/i915/ehl: Add EHL platform info and PCI IDs

2019-03-21 Thread Rodrigo Vivi
From: James Ausmus Add known EHL PCI IDs. v2 (Rodrigo): Removed x86 early quirk. To be sent in a separated patch cc'ing the appropriated list and maintainers for proper ack. v3: (Rodrigo): - Removed .num_pipes = 3 that is coming since GEN&_FEATURES. -

[Intel-gfx] [CI 1/6] drm/i915/ehl: Add EHL platform info and PCI IDs

2019-03-21 Thread Rodrigo Vivi
From: James Ausmus Add known EHL PCI IDs. v2 (Rodrigo): Removed x86 early quirk. To be sent in a separated patch cc'ing the appropriated list and maintainers for proper ack. v3: (Rodrigo): - Removed .num_pipes = 3 that is coming since GEN&_FEATURES. -

[Intel-gfx] [CI 6/6] drm/i915/ehl: Add Support for DMC on EHL

2019-03-21 Thread Rodrigo Vivi
From: Anusha Srivatsa EHL uses the same firmware as ICL. Cc: Bob Paauwe Signed-off-by: Anusha Srivatsa Signed-off-by: Rodrigo Vivi Reviewed-by: Lucas De Marchi Reviewed-by: Bob Paauwe Reviewed-by: José Roberto de Souza --- drivers/gpu/drm/i915/intel_csr.c | 2 +- 1 file changed, 1

[Intel-gfx] [CI 5/6] drm/i915/ehl: Set proper eu slice/subslice parameters for EHL

2019-03-21 Thread Rodrigo Vivi
From: Bob Paauwe EHL has a different number of subslices. Cc: Lucas De Marchi Signed-off-by: Bob Paauwe Signed-off-by: Rodrigo Vivi Reviewed-by: Lucas De Marchi Reviewed-by: José Roberto de Souza --- drivers/gpu/drm/i915/intel_device_info.c | 12 +--- 1 file changed, 9

[Intel-gfx] [CI 2/6] drm/i915/ehl: Add ElkhartLake platform

2019-03-21 Thread Rodrigo Vivi
From: Bob Paauwe Add ElkhartLake as a unique platform as there are some differences between it and Icelake. Signed-off-by: Bob Paauwe Signed-off-by: Rodrigo Vivi Reviewed-by: Lucas De Marchi Reviewed-by: José Roberto de Souza --- drivers/gpu/drm/i915/i915_drv.h | 1 +

[Intel-gfx] [CI 4/6] drm/i915/ehl: EHL outputs are different from ICL

2019-03-21 Thread Rodrigo Vivi
From: Bob Paauwe Configure the correct set of outputs for EHL. EHL has three DDI's plus MIPI. Cc: Lucas De Marchi Signed-off-by: Bob Paauwe Signed-off-by: Rodrigo Vivi Reviewed-by: Lucas De Marchi Reviewed-by: José Roberto de Souza --- drivers/gpu/drm/i915/intel_display.c | 7 ++- 1

[Intel-gfx] [CI 3/6] drm/i915/ehl: Add dpll mgr

2019-03-21 Thread Rodrigo Vivi
From: Lucas De Marchi Elkhart Lake has a different set of PLLs as compared to Ice Lake, although programming them is very similar. v2: Rebase on top of s/icl_pll_funcs/combo_pll_funcs Signed-off-by: Lucas De Marchi Signed-off-by: Rodrigo Vivi Reviewed-by: José Roberto de Souza

[Intel-gfx] linux-next: manual merge of the drm-intel tree with the drm-intel-fixes tree

2019-03-21 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the drm-intel tree got a conflict in: drivers/gpu/drm/i915/gvt/mmio_context.c between commit: 1e8b15a1988e ("drm/i915/gvt: Add in context mmio 0x20D8 to gen9 mmio list") from the drm-intel-fixes tree and commit: 8a68d464366e ("drm/i915: Store the

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/2] drm/i915: Add missing scanline case for 8 cpp watermarks.

2019-03-21 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915: Add missing scanline case for 8 cpp watermarks. URL : https://patchwork.freedesktop.org/series/58342/ State : success == Summary == CI Bug Log - changes from CI_DRM_5787_full -> Patchwork_12544_full

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [CI,1/2] drm/i915/selftests: Calculate maximum ring size for preemption chain

2019-03-21 Thread Patchwork
== Series Details == Series: series starting with [CI,1/2] drm/i915/selftests: Calculate maximum ring size for preemption chain URL : https://patchwork.freedesktop.org/series/58376/ State : success == Summary == CI Bug Log - changes from CI_DRM_5791 -> Patchwork_12557

Re: [Intel-gfx] [CI 4/4] drm/i915: Skip modeset for cdclk changes if possible

2019-03-21 Thread Clinton Taylor
On 3/20/19 6:54 AM, Imre Deak wrote: From: Ville Syrjälä If we have only a single active pipe and the cdclk change only requires the cd2x divider to be updated bxt+ can do the update with forcing a full modeset on the pipe. Try to hook that up. v2: - Wait for vblank after an optimized CDCLK

Re: [Intel-gfx] [PATCH] drm/i915: set vdbox/vebox enable masks on all gens

2019-03-21 Thread Daniele Ceraolo Spurio
On 3/21/19 3:39 PM, Chris Wilson wrote: Quoting Daniele Ceraolo Spurio (2019-03-21 22:14:20) On 3/21/19 3:11 PM, Chris Wilson wrote: Quoting Daniele Ceraolo Spurio (2019-03-21 22:02:06) The upcoming unified GuC FW will require us to send video engine enable masks to GuC for its

Re: [Intel-gfx] [PATCH] drm/i915: set vdbox/vebox enable masks on all gens

2019-03-21 Thread Chris Wilson
Quoting Daniele Ceraolo Spurio (2019-03-21 22:14:20) > > > On 3/21/19 3:11 PM, Chris Wilson wrote: > > Quoting Daniele Ceraolo Spurio (2019-03-21 22:02:06) > >> The upcoming unified GuC FW will require us to send video engine enable > >> masks to GuC for its initialization. > >> > >> For

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/8] drm/i915/psr: Remove partial PSR support on multiple transcoders

2019-03-21 Thread Patchwork
== Series Details == Series: series starting with [1/8] drm/i915/psr: Remove partial PSR support on multiple transcoders URL : https://patchwork.freedesktop.org/series/58373/ State : success == Summary == CI Bug Log - changes from CI_DRM_5791 -> Patchwork_12556

Re: [Intel-gfx] [CI 1/4] drm/i915: Force 2*96 MHz cdclk on glk/cnl when audio power is enabled

2019-03-21 Thread Clinton Taylor
On 3/20/19 6:54 AM, Imre Deak wrote: From: Ville Syrjälä CDCLK has to be at least twice the BLCK regardless of audio. Audio driver has to probe using this hook and increase the clock even in absence of any display. v2: Use atomic refcount for get_power, put_power so that we can call

Re: [Intel-gfx] [PATCH] drm/i915: set vdbox/vebox enable masks on all gens

2019-03-21 Thread Daniele Ceraolo Spurio
On 3/21/19 3:11 PM, Chris Wilson wrote: Quoting Daniele Ceraolo Spurio (2019-03-21 22:02:06) The upcoming unified GuC FW will require us to send video engine enable masks to GuC for its initialization. For consistency, just set the runtime_info enable masks for all gens. We'll then be able

Re: [Intel-gfx] [PATCH] drm/i915: set vdbox/vebox enable masks on all gens

2019-03-21 Thread Chris Wilson
Quoting Daniele Ceraolo Spurio (2019-03-21 22:02:06) > The upcoming unified GuC FW will require us to send video engine enable > masks to GuC for its initialization. > > For consistency, just set the runtime_info enable masks for all gens. > We'll then be able to directly use those in the GuC

Re: [Intel-gfx] [PATCH] drm/i915: set vdbox/vebox enable masks on all gens

2019-03-21 Thread Chris Wilson
Quoting Daniele Ceraolo Spurio (2019-03-21 22:02:06) > The upcoming unified GuC FW will require us to send video engine enable > masks to GuC for its initialization. > > For consistency, just set the runtime_info enable masks for all gens. > We'll then be able to directly use those in the GuC

Re: [Intel-gfx] [PATCH i-g-t] lib: sync with the newer i915_pciids.h from the Kernel

2019-03-21 Thread Souza, Jose
This should be sent to igt-...@lists.freedesktop.org On Thu, 2019-03-21 at 11:02 -0700, Anusha wrote: > From: Anusha Srivatsa > > Add CML IDS and additional CNL ID. > > v2: Copy header from kernel (Jose) > - Change commit message (Lucas) > > Cc: José Roberto de Souza > Cc: Lucas De Marchi >

Re: [Intel-gfx] [RFC v1 7/7] drm/i915: Add multi segment gamma for icl

2019-03-21 Thread Matt Roper
On Tue, Mar 19, 2019 at 02:00:18PM +0530, Uma Shankar wrote: > Added support for ICL platform multi segment gamma > capabilties and attached the property, exposing the > same to userspace. > > Signed-off-by: Uma Shankar > --- > drivers/gpu/drm/i915/intel_color.c | 22 +- > 1

Re: [Intel-gfx] [PATCH libdrm] intel: sync i915_pciids.h with kernel

2019-03-21 Thread Souza, Jose
This should be sent to dri-de...@lists.freedesktop.org On Thu, 2019-03-21 at 10:58 -0700, Anusha wrote: > Straight copy from the kernel file. > > Add PCI IDs for CML, add additional PCI ID > for CNL. > > v2: Do a copy from kernel header (Jose) > - Change commit message (Lucas) > > Cc: José

[Intel-gfx] [PATCH 3/3] drm/i915/icl: use previous pll hw readout

2019-03-21 Thread Lucas De Marchi
By the time icl_ddi_clock_get() is called we've just got the hw state from the pll registers. We don't need to read them again: we can rather reuse what was cached in the dpll_hw_state. Signed-off-by: Lucas De Marchi --- drivers/gpu/drm/i915/intel_ddi.c | 39 +++- 1

[Intel-gfx] [PATCH 0/3] Do not re-read dpll registers

2019-03-21 Thread Lucas De Marchi
Instead of re-reading the registers we just read on the hw state readout, use the values saved on intel_shared_dpll. Besides not doing the MMIO, this helps on sharing code since we don't have to differentiate e.g. ICL and CNL because they have different registers for the same thing. I'm a little

[Intel-gfx] [PATCH 2/3] drm/i915/cnl: use previous pll hw readout

2019-03-21 Thread Lucas De Marchi
By the time cnl_ddi_clock_get() is called we've just got the hw state from the pll registers. We don't need to read them again: we can rather reuse what was cached in the dpll_hw_state. This also affects the code for ICL since it partially reuses the CNL code. However the more intricate part on

[Intel-gfx] [PATCH 1/3] drm/i915/skl: use previous pll hw readout

2019-03-21 Thread Lucas De Marchi
By the time skl_ddi_clock_get() is called - and thus skl_calc_wrpll_link() - we've just got the hw state from the pll registers. We don't need to read them again: we can rather reuse what was cached in the dpll_hw_state. Signed-off-by: Lucas De Marchi --- drivers/gpu/drm/i915/intel_ddi.c | 49

[Intel-gfx] [PATCH] drm/i915: set vdbox/vebox enable masks on all gens

2019-03-21 Thread Daniele Ceraolo Spurio
The upcoming unified GuC FW will require us to send video engine enable masks to GuC for its initialization. For consistency, just set the runtime_info enable masks for all gens. We'll then be able to directly use those in the GuC setup Cc: Michal Wajdeczko Cc: John Spotswood Cc: Eric

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for series starting with [1/8] drm/i915/psr: Remove partial PSR support on multiple transcoders

2019-03-21 Thread Patchwork
== Series Details == Series: series starting with [1/8] drm/i915/psr: Remove partial PSR support on multiple transcoders URL : https://patchwork.freedesktop.org/series/58373/ State : warning == Summary == $ dim sparse origin/drm-tip Sparse version: v0.5.2 Commit: drm/i915/psr: Remove partial

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/8] drm/i915/psr: Remove partial PSR support on multiple transcoders

2019-03-21 Thread Patchwork
== Series Details == Series: series starting with [1/8] drm/i915/psr: Remove partial PSR support on multiple transcoders URL : https://patchwork.freedesktop.org/series/58373/ State : warning == Summary == $ dim checkpatch origin/drm-tip 95c7ad172d42 drm/i915/psr: Remove partial PSR support

Re: [Intel-gfx] [RFC v1 3/7] drm/i915: Add Support for Multi Segment Gamma Mode

2019-03-21 Thread Matt Roper
On Wed, Mar 20, 2019 at 10:03:16AM -0700, Shankar, Uma wrote: > > > >-Original Message- > >From: Syrjala, Ville > >Sent: Tuesday, March 19, 2019 10:29 PM > >To: Lankhorst, Maarten > >Cc: Shankar, Uma ; intel-gfx@lists.freedesktop.org; > >Sharma, Shashank ; Roper, Matthew D > > >

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Really calculate the cursor ddb based on the highest enabled wm level

2019-03-21 Thread Patchwork
== Series Details == Series: drm/i915: Really calculate the cursor ddb based on the highest enabled wm level URL : https://patchwork.freedesktop.org/series/58372/ State : success == Summary == CI Bug Log - changes from CI_DRM_5791 -> Patchwork_12555

[Intel-gfx] [PATCH v3] drm/i915/icl: Fix clockgating issue when using scalers

2019-03-21 Thread Radhakrishna Sripada
Fixes the clock-gating issue when pipe scaling is enabled. (Lineage #2006604312) V2: Fix typo in headline(Chris) Handle the non double buffered nature of the register(Ville) V3: Fix checkpatch warning. BAT failure for V2 on gen3 looks unrelated. Cc: Chris Wilson Cc: Ville Syrjala Cc:

[Intel-gfx] [PATCH] drm/i915: Pass intel_context to i915_request_create()

2019-03-21 Thread Chris Wilson
Start acquiring the logical intel_context and using that as our primary means for request allocation. This is the initial step to allow us to avoid requiring struct_mutex for request allocation along the perma-pinned kernel context, but it also provides a foundation for breaking up the complex

Re: [Intel-gfx] [RFC v1 1/7] drm/i915: Add gamma mode property

2019-03-21 Thread Matt Roper
On Tue, Mar 19, 2019 at 02:00:12PM +0530, Uma Shankar wrote: > Gen platforms support multiple gamma modes, currently > it's hard coded to operate only in 1 specific mode. > This patch adds a property to make gamma mode programmable. > User can select which mode should be used for a particular >

Re: [Intel-gfx] [PATCH i-g-t] lib: sync with the newer i915_pciids.h from the Kernel

2019-03-21 Thread Antonio Argenziano
On 21/03/19 11:02, Anusha wrote: From: Anusha Srivatsa Add CML IDS and additional CNL ID. Please add the kernel commits to be consistent with the previous updates. Acked-by: Antonio Argenziano v2: Copy header from kernel (Jose) - Change commit message (Lucas) Cc: José Roberto de

[Intel-gfx] [PATCH v2] drm/i915/guc: GuC suspend path cleanup

2019-03-21 Thread Sujaritha Sundaresan
Adding a call to intel_uc_suspend in i915_gem_suspend, which is a common point for the suspend/resume and hibernate paths. This fixes an unbalanced call that causes issues with the CTB register/deregister. v2: Making the call unconditional (Daniele) Moving the call to after the GEM_BUG_ON

Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/guc: GuC suspend path cleanup

2019-03-21 Thread Sujaritha
On 3/21/19 1:23 PM, Chris Wilson wrote: Quoting Sujaritha (2019-03-21 20:02:36) On 3/21/19 1:08 PM, Chris Wilson wrote: Quoting Sujaritha (2019-03-21 19:41:17) On 3/21/19 12:37 PM, Chris Wilson wrote: Quoting Patchwork (2019-03-21 19:26:27) == Series Details == Series: drm/i915/guc: GuC

Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/guc: GuC suspend path cleanup

2019-03-21 Thread Chris Wilson
Quoting Sujaritha (2019-03-21 20:02:36) > > On 3/21/19 1:08 PM, Chris Wilson wrote: > > Quoting Sujaritha (2019-03-21 19:41:17) > >> On 3/21/19 12:37 PM, Chris Wilson wrote: > >>> Quoting Patchwork (2019-03-21 19:26:27) > == Series Details == > > Series: drm/i915/guc: GuC suspend

Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/guc: GuC suspend path cleanup

2019-03-21 Thread Sujaritha
On 3/21/19 1:08 PM, Chris Wilson wrote: Quoting Sujaritha (2019-03-21 19:41:17) On 3/21/19 12:37 PM, Chris Wilson wrote: Quoting Patchwork (2019-03-21 19:26:27) == Series Details == Series: drm/i915/guc: GuC suspend path cleanup URL : https://patchwork.freedesktop.org/series/58370/ State

Re: [Intel-gfx] [PATCH] drm/i915: Really calculate the cursor ddb based on the highest enabled wm level

2019-03-21 Thread Rodrigo Vivi
On Thu, Mar 21, 2019 at 07:51:28PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > I added the loop but neglected to actually pass the level to the > function. So we were just looping 8 times calculating the exact > same thing every time. > > Fixes: df331de3f8aa ("drm/i915: Allocate

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [v1,1/1] drm/i915/sleftests: live_execlists subtest faster

2019-03-21 Thread Patchwork
== Series Details == Series: series starting with [v1,1/1] drm/i915/sleftests: live_execlists subtest faster URL : https://patchwork.freedesktop.org/series/58371/ State : success == Summary == CI Bug Log - changes from CI_DRM_5789 -> Patchwork_12554

Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/guc: GuC suspend path cleanup

2019-03-21 Thread Chris Wilson
Quoting Sujaritha (2019-03-21 19:41:17) > > On 3/21/19 12:37 PM, Chris Wilson wrote: > > Quoting Patchwork (2019-03-21 19:26:27) > >> == Series Details == > >> > >> Series: drm/i915/guc: GuC suspend path cleanup > >> URL : https://patchwork.freedesktop.org/series/58370/ > >> State : failure >

Re: [Intel-gfx] [PATCH] drm/i915/guc: GuC suspend path cleanup

2019-03-21 Thread Sujaritha
On 3/21/19 10:54 AM, Chris Wilson wrote: Quoting Daniele Ceraolo Spurio (2019-03-21 17:49:55) On 3/21/19 10:14 AM, Sujaritha Sundaresan wrote: Adding a call to intel_uc_suspend in i915_gem_suspend, which is a common point for the suspend/resume and hibernate paths. This fixes an unbalanced

Re: [Intel-gfx] [PULL] gvt-fixes

2019-03-21 Thread Rodrigo Vivi
On Thu, Mar 21, 2019 at 11:50:18AM +0800, Zhenyu Wang wrote: > > Hi, > > Sorry that I might miss 5.1-rc1 window as I was in full day event > during this week. hmm... my bad. I should had remembered that you had stuff queued for -fixes that we didn't put in -next-fixes I'm sorry. > Here's

Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/guc: GuC suspend path cleanup

2019-03-21 Thread Sujaritha
On 3/21/19 12:37 PM, Chris Wilson wrote: Quoting Patchwork (2019-03-21 19:26:27) == Series Details == Series: drm/i915/guc: GuC suspend path cleanup URL : https://patchwork.freedesktop.org/series/58370/ State : failure == Summary == CI Bug Log - changes from CI_DRM_5789 -> Patchwork_12553

[Intel-gfx] [CI 2/2] drm/i915/selftests: Mark up preemption tests for hang detection

2019-03-21 Thread Chris Wilson
Use the igt_live_test framework for detecting whether an unwanted hang occurred during test execution, and report failure if it does. Signed-off-by: Chris Wilson Reviewed-by: Joonas Lahtinen --- drivers/gpu/drm/i915/selftests/intel_lrc.c | 38 -- 1 file changed, 35

[Intel-gfx] [CI 1/2] drm/i915/selftests: Calculate maximum ring size for preemption chain

2019-03-21 Thread Chris Wilson
32 is too many for the likes of kbl, and in order to insert that many requests into the ring requires us to declare the first few hung -- understandably a slow and unexpected process. Instead, measure the size of a singe requests and use that to estimate the upper bound on the chain length we can

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [v1,1/1] drm/i915/sleftests: live_execlists subtest faster

2019-03-21 Thread Patchwork
== Series Details == Series: series starting with [v1,1/1] drm/i915/sleftests: live_execlists subtest faster URL : https://patchwork.freedesktop.org/series/58371/ State : warning == Summary == $ dim checkpatch origin/drm-tip 317b9648fbbb drm/i915/sleftests: live_execlists subtest faster

Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/guc: GuC suspend path cleanup

2019-03-21 Thread Chris Wilson
Quoting Patchwork (2019-03-21 19:26:27) > == Series Details == > > Series: drm/i915/guc: GuC suspend path cleanup > URL : https://patchwork.freedesktop.org/series/58370/ > State : failure > > == Summary == > > CI Bug Log - changes from CI_DRM_5789 -> Patchwork_12553 >

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/guc: GuC suspend path cleanup

2019-03-21 Thread Patchwork
== Series Details == Series: drm/i915/guc: GuC suspend path cleanup URL : https://patchwork.freedesktop.org/series/58370/ State : failure == Summary == CI Bug Log - changes from CI_DRM_5789 -> Patchwork_12553 Summary --- **FAILURE**

[Intel-gfx] ✓ Fi.CI.BAT: success for lib: sync with the newer i915_pciids.h from the Kernel

2019-03-21 Thread Patchwork
== Series Details == Series: lib: sync with the newer i915_pciids.h from the Kernel URL : https://patchwork.freedesktop.org/series/58375/ State : success == Summary == CI Bug Log - changes from IGT_4897 -> IGTPW_2686 Summary ---

Re: [Intel-gfx] [PATCH 1/2] drm/i915/selftests: Calculate maximum ring size for preemption chain

2019-03-21 Thread Chris Wilson
Quoting Chris Wilson (2019-03-21 18:38:53) > Quoting Caz Yokoyama (2019-03-21 18:41:10) > > inline > > -caz > > On Thu, 2019-03-21 at 07:37 +, Chris Wilson wrote: > > > + > > > + rq = i915_request_alloc(engine, lo.ctx); > > > + if (IS_ERR(rq)) > > > +

Re: [Intel-gfx] [PATCH 1/2] drm/i915/selftests: Calculate maximum ring size for preemption chain

2019-03-21 Thread Caz Yokoyama
Reviewed-by: Yokoyama, Caz -caz On Thu, 2019-03-21 at 18:42 +, Chris Wilson wrote: > Quoting Chris Wilson (2019-03-21 18:38:53) > > Quoting Caz Yokoyama (2019-03-21 18:41:10) > > > inline > > > -caz > > > On Thu, 2019-03-21 at 07:37 +, Chris Wilson wrote: > > > > 32 is too many for the

Re: [Intel-gfx] [PATCH 1/2] drm/i915/selftests: Calculate maximum ring size for preemption chain

2019-03-21 Thread Chris Wilson
Quoting Chris Wilson (2019-03-21 18:38:53) > Quoting Caz Yokoyama (2019-03-21 18:41:10) > > inline > > -caz > > On Thu, 2019-03-21 at 07:37 +, Chris Wilson wrote: > > > 32 is too many for the likes of kbl, and in order to insert that many > > Not only kbl. ring_size is 25 on my cfl. > > > > >

Re: [Intel-gfx] [PATCH 1/2] drm/i915/selftests: Calculate maximum ring size for preemption chain

2019-03-21 Thread Chris Wilson
Quoting Caz Yokoyama (2019-03-21 18:41:10) > inline > -caz > On Thu, 2019-03-21 at 07:37 +, Chris Wilson wrote: > > 32 is too many for the likes of kbl, and in order to insert that many > Not only kbl. ring_size is 25 on my cfl. > > > requests into the ring requires us to declare the first

Re: [Intel-gfx] [PATCH 1/2] drm/i915/selftests: Calculate maximum ring size for preemption chain

2019-03-21 Thread Caz Yokoyama
inline -caz On Thu, 2019-03-21 at 07:37 +, Chris Wilson wrote: > 32 is too many for the likes of kbl, and in order to insert that many Not only kbl. ring_size is 25 on my cfl. > requests into the ring requires us to declare the first few hung -- The hung is not caused by 32. It is caused by

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915: Flush pages on acquisition

2019-03-21 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915: Flush pages on acquisition URL : https://patchwork.freedesktop.org/series/58367/ State : success == Summary == CI Bug Log - changes from CI_DRM_5789 -> Patchwork_12552

Re: [Intel-gfx] [PATCH 1/8] drm/i915/psr: Remove partial PSR support on multiple transcoders

2019-03-21 Thread Ville Syrjälä
On Thu, Mar 21, 2019 at 11:01:36AM -0700, José Roberto de Souza wrote: > PSR is only support in eDP transcoder and there is only one instance > of it, so lets drop all of this code. One instance? Are you talking about PSR2? Also the EDP transcoder is already doomed is it not? > > Cc:

[Intel-gfx] [PATCH i-g-t] lib: sync with the newer i915_pciids.h from the Kernel

2019-03-21 Thread Anusha
From: Anusha Srivatsa Add CML IDS and additional CNL ID. v2: Copy header from kernel (Jose) - Change commit message (Lucas) Cc: José Roberto de Souza Cc: Lucas De Marchi Signed-off-by: Anusha Srivatsa --- lib/i915_pciids.h | 31 +-- 1 file changed, 29

[Intel-gfx] [PATCH libdrm] intel: sync i915_pciids.h with kernel

2019-03-21 Thread Anusha
Straight copy from the kernel file. Add PCI IDs for CML, add additional PCI ID for CNL. v2: Do a copy from kernel header (Jose) - Change commit message (Lucas) Cc: José Roberto de Souza Cc: Lucas De Marchi Signed-off-by: Anusha Srivatsa --- intel/i915_pciids.h | 31

[Intel-gfx] [PATCH 3/8] drm/i915/psr: Make all PSR register relative to mmio base

2019-03-21 Thread José Roberto de Souza
Right now it have a mix of PSR registers that are relative to PSR mmio base and other register with a hardcoded address, lets keep it consistented and have it all relative to mmio base. Cc: Dhinakaran Pandiyan Cc: Rodrigo Vivi Signed-off-by: José Roberto de Souza ---

[Intel-gfx] [PATCH 4/8] drm/i915/psr: Make mmio base relative to transcoder offset

2019-03-21 Thread José Roberto de Souza
From BDW+ most of the PSR registers is relative to eDP transcoder offset just PSR_IMR/IIR that have a fixed address, so lets set mmio_base with the transcoder offset and adjust all the others macros to the registers. Also removing BDW_EDP_PSR_BASE from GVT because it is not used as the only PSR

[Intel-gfx] [PATCH 5/8] drm/i915/psr: Initialize PSR mutex even when sink is not reliable

2019-03-21 Thread José Roberto de Souza
Even when driver is reload and hits this scenario the PSR mutex should be initialized, otherwise reading PSR debugfs status will execute mutex_lock() over a mutex that was not initialized. Cc: Dhinakaran Pandiyan Cc: Rodrigo Vivi Signed-off-by: José Roberto de Souza ---

[Intel-gfx] [PATCH 8/8] drm/i915/bdw+: Move misc display IRQ handling to it own function

2019-03-21 Thread José Roberto de Souza
Just moving it to reduce the tabs and avoid break code lines into several lines. No behavior changes intended here. Signed-off-by: José Roberto de Souza --- drivers/gpu/drm/i915/i915_irq.c | 63 +++-- 1 file changed, 36 insertions(+), 27 deletions(-) diff --git

  1   2   3   >