On Thu, Dec 10, 2015 at 10:15:41AM +0530, Sonika Jindal wrote:
> From: Shashank Sharma
>
> This patch adds a separate probe function for HDMI
> EDID read over DDC channel. This function has been
> registered as a .hot_plug handler for HDMI encoder.
>
> The current implementation of hdmi_detect()
On Mon, Nov 23, 2015 at 10:25:28AM +0100, Maarten Lankhorst wrote:
> Op 13-11-15 om 14:28 schreef Ander Conselvan De Oliveira:
> > On Thu, 2015-11-12 at 14:58 +0100, Maarten Lankhorst wrote:
> >> Op 12-11-15 om 14:37 schreef Ander Conselvan De Oliveira:
> >>> On Wed, 2015-11-11 at 15:36 +0100, Maar
On Tue, Dec 08, 2015 at 01:30:50PM +, Dave Gordon wrote:
> Way back at the beginning of October, Chris Wilson suggested that cleaning
> up these macros by removing the redundant 'temp' might be worthwhile. So
> here's the patch.
>
> There's one more thing that might be cleaned up here (but for
Sure, I will start a new thread for this.
Regarding the regression, since it is a sandybridge system, I am not sure if I
can debug it anyhow because of the unavailability of the system.
For the live status check, we earlier suggested to use platforms gen8 and above.
But for wider testing, we remo
On Tue, Dec 08, 2015 at 01:30:51PM +, Dave Gordon wrote:
> All of these iterator macros require a 'temp' argument, used merely to
> hold internal partial results. We can instead declare the temporary
> variable inside the macro, so the caller need not provide it.
>
> Some of the old code conta
On Mon, Dec 07, 2015 at 01:43:28PM +, Tvrtko Ursulin wrote:
> On 11/11/15 10:36, ankitprasad.r.sha...@intel.com wrote:
> >From: Ankitprasad Sharma
> >
> >Extend the drm_i915_gem_create structure to add support for
> >creating Stolen memory backed objects. Added a new flag through
> >which user
On Tue, Dec 08, 2015 at 04:14:58PM +0200, Ville Syrjälä wrote:
> On Wed, Nov 25, 2015 at 11:44:48AM +0200, Ander Conselvan De Oliveira wrote:
> > On Thu, 2015-11-19 at 16:07 +0100, Maarten Lankhorst wrote:
> > > wait_vblank is already set in intel_plane_atomic_calc_changes
> > > for broadwell, wait
On Tue, Dec 08, 2015 at 01:20:39PM +0200, Jani Nikula wrote:
> On Mon, 07 Dec 2015, Nabendu Maiti wrote:
> > On 11/26/2015 11:33 PM, Nabendu Maiti wrote:
> >>
> >>
> >> On 11/18/2015 10:56 PM, Ville Syrjälä wrote:
> >>> On Wed, Nov 18, 2015 at 10:33:55PM +0530, Maiti, Nabendu Bikash wrote:
>
On Mon, Dec 07, 2015 at 12:04:18PM +, Dave Gordon wrote:
> On 07/12/15 08:29, Daniel Vetter wrote:
> >On Fri, Dec 04, 2015 at 05:28:29PM +, Dave Gordon wrote:
> >>On 04/12/15 09:57, Daniel Vetter wrote:
> >>>On Tue, Dec 01, 2015 at 01:21:07PM +, Dave Gordon wrote:
> On 01/12/15 13:0
On Thu, Dec 10, 2015 at 08:35:30AM +, Jindal, Sonika wrote:
> Sure, I will start a new thread for this.
>
> Regarding the regression, since it is a sandybridge system, I am not
> sure if I can debug it anyhow because of the unavailability of the
> system. For the live status check, we earlier
On Mon, Dec 07, 2015 at 12:51:49PM +, Dave Gordon wrote:
> I think I missed i915_gem_phys_pwrite().
>
> i915_gem_gtt_pwrite_fast() marks the object dirty for most cases (vit
> set_to_gtt_domain(), but isn't called for all cases (or can return before
> the set_domain). Then we try i915_gem_shme
On Tue, Dec 08, 2015 at 04:01:20PM +0800, Libin Yang wrote:
> Hi all,
>
> Any comments on the patches?
Sorry, simply fell through the cracks since Ander is on vacation. Takashi
is working on some cleanup patches to have a port->encoder mapping for the
audio side of i915. His patch cleans up all t
On Mon, Dec 07, 2015 at 02:04:51PM +, Zanoni, Paulo R wrote:
> Em Sex, 2015-12-04 às 16:28 +0100, Daniel Vetter escreveu:
> > On Wed, Dec 02, 2015 at 10:47:01AM -0200, Paulo Zanoni wrote:
> > > We want to make sure that both tiled and untiled buffers have the
> > > same
> > > size for the same
By "we" I meant me :)
Also, since it isn't possible to check all the platforms for live status, it is
safer to put a gen check to platforms which we are sure.
Anyways, lets see if increasing the delay helps there, then you can take a call.
Regards,
Sonika
-Original Message-
From: Daniel
On Mon, Dec 07, 2015 at 03:12:15PM +, Thomas Wood wrote:
> v2: generate the .gitignore file with the default target
>
> Signed-off-by: Thomas Wood
Hm, committing files that the build system updates is generally a bad
idea. I think if we do this, we should remove the .gitignore here. Not
sure
On Wed, Dec 09, 2015 at 04:34:54PM +0200, Ville Syrjälä wrote:
> On Wed, Dec 09, 2015 at 05:05:52PM +0530, Deepak M wrote:
> > From: Gaurav K Singh
> >
> > Before sending TURN ON packet,check the DPI
> > FIFO empty status.
> >
> > v2: Change in commit message
> > Checking for FIFO empty onl
>
>
>-Original Message-
>From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf Of
>Daniel Vetter
>Sent: Thursday, December 10, 2015 9:07 AM
>To: Wood, Thomas
>Cc: intel-gfx@lists.freedesktop.org
>Subject: Re: [Intel-gfx] [PATCH i-g-t v2] tests: update .gitignore and ad
On 12/09/2015 09:35 PM, Ville Syrjälä wrote:
On Wed, Dec 09, 2015 at 08:59:26PM +0530, Shobhit Kumar wrote:
On Wed, Dec 9, 2015 at 8:34 PM, Chris Wilson wrote:
On Wed, Dec 09, 2015 at 08:07:10PM +0530, Shobhit Kumar wrote:
On Wed, Dec 9, 2015 at 7:27 PM, Ville Syrjälä
wrote:
On Wed, Dec 09,
On Fri, Dec 04, 2015 at 11:09:52AM -0500, Johannes Weiner wrote:
> On Fri, Dec 04, 2015 at 03:58:53PM +, Chris Wilson wrote:
> > Some modules, like i915.ko, use swappable objects and may try to swap
> > them out under memory pressure (via the shrinker). Before doing so, they
> > want to check u
On Wed, 2015-12-09 at 16:19 +, Chris Wilson wrote:
> On Wed, Dec 09, 2015 at 04:10:03PM +0200, Imre Deak wrote:
> > On ke, 2015-12-09 at 15:56 +0200, Joonas Lahtinen wrote:
> > > In order to avoid accessing GPU registers while GPU is suspended
> > > cancel
> > > the hangcheck work before callin
On Fri, Dec 04, 2015 at 03:58:54PM +, Chris Wilson wrote:
> If the system has no available swap pages, we cannot make forward
> progress in the shrinker by releasing active pages, only by releasing
> purgeable pages which are immediately reaped. Take total_swap_pages into
> account when countin
On Tue, Dec 08, 2015 at 06:42:09PM +0100, Takashi Iwai wrote:
> On Mon, 07 Dec 2015 10:41:51 +0100,
> Takashi Iwai wrote:
> >
> > On Mon, 07 Dec 2015 09:44:45 +0100,
> > Daniel Vetter wrote:
> > >
> > > On Fri, Dec 04, 2015 at 06:12:47PM +0100, Takashi Iwai wrote:
> > > > This patch adds a revers
On Tue, Dec 08, 2015 at 07:32:35AM -0800, Rodrigo Vivi wrote:
> On Tue, Dec 8, 2015 at 2:45 AM, Daniel Vetter wrote:
> > On Mon, Dec 07, 2015 at 02:06:50AM -0800, Rodrigo Vivi wrote:
> >> It takes from 2 to 5 seconds to run.
> >>
> >> Cc: Daniel Vetter
> >> Signed-off-by: Rodrigo Vivi
> >> ---
>
Hello,
Is there any special reason this has not been merged yet?
Regards
Tomas
On Tue, Dec 1, 2015 at 6:07 AM, Jani Nikula
wrote:
> On Mon, 23 Nov 2015, Maarten Lankhorst
> wrote:
> > Op 13-11-15 om 14:28 schreef Ander Conselvan De Oliveira:
> >> On Thu, 2015-11-12 at 14:58 +0100, Maarten La
Hi,
Two more comments below:
On 09/12/15 12:46, ankitprasad.r.sha...@intel.com wrote:
From: Chris Wilson
Ville reminded us that stolen memory is not preserved across
hibernation, and a result of this was that context objects now being
allocated from stolen were being corrupted on S4 and promp
On Thu, 2015-12-10 at 01:58 +0100, Rafael J. Wysocki wrote:
> On Wednesday, December 09, 2015 06:22:19 PM Joonas Lahtinen wrote:
> > Introduce pm_runtime_get_noidle to for situations where it is not
> > desireable to touch an idling device. One use scenario is periodic
> > hangchecks performed by t
On Wed, 09 Dec 2015, Ville Syrjälä wrote:
> On Wed, Dec 09, 2015 at 12:29:35PM -0800, Wayne Boyer wrote:
>> The cherryview device shares many characteristics with the valleyview
>> device. When support was added to the driver for cherryview, the
>> corresponding device info structure included .is
On Tue, Dec 08, 2015 at 01:48:23PM +0200, Ville Syrjälä wrote:
> On Mon, Dec 07, 2015 at 03:02:42PM -0800, Wayne Boyer wrote:
> > Do some further clean up based on the initial review of
> > drm/i915: Separate cherryview from valleyview.
> >
> > In this case remove the BUG_ON call in vlv_enable_pll
On Thu, 10 Dec 2015 10:38:14 +0100,
Daniel Vetter wrote:
>
> On Tue, Dec 08, 2015 at 06:42:09PM +0100, Takashi Iwai wrote:
> > On Mon, 07 Dec 2015 10:41:51 +0100,
> > Takashi Iwai wrote:
> > >
> > > On Mon, 07 Dec 2015 09:44:45 +0100,
> > > Daniel Vetter wrote:
> > > >
> > > > On Fri, Dec 04, 20
On Thu, Dec 10, 2015 at 03:01:02PM +0530, Kumar, Shobhit wrote:
> On 12/09/2015 09:35 PM, Ville Syrjälä wrote:
> >On Wed, Dec 09, 2015 at 08:59:26PM +0530, Shobhit Kumar wrote:
> >>On Wed, Dec 9, 2015 at 8:34 PM, Chris Wilson
> >>wrote:
> >>>On Wed, Dec 09, 2015 at 08:07:10PM +0530, Shobhit Kumar
On Mon, Dec 07, 2015 at 11:13:20PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> Atomic changes broke check_digital_port_conflicts(). It needs to look
> at the global situation instead of just trying to find a conflict
> within the current atomic state.
>
> This bug made
On Thu, Dec 10, 2015 at 10:47:50AM +0100, Takashi Iwai wrote:
> On Thu, 10 Dec 2015 10:38:14 +0100,
> Daniel Vetter wrote:
> >
> > On Tue, Dec 08, 2015 at 06:42:09PM +0100, Takashi Iwai wrote:
> > > On Mon, 07 Dec 2015 10:41:51 +0100,
> > > Takashi Iwai wrote:
> > > >
> > > > On Mon, 07 Dec 2015
On Tue, Dec 08, 2015 at 05:05:12PM -0200, Paulo Zanoni wrote:
> 2015-12-08 11:42 GMT-02:00 Ville Syrjälä :
> > On Tue, Dec 08, 2015 at 10:50:39AM +0200, David Weinehall wrote:
> >> Since the defaults for some external power management related settings
> >> prevents us from testing our power managem
On Tue, Dec 08, 2015 at 12:44:44PM +, Derek Morton wrote:
> gem_flink_race and prime_self_import have subtests which read the
> number of open gem objects from debugfs to determine if objects have
> leaked during the test. However the test can fail sporadically if
> the number of gem objects ch
On Tue, Dec 08, 2015 at 04:47:55PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> Bspec is confused w.r.t. the HSW/BDW FDI disable sequence. It lists
> FDI RX disable both as step 13 and step 18 in the sequence. But I dug
> up an old BUN mail from Art that moved the FDI RX
On Tue, Dec 08, 2015 at 03:52:56PM +0100, Maarten Lankhorst wrote:
> This fixes a spurious warning from an integer overflow on 64-bits systems.
> The function may return MAX_SCHEDULE_TIMEOUT which gets truncated to -1.
>
> Explicitly handling this by casting to lret fixes it.
> Signed-off-by: Maar
On 12/10/2015 03:20 PM, Daniel Vetter wrote:
On Thu, Dec 10, 2015 at 03:01:02PM +0530, Kumar, Shobhit wrote:
On 12/09/2015 09:35 PM, Ville Syrjälä wrote:
On Wed, Dec 09, 2015 at 08:59:26PM +0530, Shobhit Kumar wrote:
On Wed, Dec 9, 2015 at 8:34 PM, Chris Wilson wrote:
On Wed, Dec 09, 2015 at
Chris Wilson writes:
> Following a GPU reset, we may leave the context in a poorly defined
> state, and reloading from that context will leave the GPU flummoxed. For
> secondary contexts, this will lead to that context being banned - but
> currently it is also causing the default context to becom
On Wed, 2015-12-09 at 13:26 +, Dave Gordon wrote:
> On 09/12/15 12:46, ankitprasad.r.sha...@intel.com wrote:
> > From: Ankitprasad Sharma
> >
> > This patch adds support for clearing buffer objects via CPU/GTT. This
> > is particularly useful for clearing out the non shmem backed objects.
> >
On Tue, Dec 08, 2015 at 03:02:36PM +, Dave Gordon wrote:
> Based on Chris Wilson's patch from 6 months ago, rebased and adapted.
>
> The current implementation of intel_ring_initialized() is too heavyweight;
> it's a non-inlined function that chases several levels of pointers. This
> wouldn't
Describe the intent of boosting the GPU frequency to maximum before
waiting on the GPU.
RPS waitboosting was introduced with
commit b29c19b645287f7062e17d70fa4e9781a01a5d88
Author: Chris Wilson
Date: Wed Sep 25 17:34:56 2013 +0100
drm/i915: Boost RPS frequency for CPU stalls
but lacked a
On Wed, 2015-12-09 at 13:57 +, Tvrtko Ursulin wrote:
> On 09/12/15 12:46, ankitprasad.r.sha...@intel.com wrote:
> > From: Ankitprasad Sharma
> >
> > This patch adds support for clearing buffer objects via CPU/GTT. This
> > is particularly useful for clearing out the non shmem backed objects.
>
On Thu, Dec 10, 2015 at 12:19:29PM +0200, Mika Kuoppala wrote:
> Chris Wilson writes:
>
> > Following a GPU reset, we may leave the context in a poorly defined
> > state, and reloading from that context will leave the GPU flummoxed. For
> > secondary contexts, this will lead to that context being
On Wed, 2015-12-09 at 13:57 +, Chris Wilson wrote:
> On Wed, Dec 09, 2015 at 06:16:17PM +0530, ankitprasad.r.sha...@intel.com
> wrote:
> > From: Ankitprasad Sharma
> >
> > This patch adds support for clearing buffer objects via CPU/GTT. This
> > is particularly useful for clearing out the no
On Tue, Dec 08, 2015 at 06:06:42PM +, Dave Gordon wrote:
> On 08/12/15 17:00, Chris Wilson wrote:
> >On Tue, Dec 08, 2015 at 04:51:16PM +, Dave Gordon wrote:
> >>In various places, a GEM object is filled with data by means of CPU
> >>writes. In such cases, the object should be marked dirty,
On 12/10/2015 3:20 PM, Daniel Vetter wrote:
On Thu, Dec 10, 2015 at 03:01:02PM +0530, Kumar, Shobhit wrote:
On 12/09/2015 09:35 PM, Ville Syrjälä wrote:
On Wed, Dec 09, 2015 at 08:59:26PM +0530, Shobhit Kumar wrote:
On Wed, Dec 9, 2015 at 8:34 PM, Chris Wilson wrote:
On Wed, Dec 09, 2015 a
Hi Jani,
I have a question below.
> -Original Message-
> From: Nikula, Jani
> Sent: Friday, November 20, 2015 3:04 PM
> To: Adebisi, YetundeX; intel-gfx@lists.freedesktop.org
> Cc: Adebisi, YetundeX; Paauwe, Bob J
> Subject: Re: [PATCH] drm/i915: Add Backlight Control using DPCD for eDP
>
On Wed, 2015-12-09 at 16:15 +, Tvrtko Ursulin wrote:
> Hi,
>
> On 09/12/15 12:46, ankitprasad.r.sha...@intel.com wrote:
> > From: Ankitprasad Sharma
> >
> > This patch adds support for extending the pread/pwrite functionality
> > for objects not backed by shmem. The access will be made throug
On Thu, 2015-12-10 at 16:24 +0530, Ankitprasad Sharma wrote:
Missed Chris in last mail, adding him
On Wed, 2015-12-09 at 16:15 +, Tvrtko Ursulin wrote:
> Hi,
>
> On 09/12/15 12:46, ankitprasad.r.sha...@intel.com wrote:
> > From: Ankitprasad Sharma
> >
> > This patch adds support for extending
On Wed, 2015-12-09 at 19:39 +, Dave Gordon wrote:
> On 09/12/15 16:15, Tvrtko Ursulin wrote:
> >
> > Hi,
> >
> > On 09/12/15 12:46, ankitprasad.r.sha...@intel.com wrote:
> >> From: Ankitprasad Sharma
> >>
> >> This patch adds support for extending the pread/pwrite functionality
> >> for object
On Thu, Dec 10, 2015 at 12:02:55AM +0200, Imre Deak wrote:
> On Wed, 2015-12-09 at 20:52 +, Chris Wilson wrote:
> > On Wed, Dec 09, 2015 at 07:47:29PM +0200, Imre Deak wrote:
> > > > void gen6_rps_idle(struct drm_i915_private *dev_priv)
> > > > {
> > > > - struct drm_device *dev = dev_p
On Thu, 10 Dec 2015, Daniel Vetter wrote:
> On Mon, Dec 07, 2015 at 11:13:20PM +0200, ville.syrj...@linux.intel.com wrote:
>> From: Ville Syrjälä
>>
>> Atomic changes broke check_digital_port_conflicts(). It needs to look
>> at the global situation instead of just trying to find a conflict
>> wi
>
>
>-Original Message-
>From: Daniel Vetter [mailto:daniel.vet...@ffwll.ch] On Behalf Of Daniel Vetter
>Sent: Thursday, December 10, 2015 10:13 AM
>To: Morton, Derek J
>Cc: intel-gfx@lists.freedesktop.org; Wood, Thomas
>Subject: Re: [Intel-gfx] [PATCH i-g-t] gem_flink_race/prime_self_impor
On Wed, Dec 09, 2015 at 09:30:42AM -0800, Jesse Barnes wrote:
> On 12/09/2015 09:10 AM, Chris Wilson wrote:
> > Make sure that the RPS bottom-half is flushed before we set the idle
> > frequency when we decide the GPU is idle. This should prevent any races
> > with the bottom-half and setting the i
On 25/11/15 09:23, Daniel Vetter wrote:
On Tue, Nov 24, 2015 at 11:47:26PM +, Chris Wilson wrote:
On Tue, Nov 24, 2015 at 10:26:01PM +, Chris Wilson wrote:
On Tue, Nov 24, 2015 at 07:36:25PM +0200, Jani Nikula wrote:
/* Iterate over initialised rings */
#define for_each_ring(ring__
On Thu, Dec 10, 2015 at 12:32:03PM +, Dave Gordon wrote:
> On 25/11/15 09:23, Daniel Vetter wrote:
> >On Tue, Nov 24, 2015 at 11:47:26PM +, Chris Wilson wrote:
> >>On Tue, Nov 24, 2015 at 10:26:01PM +, Chris Wilson wrote:
> >>>On Tue, Nov 24, 2015 at 07:36:25PM +0200, Jani Nikula wrote:
On Thu, 10 Dec 2015, Daniel Vetter wrote:
> On Mon, Nov 23, 2015 at 10:25:28AM +0100, Maarten Lankhorst wrote:
>> Op 13-11-15 om 14:28 schreef Ander Conselvan De Oliveira:
>> > On Thu, 2015-11-12 at 14:58 +0100, Maarten Lankhorst wrote:
>> >> Op 12-11-15 om 14:37 schreef Ander Conselvan De Oliveir
On Fri, Dec 04, 2015 at 04:20:43PM +0100, Michał Winiarski wrote:
> According to bspec, some parts of HW expect the addresses to be in
> a canonical form, where bits [63:48] == [47]. Let's convert addresses to
> canonical form prior to relocating and return converted offsets to
> userspace.
>
> v2
On Thu, Dec 10, 2015 at 11:51:29AM +, Morton, Derek J wrote:
> >
> >
> >-Original Message-
> >From: Daniel Vetter [mailto:daniel.vet...@ffwll.ch] On Behalf Of Daniel
> >Vetter
> >Sent: Thursday, December 10, 2015 10:13 AM
> >To: Morton, Derek J
> >Cc: intel-gfx@lists.freedesktop.org; W
On Tue, Dec 08, 2015 at 06:24:40PM +, Dave Gordon wrote:
> On 08/12/15 17:03, Chris Wilson wrote:
> >On Tue, Dec 08, 2015 at 04:51:18PM +, Dave Gordon wrote:
> >>This patch covers a couple more places where a GEM object is (or may be)
> >>modified by means of CPU writes, and should therefor
On Thu, Dec 10, 2015 at 03:01:02PM +0530, Kumar, Shobhit wrote:
> On 12/09/2015 09:35 PM, Ville Syrjälä wrote:
> > On Wed, Dec 09, 2015 at 08:59:26PM +0530, Shobhit Kumar wrote:
> >> On Wed, Dec 9, 2015 at 8:34 PM, Chris Wilson
> >> wrote:
> >>> On Wed, Dec 09, 2015 at 08:07:10PM +0530, Shobhit K
Hi Daniel,
please pull the get_eld op addition from:
git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git
tags/drm-i915-get-eld
The topmost commit is 0bdf5a05647a66dcc6394986e061daeac9b1cf96
Add get_eld audio componen
On Tue, Dec 08, 2015 at 07:59:36PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> We're supposed to pass the primary DP encoder to intel_ddi_clk_select(),
> not the fake MST encoder. Do so.
>
> There's no real bug here though, since intel_ddi_clk_select() only
> checks if
On Thu, Dec 10, 2015 at 01:39:47PM +0200, Jani Nikula wrote:
> On Thu, 10 Dec 2015, Daniel Vetter wrote:
> > On Mon, Dec 07, 2015 at 11:13:20PM +0200, ville.syrj...@linux.intel.com
> > wrote:
> >> From: Ville Syrjälä
> >>
> >> Atomic changes broke check_digital_port_conflicts(). It needs to loo
On Tue, Dec 08, 2015 at 07:59:37PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> DDI A and E share some of the lanes, so check that we have enough
> lanes for the purpose we need before registering the encoders.
>
> Signed-off-by: Ville Syrjälä
Oh, trusting the VBT. Ind
On Tue, Dec 08, 2015 at 07:59:38PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> Rather than having open coded checks for the DDI A/E configuration,
> just store the max supported lane count in intel_digital_port.
>
> We had an open coded check for DDI A, but not for DDI
On Tue, Dec 08, 2015 at 07:59:39PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> skl_get_buf_trans_*() don't need the 'ddi_translations' local variable
> since all they with is assign and return. Just return the right thing
> directly and get rid of the local variable.
>
On Wed, Dec 09, 2015 at 03:52:52PM +, Dave Gordon wrote:
> In a few places, we fill a GEM object with data, or overwrite some
> portion of its contents other than a single page. In such cases, we
> should mark the object dirty so that its pages in the pagecache are
> written to backing store (r
On Tue, Dec 08, 2015 at 07:59:40PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> skl_get_buf_trans_edp() effectively contains another copy of
> skl_get_buf_trans_dp(). Remove the duplication and just call
> skl_get_buf_trans_dp() from skl_get_buf_trans_edp().
>
> Signed-
Mika Kuoppala writes:
> Chris Wilson writes:
>
>> Following a GPU reset, we may leave the context in a poorly defined
>> state, and reloading from that context will leave the GPU flummoxed. For
>> secondary contexts, this will lead to that context being banned - but
>> currently it is also causi
On Tue, Dec 08, 2015 at 07:59:41PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> Make the ddi buffer programming code a bit more neat by passing
> around dev_priv instead of dev.
>
> Signed-off-by: Ville Syrjälä
Too lazy to check you didn't upset the compiler here. Assu
On Tue, Dec 08, 2015 at 07:59:42PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> The DDI translation tables are supposed to be of certain size,
> so let's add some compile time asserts to enforce that..
>
> Signed-off-by: Ville Syrjälä
Not sure this is worth it really.
On Wed, Dec 09, 2015 at 03:52:51PM +, Dave Gordon wrote:
> In various places, a single page of a (regular) GEM object is mapped into
> CPU address space and updated. In each such case, either the page or the
> the object should be marked dirty, to ensure that the modifications are
> not discard
On Tue, Dec 08, 2015 at 07:59:43PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> Only DDI A and E support 10 translation entries in DP mode. For the
> other ports the tenth entry is reserved for HDMI..
>
> Signed-off-by: Ville Syrjälä
> ---
> drivers/gpu/drm/i915/intel_
On Thu, Dec 10, 2015 at 02:22:01PM +0100, Daniel Vetter wrote:
> On Tue, Dec 08, 2015 at 07:59:38PM +0200, ville.syrj...@linux.intel.com wrote:
> > From: Ville Syrjälä
> >
> > Rather than having open coded checks for the DDI A/E configuration,
> > just store the max supported lane count in intel_
On Wed, 2015-12-09 at 17:25 +, Tvrtko Ursulin wrote:
> Hi,
>
> On 09/12/15 12:46, ankitprasad.r.sha...@intel.com wrote:
> > From: Chris Wilson
> >
> > Ville reminded us that stolen memory is not preserved across
> > hibernation, and a result of this was that context objects now being
> > allo
On Thu, 2015-12-10 at 09:43 +, Tvrtko Ursulin wrote:
> Hi,
>
> Two more comments below:
>
> On 09/12/15 12:46, ankitprasad.r.sha...@intel.com wrote:
> > From: Chris Wilson
> >
> > Ville reminded us that stolen memory is not preserved across
> > hibernation, and a result of this was that cont
On Tue, Dec 08, 2015 at 07:59:44PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> Move the ddi buffer translation programming to occur from the encoder
> .pre_enable() hook, for just the ddi port we are enabling. Previously
> we used to reprogram the translations for all dd
On Thu, Dec 10, 2015 at 03:15:37PM +0200, Ville Syrjälä wrote:
> On Thu, Dec 10, 2015 at 03:01:02PM +0530, Kumar, Shobhit wrote:
> > On 12/09/2015 09:35 PM, Ville Syrjälä wrote:
> > > On Wed, Dec 09, 2015 at 08:59:26PM +0530, Shobhit Kumar wrote:
> > >> On Wed, Dec 9, 2015 at 8:34 PM, Chris Wilson
On Thu, Dec 10, 2015 at 03:24:52PM +0200, Francisco Jerez wrote:
> Mika Kuoppala writes:
>
> > Chris Wilson writes:
> >
> >> Following a GPU reset, we may leave the context in a poorly defined
> >> state, and reloading from that context will leave the GPU flummoxed. For
> >> secondary contexts,
On Thu, Dec 10, 2015 at 02:30:34PM +0100, Daniel Vetter wrote:
> On Tue, Dec 08, 2015 at 07:59:43PM +0200, ville.syrj...@linux.intel.com wrote:
> > From: Ville Syrjälä
> >
> > Only DDI A and E support 10 translation entries in DP mode. For the
> > other ports the tenth entry is reserved for HDMI.
On Tue, Dec 08, 2015 at 07:59:45PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> Eliminate the troublesome role switching DDI encoder, and just register
> a separate encoder for each role (DP and HDMI).
>
> Signed-off-by: Ville Syrjälä
Caveat about max_lanes and pre-DDI
On Tue, Dec 08, 2015 at 07:59:46PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> When the DDI port is in HDMI/DVI mode, it automagically uses the buffer
> translations values from entry 9. Let's make that explicit in the code.
>
> Signed-off-by: Ville Syrjälä
Or WARN_ON
On Tue, Dec 08, 2015 at 07:59:47PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> DDI buffer prorgramming works quite differently depending on
> the mode of the DDI port (DP/eDP/FDI vs. HDMI/DVI). Let's split
> the function that does the programming into two matching varian
There seem to be BIOSes with that set an invalid PWM frequency, assuming the
wrong base frequency. This base frequency is 19.2MHz on BXT except for A
stepping where it's the current CD clock frequency. The BIOSes in question
don't take this difference into account and program the frequency
assuming
On Tue, Dec 08, 2015 at 07:59:48PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> Signed-off-by: Ville Syrjälä
Reviewed-by: Daniel Vetter
> ---
> drivers/gpu/drm/i915/intel_ddi.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/intel_dd
On Tue, Dec 01, 2015 at 05:56:12PM +0200, Mika Kuoppala wrote:
> If head seems stuck and engine in question is rcs,
> inspect subunit state transitions from undone to done,
> before deciding that this really is a hang instead of limited
> progress. Only account the transitions of subunits from
> un
On Tue, Dec 08, 2015 at 07:59:49PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> Program the 'iboost_bit' based on what the VBT says it should be for
> the specific port type, rather than assume it's alwasy the same
> for DP and HDMI.
>
> Signed-off-by: Ville Syrjälä
I
On Tue, Dec 08, 2015 at 07:59:50PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> We no longer have any need to look up the intel_digital_port based
> on the passed in intel_encoder, but we still want to look up the port.
> Let's just move that logic into intel_ddi_get_enco
Chris Wilson writes:
> On Thu, Dec 10, 2015 at 03:24:52PM +0200, Francisco Jerez wrote:
>> Mika Kuoppala writes:
>>
>> > Chris Wilson writes:
>> >
>> >> Following a GPU reset, we may leave the context in a poorly defined
>> >> state, and reloading from that context will leave the GPU flummoxed
On Thu, Dec 10, 2015 at 02:37:01PM +0100, Daniel Vetter wrote:
> On Tue, Dec 08, 2015 at 07:59:44PM +0200, ville.syrj...@linux.intel.com wrote:
> > From: Ville Syrjälä
> >
> > Move the ddi buffer translation programming to occur from the encoder
> > .pre_enable() hook, for just the ddi port we ar
On Thu, Dec 10, 2015 at 01:37:56PM +, Chris Wilson wrote:
> On Thu, Dec 10, 2015 at 03:24:52PM +0200, Francisco Jerez wrote:
> > Mika Kuoppala writes:
> >
> > > Chris Wilson writes:
> > >
> > >> Following a GPU reset, we may leave the context in a poorly defined
> > >> state, and reloading f
On Wed, Dec 09, 2015 at 03:52:52PM +, Dave Gordon wrote:
> In a few places, we fill a GEM object with data, or overwrite some
> portion of its contents other than a single page. In such cases, we
> should mark the object dirty so that its pages in the pagecache are
> written to backing store (r
On Thu, Dec 10, 2015 at 03:31:10PM +0200, Ville Syrjälä wrote:
> On Thu, Dec 10, 2015 at 02:22:01PM +0100, Daniel Vetter wrote:
> > On Tue, Dec 08, 2015 at 07:59:38PM +0200, ville.syrj...@linux.intel.com
> > wrote:
> > > From: Ville Syrjälä
> > >
> > > Rather than having open coded checks for th
On Thu, Dec 10, 2015 at 03:42:15PM +0200, Ville Syrjälä wrote:
> On Thu, Dec 10, 2015 at 02:30:34PM +0100, Daniel Vetter wrote:
> > On Tue, Dec 08, 2015 at 07:59:43PM +0200, ville.syrj...@linux.intel.com
> > wrote:
> > > From: Ville Syrjälä
> > >
> > > Only DDI A and E support 10 translation ent
On Thu, Dec 10, 2015 at 02:47:12PM +0100, Daniel Vetter wrote:
> On Tue, Dec 08, 2015 at 07:59:45PM +0200, ville.syrj...@linux.intel.com wrote:
> > From: Ville Syrjälä
> >
> > Eliminate the troublesome role switching DDI encoder, and just register
> > a separate encoder for each role (DP and HDMI
On Tue, Dec 08, 2015 at 02:48:51PM -0800, Matt Roper wrote:
> In all of our various SDVO setup functions, we allocate an SDVO
> connector (along with an associated connector->state) object, then
> perform initialization. If that initialization fails, we need to make
> sure to free the state object
On Tue, Dec 08, 2015 at 02:48:52PM -0800, Matt Roper wrote:
> We can never be passed a NULL crtc_state (and if we could there are
> plenty of other places later in this function where we access it
> without testing for NULL).
>
> Signed-off-by: Matt Roper
Yup, atomic state structures better exis
On Thu, Dec 10, 2015 at 02:52:29PM +0100, Daniel Vetter wrote:
> On Tue, Dec 08, 2015 at 07:59:47PM +0200, ville.syrj...@linux.intel.com wrote:
> > From: Ville Syrjälä
> >
> > DDI buffer prorgramming works quite differently depending on
> > the mode of the DDI port (DP/eDP/FDI vs. HDMI/DVI). Let'
On 10/12/15 13:17, Ankitprasad Sharma wrote:
On Thu, 2015-12-10 at 09:43 +, Tvrtko Ursulin wrote:
Hi,
Two more comments below:
On 09/12/15 12:46, ankitprasad.r.sha...@intel.com wrote:
From: Chris Wilson
Ville reminded us that stolen memory is not preserved across
hibernation, and a res
1 - 100 of 163 matches
Mail list logo