[Bug 104194] HDMI with RX460 not working on linux 4.15-rc2 with DC enabled and linux-amd-staging-drm-next

2017-12-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104194 --- Comment #1 from laichiah...@outlook.com --- I've found a solution to make it work, just re-plug-in the HDMI cable from the GPU side (not the screen side) -- You are receiving this mail because: You are the assignee for the

Re: tinydrm: page allocation failure

2017-12-11 Thread Noralf Trønnes
Den 11.12.2017 04.28, skrev David Lechner: I'm using drm-misc/drm-misc-next and occasionally getting errors as seen in the stack traces below. I'm not really sure what to make of it. Any ideas? The spi controller driver has told the spi core to allocate dummy buffers for tx/rx: spi_sync ->

[PATCH] [v2] drm/i915: use static const array for PICK macro

2017-12-11 Thread Arnd Bergmann
The varargs macro trick in _PIPE3/_PHY3/_PORT3 was meant as an optimization to shrink the i915 kernel module by around 1000 bytes. However, the downside is a size regression with CONFIG_KASAN, as I found from stack size warnings with gcc-7.0.1: before: drivers/gpu/drm/i915/intel_dpll_mgr.c: In

[Bug 104192] [amdgpu][VEGA10] regular lockups with VM_L2_PROTECTION_FAULT_STATUS

2017-12-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104192 --- Comment #11 from Tom Englund --- (In reply to Tom Englund from comment #10) > (In reply to Michel Dänzer from comment #9) > > FWIW, I bisected some piglit regressions to LLVM SVN r319894. > > thanks, that gave me

Re: [Intel-gfx] [PATCH] drm/i915: use static const array for PICK macro

2017-12-11 Thread Arnd Bergmann
On Tue, Mar 21, 2017 at 12:23 PM, Jani Nikula wrote: > On Tue, 21 Mar 2017, Daniel Vetter wrote: >> On Tue, Mar 21, 2017 at 09:44:07AM +0100, Arnd Bergmann wrote: >>> On Tue, Mar 21, 2017 at 9:26 AM, Jani Nikula >>>

[Bug 104192] [amdgpu][VEGA10] regular lockups with VM_L2_PROTECTION_FAULT_STATUS

2017-12-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104192 --- Comment #10 from Tom Englund --- (In reply to Michel Dänzer from comment #9) > FWIW, I bisected some piglit regressions to LLVM SVN r319894. thanks, that gave me some starting points to bisecting, narrowed things

Re: [PATCH v14 0/3] Move backlight helper functions from tinydrm-helpers to linux/backlight

2017-12-11 Thread Meghana Madhyastha
On Sat, Dec 09, 2017 at 03:09:28PM +0100, Noralf Trønnes wrote: > > Den 21.10.2017 13.55, skrev Meghana Madhyastha: > >Changes in v14: > >- s/backlight_get/of_find_backlight/ in patch 2/3 > >- Change commit message in patch 3/3 from requiring to acquiring > > > >Meghana Madhyastha (3): > >

Re: [PATCH 2/3] uapi: drm: New fourcc codes needed by Xilinx Video IP

2017-12-11 Thread Laurent Pinchart
Hi Ville, On Tuesday, 28 November 2017 20:11:42 EET Ville Syrjälä wrote: > On Tue, Nov 28, 2017 at 05:25:46PM +, Hyun Kwon wrote: > > On Tuesday, November 28, 2017 6:44 AM Ville Syrjälä wrote: > >> On Mon, Nov 27, 2017 at 06:27:32PM -0800, Hyun Kwon wrote: > >>> From: Jeffrey Mouroux

Re: [PATCH v14 0/3] Move backlight helper functions from tinydrm-helpers to linux/backlight

2017-12-11 Thread Noralf Trønnes
Den 11.12.2017 14.17, skrev Meghana Madhyastha: On Sat, Dec 09, 2017 at 03:09:28PM +0100, Noralf Trønnes wrote: Den 21.10.2017 13.55, skrev Meghana Madhyastha: Changes in v14: - s/backlight_get/of_find_backlight/ in patch 2/3 - Change commit message in patch 3/3 from requiring to acquiring

Re: [PATCH 2/3] uapi: drm: New fourcc codes needed by Xilinx Video IP

2017-12-11 Thread Ville Syrjälä
On Mon, Dec 11, 2017 at 04:08:07PM +0200, Laurent Pinchart wrote: > Hi Ville, > > On Tuesday, 28 November 2017 20:11:42 EET Ville Syrjälä wrote: > > On Tue, Nov 28, 2017 at 05:25:46PM +, Hyun Kwon wrote: > > > On Tuesday, November 28, 2017 6:44 AM Ville Syrjälä wrote: > > >> On Mon, Nov 27,

Re: [Intel-gfx] [PATCH v6 0/5] drm/i915: Expose more GPU properties through sysfs

2017-12-11 Thread Lionel Landwerlin
On 11/12/17 10:50, Joonas Lahtinen wrote: + Daniel, Chris On Thu, 2017-12-07 at 09:21 +, Tvrtko Ursulin wrote: On 04/12/2017 15:02, Lionel Landwerlin wrote: Hi, After discussion with Chris, Joonas & Tvrtko, this series adds an additional commit to link the render node back to the card

[Bug 104206] [drm:construct [amdgpu]] *ERROR* construct: Invalid Connector ObjectID from Adapter Service for connector index:2!

2017-12-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104206 --- Comment #3 from Andrew --- Michael, Thank you for your reply. Why pastebin is not good? ( Thought fedora's one is blessed/approved) The rawhide mesa rpms versions are the same as in f27. Should i recompile/reconfigure

Re: [PATCH 1/3] drm: drm_fourcc: Add scaling and padding factor to drm_format_info

2017-12-11 Thread Laurent Pinchart
Hi Hyun, Thank you for the patch. On Tuesday, 28 November 2017 04:27:31 EET Hyun Kwon wrote: > From: Satish Kumar Nagireddy > > 'cpp_scale' can be used as a multiplying factor to calculate > bytes per component based on color format. > For eg. scaling

Re: [Intel-gfx] [PATCH v6 0/5] drm/i915: Expose more GPU properties through sysfs

2017-12-11 Thread Tvrtko Ursulin
On 11/12/2017 10:50, Joonas Lahtinen wrote: + Daniel, Chris On Thu, 2017-12-07 at 09:21 +, Tvrtko Ursulin wrote: On 04/12/2017 15:02, Lionel Landwerlin wrote: Hi, After discussion with Chris, Joonas & Tvrtko, this series adds an additional commit to link the render node back to the card

Re: [Intel-gfx] [PATCH v6 0/5] drm/i915: Expose more GPU properties through sysfs

2017-12-11 Thread Lionel Landwerlin
On 11/12/17 14:38, Tvrtko Ursulin wrote: On 11/12/2017 10:50, Joonas Lahtinen wrote: + Daniel, Chris On Thu, 2017-12-07 at 09:21 +, Tvrtko Ursulin wrote: On 04/12/2017 15:02, Lionel Landwerlin wrote: Hi, After discussion with Chris, Joonas & Tvrtko, this series adds an additional

Re: [PATCH] kthread: finer-grained lockdep/cross-release completion

2017-12-11 Thread Daniel Vetter
On Fri, Dec 08, 2017 at 11:54:19AM +0100, Peter Zijlstra wrote: > On Thu, Dec 07, 2017 at 11:08:49AM +0100, Daniel Vetter wrote: > > Since -rc1 we're hitting a bunch of lockdep splats using the new > > cross-release stuff around the 2 kthread completions. In all cases > > they are because totally

Re: [PATCH] drm: Update edid-derived drm_display_info fields at edid property set

2017-12-11 Thread Daniel Vetter
On Thu, Dec 07, 2017 at 04:42:57PM -0800, Keith Packard wrote: > There are a set of values in the drm_display_info structure for each > connector which hold information derived from EDID. These are computed > in drm_add_display_info. Before this patch, that was only called in > drm_add_edid_modes.

[PATCH] drm: Give the DRM device's anon_inode a unique lockclass for its mmap_rswem

2017-12-11 Thread Chris Wilson
Teach lockdep to track the device's internal mmapping separately from the generic lockclass over all other inodes. Since this is device private we wish to allow a different locking hierarchy than is typified by the requirement for the mmap_rwsem being the outermost lock for handling pagefaults. By

Re: [PATCH 20/27] drm/etnaviv: refcount the submit object

2017-12-11 Thread Philipp Zabel
On Fri, 2017-12-01 at 11:36 +0100, Lucas Stach wrote: > The submit object lifetime will get extended to the actual GPU > execution. As multiple users will depend on this, add a kref to > properly control destuction of the object. > > Signed-off-by: Lucas Stach

Re: [PATCH 17/27] drm/etnaviv: attach in fence to submit and move fence wait to fence_sync

2017-12-11 Thread Philipp Zabel
On Fri, 2017-12-01 at 11:36 +0100, Lucas Stach wrote: > Simplifies the cleanup path and moves fence waiting to a central location. > > Signed-off-by: Lucas Stach Reviewed-by: Philipp Zabel regards Philipp > --- >

[PATCH 21/25] drm/armada: wait for previous work when moving overlay window

2017-12-11 Thread Russell King
We must wait for the previous plane work to complete before moving the overlay window, as it could overwrite our positioning update. Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_overlay.c | 8 1 file changed, 4 insertions(+), 4 deletions(-)

Re: [PATCH] drm/nouveau/imem/nv50: fix incorrect use of refcount API

2017-12-11 Thread Adam Borowski
On Fri, Dec 08, 2017 at 06:30:34PM +, Ard Biesheuvel wrote: > Commit be55287aa5b ("drm/nouveau/imem/nv50: embed nvkm_instobj directly > into nv04_instobj") introduced some new calls to the refcount api to > the nv50 mapping code. In one particular instance, it does the > following: > > if

Re: [PATCHv1 00/14] omapdrm: DSI command mode panel support

2017-12-11 Thread Sebastian Reichel
Hi Tomi, On Fri, Dec 01, 2017 at 02:58:13PM +0200, Tomi Valkeinen wrote: > Hi, > > On 24/07/17 20:32, Sebastian Reichel wrote: > > Hi, > > > > This adds support for command mode DSI panels to > > omapdrm. I tested the patches on Nokia N950 (omap3) > > and Motorola Droid 4 (omap4). This is

[PATCH 13/25] drm/armada: move writes of LCD_SPU_SRAM_PARA1 under lock

2017-12-11 Thread Russell King
Move writes of LCD_SPU_SRAM_PARA1 under the irq lock, so that we can add this to the frame updates at interrupt time when disabling a plane. Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_crtc.c| 10 ++

[PATCH 2/5] drm/armada: fix SRAM powerdown

2017-12-11 Thread Russell King
Avoid powering down the overlay SRAM banks when disabling the primary plane, thereby masking any overlay video. This feature is supposed to allow us to cut the bandwidth required while displaying full-frame overlay video. Signed-off-by: Russell King ---

drm: vc4: WARNING: CPU: 2 PID: 172 at kernel/irq/chip.c:244 __irq_startup+0x9c/0xa8

2017-12-11 Thread Stefan Wahren
Hi, the commit 253696ccd613 ("drm/vc4: Account for interrupts in flight") triggers a warning during boot of Raspberry Pi 2 with multi_v7_defconfig: [7.962699] vc4_hdmi 3f902000.hdmi: vc4-hdmi-hifi <-> 3f902000.hdmi mapping ok [7.962732] vc4_hdmi 3f902000.hdmi: ASoC: no DMI vendor name!

[PATCH 19/25] drm/armada: re-organise overlay register update generation

2017-12-11 Thread Russell King
Re-organise overlay register generation so that we do not have to wait for the previous update to complete while creating the new state. This allows the update to be fully prepared before queueing it for the next interrupt. Signed-off-by: Russell King ---

[PATCH 4/5] drm/armada: improve efficiency of armada_drm_plane_calc_addrs()

2017-12-11 Thread Russell King
Lookup the drm_format_info structure once when computing all the framebuffer plane addresses by using drm_format_info(), rather than repetitive lookups via drm_format_plane_cpp(). Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_crtc.c | 5 +++-- 1 file

[PATCH 07/25] drm/armada: clean up armada_drm_crtc_plane_disable()

2017-12-11 Thread Russell King
Merge armada_drm_primary_disable() into armada_drm_crtc_plane_disable() and rename to armada_drm_plane_disable(). Use this to simplify armada_ovl_plane_disable(). Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_crtc.c| 21 +

[PATCH 24/25] drm/armada: expand overlay trace entry

2017-12-11 Thread Russell King
Add CRTC and source positions to the Armada overlay trace entry. Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_trace.h | 24 ++-- 1 file changed, 22 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/armada/armada_trace.h

[PATCH 16/25] drm/armada: allow armada_drm_plane_work_queue() to silently fail

2017-12-11 Thread Russell King
Avoid printing an error message when armada_drm_plane_work_queue() is unable to get the vblank (eg, because we're doing a modeset.) Continue to report the failure to the caller, so the caller can handle this. Move the error message into armada_ovl_plane_update(). Signed-off-by: Russell King

[PATCH 09/25] drm/armada: move overlay plane work out from under spinlock

2017-12-11 Thread Russell King
Move the overlay plane work out from under the spinlock so that both the primary and overlay planes run their work in the same context. This is necessary so that we can use frame works with the overlay plane. However, we must update the CRTC registers under the spinlock, so fix up the overlay

[PATCH 25/25] drm/armada: add iturbt_709 plane property to control YUV colorspace

2017-12-11 Thread Russell King
Add the defacto-standard "iturbt_709" property to the overlay plane to control the YUV to RGB colorspace conversion. This is mutually exclusive with the CSC_YUV CRTC property - the last property to be set determines the resulting colorspace conversion. Signed-off-by: Russell King

[Bug 102985] HDMI 2.0 4k EDIDs not available (Radeon RX Vega 56)

2017-12-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102985 Michel Dänzer changed: What|Removed |Added Resolution|--- |FIXED

Re: [PATCH] drm/i915/gvt/fb_decoder: Fix out-of-bounds read

2017-12-11 Thread Zhenyu Wang
On 2017.12.09 00:37:59 -0600, Gustavo A. R. Silva wrote: > In case function skl_format_to_drm returns -EINVAL, fmt turns into a huge > number as fmt is of type u32, hence there is an out-of-bounds read when > using fmt as an index for array skl_pixel_formats at line 225: > plane->bpp =

[Bug 104159] llvm 6.0svn: firefox GPU fault/hang with r319975

2017-12-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104159 Christoph Haag changed: What|Removed |Added Summary|llvm 6.0svn: GPU fault/hang |llvm 6.0svn:

Re: [PATCH 18/27] drm/etnaviv: move object unpinning to submit cleanup

2017-12-11 Thread Philipp Zabel
On Fri, 2017-12-01 at 11:36 +0100, Lucas Stach wrote: > This is safe to call in all paths, as the BO_PINNED flag tells us if the BO > needs unpinning. > > Signed-off-by: Lucas Stach Reviewed-by: Philipp Zabel regards Philipp > --- >

Re: [PATCH 03/27] drm/etnaviv: add lockdep annotation for userptr object population

2017-12-11 Thread Christian Gmeiner
2017-12-01 11:36 GMT+01:00 Lucas Stach : > The current userptr page population will defer work to a work item if > needed to avoid ever taking the mmap_sem in the direct call path. With > the more fine-grained locking in etnaviv this isn't needed anymore, so > a future

Re: [PATCH 04/27] drm/etnaviv: fold __etnaviv_gem_new into caller

2017-12-11 Thread Christian Gmeiner
2017-12-01 11:36 GMT+01:00 Lucas Stach : > This function has only one caller and it isn't expected that there will > be any more in the future. Folding this function into the caller is > helping the readability. > > Signed-off-by: Lucas Stach

[Bug 104159] llvm 6.0svn: firefox GPU fault/hang with r319975

2017-12-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104159 --- Comment #3 from Michel Dänzer --- FWIW, I bisected some piglit regressions to LLVM SVN r319894. -- You are receiving this mail because: You are the assignee for the bug.___

Re: [Intel-gfx] [PATCH] drm/i915: Use copy_from_user() in fence copying

2017-12-11 Thread Joonas Lahtinen
On Wed, 2017-12-06 at 12:28 -0800, Kees Cook wrote: > There's no good reason to separate the access_ok() from the copy, > especially since the access_ok() size is hard-coded instead of using > sizeof(). Instead, just use copy_from_user() directly. > > Fixes: cf6e7bac6357 ("drm/i915: Add support

Re: [PATCH v2 1/2] dt-bindings: Add binding for Sitronix ST7735R display panels

2017-12-11 Thread Linus Walleij
On Sun, Dec 10, 2017 at 11:10 PM, David Lechner wrote: > This adds a new device tree binding for Sitronix ST7735R display panels, > such as the Adafruit 1.8" TFT. > > Signed-off-by: David Lechner > Acked-by: Rob Herring (...) >

[PATCH 11/25] drm/armada: move event sending into armada_plane_work

2017-12-11 Thread Russell King
Move the sending of events into the armada_plane_work structure, and combine the processing in armada_drm_plane_work_call(). Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_crtc.c | 27 +-- drivers/gpu/drm/armada/armada_crtc.h |

[PATCH v2 0/5] Armada DRM fixes

2017-12-11 Thread Russell King - ARM Linux
Hi David, This series fixes some issues in the Armada DRM driver: - A memory leak while cleaning up in the CRTC initialisation path - Avoid powering down YUV FIFOs when disabling the graphics plane - Several fixes for the overlay plane positioning Minor updates for the comments from Sean Paul,

[PATCH 14/25] drm/armada: only enable HSMOOTH if scaling horizontally

2017-12-11 Thread Russell King
Only enable the HSMOOTH control bit if we are scaling horizontally, otherwise it makes no sense to enable the horizontal scaler. Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_crtc.c| 5 +++-- drivers/gpu/drm/armada/armada_overlay.c | 10

[PATCH 04/25] drm/armada: add work cancel callback

2017-12-11 Thread Russell King
Add a work cancel callback, so that work items can add functionality to clean themselves up when they are cancelled. Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_crtc.c | 23 --- drivers/gpu/drm/armada/armada_crtc.h | 1 + 2

[PATCH 22/25] drm/armada: extract register generation from armada_drm_primary_set()

2017-12-11 Thread Russell King
Extract the register generation from armada_drm_primary_set(), so that it can be re-used. Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_crtc.c | 25 - 1 file changed, 16 insertions(+), 9 deletions(-) diff --git

Re: [PATCH v2 6/7] drm/i2c: tda998x: add CEC support

2017-12-11 Thread Russell King - ARM Linux
On Wed, Dec 06, 2017 at 02:50:44PM +0100, Hans Verkuil wrote: > Hi Russell, > > Thanks for this patch series! > > On 12/06/17 13:35, Russell King wrote: > > The TDA998x is a HDMI transmitter with a TDA9950 CEC engine integrated > > onto the same die. Add support for the TDA9950 CEC engine to

[PATCH 20/25] drm/armada: move overlay plane register update generation

2017-12-11 Thread Russell King
Move the overlay plane register update generation to a separate function as this is independent of the legacy or atomic update. Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_crtc.h| 2 + drivers/gpu/drm/armada/armada_overlay.c | 203

[PATCH 18/25] drm/armada: disable planes at next blanking period

2017-12-11 Thread Russell King
Disable planes at the next blanking period rather than immediately. In order to achieve this, we need to delay the clearing of dcrtc->plane until after the next blanking period, so move that into a separate work function. To avoid races, we also need to move its assignment in the overlay code.

[PATCH 05/25] drm/armada: wait and cancel any pending frame work at disable

2017-12-11 Thread Russell King
Wait for a second, and if we time out, cancel any pending work when disabling the primary plane. This ensures that any pending work is completed or cleaned up prior to the disable taking effect. Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_crtc.c

Re: [PATCH v2 2/7] drm/i2c: tda998x: move CEC device initialisation later

2017-12-11 Thread Russell King - ARM Linux
On Wed, Dec 06, 2017 at 02:54:04PM +0100, Hans Verkuil wrote: > On 12/06/17 13:35, Russell King wrote: > > We no longer use the CEC client to access the CEC part itself, so we can > > move this later in the initialisation sequence. > > > > Signed-off-by: Russell King

[PATCH 17/25] drm/armada: avoid work allocation

2017-12-11 Thread Russell King
Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_crtc.c| 24 +--- drivers/gpu/drm/armada/armada_crtc.h| 3 +++ drivers/gpu/drm/armada/armada_overlay.c | 11 +++ 3 files changed, 23 insertions(+), 15 deletions(-) diff

RE: [PATCH] drm/i915: Use copy_from_user() in fence copying

2017-12-11 Thread David Laight
From: Kees Cook > Sent: 06 December 2017 20:29 > > There's no good reason to separate the access_ok() from the copy, > especially since the access_ok() size is hard-coded instead of using > sizeof(). Instead, just use copy_from_user() directly. Looks like an optimisation to save doing the

[PATCH 08/25] drm/armada: clear plane enable bit when disabling

2017-12-11 Thread Russell King
Clear the plane enable bit in the software state within armada_drm_plane_disable() when disabling either the primary or overlay planes. Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_crtc.c| 10 ++ drivers/gpu/drm/armada/armada_overlay.c |

[PATCH 03/25] drm/armada: store plane in armada_plane_work

2017-12-11 Thread Russell King
Store the plane in the armada_plane_work structure rather than passing it around; it doesn't get used very much in the work structures, so passing it around is a needless expense. Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_crtc.c| 27

Re: [PATCH 10/27] drm/etnaviv: remove switch_context member from etnaviv_gpu

2017-12-11 Thread Christian Gmeiner
2017-12-01 11:36 GMT+01:00 Lucas Stach : > There is no need to store this in the gpu struct. MMU flushes are triggered > correctly in reaction to MMU maps and unmaps, independent of the current ctx > and required pipe switches can be infered from the current and the desired

Re: [PATCH 08/27] drm/etnaviv: remove stale TODO in etnaviv_gpu_submit

2017-12-11 Thread Christian Gmeiner
2017-12-01 11:36 GMT+01:00 Lucas Stach : > Flush and prefetch are properly handled in the buffer code, data endianess > would need much wider changes than adding something to this single function. > > Signed-off-by: Lucas Stach Reviewed-by:

Re: [PATCH 12/27] drm/etnaviv: hold GPU lock while inserting END command

2017-12-11 Thread Christian Gmeiner
2017-12-01 11:36 GMT+01:00 Lucas Stach : > Inserting the END command when suspending the GPU is changing the > command buffer state, which requires the GPU to be held. > > Signed-off-by: Lucas Stach Reviewed-by: Christian Gmeiner

[Bug 104206] [drm:construct [amdgpu]] *ERROR* construct: Invalid Connector ObjectID from Adapter Service for connector index:2!

2017-12-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104206 Michel Dänzer changed: What|Removed |Added CC|

Re: [PATCHv5 0/3] drm/i915: add DisplayPort CEC-Tunneling-over-AUX support

2017-12-11 Thread Hans Verkuil
Ping again. Added a CC to Ville whom I inexplicably forgot to add when I sent the v5 patch series. Regards, Hans On 01/12/17 08:23, Hans Verkuil wrote: > Ping! > > I really like to get this in for 4.16 so I can move forward with hooking > this up for nouveau/amd. > > Regards, > >

Re: [PATCH 15/27] drm/etnaviv: move object fence attachment to gem_submit path

2017-12-11 Thread Philipp Zabel
On Fri, 2017-12-01 at 11:36 +0100, Lucas Stach wrote: > The object fencing has nothing to do with the actual GPU buffer submit, > so move it to the gem submit path to have a cleaner split. > > Signed-off-by: Lucas Stach Reviewed-by: Philipp Zabel

[PATCH 5/5] drm/armada: fix YUV planar format framebuffer offsets

2017-12-11 Thread Russell King
We weren't correctly calculating the YUV planar offsets for subsampled chroma planes correctly - fix up the coordinates for planes 1 and 2. Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_crtc.c | 8 +++- 1 file changed, 7 insertions(+), 1

[PATCH 23/25] drm/armada: implement primary plane update

2017-12-11 Thread Russell King
Implement primary plane update without having to go through a modeset to achieve that; the hardware does not require such complexity. This means we treat the primary plane as any other, allowing the format, size, position and scaling to be updated via the normal plane ioctls. This also allows us

[PATCH 02/25] drm/armada: add a common frame work allocator

2017-12-11 Thread Russell King
Add and use a common frame work allocator, initialising the frame work to a sane state. Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_crtc.c | 24 +--- 1 file changed, 17 insertions(+), 7 deletions(-) diff --git

[PATCH 1/5] drm/armada: fix leak of crtc structure

2017-12-11 Thread Russell King
Fix the leak of the CRTC structure in the failure paths of armada_drm_crtc_create(). Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_crtc.c | 25 + 1 file changed, 13 insertions(+), 12 deletions(-) diff --git

Re: drm: vc4: WARNING: CPU: 2 PID: 172 at kernel/irq/chip.c:244 __irq_startup+0x9c/0xa8

2017-12-11 Thread Marc Zyngier
On Sun, 10 Dec 2017 03:05:07 +, Stefan Schake wrote: > > On Fri, Dec 8, 2017 at 11:44 PM, Stefan Wahren wrote: > > Hi, > > > > the commit 253696ccd613 ("drm/vc4: Account for interrupts in flight") > > triggers a warning during boot of Raspberry Pi 2 with

[PATCH 01/25] drm/armada: remove armada_drm_plane_work_cancel() return value

2017-12-11 Thread Russell King
armada_drm_plane_work_cancel()'s returned work structure is never used or referenced, so it's pointless returning it. It's also pointless because the caller doesn't have a clue what kind of work structure it is. Signed-off-by: Russell King ---

Re: [Intel-gfx] linux-next: Signed-off-by missing for commit in the drm-intel-fixes tree

2017-12-11 Thread Joonas Lahtinen
+ GVT folks. On Fri, 2017-12-08 at 09:15 +1100, Stephen Rothwell wrote: > Hi all, > > Commit > > 365ad5df9caa ("drm/i915/gvt: Export > intel_gvt_render_mmio_to_ring_id()") > > is missing a Signed-off-by from its committer. > -- Joonas Lahtinen Open Source Technology Center Intel

[Bug 104159] llvm 6.0svn: GPU fault/hang with r319975

2017-12-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104159 --- Comment #1 from Lorenzo Bona --- I can confirm this bug on svn320332 too. I'm using vanilla kernel. -- You are receiving this mail because: You are the assignee for the

[Bug 104206] [drm:construct [amdgpu]] *ERROR* construct: Invalid Connector ObjectID from Adapter Service for connector index:2!

2017-12-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104206 --- Comment #2 from Michel Dänzer --- ... instead of referencing external sites, especially pastebins. Looks like there's a Mesa installation / configuration issue which prevents hardware acceleration from working, and the

Re: [PATCH 2/2] drm: clean up internally created framebuffer on CRTC disable

2017-12-11 Thread Daniel Vetter
On Fri, Dec 08, 2017 at 02:17:39PM -0800, Gurchetan Singh wrote: > The problem I'm trying to solve is that the internal cursor fb is leaky and > even present after closing the drm driver fd. This can be seen by running > modetest after this test case: > >

Re: [RESEND PATCH 0/4] drm/meson: power domain init related fixes

2017-12-11 Thread Jerome Brunet
On Wed, 2017-12-06 at 12:54 +0100, Neil Armstrong wrote: > On the Amlogic Gx SoCs (GXBB, GXL & GXM), the VPU power domain is initialized > by the vendor U-Boot code, but running mainline U-boot has been possible > on these SoCs. But lacking such init made the system lock at kernel boot. > > A PM

Re: [PATCH] drm/i915: Use copy_from_user() in fence copying

2017-12-11 Thread Kees Cook
On Fri, Dec 8, 2017 at 2:17 AM, David Laight wrote: > From: Kees Cook >> Sent: 06 December 2017 20:29 >> >> There's no good reason to separate the access_ok() from the copy, >> especially since the access_ok() size is hard-coded instead of using >> sizeof(). Instead, just

Re: [git pull] drm fixes for v4.15-rc3

2017-12-11 Thread Philippe Ombredanne
Linus, On Fri, Dec 8, 2017 at 10:16 PM, Linus Torvalds wrote: > On Thu, Dec 7, 2017 at 5:20 PM, Dave Airlie wrote: >> >> This pull is a bit larger than I'd like but a large bunch of it is >> license fixes, AMD wanted to fix the licenses for a

[PATCH 10/25] drm/armada: move fb retirement into armada_plane_work

2017-12-11 Thread Russell King
Both the primary and overlay planes retire framebuffers in a similar manner; this can be consolidated by moving the retirement up to the armada_plane_work layer. Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_crtc.c| 12 +++-

[PATCH 12/25] drm/armada: move regs into armada_plane_work

2017-12-11 Thread Russell King
Move the register update structure out of the overlay private structure into armada_plane_work, as this is common to both the primary and overlay planes. Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_crtc.c| 42 --

[PATCH 00/25] Armada DRM development for 4.16

2017-12-11 Thread Russell King - ARM Linux
Hi David, This series builds upon the set of fixes previously submitted to move Armada DRM closer to atomic modeset. We're nowhere near yet, but this series helps to get us closer by unifying some of the differences between the primary and overlay planes. New features added allows userspace to

[PATCH 15/25] drm/armada: use drm_plane_helper_check_state()

2017-12-11 Thread Russell King
Use drm_plane_helper_check_state() to check the overlay plane state rather than drm_plane_helper_check_update(), as: (1) using drm_plane_helper_check_state() provides a better migration path to atomic modeset (2) it avoids needless copies of drm rectangle structures, and so is more

[PATCH 3/5] drm/armada: fix UV swap code

2017-12-11 Thread Russell King
The UV swap code was not always programming things correctly when the source origin box has been offset. Fix this. Signed-off-by: Russell King --- drivers/gpu/drm/armada/armada_crtc.h| 2 ++ drivers/gpu/drm/armada/armada_overlay.c | 38

[PATCH] drm/i915/gvt/fb_decoder: Fix out-of-bounds read

2017-12-11 Thread Gustavo A. R. Silva
In case function skl_format_to_drm returns -EINVAL, fmt turns into a huge number as fmt is of type u32, hence there is an out-of-bounds read when using fmt as an index for array skl_pixel_formats at line 225: plane->bpp = skl_pixel_formats[fmt].bpp; Fix this by comparing the value returned by

Re: drm: vc4: WARNING: CPU: 2 PID: 172 at kernel/irq/chip.c:244 __irq_startup+0x9c/0xa8

2017-12-11 Thread Marc Zyngier
On Sat, 09 Dec 2017 14:17:58 +, Stefan Schake wrote: > > On Sat, Dec 9, 2017 at 1:34 PM, Marc Zyngier wrote: > > On Fri, 08 Dec 2017 22:44:27 +, > > Stefan Wahren wrote: > >> > >> Hi, > >> > >> the commit 253696ccd613 ("drm/vc4: Account for interrupts in > >>

Re: [PATCH v2 2/2] drm/tinydrm: add driver for ST7735R panels

2017-12-11 Thread Philippe Ombredanne
David, On Sun, Dec 10, 2017 at 11:10 PM, David Lechner wrote: > This adds a new driver for Sitronix ST7735R display panels. > > This has been tested using an Adafruit 1.8" TFT. > > Signed-off-by: David Lechner > --- /dev/null > +++

RE: [PATCH] drm/i915: Use copy_from_user() in fence copying

2017-12-11 Thread David Laight
From: Kees Cook > Sent: 08 December 2017 21:10 > >> There's no good reason to separate the access_ok() from the copy, > >> especially since the access_ok() size is hard-coded instead of using > >> sizeof(). Instead, just use copy_from_user() directly. > > > > Looks like an optimisation to save

[PATCH] drm/nouveau/imem/nv50: fix incorrect use of refcount API

2017-12-11 Thread Ard Biesheuvel
Commit be55287aa5b ("drm/nouveau/imem/nv50: embed nvkm_instobj directly into nv04_instobj") introduced some new calls to the refcount api to the nv50 mapping code. In one particular instance, it does the following: if (!refcount_inc_not_zero(>maps)) { ...

Re: [PATCH 05/27] drm/etnaviv: change return type of etnaviv_gem_obj_add to void

2017-12-11 Thread Christian Gmeiner
2017-12-01 11:36 GMT+01:00 Lucas Stach : > This function never fails, as it does nothing more than adding the GEM > object to the global device list. Making this explicit through the void > return type allows to drop some unnecessary error handling. > > Signed-off-by: Lucas

[Bug 104192] [amdgpu][VEGA10] regular lockups with VM_L2_PROTECTION_FAULT_STATUS

2017-12-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104192 --- Comment #9 from Michel Dänzer --- FWIW, I bisected some piglit regressions to LLVM SVN r319894. -- You are receiving this mail because: You are the assignee for the bug.___

[Bug 104205] displayport connected monitor doesn't initialize display

2017-12-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104205 Michel Dänzer changed: What|Removed |Added QA Contact|xorg-t...@lists.x.org |

[Bug 104206] [drm:construct [amdgpu]] *ERROR* construct: Invalid Connector ObjectID from Adapter Service for connector index:2!

2017-12-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104206 --- Comment #7 from Michel Dänzer --- (In reply to Andrew from comment #6) > the X doesn't hard crash when i turn an acceleration off. Normally you'd want Option "Accel" "off" to disable accleration, not "AccelMethod"

Re: [PATCH v14 0/3] Move backlight helper functions from tinydrm-helpers to linux/backlight

2017-12-11 Thread Meghana Madhyastha
On Mon, Dec 11, 2017 at 03:12:06PM +0100, Noralf Trønnes wrote: > > Den 11.12.2017 14.17, skrev Meghana Madhyastha: > >On Sat, Dec 09, 2017 at 03:09:28PM +0100, Noralf Trønnes wrote: > >>Den 21.10.2017 13.55, skrev Meghana Madhyastha: > >>>Changes in v14: > >>>- s/backlight_get/of_find_backlight/

Re: [PATCH 2/3] uapi: drm: New fourcc codes needed by Xilinx Video IP

2017-12-11 Thread Laurent Pinchart
Hi Ville, On Monday, 11 December 2017 16:24:32 EET Ville Syrjälä wrote: > On Mon, Dec 11, 2017 at 04:08:07PM +0200, Laurent Pinchart wrote: > > On Tuesday, 28 November 2017 20:11:42 EET Ville Syrjälä wrote: > >> On Tue, Nov 28, 2017 at 05:25:46PM +, Hyun Kwon wrote: > >>> On Tuesday, November

[Bug 104206] [drm:construct [amdgpu]] *ERROR* construct: Invalid Connector ObjectID from Adapter Service for connector index:2!

2017-12-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104206 --- Comment #4 from Andrew --- Created attachment 136081 --> https://bugs.freedesktop.org/attachment.cgi?id=136081=edit Xlog -- You are receiving this mail because: You are the assignee for the

[Bug 103987] [DC] drm:drm_atomic_helper_wait_for_dependencies - flip_done timed out

2017-12-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103987 --- Comment #6 from Harry Wentland --- Been busy and haven't gotten it in yet. It's still staged. Hope to queue it up for next RC this week. -- You are receiving this mail because: You are the assignee for the

[Bug 104206] [drm:construct [amdgpu]] *ERROR* construct: Invalid Connector ObjectID from Adapter Service for connector index:2!

2017-12-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104206 --- Comment #5 from Andrew --- Created attachment 136082 --> https://bugs.freedesktop.org/attachment.cgi?id=136082=edit messages.log -- You are receiving this mail because: You are the assignee for the

[Bug 104206] [drm:construct [amdgpu]] *ERROR* construct: Invalid Connector ObjectID from Adapter Service for connector index:2!

2017-12-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104206 Michel Dänzer changed: What|Removed |Added Attachment #136083|text/x-log |text/plain

[Bug 104206] [drm:construct [amdgpu]] *ERROR* construct: Invalid Connector ObjectID from Adapter Service for connector index:2!

2017-12-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104206 --- Comment #6 from Andrew --- Created attachment 136083 --> https://bugs.freedesktop.org/attachment.cgi?id=136083=edit x log with "Aceel" turned off the X doesn't hard crash when i turn an acceleration off. X still

Re: tinydrm: page allocation failure

2017-12-11 Thread David Lechner
On 12/11/2017 07:37 AM, Noralf Trønnes wrote: Den 11.12.2017 13.45, skrev Noralf Trønnes: Den 11.12.2017 04.28, skrev David Lechner: I'm using drm-misc/drm-misc-next and occasionally getting errors as seen in the stack traces below. I'm not really sure what to make of it. Any ideas? The

[Bug 95306] Random Blank(black) screens on "Carrizo"

2017-12-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=95306 --- Comment #77 from Kelly Anderson --- 4.15rc3 also seems to be fine with respect to this problem. -- You are receiving this mail because: You are the assignee for the bug.___

Re: [PATCH v14 0/3] Move backlight helper functions from tinydrm-helpers to linux/backlight

2017-12-11 Thread Noralf Trønnes
Den 11.12.2017 18.45, skrev Noralf Trønnes: Den 11.12.2017 15.58, skrev Meghana Madhyastha: On Mon, Dec 11, 2017 at 03:12:06PM +0100, Noralf Trønnes wrote: Den 11.12.2017 14.17, skrev Meghana Madhyastha: On Sat, Dec 09, 2017 at 03:09:28PM +0100, Noralf Trønnes wrote: Den 21.10.2017 13.55,

Re: [PATCH v2 1/2] dt-bindings: Add binding for Sitronix ST7735R display panels

2017-12-11 Thread Noralf Trønnes
Den 10.12.2017 23.10, skrev David Lechner: This adds a new device tree binding for Sitronix ST7735R display panels, such as the Adafruit 1.8" TFT. Signed-off-by: David Lechner Acked-by: Rob Herring --- v2: changes: * None, but... I'm wondering about

  1   2   >