Re: [Intel-gfx] [PATCH 2/2] drm/i915: fully apply WaSkipStolenMemoryFirstPage

On Wed, Dec 14, 2016 at 05:55:39PM -0200, Paulo Zanoni wrote: > Don't even tell the mm allocator to handle the first page of stolen on > the affected platforms. This means that we won't inherit the FB in > case the BIOS decides to put it at the start of stolen. But the BIOS > should not be putting

Re: [Intel-gfx] [PATCH] drm/i915: tune down the fast link training vs boot fail

On Wed, Dec 14, 2016 at 04:56:33PM +0200, Jani Nikula wrote: > On Wed, 14 Dec 2016, Daniel Vetter wrote: > > On Tue, Dec 13, 2016 at 10:23:05PM +0200, Ville Syrjälä wrote: > >> On Tue, Dec 13, 2016 at 08:54:14PM +0100, Daniel Vetter wrote: > >> > It's been unfixed since a while

Re: [Intel-gfx] [PATCH 1/7] drm/i915: setup bridge for HDMI LPE audio driver

> -Original Message- > From: Daniel Vetter [mailto:daniel.vet...@ffwll.ch] On Behalf Of Daniel > Vetter > Sent: Wednesday, December 14, 2016 6:22 PM > To: Anand, Jerome > Cc: intel-gfx@lists.freedesktop.org; alsa-de...@alsa-project.org; > ti...@suse.de;

Re: [Intel-gfx] [PATCH 1/7] drm/i915: setup bridge for HDMI LPE audio driver

> -Original Message- > From: Takashi Iwai [mailto:ti...@suse.de] > Sent: Wednesday, December 14, 2016 5:13 PM > To: Anand, Jerome > Cc: intel-gfx@lists.freedesktop.org; alsa-de...@alsa-project.org; > ville.syrj...@linux.intel.com; broo...@kernel.org; pierre- >

Re: [Intel-gfx] [alsa-devel] [PATCH 5/7] ALSA: x86: hdmi: Improve position reporting

On 12/14/16 8:36 AM, Takashi Iwai wrote: On Wed, 14 Dec 2016 15:09:20 +0100, Pierre-Louis Bossart wrote: On 12/14/16 6:57 AM, Takashi Iwai wrote: On Mon, 12 Dec 2016 19:10:41 +0100, Jerome Anand wrote: Use a hw register to calculate sub-period position reports. This makes PulseAudio

Re: [Intel-gfx] [PATCH v2 00/37] drm: Deduplicate fb format information (v2)

On Fri, Nov 18, 2016 at 09:52:36PM +0200, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > Second installment of my effort to remove the duplicated > depth/bpp/pixel_format from drm_framebuffer and just use > struct drm_format_info instead. > > I

Re: [Intel-gfx] [PATCH] drm/i915: disable PSR by default on HSW/BDW

Em Ter, 2016-12-13 às 21:22 +, Vivi, Rodrigo escreveu: > Acked-by: Rodrigo Vivi > Reviewed-by: Rodrigo Vivi > > Patch pushed. Thanks everybody. I'll just go ahead and close the bugzilla tickets. Even though we want to fix PSR for real and

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915: skip the first 4k of stolen memory on everything >= gen8 (rev2)

== Series Details == Series: series starting with [1/2] drm/i915: skip the first 4k of stolen memory on everything >= gen8 (rev2) URL : https://patchwork.freedesktop.org/series/16800/ State : success == Summary == Series 16800v2 Series without cover letter

[Intel-gfx] [PATCH 2/2] drm/i915: fully apply WaSkipStolenMemoryFirstPage

Don't even tell the mm allocator to handle the first page of stolen on the affected platforms. This means that we won't inherit the FB in case the BIOS decides to put it at the start of stolen. But the BIOS should not be putting it at the start of stolen since it's going to get corrupted. I

Re: [Intel-gfx] [PATCH] drm/i915: disable PSR by default on HSW/BDW

> > On Wed, 2016-12-14 at 08:00 +, Argotti, Yann wrote: > > > > > > Acked-by: Rodrigo Vivi > > > Reviewed-by: Rodrigo Vivi > > > > > > > > > > > > On Tue, 2016-12-13 at 18:57 -0200, Paulo Zanoni wrote: > > > > We've been ignoring the poor

Re: [Intel-gfx] [PATCH] drm/i915: disable PSR by default on HSW/BDW

On Wed, 2016-12-14 at 08:00 +, Argotti, Yann wrote: > > > > Acked-by: Rodrigo Vivi > > Reviewed-by: Rodrigo Vivi > > > > > > > > On Tue, 2016-12-13 at 18:57 -0200, Paulo Zanoni wrote: > > > We've been ignoring the poor bugzilla reporters

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Prevent PPS stealing from a normal DP port on VLV/CHV (rev2)

== Series Details == Series: drm/i915: Prevent PPS stealing from a normal DP port on VLV/CHV (rev2) URL : https://patchwork.freedesktop.org/series/16698/ State : success == Summary == Series 16698v2 drm/i915: Prevent PPS stealing from a normal DP port on VLV/CHV

[Intel-gfx] [PATCH v2] drm/i915: Prevent PPS stealing from a normal DP port on VLV/CHV

From: Ville Syrjälä VLV apparently gets upset if the PPS for a pipe currently driving an external DP port gets used for VDD stuff on another eDP port. The DP port falls over and fails to retrain when this happens, leaving the user staring at a black screen. Let's

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915: Don't assume 60Hz when accounting for DSI pixel overlap (rev2)

== Series Details == Series: series starting with [1/2] drm/i915: Don't assume 60Hz when accounting for DSI pixel overlap (rev2) URL : https://patchwork.freedesktop.org/series/16760/ State : success == Summary == Series 16760v2 Series without cover letter

[Intel-gfx] [PATCH 2/2] drm/i915: Dump more configuration information for DSI

From: Ville Syrjälä Dump out more of the DSI configuration details during init. This includes pclk, burst_mode_ratio, lane_count, pixel_overlap, video_mode_format and reset_timer_val. v2: Dump more info (Chris) Cc: Chris Wilson

Re: [Intel-gfx] [PATCH 03/13] drm/radeon|amdgpu: Remove redundant num_connectors check

On Tue, Dec 13, 2016 at 6:08 PM, Daniel Vetter wrote: > The list walk will shortcircuit anyway. > > Cc: Alex Deucher > Signed-off-by: Daniel Vetter Reviewed-by: Alex Deucher > --- >

Re: [Intel-gfx] [PATCH i-g-t v2 10/12] tests/kms_atomic_transition: add out_fences tests

On Wed, Dec 14, 2016 at 04:05:07AM -0500, Robert Foss wrote: From: Gustavo Padovan Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- lib/igt_kms.c | 22

Re: [Intel-gfx] [PATCH 2/2] drm/i915: fully apply WaSkipStolenMemoryFirstPage

On Wed, Dec 14, 2016 at 06:39:08PM +0200, Ville Syrjälä wrote: > On Wed, Dec 14, 2016 at 12:55:38PM -0200, Paulo Zanoni wrote: > > Reserve the first page of stolen memory right after initializing the > > mm allocator. This means that we won't inherit the FB in case the BIOS > > decides to put it

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Dump the pclk and burst_mode_ratio for DSI

On Wed, Dec 14, 2016 at 04:24:02PM +, Chris Wilson wrote: > On Tue, Dec 13, 2016 at 11:09:18PM +0200, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > Dump out the computed pclk and burst_mode_ratio for DSI. These are > > pretty important in

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Don't assume 60Hz when accounting for DSI pixel overlap

On Wed, Dec 14, 2016 at 04:22:22PM +, Chris Wilson wrote: > On Tue, Dec 13, 2016 at 11:09:17PM +0200, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > Compute the increase in the DSI clock due to pixel overlap mode in > > a way that's

Re: [Intel-gfx] [PATCH i-g-t v2 08/12] tests/kms_atomic: stress possible fence settings

Hi, On Wed, Dec 14, 2016 at 04:05:05AM -0500, Robert Foss wrote: From: Gustavo Padovan Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- tests/kms_atomic.c | 186

Re: [Intel-gfx] [PATCH 2/2] drm/i915: fully apply WaSkipStolenMemoryFirstPage

On Wed, Dec 14, 2016 at 12:55:38PM -0200, Paulo Zanoni wrote: > Reserve the first page of stolen memory right after initializing the > mm allocator. This means that we won't inherit the FB in case the BIOS > decides to put it at the start of stolen. But the BIOS should not be > putting it at the

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Dump the pclk and burst_mode_ratio for DSI

On Tue, Dec 13, 2016 at 11:09:18PM +0200, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > Dump out the computed pclk and burst_mode_ratio for DSI. These are > pretty important in figuring out timing related stuff. Pixel overlap? I don't see that

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Don't assume 60Hz when accounting for DSI pixel overlap

On Tue, Dec 13, 2016 at 11:09:17PM +0200, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > Compute the increase in the DSI clock due to pixel overlap mode in > a way that's refresh rate agnostic. So far the computation assumed > a 60Hz refresh rate.

Re: [Intel-gfx] ✗ Fi.CI.BAT: warning for drm/i915: Fix setting of boost freq tunable (rev2)

> == Series Details == > > Series: drm/i915: Fix setting of boost freq tunable (rev2) > URL : https://patchwork.freedesktop.org/series/16748/ > State : warning > > == Summary == > > Series 16748v2 drm/i915: Fix setting of boost freq tunable >

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/psr: fix blank screen issue for psr2

== Series Details == Series: drm/i915/psr: fix blank screen issue for psr2 URL : https://patchwork.freedesktop.org/series/16801/ State : success == Summary == Series 16801v1 drm/i915/psr: fix blank screen issue for psr2 https://patchwork.freedesktop.org/api/1.0/series/16801/revisions/1/mbox/

Re: [Intel-gfx] [PATCH i-g-t v2 07/12] lib/igt_kms: Add support for the OUT_FENCE_PTR property

Hi, On Wed, Dec 14, 2016 at 04:05:04AM -0500, Robert Foss wrote: From: Gustavo Padovan Add support for the OUT_FENCE_PTR property to enable setting out fences for atomic commits. Signed-off-by: Gustavo Padovan Signed-off-by:

Re: [Intel-gfx] [PATCH i-g-t v2 06/12] lib/igt_kms: Add support for the IN_FENCE_FD property

Hi, On Wed, Dec 14, 2016 at 04:05:03AM -0500, Robert Foss wrote: Add support dor the IN_FENCE_FD property to enable setting in fences for atomic commits. Signed-off-by: Robert Foss --- lib/igt_kms.c | 20 lib/igt_kms.h | 5 + 2 files changed,

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915: skip the first 4k of stolen memory on everything >= gen8

== Series Details == Series: series starting with [1/2] drm/i915: skip the first 4k of stolen memory on everything >= gen8 URL : https://patchwork.freedesktop.org/series/16800/ State : success == Summary == Series 16800v1 Series without cover letter

Re: [Intel-gfx] [PATCH 1/2] drm/i915: skip the first 4k of stolen memory on everything >= gen8

On Wed, Dec 14, 2016 at 12:55:37PM -0200, Paulo Zanoni wrote: > BSpec got updated and this workaround is now listed as standard > required programming for all subsequent projects. This is confirmed to > fix Skylake screen flickering issues (probably caused by the fact that > we initialized a ring

Re: [Intel-gfx] [PATCH 2/2] drm/i915: fully apply WaSkipStolenMemoryFirstPage

On Wed, Dec 14, 2016 at 12:55:38PM -0200, Paulo Zanoni wrote: > Reserve the first page of stolen memory right after initializing the > mm allocator. This means that we won't inherit the FB in case the BIOS > decides to put it at the start of stolen. But the BIOS should not be > putting it at the

[Intel-gfx] [PATCH] drm/i915/psr: fix blank screen issue for psr2

when psr2 is enabled, psr idle condition is taken from psr1 register(SRD_STATUS) instead of psr2_status register, resulting in looping and blank screen. code changed to lookup from psr2_status and psr2_ctl instead of srd_status and srd_ctl for psr2 scenario. Cc: Rodrigo Vivi

Re: [Intel-gfx] [PATCH 3/8] drm/i915/huc: Add HuC fw loading support

>From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf Of >Jani Nikula >Sent: Wednesday, December 14, 2016 7:20 AM >To: Tvrtko Ursulin ; Srivatsa, Anusha >; intel-gfx@lists.freedesktop.org >Subject: Re: [Intel-gfx]

Re: [Intel-gfx] [PATCH 3/8] drm/i915/huc: Add HuC fw loading support

On Mon, 12 Dec 2016, Tvrtko Ursulin wrote: > Hi all, > > Executive decision required below: > > On 08/12/2016 23:02, anushasr wrote: > > [snip] > >> + >> +/** >> + * DOC: HuC Firmware >> + * >> + * Motivation: >> + * GEN9 introduces a new dedicated firmware for

Re: [Intel-gfx] [PATCH 05/13] drm: locking iterators for connector_list

On Wed, 14 Dec 2016, Daniel Vetter wrote: > On Wed, Dec 14, 2016 at 01:22:15PM +0200, Jani Nikula wrote: >> On Wed, 14 Dec 2016, Daniel Vetter wrote: >> > +/** >> > + * drm_for_each_connector_iter - connector_list iterator macro >> > + * @connector:

Re: [Intel-gfx] [PATCH] drm/i915: tune down the fast link training vs boot fail

On Wed, 14 Dec 2016, Daniel Vetter wrote: > On Tue, Dec 13, 2016 at 10:23:05PM +0200, Ville Syrjälä wrote: >> On Tue, Dec 13, 2016 at 08:54:14PM +0100, Daniel Vetter wrote: >> > It's been unfixed since a while and no one is immediately working on >> > this. And we have the FIXME

[Intel-gfx] [PATCH 2/2] drm/i915: fully apply WaSkipStolenMemoryFirstPage

Reserve the first page of stolen memory right after initializing the mm allocator. This means that we won't inherit the FB in case the BIOS decides to put it at the start of stolen. But the BIOS should not be putting it at the start of stolen since it's going to get corrupted. I suppose the bug

[Intel-gfx] [PATCH 1/2] drm/i915: skip the first 4k of stolen memory on everything >= gen8

BSpec got updated and this workaround is now listed as standard required programming for all subsequent projects. This is confirmed to fix Skylake screen flickering issues (probably caused by the fact that we initialized a ring in the first page of stolen, but I didn't 100% confirm this theory).

Re: [Intel-gfx] [PATCH 10/13] drm/i915: Use drm_connector_list_iter in debugfs

On Wed, Dec 14, 2016 at 3:44 PM, Jani Nikula wrote: > On Wed, 14 Dec 2016, Daniel Vetter wrote: >> diff --git a/drivers/gpu/drm/i915/i915_drv.h >> b/drivers/gpu/drm/i915/i915_drv.h >> index 20bc04d5e617..7e25a7fffb1c 100644 >> ---

Re: [Intel-gfx] [PATCH 10/13] drm/i915: Use drm_connector_list_iter in debugfs

On Wed, 14 Dec 2016, Daniel Vetter wrote: > diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h > index 20bc04d5e617..7e25a7fffb1c 100644 > --- a/drivers/gpu/drm/i915/i915_drv.h > +++ b/drivers/gpu/drm/i915/i915_drv.h > @@ -481,6 +481,9 @@ struct

Re: [Intel-gfx] [PATCH 5/7] ALSA: x86: hdmi: Improve position reporting

On Wed, 14 Dec 2016 15:09:20 +0100, Pierre-Louis Bossart wrote: > > On 12/14/16 6:57 AM, Takashi Iwai wrote: > > On Mon, 12 Dec 2016 19:10:41 +0100, > > Jerome Anand wrote: > >> > >> Use a hw register to calculate sub-period position reports. > >> This makes PulseAudio happier. > >> > >>

Re: [Intel-gfx] [PATCH 5/7] ALSA: x86: hdmi: Improve position reporting

On 12/14/16 6:57 AM, Takashi Iwai wrote: On Mon, 12 Dec 2016 19:10:41 +0100, Jerome Anand wrote: Use a hw register to calculate sub-period position reports. This makes PulseAudio happier. Signed-off-by: David Henningsson Signed-off-by: Pierre-Louis Bossart

[Intel-gfx] [PATCH v13 2/3] drm/i915: Use new CRC debugfs API

The core provides now an ABI to userspace for generation of frame CRCs, so implement the ->set_crc_source() callback and reuse as much code as possible with the previous ABI implementation. When handling the pageflip interrupt, we skip 1 or 2 frames depending on the HW because they contain wrong

[Intel-gfx] [PATCH v13 3/3] drm/i915: Put "cooked" vlank counters in frame CRC lines

Use drm_accurate_vblank_count so we have the full 32 bit to represent the frame counter and userspace has a simpler way of knowing when the counter wraps around. Signed-off-by: Tomeu Vizoso Reviewed-by: Emil Velikov ---

[Intel-gfx] [PATCH v13 0/3] New debugfs API for capturing CRC of frames

Hi, here are the patches that remain to be merged in this series. I have only rebased them since last time. Thanks, Tomeu Tomeu Vizoso (3): drm: Move locking into drm_debugfs_crtc_crc_add drm/i915: Use new CRC debugfs API drm/i915: Put "cooked" vlank counters in frame CRC lines

Re: [Intel-gfx] [PATCH] drm/i915: simplify check for I915G/I945G in bit 6 swizzling detection

On Wed, 14 Dec 2016, Chris Wilson wrote: > On Wed, Dec 14, 2016 at 02:00:11PM +0100, Daniel Vetter wrote: >> On Tue, Dec 13, 2016 at 01:10:59PM +0200, Jani Nikula wrote: >> > c9c4b6f6c283 ("drm/i915: fix swizzle detection for gen3") added a >> > complicated check for

Re: [Intel-gfx] [PATCH] drm/i915: simplify check for I915G/I945G in bit 6 swizzling detection

On Wed, Dec 14, 2016 at 02:00:11PM +0100, Daniel Vetter wrote: > On Tue, Dec 13, 2016 at 01:10:59PM +0200, Jani Nikula wrote: > > c9c4b6f6c283 ("drm/i915: fix swizzle detection for gen3") added a > > complicated check for I915G/I945G. Pineview and other gen3 devices match > > IS_MOBILE() anyway.

Re: [Intel-gfx] [PATCH 2/7] drm/i915: Add support for audio driver notifications

On Wed, 14 Dec 2016 13:55:52 +0100, Daniel Vetter wrote: > > Only noticed it here, but why again do we need to re-roll our intel-only > hdmi/eld notification? The one we have for hda is somewhat justified since > it went in at roughly the same time as the new shared one across a bunch > of soc.

Re: [Intel-gfx] [PATCH 1/7] drm/i915: setup bridge for HDMI LPE audio driver

On Wed, 14 Dec 2016 13:52:27 +0100, Daniel Vetter wrote: > > ... for merging through sound tree. Since we're super early in 4.11 a > topic branch for me to pull in to avoid sync headaches would be good. Sure, I'll branch from some vanilla rc like 4.10-rc1. thanks, Takashi

Re: [Intel-gfx] [PATCH 7/7] ALSA: x86: hdmi: continue playback even when display resolution changes

On Mon, 12 Dec 2016 19:10:43 +0100, Jerome Anand wrote: > > When the display resolution changes, the drm disables the > display pipes due to which audio rendering stops. At this > time, we need to ensure the existing audio pointers and > buffers are cleared out so that the playback can restarted

Re: [Intel-gfx] [PATCH] drm/i915: simplify check for I915G/I945G in bit 6 swizzling detection

On Tue, Dec 13, 2016 at 01:10:59PM +0200, Jani Nikula wrote: > c9c4b6f6c283 ("drm/i915: fix swizzle detection for gen3") added a > complicated check for I915G/I945G. Pineview and other gen3 devices match > IS_MOBILE() anyway. Simplify. > > Cc: Daniel Vetter > Signed-off-by: Jani

Re: [Intel-gfx] [PATCH 6/7] ALSA: x86: hdmi: Fixup some monitor

On Mon, 12 Dec 2016 19:10:42 +0100, Jerome Anand wrote: > > This change was given to Canonical apparently to fix an issue with > on some monitor brand. It's not clear what this patch does but it doesn't > seem to have side effects. > > Signed-off-by: David Henningsson

Re: [Intel-gfx] [PATCH 5/7] ALSA: x86: hdmi: Improve position reporting

On Mon, 12 Dec 2016 19:10:41 +0100, Jerome Anand wrote: > > Use a hw register to calculate sub-period position reports. > This makes PulseAudio happier. > > Signed-off-by: David Henningsson > Signed-off-by: Pierre-Louis Bossart

Re: [Intel-gfx] [PATCH 4/7] ALSA: x86: hdmi: Add audio support for BYT and CHT

On Mon, 12 Dec 2016 19:10:40 +0100, Jerome Anand wrote: > > Hdmi audio driver based on the child platform device > created by gfx driver is implemented. > This audio driver is derived from legacy intel > hdmi audio driver. > > The interfaces for interaction between gfx and audio > are updated

Re: [Intel-gfx] [PATCH 2/7] drm/i915: Add support for audio driver notifications

On Mon, Dec 12, 2016 at 11:40:38PM +0530, Jerome Anand wrote: > Notifiations like mode change, hot plug and edid to > the audio driver are added. This is inturn used by the > audio driver for its functionality. > > A new interface file capturing the notifications needed by the > audio driver is

Re: [Intel-gfx] [PATCH 1/2] drm/i915: skip the first 4k of stolen memory on everything >= gen8

Em Ter, 2016-12-13 às 18:01 -0200, Paulo Zanoni escreveu: > Em Ter, 2016-12-13 às 17:57 -0200, Paulo Zanoni escreveu: > > > > BSpec got updated and this workaround is now listed as standard > > required programming for all subsequent projects. > > Ok, so I got confirmation from the bug reporter

Re: [Intel-gfx] [PATCH 1/7] drm/i915: setup bridge for HDMI LPE audio driver

On Mon, Dec 12, 2016 at 11:40:37PM +0530, Jerome Anand wrote: > Enable support for HDMI LPE audio mode on Baytrail and > Cherrytrail when HDaudio controller is not detected > > Setup minimum required resources during i915_driver_load: > 1. Create a platform device to share MMIO/IRQ resources > 2.

Re: [Intel-gfx] [PATCH 09/34] drm: kselftest for drm_mm_insert_node()

On Wed, Dec 14, 2016 at 02:26:36PM +0200, Joonas Lahtinen wrote: > On ma, 2016-12-12 at 11:53 +, Chris Wilson wrote: > > Exercise drm_mm_insert_node(), check that we can't overfill a range and > > that the lists are correct after reserving/removing. > > > > Signed-off-by: Chris Wilson

[Intel-gfx] ✗ Fi.CI.BAT: warning for drm/i915: Fix setting of boost freq tunable (rev2)

== Series Details == Series: drm/i915: Fix setting of boost freq tunable (rev2) URL : https://patchwork.freedesktop.org/series/16748/ State : warning == Summary == Series 16748v2 drm/i915: Fix setting of boost freq tunable

Re: [Intel-gfx] [PATCH 3/7] ALSA: add shell for Intel HDMI LPE audio driver

On Mon, 12 Dec 2016 19:10:39 +0100, Jerome Anand wrote: > > On Baytrail and Cherrytrail, HDaudio may be fused out or disabled > by the BIOS. This driver enables an alternate path to the i915 > display registers and DMA. > > Although there is no hardware path between i915 display and LPE/SST >

[Intel-gfx] [PATCH v2] drm/i915: Fix setting of boost freq tunable

For limiting the max frequency of gpu, the max freq tunable is not enough to hard limit the max gap. We now have also per client boost max freq. When this tunable was introduced, it was mistakenly made read only. Allow user to gain control by setting it writable. Fixes: 29ecd78d3b79 ("drm/i915:

Re: [Intel-gfx] [PATCH 09/34] drm: kselftest for drm_mm_insert_node()

On ma, 2016-12-12 at 11:53 +, Chris Wilson wrote: > Exercise drm_mm_insert_node(), check that we can't overfill a range and > that the lists are correct after reserving/removing. > > Signed-off-by: Chris Wilson > +static int __igt_insert(int count, u64 size) > +{

Re: [Intel-gfx] [PATCH 05/13] drm: locking iterators for connector_list

On Wed, Dec 14, 2016 at 01:22:15PM +0200, Jani Nikula wrote: > On Wed, 14 Dec 2016, Daniel Vetter wrote: > > +/** > > + * drm_for_each_connector_iter - connector_list iterator macro > > + * @connector: struct _connector pointer used as cursor > > + * @iter: struct

Re: [Intel-gfx] [GLK MIPI DSI V1 1/9] drm/i915/glk: Add new bit fields in MIPI CTRL register

> -Original Message- > From: Nikula, Jani > Sent: Wednesday, December 14, 2016 5:17 PM > To: Chauhan, Madhav ; intel- > g...@lists.freedesktop.org > Cc: Conselvan De Oliveira, Ander ; > Saarinen, Jani

Re: [Intel-gfx] [PATCH 03/34] drm: Add drm_mm_for_each_node_safe()

On ke, 2016-12-14 at 11:39 +, Chris Wilson wrote: > On Tue, Dec 13, 2016 at 11:35:42AM +0200, Joonas Lahtinen wrote: > > > > On ma, 2016-12-12 at 11:53 +, Chris Wilson wrote: > > > > > > +#define __drm_mm_nodes(mm) (&(mm)->head_node.node_list) > > > > Not static inline for type

Re: [Intel-gfx] [PATCH 10/34] drm: kselftest for drm_mm_replace_node()

On ma, 2016-12-12 at 11:53 +, Chris Wilson wrote: > Reuse drm_mm_insert_node() with a temporary node to exercise > drm_mm_replace_node(). We use the previous test in order to exercise the > various lists following replacement. > > Signed-off-by: Chris Wilson >

Re: [Intel-gfx] [PATCH] drm/i915: disable PSR by default on HSW/BDW

On Tue, 13 Dec 2016, Paulo Zanoni wrote: > We've been ignoring the poor bugzilla reporters that say PSR causes > system lockups and all other sorts of problems. The earliest bug > report is from April, so I think we can use the "revert the offending > commit if no fixes

Re: [Intel-gfx] [PATCH 16/34] drm: kselftest for drm_mm and top-down alignment

On ma, 2016-12-12 at 11:53 +, Chris Wilson wrote: > Check that if we request top-down allocation with a particular alignment > from drm_mm_insert_node() that the start of the node matches our > request. > > Signed-off-by: Chris Wilson Assuming magic numbers be gone

Re: [Intel-gfx] [PATCH 2/7] drm/i915: Add support for audio driver notifications

On Mon, 12 Dec 2016 19:10:38 +0100, Jerome Anand wrote: > > Notifiations like mode change, hot plug and edid to > the audio driver are added. This is inturn used by the > audio driver for its functionality. > > A new interface file capturing the notifications needed by the > audio driver is

Re: [Intel-gfx] [GLK MIPI DSI V1 1/9] drm/i915/glk: Add new bit fields in MIPI CTRL register

On Wed, 14 Dec 2016, "Chauhan, Madhav" wrote: > Overall, any bit field/register (added in this series) specific to GLK > should have GLK_ and common to GLK/BXT should have GEN9LP_ ?? It's a long standing convention in the driver to name things after the first platform

Re: [Intel-gfx] [PATCH 1/7] drm/i915: setup bridge for HDMI LPE audio driver

On Mon, 12 Dec 2016 19:10:37 +0100, Jerome Anand wrote: > > --- /dev/null > +++ b/drivers/gpu/drm/i915/intel_lpe_audio.c (snip) > +static struct platform_device* Missing space. > +lpe_audio_platdev_create(struct drm_i915_private *dev_priv) > +{ > + struct drm_device *dev = _priv->drm; > +

Re: [Intel-gfx] [PATCH 03/34] drm: Add drm_mm_for_each_node_safe()

On Tue, Dec 13, 2016 at 11:35:42AM +0200, Joonas Lahtinen wrote: > On ma, 2016-12-12 at 11:53 +, Chris Wilson wrote: > > A complement to drm_mm_for_each_node(), wraps list_for_each_entry_safe() > > for walking the list of nodes safe against removal. > > > > Signed-off-by: Chris Wilson

Re: [Intel-gfx] [PATCH 15/34] drm: kselftest for drm_mm and top-down allocation

On ma, 2016-12-12 at 11:53 +, Chris Wilson wrote: > Check that if we request top-down allocation from drm_mm_insert_node() > we receive the next available hole from the top. > > Signed-off-by: Chris Wilson Reviewed-by: Joonas Lahtinen

Re: [Intel-gfx] [PATCH 05/13] drm: locking iterators for connector_list

On Wed, 14 Dec 2016, Daniel Vetter wrote: > The requirements for connector_list locking are a bit tricky: > - We need to be able to jump over zombie conectors (i.e. with refcount > == 0, but not yet removed from the list). If instead we require that > there's no

Re: [Intel-gfx] [PATCH 0/7] Add support for Legacy HDMI audio drivers

On Mon, 12 Dec 2016 19:10:36 +0100, Jerome Anand wrote: > > Legacy (CherryTrail/ Baytrail) HDMI audio drivers added > > Legacy hdmi audio-Gfx interaction/ interfacing is updated to use > irq chip framework > > Jerome Anand (7): > drm/i915: setup bridge for HDMI LPE audio driver > drm/i915:

Re: [Intel-gfx] [GLK MIPI DSI V1 1/9] drm/i915/glk: Add new bit fields in MIPI CTRL register

> -Original Message- > From: Nikula, Jani > Sent: Tuesday, December 13, 2016 5:07 PM > To: Chauhan, Madhav ; intel- > g...@lists.freedesktop.org > Cc: Conselvan De Oliveira, Ander ; > Saarinen, Jani ;

Re: [Intel-gfx] [PATCH] drm/i915: disable PSR by default on HSW/BDW

Em Qua, 2016-12-14 às 08:00 +, Argotti, Yann escreveu: > > > > > > Acked-by: Rodrigo Vivi > > Reviewed-by: Rodrigo Vivi > > > > > > > > On Tue, 2016-12-13 at 18:57 -0200, Paulo Zanoni wrote: > > > > > > We've been ignoring the poor

Re: [Intel-gfx] [PATCH 08/34] drm: kselftest for drm_mm_reserve_node()

On ma, 2016-12-12 at 11:53 +, Chris Wilson wrote: > Exercise drm_mm_reserve_node(), check that we can't reserve an already > occupied range and that the lists are correct after reserving/removing. > > Signed-off-by: Chris Wilson > @@ -105,6 +108,135 @@ static

Re: [Intel-gfx] [PATCH igt] igt/kms_atomic: Match CRTC harder for special planes

Pushed, thanks. Rob. On 2016-12-13 01:15 PM, Daniel Stone wrote: Our heuristic for finding planes was previously matching the type, and ensuring that the plane was valid for that CRTC. However, VC4 now has primary/cursor planes which can wander multiple CRTCs, so we could pick a PRIMARY plane

Re: [Intel-gfx] [PATCH 04/13] drm: Drop locking cargo-cult from drm_mode_config_init

On 13 December 2016 at 23:08, Daniel Vetter wrote: > This is single-threaded setup code, no need for locks. And anyway, > all properties need to be set up before the driver is registered > anyway, they can't be hot-added. > > Signed-off-by: Daniel Vetter

Re: [Intel-gfx] [PATCH 34/34] drm: kselftest for drm_mm and bottom-up allocation

On ma, 2016-12-12 at 11:53 +, Chris Wilson wrote: > Check that if we request bottom-up allocation from drm_mm_insert_node() > we receive the next available hole from the bottom. > > Signed-off-by: Chris Wilson > @@ -1304,6 +1304,95 @@ static int igt_topdown(void

[Intel-gfx] [PATCH i-g-t v2 12/12] tests/kms_atomic_transition: set out_fence for all crtcs

From: Gustavo Padovan Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- tests/kms_atomic_transition.c | 22 +- 1 file changed, 17 insertions(+), 5 deletions(-) diff

[Intel-gfx] [PATCH i-g-t v2 08/12] tests/kms_atomic: stress possible fence settings

From: Gustavo Padovan Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- tests/kms_atomic.c | 186 ++--- 1 file changed, 176 insertions(+),

[Intel-gfx] [PATCH i-g-t v2 06/12] lib/igt_kms: Add support for the IN_FENCE_FD property

Add support dor the IN_FENCE_FD property to enable setting in fences for atomic commits. Signed-off-by: Robert Foss --- lib/igt_kms.c | 20 lib/igt_kms.h | 5 + 2 files changed, 25 insertions(+) diff --git a/lib/igt_kms.c b/lib/igt_kms.c

[Intel-gfx] [PATCH i-g-t v2 03/12] lib/igt_kms: move igt_kms_get_alt_edid() to the right place

From: Gustavo Padovan Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- lib/igt_kms.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git

[Intel-gfx] [PATCH i-g-t v2 09/12] tests/kms_atomic_transition: add fencing parameter to run_transition_tests

From: Gustavo Padovan Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- tests/kms_atomic_transition.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git

[Intel-gfx] [PATCH i-g-t v2 07/12] lib/igt_kms: Add support for the OUT_FENCE_PTR property

From: Gustavo Padovan Add support for the OUT_FENCE_PTR property to enable setting out fences for atomic commits. Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- lib/igt_kms.c | 21

[Intel-gfx] [PATCH i-g-t v2 10/12] tests/kms_atomic_transition: add out_fences tests

From: Gustavo Padovan Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- lib/igt_kms.c | 22 ++ tests/kms_atomic_transition.c | 32

[Intel-gfx] [PATCH i-g-t v2 04/12] lib/igt_kms: export properties names

From: Gustavo Padovan Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- lib/igt_kms.c | 6 +++--- lib/igt_kms.h | 23 +++ 2 files changed, 26 insertions(+), 3

[Intel-gfx] [PATCH i-g-t v2 01/12] tests/kms_atomic_transition: use igt timeout instead of blocking

From: Gustavo Padovan If the event never arrives we can timeout and end the test. Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- tests/kms_atomic_transition.c | 2 ++ 1 file changed, 2

[Intel-gfx] [PATCH i-g-t v2 05/12] tests/kms_atomic: use global atomic properties definitions

From: Gustavo Padovan Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- tests/kms_atomic.c | 123 - 1 file changed, 37 insertions(+), 86

[Intel-gfx] [PATCH i-g-t v2 00/12] tests/kms_atomic_transition add fence testing

This series adds in/out fence testing to kms_atomic_transition test and makes some minor cleanups. This series can be found here: https://git.collabora.com/cgit/user/robertfoss/intel-gpu-tools.git/log/?h=fences_v2 Gustavo Padovan (11): tests/kms_atomic_transition: use igt timeout instead of

[Intel-gfx] [PATCH i-g-t v2 02/12] tests/kms_atomic_transition: don't assume max pipes

From: Gustavo Padovan Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- tests/kms_atomic_transition.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Intel-gfx] [PATCH i-g-t v2 11/12] tests/kms_atomic_transition: add in_fences tests

From: Gustavo Padovan Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- tests/kms_atomic_transition.c | 95 ++- 1 file changed, 85 insertions(+), 10

Re: [Intel-gfx] [RFC 0/4] Introduce drmfs pseudo filesystem for drm subsystem

On Mon, 2016-12-12 at 07:33 -0800, Alex Deucher wrote: > On Mon, Dec 12, 2016 at 1:14 AM, sourab gupta wrote: > > On Mon, 2016-12-05 at 03:06 -0800, Dhingra, Swati wrote: > >> From: Swati Dhingra > >> > >> Currently, we don't have a stable ABI

Re: [Intel-gfx] [PATCH 05/13] drm: locking iterators for connector_list

On Wed, Dec 14, 2016 at 12:08:06AM +0100, Daniel Vetter wrote: > The requirements for connector_list locking are a bit tricky: > - We need to be able to jump over zombie conectors (i.e. with refcount > == 0, but not yet removed from the list). If instead we require that > there's no zombies on

Re: [Intel-gfx] [PATCH 31/34] drm: Simplify drm_mm scan-list manipulation

On ma, 2016-12-12 at 11:53 +, Chris Wilson wrote: > Since we mandate a strict reverse-order of drm_mm_scan_remove_block() kerneldoc speaks of forward-order, so better update that. > after drm_mm_scan_add_block() we can further simplify the list > manipulations when generating the temporary

Re: [Intel-gfx] [PATCH] drm/i915: disable PSR by default on HSW/BDW

> > Acked-by: Rodrigo Vivi > Reviewed-by: Rodrigo Vivi > > > > On Tue, 2016-12-13 at 18:57 -0200, Paulo Zanoni wrote: > > We've been ignoring the poor bugzilla reporters that say PSR causes > > system lockups and all other sorts of problems.