[PATCH] drm/exynos/hdmi: fix pipeline disable order

2017-04-05 Thread Andrzej Hajda
Hardware require that MIXER(crtc) should be disabled prior to HDMI(encoder). It was achieved by disabling crtc from encoder disable callback, bypassing drm core. As a result drm core tried to call vblank related routines on disabled crtc. The patch fixes it by simplifying hdmi_disable routine - now

Re: linux-next: build warning after merge of the drm-misc tree

2017-04-05 Thread Neil Armstrong
On 04/05/2017 02:41 AM, Stephen Rothwell wrote: > Hi all, > > After merging the drm-misc tree, today's linux-next build > (arm_multi_v7_defconfig) produced this warning: > > drivers/gpu/drm/bridge/synopsys/dw-hdmi.c:608:13: warning: > 'hdmi_bus_fmt_is_yuv420' defined but not used [-Wunused-funct

[PATCH 8/9] drm/exynos/decon5433: kill BIT_SUSPENDED flag

2017-04-05 Thread Andrzej Hajda
Exynos tracked suspend state to prevent touching disabled HW. After fixing disable order in HDMI and moving TE handling to DECON it is not needed anymore - all IRQ handlers and callbacks touching HW are called only with enabled DECON. Signed-off-by: Andrzej Hajda --- drivers/gpu/drm/exynos/exyno

[PATCH 2/9] dt-bindings: exynos5433-decon: fix interrupts bindings

2017-04-05 Thread Andrzej Hajda
DECON requires different interrupts depending on mode of work, which depends on panel it is connected to. Signed-off-by: Andrzej Hajda --- .../devicetree/bindings/display/exynos/exynos5433-decon.txt | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/Documentation/

[PATCH 1/9] drm/exynos/decon5433: always do sw-trigger when vblanks enabled

2017-04-05 Thread Andrzej Hajda
When vblanks are enabled userspace and/or kernel can expect vblank interrupt at declared period of time. To generate vblank interrupt image transfer must be triggered. This patch fixes vblank timeouts in case of sw-trigger mode. Signed-off-by: Andrzej Hajda --- drivers/gpu/drm/exynos/exynos5433_

[PATCH 9/9] drm/exynos/decon5433: remove useless check

2017-04-05 Thread Andrzej Hajda
TE IRQ is enabled only in case of sw-trigger, so trigger check is redundant. Signed-off-by: Andrzej Hajda --- drivers/gpu/drm/exynos/exynos5433_drm_decon.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/gpu/drm/exynos/exynos5433_drm_decon.c b/drivers/gpu/drm/exynos/exynos5433_drm

[PATCH 3/9] dt-bindings: exynos5433-decon: add TE interrupt binding

2017-04-05 Thread Andrzej Hajda
DECON command mode can use hardware trigger where transmission is triggered automatically, or software trigger - where TE interrupt handler should trigger transmission. DECON will use software trigger if TE interrupt is specified. Signed-off-by: Andrzej Hajda --- Documentation/devicetree/binding

[PATCH 7/9] drm/exynos/decon5433: kill BIT_WIN_UPDATED flag

2017-04-05 Thread Andrzej Hajda
The flag was used to trigger software update in TE IRQ handler only if framebuffers were replaced. Since TE update is triggered always when VBLANKs are enabled and after framebuffer replacement VBLANKs are always enabled the flag becomes redundant. Signed-off-by: Andrzej Hajda --- drivers/gpu/dr

[PATCH 6/9] drm/exynos/decon5433: kill BIT_CLKS_ENABLED flag

2017-04-05 Thread Andrzej Hajda
The flag was used to check if IRQ handlers can touch HW. Since driver enables IRQs only if hardware is enabled the flag becomes redundant. Signed-off-by: Andrzej Hajda --- drivers/gpu/drm/exynos/exynos5433_drm_decon.c | 12 +--- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git

[PATCH 4/9] drm/exynos/decon5433: move TE handling to DECON

2017-04-05 Thread Andrzej Hajda
DECON is the only user of TE signal, moving all TE related code to DECON driver allows to precise control of IRQ handlers. This control allows to fix race between IRQ handler and DECON disable code - now it is possible to disable DECON during IRQ handling which can result in kernel crash. Beside ra

[PATCH 5/9] drm/exynos/decon5433: kill BIT_IRQS_ENABLED flag

2017-04-05 Thread Andrzej Hajda
Since DECON uses enable_irq/disable_irq to full control IRQs, there is no point in having flags to trace it separately. As a bonus condition for software trigger becomes always true, so it can be removed. Signed-off-by: Andrzej Hajda --- drivers/gpu/drm/exynos/exynos5433_drm_decon.c | 8 +---

[PATCH 0/9] drm/exynos/decon5433: move TE handling to DECON

2017-04-05 Thread Andrzej Hajda
Hi Inki, This patchset contains fixes and improvements related to SW-TRIGGER. The core patch moves all TE related stuff from panel(bindings) and HDMI(irq handler) to DECON. This move allows to fix races between interrupt handlers and DECON disable. It allows also to eliminate all DECON flags and r

[PATCH] drm: bridge: dw-hdmi: remove unused hdmi_bus_fmt_is_yuv420

2017-04-05 Thread Neil Armstrong
Remove usused yet hdmi_bus_fmt_is_yuv420 function. Fixes: def23aa7e982 ("drm: bridge: dw-hdmi: Switch to V4L bus format and encodings") Signed-off-by: Neil Armstrong --- drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 14 -- 1 file changed, 14 deletions(-) diff --git a/drivers/gpu/drm/

Re: [PATCH] drm: panels: Add MAINTAINERS entry for LVS panel driver

2017-04-05 Thread Jani Nikula
On Wed, 05 Apr 2017, Laurent Pinchart wrote: > Hi Dave, > > On Wednesday 05 Apr 2017 09:51:27 Laurent Pinchart wrote: >> As the DRM LVDS panel driver uses a different approach to DT bindings >> compared to what Thierry Reding advocates, add a specific MAINTAINERS >> entry to avoid bothering Thierr

Re: [PATCH] drm: rcar-du: Document the vsps property in the DT bindings

2017-04-05 Thread Geert Uytterhoeven
Hi Laurent, On Fri, Mar 31, 2017 at 11:21 AM, Geert Uytterhoeven wrote: > On Fri, Mar 31, 2017 at 11:19 AM, Laurent Pinchart > wrote: >> On Monday 27 Mar 2017 13:05:48 Geert Uytterhoeven wrote: >>> On Mon, Mar 27, 2017 at 11:56 AM, Laurent Pinchart wrote: >>> > The property is used by the driver

Re: [linux-sunxi] Re: [PATCH v3 04/11] drm/sun4i: abstract the layer type

2017-04-05 Thread Maxime Ripard
On Wed, Apr 05, 2017 at 01:23:15PM +0800, Icenowy Zheng wrote: > > 2017年4月5日 10:27于 Chen-Yu Tsai 写道: > > > > On Wed, Apr 5, 2017 at 3:53 AM, Icenowy Zheng wrote: > > > > > > > > > 在 2017年04月05日 03:28, Sean Paul 写道: > > >> > > >> On Thu, Mar 30, 2017 at 03:46:06AM +0800, Icenowy Zheng wrote:

[PATCH] drm/atomic: Acquire connection_mutex lock in drm_helper_probe_single_connector_modes, v3.

2017-04-05 Thread Maarten Lankhorst
mode_valid() called from drm_helper_probe_single_connector_modes() may need to look at connector->state because what a valid mode is may depend on connector properties being set. For example some HDMI modes might be rejected when a connector property forces the connector into DVI mode. Some implem

Re: [PATCH] drm: bridge: dw-hdmi: remove unused hdmi_bus_fmt_is_yuv420

2017-04-05 Thread Laurent Pinchart
Hi Neil, Thank you for the patch. On Wednesday 05 Apr 2017 09:32:59 Neil Armstrong wrote: > Remove usused yet hdmi_bus_fmt_is_yuv420 function. > > Fixes: def23aa7e982 ("drm: bridge: dw-hdmi: Switch to V4L bus format and > encodings") Signed-off-by: Neil Armstrong Acked-by: Laurent Pinchart >

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-05 Thread Lucas Stach
Am Mittwoch, den 05.04.2017, 00:20 + schrieb Christopher James Halse Rogers: > > > On Tue, Apr 4, 2017 at 9:53 PM Daniel Vetter wrote: > > On Tue, Apr 4, 2017 at 12:43 PM, Lucas Stach > wrote: > >> If I could guarantee that I'd only ever run on > 4.13-or-lat

Re: [PATCH] drm: rcar-du: Document the vsps property in the DT bindings

2017-04-05 Thread Laurent Pinchart
Hi Geert, On Wednesday 05 Apr 2017 09:54:38 Geert Uytterhoeven wrote: > On Fri, Mar 31, 2017 at 11:21 AM, Geert Uytterhoeven wrote: > > On Fri, Mar 31, 2017 at 11:19 AM, Laurent Pinchart wrote: > >> On Monday 27 Mar 2017 13:05:48 Geert Uytterhoeven wrote: > >>> On Mon, Mar 27, 2017 at 11:56 AM, La

Re: [PATCH v2 0/5] Thunderbolt GPU fixes

2017-04-05 Thread Lukas Wunner
On Fri, Mar 31, 2017 at 06:30:48AM -0700, Bjorn Helgaas wrote: > On Fri, Mar 31, 2017 at 3:11 AM, Lukas Wunner wrote: > > Bjorn, I haven't heard back after my reply to your e-mail of March 10 > > (regarding the "usually soldered to the mainboard" comment). I hope > > my reply was satisfactory. If

[PATCH] drm/atomic: Add connector atomic_check function.

2017-04-05 Thread Maarten Lankhorst
The connector atomic check function may be called multiple times, or not at all. It's mostly useful for implementing properties but if you call check_modeset twice it can be used for other modeset related checks as well. Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/drm_atomic_helper.c

Re: [PATCH v2 09/13] arm64: dts: r8a7795: salvator-x: Add panel backlight support

2017-04-05 Thread Laurent Pinchart
Hi Geert, On Monday 21 Nov 2016 11:59:47 Laurent Pinchart wrote: > On Monday 21 Nov 2016 10:23:46 Geert Uytterhoeven wrote: > > On Mon, Nov 21, 2016 at 10:19 AM, Laurent Pinchart wrote: > >> On Monday 21 Nov 2016 09:36:22 Geert Uytterhoeven wrote: > >>> On Sat, Nov 19, 2016 at 4:28 AM, Laurent Pin

Re: [PATCH] drm: don't link DP aux i2c adapter to the hardware device node

2017-04-05 Thread Lucas Stach
Hi Rob, Am Mittwoch, den 29.03.2017, 08:56 -0500 schrieb Rob Herring: > On Mon, Jan 23, 2017 at 10:33 AM, Thierry Reding > wrote: > > On Fri, Jan 13, 2017 at 06:36:30PM +0100, Lucas Stach wrote: > >> The i2c adapter on DP AUX is purely a software construct. Linking > >> it to the device node of t

Re: [PATCH v2 09/13] arm64: dts: r8a7795: salvator-x: Add panel backlight support

2017-04-05 Thread Geert Uytterhoeven
Hi Laurent, On Wed, Apr 5, 2017 at 10:45 AM, Laurent Pinchart wrote: > On Monday 21 Nov 2016 11:59:47 Laurent Pinchart wrote: >> On Monday 21 Nov 2016 10:23:46 Geert Uytterhoeven wrote: >> > On Mon, Nov 21, 2016 at 10:19 AM, Laurent Pinchart wrote: >> >> On Monday 21 Nov 2016 09:36:22 Geert Uytte

Re: [PATCH] drm/atomic: Acquire connection_mutex lock in drm_helper_probe_single_connector_modes, v3.

2017-04-05 Thread Daniel Vetter
On Wed, Apr 05, 2017 at 10:11:18AM +0200, Maarten Lankhorst wrote: > mode_valid() called from drm_helper_probe_single_connector_modes() > may need to look at connector->state because what a valid mode is may > depend on connector properties being set. For example some HDMI modes > might be rejected

Re: [PATCH libdrm 2/2] Add CRTC ID to vblank event

2017-04-05 Thread Pekka Paalanen
On Tue, 4 Apr 2017 17:52:20 +0100 Daniel Stone wrote: > From: Ander Conselvan de Oliveira > > When using the atomic API, one request can span multiple CRTCs, however > one event is generated per CRTC. As we cannot disambiguate the CRTC with > user data (since we only have one piece of user dat

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-05 Thread Daniel Vetter
On Wed, Apr 05, 2017 at 10:15:44AM +0200, Lucas Stach wrote: > Am Mittwoch, den 05.04.2017, 00:20 + schrieb Christopher James Halse > Rogers: > > > > > > On Tue, Apr 4, 2017 at 9:53 PM Daniel Vetter wrote: > > > > On Tue, Apr 4, 2017 at 12:43 PM, Lucas Stach > > wrote: > >

Re: imx-drm: vblank wait timed out

2017-04-05 Thread Philipp Zabel
On Tue, 2017-04-04 at 14:50 +0200, Christian Gmeiner wrote: [...] > > Is this on a non-plus i.MX6? Maybe are missing the LDB DI clock glitch > > fixes (commits 5d283b083800, 03d576f202e8, and f13abeff2cde)? > > Yes it is a non-plus i.MX6D and all those three patches are included in > 4.9.20 (which

Re: [PATCH] drm/atomic: Add connector atomic_check function.

2017-04-05 Thread Daniel Vetter
On Wed, Apr 05, 2017 at 10:41:24AM +0200, Maarten Lankhorst wrote: > The connector atomic check function may be called multiple times, > or not at all. It's mostly useful for implementing properties but if you > call check_modeset twice it can be used for other modeset related checks > as well. >

[GIT PULL] imx-drm: module/dependency changes

2017-04-05 Thread Philipp Zabel
Hi Dave, this tag contains a patch to combine the imxdrm and imx-ipuv3-crtc modules into a single imxdrm module, as after commits 44b460cfe554 ("drm: imx: remove struct imx_drm_crtc and imx_drm_crtc_helper_funcs") and especially eb8c88808c83 ("drm/imx: add deferred plane disabling") there is no cl

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-05 Thread Lucas Stach
Am Mittwoch, den 05.04.2017, 11:59 +0200 schrieb Daniel Vetter: > On Wed, Apr 05, 2017 at 10:15:44AM +0200, Lucas Stach wrote: > > Am Mittwoch, den 05.04.2017, 00:20 + schrieb Christopher James Halse > > Rogers: > > > > > > > > > On Tue, Apr 4, 2017 at 9:53 PM Daniel Vetter wrote: > > > > >

Re: [PATCH] drm/atomic: Add connector atomic_check function.

2017-04-05 Thread Maarten Lankhorst
Op 05-04-17 om 12:06 schreef Daniel Vetter: > On Wed, Apr 05, 2017 at 10:41:24AM +0200, Maarten Lankhorst wrote: >> The connector atomic check function may be called multiple times, >> or not at all. It's mostly useful for implementing properties but if you >> call check_modeset twice it can be use

[GIT PULL] etnaviv-next for 4.12

2017-04-05 Thread Lucas Stach
Hi Dave, etnaviv feature pull for 4.12. Highlights: - Cooling device support from Russell, to allow GPU throttling on system thermal overload. - Explicit fencing support from Philipp, implemented in a similar way to drm/msm. Regards, Lucas The following changes since commit 65d1086c44791112188f

[Bug 100577] DC + TearFree display lock

2017-04-05 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100577 Bug ID: 100577 Summary: DC + TearFree display lock Product: DRI Version: DRI git Hardware: Other OS: All Status: NEW Severity: normal Priority

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-05 Thread Christopher James Halse Rogers
On Wed, Apr 5, 2017 at 8:14 PM Lucas Stach wrote: > Am Mittwoch, den 05.04.2017, 11:59 +0200 schrieb Daniel Vetter: > > On Wed, Apr 05, 2017 at 10:15:44AM +0200, Lucas Stach wrote: > > > Am Mittwoch, den 05.04.2017, 00:20 + schrieb Christopher James > Halse > > > Rogers: > > > > > > > > > > >

[PATCH maintainer-tools 2/3] dim: send 'not configured' error to stderr

2017-04-05 Thread Eric Engestrom
This fixes `make check` when dim is not configured. Signed-off-by: Eric Engestrom --- dim | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dim b/dim index 588e859..b373901 100755 --- a/dim +++ b/dim @@ -197,7 +197,7 @@ export __dim_running=1 if [ "$subcommand" != "setup" ] &&

[PATCH maintainer-tools 1/3] make: run check on current `dim`, not the installed one

2017-04-05 Thread Eric Engestrom
Signed-off-by: Eric Engestrom --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index b4cea98..4291049 100644 --- a/Makefile +++ b/Makefile @@ -33,7 +33,7 @@ shellcheck: shellcheck $(SC_EXCLUDE) dim bash_completion mancheck: - @f

[PATCH maintainer-tools 3/3] dim.rst: update CONTRIBUTING instructions

2017-04-05 Thread Eric Engestrom
Suggest using git-config instead of a flag on format-patch. While at it, use the more common "PATCH foo" subject prefix. Signed-off-by: Eric Engestrom --- dim.rst | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/dim.rst b/dim.rst index b99248e..c650317 100644 --- a/dim.rs

Re: [PATCH 08/15] drm/i915: Nuke intel_atomic_legacy_gamma_set

2017-04-05 Thread Daniel Vetter
On Tue, Apr 04, 2017 at 12:44:26PM +0200, Maarten Lankhorst wrote: > Op 03-04-17 om 10:32 schreef Daniel Vetter: > > We do set DRIVER_ATOMIC now. > > > > Signed-off-by: Daniel Vetter > > --- > > drivers/gpu/drm/i915/intel_display.c | 44 > > +--- > > 1 file change

[PATCH] MAINTAINERS: Add Laurent and Andrzej as maintainers for bridge drivers

2017-04-05 Thread Archit Taneja
Add Laurent and Andrzej as maintainers for DRM bridge chip drivers. They actively review and contribute to bridge drivers and the bridge API. Cc: Laurent Pinchart Cc: Andrzej Hajda Signed-off-by: Archit Taneja --- MAINTAINERS | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MAINTAINERS b/

Re: [PATCH maintainer-tools 3/3] dim.rst: update CONTRIBUTING instructions

2017-04-05 Thread Jani Nikula
On Wed, 05 Apr 2017, Eric Engestrom wrote: > Suggest using git-config instead of a flag on format-patch. > While at it, use the more common "PATCH foo" subject prefix. More common? That's all so subjective. And I've mostly used [dim PATCH]... BR, Jani. > > Signed-off-by: Eric Engestrom > ---

Re: [PATCH maintainer-tools 1/3] make: run check on current `dim`, not the installed one

2017-04-05 Thread Jani Nikula
On Wed, 05 Apr 2017, Eric Engestrom wrote: > Signed-off-by: Eric Engestrom For me "current dim" *is* the "installed one". ;) Pushed patches 1-2. BR, Jani. > --- > Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Makefile b/Makefile > index b4cea98..4291049 10

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-05 Thread Christian König
Am 05.04.2017 um 13:13 schrieb Christopher James Halse Rogers: On Wed, Apr 5, 2017 at 8:14 PM Lucas Stach > wrote: Am Mittwoch, den 05.04.2017, 11:59 +0200 schrieb Daniel Vetter: > On Wed, Apr 05, 2017 at 10:15:44AM +0200, Lucas Stach wrote: > > Am Mitt

Re: [PATCH] drm: don't link DP aux i2c adapter to the hardware device node

2017-04-05 Thread Thierry Reding
On Wed, Apr 05, 2017 at 10:52:32AM +0200, Lucas Stach wrote: > Hi Rob, > > Am Mittwoch, den 29.03.2017, 08:56 -0500 schrieb Rob Herring: > > On Mon, Jan 23, 2017 at 10:33 AM, Thierry Reding > > wrote: > > > On Fri, Jan 13, 2017 at 06:36:30PM +0100, Lucas Stach wrote: > > >> The i2c adapter on DP

Re: [PATCH maintainer-tools 3/3] dim.rst: update CONTRIBUTING instructions

2017-04-05 Thread Daniel Vetter
On Wed, Apr 05, 2017 at 12:15:25PM +0100, Eric Engestrom wrote: > Suggest using git-config instead of a flag on format-patch. > While at it, use the more common "PATCH foo" subject prefix. > > Signed-off-by: Eric Engestrom > --- > dim.rst | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions

Re: DVI output on i.MX51 EVP board not working?

2017-04-05 Thread Fabio Estevam
On Sat, Apr 1, 2017 at 6:40 PM, Fabio Estevam wrote: > Hi Wladimir, > > On Fri, Mar 31, 2017 at 2:36 AM, Wladimir J. van der Laan > wrote: > >> - Went as far back as kernel v4.0, even to v3.12 or so (commit 493a863, "ARM: >> dts: imx51-babbage: Make DVI and WVGA panel functional"). No differenc

Re: DVI output on i.MX51 EVP board not working?

2017-04-05 Thread Wladimir J. van der Laan
On Wed, Apr 05, 2017 at 09:50:23AM -0300, Fabio Estevam wrote: > On Sat, Apr 1, 2017 at 6:40 PM, Fabio Estevam wrote: > > Hi Wladimir, > > > > On Fri, Mar 31, 2017 at 2:36 AM, Wladimir J. van der Laan > > wrote: > > > >> - Went as far back as kernel v4.0, even to v3.12 or so (commit 493a863, > >

Re: DVI output on i.MX51 EVP board not working?

2017-04-05 Thread Fabio Estevam
On Wed, Apr 5, 2017 at 10:05 AM, Wladimir J. van der Laan wrote: > Ok, thanks for testing. > > To be clear: was this result with v3.x, or with mainline? I tested with 4.11-rc5 and the kernel is driving the DVI monitor correctly. ___ dri-devel mailing l

Re: [RFC, v2, 10/11] vb2: dma-contig: Let drivers decide DMA attrs of MMAP and USERPTR bufs

2017-04-05 Thread Sakari Ailus
Hi Ricky, On Mon, Dec 26, 2016 at 03:58:07PM +0800, Ricky Liang wrote: > Hi Laurent, > > On Fri, Dec 16, 2016 at 9:24 AM, Laurent Pinchart > wrote: > > From: Sakari Ailus > > > > The desirable DMA attributes are not generic for all devices using > > Videobuf2 contiguous DMA ops. Let the drivers

Re: [PATCH 09/15] drm/msm: Nerf zpos property

2017-04-05 Thread Daniel Vetter
On Mon, Apr 03, 2017 at 10:32:58AM +0200, Daniel Vetter wrote: > It's not wired up, and if it is, it should be moved over to the new > fancy standardized zpos property exposed through > drm_plane_create_zpos_property(). > > Cc: Rob Clark > Signed-off-by: Daniel Vetter Discussed this with Rob Cl

Re: [PATCH 0/4] Fix DP busy wait and defer disabling overlay plane

2017-04-05 Thread Philipp Zabel
On Tue, 2017-04-04 at 19:25 +0100, Dan MacDonald wrote: > It took just under 11 hours to build the Arch 4.11 armv7h kernel on my > SL. The good news is that the patch both applied and built OK this > time, and the kernel package installed fine. What didn't seem right is > that boot pauses for a min

[Bug 100563] Compile fails with radeonsi / LLVM (git)

2017-04-05 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100563 --- Comment #1 from Nicolai Hähnle --- This has nothing to do with LLVM. It looks like your sid_tables.h is stale -- please remove it (it's auto-generated) and try again. -- You are receiving this mail because: You are the assignee for the bug

[Bug 100563] Compile fails with radeonsi / LLVM (git)

2017-04-05 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100563 --- Comment #2 from Nicolai Hähnle --- To add a bit more explanation: sid_tables.h is usually updated automatically; but it was moved some time ago, and it's possible that you have an old copy in the old location. -- You are receiving this mai

Re: [PATCH 4/4] drm: zte: add VGA driver support

2017-04-05 Thread Shawn Guo
On Tue, Apr 04, 2017 at 01:46:48PM -0400, Sean Paul wrote: > > +static int zx_vga_register(struct drm_device *drm, struct zx_vga *vga) > > +{ > > + struct drm_encoder *encoder = &vga->encoder; > > + struct drm_connector *connector = &vga->connector; > > + > > + encoder->possible_crtcs = VOU_C

[PATCH] exynos: add C++ support to exynos_drmif header

2017-04-05 Thread Tobias Jakobi
Add the usual extern "C" when compiling in C++ mode. Signed-off-by: Tobias Jakobi --- exynos/exynos_drmif.h | 8 1 file changed, 8 insertions(+) diff --git a/exynos/exynos_drmif.h b/exynos/exynos_drmif.h index 626e399..154439b 100644 --- a/exynos/exynos_drmif.h +++ b/exynos/exynos_drmi

Re: [PATCH] MAINTAINERS: Add Laurent and Andrzej as maintainers for bridge drivers

2017-04-05 Thread Sean Paul
On Wed, Apr 05, 2017 at 04:53:50PM +0530, Archit Taneja wrote: > Add Laurent and Andrzej as maintainers for DRM bridge chip drivers. They > actively review and contribute to bridge drivers and the bridge API. > > Cc: Laurent Pinchart > Cc: Andrzej Hajda Acked-by: Sean Paul > Signed-off-by: Ar

[PATCH libdrm 2/2] etnaviv: add fence fd support

2017-04-05 Thread Philipp Zabel
Add etna_cmd_stream_flush_explicit with in-fence fd and out-fence fd support for explicit fencing. Signed-off-by: Philipp Zabel --- etnaviv/etnaviv_cmd_stream.c | 33 + etnaviv/etnaviv_drmif.h | 2 ++ 2 files changed, 31 insertions(+), 4 deletions(-) diff -

[PATCH libdrm 1/2] etnaviv: sync uapi header

2017-04-05 Thread Philipp Zabel
Import the etnaviv header changes from kernel commits 9ad59fea162c ("drm/etnaviv: submit support for in-fences") and 78ec187f64fa ("drm/etnaviv: submit support for out-fences") for fence fd support. Signed-off-by: Philipp Zabel --- etnaviv/etnaviv_drm.h | 8 1 file changed, 8 insertions

Re: [PATCH] exynos: add C++ support to exynos_drmif header

2017-04-05 Thread Eric Engestrom
On Wednesday, 2017-04-05 16:22:24 +0200, Tobias Jakobi wrote: > Add the usual extern "C" when compiling in C++ mode. Thanks, but why specifically this header? The other exynos/*.h headers also lack the c++ mangling guard. A quick grep shows that only 15/101 headers in libdrm have it. Can I intere

Re: [PATCH libdrm 2/2] etnaviv: add fence fd support

2017-04-05 Thread Eric Engestrom
On Wednesday, 2017-04-05 14:34:56 +0200, Philipp Zabel wrote: > Add etna_cmd_stream_flush_explicit with in-fence fd and out-fence fd > support for explicit fencing. > > Signed-off-by: Philipp Zabel Series is: Reviewed-by: Eric Engestrom Do you have push access? > --- > etnaviv/etnaviv_cmd_st

Re: imx-drm: vblank wait timed out

2017-04-05 Thread Christian Gmeiner
2017-04-05 12:05 GMT+02:00 Philipp Zabel : > On Tue, 2017-04-04 at 14:50 +0200, Christian Gmeiner wrote: > [...] >> > Is this on a non-plus i.MX6? Maybe are missing the LDB DI clock glitch >> > fixes (commits 5d283b083800, 03d576f202e8, and f13abeff2cde)? >> >> Yes it is a non-plus i.MX6D and all t

[PATCH libdrm v3 1/2] Headers: Sync drm{,_mode}.h with the kernel

2017-04-05 Thread Daniel Stone
Generated using make headers_install, based on drm-misc-next commit 5db06a8a98f515f67446a69c57577c4c363ec65d. This clarifies the comments around modifiers such that they are per-framebuffer rather than per-plane, adds the beginnings of aspect ratio mode flags, link status properties, and updates t

[PATCH libdrm v3 2/2] Add CRTC ID to vblank event

2017-04-05 Thread Daniel Stone
From: Ander Conselvan de Oliveira When using the atomic API, one request can span multiple CRTCs, however one event is generated per CRTC. As we cannot disambiguate the CRTC with user data (since we only have one piece of user data to pass in), newer kernels can include the CRTC ID in the page fl

Re: [PATCH] exynos: add C++ support to exynos_drmif header

2017-04-05 Thread Tobias Jakobi
Hello Eric, Eric Engestrom wrote: > On Wednesday, 2017-04-05 16:22:24 +0200, Tobias Jakobi wrote: >> Add the usual extern "C" when compiling in C++ mode. > > Thanks, but why specifically this header? The other exynos/*.h headers > also lack the c++ mangling guard. I'm currently writing a small C

Re: [PATCH libdrm 2/2] etnaviv: add fence fd support

2017-04-05 Thread Philipp Zabel
On Wed, 2017-04-05 at 17:07 +0100, Eric Engestrom wrote: > On Wednesday, 2017-04-05 14:34:56 +0200, Philipp Zabel wrote: > > Add etna_cmd_stream_flush_explicit with in-fence fd and out-fence fd > > support for explicit fencing. > > > > Signed-off-by: Philipp Zabel > > Series is: > Reviewed-by: E

Re: [PATCH v3 8/9] drm/rockchip: gem: Don't alloc/free gem buf when dev_private is invalid

2017-04-05 Thread Sean Paul
On Wed, Apr 05, 2017 at 04:29:26PM +0800, Jeffy Chen wrote: > After unbinding drm, the userspace may still has a chance to access > gem buf. > > Add a sanity check for a NULL dev_private to prevent that from > happening. I still don't understand how this is happening. You're saying that these hoo

[PATCH libdrm] configure.ac: pthread-stubs is not a thing on GNU/kFreeBSD

2017-04-05 Thread Emil Velikov
From: Emil Velikov As mentioned on the xcb mailing list, the platform uses the GLIBC forwarding mechanism. https://lists.freedesktop.org/archives/xcb/2016-November/010896.html Cc: Andreas Boll Reported-by: Andreas Boll Signed-off-by: Emil Velikov --- I'm slightly inclined to go a single *gnu

Re: [PATCH libdrm] configure.ac: pthread-stubs is not a thing on GNU/kFreeBSD

2017-04-05 Thread Eric Engestrom
On Wednesday, 2017-04-05 17:24:31 +0100, Emil Velikov wrote: > From: Emil Velikov > > As mentioned on the xcb mailing list, the platform uses the GLIBC > forwarding mechanism. > > https://lists.freedesktop.org/archives/xcb/2016-November/010896.html > > Cc: Andreas Boll > Reported-by: Andreas B

Re: [PATCH libdrm] configure.ac: pthread-stubs is not a thing on GNU/kFreeBSD

2017-04-05 Thread Andreas Boll
2017-04-05 18:43 GMT+02:00 Eric Engestrom : > On Wednesday, 2017-04-05 17:24:31 +0100, Emil Velikov wrote: >> From: Emil Velikov >> >> As mentioned on the xcb mailing list, the platform uses the GLIBC >> forwarding mechanism. >> >> https://lists.freedesktop.org/archives/xcb/2016-November/010896.ht

[PATCH v6 4/5] drm: Connector helper function to release resources

2017-04-05 Thread Dhinakaran Pandiyan
From: "Pandiyan, Dhinakaran" Having an ->atomic_release callback is useful to release shared resources that get allocated in compute_config(). This function is expected to be called in the atomic_check() phase before new resources are acquired. v5: Return an int (Maarten) v4: Document that the f

[PATCH v3 1/9] drm: bridge: analogix: Detach panel when unbinding analogix dp

2017-04-05 Thread Jeffy Chen
The panel is attached when binding analogix dp. Signed-off-by: Jeffy Chen --- Changes in v3: None Changes in v2: Fix some commit messages. drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c

[PATCH v6 2/5] drm/dp: Introduce MST topology state to track available link bandwidth

2017-04-05 Thread Dhinakaran Pandiyan
From: "Pandiyan, Dhinakaran" Link bandwidth is shared between multiple display streams in DP MST configurations. The DP MST topology manager structure maintains the shared link bandwidth for a primary link directly connected to the GPU. For atomic modesetting drivers, checking if there is suffici

Re: [PATCH 2/4] drm: Add drm_object lease infrastructure

2017-04-05 Thread Keith Packard
Daniel Vetter writes: > On Sat, Apr 01, 2017 at 10:08:39AM -0700, Keith Packard wrote: >> +BUG_ON(__mutex_owner(&master->dev->mode_config.idr_mutex) != current); > > Forgot to reply on this: > > lockdep_assert_held + enable lockdep. Thanks. Will fix. -- -keith signature.asc Description:

Re: [PATCH 0/2] virtio: fix virtio_gpu_mode_dumb_create

2017-04-05 Thread Laurent Vivier
On 03/04/2017 09:08, Gerd Hoffmann wrote: > Hi, > > Little series to fix xorg display on virtio on bigendian machines. > First patch factors out the drm -> virtio format mapping code into > a helper function, second patch actually fixes the issue using the > new helper. > > cheers, > Gerd >

Re: [PULL] Synopsys Media Formats

2017-04-05 Thread Mauro Carvalho Chehab
Em Mon, 3 Apr 2017 12:35:44 -0400 Sean Paul escreveu: > Hi Mauro, > > Here's the pull for Neil's new media formats. We're using a topic branch in > drm-misc, so it will not change. Once you have acked, we'll pull this in and > apply the rest of Neil's set. Thanks! Acked-by: Mauro Carvalho Cheh

Re: [PATCH v2 6/9] drm/rockchip: Reoder unload sequence

2017-04-05 Thread jeffy
Hi Sean, On 04/05/2017 03:44 AM, Sean Paul wrote: On Sat, Apr 01, 2017 at 07:35:26PM +0800, Jeffy Chen wrote: We should not cleanup iommu before cleanup other resources. Reorder unload sequence, follow exynos drm. This doesn't match the cleanup sequence in rockchip_drm_bind. Also make sure t

[PATCH v3 6/9] drm/rockchip: Reoder drm bind/unbind sequence

2017-04-05 Thread Jeffy Chen
Current drm bind/unbind sequence would cause some memory issues. For example we should not cleanup iommu before cleanup mode config. Reorder bind/unbind sequence, follow exynos drm. Signed-off-by: Jeffy Chen --- Changes in v3: Address Sean Paul 's comments. Update commit message. Changes in v2

Re: Proposal for RandR version 1.6, Leases and EDID-based output grabs

2017-04-05 Thread Keith Packard
Daniel Vetter writes: > The multi-seat thing sounds like vapourware, I think we should care about > the vr use-case for now, and only that one. Ok, I can live with that, even if I like the idea of a slightly more general solution. > For VR itself I'd go as far as saying that probably our "creat

RE: DRM Display driver for Intel FPGA Video and Image Processing Suite

2017-04-05 Thread Ong, Hean Loong
Hi All, Apologies for the attachment earlier. Below are the inline changes for the patch From 0de293e3646a1780ed603cf8e1f2a19d9aebbe83 Mon Sep 17 00:00:00 2001 From: Ong, Hean Loong Date: Thu, 30 Mar 2017 18:02:22 +0800 Subject: [PATCHv0] Intel FPGA Video and Image Processing Suite Frame Buffer

Google Summer of Code time conflict

2017-04-05 Thread Jacob Lifshay
I forgot to mention in my application that I have classes until june 7. Also, Dave Airlie said he might be willing to mentor me: https://lists.freedesktop.org/archives/mesa-dev/2017-February/144078.html Jacob Lifshay ___ dri-devel mailing list dri-devel@l

Re: Proposal for RandR version 1.6, Leases and EDID-based output grabs

2017-04-05 Thread Keith Packard
Daniel Vetter writes: > Hm, if you restrict getresources and getplanes, you'll get your leased > objects query api. Iirc that part was missing in your kernel patch. And it > gives you exaclty what you want: per-type list of object ids. Hrm. I think that's one Dave didn't want to restrict so that

Re: [Intel-gfx] [PATCH v5 3/5] drm/dp: Add DP MST helpers to atomically find and release vcpi slots

2017-04-05 Thread Pandiyan, Dhinakaran
On Thu, 2017-03-30 at 01:42 -0700, Dhinakaran Pandiyan wrote: > From: "Pandiyan, Dhinakaran" > > drm_dp_atomic_find_vcpi_slots() should be called from ->atomic_check() to > check there are sufficient vcpi slots for a mode and to add that to the > state. This should be followed by a call to drm_dp

Re: [PATCH v2 8/9] drm/rockchip: gem: Don't alloc/free gem buf before drm dev registered

2017-04-05 Thread jeffy
Hi Daniel, On 04/03/2017 03:41 PM, Daniel Vetter wrote: On Sat, Apr 01, 2017 at 07:35:28PM +0800, Jeffy Chen wrote: Signed-off-by: Jeffy Chen --- Changes in v2: None Wut? How is this even possible? If you haven't registered the driver yet, there's no way for userspace to call allocation fun

[PATCH v3 4/9] drm/rockchip: cdn-dp: Don't try to release firmware when not loaded

2017-04-05 Thread Jeffy Chen
Signed-off-by: Jeffy Chen --- Changes in v3: None Changes in v2: None drivers/gpu/drm/rockchip/cdn-dp-core.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/rockchip/cdn-dp-core.c b/drivers/gpu/drm/rockchip/cdn-dp-core.c index 4e55d63..ee4195d 100644 ---

Re: Vulkan WSI+VK_KHR_display for KMS/DRM?

2017-04-05 Thread Keith Packard
Jason Ekstrand writes: > Interesting question. To my knowledge, no one has actually implemented the > Vulkan WSI direct-to-display extensions. (I tried to prevent them from > getting released with 1.0 but failed.) I believe the correct answer is to > use the external memory dma-buf stuff that

Re: [PATCH v2 7/9] drm/rockchip: Force disable all crtc when unload

2017-04-05 Thread jeffy
Hi Daniel, On 04/03/2017 03:58 PM, Daniel Vetter wrote: On Sat, Apr 1, 2017 at 1:35 PM, Jeffy Chen wrote: diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c index a5d83cb..5dbf011 100644 --- a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c +++

Re: [PATCH 0/4] Fix DP busy wait and defer disabling overlay plane

2017-04-05 Thread Dan MacDonald
I should've said that 4.10 didn't pause at: [ 4.458572] panel-simple panel-lcd: panel-lcd supply power not found, using dummy regulator [ 4.467736] panel-simple panel-lvds0: panel-lvds0 supply power not found, using dummy regulator If it showed those lines at all and I've never seen those flip_do

Re: [PATCH 0/4] Fix DP busy wait and defer disabling overlay plane

2017-04-05 Thread Dan MacDonald
Hi Phillipp My SL was attached to a 27" UHD Acer S277HK Monitor. I can disable those for the next patch but I get the impression thats not the only issue here as I've gone from being able to get a picture (but not the right res) in 4.10 to not getting any HDMI signal at all with this patch. Thos

[PATCH v6 1/5] drm: Add driver-private objects to atomic state

2017-04-05 Thread Dhinakaran Pandiyan
From: "Pandiyan, Dhinakaran" It is necessary to track states for objects other than connector, crtc and plane for atomic modesets. But adding objects like DP MST link bandwidth to drm_atomic_state would mean that a non-core object will be modified by the core helper functions for swapping and cle

Re: [PATCH v2] drm: hdlcd: Fix the calculation of the scanout start address

2017-04-05 Thread Russell King - ARM Linux
On Mon, Apr 03, 2017 at 11:31:34AM +0100, Liviu Dudau wrote: > On Fri, Mar 31, 2017 at 02:48:10PM +0100, Russell King - ARM Linux wrote: > > I sent a reminder on 20th February about it, and we discussed it, and I > > said at the time I did not have time to test your patch. Ville commented > > on y

[PATCH v3 5/9] drm/rockchip: vop: Enable pm domain before vop_initial

2017-04-05 Thread Jeffy Chen
We're trying to access vop registers here, so need to make sure the pm domain is on. Normally it should be enabled by the bootloader, but there's no guarantee of it. And if we wanna do unbind/bind, it would also cause the device to hang. And this patch also does these: 1/ move vop_initial to the

[PATCH v3 3/9] drm: bridge: analogix: Destroy connector when unbinding

2017-04-05 Thread Jeffy Chen
Normally we do this in drm_mode_config_cleanup. But analogix dp's connector is allocated in bind, and freed after unbind. So we need to destroy it in unbind to avoid further access. Signed-off-by: Jeffy Chen --- Changes in v3: None Changes in v2: None drivers/gpu/drm/bridge/analogix/analogix_d

Re: Vulkan WSI+VK_KHR_display for KMS/DRM?

2017-04-05 Thread Keith Packard
Chad Versace writes: > The real path forward should be implemented on top of > VK_KHX_external_memory. If you want to start experimenting now with > Vulkan+KMS, you may want to look at > VK_EXTERNAL_MEMORY_HANDLE_OPAQUE_FD_KHX. It seems like the Vulkan spec for WSI with devices has a bunch of wh

[PATCH v6 5/5] drm/dp: Track MST link bandwidth

2017-04-05 Thread Dhinakaran Pandiyan
From: "Pandiyan, Dhinakaran" Use the added helpers to track MST link bandwidth for atomic modesets. Link bw is acquired in the ->atomic_check() phase when CRTCs are being enabled with drm_atomic_find_vcpi_slots() instead of drm_find_vcpi_slots(). Similarly, link bw is released during ->atomic_che

Re: [PATCH v3 01/11] dt-bindings: add binding for the Allwinner DE2 CCU

2017-04-05 Thread Icenowy Zheng
在 2017年04月03日 23:33, Rob Herring 写道: On Thu, Mar 30, 2017 at 03:46:03AM +0800, Icenowy Zheng wrote: From: Icenowy Zheng Allwinner "Display Engine 2.0" contains some clock controls in it. In order to add them as clock drivers, we need a device tree binding. Add the binding here. Signed-off-

[PATCH] drm/vmwgfx: Convert macro to octal representation

2017-04-05 Thread Øyvind A . Holm
Change "S_IRUSR | S_IWUSR" to "0600", it's easier to parse mentally. This change should be part of commit 50f837371dd9 ("drm/vmwgfx: Revert "drm/vmwgfx: Replace numeric parameter like 0444 with macro""), but the patch was truncated somewhere in the patch route, so add the missing change. Extract f

Re: [PATCH 0/4] Fix DP busy wait and defer disabling overlay plane

2017-04-05 Thread Dan MacDonald
Hi Phillipp Thanks for the patch, but did you mean v4.11-rc4 or is it really/only for v4.11-rc1, if it makes a difference? I was trying to build rc4. Thanks On Mon, Apr 3, 2017 at 12:46 PM, Philipp Zabel wrote: > Hi Dan, > > On Sat, 2017-04-01 at 11:50 +0100, Dan MacDonald wrote: >> No such luc

Re: Proposal for RandR version 1.6, Leases and EDID-based output grabs

2017-04-05 Thread Keith Packard
Daniel Vetter writes: > Also if this confuses VR, then another reason why we want to make leases > invariant and only allow pure revoke, not changing the list. I'm not sure why you want this to be asymmetrical, nor why you would expect lessees to be any more competent at dealing with hotplug tha

Re: [PATCH 0/4] Fix DP busy wait and defer disabling overlay plane

2017-04-05 Thread Dan MacDonald
It took just under 11 hours to build the Arch 4.11 armv7h kernel on my SL. The good news is that the patch both applied and built OK this time, and the kernel package installed fine. What didn't seem right is that boot pauses for a minute or so on these lines: [ 4.458572] panel-simple panel-lcd: p

  1   2   >