>
> 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. The earliest bug
> > report is from April, so I thi
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 sc
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
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 which can be used for the purpose of
> >> providing o
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 deletions(-)
diff --git a/tests/kms_atomic_transition.c b/tests/kms_atomic_transition.c
inde
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 a/tests/kms_atomic_transition.c b/tests/kms_atomic_transition.c
index fb8864e8..5575ac18 100644
--- a/tests/kms_at
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 deletions(-)
diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index aa9fd161..8aaff5b8 100644
--- a/lib/igt
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 insertions(+)
diff --git a/tests/kms_atomic_transition.c b/tests/kms_atomic_transition.c
ind
From: Gustavo Padovan
Signed-off-by: Gustavo Padovan
Signed-off-by: Robert Foss
---
tests/kms_atomic.c | 123 -
1 file changed, 37 insertions(+), 86 deletions(-)
diff --git a/tests/kms_atomic.c b/tests/kms_atomic.c
index 1441fdfd..8b648eba 1
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 bl
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 -
lib/igt_kms.h | 3 +++
2 files changed, 23 insertions(+), 1 deletion(-)
d
From: Gustavo Padovan
Signed-off-by: Gustavo Padovan
Signed-off-by: Robert Foss
---
lib/igt_kms.c | 22 ++
tests/kms_atomic_transition.c | 32 ++--
2 files changed, 52 insertions(+), 2 deletions(-)
diff --git a/lib/igt_kms.c b/li
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 a/tests/kms_atomic_transition.c b/tests/kms_atomic_transition.c
index 5575ac18..b7d99975 100644
--- a/t
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
index 8aaff5b8..8ca49d86 100644
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 a/lib/igt_kms.c b/lib/igt_kms.c
index 989704e1..aa9fd161 100644
--- a/lib/igt_kms.c
+++ b/lib
From: Gustavo Padovan
Signed-off-by: Gustavo Padovan
Signed-off-by: Robert Foss
---
tests/kms_atomic.c | 186 ++---
1 file changed, 176 insertions(+), 10 deletions(-)
diff --git a/tests/kms_atomic.c b/tests/kms_atomic.c
index 8b648eba..a557ac60
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 --git a/tests/kms_atomic_transition.c b/tests/kms_atomic_transition.c
index c2ec89d7..d7f599b4 1
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 *ignored)
> > return re
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
Reviewed-by: Daniel Stone
__
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
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 int igt_debug(void *ignored)
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 bugzilla reporters that say PSR
> > > causes
> > > syst
> -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 ; Konduru, Chandra
> ; Shankar, Uma ;
> Mukherjee, Indranil ; Kumar, Shobhit
> ; Deepak, M ;
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: A
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 zombies on the list then the fi
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
Regards, Joonas
--
Joonas Lahtinen
Open Source Technol
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
>
>
>
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 = &dev_priv->drm;
>
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 they were introduced in.
BR
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 added
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 in next iteration.
> +sta
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 are presented within 8 mont
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
> +static int __igt_insert(int
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 checking?
> -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 ; Konduru, Chandra
> ; Shankar, Uma ;
> Mukherjee, Indranil ; Kumar, Shobhit
> ; Deepak, M
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 &drm_connector pointer used as cursor
> > + * @iter: struct &drm_connector_list_iter
> >
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)
> +{
> + for (n = 0; n <
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: De
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
> aud
== 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
https://patchwork.freedesktop.org/api/1.0/series/16748/revisions/2/mbox
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
>
>
>
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.
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
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 ad
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 and
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
> Signed-off-by: Jerome Anand
There is no big merit to keep this
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
> Signed-off-by: Pierre-Lou
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 Nikula
> ---
>
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
>
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
_
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. Bu
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. Si
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 I915G/I945G. Pineview and othe
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
---
drivers/gpu/drm/i915/i915_irq.c | 6 +++---
1 file changed, 3 insertions(+
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
driver
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 v
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
Signed-off-by: Jerome Anand
Ther
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.
> >>
> >> Signed-of
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 i915_hotplug {
>
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
>> --- a/drivers/gpu/drm/i915/i915_drv.h
>> +++ b/drivers/gpu/drm/i915/i
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 already. And now a
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 her
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).
v
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: struct &drm_connector pointer used as cursor
>>
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 usage in media HEVC (High
>> + * Eff
>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] [PATCH 3/8] drm/i915/huc: Add HuC fw loading support
>
>On Mon,
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
Cc: Jim Bride
Signed
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 st
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 i
== 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
https://patchwork.freedesktop.org/api/1.0/s
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, 25 insertions(+)
diff --gi
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: Robert Foss
---
lib/igt_kms.c | 21 -
lib/igt_k
== 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/
> == 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
> https://patchwork.freedesktop.org/api/1.0/series/16
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. And let's switch to round to clos
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 being dumped anyway and seems relev
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 st
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 ++---
1 file changed, 176 insertions(+), 10 deletions(-)
diff --git a
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 refresh rate agnostic. So far the comp
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 figuring out timing related stu
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 at
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 ++
tests/kms_atomic_transition.c | 32 ++--
2 files changed, 52 in
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
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_irq.c | 6 ++
> drivers/gpu/drm/radeon/radeon_irq_kms.c | 12 --
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
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/intel_dsi_
== 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
https://patchwork.freedesktop.org/api
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 fix it by also tracking which pi
== 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
https://patchwork.freedesktop.o
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 that say PSR causes
> > > system lockups and all ot
>
> 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 that say PSR
> > > > causes system
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 suppose
== 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
https://patchwork.freedesktop.org/ap
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 those bugs could be useful as a documentation
of w
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 tried to address all of the review
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 happier.
> -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-
> louis.boss...@linux.intel.c
> -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; broo...@kernel.org; Ughreja, Rakes
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 and no one is immed
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
98 matches
Mail list logo
