Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/gvt: Fix build failure after intel_engine_cs change

2016-10-17 Thread Zhenyu Wang
On 2016.10.18 05:39:11 +, Saarinen, Jani wrote: > > == Series Details == > > > > Series: drm/i915/gvt: Fix build failure after intel_engine_cs change > > URL : https://patchwork.freedesktop.org/series/13910/ > > State : failure > > > > == Summary == > > > > Series 13910v1 drm/i915/gvt:

Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/gvt: Fix build failure after intel_engine_cs change

2016-10-17 Thread Saarinen, Jani
> == Series Details == > > Series: drm/i915/gvt: Fix build failure after intel_engine_cs change > URL : https://patchwork.freedesktop.org/series/13910/ > State : failure > > == Summary == > > Series 13910v1 drm/i915/gvt: Fix build failure after intel_engine_cs change >

[Intel-gfx] [i-g-t PATCH] igt/tools: Update intel_watermark with SKL support

2016-10-17 Thread Dhinakaran Pandiyan
Added support to print SKL watermark and DDB registers. Signed-off-by: Dhinakaran Pandiyan --- tools/intel_watermark.c | 104 +++- 1 file changed, 103 insertions(+), 1 deletion(-) diff --git a/tools/intel_watermark.c

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/gvt: Fix build failure after intel_engine_cs change

2016-10-17 Thread Patchwork
== Series Details == Series: drm/i915/gvt: Fix build failure after intel_engine_cs change URL : https://patchwork.freedesktop.org/series/13910/ State : failure == Summary == Series 13910v1 drm/i915/gvt: Fix build failure after intel_engine_cs change

Re: [Intel-gfx] [PULL] GVT-g device model core

2016-10-17 Thread Zhenyu Wang
On 2016.10.17 16:07:50 +0200, Daniel Vetter wrote: > Yeah might be best to have a new branch with upstreaming stuff (now you > need to at least split out bugfixes for the already merged stuff) and > treat that like a mostly stable branch. And the still unmerged features > (vfio and all that) would

[Intel-gfx] [PATCH] drm/i915/gvt: Fix build failure after intel_engine_cs change

2016-10-17 Thread Zhenyu Wang
Change GVT-g code reference for intel_engine_cs from static array to allocated pointer after commit 3b3f1650b1ca ("drm/i915: Allocate intel_engine_cs structure only for the enabled engines"). Signed-off-by: Zhenyu Wang --- drivers/gpu/drm/i915/gvt/execlist.c | 2 +-

Re: [Intel-gfx] [PATCH] drm/i915: KBL - Recommended buffer translation programming for DisplayPort

2016-10-17 Thread Manasi Navare
On Thu, Oct 06, 2016 at 10:54:01AM -0700, Rodrigo Vivi wrote: > According to spec: "KBL re-uses SKL values, except where > specific KBL values are listed." > > And recently spec has changed adding different table for Display Port only. > But for all SKUs (H,S,U,Y) we have slightly different

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

2016-10-17 Thread Stephen Rothwell
tructure only for the enabled engines") interacting with some other commits that were merged without fixes being applied (probably) in commit 06a75ace46e2 ("Merge tag 'gvt-next-2016-10-14' of https://github.com/01org/gvt-linux into drm-intel-next-queued") I have used the versi

[Intel-gfx] [drm-intel:for-linux-next 0/3] drivers/gpu/drm/i915/gvt/handlers.c:137: error: request for member 'mmio_base' in something not a structure or union

2016-10-17 Thread kbuild test robot
tree: git://anongit.freedesktop.org/drm-intel for-linux-next head: bfd02b3c557caa083be0d55a3164ede706a446e1 commit: 06a75ace46e2fdd1d93b06228df0e2dfe526cc27 [0/3] Merge tag 'gvt-next-2016-10-14' of https://github.com/01org/gvt-linux into drm-intel-next-queued config:

Re: [Intel-gfx] [PATCH 12/15] drm: RIP mode_config->rotation_property

2016-10-17 Thread Laurent Pinchart
Hi Ville, On Friday 22 Jul 2016 16:43:13 ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > Now that all drivers have been converted over to the per-plane rotation > property, we can just nuke the global rotation property. > > Signed-off-by: Ville

[Intel-gfx] [PATCH xf86-video-intel] sna: Reprobe if kernel updates the connector mode list

2016-10-17 Thread Manasi Navare
Output_check_status() should return a false if it detects that the connector mode list has changed so that sna_mode_discover can reprobe. Fixes: eb01cc549d4d (sna: Refresh mode list if the kernel updates) Signed-off-by: Manasi Navare Cc: Chris Wilson

Re: [Intel-gfx] [PATCH 1/2] drm/i915/gen9: fix watermarks when using the pipe scaler

2016-10-17 Thread Matt Roper
On Fri, Oct 07, 2016 at 05:28:57PM -0300, Paulo Zanoni wrote: > Luckily, the necessary adjustments for when we're using the scaler are > exactly the same as the ones needed on ILK+, so just reuse the > function we already have. > > v2: Invert the patch order so stable backports get easier. > >

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Trust VBT aux/ddc information (rev2)

2016-10-17 Thread Patchwork
== Series Details == Series: drm/i915: Trust VBT aux/ddc information (rev2) URL : https://patchwork.freedesktop.org/series/13600/ State : failure == Summary == Series 13600v2 drm/i915: Trust VBT aux/ddc information https://patchwork.freedesktop.org/api/1.0/series/13600/revisions/2/mbox/ Test

[Intel-gfx] [PATCH v3 3/4] drm/i915: Clean up DDI DDC/AUX CH sanitation

2016-10-17 Thread ville . syrjala
From: Ville Syrjälä Now that we use the AUX and GMBUS assignment from VBT for all ports, let's clean up the sanitization of the port information a bit. Previosuly we only did this for port E, and only complained about a non-standard assignment for the other ports.

Re: [Intel-gfx] [PATCH 3/4] drm/i915: Clean up DDI DDC/AUX CH sanitation

2016-10-17 Thread Ville Syrjälä
On Tue, Oct 11, 2016 at 08:52:47PM +0300, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > Now that we use the AUX and GMBUS assignment from VBT for all ports, > let's clean up the sanitization of the port information a bit. > Previosuly we only did

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Restrict pagefault disabling to just around copy_from_user()

2016-10-17 Thread Patchwork
== Series Details == Series: drm/i915: Restrict pagefault disabling to just around copy_from_user() URL : https://patchwork.freedesktop.org/series/13880/ State : failure == Summary == Series 13880v1 drm/i915: Restrict pagefault disabling to just around copy_from_user()

Re: [Intel-gfx] [PATCH 2/2] drm/i915/DMC/KBL: Load DMC on KBL using he no_stepping_info array

2016-10-17 Thread Vivi, Rodrigo
On Fri, 2016-10-14 at 16:56 -0700, Anusha Srivatsa wrote: > Currently, for display there is only one DMC image for KBL. > Remove the stepping_info table for KBL and use the no_stepping_info > for loading the firmware image. > > Cc: Rodrigo Vivi > Signed-off-by: Anusha

Re: [Intel-gfx] [PATCH 1/2] drm/i915/dmc: Make no_stepping_info an array

2016-10-17 Thread Vivi, Rodrigo
Reviewed-by: Rodrigo Vivi On Fri, 2016-10-14 at 16:56 -0700, Anusha Srivatsa wrote: > Make no_stepping_info an array of structs so that > on plaforms that have only one binary of DMC, we can > iterate through this array by having the same logic > for firmware loads > >

Re: [Intel-gfx] [PATCH v4 2/4] drm: Add aspect ratio parsing in DRM layer

2016-10-17 Thread Sharma, Shashank
Regards Shashank On 10/17/2016 8:30 PM, Ville Syrjälä wrote: On Mon, Oct 17, 2016 at 08:21:21PM +0530, Sharma, Shashank wrote: Regards Shashank On 10/17/2016 7:36 PM, Ville Syrjälä wrote: On Mon, Oct 17, 2016 at 07:10:42PM +0530, Sharma, Shashank wrote: Regards Shashank On 10/17/2016

Re: [Intel-gfx] [PATCH] intel-ci: Add gem_exec_suspend/basic-S3/S4-devices to BAT

2016-10-17 Thread Imre Deak
On ma, 2016-10-17 at 18:01 +0300, Saarinen, Jani wrote: > HI, > > -Original Message- > > From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On > > Behalf > > Of Imre Deak > > Sent: Monday, October 17, 2016 5:47 PM > > To: Daniel Vetter > > Cc:

Re: [Intel-gfx] [PATCH] intel-ci: Add gem_exec_suspend/basic-S3/S4-devices to BAT

2016-10-17 Thread Saarinen, Jani
HI, > -Original Message- > From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf > Of Imre Deak > Sent: Monday, October 17, 2016 5:47 PM > To: Daniel Vetter > Cc: intel-gfx@lists.freedesktop.org > Subject: Re: [Intel-gfx] [PATCH] intel-ci: Add

Re: [Intel-gfx] [PATCH v4 2/4] drm: Add aspect ratio parsing in DRM layer

2016-10-17 Thread Ville Syrjälä
On Mon, Oct 17, 2016 at 08:21:21PM +0530, Sharma, Shashank wrote: > Regards > > Shashank > > > On 10/17/2016 7:36 PM, Ville Syrjälä wrote: > > On Mon, Oct 17, 2016 at 07:10:42PM +0530, Sharma, Shashank wrote: > >> Regards > >> > >> Shashank > >> > >> > >> On 10/17/2016 6:01 PM, Ville Syrjälä

Re: [Intel-gfx] [PATCH] intel-ci: Add gem_exec_suspend/basic-S3/S4-devices to BAT

2016-10-17 Thread Jani Nikula
On Mon, 17 Oct 2016, Imre Deak wrote: > On ma, 2016-10-17 at 16:32 +0200, Daniel Vetter wrote: >> On Mon, Oct 17, 2016 at 03:39:04PM +0300, Imre Deak wrote: >> > Add gem_exec_suspend/basic-s3-devices and basic-s4-devices subtests >> > to >> > BAT. At the same time remove

Re: [Intel-gfx] [PATCH v4 2/4] drm: Add aspect ratio parsing in DRM layer

2016-10-17 Thread Sharma, Shashank
Regards Shashank On 10/17/2016 7:36 PM, Ville Syrjälä wrote: On Mon, Oct 17, 2016 at 07:10:42PM +0530, Sharma, Shashank wrote: Regards Shashank On 10/17/2016 6:01 PM, Ville Syrjälä wrote: On Mon, Oct 17, 2016 at 05:34:38PM +0530, Shashank Sharma wrote: Current DRM layer functions don't

Re: [Intel-gfx] [PATCH] intel-ci: Add gem_exec_suspend/basic-S3/S4-devices to BAT

2016-10-17 Thread Imre Deak
On ma, 2016-10-17 at 16:32 +0200, Daniel Vetter wrote: > On Mon, Oct 17, 2016 at 03:39:04PM +0300, Imre Deak wrote: > > Add gem_exec_suspend/basic-s3-devices and basic-s4-devices subtests > > to > > BAT. At the same time remove basic-s4 from the list, which is atm > > implicitly disabled via

Re: [Intel-gfx] [PATCH] intel-ci: Add gem_exec_suspend/basic-S3/S4-devices to BAT

2016-10-17 Thread Daniel Vetter
On Mon, Oct 17, 2016 at 03:39:04PM +0300, Imre Deak wrote: > Add gem_exec_suspend/basic-s3-devices and basic-s4-devices subtests to > BAT. At the same time remove basic-s4 from the list, which is atm > implicitly disabled via HIBERNATION=n in kconfig. We would need at least > basic S4 coverage

Re: [Intel-gfx] [PATCH i-g-t] tests/kms_plane_multiple: CRC based atomic correctness test

2016-10-17 Thread Daniel Vetter
On Mon, Oct 17, 2016 at 02:28:37PM +0300, Mika Kahola wrote: > + for (int i = 0; i < iterations; i++) { > + igt_info("%d/%d: Testing connector %s using pipe %s with %d > planes\n", > + i+1, iterations, igt_output_name(output), > +

Re: [Intel-gfx] [PATCH] RFC drm/i915: Add a sunset clause to GPU hang logging

2016-10-17 Thread Joonas Lahtinen
On ma, 2016-10-17 at 16:10 +0200, Daniel Vetter wrote: > On Mon, Oct 17, 2016 at 03:33:43PM +0300, Joonas Lahtinen wrote: > > Maybe we could even explicitly state that bugs should be reported to > > the distro bugzilla because of running an old kernel? > > Distro's already shut down our warnings

Re: [Intel-gfx] [PATCH igt] lib/igt_aux: Improve documentation for igt_system_suspend_autoresume()

2016-10-17 Thread Daniel Vetter
On Fri, Oct 14, 2016 at 05:51:14PM +0300, Imre Deak wrote: > While at it fix the order of states for consistency. > > Suggested by Daniel. > > Cc: Daniel Vetter > Signed-off-by: Imre Deak Going ocd with docs makes me really happy \o/ And I think it

Re: [Intel-gfx] ✗ Fi.CI.BAT: warning for drm/i915: GMBUS don't need no forcewake

2016-10-17 Thread Ville Syrjälä
On Wed, Oct 12, 2016 at 01:49:20PM -, Patchwork wrote: > == Series Details == > > Series: drm/i915: GMBUS don't need no forcewake > URL : https://patchwork.freedesktop.org/series/13641/ > State : warning > > == Summary == > > Series 13641v1 drm/i915: GMBUS don't need no forcewake >

[Intel-gfx] [PATCH] drm/i915: Restrict pagefault disabling to just around copy_from_user()

2016-10-17 Thread Chris Wilson
When handling execbuf relocations, we play a delicate dance with pagefault. We first try to access the user pages underneath our struct_mutex. However, if those pages were inside a GEM object, we may trigger a pagefault and deadlock as i915_gem_fault() tries to recursively acquire struct_mutex.

Re: [Intel-gfx] [PATCH] RFC drm/i915: Add a sunset clause to GPU hang logging

2016-10-17 Thread Daniel Vetter
On Mon, Oct 17, 2016 at 03:33:43PM +0300, Joonas Lahtinen wrote: > On pe, 2016-10-14 at 14:44 +0100, Chris Wilson wrote: > > If the kernel is old, more than a few releases old, chances are that the > > user is using an old kernel for a good reason, despite there being GPU > > hangs. After 180days

Re: [Intel-gfx] [PATCH 0/2] drm/i915: Suppress underrun reporting around DP link retraining

2016-10-17 Thread Ville Syrjälä
On Fri, Oct 14, 2016 at 08:02:52PM +0300, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > This series should reduce the underrn spam in CI. I expect it won't eliminate > it entirely > as we seem to have two classes of dmesg warns: from link

Re: [Intel-gfx] [PATCH 17/41] drm/i915: Pass around sg_table to get_pages/put_pages backend

2016-10-17 Thread Chris Wilson
On Mon, Oct 17, 2016 at 02:51:01PM +0100, Tvrtko Ursulin wrote: > > On 17/10/2016 12:31, Chris Wilson wrote: > >On Mon, Oct 17, 2016 at 11:55:54AM +0100, Tvrtko Ursulin wrote: > >>On 14/10/2016 13:18, Chris Wilson wrote: > >>> static void > >>>-i915_gem_object_put_pages_gtt(struct

Re: [Intel-gfx] [PULL] GVT-g device model core

2016-10-17 Thread Daniel Vetter
On Mon, Oct 17, 2016 at 03:45:47PM +0800, Zhenyu Wang wrote: > On 2016.10.17 09:33:19 +0200, Daniel Vetter wrote: > > On Mon, Oct 17, 2016 at 09:30:45AM +0200, Daniel Vetter wrote: > > > On Fri, Oct 14, 2016 at 06:30:30PM +0800, Zhenyu Wang wrote: > > > > > > > > Hi, > > > > > > > > This is

Re: [Intel-gfx] [PULL] GVT-g device model core

2016-10-17 Thread Jani Nikula
On Mon, 17 Oct 2016, Daniel Vetter wrote: > Ok applied, but a few things to keep in mind before your next pull > request: > > - Dont rebase everything 5 seconds before sending out the pull request. > That just invalidates all the testing you've done, so not a good idea. > In

Re: [Intel-gfx] [PATCH v4 2/4] drm: Add aspect ratio parsing in DRM layer

2016-10-17 Thread Ville Syrjälä
On Mon, Oct 17, 2016 at 07:10:42PM +0530, Sharma, Shashank wrote: > Regards > > Shashank > > > On 10/17/2016 6:01 PM, Ville Syrjälä wrote: > > On Mon, Oct 17, 2016 at 05:34:38PM +0530, Shashank Sharma wrote: > >> Current DRM layer functions don't parse aspect ratio information > >> while

Re: [Intel-gfx] [PATCH v6 3/5] drm/i915: Parse VBT data for lspcon

2016-10-17 Thread Sharma, Shashank
Regards Shashank On 10/17/2016 6:04 PM, Jani Nikula wrote: On Fri, 14 Oct 2016, "Sharma, Shashank" wrote: Regards Shashank On 10/14/2016 8:02 PM, Jani Nikula wrote: On Fri, 14 Oct 2016, Shashank Sharma wrote: Many GEN9 boards come

Re: [Intel-gfx] [PATCH 17/41] drm/i915: Pass around sg_table to get_pages/put_pages backend

2016-10-17 Thread Tvrtko Ursulin
On 17/10/2016 12:31, Chris Wilson wrote: On Mon, Oct 17, 2016 at 11:55:54AM +0100, Tvrtko Ursulin wrote: On 14/10/2016 13:18, Chris Wilson wrote: static void -i915_gem_object_put_pages_gtt(struct drm_i915_gem_object *obj) +i915_gem_object_put_pages_gtt(struct drm_i915_gem_object *obj, +

Re: [Intel-gfx] [PATCH v4 2/4] drm: Add aspect ratio parsing in DRM layer

2016-10-17 Thread Sharma, Shashank
Regards Shashank On 10/17/2016 6:01 PM, Ville Syrjälä wrote: On Mon, Oct 17, 2016 at 05:34:38PM +0530, Shashank Sharma wrote: Current DRM layer functions don't parse aspect ratio information while converting a user mode->kernel mode or vice versa. This causes modeset to pick mode with wrong

[Intel-gfx] ✗ Fi.CI.BAT: warning for drm/atomic: Use less confusing iterator names.

2016-10-17 Thread Patchwork
== Series Details == Series: drm/atomic: Use less confusing iterator names. URL : https://patchwork.freedesktop.org/series/13869/ State : warning == Summary == Series 13869v1 drm/atomic: Use less confusing iterator names.

Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for Picture aspect ratio support in DRM layer

2016-10-17 Thread Saarinen, Jani
> == Series Details == > > Series: Picture aspect ratio support in DRM layer > URL : https://patchwork.freedesktop.org/series/13868/ > State : failure > > == Summary == > > Series 13868v1 Picture aspect ratio support in DRM layer >

Re: [Intel-gfx] [PATCH v4 0/4] Picture aspect ratio support in DRM layer

2016-10-17 Thread Sharma, Shashank
Regards Shashank On 10/17/2016 5:55 PM, Daniel Vetter wrote: On Mon, Oct 17, 2016 at 05:34:36PM +0530, Shashank Sharma wrote: This patch series adds 4 patches. - The first two patches add aspect ratio support in DRM layes - Next two patches add new aspect ratios defined in CEA-861-F

[Intel-gfx] [drm-intel:drm-intel-next-queued 1/1] drivers/gpu/drm/i915/gvt/handlers.c:137:3: note: in expansion of macro 'if'

2016-10-17 Thread kbuild test robot
tree: git://anongit.freedesktop.org/drm-intel drm-intel-next-queued head: 06a75ace46e2fdd1d93b06228df0e2dfe526cc27 commit: 06a75ace46e2fdd1d93b06228df0e2dfe526cc27 [1/1] Merge tag 'gvt-next-2016-10-14' of https://github.com/01org/gvt-linux into drm-intel-next-queued config:

[Intel-gfx] ✗ Fi.CI.BAT: failure for Picture aspect ratio support in DRM layer

2016-10-17 Thread Patchwork
== Series Details == Series: Picture aspect ratio support in DRM layer URL : https://patchwork.freedesktop.org/series/13868/ State : failure == Summary == Series 13868v1 Picture aspect ratio support in DRM layer https://patchwork.freedesktop.org/api/1.0/series/13868/revisions/1/mbox/ Test

[Intel-gfx] [PATCH] intel-ci: Add gem_exec_suspend/basic-S3/S4-devices to BAT

2016-10-17 Thread Imre Deak
Add gem_exec_suspend/basic-s3-devices and basic-s4-devices subtests to BAT. At the same time remove basic-s4 from the list, which is atm implicitly disabled via HIBERNATION=n in kconfig. We would need at least basic S4 coverage provided by basic-s4-devices, which requires HIBERNATION=y.

Re: [Intel-gfx] [PATCH v4 3/4] video: Add new aspect ratios for HDMI 2.0

2016-10-17 Thread Ville Syrjälä
On Mon, Oct 17, 2016 at 05:34:39PM +0530, Shashank Sharma wrote: > HDMI 2.0/CEA-861-F introduces two new aspect ratios: > - 64:27 > - 256:135 > > This patch adds enumeration for the new aspect ratios > in the existing aspect ratio list. > > Signed-off-by: Shashank Sharma

[Intel-gfx] [PATCH 14/19] drm/mediatek: Use new atomic iterator macros

2016-10-17 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/mediatek/mtk_drm_drv.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/mediatek/mtk_drm_drv.c b/drivers/gpu/drm/mediatek/mtk_drm_drv.c index db61aa5f32ef..414e848d8cbf

[Intel-gfx] [PATCH 03/19] drm/exynos: Use new atomic iterator macros.

2016-10-17 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/exynos/exynos_drm_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/exynos/exynos_drm_drv.c b/drivers/gpu/drm/exynos/exynos_drm_drv.c index 4a21a745c373..fc54f3f0a42c

[Intel-gfx] [PATCH 00/19] drm/atomic: Use less confusing iterator names.

2016-10-17 Thread Maarten Lankhorst
Add for_each_(old,new)_(crtc,connector,plane)_in_state iterators, and re-use for_each_*_in_state to be a iterator with the old and the new state. This will be less confusing than the previous case, in which for_each_*_in_state could refer to the old or the new state. The oldnew macro names are

[Intel-gfx] [PATCH 16/19] drm/i915: Use new atomic iterator macros in fbc

2016-10-17 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/i915/intel_fbc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_fbc.c b/drivers/gpu/drm/i915/intel_fbc.c index faa67624e1ed..0028335fc1bb 100644 ---

[Intel-gfx] [PATCH 17/19] drm/i915: Use new atomic iterator macros in wm code

2016-10-17 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/i915/intel_pm.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c index 2df06b703e3d..163b73b493bf 100644 ---

[Intel-gfx] [PATCH 06/19] drm/atomic: Use new atomic iterator macros.

2016-10-17 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/drm_atomic.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c index 1915ec44f7eb..6672ea93ee73 100644 ---

[Intel-gfx] [PATCH 11/19] drm/omap: Use new atomic iterator macros

2016-10-17 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/omapdrm/omap_drv.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/omapdrm/omap_drv.c b/drivers/gpu/drm/omapdrm/omap_drv.c index 1735c7accf72..74f9519878a2 100644

[Intel-gfx] [PATCH 05/19] drm/atomic: Make add_affected_connectors look at crtc_state.

2016-10-17 Thread Maarten Lankhorst
This kills another dereference of connector->state. connector_mask holds all unchanged connectors at least and any changed connectors are already in state anyway. Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/drm_atomic.c | 11 --- 1 file

[Intel-gfx] [PATCH 10/19] drm/rcar-du: Use new atomic iterator macros.

2016-10-17 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/rcar-du/rcar_du_kms.c | 2 +- drivers/gpu/drm/rcar-du/rcar_du_plane.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/rcar-du/rcar_du_kms.c

[Intel-gfx] [PATCH 01/19] drm/atomic: Add new iterators over all state

2016-10-17 Thread Maarten Lankhorst
Add for_each_(old)(new)_(plane,connector,crtc)_in_state iterators to replace the old for_each_xxx_in_state ones. This is useful for >1 flip depth and getting rid of all xxx->state dereferences. Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/drm_atomic.c

[Intel-gfx] [PATCH 04/19] drm/blend: Use new atomic iterator macros.

2016-10-17 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/drm_blend.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/drm_blend.c b/drivers/gpu/drm/drm_blend.c index 85172a977bf3..70c03eb032fc 100644 ---

[Intel-gfx] [PATCH 08/19] drm/vc4: Use new atomic iterator macros.

2016-10-17 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/vc4/vc4_kms.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/vc4/vc4_kms.c b/drivers/gpu/drm/vc4/vc4_kms.c index f31f72af8551..76cc9b374215 100644 ---

[Intel-gfx] [PATCH 02/19] drm/atmel-hlcdc: Use new atomic iterator macros.

2016-10-17 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c index

[Intel-gfx] [PATCH 19/19] drm/atomic: Rename atomic oldnew iterator

2016-10-17 Thread Maarten Lankhorst
With the old users of for_each_obj_in_state gone, we can rename for_each_oldnew_obj_in_state back to that name. It's slightly less ugly. Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/drm_atomic_helper.c | 34 ++---

[Intel-gfx] [PATCH 09/19] drm/rockchip: Use new atomic iterator macros.

2016-10-17 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c index

[Intel-gfx] [PATCH 12/19] drm/msm: Use new atomic iterator macros

2016-10-17 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/msm/mdp/mdp4/mdp4_kms.c | 4 ++-- drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c | 6 +++--- drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.h | 3 ++- drivers/gpu/drm/msm/mdp/mdp5/mdp5_plane.c | 4 ++--

[Intel-gfx] [PATCH 13/19] drm/imx: Use new atomic iterator macros

2016-10-17 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/imx/imx-drm-core.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/imx/imx-drm-core.c b/drivers/gpu/drm/imx/imx-drm-core.c index 98df09c2b388..d484af773460

[Intel-gfx] [PATCH 07/19] drm/atomic: Fix atomic helpers to use the new iterator macros.

2016-10-17 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/drm_atomic_helper.c | 302 +++- 1 file changed, 157 insertions(+), 145 deletions(-) diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c

[Intel-gfx] [PATCH 15/19] drm/i915: Use new atomic iterator macros in ddi

2016-10-17 Thread Maarten Lankhorst
Also make the function static, it's only used inside intel_ddi.c Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/i915/intel_ddi.c | 4 ++-- drivers/gpu/drm/i915/intel_drv.h | 2 -- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git

[Intel-gfx] [PATCH 18/19] drm/i915: Use new atomic iterator macros in display code

2016-10-17 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/i915/intel_display.c | 156 ++- 1 file changed, 80 insertions(+), 76 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c

Re: [Intel-gfx] [PATCH v6 3/5] drm/i915: Parse VBT data for lspcon

2016-10-17 Thread Jani Nikula
On Fri, 14 Oct 2016, "Sharma, Shashank" wrote: > Regards > > Shashank > > > On 10/14/2016 8:02 PM, Jani Nikula wrote: >> On Fri, 14 Oct 2016, Shashank Sharma wrote: >>> Many GEN9 boards come with on-board lspcon cards. >>> Fot these boards,

Re: [Intel-gfx] [PATCH] RFC drm/i915: Add a sunset clause to GPU hang logging

2016-10-17 Thread Joonas Lahtinen
On pe, 2016-10-14 at 14:44 +0100, Chris Wilson wrote: > If the kernel is old, more than a few releases old, chances are that the > user is using an old kernel for a good reason, despite there being GPU > hangs. After 180days since driver release stop suggesting that they > should send those

Re: [Intel-gfx] [PATCH v4 2/4] drm: Add aspect ratio parsing in DRM layer

2016-10-17 Thread Ville Syrjälä
On Mon, Oct 17, 2016 at 05:34:38PM +0530, Shashank Sharma wrote: > Current DRM layer functions don't parse aspect ratio information > while converting a user mode->kernel mode or vice versa. This > causes modeset to pick mode with wrong aspect ratio, eventually > causing failures in HDMI

Re: [Intel-gfx] [PATCH 08/41] drm/i915: Rearrange i915_wait_request() accounting with callers

2016-10-17 Thread Joonas Lahtinen
On pe, 2016-10-14 at 13:18 +0100, Chris Wilson wrote: > Our low-level wait routine has evolved from our generic wait interface > that handled unlocked, RPS boosting, waits with time tracking. If we > push our GEM fence tracking to use reservation_objects (required for > handling multiple

Re: [Intel-gfx] [PATCH v4 0/4] Picture aspect ratio support in DRM layer

2016-10-17 Thread Daniel Vetter
On Mon, Oct 17, 2016 at 05:34:36PM +0530, Shashank Sharma wrote: > This patch series adds 4 patches. > - The first two patches add aspect ratio support in DRM layes > - Next two patches add new aspect ratios defined in CEA-861-F > supported for HDMI 2.0 4k modes. > > Adding aspect ratio support

Re: [Intel-gfx] [PATCH 06/41] drm/i915: Support asynchronous waits on struct fence from i915_gem_request

2016-10-17 Thread Joonas Lahtinen
On pe, 2016-10-14 at 13:17 +0100, Chris Wilson wrote: > We will need to wait on DMA completion (as signaled via struct fence) > before executing our i915_gem_request. Therefore we want to expose a > method for adding the await on the fence itself to the request. > > v2: Add a comment detailing a

[Intel-gfx] ✗ Fi.CI.BAT: warning for mm/vmalloc: Replace opencoded 4-level page walkers

2016-10-17 Thread Patchwork
== Series Details == Series: mm/vmalloc: Replace opencoded 4-level page walkers URL : https://patchwork.freedesktop.org/series/13822/ State : warning == Summary == Series 13822v1 mm/vmalloc: Replace opencoded 4-level page walkers

Re: [Intel-gfx] [PATCH 25/41] drm/i915: Move GEM activity tracking into a common struct reservation_object

2016-10-17 Thread Joonas Lahtinen
On pe, 2016-10-14 at 13:18 +0100, Chris Wilson wrote: > In preparation to support many distinct timelines, we need to expand the > activity tracking on the GEM object to handle more than just a request > per engine. We already use the struct reservation_object on the dma-buf > to handle many fence

Re: [Intel-gfx] [PATCH 38/41] drm/i915: Defer setting of global seqno on request to submission

2016-10-17 Thread Joonas Lahtinen
On pe, 2016-10-14 at 13:18 +0100, Chris Wilson wrote: > Defer the assignment of the global seqno on a request to its submission. > In the next patch, we will only allocate the global seqno at that time, > here we are just enabling the wait-for-submission before wait-for-seqno > paths. > >

Re: [Intel-gfx] [PATCH 32/41] drm/i915: Rename ->emit_request to ->emit_breadcrumb

2016-10-17 Thread Joonas Lahtinen
On pe, 2016-10-14 at 13:18 +0100, Chris Wilson wrote: > Now that the emission of the request tail and its submission to hardware > are two separate steps, engine->emit_request() is confusing. > engine->emit_request() is called to emit the breadcrumb commands for the > request into the ring, name

Re: [Intel-gfx] ✗ Fi.CI.BAT: warning for : Picture aspect ratio support in DRM layer (rev2)

2016-10-17 Thread Saarinen, Jani
> == Series Details == > > Series: : Picture aspect ratio support in DRM layer (rev2) > URL : https://patchwork.freedesktop.org/series/10587/ > State : warning > > == Summary == > > Series 10587v2 : Picture aspect ratio support in DRM layer >

[Intel-gfx] [PATCH v4 4/4] drm: Add and handle new aspect ratios in DRM layer

2016-10-17 Thread Shashank Sharma
HDMI 2.0/CEA-861-F introduces two new aspect ratios: - 64:27 - 256:135 This patch: - Adds new DRM flags for to represent these new aspect ratios. - Adds new cases to handle these aspect ratios while converting from user->kernel mode or vise versa. Signed-off-by: Shashank Sharma

[Intel-gfx] [PATCH v4 2/4] drm: Add aspect ratio parsing in DRM layer

2016-10-17 Thread Shashank Sharma
Current DRM layer functions don't parse aspect ratio information while converting a user mode->kernel mode or vice versa. This causes modeset to pick mode with wrong aspect ratio, eventually causing failures in HDMI compliance test cases, due to wrong VIC. This patch adds aspect ratio information

[Intel-gfx] [PATCH v4 0/4] Picture aspect ratio support in DRM layer

2016-10-17 Thread Shashank Sharma
This patch series adds 4 patches. - The first two patches add aspect ratio support in DRM layes - Next two patches add new aspect ratios defined in CEA-861-F supported for HDMI 2.0 4k modes. Adding aspect ratio support in DRM layer: - The CEA videmodes contain aspect ratio information, which we

[Intel-gfx] [PATCH v4 3/4] video: Add new aspect ratios for HDMI 2.0

2016-10-17 Thread Shashank Sharma
HDMI 2.0/CEA-861-F introduces two new aspect ratios: - 64:27 - 256:135 This patch adds enumeration for the new aspect ratios in the existing aspect ratio list. Signed-off-by: Shashank Sharma Reviewed-by: Sean Paul Reviewed-by: Jose Abreu

[Intel-gfx] [PATCH v4 1/4] drm: add picture aspect ratio flags

2016-10-17 Thread Shashank Sharma
This patch adds drm flag bits for aspect ratio information Currently drm flag bits don't have field for mode's picture aspect ratio. This field will help the driver to pick mode with right aspect ratio, and help in setting right VIC field in avi infoframes. Signed-off-by: Shashank Sharma

[Intel-gfx] ✗ Fi.CI.BAT: warning for : Picture aspect ratio support in DRM layer (rev2)

2016-10-17 Thread Patchwork
== Series Details == Series: : Picture aspect ratio support in DRM layer (rev2) URL : https://patchwork.freedesktop.org/series/10587/ State : warning == Summary == Series 10587v2 : Picture aspect ratio support in DRM layer

Re: [Intel-gfx] ✗ Fi.CI.BAT: warning for drm/i915: Suppress underrun reporting around DP link retraining

2016-10-17 Thread Ville Syrjälä
On Mon, Oct 17, 2016 at 02:38:01PM +0300, Tomi Sarvela wrote: > On Monday, 17 October 2016 14:33:28 EEST Ville Syrjälä wrote: > > On Fri, Oct 14, 2016 at 06:51:37PM -, Patchwork wrote: > > > == Series Details == > > > > > > Series: drm/i915: Suppress underrun reporting around DP link

Re: [Intel-gfx] [PATCH v3 0/4]: Picture aspect ratio support in DRM layer

2016-10-17 Thread Jose Abreu
Hi Shashank, On 17-10-2016 12:32, Shashank Sharma wrote: > This patch series adds 4 patches. > - The first two patches add aspect ratio support in DRM layes > - Next two patches add new aspect ratios defined in CEA-861-F > supported for HDMI 2.0 4k modes. > > Adding aspect ratio support in DRM

Re: [Intel-gfx] ✗ Fi.CI.BAT: warning for drm/i915: Suppress underrun reporting around DP link retraining

2016-10-17 Thread Tomi Sarvela
On Monday, 17 October 2016 14:33:28 EEST Ville Syrjälä wrote: > On Fri, Oct 14, 2016 at 06:51:37PM -, Patchwork wrote: > > == Series Details == > > > > Series: drm/i915: Suppress underrun reporting around DP link retraining > > URL : https://patchwork.freedesktop.org/series/13805/ > > State

Re: [Intel-gfx] [PATCH v3 0/4]: Picture aspect ratio support in DRM layer

2016-10-17 Thread Sharma, Shashank
Regards Shashank On 10/17/2016 5:01 PM, Jose Abreu wrote: Hi Shashank, On 17-10-2016 12:32, Shashank Sharma wrote: This patch series adds 4 patches. - The first two patches add aspect ratio support in DRM layes - Next two patches add new aspect ratios defined in CEA-861-F supported for

Re: [Intel-gfx] ✗ Fi.CI.BAT: warning for drm/i915: Suppress underrun reporting around DP link retraining

2016-10-17 Thread Ville Syrjälä
On Fri, Oct 14, 2016 at 06:51:37PM -, Patchwork wrote: > == Series Details == > > Series: drm/i915: Suppress underrun reporting around DP link retraining > URL : https://patchwork.freedesktop.org/series/13805/ > State : warning OK, but... > > == Summary == > > Series 13805v1 drm/i915:

[Intel-gfx] [PATCH i-g-t] tests/kms_plane_multiple: CRC based atomic correctness test

2016-10-17 Thread Mika Kahola
This is a testcase with multiple planes. The idea here is the following - draw a uniform frame with blue color - grab crc for reference - put planes randomly on top with the same blue color - punch holes with black color into the primary framebuffer - ideally the planes should cover these

Re: [Intel-gfx] [PATCH 17/41] drm/i915: Pass around sg_table to get_pages/put_pages backend

2016-10-17 Thread Chris Wilson
On Mon, Oct 17, 2016 at 11:55:54AM +0100, Tvrtko Ursulin wrote: > On 14/10/2016 13:18, Chris Wilson wrote: > > static void > >-i915_gem_object_put_pages_gtt(struct drm_i915_gem_object *obj) > >+i915_gem_object_put_pages_gtt(struct drm_i915_gem_object *obj, > >+ struct

Re: [Intel-gfx] [PATCH i-g-t] tests/kms_plane_multiple: CRC based atomic correctness test

2016-10-17 Thread Mika Kahola
On Wed, 2016-10-12 at 14:54 +0200, Maarten Lankhorst wrote: > Op 07-10-16 om 13:45 schreef Mika Kahola: > > > > This is a testcase with multiple planes. The idea here is the > > following > > > >  - draw a uniform frame with blue color > >  - grab crc for reference > >  - put planes randomly on

[Intel-gfx] [PATCH v3 4/4] drm: Add and handle new aspect ratios in DRM layer

2016-10-17 Thread Shashank Sharma
HDMI 2.0/CEA-861-F introduces two new aspect ratios: - 64:27 - 256:135 This patch: - Adds new DRM flags for to represent these new aspect ratios. - Adds new cases to handle these aspect ratios while converting from user->kernel mode or vise versa. Signed-off-by: Shashank Sharma

[Intel-gfx] [PATCH v3 3/4] video: Add new aspect ratios for HDMI 2.0

2016-10-17 Thread Shashank Sharma
HDMI 2.0/CEA-861-F introduces two new aspect ratios: - 64:27 - 256:135 This patch adds enumeration for the new aspect ratios in the existing aspect ratio list. Signed-off-by: Shashank Sharma Reviewed-by: Sean Paul Cc: Daniel Vetter

[Intel-gfx] [PATCH v3 1/4] drm: add picture aspect ratio flags

2016-10-17 Thread Shashank Sharma
This patch adds drm flag bits for aspect ratio information Currently drm flag bits don't have field for mode's picture aspect ratio. This field will help the driver to pick mode with right aspect ratio, and help in setting right VIC field in avi infoframes. Signed-off-by: Shashank Sharma

[Intel-gfx] [PATCH v3 2/4] drm: Add aspect ratio parsing in DRM layer

2016-10-17 Thread Shashank Sharma
Current DRM layer functions don't parse aspect ratio information while converting a user mode->kernel mode or vice versa. This causes modeset to pick mode with wrong aspect ratio, eventually causing failures in HDMI compliance test cases, due to wrong VIC. This patch adds aspect ratio information

[Intel-gfx] [PATCH v3 0/4]: Picture aspect ratio support in DRM layer

2016-10-17 Thread Shashank Sharma
This patch series adds 4 patches. - The first two patches add aspect ratio support in DRM layes - Next two patches add new aspect ratios defined in CEA-861-F supported for HDMI 2.0 4k modes. Adding aspect ratio support in DRM layer: - The CEA videmodes contain aspect ratio information, which we

Re: [Intel-gfx] [PATCH v3 0/4] Picture aspect ratio support in DRM layer

2016-10-17 Thread Sharma, Shashank
Please ignore this series, I mixed up one patch from LSPCON in here. Will send the right series in few minutes. Regards Shashank -Original Message- From: Sharma, Shashank Sent: Monday, October 17, 2016 4:52 PM To: dri-de...@lists.freedesktop.org; jim.br...@linux.intel.com Cc:

[Intel-gfx] [PATCH v3 0/4] Picture aspect ratio support in DRM layer

2016-10-17 Thread Shashank Sharma
This patch series adds 4 patches. - The first two patches add aspect ratio support in DRM layes - Next two patches add new aspect ratios defined in CEA-861-F supported for HDMI 2.0 4k modes. Adding aspect ratio support in DRM layer: - The CEA videmodes contain aspect ratio information, which we

[Intel-gfx] [PATCH v3 1/4] drm/i915: Add lspcon resume function

2016-10-17 Thread Shashank Sharma
As per the software design, we are driving lspcon in PCON mode. But while resuming from suspend, lspcon can go in LS mode (which is its default operating mode on power on) This patch adds a resume function for lspcon, which makes sure its operating in PCON mode, post resume. V2: Address review

  1   2   >