[Bug 97157] MST displays fail to wake

2016-08-21 Thread bugzilla-dae...@freedesktop.org
was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160821/0f0ecd76/attachment.html>

[Bug 97157] MST displays fail to wake

2016-08-21 Thread bugzilla-dae...@freedesktop.org
bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160821/18b278ad/attachment.html>

[Bug 97428] Specific OpenGL applications deadlock on AMD GPU drivers

2016-08-21 Thread bugzilla-dae...@freedesktop.org
the bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160821/7a2a988c/attachment.html>

[Bug 97428] Specific OpenGL applications deadlock on AMD GPU drivers

2016-08-21 Thread bugzilla-dae...@freedesktop.org
, let me know. -- 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/20160821/3e7e3e0a/attachment.html>

[PATCH 1/2] Revert "include/uapi/drm/amdgpu_drm.h: use __u32 and __u64 from "

2016-08-21 Thread Christian König
Am 20.08.2016 um 19:58 schrieb Mikko Rapeli: > Cc'ing lkml too. > > On Fri, Aug 19, 2016 at 11:54:21PM +0100, Emil Velikov wrote: >> Story time: >> I was dreaming of a day were we can stop installing these headers, >> thus making deprecation a bit easier process. >> Yet after failing to convince

[PATCH] drm/tegra: Fix window[0] base address corruption

2016-08-21 Thread Dmitry Osipenko
Window uses shared stride for UV planes and tegra_dc_window struct defines array of 2 strides per window. That's not taken in account during setting up of the window addresses and strides, resulting in out-of-bounds write of the 3-rd (non-existent) V plane stride that overwrites Y plane base

[Bug 93475] Saints Row IV causes GPU lockup on Linux

2016-08-21 Thread bugzilla-dae...@freedesktop.org
//lists.freedesktop.org/archives/dri-devel/attachments/20160821/f46d0c72/attachment.html>

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

2016-08-21 Thread bugzilla-dae...@freedesktop.org
... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160821/c3f8699a/attachment.html>

[PATCH 1/1] drm/amdgpu/gmc7: remove dead code

2016-08-21 Thread Joe Perches
On Sun, 2016-08-21 at 20:06 +0200, Heinrich Schuchardt wrote: > In an if block for (running == 0) running cannot be non-zero. This code could also be better unindented by one level (all of the block would fit 80 columns) by changing: if (running == 0) { [code...]

[PATCH] drm/gma500: dont expose bytes from kernel stack

2016-08-21 Thread Joe Perches
On Sun, 2016-08-21 at 20:39 +0200, Heinrich Schuchardt wrote: > Components m1, m2, p2, dot, vco of variable clock should be > initialized to avoid bytes from the kernel stack to be > exposed. How was this found? visual code inspection? And isn't this true for mrst_lvds_find_best_pll as well? >

[PATCH 1/1] drm/amdgpu/gmc7: remove dead code

2016-08-21 Thread Joe Perches
On Sun, 2016-08-21 at 20:45 +0200, Heinrich Schuchardt wrote: > On 08/21/2016 08:29 PM, Joe Perches wrote: > > On Sun, 2016-08-21 at 20:06 +0200, Heinrich Schuchardt wrote: > > > In an if block for (running == 0) running cannot be non-zero. > > This code could also be better unindented by one

[PATCH] drm/gma500: dont expose bytes from kernel stack

2016-08-21 Thread Joe Perches
On Sun, 2016-08-21 at 21:35 +0200, Heinrich Schuchardt wrote: > On 08/21/2016 08:46 PM, Joe Perches wrote: > > On Sun, 2016-08-21 at 20:39 +0200, Heinrich Schuchardt wrote: > > > Components m1, m2, p2, dot, vco of variable clock should be > > > initialized to avoid bytes from the kernel stack to

[PATCH 1/1] drm/radeon: avoid NULL dereference, si_get_vce_clock_voltage

2016-08-21 Thread Joe Perches
On Sun, 2016-08-21 at 22:52 +0200, Heinrich Schuchardt wrote: > It does not make sense to check if table is NULL > and afterwards to dereference it without > considering the result. This makes no sense. > The inconsistency was indicated by cppcheck. Perhaps this is a defect in cppcheck? > An

[PATCH 1/1] drm/radeon: avoid NULL dereference, si_get_vce_clock_voltage

2016-08-21 Thread Joe Perches
On Sun, 2016-08-21 at 23:20 +0200, Heinrich Schuchardt wrote: > On 08/21/2016 11:06 PM, Joe Perches wrote: > > On Sun, 2016-08-21 at 22:52 +0200, Heinrich Schuchardt wrote: > > > > > > It does not make sense to check if table is NULL > > > and afterwards to dereference it without > > >

[PATCH 1/3] drm/etnaviv: check for errors when enabling clocks

2016-08-21 Thread Fabio Estevam
clk_prepare_enable() may fail, so we should better check for its return value and propagate it in the case of failure. Signed-off-by: Fabio Estevam --- drivers/gpu/drm/etnaviv/etnaviv_gpu.c | 30 -- 1 file changed, 24 insertions(+), 6 deletions(-) diff --git

[PATCH 2/3] drm/etnaviv: remove unneeded 'fail' label

2016-08-21 Thread Fabio Estevam
In the etnaviv_gpu_platform_probe() error path the 'fail' label is used to just return the error code. This can be simplified by returning the error code immediately, so get rid of the unneeded 'fail' label. Signed-off-by: Fabio Estevam --- drivers/gpu/drm/etnaviv/etnaviv_gpu.c | 12

[PATCH 3/3] drm/etnaviv: remove unneeded variable initialization

2016-08-21 Thread Fabio Estevam
There is no need to initialize variable 'err' with 0 because it will be properly assigned later on. Signed-off-by: Fabio Estevam --- drivers/gpu/drm/etnaviv/etnaviv_gpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/etnaviv/etnaviv_gpu.c

[PATCH -next] drm/i915: Fix non static symbol warning

2016-08-21 Thread Wei Yongjun
From: Wei Yongjun Fixes the following sparse warning: drivers/gpu/drm/i915/intel_hotplug.c:480:6: warning: symbol 'i915_hpd_poll_init_work' was not declared. Should it be static? Also move the '{' to new line. Signed-off-by: Wei Yongjun ---

[PATCH 1/1] drm/amdgpu/gmc7: remove dead code

2016-08-21 Thread Heinrich Schuchardt
In an if block for (running == 0) running cannot be non-zero. Signed-off-by: Heinrich Schuchardt --- drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c | 8 1 file changed, 8 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c index

[PATCH 1/1] drm/amd/powerplay: avoid NULL pointer dereference

2016-08-21 Thread Heinrich Schuchardt
if (a == NULL || a->b == NULL) leads to a NULL pointer dereference if a == NULL. Signed-off-by: Heinrich Schuchardt --- drivers/gpu/drm/amd/powerplay/amd_powerplay.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/powerplay/amd_powerplay.c

[PATCH 1/1] drm/amd/powerplay: avoid NULL dereference, cz_hwmgr.c

2016-08-21 Thread Heinrich Schuchardt
if (a == NULL || a->b == NULL) leads to a NULL pointer dereference if a == NULL. Signed-off-by: Heinrich Schuchardt --- drivers/gpu/drm/amd/powerplay/hwmgr/cz_hwmgr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/cz_hwmgr.c

[PATCH] drm/gma500: dont expose bytes from kernel stack

2016-08-21 Thread Heinrich Schuchardt
On 08/21/2016 08:46 PM, Joe Perches wrote: > On Sun, 2016-08-21 at 20:39 +0200, Heinrich Schuchardt wrote: >> Components m1, m2, p2, dot, vco of variable clock should be >> initialized to avoid bytes from the kernel stack to be >> exposed. > > How was this found? visual code inspection? cppcheck

[PATCH 1/1] drm/radeon/cik: remove dead code

2016-08-21 Thread Heinrich Schuchardt
In an if block for (running == 0) running cannot be non-zero. Signed-off-by: Heinrich Schuchardt --- drivers/gpu/drm/radeon/cik.c | 8 1 file changed, 8 deletions(-) diff --git a/drivers/gpu/drm/radeon/cik.c b/drivers/gpu/drm/radeon/cik.c index 0c1b9ff..c8e1515 100644 ---

[PATCH 1/1] drm/gma500: dont expose stack, mrst_lvds_find_best_pll

2016-08-21 Thread Heinrich Schuchardt
All components of variable clock should be initialized to avoid bytes from the kernel stack to be exposed. Reported-by: Joe Perches Signed-off-by: Heinrich Schuchardt --- drivers/gpu/drm/gma500/oaktrail_crtc.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 1/1] drm/radeon: remove dead code, si_mc_load_microcode

2016-08-21 Thread Heinrich Schuchardt
In an if block for (running == 0) running cannot be non-zero. Signed-off-by: Heinrich Schuchardt --- drivers/gpu/drm/radeon/si.c | 8 1 file changed, 8 deletions(-) diff --git a/drivers/gpu/drm/radeon/si.c b/drivers/gpu/drm/radeon/si.c index 2523ca9..41da47d 100644 ---

[PATCH 1/1] drm/amdgpu/gmc8: remove dead code

2016-08-21 Thread Heinrich Schuchardt
In an if block for (running == 0) running cannot be non-zero. Signed-off-by: Heinrich Schuchardt --- drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c | 8 1 file changed, 8 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c index

[PATCH] drm/gma500: dont expose bytes from kernel stack

2016-08-21 Thread Heinrich Schuchardt
Components m1, m2, p2, dot, vco of variable clock should be initialized to avoid bytes from the kernel stack to be exposed. Signed-off-by: Heinrich Schuchardt --- drivers/gpu/drm/gma500/oaktrail_crtc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/gma500/oaktrail_crtc.c

[PATCH 1/1] drm/amdgpu/gmc7: remove dead code

2016-08-21 Thread Heinrich Schuchardt
On 08/21/2016 08:29 PM, Joe Perches wrote: > On Sun, 2016-08-21 at 20:06 +0200, Heinrich Schuchardt wrote: >> In an if block for (running == 0) running cannot be non-zero. > > This code could also be better unindented by one level > (all of the block would fit 80 columns) by changing: > >

[PATCH 1/1] drm/radeon: avoid NULL dereference, si_get_vce_clock_voltage

2016-08-21 Thread Heinrich Schuchardt
It does not make sense to check if table is NULL and afterwards to dereference it without considering the result. The inconsistency was indicated by cppcheck. An actual NULL pointer dereference was not observed. Signed-off-by: Heinrich Schuchardt --- drivers/gpu/drm/radeon/si_dpm.c | 2 +- 1

[PATCH 1/1] drm: avoid exposing kernel stack in compat_drm_getstats

2016-08-21 Thread Heinrich Schuchardt
The C standard does not specify the size of the integer used to store an enum. Hence in structure drm_stats32_t alignment bytes may exist. To avoid exposing bytes from the kernel stack it is necessary to initialize variable s32 completely. Signed-off-by: Heinrich Schuchardt ---

[v13 PATCH 5/5] drm/rockchip: cdn-dp: add cdn DP support for rk3399

2016-08-21 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

[v13 PATCH 0/5] Rockchip Type-C and DisplayPort driver

2016-08-21 Thread Chris Zhong
Hi all This series patch is for rockchip Type-C phy and DisplayPort controller driver. The USB Type-C PHY is designed to support the USB3 and DP applications. The PHY basically has two main components: USB3 and DisplyPort. USB3 operates in SuperSpeed mode and the DP can operate at RBR, HBR and