[PATCH v3 04/12] drm/i915: Intercept Aksv writes in the aux hooks

2020-01-17 Thread Sean Paul
From: Sean Paul Instead of hand rolling the transfer ourselves in the hdcp hook, inspect aux messages and add the aksv flag in the aux transfer hook. IIRC, this was the original implementation and folks wanted this hack to be isolated to the hdcp code, which makes sense. However in testing an L

[PATCH v3 06/12] drm/i915: Factor out hdcp->value assignments

2020-01-17 Thread Sean Paul
From: Sean Paul This is a bit of housecleaning for a future patch. Instead of sprinkling hdcp->value assignments and prop_work scheduling everywhere, introduce a function to do it for us. Reviewed-by: Ramalingam C Signed-off-by: Sean Paul Link: https://patchwork.freedesktop.org/patch/msgid/20

[PATCH v3 11/12] drm/i915: Factor out HDCP shim functions from dp for use by dp_mst

2020-01-17 Thread Sean Paul
From: Sean Paul These functions are all the same for dp and dp_mst, so move them into a dedicated file for both sst and mst to use. Signed-off-by: Sean Paul Link: https://patchwork.freedesktop.org/patch/msgid/20191203173638.94919-11-s...@poorly.run #v1 Link: https://patchwork.freedesktop.org

Re: [PATCH 0/6] video: fbdev: controlfb: small cleanup

2020-01-17 Thread Sam Ravnborg
Hi Bartlomiej On Thu, Jan 16, 2020 at 03:08:54PM +0100, Bartlomiej Zolnierkiewicz wrote: > Hi, > > Small cleanup for controlfb driver: > > - fix sparse warnings > - remove not working module support > - add COMPILE_TEST support > - remove redundant function prototypes Nice cleanup - with a few

Re: [Bug 206231] R9 280X low performance with all games

2020-01-17 Thread sylvain . bertrand
Wow, this from the first tomb raider, the one from 2013!! (7 years ago). Sorry I read the emails in the wrong order. I don't know own this game, sorry. Another game perhaps? ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freede

[Bug 206231] R9 280X low performance with all games

2020-01-17 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206231 --- Comment #10 from Sylvain BERTRAND (sylvain.bertr...@gmail.com) --- Wow, this from the first tomb raider, the one from 2013!! (7 years ago). Sorry I read the emails in the wrong order. I don't know own this game, sorry. Another game perhaps?

[Bug 206231] R9 280X low performance with all games

2020-01-17 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206231 --- Comment #11 from Sylvain BERTRAND (sylvain.bertr...@gmail.com) --- Dota2? (vulkan and GL) CS:GO? (GL) Those games are free. -- You are receiving this mail because: You are watching the assignee of the bug. __

Re: [Bug 206231] R9 280X low performance with all games

2020-01-17 Thread sylvain . bertrand
Dota2? (vulkan and GL) CS:GO? (GL) Those games are free. ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH 2/4] drm/mst: Some style improvements in drm_dp_mst_topology_mgr_set_mst()

2020-01-17 Thread Lyude Paul
Reviewed-by: Lyude Paul On Thu, 2020-01-16 at 17:58 -0800, José Roberto de Souza wrote: > Removing this lose code block and removing unnecessary bracket. > > Cc: Lyude Paul > Signed-off-by: José Roberto de Souza > --- > drivers/gpu/drm/drm_dp_mst_topology.c | 14 ++ > 1 file chang

Re: [PATCH 1/4] drm/mst: Don't do atomic checks over disabled managers

2020-01-17 Thread Lyude Paul
Reviewed-by: Lyude Paul On Thu, 2020-01-16 at 17:58 -0800, José Roberto de Souza wrote: > When a main MST port is disconnected drivers should call > drm_dp_mst_topology_mgr_set_mst() disabling the MST manager, this > function will set manager mst_primary to NULL and it will cause the > crash bell

Re: [PATCH v3 4/5] drm/i915: Don't use VBT for detecting DPCD backlight controls

2020-01-17 Thread Lyude Paul
On Fri, 2020-01-17 at 13:36 +0200, Jani Nikula wrote: > On Thu, 16 Jan 2020, Lyude Paul wrote: > > Despite the fact that the VBT appears to have a field for specifying > > that a system is equipped with a panel that supports standard VESA > > backlight controls over the DP AUX channel, so far ever

Re: [PATCH v9 12/18] drm/dp_mst: Add branch bandwidth validation to MST atomic check

2020-01-17 Thread Lyude Paul
fwiw - https://patchwork.freedesktop.org/patch/349111/ is the proper fix that I was talking about, we should probably go with that one instead if you haven't already pushed this On Fri, 2020-01-17 at 10:39 -0500, Sean Paul wrote: > On Fri, Jan 17, 2020 at 10:26 AM Mikita Lipski wrote: > > > > >

Re: [PATCH v2] drm/dp_mst: Remove VCPI while disabling topology mgr

2020-01-17 Thread Lyude Paul
On Fri, 2020-01-17 at 11:19 -0500, Sean Paul wrote: > On Mon, Dec 9, 2019 at 12:56 AM Lin, Wayne wrote: > > > > > > > -Original Message- > > > From: Lyude Paul > > > Sent: Saturday, December 7, 2019 3:57 AM > > > To: Lin, Wayne ; dri-devel@lists.freedesktop.org; > > > amd-...@lists.free

Re: [PATCH v2] drm/dp_mst: Remove VCPI while disabling topology mgr

2020-01-17 Thread Sean Paul
On Fri, Jan 17, 2020 at 3:27 PM Lyude Paul wrote: > > On Fri, 2020-01-17 at 11:19 -0500, Sean Paul wrote: > > On Mon, Dec 9, 2019 at 12:56 AM Lin, Wayne wrote: > > > > > > > > > > -Original Message- > > > > From: Lyude Paul > > > > Sent: Saturday, December 7, 2019 3:57 AM > > > > To: Lin

Re: [PATCH v2] drm/dp_mst: Remove VCPI while disabling topology mgr

2020-01-17 Thread Lyude Paul
Yeah that's fine with me, I'll send out a revert for this in just a moment On Fri, 2020-01-17 at 15:43 -0500, Sean Paul wrote: > On Fri, Jan 17, 2020 at 3:27 PM Lyude Paul wrote: > > On Fri, 2020-01-17 at 11:19 -0500, Sean Paul wrote: > > > On Mon, Dec 9, 2019 at 12:56 AM Lin, Wayne wrote: > > >

[PATCH] Revert "drm/dp_mst: Remove VCPI while disabling topology mgr"

2020-01-17 Thread Lyude Paul
This reverts commit 64e62bdf04ab8529f45ed0a85122c703035dec3a. This commit ends up causing some lockdep splats due to trying to grab the payload lock while holding the mgr's lock: [ 54.010099] [ 54.011765] == [ 54.018670] WARNING: possible

Re: [PATCH] Revert "drm/dp_mst: Remove VCPI while disabling topology mgr"

2020-01-17 Thread Sean Paul
On Fri, Jan 17, 2020 at 3:51 PM Lyude Paul wrote: > > This reverts commit 64e62bdf04ab8529f45ed0a85122c703035dec3a. > > This commit ends up causing some lockdep splats due to trying to grab the > payload lock while holding the mgr's lock: > > [ 54.010099] > [ 54.011765] ===

Re: [PATCH v2 3/5] drm/mipi_dbi: Add support for display offsets

2020-01-17 Thread Sam Ravnborg
Hi Noralf. If you could find time to review this patch it would be great. I am reluctant to apply it until you have acked/reviewed it. Thanks in advance, Sam On Wed, Jan 15, 2020 at 01:45:46PM +0100, Geert Uytterhoeven wrote: > If the resolution of the TFT display is smaller than the ma

[Bug 206141] VCE UVD ring test failed -110

2020-01-17 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206141 --- Comment #8 from Alex Deucher (alexdeuc...@gmail.com) --- (In reply to Janpieter Sollie from comment #7) > Is it possible to disable the UVD/VCE engine on the original GPU? > I mean, it's not used anyway, so I might as well disable it completel

Re: [PATCH 2/4] drm/mst: Some style improvements in drm_dp_mst_topology_mgr_set_mst()

2020-01-17 Thread Lyude Paul
JFYI: I'm going to go ahead and push this patch by itself to drm-misc-next since it applies cleanly, the other patches in this series don't depend on this, and I'm about to send out a patch that modifies the code around these hunks anyway. On Thu, 2020-01-16 at 17:58 -0800, José Roberto de Souza w

Re: [PATCH 1/4] drm/mst: Don't do atomic checks over disabled managers

2020-01-17 Thread Alex Deucher
On Fri, Jan 17, 2020 at 3:21 PM Lyude Paul wrote: > > Reviewed-by: Lyude Paul > Applied. I'll send a PR shortly. Thanks! Alex > On Thu, 2020-01-16 at 17:58 -0800, José Roberto de Souza wrote: > > When a main MST port is disconnected drivers should call > > drm_dp_mst_topology_mgr_set_mst() d

[pull] amdgpu, amdkfd, scheduler, drm drm-next-5.6

2020-01-17 Thread Alex Deucher
Hi Dave, Daniel, Same PR as yesterday, this just adds a DP MST fix for the DSC code and a mutex init regression on top. The following changes since commit 688486a49cf500a193dfe15be9eb5aa468887769: Merge tag 'amd-drm-next-5.6-2020-01-10-dp-mst-dsc' of git://people.freedesktop.org/~agd5f/linux

[PATCH 1/2] drm/dp_mst: Fix indenting in drm_dp_mst_topology_mgr_set_mst()

2020-01-17 Thread Lyude Paul
This has always bugged me but somehow I've never remembered to actually fix it. So let's do that. Signed-off-by: Lyude Paul --- drivers/gpu/drm/drm_dp_mst_topology.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c b/drivers/gpu/drm/dr

[PATCH 2/2] drm/dp_mst: Fix clearing payload state on topology disable

2020-01-17 Thread Lyude Paul
The issues caused by: 64e62bdf04ab ("drm/dp_mst: Remove VCPI while disabling topology mgr") Prompted me to take a closer look at how we clear the payload state in general when disabling the topology, and it turns out there's actually two subtle issues here. The first is that we're not grabbing &

[PATCH v2] dt-bindings: display: Convert a bunch of panels to DT schema

2020-01-17 Thread Rob Herring
Convert all the 'simple' panels which match the constraints of the common panel-simple.yaml schema. This conversion is based on how the panels are documented. Some may turn out to be more complex once the schema is applied to actual dts files. Cc: Maxime Ripard Cc: Chen-Yu Tsai Cc: Thierry Redin

[PATCH v4] drm/i915: Don't use VBT for detecting DPCD backlight controls

2020-01-17 Thread Lyude Paul
Despite the fact that the VBT appears to have a field for specifying that a system is equipped with a panel that supports standard VESA backlight controls over the DP AUX channel, so far every system we've spotted DPCD backlight control support on doesn't actually set this field correctly and all h

Re: [PATCH v2 1/1] drm/dp_mst: Handle SST-only branch device case

2020-01-17 Thread Lyude Paul
Pushed to drm-misc-fixes, thanks! On Fri, 2020-01-17 at 14:03 +0800, Wayne Lin wrote: > [Why] > While handling LINK_ADDRESS reply, current code expects a peer device > can handle sideband message once the peer device type is reported as > DP_PEER_DEVICE_MST_BRANCHING. However, when the connected d

Re: [PATCH -next] drm/nouveau/kms/nv50: remove set but not unused variable 'nv_connector'

2020-01-17 Thread Lyude Paul
Reviewed-by: Lyude Paul On Fri, 2020-01-17 at 11:36 +0800, YueHaibing wrote: > drivers/gpu/drm/nouveau/dispnv50/disp.c: In function nv50_pior_enable: > drivers/gpu/drm/nouveau/dispnv50/disp.c:1672:28: warning: > variable nv_connector set but not used [-Wunused-but-set-variable] > > commit ac2d9

Re: [PATCH v3] drm/dp_mst: Fix W=1 warnings

2020-01-17 Thread Lyude Paul
Hey, does this still need review? On Tue, 2020-01-07 at 14:11 +0100, Benjamin Gaignard wrote: > Le ven. 20 déc. 2019 à 15:03, Benjamin Gaignard > a écrit : > > Le lun. 16 déc. 2019 à 09:28, Benjamin Gaignard > > a écrit : > > > Le mer. 4 déc. 2019 à 17:47, Jani Nikula a > > > écrit : > > > > On

[Bug 206141] VCE UVD ring test failed -110

2020-01-17 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206141 Janpieter Sollie (janpieter.sol...@dommel.be) changed: What|Removed |Added Status|NEW |RESOLVED

[Bug 206155] amdgpu several warnings while booting Fiji GPU, GPU not activated

2020-01-17 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206155 Janpieter Sollie (janpieter.sol...@dommel.be) changed: What|Removed |Added Status|NEW |RESOLVED

<    1   2