Re: [Intel-gfx] [alsa-devel] [PATCH 3/4] snd: add support for displayport multi-stream to hda codec.

2015-06-26 Thread Raymond Yau
> > > > > > Shall we move or cc this discussion on audio driver side to ALSA ML? > > > > Oops I thought I had cc'ed these patches to alsa-devel as well when I sent them. > > > > > I think we also need to decide how to manage PCM devices for DP MST. > > > Now the HD-A driver create a PCM device for

Re: [Intel-gfx] [git pull] drm tree for 4.2

2015-06-26 Thread Linus Torvalds
On Thu, Jun 25, 2015 at 6:00 PM, Dave Airlie wrote: > > This is the main drm pull request for v4.2. It seems to work ok for me, but it causes quite a few new warnings on my Sony VAIO Pro laptop. It's (once more) a regular i5-4200U CPU (aka Haswell, aka 4th gen Intel Core i5) Most of them are in

[Intel-gfx] [PATCH] drm/i915: Fix IPS related flicker

2015-06-26 Thread Rodrigo Vivi
We cannot let IPS enabled with no plane on the pipe: BSpec: "IPS cannot be enabled until after at least one plane has been enabled for at least one vertical blank." and "IPS must be disabled while there is still at least one plane enabled on the same pipe as IPS." This restriction apply to HSW and

Re: [Intel-gfx] [PATCH v2 1/2] i965/gen9: Pass alignment as function parameter in drm_intel_gem_bo_alloc_internal()

2015-06-26 Thread Anuj Phogat
On Thu, Jun 25, 2015 at 12:17 PM, Ben Widawsky wrote: > On Thu, Jun 25, 2015 at 07:11:21PM +0100, Chris Wilson wrote: >> On Thu, Jun 25, 2015 at 11:01:44AM -0700, Ben Widawsky wrote: >> > On Wed, Jun 24, 2015 at 08:28:13AM +0100, Chris Wilson wrote: >> > > On Tue, Jun 23, 2015 at 04:44:52PM -0700,

Re: [Intel-gfx] [PATCH 10/10] drm/i915: Zero unused WM1 watermarks on VLV/CHV

2015-06-26 Thread Clint Taylor
On 06/24/2015 12:00 PM, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä The hardware supposedly ignores the WM1 watermarks while the PND deadline mode is enabled, but clear out the register just in case. This is what the other OS does, and it does make register dumps look more consiste

Re: [Intel-gfx] [PATCH 06/10] drm/i915: Use the memory latency based WM computation on VLV too

2015-06-26 Thread Clint Taylor
On 06/24/2015 12:00 PM, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä In order to get decnet memory self refresh residency on VLV, flip it over to the new CHV way of doing things. VLV doesn't do PM5 or DDR DVFS so it's a bit simpler. I'm not sure the currently memory latency used fo

Re: [Intel-gfx] [PATCH 09/10] drm/i915: Add debugfs knobs for VLVCHV memory latency values

2015-06-26 Thread Clint Taylor
On 06/24/2015 12:00 PM, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä Allow tweaking the VLV/CHV memory latencies thorugh sysfs, like we do for ILK+. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/i915_debugfs.c | 24 +++- 1 file changed, 19 insertions

Re: [Intel-gfx] [PATCH 07/10] drm/i915: Try to make sure cxsr is disabled around plane enable/disable

2015-06-26 Thread Clint Taylor
On 06/24/2015 12:00 PM, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä CxSR (or maxfifo on VLV/CHV) blocks somne changes to the plane control register (enable bit at least, not quite sure about the rest). So in order to have the plane enable/disable when we want we need to first kick

Re: [Intel-gfx] [PATCH 05/10] drm/i915: Compute display FIFO split dynamically for CHV

2015-06-26 Thread Clint Taylor
On 06/24/2015 12:00 PM, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä Consider which planes are active and compute the FIFO split based on the relative data rates. Since we only consider the pipe src width rather than the plane width when computing watermarks it seems best to do the

Re: [Intel-gfx] [PATCH 08/10] drm/i915: Don't do PM5/DDR DVFS with multiple pipes

2015-06-26 Thread Clint Taylor
On 06/24/2015 12:00 PM, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä Enabling PM5/DDR DVFS with multiple active pipes isn't a validated configuration. It does seem to work most of the time at least, but there is clearly an additional risk of underruns, so let's not play with fire.

Re: [Intel-gfx] [PATCH 03/10] drm/i915: Read wm values from hardware at init on CHV

2015-06-26 Thread Clint Taylor
On 06/24/2015 12:00 PM, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä Read out the current watermark settings from the hardware at driver init time. This will allow us to compare the newly calculated values against the currrent ones and potentially avoid needless WM updates. Signed-

Re: [Intel-gfx] [PATCH 02/10] drm/i915: Split atomic wm update to pre and post variants

2015-06-26 Thread Clint Taylor
On 06/24/2015 12:00 PM, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä Try to update the watermarks on the right side of the plane update. This is just a temporary hack until we get the proper two part update into place. However in the meantime this might have some chance of at least

Re: [Intel-gfx] [PATCH 01/10] drm/i915: POSTING_READ() in intel_set_memory_cxsr()

2015-06-26 Thread Clint Taylor
On 06/24/2015 12:00 PM, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä We want cxsr exit to happen ASAP, so toss in some POSTING_READ()s to make sure things are really kicked off. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/intel_pm.c | 5 + 1 file changed, 5 insert

Re: [Intel-gfx] [PATCH 04/10] drm/i915: CHV DDR DVFS support and another watermark rewrite

2015-06-26 Thread Clint Taylor
On 06/26/2015 12:48 PM, Ville Syrjälä wrote: On Fri, Jun 26, 2015 at 10:56:33AM -0700, Clint Taylor wrote: On 06/24/2015 12:00 PM, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä Turns out the VLV/CHV system agent doesn't understand memory latencies, so trying to rely on the PND dead

Re: [Intel-gfx] [PATCH] drm/i915: Unconditionally do fb tracking invalidate in set_domain

2015-06-26 Thread Paulo Zanoni
2015-06-26 14:35 GMT-03:00 Chris Wilson : > On Fri, Jun 26, 2015 at 07:35:16PM +0200, Daniel Vetter wrote: >> We can't elide the fb tracking invalidate if the buffer is already in >> the right domain since that would lead to missed screen updates. I'm >> pretty sure I've written this already before

Re: [Intel-gfx] [PATCH 04/10] drm/i915: CHV DDR DVFS support and another watermark rewrite

2015-06-26 Thread Ville Syrjälä
On Fri, Jun 26, 2015 at 10:56:33AM -0700, Clint Taylor wrote: > On 06/24/2015 12:00 PM, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > Turns out the VLV/CHV system agent doesn't understand memory > > latencies, so trying to rely on the PND deadline mechanism is not > > going

Re: [Intel-gfx] [PATCH 3/4] snd: add support for displayport multi-stream to hda codec.

2015-06-26 Thread Liam Girdwood
On Fri, 2015-06-26 at 11:50 +0100, Kaskinen, Tanu wrote: > On Tue, 2015-06-23 at 10:51 +0300, Tanu Kaskinen wrote: > > (Added pulseaudio-discuss to CC.) > > > > On Mon, 2015-06-22 at 17:44 +0200, Takashi Iwai wrote: > > > At Mon, 22 Jun 2015 15:21:16 +, > > > Kaskinen, Tanu wrote: > > > > > >

Re: [Intel-gfx] [PATCH] drm/i915: Reserve space improvements

2015-06-26 Thread Dave Gordon
On 25/06/15 19:38, Tomas Elf wrote: > On 24/06/2015 18:03, john.c.harri...@intel.com wrote: >> From: John Harrison >> >> An earlier patch was added to reserve space in the ring buffer for the >> commands issued during 'add_request()'. The initial version was >> pessimistic in the way it handled bu

Re: [Intel-gfx] [PATCH 04/10] drm/i915: CHV DDR DVFS support and another watermark rewrite

2015-06-26 Thread Clint Taylor
On 06/24/2015 12:00 PM, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä Turns out the VLV/CHV system agent doesn't understand memory latencies, so trying to rely on the PND deadline mechanism is not going to fly especially when DDR DVFS is enabled. Currently we try to avoid the problem

Re: [Intel-gfx] [v3 0/7] Crystalcove (CRC) PMIC based panel and pwm control

2015-06-26 Thread Ville Syrjälä
On Fri, Jun 26, 2015 at 06:31:37PM +0200, Daniel Vetter wrote: > On Fri, Jun 26, 2015 at 02:32:03PM +0530, Shobhit Kumar wrote: > > Hi, > > Next update of the series reviewed at > > https://lkml.org/lkml/2015/6/22/155 > > > > Major changes are few review comments from Varka and Ville being addres

Re: [Intel-gfx] [PATCH v2] drm/i915/skl: Skip remaining dividers when deviation is 0

2015-06-26 Thread Daniel Vetter
On Fri, Jun 26, 2015 at 06:34:29PM +0100, Damien Lespiau wrote: > We can't improve a 0 deviation, so when we find such a divider, skip the > remaining ones they won't be better. > > This short-circuit the search for 34 of the 373 test frequencies in the > corresponding i-g-t test (tools/skl_comput

Re: [Intel-gfx] [PATCH 13/13 v2] drm/i915/skl: Prefer even dividers for SKL DPLLs

2015-06-26 Thread Daniel Vetter
On Fri, Jun 26, 2015 at 02:08:30PM -0300, Paulo Zanoni wrote: > 2015-06-25 12:19 GMT-03:00 Damien Lespiau : > > Currently, if an odd divider improves the deviation (minimizes it), we > > take that divider. The recommendation is to prefer even dividers. > > > > v2: Move the check at the right place

Re: [Intel-gfx] [RFC PATCH 09/18] drm/i915: Cloned mode check

2015-06-26 Thread Daniel Vetter
On Fri, Jun 26, 2015 at 06:14:07PM +0100, Chris Wilson wrote: > On Fri, Jun 26, 2015 at 07:08:40PM +0200, Daniel Vetter wrote: > > On Fri, Jun 26, 2015 at 07:21:53PM +0530, Ramalingam C wrote: > > > If crtc is in clone mode, DRRS will be disabled. Because if the both > > > the displays are not shar

Re: [Intel-gfx] [PATCH] drm/i915: Unconditionally do fb tracking invalidate in set_domain

2015-06-26 Thread Chris Wilson
On Fri, Jun 26, 2015 at 07:35:16PM +0200, Daniel Vetter wrote: > We can't elide the fb tracking invalidate if the buffer is already in > the right domain since that would lead to missed screen updates. I'm > pretty sure I've written this already before but must have gotten lost > unfortunately :( >

[Intel-gfx] [PATCH v2] drm/i915/skl: Skip remaining dividers when deviation is 0

2015-06-26 Thread Damien Lespiau
We can't improve a 0 deviation, so when we find such a divider, skip the remaining ones they won't be better. This short-circuit the search for 34 of the 373 test frequencies in the corresponding i-g-t test (tools/skl_compute_wrpll) v2: Place the short-circuiting code in skl_compute_wrpll() (Paul

[Intel-gfx] [PATCH] drm/i915: Unconditionally do fb tracking invalidate in set_domain

2015-06-26 Thread Daniel Vetter
We can't elide the fb tracking invalidate if the buffer is already in the right domain since that would lead to missed screen updates. I'm pretty sure I've written this already before but must have gotten lost unfortunately :( v2: Chris observed that all internal set_domain users already correctly

Re: [Intel-gfx] [PATCH 12/18] drm/i915/gtt: Fill scratch page

2015-06-26 Thread Dave Gordon
On 25/06/15 18:51, Chris Wilson wrote: > On Thu, Jun 25, 2015 at 06:35:14PM +0300, Mika Kuoppala wrote: >> During review of dynamic page tables series, I was able >> to hit a lite restore bug with execlists. I assume that >> due to incorrect pd, the batch run out of legit address space >> and into

Re: [Intel-gfx] Adding custom bugzilla fields

2015-06-26 Thread Daniel Vetter
On Fri, Jun 26, 2015 at 06:23:39PM +0100, Chris Wilson wrote: > On Fri, Jun 26, 2015 at 07:05:20PM +0200, Daniel Vetter wrote: > > Maybe we need a bit more polish, but probably not worth it to spend too > > much time on the exact feature list. If we spot serious gaps we can always > > add more. And

Re: [Intel-gfx] [PATCH 2/2] drm/i915: In DBUF/WM calcs for 90/270, swap w & h

2015-06-26 Thread Daniel Vetter
On Fri, Jun 26, 2015 at 06:53:49AM -0700, Chandra Konduru wrote: > This patch swaps src width and height for dbuf/wm calculations > when rotation is 90/270 as per hw requirements. > > Signed-off-by: Chandra Konduru Do we have an igt which provokes underruns and hence can test this automatically?

Re: [Intel-gfx] Adding custom bugzilla fields

2015-06-26 Thread Chris Wilson
On Fri, Jun 26, 2015 at 07:05:20PM +0200, Daniel Vetter wrote: > Maybe we need a bit more polish, but probably not worth it to spend too > much time on the exact feature list. If we spot serious gaps we can always > add more. And remove old ones which have gone out of favour (having that > script h

Re: [Intel-gfx] [PATCH] drm/i915/skl: Skip remaining dividers when deviation is 0

2015-06-26 Thread Damien Lespiau
On Fri, Jun 26, 2015 at 02:18:39PM -0300, Paulo Zanoni wrote: > > @@ -1311,13 +1320,15 @@ skl_ddi_calculate_wrpll(int clock /* in Hz */, > > unsigned int p = dividers[d].list[i]; > > uint64_t dco_freq = p * afe_clock; > > > > -

Re: [Intel-gfx] [PATCH 3/4] drm/i915: Interrupt driven fences

2015-06-26 Thread Chris Wilson
On Fri, Jun 26, 2015 at 06:00:03PM +0100, John Harrison wrote: > On 26/06/2015 14:34, Chris Wilson wrote: > >On Fri, Jun 26, 2015 at 01:58:11PM +0100, john.c.harri...@intel.com wrote: > >>From: John Harrison > >> > >>The intended usage model for struct fence is that the signalled status > >>shoul

Re: [Intel-gfx] [PATCH] drm/i915/skl: Skip remaining dividers when deviation is 0

2015-06-26 Thread Paulo Zanoni
2015-06-25 14:08 GMT-03:00 Damien Lespiau : > We can't improve a 0 deviation, so when we find such a divider, skip the > remaining ones they won't be better. > > This short-circuit the search for 34 of the 373 test frequencies in the > corresponding i-g-t test (tools/skl_compute_wrpll) > > Suggeste

Re: [Intel-gfx] [RFC PATCH 09/18] drm/i915: Cloned mode check

2015-06-26 Thread Chris Wilson
On Fri, Jun 26, 2015 at 07:08:40PM +0200, Daniel Vetter wrote: > On Fri, Jun 26, 2015 at 07:21:53PM +0530, Ramalingam C wrote: > > If crtc is in clone mode, DRRS will be disabled. Because if the both > > the displays are not sharing the same vrefresh, then userspace > > activities based on vsync wi

Re: [Intel-gfx] [RFC PATCH 00/18] Generic DRRS implementation across the encoders

2015-06-26 Thread Daniel Vetter
On Fri, Jun 26, 2015 at 07:21:44PM +0530, Ramalingam C wrote: > Display Refresh Rate Switching (DRRS) is a power conservation feature > which enables swtching between low and high refresh rates, > dynamically, based on the usage scenario to save power. > > This feature is applicable for internal p

Re: [Intel-gfx] [PATCH 12/13 v2] drm/i915/skl: Replace the HDMI DPLL divider computation algorithm

2015-06-26 Thread Paulo Zanoni
2015-06-25 12:15 GMT-03:00 Damien Lespiau : > The HW validation team came back from further testing with a slightly > changed constraint on the deviation between the DCO frequency and the > central frequency. Instead of +-4%, it's now +1%/-6%. > > Unfortunately, the previous algorithm didn't quite

Re: [Intel-gfx] [RFC PATCH 10/18] drm/i915: Initializing DRRS for all connectors

2015-06-26 Thread Daniel Vetter
On Fri, Jun 26, 2015 at 07:21:54PM +0530, Ramalingam C wrote: > For all the connectors drrs init is invoked. drrs_init will > initialize the drrs for those connectors that support DRRS. > > Signed-off-by: Ramalingam C > --- > drivers/gpu/drm/i915/intel_display.c |5 + > 1 file changed, 5

Re: [Intel-gfx] [RFC PATCH 11/18] drm/i915: Updating the crtc modes in DRRS transitions

2015-06-26 Thread Daniel Vetter
On Fri, Jun 26, 2015 at 07:21:55PM +0530, Ramalingam C wrote: > During the DRRS state transitions we are modifying the clock and > hence the vrefresh rate. > > So we need to update the drm_crtc->mode and the adjusted > mode in intel_crtc. So that watermark calculations will be as per the > new mod

Re: [Intel-gfx] [PATCH 13/13 v2] drm/i915/skl: Prefer even dividers for SKL DPLLs

2015-06-26 Thread Paulo Zanoni
2015-06-25 12:19 GMT-03:00 Damien Lespiau : > Currently, if an odd divider improves the deviation (minimizes it), we > take that divider. The recommendation is to prefer even dividers. > > v2: Move the check at the right place after having inverted the two for > loops in the previous patch. > >

Re: [Intel-gfx] [RFC PATCH 09/18] drm/i915: Cloned mode check

2015-06-26 Thread Daniel Vetter
On Fri, Jun 26, 2015 at 07:21:53PM +0530, Ramalingam C wrote: > If crtc is in clone mode, DRRS will be disabled. Because if the both > the displays are not sharing the same vrefresh, then userspace > activities based on vsync will go for toss. > > Clone mode will be rechecked on every restarting I

Re: [Intel-gfx] Color management in DRM framework

2015-06-26 Thread Hans Verkuil
Hi Shashank, I have been working on support for colorspace handling for V4L2 (video capture), basically the flip-side of the coin that you are working on. While both the V4L2 and DRM/KMS APIs are completely different, the part that does the matrix calculations can be shared between the two. I wo

Re: [Intel-gfx] [PATCH] drm/vblank: WARN_ON nested use of drm_vblank_on/off

2015-06-26 Thread Tomeu Vizoso
On 22 June 2015 at 14:02, Daniel Vetter wrote: > We should never nest these since in theory kms drivers should know > when a pipe is on/off and call the corresponding enable/disable > functions for the vblank helper code only once. But for historical > reasons (the shared-with-ums version of this

Re: [Intel-gfx] Adding custom bugzilla fields

2015-06-26 Thread Daniel Vetter
On Fri, Jun 26, 2015 at 06:28:39PM +0300, Ander Conselvan De Oliveira wrote: > Hi all, > > I've been looking into creating custom fields in Bugzilla to help sort > our bugs in a more manageable way. I did some testing in a private > installation and came up with this proposal. In a nut shell, we w

Re: [Intel-gfx] [PATCH 3/4] drm/i915: Interrupt driven fences

2015-06-26 Thread John Harrison
On 26/06/2015 14:34, Chris Wilson wrote: On Fri, Jun 26, 2015 at 01:58:11PM +0100, john.c.harri...@intel.com wrote: From: John Harrison The intended usage model for struct fence is that the signalled status should be set on demand rather than polled. That is, there should not be a need for a '

Re: [Intel-gfx] [PATCH v4 4/6] drm/i915/gen8: Add WaFlushCoherentL3CacheLinesAtContextSwitch workaround

2015-06-26 Thread Chris Wilson
On Fri, Jun 26, 2015 at 05:45:08PM +0100, Dave Gordon wrote: > On 16/06/15 21:38, Chris Wilson wrote: > > On Tue, Jun 16, 2015 at 08:25:23PM +0100, Arun Siluvery wrote: > >> + /* WaFlushCoherentL3CacheLinesAtContextSwitch:bdw */ > >> + if (IS_BROADWELL(ring->dev)) { > >> + struct drm_i91

Re: [Intel-gfx] [RFC PATCH 01/18] drm/i915: Removing the eDP specific DRRS implementation

2015-06-26 Thread Daniel Vetter
On Fri, Jun 26, 2015 at 07:21:45PM +0530, Ramalingam C wrote: > EDP specific DRRS implementation is removed to implement a > generic DRRS stack extentable accross the supportable encoders. > > Signed-off-by: Ramalingam C Nack. You don't make something generic by first throwing out the existing s

[Intel-gfx] [PATCH igt v3] lib: Update intel_require_memory to handle +4GB cases

2015-06-26 Thread Michel Thierry
Changed size from u32 to u64 to support +4GB. 48-bit PPGTT test cases may need extra memory available. v2: Use thousands separator (Chris) v3: Moved igt_skip_on_simulation() to the start (Chris) Include Chris' usage hint in the gtk-doc section (Daniel) Cc: Daniel Vetter Reviewed-by: Chris Wi

Re: [Intel-gfx] [PATCH v4 4/6] drm/i915/gen8: Add WaFlushCoherentL3CacheLinesAtContextSwitch workaround

2015-06-26 Thread Dave Gordon
On 16/06/15 21:38, Chris Wilson wrote: > On Tue, Jun 16, 2015 at 08:25:23PM +0100, Arun Siluvery wrote: >> +/* WaFlushCoherentL3CacheLinesAtContextSwitch:bdw */ >> +if (IS_BROADWELL(ring->dev)) { >> +struct drm_i915_private *dev_priv = ring->dev->dev_private; > > dev_priv = to_

Re: [Intel-gfx] [PATCH 16/18] drm/i915/gtt: One instance of scratch page table/directory

2015-06-26 Thread Daniel Vetter
On Fri, Jun 26, 2015 at 03:05:29PM +0300, Mika Kuoppala wrote: > Daniel Vetter writes: > > > On Thu, Jun 25, 2015 at 06:35:18PM +0300, Mika Kuoppala wrote: > >> +static int setup_scratch(struct i915_address_space *vm) > >> +{ > >> + struct i915_address_space *ggtt_vm = &to_i915(vm->dev)->gtt.bas

Re: [Intel-gfx] [PATCH v2 02/18] drm/i915/gtt: Switch gen8_free_page_tables params

2015-06-26 Thread Daniel Vetter
On Thu, Jun 11, 2015 at 09:05:38PM +0300, Mika Kuoppala wrote: > Michel Thierry writes: > > > After Mika's ppgtt cleanup series, all the other free functions have > > drm_device as the first parameter, except this one. > > > > No functional changes. > > > > Signed-off-by: Michel Thierry > > Rev

Re: [Intel-gfx] [v3 0/7] Crystalcove (CRC) PMIC based panel and pwm control

2015-06-26 Thread Daniel Vetter
On Fri, Jun 26, 2015 at 02:32:03PM +0530, Shobhit Kumar wrote: > Hi, > Next update of the series reviewed at > https://lkml.org/lkml/2015/6/22/155 > > Major changes are few review comments from Varka and Ville being addressed. > Also except > for intel-gfx patches, all patches reviesion history

Re: [Intel-gfx] [PATCH 3/4] snd: add support for displayport multi-stream to hda codec.

2015-06-26 Thread Kaskinen, Tanu
On Mon, 2015-06-22 at 17:44 +0200, Takashi Iwai wrote: > At Mon, 22 Jun 2015 15:21:16 +, > Kaskinen, Tanu wrote: > > PulseAudio (mostly) doesn't use the hw:X devices directly. Instead, it > > uses logical names like "front", "hdmi", "iec958", etc. Speaking of HDMI > > specifically, PulseAudio u

[Intel-gfx] [PATCH v2] scripts/kernel-doc: Adding cross-reference links to html documentation.

2015-06-26 Thread Danilo Cesar Lemes de Paula
Functions, Structs and Parameters definitions on kernel documentation are pure cosmetic, it only highlights the element. To ease the navigation in the documentation we should use inside those tags so readers can easily jump between methods directly. This was discussed in 2014[1] and is implement

Re: [Intel-gfx] [PATCH libdrm] intel: Add EXEC_OBJECT_SUPPORTS_48BADDRESS flag.

2015-06-26 Thread Dave Gordon
On 23/06/15 13:21, Michel Thierry wrote: > Gen8+ supports 48-bit virtual addresses, but some objects must always be > allocated inside the 32-bit address range. > > In specific, any resource used with flat/heapless (0x-0xf000) > General State Heap (GSH) or Intruction State Heap (ISH) m

Re: [Intel-gfx] [PATCH v3] drm/i915/lrc: Update PDPx registers with lri commands

2015-06-26 Thread Mika Kuoppala
Michel Thierry writes: > A safer way to update the PDPx registers is sending lri commands, added > in the ring before the batchbuffer start. Otherwise, the ctx must be idle > before trying to change anything (but the ring-tail) in the ctx image. An > example where the ctx won't be idle is lite-re

Re: [Intel-gfx] [PATCH] sna: Correct misspelled cacheing->caching in comments

2015-06-26 Thread Dave Gordon
On 24/06/15 11:04, Chris Wilson wrote: > On Tue, Jun 23, 2015 at 07:45:42AM +0200, Sedat Dilek wrote: >> I have seen this typo once and added an entry to codespell's dictionary.txt >> file. >> >> $ diff -uprN /usr/share/codespell/dictionary.txt.orig >> /usr/share/codespell/dictionary.txt >> --- /

Re: [Intel-gfx] [PATCH] scripts/kernel-doc: Adding cross-reference links to html documentation.

2015-06-26 Thread Danilo Cesar Lemes de Paula
On 06/24/2015 06:22 PM, Daniel Vetter wrote: > On Wed, Jun 24, 2015 at 04:10:24PM -0300, Danilo Cesar Lemes de Paula wrote: >> Functions, Structs and Parameters definitions on kernel documentation >> are pure cosmetic, it only highlights the element. >> >> To ease the navigation in the documentatio

Re: [Intel-gfx] [PATCH resend 3/5] drm/i915: Enable resource streamer on Execlists

2015-06-26 Thread Siluvery, Arun
On 16/06/2015 11:39, Abdiel Janulgue wrote: GEN8 and above uses Execlists by default instead of the legacy ringbuffer for batch execution. This patch enables the resource streamer bits when required. Patch is based on the initial work by Minu Mathai This version also adds the required bits to e

[Intel-gfx] [RFC PATCH 00/18] Generic DRRS implementation across the encoders

2015-06-26 Thread Ramalingam C
Display Refresh Rate Switching (DRRS) is a power conservation feature which enables swtching between low and high refresh rates, dynamically, based on the usage scenario to save power. This feature is applicable for internal panels. Indication that the panel supports DRRS is given by the panel ED

[Intel-gfx] [RFC PATCH 18/18] drm/i915: Connector property for DRRS capability

2015-06-26 Thread Ramalingam C
DRRS capability on each connector is exposed to userspace through drm connector property. In this change one drm property is created and attached with each connector. And when DRRS is successfully initialized for a connector, drrs connector property is set with appropriate value. Signed-off-by: R

[Intel-gfx] [RFC PATCH 13/18] drm/i915: MEDIA_RR support in general DRRS state machine

2015-06-26 Thread Ramalingam C
Based on the minimum requirement of the vrefresh to support a content's playback, we can place a drrs request from userspace to kernel. For example 24FPS video content can be played at 48 vrefresh without any quality degradation. This patch implements required changes in generic DRRS state machine

[Intel-gfx] [RFC PATCH 17/18] drm/i915: Addition of downclock mode to connector modelist

2015-06-26 Thread Ramalingam C
If DSI panel has the downclock mode supported, add that in the connector's mode list. This is needed by userspace apps to identify the range of vrefresh rates supported by panel incase of the media playback DRRS. Signed-off-by: Ramalingam C --- drivers/gpu/drm/i915/intel_dsi.c | 21 ++

[Intel-gfx] [RFC PATCH 15/18] drm/i915: MEDIA_RR support in DSI DRRS module

2015-06-26 Thread Ramalingam C
Content based DRRS support is implemented in DSI DRRS module also. Signed-off-by: Ramalingam C --- drivers/gpu/drm/i915/intel_dsi_drrs.c | 60 +++-- 1 file changed, 58 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dsi_drrs.c b/drivers/gpu/d

[Intel-gfx] [RFC PATCH 16/18] drm/i915: Filtering media playback DRRS requests

2015-06-26 Thread Ramalingam C
If the platform supports the media playback DRRS, Userspace can provide a request for Media playback DRRS. This is done by placing the modeset request with the same mode and FB but with different vrefresh. This change implements the algorithm to identify the Media playback DRRS requests from the n

[Intel-gfx] [RFC PATCH 14/18] drm/i915: MEDIA_RR support in eDP DRRS module

2015-06-26 Thread Ramalingam C
Content based DRRS support is implemented in eDP DRRS module also. Signed-off-by: Ramalingam C --- drivers/gpu/drm/i915/intel_drv.h |3 +++ drivers/gpu/drm/i915/intel_edp_drrs.c | 29 + 2 files changed, 32 insertions(+) diff --git a/drivers/gpu/drm/i915/in

[Intel-gfx] [RFC PATCH 12/18] drm/i915: Redesigning dp_set_m_n to receive divider values

2015-06-26 Thread Ramalingam C
This will enable the called to decide the combination of divider values that needs to be programmed. First divider value will go into primary divider and the second divider value will be taken for drrs divider register, if that is supported on platform. Signed-off-by: Ramalingam C --- drivers/g

[Intel-gfx] [RFC PATCH 07/18] drm/i915: Generic eDP DRRS implementation

2015-06-26 Thread Ramalingam C
This patch Implements the generic eDP DRRS functions and registers them with Generic DRRS state machine. Platform specific eDP DRRS functions will be implemented and registered with this generic eDP DRRS implementation. Hence extending the eDP DRRS to new platform is made simple. Signed-off-by: R

[Intel-gfx] [RFC PATCH 11/18] drm/i915: Updating the crtc modes in DRRS transitions

2015-06-26 Thread Ramalingam C
During the DRRS state transitions we are modifying the clock and hence the vrefresh rate. So we need to update the drm_crtc->mode and the adjusted mode in intel_crtc. So that watermark calculations will be as per the new modified clock. Signed-off-by: Ramalingam C --- drivers/gpu/drm/i915/intel

[Intel-gfx] [RFC PATCH 09/18] drm/i915: Cloned mode check

2015-06-26 Thread Ramalingam C
If crtc is in clone mode, DRRS will be disabled. Because if the both the displays are not sharing the same vrefresh, then userspace activities based on vsync will go for toss. Clone mode will be rechecked on every restarting Idleness DRRS events. Signed-off-by: Ramalingam C --- drivers/gpu/drm/

[Intel-gfx] [RFC PATCH 10/18] drm/i915: Initializing DRRS for all connectors

2015-06-26 Thread Ramalingam C
For all the connectors drrs init is invoked. drrs_init will initialize the drrs for those connectors that support DRRS. Signed-off-by: Ramalingam C --- drivers/gpu/drm/i915/intel_display.c |5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers

[Intel-gfx] [RFC PATCH 08/18] drm/i915: VLV eDP DRRS methods

2015-06-26 Thread Ramalingam C
VLV related ePD drrs functions are implemented and registered with generic eDP drrs module. This will provide the platform specific services to generic ePD drrs stack, like program the pll registers for DRRS functionality. Signed-off-by: Ramalingam C --- drivers/gpu/drm/i915/intel_edp_drrs.c |

[Intel-gfx] [RFC PATCH 06/18] drm/i915: VLV dsi drrs support

2015-06-26 Thread Ramalingam C
VLV related dsi drrs functions are implemented and registered with generic dsi drrs. This will provide the service to generic dsi drrs stack to calculate the pll divider values and program the pll registers for DRRS functionality. Signed-off-by: Ramalingam C --- drivers/gpu/drm/i915/i915_reg.h

[Intel-gfx] [RFC PATCH 04/18] drm/i915: Implementation of Generic DSI DRRS

2015-06-26 Thread Ramalingam C
This patch Implements the generic DSI DRRS functions and registers them with Generic DRRS state machine. Platform specific DSI DRRS functions will be implemented and registered with this generic DSI DRRS implementation. Hence extending the DSI DRRS to new platform is made simple. Signed-off-by: R

[Intel-gfx] [RFC PATCH 05/18] drm/i915: Adjusting the pclk for dual link and burst mode

2015-06-26 Thread Ramalingam C
dsi_clk is calculated for the clock of passed drm_display_mode and pclk is adjusted considering dual link and the burst mode. This change is required to make the drrs to co-exist with dual link and Burst mode. Signed-off-by: Ramalingam C --- drivers/gpu/drm/i915/intel_dsi_pll.c | 77 +

[Intel-gfx] [RFC PATCH 01/18] drm/i915: Removing the eDP specific DRRS implementation

2015-06-26 Thread Ramalingam C
EDP specific DRRS implementation is removed to implement a generic DRRS stack extentable accross the supportable encoders. Signed-off-by: Ramalingam C --- drivers/gpu/drm/i915/i915_debugfs.c | 110 - drivers/gpu/drm/i915/i915_drv.h | 22 -- drivers/gpu/drm/i915/intel_ddi

[Intel-gfx] [RFC PATCH 03/18] drm/i915: Addition of the drrs_min_vrefresh in VBT

2015-06-26 Thread Ramalingam C
VBT structure's block 42 is updated as per the current VBT v188 specification and additonal member drrs_min_vrefresh is added. Corresponding logic for parsing drrs_min_vrefresh is added. drrs_min_vrefresh is nothing but lowest vrefresh supported by the panel. Signed-off-by: Ramalingam C --- dri

[Intel-gfx] [RFC PATCH 02/18] drm/i915: Generic DRRS state Machine

2015-06-26 Thread Ramalingam C
DRRS is a power saving feature, which will refresh the display at the lowest supported refresh rate, based on the rate of change of display content to be rendered. This patch implements the Generic state machine for the Idleness DRRS. Idleness DRRS is nothing but, when the content of the Display i

[Intel-gfx] [PATCH 2/2] drm/i915: In DBUF/WM calcs for 90/270, swap w & h

2015-06-26 Thread Chandra Konduru
This patch swaps src width and height for dbuf/wm calculations when rotation is 90/270 as per hw requirements. Signed-off-by: Chandra Konduru --- drivers/gpu/drm/i915/intel_pm.c | 32 1 file changed, 28 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/dr

[Intel-gfx] [PATCH 1/2] drm/i915: Allocate min dbuf blocks per bspec

2015-06-26 Thread Chandra Konduru
Properly allocate min blocks per hw requirements. Signed-off-by: Chandra Konduru --- drivers/gpu/drm/i915/intel_pm.c | 39 +-- 1 file changed, 37 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c in

[Intel-gfx] [PATCH 0/2] Fix min dbuf, 90/270 wm calcs

2015-06-26 Thread Chandra Konduru
Resending with correct patches. This patch series allocates minimum dbuf blocks required for tile-y, tile-yf buffers correctly as per bspec. Also in WM calculations, for 90/270, swaps source width and height. Chandra Konduru (2): drm/i915: Allocate min dbuf blocks per bspec drm/i915: In DBUF

[Intel-gfx] [PATCH 2/2] drm/i915: In DBUF/WM calcs for 90/270, swap w & h

2015-06-26 Thread Chandra Konduru
This patch swaps src width and height for dbuf/wm calculations when rotation is 90/270 as per hw requirements. Signed-off-by: Chandra Konduru --- drivers/gpu/drm/i915/intel_pm.c | 32 1 file changed, 28 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/dr

[Intel-gfx] [PATCH 1/2] drm/i915: Allocate min dbuf blocks per bspec

2015-06-26 Thread Chandra Konduru
Properly allocate min blocks per hw requirements. Signed-off-by: Chandra Konduru --- drivers/gpu/drm/i915/intel_pm.c | 39 +-- 1 file changed, 37 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c in

[Intel-gfx] [PATCH 0/2] Fix min dbuf, 90/270 wm calcs

2015-06-26 Thread Chandra Konduru
This patch series allocates minimum dbuf blocks required for tile-y, tile-yf buffers correctly as per bspec. Also in WM calculations, for 90/270, swaps source width and height. Chandra Konduru (2): drm/i915: Allocate min dbuf blocks per bspec drm/i915: In DBUF/WM calcs for 90/270, swap w & h

[Intel-gfx] [PATCH v4] drm/i915: Add support for SKL background color

2015-06-26 Thread Chandra Konduru
This patch adds support for Skylake display pipe background color. Signed-off-by: Chandra Konduru --- Documentation/DocBook/drm.tmpl | 10 - drivers/gpu/drm/i915/i915_reg.h | 10 + drivers/gpu/drm/i915/intel_display.c | 73 ++ drivers/gpu/

Re: [Intel-gfx] [PATCH 3/4] drm/i915: Interrupt driven fences

2015-06-26 Thread Chris Wilson
On Fri, Jun 26, 2015 at 01:58:11PM +0100, john.c.harri...@intel.com wrote: > From: John Harrison > > The intended usage model for struct fence is that the signalled status should > be > set on demand rather than polled. That is, there should not be a need for a > 'signaled' function to be called

Re: [Intel-gfx] [PATCH 09/17 v2] drm/i915: Expose two LRC functions for GuC submission mode

2015-06-26 Thread Dave Gordon
On 26/06/15 08:31, Chris Wilson wrote: > On Thu, Jun 25, 2015 at 01:57:16PM -0700, Yu Dai wrote: >> >> On 06/25/2015 07:40 AM, Dave Gordon wrote: >>> GuC submission is basically execlist submission, but with the GuC >>> handling the actual writes to the ELSP and the resulting context >>> switch int

[Intel-gfx] [PATCH 1/4] drm/i915: Convert requests to use struct fence

2015-06-26 Thread John . C . Harrison
From: John Harrison There is a construct in the linux kernel called 'struct fence' that is intended to keep track of work that is executed on hardware. I.e. it solves the basic problem that the drivers 'struct drm_i915_gem_request' is trying to address. The request structure does quite a lot more

[Intel-gfx] [PATCH 2/4] drm/i915: Removed now redudant parameter to i915_gem_request_completed()

2015-06-26 Thread John . C . Harrison
From: John Harrison The change to the implementation of i915_gem_request_completed() means that the lazy coherency flag is no longer used. This can now be removed to simplify the interface. For: VIZ-5190 Signed-off-by: John Harrison --- drivers/gpu/drm/i915/i915_debugfs.c | 2 +- drivers/gpu

[Intel-gfx] [PATCH 4/4] drm/i915: Updated request structure tracing

2015-06-26 Thread John . C . Harrison
From: John Harrison Added the '_complete' trace event which occurs when a fence/request is signaled as complete. Also moved the notify event from the IRQ handler code to inside the notify function itself. For: VIZ-5190 Signed-off-by: John Harrison --- drivers/gpu/drm/i915/i915_gem.c | 3 +++

[Intel-gfx] [PATCH 0/4] Convert requests to use struct fence

2015-06-26 Thread John . C . Harrison
From: John Harrison There is a construct in the linux kernel called 'struct fence' that is intended to keep track of work that is executed on hardware. I.e. it solves the basic problem that the drivers 'struct drm_i915_gem_request' is trying to address. The request structure does quite a lot more

[Intel-gfx] [PATCH 3/4] drm/i915: Interrupt driven fences

2015-06-26 Thread John . C . Harrison
From: John Harrison The intended usage model for struct fence is that the signalled status should be set on demand rather than polled. That is, there should not be a need for a 'signaled' function to be called everytime the status is queried. Instead, 'something' should be done to enable a signal

Re: [Intel-gfx] [PATCH] drm/i915: Update rps frequencies for BXT

2015-06-26 Thread Imre Deak
On to, 2015-06-25 at 14:54 -0700, Bob Paauwe wrote: > Broxton is using a different register and different bit ordering > for rps status capabilities. > > Also GT perf freqency register is different for Broxton so update > that. > > Signed-off-by: Bob Paauwe > --- > drivers/gpu/drm/i915/i915_deb

[Intel-gfx] [PATCH v3] drm/i915/lrc: Update PDPx registers with lri commands

2015-06-26 Thread Michel Thierry
A safer way to update the PDPx registers is sending lri commands, added in the ring before the batchbuffer start. Otherwise, the ctx must be idle before trying to change anything (but the ring-tail) in the ctx image. An example where the ctx won't be idle is lite-restore. This patch depends on 5b7

Re: [Intel-gfx] [v3 5/7] pwm: crc: Add Crystalcove (CRC) PWM driver

2015-06-26 Thread Varka Bhadram
On 06/26/2015 02:32 PM, Shobhit Kumar wrote: The Crystalcove PMIC provides three PWM signals and this driver exports one of them on the BYT platform which is used to control backlight for DSI panel. This is platform device implementation of the drivers/mfd cell device for CRC PMIC. CC: Samuel Or

Re: [Intel-gfx] [PATCH 16/18] drm/i915/gtt: One instance of scratch page table/directory

2015-06-26 Thread Mika Kuoppala
Daniel Vetter writes: > On Thu, Jun 25, 2015 at 06:35:18PM +0300, Mika Kuoppala wrote: >> +static int setup_scratch(struct i915_address_space *vm) >> +{ >> +struct i915_address_space *ggtt_vm = &to_i915(vm->dev)->gtt.base; >> + >> +if (i915_is_ggtt(vm)) >> +return setup_scratc

[Intel-gfx] [PATCH i-g-t v3] tests/gem_ringfill: Add {render, blitter}-forked-1 subtests.

2015-06-26 Thread Joonas Lahtinen
Add forking subtests to gem_ringfill. Tests cause consistent GPU hangs on SKL. v2: Removed noop parts. v3: - Allow executing the tests in order too (Chris Wilson). - Rename the tests to -forked-1 Cc: Mika Kuoppala Cc: Chris Wilson Signed-off-by: Joonas Lahtinen Bugzilla: https://bugs.freedeskt

[Intel-gfx] [PATCH v3] drm/i915: fix backlight after resume on 855gm

2015-06-26 Thread Jani Nikula
Some 855gm models (at least ThinkPad X40) regressed because of commit b0cd324faed23d10d66ba6ade66579c681feef6f Author: Jani Nikula Date: Wed Nov 12 16:25:43 2014 +0200 drm/i915: don't save/restore backlight hist ctl registers which tried to make our driver more robust by not blindly savin

Re: [Intel-gfx] [PATCH] drm/i915: fix backlight after resume on 855gm

2015-06-26 Thread Chris Wilson
On Fri, Jun 26, 2015 at 01:54:18PM +0300, Jani Nikula wrote: > Some 855gm models (at least ThinkPad X40) regressed because of > > commit b0cd324faed23d10d66ba6ade66579c681feef6f > Author: Jani Nikula > Date: Wed Nov 12 16:25:43 2014 +0200 > > drm/i915: don't save/restore backlight hist ctl

Re: [Intel-gfx] Color management in DRM framework

2015-06-26 Thread Sharma, Shashank
Hi Emil, Thanks your time and suggestions. We actually have the complete implementation ready with us, but we wanted to know if community is agreeing to the design itself. In fact that was our plan ahead, that if we don’t get enough comments on the design, we will start pushing the patches.

[Intel-gfx] [PATCH] drm/i915: fix backlight after resume on 855gm

2015-06-26 Thread Jani Nikula
Some 855gm models (at least ThinkPad X40) regressed because of commit b0cd324faed23d10d66ba6ade66579c681feef6f Author: Jani Nikula Date: Wed Nov 12 16:25:43 2014 +0200 drm/i915: don't save/restore backlight hist ctl registers which tried to make our driver more robust by not blindly savin

  1   2   >