Re: DRM Client capability for aspect ratio

2017-10-30 Thread Nautiyal, Ankit K
Hi Daniel, Thanks for pointing out the places which will be impacted if aspect ratio support is added as DRM Client Cap. Thanks also to Daniel Vetter, Ville Syrjala, and Shashank Sharma for suggestions and recommendations in various discussions, to identify the places where the aspect-ratio

Re: [PATCH v6 00/10] rockchip: kevin: Enable edp display

2017-10-30 Thread JeffyChen
Hi Heiko, On 10/31/2017 07:01 AM, Heiko Stuebner wrote: As I was just looking at the edp dts change in patch1 again, does this series also contain a fix for the issue below [0] ? I'm still seeing this on 4.14-rc6 with the most recent drm tree merged in. i saw that too, it should due to our psr

Re: [PATCH] drm/panel: rpi-touchscreen: propagate errors in rpi_touchscreen_i2c_read()

2017-10-30 Thread Eric Anholt
Dan Carpenter writes: > There is one caller which checks whether rpi_touchscreen_i2c_read() > returns negative error codes. Currently it can't because negative > error codes are truncated to u8, but that's easy to fix if we change the > type to int. Reviewed and applied to -next. Thanks! (Not

Re: [PATCH] drm/vc4: Convert timers to use timer_setup()

2017-10-30 Thread Eric Anholt
Kees Cook writes: > In preparation for unconditionally passing the struct timer_list pointer to > all timer callbacks, switch to using the new timer_setup() and from_timer() > to pass the timer pointer explicitly. Reviewed and applied to drm-misc-next. Thanks! signature.asc Description: PGP s

Re: [PATCH v6 00/10] rockchip: kevin: Enable edp display

2017-10-30 Thread Heiko Stuebner
Hi Jeffy, Sean, Am Donnerstag, 19. Oktober 2017, 11:48:02 CET schrieb Jeffy Chen: > > Make edp display works on chromebook kevin(at least for boot animation). > > Also solve some issues i meet during the bringup. > > Changes in v6: > Don't change order of rockchip_drm_psr_register(). > > Chang

[PATCH] drm/dp: Update SET_POWER_MASK to include the D3 Aux-On state too.

2017-10-30 Thread Dhinakaran Pandiyan
Updating the mask is needed to clear all the three power state bits before setting the required power state. Also add a comment documenting that D3 Aux-On state has been available DPCD v1.2 onwards. Thanks to Ville for pointing this out. Cc: Ville Syrjälä Cc: Jani Nikula Signed-off-by: Dhinakara

[Bug 103519] wayland egl apps crash on start with mesa 17.2

2017-10-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103519 Derek Foreman changed: What|Removed |Added Summary|EFL apps crash on start |wayland egl apps crash on

[Bug 103519] EFL apps crash on start with mesa 17.2

2017-10-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103519 --- Comment #1 from Derek Foreman --- I've been trying to debug this a little bit. It's worth noting that this crash occurs for Apostolos when running weston-simple-egl under weston as well (which removes Enlightenment and EFL from the equation

[maintainer-tools PATCH] dim: Sign commits in addition to tags

2017-10-30 Thread Sean Paul
Expanding on Jani's work to sign tags, this patch adds signing for git commit/am. Signed-off-by: Sean Paul --- This has been lightly tested with dim apply-branch/dim push-branch. Sean dim | 78 + 1 file changed, 51 insertions

Re: [PATCH] drm: Enable pr_debug() for drm_printer

2017-10-30 Thread Sean Paul
On Fri, Oct 27, 2017 at 12:06:02PM +0100, Chris Wilson wrote: > pr_debug() is conditionally compiled and requires either > dynamic-debugging to be enabled or for the code to opt-in using #define > DEBUG. Since drm_print provides a central debugging facility using > pr_debug(), make sure it will alw

[Bug 103463] [drm:amdgpu_get_bios [amdgpu]] *ERROR* ACPI VFCT table present but broken (too short #2)

2017-10-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103463 --- Comment #14 from Dennis Schridde --- Created attachment 135171 --> https://bugs.freedesktop.org/attachment.cgi?id=135171&action=edit photo of screen when kernel hangs Please find attached a photo of the screen contents, with "verbose debu

[Bug 103519] EFL apps crash on start with mesa 17.2

2017-10-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103519 Bug ID: 103519 Summary: EFL apps crash on start with mesa 17.2 Product: Mesa Version: 17.2 Hardware: Other OS: All Status: NEW Severity: normal

[Bug 103489] [Regression bisected]: vblank time outs and machine hangs on RX560 drm-next-4.15-dc

2017-10-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103489 --- Comment #2 from osch...@web.de --- OK, I took another look and the first commit which boots and exhibits the problem is 8459f633b65a81979b9b233ed0a91a9fdc98b7bd -- You are receiving this mail because: You are the assignee for the bug.__

RE: [PATCH] drm/ttm: once more fix ttm_buffer_object_transfer

2017-10-30 Thread Deucher, Alexander
> -Original Message- > From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf > Of Christian König > Sent: Monday, October 30, 2017 10:00 AM > To: dri-devel@lists.freedesktop.org; amd-...@lists.freedesktop.org > Subject: [PATCH] drm/ttm: once more fix ttm_buffer_object_trans

[Bug 78562] [gma500] wrong pixel clock on LVDS (half the correct frequency)

2017-10-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=78562 mirh changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

[PATCH v4 10/11] drm/tinydrm: Use drm_gem_cma_print_info()

2017-10-30 Thread Noralf Trønnes
There is a new core debugfs file that prints fb/gem info: /dri//framebuffer Use drm_gem_cma_print_info() to provide info to that output instead of using drm_fb_cma_debugfs_show(). Reviewed-by: Daniel Vetter Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/tinydrm/mipi-dbi.c | 8 +--- incl

[PATCH v4 11/11] drm/cma-helper: Remove drm_fb_cma_debugfs_show()

2017-10-30 Thread Noralf Trønnes
drm_fb_cma_debugfs_show() and drm_gem_cma_describe() are superseded by drm_framebuffer_debugfs_init() and drm_gem_cma_print_info(). Cc: Laurent Pinchart Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/drm_fb_cma_helper.c | 37 drivers/gpu/drm/drm_gem_cma_

[PATCH v4 05/11] drm/atomic: Use drm_framebuffer_print_info()

2017-10-30 Thread Noralf Trønnes
Use drm_framebuffer_print_info() to print framebuffer info in drm_atomic_plane_print_state(). This will give optional GEM info as well. Reviewed-by: Daniel Vetter Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/drm_atomic.c | 18 +++--- 1 file changed, 3 insertions(+), 15 deletion

[PATCH v4 03/11] drm/print: Add drm_printf_indent()

2017-10-30 Thread Noralf Trønnes
Add drm_printf_indent() that adds tab indentation according to argument. Indentation overflow is marked with an X. Reviewed-by: Daniel Vetter Signed-off-by: Noralf Trønnes --- Changes since version 3: - Turn drm_printf_indent() into a macro (Ville) - drm_printf_indent() add overflow marker (Jan

[PATCH v4 09/11] drm/tilcdc: Use drm_gem_cma_print_info()

2017-10-30 Thread Noralf Trønnes
There is a new core debugfs file that prints fb/gem info: /dri//framebuffer Use drm_gem_cma_print_info() to provide info to that output instead of using drm_fb_cma_debugfs_show(). Cc: Jyri Sarha Cc: Tomi Valkeinen Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/tilcdc/tilcdc_drv.c | 2 +- 1

[PATCH v4 00/11] drm/framebuffer: Add framebuffer debugfs file

2017-10-30 Thread Noralf Trønnes
This patchset adds a debugfs file that prints info about the framebuffers. I have followed the style of drm_state_info(). This means that the added drm_framebuffer_print_info() could also be used in drm_atomic_plane_print_state(). In order to do that I had to add a way to vary indentation to fit b

[PATCH v4 06/11] drm/cma-helper: Add drm_gem_cma_print_info()

2017-10-30 Thread Noralf Trønnes
Add drm_gem_cma_print_info() for debugfs printing struct drm_gem_cma_object specific info. Reviewed-by: Daniel Vetter Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/drm_gem_cma_helper.c | 19 +++ include/drm/drm_gem_cma_helper.h | 5 - 2 files changed, 23 insertions(

[PATCH v4 04/11] drm/framebuffer: Add framebuffer debugfs file

2017-10-30 Thread Noralf Trønnes
Add debugfs file that dumps info about the framebuffers and its planes. Also dump info about any connected gem object(s). Reviewed-by: Daniel Vetter Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/drm_debugfs.c | 6 drivers/gpu/drm/drm_framebuffer.c | 59

[PATCH v4 07/11] drm/arc: Use drm_gem_cma_print_info()

2017-10-30 Thread Noralf Trønnes
There is a new core debugfs file that prints fb/gem info: /dri//framebuffer Use drm_gem_cma_print_info() to provide info to that output instead of using drm_fb_cma_debugfs_show(). Cc: Alexey Brodkin Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/arc/arcpgu_drv.c | 2 +- 1 file changed, 1 in

[PATCH v4 08/11] drm/arm/hdlcd: Use drm_gem_cma_print_info()

2017-10-30 Thread Noralf Trønnes
There is a new core debugfs file that prints fb/gem info: /dri//framebuffer Use drm_gem_cma_print_info() to provide info to that output instead of using drm_fb_cma_debugfs_show(). Cc: Liviu Dudau Cc: Brian Starkey Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/arm/hdlcd_drv.c | 2 +- 1 fil

[PATCH v4 01/11] drm/vma-manager: drm_vma_node_start() constify argument

2017-10-30 Thread Noralf Trønnes
Constify argument so functions calling into this take a const argument. Reviewed-by: Ville Syrjälä Signed-off-by: Noralf Trønnes --- include/drm/drm_vma_manager.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/drm/drm_vma_manager.h b/include/drm/drm_vma_manager.h in

[PATCH v4 02/11] drm/framebuffer: drm_framebuffer_read_refcount() constify argument

2017-10-30 Thread Noralf Trønnes
Constify argument so functions calling into this take a const argument. Reviewed-by: Ville Syrjälä Signed-off-by: Noralf Trønnes --- include/drm/drm_framebuffer.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/drm/drm_framebuffer.h b/include/drm/drm_framebuffer.h in

[Bug 103463] [drm:amdgpu_get_bios [amdgpu]] *ERROR* ACPI VFCT table present but broken (too short #2)

2017-10-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103463 --- Comment #13 from Dennis Schridde --- Created attachment 135165 --> https://bugs.freedesktop.org/attachment.cgi?id=135165&action=edit Linux 4.13.10-gentoo config -- You are receiving this mail because: You are the assignee for the bug.___

[Bug 103463] [drm:amdgpu_get_bios [amdgpu]] *ERROR* ACPI VFCT table present but broken (too short #2)

2017-10-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103463 --- Comment #12 from Dennis Schridde --- P.S. If you have ANY hint, on how to gather more information about this issue, I would be most grateful. Maybe there is some way to make the kernel dump stacktraces somewhere, or to make the hardware its

Re: [PATCH] drm: Fix checkpatch issue: "WARNING: braces {} are not necessary for single statement blocks."

2017-10-30 Thread Liviu Dudau
Hi Vitor, Thanks for the patch! Acked-by: Liviu Dudau I will pull the patch and send it to David Airlie in this weeks' pull request. Best regards, Liviu On Mon, Oct 30, 2017 at 11:36:54AM -0200, Vitor Massaru Iha wrote: > Signed-off-by: Vitor Massaru Iha > --- > drivers/gpu/drm/arm/hdlcd_cr

[Bug 103463] [drm:amdgpu_get_bios [amdgpu]] *ERROR* ACPI VFCT table present but broken (too short #2)

2017-10-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103463 --- Comment #11 from Dennis Schridde --- One more thing: It appears as if the first "cold" boot usually fails -- the kernel hanging after "[drm:amdgpu_get_bios [amdgpu]] *ERROR* ACPI VFCT table present but broken (too short #2)" and the blinking

[PATCH v2 12/15] drm/radeon: Use drm_fb_helper_lastclose() and _poll_changed()

2017-10-30 Thread Noralf Trønnes
This driver can use drm_fb_helper_lastclose() in its .lastclose function. It can also use drm_fb_helper_output_poll_changed() as its .output_poll_changed callback. Cc: Alex Deucher Cc: "Christian König" Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/radeon/radeon_display.c | 9 ++--- d

Re: [PATCH] drm/TODO: Clarify DRM_DEV_* conversion

2017-10-30 Thread Sean Paul
On Mon, Oct 30, 2017 at 9:15 AM, Daniel Vetter wrote: > Not everyone agrees this is the best thing, so make it really clear > that maintainers need to be asked first, then the conversion. We've > had a few newbies that did this the other way round, got their patches > rejected, which isn't the bes

[PATCH v2 13/15] drm/rockchip: Use drm_fb_helper_lastclose() and _poll_changed()

2017-10-30 Thread Noralf Trønnes
This driver can use drm_fb_helper_lastclose() as its .lastclose callback. It can also use drm_fb_helper_output_poll_changed() as its .output_poll_changed callback. Cc: Mark Yao Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 9 + drivers/gpu/drm/rockchip/

[PATCH v2 15/15] staging: vboxvideo: Use drm_fb_helper_lastclose()

2017-10-30 Thread Noralf Trønnes
This driver can use drm_fb_helper_lastclose() as its .lastclose callback. Cc: Hans de Goede Signed-off-by: Noralf Trønnes Reviewed-by: Hans de Goede --- drivers/staging/vboxvideo/vbox_drv.c | 2 +- drivers/staging/vboxvideo/vbox_drv.h | 1 - drivers/staging/vboxvideo/vbox_main.c | 12 -

[PATCH v2 14/15] drm/tegra: Use drm_fb_helper_lastclose() and _poll_changed()

2017-10-30 Thread Noralf Trønnes
This driver can use drm_fb_helper_lastclose() as its .lastclose callback. It can also use drm_fb_helper_output_poll_changed() as its .output_poll_changed callback. Cc: Thierry Reding Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/tegra/drm.c | 13 ++--- drivers/gpu/drm/tegra/drm.h |

[PATCH v2 11/15] drm/omap: Use drm_fb_helper_lastclose() and _poll_changed()

2017-10-30 Thread Noralf Trønnes
This driver can use drm_fb_helper_lastclose() as its .lastclose callback. It can also use drm_fb_helper_output_poll_changed() as its .output_poll_changed callback. Cc: Tomi Valkeinen Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/omapdrm/omap_drv.c | 34 ++ 1

[PATCH v2 10/15] drm/nouveau: Use drm_fb_helper_output_poll_changed()

2017-10-30 Thread Noralf Trønnes
This driver can use drm_fb_helper_output_poll_changed() instead of its own nouveau_fbcon_output_poll_changed(). Cc: Ben Skeggs Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/nouveau/nouveau_display.c | 3 ++- drivers/gpu/drm/nouveau/nouveau_fbcon.c | 8 drivers/gpu/drm/nouveau/nou

[PATCH v2 04/15] drm/amdgpu: Use drm_fb_helper_lastclose() and _poll_changed()

2017-10-30 Thread Noralf Trønnes
This driver can use drm_fb_helper_lastclose() in its .lastclose function. It can also use drm_fb_helper_output_poll_changed() as its .output_poll_changed callback. Cc: Alex Deucher Cc: "Christian König" Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/amd/amdgpu/amdgpu_display.c | 9 ++--

[PATCH v2 09/15] drm/msm: Use drm_fb_helper_lastclose() and _poll_changed()

2017-10-30 Thread Noralf Trønnes
This driver can use drm_fb_helper_lastclose() as its .lastclose callback. It can also use drm_fb_helper_output_poll_changed() as its .output_poll_changed callback. Cc: Rob Clark Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/msm/msm_drv.c | 18 ++ 1 file changed, 2 insertions

[PATCH v2 06/15] drm/exynos: Use drm_fb_helper_lastclose() and _poll_changed()

2017-10-30 Thread Noralf Trønnes
This driver can use drm_fb_helper_lastclose() as its .lastclose callback. It can also use drm_fb_helper_output_poll_changed() as its .output_poll_changed callback. Cc: Inki Dae Cc: Joonyoung Shim Cc: Seung-Woo Kim Cc: Kyungmin Park Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/exynos/exy

[PATCH v2 03/15] drm/fb-helper: Add .last_close and .output_poll_changed helpers

2017-10-30 Thread Noralf Trønnes
This adds helpers for the drm_driver->last_close and the drm_mode_config_funcs->output_poll_changed callbacks. Signed-off-by: Noralf Trønnes Reviewed-by: Daniel Vetter --- drivers/gpu/drm/drm_fb_helper.c | 28 include/drm/drm_fb_helper.h | 11 +++ 2 file

[PATCH v2 08/15] drm/i915: Use drm_fb_helper_output_poll_changed()

2017-10-30 Thread Noralf Trønnes
This driver can use drm_fb_helper_output_poll_changed() as its .output_poll_changed callback. Cc: Jani Nikula Cc: Joonas Lahtinen Cc: Rodrigo Vivi Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/i915/intel_display.c | 2 +- drivers/gpu/drm/i915/intel_drv.h | 5 - drivers/gpu/drm/i91

[PATCH v2 07/15] drm/gma500: Use drm_fb_helper_lastclose() and _poll_changed()

2017-10-30 Thread Noralf Trønnes
This driver can use drm_fb_helper_lastclose() as its .lastclose callback. It can also use drm_fb_helper_output_poll_changed() as its .output_poll_changed callback. Cc: Patrik Jakobsson Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/gma500/framebuffer.c | 9 + drivers/gpu/drm/gma500/

[PATCH v2 00/15] drm/fb-helper: Add .last_close and .output_poll_changed helpers

2017-10-30 Thread Noralf Trønnes
This patchset adds fbdev .last_close and .output_poll_changed helpers to reduce fbdev emulation footprint in drivers. I don't know which drivers have their own tree or not, so if you want me to apply your patch to drm-misc, please let me know. I will do a separate patchset for the cma helper driv

[PATCH v2 02/15] drm: Add drm_device->fb_helper pointer

2017-10-30 Thread Noralf Trønnes
drm_fb_helper is *the* way of doing fbdev emulation so add a pointer to struct drm_device. This makes it possible to add callback helpers for .last_close and .output_poll_changed further reducing fbdev emulation footprint in drivers. The pointer is set by drm_fb_helper_init() and cleared by drm_fb_

[PATCH v2 01/15] drm/fb-helper: Handle function NULL argument

2017-10-30 Thread Noralf Trønnes
Make functions tolerate that the drm_fb_helper argument is NULL. This is useful for drivers that continue probing when fbdev emulation fails and not having to do this check themselves. Update docs for functions that already handles this. Signed-off-by: Noralf Trønnes Reviewed-by: Daniel Vetter -

[PATCH v2 05/15] drm/armada: Use drm_fb_helper_lastclose() and _poll_changed()

2017-10-30 Thread Noralf Trønnes
This driver can use drm_fb_helper_lastclose() as its .lastclose callback. It can also use drm_fb_helper_output_poll_changed() as its .output_poll_changed callback. Cc: Russell King Signed-off-by: Noralf Trønnes Acked-by: Russell King --- drivers/gpu/drm/armada/armada_drm.h | 1 - drivers/gp

Re: [Intel-gfx] [PATCH v2 1/2] drm/dp: Bit definition for D3 power state that keeps AUX fully powered

2017-10-30 Thread Jani Nikula
On Fri, 27 Oct 2017, Ville Syrjälä wrote: > On Fri, Oct 27, 2017 at 01:25:01PM +0300, Jani Nikula wrote: >> On Mon, 14 Aug 2017, Harry Wentland wrote: >> > On 2017-08-11 02:10 PM, Dhinakaran Pandiyan wrote: >> >> DPCD 600h - SET_POWER & SET_DP_PWR_VOLTAGE defines power state >> >> >> >> 101 = Se

[PATCH 2/2] dma-buf: try to replace a signaled fence in reservation_object_add_shared_inplace

2017-10-30 Thread Christian König
From: Christian König The amdgpu issue to also need signaled fences in the reservation objects should be fixed by now. Optimize the handling by replacing a signaled fence when adding a new shared one. Signed-off-by: Christian König --- drivers/dma-buf/reservation.c | 18 +++--- 1

[PATCH 1/2] dma-buf: keep only not signaled fence in reservation_object_add_shared_replace v2

2017-10-30 Thread Christian König
From: Christian König The amdgpu issue to also need signaled fences in the reservation objects should be fixed by now. Optimize the list by keeping only the not signaled yet fences around. v2: temporary put the signaled fences at the end of the new container Signed-off-by: Christian König ---

Re: WARNING in drm_modeset_lock_all

2017-10-30 Thread Chris Wilson
Quoting syzbot (2017-10-27 09:09:50) > Hello, > > syzkaller hit the following crash on > 6f20b7a58cb9c0fe00badcdfd65b1f4a8f28dfc6 > git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/master > compiler: gcc (GCC) 7.1.1 20170620 > .config is attached > Raw console output is attached

[Bug 103404] HDMI Audio with RX Vega not working on drm-next-4.15-dc (device listed as unconnected)

2017-10-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103404 --- Comment #5 from Harry Wentland --- It looks like we missed this when bringing up Vega. We got a patch in the works. It should land sometime this week. -- You are receiving this mail because: You are the assignee for the bug.___

Re: [PATCH] drm/ttm: once more fix ttm_buffer_object_transfer

2017-10-30 Thread Dieter Nützel
Hello Christian, is this for these warnings flooding my dmesg-log or do I have to open a new ticket: Current amd-staging-drm-next commit #7781a8c68a6a from you ;-) [ 6588.429302] WARNING: CPU: 5 PID: 2957 at drivers/gpu/drm/amd/amdgpu/../display/dc/dm_services.h:132 generic_reg_update_ex+0x

[PATCH] drm/ttm: once more fix ttm_buffer_object_transfer

2017-10-30 Thread Christian König
From: Christian König When the mutex is locked just in the moment we copy it we end up with a warning that we release a locked mutex. Fix this by properly reinitializing the mutex. Signed-off-by: Christian König --- drivers/gpu/drm/ttm/ttm_bo_util.c | 1 + 1 file changed, 1 insertion(+) diff

Re: [PATCH] MAINTAINERS: Gustavo as drm-misc co-maintainer

2017-10-30 Thread Sean Paul
On Mon, Oct 30, 2017 at 9:10 AM, Daniel Vetter wrote: > Gustavo volunteered to become a drm-misc co-maintainer, he'll take > care of 4.16 to get started. > > Cc: Gustavo Padovan > Cc: Daniel Vetter > Cc: Sean Paul > Cc: Jani Nikula > Cc: Dave Airlie > Signed-off-by: Daniel Vetter Acked-by:

Re: [PATCH 1/3] dt-bindings: display/ti: Add optional property to set memory bandwidth limit

2017-10-30 Thread Peter Ujfalusi
Rob, On 10/27/2017 05:27 AM, Rob Herring wrote: > On Fri, Oct 20, 2017 at 04:12:56PM +0300, Peter Ujfalusi wrote: >> max-memory-bandwidth can be used to specify the maximum bandwidth dispc >> can use when reading display data from main memory. >> >> In some SoC (am437x for example) we have memory

Re: [PATCH] MAINTAINERS: Gustavo as drm-misc co-maintainer

2017-10-30 Thread Jani Nikula
On Mon, 30 Oct 2017, Daniel Vetter wrote: > Gustavo volunteered to become a drm-misc co-maintainer, he'll take > care of 4.16 to get started. > > Cc: Gustavo Padovan > Cc: Daniel Vetter > Cc: Sean Paul > Cc: Jani Nikula > Cc: Dave Airlie > Signed-off-by: Daniel Vetter Acked-by: Jani Nikula

[PATCH] drm: Fix checkpatch issue: "WARNING: braces {} are not necessary for single statement blocks."

2017-10-30 Thread Vitor Massaru Iha
Signed-off-by: Vitor Massaru Iha --- drivers/gpu/drm/arm/hdlcd_crtc.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/arm/hdlcd_crtc.c b/drivers/gpu/drm/arm/hdlcd_crtc.c index 72b22b805412..5a5427bbd70e 100644 --- a/drivers/gpu/drm/arm/hdlcd_crtc.c +++ b/driv

Re: [PATCH] drm/vgem: Fix vgem_init to get drm device avaliable.

2017-10-30 Thread Emil Velikov
On 26 October 2017 at 00:02, Deepak Sharma wrote: > From: Deepak Sharma > > Modify vgem_init to take platform dev as parent in drm_dev_init. > This will make drm device available at "/sys/devices/platform/vgem" > in x86 chromebook. > Shouldn't one update the drm_dev_init/drm_dev_alloc documentati

Re: [PATCH 1/2] drm/rcar-du: Use common error handling code in rcar_du_encoders_init()

2017-10-30 Thread Laurent Pinchart
Hi Jani, On Monday, 30 October 2017 11:52:07 EET Jani Nikula wrote: > On Sun, 29 Oct 2017, Laurent Pinchart wrote: > > On Friday, 27 October 2017 21:45:17 EET Jani Nikula wrote: > >> On Tue, 24 Oct 2017, SF Markus Elfring wrote: > >>> Add a jump target so that a bit of exception handling can be be

[PATCH] drm/TODO: Clarify DRM_DEV_* conversion

2017-10-30 Thread Daniel Vetter
Not everyone agrees this is the best thing, so make it really clear that maintainers need to be asked first, then the conversion. We've had a few newbies that did this the other way round, got their patches rejected, which isn't the best newbie experience. Cc: Sean Paul Signed-off-by: Daniel Vett

Re: [PATCH v5 1/2] x86 / i915 iosf_mbi / PMIC bus access fixes

2017-10-30 Thread Hans de Goede
Thomas, Ingo, Ping? Can we please get an ack for taking this upstream through the drm subsys tree ? Regards, Hans On 19-10-17 13:16, Hans de Goede wrote: Hi All, Here is a split-up version of my "drm/i915: Acquire PUNIT->PMIC bus for intel_uncore_forcewake_reset()" patch, this time with the

Re: [PATCH resend v3 0/3] drm/i915: Misc. PMIC bus access notifier fixes

2017-10-30 Thread Hans de Goede
Hi, On 19-10-17 10:34, Hans de Goede wrote: Hi All, Here is a resend of the first 3 of the 4 patches in my Misc. PMIC bus access notifier fixes series. I'm leaving out the 4th patch this time, because that includes some changes to the iosf_mbi code under arch/x86 and that has been blocking this

[PATCH] MAINTAINERS: Gustavo as drm-misc co-maintainer

2017-10-30 Thread Daniel Vetter
Gustavo volunteered to become a drm-misc co-maintainer, he'll take care of 4.16 to get started. Cc: Gustavo Padovan Cc: Daniel Vetter Cc: Sean Paul Cc: Jani Nikula Cc: Dave Airlie Signed-off-by: Daniel Vetter --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/

Re: [PATCH] drm/vgem: Fix vgem_init to get drm device avaliable.

2017-10-30 Thread Sean Paul
On Wed, Oct 25, 2017 at 04:02:26PM -0700, Deepak Sharma wrote: > From: Deepak Sharma > > Modify vgem_init to take platform dev as parent in drm_dev_init. > This will make drm device available at "/sys/devices/platform/vgem" > in x86 chromebook. > > Signed-off-by: Deepak Sharma Reviewed-by: Sea

Re: [Intel-gfx] [PATCH v2 2/2] Test case for drm_vblank_cleanup refcount validation patch

2017-10-30 Thread PrasannaKumar Muralidharan
Hi Daniel, On 30 October 2017 at 15:40, Daniel Vetter wrote: > On Wed, Oct 25, 2017 at 08:44:45PM +0530, PrasannaKumar Muralidharan wrote: >> Hi All, >> >> On 24 October 2017 at 22:18, PrasannaKumar Muralidharan >> wrote: >> > In i915 driver unload drm_vblank_get is added to test whether >> > dr

Re: [Intel-gfx] [PATCH v3 6/7] efifb: Set info->fbcon_rotate_hint based on drm_get_panel_orientation_quirk

2017-10-30 Thread Hans de Goede
Hi, On 30-10-17 10:53, Daniel Vetter wrote: On Mon, Oct 23, 2017 at 09:14:24AM +0200, Hans de Goede wrote: On some hardware the LCD panel is not mounted upright in the casing, but rotated by 90 degrees. In this case we want the console to automatically be rotated to compensate. The drm subsys

Re: [Intel-gfx] [PATCH v3 4/7] drm/fb-helper: Apply panel orientation connector prop to the primary plane

2017-10-30 Thread Hans de Goede
Hi, On 30-10-17 10:52, Daniel Vetter wrote: On Mon, Oct 23, 2017 at 09:14:22AM +0200, Hans de Goede wrote: Apply the "panel orientation" drm connector prop to the primary plane so that fbcon and fbdev using userspace programs display the right way up. Fixes: https://bugs.freedesktop.org/show_b

Re: [Intel-gfx] [PATCH v3 3/7] drm: Add support for a panel-orientation connector property

2017-10-30 Thread Hans de Goede
Hi, On 30-10-17 10:43, Daniel Vetter wrote: On Mon, Oct 23, 2017 at 09:14:21AM +0200, Hans de Goede wrote: On some devices the LCD panel is mounted in the casing in such a way that the up/top side of the panel does not match with the top side of the device (e.g. it is mounted upside-down). Thi

Re: [Intel-gfx] [PATCH v3 2/7] drm: Add panel orientation quirks

2017-10-30 Thread Hans de Goede
Hi, On 30-10-17 10:39, Daniel Vetter wrote: On Mon, Oct 23, 2017 at 09:14:20AM +0200, Hans de Goede wrote: Some x86 clamshell design devices use portrait tablet screens and a display engine which cannot rotate in hardware, so the firmware just leaves things as is and we cannot figure out that t

Re: [PATCH v3 8/8] drm/tinydrm: Use drm_gem_cma_print_info()

2017-10-30 Thread Daniel Vetter
On Thu, Oct 26, 2017 at 06:57:31PM +0200, Noralf Trønnes wrote: > Use drm_gem_cma_print_info() instead of drm_fb_cma_debugfs_show() > to print framebuffer/gem info to debugfs. The debugfs file is now: > /dri//framebuffer > > Signed-off-by: Noralf Trønnes > --- > drivers/gpu/drm/tinydrm/mipi-dbi.

Re: gma500: mmu: unmap the correct address

2017-10-30 Thread Daniel Vetter
On Sun, Oct 29, 2017 at 01:55:12PM +0800, Pan Bian wrote: > The function kunmap_atomatic() is called on the same variable twice, > i.e. pt->v. In the second call, its parameter should be variable v > rather than pt->v. > > Signed-off-by: Pan Bian Makes sense, not sure this ever worked or how it

Re: [PATCH v2] drm/framebuffer: Add framebuffer debugfs file

2017-10-30 Thread Daniel Vetter
On Wed, Oct 25, 2017 at 07:18:28PM +0300, Ville Syrjälä wrote: > It also occurs to me that we might want to combine this fb dumper with > the atomic state dumper. That one already prints out stuf about > framebuffers in drm_atomic_plane_print_state(). Not sure if there > would be a nice way to keep

Re: [PATCH 1/3] drm: add connector info/property for non-desktop displays

2017-10-30 Thread Daniel Vetter
On Wed, Oct 25, 2017 at 01:23:58PM -0700, Keith Packard wrote: > Daniel Stone writes: > > > I still think putting these behind a client cap is a good idea: > > https://lists.freedesktop.org/archives/dri-devel/2017-October/155616.html > > Assuming that everyone eventually catches up, this should

Re: [Intel-gfx] [PATCH v2 2/2] Test case for drm_vblank_cleanup refcount validation patch

2017-10-30 Thread Daniel Vetter
On Wed, Oct 25, 2017 at 08:44:45PM +0530, PrasannaKumar Muralidharan wrote: > Hi All, > > On 24 October 2017 at 22:18, PrasannaKumar Muralidharan > wrote: > > In i915 driver unload drm_vblank_get is added to test whether > > drm_vblank_cleanup refcount validation patch is working. > > > > Signed-

[Bug 103489] [Regression bisected]: vblank time outs and machine hangs on RX560 drm-next-4.15-dc

2017-10-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103489 --- Comment #1 from Michel Dänzer --- (In reply to oschowa from comment #0) > Bisecting produced d050f8ed14494ea23c6e10b5893010946ef0f78b as the first bad > commit, although on this commit the machine hangs immediatly after boot, so > it might b

Re: [PATCH] drm: gma500: Convert timers to use timer_setup()

2017-10-30 Thread Daniel Vetter
On Tue, Oct 24, 2017 at 08:16:09AM -0700, Kees Cook wrote: > In preparation for unconditionally passing the struct timer_list pointer to > all timer callbacks, switch to using the new timer_setup() and from_timer() > to pass the timer pointer explicitly. > > Cc: Patrik Jakobsson > Cc: David Airli

Re: [PATCH] drm: virtio: replace reference/unreference with get/put

2017-10-30 Thread Daniel Vetter
On Mon, Oct 23, 2017 at 10:08:06PM +0530, Aastha Gupta wrote: > DRM core uses reference/unreference suffixes for refcounting > functions, but kernel uses get/put (e.g. kref_get/put()). > Replace reference/unreference with get/put for consistency > and also it's shorter. > > The following cocci scr

Re: [PATCH] drm/exynos: Fix dma-buf import

2017-10-30 Thread Tobias Jakobi
Marek Szyprowski wrote: > When IOMMU support was enabled, dma-buf import in Exynos DRM was broken > since commit f43c35966a5a ("drm/exynos: use real device for DMA-mapping > operations") due to using wrong struct device in drm_gem_prime_import() > function. This patch fixes following kernel BUG cau

Re: [PATCH] drm/framebuffer: Add framebuffer debugfs file

2017-10-30 Thread Daniel Vetter
On Tue, Oct 24, 2017 at 04:03:46PM -0700, Kristian Høgsberg wrote: > On Tue, Oct 24, 2017 at 9:39 AM, Noralf Trønnes wrote: > > > > Den 23.10.2017 23.32, skrev Kristian Høgsberg: > >> > >> On Mon, Oct 23, 2017 at 9:47 AM, Noralf Trønnes > >> wrote: > >>> > >>> Add debugfs file that dumps info abo

Re: drm/rcar-du: Use common error handling code in rcar_du_encoders_init()

2017-10-30 Thread SF Markus Elfring
> In general, I'll pick up any patches that are good, This is usual. > but the current track record is that Markus' patches need extra scrutiny, I find that this can be fine according to a safe review for presented update suggestions. > and many of the patches contain subjective changes that

Re: [Intel-gfx] [PATCH v3 0/7] drm/fbdev: Panel orientation connector property support

2017-10-30 Thread Daniel Vetter
On Mon, Oct 23, 2017 at 09:14:18AM +0200, Hans de Goede wrote: > Hi All, > > Here is v3 of my series to add a "panel orientation" property to > the drm-connector for the LCD panel to let userspace know about LCD > panels which are not mounted upright, as well as detecting upside-down > panels with

Re: [Intel-gfx] [PATCH v3 6/7] efifb: Set info->fbcon_rotate_hint based on drm_get_panel_orientation_quirk

2017-10-30 Thread Daniel Vetter
On Mon, Oct 23, 2017 at 09:14:24AM +0200, Hans de Goede wrote: > On some hardware the LCD panel is not mounted upright in the casing, > but rotated by 90 degrees. In this case we want the console to > automatically be rotated to compensate. > > The drm subsys has a quirk table for this, use the >

Re: [Intel-gfx] [PATCH v3 4/7] drm/fb-helper: Apply panel orientation connector prop to the primary plane

2017-10-30 Thread Daniel Vetter
On Mon, Oct 23, 2017 at 09:14:22AM +0200, Hans de Goede wrote: > Apply the "panel orientation" drm connector prop to the primary plane so > that fbcon and fbdev using userspace programs display the right way up. > > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=94894 > Signed-off-by: Hans de

Re: [PATCH 1/2] drm/rcar-du: Use common error handling code in rcar_du_encoders_init()

2017-10-30 Thread Jani Nikula
On Sun, 29 Oct 2017, Laurent Pinchart wrote: > Hi Jani, > > On Friday, 27 October 2017 21:45:17 EET Jani Nikula wrote: >> On Tue, 24 Oct 2017, SF Markus Elfring wrote: >> > Add a jump target so that a bit of exception handling can be better reused >> > at the end of this function. >> > >> > This

Re: [Intel-gfx] [PATCH v3 3/7] drm: Add support for a panel-orientation connector property

2017-10-30 Thread Daniel Vetter
On Mon, Oct 23, 2017 at 09:14:21AM +0200, Hans de Goede wrote: > On some devices the LCD panel is mounted in the casing in such a way that > the up/top side of the panel does not match with the top side of the > device (e.g. it is mounted upside-down). > > This commit adds the necessary infra for

[Bug 91666] build fail with cunit - OK on amd64 - KO for i386

2017-10-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=91666 --- Comment #3 from Michel Dänzer --- (In reply to network723 from comment #2) > Same error while 32-bit build with git master commit > 33dcc29f7ccc554de1fc42f4b95adf056822eabb: It's not exactly the same, so it would have been better to file you

Re: [Intel-gfx] [PATCH v3 2/7] drm: Add panel orientation quirks

2017-10-30 Thread Daniel Vetter
On Mon, Oct 23, 2017 at 09:14:20AM +0200, Hans de Goede wrote: > Some x86 clamshell design devices use portrait tablet screens and a display > engine which cannot rotate in hardware, so the firmware just leaves things > as is and we cannot figure out that the display is oriented non upright > from

Re: RFC: Add drm_dev_suspend/resume() ?

2017-10-30 Thread Daniel Vetter
Hi Noralf, On Sun, Oct 22, 2017 at 06:52:41PM +0200, Noralf Trønnes wrote: > Hi, > > I've spent some time in the fbdev emulation code and discovered a > recurring pattern around suspend/resume. > Should we add some more helpers :-) You're maybe a bit too good at spotting these for your own good

Re: [PATCH 02/15] drm: Add drm_device->drm_fb_helper_private pointer

2017-10-30 Thread Daniel Vetter
On Fri, Oct 20, 2017 at 07:11:18PM +0200, Noralf Trønnes wrote: > > Den 20.10.2017 18.00, skrev Ville Syrjälä: > > On Fri, Oct 20, 2017 at 05:44:15PM +0200, Noralf Trønnes wrote: > > > Den 20.10.2017 15.52, skrev Ville Syrjälä: > > > > On Fri, Oct 20, 2017 at 01:02:00AM +0200, Noralf Trønnes wrote

Re: [PATCH] drm/vblank: Tune drm_crtc_accurate_vblank_count() WARN down to a debug

2017-10-30 Thread Daniel Vetter
On Tue, Oct 24, 2017 at 11:01:32AM +0200, Benjamin Gaignard wrote: > 2017-10-23 17:25 GMT+02:00 Ville Syrjala : > > From: Ville Syrjälä > > > > Since commit 632c6e4edef1 ("drm/vblank: Fix flip event vblank count") > > even drivers that don't implement accurate vblank timestamps will end > > up usi

[PATCH v2 04/10] dt-bindings: simplefb-sunxi: add pipelines for DE2

2017-10-30 Thread Icenowy Zheng
As we're going to add simplefb support for Allwinner SoCs with DE2, add suitable pipeline strings in the device tree binding. Acked-by: Rob Herring Signed-off-by: Icenowy Zheng --- Changes in v2: - Adds Rob's ACK. .../devicetree/bindings/display/simple-framebuffer-sunxi.txt | 4

[PATCH v2 06/10] arm64: allwinner: h5: add compatible string for DE2 CCU

2017-10-30 Thread Icenowy Zheng
The DE2 CCU on Allwinner H5 SoC has a slightly different behavior than the one on H3, so the compatible string is not set in the common DTSI file. Add the compatible string of H5 DE2 CCU in H5 DTSI file. Signed-off-by: Icenowy Zheng --- arch/arm64/boot/dts/allwinner/sun50i-h5.dtsi | 4 1 f

[PATCH 1/3] drm/i915: Reject unknown syncobj flags

2017-10-30 Thread Tvrtko Ursulin
From: Tvrtko Ursulin We have to reject unknown flags for uAPI considerations, and also because the curent implementation limits their i915 storage space to two bits. Signed-off-by: Tvrtko Ursulin Fixes: cf6e7bac6357 ("drm/i915: Add support for drm syncobjs") Cc: Jason Ekstrand Cc: Chris Wilson

[PATCH v2 00/10] Allwinner H3/H5/A64(DE2) SimpleFB support

2017-10-30 Thread Icenowy Zheng
This patchset adds support for the SimpleFB on Allwinner SoCs with "Display Engine 2.0". PATCH 1 to PATCH 3 are DE2 CCU fixes for H3/H5 SoCs. PATCH 4 adds the pipeline strings for DE2 SimpleFB. PATCH 5 to 7 adds necessary device tree nodes (DE2 CCU and SimpleFB) for H3/H5 SoCs. PATCH 8 to 10 ar

gma500: mmu: unmap the correct address

2017-10-30 Thread Pan Bian
The function kunmap_atomatic() is called on the same variable twice, i.e. pt->v. In the second call, its parameter should be variable v rather than pt->v. Signed-off-by: Pan Bian --- drivers/gpu/drm/gma500/mmu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/

[PATCH v2 03/10] clk: sunxi-ng: fix the A64/H5 clock description of DE2 CCU

2017-10-30 Thread Icenowy Zheng
The clocks of A64/H5 SoCs in the DE2 CCU is the same as the clocks in H3 DE2 CCU rather than the A83T DE2 CCU (the parent of them is the DE module clock). Fix this by change the clock descriptions to use the clocks of H3. Fixes: 763c5bd045b1 ("clk: sunxi-ng: add support for DE2 CCU") Signed-off-b

[PATCH v2 01/10] dt-bindings: fix the binding of Allwinner DE2 CCU of A83T and H3

2017-10-30 Thread Icenowy Zheng
The DE2 CCU is different on A83T and H3 -- the parent of the clocks on A83T is PLL_DE but on H3 it's the DE module clock. This is not noticed when I develop the DE2 CCU driver. Fix the binding by using different compatibles for A83T and H3, adding notes for the PLL_DE usage on A83T, and change the

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

2017-10-30 Thread Sebastian Reichel
Hi, On Thu, Oct 26, 2017 at 02:59:10PM +0300, Tomi Valkeinen wrote: > On 24/10/17 01:01, Sebastian Reichel wrote: > > Hi, > > > > On Fri, Oct 13, 2017 at 10:12:08AM -0700, Tony Lindgren wrote: > >> * Tomi Valkeinen [171012 01:46]: > >>> On 29/09/17 16:26, Sebastian Reichel wrote: > Hi Tomi

  1   2   >