Re: [Intel-gfx] [PATCH v2 5/7] drm/i915: Move intel_dp->lane_count into pipe_config

2015-08-13 Thread Sivakumar Thulasimani
Reviewed-by: Sivakumar Thulasimani On Monday 06 July 2015 07:09 PM, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä Currently we clobber intel_dp->lane_count in compute config, which means after a rejected modeset we may no longer be able to retrain the current link. Move lane_count

Re: [Intel-gfx] [PATCH 3/7] prime_mmap: Add basic tests to write in a bo using CPU

2015-08-13 Thread Daniel Vetter
On Wed, Aug 12, 2015 at 08:29:16PM -0300, Tiago Vignatti wrote: > This patch adds test_correct_cpu_write, which maps the texture buffer through > a > prime fd and then writes directly to it using the CPU. It stresses the driver > to guarantee cache synchronization among the different domains. > >

Re: [Intel-gfx] [PATCH 1/7] prime_mmap: Add new test for calling mmap() on dma-buf fds

2015-08-13 Thread Daniel Vetter
On Wed, Aug 12, 2015 at 08:29:14PM -0300, Tiago Vignatti wrote: > From: Rob Bradford > > This test has the following subtests: > - test_correct for correctness of the data > - test_map_unmap checks for mapping idempotency > - test_reprime checks for dma-buf creation idempotency > - test_forke

Re: [Intel-gfx] [PATCH] lib/rendercopy_gen9: Setup Push constant pointer before sending BTP commands

2015-08-13 Thread Joonas Lahtinen
Hi, On ke, 2015-08-12 at 18:35 -0700, Ben Widawsky wrote: > On Wed, Aug 12, 2015 at 03:10:18PM +0300, Joonas Lahtinen wrote: > > On ke, 2015-08-12 at 12:26 +0100, Arun Siluvery wrote: > > > From Gen9, by default push constant command is not committed to > > > the > > > shader unit > > > untill t

[Intel-gfx] [PULL] topic/drm-misc

2015-08-13 Thread Daniel Vetter
Hi Dave, Final drm-misc pull for 4.3: - fbdev emulation Kconfig option for everyone thanks to Archit. It's not everything yet bit this is fairly tricky since it spawns all drivers. - vgaarb & vgaswitcheroo polish from Thierry - some drm_irq.c cleanups (Thierry) - struct_mutex crusade from me - m

Re: [Intel-gfx] [PATCH v3 01/11] drm/i915: Store max dotclock

2015-08-13 Thread Mika Kahola
On Wed, 2015-08-12 at 22:01 +0300, Ville Syrjälä wrote: > On Wed, Aug 12, 2015 at 08:30:23PM +0300, Ville Syrjälä wrote: > > On Fri, Jul 31, 2015 at 03:13:50PM +0300, Mika Kahola wrote: > > > Store max dotclock into dev_priv structure so we are able > > > to filter out the modes that are not suppor

Re: [Intel-gfx] [PATCH] drm/i915: Set alternate aux for DDI-E

2015-08-13 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 7119 -Summary- Platform Delta drm-intel-nightly Series Applied ILK -1

Re: [Intel-gfx] [PATCH] drm/i915: Only dither on 6bpc panels

2015-08-13 Thread Jani Nikula
On Thu, 13 Aug 2015, Mario Kleiner wrote: > Thanks for the quick fix! Comments below... > > On 08/12/2015 11:43 AM, Daniel Vetter wrote: >> In >> >> commit d328c9d78d64ca11e744fe227096990430a88477 >> Author: Daniel Vetter >> Date: Fri Apr 10 16:22:37 2015 +0200 >> >> drm/i915: Select start

Re: [Intel-gfx] [PATCH 02/21] drm/i915/gtt: Workaround for HW preload not flushing pdps

2015-08-13 Thread Zhiyuan Lv
Hi Michel, Thanks for the reply! I yet have another question: right now the mark_tlb_dirty() will be called if any level of PPGTT table is changed. But for the EXECLIST context submission, we only need LRI commands if there are L3 PDP root pointer changes right? Thanks! Regards, -Zhiyuan On Wed

[Intel-gfx] [PATCH i-g-t] benckmarks/Android.mk: Fix building benchmarks for Android

2015-08-13 Thread Derek Morton
The commit "benchmarks: Do not install to system-wide bin/" changed the benchmark file list from bin_PROGRAMS to benchmarks_PROGRAMS. However Android.mk was not updated, resulting in IGT failing to build for Android. This commit adds that change. It also adds LOCAL_MODULE_PATH to specify where the

Re: [Intel-gfx] [PATCH 02/21] drm/i915/gtt: Workaround for HW preload not flushing pdps

2015-08-13 Thread Zhiyuan Lv
Hi Dave, On Wed, Aug 12, 2015 at 04:09:18PM +0100, Dave Gordon wrote: > On 12/08/15 08:56, Thierry, Michel wrote: > >On 8/11/2015 1:05 PM, Zhiyuan Lv wrote: > >>Hi Mika/Dave/Michel, > >> > >>I saw the patch of using LRI for root pointer update has been merged to > >>drm-intel. When we consider i91

[Intel-gfx] [PATCH i-g-t] benckmarks/Android.mk: Fix building benchmarks for Android

2015-08-13 Thread Derek Morton
The commit "benchmarks: Do not install to system-wide bin/" changed the benchmark file list from bin_PROGRAMS to benchmarks_PROGRAMS. However Android.mk was not updated, resulting in IGT failing to build for Android. This commit adds that change. It also adds LOCAL_MODULE_PATH to specify where the

Re: [Intel-gfx] [PATCH i-g-t] benckmarks/Android.mk: Fix building benchmarks for Android

2015-08-13 Thread Morton, Derek J
This is the same as the previously sent patch but with Signed-off-by added to the commit message. //Derek > > >-Original Message- >From: Morton, Derek J >Sent: Thursday, August 13, 2015 10:51 AM >To: intel-gfx@lists.freedesktop.org >Cc: Wood, Thomas; Gore, Tim; ch...@chris-wilson.co.uk;

[Intel-gfx] [PATCH 1/2] drm/i915: Adding break for one case

2015-08-13 Thread Xiong Zhang
Signed-off-by: Xiong Zhang --- drivers/gpu/drm/i915/intel_display.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index 65cc5b1..801187c 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/

[Intel-gfx] [PATCH 2/2] drm/i915/skl: Adding DDI_E power well domain

2015-08-13 Thread Xiong Zhang
From B spec, DDI_E port belong to PowerWell 2, but DDI_E share the powerwell_req/staus register bit with DDI_A which belong to DDI_A_E_POWER_WELL. In order to communicate with the connector on DDI-E, both DDI_A_E_POWER_WELL and POWER_WELL_2 must be enabled. Currently intel_dp_power_get(DDI_E) onl

Re: [Intel-gfx] [PATCH 02/21] drm/i915/gtt: Workaround for HW preload not flushing pdps

2015-08-13 Thread Michel Thierry
On 8/13/2015 5:36 PM, Zhiyuan Lv wrote: Hi Dave, On Wed, Aug 12, 2015 at 04:09:18PM +0100, Dave Gordon wrote: On 12/08/15 08:56, Thierry, Michel wrote: On 8/11/2015 1:05 PM, Zhiyuan Lv wrote: Hi Mika/Dave/Michel, I saw the patch of using LRI for root pointer update has been merged to drm-int

Re: [Intel-gfx] [PATCH] lib/rendercopy_gen9: Setup Push constant pointer before sending BTP commands

2015-08-13 Thread Mika Kuoppala
Arun Siluvery writes: > From Gen9, by default push constant command is not committed to the shader > unit > untill the corresponding shader's BTP_* command is parsed. This is the > behaviour when set shader is enabled. This patch updates the batch to follow > this requirement otherwise it result

Re: [Intel-gfx] [PATCH 02/21] drm/i915/gtt: Workaround for HW preload not flushing pdps

2015-08-13 Thread Michel Thierry
On 8/13/2015 5:08 PM, Zhiyuan Lv wrote: Hi Michel, Thanks for the reply! I yet have another question: right now the mark_tlb_dirty() will be called if any level of PPGTT table is changed. But for the EXECLIST context submission, we only need LRI commands if there are L3 PDP root pointer changes

[Intel-gfx] [PATCH i-g-t v2] benckmarks/Android.mk: Fix building benchmarks for Android

2015-08-13 Thread Derek Morton
The commit "benchmarks: Do not install to system-wide bin/" changed the benchmark file list from bin_PROGRAMS to benchmarks_PROGRAMS. However Android.mk was not updated, resulting in IGT failing to build for Android. This commit adds that change. It also adds LOCAL_MODULE_PATH to specify where the

Re: [Intel-gfx] [PATCH] drm/i915: Spam less on dp aux send/receive problems

2015-08-13 Thread Mika Kuoppala
Daniel Vetter writes: > On Mon, Aug 10, 2015 at 02:01:56PM +0300, Jani Nikula wrote: >> On Mon, 10 Aug 2015, Jani Nikula wrote: >> > On Thu, 06 Aug 2015, Mika Kuoppala wrote: >> >> If we encounter frequent problems with dp aux channel >> >> communications, we end up spamming the dmesg with the

Re: [Intel-gfx] [PATCH 1/2 v2 addendum] drm/i915: Allow parsing of variable size child device entries from VBT

2015-08-13 Thread David Weinehall
On Wed, Aug 12, 2015 at 03:13:35PM +0300, Jani Nikula wrote: > On Wed, 12 Aug 2015, David Weinehall wrote: > > Some more fixup is needed; the bits from Antti's patch > > that actually expanded the struct to fully fit the newer > > versions of the child_device_config was part of the second > > patc

Re: [Intel-gfx] [PATCH i-g-t v2] benckmarks/Android.mk: Fix building benchmarks for Android

2015-08-13 Thread Thomas Wood
On 13 August 2015 at 11:27, Derek Morton wrote: > The commit "benchmarks: Do not install to system-wide bin/" changed > the benchmark file list from bin_PROGRAMS to benchmarks_PROGRAMS. > However Android.mk was not updated, resulting in IGT failing to > build for Android. > This commit adds that c

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Adding break for one case

2015-08-13 Thread Timo Aaltonen
On 13.08.2015 13:36, Timo Aaltonen wrote: > On 13.08.2015 13:00, Xiong Zhang wrote: >> Signed-off-by: Xiong Zhang >> --- >> drivers/gpu/drm/i915/intel_display.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/drivers/gpu/drm/i915/intel_display.c >> b/drivers/gpu/drm/i915/intel_displa

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Adding break for one case

2015-08-13 Thread Timo Aaltonen
On 13.08.2015 13:00, Xiong Zhang wrote: > Signed-off-by: Xiong Zhang > --- > drivers/gpu/drm/i915/intel_display.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/gpu/drm/i915/intel_display.c > b/drivers/gpu/drm/i915/intel_display.c > index 65cc5b1..801187c 100644 > --- a/driver

[Intel-gfx] [PATCH] drm/i915/bdw: Check for slice, subslice and EU count for BDW

2015-08-13 Thread Łukasz Daniluk
Added checks for available slices, subslices and EUs for Broadwell. This information is filled in intel_device_info and is available to user with GET_PARAM. Added checks for enabled slices, subslices and EU for Broadwell. This information is based on available counts but takes power gated slices in

Re: [Intel-gfx] [PATCH 1/2 v2 addendum] drm/i915: Allow parsing of variable size child device entries from VBT

2015-08-13 Thread Jani Nikula
On Thu, 13 Aug 2015, David Weinehall wrote: > On Wed, Aug 12, 2015 at 03:13:35PM +0300, Jani Nikula wrote: >> On Wed, 12 Aug 2015, David Weinehall wrote: >> > Some more fixup is needed; the bits from Antti's patch >> > that actually expanded the struct to fully fit the newer >> > versions of the

Re: [Intel-gfx] [PATCH 02/21] drm/i915/gtt: Workaround for HW preload not flushing pdps

2015-08-13 Thread Dave Gordon
On 13/08/15 11:12, Michel Thierry wrote: On 8/13/2015 5:08 PM, Zhiyuan Lv wrote: Hi Michel, Thanks for the reply! I yet have another question: right now the mark_tlb_dirty() will be called if any level of PPGTT table is changed. But for the EXECLIST context submission, we only need LRI command

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Commit planes on each crtc separately.

2015-08-13 Thread Jani Nikula
On Wed, 12 Aug 2015, Ander Conselvan De Oliveira wrote: > For both patches, > > Reviewed-by: Ander Conselvan de Oliveira Both pushed to drm-intel-fixes, thanks for the patches and review. BR, Jani. > > On Tue, 2015-08-11 at 12:31 +0200, Maarten Lankhorst wrote: >> This patch is based on the up

Re: [Intel-gfx] [4.2-rc4] acpi|drm|i915: circular locking dependency: acpi_video_get_backlight_type

2015-08-13 Thread Ville Syrjälä
On Thu, Aug 13, 2015 at 08:52:23AM +0200, Sedat Dilek wrote: > On Wed, Aug 12, 2015 at 9:26 PM, Ville Syrjälä > wrote: > > On Mon, Aug 10, 2015 at 08:29:00PM +0200, Sedat Dilek wrote: > >> On Sat, Aug 1, 2015 at 2:23 PM, Sedat Dilek wrote: > >> > On Mon, Jul 27, 2015 at 12:33 AM, Sedat Dilek >

Re: [Intel-gfx] [PATCH 1/7] drm/i915: Clean up DP/HDMI limited color range handling

2015-08-13 Thread Ville Syrjälä
On Thu, Aug 13, 2015 at 11:46:56AM +0530, Sivakumar Thulasimani wrote: > sdvo is still using color_range name in it's functions. would be good to > rename that as well along with dp & hdmi done here. Doh. I forgot about sdvo completely. I'll take a look to make sure it conforms to the same style.

Re: [Intel-gfx] [PATCH 02/21] drm/i915/gtt: Workaround for HW preload not flushing pdps

2015-08-13 Thread Dave Gordon
On 13/08/15 12:42, Dave Gordon wrote: On 13/08/15 11:12, Michel Thierry wrote: On 8/13/2015 5:08 PM, Zhiyuan Lv wrote: Hi Michel, Thanks for the reply! I yet have another question: right now the mark_tlb_dirty() will be called if any level of PPGTT table is changed. But for the EXECLIST conte

Re: [Intel-gfx] [PATCH] tools/null_state/gen9: Send atleast one valid component in VF state

2015-08-13 Thread Mika Kuoppala
Ben Widawsky writes: > On Fri, Jul 31, 2015 at 04:27:07PM +0100, Arun Siluvery wrote: >> A programming restriction exists for this instruction, atleast one component >> of one valid vertex element must be enabled. >> >> Cc: Ben Widawsky >> Cc: Chris Wilson >> Signed-off-by: Arun Siluvery >> -

Re: [Intel-gfx] [PATCH 3/3] drm/i915/bxt: WA for swapped HPD pins in A stepping

2015-08-13 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 7125 -Summary- Platform Delta drm-intel-nightly Series Applied ILK -1

[Intel-gfx] [PATCH] tools/null_state/gen9: Send all components in VF state

2015-08-13 Thread Mika Kuoppala
Fix commit 59cdc16b1a6f069f944ff17851a59edf8f72d45d Author: Arun Siluvery Date: Fri Jul 31 16:27:07 2015 +0100 tools/null_state/gen9: Send atleast one valid component in VF state to honor the Reviewed-by, send all four components as noted by Ben in his review. Cc: Ben Widawsky Cc: Arun

Re: [Intel-gfx] [PATCH 1/2 v2 addendum v2] drm/i915: Allow parsing of variable size child device entries from VBT

2015-08-13 Thread David Weinehall
On Wed, Aug 12, 2015 at 05:19:35PM +0300, Jani Nikula wrote: > On Wed, 12 Aug 2015, David Weinehall wrote: > > Some more fixup is needed; the bits from Antti's patch > > that actually expanded the struct to fully fit the newer > > versions of the child_device_config was part of the second > > patc

Re: [Intel-gfx] [PATCH v4] mmap on the dma-buf directly

2015-08-13 Thread Tiago Vignatti
On 08/13/2015 08:09 AM, Thomas Hellstrom wrote: Tiago, I take it, this is intended to be a generic interface used mostly for 2D rendering. yup. "generic" is an important point that I've actually forgot to mention in the description, which is probably the whole motivation for bringing this up

Re: [Intel-gfx] [PATCH 3/7] prime_mmap: Add basic tests to write in a bo using CPU

2015-08-13 Thread Tiago Vignatti
On 08/13/2015 04:01 AM, Daniel Vetter wrote: On Wed, Aug 12, 2015 at 08:29:16PM -0300, Tiago Vignatti wrote: This patch adds test_correct_cpu_write, which maps the texture buffer through a prime fd and then writes directly to it using the CPU. It stresses the driver to guarantee cache synchroniz

Re: [Intel-gfx] [4.2-rc4] acpi|drm|i915: circular locking dependency: acpi_video_get_backlight_type

2015-08-13 Thread Hans de Goede
Hi, On 12-08-15 21:26, Ville Syrjälä wrote: On Mon, Aug 10, 2015 at 08:29:00PM +0200, Sedat Dilek wrote: On Sat, Aug 1, 2015 at 2:23 PM, Sedat Dilek wrote: On Mon, Jul 27, 2015 at 12:33 AM, Sedat Dilek wrote: Hi, this my first build of a 4.2-rcN Linux-kernel and I see this... Just FYI:

Re: [Intel-gfx] [PATCH 3/7] prime_mmap: Add basic tests to write in a bo using CPU

2015-08-13 Thread Daniel Vetter
On Thu, Aug 13, 2015 at 11:26:57AM -0300, Tiago Vignatti wrote: > On 08/13/2015 04:01 AM, Daniel Vetter wrote: > >On Wed, Aug 12, 2015 at 08:29:16PM -0300, Tiago Vignatti wrote: > >>This patch adds test_correct_cpu_write, which maps the texture buffer > >>through a > >>prime fd and then writes dir

Re: [Intel-gfx] [PATCH v4] mmap on the dma-buf directly

2015-08-13 Thread Daniel Vetter
On Thu, Aug 13, 2015 at 11:09:07AM -0300, Tiago Vignatti wrote: > On 08/13/2015 08:09 AM, Thomas Hellstrom wrote: > >Tiago, > > > >I take it, this is intended to be a generic interface used mostly for 2D > >rendering. > > yup. "generic" is an important point that I've actually forgot to mention in

[Intel-gfx] [PATCH 0/4] drm/i915: SDVO limited color range and pixel repeat fixes

2015-08-13 Thread ville . syrjala
From: Ville Syrjälä While looking at cleaning up the SDVO limited color range handling I noticed the new FIXMEs that we're not supposed to use the limited color range bit in the port registers. After a bit of work I managed to produce a few patches that should hopefully deal with this correctly.

[Intel-gfx] [PATCH 3/4] drm/i915: Implement limited color range for SDVO properly

2015-08-13 Thread ville . syrjala
From: Ville Syrjälä The SDVO/HDMI port register limited color range bit can only be used with TMDS encoding and not SDVO encoding, ie. to be used only when using the port as a HDMI port as opposed to a SDVO port. To implement limited color range support for SDVO->HDMI we need to ask the SDVO dev

[Intel-gfx] [PATCH 2/4] drm/i915: Fix SDVO colorimetry bit defines

2015-08-13 Thread ville . syrjala
From: Ville Syrjälä Fix up the SDVO colorimetry bits to match the spec. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/intel_sdvo_regs.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_sdvo_regs.h b/drivers/gpu/drm/i915/intel_sdvo_

[Intel-gfx] [PATCH 1/4] drm/i915: Clean up SDVO limited color range handling

2015-08-13 Thread ville . syrjala
From: Ville Syrjälä Clean up the limited color range handling for SDVO as was done for HDMI/DP previously. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/intel_sdvo.c | 27 --- 1 file changed, 12 insertions(+), 15 deletions(-) diff --git a/drivers/gpu/drm/i915/i

[Intel-gfx] [PATCH 4/4] drm/i915: Make SDVO deal with HDMI pixel repeat

2015-08-13 Thread ville . syrjala
From: Ville Syrjälä With SDVO the pipe config pixel_multiplier only concerns itself with the data on the SDVO bus. Any HDMI specific pixel repeat must be handled by the SDVO device itself. To do that simply configure the SDVO pixel replication factor appropriately. We already set up the infoframe

Re: [Intel-gfx] [PATCH 1/3] drm/i915: Dont enable hpd for eDP

2015-08-13 Thread Ville Syrjälä
On Thu, Aug 13, 2015 at 09:35:21AM +0530, Sivakumar Thulasimani wrote: > > > On 8/12/2015 6:26 PM, Daniel Vetter wrote: > > On Mon, Aug 10, 2015 at 05:51:48PM +0530, Sivakumar Thulasimani wrote: > >> > >> On 8/10/2015 5:44 PM, Jani Nikula wrote: > >>> On Mon, 10 Aug 2015, Sivakumar Thulasimani >

Re: [Intel-gfx] [PATCH 02/21] drm/i915/gtt: Workaround for HW preload not flushing pdps

2015-08-13 Thread Zhiyuan Lv
On Thu, Aug 13, 2015 at 01:03:30PM +0100, Dave Gordon wrote: > On 13/08/15 12:42, Dave Gordon wrote: > >On 13/08/15 11:12, Michel Thierry wrote: > >>On 8/13/2015 5:08 PM, Zhiyuan Lv wrote: > >>>Hi Michel, > >>> > >>>Thanks for the reply! > >>> > >>>I yet have another question: right now the mark_tl

Re: [Intel-gfx] [PATCH 2/4] drm/i915: Adding Panel Filter function for DP

2015-08-13 Thread Ville Syrjälä
On Mon, Aug 10, 2015 at 03:26:09PM +0800, Xiong Zhang wrote: > Only internal eDP, LVDS, DVI screen could set scalling mode, some > customers need to set scalling mode for external DP, HDMI, VGA > screen. Let's fulfill this. > > bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90989 > > Sign

Re: [Intel-gfx] [PATCH 2/2] drm/doc: Update docs about device instance setup

2015-08-13 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 7129 -Summary- Platform Delta drm-intel-nightly Series Applied ILK -1

[Intel-gfx] [PATCH] ACPI / video: Fix circular lock dependency issue in the video-detect code

2015-08-13 Thread Hans de Goede
Before this commit, the following would happen: a) acpi_video_get_backlight_type() gets called b) acpi_video_get_backlight_type() calls acpi_video_init_backlight_type() c) acpi_video_init_backlight_type() locks its function static init_mutex d) acpi_video_init_backlight_type() calls backlight_regi

[Intel-gfx] [PATCH igt] lib/igt_core: use print("%s", #expr) instead of print(#expr)

2015-08-13 Thread Paulo Zanoni
If I have a program with the following: igt_skip_on(i % 2 == 0); igt_skip_on_f(i % 2 == 0, "i:%d\n", i); igt_require(i % 2 == 0); igt_require_f(i % 2 == 0, "i:%d\n", i); then I'll get compiler error messages complaining about format conversions related to the '%' character used in

[Intel-gfx] [PATCH 1/2] drm/i915: Assign hwmode after encoder state readout

2015-08-13 Thread ville . syrjala
From: Ville Syrjälä The dotclock is often calculated in encoder .get_config(), so we shouldn't copy the adjusted_mode to hwmode until we have read out the dotclock. Gets rid of some warnings like these: [drm:drm_calc_timestamping_constants [drm]] *ERROR* crtc 21: Can't calculate constants, dotc

[Intel-gfx] [PATCH 2/2] drm/i915: Fix clock readout when pipes are enabld w/o ports

2015-08-13 Thread ville . syrjala
From: Ville Syrjälä The BIOS sometimes likes to enable pipes w/o any ports, at least on older machines. Currently we fail to assign anything sensible to crtc->hwmode.crtc_clock which leads to complaints from the vblank code. Deal with active pipes w/o ports and assign something sensible to crtc_c

[Intel-gfx] [drm-intel:topic/drm-misc 71/71] drivers/gpu/drm/imx/imx-drm-core.c:490:2: warning: initialization from incompatible pointer type

2015-08-13 Thread kbuild test robot
tree: git://anongit.freedesktop.org/drm-intel topic/drm-misc head: b90180b057f77fe5c47a033aaa13a0f6578e5a01 commit: b90180b057f77fe5c47a033aaa13a0f6578e5a01 [71/71] drm/irq: More pipe/crtc consistency cleanups config: arm-imx_v6_v7_defconfig (attached as .config) reproduce: wget https://git

[Intel-gfx] [drm-intel:topic/drm-misc 71/71] drivers/gpu/drm/rcar-du/rcar_du_drv.c:262:2: warning: initialization from incompatible pointer type

2015-08-13 Thread kbuild test robot
tree: git://anongit.freedesktop.org/drm-intel topic/drm-misc head: b90180b057f77fe5c47a033aaa13a0f6578e5a01 commit: b90180b057f77fe5c47a033aaa13a0f6578e5a01 [71/71] drm/irq: More pipe/crtc consistency cleanups config: arm-marzen_defconfig (attached as .config) reproduce: wget https://git.ke

Re: [Intel-gfx] [PATCH v2 5/5] tests: make drm_read platform agnostic

2015-08-13 Thread Micah Fedke
Yup, in fact I see that if the console blanking kicks on, that the drm_read test fails the first time, and then succeed on the second run. It must wake up the console when it runs? (Chris, I tried drm_read without the vblank api switch and it does work just fine - I must have carried that ove

Re: [Intel-gfx] [PATCH] drm/i915: Use CONFIG_DRM_FBDEV_EMULATION

2015-08-13 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 7131 -Summary- Platform Delta drm-intel-nightly Series Applied ILK -2

[Intel-gfx] [PATCH 08/18] tests/gem_mmap: mark basic object creation tests as basic

2015-08-13 Thread Jesse Barnes
We should be able to create small and moderate sized objects quickly and without errors. Signed-off-by: Jesse Barnes --- tests/gem_mmap.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/gem_mmap.c b/tests/gem_mmap.c index 095f5b9..dd66ad6 100644 --- a/tests/gem_mmap

[Intel-gfx] [PATCH 07/18] tests/gem_ctx_exec: mark lrc lite restore as basic

2015-08-13 Thread Jesse Barnes
Need some LRC tests in the 'basic' subset, and this is a good simple one. Signed-off-by: Jesse Barnes --- tests/gem_ctx_exec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/gem_ctx_exec.c b/tests/gem_ctx_exec.c index 3df939c..ea0fb7f 100644 --- a/tests/gem_ctx_exec.c

[Intel-gfx] [PATCH 05/18] tests/drv_getparams: mark EU and subslice fetch as basic

2015-08-13 Thread Jesse Barnes
Fundamental and simple functionality. Signed-off-by: Jesse Barnes --- tests/drv_getparams.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/drv_getparams.c b/tests/drv_getparams.c index 31382e9..5a5c8bb 100644 --- a/tests/drv_getparams.c +++ b/tests/drv_getparams.c

[Intel-gfx] [PATCH 04/18] tests/drm_import_export: mark flink and prime tests as basic

2015-08-13 Thread Jesse Barnes
They're testing basic functionality and don't involve stress or race induction. Signed-off-by: Jesse Barnes --- tests/drm_import_export.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/drm_import_export.c b/tests/drm_import_export.c index e24e0df..6a63ef9 100644 --

[Intel-gfx] [PATCH 03/18] tests/drv_module_reload_basic: use linear_blits after module_reload for sanity check

2015-08-13 Thread Jesse Barnes
Reduces runtime a lot... Signed-off-by: Jesse Barnes --- tests/drv_module_reload_basic | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/drv_module_reload_basic b/tests/drv_module_reload_basic index bb29a64..cf7f2b8 100755 --- a/tests/drv_module_reload_basic +++ b/tests/d

[Intel-gfx] [PATCH 09/18] tests/gem_mmap_gtt: mark basic access and copy tests as basic

2015-08-13 Thread Jesse Barnes
These ones should always pass and are fairly quick. Signed-off-by: Jesse Barnes --- tests/gem_mmap_gtt.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/gem_mmap_gtt.c b/tests/gem_mmap_gtt.c index a95b98a..f964b39 100644 --- a/tests/gem_mmap_gtt.c +++ b/tests/ge

[Intel-gfx] [PATCH 10/18] tests/gem_pread/pwrite: mark normal tests as basic

2015-08-13 Thread Jesse Barnes
These should always pass. Signed-off-by: Jesse Barnes --- tests/gem_pread.c | 2 +- tests/gem_pwrite.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/gem_pread.c b/tests/gem_pread.c index cc83948..3ec5fb1 100644 --- a/tests/gem_pread.c +++ b/tests/gem_pread.c @@ -1

[Intel-gfx] [PATCH 06/18] tests/drv_suspend: mark sysfs tests as basic

2015-08-13 Thread Jesse Barnes
debugfs may not be mounted, but sysfs should always be restored after suspend or hibernate. Signed-off-by: Jesse Barnes --- tests/drv_suspend.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/drv_suspend.c b/tests/drv_suspend.c index d67a794..60ca8e3 100644 --- a/te

[Intel-gfx] [PATCH 01/18] tests/gem_storedw_loop: add new store_dword test to unify per-ring ones

2015-08-13 Thread Jesse Barnes
There was a lot of duplication going on... Mark as basic while we're at it as these should never fail. Signed-off-by: Jesse Barnes --- tests/Makefile.sources | 1 + tests/gem_storedw_loop.c | 181 +++ 2 files changed, 182 insertions(+) create mod

[Intel-gfx] [PATCH 02/18] tests/drv_module_reload: rename drv_module_reload to include in BATs

2015-08-13 Thread Jesse Barnes
Signed-off-by: Jesse Barnes --- tests/Makefile.sources| 2 +- tests/drv_module_reload | 60 --- tests/drv_module_reload_basic | 60 +++ 3 files changed, 61 insertions(+), 61 deletions(-) delete mode 10

Re: [Intel-gfx] [4.2-rc4] acpi|drm|i915: circular locking dependency: acpi_video_get_backlight_type

2015-08-13 Thread Sedat Dilek
On Thu, Aug 13, 2015 at 4:33 PM, Hans de Goede wrote: > Hi, > > On 12-08-15 21:26, Ville Syrjälä wrote: >> >> On Mon, Aug 10, 2015 at 08:29:00PM +0200, Sedat Dilek wrote: >>> >>> On Sat, Aug 1, 2015 at 2:23 PM, Sedat Dilek >>> wrote: On Mon, Jul 27, 2015 at 12:33 AM, Sedat Dilek w

[Intel-gfx] [PATCH 14/18] tests/pm_backlight: mark simple test as basic

2015-08-13 Thread Jesse Barnes
We should be able to adjust the backlight and observe changes in sysfs. Signed-off-by: Jesse Barnes --- tests/pm_backlight.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/pm_backlight.c b/tests/pm_backlight.c index d02336d..cff2694 100644 --- a/tests/pm_backlight.c ++

[Intel-gfx] [PATCH 16/18] tests/kms_flip: add basic tests for flip, flip vs dpms, and flip modeset

2015-08-13 Thread Jesse Barnes
Simple variants that don't do multiple output or interruptible testing. Signed-off-by: Jesse Barnes --- tests/kms_flip.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/tests/kms_flip.c b/tests/kms_flip.c index a595d9f..a0e4112 100644 --- a/tests/kms_flip.c +++ b/

[Intel-gfx] [PATCH 15/18] tests/pm_rpm: mark RTE and D3 tests as basic

2015-08-13 Thread Jesse Barnes
These always need to pass for basic PM functionality. Signed-off-by: Jesse Barnes --- tests/pm_rpm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/pm_rpm.c b/tests/pm_rpm.c index d509fa8..7ae5806 100644 --- a/tests/pm_rpm.c +++ b/tests/pm_rpm.c @@ -1832,11 +1832,1

[Intel-gfx] [PATCH 18/18] gitignore: ignore more files

2015-08-13 Thread Jesse Barnes
--- .gitignore | 3 +++ tests/.gitignore | 13 + tools/.gitignore | 8 3 files changed, 24 insertions(+) diff --git a/.gitignore b/.gitignore index a438c1c..533f6f1 100644 --- a/.gitignore +++ b/.gitignore @@ -90,3 +90,6 @@ gtk-doc.m4 piglit results + +*.orig +ver

[Intel-gfx] [PATCH 13/18] tests/kms_vblank: mark accuracy test as basic

2015-08-13 Thread Jesse Barnes
Need some simple vblank coverage in the BAT list. Signed-off-by: Jesse Barnes --- tests/kms_vblank.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/kms_vblank.c b/tests/kms_vblank.c index 6946177..ef2f7ca 100644 --- a/tests/kms_vblank.c +++ b/tests/kms_vblank.c @@ -188

[Intel-gfx] [PATCH 11/18] tests/gem_tiled_pread/pwrite: mark normal tests as basic

2015-08-13 Thread Jesse Barnes
These simple tests should always pass. Signed-off-by: Jesse Barnes --- tests/gem_tiled_pread.c| 167 + tests/gem_tiled_pread_pwrite.c | 48 ++-- 2 files changed, 112 insertions(+), 103 deletions(-) diff --git a/tests/gem_tiled_pread.c b/t

[Intel-gfx] [PATCH 12/18] tests/kms_addfb: mark simple fb creation tests as basic

2015-08-13 Thread Jesse Barnes
We should always be able to create simple and tiled objects. Signed-off-by: Jesse Barnes --- tests/kms_addfb.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/kms_addfb.c b/tests/kms_addfb.c index 42ee632..f10e12b 100644 --- a/tests/kms_addfb.c +++ b/tests/kms_add

[Intel-gfx] [PATCH 17/18] tests/kms_setmode: mark simple clone test as basic

2015-08-13 Thread Jesse Barnes
Should cover simple, single CRTC mode sets. Signed-off-by: Jesse Barnes --- tests/kms_setmode.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/kms_setmode.c b/tests/kms_setmode.c index 82769ab..2c0fe45 100644 --- a/tests/kms_setmode.c +++ b/tests/kms_setmode.c @@ -685,

Re: [Intel-gfx] [PATCH] drm/i915/bdw: Check for slice, subslice and EU count for BDW

2015-08-13 Thread Bish, Jim
On Thu, 2015-08-13 at 14:38 +0200, Łukasz Daniluk wrote: > Added checks for available slices, subslices and EUs for Broadwell. > This > information is filled in intel_device_info and is available to user > with > GET_PARAM. > Added checks for enabled slices, subslices and EU for Broadwell. This >

Re: [Intel-gfx] [PATCH] lib/rendercopy_gen9: Setup Push constant pointer before sending BTP commands

2015-08-13 Thread Ben Widawsky
On Thu, Aug 13, 2015 at 10:33:00AM +0300, Joonas Lahtinen wrote: > Hi, > > On ke, 2015-08-12 at 18:35 -0700, Ben Widawsky wrote: > > On Wed, Aug 12, 2015 at 03:10:18PM +0300, Joonas Lahtinen wrote: > > > On ke, 2015-08-12 at 12:26 +0100, Arun Siluvery wrote: > > > > From Gen9, by default push cons

Re: [Intel-gfx] [PATCH 0/4] Add support for Hyperlinks and Markup on kernel-doc

2015-08-13 Thread Danilo Cesar Lemes de Paula
On 07/23/2015 05:29 PM, Jonathan Corbet wrote: > On Thu, 23 Jul 2015 15:16:23 -0300 > Danilo Cesar Lemes de Paula wrote: > >> This series add supports for hyperlink cross-references on Docbooks and >> an optional markup syntax for in-source Documentation. > > I like the idea; just be warned that

Re: [Intel-gfx] [PATCH i-g-t 4/3] tests/gem_ctx_param_basic: Expand ctx_param tests

2015-08-13 Thread Jesse Barnes
On 08/10/2015 07:15 AM, David Weinehall wrote: > On Thu, Aug 06, 2015 at 02:33:31PM -0700, Jesse Barnes wrote: >> On 08/06/2015 02:30 PM, Daniel Vetter wrote: >>> On Fri, May 29, 2015 at 09:52:52AM +0200, Daniel Vetter wrote: On Thu, May 28, 2015 at 05:53:17PM +0300, David Weinehall wrote: >>>

Re: [Intel-gfx] [PATCH 0/4] Add support for Hyperlinks and Markup on kernel-doc

2015-08-13 Thread Jonathan Corbet
On Thu, 13 Aug 2015 20:09:35 -0300 Danilo Cesar Lemes de Paula wrote: > Did you find time to take a look on this? No. Just when I thought things couldn't get crazier, my laptop died. https://plus.google.com/+JonathanCorbet/posts/FBHp48dPb95 What spare time I had has been dedicated t

[Intel-gfx] [PATCH] sna: Fix the reduction of xy reflection onto rotations.

2015-08-13 Thread Bob Paauwe
When reducing a xy reflection to a 180 degree rotation, make sure only one rotation bit is set. Also by rotating the bit left, we can support cases where xy reflection happens with 90/270 degree rotation. Signed-off-by: Bob Paauwe --- src/sna/sna_display.c | 5 - 1 file changed, 4 insertion

Re: [Intel-gfx] [PATCH] drm/i915: fix stolen bios_reserved checks

2015-08-13 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 7138 -Summary- Platform Delta drm-intel-nightly Series Applied ILK -1

Re: [Intel-gfx] [PATCH] drm/i915/bdw: Check for slice, subslice and EU count for BDW

2015-08-13 Thread Jeff McGee
On Thu, Aug 13, 2015 at 02:38:48PM +0200, Łukasz Daniluk wrote: > Added checks for available slices, subslices and EUs for Broadwell. This > information is filled in intel_device_info and is available to user with > GET_PARAM. > Added checks for enabled slices, subslices and EU for Broadwell. This

[Intel-gfx] i915/kasan: out of bounds access in i915_cmd_parser_init_ring

2015-08-13 Thread Dave Jones
I finally got around to playing with kasan. It didn't end well. I added some debugging to validate_cmds_sorted to print out the table sizes right before the stack traces. Dave validate_cmds_sorted: table:a1fb4220 cmd_table_count:3 validate_cmds_sorted: table:a1fb4220 tabl

[Intel-gfx] linux-next: manual merge of the drm-tegra tree with the drm-misc tree

2015-08-13 Thread Stephen Rothwell
Hi Thierry, Today's linux-next merge of the drm-tegra tree got conflicts in: drivers/gpu/drm/tegra/dsi.c drivers/gpu/drm/tegra/hdmi.c drivers/gpu/drm/tegra/rgb.c drivers/gpu/drm/tegra/sor.c between commit: 9a69a9ac20f7 ("drm: Make the connector dpms callback return a value, v2") from

[Intel-gfx] linux-next: manual merge of the drm-misc tree with Linus' tree

2015-08-13 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the drm-misc tree got a conflict in: drivers/gpu/drm/drm_irq.c between commit: 209e4dbc8dcd ("drm/vblank: Use u32 consistently for vblank counters") from Linus' tree and commit: b90180b057f7 ("drm/irq: More pipe/crtc consistency cleanups") from the d

[Intel-gfx] [PATCH] drm/i915: fix typo causing bad memory access in ring init

2015-08-13 Thread Dave Airlie
From: Dave Airlie This is validating from the wrong index. testing with KASAN found it. Reported-by: Dave Jones Signed-off-by: Dave Airlie --- drivers/gpu/drm/i915/i915_cmd_parser.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/i915_cmd_parser.c b/

Re: [Intel-gfx] i915/kasan: out of bounds access in i915_cmd_parser_init_ring

2015-08-13 Thread Dave Airlie
On 14 August 2015 at 11:09, Dave Jones wrote: > I finally got around to playing with kasan. It didn't end well. > > I added some debugging to validate_cmds_sorted to print out the table > sizes right before the stack traces. oops typo, patch sent. Dave. _

[Intel-gfx] [drm-intel:topic/drm-misc 71/71] drivers/gpu/drm/nouveau/nouveau_drm.c:961:31: sparse: incorrect type in initializer (incompatible argument 2 (different signedness))

2015-08-13 Thread kbuild test robot
tree: git://anongit.freedesktop.org/drm-intel topic/drm-misc head: b90180b057f77fe5c47a033aaa13a0f6578e5a01 commit: b90180b057f77fe5c47a033aaa13a0f6578e5a01 [71/71] drm/irq: More pipe/crtc consistency cleanups reproduce: # apt-get install sparse git checkout b90180b057f77fe5c47a033aaa13a0f

Re: [Intel-gfx] [PATCH 2/4] drm/i915: Adding Panel Filter function for DP

2015-08-13 Thread Zhang, Xiong Y
> On Mon, Aug 10, 2015 at 03:26:09PM +0800, Xiong Zhang wrote: > > Only internal eDP, LVDS, DVI screen could set scalling mode, some > > customers need to set scalling mode for external DP, HDMI, VGA screen. > > Let's fulfill this. > > > > bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=9098

Re: [Intel-gfx] [PATCH 1/4] drm/i915: Adding intel_panel_scale_none() helper function

2015-08-13 Thread Zhang, Xiong Y
> On Mon, Aug 10, 2015 at 06:23:19PM +, Rodrigo Vivi wrote: > > I believe this function could be added along with the next patch that > > is the first to use it... > > Or it would be good to have a good commit message explaining why this > > function is needed and what is be used for... > > Ye

[Intel-gfx] linux-next: build warnings after merge of the drm-misc tree

2015-08-13 Thread Stephen Rothwell
Hi all, After merging the drm-misc tree, today's linux-next build (arm multi_v7_defconfig) produced these warnings: drivers/gpu/drm/exynos/exynos_drm_drv.c:290:2: warning: initialization from incompatible pointer type .get_vblank_counter = drm_vblank_count, ^ drivers/gpu/drm/exynos/exynos_dr

Re: [Intel-gfx] [PATCH 1/2] Revert "drm/i915: Add eDP intermediate frequencies for CHV"

2015-08-13 Thread Jani Nikula
On Wed, 12 Aug 2015, Daniel Vetter wrote: > On Wed, Aug 12, 2015 at 04:02:17PM +0300, Ville Syrjälä wrote: >> On Wed, Aug 12, 2015 at 05:31:55PM +0530, Sivakumar Thulasimani wrote: >> > >> > >> > On 8/12/2015 5:02 PM, Ville Syrjälä wrote: >> > > On Fri, Jul 31, 2015 at 11:32:52AM +0530, Sivakuma