On Tue, Feb 07, 2017 at 06:51:25PM +0100, Maarten Lankhorst wrote:
> Op 07-02-17 om 16:35 schreef Ville Syrjälä:
> > On Tue, Feb 07, 2017 at 05:03:09PM +0200, Martin Peres wrote:
> >> On 07/02/17 15:22, Uwe Kleine-König wrote:
> >>> Hello,
> >>>
> >>> On 02/01/2017 03:37 PM, Ville Syrjälä wrote:
>
On Wed, Feb 01, 2017 at 12:50:26AM +0100, Arthur Heymans wrote:
> This is according to Mobile Intel® 945 Express Chipset
> Family datasheet.
>
> Signed-off-by: Arthur Heymans
Reviewed-by: Ville Syrjälä
and pushed to dinq. Thanks for the patch.
> ---
> drivers/gpu/drm/i915/i915_reg.h |
This allows us to be a little more flexible with how we use
igt_warn_on() and igt_warn_on_f() by allowing us to write statements
like this:
if (igt_warn_on(scary_condition)) {
/* some error handling... */
}
Signed-off-by: Lyude
---
lib/igt_core.h | 25 +++
== Series Details ==
Series: drm/i915/guc: Make intel_guc_send a function pointer (rev2)
URL : https://patchwork.freedesktop.org/series/19024/
State : failure
== Summary ==
Series 19024v2 drm/i915/guc: Make intel_guc_send a function pointer
https://patchwork.freedesktop.org/api/1.0/series/1902
From: Ville Syrjälä
Rename the .get_display_clock_speed() hook to .get_cdclk().
.get_cdclk() is more specific (which clock) and it's much
shorter.
v2: Deal with IS_GEN9_BC()
v3: Deal with i945gm_get_display_clock_speed()
Signed-off-by: Ville Syrjälä
Reviewed-by: Ander Conselvan de Oliveira
Li
From: Ville Syrjälä
Let's clean up the mess we have in the if ladder that assigns the
.get_cdclk() hooks. The grouping of the platforms by the function
results in a thing that's not really legible, so let's do it the
other way around and order the if ladder by platform and duplicate
whatever assi
From: Ville Syrjälä
Let's try to shrink intel_display.c a bit by moving the cdclk/rawclk
stuff to a new file. It's all reasonably self contained so we don't
even have to add that many non-static symbols.
We'll also take the opportunity to shuffle around the functions a bit
to get things in a mor
From: Ville Syrjälä
Introduce intel_cdclk state which for now will track the cdclk
frequency, the vco frequency and the reference frequency (not sure we
want the last one, but I put it there anyway). We'll also make the
.get_cdclk() function fill out this state structure rather than
just returnin
On Thu, 2017-02-02 at 10:44 +0200, Jani Nikula wrote:
> On Wed, 01 Feb 2017, "Pandiyan, Dhinakaran"
> wrote:
> > On Thu, 2017-01-26 at 21:44 +0200, Jani Nikula wrote:
> >> Rename the function, move it at the top, and reuse in
> >> intel_dp_link_rate_index(). If there was a reason in the past to u
We're using non-canonical addresses in drm_mm, and we're making sure that
userspace is using canonical addressing - both in case of softpin
(verifying incoming offset) and when relocating (converting to canonical
when updating offset returned to userspace).
Unfortunately when considering the need f
Hi Jani,
Thanks for this patch series. This definitely makes use
of lane count and link rate cleaner while handling the link failures.
I have tested these patches with compliance device along with my pending
DRM link-status patches and it does the fallback as expected.
It does not solve the proble
This has been tested to correctly execute the compliance request.
Reviewed-by: Manasi Navare
Manasi
On Fri, Feb 03, 2017 at 04:19:35PM +0200, Jani Nikula wrote:
> Localize link_rate_index to the if block, and rename to just index to
> reduce indent.
>
> Cc: Manasi Navare
> Cc: Ville Syrjälä
== Series Details ==
Series: drm/i915: Introduce intel_cdclk_state (rev12)
URL : https://patchwork.freedesktop.org/series/16994/
State : warning
== Summary ==
Series 16994v12 drm/i915: Introduce intel_cdclk_state
https://patchwork.freedesktop.org/api/1.0/series/16994/revisions/12/mbox/
Test k
On Tue, Feb 07, 2017 at 08:55:59PM +0100, Michał Winiarski wrote:
> We're using non-canonical addresses in drm_mm, and we're making sure that
> userspace is using canonical addressing - both in case of softpin
> (verifying incoming offset) and when relocating (converting to canonical
> when updatin
On Tue, Feb 07, 2017 at 01:37:52AM -0800, Oscar Mateo wrote:
>
>
> On 02/07/2017 09:25 AM, Chris Wilson wrote:
> >On Tue, Feb 07, 2017 at 12:55:21AM -0800, Oscar Mateo wrote:
> >>
> >>On 02/02/2017 11:33 PM, Chris Wilson wrote:
> >>>On Thu, Feb 02, 2017 at 07:27:45AM -0800, Oscar Mateo wrote:
> >
== Series Details ==
Series: drm/i915: Always convert incoming exec offsets to non-canonical
URL : https://patchwork.freedesktop.org/series/19259/
State : failure
== Summary ==
Series 19259v1 drm/i915: Always convert incoming exec offsets to non-canonical
https://patchwork.freedesktop.org/api/
Currently we do a reset prepare/finish around the call to reset the GPU,
but it looks like we need a later stage after the hw has been
reinitialised to allow GEM to restart itself. Start by splitting the 2
GEM phases into 3:
prepare - before the reset, check if GEM recovered, then stop GEM
re
Just a simple refactor to move a loop and some support code out of
i915_gem_init_hw(). This is in preparation for avoiding a race between
the tasklet writing to ELSP whilst simultaneously being written by
engine->init_hw() following a GPU reset.
Signed-off-by: Chris Wilson
Cc: Mika Kuoppala
---
When we restart the engines, and we have active requests, a request on
the first engine may complete and queue a request to the second engine
before we try to restart the second engine. That queueing of the
request may race with the engine to restart, and so may corrupt the
current state. Disabling
On Tue, Feb 07, 2017 at 09:12:25PM +, Chris Wilson wrote:
> Currently we do a reset prepare/finish around the call to reset the GPU,
> but it looks like we need a later stage after the hw has been
> reinitialised to allow GEM to restart itself. Start by splitting the 2
> GEM phases into 3:
>
>
On Tue, Feb 07, 2017 at 05:54:10PM +0200, Mika Kuoppala wrote:
> Chris Wilson writes:
>
> > Following a reset, the context and page directory registers are lost.
> > However, the queue of requests that we resubmit after the reset may
> > depend upon them - the registers are restored from a contex
Hi,
Please consider pulling i915 updates to linux-firmware.git.
The following changes since commit 6d3bc8886517d171068fd1263176b8b5c51df204:
Fix permissions on ti-connectivity firmware from 05e9fe59 (2017-01-13
10:14:07 -0500)
are available in the git repository at:
https://github.com/anu
== Series Details ==
Series: series starting with [1/3] drm/i915: Move calling engine->init_hw() to
its own function
URL : https://patchwork.freedesktop.org/series/19263/
State : success
== Summary ==
Series 19263v1 Series without cover letter
https://patchwork.freedesktop.org/api/1.0/series/
Detect_done flag was added previously when intel_dp_long_pulse
was called from multiple places. However now it is called only
from intel_dp_detect(). So this flag becomes redundant.
Anyway now, we set this to false right away after intel_dp_long_pulse
so it calls long pulse handler always for all c
On 7 February 2017 at 14:49, Chris Wilson wrote:
> On Tue, Feb 07, 2017 at 02:38:16PM +, Emil Velikov wrote:
>> On 7 February 2017 at 14:29, Daniel Vetter wrote:
>> > Noticed that everyone duplicates the same logic here and we could safe
>> > a few lines per driver. Yay for lots of drivers to
== Series Details ==
Series: drm/i915/dp: Clean up/Get rid of detect_done flag
URL : https://patchwork.freedesktop.org/series/19276/
State : success
== Summary ==
Series 19276v1 drm/i915/dp: Clean up/Get rid of detect_done flag
https://patchwork.freedesktop.org/api/1.0/series/19276/revisions/1
The max link parameters should be set/reset only on HPD or
connected boot case or on system resume.
Add a flag reset_link_params to intel_dp to decide when
to reset the max link parameters. This prevents the parameters
from getting reset/overwritten through all other
connector->funcs->detect() cal
== Series Details ==
Series: drm/i915/dp: Reset the link params on HPD/connected boot/resume
URL : https://patchwork.freedesktop.org/series/19277/
State : success
== Summary ==
Series 19277v1 drm/i915/dp: Reset the link params on HPD/connected boot/resume
https://patchwork.freedesktop.org/api/
As that is what they are meant to be. It will prevent any confusion if we
have to add other flags in the future.
Signed-off-by: Michel Thierry
---
lib/igt_gt.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/igt_gt.h b/lib/igt_gt.h
index e44b6db1..d83e23a1 100644
---
101 - 129 of 129 matches
Mail list logo
