Re: [Intel-gfx] [PATCH 47/89] drm/i915/skl: SKL Watermark Computation

2014-09-23 Thread Satheeshakrishna M
On Tuesday 23 September 2014 04:06 AM, Damien Lespiau wrote: Hi Satheesh, On Wed, Sep 17, 2014 at 03:07:51PM +0300, Ville Syrjälä wrote: +static bool skl_compute_plane_wm(struct skl_pipe_wm_parameters *p, + struct intel_plane_wm_parameters *p_params, +

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

2014-09-23 Thread Chris Wilson
On Mon, Sep 22, 2014 at 07:23:08PM -0300, Gustavo Padovan wrote: From: Gustavo Padovan gustavo.pado...@collabora.co.uk 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

Re: [Intel-gfx] [PATCH v4] drm/i915/ppgtt: Load address space after mi_set_context

2014-09-23 Thread Zhi Wang
Hi Michel: I'm doing a task which require LRIs to load root pointer around RCS context switch.My BDW CPU is C stepping. And what I found is MI_SET_CONTEXT instruction will save/restore PDP0/1/2/3. It's weired that as part of EXECLIST context, the EXECLIST PPGTT base context also appear in

Re: [Intel-gfx] [PATCH] drm/i915: WaRsClearFWBitsAtReset - WA for blitter, render and media forcewake on BDW and GEN9 platforms.

2014-09-23 Thread Ville Syrjälä
On Mon, Sep 22, 2014 at 02:43:02PM -0400, Suketu Shah wrote: The newly loaded Gfx driver must first initialize the forcewake request register for render, media and blitter engines by clearing all forcewake bits (0x). This applies to BDW and GEN9 platforms. Already done

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

2014-09-23 Thread Ville Syrjälä
On Mon, Sep 22, 2014 at 07:23:08PM -0300, Gustavo Padovan wrote: From: Gustavo Padovan gustavo.pado...@collabora.co.uk 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

Re: [Intel-gfx] [PATCH v2 02/10] drm/i915: remove leftover from pre-universal planes days

2014-09-23 Thread Ville Syrjälä
On Mon, Sep 22, 2014 at 07:23:09PM -0300, Gustavo Padovan wrote: From: Gustavo Padovan gustavo.pado...@collabora.co.uk Now that universal planes are in place we don't need this plane unref on failures. Suggested-by: Ville Syrjälä ville.syrj...@linux.intel.com Signed-off-by: Gustavo

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

2014-09-23 Thread Ville Syrjälä
On Mon, Sep 22, 2014 at 07:23:10PM -0300, Gustavo Padovan wrote: From: Gustavo Padovan gustavo.pado...@collabora.co.uk 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_obj() into

Re: [Intel-gfx] [PATCH 1/4] drm/i915: Suppress redundant syncs with mmio page flips

2014-09-23 Thread Daniel Vetter
On Fri, Sep 19, 2014 at 08:58:58PM +0300, Ville Syrjälä wrote: On Wed, Jun 18, 2014 at 12:23:14PM +0100, Chris Wilson wrote: Since mmio-flips do not occur on the suggested ring, we are introducing an extra sync operation where none is required. Pass the current obj-ring, which is what mmio

Re: [Intel-gfx] [PATCH 45/89] drm/i915/skl: Definition of SKL WM param structs for pipe/plane

2014-09-23 Thread Daniel Vetter
On Mon, Sep 22, 2014 at 03:21:22PM +0100, Damien Lespiau wrote: On Mon, Sep 22, 2014 at 05:06:11PM +0300, Ville Syrjälä wrote: +struct skl_pipe_wm { + struct skl_wm_level wm[8]; + struct skl_wm_level trans_wm; + uint32_t linetime; +}; + struct intel_crtc { struct

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

2014-09-23 Thread Daniel Vetter
On Fri, Sep 19, 2014 at 08:49:06PM +0300, Ville Syrjälä wrote: On Fri, Sep 19, 2014 at 08:05:26PM +0300, Mika Kuoppala wrote: as these have been fixed in production hw and hurt performance if applied. v2: adjust requested ring space (Ville) Bugzilla:

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

2014-09-23 Thread Daniel Vetter
On Fri, Sep 19, 2014 at 09:00:00PM +0100, Chris Wilson wrote: On Fri, Sep 19, 2014 at 06:21:46PM +, Tian, Kevin wrote: From: Chris Wilson The implementation also looks backwards. To work correctly with the GTT allocator, you need to preallocate the reserved space such that it can

Re: [Intel-gfx] [PATCH 2/2] drm/i915: add SW tracking to FBC enabling

2014-09-23 Thread Daniel Vetter
On Fri, Sep 19, 2014 at 12:44:08PM -0700, Rodrigo Vivi wrote: Reviewed-by: Rodrigo Vivi rodrigo.v...@intel.com Both patches merged to dinq. But I think it would be good now to change fbc_status interface on debugfs to show the current bit state as well. Seconded. Also some locking for fbc

Re: [Intel-gfx] [PATCH] drm/i915: Don't leak command parser tables on suspend/resume

2014-09-23 Thread Daniel Vetter
On Mon, Sep 22, 2014 at 08:25:21AM -0700, bradley.d.vol...@intel.com wrote: From: Brad Volkin bradley.d.vol...@intel.com Ring init and cleanup are not balanced because we re-init the rings on resume without having cleaned them up on suspend. This leads to the driver leaking the parser's hash

Re: [Intel-gfx] [PATH] Correct GPU timestamp read

2014-09-23 Thread Daniel Vetter
On Mon, Sep 22, 2014 at 06:22:53PM +0200, Jacek Danecki wrote: Current implementation of reading GPU timestamp is broken. It returns lower 32 bits shifted by 32 bits ( instead of ). Below change is adding possibility to read hi part of that register

Re: [Intel-gfx] [PATCH] drm/i915: Add RCS General Purpose Registers to parser whitelist

2014-09-23 Thread Daniel Vetter
On Tue, Sep 23, 2014 at 12:41:50AM +0200, Michał Winiarski wrote: These registers are used as a temporary storage by MI_MATH command when performing ALU operations. Signed-off-by: Michał Winiarski michal.winiar...@intel.com Needs to come with corresponding userspace using this. Also we need

Re: [Intel-gfx] [RFC 1/6] drm/i915: Modifying structures related to DRRS

2014-09-23 Thread Daniel Vetter
On Fri, Sep 12, 2014 at 11:13:04PM +0530, Vandana Kannan wrote: Moving around and changing some data related to DRRS to support DRRS based on frontbuffer tracking in the following patches Signed-off-by: Vandana Kannan vandana.kan...@intel.com The commit message is a bit too sparse imo. You

Re: [Intel-gfx] [RFC 4/6] drm/i915: DRRS calls based on frontbuffer

2014-09-23 Thread Daniel Vetter
On Fri, Sep 12, 2014 at 11:13:07PM +0530, Vandana Kannan wrote: Calls to switch between high and low refresh rates based on calls made to fb_invalidate and fb_flush. Signed-off-by: Vandana Kannan vandana.kan...@intel.com A few comments below. -Daniel ---

Re: [Intel-gfx] [RFC 0/6] eDP DRRS based on frontbuffer tracking

2014-09-23 Thread Daniel Vetter
On Fri, Sep 12, 2014 at 11:13:03PM +0530, Vandana Kannan wrote: As part of implementing DRRS based on frontbuffer tracking, this patch series includes some modifications related to the data for DRRS, code to enable/ disable DRRS during init or uninit, calls to switch between high and low

Re: [Intel-gfx] [PATCH] drm/i915: Don't leak command parser tables on suspend/resume

2014-09-23 Thread Chris Wilson
On Tue, Sep 23, 2014 at 10:34:46AM +0200, Daniel Vetter wrote: On Mon, Sep 22, 2014 at 08:25:21AM -0700, bradley.d.vol...@intel.com wrote: From: Brad Volkin bradley.d.vol...@intel.com Ring init and cleanup are not balanced because we re-init the rings on resume without having cleaned

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

2014-09-23 Thread Chris Wilson
On Tue, Sep 23, 2014 at 10:26:26AM +0200, Daniel Vetter wrote: On Fri, Sep 19, 2014 at 09:00:00PM +0100, Chris Wilson wrote: On Fri, Sep 19, 2014 at 06:21:46PM +, Tian, Kevin wrote: From: Chris Wilson The implementation also looks backwards. To work correctly with the GTT

Re: [Intel-gfx] [PATCH] drm/i915: Add RCS General Purpose Registers to parser whitelist

2014-09-23 Thread Zhenyu Wang
On 2014.09.23 10:38:49 +0200, Daniel Vetter wrote: On Tue, Sep 23, 2014 at 12:41:50AM +0200, Michał Winiarski wrote: These registers are used as a temporary storage by MI_MATH command when performing ALU operations. Signed-off-by: Michał Winiarski michal.winiar...@intel.com Needs to

Re: [Intel-gfx] [PATCH 47/89] drm/i915/skl: SKL Watermark Computation

2014-09-23 Thread Damien Lespiau
On Wed, Sep 17, 2014 at 03:07:51PM +0300, Ville Syrjälä wrote: On Thu, Sep 04, 2014 at 12:27:13PM +0100, Damien Lespiau wrote: From: Pradeep Bhat pradeep.b...@intel.com This patch implements the watermark algorithm and its necessary functions. Two function pointers skl_update_wm and

[Intel-gfx] [PATCH 47/89 v11] drm/i915/skl: SKL Watermark Computation

2014-09-23 Thread Damien Lespiau
From: Pradeep Bhat pradeep.b...@intel.com This patch implements the watermark algorithm and its necessary functions. Two function pointers skl_update_wm and skl_update_sprite_wm are provided. The skl_update_wm will update the watermarks for the crtc provided as an argument and then checks for

[Intel-gfx] [PATCH 44/89 v5] drm/i915/skl: Register definitions and macros for SKL Watermark regs

2014-09-23 Thread Damien Lespiau
From: Pradeep Bhat pradeep.b...@intel.com This patch defines SKL specific PLANE_WM Watermark registers. It also defines macros to get the addresses of different LP levels within a pipe. v2: Reworked the register definitions and associated macros to make it more generic and be able to use

[Intel-gfx] [PATCH 48/89 v4] drm/i915/skl: Allocate DDB portions for display planes

2014-09-23 Thread Damien Lespiau
v2: Fix the 3rd plane/cursor logic (Pradeep Bhat) v3: Fix one-by-one error in the DDB allocation code v4: Rebase on top of the skl_pipe_pixel_rate() argument change Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/drm/i915/intel_pm.c | 150

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

2014-09-23 Thread Daniel Vetter
On Tue, Sep 23, 2014 at 10:19:02AM +0100, Chris Wilson wrote: On Tue, Sep 23, 2014 at 10:26:26AM +0200, Daniel Vetter wrote: On Fri, Sep 19, 2014 at 09:00:00PM +0100, Chris Wilson wrote: On Fri, Sep 19, 2014 at 06:21:46PM +, Tian, Kevin wrote: From: Chris Wilson The

Re: [Intel-gfx] [PATCH] drm/i915: Don't leak command parser tables on suspend/resume

2014-09-23 Thread Daniel Vetter
On Tue, Sep 23, 2014 at 10:14:33AM +0100, Chris Wilson wrote: On Tue, Sep 23, 2014 at 10:34:46AM +0200, Daniel Vetter wrote: On Mon, Sep 22, 2014 at 08:25:21AM -0700, bradley.d.vol...@intel.com wrote: From: Brad Volkin bradley.d.vol...@intel.com Ring init and cleanup are not balanced

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

2014-09-23 Thread Jani Nikula
On Tue, 23 Sep 2014, Daniel Vetter dan...@ffwll.ch wrote: On Fri, Sep 19, 2014 at 08:49:06PM +0300, Ville Syrjälä wrote: On Fri, Sep 19, 2014 at 08:05:26PM +0300, Mika Kuoppala wrote: as these have been fixed in production hw and hurt performance if applied. v2: adjust requested ring

Re: [Intel-gfx] [PATCH] drm/i915: Flush the PTEs after updating them before suspend

2014-09-23 Thread Jani Nikula
On Thu, 18 Sep 2014, Daniel Vetter dan...@ffwll.ch wrote: On Thu, Sep 18, 2014 at 07:03:32AM +0100, Chris Wilson wrote: As we use WC updates of the PTE, we are responsible for notifying the hardware when to flush its TLBs. Do so after we zap all the PTEs before suspend (and the BIOS tries to

Re: [Intel-gfx] No display at boot

2014-09-23 Thread Jani Nikula
On Tue, 23 Sep 2014, 陳勇秀 greatshow.c...@gmail.com wrote: Hi All, I am using a baytrail cpu and linux kernel 3.14.7, and will echo some message to /dev/tty1 during boot. There is chance that no display output during boot, but I can use linux via ttyS4, the HSUART. Attached is output of

Re: [Intel-gfx] [PATCH 14/19] drm: Don't update vblank timestamp when the counter didn't change

2014-09-23 Thread Jani Nikula
On Mon, 15 Sep 2014, Daniel Vetter dan...@ffwll.ch wrote: On Sat, Sep 13, 2014 at 06:25:54PM +0200, Mario Kleiner wrote: The current drm-next misses Ville's original Patch 14/19, the one i first objected, then objected to my objection. It is needed to avoid actual regressions. Attached a

Re: [Intel-gfx] [PATCH] drm/i915: Don't leak command parser tables on suspend/resume

2014-09-23 Thread Jani Nikula
On Tue, 23 Sep 2014, Daniel Vetter dan...@ffwll.ch wrote: On Mon, Sep 22, 2014 at 08:25:21AM -0700, bradley.d.vol...@intel.com wrote: From: Brad Volkin bradley.d.vol...@intel.com Ring init and cleanup are not balanced because we re-init the rings on resume without having cleaned them up on

[Intel-gfx] [PATCH] tests/kms_psr_sink_crc: Use options

2014-09-23 Thread Daniel Vetter
Env variables are a bit more annoying since much harder to discover. With options you can just see what they do with --help. Cc: Rodrigo Vivi rodrigo.v...@intel.com Signed-off-by: Daniel Vetter daniel.vet...@intel.com --- tests/kms_psr_sink_crc.c | 30 -- 1 file

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

2014-09-23 Thread Jani Nikula
On Wed, 17 Sep 2014, Michel Thierry michel.thie...@intel.com wrote: On 9/17/2014 11:20 AM, Jani Nikula wrote: IMHO it would be perilous to apply these patches before we have root caused https://bugs.freedesktop.org/show_bug.cgi?id=83482. I think we need to be able to revert those changes if

Re: [Intel-gfx] [PATCH 14/19] drm: Don't update vblank timestamp when the counter didn't change

2014-09-23 Thread Daniel Vetter
On Tue, Sep 23, 2014 at 03:48:25PM +0300, Jani Nikula wrote: On Mon, 15 Sep 2014, Daniel Vetter dan...@ffwll.ch wrote: On Sat, Sep 13, 2014 at 06:25:54PM +0200, Mario Kleiner wrote: The current drm-next misses Ville's original Patch 14/19, the one i first objected, then objected to my

Re: [Intel-gfx] [PATCH] drm/i915/hdmi: fix hdmi audio state readout

2014-09-23 Thread Jani Nikula
On Wed, 17 Sep 2014, Daniel Vetter dan...@ffwll.ch wrote: On Wed, Sep 17, 2014 at 03:34:58PM +0300, Jani Nikula wrote: Check the correct bit for audio. Seems like a copy-paste error from the start: commit 9ed109a7b445e3f073d8ea72f888ec80c0532465 Author: Daniel Vetter daniel.vet...@ffwll.ch

Re: [Intel-gfx] [PATCH 14/19] drm: Don't update vblank timestamp when the counter didn't change

2014-09-23 Thread Mario Kleiner
On 23/09/14 15:51, Daniel Vetter wrote: On Tue, Sep 23, 2014 at 03:48:25PM +0300, Jani Nikula wrote: On Mon, 15 Sep 2014, Daniel Vetter dan...@ffwll.ch wrote: On Sat, Sep 13, 2014 at 06:25:54PM +0200, Mario Kleiner wrote: The current drm-next misses Ville's original Patch 14/19, the one i

Re: [Intel-gfx] [PATCH 63/89] drm/i915/skl: Define shared DPLLs for Skylake

2014-09-23 Thread Paulo Zanoni
2014-09-04 8:27 GMT-03:00 Damien Lespiau damien.lesp...@intel.com: From: Satheeshakrishna M satheeshakrishn...@intel.com On skylake, DPLL 1, 2 and 3 can be used for DP and HDMI. The shared dpll framework allows us to share those DPLLs among DDIs when possible. The most tricky part is to

Re: [Intel-gfx] [PATCH 64/89] drm/i915/skl: Adjust the port PLL selection code

2014-09-23 Thread Paulo Zanoni
2014-09-04 8:27 GMT-03:00 Damien Lespiau damien.lesp...@intel.com: From: Satheeshakrishna M satheeshakrishn...@intel.com Skylake deprecates the usage of PORT_CLK_SEL and we are advised to use the new DPLL_CRTL2 for the DDI-PLL mapping. Signed-off-by: Satheeshakrishna M

Re: [Intel-gfx] [PATCH 65/89] drm/i915/skl: Always use DPLL0 for eDP

2014-09-23 Thread Paulo Zanoni
2014-09-04 8:27 GMT-03:00 Damien Lespiau damien.lesp...@intel.com: From: Satheeshakrishna M satheeshakrishn...@intel.com DPLL0 is not part of the shared PLL infrastructure. We'll use on for eDP and rely on what the BIOS does for now. Signed-off-by: Satheeshakrishna M

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

2014-09-23 Thread Michel Thierry
On 9/23/2014 2:21 PM, Jani Nikula wrote: On Wed, 17 Sep 2014, Michel Thierry michel.thie...@intel.com wrote: On 9/17/2014 11:20 AM, Jani Nikula wrote: FYI, Mika has posted the fix: http://mid.gmane.org/1411146326-9884-1-git-send-email-mika.kuopp...@intel.com Thanks, (Has it been merged? I

[Intel-gfx] [PATCH 45/89 v4] drm/i915/skl: Definition of SKL WM param structs for pipe/plane

2014-09-23 Thread Damien Lespiau
From: Pradeep Bhat pradeep.b...@intel.com This patch defines the structures needed for computation of watermarks of pipes and planes for SKL. v2: Incorporated Damien's review comments and removed unused fields in structs for future features like rotation, drrs and scaling. The

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

2014-09-23 Thread Ville Syrjälä
On Tue, Sep 23, 2014 at 12:41:56PM -0300, Gustavo Padovan wrote: 2014-09-23 Ville Syrjälä ville.syrj...@linux.intel.com: On Mon, Sep 22, 2014 at 07:23:10PM -0300, Gustavo Padovan wrote: From: Gustavo Padovan gustavo.pado...@collabora.co.uk Move checks inside

Re: [Intel-gfx] [PATH] Correct GPU timestamp read

2014-09-23 Thread Jacek Danecki
On 09/23/14 10:37, Daniel Vetter wrote: On Mon, Sep 22, 2014 at 06:22:53PM +0200, Jacek Danecki wrote: Current implementation of reading GPU timestamp is broken. It returns lower 32 bits shifted by 32 bits ( instead of ). Below change is adding possibility to

Re: [Intel-gfx] [PATCH 66/89] drm/i915/skl: Implementation of SKL DPLL programming

2014-09-23 Thread Paulo Zanoni
2014-09-04 8:27 GMT-03:00 Damien Lespiau damien.lesp...@intel.com: From: Satheeshakrishna M satheeshakrishn...@intel.com This patch implements SKL DPLL programming that includes: - DPLL allocation - wide range PLL calculation and programming - DP link rate programming

Re: [Intel-gfx] [PATCH 67/89] drm/i915/skl: Provide skl-specific pll hw state cross-checking

2014-09-23 Thread Paulo Zanoni
2014-09-04 8:27 GMT-03:00 Damien Lespiau damien.lesp...@intel.com: v2: rebase on top of the hw state flattening. Reviewed-by: Paulo Zanoni paulo.r.zan...@intel.com Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/drm/i915/intel_display.c | 3 +++ 1 file changed, 3

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

2014-09-23 Thread clinton . a . taylor
From: Clint Taylor clinton.a.tay...@intel.com 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

Re: [Intel-gfx] [PATCH 68/89] drm/i915/skl: Apply eDP WA only for gen 9

2014-09-23 Thread Paulo Zanoni
2014-09-04 8:27 GMT-03:00 Damien Lespiau damien.lesp...@intel.com: From: Vandana Kannan vandana.kan...@intel.com The eDP WA to stop link train based on port type is for HSW/BDW, not required for SKL+. Suggested by Satheesh v2: Simplified the check befoe stop_link_train. Suggested by

Re: [Intel-gfx] [PATCH 74/89] drm/i915/skl: Implement queue_flip

2014-09-23 Thread Paulo Zanoni
2014-09-04 8:27 GMT-03:00 Damien Lespiau damien.lesp...@intel.com: A few bits have changed in MI_DISPLAY_FLIP to accomodate the new planes. DE_RRMR seems to have kept its plane flip bits backward compatible. v2: Rebase on top of nightly v2: Rebase on top of nightly (minor conflict in

[Intel-gfx] [PATCH 1/2] drm/i915: Put logical pipe_control emission into a helper.

2014-09-23 Thread Rodrigo Vivi
To be used for a Workaroud. Similar to: commit 884ceacee308f0e4616d0c933518af2639f7b1d8 Author: Kenneth Graunke kenn...@whitecape.org Date: Sat Jun 28 02:04:20 2014 +0300 drm/i915: Refactor Broadwell PIPE_CONTROL emission into a helper. Signed-off-by: Rodrigo Vivi rodrigo.v...@intel.com

[Intel-gfx] [PATCH 2/2] drm/i915: Add WaCsStallBeforeStateCacheInvalidate:bdw, chv to logical ring

2014-09-23 Thread Rodrigo Vivi
Similar to: commit 02c9f7e3cfe76a7f54ef03438c36aade86cc1c8b Author: Kenneth Graunke kenn...@whitecape.org Date: Mon Jan 27 14:20:16 2014 -0800 drm/i915: Add the WaCsStallBeforeStateCacheInvalidate:bdw workaround. On Broadwell, any PIPE_CONTROL with the State Cache Invalidate bit set

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

2014-09-23 Thread Hans de Goede
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. More information about this problem: For configuring brightness on Dell

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

2014-09-23 Thread Paulo Zanoni
2014-09-04 8:27 GMT-03:00 Damien Lespiau damien.lesp...@intel.com: From: Jesse Barnes jbar...@virtuousgeek.org This moved around on SKL, so we need to make sure we read/write the correct regs. Signed-off-by: Jesse Barnes jbar...@virtuougseek.org Signed-off-by: Damien Lespiau

Re: [Intel-gfx] [PATCH 1/4] drm/i915: PSR: organize setup function.

2014-09-23 Thread Paulo Zanoni
2014-09-16 20:19 GMT-03:00 Rodrigo Vivi rodrigo.v...@gmail.com: psr_enabled is already by itself a setup once so let's put the W/As there and rename old setup once to setup_vsc. Yeah, I prefer the new way too. Reviewed-by: Paulo Zanoni paulo.r.zan...@intel.com Signed-off-by: Rodrigo Vivi

Re: [Intel-gfx] [PATCH] tests/kms_psr_sink_crc: Use options

2014-09-23 Thread Vivi, Rodrigo
I had tried options but didn't liked because I was unable to list subtests... so I gave back to env var... But now I see that I was probably forgetting igt_subtest_init_parse_opts() Thanks Acked/Reviewed-by: Rodrigo Vivi rodrigo.v...@intel.com From:

Re: [Intel-gfx] [PATCH 2/4] drm/i915: PSR: Organize PSR enable function

2014-09-23 Thread Paulo Zanoni
2014-09-16 20:19 GMT-03:00 Rodrigo Vivi rodrigo.v...@gmail.com: We don't need to setup everything else if it doesn't match all conditions. Reviewed-by: Paulo Zanoni paulo.r.zan...@intel.com Signed-off-by: Rodrigo Vivi rodrigo.v...@intel.com --- drivers/gpu/drm/i915/intel_dp.c | 10

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

2014-09-23 Thread Joe Konno
From: Joe Konno joe.ko...@intel.com Improper integer division-- truncated rather than rounded-- 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

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

2014-09-23 Thread Eoff, Ullysses A
On Tue, 2014-09-23 at 15:50 -0700, Joe Konno wrote: From: Joe Konno joe.ko...@intel.com Improper integer division-- truncated rather than rounded-- in the scale() function causes actual_brightness != brightness. This (partial) work-around should be sufficient for a majority of use-cases, but