Re: LED backlight on Droid 4 and others

2020-02-18 Thread Lee Jones
On Wed, 19 Feb 2020, Pavel Machek wrote: > Hi! > > > > > > > > It would be good to get LED backlight to work in clean way for 5.6 > > > > > > > kernel. > > > > > ... > > > > > > > [If you have an idea what else is needed, it would be welcome; it > > > > > > > works for me in development tree but

Re: [PATCH 5/5] drm/amdgpu: implement amdgpu_gem_prime_move_notify v2

2020-02-18 Thread VMware
On 2/18/20 10:01 PM, Daniel Vetter wrote: On Tue, Feb 18, 2020 at 9:17 PM Thomas Hellström (VMware) wrote: On 2/17/20 6:55 PM, Daniel Vetter wrote: On Mon, Feb 17, 2020 at 04:45:09PM +0100, Christian König wrote: Implement the importer side of unpinned DMA-buf handling. v2: update page

Re: [PATCH] drm/vram-helper: make drm_vram_mm_debugfs_init return 0

2020-02-18 Thread Thomas Zimmermann
Am 18.02.20 um 18:28 schrieb Wambui Karuga: > As drm_debugfs_create_files() should return 0, remove its use as the > return value of drm_vram_mm_debugfs_init(), and have the function return > 0 directly. > > Signed-off-by: Wambui Karuga Acked-by: Thomas Zimmermann > --- >

Re: [Nouveau] [PATCH] drm/nouveau: remove checks for return value of debugfs functions

2020-02-18 Thread Ben Skeggs
On Wed, 19 Feb 2020 at 03:28, Wambui Karuga wrote: > > As there is no need to check for the return value of debugfs_create_file > and drm_debugfs_create_files, remove unnecessary checks and error > handling in nouveau_drm_debugfs_init. Thanks! > > Signed-off-by: Wambui Karuga > --- >

[PATCH] drm: kirin: Revert "Fix for hikey620 display offset problem"

2020-02-18 Thread John Stultz
This reverts commit ff57c6513820efe945b61863cf4a51b79f18b592. With the commit ff57c6513820 ("drm: kirin: Fix for hikey620 display offset problem") we added support for handling LDI overflows by resetting the hardware. However, its been observed that when we do hit the LDI overflow condition, the

Re: [PATCH v3 4/5] dt-bindings: display: add data-mapping to panel-dpi

2020-02-18 Thread Rob Herring
On Tue, Feb 18, 2020 at 11:16:38PM +0100, Sam Ravnborg wrote: > On Tue, Feb 18, 2020 at 02:13:45PM -0600, Rob Herring wrote: > > On Sun, Feb 16, 2020 at 12:15 PM Sam Ravnborg wrote: > > > > > > Add data-mapping property that can be used to specify > > > the media format used for the connection

[PATCH 3/5 v4] drm/virtio: track whether or not a context has been initiated

2020-02-18 Thread Gurchetan Singh
Use a boolean variable to track whether a context has been initiated. v3: Fix possible race via spinlock (@olv) v4: Fix commit message Signed-off-by: Gurchetan Singh --- drivers/gpu/drm/virtio/virtgpu_drv.h | 2 ++ drivers/gpu/drm/virtio/virtgpu_ioctl.c | 8

[PATCH 4/5 v3] drm/virtio: enqueue virtio_gpu_create_context after the first 3D ioctl

2020-02-18 Thread Gurchetan Singh
For old userspace, initialization will still be implicit. For backwards compatibility, enqueue virtio_gpu_cmd_context_create after the first 3D ioctl. v3: staticify virtio_gpu_create_context remove notify to batch vm-exit Signed-off-by: Gurchetan Singh ---

[PATCH 2/5 v3] drm/virtio: factor out context create hyercall

2020-02-18 Thread Gurchetan Singh
We currently create an OpenGL context when opening the DRM fd if 3D is available. We may need other context types (VK,..) in the future, and the plan is to have explicit initialization for that. For explicit initialization to work, we need to factor out virtio_gpu_create_context from driver

[PATCH 3/5 v3] drm/virtio: track whether or not a context has been initiated

2020-02-18 Thread Gurchetan Singh
Use an atomic variable to track whether a context has been initiated. v3: Fix possible race via spinlock (@olv) Signed-off-by: Gurchetan Singh --- drivers/gpu/drm/virtio/virtgpu_drv.h | 2 ++ drivers/gpu/drm/virtio/virtgpu_ioctl.c | 8 drivers/gpu/drm/virtio/virtgpu_kms.c | 2 ++

[PATCH 5/5 v3] drm/virtio: add virtio_gpu_context_type

2020-02-18 Thread Gurchetan Singh
We'll have to do something like this eventually, and this conveys we want a Virgl context by default. Signed-off-by: Gurchetan Singh --- drivers/gpu/drm/virtio/virtgpu_ioctl.c | 23 +-- 1 file changed, 17 insertions(+), 6 deletions(-) diff --git

[PATCH 1/5 v3] drm/virtio: use consistent names for drm_files

2020-02-18 Thread Gurchetan Singh
Minor cleanup, change: - file_priv--> file, - drm_file --> file. Signed-off-by: Gurchetan Singh --- drivers/gpu/drm/virtio/virtgpu_ioctl.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/gpu/drm/virtio/virtgpu_ioctl.c

[PATCH v2 2/2] drm/i915/gen7: Clear all EU/L3 residual contexts

2020-02-18 Thread Akeem G Abodunrin
From: Prathap Kumar Valsan On gen7 and gen7.5 devices, there could be leftover data residuals in EU/L3 from the retiring context. This patch introduces workaround to clear that residual contexts, by submitting a batch buffer with dedicated HW context to the GPU with ring allocation for each

[PATCH v2 0/2] Security mitigation for Intel Gen7/7.5 HWs

2020-02-18 Thread Akeem G Abodunrin
Intel ID: PSIRT-TA-201910-001 CVEID: CVE-2019-14615 Summary of Vulnerability Insufficient control flow in certain data structures for some Intel(R) Processors with Intel Processor Graphics may allow an unauthenticated user to potentially enable information disclosure via

[PATCH v2 1/2] drm/i915: Add mechanism to submit a context WA on ring submission

2020-02-18 Thread Akeem G Abodunrin
From: Mika Kuoppala This patch adds framework to submit an arbitrary batchbuffer on each context switch to clear residual state for render engine on Gen7/7.5 devices. The idea of always emitting the context and vm setup around each request is primary to make reset recovery easy, and not require

Re: LED backlight on Droid 4 and others

2020-02-18 Thread Pavel Machek
Hi! > > > > > > It would be good to get LED backlight to work in clean way for 5.6 > > > > > > kernel. > > > > ... > > > > > > [If you have an idea what else is needed, it would be welcome; it > > > > > > works for me in development tree but not in tree I'd like to > > > > > > upstream.] > > > >

Re: [PATCH v2] drm/msm: devcoredump should dump MSM_SUBMIT_BO_DUMP buffers

2020-02-18 Thread Jordan Crouse
On Tue, Feb 18, 2020 at 01:20:12PM -0800, Rob Clark wrote: > From: Rob Clark > > Also log buffers with the DUMP flag set, to ensure we capture all useful > cmdstream in crashdump state with modern mesa. > > Otherwise we miss out on the contents of "state object" cmdstream > buffers. > > v2:

Re: [Freedreno] [PATCH] drm/msm: devcoredump should dump MSM_SUBMIT_BO_DUMP buffers

2020-02-18 Thread Jordan Crouse
On Tue, Feb 18, 2020 at 01:00:21PM -0800, Rob Clark wrote: > From: Rob Clark > > Also log buffers with the DUMP flag set, to ensure we capture all useful > cmdstream in crashdump state with modern mesa. > > Otherwise we miss out on the contents of "state object" cmdstream > buffers. One nit,

Re: [RFC PATCH 0/3] KVM: x86: honor guest memory type

2020-02-18 Thread Sean Christopherson
On Tue, Feb 18, 2020 at 05:28:51PM +0100, Paolo Bonzini wrote: > On 14/02/20 23:03, Sean Christopherson wrote: > >> On Fri, Feb 14, 2020 at 1:47 PM Chia-I Wu wrote: > >>> AFAICT, it is currently allowed on ARM (verified) and AMD (not > >>> verified, but svm_get_mt_mask returns 0 which supposedly

Re: [PATCH v3 4/5] dt-bindings: display: add data-mapping to panel-dpi

2020-02-18 Thread Sam Ravnborg
On Tue, Feb 18, 2020 at 02:13:45PM -0600, Rob Herring wrote: > On Sun, Feb 16, 2020 at 12:15 PM Sam Ravnborg wrote: > > > > Add data-mapping property that can be used to specify > > the media format used for the connection betwwen the > > display controller (connector) and the panel. > >

[Bug 205915] AMDGPU: Screen flicker after resume from suspend

2020-02-18 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205915 --- Comment #15 from Manuel Ullmann (la...@posteo.de) --- That fixed it. Thanks. -- You are receiving this mail because: You are watching the assignee of the bug. ___ dri-devel mailing list

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

2020-02-18 Thread Sean Paul
On Sun, Feb 2, 2020 at 2:03 PM Ramalingam C wrote: > > On 2020-01-17 at 14:30:59 -0500, Sean Paul wrote: > > 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

[Bug 205915] AMDGPU: Screen flicker after resume from suspend

2020-02-18 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205915 --- Comment #14 from Alex Deucher (alexdeuc...@gmail.com) --- See also: https://bugzilla.kernel.org/show_bug.cgi?id=205915 https://bugzilla.kernel.org/show_bug.cgi?id=206393 https://bugzilla.kernel.org/show_bug.cgi?id=206575 -- You are

[Bug 206393] amdgpu: garbled screen after resume

2020-02-18 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206393 --- Comment #6 from Alex Deucher (alexdeuc...@gmail.com) --- See also: https://bugzilla.kernel.org/show_bug.cgi?id=205915 https://bugzilla.kernel.org/show_bug.cgi?id=206393 https://bugzilla.kernel.org/show_bug.cgi?id=206575 -- You are receiving

[Bug 206575] [amdgpu] [drm] No video signal on resume from suspend, R9 380

2020-02-18 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206575 --- Comment #9 from Noel Maersk (veox+ker...@veox.pw) --- Some of the commits that got skipped in that `git bisect log` of mine actually come before the one above when viewing `git log`. :/ Guess I'll try the bisect again in coming days. --

[PATCH v4 12/14] drm/i915: Add connector to hdcp_shim->check_link()

2020-02-18 Thread Sean Paul
From: Sean Paul Currently we derive the connector from digital port in check_link(). For MST, this isn't sufficient since the digital port passed into the function can have multiple connectors downstream. This patch adds connector to the check_link() arguments so we have it when we need it.

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

2020-02-18 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:

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

2020-02-18 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:

[PATCH v4 10/14] drm/i915: Use ddi_update_pipe in intel_dp_mst

2020-02-18 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

[PATCH v4 14/14] drm/i915: Add HDCP 1.4 support for MST connectors

2020-02-18 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 Cc: Juston Li Signed-off-by: Sean Paul Link:

[PATCH v4 13/14] drm/mst: Add support for QUERY_STREAM_ENCRYPTION_STATUS MST sideband message

2020-02-18 Thread Sean Paul
From: Sean Paul Used to query whether an MST stream is encrypted or not. Cc: Lyude Paul Signed-off-by: Sean Paul Changes in v4: -Added to the set --- drivers/gpu/drm/drm_dp_mst_topology.c | 117 ++ include/drm/drm_dp_helper.h | 3 +

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

2020-02-18 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

[PATCH v4 06/14] drm/i915: Factor out hdcp->value assignments

2020-02-18 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:

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

2020-02-18 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 Link:

[PATCH v4 01/14] drm/i915: Fix sha_text population code

2020-02-18 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

[PATCH v4 07/14] drm/i915: Protect workers against disappearing connectors

2020-02-18 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

[PATCH v4 04/14] drm/i915: Intercept Aksv writes in the aux hooks

2020-02-18 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

[PATCH v4 02/14] drm/i915: Clear the repeater bit on HDCP disable

2020-02-18 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

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

2020-02-18 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

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

2020-02-18 Thread Sean Paul
From: Sean Paul Hey all, Back with a v4. Rebased on latest drm-tip. Biggest change was adding the QUERY_STREAM_ENCRYPTION_STATUS check which ensures not only the link to the first branch is encrypted, but also that the channel iteself is also protected. Sean Sean Paul (14): drm/i915: Fix

Re: [PATCH] drm/msm/a5xx: Always set an OPP supported hardware value

2020-02-18 Thread Eric Anholt
On Fri, Feb 14, 2020 at 10:36 AM Jordan Crouse wrote: > > If the opp table specifies opp-supported-hw as a property but the driver > has not set a supported hardware value the OPP subsystem will reject > all the table entries. > > Set a "default" value that will match the default table entries

[Bug 206575] [amdgpu] [drm] No video signal on resume from suspend, R9 380

2020-02-18 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206575 --- Comment #8 from Noel Maersk (veox+ker...@veox.pw) --- The *ERROR* Couldn't read SADs: -2 in my and Thomas' logs are unrelated to the issue, I believe, and pertain to sound (HDMI sound?..). The error comes from

[Bug 206575] [amdgpu] [drm] No video signal on resume from suspend, R9 380

2020-02-18 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206575 --- Comment #7 from Noel Maersk (veox+ker...@veox.pw) --- `git bisect log` output at: https://gist.github.com/veox/36aeb77acfbcaea9c4ba1cc70052329a Had to `skip` a few because of system instability on v5.5.4 (cause unknown, likely unrelated to

Re: [RFC 0/9] Regmap over USB for Multifunction USB Device (gpio, display, ...)

2020-02-18 Thread Noralf Trønnes
Den 18.02.2020 21.57, skrev Andy Shevchenko: > On Sun, Feb 16, 2020 at 7:30 PM Noralf Trønnes wrote: >> >> Hi, >> >> A while back I had the idea to turn a Raspberry Pi Zero into a $5 >> USB to HDMI/SDTV/DSI/DPI display adapter. >> >> Thinking about how to represent the display to the driver I

[PATCH v2] drm/msm: devcoredump should dump MSM_SUBMIT_BO_DUMP buffers

2020-02-18 Thread Rob Clark
From: Rob Clark Also log buffers with the DUMP flag set, to ensure we capture all useful cmdstream in crashdump state with modern mesa. Otherwise we miss out on the contents of "state object" cmdstream buffers. v2: add missing 'inline' Signed-off-by: Rob Clark ---

[PATCH] drm/msm: devcoredump should dump MSM_SUBMIT_BO_DUMP buffers

2020-02-18 Thread Rob Clark
From: Rob Clark Also log buffers with the DUMP flag set, to ensure we capture all useful cmdstream in crashdump state with modern mesa. Otherwise we miss out on the contents of "state object" cmdstream buffers. Signed-off-by: Rob Clark --- drivers/gpu/drm/msm/msm_gem.h | 10 ++

Re: [PATCH 5/5] drm/amdgpu: implement amdgpu_gem_prime_move_notify v2

2020-02-18 Thread Daniel Vetter
On Tue, Feb 18, 2020 at 9:17 PM Thomas Hellström (VMware) wrote: > > On 2/17/20 6:55 PM, Daniel Vetter wrote: > > On Mon, Feb 17, 2020 at 04:45:09PM +0100, Christian König wrote: > >> Implement the importer side of unpinned DMA-buf handling. > >> > >> v2: update page tables immediately > >> > >>

Re: [RFC 0/9] Regmap over USB for Multifunction USB Device (gpio, display, ...)

2020-02-18 Thread Andy Shevchenko
On Sun, Feb 16, 2020 at 7:30 PM Noralf Trønnes wrote: > > Hi, > > A while back I had the idea to turn a Raspberry Pi Zero into a $5 > USB to HDMI/SDTV/DSI/DPI display adapter. > > Thinking about how to represent the display to the driver I realised > that hardware use registers as API. And Linux

Re: [PATCH v7 02/13] dt-bindings: display: mediatek: Add mmsys binding description

2020-02-18 Thread Rob Herring
On Thu, Feb 13, 2020 at 09:19:42PM +0100, matthias@kernel.org wrote: > From: Matthias Brugger > > The MediaTek DRM has a block called mmsys, which sets > the routing and enables the different blocks. > This patch adds one line for the mmsys bindings description and changes > the mmsys

Re: [PATCH 5/5] drm/amdgpu: implement amdgpu_gem_prime_move_notify v2

2020-02-18 Thread VMware
On 2/17/20 6:55 PM, Daniel Vetter wrote: On Mon, Feb 17, 2020 at 04:45:09PM +0100, Christian König wrote: Implement the importer side of unpinned DMA-buf handling. v2: update page tables immediately Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_dma_buf.c | 66

Re: [PATCH v3 4/5] dt-bindings: display: add data-mapping to panel-dpi

2020-02-18 Thread Rob Herring
On Sun, Feb 16, 2020 at 12:15 PM Sam Ravnborg wrote: > > Add data-mapping property that can be used to specify > the media format used for the connection betwwen the > display controller (connector) and the panel. > Signed-off-by: Sam Ravnborg Missing blank line. > --- >

Re: [PATCH v3 3/5] dt-bindings: display: convert panel-dpi to DT schema

2020-02-18 Thread Rob Herring
On Sun, Feb 16, 2020 at 12:15 PM Sam Ravnborg wrote: > > With panel-timing converted, now convert the single > remaining .txt user in panel/ of panel-timing to DT schema. > > v2: > - Drop Thierry as maintainer, as this is not a general panel binding > and I have no acks. > - Drop

Re: [PATCH] drm/arc: make arcpgu_debugfs_init return 0

2020-02-18 Thread Daniel Vetter
On Tue, Feb 18, 2020 at 08:00:30PM +0100, Daniel Vetter wrote: > On Tue, Feb 18, 2020 at 6:28 PM Wambui Karuga > wrote: > > > > As drm_debugfs_create_files should return void, remove its use as the > > return value of arcpgu_debugfs_init and have the latter function > > return 0 directly. > > >

Re: [RESEND][PATCH] drm/bridge: analogix-anx78xx: Fix drm_dp_link helper removal

2020-02-18 Thread Daniel Vetter
On Tue, Feb 18, 2020 at 04:57:44PM +0100, Torsten Duwe wrote: > drm_dp_link_rate_to_bw_code and ...bw_code_to_link_rate simply divide by > and multiply with 27000, respectively. Avoid an overflow in the u8 dpcd[0] > and the multiply+divide alltogether. > > fixes: ff1e8fb68ea06027

Re: [PATCH 1/2] drm/debugfs: remove checks for return value of drm_debugfs functions.

2020-02-18 Thread Daniel Vetter
On Tue, Feb 18, 2020 at 08:28:14PM +0300, Wambui Karuga wrote: > As there is no need to check the return value of > drm_debugfs_create_files, remove unnecessary checks and error handling > statement blocks. > > Signed-off-by: Wambui Karuga I'd split this up a bit differently, with a patch per

Re: [Nouveau] [PATCH 8/8] drm/ttm: do not keep GPU dependent addresses

2020-02-18 Thread Daniel Vetter
On Tue, Feb 18, 2020 at 07:37:44PM +0100, Christian König wrote: > Am 18.02.20 um 19:28 schrieb Thomas Zimmermann: > > Hi > > > > Am 18.02.20 um 19:23 schrieb Christian König: > > > Am 18.02.20 um 19:16 schrieb Thomas Zimmermann: > > > > Hi > > > > > > > > Am 18.02.20 um 18:13 schrieb Nirmoy: >

Re: [PATCH] drm/arc: make arcpgu_debugfs_init return 0

2020-02-18 Thread Daniel Vetter
On Tue, Feb 18, 2020 at 6:28 PM Wambui Karuga wrote: > > As drm_debugfs_create_files should return void, remove its use as the > return value of arcpgu_debugfs_init and have the latter function > return 0 directly. > > Signed-off-by: Wambui Karuga > --- > drivers/gpu/drm/arc/arcpgu_drv.c | 6

Re: [PATCH 8/8] drm/ttm: do not keep GPU dependent addresses

2020-02-18 Thread Christian König
Am 18.02.20 um 19:28 schrieb Thomas Zimmermann: Hi Am 18.02.20 um 19:23 schrieb Christian König: Am 18.02.20 um 19:16 schrieb Thomas Zimmermann: Hi Am 18.02.20 um 18:13 schrieb Nirmoy: On 2/18/20 1:44 PM, Christian König wrote: Am 18.02.20 um 13:40 schrieb Thomas Zimmermann: Hi Am

Re: [PATCH 8/8] drm/ttm: do not keep GPU dependent addresses

2020-02-18 Thread Thomas Zimmermann
Hi Am 18.02.20 um 19:23 schrieb Christian König: > Am 18.02.20 um 19:16 schrieb Thomas Zimmermann: >> Hi >> >> Am 18.02.20 um 18:13 schrieb Nirmoy: >>> On 2/18/20 1:44 PM, Christian König wrote: Am 18.02.20 um 13:40 schrieb Thomas Zimmermann: > Hi > > Am 17.02.20 um 16:04 schrieb

Re: [PATCH 8/8] drm/ttm: do not keep GPU dependent addresses

2020-02-18 Thread Nirmoy
On 2/18/20 7:16 PM, Thomas Zimmermann wrote: Hi Am 18.02.20 um 18:13 schrieb Nirmoy: On 2/18/20 1:44 PM, Christian König wrote: Am 18.02.20 um 13:40 schrieb Thomas Zimmermann: Hi Am 17.02.20 um 16:04 schrieb Nirmoy Das: GPU address handling is device specific and should be handle by its

[Bug 206519] [amdgpu] kernel NULL pointer dereference on shutdown when CONFIG_DRM_AMD_DC_HDCP=y

2020-02-18 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206519 --- Comment #5 from Alex Deucher (alexdeuc...@gmail.com) --- Created attachment 287487 --> https://bugzilla.kernel.org/attachment.cgi?id=287487=edit possible fix I think this patch should fix it. -- You are receiving this mail because: You

Re: [PATCH 8/8] drm/ttm: do not keep GPU dependent addresses

2020-02-18 Thread Christian König
Am 18.02.20 um 19:16 schrieb Thomas Zimmermann: Hi Am 18.02.20 um 18:13 schrieb Nirmoy: On 2/18/20 1:44 PM, Christian König wrote: Am 18.02.20 um 13:40 schrieb Thomas Zimmermann: Hi Am 17.02.20 um 16:04 schrieb Nirmoy Das: GPU address handling is device specific and should be handle by its

Re: [PATCH] drm/arc: make arcpgu_debugfs_init return 0

2020-02-18 Thread Greg KH
On Tue, Feb 18, 2020 at 08:28:12PM +0300, Wambui Karuga wrote: > As drm_debugfs_create_files should return void, remove its use as the > return value of arcpgu_debugfs_init and have the latter function > return 0 directly. > > Signed-off-by: Wambui Karuga > --- >

Re: [PATCH 8/8] drm/ttm: do not keep GPU dependent addresses

2020-02-18 Thread Thomas Zimmermann
Hi Am 18.02.20 um 18:13 schrieb Nirmoy: > > On 2/18/20 1:44 PM, Christian König wrote: >> Am 18.02.20 um 13:40 schrieb Thomas Zimmermann: >>> Hi >>> >>> Am 17.02.20 um 16:04 schrieb Nirmoy Das: GPU address handling is device specific and should be handle by its device driver.

[Bug 206575] [amdgpu] [drm] No video signal on resume from suspend, R9 380

2020-02-18 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206575 Thomas Frank (thfr...@e.mail.de) changed: What|Removed |Added CC||thfr...@e.mail.de ---

Re: [PATCH] drm/etnaviv: remove check for return value of drm_debugfs function

2020-02-18 Thread Lucas Stach
On Di, 2020-02-18 at 20:28 +0300, Wambui Karuga wrote: > As there is no need to check the return value if > drm_debugfs_create_files, And here is where the commit message skips a very important information: Since 987d65d01356 (drm: debugfs: make drm_debugfs_create_files() never fail) this

Re: [PATCH 8/8] drm/ttm: do not keep GPU dependent addresses

2020-02-18 Thread Christian König
Am 18.02.20 um 18:13 schrieb Nirmoy: On 2/18/20 1:44 PM, Christian König wrote: Am 18.02.20 um 13:40 schrieb Thomas Zimmermann: Hi Am 17.02.20 um 16:04 schrieb Nirmoy Das: GPU address handling is device specific and should be handle by its device driver. Signed-off-by: Nirmoy Das ---  

[Bug 206519] [amdgpu] kernel NULL pointer dereference on shutdown

2020-02-18 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206519 --- Comment #4 from Shlomo (shl...@fastmail.com) --- I bisected the bug. The first bad commit is 96a3b32e67236f547cc8acd69d5a3cef125b2295 (drm/amd/display: only enable HDCP for DCN+) with ea268870d6f548d0661e896e9746673210c1fa79

[Bug 206519] [amdgpu] kernel NULL pointer dereference on shutdown when CONFIG_DRM_AMD_DC_HDCP=y

2020-02-18 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206519 Shlomo (shl...@fastmail.com) changed: What|Removed |Added Summary|[amdgpu] kernel NULL|[amdgpu] kernel NULL

Re: [PATCH 3/3] drm/dp_mst: Remove single tx msg restriction.

2020-02-18 Thread Sean Paul
On Tue, Feb 18, 2020 at 10:52:06AM -0500, Sean Paul wrote: > On Mon, Feb 17, 2020 at 07:08:37AM +, Lin, Wayne wrote: > > [AMD Public Use] > > > > > > > > > -Original Message- > > > From: Sean Paul > > > Sent: Saturday, February 15, 2020 12:09 AM > > > To: Lin, Wayne > > > Cc:

Re: [PATCH 8/8] drm/ttm: do not keep GPU dependent addresses

2020-02-18 Thread Nirmoy
On 2/18/20 1:44 PM, Christian König wrote: Am 18.02.20 um 13:40 schrieb Thomas Zimmermann: Hi Am 17.02.20 um 16:04 schrieb Nirmoy Das: GPU address handling is device specific and should be handle by its device driver. Signed-off-by: Nirmoy Das ---   drivers/gpu/drm/ttm/ttm_bo.c    | 7

Re: [RFC PATCH 0/3] KVM: x86: honor guest memory type

2020-02-18 Thread Paolo Bonzini
On 14/02/20 23:03, Sean Christopherson wrote: >> On Fri, Feb 14, 2020 at 1:47 PM Chia-I Wu wrote: >>> AFAICT, it is currently allowed on ARM (verified) and AMD (not >>> verified, but svm_get_mt_mask returns 0 which supposedly means the NPT >>> does not restrict what the guest PAT can do). This

Re: [PATCH 0/3] drm/mst: Add support for simultaneous down replies

2020-02-18 Thread Sean Paul
On Fri, Feb 14, 2020 at 06:33:17PM -0500, Lyude Paul wrote: > On Thu, 2020-02-13 at 16:15 -0500, Sean Paul wrote: > > From: Sean Paul > > > > Hey all, > > Earlier this week on my 5.5 kernel (I can't seem to get a 5.6 kernel to > > behave on any of my devices), I ran into the multi-reply problem

[RESEND][PATCH] drm/bridge: analogix-anx78xx: Fix drm_dp_link helper removal

2020-02-18 Thread Torsten Duwe
drm_dp_link_rate_to_bw_code and ...bw_code_to_link_rate simply divide by and multiply with 27000, respectively. Avoid an overflow in the u8 dpcd[0] and the multiply+divide alltogether. fixes: ff1e8fb68ea06027 ("analogix-anx78xx: Avoid drm_dp_link helpers") Signed-off-by: Torsten Duwe

Re: [PATCH 0/8] do not store GPU address in TTM

2020-02-18 Thread Nirmoy
On 2/18/20 1:42 PM, Thomas Zimmermann wrote: Hi, the email's subject line should contain the version of the patchset (e.g, v2, v3, etc). git send-mail adds this with the -v switch Thanks Thomas. I over looked that,  will add version tag from my next emails. Nirmoy Best regards Thomas

[RESEND][PATCH] drm/bridge: analogix-anx6345: Avoid duplicate -supply suffix

2020-02-18 Thread Torsten Duwe
of_get_regulator() will unconditionally add "-supply" to form the property name. This is documented in commit 69511a452e6dc ("map consumer regulator based on device tree"). Remove the suffix from the requests. Signed-off-by: Torsten Duwe Reviewed-by: Mark Brown --- Mark has actually reviewed

Re: [PATCH 3/3] drm/dp_mst: Remove single tx msg restriction.

2020-02-18 Thread Sean Paul
On Mon, Feb 17, 2020 at 07:08:37AM +, Lin, Wayne wrote: > [AMD Public Use] > > > > > -Original Message- > > From: Sean Paul > > Sent: Saturday, February 15, 2020 12:09 AM > > To: Lin, Wayne > > Cc: dri-devel@lists.freedesktop.org; ly...@redhat.com; Sean Paul > > ; Maarten

Re: [PATCH v2] drm/bridge: Fix the bridge kernel doc

2020-02-18 Thread Neil Armstrong
On 18/02/2020 16:15, Boris Brezillon wrote: > Commit 751465913f04 ("drm/bridge: Add a drm_bridge_state object") > introduced new helpers and hooks but the kernel was slightly broken. > Fix that now. > > v2: > * Fix the drm_atomic_add_encoder_bridges() doc > > Fixes: 751465913f04 ("drm/bridge:

[RESEND][PATCH] drm/bridge: analogix-anx6345: Fix drm_dp_link helper removal

2020-02-18 Thread Torsten Duwe
drm_dp_link_rate_to_bw_code and ...bw_code_to_link_rate simply divide by and multiply with 27000, respectively. Avoid an overflow in the u8 dpcd[0] and the multiply+divide alltogether. fixes: e1cff82c1097bda2478 ("fix anx6345 compilation for v5.5") Signed-off-by: Torsten Duwe ---

[RESEND][PATCH] drm/bridge: analogix-anx6345: Avoid duplicate -supply suffix

2020-02-18 Thread Torsten Duwe
of_get_regulator() will unconditionally add "-supply" to form the property name. This is documented in commit 69511a452e6dc ("map consumer regulator based on device tree"). Remove the suffix from the requests. Signed-off-by: Torsten Duwe --- https://patchwork.freedesktop.org/patch/343005/

[RESEND][PATCH] drm/bridge: analogix-anx78xx: Fix drm_dp_link helper removal

2020-02-18 Thread Torsten Duwe
drm_dp_link_rate_to_bw_code and ...bw_code_to_link_rate simply divide by and multiply with 27000, respectively. Avoid an overflow in the u8 dpcd[0] and the multiply+divide alltogether. fixes: ff1e8fb68ea06027 ("analogix-anx78xx: Avoid drm_dp_link helpers") Signed-off-by: Torsten Duwe ---

Re: [Intel-gfx] [PATCH v2 1/5] drm/hdcp: optimizing the srm handling

2020-02-18 Thread Sean Paul
On Wed, Feb 12, 2020 at 03:59:38PM +0530, Ramalingam C wrote: > As we are not using the sysfs infrastructure anymore, link to it is > removed. And global srm data and mutex to protect it are removed, > with required handling at revocation check function. > > v2: > srm_data is dropped and few

Re: [Intel-gfx] [PATCH v2 4/5] drm/i915: dont retry stream management at seq_num_m roll over

2020-02-18 Thread Sean Paul
On Wed, Feb 12, 2020 at 03:59:41PM +0530, Ramalingam C wrote: > When roll over detected for seq_num_m, we shouldn't continue with stream > management with rolled over value. > > So we are terminating the stream management retry, on roll over of the > seq_num_m. > > v2: > using drm_dbg_kms

Re: [PATCH v2 3/5] drm/i915: terminate reauth at stream management failure

2020-02-18 Thread Sean Paul
On Wed, Feb 12, 2020 at 03:59:40PM +0530, Ramalingam C wrote: > As per the HDCP2.2 compliance test 1B-10 expectation, when stream > management for a repeater fails, we retry thrice and when it fails > in all retries, HDCP2.2 reauthentication aborted at kernel. > > v2: > seq_num_m++ is extended

Re: [PATCH v2] drm/bridge: Fix the bridge kernel doc

2020-02-18 Thread Neil Armstrong
On 18/02/2020 16:15, Boris Brezillon wrote: > Commit 751465913f04 ("drm/bridge: Add a drm_bridge_state object") > introduced new helpers and hooks but the kernel was slightly broken. > Fix that now. > > v2: > * Fix the drm_atomic_add_encoder_bridges() doc > > Fixes: 751465913f04 ("drm/bridge:

[PATCH v2] drm/bridge: Fix the bridge kernel doc

2020-02-18 Thread Boris Brezillon
Commit 751465913f04 ("drm/bridge: Add a drm_bridge_state object") introduced new helpers and hooks but the kernel was slightly broken. Fix that now. v2: * Fix the drm_atomic_add_encoder_bridges() doc Fixes: 751465913f04 ("drm/bridge: Add a drm_bridge_state object") Signed-off-by: Boris Brezillon

[PATCH] drm/bridge: Fix the bridge kernel doc

2020-02-18 Thread Boris Brezillon
Commit 751465913f04 ("drm/bridge: Add a drm_bridge_state object") introduced new helpers and hooks but the kernel was slightly broken. Fix that now. Fixes: 751465913f04 ("drm/bridge: Add a drm_bridge_state object") Signed-off-by: Boris Brezillon --- drivers/gpu/drm/drm_atomic.c | 5 +++--

Re: [PATCH 1/5] dma-buf: add dynamic DMA-buf handling v14

2020-02-18 Thread Daniel Vetter
On Tue, Feb 18, 2020 at 2:20 PM Christian König wrote: > > Am 05.11.19 um 11:20 schrieb Daniel Vetter: > > On Tue, Oct 29, 2019 at 11:40:45AM +0100, Christian König wrote: > > [SNIP] > >> diff --git a/drivers/dma-buf/dma-buf.c b/drivers/dma-buf/dma-buf.c > >> index d377b4ca66bf..ce293cee76ed

Re: LED backlight on Droid 4 and others

2020-02-18 Thread Lee Jones
On Wed, 12 Feb 2020, Pavel Machek wrote: > Hi! > > > > > It would be good to get LED backlight to work in clean way for 5.6 > > > > kernel. > > ... > > > > [If you have an idea what else is needed, it would be welcome; it > > > > works for me in development tree but not in tree I'd like to > > >

Re: [PATCH 1/5] dma-buf: add dynamic DMA-buf handling v14

2020-02-18 Thread Christian König
Am 05.11.19 um 11:20 schrieb Daniel Vetter: On Tue, Oct 29, 2019 at 11:40:45AM +0100, Christian König wrote: [SNIP] diff --git a/drivers/dma-buf/dma-buf.c b/drivers/dma-buf/dma-buf.c index d377b4ca66bf..ce293cee76ed 100644 --- a/drivers/dma-buf/dma-buf.c +++ b/drivers/dma-buf/dma-buf.c @@

Re: [PATCH 8/8] drm/ttm: do not keep GPU dependent addresses

2020-02-18 Thread Christian König
Am 18.02.20 um 13:40 schrieb Thomas Zimmermann: Hi Am 17.02.20 um 16:04 schrieb Nirmoy Das: GPU address handling is device specific and should be handle by its device driver. Signed-off-by: Nirmoy Das --- drivers/gpu/drm/ttm/ttm_bo.c| 7 --- include/drm/ttm/ttm_bo_api.h| 2 --

Re: [PATCH 0/8] do not store GPU address in TTM

2020-02-18 Thread Thomas Zimmermann
Hi, the email's subject line should contain the version of the patchset (e.g, v2, v3, etc). git send-mail adds this with the -v switch Best regards Thomas Am 17.02.20 um 16:04 schrieb Nirmoy Das: > With this patch series I am trying to remove GPU address dependency in > TTM and moving GPU

Re: [PATCH 8/8] drm/ttm: do not keep GPU dependent addresses

2020-02-18 Thread Thomas Zimmermann
Hi Am 17.02.20 um 16:04 schrieb Nirmoy Das: > GPU address handling is device specific and should be handle by its device > driver. > > Signed-off-by: Nirmoy Das > --- > drivers/gpu/drm/ttm/ttm_bo.c| 7 --- > include/drm/ttm/ttm_bo_api.h| 2 -- > include/drm/ttm/ttm_bo_driver.h | 1

Re: [PATCH v2 RESEND] drm/i915/gvt: make gvt oblivious of kvmgt data structures

2020-02-18 Thread Zhenyu Wang
On 2020.02.17 17:38:58 +0100, Julian Stecklina wrote: > Instead of defining KVMGT per-device state in struct intel_vgpu > directly, add an indirection. This makes the GVT code oblivious of > what state KVMGT needs to keep. > > The intention here is to eventually make it possible to build >

[PATCH v2 4/4] drm/qxl: Use simple encoder

2020-02-18 Thread Thomas Zimmermann
The qxl driver uses an empty implementation for its encoder. Replace the code with the generic simple encoder. v2: * rebase onto new simple-encoder interface Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/qxl/qxl_display.c | 18 +++--- 1 file changed, 3 insertions(+),

[PATCH v2 3/4] drm/mgag200: Use simple encoder

2020-02-18 Thread Thomas Zimmermann
The mgag200 driver uses an empty implementation for its encoder. Replace the code with the generic simple encoder. v2: * rebase onto new simple-encoder interface Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/mgag200/mgag200_drv.h | 7 --- drivers/gpu/drm/mgag200/mgag200_mode.c

[PATCH v2 1/4] drm/simple-kms: Add drm_simple_encoder_{init, create}()

2020-02-18 Thread Thomas Zimmermann
This patch makes the internal encoder implementation of the simple KMS helpers available to drivers. These simple-encoder helpers initialize an encoder with an empty implementation. This covers the requirements of most of the existing DRM drivers. A call to drm_simple_encoder_create() allocates

[PATCH v2 0/4] drm: Provide a simple encoder

2020-02-18 Thread Thomas Zimmermann
Many DRM drivers implement an encoder with an empty implementation. This patchset adds drm_simple_encoder_init() and drm_simple_encoder_create(), which can be used by drivers instead. Except for the destroy callback, the simple encoder's implementation is empty. The patchset also converts 4

[PATCH v2 2/4] drm/ast: Use simple encoder

2020-02-18 Thread Thomas Zimmermann
The ast driver uses an empty implementation for its encoder. Replace the code with the generic simple encoder. v2: * rebase onto new simple-encoder interface Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/ast/ast_drv.h | 6 +- drivers/gpu/drm/ast/ast_mode.c | 25

Re: [PATCH 2/3] drm/mediatek: make sure previous message done or be aborted before send

2020-02-18 Thread CK Hu
Hi, Bibby: On Fri, 2020-02-14 at 14:48 +0800, CK Hu wrote: > Hi, Bibby: > > On Fri, 2020-02-14 at 12:49 +0800, Bibby Hsieh wrote: > > Mediatek CMDQ driver removed atomic parameter and implementation > > related to atomic. DRM driver need to make sure previous message > > done or be aborted

  1   2   >