because revision is a u8. Tweak a bit the condition then.
Cc: Nick Hoath nicholas.ho...@intel.com
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_ringbuffer.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915
v2: Use the recently introduced INTEL_REVID() and SKL_REVID defines
(Nick Hoath)
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_dp.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu
On Wed, Feb 11, 2015 at 03:29:51PM +, Nick Hoath wrote:
On 09/02/2015 19:33, Damien Lespiau wrote:
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_reg.h | 5 +++--
drivers/gpu/drm/i915/intel_ringbuffer.c | 8
2 files changed, 11
On Tue, Feb 10, 2015 at 07:32:17PM +, Damien Lespiau wrote:
This function is only used in intel_ringbuffer.c, so restrict it to that
file. The function was moved around to avoid a forward declaration and
group it with its user.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
On Mon, Feb 09, 2015 at 07:03:26PM +0100, Daniel Vetter wrote:
Just a little demo really. We probably need to introduce skl specific
functions for a lot of the format validation stuff, or at least
helpers. Specifically I think intel_framebuffer_init and
intel_fb_align_height must be adjusted
On Mon, Feb 09, 2015 at 09:14:26PM +, Chris Wilson wrote:
On Mon, Feb 09, 2015 at 07:25:56PM +, Damien Lespiau wrote:
v2: Reorder defines (Ben)
Bikeshed time?
Oh no, why did I give in the first time? now I'm being bullied!
/* WaCatErrorRejectionIssue */
#define
v2: Reorder defines (Ben)
v3: More bikesheds, this time re-ordering comments! (Chris)
Reviewed-by: Ben Widawsky b...@bwidawsk.net
Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_reg.h | 5
On Fri, Nov 14, 2014 at 08:52:30AM -0800, Rodrigo Vivi wrote:
+struct psr_table {
+ /* Feature bits */
+ u8 full_link:1;
+ u8 require_aux_to_wakeup:1;
+ u8 feature_bits_rsvd:6;
+
+ /* Wait times */
+ u8 idle_frames:4;
+ u8 lines_to_wait:3;
+ u8
This function is only used in intel_lrc.c, so restrict it to that file. The
function was moved around to avoid a forward declaration and group it with its
user.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_lrc.c | 9 +
drivers/gpu/drm/i915
The last (only?) user of this was removed in:
commit 2208d655a91f9879bd9a39ff9df05dd668b3512c
Author: Daniel Vetter daniel.vet...@ffwll.ch
Date: Fri Nov 14 09:25:29 2014 +0100
drm/i915: drop WaSetupGtModeTdRowDispatch:snb
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
This header has been unusued since:
commit 063c86f60ad4064b2cf62041bee8c6389e180b76
Author: Jani Nikula jani.nik...@intel.com
Date: Fri Jan 16 14:27:27 2015 +0200
drm/i915/dsi: remove intel_dsi_cmd.c and the unused functions therein
Signed-off-by: Damien Lespiau damien.lesp
from the documentation
generation as it's not exposed from its compilation unit anymore.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_lrc.c | 218 +++
drivers/gpu/drm/i915/intel_lrc.h | 3 -
2 files changed, 109
This function is only used in intel_ringbuffer.c, so restrict it to that
file. The function was moved around to avoid a forward declaration and
group it with its user.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_ringbuffer.c | 124
This function is only used in intel_lrc.c, so restrict it to that file. The
function was moved around to avoid a forward declaration and group it with its
user.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_lrc.c | 66
This function is only used in intel_dp.c since:
commit 0e32b39ceed665bfa4a77a4bc307b6652b991632
Author: Dave Airlie airl...@redhat.com
Date: Fri May 2 14:02:48 2014 +1000
drm/i915: add DP 1.2 MST support (v0.7)
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers
There have been quite a bit of development lately, leaving behing lonely
protypes. Time to bid them farewell.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_drv.h | 4
drivers/gpu/drm/i915/intel_drv.h | 9 -
2 files changed, 13 deletions
This function is not used outside of intel_display.c since;
commit cf4c7c12258ed9367f4fc45238f5f50d2db892c1
Author: Matt Roper matthew.d.ro...@intel.com
Date: Thu Dec 4 10:27:42 2014 -0800
drm/i915: Make all plane disables use 'update_plane' (v5)
Signed-off-by: Damien Lespiau
Ran my cleanup script again and caught a few tiny oversights.
--
Damien
Damien Lespiau (10):
drm/i915: Garbage collect orphaned prototypes
drm/i915: Make intel_ring_setup_status_page() static
drm/i915: Remove intel_dsi_cmd.h
drm/i915: Make intel_lr_context_render_state_init() static
This was introduced in:
commit 0bc12bcb1b9686d7011f16410ba17ed0740167c3
Author: Rodrigo Vivi rodrigo.v...@intel.com
Date: Fri Nov 14 08:52:28 2014 -0800
drm/i915: Introduce intel_psr.c
Put the unpack function is unused at this date.
Signed-off-by: Damien Lespiau damien.lesp
v2: Reorder defines (Ben)
Reviewed-by: Ben Widawsky b...@bwidawsk.net
Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_reg.h | 5 +++--
drivers/gpu/drm/i915/intel_ringbuffer.c | 2 ++
2 files
WaDisableAsyncFlipPerfMode isn't listed for SKL and
INSTPM_FORCE_ORDERING is MBZ so let's make a gen9 specific render init
function.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_lrc.c | 16 +++-
1 file changed, 15 insertions(+), 1 deletion
This function will host SKL-only W/As.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_ringbuffer.c | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c
b/drivers/gpu/drm/i915
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_reg.h | 1 +
drivers/gpu/drm/i915/intel_ringbuffer.c | 4
2 files changed, 5 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
index 610fcd4..090ddd7
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_drv.h | 1 +
drivers/gpu/drm/i915/i915_reg.h | 3 +++
drivers/gpu/drm/i915/intel_pm.c | 5 +
3 files changed, 9 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_reg.h | 1 +
drivers/gpu/drm/i915/intel_ringbuffer.c | 3 +++
2 files changed, 4 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
index fdfbdb3..d519ed9 100644
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_reg.h | 1 +
drivers/gpu/drm/i915/intel_pm.c | 6 ++
2 files changed, 7 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
index 3ae7a09..b363c5e 100644
This W/A is put in a gen9 specific function because it may well be
needed on other gen9 platforms.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_reg.h | 3 +++
drivers/gpu/drm/i915/intel_pm.c | 11 +++
2 files changed, 14 insertions(+)
diff --git
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_reg.h | 4
drivers/gpu/drm/i915/intel_ringbuffer.c | 8
2 files changed, 12 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
index 578fd90
When enabling new platforms, we may not have any W/A to apply,
especially that, now, a bunch of them have to be done from the ring.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_pm.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_dp.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c
index d4c82d7..4a60c6a 100644
--- a/drivers/gpu/drm/i915/intel_dp.c
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_reg.h | 1 +
drivers/gpu/drm/i915/intel_pm.c | 4
2 files changed, 5 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
index 2043e82..a457c28 100644
--- a/drivers
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_reg.h | 3 +++
drivers/gpu/drm/i915/intel_pm.c | 7 ++-
2 files changed, 9 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
index b363c5e..610fcd4
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_reg.h | 1 +
drivers/gpu/drm/i915/intel_ringbuffer.c | 7 +++
2 files changed, 8 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
index 090ddd7..b4abd50
On Mon, Feb 09, 2015 at 03:56:20PM +0200, Ville Syrjälä wrote:
@@ -2544,8 +2557,9 @@ skl_allocate_pipe_ddb(struct drm_crtc *crtc,
* promote the expression to 64 bits to avoid overflowing, the
* result is available as data_rate / total_data_rate 1
*/
On Mon, Feb 09, 2015 at 03:06:09PM +0530, Kumar, Mahesh wrote:
Fifo Underrun is observed when allocating minimum allocable blocks
for any plane, This patch calculate checks for upper lower DDB
bound for each plane according to total allocated DDB for that Pipe.
Signed-off-by: Kumar,
).
So, instead, start by allocating 8 blocks to each active display plane
and then allocate the remaining blocks like before.
v2: Rebase on top of -nightly
Cc: Mahesh Kumar mahesh1.ku...@intel.com
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_pm.c | 22
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_uncore.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/i915/intel_uncore.c
b/drivers/gpu/drm/i915/intel_uncore.c
index c47a3ba..ad71575 100644
--- a/drivers/gpu/drm/i915/intel_uncore.c
Let's also take the opportunity the remove the comment telling it's a
pre-prod W/A, it should be obvious from the stepping test.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_reg.h | 1 +
drivers/gpu/drm/i915/intel_pm.c | 3 ++-
2 files changed, 3
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_pm.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index a3b979d..6fd6f26 100644
--- a/drivers/gpu/drm/i915/intel_pm.c
+++ b/drivers/gpu
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_reg.h | 5 +++--
drivers/gpu/drm/i915/intel_ringbuffer.c | 8
2 files changed, 11 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
Following Nick's work, I found a few more.
--
Damien
Damien Lespiau (18):
drm/i915: Support not having an init clock gating function defined
drm/i915/skl: Implement WaDisableHBR2
drm/i915/skl: Document the WM read latency W/A with its name
drm/i915/skl: Provide a gen9 specific
We'll gather cross-gen9 W/A in a separate function later.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_pm.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index
On Fri, Feb 06, 2015 at 11:30:04AM +, Nick Hoath wrote:
Add:
WaEnableForceRestoreInCtxtDescForVCS
v2: Add stepping check.
v3: Fixed stepping check direction. Cleaned up indentation.
Signed-off-by: Nick Hoath nicholas.ho...@intel.com
Reviewed-by: Damien Lespiau damien.lesp
looks really better as x == 0 || x == 1
Otherwise:
Reviewed-by: Damien Lespiau damien.lesp...@intel.com
/*
* WaDisableDgMirrorFixInHalfSliceChicken5:skl
* This is a pre-production w/a.
--
2.1.1
On Thu, Feb 05, 2015 at 11:12:20AM -0800, Matt Roper wrote:
I think the only thing we need to close on is Ville's concern about the
plane winding up with non-NULL fb and crtc pointers even though a crazy
firmware or bootloader somehow left the plane turned off. I'm not super
familiar with the
take over from HW state just yet, so
fake enough of the plane atomic state to pass the checks.
v2: Fix the state on which we set the CRTC in the case we're sharing the
initial fb with another pipe. (Matt)
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915
the state in struct intel_initial_plane_config
until we know we can reuse that framebuffer.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_display.c | 14 +++---
drivers/gpu/drm/i915/intel_drv.h | 1 +
2 files changed, 8 insertions(+), 7 deletions
take over from HW state just yet, so
fake enough of the plane atomic state to pass the checks.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_display.c | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915
there then.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_display.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_display.c
b/drivers/gpu/drm/i915/intel_display.c
index e5c0579..8900ce0 100644
--- a/drivers
On Thu, Feb 05, 2015 at 10:47:16AM +, Nick Hoath wrote:
From: Hoath, Nicholas nicholas.ho...@intel.com
Add framework for gen 9 HW WAs
v1: Changed SOC specific WA function to gen 9 common function (Req: Damien
Lespiau)
Signed-off-by: Nick Hoath nicholas.ho...@intel.com
Reviewed
Ursulin tvrtko.ursu...@intel.com
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_display.c | 35 +--
1 file changed, 17 insertions(+), 18 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_display.c
b/drivers/gpu/drm/i915
with plane-fb
The series tries to address those.
--
Damien
Damien Lespiau (5):
drm/i915: Put update_state_fb() next to the fb update
drm/i915: Use an intermediate variable to avoid repeating ourselves
drm/i915: Don't try to reference the fb in get_initial_plane_config()
drm/i915: Store
-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_drv.h | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index ca64b99..39e8b07 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu
and mention SKL:
drm/i915/skl: Implement WaDisablePartialInstShootdown
v1: Dont add WaDisableThreadStallDopClockGating as not SKL WA. (Found by
Damien Lespiau)
v2, v1 was the original patch. Commit messages should also be wrapped
like emails, so 90 chars is too long.
Reviewed-by: Damien
On Thu, Feb 05, 2015 at 10:47:19AM +, Nick Hoath wrote:
Move WaDisableDgMirrorFixInHalfSliceChicken5 to gen9_init_workarounds
v1: Added stepping check
v2: Removed unused register bitmap
Signed-off-by: Nick Hoath nicholas.ho...@intel.com
---
drivers/gpu/drm/i915/intel_pm.c
On Thu, Feb 05, 2015 at 05:55:06PM +, Damien Lespiau wrote:
+ if (INTEL_REVID(dev) == SKL_A0_REVID) {
for SKL, I read B0 only.
Well B0 only in the W/A db, but A0 and B0 in BSpec. I'd trust BSpec on
those.
--
Damien
___
Intel-gfx mailing
On Thu, Feb 05, 2015 at 10:10:26AM -0800, Matt Roper wrote:
On Thu, Feb 05, 2015 at 05:22:19PM +, Damien Lespiau wrote:
Right now, we get a warning when taking over the firmware fb:
[drm:drm_atomic_plane_check] FB set but no CRTC
with the following backtrace
On Thu, Feb 05, 2015 at 10:47:20AM +, Nick Hoath wrote:
Added:
Syncing dependencies between camera and graphics
v1: Added missing register bitmap
Signed-off-by: Nick Hoath nicholas.ho...@intel.com
For the record, this W/A has no name nor documentation. So well...
Reviewed-by: Damien
On Thu, Feb 05, 2015 at 10:47:21AM +, Nick Hoath wrote:
From: Hoath, Nicholas nicholas.ho...@intel.com
Add stepping check for WaDisableSDEUnitClockGating.
Signed-off-by: Nick Hoath nicholas.ho...@intel.com
Reviewed-by: Damien Lespiau damien.lesp...@intel.com
--
Damien
---
drivers
On Thu, Feb 05, 2015 at 10:47:22AM +, Nick Hoath wrote:
Move WaEnableYV12BugFixInHalfSliceChicken7 to gen9_init_workarounds
v1: Add stepping check.
Signed-off-by: Nick Hoath nicholas.ho...@intel.com
Reviewed-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915
On Thu, Feb 05, 2015 at 10:47:23AM +, Nick Hoath wrote:
From: Hoath, Nicholas nicholas.ho...@intel.com
Move Wa4x4STCOptimizationDisable to gen9_init_workarounds
v1: rebase
Signed-off-by: Nick Hoath nicholas.ho...@intel.com
Reviewed-by: Damien Lespiau damien.lesp...@intel.com
Reviewed-by: Damien Lespiau damien.lesp...@intel.com
--
Damien
---
drivers/gpu/drm/i915/intel_ringbuffer.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c
b/drivers/gpu/drm/i915/intel_ringbuffer.c
index 066b63d..36885b4 100644
On Thu, Feb 05, 2015 at 10:47:25AM +, Nick Hoath wrote:
From: Hoath, Nicholas nicholas.ho...@intel.com
Add:
WaEnableForceRestoreInCtxtDescForVCS
v1: Add stepping check.
Signed-off-by: Nick Hoath nicholas.ho...@intel.com
---
drivers/gpu/drm/i915/intel_lrc.c | 15 ---
() in intel_alloc_plane_obj() before
finding a better place. (Matt)
Reviewed-by: Matt Roper matthew.d.ro...@intel.com
Reported-by: Tvrtko Ursulin tvrtko.ursu...@intel.com
Cc: Matt Roper matthew.d.ro...@intel.com
Cc: Tvrtko Ursulin tvrtko.ursu...@intel.com
Signed-off-by: Damien Lespiau damien.lesp
Syrjälä ville.syrj...@linux.intel.com
Suggested-by: Matt Roper matthew.d.ro...@intel.com
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_display.c | 15 +++
1 file changed, 11 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/i915
On Tue, Feb 03, 2015 at 01:06:31AM +0200, Imre Deak wrote:
+static struct i915_power_well skl_power_wells[] = {
+ {
+ .name = always-on,
+ .always_on = 1,
+ .domains = SKL_DISPLAY_ALWAYS_ON_POWER_DOMAINS,
+ .ops = i9xx_always_on_power_well_ops,
On Tue, Feb 03, 2015 at 05:51:29PM -0800, Rodrigo Vivi wrote:
On Thu, Jan 29, 2015 at 6:13 AM, Damien Lespiau
damien.lesp...@intel.com wrote:
We need to have a separate GT3 struct intel_device_info to declare they
have a second VCS. Let's start by splitting the PCI ids per-GT.
Signed-off
v2: leave intel_skylake_info alone (Rodrigo, Daniel)
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_drv.c | 17 -
1 file changed, 16 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c
.
Reviewed-by: Imre Deak imre.d...@intel.com
Signed-off-by: Satheeshakrishna M satheeshakrishn...@intel.com (v3,v6,v7)
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_reg.h | 20 +++
drivers/gpu/drm/i915/intel_runtime_pm.c | 220
On Wed, Feb 04, 2015 at 04:20:28PM +0200, Imre Deak wrote:
On ke, 2015-02-04 at 13:53 +, Damien Lespiau wrote:
On Tue, Feb 03, 2015 at 01:06:31AM +0200, Imre Deak wrote:
+static struct i915_power_well skl_power_wells[] = {
+ {
+ .name = always
On Tue, Feb 03, 2015 at 05:51:29PM -0800, Rodrigo Vivi wrote:
On Thu, Jan 29, 2015 at 6:13 AM, Damien Lespiau
damien.lesp...@intel.com wrote:
We need to have a separate GT3 struct intel_device_info to declare they
have a second VCS. Let's start by splitting the PCI ids per-GT.
Signed-off
At the moment we compare the whole EDRAM_PRESENT/EDRAMCAP register value
to 1 while EDRAM_PRESENT is only bit 0 (the rest may be used to describe
eDRAM capabilities).
To be more future proof, only look at bit 0 to detect eDRAM presence.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
$ make arch/x86/kernel/early-quirks.s
before and after the patch. No change.
I'm sold by the test so:
Reviewed-by: Damien Lespiau damien.lesp...@intel.com
I did double check with the specs though and found:
- 0x160B is marked as reserved
- 0x161B is marked as reserved
- A few more ids
Suggested-by: Daniel Vetter dan...@ffwll.ch
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_uncore.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_uncore.c
b/drivers/gpu/drm/i915/intel_uncore.c
index 00c91be
On Mon, Feb 02, 2015 at 02:33:48PM +0200, Ville Syrjälä wrote:
On Thu, Jan 08, 2015 at 07:59:10PM -0800, Ben Widawsky wrote:
Implements a required workaround whose implications aren't entirely clear
to me
from the description. In particular I do not know if this effects legacy
contexts,
On Fri, Jan 30, 2015 at 09:30:07AM +0200, Jani Nikula wrote:
On a related note, I'm contemplating sending a patch to obliterate the
_INTEL_BDW_M and _INTEL_BDW_D macros from i915_pciids.h because it hides
the IDs from a simple grep.
Yes, please!
--
Damien
On Fri, Jan 30, 2015 at 05:23:06PM +0100, Daniel Vetter wrote:
On Thu, Jan 29, 2015 at 12:42:35PM +, Damien Lespiau wrote:
Suggested-by: Daniel Vetter dan...@ffwll.ch
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
Hm, I've thought the magic bit moved ... or have you found
On Thu, Jan 29, 2015 at 12:03:19AM -0800, Imre Deak wrote:
Without emitting the default 3DSTATE_WM_DEPTH_STENCIL state the test
will fail.
Signed-off-by: Imre Deak imre.d...@intel.com
Question: Wasn't the golden context supposed to paper over those?
--
Damien
---
lib/rendercopy_gen9.c
On Thu, Jan 29, 2015 at 11:32:46AM +, Chris Wilson wrote:
On Thu, Jan 29, 2015 at 11:17:04AM +, Damien Lespiau wrote:
On Thu, Jan 29, 2015 at 11:12:46AM +, Chris Wilson wrote:
On Thu, Jan 29, 2015 at 03:01:50AM -0800, Imre Deak wrote:
On Thu, 2015-01-29 at 10:51 +, Damien
On Thu, Jan 29, 2015 at 11:12:46AM +, Chris Wilson wrote:
On Thu, Jan 29, 2015 at 03:01:50AM -0800, Imre Deak wrote:
On Thu, 2015-01-29 at 10:51 +, Damien Lespiau wrote:
On Thu, Jan 29, 2015 at 12:03:19AM -0800, Imre Deak wrote:
Without emitting the default
On Wed, Jan 28, 2015 at 11:21:17AM +0200, Jani Nikula wrote:
On Tue, 27 Jan 2015, Damien Lespiau damien.lesp...@intel.com wrote:
Missing commit message. I need some description to decide whether this
is required for fixes/stable or not.
This patch wasn't prompted by an actual problem I
Suggested-by: Daniel Vetter dan...@ffwll.ch
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_uncore.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_uncore.c
b/drivers/gpu/drm/i915/intel_uncore.c
index 0e9bf82
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_drv.c | 19 +--
1 file changed, 17 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c
index 6484229..9fdaf64 100644
--- a/drivers/gpu/drm
We need to have a separate GT3 struct intel_device_info to declare they
have a second VCS. Let's start by splitting the PCI ids per-GT.
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
include/drm/i915_pciids.h | 28 +++-
1 file changed, 19 insertions(+), 9
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_ringbuffer.c | 8 +---
1 file changed, 1 insertion(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c
b/drivers/gpu/drm/i915/intel_ringbuffer.c
index d393026..bbe439d 100644
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_reg.h | 1 +
drivers/gpu/drm/i915/intel_ringbuffer.c | 2 ++
2 files changed, 3 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
index ae8ea42..47bc4e2 100644
On Thu, Jan 22, 2015 at 01:41:48PM +, Tvrtko Ursulin wrote:
@@ -718,7 +718,7 @@ struct drm_i915_gem_execbuffer2 {
#define I915_EXEC_CONSTANTS_REL_SURFACE (26) /* gen4/5 only */
__u64 flags;
__u64 rsvd1; /* now used for context info */
- __u64 rsvd2;
+ __u64 rsvd2; /* now
On Thu, Jan 22, 2015 at 03:28:04PM +, Tvrtko Ursulin wrote:
On 01/22/2015 02:04 PM, Damien Lespiau wrote:
On Thu, Jan 22, 2015 at 01:41:48PM +, Tvrtko Ursulin wrote:
@@ -718,7 +718,7 @@ struct drm_i915_gem_execbuffer2 {
#define I915_EXEC_CONSTANTS_REL_SURFACE (26) /* gen4/5 only
On Thu, Jan 22, 2015 at 03:54:29PM +, Tvrtko Ursulin wrote:
On 01/22/2015 03:47 PM, Damien Lespiau wrote:
On Thu, Jan 22, 2015 at 03:28:04PM +, Tvrtko Ursulin wrote:
On 01/22/2015 02:04 PM, Damien Lespiau wrote:
On Thu, Jan 22, 2015 at 01:41:48PM +, Tvrtko Ursulin wrote
Suggested-by: Ville Syrjälä ville.syrj...@linux.intel.com
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_display.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_display.c
b/drivers/gpu/drm/i915
-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_display.c | 21 +++--
1 file changed, 15 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_display.c
b/drivers/gpu/drm/i915/intel_display.c
index 65fce56..01dc80b 100644
--- a/drivers/gpu/drm
On Tue, Jan 20, 2015 at 11:14:22AM +0100, Daniel Vetter wrote:
On Fri, Jan 16, 2015 at 06:07:26PM +, Damien Lespiau wrote:
v2: Use the new function, gen6_init_rps_frequencies() (Damien)
Reviewed-by: Mika Kuoppala mika.kuopp...@intel.com (v1)
Signed-off-by: Damien Lespiau damien.lesp
v2: Use the new function, gen6_init_rps_frequencies() (Damien)
Reviewed-by: Mika Kuoppala mika.kuopp...@intel.com (v1)
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_pm.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/i915/intel_pm.c
by the
forcewake framework
--
Damien
Damien Lespiau (1):
drm/i915/skl: Retrieve the frequency limits
Jesse Barnes (1):
drm/i915/skl: add turbo support
Zhe Wang (1):
drm/i915/skl: Gen9 coarse power gating
drivers/gpu/drm/i915/i915_reg.h | 3 +++
drivers/gpu/drm/i915/intel_pm.c | 40
(Damien)
Reviewed-by: Damien Lespiau damien.lesp...@intel.com
Signed-off-by: Zhe Wang zhe1.w...@intel.com
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_reg.h | 3 +++
drivers/gpu/drm/i915/intel_pm.c | 8
2 files changed, 11 insertions(+)
diff --git
forcewake series
Reviewed-by: Mika Kuoppala mika.kuopp...@intel.com
Signed-off-by: Jesse Barnes jbar...@virtuousgeek.org
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_pm.c | 30 ++
1 file changed, 30 insertions(+)
diff --git
...@ffwll.ch
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/i915_drv.h | 6 ++---
drivers/gpu/drm/i915/intel_display.c | 44 ++--
drivers/gpu/drm/i915/intel_drv.h | 4 ++--
drivers/gpu/drm/i915/intel_fbdev.c | 2 +-
4 files
-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_display.c | 114 ---
1 file changed, 107 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_display.c
b/drivers/gpu/drm/i915/intel_display.c
index 8bc260c..84e79b8 100644
v2: Fix conflict caused by the introduction of struct intel_crtc_state
Reviewed-By: Tvrtko Ursulin tvrtko.ursu...@intel.com (v1)
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
drivers/gpu/drm/i915/intel_display.c | 2 +-
drivers/gpu/drm/i915/intel_drv.h | 1 -
2 files changed, 1
crtc-base.primary-fb was used everywhere. Use fb to temporarily point
there and don't forget to assign fb to its final destination at the end.
v2: Rebase on top of misc changes (mask of DSPSURF, PAGE_ALIGN)
Reviewed-By: Tvrtko Ursulin tvrtko.ursu...@intel.com
Signed-off-by: Damien Lespiau
501 - 600 of 2450 matches
Mail list logo