Re: [Intel-gfx] [PATCH] drm/i915: Audio N value computed for pixel doubled modes

2014-09-24 Thread Jani Nikula
On Thu, 25 Sep 2014, clinton.a.tay...@intel.com wrote: > From: Clint Taylor > > HDMI audio clock config was incorrectly choosing the default for > pixel doubled interlaced modes. The table was missing pixel clock > values 13.500 (27.000) and 13.513 (27.027). Luckily the default N > value for 25.20

Re: [Intel-gfx] [PATCH] drm/i915: Enable pixel replicated modes on BDW and HSW.

2014-09-24 Thread Ville Syrjälä
On Wed, Sep 24, 2014 at 09:42:18PM +0200, Daniel Vetter wrote: > On Wed, Sep 24, 2014 at 08:44:42AM -0700, Clint Taylor wrote: > > On 09/24/2014 01:51 AM, Daniel Vetter wrote: > > >On Tue, Sep 23, 2014 at 11:06:56AM -0700, clinton.a.tay...@intel.com wrote: > > >>From: Clint Taylor > > >> > > >>Has

Re: [Intel-gfx] ACPI/i915: Cannot configure display brightness on Dell Latitude E6440

2014-09-24 Thread Aaron Lu
Hi Hans, Thanks for following up and explaining the situation to Pali. On 09/25/2014 02:21 AM, Pali Rohár wrote: > On Wednesday 24 September 2014 16:34:21 Hans de Goede wrote: >> Ok, so the dell-laptop interface is just an obsolete wrapper >> around the i915 opregion code, which shows that the ri

Re: [Intel-gfx] [PATCH 56/89] drm/i915: Gen9 shadowed registers

2014-09-24 Thread Bob Wang
On 09/24/2014 02:36 PM, Mika Kuoppala wrote: Damien Lespiau writes: From: Zhe Wang For MMIO registers which are shadowed, force wake is not needed to write to these registers. v2: Rebase on top of nightly (Damien) Signed-off-by: Zhe Wang Signed-off-by: Damien Lespiau --- drivers/gpu/dr

Re: [Intel-gfx] [PATCH 55/89] drm/i915/skl: Gen9 multi-engine forcewake

2014-09-24 Thread Bob Wang
On 09/22/2014 04:11 PM, Mika Kuoppala wrote: Damien Lespiau writes: From: Zhe Wang Enable multi-engine forcewake for Gen9. v2: Rebase on top of nightly Move the register range definitions to intel_uncore.c Whitespace fixes (Damien) Signed-off-by: Zhe Wang Signed-off-by: Dam

Re: [Intel-gfx] [PATCH 54/89] drm/i915/skl: Enable Gen9 RC6

2014-09-24 Thread Bob Wang
On 09/22/2014 02:15 PM, Mika Kuoppala wrote: Damien Lespiau writes: From: Zhe Wang Configure and enable RC6 for Gen9. v2: Rebase on top of BDW rc6 support (Damien) Signed-off-by: Zhe Wang Signed-off-by: Damien Lespiau --- drivers/gpu/drm/i915/intel_pm.c | 52 +++

[Intel-gfx] [PATCH 2/2] drm/i915: Broadwell DDI Buffer translation - more tuning

2014-09-24 Thread Rodrigo Vivi
BDW display - DP buffer translation values changed to give better margin. Further change to entry 6; set dword 0 bit 31=1. Both changes were approved already but this one didn't landed BSpec yet this is why it is in a separated patch. Making reviewer's life easier. Also alowing separated tests an

[Intel-gfx] [PATCH 1/2] drm/i915: Broadwell DDI Buffer translation changed to give better margin.

2014-09-24 Thread Rodrigo Vivi
Reference: Predator r73977 / HSD 4394389 Cc: Arthur Runyan Cc: Paulo Zanoni Signed-off-by: Rodrigo Vivi --- drivers/gpu/drm/i915/intel_ddi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_ddi.c b/drivers/gpu/drm/i915/intel_ddi.c index ba1103f

[Intel-gfx] [PATCH v2] drm/i915: Enable pixel replicated modes on BDW and HSW.

2014-09-24 Thread clinton . a . taylor
From: Clint Taylor Haswell and later silicon has added a new pixel replication register to the pipe timings for each transcoder. Now in addition to the DPLL_A_MD register for the pixel clock double, we also need to write to the TRANS_MULT_n (0x6002c) register to double the pixel data. Writing to

[Intel-gfx] [PATCH] drm/i915: Minimize the huge amount of unecessary fbc sw cache clean.

2014-09-24 Thread Rodrigo Vivi
The sw cache clean on BDW is a tempoorary workaround because we cannot set cache clean on blt ring with risk of hungs. So we are doing the cache clean on sw. However we are doing much more than needed. Not only when using blt ring. So, with this extra w/a we minimize the ammount of cache cleans an

[Intel-gfx] [PATCH] drm/i915: Minimize the huge amount of unecessary fbc sw cache clean.

2014-09-24 Thread Rodrigo Vivi
The sw cache clean on BDW is a tempoorary workaround because we cannot set cache clean on blt ring with risk of hungs. So we are doing the cache clean on sw. However we are doing much more than needed. Not only when using blt ring. So, with this extra w/a we minimize the ammount of cache cleans an

[Intel-gfx] [PATCH] drm/i915/hdmi: Compute port_clock for 27.027 pixel replicated modes

2014-09-24 Thread clinton . a . taylor
From: Clint Taylor port_clock was being incorrectly computed and WRPLL was incorrectly programmed for pixel doubled modes using a 27.027MHz pixel clock. port_clock was set to 27.026 resulting in an output pixel clock matching 27.000MHz. Since there is no way to correctly half the 27.027 frequency

[Intel-gfx] [PATCH] drm/i915/bdw: Remove BDW preproduction W/As until C stepping.

2014-09-24 Thread Rodrigo Vivi
Let's clean this a bit v2: Rebase after other Mika's patch that removed some BDW production workarounds. Signed-off-by: Rodrigo Vivi --- drivers/gpu/drm/i915/intel_pm.c | 10 -- drivers/gpu/drm/i915/intel_ringbuffer.c | 5 ++--- 2 files changed, 2 insertions(+), 13 deletions(-

[Intel-gfx] [PATCH] drm/i915: Let number of workarounds more clear

2014-09-24 Thread Rodrigo Vivi
This helps when including or removing cs workarounds. v2: Rebase after other Mika's patch that removed some BDW production workarounds. Signed-off-by: Rodrigo Vivi --- drivers/gpu/drm/i915/intel_ringbuffer.c | 16 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/

[Intel-gfx] [PATCH] drm/i915: Audio N value computed for pixel doubled modes

2014-09-24 Thread clinton . a . taylor
From: Clint Taylor HDMI audio clock config was incorrectly choosing the default for pixel doubled interlaced modes. The table was missing pixel clock values 13.500 (27.000) and 13.513 (27.027). Luckily the default N value for 25.200 is the same N value for both 27MHz pixel clocks, a warning messa

[Intel-gfx] [PATCH] drm/i915: Make sure PSR is ready for been re-enabled.

2014-09-24 Thread Rodrigo Vivi
Let's make sure PSR is propperly disabled before to re-enabled it. According to Spec, after disabled PSR CTL, the Idle state might occur up to 24ms, that is one full frame time (1/refresh rate), plus SRD exit training time (max of 6ms), plus SRD aux channel handshake (max of 1.5ms). So if somethi

Re: [Intel-gfx] [Qemu-devel] [IGDVFIO] [PATCH 3/8] RFC and help completing: Intel IGD Direct Assignment with VFIO

2014-09-24 Thread Paolo Bonzini
Il 24/09/2014 22:57, Alex Williamson ha scritto: > Right, that's the physical mapping, Andy's patches are mimicking that > behavior virtually. Seabios reserves memory, creates e820 entries, and > "maps" the hardware by writing to these registers. That triggers QEMU > to adjust the MemoryRegion in

Re: [Intel-gfx] [PULL] topic/core-stuff

2014-09-24 Thread Dave Airlie
On 25 September 2014 03:59, Daniel Vetter wrote: > On Wed, Sep 24, 2014 at 6:24 PM, Ilia Mirkin wrote: >> On Wed, Sep 24, 2014 at 6:24 AM, Daniel Vetter >> wrote: >>> Hi Dave, >>> >>> Just noticed that you've picked up the header rework stuff already, so >>> I've rebased that out again. Otherwi

Re: [Intel-gfx] [PATCH 3/3] tools/null_state_gen: Add GEN9 golden context batch buffer creation

2014-09-24 Thread Volkin, Bradley D
I went through and compared this against both the spec (the state commands listed in 3D-Media-GPGPU chapter 3D Pipeline Stages section) and the other information I've seen on recommended setup, and as far as I can tell this looks good. It still might be worth getting another set of eyes on this, bu

Re: [Intel-gfx] [Qemu-devel] [IGDVFIO] [PATCH 3/8] RFC and help completing: Intel IGD Direct Assignment with VFIO

2014-09-24 Thread Alex Williamson
On Wed, 2014-09-24 at 22:31 +0200, Paolo Bonzini wrote: > Il 24/09/2014 21:47, Alex Williamson ha scritto: > > So the opregion is mapped by a config write on the IGD device itself and > > the other 3 regions, that we know about so far, are mapped via writes to > > the host bridge. > > AFAIU the op

Re: [Intel-gfx] [IGDVFIO] [PATCH 3/8] RFC and help completing: Intel IGD Direct Assignment with VFIO

2014-09-24 Thread Paolo Bonzini
Il 24/09/2014 21:47, Alex Williamson ha scritto: > So the opregion is mapped by a config write on the IGD device itself and > the other 3 regions, that we know about so far, are mapped via writes to > the host bridge. AFAIU the opregion is mapped by the (host) BIOS, that writes the address to a we

Re: [Intel-gfx] [PATCH] drm: Drop grab fpriv->fbs_lock in drm_fb_release

2014-09-24 Thread Paulo Zanoni
2014-09-24 16:55 GMT-03:00 Daniel Vetter : > Paulo Zanoni reported a lockdep splat with a locking inversion between > fpriv->fbs_lock and the modeset locks. This issue was introduced in > > commit f2b50c1161590c3bcdbf3455fe4c575f1c1bd293 > Author: Daniel Vetter > Date: Fri Sep 12 17:07:32 2014 +

Re: [Intel-gfx] [PATCH 00/27] add pm_runtime_last_busy_and_autosuspend() helper

2014-09-24 Thread Rafael J. Wysocki
On Wednesday, September 24, 2014 03:15:58 PM Felipe Balbi wrote: > On Wed, Sep 24, 2014 at 10:28:07PM +0200, Rafael J. Wysocki wrote: > > On Wednesday, September 24, 2014 09:44:50 PM Vinod Koul wrote: > > > This patch series adds a simple macro > > > pm_runtime_last_busy_and_autosuspend() > > > wh

Re: [Intel-gfx] [PATCH 00/27] add pm_runtime_last_busy_and_autosuspend() helper

2014-09-24 Thread Rafael J. Wysocki
On Wednesday, September 24, 2014 09:44:50 PM Vinod Koul wrote: > This patch series adds a simple macro pm_runtime_last_busy_and_autosuspend() > which invokes pm_runtime_mark_last_busy() and pm_runtime_put_autosuspend() > sequentially. Then we do a tree wide update of current patterns which are > pr

[Intel-gfx] [PATCH] drm: Drop grab fpriv->fbs_lock in drm_fb_release

2014-09-24 Thread Daniel Vetter
Paulo Zanoni reported a lockdep splat with a locking inversion between fpriv->fbs_lock and the modeset locks. This issue was introduced in commit f2b50c1161590c3bcdbf3455fe4c575f1c1bd293 Author: Daniel Vetter Date: Fri Sep 12 17:07:32 2014 +0200 drm: Fixup locking for universal cursor plan

Re: [Intel-gfx] [IGDVFIO] [PATCH 4/8] RFC and help completing: Intel IGD Direct Assignment with VFIO

2014-09-24 Thread Alex Williamson
On Wed, 2014-09-24 at 14:20 +0100, Andrew Barnes wrote: > hw/pci/pci.c > > this patch adds: > * read / write host PCI config space > > patch > - > > diff --git a/hw/pci/pci.c b/hw/pci/pci.c > index 42995e6..041f6f1 100644 > --- a/hw/pci/pci.c > +++ b/hw/pci/pci.c > @@ -2,6 +2

Re: [Intel-gfx] [IGDVFIO] [PATCH 3/8] RFC and help completing: Intel IGD Direct Assignment with VFIO

2014-09-24 Thread Alex Williamson
On Wed, 2014-09-24 at 14:20 +0100, Andrew Barnes wrote: > hw/pci-host/q35.c > > this patch adds: > * redirect some PCI config reads/writes to host > * memory map BSDM, BGSM, TSEG > > patch > - > > diff --git a/hw/pci-host/q35.c b/hw/pci-host/q35.c > index a0a3068..05348ac 100

Re: [Intel-gfx] [PULL] topic/core-stuff

2014-09-24 Thread Daniel Vetter
On Wed, Sep 24, 2014 at 12:24:53PM +0200, Daniel Vetter wrote: > Hi Dave, > > Just noticed that you've picked up the header rework stuff already, so > I've rebased that out again. Otherwise just two stragglers from the vblank > rework and the universal cursor planes locking fix. Plus sprinkling >

Re: [Intel-gfx] [PATCH] drm/i915: Enable pixel replicated modes on BDW and HSW.

2014-09-24 Thread Daniel Vetter
On Wed, Sep 24, 2014 at 08:44:42AM -0700, Clint Taylor wrote: > On 09/24/2014 01:51 AM, Daniel Vetter wrote: > >On Tue, Sep 23, 2014 at 11:06:56AM -0700, clinton.a.tay...@intel.com wrote: > >>From: Clint Taylor > >> > >>Haswell and later silicon has added a new pixel replication register > >>to th

Re: [Intel-gfx] [IGDVFIO] [PATCH 2/8] RFC and help completing: Intel IGD Direct Assignment with VFIO

2014-09-24 Thread Alex Williamson
On Wed, 2014-09-24 at 14:20 +0100, Andrew Barnes wrote: > hw/misc/vfio.c > > this patch adds: > * memory map intel opregion > * mirroring of bdsm to guest's device 0 not hosts. > > patch > - > > diff --git a/hw/misc/vfio.c b/hw/misc/vfio.c > index e88b610..54e549b 100644 > --

Re: [Intel-gfx] [PATCH 1/3] tools/null_state_gen: add macro to emit commands with null state

2014-09-24 Thread Volkin, Bradley D
On Wed, Sep 24, 2014 at 05:50:30AM -0700, Mika Kuoppala wrote: > In null/golden context there are multiple state commands where > the actual state is always zero. For more compact batch representation > add a macro which just emits command and the rest of the state as zero. > > Signed-off-by: Mika

Re: [Intel-gfx] [PATCH] drm/i915: Make sure PSR is ready for been re-enabled.

2014-09-24 Thread Daniel Vetter
On Wed, Sep 24, 2014 at 12:40:20PM -0300, Paulo Zanoni wrote: > 2014-09-17 14:23 GMT-03:00 Rodrigo Vivi : > > Let's make sure PSR is propperly disabled before to re-enabled it. > > > > According to Spec, after disabled PSR CTL, the Idle state might occur > > up to 24ms, that is one full frame time

Re: [Intel-gfx] [IGDVFIO] [PATCH 1/8] RFC and help completing: Intel IGD Direct Assignment with VFIO

2014-09-24 Thread Alex Williamson
On Wed, 2014-09-24 at 14:20 +0100, Andrew Barnes wrote: > hw/isa/lpc_ich9.c > > this patch adds: > * debug output, if enabled > * enforces correct intel config, if enabled. (unsure if this is needed) > * redirects some PCI Config to host > * uses hosts LPC device id > > patch > --

Re: [Intel-gfx] [PATCH 04/27] drm/i915: use pm_runtime_last_busy_and_autosuspend helper

2014-09-24 Thread Daniel Vetter
On Wed, Sep 24, 2014 at 09:44:54PM +0530, Vinod Koul wrote: > Use the new pm_runtime_last_busy_and_autosuspend helper instead of open > coding the same code > > Signed-off-by: Vinod Koul Ack to merge through whatever tree is appropriate for this. Or tell me when I should pick this up for drm-int

Re: [Intel-gfx] ACPI/i915: Cannot configure display brightness on Dell Latitude E6440

2014-09-24 Thread Pali Rohár
On Wednesday 24 September 2014 16:34:21 Hans de Goede wrote: > Hi, > > On 09/24/2014 02:53 PM, Pali Rohár wrote: > > On Wednesday 24 September 2014 14:04:36 Hans de Goede wrote: > >> Hi, > >> > >> On 09/24/2014 11:14 AM, Pali Rohár wrote: > >>> On Wednesday 24 September 2014 10:59:41 Pali Rohár w

Re: [Intel-gfx] [PULL] topic/core-stuff

2014-09-24 Thread Daniel Vetter
On Wed, Sep 24, 2014 at 6:24 PM, Ilia Mirkin wrote: > On Wed, Sep 24, 2014 at 6:24 AM, Daniel Vetter wrote: >> Hi Dave, >> >> Just noticed that you've picked up the header rework stuff already, so >> I've rebased that out again. Otherwise just two stragglers from the vblank >> rework and the univ

Re: [Intel-gfx] [PATCH] drm/i915/bdw: Cleanup pre prod workarounds

2014-09-24 Thread Rodrigo Vivi
Oh just noticed we duplicated effort on this.. [5/5] drm/i915/bdw: Remove BDW preproduction W/As until C stepping. After this land -nightly I rebase my series on top. On Wed, Sep 24, 2014 at 1:41 AM, Daniel Vetter wrote: > On Tue, Sep 23, 2014 at 02:48:47PM +0300, Jani Nikula wrote: > > On Tue,

Re: [Intel-gfx] [PATCH] drm/i915: intel_backlight scale() math WA v2

2014-09-24 Thread Eoff, Ullysses A
> -Original Message- > From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf Of > Jani Nikula > Sent: Wednesday, September 24, 2014 10:08 AM > To: Hans de Goede; Joe Konno; intel-gfx@lists.freedesktop.org > Subject: Re: [Intel-gfx] [PATCH] drm/i915: intel_backlight sca

[Intel-gfx] [PATCH v3 09/11] drm/i915: create a prepare phase for sprite plane updates

2014-09-24 Thread Gustavo Padovan
From: Gustavo Padovan take out pin_fb code so the commit phase can't fail anymore. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/i915/intel_sprite.c | 63 +++-- 1 file changed, 40 insertions(+), 23 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_sprit

[Intel-gfx] [PATCH v3 08/11] drm/i915: create a prepare step for primary planes updates

2014-09-24 Thread Gustavo Padovan
From: Gustavo Padovan Take out the pin_fb code so commit phase can't fail anymore. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/i915/intel_display.c | 35 ++- 1 file changed, 26 insertions(+), 9 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_display

[Intel-gfx] [PATCH v3 11/11] drm/i915: remove intel_pipe_set_base()

2014-09-24 Thread Gustavo Padovan
From: Gustavo Padovan After some refactor intel_primary_plane_setplane() does the same as intel_pipe_set_base() so we can get rid of it and replace the calls with intel_primary_plane_setplane(). v2: take Ville's comments: - get the right arguments for update_plane() - use drm_crt

[Intel-gfx] [PATCH v3 02/11] drm/i915: remove leftover from pre-universal planes days

2014-09-24 Thread Gustavo Padovan
From: Gustavo Padovan Now that universal planes are in place we don't need this plane unref on failures. Suggested-by: Ville Syrjälä Signed-off-by: Gustavo Padovan Reviewed-by: Ville Syrjälä --- drivers/gpu/drm/i915/intel_display.c | 12 +--- 1 file changed, 1 insertion(+), 11 deleti

[Intel-gfx] [PATCH v3 01/11] drm/i915: Merge of visible and !visible paths for primary planes

2014-09-24 Thread Gustavo Padovan
From: Gustavo Padovan Fold intel_pipe_set_base() in the update primary plane path merging pieces of code that are common to both paths. Basically the the pin/unpin procedures are the same for both paths and some checks can also be shared (some of the were moved to the check() stage) v2: take Vi

[Intel-gfx] [PATCH v3 06/11] drm/i915: split intel_crtc_page_flip() into check() and commit()

2014-09-24 Thread Gustavo Padovan
From: Daniel Stone Start the work of splitting the intel_crtc_page_flip() for later use by the atomic modesetting API. Signed-off-by: Daniel Stone Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/i915/intel_display.c | 51 ++-- 1 file changed, 37 insertions(+

[Intel-gfx] [PATCH v3 10/11] drm/i915: use intel_fb_obj() macros to assign gem objects

2014-09-24 Thread Gustavo Padovan
From: Gustavo Padovan Use the macros makes the code cleaner and it also checks for a NULL fb. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/i915/intel_sprite.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_sprite.c b/drivers/gpu/dr

[Intel-gfx] [PATCH v3 07/11] drm: add helper to get crtc timings

2014-09-24 Thread Gustavo Padovan
From: Gustavo Padovan We need to get hdisplay and vdisplay in a few places so create a helper to make our job easier. Suggested-by: Ville Syrjälä Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/drm_crtc.c | 20 +--- drivers/gpu/drm/i915/intel_display.c | 6 +++---

[Intel-gfx] [PATCH v3 03/11] drm/i915: Fix not checking cursor and object sizes

2014-09-24 Thread Gustavo Padovan
From: Gustavo Padovan Even if the fb is the same we should still check if the sizes are valid to be set. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/i915/intel_display.c | 61 1 file changed, 41 insertions(+), 20 deletions(-) diff --git a/drivers/gp

[Intel-gfx] [PATCH v3 05/11] drm/i915: remove intel_crtc_cursor_set_obj()

2014-09-24 Thread Gustavo Padovan
From: Gustavo Padovan Merge it into the plane update_plane() callback and make other users use the update_plane() functions instead. The fb != crtc->cursor->fb was already inside intel_crtc_cursor_set_obj() so we fold intel_crtc_cursor_set_obj() inside intel_commit_cursor_plane() and merge both

[Intel-gfx] [PATCH v3 04/11] drm/i915: move check of intel_crtc_cursor_set_obj() out

2014-09-24 Thread Gustavo Padovan
From: Gustavo Padovan Move check inside intel_crtc_cursor_set_obj() to intel_check_cursor_plane(), we only use it there so move them out to make the merge of intel_crtc_cursor_set_obj() into intel_check_cursor_plane() easier. This is another step toward the atomic modesetting support and unifica

Re: [Intel-gfx] [PATCH] drm/i915: intel_backlight scale() math WA v2

2014-09-24 Thread Jani Nikula
On Wed, 24 Sep 2014, Hans de Goede wrote: > Hi, > > On 09/24/2014 05:54 PM, Joe Konno wrote: >> From: Joe Konno >> >> Improper truncated integer division in the scale() function causes >> actual_brightness != brightness. This (partial) work-around should be >> sufficient for a majority of use-ca

[Intel-gfx] [PULL] drm-intel-fixes

2014-09-24 Thread Jani Nikula
Hi Dave, a couple of small fixes for 3.17 still. BR, Jani. The following changes since commit 0f33be009b89d2268e94194dc4fd01a7851b6d51: Linux 3.17-rc6 (2014-09-21 15:43:02 -0700) are available in the git repository at: git://anongit.freedesktop.org/drm-intel tags/drm-intel-fixes-2014-09-

[Intel-gfx] [PATCH 04/27] drm/i915: use pm_runtime_last_busy_and_autosuspend helper

2014-09-24 Thread Vinod Koul
Use the new pm_runtime_last_busy_and_autosuspend helper instead of open coding the same code Signed-off-by: Vinod Koul --- drivers/gpu/drm/i915/intel_pm.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c

[Intel-gfx] [PATCH 00/27] add pm_runtime_last_busy_and_autosuspend() helper

2014-09-24 Thread Vinod Koul
This patch series adds a simple macro pm_runtime_last_busy_and_autosuspend() which invokes pm_runtime_mark_last_busy() and pm_runtime_put_autosuspend() sequentially. Then we do a tree wide update of current patterns which are present. As evident from log below this pattern is frequent in the kernel

Re: [Intel-gfx] [Qemu-devel] [IGDVFIO] [PATCH 1/8] RFC and help completing: Intel IGD Direct Assignment with VFIO

2014-09-24 Thread Eric Blake
On 09/24/2014 07:20 AM, Andrew Barnes wrote: > hw/isa/lpc_ich9.c > > this patch adds: > * debug output, if enabled > * enforces correct intel config, if enabled. (unsure if this is needed) > * redirects some PCI Config to host > * uses hosts LPC device id > > patch > - > Sty

Re: [Intel-gfx] [PULL] topic/core-stuff

2014-09-24 Thread Ilia Mirkin
On Wed, Sep 24, 2014 at 6:24 AM, Daniel Vetter wrote: > Hi Dave, > > Just noticed that you've picked up the header rework stuff already, so > I've rebased that out again. Otherwise just two stragglers from the vblank > rework and the universal cursor planes locking fix. Plus sprinkling > container

Re: [Intel-gfx] [PATCH] drm/i915: intel_backlight scale() math WA v2

2014-09-24 Thread Hans de Goede
Hi, On 09/24/2014 05:54 PM, Joe Konno wrote: > From: Joe Konno > > Improper truncated integer division in the scale() function causes > actual_brightness != brightness. This (partial) work-around should be > sufficient for a majority of use-cases, but it is by no means a complete > solution. >

Re: [Intel-gfx] [PATCH] drm/i915: Enable pixel replicated modes on BDW and HSW.

2014-09-24 Thread Clint Taylor
On 09/24/2014 01:51 AM, Daniel Vetter wrote: On Tue, Sep 23, 2014 at 11:06:56AM -0700, clinton.a.tay...@intel.com wrote: From: Clint Taylor Haswell and later silicon has added a new pixel replication register to the pipe timings for each transcoder. Now in addition to the DPLL_A_MD register fo

[Intel-gfx] [PATCH] drm/i915: intel_backlight scale() math WA v2

2014-09-24 Thread Joe Konno
From: Joe Konno Improper truncated integer division in the scale() function causes actual_brightness != brightness. This (partial) work-around should be sufficient for a majority of use-cases, but it is by no means a complete solution. TODO: Determine how best to scale "user" values to "hw" valu

Re: [Intel-gfx] [PATCH] drm/i915: intel_backlight scale() math WA

2014-09-24 Thread Joe Konno
Many thanks, Chris. I'll incorporate your feedback into v2. On 09/23/2014 11:04 PM, Chris Wilson wrote: > On Tue, Sep 23, 2014 at 03:50:25PM -0700, Joe Konno wrote: >> From: Joe Konno >> >> Improper integer division-- truncated rather than rounded-- in the > That's confusing. It reads that you ar

Re: [Intel-gfx] [PATCH] drm/i915: Make sure PSR is ready for been re-enabled.

2014-09-24 Thread Paulo Zanoni
2014-09-17 14:23 GMT-03:00 Rodrigo Vivi : > Let's make sure PSR is propperly disabled before to re-enabled it. > > According to Spec, after disabled PSR CTL, the Idle state might occur > up to 24ms, that is one full frame time (1/refresh rate), > plus SRD exit training time (max of 6ms), > plus SRD

Re: [Intel-gfx] ACPI/i915: Cannot configure display brightness on Dell Latitude E6440

2014-09-24 Thread Pali Rohár
On Wednesday 24 September 2014 14:04:36 Hans de Goede wrote: > Hi, > > On 09/24/2014 11:14 AM, Pali Rohár wrote: > > On Wednesday 24 September 2014 10:59:41 Pali Rohár wrote: > >> On Wednesday 24 September 2014 10:19:38 Hans de Goede wrote: > >>> Hi, > >>> > >>> On 09/23/2014 10:44 PM, Pali Rohár

[Intel-gfx] ACPI/i915: Cannot configure display brightness on Dell Latitude E6440

2014-09-24 Thread Pali Rohár
Hello, after big changes in acpi video/i915 code I cannot change display brightness on my Dell Latitude E6440 with kernel 3.17-rc6. With kernel 3.13 everything worked fine. More information about this problem: For configuring brightness on Dell laptops there are 4 ways: 1) via acpi video drive

Re: [Intel-gfx] ACPI/i915: Cannot configure display brightness on Dell Latitude E6440

2014-09-24 Thread Pali Rohár
On Wednesday 24 September 2014 10:59:41 Pali Rohár wrote: > On Wednesday 24 September 2014 10:19:38 Hans de Goede wrote: > > Hi, > > > > On 09/23/2014 10:44 PM, Pali Rohár wrote: > > > On Tuesday 23 September 2014 22:31:31 you wrote: > > >> Hi, > > >> > > >> On 09/23/2014 10:06 PM, Pali Rohár wro

Re: [Intel-gfx] ACPI/i915: Cannot configure display brightness on Dell Latitude E6440

2014-09-24 Thread Pali Rohár
On Wednesday 24 September 2014 10:19:38 Hans de Goede wrote: > Hi, > > On 09/23/2014 10:44 PM, Pali Rohár wrote: > > On Tuesday 23 September 2014 22:31:31 you wrote: > >> Hi, > >> > >> On 09/23/2014 10:06 PM, Pali Rohár wrote: > >>> Hello, > >>> > >>> after big changes in acpi video/i915 code I

Re: [Intel-gfx] ACPI/i915: Cannot configure display brightness on Dell Latitude E6440

2014-09-24 Thread Pali Rohár
On Tuesday 23 September 2014 22:31:31 you wrote: > Hi, > > On 09/23/2014 10:06 PM, Pali Rohár wrote: > > Hello, > > > > after big changes in acpi video/i915 code I cannot change > > display brightness on my Dell Latitude E6440 with kernel > > 3.17-rc6. With kernel 3.13 everything worked fine. > >

Re: [Intel-gfx] [PATCH v2 03/10] drm/i915: move checks of intel_crtc_cursor_set_obj() out

2014-09-24 Thread Gustavo Padovan
2014-09-23 Ville Syrjälä : > On Mon, Sep 22, 2014 at 07:23:10PM -0300, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > Move checks inside intel_crtc_cursor_set_obj() to > > intel_check_cursor_plane(), we only use they there so move them out to > > make the merge of intel_crtc_cursor_set

Re: [Intel-gfx] [RFC] drm/i915: Add sysfs entry for reading GPU Timestamp

2014-09-24 Thread Chris Wilson
On Wed, Sep 24, 2014 at 03:41:08PM +0200, Michał Winiarski wrote: > Reading timestamp register using I915_READ64 returns incorrect value. > Unfortunately, that's how I915_REG_READ ioctl is handling it on x86_64, > resulting in different counter size (we can only get 32 usable bits on > x86_64 vs 36

Re: [Intel-gfx] [Qemu-devel] [IGDVFIO] [PATCH 1/8] RFC and help completing: Intel IGD Direct Assignment with VFIO

2014-09-24 Thread Eric Blake
[meta-comment] On 09/24/2014 07:20 AM, Andrew Barnes wrote: > hw/isa/lpc_ich9.c > Your patches came through unthreaded (mail 1 through 8 are all lacking In-Reply-To the message id of 0/8, ). This makes it harder to review. You may want to double-check your git send-email settings, or look for a

Re: [Intel-gfx] ACPI/i915: Cannot configure display brightness on Dell Latitude E6440

2014-09-24 Thread Hans de Goede
Hi, On 09/24/2014 02:53 PM, Pali Rohár wrote: > On Wednesday 24 September 2014 14:04:36 Hans de Goede wrote: >> Hi, >> >> On 09/24/2014 11:14 AM, Pali Rohár wrote: >>> On Wednesday 24 September 2014 10:59:41 Pali Rohár wrote: On Wednesday 24 September 2014 10:19:38 Hans de Goede wrote: >

Re: [Intel-gfx] [PATCH 52/89] drm/i915/gen9: Disable WM if corresponding latency is 0

2014-09-24 Thread Damien Lespiau
On Fri, Sep 19, 2014 at 01:05:02PM +0300, Ville Syrjälä wrote: > > If we're going to be paranoid I think we should disable all higher WM > levels whose latency is lower than any of the lower levels. And I > think we'll want something like dev_priv->wm.max_wm_level instead of > relying on the zero

Re: [Intel-gfx] [PATCH 3/4] drm/i915: Avoid re-configure panel on every PSR re-enable.

2014-09-24 Thread Paulo Zanoni
2014-09-16 20:19 GMT-03:00 Rodrigo Vivi : > The panel has to be reconfigured only when it really loose the power. > The traditional enable/disable sequence already take care of this so we can > minimize the time spend on every re-enable. Reviewed-by: Paulo Zanoni > > Signed-off-by: Rodrigo Vivi

[Intel-gfx] [RFC] drm/i915: Add sysfs entry for reading GPU Timestamp

2014-09-24 Thread Michał Winiarski
Reading timestamp register using I915_READ64 returns incorrect value. Unfortunately, that's how I915_REG_READ ioctl is handling it on x86_64, resulting in different counter size (we can only get 32 usable bits on x86_64 vs 36 bits on x86). Propose new sysfs interface for accessing full 36 bits of t

Re: [Intel-gfx] [PATCH 56/89] drm/i915: Gen9 shadowed registers

2014-09-24 Thread Mika Kuoppala
Damien Lespiau writes: > From: Zhe Wang > > For MMIO registers which are shadowed, force wake is not needed to > write to these registers. > > v2: Rebase on top of nightly (Damien) > > Signed-off-by: Zhe Wang > Signed-off-by: Damien Lespiau > --- > drivers/gpu/drm/i915/intel_uncore.c | 26 +++

Re: [Intel-gfx] [PATCH 2/8] drm/i915: Adds graphic address space ballooning logic

2014-09-24 Thread Daniel Vetter
On Wed, Sep 24, 2014 at 08:35:50PM +0800, Zhang, Yu wrote: > Hi Daniel & Chris, > > Thank you very much for your comments, And sorry for my late reply.:) I > was focusing on other tasks previously. > See my questions below: > > On 9/23/2014 7:25 PM, Daniel Vetter wrote: > >On Tue, Sep 23, 201

Re: [Intel-gfx] [PATCH 2/8] drm/i915: Adds graphic address space ballooning logic

2014-09-24 Thread Chris Wilson
On Wed, Sep 24, 2014 at 08:35:50PM +0800, Zhang, Yu wrote: > Hi Daniel & Chris, > > Thank you very much for your comments, And sorry for my late > reply.:) I was focusing on other tasks previously. > See my questions below: > > On 9/23/2014 7:25 PM, Daniel Vetter wrote: > >On Tue, Sep 23, 201

[Intel-gfx] [IGDVFIO] [PATCH 6/8] RFC and help completing: Intel IGD Direct Assignment with VFIO

2014-09-24 Thread Andrew Barnes
include/hw/pci/pci.h patch - diff --git a/include/hw/pci/pci.h b/include/hw/pci/pci.h index c352c7b..d3bc224 100644 --- a/include/hw/pci/pci.h +++ b/include/hw/pci/pci.h @@ -338,6 +338,9 @@ typedef PCIINTxRoute (*pci_route_irq_fn)(void *opaque, int pin); #define PCI_BUS(obj)

[Intel-gfx] [IGDVFIO] [PATCH 8/8] RFC and help completing: Intel IGD Direct Assignment with VFIO

2014-09-24 Thread Andrew Barnes
Seabios patch This patch covers all the changes to seabios. The intentions are to make seabios behave like the hosts BIOS. and like the hosts BIOS, configure the Q35 chipset more thoroughly. This patch configures TOM, TOUUD, TOLUD, BDSM, BGSM, TSEG, IGD-OPREGION. by writing new base addresses, V

[Intel-gfx] [IGDVFIO] [PATCH 4/8] RFC and help completing: Intel IGD Direct Assignment with VFIO

2014-09-24 Thread Andrew Barnes
hw/pci/pci.c this patch adds: * read / write host PCI config space patch - diff --git a/hw/pci/pci.c b/hw/pci/pci.c index 42995e6..041f6f1 100644 --- a/hw/pci/pci.c +++ b/hw/pci/pci.c @@ -2,6 +2,8 @@ * QEMU PCI bus manager * * Copyright (c) 2004 Fabrice Bellard + *2

[Intel-gfx] [IGDVFIO] [PATCH 7/8] RFC and help completing: Intel IGD Direct Assignment with VFIO

2014-09-24 Thread Andrew Barnes
include/hw/pci/pci_ids.h A very last minute realisation is that this is not needed. Its purpose is to make sure anything else that references LPC device ID is given the hacked-in host's ID. but in verifying the patch validity it appears this not referenced anywhere, other than where its taken care

[Intel-gfx] [IGDVFIO] [PATCH 5/8] RFC and help completing: Intel IGD Direct Assignment with VFIO

2014-09-24 Thread Andrew Barnes
include/hw/pci-host/q35.h this patch adds: * #defines for q35 pci config. not all are used, added for completeness patch - diff --git a/include/hw/pci-host/q35.h b/include/hw/pci-host/q35.h index d9ee978..abb795e 100644 --- a/include/hw/pci-host/q35.h +++ b/include/hw/pci-hos

[Intel-gfx] [IGDVFIO] [PATCH 3/8] RFC and help completing: Intel IGD Direct Assignment with VFIO

2014-09-24 Thread Andrew Barnes
hw/pci-host/q35.c this patch adds: * redirect some PCI config reads/writes to host * memory map BSDM, BGSM, TSEG patch - diff --git a/hw/pci-host/q35.c b/hw/pci-host/q35.c index a0a3068..05348ac 100644 --- a/hw/pci-host/q35.c +++ b/hw/pci-host/q35.c @@ -6,6 +6,7 @@ *

[Intel-gfx] [IGDVFIO] [PATCH 2/8] RFC and help completing: Intel IGD Direct Assignment with VFIO

2014-09-24 Thread Andrew Barnes
hw/misc/vfio.c this patch adds: * memory map intel opregion * mirroring of bdsm to guest's device 0 not hosts. patch - diff --git a/hw/misc/vfio.c b/hw/misc/vfio.c index e88b610..54e549b 100644 --- a/hw/misc/vfio.c +++ b/hw/misc/vfio.c @@ -5,6 +5,7 @@ * * Authors: * Al

[Intel-gfx] [IGDVFIO] [PATCH 1/8] RFC and help completing: Intel IGD Direct Assignment with VFIO

2014-09-24 Thread Andrew Barnes
hw/isa/lpc_ich9.c this patch adds: * debug output, if enabled * enforces correct intel config, if enabled. (unsure if this is needed) * redirects some PCI Config to host * uses hosts LPC device id patch - diff --git a/hw/isa/lpc_ich9.c b/hw/isa/lpc_ich9.c index b846d81..e6a7f

[Intel-gfx] [IGDVFIO] [PATCH 0/8] RFC and help completing: Intel IGD Direct Assignment with VFIO

2014-09-24 Thread Andrew Barnes
https://github.com/UmbraMalison/qemu-igdvfio Hi all, I am publishing my work on IGD Direct Assignment in a semi-working state as I need some help from the community to finish these. It may also be a good time to take in comments from the community. There has been an increased interest in IGD DA

[Intel-gfx] [PATCH 2/3] tools/null_state_gen: gen8 golden state

2014-09-24 Thread Mika Kuoppala
Previously we didn't have a clear understanding what is necessary for a pipeline state to be properly initialized. So we had to improvise and use a stripped out render copy. Now we have a more clear understanding so switch out render copy based frankenstate to state we can call golden state. ---

[Intel-gfx] [PATCH 1/3] tools/null_state_gen: add macro to emit commands with null state

2014-09-24 Thread Mika Kuoppala
In null/golden context there are multiple state commands where the actual state is always zero. For more compact batch representation add a macro which just emits command and the rest of the state as zero. Signed-off-by: Mika Kuoppala --- tools/null_state_gen/intel_batchbuffer.c| 12

[Intel-gfx] [PATCH 3/3] tools/null_state_gen: Add GEN9 golden context batch buffer creation

2014-09-24 Thread Mika Kuoppala
From: Armin Reese Modifications to 'null_state_gen' so it can generate GEN9 golden context batch buffer source for SKL. v2: - rebased on top of gen8 changes (Mika) - fixed state base address command size (Mika) - base address size macro as pages (Mika) Signed-off-by: Armin Reese Signed

Re: [Intel-gfx] [PATCH 2/8] drm/i915: Adds graphic address space ballooning logic

2014-09-24 Thread Zhang, Yu
Hi Daniel & Chris, Thank you very much for your comments, And sorry for my late reply.:) I was focusing on other tasks previously. See my questions below: On 9/23/2014 7:25 PM, Daniel Vetter wrote: On Tue, Sep 23, 2014 at 10:19:02AM +0100, Chris Wilson wrote: On Tue, Sep 23, 2014 at 10:2

Re: [Intel-gfx] ACPI/i915: Cannot configure display brightness on Dell Latitude E6440

2014-09-24 Thread Hans de Goede
Hi, On 09/24/2014 11:14 AM, Pali Rohár wrote: > On Wednesday 24 September 2014 10:59:41 Pali Rohár wrote: >> On Wednesday 24 September 2014 10:19:38 Hans de Goede wrote: >>> Hi, >>> >>> On 09/23/2014 10:44 PM, Pali Rohár wrote: On Tuesday 23 September 2014 22:31:31 you wrote: > Hi, >

[Intel-gfx] [PATCH v2 2/2] drm/i915: Initialize chv workarounds in logical ring mode too

2014-09-24 Thread Michel Thierry
Also enable the ring->init_context() hook for chv in execlist submission mode. For: VIZ-4092 Signed-off-by: Michel Thierry --- drivers/gpu/drm/i915/intel_lrc.c| 37 ++- drivers/gpu/drm/i915/intel_ringbuffer.c | 39 +++-- drivers/gpu

[Intel-gfx] [PATCH v2 1/2] drm/i915: Initialize bdw workarounds in logical ring mode too

2014-09-24 Thread Michel Thierry
Following the legacy ring submission example, update the ring->init_context() hook to support the execlist submission mode. Workarounds are defined in bdw_emit_workarounds(), but the emit now depends on the ring submission mode. v2: Updated after "Cleanup pre prod workarounds" For: VIZ-4092 Sign

Re: [Intel-gfx] [PATCH 75/89] drm/i915/skl: fetch, enable/disable pfit as needed

2014-09-24 Thread Damien Lespiau
Hi Jesse, Mind looking at those review comments? -- Damien On Tue, Sep 23, 2014 at 05:50:29PM -0300, Paulo Zanoni wrote: > 2014-09-04 8:27 GMT-03:00 Damien Lespiau : > > From: Jesse Barnes > > > > This moved around on SKL, so we need to make sure we read/write the > > correct regs. > > > > Sig

[Intel-gfx] [PULL] topic/core-stuff

2014-09-24 Thread Daniel Vetter
Hi Dave, Just noticed that you've picked up the header rework stuff already, so I've rebased that out again. Otherwise just two stragglers from the vblank rework and the universal cursor planes locking fix. Plus sprinkling container_of all over fbdev emulation from Fabian. Aside: I only have just

Re: [Intel-gfx] [PATCH 6/9] drm/i915: Dsipll clk to be enabled for DSI1 in case of dual link configuration

2014-09-24 Thread Jani Nikula
On Wed, 24 Sep 2014, Gaurav K Singh wrote: > Signed-off-by: Gaurav K Singh > Signed-off-by: Shobhit Kumar > --- > drivers/gpu/drm/i915/intel_dsi_pll.c | 14 -- > 1 file changed, 8 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_dsi_pll.c > b/drivers/gpu/

Re: [Intel-gfx] [PATCH 5/9] drm/i915: SHUTDOWN & Turn ON packets to be sent for both MIPI Ports in case of dual link Configuration

2014-09-24 Thread Jani Nikula
On Wed, 24 Sep 2014, Gaurav K Singh wrote: > Signed-off-by: Gaurav K Singh > Signed-off-by: Shobhit Kumar > --- > drivers/gpu/drm/i915/intel_dsi_cmd.c | 35 > ++ > 1 file changed, 23 insertions(+), 12 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_ds

Re: [Intel-gfx] [PATCH 3/9] drm/i915: MIPI Port Ctrl related changes for dual link configuration

2014-09-24 Thread Jani Nikula
On Wed, 24 Sep 2014, Gaurav K Singh wrote: > Signed-off-by: Gaurav K Singh > Signed-off-by: Shobhit Kumar > --- > drivers/gpu/drm/i915/i915_reg.h|1 + > drivers/gpu/drm/i915/intel_dsi.c | 53 > ++-- > drivers/gpu/drm/i915/intel_dsi.h

Re: [Intel-gfx] [PATCH 4/9] drm/i915: Pixel Clock and pixel overlap related changes for dual link Configuration

2014-09-24 Thread Jani Nikula
On Wed, 24 Sep 2014, Gaurav K Singh wrote: > Signed-off-by: Gaurav K Singh > Signed-off-by: Shobhit Kumar > --- > drivers/gpu/drm/i915/i915_reg.h|4 > drivers/gpu/drm/i915/intel_bios.h |3 ++- > drivers/gpu/drm/i915/intel_dsi.c |8 > driv

Re: [Intel-gfx] [PATCH 0/9] BYT DSI Dual Link Support

2014-09-24 Thread Daniel Vetter
On Wed, Sep 24, 2014 at 02:16:49PM +0530, Gaurav K Singh wrote: > Hi, > These set of patches build on top of the existing DSI Video mode support to > enable dual link MIPI panels with high resolutions. These patches have been > tested on a 25x16 panel and works well. Except for the first patch (w

Re: [Intel-gfx] [PATCH] drm/i915: Enable pixel replicated modes on BDW and HSW.

2014-09-24 Thread Daniel Vetter
On Tue, Sep 23, 2014 at 11:06:56AM -0700, clinton.a.tay...@intel.com wrote: > From: Clint Taylor > > Haswell and later silicon has added a new pixel replication register > to the pipe timings for each transcoder. Now in addition to the > DPLL_A_MD register for the pixel clock double, we also need

[Intel-gfx] [PATCH 5/9] drm/i915: SHUTDOWN & Turn ON packets to be sent for both MIPI Ports in case of dual link Configuration

2014-09-24 Thread Gaurav K Singh
Signed-off-by: Gaurav K Singh Signed-off-by: Shobhit Kumar --- drivers/gpu/drm/i915/intel_dsi_cmd.c | 35 ++ 1 file changed, 23 insertions(+), 12 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dsi_cmd.c b/drivers/gpu/drm/i915/intel_dsi_cmd.c index eb698b

  1   2   >