[Intel-gfx] ✗ Fi.CI.SPARSE: warning for series starting with [CI,v13,01/17] drm/i915/dp: Add DSC params and DSC config to intel_crtc_state (rev3)

2018-11-29 Thread Patchwork
== Series Details == Series: series starting with [CI,v13,01/17] drm/i915/dp: Add DSC params and DSC config to intel_crtc_state (rev3) URL : https://patchwork.freedesktop.org/series/53184/ State : warning == Summary == $ dim sparse origin/drm-tip Sparse version: v0.5.2 Commit: drm/i915/dp:

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [CI,v13,01/17] drm/i915/dp: Add DSC params and DSC config to intel_crtc_state (rev3)

2018-11-29 Thread Patchwork
== Series Details == Series: series starting with [CI,v13,01/17] drm/i915/dp: Add DSC params and DSC config to intel_crtc_state (rev3) URL : https://patchwork.freedesktop.org/series/53184/ State : success == Summary == CI Bug Log - changes from CI_DRM_5224 -> Patchwork_10963

Re: [Intel-gfx] [CI v13 01/17] drm/i915/dp: Add DSC params and DSC config to intel_crtc_state

2018-11-29 Thread Manasi Navare
Pushed to dinq, thanks for the patch and the review. Manasi On Wed, Nov 28, 2018 at 12:26:12PM -0800, Manasi Navare wrote: > Basic DSC parameters and DSC configuration data needs to be computed > for each of the requested mode during atomic check. This is > required since for certain modes,

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915: s/PUNIT_REG_DSPFREQ/PUNIT_REG_DSPSSPM/

2018-11-29 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915: s/PUNIT_REG_DSPFREQ/PUNIT_REG_DSPSSPM/ URL : https://patchwork.freedesktop.org/series/53275/ State : success == Summary == CI Bug Log - changes from CI_DRM_5224 -> Patchwork_10961

Re: [Intel-gfx] [PATCH 1/3] drm/fourcc: Add 64 bpp half float formats

2018-11-29 Thread Ville Syrjälä
On Wed, Nov 28, 2018 at 10:38:11PM -0800, Kevin Strasser wrote: > Add 64 bpp 16:16:16:16 half float pixel formats. Each 16 bit component is > formatted in IEEE-754 half-precision float (binary16) 1:5:10 > MSb-sign:exponent:fraction form. > > An 'is_fp' attribute is added to drm_format_info so

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/lease: Send a distinct uevent

2018-11-29 Thread Patchwork
== Series Details == Series: drm/lease: Send a distinct uevent URL : https://patchwork.freedesktop.org/series/53228/ State : success == Summary == CI Bug Log - changes from CI_DRM_5221_full -> Patchwork_10948_full Summary ---

[Intel-gfx] [PATCH 4/6] drm/i915/selftests: Terminate hangcheck sanitycheck forcibly

2018-11-29 Thread Chris Wilson
If all else fails and we are stuck eternally waiting for the undying request, abandon all hope. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/selftests/intel_hangcheck.c | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git

Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/psr: Get pipe id following atomic guidelines (rev2)

2018-11-29 Thread Rodrigo Vivi
On Wed, Nov 28, 2018 at 11:52:49PM -0800, Saarinen, Jani wrote: > Hi, > > > -Original Message- > > From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf > > Of > > Rodrigo Vivi > > Sent: torstai 29. marraskuuta 2018 8.18 > > To: Souza, Jose > > Cc:

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Remove whitelist application from ringbuffer backend (rev3)

2018-11-29 Thread Patchwork
== Series Details == Series: drm/i915: Remove whitelist application from ringbuffer backend (rev3) URL : https://patchwork.freedesktop.org/series/53243/ State : success == Summary == CI Bug Log - changes from CI_DRM_5224 -> Patchwork_10960

Re: [Intel-gfx] [PATCH] drm/dp-mst-helper: Remove hotplug callback

2018-11-29 Thread Lyude Paul
Reviewed-by: Lyude Paul On Wed, 2018-11-28 at 23:12 +0100, Daniel Vetter wrote: > When everyone implements it exactly the same way, among all 4 > implementations, there's not really a need to overwrite this at all. > > Aside: drm_kms_helper_hotplug_event is pretty much core functionality > at

[Intel-gfx] [PATCH v8] drm/i915: implement EXTENDED_RECEIVER_CAPABILITY_FIELD_PRESENT

2018-11-29 Thread Manasi Navare
From: Matt Atwood According to DP spec (2.9.3.1 of DP 1.4) if EXTENDED_RECEIVER_CAPABILITY_FIELD_PRESENT is set the addresses in DPCD 02200h through 0220Fh shall contain the DPRX's true capability. These values will match 0h through Fh, except for DPCD_REV, MAX_LINK_RATE,

Re: [Intel-gfx] [PATCH v8] drm/i915: implement EXTENDED_RECEIVER_CAPABILITY_FIELD_PRESENT

2018-11-29 Thread Rodrigo Vivi
On Thu, Nov 29, 2018 at 10:34:05AM -0800, Manasi Navare wrote: > From: Matt Atwood > > According to DP spec (2.9.3.1 of DP 1.4) if > EXTENDED_RECEIVER_CAPABILITY_FIELD_PRESENT is set the addresses in DPCD > 02200h through 0220Fh shall contain the DPRX's true capability. These > values will match

Re: [Intel-gfx] [PATCH v10 00/23] Respin of remaining DSC + FEC patches

2018-11-29 Thread Manasi Navare
pushed the latest version of this series (https://patchwork.freedesktop.org/series/53184/) and first 5 from https://patchwork.freedesktop.org/series/53113/ to dinq Thanks for the patches and reviews! Manasi On Tue, Nov 20, 2018 at 10:37:13AM -0800, Manasi Navare wrote: > This patch series

Re: [Intel-gfx] [PATCH 1/3] drm/i915/icl: Release TC ports when unloading or suspending driver

2018-11-29 Thread Imre Deak
On Thu, Nov 29, 2018 at 10:18:40PM +0200, Souza, Jose wrote: > On Thu, 2018-11-29 at 15:52 +0200, Imre Deak wrote: > > On Wed, Nov 28, 2018 at 11:54:21PM +0200, Souza, Jose wrote: > > > On Wed, 2018-11-28 at 13:34 +0200, Imre Deak wrote: > > > > On Wed, Nov 07, 2018 at 04:05:52PM -0800, José

Re: [Intel-gfx] [PATCH v3] Return only active connectors for get_resources ioctl

2018-11-29 Thread Lyude Paul
On Thu, 2018-11-29 at 13:09 +0200, Stanislav Lisovskiy wrote: > Currently kernel might allocate different connector ids > for the same outputs in case of DP MST, which seems to > confuse userspace. There are can be different connector > ids in the list, which could be assigned to the same >

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [CI,v13,01/17] drm/i915/dp: Add DSC params and DSC config to intel_crtc_state (rev3)

2018-11-29 Thread Patchwork
== Series Details == Series: series starting with [CI,v13,01/17] drm/i915/dp: Add DSC params and DSC config to intel_crtc_state (rev3) URL : https://patchwork.freedesktop.org/series/53184/ State : warning == Summary == $ dim checkpatch origin/drm-tip cf74ed403002 drm/i915/dp: Add DSC params

Re: [Intel-gfx] [PATCH v8] drm/i915: implement EXTENDED_RECEIVER_CAPABILITY_FIELD_PRESENT

2018-11-29 Thread Rodrigo Vivi
On Thu, Nov 29, 2018 at 11:43:18AM -0800, Manasi Navare wrote: > On Thu, Nov 29, 2018 at 11:28:34AM -0800, Rodrigo Vivi wrote: > > On Thu, Nov 29, 2018 at 10:34:05AM -0800, Manasi Navare wrote: > > > From: Matt Atwood > > > > > > According to DP spec (2.9.3.1 of DP 1.4) if > > >

Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/psr: Get pipe id following atomic guidelines (rev3)

2018-11-29 Thread Souza, Jose
On Thu, 2018-11-29 at 06:33 +, Patchwork wrote: > == Series Details == > > Series: drm/i915/psr: Get pipe id following atomic guidelines (rev3) > URL : https://patchwork.freedesktop.org/series/53132/ > State : failure > > == Summary == > > CI Bug Log - changes from CI_DRM_5220 ->

[Intel-gfx] [PATCH] drm/i915: Fixup stub definitions for intel_opregion_suspend|resume

2018-11-29 Thread Chris Wilson
248 "multiple definition of ...". E.g.: LD [M] drivers/gpu/drm/i915/i915.o ld: drivers/gpu/drm/i915/i915_irq.o: in function `intel_opregion_resume': i915_irq.c:(.text+0x58f0): multiple definition of `intel_opregion_resume'; drivers/gpu/drm/i915/i915_drv.o:i915_drv.c:(.text+0x2d40):

Re: [Intel-gfx] [PATCH] drm/dp-mst-helper: Remove hotplug callback

2018-11-29 Thread Alex Deucher
On Wed, Nov 28, 2018 at 5:12 PM Daniel Vetter wrote: > > When everyone implements it exactly the same way, among all 4 > implementations, there's not really a need to overwrite this at all. > > Aside: drm_kms_helper_hotplug_event is pretty much core functionality > at this point. Probably should

Re: [Intel-gfx] [PATCH 2/3] drm: Add optional PIXEL_NORMALIZE_RANGE property to drm_plane

2018-11-29 Thread Ville Syrjälä
On Wed, Nov 28, 2018 at 10:38:12PM -0800, Kevin Strasser wrote: > Add an optional property to allow applications to indicate what range their > floating point pixel data is normalized to. Drivers are free to choose what > ranges they want to support and can attach this property to each plane that

Re: [Intel-gfx] [PATCH 0/3] Support 64 bpp half float formats

2018-11-29 Thread Ville Syrjälä
On Wed, Nov 28, 2018 at 10:38:10PM -0800, Kevin Strasser wrote: > This series defines new formats and adds a plane property to be used for > floating point framebuffer content. Implementation is then added to i915. > > I have shared an IGT branch which adds test coverage for the new formats: >

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Implement half float formats and pixel normalize property

2018-11-29 Thread Strasser, Kevin
Daniel Vetter wrote: > Do we have end-to-end userspace for this? I have patches for IGT and I'm planning on adding usage code to Weston. Apart from that there is a Windows use case that Tina mentioned previously. I take it that you will need to see the Weston part before accepting this. Thanks,

Re: [Intel-gfx] [PATCH 1/3] drm/i915/icl: Release TC ports when unloading or suspending driver

2018-11-29 Thread Souza, Jose
On Thu, 2018-11-29 at 15:52 +0200, Imre Deak wrote: > On Wed, Nov 28, 2018 at 11:54:21PM +0200, Souza, Jose wrote: > > On Wed, 2018-11-28 at 13:34 +0200, Imre Deak wrote: > > > On Wed, Nov 07, 2018 at 04:05:52PM -0800, José Roberto de Souza > > > wrote: > > > > When suspending or unloading the

[Intel-gfx] [PATCH 2/6] drm/i915/ringbuffer: Clear semaphore sync registers on ring init

2018-11-29 Thread Chris Wilson
Ensure that the sync registers are cleared every time we restart the ring to avoid stale values from creeping in from random neutrinos. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/intel_ringbuffer.c | 7 +++ 1 file changed, 7 insertions(+) diff --git

[Intel-gfx] [PATCH 1/6] drm/i915: Signal the fences as they are cancelled due to wedging

2018-11-29 Thread Chris Wilson
We inspect the requests under the assumption that they will be marked as completed/signaled when they are removed from the queue. Currently however, in the process of wedging the requests will be removed from the queue before they are completed, so rearrange the code to signal the fences before

[Intel-gfx] [PATCH 6/6] drm/i915: Pipeline PDP updates for Braswell

2018-11-29 Thread Chris Wilson
Currently we face a severe problem on Braswell that manifests as invalid ppGTT accesses. The code tries to maintain the PDP (page directory pointers) inside the context in two ways, direct write into the context and a pipelined LRI update. The direct write into the context is fundamentally racy as

[Intel-gfx] [PATCH 5/6] drm/i915/selftests: Reorder request allocation vs vma pinning

2018-11-29 Thread Chris Wilson
Impose a restraint that we have all vma pinned for a request prior to its allocation. This is to simplify request construction, and should facilitate unravelling the lock interdependencies later. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/selftests/huge_pages.c | 31 +++--

[Intel-gfx] [PATCH 3/6] drm/i915: Allocate a common scratch page

2018-11-29 Thread Chris Wilson
Currently we allocate a scratch page for each engine, but since we only ever write into it for post-sync operations, it is not exposed to userspace nor do we care for coherency. As we then do not care about its contents, we can use one page for all, reducing our allocations and avoid complications

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for HuC Updates [BXT,SKL,KBL,GLK] GuC [GLK] (rev3)

2018-11-29 Thread Patchwork
== Series Details == Series: HuC Updates [BXT,SKL,KBL,GLK] GuC [GLK] (rev3) URL : https://patchwork.freedesktop.org/series/53191/ State : warning == Summary == $ dim checkpatch origin/drm-tip 31a2d0868104 firmware/huc/BXT: Update the HuC version 6615988330bb firmware/huc/SKL: Update HuC

Re: [Intel-gfx] [PATCH] drm/i915: Fix the HDMI hot plug disconnection failure (v4)

2018-11-29 Thread Guang Bai
On Thu, 29 Nov 2018 10:17:49 +0200 Jani Nikula wrote: > On Wed, 28 Nov 2018, Guang Bai wrote: > > On some GEN9 platforms, slowly unplugging (wiggling) the HDMI cable > > makes the kernel to believe the HDMI display is still connected. > > This is because the HDMI DDC lines are disconnected a

Re: [Intel-gfx] [PATCH v8] drm/i915: implement EXTENDED_RECEIVER_CAPABILITY_FIELD_PRESENT

2018-11-29 Thread Manasi Navare
On Thu, Nov 29, 2018 at 11:28:34AM -0800, Rodrigo Vivi wrote: > On Thu, Nov 29, 2018 at 10:34:05AM -0800, Manasi Navare wrote: > > From: Matt Atwood > > > > According to DP spec (2.9.3.1 of DP 1.4) if > > EXTENDED_RECEIVER_CAPABILITY_FIELD_PRESENT is set the addresses in DPCD > > 02200h through

Re: [Intel-gfx] linux-next: Tree for Nov 29 (drm/i915)

2018-11-29 Thread Stephen Rothwell
Hi all, On Thu, 29 Nov 2018 09:09:25 -0800 Randy Dunlap wrote: > > on i386: > > 248 "multiple definition of ...". E.g.: > > LD [M] drivers/gpu/drm/i915/i915.o > ld: drivers/gpu/drm/i915/i915_irq.o: in function `intel_opregion_resume': > i915_irq.c:(.text+0x58f0): multiple definition of

[Intel-gfx] ✓ Fi.CI.IGT: success for Return only active connectors for get_resources ioctl (rev4)

2018-11-29 Thread Patchwork
== Series Details == Series: Return only active connectors for get_resources ioctl (rev4) URL : https://patchwork.freedesktop.org/series/53163/ State : success == Summary == CI Bug Log - changes from CI_DRM_5221_full -> Patchwork_10949_full

[Intel-gfx] [PATCH 2/6] firmware/huc/SKL: Update HuC versiom for SKL

2018-11-29 Thread Anusha
From: Anusha Srivatsa We have an update of huC for SKL. Load the latest verion. Cc: Rodrigo Vivi Signed-off-by: Anusha Srivatsa --- drivers/gpu/drm/i915/intel_huc_fw.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_huc_fw.c

[Intel-gfx] [PATCH 3/6] firmware/huc/KBL: Update HuC for KBL

2018-11-29 Thread Anusha
From: Anusha Srivatsa We have an update of HuC for KBL. Load the latest version. cc: Rodrigo Vivi Signed-off-by: Anusha Srivatsa --- drivers/gpu/drm/i915/intel_huc_fw.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_huc_fw.c

[Intel-gfx] [PATCH 5/6] firmware/guc/glk: Load GuC v11.98 for Geminilake.

2018-11-29 Thread Anusha
From: John Spotswood load the v11.98 guC on geminilake. v2: rebased. v3: Change subject prefix. (Anusha) Cc: Tomi Sarvela Cc: Jani Saarinen Signed-off-by: Anusha Srivatsa Signed-off-by: John Spotswood --- drivers/gpu/drm/i915/intel_guc_fw.c | 10 ++ 1 file changed, 10

[Intel-gfx] [PATCH 6/6] HAX enable HuC for CI

2018-11-29 Thread Anusha
From: Anusha Srivatsa Signed-off-by: Anusha Srivatsa --- drivers/gpu/drm/i915/i915_params.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/i915_params.h b/drivers/gpu/drm/i915/i915_params.h index 7e56c516c815..c681537bcb92 100644 ---

[Intel-gfx] [PATCH 4/6] firmware/huc/GLK: Load HuC for GLK

2018-11-29 Thread Anusha
From: Anusha Srivatsa Load Huc for GLK. Cc: Rodrigo Vivi Signed-off-by: Anusha Srivatsa --- drivers/gpu/drm/i915/intel_huc_fw.c | 12 1 file changed, 12 insertions(+) diff --git a/drivers/gpu/drm/i915/intel_huc_fw.c b/drivers/gpu/drm/i915/intel_huc_fw.c index

[Intel-gfx] [PATCH 0/6] HuC Updates [BXT,SKL,KBL,GLK] GuC [GLK]

2018-11-29 Thread Anusha
Adding the PR- The following changes since commit 1baa34868b2c0a004dc595b20678145e3fff83e7: Merge branch 'nxp_mc' of https://github.com/NXP/linux-firmware (2018-10-26 08:13:19 -0400) are available in the Git repository at: git://anongit.freedesktop.org/drm/drm-firmware firmware_updates

[Intel-gfx] [PATCH 1/6] firmware/huc/BXT: Update the HuC version

2018-11-29 Thread Anusha
From: Anusha Srivatsa We have an update for HuC for BXT. Load the latest version. Cc: Rodrigo Vivi Signed-off-by: Anusha Srivatsa --- drivers/gpu/drm/i915/intel_huc_fw.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_huc_fw.c

[Intel-gfx] ✗ Fi.CI.BAT: failure for HuC Updates [BXT,SKL,KBL,GLK] GuC [GLK] (rev3)

2018-11-29 Thread Patchwork
== Series Details == Series: HuC Updates [BXT,SKL,KBL,GLK] GuC [GLK] (rev3) URL : https://patchwork.freedesktop.org/series/53191/ State : failure == Summary == CI Bug Log - changes from CI_DRM_5227 -> Patchwork_10969 Summary ---

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/2] drm/i915: Check if PSR is globally enabled before change PSR variables

2018-11-29 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915: Check if PSR is globally enabled before change PSR variables URL : https://patchwork.freedesktop.org/series/53293/ State : warning == Summary == $ dim checkpatch origin/drm-tip 3558d3c4ede6 drm/i915: Check if PSR is globally

[Intel-gfx] ✓ Fi.CI.IGT: success for Add support for Gen 11 pipe color features (rev3)

2018-11-29 Thread Patchwork
== Series Details == Series: Add support for Gen 11 pipe color features (rev3) URL : https://patchwork.freedesktop.org/series/51408/ State : success == Summary == CI Bug Log - changes from CI_DRM_5222_full -> Patchwork_10957_full Summary

Re: [Intel-gfx] [PATCH 2/9] drm/i915/psr: Don't tell sink that main link will be active while is active PSR2

2018-11-29 Thread Rodrigo Vivi
On Wed, Nov 28, 2018 at 12:13:30PM -0800, Souza, Jose wrote: > On Wed, 2018-11-28 at 11:02 -0800, Rodrigo Vivi wrote: > > On Mon, Nov 26, 2018 at 04:37:03PM -0800, José Roberto de Souza > > wrote: > > > For PSR2 there is no register to tell HW to keep main link enabled > > > while PSR2 is active,

[Intel-gfx] [PATCH 1/2] drm/i915: Check if PSR is globally enabled before change PSR variables

2018-11-29 Thread José Roberto de Souza
There is no issues changing the PSR variables even if PSR will be not enabled but it avoid having misleading values like have psr2_enabled set but enabled unset. Cc: Maarten Lankhorst Cc: Dhinakaran Pandiyan Cc: Rodrigo Vivi Signed-off-by: José Roberto de Souza ---

[Intel-gfx] [PATCH 2/2] drm/i915/psr: Execute the default PSR code path when setting i915_edp_psr_debug

2018-11-29 Thread José Roberto de Souza
Changing the i915_edp_psr_debug was enabling, disabling or switching PSR version by directly calling intel_psr_disable_locked() and intel_psr_enable_locked(), what is not the default PSR path that is executed in a regular modesets. So lets force a modeset in the PSR CRTC to trigger the requested

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [v2,01/11] drm/i915: Disable PSR in Apple panels

2018-11-29 Thread Patchwork
== Series Details == Series: series starting with [v2,01/11] drm/i915: Disable PSR in Apple panels URL : https://patchwork.freedesktop.org/series/53291/ State : success == Summary == CI Bug Log - changes from CI_DRM_5227 -> Patchwork_10970

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/icl: dsi enabling (rev6)

2018-11-29 Thread Patchwork
== Series Details == Series: drm/i915/icl: dsi enabling (rev6) URL : https://patchwork.freedesktop.org/series/51011/ State : success == Summary == CI Bug Log - changes from CI_DRM_5222_full -> Patchwork_10956_full Summary ---

Re: [Intel-gfx] [PATCH 7/9] drm/i915/psr: Rename PSR2 macros to better match meaning

2018-11-29 Thread Souza, Jose
On Thu, 2018-11-29 at 15:25 -0800, Dhinakaran Pandiyan wrote: > On Thu, 2018-11-29 at 15:07 -0800, Rodrigo Vivi wrote: > > On Mon, Nov 26, 2018 at 04:37:08PM -0800, José Roberto de Souza > > wrote: > > > The first 8 bits of PSR2_CTL have 2 fields to set frames count, > > > the > > > first one is

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/2] drm/i915/backlight: Restore backlight on resume (rev2)

2018-11-29 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915/backlight: Restore backlight on resume (rev2) URL : https://patchwork.freedesktop.org/series/53239/ State : success == Summary == CI Bug Log - changes from CI_DRM_5222_full -> Patchwork_10953_full

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/icl: Remove Wa_1604302699

2018-11-29 Thread Patchwork
== Series Details == Series: drm/i915/icl: Remove Wa_1604302699 URL : https://patchwork.freedesktop.org/series/53244/ State : success == Summary == CI Bug Log - changes from CI_DRM_5222_full -> Patchwork_10955_full Summary ---

[Intel-gfx] [PATCH v2 04/11] drm/i915/psr: Enable sink to trigger a interruption on PSR2 CRC mismatch

2018-11-29 Thread José Roberto de Souza
eDP spec states 2 different bits to enable sink to trigger a interruption when there is a CRC mismatch. DP_PSR_CRC_VERIFICATION is for PSR only and DP_PSR_IRQ_HPD_WITH_CRC_ERRORS is for PSR2 only. Cc: Dhinakaran Pandiyan Reviewed-by: Rodrigo Vivi Signed-off-by: José Roberto de Souza ---

[Intel-gfx] [PATCH v2 09/11] drm/i915: Remove old PSR2 FIXME about frontbuffer tracking

2018-11-29 Thread José Roberto de Souza
Our frontbuffer tracking improved over the years + the WA #0884 helped us keep PSR2 enabled while triggering screen updates when necessary so this FIXME is not valid anymore. Acked-by: Dhinakaran Pandiyan Reviewed-by: Rodrigo Vivi Signed-off-by: José Roberto de Souza ---

[Intel-gfx] [PATCH v2 10/11] drm/i915: Improve PSR2 CTL macros

2018-11-29 Thread José Roberto de Souza
- Reusing the EDP_PSR2_FRAME_BEFORE_SU_SHIFT in EDP_PSR2_FRAME_BEFORE_SU - Removing unused EDP_PSR2_FRAME_BEFORE_SU_MASK - Adding EDP_PSR2_FRAME_BEFORE_SU_MAX - Adding EDP_PSR2_IDLE_FRAME() - Adding EDP_PSR2_IDLE_FRAME_MAX In the next patch the new macros will be used. Cc: Dhinakaran Pandiyan

[Intel-gfx] [PATCH v2 06/11] drm: Add the PSR SU granularity registers offsets

2018-11-29 Thread José Roberto de Souza
Source is required to comply to sink SU granularity when DP_PSR2_SU_GRANULARITY_REQUIRED is set in DP_PSR_CAPS, so adding the registers offsets. v2: Also adding DP_PSR2_SU_Y_GRANULARITY(Rodrigo) Cc: Dhinakaran Pandiyan Cc: Rodrigo Vivi Signed-off-by: José Roberto de Souza ---

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for series starting with [v2,01/11] drm/i915: Disable PSR in Apple panels

2018-11-29 Thread Patchwork
== Series Details == Series: series starting with [v2,01/11] drm/i915: Disable PSR in Apple panels URL : https://patchwork.freedesktop.org/series/53291/ State : warning == Summary == $ dim sparse origin/drm-tip Sparse version: v0.5.2 Commit: drm/i915: Disable PSR in Apple panels Okay!

Re: [Intel-gfx] [PATCH 8/9] drm/i915/psr: Set the right frames values

2018-11-29 Thread Souza, Jose
On Thu, 2018-11-29 at 15:10 -0800, Rodrigo Vivi wrote: > On Mon, Nov 26, 2018 at 04:37:09PM -0800, José Roberto de Souza > wrote: > > EDP_PSR2_IDLE_FRAMES_TO_DEEP_SLEEP() was being set with the number > > of > > frames that it should wait to enter PSR, what is wrong. > > Here it is setting this

Re: [Intel-gfx] [PATCH 8/9] drm/i915/psr: Set the right frames values

2018-11-29 Thread Souza, Jose
On Thu, 2018-11-29 at 15:33 -0800, Dhinakaran Pandiyan wrote: > On Mon, 2018-11-26 at 16:37 -0800, José Roberto de Souza wrote: > > EDP_PSR2_IDLE_FRAMES_TO_DEEP_SLEEP() was being set with the number > > of > > frames that it should wait to enter PSR, what is wrong. > > Here it is setting this

[Intel-gfx] [PATCH v2 08/11] drm/i915/psr: Check if source supports sink specific SU granularity

2018-11-29 Thread José Roberto de Souza
According to eDP spec, sink can required specific selective update granularity that source must comply. Here caching the value if required and checking if source supports it. Cc: Rodrigo Vivi Cc: Dhinakaran Pandiyan Signed-off-by: José Roberto de Souza --- drivers/gpu/drm/i915/i915_drv.h |

[Intel-gfx] [PATCH v2 01/11] drm/i915: Disable PSR in Apple panels

2018-11-29 Thread José Roberto de Souza
i915 yet don't support PSR in Apple panels, so lets keep it disabled while we work on that. v2: Renamed DP_DPCD_QUIRK_PSR_NOT_CURRENTLY_SUPPORTED to DP_DPCD_QUIRK_NO_PSR (Ville) Fixes: 598c6cfe0690 (drm/i915/psr: Enable PSR1 on gen-9+ HW) Cc: Ville Syrjälä Cc: Rodrigo Vivi Cc: Dhinakaran

[Intel-gfx] [PATCH v2 05/11] drm/i915/icl: Do not change reserved registers related to PSR2

2018-11-29 Thread José Roberto de Souza
For ICL the bit 12 of CHICKEN_TRANS is reserved so we should not touch it and as by default VSC_DATA_SEL_SOFTWARE_CONTROL is already unset in gen10 + GLK we can just drop it and fix for both gens. Cc: Dhinakaran Pandiyan Reviewed-by: Rodrigo Vivi Signed-off-by: José Roberto de Souza ---

[Intel-gfx] [PATCH v2 03/11] drm/i915/psr: Set PSR CRC verification bit in sink inside PSR1 block

2018-11-29 Thread José Roberto de Souza
As we have a else block for the 'if (dev_priv->psr.psr2_enabled) {' and this bit is only set for PSR1 move it to that block to make it more easy to read. Cc: Dhinakaran Pandiyan Cc: Rodrigo Vivi Signed-off-by: José Roberto de Souza --- drivers/gpu/drm/i915/intel_psr.c | 5 +++-- 1 file

[Intel-gfx] [PATCH v2 11/11] drm/i915/psr: Set the right frames values

2018-11-29 Thread José Roberto de Souza
EDP_PSR2_FRAME_BEFORE_SU() is the number of frames that PSR2 HW will wait before enter in PSR2 activation state, important to note here is that it will wait for X frames not X idle frames. So lets reuse the previous approch to get the maximum number of frames between 6 and sink_sync_latency to

[Intel-gfx] [PATCH v2 07/11] drm/i915/psr: Check if resolution is supported by default SU granularity

2018-11-29 Thread José Roberto de Souza
Selective updates have a default granularity requirements as stated by eDP spec, so check if HW can match those requirements before enable PSR2. Cc: Dhinakaran Pandiyan Cc: Rodrigo Vivi Signed-off-by: José Roberto de Souza --- drivers/gpu/drm/i915/intel_psr.c | 12 1 file

[Intel-gfx] [PATCH v2 02/11] drm/i915/psr: Don't tell sink that main link will be active while is active PSR2

2018-11-29 Thread José Roberto de Souza
For PSR2 there is no register to tell HW to keep main link enabled while PSR2 is active, so don't configure sink DPCD with a misleading value. v2: Moving the set of DP_PSR_CRC_VERIFICATION to the else block of 'if (dev_priv->psr.psr2_enabled)' to another patch. (Rodrigo) Cc: Dhinakaran Pandiyan

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915: Check if PSR is globally enabled before change PSR variables

2018-11-29 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915: Check if PSR is globally enabled before change PSR variables URL : https://patchwork.freedesktop.org/series/53293/ State : success == Summary == CI Bug Log - changes from CI_DRM_5227 -> Patchwork_10971

Re: [Intel-gfx] [PATCH] drm/i915: Fix the HDMI hot plug disconnection failure (v4)

2018-11-29 Thread Chris Chiu
On Thu, Nov 29, 2018 at 9:52 AM Guang Bai wrote: > On some GEN9 platforms, slowly unplugging (wiggling) the HDMI cable makes > the kernel to believe the HDMI display is still connected. This is because > the HDMI DDC lines are disconnected a little bit later after the hot-plug > interrupt

Re: [Intel-gfx] [PATCH v5 4/5] drm: Add library for shmem backed GEM objects

2018-11-29 Thread Daniel Vetter
On Wed, Nov 28, 2018 at 01:52:56PM -0800, Eric Anholt wrote: > Daniel Vetter writes: > > > On Tue, Nov 27, 2018 at 12:38:44PM -0800, Eric Anholt wrote: > >> Daniel Vetter writes: > >> > >> > On Mon, Nov 26, 2018 at 04:36:21PM -0800, Eric Anholt wrote: > >> >> Noralf Trønnes writes: > >> >> >

Re: [Intel-gfx] [PATCH] drm/i915: Fix the HDMI hot plug disconnection failure (v4)

2018-11-29 Thread Jani Nikula
On Wed, 28 Nov 2018, Guang Bai wrote: > On some GEN9 platforms, slowly unplugging (wiggling) the HDMI cable makes > the kernel to believe the HDMI display is still connected. This is because > the HDMI DDC lines are disconnected a little bit later after the hot-plug > interrupt triggered thus an

Re: [Intel-gfx] [PATCH v2] Return only active connectors for get_resources ioctl

2018-11-29 Thread Lisovskiy, Stanislav
On Thu, 2018-11-29 at 07:37 +, Lisovskiy, Stanislav wrote: > On Wed, 2018-11-28 at 22:21 +0100, Daniel Vetter wrote: > > > > > I tried to read the bug and I have no idea what's going on here. > > > Userspace > > > is supposed to shut off outputs that are disconnected, whether > > > that's DP,

Re: [Intel-gfx] [PATCH] drm/i915/selftests: Terminate hangcheck sanitycheck forcibly

2018-11-29 Thread kbuild test robot
://github.com/0day-ci/linux/commits/Chris-Wilson/drm-i915-selftests-Terminate-hangcheck-sanitycheck-forcibly/20181129-053109 base: git://anongit.freedesktop.org/drm-intel for-linux-next config: i386-allmodconfig (attached as .config) compiler: gcc-7 (Debian 7.3.0-1) 7.3.0 reproduce: # save

Re: [Intel-gfx] [PATCH v2] Return only active connectors for get_resources ioctl

2018-11-29 Thread Daniel Vetter
On Thu, Nov 29, 2018 at 8:37 AM Lisovskiy, Stanislav wrote: > > On Wed, 2018-11-28 at 22:21 +0100, Daniel Vetter wrote: > > On Wed, Nov 28, 2018 at 09:51:13PM +0100, Daniel Vetter wrote: > > > On Wed, Nov 28, 2018 at 03:55:58PM +0200, Stanislav Lisovskiy > > > wrote: > > > > Currently kernel

Re: [Intel-gfx] [PATCH 1/4] drm/edid: Pass connector to AVI inforframe functions

2018-11-29 Thread Andrzej Hajda
On 21.11.2018 19:19, Laurent Pinchart wrote: > Hi Ville, > > Thank you for the patch. > > On Tuesday, 20 November 2018 18:13:42 EET Ville Syrjala wrote: >> From: Ville Syrjälä >> >> Make life easier for drivers by simply passing the connector >> to drm_hdmi_avi_infoframe_from_display_mode() and

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Implement half float formats and pixel normalize property

2018-11-29 Thread Daniel Vetter
On Wed, Nov 28, 2018 at 10:38:13PM -0800, Kevin Strasser wrote: > 64 bpp half float formats are supported on hdr planes only and are subject > to the following restrictions: > * 90/270 rotation not supported > * Yf Tiling not supported > * Frame Buffer Compression not supported > * Color

Re: [Intel-gfx] [PATCH v2] drm/i915/cnl: Fix the formulae for register offsets

2018-11-29 Thread Jani Nikula
On Wed, 28 Nov 2018, Radhakrishna Sripada wrote: > For gen10+ the offsets for Slice PG cntl/ EU PG cntl donot scale well > for higher slices. > > v2: Use _PICK instead of formulae(Jani) > > Cc: Jani Nikula > Cc: Rodrigo Vivi > Cc: Lucs De Marchi > Cc: Daniele Ceraolo Spurio > Signed-off-by:

Re: [Intel-gfx] [PATCH v2] Return only active connectors for get_resources ioctl

2018-11-29 Thread Lisovskiy, Stanislav
On Thu, 2018-11-29 at 09:48 +0100, Daniel Vetter wrote: > I didn't check wayland (well, weston or any of the others, wayland > doesn't exist as an implementation), but -modesetting and -amdgpu. > And > they all have the same issue. It's roughly: > - You unplug > - Kernel sends out uevent > -

Re: [Intel-gfx] [PATCH 1/4] drm/edid: Pass connector to AVI inforframe functions

2018-11-29 Thread Andrzej Hajda
Quite late, hopefully not too late. On 21.11.2018 12:51, Ville Syrjälä wrote: > On Wed, Nov 21, 2018 at 01:40:43PM +0200, Jani Nikula wrote: >> >>> return; >>> diff --git a/drivers/gpu/drm/bridge/sil-sii8620.c >>> b/drivers/gpu/drm/bridge/sil-sii8620.c >>> index

Re: [Intel-gfx] [PATCH] drm/i915/gvt: fix spelling mistake "Interupts" -> "Interrupts"

2018-11-29 Thread Zhenyu Wang
On 2018.11.25 23:25:24 +, Colin King wrote: > From: Colin Ian King > > There is a spelling mistake in an error message, fix it. > > Signed-off-by: Colin Ian King > --- > drivers/gpu/drm/i915/gvt/interrupt.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [Intel-gfx] [PATCH v6 3/6] drm/dp_mst: Start tracking per-port VCPI allocations

2018-11-29 Thread Daniel Vetter
On Wed, Nov 28, 2018 at 08:14:13PM -0500, Lyude Paul wrote: > > On Wed, 2018-11-28 at 09:17 +0100, Daniel Vetter wrote: > > On Tue, Nov 27, 2018 at 08:44:14PM -0500, Lyude Paul wrote: > > > On Tue, 2018-11-27 at 20:44 +0100, Daniel Vetter wrote: > > > > > > We could do this the other way around

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/6] drm/i915: Signal the fences as they are cancelled due to wedging

2018-11-29 Thread Patchwork
== Series Details == Series: series starting with [1/6] drm/i915: Signal the fences as they are cancelled due to wedging URL : https://patchwork.freedesktop.org/series/53283/ State : failure == Summary == CI Bug Log - changes from CI_DRM_5226 -> Patchwork_10964

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/psr: Get pipe id following atomic guidelines (rev4)

2018-11-29 Thread Patchwork
== Series Details == Series: drm/i915/psr: Get pipe id following atomic guidelines (rev4) URL : https://patchwork.freedesktop.org/series/53132/ State : failure == Summary == CI Bug Log - changes from CI_DRM_5226 -> Patchwork_10966 Summary

Re: [Intel-gfx] [PATCH] drm/lease: Send a distinct uevent

2018-11-29 Thread Daniel Vetter
On Thu, Nov 29, 2018 at 10:42 AM Daniel Vetter wrote: > > Sending the exact same hotplug event is not great uapi. Luckily the > only already merged implementation of leases (in the -modesetting > driver) doesn't care about what kind of uevent it gets, and > unconditionally processes both hotplug

Re: [Intel-gfx] [PATCH v2] drm/i915: Pipeline PDP updates for Braswell

2018-11-29 Thread kbuild test robot
Hi Chris, Thank you for the patch! Yet something to improve: [auto build test ERROR on drm-intel/for-linux-next] [also build test ERROR on next-20181129] [cannot apply to v4.20-rc4] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url

Re: [Intel-gfx] [PATCH 8/9] drm/i915/psr: Set the right frames values

2018-11-29 Thread Dhinakaran Pandiyan
On Mon, 2018-11-26 at 16:37 -0800, José Roberto de Souza wrote: > EDP_PSR2_IDLE_FRAMES_TO_DEEP_SLEEP() was being set with the number of > frames that it should wait to enter PSR, what is wrong. > Here it is setting this field with the highest value to avoid PSR2 > exits frequently, as when HW exit

Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/psr: Get pipe id following atomic guidelines (rev4)

2018-11-29 Thread Rodrigo Vivi
On Thu, Nov 29, 2018 at 10:26:04PM -, Patchwork wrote: > == Series Details == > > Series: drm/i915/psr: Get pipe id following atomic guidelines (rev4) > URL : https://patchwork.freedesktop.org/series/53132/ > State : failure > > == Summary == > > CI Bug Log - changes from CI_DRM_5226 ->

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for series starting with [1/6] drm/i915: Signal the fences as they are cancelled due to wedging

2018-11-29 Thread Patchwork
== Series Details == Series: series starting with [1/6] drm/i915: Signal the fences as they are cancelled due to wedging URL : https://patchwork.freedesktop.org/series/53283/ State : warning == Summary == $ dim sparse origin/drm-tip Sparse version: v0.5.2 Commit: drm/i915: Signal the fences

Re: [Intel-gfx] [PATCH 0/3] Support 64 bpp half float formats

2018-11-29 Thread Strasser, Kevin
Ville Syrjälä wrote: > On Wed, Nov 28, 2018 at 10:38:10PM -0800, Kevin Strasser wrote: >> This series defines new formats and adds a plane property to be used for >> floating point framebuffer content. Implementation is then added to i915. >> >> I have shared an IGT branch which adds test coverage

Re: [Intel-gfx] [PATCH 4/9] drm/i915/icl: Do not change reserved registers related to PSR2

2018-11-29 Thread Rodrigo Vivi
On Mon, Nov 26, 2018 at 04:37:05PM -0800, José Roberto de Souza wrote: > For ICL the bit 12 of CHICKEN_TRANS is reserved so we should not > touch it and as by default VSC_DATA_SEL_SOFTWARE_CONTROL is already > unset in gen10 + GLK we can just drop it and fix for both gens. > > Cc: Dhinakaran

Re: [Intel-gfx] [PATCH v9] drm/i915: implement EXTENDED_RECEIVER_CAPABILITY_FIELD_PRESENT

2018-11-29 Thread Atwood, Matthew S
On Thu, 2018-11-29 at 14:00 -0800, Manasi Navare wrote: > From: Matt Atwood > > According to DP spec (2.9.3.1 of DP 1.4) if > EXTENDED_RECEIVER_CAPABILITY_FIELD_PRESENT is set the addresses in > DPCD > 02200h through 0220Fh shall contain the DPRX's true capability. These > values will match

Re: [Intel-gfx] [PATCH 7/9] drm/i915/psr: Rename PSR2 macros to better match meaning

2018-11-29 Thread Rodrigo Vivi
On Mon, Nov 26, 2018 at 04:37:08PM -0800, José Roberto de Souza wrote: > The first 8 bits of PSR2_CTL have 2 fields to set frames count, the > first one is to set how many idle frames PSR2 HW needs to wait before > enter in deep sleep and the second one it is how many frames(it don't > need to be

Re: [Intel-gfx] [v3 1/3] drm/i915/icl: Add icl pipe degamma and gamma support

2018-11-29 Thread Matt Roper
On Thu, Nov 29, 2018 at 08:21:41PM +0530, Uma Shankar wrote: > Add support for icl pipe degamma and gamma. > > v2: Removed a POSTING_READ and corrected the Bit > Definition as per Maarten's comments. > > v3: Addressed Matt's review comments. Removed rmw patterns > as suggested by Matt. > >

Re: [Intel-gfx] [PATCH 3/9] drm/i915/psr: Enable sink to trigger a interruption on PSR2 CRC mismatch

2018-11-29 Thread Dhinakaran Pandiyan
On Thu, 2018-11-29 at 14:04 -0800, Rodrigo Vivi wrote: > On Mon, Nov 26, 2018 at 04:37:04PM -0800, José Roberto de Souza > wrote: > > eDP spec states 2 different bits to enable sink to trigger a > > interruption when there is a CRC mismatch. > > DP_PSR_CRC_VERIFICATION is for PSR only and > >

Re: [Intel-gfx] [PATCH 1/3] drm/fourcc: Add 64 bpp half float formats

2018-11-29 Thread Strasser, Kevin
Ville Syrjälä wrote: > On Wed, Nov 28, 2018 at 10:38:11PM -0800, Kevin Strasser wrote: >> + { .format = DRM_FORMAT_XRGB16161616H, .depth = 48, >> .num_planes = 1, .cpp = { 4, 0, 0 }, .hsub = 1, .vsub = 1, .is_fp = true }, >> + { .format = DRM_FORMAT_XBGR16161616H,

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/icl: push pll to port mapping/unmapping to ddi encoder hooks

2018-11-29 Thread Patchwork
== Series Details == Series: drm/i915/icl: push pll to port mapping/unmapping to ddi encoder hooks URL : https://patchwork.freedesktop.org/series/53237/ State : success == Summary == CI Bug Log - changes from CI_DRM_5221_full -> Patchwork_10950_full

Re: [Intel-gfx] [PATCH] drm/i915: Fixup stub definitions for intel_opregion_suspend|resume

2018-11-29 Thread Randy Dunlap
On 11/29/18 1:05 PM, Chris Wilson wrote: > 248 "multiple definition of ...". E.g.: > > LD [M] drivers/gpu/drm/i915/i915.o > ld: drivers/gpu/drm/i915/i915_irq.o: in function `intel_opregion_resume': > i915_irq.c:(.text+0x58f0): multiple definition of `intel_opregion_resume'; >

Re: [Intel-gfx] [PATCH 6/9] drm/i915/psr: Check if source supports sink specific SU granularity

2018-11-29 Thread Rodrigo Vivi
On Mon, Nov 26, 2018 at 04:37:07PM -0800, José Roberto de Souza wrote: > According to eDP spec, sink can required specific selective update > granularity that source must comply. > Here caching the value if required and checking source supports it. > > Cc: Rodrigo Vivi > Cc: Dhinakaran Pandiyan

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: implement EXTENDED_RECEIVER_CAPABILITY_FIELD_PRESENT (rev4)

2018-11-29 Thread Patchwork
== Series Details == Series: drm/i915: implement EXTENDED_RECEIVER_CAPABILITY_FIELD_PRESENT (rev4) URL : https://patchwork.freedesktop.org/series/49669/ State : success == Summary == CI Bug Log - changes from CI_DRM_5226 -> Patchwork_10967

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: Signal the fences as they are cancelled due to wedging (rev2)

2018-11-29 Thread Patchwork
== Series Details == Series: drm/i915: Signal the fences as they are cancelled due to wedging (rev2) URL : https://patchwork.freedesktop.org/series/53226/ State : failure == Summary == CI Bug Log - changes from CI_DRM_5221_full -> Patchwork_10952_full

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/psr: Get pipe id following atomic guidelines (rev5)

2018-11-29 Thread Patchwork
== Series Details == Series: drm/i915/psr: Get pipe id following atomic guidelines (rev5) URL : https://patchwork.freedesktop.org/series/53132/ State : success == Summary == CI Bug Log - changes from CI_DRM_5227 -> Patchwork_10968 Summary

  1   2   3   >