[PATCH v5 1/3] drm/bridge/synopsys: dsi: stop clobbering drvdata

2017-12-06 Thread Nickey Yang
From: Brian Norris Bridge drivers/helpers shouldn't be clobbering the drvdata, since a parent driver might need to own this. Instead, let's return our 'dw_mipi_dsi' object and have callers pass that back to us for removal. Signed-off-by: Brian Norris Signed-off-by: Nickey Yang Link:https://pat

[PATCH v5 0/3] Update ROCKCHIP DSI driver that uses dw-mipi-dsi bridge

2017-12-06 Thread Nickey Yang
We now have a generic dw-mipi-dsi bridge driver.So we send this patchs to moving rockchip dw-mipi-dsi driver to that in order to add new features(dual mipi support). Update ROCKCHIP DSI controller driver that uses the Synopsys DesignWare MIPI DSI host controller bridge. ChangeLog: v2: add err_p

[PATCH v5 3/3] drm/rockchip: Add ROCKCHIP DW MIPI DSI controller driver

2017-12-06 Thread Nickey Yang
Add the ROCKCHIP DSI controller driver that uses the Synopsys DesignWare MIPI DSI host controller bridge. Signed-off-by: Nickey Yang Signed-off-by: Brian Norris Reviewed-by: Brian Norris Reviewed-by: Sean Paul --- Changes: v2: add err_pllref, remove unnecessary encoder.enable & disable

[PATCH v5 2/3] dt-bindings: display: rockchip: update DSI controller

2017-12-06 Thread Nickey Yang
This patch update describe panel/port links, including unit addresses in documentation of device tree bindings for the rockchip DSI controller based on the Synopsys DesignWare MIPI DSI host controller. Signed-off-by: Nickey Yang Reviewed-by: Brian Norris --- Changes v5: - add review tag ...

Re: [PATCH 04/10] gpu: host1x: Lock classes during job submission

2017-12-06 Thread Mikko Perttunen
On 07.11.2017 23:23, Dmitry Osipenko wrote: On 07.11.2017 15:28, Mikko Perttunen wrote: On 05.11.2017 18:46, Dmitry Osipenko wrote: On 05.11.2017 14:01, Mikko Perttunen wrote: ... +static int mlock_id_for_class(unsigned int class) +{ +#if HOST1X_HW >= 6 +switch (class) +{ +case HO

Re: [PATCH v2 00/27] Improve DE2 support

2017-12-06 Thread Jernej Škrabec
Hi Maxime, Dne torek, 05. december 2017 ob 11:36:18 CET je Maxime Ripard napisal(a): > Hi, > > On Fri, Dec 01, 2017 at 07:05:23AM +0100, Jernej Skrabec wrote: > > Current DE2 driver is very basic and uses a lot of magic constants since > > there is no documentation and knowledge about it was limi

Re: [PATCH] drm/bridge: analogix dp: Fix runtime PM state in get_modes() callback

2017-12-06 Thread Misha Komarovskiy
Hello Marek, On Tue, Dec 5, 2017 at 5:12 PM, Marek Szyprowski wrote: > Hi Misha, > > On 2017-12-05 15:06, Misha Komarovskiy wrote: >> >> On Mon, Nov 27, 2017 at 6:40 AM, Archit Taneja >> wrote: >>> >>> On 11/21/2017 01:19 PM, Marek Szyprowski wrote: get_modes() callback might be called

Re: [PATCH 04/10] gpu: host1x: Lock classes during job submission

2017-12-06 Thread Dmitry Osipenko
On 05.12.2017 16:21, Mikko Perttunen wrote: > On 07.11.2017 23:23, Dmitry Osipenko wrote: >> On 07.11.2017 15:28, Mikko Perttunen wrote: >>> On 05.11.2017 18:46, Dmitry Osipenko wrote: On 05.11.2017 14:01, Mikko Perttunen wrote: > ... > > +static int mlock_id_for_class(unsigned int

Re: [PATCH] drm/bridge: analogix dp: Fix runtime PM state in get_modes() callback

2017-12-06 Thread Misha Komarovskiy
Hello, On Mon, Nov 27, 2017 at 6:40 AM, Archit Taneja wrote: > > > On 11/21/2017 01:19 PM, Marek Szyprowski wrote: >> >> get_modes() callback might be called asynchronously from the DRM core and >> it is not synchronized with bridge_enable(), which sets proper runtime PM >> state of the main DP d

Re: [PATCH] drm: rcar-du: calculate DPLLCR to be more small jitter

2017-12-06 Thread Kuninori Morimoto
Hi I noticed 1 typo, 1 bug on this patch. I will post v2 patch > From: Kuninori Morimoto > > In general, PLL has VCO (= Voltage controlled oscillator), > one of the very important electronic feature called as "jitter" > is related to this VCO. > In academic generalism, VCO should be maximum to

[PATCH 0/1] drm: Document that drm_panel_orientation_quirks.c is shared with fbdev

2017-12-06 Thread Hans de Goede
Hi All, This is a resend from my gmail mail address so that patchwork and the CI pick this up. Regards, Hans ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: xf86-video-armada via UDL [was: Re: UDL's fbdev doesn't work for user-space apps]

2017-12-06 Thread Alexey Brodkin
Hi Jose, On Tue, 2017-12-05 at 12:26 +, Jose Abreu wrote: > On 05-12-2017 11:53, Alexey Brodkin wrote: > > > > > > From my note above about udl_drm_gem_mmap() being only used in case of > > Xserver > > I barely may conclude anything. Given my lack of knowledge of DRM guts > > especially > >

Re: [PATCH v3 09/15] drm/sun4i: Add A83T support

2017-12-06 Thread Jernej Škrabec
Hi Maxime, Dne torek, 05. december 2017 ob 16:10:21 CET je Maxime Ripard napisal(a): > Add support for the A83T display pipeline. > > Reviewed-by: Chen-Yu Tsai > Signed-off-by: Maxime Ripard > --- > drivers/gpu/drm/sun4i/sun4i_drv.c | 1 + > drivers/gpu/drm/sun4i/sun4i_tcon.c | 5 + >

Re: [PATCH] drm/vmwgfx_kms: Fix potential NULL pointer dereference

2017-12-06 Thread Gustavo A. R. Silva
Hi Ville, Quoting Ville Syrjälä : On Mon, Dec 04, 2017 at 03:54:18PM -0600, Gustavo A. R. Silva wrote: crtc_state is being null checked in a previous code block, which implies that such pointer might be null. crtc_state is dereferenced in drm_atomic_helper_check_plane_state, hence there is a

Re: etnaviv: PHYS_OFFSET usage

2017-12-06 Thread Alexey Brodkin
Hi Lucas, On Wed, 2017-11-15 at 18:56 +0100, Lucas Stach wrote: > Am Mittwoch, den 15.11.2017, 17:36 + schrieb Alexey Brodkin: [snip] > I'm not keen on having a private memory region for the GPU. Normally we > just use the shared system CMA memory region (and we will point the > linear memor

[PATCH 2/2] drm: omapdrm: Simplify platform registration

2017-12-06 Thread Andrew F. Davis
Currently, calls into each file are used to register the various platform drivers. Change this to a table of pointers to platform_driver structs to allow using platform_register_drivers. Signed-off-by: Andrew F. Davis --- drivers/gpu/drm/omapdrm/dss/core.c | 51 +++--

Re: xf86-video-armada via UDL [was: Re: UDL's fbdev doesn't work for user-space apps]

2017-12-06 Thread Alexey Brodkin
Hi Jose, On Tue, 2017-12-05 at 10:39 +, Jose Abreu wrote: > Hi Alexey, > > On 04-12-2017 17:29, Alexey Brodkin wrote: > > > > > > Indeed, in case of kmscube etnaviv is a renderer while UDL > > outputs the picture on the screen. > > Thats nice :) > > Ok, from your logs I was not able to se

[PATCH 1/2] drm: omapdrm: Remove filename from header and fix copyright tag

2017-12-06 Thread Andrew F. Davis
Having the filename in the header serves little purpose and is often wrong after renames as it is here in several places, just drop it from all omapdrm files. While we are here unify the copyright tags to the TI recommended style. Signed-off-by: Andrew F. Davis --- .../gpu/drm/omapdrm/displays/

[PATCH] drm/fb-helper: Fix potential NULL pointer dereference

2017-12-06 Thread Gustavo A. R. Silva
fb_helper is being dereferenced before it is null checked, hence there is a potential null pointer dereference. Fix this by moving the pointer dereference after fb_helper has been null checked. This issue was detected with the help of Coccinelle. Fixes: c777990fb45b ("drm/fb-helper: Handle funct

[PATCH] drm: Document that drm_panel_orientation_quirks.c is shared with fbdev

2017-12-06 Thread Hans de Goede
Add a comment to the DRM_PANEL_ORIENTATION_QUIRKS documenting that the reason for a separate Kconfig for this is because drm_panel_orientation_quirks.c code is shared with fbdev. Suggested-by: Bartlomiej Zolnierkiewicz Reviewed-by: Daniel Vetter Signed-off-by: Hans de Goede --- drivers/gpu/drm

[PATCH v2] drm: rcar-du: calculate DPLLCR to be more small jitter

2017-12-06 Thread Kuninori Morimoto
From: Kuninori Morimoto In general, PLL has VCO (= Voltage controlled oscillator), one of the very important electronic feature called as "jitter" is related to this VCO. In academic generalism, VCO should be maximum to be more small jitter. In high frequency clock, jitter will be large impact.

Re: [PATCH] drm/ttm: swap consecutive allocated pooled pages v4

2017-12-06 Thread Dieter Nützel
Tested-by: Dieter Nützel Dieter Am 05.12.2017 13:54, schrieb Christian König: When we detect consecutive allocation of pages swap them to avoid accidentally freeing them as huge page. v2: use swap v3: check if it's really the first allocated page v4: don't touch the loop variable Signed-off-

[Bug 103913] DRM/Radeon GPU hang

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103913 ro...@beardandsandals.co.uk changed: What|Removed |Added Status|NEW |RESOLVED Resolutio

[PATCH v6 1/3] drm/bridge/synopsys: dsi: stop clobbering drvdata

2017-12-06 Thread Nickey Yang
From: Brian Norris Bridge drivers/helpers shouldn't be clobbering the drvdata, since a parent driver might need to own this. Instead, let's return our 'dw_mipi_dsi' object and have callers pass that back to us for removal. Signed-off-by: Brian Norris Signed-off-by: Nickey Yang Link:https://pat

[PATCH v6 0/3] Update ROCKCHIP DSI driver that uses dw-mipi-dsi bridge

2017-12-06 Thread Nickey Yang
We now have a generic dw-mipi-dsi bridge driver.So we send this patchs to moving rockchip dw-mipi-dsi driver to that in order to add new features(dual mipi support). Update ROCKCHIP DSI controller driver that uses the Synopsys DesignWare MIPI DSI host controller bridge. ChangeLog: v2: add err_p

[PATCH v6 2/3] dt-bindings: display: rockchip: update DSI controller

2017-12-06 Thread Nickey Yang
This patch update describe panel/port links, including unit addresses in documentation of device tree bindings for the rockchip DSI controller based on the Synopsys DesignWare MIPI DSI host controller. Signed-off-by: Nickey Yang Reviewed-by: Brian Norris --- .../display/rockchip/dw_mipi_dsi_roc

[PATCH v6 3/3] drm/rockchip: Add ROCKCHIP DW MIPI DSI controller driver

2017-12-06 Thread Nickey Yang
Add the ROCKCHIP DSI controller driver that uses the Synopsys DesignWare MIPI DSI host controller bridge. Signed-off-by: Nickey Yang Signed-off-by: Brian Norris Reviewed-by: Brian Norris Reviewed-by: Sean Paul --- change: v2: add err_pllref, remove unnecessary encoder.enable & disable c

Re: [PATCH 0/1] drm: Document that drm_panel_orientation_quirks.c is shared with fbdev

2017-12-06 Thread Daniel Vetter
On Tue, Dec 05, 2017 at 12:00:34PM +0100, Hans de Goede wrote: > Hi All, > > This is a resend from my gmail mail address so that patchwork and the CI > pick this up. If you mean the intel CI farm, then you need to cc: intel-gfx. Unfortunately our CI does not (yet) watch what's going on on dri-dev

Re: [PATCH] drm/fb-helper: Fix potential NULL pointer dereference

2017-12-06 Thread Daniel Vetter
On Tue, Dec 05, 2017 at 11:46:28AM -0600, Gustavo A. R. Silva wrote: > fb_helper is being dereferenced before it is null checked, > hence there is a potential null pointer dereference. > > Fix this by moving the pointer dereference after fb_helper > has been null checked. > > This issue was detec

Re: [PATCH v3 00/11] drm/fb-helper: Add .last_close and .output_poll_changed helpers

2017-12-06 Thread Daniel Vetter
On Tue, Dec 05, 2017 at 04:11:26PM -0500, Alex Deucher wrote: > On Tue, Dec 5, 2017 at 1:24 PM, Noralf Trønnes wrote: > > The helpers are applied and have reached airlied/drm-next. > > > > amd has gained another .poll_changed user since last. > > Patches 1, 2, 9 applied to my -next tree. Thanks!

Re: [Nouveau] [PATCH] drm/nouveau/mmu: fix odd_ptr_err.cocci warnings

2017-12-06 Thread Karol Herbst
Reviewed-by: Karol Herbst On Thu, Nov 30, 2017 at 8:53 PM, Christoph Böhmwalder wrote: > The kbuild test bot complained about a new coccinelle warning nearby, > which sparked a discussion about the assignment to 'memory' inside of > the conditional expression. See Link below for the original po

Re: [PATCH 1/2] drm/nouveau/bar/gf100: fix hang when calling ->fini() before ->init()

2017-12-06 Thread Guillaume Tucker
On 05/12/17 18:32, Ben Skeggs wrote: On Wed, Dec 6, 2017 at 12:30 AM, Jon Hunter wrote: On 04/12/17 18:37, Guillaume Tucker wrote: If the firmware fails to load then ->fini() will be called before the device has been initialised, causing the kernel to hang while trying to write to a register

[Bug 104142] Stack trace in runpm when Tonga card powers down

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104142 Bug ID: 104142 Summary: Stack trace in runpm when Tonga card powers down Product: DRI Version: XOrg git Hardware: Other OS: All Status: NEW Severity: nor

Re: [Outreachy kernel] [PATCH v14 2/3] drm/tinydrm: Move tinydrm_of_find_backlight to backlight.c

2017-12-06 Thread Meghana Madhyastha
On Tue, Oct 24, 2017 at 06:45:34PM +0200, Noralf Trønnes wrote: > > Den 24.10.2017 17.42, skrev Sean Paul: > >On Sat, Oct 21, 2017 at 05:27:33PM +0530, Meghana Madhyastha wrote: > >>Rename tinydrm_of_find_backlight to of_find_backlight and move > >>it to linux/backlight.c so that it can be used by

[PATCH] gpu: drm: stm: Adopt SPDX identifiers

2017-12-06 Thread Benjamin Gaignard
Add SPDX identifiers to files under stm directory Signed-off-by: Benjamin Gaignard --- drivers/gpu/drm/stm/drv.c | 3 +-- drivers/gpu/drm/stm/dw_mipi_dsi-stm.c | 3 +-- drivers/gpu/drm/stm/ltdc.c| 3 +-- drivers/gpu/drm/stm/ltdc.h| 3 +-- 4 files changed, 4 in

[Bug 104143] r600/sb: clobbers gl_Position -> gl_FragCoord

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104143 Bug ID: 104143 Summary: r600/sb: clobbers gl_Position -> gl_FragCoord Product: Mesa Version: git Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW Sev

[PATCH] gpu: drm: sti: Adopt SPDX identifiers

2017-12-06 Thread Benjamin Gaignard
Add SPDX identifiers to files under sti directory Signed-off-by: Benjamin Gaignard --- drivers/gpu/drm/sti/sti_awg_utils.c| 2 +- drivers/gpu/drm/sti/sti_awg_utils.h| 2 +- drivers/gpu/drm/sti/sti_compositor.c | 2 +- drivers/gpu/drm/sti/sti_compositor.h | 2 +- drive

Re: nvidia vgaarb bug (was: Re: Static inline DRM functions calling into GPL-only code)

2017-12-06 Thread Lukas Wunner
On Wed, Apr 12, 2017 at 03:46:05PM -0700, Andy Ritger wrote: > On Tue, Apr 11, 2017 at 09:24:33AM +0200, Lukas Wunner wrote: > > Sorry to hijack this thread, but there's an egregious, long-standing bug > > in your driver with regards to vgaarb usage: nvidia/nv.c calls > > vga_tryget() but never ca

[Bug 104143] r600/sb: clobbers gl_Position -> gl_FragCoord

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104143 --- Comment #1 from Gert Wollny --- Created attachment 135999 --> https://bugs.freedesktop.org/attachment.cgi?id=135999&action=edit Version of the original piglit that passes -- You are receiving this mail because: You are the assignee for t

[Bug 104143] r600/sb: clobbers gl_Position -> gl_FragCoord

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104143 --- Comment #2 from Gert Wollny --- Created attachment 136000 --> https://bugs.freedesktop.org/attachment.cgi?id=136000&action=edit piglit screen output of simplified piglit -- You are receiving this mail because: You are the assignee for th

[Bug 104143] r600/sb: clobbers gl_Position -> gl_FragCoord

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104143 --- Comment #3 from Gert Wollny --- Created attachment 136001 --> https://bugs.freedesktop.org/attachment.cgi?id=136001&action=edit Version of the piglit that uses interleaved array and fails with sb -- You are receiving this mail because: Y

[Bug 104143] r600/sb: clobbers gl_Position -> gl_FragCoord

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104143 --- Comment #6 from Gert Wollny --- Created attachment 136004 --> https://bugs.freedesktop.org/attachment.cgi?id=136004&action=edit Piglit output with extra pos parameter and sb enabled In this image on can see that the (corrected) position p

[Bug 104143] r600/sb: clobbers gl_Position -> gl_FragCoord

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104143 --- Comment #4 from Gert Wollny --- Created attachment 136002 --> https://bugs.freedesktop.org/attachment.cgi?id=136002&action=edit Version of the piglit that passes copy of gl_Position and tests it -- You are receiving this mail because: Yo

[Bug 104143] r600/sb: clobbers gl_Position -> gl_FragCoord

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104143 --- Comment #5 from Gert Wollny --- Created attachment 136003 --> https://bugs.freedesktop.org/attachment.cgi?id=136003&action=edit Piglit screen output with R600_DEBUG=nosb of shaders with extra pos parameter -- You are receiving this mail

[Bug 104143] r600/sb: clobbers gl_Position -> gl_FragCoord

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104143 --- Comment #7 from Gert Wollny --- Created attachment 136005 --> https://bugs.freedesktop.org/attachment.cgi?id=136005&action=edit Shader dump with pos test -- You are receiving this mail because: You are the assignee for the bug.__

[Bug 100200] Default Unreal Engine 4 frag shader fails to compile

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100200 --- Comment #1 from Gert Wollny --- This shader uses many arrays, and hence, reserves more registers that the r600 hardware supports. A work-in-progress patch series that fixes this can be found at https://github.com/gerddie/mesa/tree/merge-ar

[RESEND PATCH 1/4] dt-bindings: display: amlogic, meson-vpu: Add optional power domain property

2017-12-06 Thread Neil Armstrong
The Video Processing Unit power domain was setup by the Vendor U-Boot, add support for an optional Power Domain phandle to setup it from the kernel. Acked-by: Rob Herring Signed-off-by: Neil Armstrong --- Documentation/devicetree/bindings/display/amlogic,meson-vpu.txt | 4 1 file changed,

[RESEND PATCH 4/4] drm/meson: Add missing VPU init

2017-12-06 Thread Neil Armstrong
The VPU init misses these configurations values. Signed-off-by: Neil Armstrong --- drivers/gpu/drm/meson/meson_drv.c | 9 + drivers/gpu/drm/meson/meson_registers.h | 4 2 files changed, 13 insertions(+) diff --git a/drivers/gpu/drm/meson/meson_drv.c b/drivers/gpu/drm/meson/m

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

2017-12-06 Thread Neil Armstrong
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 Power Domain driver has been pushed at [1] to solve the main issue.

[RESEND PATCH 2/4] dt-bindings: display: amlogic, meson-dw-hdmi: Add optional HDMI 5V regulator

2017-12-06 Thread Neil Armstrong
On reference boards and derivatives, the HDMI Logic is powered by an external 5V regulator. This regulator was set by the Vendor U-Boot, add optional support for it. Acked-by: Rob Herring Signed-off-by: Neil Armstrong --- Documentation/devicetree/bindings/display/amlogic,meson-dw-hdmi.txt | 4 +

[RESEND PATCH 3/4] drm/meson: dw_hdmi: Add support for an optional external 5V regulator

2017-12-06 Thread Neil Armstrong
On reference boards and derivatives, the HDMI Logic is powered by an external 5V regulator. This regulator was set by the Vendor U-Boot, add optional support for it. Signed-off-by: Neil Armstrong --- drivers/gpu/drm/meson/meson_dw_hdmi.c | 13 + 1 file changed, 13 insertions(+) diff

Re: [RESEND PATCH 4/4] drm/meson: Add missing VPU init

2017-12-06 Thread Chris Wilson
Quoting Neil Armstrong (2017-12-06 11:54:28) > The VPU init misses these configurations values. > > Signed-off-by: Neil Armstrong > --- > drivers/gpu/drm/meson/meson_drv.c | 9 + > drivers/gpu/drm/meson/meson_registers.h | 4 > 2 files changed, 13 insertions(+) > > diff --git

Re: [RESEND PATCH 4/4] drm/meson: Add missing VPU init

2017-12-06 Thread Neil Armstrong
On 06/12/2017 13:02, Chris Wilson wrote: > Quoting Neil Armstrong (2017-12-06 11:54:28) >> The VPU init misses these configurations values. >> >> Signed-off-by: Neil Armstrong >> --- >> drivers/gpu/drm/meson/meson_drv.c | 9 + >> drivers/gpu/drm/meson/meson_registers.h | 4 >>

Re: [RESEND PATCH 4/4] drm/meson: Add missing VPU init

2017-12-06 Thread Chris Wilson
Quoting Neil Armstrong (2017-12-06 12:03:59) > On 06/12/2017 13:02, Chris Wilson wrote: > > Quoting Neil Armstrong (2017-12-06 11:54:28) > >> The VPU init misses these configurations values. > >> > >> Signed-off-by: Neil Armstrong > >> --- > >> drivers/gpu/drm/meson/meson_drv.c | 9

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

2017-12-06 Thread Hans Verkuil
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 the > TDA998x driver. > > Signed-off-by: Russell King > --- > drivers/gpu/dr

Re: [PATCH v2 1/7] drm/i2c: tda998x: move mutex/waitqueue/timer/work init early

2017-12-06 Thread Hans Verkuil
On 12/06/17 13:35, Russell King wrote: > Move the mutex, waitqueue, timer and detect work initialisation early > in the driver's initialisation, rather than being after we've registered > the CEC device. > > Signed-off-by: Russell King Acked-by: Hans Verkuil Regards, Hans > --- > dr

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

2017-12-06 Thread Hans Verkuil
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 > --- > drivers/gpu/drm/i2c/tda998x_drv.c | 7 --- > 1 file changed, 4 insertions(+), 3 deletion

Re: [PATCH v2 3/7] drm/i2c: tda998x: fix error cleanup paths

2017-12-06 Thread Hans Verkuil
On 12/06/17 13:35, Russell King wrote: > If tda998x_get_audio_ports() fails, and we requested the interrupt, we > fail to free the interrupt before returning failure. Rework the failure > cleanup code and exit paths so that we always clean up properly after an > error, and always propagate the err

Re: [PATCH v2 4/7] drm/i2c: tda998x: always disable and clear interrupts at probe

2017-12-06 Thread Hans Verkuil
On 12/06/17 13:35, Russell King wrote: > Always disable and clear interrupts at probe time to ensure that the > TDA998x is in a sane state. This ensures that the interrupt line, > which is also the CEC clock calibration signal, is always deasserted. > > Signed-off-by: Russell King Acked-by: Han

Re: [PATCH v2 5/7] drm/i2c: tda9950: add CEC driver

2017-12-06 Thread Hans Verkuil
Hi Russell, Some small comments below: On 12/06/17 13:35, Russell King wrote: > Add a CEC driver for the TDA9950, which is a stand-alone I2C CEC device, > but is also integrated into HDMI transceivers such as the TDA9989 and > TDA19989. > > The TDA9950 contains a command processor which handles

Re: [PATCH] gpu: drm: stm: Adopt SPDX identifiers

2017-12-06 Thread Vincent ABRIOU
Benjamin, The patch is fine for me: Acked-by: Vincent Abriou Vincent On 12/06/2017 12:29 PM, Benjamin Gaignard wrote: > Add SPDX identifiers to files under stm directory > > Signed-off-by: Benjamin Gaignard > --- > drivers/gpu/drm/stm/drv.c | 3 +-- > drivers/gpu/drm/stm/dw_mi

Re: [PATCH] gpu: drm: sti: Adopt SPDX identifiers

2017-12-06 Thread Vincent ABRIOU
Benjamin, The patch is fine for me: Acked-by: Vincent Abriou Vincent On 12/06/2017 12:29 PM, Benjamin Gaignard wrote: > Add SPDX identifiers to files under sti directory > > Signed-off-by: Benjamin Gaignard > --- > drivers/gpu/drm/sti/sti_awg_utils.c| 2 +- > drivers/gpu/drm/sti/s

[Bug 104143] r600/sb: clobbers gl_Position -> gl_FragCoord

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104143 --- Comment #8 from Gert Wollny --- I found the problem: if KC0[0].x == index (=0): 1 x: ADD_INTT0.x, KC0[0].x, [0xfffe -nan].x 2 x: MOVA_INT __.x, T0.x Address register is now -2 and hence, in the ne

Re: [Intel-gfx] [PATCH v3 4/9] drm: Add some HDCP related #defines

2017-12-06 Thread Alex Deucher
On Tue, Dec 5, 2017 at 6:12 PM, Chris Wilson wrote: > Quoting Sean Paul (2017-12-05 05:15:03) >> In preparation for implementing HDCP in i915, add some HDCP related >> register offsets and defines. The dpcd register offsets will go in >> drm_dp_helper.h whereas the ddc offsets along with generic H

Re: [PATCH] video: hd44780: Add hd44780 lcd display driver

2017-12-06 Thread Geert Uytterhoeven
Hi Lars, On Wed, Dec 6, 2017 at 2:52 PM, Lars Poeschel wrote: > This adds a console driver for hd44780 based character lcd displays and > clones. The driver currently supports 20x4 character displays with > character ROMs A00 and A02. > The hardware wirings to the display have to be supplied to t

Re: [PATCH] gpu: drm: stm: Adopt SPDX identifiers

2017-12-06 Thread Benjamin Gaignard
2017-12-06 15:26 GMT+01:00 Vincent ABRIOU : > Benjamin, > > The patch is fine for me: > > Acked-by: Vincent Abriou > > Vincent Thanks, Pushed in drm-misc-next Benjamin > > On 12/06/2017 12:29 PM, Benjamin Gaignard wrote: >> Add SPDX identifiers to files under stm directory >> >> Signed-off-by:

Re: [PATCH] gpu: drm: sti: Adopt SPDX identifiers

2017-12-06 Thread Benjamin Gaignard
2017-12-06 15:26 GMT+01:00 Vincent ABRIOU : > Benjamin, > > The patch is fine for me: > > Acked-by: Vincent Abriou > > Vincent Thanks, Pushed in drm-misc-next Benjamin > > On 12/06/2017 12:29 PM, Benjamin Gaignard wrote: >> Add SPDX identifiers to files under sti directory >> >> Signed-off-by:

[Bug 104090] Reduced colors on RX580 through eDP on Asus GL702ZC laptop

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104090 --- Comment #12 from Hein-Pieter van Braam --- How do I establish whether that is what is happening or not? It sure looks like that's what's going on though. -- You are receiving this mail because: You are the assignee for the bug.

Re: [PATCH] drm/ttm: swap consecutive allocated pooled pages v4

2017-12-06 Thread kbuild test robot
Hi Christian, I love your patch! Yet something to improve: [auto build test ERROR on drm/drm-next] [also build test ERROR on v4.15-rc2 next-20171206] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux

Re: [PATCH v3 3/9] drm: Add Content Protection property

2017-12-06 Thread Sean Paul
On Tue, Dec 5, 2017 at 12:11 PM, C, Ramalingam wrote: > > > > Best Regards, > Ramalingam C > >> -Original Message- >> From: Sean Paul [mailto:seanp...@chromium.org] >> Sent: Tuesday, December 5, 2017 8:07 PM >> To: C, Ramalingam >> Cc: dri-devel ; Hans Verkuil >> >> Subject: Re: [PATCH v

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

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=95306 --- Comment #70 from Andrew --- this bug/thread looks very simillar to what i ahve been expiriencing once i tried to move from fglx + fedora 22 to over amdgpu + fedora27. Please take a look@ the bug i filed : https://bugzilla.redhat.com/show_bug.

[PATCH] drm/fb-helper: Fix a potential NULL dereference

2017-12-06 Thread Dan Carpenter
We recently modified drm_fb_helper_single_add_all_connectors() to allow NULL "fb_helper" pointers. But the problem is that it gets dereferenced before we checked for NULL. Fixes: c777990fb45b ("drm/fb-helper: Handle function NULL argument") Signed-off-by: Dan Carpenter diff --git a/drivers/gpu

[PATCH v2 2/2] drm/sched: move fence slab handling to module init/exit

2017-12-06 Thread Lucas Stach
This is the only part of the scheduler which must not be called from different drivers. Move it to module init/exit so it is done a single time when loading the scheduler. Signed-off-by: Lucas Stach --- drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 8 drivers/gpu/drm/scheduler/sched_fence.

[Bug 104143] r600/sb: clobbers gl_Position -> gl_FragCoord

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104143 --- Comment #9 from Gert Wollny --- Patch: https://patchwork.freedesktop.org/patch/192036/ -- You are receiving this mail because: You are the assignee for the bug.___ dri-devel mailing list dri-deve

[PATCH v2 1/2] drm: move amd_gpu_scheduler into common location

2017-12-06 Thread Lucas Stach
This moves and renames the AMDGPU scheduler to a common location in DRM in order to facilitate re-use by other drivers. This is mostly a straight forward rename with no code changes. One notable exception is the function to_drm_sched_fence(), which is no longer a inline header function to avoid th

[PATCH v2 0/2] AMDGPU scheduler move, take 2

2017-12-06 Thread Lucas Stach
Hi all, second try to move the AMDGPU scheduler into a common location, this time rebased onto drm-next-4.16-wip. I've tested my etnaviv series on top of this and things seem to work fine. I checked that AMDGPU still builds, but I don't have any means to actually runtime test this currently, so I

[pull] amdgpu and ttm drm-fixes-4.15

2017-12-06 Thread Alex Deucher
Hi Dave, Fixes for 4.15: - Add licenses to files that were missing it - huge page fixes for ttm The following changes since commit 503505bfea19b7d69e2572297e6defa0f9c2404e: Merge branch 'drm-fixes-4.15' of git://people.freedesktop.org/~agd5f/linux into drm-fixes (2017-12-01 09:15:57 +1000) a

[PATCH 4/4] drm/etnaviv: replace hangcheck with scheduler timeout

2017-12-06 Thread Lucas Stach
This replaces the etnaviv internal hangcheck logic with the job timeout handling provided by the DRM scheduler. This simplifies the driver further and allows to replay jobs after a GPU reset, so only minimal state is lost. This introduces a user-visible change in that we don't allow jobs to run in

[PATCH 2/4] drm/etnaviv: move dependency handling to scheduler

2017-12-06 Thread Lucas Stach
Move the fence dependency handling to the scheduler where it belongs. Jobs with unsignaled dependencies just get to sit in the scheduler queue without holding any locks. Signed-off-by: Lucas Stach --- drivers/gpu/drm/etnaviv/etnaviv_gem.h| 3 ++ drivers/gpu/drm/etnaviv/etnaviv_gem_submi

[PATCH 0/4] Etnaviv GPU scheduler

2017-12-06 Thread Lucas Stach
Hi all, this hooks up the AMDGPU/DRM GPU scheduler in etnaviv. This depends on both the etnaviv job lifetime fixes series, as well as the AMDGPU scheduler move. Please keep this in mind while reviewing. The scheduler has 4 main benefits to etnaviv: 1. Cross GPU/device synchronization is handled i

[PATCH 1/4] drm/etnaviv: hook up DRM GPU scheduler

2017-12-06 Thread Lucas Stach
This hooks in the DRM GPU scheduler. No improvement yet, as all the dependency handling is still done in etnaviv_gem_submit. This just replaces the actual GPU submit by passing through the scheduler. Allows to get rid of the retire worker, as this is now driven by the scheduler. Signed-off-by: Lu

[PATCH 3/4] drm/etnaviv: lock BOs after all other submit work is done

2017-12-06 Thread Lucas Stach
Populating objects, adding them to the GPU VM and patching/validating the command stream might take a lot of CPU time. There is no reason to hold all object reservations during that time. Signed-off-by: Lucas Stach --- drivers/gpu/drm/etnaviv/etnaviv_gem_submit.c | 16 1 file ch

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

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=95306 --- Comment #71 from Andrew --- adding amdgpu.cg_mask=0xFFFB semed to keep boot messages for a bit longer, but then again - black screen of beauty :( adding powerplay) made no difference -- You are receiving this mail because: You are the

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

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=95306 --- Comment #72 from Andrew --- Created attachment 136007 --> https://bugs.freedesktop.org/attachment.cgi?id=136007&action=edit log files with amdgpu.cg_mask set -- You are receiving this mail because: You are the assignee for the bug.___

Re: [PATCH 1/2] drm/nouveau/bar/gf100: fix hang when calling ->fini() before ->init()

2017-12-06 Thread Jon Hunter
On 06/12/17 09:22, Guillaume Tucker wrote: > On 05/12/17 18:32, Ben Skeggs wrote: >> On Wed, Dec 6, 2017 at 12:30 AM, Jon Hunter wrote: >> >>> >>> On 04/12/17 18:37, Guillaume Tucker wrote: If the firmware fails to load then ->fini() will be called before the device has been initialised

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

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=95306 --- Comment #73 from Andrew --- i also noticed that the grub screen is in high resolution (probably 1024 ) without me specifying any params. Also the boot messages are flying in the high resolution, until something is loaded and then all goes dar

[PATCH v2 0/7] TDA998x CEC support

2017-12-06 Thread Russell King - ARM Linux
Hi, This patch series adds CEC support to the DRM TDA998x driver. The TDA998x family of devices integrate a TDA9950 CEC at a separate I2C address from the HDMI encoder. Implementation of the CEC part is separate to allow independent CEC implementations, or independent HDMI implementations (since

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

2017-12-06 Thread Russell King
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 --- drivers/gpu/drm/i2c/tda998x_drv.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c

[PATCH v2 5/7] drm/i2c: tda9950: add CEC driver

2017-12-06 Thread Russell King
Add a CEC driver for the TDA9950, which is a stand-alone I2C CEC device, but is also integrated into HDMI transceivers such as the TDA9989 and TDA19989. The TDA9950 contains a command processor which handles retransmissions and the low level bus protocol. The driver just has to read and write the

[PATCH v2 1/7] drm/i2c: tda998x: move mutex/waitqueue/timer/work init early

2017-12-06 Thread Russell King
Move the mutex, waitqueue, timer and detect work initialisation early in the driver's initialisation, rather than being after we've registered the CEC device. Signed-off-by: Russell King --- drivers/gpu/drm/i2c/tda998x_drv.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff

[PATCH] drm/sun4i: Fix uninitialized variables in vi layer

2017-12-06 Thread Jernej Skrabec
min_scale and max_scale in sun8i_vi_layer_atomic_check() can be used without initialization. Fix that. Fixes: b862a648de3b (drm/sun4i: Add support for HW scaling to DE2) Signed-off-by: Jernej Skrabec --- drivers/gpu/drm/sun4i/sun8i_vi_layer.c | 3 +++ 1 file changed, 3 insertions(+) diff --g

[PATCH] video: hd44780: Add hd44780 lcd display driver

2017-12-06 Thread Lars Poeschel
This adds a console driver for hd44780 based character lcd displays and clones. The driver currently supports 20x4 character displays with character ROMs A00 and A02. The hardware wirings to the display have to be supplied to the kernel in the devicetree. The binding doc has the necessary informati

[PATCH v2 4/7] drm/i2c: tda998x: always disable and clear interrupts at probe

2017-12-06 Thread Russell King
Always disable and clear interrupts at probe time to ensure that the TDA998x is in a sane state. This ensures that the interrupt line, which is also the CEC clock calibration signal, is always deasserted. Signed-off-by: Russell King --- drivers/gpu/drm/i2c/tda998x_drv.c | 14 +- 1 f

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

2017-12-06 Thread Russell King
The TDA998x is a HDMI transmitter with a TDA9950 CEC engine integrated onto the same die. Add support for the TDA9950 CEC engine to the TDA998x driver. Signed-off-by: Russell King --- drivers/gpu/drm/i2c/Kconfig | 1 + drivers/gpu/drm/i2c/tda998x_drv.c | 209

[PATCH v2 3/7] drm/i2c: tda998x: fix error cleanup paths

2017-12-06 Thread Russell King
If tda998x_get_audio_ports() fails, and we requested the interrupt, we fail to free the interrupt before returning failure. Rework the failure cleanup code and exit paths so that we always clean up properly after an error, and always propagate the error code. Signed-off-by: Russell King --- dri

[PATCH v2 7/7] dt-bindings: tda998x: add the calibration gpio

2017-12-06 Thread Russell King
Add the optional calibration gpio for integrated TDA9950 CEC support. This GPIO corresponds with the interrupt from the TDA998x, as the calibration requires driving the interrupt pin low. Signed-off-by: Russell King --- Documentation/devicetree/bindings/display/bridge/tda998x.txt | 3 +++ 1 file

[Bug 103791] Tearing after screen wakeup/on

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103791 --- Comment #17 from Michel Dänzer --- Thanks. I think I see what's happening, but I need some time to think about how to address it. Meanwhile, you should be able to re-enable TearFree by forcing a modeset, e.g. by re-enabling the TearFree pro

Re: [PATCH v3 09/15] drm/sun4i: Add A83T support

2017-12-06 Thread Jernej Škrabec
Hi, Dne torek, 05. december 2017 ob 16:42:55 CET je Jernej Škrabec napisal(a): > Hi Maxime, > > Dne torek, 05. december 2017 ob 16:10:21 CET je Maxime Ripard napisal(a): > > Add support for the A83T display pipeline. > > > > Reviewed-by: Chen-Yu Tsai > > Signed-off-by: Maxime Ripard > > --- >

[Bug 103769] Unity based games do not start

2017-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103769 --- Comment #5 from letha...@gmail.com --- other unity games affected: candle silence grow home pulse mousecraft wasteland 2 broforce habitat munin the last tinker agatha christie the abc murders plague inc evolved pillars of eternity monochroma

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

2017-12-06 Thread Noralf Trønnes
Den 29.11.2017 04.01, skrev David Lechner: 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 --- MAINTAINERS | 6 + drivers/gpu/drm/tinydrm/Kconfig | 10 ++ drivers/gpu/drm/t

  1   2   >