Re: [Intel-gfx] [PATCH v3 10/20] drm/i915: Set csc coefficients in intel_sanitize_crtc.

2015-07-13 Thread Daniel Stone
Hi, On 13 July 2015 at 15:30, Maarten Lankhorst wrote: > This might not have been set during boot, and when we preserve > the initial mode this can result in a black screen. > > Cc: Daniel Stone > Signed-off-by: Maarten Lankhorst > --- > drivers/gpu/drm/i915/intel_display.c | 3 +++ > 1 file c

Re: [Intel-gfx] [PATCH v3 06/20] drm/i915: Remove plane_config from struct intel_crtc.

2015-07-13 Thread Daniel Stone
Hi, On 13 July 2015 at 15:30, Maarten Lankhorst wrote: > diff --git a/drivers/gpu/drm/i915/intel_display.c > b/drivers/gpu/drm/i915/intel_display.c > index 037a85f1b127..e4d8acc63823 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -15204,7 +1

[Intel-gfx] [RFC 0/1] drm/i915 : Wait until SYSTEM_RUNNING before loading CSR firmware

2015-07-13 Thread jay . p . patel
From: jay Chrome OS is "noinitrd" OS. It does not normally allow in-built firmware in kernel. i915 driver initialization precedes the initialization of file system. Hence, loading fails whenever driver tries to load CSR firmware from file system. The current implementation uses "request_firmwa

[Intel-gfx] [RFC 1/1] drm/i915 : Wait until SYSTEM_RUNNING before loading CSR firmware

2015-07-13 Thread jay . p . patel
From: Jay Patel NOTE: This is an interim solution which is targeted towards Chrome OS/Android to be used until a long term solution is available. In this patch, request_firmware() is called in a worker thread which initially waits for file system to be initialized and then attempts to load the f

Re: [Intel-gfx] [PATCH v3 19/20] drm/i915: always disable irqs in intel_pipe_update_start

2015-07-13 Thread Daniel Stone
Hi, On 13 July 2015 at 15:30, Maarten Lankhorst wrote: > @@ -13649,9 +13647,7 @@ static void intel_begin_crtc_commit(struct drm_crtc > *crtc) > > /* Perform vblank evasion around commit operation */ > if (crtc->state->active) > - intel_crtc->atomic.evade = > -

Re: [Intel-gfx] [PATCH] drm: Provide compat ioctl for addfb2.1

2015-07-13 Thread Rob Clark
btw, I mentioned on irc, but someone *really* ought to go through the ioclt structs and split each struct member into it's own line, so things like '__u32 width, height' aren't so sneaky ;-) On Mon, Jul 13, 2015 at 11:56 AM, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin > > Frame buffer modifiers

Re: [Intel-gfx] [PATCH] drm/i915: Adjust BXT HDMI port clock limits

2015-07-13 Thread Bish, Jim
On Mon, 2015-07-13 at 11:05 +0200, Daniel Vetter wrote: > On Fri, Jul 10, 2015 at 02:27:48PM +0100, Damien Lespiau wrote: > > On Fri, Jul 10, 2015 at 04:21:27PM +0300, Ville Syrjälä wrote: > > > On Fri, Jul 10, 2015 at 02:18:57PM +0100, Damien Lespiau wrote: > > > > On Fri, Jul 10, 2015 at 04:09:42

Re: [Intel-gfx] [PATCH v3 00/20] Convert to atomic, part 4.

2015-07-13 Thread Daniel Stone
Hi, On Monday, July 13, 2015, Maarten Lankhorst < maarten.lankho...@linux.intel.com> wrote: > Atomic suspend/resume, full hardware readout and atomic ioctl support. > > Changes from the previous version: > - The fastboot changes from the previous patch have been removed, > fastboot will have to

Re: [Intel-gfx] [RFC 0/2] Add Pooled EU support

2015-07-13 Thread Chris Wilson
On Mon, Jul 13, 2015 at 04:00:08PM +0100, Siluvery, Arun wrote: > On 11/07/2015 20:09, Chris Wilson wrote: > >On Sat, Jul 11, 2015 at 08:05:05PM +0100, Chris Wilson wrote: > >>On Fri, Jul 10, 2015 at 06:35:18PM +0100, Arun Siluvery wrote: > >>>These patches enabled Pooled EU support for BXT, they a

Re: [Intel-gfx] [PATCH v4 03/18] drm/i915/gen8: Add PML4 structure

2015-07-13 Thread Chris Wilson
On Mon, Jul 13, 2015 at 03:41:38PM +0100, Michel Thierry wrote: > On 7/11/2015 9:02 PM, Chris Wilson wrote: > >On Tue, Jul 07, 2015 at 04:14:48PM +0100, Michel Thierry wrote: > >>Introduces the Page Map Level 4 (PML4), ie. the new top level structure > >>of the page tables. > >> > >>To facilitate t

Re: [Intel-gfx] [PATCH v2] scripts/kernel-doc: Adding cross-reference links to html documentation.

2015-07-13 Thread Danilo Cesar Lemes de Paula
On 07/09/2015 08:56 PM, Jonathan Corbet wrote: > On Fri, 26 Jun 2015 12:08:57 -0300 > Danilo Cesar Lemes de Paula wrote: > >> To ease the navigation in the documentation we should use inside >> those tags so readers can easily jump between methods directly. >> >> This was discussed in 2014[1] an

[Intel-gfx] [PATCH v3] scripts/kernel-doc: Adding cross-reference links to html documentation.

2015-07-13 Thread Danilo Cesar Lemes de Paula
Functions, Structs and Parameters definitions on kernel documentation are pure cosmetic, it only highlights the element. To ease the navigation in the documentation we should use inside those tags so readers can easily jump between methods directly. This was discussed in 2014[1] and is implement

[Intel-gfx] [drm-intel:drm-intel-nightly 9/12] drivers/gpu/drm/exynos/exynos_drm_fbdev.c:192:1: warning: label 'err_dealloc_cmap' defined but not used

2015-07-13 Thread kbuild test robot
tree: git://anongit.freedesktop.org/drm-intel drm-intel-nightly head: 39c8d0f886e9496eba420b1b5554bd1fa0c5c337 commit: 76c6e302feb1d6451c6bf4676fd1e98dfcd31316 [9/12] Merge remote-tracking branch 'origin/topic/drm-misc' into drm-intel-nightly config: arm-multi_v7_defconfig (attached as .config

[Intel-gfx] linux-next: manual merge of the drm-intel tree with the drm-intel-fixes tree

2015-07-13 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the drm-intel tree got a conflict in: drivers/gpu/drm/i915/intel_display.c between commit: 8aa3053bf731 ("drm/i915: fix oops in primary_check_plane") from the drm-intel-fixes tree and commit: da20eabd2c69 ("drm/i915: Split plane updates of crtc->atomi

[Intel-gfx] linux-next: build failure after merge of the drm-misc tree

2015-07-13 Thread Stephen Rothwell
on of function 'drm_fb_helper_remove_conflicting_framebuffers' [-Werror=implicit-function-declaration] drm_fb_helper_remove_conflicting_framebuffers(ap, "virtiodrmfb", ^ Caused by commit 7bd870e7b1c8 ("drm/virtio: Use new drm_fb_helper functions") I have used the drm-m

[Intel-gfx] linux-next: build warnings after merge of the drm-misc tree

2015-07-13 Thread Stephen Rothwell
Hi all, After merging the drm-misc tree, today's linux-next build (x86_64 allmodconfig) produced these warnings: drivers/gpu/drm/bochs/bochs_fbdev.c: In function 'bochsfb_create': drivers/gpu/drm/bochs/bochs_fbdev.c:63:17: warning: unused variable 'device' [-Wunused-variable] struct device *de

Re: [Intel-gfx] [PATCH] drm/i915: Set edid from init and not from detect

2015-07-13 Thread Sharma, Shashank
Hi Daniel, Chris Gen7 and Gen8 platforms have a different live status register (0x61114) and we need not to rely on ISR on that. As Sonika mentioned, We have been using this register for our commercial releases, and found them reliable across a wide range of monitors. On the other hand, Bse

Re: [Intel-gfx] [PATCH] drm/i915: Check live status before reading edid

2015-07-13 Thread Jindal, Sonika
On 7/13/2015 8:25 PM, Daniel Vetter wrote: On Mon, Jul 13, 2015 at 04:35:00PM +0530, Sonika Jindal wrote: Adding this for SKL onwards. v2: Adding checks for VLV/CHV as well. Reusing old ibx and g4x functions to check digital port status. Adding a separate function to get bxt live status (Dani

[Intel-gfx] [PATCH 1/2] drm/i915/bxt: Add HPD support for DDIA

2015-07-13 Thread Sonika Jindal
Also remove redundant comments. Signed-off-by: Sonika Jindal --- drivers/gpu/drm/i915/i915_irq.c | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c index a897f68..63137dd 100644 --- a/drivers/gpu/drm/

[Intel-gfx] [PATCH 2/2] drm/i915/bxt: WA for swapped HPD pins in A stepping

2015-07-13 Thread Sonika Jindal
As per bspec, on BXT A0/A1, sw needs to activate DDIA HPD logic and interrupts to check the external panel connection and DDIC HPD logic for edp panel. Signed-off-by: Sonika Jindal --- drivers/gpu/drm/i915/intel_dp.c | 18 -- drivers/gpu/drm/i915/intel_hdmi.c |9 -

<    1   2