Re: [Intel-gfx] [PATCH v3] drm/i915: Disable the mmio.debug WARN after it fires

2014-12-26 Thread Paulo Zanoni
2014-12-18 10:47 GMT-02:00 Chris Wilson : > On Thu, Dec 18, 2014 at 02:36:54PM +0200, Jani Nikula wrote: >> On Thu, 18 Dec 2014, Chris Wilson wrote: >> > If we have a single unclaimed register, we will have lots. A WARN for >> > each one makes the machine unusable and does not aid debugging. Conve

Re: [Intel-gfx] 3.19-rc1 errors when opening LID

2014-12-26 Thread Pali Rohár
On Friday 26 December 2014 16:12:49 Rafael J. Wysocki wrote: > On Wednesday, December 24, 2014 07:51:48 PM Pali Rohár wrote: > > --nextPart5943893.pKyMBm5Emp > > Content-Type: Text/Plain; > > > > charset="utf-8" > > > > Content-Transfer-Encoding: quoted-printable > > > > Hello! > > > > With n

Re: [Intel-gfx] 3.19-rc1 errors when opening LID

2014-12-26 Thread Rafael J. Wysocki
On Wednesday, December 24, 2014 07:51:48 PM Pali Rohár wrote: > > --nextPart5943893.pKyMBm5Emp > Content-Type: Text/Plain; > charset="utf-8" > Content-Transfer-Encoding: quoted-printable > > Hello! > > With new 3.19-rc1 kernel every time when I open LID on my laptop, kernel pr= > ints these er

Re: [Intel-gfx] [PATCH 4/9] drm/i915: don't free the CFB while FBC is enabled

2014-12-26 Thread Chris Wilson
On Fri, Dec 26, 2014 at 11:46:49AM -0200, Paulo Zanoni wrote: > 2014-12-25 8:20 GMT-02:00 Chris Wilson : > > On Tue, Dec 23, 2014 at 10:35:40AM -0200, Paulo Zanoni wrote: > >> From: Paulo Zanoni > >> > >> Because that is probably not very a good idea: if we used the stolen > >> memory for more thi

Re: [Intel-gfx] [PATCH 2/9] drm/i915: fix the FBC CFB size tracking

2014-12-26 Thread Chris Wilson
On Fri, Dec 26, 2014 at 11:46:34AM -0200, Paulo Zanoni wrote: > 2014-12-25 8:16 GMT-02:00 Chris Wilson : > > On Tue, Dec 23, 2014 at 10:35:38AM -0200, Paulo Zanoni wrote: > >> From: Paulo Zanoni > >> > >> We have dev_priv->fbc.size which is supposed to contain the compressed > >> FB size, but it i

Re: [Intel-gfx] [PATCH 7/9] drm/i915: don't alloc/free fbc_work at every update

2014-12-26 Thread Paulo Zanoni
2014-12-25 8:33 GMT-02:00 Chris Wilson : > On Tue, Dec 23, 2014 at 10:35:43AM -0200, Paulo Zanoni wrote: >> From: Paulo Zanoni >> >> Because there's no need for it. Just use a static structure with a >> bool field to tell if it's in use or not. The big advantage here is >> not saving kzalloc/kfree

Re: [Intel-gfx] [PATCH 6/9] drm/i915: add the FBC mutex

2014-12-26 Thread Paulo Zanoni
2014-12-25 8:25 GMT-02:00 Chris Wilson : > On Tue, Dec 23, 2014 at 10:35:42AM -0200, Paulo Zanoni wrote: >> From: Paulo Zanoni >> >> Make sure we're not gonna have weird races in really weird cases where >> a lot of different CRTCs are doing rendering and modesets at the same >> time. > > You are

Re: [Intel-gfx] [PATCH 4/9] drm/i915: don't free the CFB while FBC is enabled

2014-12-26 Thread Paulo Zanoni
2014-12-25 8:20 GMT-02:00 Chris Wilson : > On Tue, Dec 23, 2014 at 10:35:40AM -0200, Paulo Zanoni wrote: >> From: Paulo Zanoni >> >> Because that is probably not very a good idea: if we used the stolen >> memory for more things, there could be a risk that someone would >> "allocate" the memory tha

Re: [Intel-gfx] [PATCH 2/9] drm/i915: fix the FBC CFB size tracking

2014-12-26 Thread Paulo Zanoni
2014-12-25 8:16 GMT-02:00 Chris Wilson : > On Tue, Dec 23, 2014 at 10:35:38AM -0200, Paulo Zanoni wrote: >> From: Paulo Zanoni >> >> We have dev_priv->fbc.size which is supposed to contain the compressed >> FB size, but it is not: at find_compression_threshold() we try to >> overallocate the CFB,

Re: [Intel-gfx] [PATCH] igt: Correct the return value for drm short_buffer read

2014-12-26 Thread Chris Wilson
On Fri, Dec 26, 2014 at 01:16:06AM +, Zhang, Xiong Y wrote: > > -Original Message- > > From: Chris Wilson [mailto:ch...@chris-wilson.co.uk] > > Sent: Tuesday, December 23, 2014 7:31 PM > > To: Zhang, Xiong Y > > Cc: intel-gfx@lists.freedesktop.org > > Subject: Re: [Intel-gfx] [PATCH] ig

[Intel-gfx] [RFC 3/4] drm/i915/Kconfig: By default select DRM_PANEL

2014-12-26 Thread Shobhit Kumar
This will be needed for enabling drm_panel driver over pmic Signed-off-by: Shobhit Kumar --- drivers/gpu/drm/i915/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/Kconfig b/drivers/gpu/drm/i915/Kconfig index 4e39ab3..3210dbb 100644 --- a/drivers/gpu/drm/i915/Kconf

[Intel-gfx] [RFC 4/4] drm/i915: Enable PMIC panel control as drm_panel for DSI

2014-12-26 Thread Shobhit Kumar
This initialize the drm_panel based on PMIC driver and uses drm_panel_* for controlling the panel enable/disable states Signed-off-by: Shobhit Kumar --- drivers/gpu/drm/i915/intel_dsi.c | 20 drivers/gpu/drm/i915/intel_dsi.h | 6 ++ drivers/gpu/drm/i

[Intel-gfx] [RFC 2/4] drm/i915: Add a drm_panel over INTEL_SOC_PMIC

2014-12-26 Thread Shobhit Kumar
This driver just add PANEL_ENABLE/DISABLE control using drm_panel framework. Signed-off-by: Shobhit Kumar --- drivers/gpu/drm/i915/Makefile | 1 + drivers/gpu/drm/i915/intel_drv.h| 3 + drivers/gpu/drm/i915/intel_panel_pmic.c | 157 3 files

[Intel-gfx] [RFC 1/4] drm/i915: Define a common data structure for Panel Info

2014-12-26 Thread Shobhit Kumar
As of now this includes only PPS and BLC delays. New things can be added as and when needed Signed-off-by: Shobhit Kumar --- drivers/gpu/drm/i915/intel_drv.h | 11 +++ drivers/gpu/drm/i915/intel_dsi.h | 7 +-- drivers/gpu/drm/i915/intel_dsi_panel_vbt.c | 10 +

[Intel-gfx] [RFC 0/4] PMIC based Panel and Backlight Control

2014-12-26 Thread Shobhit Kumar
Hi All, For quite some time this problem is there in front of us and I am trying to have a closure on how to go about doing this. This patch set is a crude implementation to trigger the discussion. Daniel sometime back suggested to look at drm_panel for doing this but I was not able to figure o