Re: [PATCH 3/5] drm/virtio: Return an error from connector dpms callback

2017-07-26 Thread Daniel Vetter
On Wed, Jul 26, 2017 at 10:56:34PM +0200, Takashi Iwai wrote: > The virtio drm driver doesn't treat with DPMS, so we should return an > error from the connector dpms callback so that the fbcon can fall back > to the generic blank mode. > > Signed-off-by: Takashi Iwai > --- > drivers/gpu/drm/virt

Re: [PATCH 2/5] drm/bochs: Return an error from connector dpms callback

2017-07-26 Thread Daniel Vetter
On Wed, Jul 26, 2017 at 10:56:33PM +0200, Takashi Iwai wrote: > The bochs drm driver doesn't treat with DPMS, so we should return an > error from the connector dpms callback so that the fbcon can fall back > to the generic blank mode. > > Signed-off-by: Takashi Iwai > --- > drivers/gpu/drm/bochs

Re: [PATCH 1/5] drm: Propagate error from connector dpms function in drm_fb_helper_blank()

2017-07-26 Thread Daniel Vetter
On Wed, Jul 26, 2017 at 10:56:32PM +0200, Takashi Iwai wrote: > Currently the DRM fbcon helper for console blank, > drm_fb_helper_blank(), simply calls drm_fb_helper_dpms() and always > returns zero, supposing the driver dealing with DPMS properly for > blanking the screen. However, it turned out

Re: [PATCH 17/41] drm/sun4i: Use .dumb_map_offset and .dumb_destroy defaults

2017-07-26 Thread Maxime Ripard
On Sun, Jul 23, 2017 at 09:16:33PM +0200, Noralf Trønnes wrote: > This driver can use the drm_driver.dumb_destroy and > drm_driver.dumb_map_offset defaults, so no need to set them. > > Cc: Maxime Ripard > Signed-off-by: Noralf Trønnes > --- > drivers/gpu/drm/sun4i/sun4i_drv.c | 2 -- > 1 file c

[GIT PULL] exynos-drm-fixes

2017-07-26 Thread Inki Dae
Hi Dave, Just several regression fixups and trivial cleanups. Please kindly let me know if there is any problem. Thanks, Inki Dae The following changes since commit 517069ff6ec0971e0794ab97d4fcfa8b143746e4: Merge branch 'drm-fixes-4.13' of git://people.freedesktop.org/~agd5f/linux int

Re: [PATCH] drm: hdlcd: allow HDLCD to be used without interrupt

2017-07-26 Thread Russell King - ARM Linux
On Wed, Jul 26, 2017 at 11:05:39AM +0100, Russell King wrote: > Some ARM platforms do not wire the HDLCD interrupt. Allow hdlcd to > initialise without an interrupt present. > > Signed-off-by: Russell King This isn't fully functional on ARM MPS platforms yet, but at least gets us further. With

[PATCH] drm: hdlcd: allow HDLCD to be used without interrupt

2017-07-26 Thread Russell King
Some ARM platforms do not wire the HDLCD interrupt. Allow hdlcd to initialise without an interrupt present. Signed-off-by: Russell King --- Documentation/devicetree/bindings/display/arm,hdlcd.txt | 6 +++--- drivers/gpu/drm/arm/hdlcd_drv.c | 13 - 2 files ch

Re: [PATCH v3] drm/msm: gpu: don't abuse dma_alloc for non-DMA allocations

2017-07-26 Thread Bjorn Andersson
On Wed 26 Jul 12:59 PDT 2017, Arnd Bergmann wrote: > In zap_shader_load_mdt(), we pass a pointer to a phys_addr_t > into dmam_alloc_coherent, which the compiler warns about: > > drivers/gpu/drm/msm/adreno/a5xx_gpu.c: In function 'zap_shader_load_mdt': > drivers/gpu/drm/msm/adreno/a5xx_gpu.c:54:50

Re: [PATCH 01/41] drm/gem: Add drm_gem_dumb_map_offset()

2017-07-26 Thread Emil Velikov
On 26 July 2017 at 19:41, Noralf Trønnes wrote: > > Den 26.07.2017 14.05, skrev Emil Velikov: >> >> Hi Noralf, >> >> >> On 23 July 2017 at 20:16, Noralf Trønnes wrote: >>> >>> Add a common drm_driver.dumb_map_offset function for GEM backed drivers. >>> >>> Signed-off-by: Noralf Trønnes >>> --- >

Re: [PATCH v6 3/7] drm/rockchip: vop: move line_flag_num to interrupt registers

2017-07-26 Thread Heiko Stuebner
Am Mittwoch, 26. Juli 2017, 14:19:19 CEST schrieb Mark Yao: > In the hardware design process, the design of line flags > register is associated with the interrupt register, > placing the line flags in the interrupt definition is > more reasonable, and it would make multi-vop define easilier. > > S

Re: [PATCH v6 2/7] drm/rockchip: vop: move write_relaxed flags to vop register

2017-07-26 Thread Heiko Stuebner
Am Mittwoch, 26. Juli 2017, 14:19:12 CEST schrieb Mark Yao: > Since the drm atomic framework, only a small part of the vop > register needs sync write, Currently seems only following registers > need sync write: >cfg_done, standby and interrupt related register. > > All ctrl registers are usin

Re: [PATCH v6 1/7] drm/rockchip: vop: initialize registers directly

2017-07-26 Thread Heiko Stuebner
Am Mittwoch, 26. Juli 2017, 14:19:05 CEST schrieb Mark Yao: > At present we are using init_table to initialize some > registers, but the Register init table use un-document define, > it is unreadable, and sometimes we only want to update tiny > bits, init table method is not friendly, it's diffcult

Re: [PATCH v5 2/6] drm/bridge: Add a devm_ allocator for panel bridge.

2017-07-26 Thread Eric Anholt
Philippe CORNU writes: > On 07/18/2017 11:05 PM, Eric Anholt wrote: >> This will let drivers reduce the error cleanup they need, in >> particular the "is_panel_bridge" flag. >> >> v2: Slight cleanup of remove function by Andrzej >> >> Signed-off-by: Eric Anholt >> Reviewed-by: Andrzej Hajda >

[Bug 101672] radeonsi: 3D engines causing frequent GPU lockups

2017-07-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101672 --- Comment #17 from MirceaKitsune --- My understanding is limited when it comes to drivers and core OpenGL components, so what I say might be completely irrelevant. Looking through the logs, I'm noticing something suspicious in ttm_dma_page_poo

[Bug 101672] radeonsi: 3D engines causing frequent GPU lockups

2017-07-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101672 --- Comment #16 from MirceaKitsune --- Created attachment 133061 --> https://bugs.freedesktop.org/attachment.cgi?id=133061&action=edit watch-ttm_page_pool.txt -- You are receiving this mail because: You are the assignee for the bug._

[Bug 101672] radeonsi: 3D engines causing frequent GPU lockups

2017-07-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101672 --- Comment #15 from MirceaKitsune --- Created attachment 133060 --> https://bugs.freedesktop.org/attachment.cgi?id=133060&action=edit watch-ttm_dma_page_pool.txt -- You are receiving this mail because: You are the assignee for the bug._

[Bug 101672] radeonsi: 3D engines causing frequent GPU lockups

2017-07-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101672 --- Comment #14 from MirceaKitsune --- Created attachment 133059 --> https://bugs.freedesktop.org/attachment.cgi?id=133059&action=edit watch-radeon_vram_mm.txt -- You are receiving this mail because: You are the assignee for the bug.

[Bug 101672] radeonsi: 3D engines causing frequent GPU lockups

2017-07-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101672 --- Comment #12 from MirceaKitsune --- Created attachment 133057 --> https://bugs.freedesktop.org/attachment.cgi?id=133057&action=edit watch-gem_names.txt -- You are receiving this mail because: You are the assignee for the bug._

[Bug 101672] radeonsi: 3D engines causing frequent GPU lockups

2017-07-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101672 --- Comment #13 from MirceaKitsune --- Created attachment 133058 --> https://bugs.freedesktop.org/attachment.cgi?id=133058&action=edit watch-radeon_gtt_mm.txt -- You are receiving this mail because: You are the assignee for the bug._

[Bug 101672] radeonsi: 3D engines causing frequent GPU lockups

2017-07-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101672 --- Comment #11 from MirceaKitsune --- Created attachment 133056 --> https://bugs.freedesktop.org/attachment.cgi?id=133056&action=edit watch-clients.txt -- You are receiving this mail because: You are the assignee for the bug.___

[Bug 101672] radeonsi: 3D engines causing frequent GPU lockups

2017-07-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101672 --- Comment #10 from MirceaKitsune --- I was able to use parallel SSH sessions to monitor changes in the files suggested by Max Staudt, which I did by using the command: watch -n 1 cat filename The relevant files that existed and I was able to

[pull] amdgpu drm-fixes-4.13

2017-07-26 Thread Alex Deucher
Hi Dave, A few fixes for 4.13. The following changes since commit cfd10811080f5bb0592e21d0b197d445f133ba9e: Merge branch 'linux-4.13' of git://github.com/skeggsb/linux into drm-fixes (2017-07-25 15:41:39 +1000) are available in the git repository at: git://people.freedesktop.org/~agd5f/li

[Bug 98239] saints row 3: performance is limited by flushes

2017-07-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98239 almos changed: What|Removed |Added Resolution|--- |WORKSFORME Status|NEW

[PATCH 4/5] drm/qxl: Return an error from connector dpms callback

2017-07-26 Thread Takashi Iwai
The qxl drm driver doesn't treat with DPMS, so we should return an error from the connector dpms callback so that the fbcon can fall back to the generic blank mode. Signed-off-by: Takashi Iwai --- drivers/gpu/drm/qxl/qxl_display.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) di

[PATCH 1/5] drm: Propagate error from connector dpms function in drm_fb_helper_blank()

2017-07-26 Thread Takashi Iwai
Currently the DRM fbcon helper for console blank, drm_fb_helper_blank(), simply calls drm_fb_helper_dpms() and always returns zero, supposing the driver dealing with DPMS properly for blanking the screen. However, it turned out that the console blank doesn't work at all on KVM/QEMU when DRM driver

[PATCH 3/5] drm/virtio: Return an error from connector dpms callback

2017-07-26 Thread Takashi Iwai
The virtio drm driver doesn't treat with DPMS, so we should return an error from the connector dpms callback so that the fbcon can fall back to the generic blank mode. Signed-off-by: Takashi Iwai --- drivers/gpu/drm/virtio/virtgpu_display.c | 9 - 1 file changed, 8 insertions(+), 1 delet

[PATCH 2/5] drm/bochs: Return an error from connector dpms callback

2017-07-26 Thread Takashi Iwai
The bochs drm driver doesn't treat with DPMS, so we should return an error from the connector dpms callback so that the fbcon can fall back to the generic blank mode. Signed-off-by: Takashi Iwai --- drivers/gpu/drm/bochs/bochs_kms.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-)

[PATCH 5/5] drm/cirrus: Return an error from connector dpms callback

2017-07-26 Thread Takashi Iwai
Although the cirrus drm driver handles DPMS and sets the register bits accordingly, currently KVM/QEMU ignores it, resulting in the non-functional console blank. This patch makes the connector dpms callback always returning an error so that the fbcon can fall back to the generic blank mode. Signe

[PATCH 0/5] drm: Fix console blank on QEMU

2017-07-26 Thread Takashi Iwai
Hi, this is a revised patchset after thinking of Daniel's suggestion. In this version, drm_fb_helper_blank() is modified to treat the error code from the connector dpms callback, so that the caller side, fbcon, can fall back to the generic blank code gracefully for the drivers that don't support D

[Bug 89282] new error messages 4.0-rc1 kernel

2017-07-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=89282 Samuel Pitoiset changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug 98239] saints row 3: performance is limited by flushes

2017-07-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98239 --- Comment #8 from Samuel Pitoiset --- After playing SR3 a bit on RX480 with mesa 17.1.5 and LLVM 4.0.1, it appears to perform quite well. I have 70-80 FPS almost everywhere in ultra and it's GPU-bound there. Okay it's a different configuration

Re: [PATCH 2/2] drm/msm: gpu: don't abuse dma_alloc for non-DMA allocations

2017-07-26 Thread Arnd Bergmann
On Jul 26, 2017 6:35 PM, "Jordan Crouse" wrote: > > if (!IS_ENABLED(CONFIG_ARCH_QCOM)) > > return -EINVAL; > > > > + np = of_get_child_by_name(dev->of_node, "zap-shader"); > > + if (!np) > > + return -ENODEV; > > + > > + np = of_parse_phandle(dev->of_n

[PATCH v3] drm/msm: gpu: don't abuse dma_alloc for non-DMA allocations

2017-07-26 Thread Arnd Bergmann
In zap_shader_load_mdt(), we pass a pointer to a phys_addr_t into dmam_alloc_coherent, which the compiler warns about: drivers/gpu/drm/msm/adreno/a5xx_gpu.c: In function 'zap_shader_load_mdt': drivers/gpu/drm/msm/adreno/a5xx_gpu.c:54:50: error: passing argument 3 of 'dmam_alloc_coherent' from inc

Re: [PATCHv2 3/3] drm/vc4: add HDMI CEC support

2017-07-26 Thread Eric Anholt
Hans Verkuil writes: > Hi Eric, > > On 16/07/17 12:48, Hans Verkuil wrote: >> From: Hans Verkuil >> >> This patch adds support to VC4 for CEC. >> >> Thanks to Eric Anholt for providing me with the CEC register information. >> >> To prevent the firmware from eating the CEC interrupts you need

[Bug 101484] [regression, bisected] Steam fails to render content, if mesa is compiled with -O2 -march=haswell

2017-07-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101484 --- Comment #14 from LoneVVolf --- (In reply to Gregor Münch from comment #5) > Ive tested this now, and again its a compiler flags issue. > Arch standard clfags are: > CFLAGS="-march=native -O2 -pipe -fstack-protector-strong" > for clarity, d

Re: [PATCH 7/8] drm: Nuke drm_atomic_helper_connector_dpms

2017-07-26 Thread Noralf Trønnes
Den 25.07.2017 10.01, skrev Daniel Vetter: It's dead code, the core handles all this directly now. The only special case is nouveau and tda988x which used one function for both legacy modeset code and -nv50 atomic world instead of 2 vtables. But amounts to exactly the same. v2: Rebase over the

Re: [PATCH 01/41] drm/gem: Add drm_gem_dumb_map_offset()

2017-07-26 Thread Noralf Trønnes
Den 26.07.2017 14.05, skrev Emil Velikov: Hi Noralf, On 23 July 2017 at 20:16, Noralf Trønnes wrote: Add a common drm_driver.dumb_map_offset function for GEM backed drivers. Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/drm_gem.c | 35 +++ include/drm

Re: [PATCH 2/3] ACPI/DRM: rework ACPI_VIDEO Kconfig dependencies

2017-07-26 Thread Rafael J. Wysocki
On Wednesday, July 26, 2017 03:53:11 PM Arnd Bergmann wrote: > ACPI_VIDEO keeps causing problems with circular Kconfig dependencies, > as it depends on a couple of other symbols, and it gets selected by > drivers that may end up being depending on others. > > This is an attempt to simplify this by

[PATCH 5/6] drm/i915: Add format modifiers for Intel

2017-07-26 Thread Ben Widawsky
This was based on a patch originally by Kristian. It has been modified pretty heavily to use the new callbacks from the previous patch. v2: - Add LINEAR and Yf modifiers to list (Ville) - Combine i8xx and i965 into one list of formats (Ville) - Allow 1010102 formats for Y/Yf tiled (Ville) v

[PATCH 6/6] drm/i915: Add support for CCS modifiers

2017-07-26 Thread Ben Widawsky
v2: Support sprite plane. Support pipe C/D limitation on GEN9. v3: Rename structure (Ville) Handle GLK (Ville) This requires rebase on the correct Ville patches Cc: Daniel Stone Cc: Kristian Høgsberg Signed-off-by: Ben Widawsky --- drivers/gpu/drm/i915/intel_display.c | 30 ++

[PATCH 4/6] drm: Create a format/modifier blob

2017-07-26 Thread Ben Widawsky
Updated blob layout (Rob, Daniel, Kristian, xerpi) v2: * Removed __packed, and alignment (.+) * Fix indent in drm_format_modifier fields (Liviu) * Remove duplicated modifier > 64 check (Liviu) * Change comment about modifier (Liviu) * Remove arguments to blob creation, use plane instead (Liviu) *

[PATCH 3/6] drm: Plumb modifiers through plane init

2017-07-26 Thread Ben Widawsky
This is the plumbing for supporting fb modifiers on planes. Modifiers have already been introduced to some extent, but this series will extend this to allow querying modifiers per plane. Based on this, the client to enable optimal modifications for framebuffers. This patch simply allows the DRM dr

[PATCH 2/6] drm/i915: Add render decompression support

2017-07-26 Thread Ben Widawsky
From: Ville Syrjälä SKL+ display engine can scan out certain kinds of compressed surfaces produced by the render engine. This involved telling the display engine the location of the color control surfae (CCS) which describes which parts of the main surface are compressed and which are not. The lo

[PATCH 1/6] drm/i915: Implement .get_format_info() hook for CCS

2017-07-26 Thread Ben Widawsky
From: Ville Syrjälä SKL+ display engine can scan out certain kinds of compressed surfaces produced by the render engine. This involved telling the display engine the location of the color control surfae (CCS) which describes which parts of the main surface are compressed and which are not. The lo

Re: [RFC v3] drm/hdcp: drm enum property for CP State

2017-07-26 Thread Sean Paul
On Wed, Jul 26, 2017 at 12:51 PM, C, Ramalingam wrote: > >> -Original Message- >> From: Sean Paul [mailto:seanp...@chromium.org] >> Sent: Wednesday, July 26, 2017 8:23 PM >> To: C, Ramalingam >> Cc: Sean Paul ; Vetter, Daniel >> ; Intel Graphics Development > g...@lists.freedesktop.org>;

Re: [PATCH 4/4] drm/ttm: individualize BO reservation obj when they are freed

2017-07-26 Thread Alex Deucher
On Wed, Jul 26, 2017 at 9:12 AM, Christian König wrote: > From: Christian König > > Use the BOs reservation object when it is put on the ddelete list. This way we > avoid delaying freeing up the BO because of new fences on the reservation > object. > > Signed-off-by: Christian König You might

Re: [PATCH 1/4] dma-buf: dma_fence_put is NULL save

2017-07-26 Thread Alex Deucher
On Wed, Jul 26, 2017 at 9:12 AM, Christian König wrote: > From: Christian König > > No need to check. > > Signed-off-by: Christian König Patch title: s/save/safe/ With that fixed: Reviewed-by: Alex Deucher > --- > drivers/dma-buf/reservation.c | 6 ++ > 1 file changed, 2 insertions(+), 4

RE: [RFC v3] drm/hdcp: drm enum property for CP State

2017-07-26 Thread C, Ramalingam
> -Original Message- > From: Sean Paul [mailto:seanp...@chromium.org] > Sent: Wednesday, July 26, 2017 8:23 PM > To: C, Ramalingam > Cc: Sean Paul ; Vetter, Daniel > ; Intel Graphics Development g...@lists.freedesktop.org>; dri-devel ; > Daniel > Vetter > Subject: Re: [RFC v3] drm/hdcp

Re: [PATCH 2/2] drm/msm: gpu: don't abuse dma_alloc for non-DMA allocations

2017-07-26 Thread Jordan Crouse
On Wed, Jul 26, 2017 at 05:52:45PM +0200, Arnd Bergmann wrote: > In zap_shader_load_mdt(), we pass a pointer to a phys_addr_t > into dmam_alloc_coherent, which the compiler warns about: > > drivers/gpu/drm/msm/adreno/a5xx_gpu.c: In function 'zap_shader_load_mdt': > drivers/gpu/drm/msm/adreno/a5xx_

[PATCH 2/2] drm/msm: gpu: don't abuse dma_alloc for non-DMA allocations

2017-07-26 Thread Arnd Bergmann
In zap_shader_load_mdt(), we pass a pointer to a phys_addr_t into dmam_alloc_coherent, which the compiler warns about: drivers/gpu/drm/msm/adreno/a5xx_gpu.c: In function 'zap_shader_load_mdt': drivers/gpu/drm/msm/adreno/a5xx_gpu.c:54:50: error: passing argument 3 of 'dmam_alloc_coherent' from inc

[PATCH 1/2] drm/msm: gpu: call qcom_mdt interfaces only for ARCH_QCOM

2017-07-26 Thread Arnd Bergmann
When compile-testing for something other than ARCH_QCOM, we run into a link error: drivers/gpu/drm/msm/adreno/a5xx_gpu.o: In function `a5xx_hw_init': a5xx_gpu.c:(.text.a5xx_hw_init+0x600): undefined reference to `qcom_mdt_get_size' a5xx_gpu.c:(.text.a5xx_hw_init+0x93c): undefined reference to `qc

Re: [PATCH 1/3] backlight: always select BACKLIGHT_LCD_SUPPORT for BACKLIGHT_CLASS_DEVICE

2017-07-26 Thread Andy Shevchenko
On Wed, Jul 26, 2017 at 4:53 PM, Arnd Bergmann wrote: > randconfig builds occasionally produce this Kconfig warning: > > warning: (DRM_RADEON && DRM_AMDGPU && DRM_NOUVEAU && DRM_I915 && DRM_GMA500 > && DRM_SHMOBILE && DRM_TILCDC && DRM_FSL_DCU && DRM_TINYDRM && > DRM_PARADE_PS8622 && FB_BACKLIGH

[Bug 101927] American Conquest via Wine cannot start

2017-07-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101927 --- Comment #1 from Emil Velikov --- Just sharing some ideas, I likely won't be able to help much. Possible issues include: - file is not there, has wrong permissions, is corrupted - map fails due to wrong permissions of the segment, OOM cond

Re: [PATCH] android: amdgpu: move asic id table to a separate file

2017-07-26 Thread Emil Velikov
On 25 July 2017 at 08:28, Chih-Wei Huang wrote: > 2017-07-22 17:01 GMT+08:00 Mauro Rossi : >> Changes in Android.mk makefile to avoid building errors in mesa >> due to missing LOCAL_CFLAGS variable definition for >> AMDGPU_ASIC_ID_TABLE and ASIC_ID_TABLE_NUM_ENTRIES >> >> Fixes: 7e6bf88cac ("amdgp

Re: [PATCH 1/2] drm/edid: Add helper to detect whether EDID changed

2017-07-26 Thread Paul Kocialkowski
On Tue, 2017-07-25 at 17:50 +0200, Daniel Vetter wrote: > On Tue, Jul 25, 2017 at 03:18:04PM +0300, Paul Kocialkowski wrote: > > On Tue, 2017-07-25 at 10:16 +0200, Daniel Vetter wrote: > > > On Tue, Jul 25, 2017 at 10:58:55AM +0300, Paul Kocialkowski wrote: > > > > On Tue, 2017-07-25 at 09:34 +0200

Re: [PATCH 01/14] amdgpu: powerplay: Remove unused function

2017-07-26 Thread Harry Wentland
On 2017-07-26 09:47 AM, Ricardo Ribalda Delgado wrote: > Hi Harry > > On Mon, Jul 24, 2017 at 5:45 PM, Harry Wentland > wrote: >> On 2017-07-24 10:06 AM, Ricardo Ribalda Delgado wrote: >>> Hi Harry >>> On Mon, Jul 24, 2017 at 4:01 PM, Harry Wentland >>> wrote: >>> This is used and ne

Re: [RFC v3] drm/hdcp: drm enum property for CP State

2017-07-26 Thread Sean Paul
On Wed, Jul 26, 2017 at 03:24:10PM +0530, Ramalingam C wrote: > > > On Tuesday 25 July 2017 06:04 PM, Sean Paul wrote: > > On Mon, Jul 24, 2017 at 2:12 PM, Ramalingam C > > wrote: > > > DRM connector property is created to represent the content protection > > > state of the connector and to con

Re: [PATCH RESEND] fbdev: omapfb: remove unused variable

2017-07-26 Thread Daniel Vetter
On Wed, Jul 26, 2017 at 04:31:38PM +0200, Arnd Bergmann wrote: > On Wed, Jul 26, 2017 at 4:27 PM, Bartlomiej Zolnierkiewicz > wrote: > > > > Hi, > > > > On Wednesday, July 26, 2017 03:57:55 PM Arnd Bergmann wrote: > >> Removing the default display name left a harmless warning: > >> > >> fbdev/omap

Re: [Intel-gfx] [PATCH 0/3] Kconfig dependencies: acpi-video, backlight and thermal

2017-07-26 Thread Daniel Vetter
On Wed, Jul 26, 2017 at 03:53:09PM +0200, Arnd Bergmann wrote: > Hi everyone, > > It took me a while to figure this out properly, as I kept getting > circular or missing dependencies with video drivers. > > This set of three patches should simplify the situation a bit, > mostly by cleaning up the

[Bug 101928] GRID via Wine crashes mid-race

2017-07-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101928 --- Comment #1 from almos --- For gallium nine see https://github.com/iXit/Mesa-3D/issues/285 -- You are receiving this mail because: You are the assignee for the bug.___ dri-devel mailing list dri-d

[Bug 101928] GRID via Wine crashes mid-race

2017-07-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101928 Bug ID: 101928 Summary: GRID via Wine crashes mid-race Product: Mesa Version: 17.1 Hardware: Other OS: All Status: NEW Severity: normal Priorit

[Bug 101927] American Conquest via Wine cannot start

2017-07-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101927 Bug ID: 101927 Summary: American Conquest via Wine cannot start Product: Mesa Version: 17.1 Hardware: Other OS: All Status: NEW Severity: normal

Re: [PATCH] drm: adding SDI to drm_connector_enum_list

2017-07-26 Thread Daniel Vetter
On Wed, Jul 26, 2017 at 10:22:49AM +0530, Saurabh Sengar wrote: > adding SDI to drm connector list > > Signed-off-by: Saurabh Sengar This is an uapi change, i.e. userspace needs to be updated. Do you _really_ need this? I'd recommend to just use something existing (go with VIRTUAL maybe, not sur

Re: [PATCH RESEND] fbdev: omapfb: remove unused variable

2017-07-26 Thread Arnd Bergmann
On Wed, Jul 26, 2017 at 4:27 PM, Bartlomiej Zolnierkiewicz wrote: > > Hi, > > On Wednesday, July 26, 2017 03:57:55 PM Arnd Bergmann wrote: >> Removing the default display name left a harmless warning: >> >> fbdev/omap2/omapfb/dss/core.c: In function 'omap_dss_probe': >> fbdev/omap2/omapfb/dss/core

Re: [PATCH RESEND] fbdev: omapfb: remove unused variable

2017-07-26 Thread Bartlomiej Zolnierkiewicz
Hi, On Wednesday, July 26, 2017 03:57:55 PM Arnd Bergmann wrote: > Removing the default display name left a harmless warning: > > fbdev/omap2/omapfb/dss/core.c: In function 'omap_dss_probe': > fbdev/omap2/omapfb/dss/core.c:196:30: error: unused variable 'pdata' > [-Werror=unused-variable] > >

Re: [PATCH 21/41] drm/tinydrm: Use .dumb_map_offset and .dumb_destroy defaults

2017-07-26 Thread Sean Paul
On Sun, Jul 23, 2017 at 09:16:37PM +0200, Noralf Trønnes wrote: > tinydrm can use the drm_driver.dumb_destroy and > drm_driver.dumb_map_offset defaults, so no need to set them. > > Signed-off-by: Noralf Trønnes Reviewed-by: Sean Paul > --- > include/drm/tinydrm/tinydrm.h | 4 +--- > 1 file c

Re: [PATCH 36/41] drm/hisilicon: hibmc: Use the drm_driver.dumb_destroy default

2017-07-26 Thread Sean Paul
On Sun, Jul 23, 2017 at 09:16:52PM +0200, Noralf Trønnes wrote: > drm_gem_dumb_destroy() is the drm_driver.dumb_destroy default, > so no need to set it. > > Cc: Chen Feng > Signed-off-by: Noralf Trønnes Reviewed-by: Sean Paul > --- > drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c | 1 - > 1

Re: [PATCH 02/41] drm/dumb-buffers: Add defaults for .dumb_map_offset and .dumb_destroy

2017-07-26 Thread Sean Paul
On Sun, Jul 23, 2017 at 09:16:18PM +0200, Noralf Trønnes wrote: > Almost everyone did end up using GEM as bo, so this adds defaults > for the drm_driver.dumb_destroy and drm_driver.dumb_map_offset > callbacks. > > Signed-off-by: Noralf Trønnes With Eric's suggestion: Reviewed-by: Sean Paul >

Re: [PATCH 01/41] drm/gem: Add drm_gem_dumb_map_offset()

2017-07-26 Thread Sean Paul
On Sun, Jul 23, 2017 at 09:16:17PM +0200, Noralf Trønnes wrote: > Add a common drm_driver.dumb_map_offset function for GEM backed drivers. > > Signed-off-by: Noralf Trønnes Reviewed-by: Sean Paul > --- > drivers/gpu/drm/drm_gem.c | 35 +++ > include/drm/drm_gem

Re: [PATCH 24/41] drm/rockchip: Use .dumb_map_offset and .dumb_destroy defaults

2017-07-26 Thread Sean Paul
On Sun, Jul 23, 2017 at 09:16:40PM +0200, Noralf Trønnes wrote: > This driver can use the drm_driver.dumb_destroy and > drm_driver.dumb_map_offset defaults, so no need to set them. > > Cc: Mark Yao > Signed-off-by: Noralf Trønnes > --- > drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 2 -- > dr

[PATCH 3/3] drm/etnaviv: add thermal dependency

2017-07-26 Thread Arnd Bergmann
When CONFIG_THERMAL is enabled as a loadable module, and etnaviv is built-in, we get a link error: drivers/gpu/drm/etnaviv/etnaviv_gpu.o: In function `etnaviv_gpu_bind': etnaviv_gpu.c:(.text.etnaviv_gpu_bind+0x34): undefined reference to `thermal_of_cooling_device_register' etnaviv_gpu.c:(.text.e

[PATCH RESEND] fbdev: omapfb: remove unused variable

2017-07-26 Thread Arnd Bergmann
Removing the default display name left a harmless warning: fbdev/omap2/omapfb/dss/core.c: In function 'omap_dss_probe': fbdev/omap2/omapfb/dss/core.c:196:30: error: unused variable 'pdata' [-Werror=unused-variable] This removes the now-unused variable as well. Fixes: 278cba7eaf54 ("drm: omapdrm

[PATCH 2/3] ACPI/DRM: rework ACPI_VIDEO Kconfig dependencies

2017-07-26 Thread Arnd Bergmann
ACPI_VIDEO keeps causing problems with circular Kconfig dependencies, as it depends on a couple of other symbols, and it gets selected by drivers that may end up being depending on others. This is an attempt to simplify this by changing all drivers that currently 'select ACPI_VIDEO' to use 'depend

[PATCH 1/3] backlight: always select BACKLIGHT_LCD_SUPPORT for BACKLIGHT_CLASS_DEVICE

2017-07-26 Thread Arnd Bergmann
randconfig builds occasionally produce this Kconfig warning: warning: (DRM_RADEON && DRM_AMDGPU && DRM_NOUVEAU && DRM_I915 && DRM_GMA500 && DRM_SHMOBILE && DRM_TILCDC && DRM_FSL_DCU && DRM_TINYDRM && DRM_PARADE_PS8622 && FB_BACKLIGHT && FB_ARMCLCD && FB_MX3 && USB_APPLEDISPLAY && FB_OLPC_DCON &&

[PATCH 0/3] Kconfig dependencies: acpi-video, backlight and thermal

2017-07-26 Thread Arnd Bergmann
Hi everyone, It took me a while to figure this out properly, as I kept getting circular or missing dependencies with video drivers. This set of three patches should simplify the situation a bit, mostly by cleaning up the dependencies around CONFIG_ACPI_VIDEO. With all three patches applied, I no

Re: [PATCH 01/14] amdgpu: powerplay: Remove unused function

2017-07-26 Thread Ricardo Ribalda Delgado
Hi Harry On Mon, Jul 24, 2017 at 5:45 PM, Harry Wentland wrote: > On 2017-07-24 10:06 AM, Ricardo Ribalda Delgado wrote: >> Hi Harry >> On Mon, Jul 24, 2017 at 4:01 PM, Harry Wentland >> wrote: >> >>> >>> This is used and needed by the DC display driver. See >>> display/amdgpu_dm/amdgpu_dm_serv

Re: [PATCH 01/14] amdgpu: powerplay: Remove unused function

2017-07-26 Thread Ricardo Ribalda Delgado
Hi Harry On Mon, Jul 24, 2017 at 5:45 PM, Harry Wentland wrote: > On 2017-07-24 10:06 AM, Ricardo Ribalda Delgado wrote: > > Hi Harry > > On Mon, Jul 24, 2017 at 4:01 PM, Harry Wentland > wrote: > > > >> > >> This is used and needed by the DC display driver. See > >> display/amdgpu_dm/amdgpu_dm

[PATCH 2/4] dma-buf: add reservation_object_copy_fences

2017-07-26 Thread Christian König
From: Christian König Allows us to copy all the fences in a reservation object to another one. Signed-off-by: Christian König --- drivers/dma-buf/reservation.c | 58 +++ include/linux/reservation.h | 3 +++ 2 files changed, 61 insertions(+) diff --gi

[PATCH 3/4] drm/ttm: remove nonsense wait in ttm_bo_cleanup_refs_and_unlock

2017-07-26 Thread Christian König
From: Christian König With shared reservation objects the assumption that no fence could have been added isn't true any more. Additional to that the BO is about to be destroyed, so removing the fences now has no advantage whatsoever. Signed-off-by: Christian König --- drivers/gpu/drm/ttm/ttm_

[PATCH 4/4] drm/ttm: individualize BO reservation obj when they are freed

2017-07-26 Thread Christian König
From: Christian König Use the BOs reservation object when it is put on the ddelete list. This way we avoid delaying freeing up the BO because of new fences on the reservation object. Signed-off-by: Christian König --- drivers/gpu/drm/ttm/ttm_bo.c | 54 ++---

[PATCH 1/4] dma-buf: dma_fence_put is NULL save

2017-07-26 Thread Christian König
From: Christian König No need to check. Signed-off-by: Christian König --- drivers/dma-buf/reservation.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/dma-buf/reservation.c b/drivers/dma-buf/reservation.c index 9d4316d..dfcd4ef3 100644 --- a/drivers/dma-buf/

Re: [PULL] drm-misc-next remix edition

2017-07-26 Thread Daniel Vetter
Hi Dave, On Wed, Jul 26, 2017 at 2:27 PM, Sean Paul wrote: > Hi Dave, > Here's the revised pull from -misc-next with fixed history. It's mostly > comprised of last week's PR, but has a few changes on top that I've summarized > as well. > > Once you pull this, I'll backmerge/ff to pick up the stm

[Bug 101926] kabini amdgpu: LVDS not working after resume from suspend (works with radeon)

2017-07-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101926 --- Comment #1 from Chí-Thanh Christopher Nguyễn --- Created attachment 132985 --> https://bugs.freedesktop.org/attachment.cgi?id=132985&action=edit dmesg -- You are receiving this mail because: You are the assignee for the bug._

[PATCH v3 2/2] drm/rockchip: cdn-dp: send audio infoframe to sink

2017-07-26 Thread Chris Zhong
Some DP/HDMI sink need to receive the audio infoframe to play sound, especially some multi-channel AV receiver, they need the channel_allocation from infoframe to config the speakers. Send the audio infoframe via SDP will make them work properly. Signed-off-by: Chris Zhong --- Changes in v3: No

[Bug 101926] kabini amdgpu: LVDS not working after resume from suspend (works with radeon)

2017-07-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101926 Bug ID: 101926 Summary: kabini amdgpu: LVDS not working after resume from suspend (works with radeon) Product: DRI Version: unspecified Hardware: Other OS:

[PULL] drm-misc-next remix edition

2017-07-26 Thread Sean Paul
Hi Dave, Here's the revised pull from -misc-next with fixed history. It's mostly comprised of last week's PR, but has a few changes on top that I've summarized as well. Once you pull this, I'll backmerge/ff to pick up the stm changes which will hopefully stop the spread of the rebase/bad history.

Re: linux-next: unsigned commits in the drm-misc tree

2017-07-26 Thread Daniel Vetter
On Wed, Jul 26, 2017 at 9:09 AM, Daniel Vetter wrote: > Oops, that shouldn't have happened. Actually, our maintainer tooling > ensures this doesn't happen, by auto-adding the committer sob line. > But these patches (and a bunch of others pushed by Benjamin) haven't > been pushed by our tooling it

Re: [PATCH v2] dim: add checks for author and committer sign-off-by

2017-07-26 Thread Daniel Vetter
On Wed, Jul 26, 2017 at 02:07:49PM +0200, Benjamin Gaignard wrote: > Add checks for author and committer sign-off-by. > Add a check for "Link" tag added by dim apply-branch. > > Use real names for people with many different email addresses. > > Signed-off-by: Benjamin Gaignard > --- > version 2:

[PATCH v2] dim: add checks for author and committer sign-off-by

2017-07-26 Thread Benjamin Gaignard
Add checks for author and committer sign-off-by. Add a check for "Link" tag added by dim apply-branch. Use real names for people with many different email addresses. Signed-off-by: Benjamin Gaignard --- version 2: - keep the same logic than before for reviewed/acked-by checks - use real names

Re: [PATCH 01/41] drm/gem: Add drm_gem_dumb_map_offset()

2017-07-26 Thread Emil Velikov
Hi Noralf, On 23 July 2017 at 20:16, Noralf Trønnes wrote: > Add a common drm_driver.dumb_map_offset function for GEM backed drivers. > > Signed-off-by: Noralf Trønnes > --- > drivers/gpu/drm/drm_gem.c | 35 +++ > include/drm/drm_gem.h | 2 ++ > 2 files cha

Re: [PATCH v2 3/7] drm/rcar-du: Use new iterator macros, v2.

2017-07-26 Thread Laurent Pinchart
Hi Maarten, Thank you for the patch. On Wednesday 19 Jul 2017 16:39:16 Maarten Lankhorst wrote: > for_each_obj_in_state is about to be removed, so use the correct new > iterator macros. > > Also look at new_plane_state instead of plane->state when looking up > the hw planes in use. They should b

Re: [PATCH v2 3/3] drm/vc4: Convert more lock requirement comments to lockdep assertions.

2017-07-26 Thread Daniel Vetter
On Tue, Jul 25, 2017 at 11:27:18AM -0700, Eric Anholt wrote: > Since I do my development with lockdep on, this will help make sure I > don't introduce bugs here. Yeah it's somewhat annoying that lockdep_assert_held doesn't reduce to a "no false positives, maybe some false negatives, but real fast"

Re: [PATCH v2 1/3] drm/vc4: Start using u64_to_user_ptr.

2017-07-26 Thread Daniel Vetter
On Tue, Jul 25, 2017 at 11:27:16AM -0700, Eric Anholt wrote: > Chris Wilson pointed out this little cleanup in a review of new code, > so let's fix up the code I was copying from. > > Signed-off-by: Eric Anholt Reviewed-by: Daniel Vetter > --- > drivers/gpu/drm/vc4/vc4_gem.c | 11 +--

Re: [PATCH v2] drm/hisilicon: fix build error without fbdev emulation

2017-07-26 Thread Daniel Vetter
On Wed, Jul 26, 2017 at 08:58:16AM +0200, Arnd Bergmann wrote: > On Wed, Jul 26, 2017 at 7:55 AM, Daniel Vetter wrote: > > On Tue, Jul 25, 2017 at 8:05 PM, Arnd Bergmann wrote: > >> We cannot reference priv->fbdev outside of the #ifdef: > >> > >> drivers/net/virtio_net.c:1881:12: error: 'virtnet_

Re: [PATCHv1 07/14] drm/omap: add support for physical size hints from display drivers

2017-07-26 Thread Pavel Machek
On Mon 2017-07-24 19:33:04, Sebastian Reichel wrote: > While physical size information is automatically parsed for EDID > based displays, we need to provide it manually for displays providing > one fixed mode. > > Signed-off-by: Sebastian Reichel Acked-by: Pavel Machek -- (english) http://www

Re: [PATCHv1 14/14] ARM: dts: n950: add display support

2017-07-26 Thread Pavel Machek
On Mon 2017-07-24 19:33:11, Sebastian Reichel wrote: > Add basic panel support for the Nokia N950. It must be tweaked a > little bit later, since the panel was built into the device > upside-down. Also the first 5 and the last 5 pixels are covered > by plastic. > > Signed-off-By: Sebastian Reichel

Re: [PATCHv1 02/14] drm/omap: drop incorrect comment

2017-07-26 Thread Pavel Machek
On Mon 2017-07-24 19:32:59, Sebastian Reichel wrote: > The wrappers have been removed in commit 5a35876e2830 > (drm: omapdrm: Remove manual update display support) > and will not be reintroduced, since the normal sys > functions properly call the dirty callback. > > Signed-off-by: Sebastian Reiche

Re: [PATCH] dim: add checks for author and committer sign-off-by

2017-07-26 Thread Daniel Vetter
On Wed, Jul 26, 2017 at 12:27 PM, Benjamin Gaignard wrote: > Rework checkpatch_commit_push logic to check for author and > committer sign-off-by. > Also add a check for "Link" tag added by dim apply-branch. > > Remove the committer != author check because obviously some > authors commit their patc

Re: [PATCHv1 01/14] drm/omap: remove unused function defines

2017-07-26 Thread Pavel Machek
On Mon 2017-07-24 19:32:58, Sebastian Reichel wrote: > Remove driver (un)register API defines. They do not even exist > anymore. > > Signed-off-by: Sebastian Reichel Acked-by: Pavel Machek > --- > drivers/gpu/drm/omapdrm/dss/omapdss.h | 3 --- > 1 file changed, 3 deletions(-) > > diff --git

[PATCH] dim: add checks for author and committer sign-off-by

2017-07-26 Thread Benjamin Gaignard
Rework checkpatch_commit_push logic to check for author and committer sign-off-by. Also add a check for "Link" tag added by dim apply-branch. Remove the committer != author check because obviously some authors commit their patches themselves. This test was never done anyway because the first check

  1   2   >