[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

[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

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

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 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

[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

[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 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 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

[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

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

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

[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 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

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] ===

[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 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: > > >

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
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 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 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 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 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

[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: [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?

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

[PATCH v3 05/12] drm/i915: Use the cpu_transcoder in intel_hdcp to toggle HDCP signalling

2020-01-17 Thread Sean Paul
From: Sean Paul Instead of using intel_dig_port's encoder pipe to determine which transcoder to toggle signalling on, use the cpu_transcoder field already stored in intel_hdmi. This is particularly important for MST. Suggested-by: Ville Syrjälä Reviewed-by: Ramalingam C Signed-off-by: Sean Pa

[PATCH v3 03/12] drm/i915: WARN if HDCP signalling is enabled upon disable

2020-01-17 Thread Sean Paul
From: Sean Paul HDCP signalling should not be left on, WARN if it is Cc: Ville Syrjälä Cc: Daniel Vetter Reviewed-by: Ramalingam C Signed-off-by: Sean Paul Link: https://patchwork.freedesktop.org/patch/msgid/20191212190230.188505-4-s...@poorly.run #v2 Changes in v2: - Added to the set in

[PATCH v3 08/12] drm/i915: Don't fully disable HDCP on a port if multiple pipes are using it

2020-01-17 Thread Sean Paul
From: Sean Paul This patch is required for HDCP over MST. If a port is being used for multiple HDCP streams, we don't want to fully disable HDCP on a port if one of them is disabled. Instead, we just disable the HDCP signalling on that particular pipe and exit early. The last pipe to disable HDCP

[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 10/12] drm/i915: Use ddi_update_pipe in intel_dp_mst

2020-01-17 Thread Sean Paul
From: Sean Paul In order to act upon content_protection property changes, we'll need to implement the .update_pipe() hook. We can re-use intel_ddi_update_pipe for this Signed-off-by: Sean Paul Link: https://patchwork.freedesktop.org/patch/msgid/20191203173638.94919-10-s...@poorly.run #v1 Link

[PATCH v3 01/12] drm/i915: Fix sha_text population code

2020-01-17 Thread Sean Paul
From: Sean Paul This patch fixes a few bugs: 1- We weren't taking into account sha_leftovers when adding multiple ksvs to sha_text. As such, we were or'ing the end of ksv[j - 1] with the beginning of ksv[j] 2- In the sha_leftovers == 2 and sha_leftovers == 3 case, bstatus was being pla

[PATCH v3 12/12] drm/i915: Add HDCP 1.4 support for MST connectors

2020-01-17 Thread Sean Paul
From: Sean Paul Now that all the groundwork has been laid, we can turn on HDCP 1.4 over MST. Everything except for toggling the HDCP signalling and HDCP 2.2 support is the same as the DP case, so we'll re-use those callbacks Signed-off-by: Sean Paul Link: https://patchwork.freedesktop.org/patc

[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

[PATCH v3 07/12] drm/i915: Protect workers against disappearing connectors

2020-01-17 Thread Sean Paul
From: Sean Paul This patch adds some protection against connectors being destroyed before the HDCP workers are finished. For check_work, we do a synchronous cancel after the connector is unregistered which will ensure that it is finished before destruction. In the case of prop_work, we can't do

[PATCH v3 09/12] drm/i915: Support DP MST in enc_to_dig_port() function

2020-01-17 Thread Sean Paul
From: Sean Paul Although DP_MST fake encoders are not subclassed from digital ports, they are associated with them. Support these encoders. Signed-off-by: Sean Paul Link: https://patchwork.freedesktop.org/patch/msgid/20191203173638.94919-9-s...@poorly.run #v1 Link: https://patchwork.freedesk

Re: [PATCH 3/6] video: fbdev: controlfb: add COMPILE_TEST support

2020-01-17 Thread Sam Ravnborg
Hi Bartlomiej On Thu, Jan 16, 2020 at 03:08:57PM +0100, Bartlomiej Zolnierkiewicz wrote: > Add COMPILE_TEST support to controlfb driver for better compile > testing coverage. This is not a nice patch to add COMPILE_TEST support :-( But I see why you do it. I already spent too much time being side

[PATCH v3 02/12] drm/i915: Clear the repeater bit on HDCP disable

2020-01-17 Thread Sean Paul
From: Sean Paul On HDCP disable, clear the repeater bit. This ensures if we connect a non-repeater sink after a repeater, the bit is in the state we expect. Fixes: ee5e5e7a5e0f (drm/i915: Add HDCP framework + base implementation) Cc: Chris Wilson Cc: Ramalingam C Cc: Daniel Vetter Cc: Sean Pa

[PATCH v3 00/12] drm/i915: Add support for HDCP 1.4 over MST connectors

2020-01-17 Thread Sean Paul
From: Sean Paul Hey all, Here's v3, which addresses all review comments in v2. Sean Sean Paul (12): drm/i915: Fix sha_text population code drm/i915: Clear the repeater bit on HDCP disable drm/i915: WARN if HDCP signalling is enabled upon disable drm/i915: Intercept Aksv writes in the au

[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 #9 from Sylvain BERTRAND (sylvain.bertr...@gmail.com) --- On Fri, Jan 17, 2020 at 06:45:28PM +, bugzilla-dae...@bugzilla.kernel.org wrote: > screenshot : https://i.ibb.co/PrHj3hV/tombraider.jpg This seems to be from "shadow of the

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

2020-01-17 Thread sylvain . bertrand
On Fri, Jan 17, 2020 at 06:45:28PM +, bugzilla-dae...@bugzilla.kernel.org wrote: > screenshot : https://i.ibb.co/PrHj3hV/tombraider.jpg This seems to be from "shadow of the tomb raider" which I don't "own". Do you "own" "rise of the tomb raider", the previous tomb raider game (which I "own"

Re: [PATCH 1/6] video: fbdev: controlfb: fix sparse warning about using incorrect type

2020-01-17 Thread Sam Ravnborg
Hi Bartlomiej On Thu, Jan 16, 2020 at 03:08:55PM +0100, Bartlomiej Zolnierkiewicz wrote: > Force le32_to_cpup() argument to be of proper type (const __le32 *). > > Also add missing inline keyword to control_par_to_var() definition > (to match function prototype). > > Signed-off-by: Bartlomiej Zo

Re: [PATCH 1/2] video: fbdev: wm8505fb: fix sparse warnings about using incorrect types

2020-01-17 Thread Sam Ravnborg
On Thu, Jan 16, 2020 at 03:54:42PM +0100, Bartlomiej Zolnierkiewicz wrote: > Use ->screen_buffer instead of ->screen_base to fix sparse warnings. > > [ Please see commit 17a7b0b4d974 ("fb.h: Provide alternate screen_base > pointer") for details. ] > > Signed-off-by: Bartlomiej Zolnierkiewicz A

Re: [PATCH 1/2] video: fbdev: wm8505fb: fix sparse warnings about using incorrect types

2020-01-17 Thread Sam Ravnborg
On Thu, Jan 16, 2020 at 03:56:50PM +0100, Bartlomiej Zolnierkiewicz wrote: > Use ->screen_buffer instead of ->screen_base to fix sparse warnings. > > [ Please see commit 17a7b0b4d974 ("fb.h: Provide alternate screen_base > pointer") for details. ] > > Signed-off-by: Bartlomiej Zolnierkiewicz A

Re: [PATCH 2/2] video: fbdev: wm8505fb: add COMPILE_TEST support

2020-01-17 Thread Sam Ravnborg
On Thu, Jan 16, 2020 at 03:58:08PM +0100, Bartlomiej Zolnierkiewicz wrote: > Add COMPILE_TEST support to wm8505fb driver for better compile > testing coverage. > > Signed-off-by: Bartlomiej Zolnierkiewicz Acked-by: Sam Ravnborg > --- > drivers/video/fbdev/Kconfig |2 +- > 1 file changed, 1

Re: [PATCH 2/2] video: fbdev: w100fb: add COMPILE_TEST support

2020-01-17 Thread Sam Ravnborg
On Thu, Jan 16, 2020 at 03:53:58PM +0100, Bartlomiej Zolnierkiewicz wrote: > Add COMPILE_TEST support to w100fb driver for better compile > testing coverage. > > Signed-off-by: Bartlomiej Zolnierkiewicz Acked-by: Sam Ravnborg > --- > drivers/video/fbdev/Kconfig |2 +- > 1 file changed, 1 in

Re: [PATCH 1/2] video: fbdev: w100fb: fix sparse warnings

2020-01-17 Thread Sam Ravnborg
On Thu, Jan 16, 2020 at 03:53:20PM +0100, Bartlomiej Zolnierkiewicz wrote: > * Add missing __iomem annotations where needed. > * Make w100fb_probe() static. > * Return NULL pointer (instead of using plain integer) in > w100_get_xtal_tabl(). > > Signed-off-by: Bartlomiej Zolnierkiewicz Acked-by:

Re: [PATCH] video: fbdev: arcfb: add COMPILE_TEST support

2020-01-17 Thread Sam Ravnborg
On Thu, Jan 16, 2020 at 03:49:07PM +0100, Bartlomiej Zolnierkiewicz wrote: > Add COMPILE_TEST support to arcfb driver for better compile > testing coverage. > > Signed-off-by: Bartlomiej Zolnierkiewicz Acked-by: Sam Ravnborg > --- > drivers/video/fbdev/Kconfig |2 +- > 1 file changed, 1 ins

Re: [PATCH v2 0/5] Add support for Pine64 PineTab

2020-01-17 Thread Maxime Ripard
On Fri, Jan 17, 2020 at 07:50:12PM +0100, Sam Ravnborg wrote: > Hi Icenowy > > On Thu, Jan 16, 2020 at 11:36:31AM +0800, Icenowy Zheng wrote: > > This patchset tries to add support for the PineTab tablet from Pine64. > > > > As it uses a specific MIPI-DSI panel, the support of the panel should be >

Re: [PATCH v3 0/2] drm/modes: Apply video parameters with only reflect option

2020-01-17 Thread Maxime Ripard
On Fri, Jan 17, 2020 at 04:34:27PM +0100, Stephan Gerhold wrote: > At the moment, video mode parameters like video=540x960,reflect_x, > (without rotation set) are not taken into account when applying the > mode in drm_client_modeset.c. > > One of the reasons for this is that the calculation that >

Re: [PATCH v2 0/5] Add support for Pine64 PineTab

2020-01-17 Thread Sam Ravnborg
Hi Icenowy On Thu, Jan 16, 2020 at 11:36:31AM +0800, Icenowy Zheng wrote: > This patchset tries to add support for the PineTab tablet from Pine64. > > As it uses a specific MIPI-DSI panel, the support of the panel should be > introduced first, with its DT binding. > > Then a device tree is added

[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 #8 from kentosama (kentos...@whiteninjastudio.com) --- (In reply to Sylvain BERTRAND from comment #6) > Owner and user of tahiti parts on amdgpu with a state of the art gfx stack > poping in. > > I own "rise of the tomb raider" which

[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 #7 from kentosama (kentos...@whiteninjastudio.com) --- I noticed something strange. When I do the Tomb Raider benchmark, the GPU usage is 100% and I hear the fans running at full speed after a few seconds. The bench gives a result of

Re: [PATCH v2] drm/msm: Add syncobj support.

2020-01-17 Thread Bas Nieuwenhuizen
On Fri, Jan 17, 2020 at 7:17 PM Jordan Crouse wrote: > > On Fri, Jan 17, 2020 at 12:04:17AM +0100, Bas Nieuwenhuizen wrote: > > This > > > > 1) Enables core DRM syncobj support. > > 2) Adds options to the submission ioctl to wait/signal syncobjs. > > > > Just like the wait fence fd, this does inli

Re: [PATCH v9 0/5] add driver for "boe, tv101wum-nl6", "boe, tv101wum-n53", "auo, kd101n80-45na" and "auo, b101uan08.3" panels

2020-01-17 Thread Sam Ravnborg
Hi Jitao. Thanks for your perseverance on this. With the minor issue on the binding file resolved everything is now fine and patches are all applied to drm-misc-next. Thanks, Sam On Thu, Jan 16, 2020 at 10:15:06AM +0800, Jitao Shi wrote: > Changes since v8: > - merge the panel document

Re: [PATCH v9 1/5] dt-bindings: display: panel: Add boe tv101wum-n16 panel bindings

2020-01-17 Thread Sam Ravnborg
Hi Jitao. On Fri, Jan 17, 2020 at 07:08:17PM +0100, Sam Ravnborg wrote: > Hi Jitao. > > Looks good, much better than the individual files. > Rob Herring is still listed as maintainer which I questioned in last > feedback. > > With this resolved (kept only if Rob confirms), this is > Reviewed-by:

Re: [PATCH v2] drm/msm: Add syncobj support.

2020-01-17 Thread Jordan Crouse
On Fri, Jan 17, 2020 at 12:04:17AM +0100, Bas Nieuwenhuizen wrote: > This > > 1) Enables core DRM syncobj support. > 2) Adds options to the submission ioctl to wait/signal syncobjs. > > Just like the wait fence fd, this does inline waits. Using the > scheduler would be nice but I believe it is ou

Re: [PATCH v9 1/5] dt-bindings: display: panel: Add boe tv101wum-n16 panel bindings

2020-01-17 Thread Sam Ravnborg
Hi Jitao. Looks good, much better than the individual files. Rob Herring is still listed as maintainer which I questioned in last feedback. With this resolved (kept only if Rob confirms), this is Reviewed-by: Sam Ravnborg On Thu, Jan 16, 2020 at 10:15:07AM +0800, Jitao Shi wrote: > Add document

[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 #6 from Sylvain BERTRAND (sylvain.bertr...@gmail.com) --- Owner and user of tahiti parts on amdgpu with a state of the art gfx stack poping in. I own "rise of the tomb raider" which gnu/linux port is vulkan only, and vulkan is only av

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

2020-01-17 Thread sylvain . bertrand
Owner and user of tahiti parts on amdgpu with a state of the art gfx stack poping in. I own "rise of the tomb raider" which gnu/linux port is vulkan only, and vulkan is only available with the "amdgpu" kernel module (as far as I know). I have not bought "shadow of the tomb raider", which is vulka

Re: [PATCH AUTOSEL 5.4 002/205] drm/panfrost: Add missing check for pfdev->regulator

2020-01-17 Thread Greg KH
On Fri, Jan 17, 2020 at 04:12:27PM +, Steven Price wrote: > On Thu, Jan 16, 2020 at 04:39:37PM +, Sasha Levin wrote: > > From: Steven Price > > > > [ Upstream commit 52282163dfa651849e905886845bcf6850dd83c2 ] > > This commit is effectively already in 5.4. Confusingly there were two > ver

[Bug 206141] VCE UVD ring test failed -110

2020-01-17 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206141 --- Comment #7 from Janpieter Sollie (janpieter.sol...@dommel.be) --- Created attachment 286863 --> https://bugzilla.kernel.org/attachment.cgi?id=286863&action=edit dmesg with 2 GPUs OK, so this definitely looks like a HW failure, also tried to

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

2020-01-17 Thread Sean Paul
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.freedesktop.org > > Cc: Kazlauskas, Nicholas ; Wentland, Harry > > ; Zuo

Re: [PATCH AUTOSEL 5.4 002/205] drm/panfrost: Add missing check for pfdev->regulator

2020-01-17 Thread Steven Price
On Thu, Jan 16, 2020 at 04:39:37PM +, Sasha Levin wrote: > From: Steven Price > > [ Upstream commit 52282163dfa651849e905886845bcf6850dd83c2 ] This commit is effectively already in 5.4. Confusingly there were two versions of this upstream: 52282163dfa6 ("drm/panfrost: Add missing check for

Re: [PATCH 0/5] Use new logging macros in drm/i915

2020-01-17 Thread Jani Nikula
On Thu, 09 Jan 2020, Wambui Karuga wrote: > This patchset continues the conversion to using the new struct > drm_device based logging macros in drm/i915. Pushed to drm-intel-next-queued, thanks for the patches. BR, Jani. > > Wambui Karuga (5): > drm/i915: conversion to new logging macros in i

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

2020-01-17 Thread Sean Paul
On Fri, Jan 17, 2020 at 10:26 AM Mikita Lipski wrote: > > > > On 1/17/20 10:09 AM, Sean Paul wrote: > > On Fri, Dec 13, 2019 at 3:09 PM wrote: > >> > >> From: Mikita Lipski > >> > > > > Hi Mikita, > > Unfortunately this patch causes a crash on my i915 device when I > > unplug my MST hub. The pan

[PATCH v3 2/2] drm/modes: Allow DRM_MODE_ROTATE_0 when applying video mode parameters

2020-01-17 Thread Stephan Gerhold
At the moment, only DRM_MODE_ROTATE_180 is allowed when we try to apply the rotation from the video mode parameters. It is also useful to allow DRM_MODE_ROTATE_0 in case there is only a reflect option in the video mode parameter (e.g. video=540x960,reflect_x). DRM_MODE_ROTATE_0 means "no rotation"

[PATCH v3 0/2] drm/modes: Apply video parameters with only reflect option

2020-01-17 Thread Stephan Gerhold
At the moment, video mode parameters like video=540x960,reflect_x, (without rotation set) are not taken into account when applying the mode in drm_client_modeset.c. One of the reasons for this is that the calculation that combines the panel_orientation with cmdline->rotation_reflection does not ha

[PATCH v3 1/2] drm/modes: Make sure to parse valid rotation value from cmdline

2020-01-17 Thread Stephan Gerhold
A rotation value should have exactly one rotation angle. At the moment there is no validation for this when parsing video= parameters from the command line. This causes problems later on when we try to combine the command line rotation with the panel orientation. To make sure that we generate a va

[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 #5 from kentosama (kentos...@whiteninjastudio.com) --- (In reply to Alex Deucher from comment #3) > radeonsi is the mesa OpenGL driver and runs over both radeon and amdgpu. > Are you actually using the radeon kernel driver? Yes, i us

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

2020-01-17 Thread Mikita Lipski
On 1/17/20 10:09 AM, Sean Paul wrote: On Fri, Dec 13, 2019 at 3:09 PM wrote: From: Mikita Lipski Hi Mikita, Unfortunately this patch causes a crash on my i915 device when I unplug my MST hub. The panic is below. Hi Sean, I thought this issue was fixed by Wayne Lin in https://patchwo

[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 #4 from Michel Dänzer (mic...@daenzer.net) --- (In reply to Alex Deucher from comment #3) > Are you actually using the radeon kernel driver? "DRM 2.50.0" says yes. :) -- You are receiving this mail because: You are watching the assi

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

2020-01-17 Thread Sean Paul
On Fri, Dec 13, 2019 at 3:09 PM wrote: > > From: Mikita Lipski > Hi Mikita, Unfortunately this patch causes a crash on my i915 device when I unplug my MST hub. The panic is below. [ 38.514014] BUG: kernel NULL pointer dereference, address: 0030 [ 38.521801] #PF: supervisor read

[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 #3 from Alex Deucher (alexdeuc...@gmail.com) --- radeonsi is the mesa OpenGL driver and runs over both radeon and amdgpu. Are you actually using the radeon kernel driver? -- You are receiving this mail because: You are watching the

[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 #2 from kentosama (kentos...@whiteninjastudio.com) --- Hello, with the radeonsi driver, the performance is lower than the amdgpu driver. On the same scene in Tomb Raider, the graphics card displays ~ 10fps. I have a lower score with g

Re: [PATCH][next] drm/amd/display: fix for-loop with incorrectly sized loop counter

2020-01-17 Thread walter harms
Am 17.01.2020 14:33, schrieb Colin King: > From: Colin Ian King > > A for-loop is iterating from 0 up to 1000 however the loop variable count > is a u8 and hence not large enough. Fix this by making count an int. > Also remove the redundant initialization of count since this is never used > a

Re: [PATCH] drm/i915: Fix i915_error_state_store error defination

2020-01-17 Thread Jani Nikula
On Fri, 17 Jan 2020, Andi Shyti wrote: > Hi Jani, > >> >> Reported-by: Hulk Robot >> > >> > I've never been a fan of non human accounts, we had this discussion >> > already in a different mailing list. Could you please find a >> > different way of giving credit to your CI system? >> >> I don't a

Re: [PATCH 2/2] video: fbdev: sh_mobile_lcdcfb: add COMPILE_TEST support

2020-01-17 Thread Geert Uytterhoeven
On Thu, Jan 16, 2020 at 3:52 PM Bartlomiej Zolnierkiewicz wrote: > Add COMPILE_TEST support to sh_mobile_lcdcfb driver for better compile > testing coverage. > > Signed-off-by: Bartlomiej Zolnierkiewicz Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geer

Re: [PATCH 1/2] video: fbdev: sh_mobile_lcdcfb: fix sparse warnings about using incorrect types

2020-01-17 Thread Geert Uytterhoeven
On Thu, Jan 16, 2020 at 3:51 PM Bartlomiej Zolnierkiewicz wrote: > Use ->screen_buffer instead of ->screen_base to fix sparse warnings. > > [ Please see commit 17a7b0b4d974 ("fb.h: Provide alternate screen_base > pointer") for details. ] > > Signed-off-by: Bartlomiej Zolnierkiewicz Reviewed-by

Re: [PATCH] drm/i915: Fix i915_error_state_store error defination

2020-01-17 Thread Andi Shyti
Hi Jani, > >> Reported-by: Hulk Robot > > > > I've never been a fan of non human accounts, we had this discussion > > already in a different mailing list. Could you please find a > > different way of giving credit to your CI system? > > I don't actually mind for Reported-by credits. The history

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

2020-01-17 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206231 Alex Deucher (alexdeuc...@gmail.com) changed: What|Removed |Added CC||alexdeuc...@gmail.c

Re: [PATCH 0/4] drm/i915/display: conversion to new logging macros.

2020-01-17 Thread Jani Nikula
On Thu, 16 Jan 2020, Wambui Karuga wrote: > This series converts the printk based logging macros in > drm/i915/display/intel_display.c to the new struct drm_device based > logging macros. This change was split into four for manageability and > due to the size of drm/i915/display/intel_display.c.

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

2020-01-17 Thread Mikita Lipski
Thanks for the catch! Makes sense to skip disabled managers there, but I wonder why it didn't cause a crash with amdgpu. Anyways looks good to me. Acked-by: Mikita Lipski On 1/16/20 8:58 PM, José Roberto de Souza wrote: When a main MST port is disconnected drivers should call drm_dp_mst_topol

[Bug 206017] Kernel 5.4.x unusable with GUI due to crashes (some hard)

2020-01-17 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206017 Paul (paul.e.hi...@outlook.com) changed: What|Removed |Added CC||paul.e.hi...@outlook.com

[PATCH][next] drm/amd/display: fix for-loop with incorrectly sized loop counter

2020-01-17 Thread Colin King
From: Colin Ian King A for-loop is iterating from 0 up to 1000 however the loop variable count is a u8 and hence not large enough. Fix this by making count an int. Also remove the redundant initialization of count since this is never used and add { } on the loop statement make the loop block cle

Re: [PATCH v2 19/21] drm/vkms: Convert to CRTC VBLANK callbacks

2020-01-17 Thread Thomas Zimmermann
Hi Am 16.01.20 um 00:18 schrieb Rodrigo Siqueira: > Hi, > > Thanks for the patch, I reviewed and tested it. Everything looks fine > for VKMS. > > Reviewed-by: Rodrigo Siqueira > Tested-by: Rodrigo Siqueira Thanks a lot. Best regards Thomas > > On 01/15, Thomas Zimmermann wrote: >> VBLANK c

re: drm/amd/display: make PSR static screen entry within 30 ms

2020-01-17 Thread Colin Ian King
Hi, Static analysis with Coverity has detected a division by zero in the following commit: commit 5b5abe9526073ccbf3032d27b5864520829cdd9c Author: Anthony Koo Date: Mon Dec 9 17:26:34 2019 -0500 drm/amd/display: make PSR static screen entry within 30 ms Specifically: unsigned int

Re: [PATCH] drm/i915: Fix i915_error_state_store error defination

2020-01-17 Thread Jani Nikula
On Fri, 17 Jan 2020, Andi Shyti wrote: > Hi Zhang, > > On Fri, Jan 17, 2020 at 03:34:36PM +0800, Zhang Xiaoxu wrote: >> Since commit 742379c0c4001 ("drm/i915: Start chopping up the GPU error >> capture"), function 'i915_error_state_store' was defined and used with >> only one parameter. >> >> But

Re: [PATCH] drm/bridge: tfp410: add pclk limits

2020-01-17 Thread Laurent Pinchart
Hi Tomi, Thank you for the patch. On Mon, Jan 13, 2020 at 03:45:28PM +0200, Tomi Valkeinen wrote: > Add pixel clock limits to the driver as per TFP410 datasheet: min 25MHz, > max 165MHz. > > Signed-off-by: Tomi Valkeinen > --- > drivers/gpu/drm/bridge/ti-tfp410.c | 10 ++ > 1 file chan

Re: [PATCH] drm/i915: Fix i915_error_state_store error defination

2020-01-17 Thread Andi Shyti
Hi Zhang, On Fri, Jan 17, 2020 at 03:34:36PM +0800, Zhang Xiaoxu wrote: > Since commit 742379c0c4001 ("drm/i915: Start chopping up the GPU error > capture"), function 'i915_error_state_store' was defined and used with > only one parameter. > > But if no 'CONFIG_DRM_I915_CAPTURE_ERROR', this funct

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

2020-01-17 Thread Jani Nikula
On Fri, 17 Jan 2020, 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 every system we've >

Re: [PATCH v3] drm/panfrost: Add the panfrost_gem_mapping concept

2020-01-17 Thread Steven Price
On 16/01/2020 02:15, Rob Herring wrote: > From: Boris Brezillon > > With the introduction of per-FD address space, the same BO can be mapped > in different address space if the BO is globally visible (GEM_FLINK) > and opened in different context or if the dmabuf is self-imported. The > current im

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

2020-01-17 Thread Jani Nikula
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 every system we've > spotted DPCD backlight control support on doe

Re: [RESEND v3 2/2] drm: bridge: adv7511: Add support for ADV7535

2020-01-17 Thread Andrzej Hajda
On 17.01.2020 11:27, Andrzej Hajda wrote: > On 07.01.2020 14:34, Bogdan Togorean wrote: >> ADV7535 is a DSI to HDMI bridge chip like ADV7533 but it allows >> 1080p@60Hz. v1p2 is fixed to 1.8V on ADV7535 but on ADV7533 can be 1.2V >> or 1.8V and is configurable in a register. >> >> Signed-off-by: Bo

Re: [RESEND v3 2/2] drm: bridge: adv7511: Add support for ADV7535

2020-01-17 Thread Andrzej Hajda
On 07.01.2020 14:34, Bogdan Togorean wrote: > ADV7535 is a DSI to HDMI bridge chip like ADV7533 but it allows > 1080p@60Hz. v1p2 is fixed to 1.8V on ADV7535 but on ADV7533 can be 1.2V > or 1.8V and is configurable in a register. > > Signed-off-by: Bogdan Togorean > --- > drivers/gpu/drm/bridge/ad

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

2020-01-17 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206231 Bug ID: 206231 Summary: R9 280X low performance with all games Product: Drivers Version: 2.5 Kernel Version: 5.4.0-2-amd64 Hardware: x86-64 OS: Linux Tree: Ma

Re: [PATCH] drm/bridge: tfp410: add pclk limits

2020-01-17 Thread Andrzej Hajda
On 13.01.2020 14:45, Tomi Valkeinen wrote: > Add pixel clock limits to the driver as per TFP410 datasheet: min 25MHz, > max 165MHz. > > Signed-off-by: Tomi Valkeinen Reviewed-by: Andrzej Hajda  -- Regards Andrzej > --- > drivers/gpu/drm/bridge/ti-tfp410.c | 10 ++ > 1 file changed, 10

  1   2   >