[benjamin.gaignard:generic-zpos-v6 4/4] drivers/gpu/drm/rcar-du/rcar_du_vsp.c:51:3: error: unknown field 'zpos' specified in initializer

2016-07-22 Thread kbuild test robot
Intel Corporation -- next part -- A non-text attachment was scrubbed... Name: .config.gz Type: application/octet-stream Size: 57673 bytes Desc: not available URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160722/5a1e03df/attachment-0001.obj>

[v6 PATCH 6/6] drm/rockchip: cdn-dp: add cdn DP support for rk3399

2016-07-22 Thread Mark yao
On 2016年07月21日 21:13, Chris Zhong wrote: > Add support for cdn DP controller which is embedded in the rk3399 > SoCs. The DP is compliant with DisplayPort Specification, > Version 1.3, This IP is compatible with the rockchip type-c PHY IP. > There is a uCPU in DP controller, it need a firmware

[PATCH v1 2/2] drm/bridge: analogix_dp: turn off the panel when eDP need to disable

2016-07-22 Thread Yakir Yang
rm_dev, >> >> phy_power_on(dp->phy); >> >> - if (dp->plat_data->panel) { >> - if (drm_panel_prepare(dp->plat_data->panel)) { >> - DRM_ERROR("failed to setup the panel\n"); >> - return -EBUSY; >> - } >> - } >> - >> analogix_dp_init_dp(dp); >> >> ret = devm_request_threaded_irq(&pdev->dev, dp->irq, >> -- >> 1.9.1 >> >> > > -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160722/5c4a6f5c/attachment.html>

[PATCH v1 1/2] drm/panel: simple-panel: add the delay timing for Sharp LQ123P1JX31

2016-07-22 Thread Yakir Yang
Sean, On 07/21/2016 10:30 PM, Sean Paul wrote: > On Thu, Jul 21, 2016 at 9:14 AM, Yakir Yang wrote: >> According to page 16 of Sharp LQ123P1JX31 datasheet, we need to add the >> missing delay timing. Panel prepare time should be t1 (0.5ms~10ms) plus >> t3 (0ms~100ms), and panel enable time should

[Bug 117591] amdgpu: Black screens on A10-8700P (Carrizo) + R7 M260/M265 (Topaz) Combo

2016-07-22 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=117591 --- Comment #11 from Michel Dänzer --- Thomas, since your symptoms are different from Jani's, please file your own report, preferably at https://bugs.freedesktop.org/enter_bug.cgi?product=DRI&component=DRM/AMDgpu . -- You are receiving this ma

[PATCH v2 1/3] drm/panel: simple-panel: add the delay timing for Sharp LQ123P1JX31

2016-07-22 Thread Yakir Yang
According to page 16 of Sharp LQ123P1JX31 datasheet, we need to add the missing delay timing. Panel prepare time should be t1 (0.5ms~10ms) plus t3 (0ms~100ms), and panel enable time should equal to t7 (0ms~50ms), and panel unprepare time should be t11 (1ms~50ms) plus t12 (500ms~). Signed-off-by: Y

[Bug 97025] flip queue failed: Device or resource busy

2016-07-22 Thread bugzilla-dae...@freedesktop.org
hments/20160722/7b1ebfdd/attachment.html>

[PATCH v2 2/3] drm/bridge: analogix_dp: turn off the panel when eDP need to disable

2016-07-22 Thread Yakir Yang
Some panels (like Sharp LQ123P1JX31) need to be turn off when eDP controller stop to send valid video signal, otherwhise panel would go burn in, and keep flicker and flicker. So it's better to turn off the panel when eDP need to disable, and we need to turn on the panel in connector->detect() call

[PATCH v2 3/3] drm/bridge: analogix_dp: remove the panel prepare/unprepare in suspend/resume

2016-07-22 Thread Yakir Yang
We already manager the panel power status in bridge_disable and connector->detect functions, then we don't need to manager the panel power status at suspend/resume in particular. Signed-off-by: Yakir Yang --- Changes in v2: None drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 12 --

[Bug 97022] Garbage in windows while running a game windowed

2016-07-22 Thread bugzilla-dae...@freedesktop.org
. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160722/1755e5ce/attachment.html>

[v6.1 PATCH 6/6] drm/rockchip: cdn-dp: add cdn DP support for rk3399

2016-07-22 Thread Chris Zhong
Add support for cdn DP controller which is embedded in the rk3399 SoCs. The DP is compliant with DisplayPort Specification, Version 1.3, This IP is compatible with the rockchip type-c PHY IP. There is a uCPU in DP controller, it need a firmware to work, please put the firmware file to /lib/firmware

[Bug 97038] OpenArena couple times slower using llvm 3.9

2016-07-22 Thread bugzilla-dae...@freedesktop.org
. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160722/d441578f/attachment.html>

[PATCH] drm/imx: imx-ldb: store LVDS bus configuration in ldb channel

2016-07-22 Thread Ying Liu
Hi Philipp, This patch's headline doesn't exactly reflect what the patch actually does - retrieve lvds bus format from imx_crtc_state during encoder mode_set. On Thu, Jul 21, 2016 at 9:25 PM, Philipp Zabel wrote: > The code in imx_ldb_encoder_mode_set crashes trying to access the > crtc->state-

[Bug 97039] The Talos Principle and Serious Sam 3 GPU faults

2016-07-22 Thread bugzilla-dae...@freedesktop.org
attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160722/7a65f8db/attachment.html>

[Bug 97025] flip queue failed: Device or resource busy

2016-07-22 Thread bugzilla-dae...@freedesktop.org
ure that the xorg-server update is unrelated. -- You are receiving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160722/a36a4847/attachment.html>

[Bug 97040] NAEV menu gradient render bug

2016-07-22 Thread bugzilla-dae...@freedesktop.org
-- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160722/6dba1741/attachment.html>

[Bug 97040] NAEV menu gradient render bug

2016-07-22 Thread bugzilla-dae...@freedesktop.org
the bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160722/af74cfdd/attachment-0001.html>

[PATCH] GPU-DRM-OMAP: Delete unnecessary checks before two function calls

2016-07-22 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 22 Jul 2016 08:28:31 +0200 The following functions test whether their argument is NULL and then return immediately. * backlight_device_unregister * drm_gem_object_unreference_unlocked Thus the test around the calls is not needed. This issue was detected by using

[Bug 97040] NAEV menu gradient render bug

2016-07-22 Thread bugzilla-dae...@freedesktop.org
attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160722/c09b1d11/attachment.html>

[PATCH 1/1] drm/sti: use new Reset API

2016-07-22 Thread Lee Jones
Since 0b52297f228 ("reset: Add support for shared reset controls") the new Reset API now demands consumers choose either an *_exclusive or a *_shared line when requesting reset lines. Signed-off-by: Lee Jones --- drivers/gpu/drm/sti/sti_compositor.c | 4 ++-- 1 file changed, 2 insertions(+), 2 d

[Bug 97038] OpenArena couple times slower using llvm 3.9

2016-07-22 Thread bugzilla-dae...@freedesktop.org
attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160722/82d12c39/attachment.html>

[Bug 97029] cik_sdma_copy_texture broken on Carrizo (mesa 12)

2016-07-22 Thread bugzilla-dae...@freedesktop.org
assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160722/f9018db4/attachment.html>

[PULL] topic/drm-misc

2016-07-22 Thread Daniel Vetter
Hi Dave, Suddenly everyone shows up with their trivial patch series! - piles of if (!ptr) check removals from Markus Elfring - more of_node_put fixes from Peter Chen - make fbdev support really optional in all drivers (except vmwgfx), somehow this fell through the cracks when we did all the hard

[Bug 97039] The Talos Principle and Serious Sam 3 GPU faults

2016-07-22 Thread bugzilla-dae...@freedesktop.org
achments/20160722/f676c204/attachment.html>

Some more VC4 vblank fixes and timestamping improvements.

2016-07-22 Thread Daniel Vetter
On Thu, Jul 21, 2016 at 12:58:16PM -0700, Eric Anholt wrote: > Mario Kleiner writes: > > > Hi Eric and all, > > > > this batch prevents setting modes one shouldn't set, > > adds precise vblank timestamping for interlaced video > > modes, and one fix for vblank en/disable during crtc > > en/disabl

[PATCH] GPU-DRM-GMA500: Delete unnecessary checks before two function calls

2016-07-22 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 22 Jul 2016 10:30:30 +0200 The functions pci_dev_put() and psb_intel_i2c_destroy() test whether their argument is NULL and then return immediately. Thus the tests around their calls are not needed. This issue was detected by using the Coccinelle software. Signed-

[PATCH] drm/imx: imx-ldb: store LVDS bus configuration in ldb channel

2016-07-22 Thread Philipp Zabel
Hi Liu, thank you for your comments. Am Freitag, den 22.07.2016, 12:01 +0800 schrieb Ying Liu: > Hi Philipp, > > This patch's headline doesn't exactly reflect what the patch actually > does - retrieve lvds bus format from imx_crtc_state during encoder > mode_set. Yes. I initially stored ldb_bus

[PATCH] drm/imx: imx-ldb: do not try to dereference crtc->state->state in encoder mode_set

2016-07-22 Thread Philipp Zabel
The code in imx_ldb_encoder_mode_set crashes with a NULL pointer dereference trying to access crtc->state->state, which was previously cleared by drm_atomic_helper_swap_state: Unable to handle kernel NULL pointer dereference at virtual address 0010 pgd = ae08c000 [0010] *pgd=3e

[Bug 97003] [d3dadapter+radeonsi] Dragon's Dogma: video memory leak with precompiled shaders

2016-07-22 Thread bugzilla-dae...@freedesktop.org
ed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160722/4c40a186/attachment.html>

[PATCH] drm/imx: imx-ldb: do not try to dereference crtc->state->state in encoder mode_set

2016-07-22 Thread Philipp Zabel
Am Freitag, den 22.07.2016, 10:59 +0200 schrieb Philipp Zabel: > + struct drm_crtc_state *crtc_state = encoder->crtc->state; Sorry, this is superfluous. I'll drop it.

[Bug 97003] [d3dadapter+radeonsi] Dragon's Dogma: video memory leak with precompiled shaders

2016-07-22 Thread bugzilla-dae...@freedesktop.org
sts.freedesktop.org/archives/dri-devel/attachments/20160722/6eae9902/attachment.html>

[PATCH] drm/imx: imx-ldb: store LVDS bus configuration in ldb channel

2016-07-22 Thread Ying Liu
Hi Philipp, On Fri, Jul 22, 2016 at 4:59 PM, Philipp Zabel wrote: > Hi Liu, > > thank you for your comments. > > Am Freitag, den 22.07.2016, 12:01 +0800 schrieb Ying Liu: >> Hi Philipp, >> >> This patch's headline doesn't exactly reflect what the patch actually >> does - retrieve lvds bus format

[PATCH] GPU-DRM-GMA500: Delete unnecessary checks before two function calls

2016-07-22 Thread Daniel Vetter
On Fri, Jul 22, 2016 at 10:45:30AM +0200, SF Markus Elfring wrote: > From: Markus Elfring > Date: Fri, 22 Jul 2016 10:30:30 +0200 > > The functions pci_dev_put() and psb_intel_i2c_destroy() test whether > their argument is NULL and then return immediately. > Thus the tests around their calls are

[PATCH] drm/mgag200: Delete an unnecessary check before drm_gem_object_unreference_unlocked()

2016-07-22 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 22 Jul 2016 11:20:27 +0200 The drm_gem_object_unreference_unlocked() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Ma

[PATCH] drm/imx: imx-ldb: do not try to dereference crtc->state->state in encoder mode_set

2016-07-22 Thread Daniel Vetter
On Fri, Jul 22, 2016 at 10:59:51AM +0200, Philipp Zabel wrote: > The code in imx_ldb_encoder_mode_set crashes with a NULL pointer > dereference trying to access crtc->state->state, which was previously > cleared by drm_atomic_helper_swap_state: > > Unable to handle kernel NULL pointer derefere

[Bug 97022] Garbage in windows while running a game windowed

2016-07-22 Thread bugzilla-dae...@freedesktop.org
. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160722/321bc771/attachment.html>

[PATCH v3 1/7] firmware: rockchip: sip: Add rockchip SIP runtime service

2016-07-22 Thread Heiko Stübner
Hi, Am Freitag, 22. Juli 2016, 17:07:14 schrieb Lin Huang: > From: Shengfei xu > > This patch adds support for the SiP interface, we can pass dram > paramtert to bl31, and control ddr frequency scaling in bl31. > > Signed-off-by: Shengfei xu > Signed-off-by: Lin Huang [...] > +++ b/drivers/

[PATCH v3 3/7] clk: rockchip: rk3399: add SCLK_DDRCLK ID for ddrc

2016-07-22 Thread Heiko Stübner
Am Freitag, 22. Juli 2016, 17:07:16 schrieb Lin Huang: > Signed-off-by: Lin Huang > --- > Changes in v3: > -None > > Changes in v2: > - None > Changes in v1: > - None > > include/dt-bindings/clock/rk3399-cru.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/include/dt-bindings/clock/r

[PATCH] GPU-DRM: Delete an unnecessary check before drm_property_unreference_blob()

2016-07-22 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 22 Jul 2016 12:48:12 +0200 The drm_property_unreference_blob() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus E

[PATCH] drm/imx: imx-ldb: do not try to dereference crtc->state->state in encoder mode_set

2016-07-22 Thread Philipp Zabel
Am Freitag, den 22.07.2016, 11:35 +0200 schrieb Daniel Vetter: [...] > Proper fix would be to roll out atomic_ versions of all teh encoder > callbacks where we additionally pass both the crtc state and the connector > state. Then there's no need for walking connector lists like that. And in > the a

[PATCH libdrm] automake: Don't include Android Makefiles in the release tarball

2016-07-22 Thread Andreas Boll
Currently only some Android Makefiles are included in the release tarball. To be more consistent one could either add the remaining files or don't ship Android Makefiles altogether. According to Emil the Android folk doesn't use our release tarballs. Thus it makes sense to remove those files from

[PATCH libdrm] virtgpu: Update kernel header

2016-07-22 Thread Andreas Boll
Generated using make headers_install. This brings the C++ guard, proper include path for drm.h and the switching to kernel types for fixed-with integers. Generated from drm-next commit c11dea5b0290984fa48111957ba3fdc5b3bdae5a Suggested-by: Emil Velikov Signed-off-by: Andreas Boll --- include/

[PATCH libdrm 2/6] automake: Pick up all Android files for distribution

2016-07-22 Thread Andreas Boll
2016-07-21 18:14 GMT+02:00 Emil Velikov : > On 21 July 2016 at 14:12, Andreas Boll wrote: >> Currently only some Android Makefiles are included in the release tarball. >> Add all remaining files to be more consistent. >> > Since Android folk never fully bought the idea of using actual > releases/r

[PATCH libdrm 3/6] libdrm: Fix drm.h include path in virtgpu drm header file

2016-07-22 Thread Andreas Boll
2016-07-21 18:10 GMT+02:00 Emil Velikov : > On 21 July 2016 at 14:12, Andreas Boll wrote: >> A similar change was made to mesa's copy of virtgpu_drm.h by the >> following commit: >> > Please sync this using the approach shown in commit > c745e541a9d8dfd3fb5e1ac57297e58d34d9328f. > > Namely: > - U

[PATCH] drm/vmwgfx: Delete an unnecessary check before the function call "vfree"

2016-07-22 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 22 Jul 2016 13:31:00 +0200 The vfree() function performs also input parameter validation. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpu/drm/vmwgfx/vmwgfx_exec

[PATCH v2] drm/imx: imx-ldb: do not try to dereference crtc->state->state in encoder mode_set

2016-07-22 Thread Philipp Zabel
The code in imx_ldb_encoder_mode_set crashes with a NULL pointer dereference trying to access crtc->state->state, which was previously cleared by drm_atomic_helper_swap_state: Unable to handle kernel NULL pointer dereference at virtual address 0010 pgd = ae08c000 [0010] *pgd=3e

[PATCH 1/2] drm/atomic-helper: Add atomic_mode_set version to extend encoder mode_set

2016-07-22 Thread Philipp Zabel
Some encoders need more information from crtc and connector state than just the mode. Add an atomic encoder mode setting variant that passes the crtc state (which contains the modes) and the connector state. Signed-off-by: Philipp Zabel --- drivers/gpu/drm/drm_atomic_helper.c | 6 +- i

[PATCH 2/2] drm/imx: imx-ldb: use encoder atomic_mode_set callback

2016-07-22 Thread Philipp Zabel
Using atomic_mode_set instead of mode_set allows to access crtc and connector states in addition to the modes. This allows to remove the connector list walk. Signed-off-by: Philipp Zabel --- drivers/gpu/drm/imx/imx-ldb.c | 24 ++-- 1 file changed, 10 insertions(+), 14 deletio

[PATCH] drm/qxl: Delete an unnecessary check before drm_gem_object_unreference_unlocked()

2016-07-22 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 22 Jul 2016 14:14:54 +0200 The drm_gem_object_unreference_unlocked() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Ma

[PATCH v6 4/4] drm: rcar: use generic code for managing zpos plane property

2016-07-22 Thread Benjamin Gaignard
version 6: rebased patch on top rcar-du changes for zpos version 4: fix null pointer issue while setting zpos in plane reset function This patch replaces zpos property handling custom code in rcar DRM driver with calls to generic DRM code. Signed-off-by: Benjamin Gaignard Reviewed-by: Laurent P

[PATCH] drm/bridge: ps8622: Delete an unnecessary check before backlight_device_unregister()

2016-07-22 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 22 Jul 2016 14:45:51 +0200 The backlight_device_unregister() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elf

[PATCH] drm/ttm: partial revert "cleanup ttm_tt_(unbind|destroy)" v2

2016-07-22 Thread Christian König
From: Christian König We still need to unbind explicitely during a move. This partial reverts commit ff20caa0bcbfef9f7686f8d1868a3b990921afd6. v2: remove unnecessary check and unused variable Signed-off-by: Christian König --- drivers/gpu/drm/ttm/ttm_bo_util.c | 1 + drivers/gpu/drm/ttm/t

[PATCH v3 0/7] rk3399 support ddr frequency scaling

2016-07-22 Thread Lin Huang
rk3399 platform have dfi controller can monitor ddr load, and dcf controller to handle ddr register so we can get the right ddr frequency and make ddr controller happy work(which will implement in bl31). So we do ddr frequency scaling with following flow: kernel

[PATCH v3 1/7] firmware: rockchip: sip: Add rockchip SIP runtime service

2016-07-22 Thread Lin Huang
From: Shengfei xu This patch adds support for the SiP interface, we can pass dram paramtert to bl31, and control ddr frequency scaling in bl31. Signed-off-by: Shengfei xu Signed-off-by: Lin Huang --- Changes in v3: - None Changes in v2: - None Changes in v1: - None drivers/firmware/Kconfig

[PATCH v3 2/7] clk: rockchip: add new clock-type for the ddrclk

2016-07-22 Thread Lin Huang
On new rockchip platform(rk3399 etc), there have dcf controller to do ddr frequency scaling, and this controller will implement in arm-trust-firmware. We add a special clock-type to handle that. Signed-off-by: Lin Huang --- Changes in v3: - use sip call to set/read ddr rate Changes in v2: - use

[PATCH v3 3/7] clk: rockchip: rk3399: add SCLK_DDRCLK ID for ddrc

2016-07-22 Thread Lin Huang
Signed-off-by: Lin Huang --- Changes in v3: -None Changes in v2: - None Changes in v1: - None include/dt-bindings/clock/rk3399-cru.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/dt-bindings/clock/rk3399-cru.h b/include/dt-bindings/clock/rk3399-cru.h index 50a44cf..8a0f0442 1006

[PATCH v3 4/7] clk: rockchip: rk3399: add ddrc clock support

2016-07-22 Thread Lin Huang
add ddrc clock setting, so we can do ddr frequency scaling on rk3399 platform in future. Signed-off-by: Lin Huang --- Changes in v3: - None Changes in v2: - remove clk_ddrc_dpll_src from critical clock list Changes in v1: - remove ddrc source CLK_IGNORE_UNUSED flag - move clk_ddrc and clk_ddrc_

[PATCH v3 5/7] PM / devfreq: event: support rockchip dfi controller

2016-07-22 Thread Lin Huang
on rk3399 platform, there is dfi conroller can monitor ddr load, base on this result, we can do ddr freqency scaling. Signed-off-by: Lin Huang Acked-by: Chanwoo Choi --- Changes in v3: - None Changes in v2: - use clk_disable_unprepare and clk_enable_prepare - remove clk_enable_prepare in probe

[PATCH v3 6/7] PM / devfreq: rockchip: add devfreq driver for rk3399 dmc

2016-07-22 Thread Lin Huang
base on dfi result, we do ddr frequency scaling, register dmc driver to devfreq framework, and use simple-ondemand policy. Signed-off-by: Lin Huang --- Changes in v3: - operate dram setting through sip call - imporve set rate flow Changes in v2: - None Changes in v1: - move dfi controller to ev

[PATCH v3 7/7] drm/rockchip: Add dmc notifier in vop driver

2016-07-22 Thread Lin Huang
when in ddr frequency scaling process, vop can not do enable or disable operate, since dcf will base on vop vblank time to do frequency scaling and need to get vop irq if there have vop enabled. So need register to dmc notifier, and we can get the dmc status. Signed-off-by: Lin Huang --- Changes

[Intel-gfx] [PATCH i-g-t] tests/kms_rotation_crc: Add bad-rotation subtest

2016-07-22 Thread Matthew Auld
I believe you're thinking of: https://patchwork.freedesktop.org/patch/77191/ https://patchwork.freedesktop.org/patch/77192/ Although they don't test for multiple rotation values...

[PATCH] drm/ttm: partial revert "cleanup ttm_tt_(unbind|destroy)" v2

2016-07-22 Thread Edward O'Callaghan
*ttm); > + > +/** > * ttm_tt_swapin: > * > * @ttm: The struct ttm_tt. > -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: OpenPGP digital signature URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160722/e52e227a/attachment-0001.sig>

[Intel-gfx] [PATCH i-g-t] tests/kms_rotation_crc: Add bad-rotation subtest

2016-07-22 Thread Ville Syrjälä
On Fri, Jul 22, 2016 at 01:52:32PM +0100, Matthew Auld wrote: > I believe you're thinking of: > https://patchwork.freedesktop.org/patch/77191/ > https://patchwork.freedesktop.org/patch/77192/ > > Although they don't test for multiple rotation values... I guess you could just for (rotation = 0; r

[PATCH 00/15] drm: Per-plane rotation etc.

2016-07-22 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Here's an expaned version of my earlier series [1]. This time I went as far as nuking the mode_config.rotation_property in favor of the per-plane property. Also tried to fix a few buglets in omap/msm rotation property setup. Entire series is available here: git://github.com

[PATCH 01/15] drm: Add drm_rotation_90_or_270()

2016-07-22 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä We have intel_rotation_90_or_270() in i915 to check if the rotation is 90 or 270 degrees. Similar checks are elsewhere in drm, so let's move the helper into a central place and use it everwhere. Signed-off-by: Ville Syrjälä Reviewed-by: Joonas Lahtinen Reviewed-by: Chri

[PATCH 02/15] drm/atomic: Reject attempts to use multiple rotation angles at once

2016-07-22 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä The rotation property should only accept exactly one rotation angle at once. Let's reject attempts to set none or multiple angles. Testcase: igt/kms_rotation_crc/bad-rotation Signed-off-by: Ville Syrjälä Reviewed-by: Joonas Lahtinen Reviewed-by: Chris Wilson --- drive

[PATCH v2 03/15] drm: Add support for optional per-plane rotation property

2016-07-22 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Not all planes on the ssytem may support the same rotations/reflections, so make it possible to create a separate property for each plane. This way userspace gets told exactly which rotations/reflections are possible for each plane. v2: Add drm_plane_create_rotation_propert

[PATCH 05/15] drm/atmel-hlcdc: Use per-plane rotation property

2016-07-22 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä The global mode_config.rotation_property is going away, switch over to per-plane rotation_property. Cc: Boris Brezillon Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 18 ++ 1 file changed, 6 insertions(+), 12 deletio

[PATCH 07/15] drm/omap: Use per-plane rotation property

2016-07-22 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä The global mode_config.rotation_property is going away, switch over to per-plane rotation_property. Not sure I got the annoying crtc rotation_property handling right. Might work, or migth not. Cc: Tomi Valkeinen Cc: Rob Clark Signed-off-by: Ville Syrjälä --- drivers/

[PATCH 08/15] drm/msm/mdp5: Set rotation property initial value to BIT(DRM_ROTATE_0) insted of 0

2016-07-22 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä 0 isn't a valid rotation property value, so let's set the initial value of the property to BIT(DRM_ROTATE_0) instead. In the same vein, we must always have at leat one angle as part of set of supported rotation bits, so let's include BIT(DRM_ROTATE_0) in there. Cc: Rob Cla

[PATCH 09/15] drm/msm/mdp5: Use per-plane rotation property

2016-07-22 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä The global mode_config.rotation_property is going away, switch over to per-plane rotation_property. Cc: Rob Clark Cc: Jilai Wang Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/msm/mdp/mdp5/mdp5_plane.c | 15 +-- 1 file changed, 5 insertions(+), 10 deleti

[PATCH 12/15] drm: RIP mode_config->rotation_property

2016-07-22 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Now that all drivers have been converted over to the per-plane rotation property, we can just nuke the global rotation property. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/drm_atomic.c| 6 ++ drivers/gpu/drm/drm_crtc.c | 18 -- driver

[PATCH v2 11/15] drm/i915: Use the per-plane rotation property

2016-07-22 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä On certain platforms not all planes support the same set of rotations/reflections, so let's use the per-plane property for this. This is already a problem on SKL when we use the legay cursor plane as it only supports 0|180 whereas the universal planes support 0|90|180|270,

[PATCH 13/15] drm/i915: Use & instead if == to check for rotations

2016-07-22 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Using == to check for 180 degree rotation only works as long as the reflection bits aren't set. That will change soon enough for CHV, so let's stop doing things the wrong way. Signed-off-by: Ville Syrjälä Reviewed-by: Chris Wilson Reviewed-by: Joonas Lahtinen --- driv

[PATCH 14/15] drm/i915: Clean up rotation DSPCNTR/DVSCNTR/etc. setup

2016-07-22 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Move the plane control register rotation setup away from the coordinate munging code. This will result in neater looking code once we add reflection support for CHV. Signed-off-by: Ville Syrjälä Reviewed-by: Chris Wilson Reviewed-by: Joonas Lahtinen --- drivers/gpu/dr

[PATCH 15/15] drm/i915: Add horizontal mirroring support for CHV pipe B planes

2016-07-22 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä The primary and sprite planes on CHV pipe B support horizontal mirroring. Expose it to the world. Sadly the hardware ignores the mirror bit when the rotate bit is set, so we'll have to reject the 180+X case. Signed-off-by: Ville Syrjälä Reviewed-by: Joonas Lahtinen ---

[PATCH 04/15] drm/arm: Use per-plane rotation property

2016-07-22 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä The global mode_config.rotation_property is going away, switch over to per-plane rotation_property. Cc: Liviu Dudau Cc: Brian Starkey Cc: Mali DP Maintainers Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/arm/malidp_planes.c | 13 + 1 file changed, 5 in

[PATCH 06/15] drm/omap: Set rotation property initial value to BIT(DRM_ROTATE_0) insted of 0

2016-07-22 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä 0 isn't a valid rotation property value, so let's set the initial value of the property to BIT(DRM_ROTATE_0) instead. Cc: Tomi Valkeinen Cc: Rob Clark Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/omapdrm/omap_drv.c | 6 -- drivers/gpu/drm/omapdrm/omap_plane.

[PATCH 10/15] drm/msm/mdp5: Advertize 180 degree rotation

2016-07-22 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Since the hardware can apparently do both X and Y reflection, we can advertize also 180 degree rotation as thats just X+Y reflection. Cc: Rob Clark Cc: Jilai Wang Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/msm/mdp/mdp5/mdp5_plane.c | 20 1 f

[PATCH 05/15] drm/atmel-hlcdc: Use per-plane rotation property

2016-07-22 Thread Boris Brezillon
Hi Ville, On Fri, 22 Jul 2016 16:43:06 +0300 ville.syrjala at linux.intel.com wrote: > From: Ville Syrjälä > > The global mode_config.rotation_property is going away, switch over to > per-plane rotation_property. > > Cc: Boris Brezillon > Signed-off-by: Ville Syrjälä > --- > drivers/gpu

[PATCH v3 0/3] New debugfs API for capturing CRC of frames

2016-07-22 Thread Tomeu Vizoso
Hi, this series basically takes the facility for continuously capturing CRCs of frames from the i915 driver and into the DRM core. The idea is that test suites such as IGT use this information to check that frames that are exected to be identical, also have identical CRC values. Other drivers fo

[PATCH v3 1/3] drm/i915/debugfs: Move out pipe CRC code

2016-07-22 Thread Tomeu Vizoso
In preparation to using a generic API in the DRM core for continuous CRC generation, move the related code out of i915_debugfs.c into a new file. Eventually, only the Intel-specific code will remain in this new file. v2: Rebased. Signed-off-by: Tomeu Vizoso --- drivers/gpu/drm/i915/Makefile

[PATCH v3 2/3] drm: Add API for capturing frame CRCs

2016-07-22 Thread Tomeu Vizoso
Adds files and directories to debugfs for controlling and reading frame CRCs, per CRTC: dri/0/crtc-0/crc dri/0/crtc-0/crc/control dri/0/crtc-0/crc/data Drivers can implement the set_crc_source callback() in drm_crtc_funcs to start and stop generating frame CRCs and can add entries to the output b

[PATCH v3 3/3] drm/i915: Use new CRC debugfs API

2016-07-22 Thread Tomeu Vizoso
The core provides now an ABI to userspace for generation of frame CRCs, so implement the ->set_crc_source() callback and reuse as much code as possible with the previous ABI implementation. v2: - Leave the legacy implementation in place as the ABI implementation in the core is incompatib

Why drm-mipi-dsi is built-in only?

2016-07-22 Thread Takashi Iwai
Hi, is there any reason drm-mipi-dsi can't be a module? It's fixed as a built-in since its Kconfig is bool. thanks, Takashi

[PATCH 04/15] drm/arm: Use per-plane rotation property

2016-07-22 Thread Brian Starkey
On Fri, Jul 22, 2016 at 04:43:05PM +0300, Ville Syrjälä wrote: >From: Ville Syrjälä > >The global mode_config.rotation_property is going away, switch over to >per-plane rotation_property. > >Cc: Liviu Dudau >Cc: Brian Starkey >Cc: Mali DP Maintainers >Signed-off-by: Ville Syrjälä Acked-

[PATCH v1 2/2] drm/bridge: analogix_dp: turn off the panel when eDP need to disable

2016-07-22 Thread Sean Paul
On Thu, Jul 21, 2016 at 9:00 PM, Yakir Yang wrote: > Sean, > > Thanks for your fast respond :-) > > But this patch is not the latest one, I have upgraded this to "v1.1" version > to fix the eDP can't be disabled problem: > [PATCH v1.1 2/2] drm/bridge: analogix_dp: turn off the panel when eDP >

[PATCH] drm/ttm: partial revert "cleanup ttm_tt_(unbind|destroy)" v2

2016-07-22 Thread Sean Paul
On Fri, Jul 22, 2016 at 9:10 AM, Christian König wrote: > From: Christian König > > We still need to unbind explicitely during a move. Sorry for the drive-by nit, but: s/explicitely/explicitly/ > > This partial reverts commit ff20caa0bcbfef9f7686f8d1868a3b990921afd6. > > v2: remove unnecess

[PATCH 1/1] drm/sti: use new Reset API

2016-07-22 Thread Sean Paul
On Fri, Jul 22, 2016 at 3:22 AM, Lee Jones wrote: > Since 0b52297f228 ("reset: Add support for shared reset controls") the > new Reset API now demands consumers choose either an *_exclusive or > a *_shared line when requesting reset lines. > > Signed-off-by: Lee Jones > --- > drivers/gpu/drm/sti

[PATCH] drm/mgag200: Delete an unnecessary check before drm_gem_object_unreference_unlocked()

2016-07-22 Thread Sean Paul
On Fri, Jul 22, 2016 at 5:32 AM, SF Markus Elfring wrote: > From: Markus Elfring > Date: Fri, 22 Jul 2016 11:20:27 +0200 > > The drm_gem_object_unreference_unlocked() function tests whether > its argument is NULL and then returns immediately. > Thus the test around the call is not needed. > > Thi

[PATCH] drm/vmwgfx: Delete an unnecessary check before the function call "vfree"

2016-07-22 Thread Sean Paul
On Fri, Jul 22, 2016 at 7:45 AM, SF Markus Elfring wrote: > From: Markus Elfring > Date: Fri, 22 Jul 2016 13:31:00 +0200 > > The vfree() function performs also input parameter validation. > Thus the test around the call is not needed. > > This issue was detected by using the Coccinelle software.

[PATCH] drm/qxl: Delete an unnecessary check before drm_gem_object_unreference_unlocked()

2016-07-22 Thread Sean Paul
On Fri, Jul 22, 2016 at 8:22 AM, SF Markus Elfring wrote: > From: Markus Elfring > Date: Fri, 22 Jul 2016 14:14:54 +0200 > > The drm_gem_object_unreference_unlocked() function tests whether > its argument is NULL and then returns immediately. > Thus the test around the call is not needed. > > Thi

[PATCH] drm/bridge: ps8622: Delete an unnecessary check before backlight_device_unregister()

2016-07-22 Thread Sean Paul
On Fri, Jul 22, 2016 at 8:54 AM, SF Markus Elfring wrote: > From: Markus Elfring > Date: Fri, 22 Jul 2016 14:45:51 +0200 > > The backlight_device_unregister() function tests whether its argument > is NULL and then returns immediately. > Thus the test around the call is not needed. > > This issue

[PATCH] drm/vmwgfx: Delete an unnecessary check before the function call "vfree"

2016-07-22 Thread Sinclair Yeh
Looks good. Thanks! Reviewed-by: Sinclair Yeh On Fri, Jul 22, 2016 at 01:45:40PM +0200, SF Markus Elfring wrote: > From: Markus Elfring > Date: Fri, 22 Jul 2016 13:31:00 +0200 > > The vfree() function performs also input parameter validation. > Thus the test around the call is not needed. >

[Bug 97022] Garbage in windows while running a game windowed

2016-07-22 Thread bugzilla-dae...@freedesktop.org
-- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160722/23378732/attachment-0001.html>

[Bug 97022] Garbage in windows while running a game windowed

2016-07-22 Thread bugzilla-dae...@freedesktop.org
nee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160722/5f789669/attachment.html>

[Bug 97022] Garbage in windows while running a game windowed

2016-07-22 Thread bugzilla-dae...@freedesktop.org
the bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160722/e5f2aa41/attachment.html>

[PATCH 0/4] GPU-DRM-Etnaviv: Fine-tuning for a few functions

2016-07-22 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 22 Jul 2016 17:34:32 +0200 Further update suggestions were taken into account after a patch was applied from static source code analysis. Markus Elfring (4): Delete unnecessary checks before two function calls Delete unnecessary if statement in __etnaviv_gem_n

[PATCH v2 05/15] drm/atmel-hlcdc: Use per-plane rotation property

2016-07-22 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä The global mode_config.rotation_property is going away, switch over to per-plane rotation_property. v2: Propagate error upwards (Boris) Cc: Boris Brezillon Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 37 +

[PATCH 1/4] GPU-DRM-Etnaviv: Delete unnecessary checks before two function calls

2016-07-22 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 22 Jul 2016 15:56:15 +0200 The functions drm_gem_object_unreference_unlocked() and vunmap() perform also input parameter validation. Thus the tests around their calls are not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus

  1   2   >