Don't use the vrefresh field of the DRM mode since this
one is supposed to only be used for debug purpose.
Instead use the clock field which should also provide
much more precise information.
Also sanitize the case in which the clock value
should be zero. We then just default to the maximum
clock divisor.
Signed-off-by: Tobias Jakobi <tjakobi(a)math.uni-bielefeld.de>
---
drivers/gpu/drm/exynos/exynos_drm_fimd.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/…
[View More]drivers/gpu/drm/exynos/exynos_drm_fimd.c b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
index 57020f9..9d4cae4 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c
@@ -367,9 +367,16 @@ static void fimd_clear_channels(struct exynos_drm_crtc *crtc)
static u32 fimd_calc_clkdiv(struct fimd_context *ctx,
const struct drm_display_mode *mode)
{
- unsigned long ideal_clk = mode->htotal * mode->vtotal * mode->vrefresh;
+ unsigned long ideal_clk;
u32 clkdiv;
+ if (mode->clock == 0) {
+ DRM_ERROR("Mode has zero clock value.\n");
+ return 0xff;
+ }
+
+ ideal_clk = mode->clock * 1000;
+
if (ctx->i80_if) {
/*
* The frame done interrupt should be occurred prior to the
--
2.7.3
[View Less]
Hi Linus,
Fixes for i915, amdgpu/radeon and imx.
The IMX fix is for an autoloading regression found in Fedora.
The radeon fixes, are the same fix to amdgpu/radeon to avoid
a hardware lockup in some circumstances with a bad mode, and
a double free bug I took a few hours chasing down the other
morning.
The i915 fixes are across the board, all stable material,
and fixing some hangs and suspend/resume issues, along
with a live status regressions.
Dave.
The following changes since commit …
[View More]4810d9682971e8eee659f96e4f9d9154e3c6c0b4:
Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security (2016-05-04 11:14:00 -0700)
are available in the git repository at:
git://people.freedesktop.org/~airlied/linux drm-fixes
for you to fetch changes up to fca097169faf8b9cfe92d8926da7a1fa2d3cd452:
Merge tag 'drm-intel-fixes-2016-05-02' of git://anongit.freedesktop.org/drm-intel into drm-fixes (2016-05-05 12:12:09 +1000)
----------------------------------------------------------------
Alex Deucher (2):
drm/radeon: make sure vertical front porch is at least 1
drm/amdgpu: make sure vertical front porch is at least 1
Chris Wilson (1):
drm/i915: Avoid stalling on pending flips for legacy cursor updates
Dave Airlie (3):
drm/amdgpu: set metadata pointer to NULL after freeing.
Merge branch 'drm-fixes-4.6' of git://people.freedesktop.org/~agd5f/linux into drm-fixes
Merge tag 'drm-intel-fixes-2016-05-02' of git://anongit.freedesktop.org/drm-intel into drm-fixes
Imre Deak (2):
drm/i915: Fix system resume if PCI device remained enabled
drm/i915/ddi: Fix eDP VDD handling during booting and suspend/resume
Mika Kahola (1):
drm/i915: Fix eDP low vswing for Broadwell
Philipp Zabel (1):
gpu: ipu-v3: Fix imx-ipuv3-crtc module autoloading
Shashank Sharma (1):
drm/i915: Fake HDMI live status
Ville Syrjälä (1):
drm/i915: Make RPS EI/thresholds multiple of 25 on SNB-BDW
drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 1 +
drivers/gpu/drm/amd/amdgpu/atombios_encoders.c | 4 ++++
drivers/gpu/drm/i915/i915_drv.c | 32 +++++++++++++++++++++++++-
drivers/gpu/drm/i915/i915_reg.h | 9 +++++++-
drivers/gpu/drm/i915/intel_ddi.c | 22 ++++++++++--------
drivers/gpu/drm/i915/intel_display.c | 3 +++
drivers/gpu/drm/i915/intel_dp.c | 4 ++--
drivers/gpu/drm/i915/intel_drv.h | 2 ++
drivers/gpu/drm/i915/intel_hdmi.c | 12 ++++++++--
drivers/gpu/drm/radeon/atombios_encoders.c | 4 ++++
drivers/gpu/ipu-v3/ipu-common.c | 7 +++++-
11 files changed, 84 insertions(+), 16 deletions(-)
[View Less]
https://bugs.freedesktop.org/show_bug.cgi?id=95285
Bug ID: 95285
Summary: [radeonsi] Tomb Raider: enabling TressFX makes hair
disappear
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: NEW
Severity: normal
Priority: medium
Component: Drivers/Gallium/radeonsi
Assignee: dri-devel(a)lists.freedesktop.org
Reporter: kai(a)…
[View More]dev.carbon-project.org
QA Contact: dri-devel(a)lists.freedesktop.org
Created attachment 123497
--> https://bugs.freedesktop.org/attachment.cgi?id=123497&action=edit
No hair with TressFX
When I enable TressFX in Tomb Raider with the stack detailed below, Lara's hair
disappears partially (the part rendered with TressFX). Going back to "normal"
settings makes the hair fully visible again. The attached screenshot shows how
it looks with TressFX enabled.
I thought it might help to set the GLSL and GL overrides to 430 and 4.3
respectively, but then the game launcher refuses to start with tons of
GL_INVALID_OPERATION errors (see <https://paste.debian.net/hidden/84a0115a/>).
Not sure if this is a bug in the launcher or Mesa, but since 4.3 support is
imminent I thought I mention it here as well.
The following stack (Debian testing as a base) was used:
GPU: Hawaii PRO [Radeon R9 290] (ChipID = 0x67b1)
Mesa: Git:master/76a36ac3ea
libdrm: 2.4.67-1
LLVM: SVN:trunk/r268388 (3.9 devel)
X.Org: 2:1.18.3-1
Linux: 4.5.1
Firmware: firmware-amd-graphics/20160110-1
libclc: Git:master/20d977a3e6
DDX: 1:7.7.0-1
Let me know, if you need something else.
--
You are receiving this mail because:
You are the assignee for the bug.
[View Less]
On DCE6.1 PPLL2 is exclusively available to UNIPHYA, so it should not
be taken into consideration when looking for an already enabled PLL
to be shared with other outputs.
This fixes the broken VGA port (TRAVIS DP->VGA bridge) on my Richland
based laptop, where the internal display is connected to UNIPHYA through
a TRAVIS DP->LVDS bridge.
Bug:
https://bugs.freedesktop.org/show_bug.cgi?id=78987
Signed-off-by: Lucas Stach <dev(a)lynxeye.de>
---
drivers/gpu/drm/radeon/atombios_crtc.…
[View More]c | 13 +++++++++----
1 file changed, 9 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/radeon/atombios_crtc.c b/drivers/gpu/drm/radeon/atombios_crtc.c
index b80b08f..401bcbd 100644
--- a/drivers/gpu/drm/radeon/atombios_crtc.c
+++ b/drivers/gpu/drm/radeon/atombios_crtc.c
@@ -1739,7 +1739,8 @@ static u32 radeon_get_pll_use_mask(struct drm_crtc *crtc)
* also in DP mode. For DP, a single PPLL can be used for all DP
* crtcs/encoders.
*/
-static int radeon_get_shared_dp_ppll(struct drm_crtc *crtc)
+static int radeon_get_shared_dp_ppll(struct radeon_device *rdev,
+ struct drm_crtc *crtc)
{
struct drm_device *dev = crtc->dev;
struct drm_crtc *test_crtc;
@@ -1751,6 +1752,10 @@ static int radeon_get_shared_dp_ppll(struct drm_crtc *crtc)
test_radeon_crtc = to_radeon_crtc(test_crtc);
if (test_radeon_crtc->encoder &&
ENCODER_MODE_IS_DP(atombios_get_encoder_mode(test_radeon_crtc->encoder))) {
+ /* PPLL2 is exclusive to UNIPHYA on DCE61 */
+ if (ASIC_IS_DCE61(rdev) && !ASIC_IS_DCE8(rdev) &&
+ test_radeon_crtc->pll_id == ATOM_PPLL2)
+ continue;
/* for DP use the same PLL for all */
if (test_radeon_crtc->pll_id != ATOM_PPLL_INVALID)
return test_radeon_crtc->pll_id;
@@ -1859,7 +1864,7 @@ static int radeon_atom_pick_pll(struct drm_crtc *crtc)
return ATOM_PPLL_INVALID;
else {
/* use the same PPLL for all DP monitors */
- pll = radeon_get_shared_dp_ppll(crtc);
+ pll = radeon_get_shared_dp_ppll(rdev, crtc);
if (pll != ATOM_PPLL_INVALID)
return pll;
}
@@ -1907,7 +1912,7 @@ static int radeon_atom_pick_pll(struct drm_crtc *crtc)
return ATOM_PPLL_INVALID;
else {
/* use the same PPLL for all DP monitors */
- pll = radeon_get_shared_dp_ppll(crtc);
+ pll = radeon_get_shared_dp_ppll(rdev, crtc);
if (pll != ATOM_PPLL_INVALID)
return pll;
}
@@ -1962,7 +1967,7 @@ static int radeon_atom_pick_pll(struct drm_crtc *crtc)
return ATOM_DCPLL;
else {
/* use the same PPLL for all DP monitors */
- pll = radeon_get_shared_dp_ppll(crtc);
+ pll = radeon_get_shared_dp_ppll(rdev, crtc);
if (pll != ATOM_PPLL_INVALID)
return pll;
}
--
2.5.5
[View Less]
Hi Dave, Daniel,
This is a regression causing many failures in intel-gfx CI.
[ 295.164523] [drm:drm_debugfs_init] *ERROR* Cannot create
/sys/kernel/debug/dri/0
[ 295.164531] [drm:drm_minor_register] *ERROR* DRM: Failed to
initialize /sys/kernel/debug/dri.
[ 295.167544] i915: probe of 0000:00:02.0 failed with error -1
After bisect:
commit d2307dea14a4f14a4b5db01b6d40a30fa6117e6c
Author: Dave Airlie <airlied(a)redhat.com>
Date: Wed Apr 27 11:27:39 2016 +1000
drm/atomic: …
[View More]use connector references (v3)
Take a reference when setting a crtc on a connecter,
also take one when duplicating if a crtc is set,
and drop one on destroy if a crtc is set.
v2: take Daniel Stone's advice and simplify the
ref/unref dances, also take care of NULL as connector
to state reset.
v3: remove need for connector NULL check.
Reviewed-by: Daniel Stone <daniels(a)collabora.com>
Reviewed-by: Daniel Vetter <daniel.vetter(a)ffwll.ch>
Signed-off-by: Dave Airlie <airlied(a)redhat.com>
Regards,
Gabriel
[View Less]