Re: [PULL] drm-intel-next-fixes

2018-10-18 Thread Joonas Lahtinen
Quoting Daniel Vetter (2018-10-18 22:32:00) > On Thu, Oct 18, 2018 at 6:57 PM Joonas Lahtinen > wrote: > > > > Hi Dave, > > > > Here comes the final set of fixes under -next-fixes umbrella. > > Next one will be then from -fixes, assuming a release next Sun. > > > > Fixes for bunch of display relat

Re: [PATCH 7/9] arm64: allwinner: a64: enable ANX6345 bridge on Pinebook

2018-10-18 Thread Icenowy Zheng
于 2018年10月18日 GMT+08:00 下午11:17:35, Vasily Khoruzhick 写到: >Hi Icenowy, > >On Thursday, October 18, 2018 12:33:25 AM PDT Icenowy Zheng wrote: >> Pinebook has an ANX6345 bridge connected to the RGB666 LCD output, >and >> the I2C controlling signals are connected to R_I2C bus. >> >> Enable it in t

[PATCH v2 3/6] drm/virtio: use virtio_gpu_object->hw_res_handle in virtio_gpu_mode_dumb_create()

2018-10-18 Thread Gerd Hoffmann
Drop pointless resid variable in virtio_gpu_mode_dumb_create(), just use the hw_res_handle field in virtio_gpu_object directly. Signed-off-by: Gerd Hoffmann --- drivers/gpu/drm/virtio/virtgpu_gem.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/virtio

[PATCH v2 6/6] drm/virtio: fix resource id handling

2018-10-18 Thread Gerd Hoffmann
Move virtio_gpu_resource_id_{get,put} to virtgpu_object.c and make them static. Allocate and free the id on creation and destroy, drop all other calls. That way objects have a valid handle for the whole lifetime of the object. Also fixes ids leaking. Worst offender are dumb buffers, and I think

[PATCH v2 4/6] drm/virtio: use virtio_gpu_object->hw_res_handle in virtio_gpu_resource_create_ioctl()

2018-10-18 Thread Gerd Hoffmann
Drop pointless res_id variable in virtio_gpu_resource_create_ioctl(), just use the hw_res_handle field in virtio_gpu_object directly. Signed-off-by: Gerd Hoffmann --- drivers/gpu/drm/virtio/virtgpu_ioctl.c | 24 1 file changed, 8 insertions(+), 16 deletions(-) diff --gi

Re: [Outreachy kernel] [PATCH DRM] drm: msm: adreno: Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) +PTR_ERR

2018-10-18 Thread Julia Lawall
On Fri, 19 Oct 2018, Mamta Shukla wrote: > Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR because its > better to have inlined function rather than code-opened implementation. Just for your information, people have mixed feelings about this. In your case, it's probably fine, becaus

Re: [PATCH 2/3] drm/amdgpu: Replace TTM initialization/release with ttm_global

2018-10-18 Thread Zhang, Jerry(Junwei)
On 10/19/2018 12:27 AM, Thomas Zimmermann wrote: Unified initialization and relesae of the global TTM state is provided by struct ttm_global and its interfaces. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 63 ++--- drivers/gpu/drm/amd/am

[PATCH v2 5/6] drm/virtio: drop resource_id argument.

2018-10-18 Thread Gerd Hoffmann
We pass the obj anyway, so obj->hw_res_handle can be used instead in virtio_gpu_object_attach() and virtio_gpu_cmd_create_resource(). Signed-off-by: Gerd Hoffmann --- drivers/gpu/drm/virtio/virtgpu_drv.h | 2 -- drivers/gpu/drm/virtio/virtgpu_fb.c| 4 ++-- drivers/gpu/drm/virtio/virtgpu_ge

[PATCH v2 0/6] drm/virtio: rework ttm resource handling.

2018-10-18 Thread Gerd Hoffmann
Trying to put the virtio-gpu driver's ttm resource handling into shape. Also improve the resource id handling. v2: * drop patches 7+8, back to the drawing board, they'll come with a separate patch series later. * reorder patches. Gerd Hoffmann (6): drm/virtio: track created object state

[PATCH v2 2/6] drm/virtio: use virtio_gpu_object->hw_res_handle in virtio_gpufb_create()

2018-10-18 Thread Gerd Hoffmann
Drop pointless resid variable in virtio_gpufb_create(), just use the hw_res_handle field in virtio_gpu_object directly. Signed-off-by: Gerd Hoffmann --- drivers/gpu/drm/virtio/virtgpu_fb.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/virtio/virtgpu_

[PATCH v2 1/6] drm/virtio: track created object state

2018-10-18 Thread Gerd Hoffmann
Track whenever the virtio_gpu_object is already created (i.e. host knows about it) in a new variable. Add checks to virtio_gpu_object_attach() to do nothing on objects not created yet. Make virtio_gpu_ttm_bo_destroy() use the new variable too, instead of expecting hw_res_handle indicating the obj

[git pull] drm fixes for 4.19 final

2018-10-18 Thread Dave Airlie
Hi Greg, Just a last set of misc core fixes for final. 4 fixes, one use after free, one fb integration fix, one EDID fix, and one laptop panel quirk, Thanks, Dave. drm-fixes-2018-10-19: drm: one set of misc fixes for final release The following changes since commit 35a7f35ad1b150ddf59a41dcac7b2

Re: [PATCH 3/3] drm/radeon: Replace TTM initialization/release with ttm_global

2018-10-18 Thread Huang Rui
On Fri, Oct 19, 2018 at 12:27:52AM +0800, Thomas Zimmermann wrote: > Unified initialization and release of the global TTM state is provided > by struct ttm_global and its interfaces. > > Signed-off-by: Thomas Zimmermann Reviewed-by: Huang Rui > --- > drivers/gpu/drm/radeon/radeon.h | 4 +

Re: [PATCH 2/3] drm/amdgpu: Replace TTM initialization/release with ttm_global

2018-10-18 Thread Huang Rui
On Fri, Oct 19, 2018 at 12:27:51AM +0800, Thomas Zimmermann wrote: > Unified initialization and relesae of the global TTM state is provided > by struct ttm_global and its interfaces. > > Signed-off-by: Thomas Zimmermann Reviewed-by: Huang Rui > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c |

Re: [PATCH 1/3] drm/ttm: Provide struct ttm_global for referencing TTM global state

2018-10-18 Thread Huang Rui
On Fri, Oct 19, 2018 at 12:27:50AM +0800, Thomas Zimmermann wrote: > The new struct ttm_global provides drivers with TTM's global memory and > BO in a unified way. Initialization and release is handled internally. > > The functionality provided by struct ttm_global is currently re-implemented > by

Re: [PATCH 2/7] drm: add syncobj timeline support v8

2018-10-18 Thread zhoucm1
On 2018年10月18日 19:50, Christian König wrote: Am 18.10.18 um 05:11 schrieb zhoucm1: On 2018年10月17日 18:24, Daniel Vetter wrote: On Wed, Oct 17, 2018 at 11:29 AM Koenig, Christian wrote: Am 17.10.18 um 11:17 schrieb zhoucm1: [SNIP]    +struct drm_syncobj_signal_pt { +    struct dma_fence_a

[Bug 106177] overclocking doesn't work with 4.17-rc1

2018-10-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=106177 --- Comment #11 from Alex Deucher --- (In reply to alvarex from comment #10) > check here > https://wiki.archlinux.org/index.php/AMDGPU#Overclocking > > and it's documented on the kernel source The arch wiki has some errors in it as well. He

[Bug 107928] Screen regularly turns black, reboot needed

2018-10-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107928 --- Comment #11 from Matthew Vaughn --- Downgrading from x11-drivers/xf86-video-amdgpu-18.1.0 to x11-drivers/xf86-video-amdgpu-18.0.1 has prevented the issue from occurring on my system. -- You are receiving this mail because: You are the assi

Re: [PATCH 7/8] drm/virtio: move virtio_gpu_object_{attach, detach} calls.

2018-10-18 Thread Dave Airlie
On Thu, 18 Oct 2018 at 17:00, Gerd Hoffmann wrote: > > > > > This to me feels more like a bind/unbind operation rather than a > > > > populate/unpopulate operation, > > > > > > > > bind is " Bind the backend pages into the aperture in the location" > > > > > > > > whereas populate is > > > > > > >

[Bug 107990] Got Dying Light working in Arch by changing Mesa's compile steps, how to get it working Out Of the Box?

2018-10-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107990 --- Comment #5 from Timothy Arceri --- (In reply to Thomas Crider from comment #4) > Just an update on this, you don't need to change Arch's compiler flags. > I compiled mesa with the default flags and used glvnd=false, then used > LD_PRELOAD fo

[Bug 108340] Ambient Occlusion in Two Point Hospital shows black spot artifacts

2018-10-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108340 Timothy Arceri changed: What|Removed |Added Status|NEW |NEEDINFO --- Comment #1 from Timothy A

[Bug 108111] Civilization VI Artifacts RX480

2018-10-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108111 Timothy Arceri changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

Re: [PATCH 04/11] drm/msm/dpu: Remove dpu_crtc_is_enabled()

2018-10-18 Thread Bruce Wang
On Thu, Oct 18, 2018 at 3:59 PM Jordan Crouse wrote: > > The static inline function dpu_crtc_enabled() is only called once > and the function that calls it in turn is only called once and > the return value can be easily checked in the calling functions > so collapse everything down. > > Signed-of

[PATCH DRM] drm: msm: Replace instances of *_reference/unreference() with *_get/put()

2018-10-18 Thread Mamta Shukla
Replace instances of *_reference/unreference() with *_get/put(), to maintain consistency with the kernel use of *_get/put suffixes and get/put() is shorter. Issue found with drm-get-put.cocci: WARNING: use get/put helpers to reference and dereference state Path to the Coccinelle script: scripts/

Re: [PATCH 7/9] arm64: allwinner: a64: enable ANX6345 bridge on Pinebook

2018-10-18 Thread Vasily Khoruzhick
Hi Icenowy, On Thursday, October 18, 2018 12:33:25 AM PDT Icenowy Zheng wrote: > Pinebook has an ANX6345 bridge connected to the RGB666 LCD output, and > the I2C controlling signals are connected to R_I2C bus. > > Enable it in the device tree, and add a usable EDID from the panel's > datasheet (a

[PATCH] drm/radeon: fix a missing-check bug

2018-10-18 Thread Wenwen Wang
In radeon_read_bios(), the bios rom is firstly mapped to the IO memory region 'bios' through pci_map_rom(). Then the first two bytes of 'bios' are copied to 'val1' and 'val2' respectively through readb(). After that, 'val1' and 'val2' are checked to see whether they have expected values, i.e., 0x55

Re: [PATCH 03/11] drm/msm/dpu: Remove dpu_crtc_get_mixer_height

2018-10-18 Thread Bruce Wang
On Thu, Oct 18, 2018 at 3:59 PM Jordan Crouse wrote: > > dpu_crtc_get_mixer_height() is only used once and the value it > returns can be easily derived from the calling function. > > Signed-off-by: Jordan Crouse Reviewed-by: Bruce Wang > --- > drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 3 +--

[PATCH DRM] drm: msm: adreno: Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) +PTR_ERR

2018-10-18 Thread Mamta Shukla
Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR because its better to have inlined function rather than code-opened implementation. Signed-off-by: Mamta Shukla --- drivers/gpu/drm/msm/adreno/a5xx_gpu.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/gpu/

[PATCH] drm/radeon: fix a missing-check bug

2018-10-18 Thread Wenwen Wang
In igp_read_bios_from_vram(), the start of vram is firstly remapped to the IO memory region 'bios' through ioremap(). Then the size and values of 'bios' are checked. For example, 'bios[0]' is compared against 0x55 and 'bios[1]' is compared against 0xaa. If no error happens during this checking proc

kmemleak in nvkm_iccsense_oneinit

2018-10-18 Thread Jaejoong Kim
## I forgot the write email without checking the enable text mode. So, send email again. Sorry for send E-mail twice. Hi I got a kmemleak and it is pointing to nvkm_iccsense_oneinit(). Please refer to the below kmemleak message. # uname -a Linux climb 4.19.0-rc1+ #4 SMP Thu Sep 20 11:48:55 KST 2

[PATCH DRM] drm: msm: adreno: Use DRM_DEV_* instead of dev_*

2018-10-18 Thread Mamta Shukla
Use DRM_DEV_INFO/ERROR/WARN instead of dev_info/err/debug to generate drm-formatted specific log messages so that it will be easy to differentiate in case of multiple instances of driver. Signed-off-by: Mamta Shukla --- drivers/gpu/drm/msm/adreno/a5xx_gpu.c | 10 +- 1 file changed, 5 ins

Re: [PATCH 02/11] drm/msm/dpu: Use DEFINE_SHOW_ATTRIBUTE

2018-10-18 Thread Bruce Wang
On Thu, Oct 18, 2018 at 3:58 PM Jordan Crouse wrote: > > Use the standard DEFINE_SHOW_ATTRIBUTE macro for seq_file based > debugfs files instead of custom macros and hand-coded functions. > > Signed-off-by: Jordan Crouse Sorry! Suppressed CCs by accident Reviewed-by: Bruce Wang > --- > drive

[Bug 108464] System fails to reboot after Ctrl-Alt-Del

2018-10-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108464 --- Comment #5 from Duncan Roe --- Still present at 4.19.0-rc8. Is there any other info I can provide? -- You are receiving this mail because: You are the assignee for the bug.___ dri-devel mailing l

[Bug 106177] overclocking doesn't work with 4.17-rc1

2018-10-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=106177 --- Comment #10 from alvarex --- check here https://wiki.archlinux.org/index.php/AMDGPU#Overclocking and it's documented on the kernel source -- You are receiving this mail because: You are the assignee for the bug.__

[Bug 106177] overclocking doesn't work with 4.17-rc1

2018-10-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=106177 alvarex changed: What|Removed |Added Resolution|--- |NOTABUG Status|NEW

Re: omap4: support for manually updated display

2018-10-18 Thread Pavel Machek
Hi! > > I want to make it clear that I don't want to claim any privilege in getting > > patches merged first. I am however worried that, without an easy way to > > test > > DSI support, and without enough time to focus on it, I would break whatever > > would be merged now in future reworks. I

Re: [PATCH 10/11] drm/msm/dpu: Further cleanups for static inline functions

2018-10-18 Thread Sam Ravnborg
Hi Jordan. On Thu, Oct 18, 2018 at 01:58:35PM -0600, Jordan Crouse wrote: > Remove more static inline functions that are lightly used and/or > very simple and easy to build into the calling functions. Please explain the background/analysis that resulted in dropping dpu_crtc_get_client_type(), ass

Re: [PATCH 06/11] drm/msm/dpu: Cleanup callers of dpu_hw_blk_init

2018-10-18 Thread Sam Ravnborg
Hi Jordan. On Thu, Oct 18, 2018 at 01:58:31PM -0600, Jordan Crouse wrote: > Outside of superfluous parameter checks the dpu_hw_blk_init() > doesn't have any failure paths. Switch it over to be a void > function and we can remove error handling paths in all the functions > that call it. While we're

Re: [PATCH 05/11] drm/msm/dpu: Cleanup some container_of helper functions

2018-10-18 Thread Sam Ravnborg
Hi Jordan. > -/** > - * to_dpu_hw_mixer - convert base object dpu_hw_base to container > - * @hw: Pointer to base hardware block > - * return: Pointer to hardware block container > - */ > -static inline struct dpu_hw_mixer *to_dpu_hw_mixer(struct dpu_hw_blk *hw) > -{ > - return container_of(hw

Re: [PATCH 1/2 v2] drm/panel: Add DT bindings for Samsung S6D16D0

2018-10-18 Thread Rob Herring
On Tue, 16 Oct 2018 14:34:08 +0200, Linus Walleij wrote: > This adds the device tree bindings for the Samsung S6D16D0 > panel. This is a command mode only panel using DSI. > > Cc: devicet...@vger.kernel.org > Cc: Sam Ravnborg > Cc: Andrzej Hajda > Signed-off-by: Linus Walleij > --- > ChangeLog

[PATCH libdrm 2/2] xf86drm: handle non existing OF_COMPATIBLE_N uevent entry

2018-10-18 Thread Christian Gmeiner
Default the count value to 0. Signed-off-by: Christian Gmeiner --- xf86drm.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/xf86drm.c b/xf86drm.c index 4ee1337b..a9152f0c 100644 --- a/xf86drm.c +++ b/xf86drm.c @@ -3537,17 +3537,16 @@ static int drmParsePlatformDe

[PATCH libdrm 1/2] xf86drm: fallback to DRIVER uevent entry when OF_FULLNAME fails

2018-10-18 Thread Christian Gmeiner
Signed-off-by: Christian Gmeiner --- xf86drm.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/xf86drm.c b/xf86drm.c index 10df682b..4ee1337b 100644 --- a/xf86drm.c +++ b/xf86drm.c @@ -3516,6 +3516,8 @@ static int drmParsePlatformBusInfo(int maj, int min, drmPlatformBusInfoPtr info) s

[PATCH libdrm 0/2] xf86drm: add OF_ fallback mechanism

2018-10-18 Thread Christian Gmeiner
Since kernel 4.17 (drm/etnaviv: remove the need for a gpu-subsystem DT node) the etnaviv DRM driver doesn't have an associated DT node anymore. This is technically correct, as the etnaviv device is a virtual device driving multiple hardware devices. Before 4.17 the userspace had access to the foll

[PATCH 10/11] drm/msm/dpu: Further cleanups for static inline functions

2018-10-18 Thread Jordan Crouse
Remove more static inline functions that are lightly used and/or very simple and easy to build into the calling functions. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c | 17 ++ drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 14 +++- drivers/gpu/drm/

[PATCH 11/11] drm/msm/dpu: Clean up dpu_media_info.h static inline functions

2018-10-18 Thread Jordan Crouse
Do some cleanup in the static inline functions defined in dpu_media_info.h by cleaning up gotos and unneeded local variables. Signed-off-by: Jordan Crouse --- .../gpu/drm/msm/disp/dpu1/msm_media_info.h| 164 ++ 1 file changed, 57 insertions(+), 107 deletions(-) diff --git a/

[PATCH 08/11] drm/msm/dpu: Remove dpu_irq and unused functions

2018-10-18 Thread Jordan Crouse
dpu_irq.c does some unneeded checks and passes control to dpu_core_irq.c The simple functions can be defined in the same file where we use them and the files and their associated hangers on can be deleted. Additionally the postinstall hook isn't used even in dpu_core_irq.c so zap that entire path

[PATCH 03/11] drm/msm/dpu: Remove dpu_crtc_get_mixer_height

2018-10-18 Thread Jordan Crouse
dpu_crtc_get_mixer_height() is only used once and the value it returns can be easily derived from the calling function. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 3 +-- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h | 13 - 2 files changed, 1 insertion(+)

[PATCH 06/11] drm/msm/dpu: Cleanup callers of dpu_hw_blk_init

2018-10-18 Thread Jordan Crouse
Outside of superfluous parameter checks the dpu_hw_blk_init() doesn't have any failure paths. Switch it over to be a void function and we can remove error handling paths in all the functions that call it. While we're in those functions remove unneeded initialization for a static variable. Signed-o

[PATCH 07/11] drm/msm: Make irq_postinstall optional

2018-10-18 Thread Jordan Crouse
Allow the KMS operation 'irq_postinstall' to be optional so that the target display drivers don't need to define a dummy function if they don't need one. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/msm/msm_drv.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/driver

[PATCH 05/11] drm/msm/dpu: Cleanup some container_of helper functions

2018-10-18 Thread Jordan Crouse
Cleanup a number of static inline container_of() helper functions because they are either unused or only used in one file and can be performed inline or with a simple #define. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.h | 10 -- drivers/gpu/drm/msm/di

[PATCH 09/11] drm/msm/dpu: Debugfs related cleanups

2018-10-18 Thread Jordan Crouse
Do some cleanups to the code related to debugfs files from across the DPU driver including removing destroy functions, tweeks to the initialization functions, removal of unused static inline stubs and skipping unneeded null checks. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/msm/disp/dpu1/d

[PATCH 00/11] DPU cleanups

2018-10-18 Thread Jordan Crouse
I've been working on various methods to automate code cleanup strategies and I'm using dpu as my guinea pig. I started out by trying to identify unused or lightly static inline functions and then that morphed to very small functions in general and that then identified a few general areas of cleanu

[PATCH 04/11] drm/msm/dpu: Remove dpu_crtc_is_enabled()

2018-10-18 Thread Jordan Crouse
The static inline function dpu_crtc_enabled() is only called once and the function that calls it in turn is only called once and the return value can be easily checked in the calling functions so collapse everything down. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/msm/disp/dpu1/dpu_core_pe

[PATCH 02/11] drm/msm/dpu: Use DEFINE_SHOW_ATTRIBUTE

2018-10-18 Thread Jordan Crouse
Use the standard DEFINE_SHOW_ATTRIBUTE macro for seq_file based debugfs files instead of custom macros and hand-coded functions. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/msm/disp/dpu1/dpu_core_irq.c | 14 +--- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 31 +++--- driv

[PATCH 01/11] drm/msm/dpu: Remove dpu_dbg

2018-10-18 Thread Jordan Crouse
The functions in dpu_dbg.c aren't used. The two main dump functions fail after a lookup from dpu_dbg_base.reg_base_list which turns out to never be populated and once those are removed the rest of the file doesn't make any sense. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/msm/Makefile

[PATCH] drm: panel-orientation-quirks: Do rotation quirk for new GPD Win2 FW

2018-10-18 Thread Gabriel Krisman Bertazi
I just got a new GDP Win2 device with an updated firmware, which still requires this quirk to get the rotation right, so add the new firmware date to the quirk matching table. This should go to drm-misc-next. Signed-off-by: Gabriel Krisman Bertazi --- drivers/gpu/drm/drm_panel_orientation_quirk

Re: [PULL] drm-intel-next-fixes

2018-10-18 Thread Daniel Vetter
On Thu, Oct 18, 2018 at 6:57 PM Joonas Lahtinen wrote: > > Hi Dave, > > Here comes the final set of fixes under -next-fixes umbrella. > Next one will be then from -fixes, assuming a release next Sun. > > Fixes for bunch of display related issues reported by users, then the > MST fixes that were dr

Re: [PATCH v2] drm: Get ref on CRTC commit object when waiting for flip_done

2018-10-18 Thread Wentland, Harry
On 2018-10-18 1:38 p.m., Wentland, Harry wrote: > On 2018-10-16 11:48 a.m., Alex Deucher wrote: >> On Tue, Oct 16, 2018 at 11:00 AM Li, Sun peng (Leo) >> wrote: >>> >>> >>> >>> On 2018-10-16 08:33 AM, Daniel Vetter wrote: On Mon, Oct 15, 2018 at 09:46:40AM -0400, sunpeng...@amd.com wrote: >>

[PATCH 3/3] drm/amdgpu: Refresh rq selection for job after ASIC reset

2018-10-18 Thread Andrey Grodzovsky
A ring might become unusable after reset, if that the case drm_sched_entity_select_rq will choose another, working rq to run the job if there is one. Also, skip recovery of ring which is not ready. Signed-off-by: Andrey Grodzovsky --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 16 +

[PATCH 1/3] drm/sched: Add callback to mark if sched is ready to work.

2018-10-18 Thread Andrey Grodzovsky
Problem: A particular scheduler may become unsuable (underlying HW) after some event (e.g. GPU reset). If it's later chosen by the get free sched. policy a command will fail to be submitted. Fix: Add a driver specific callback to report the sced. status so rq with bad sched. can be avoided in favo

[PATCH 2/3] drm/sched: Expose drm_sched_entity_select_rq to use in drivers.

2018-10-18 Thread Andrey Grodzovsky
Signed-off-by: Andrey Grodzovsky --- drivers/gpu/drm/scheduler/sched_entity.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/scheduler/sched_entity.c b/drivers/gpu/drm/scheduler/sched_entity.c index 320c77a..426606c 100644 --- a/drivers/gpu/drm/scheduler/sched_entity.c +++ b

Re: [PATCH] drm/vboxvideo: Move vboxvideo driver out of staging

2018-10-18 Thread Sam Ravnborg
Hi Hans. Just a bunch of random observations that I hope you find use of. Sam > diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig > index cb88528e7b10..6b4d6c957da8 100644 > --- a/drivers/gpu/drm/Kconfig > +++ b/drivers/gpu/drm/Kconfig > @@ -315,6 +315,8 @@ source "drivers/g

[Bug 108487] Wayland compositors are unable to use hardware acceleration on i915

2018-10-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108487 Bug ID: 108487 Summary: Wayland compositors are unable to use hardware acceleration on i915 Product: Mesa Version: 18.2 Hardware: Other OS: Linux (All)

Re: [PATCH v2] drm: Get ref on CRTC commit object when waiting for flip_done

2018-10-18 Thread Wentland, Harry
On 2018-10-16 11:48 a.m., Alex Deucher wrote: > On Tue, Oct 16, 2018 at 11:00 AM Li, Sun peng (Leo) > wrote: >> >> >> >> On 2018-10-16 08:33 AM, Daniel Vetter wrote: >>> On Mon, Oct 15, 2018 at 09:46:40AM -0400, sunpeng...@amd.com wrote: From: Leo Li This fixes a general protectio

[Bug 108486] Incompatible result.json generation between igt_runner and piglit summary

2018-10-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108486 --- Comment #1 from Andi --- the incompatible result.json happens also without the -d (dry-run) and with different test list inputs. I guess the igt_runner generates always an incompatible result.json -- You are receiving this mail because: Yo

Re: [PATCH v5 24/28] drm/i915/dp: Configure Display stream splitter registers during DSC enable

2018-10-18 Thread Ville Syrjälä
On Fri, Oct 05, 2018 at 04:23:02PM -0700, Manasi Navare wrote: > Display Stream Splitter registers need to be programmed to enable > the joiner if two DSC engines are used and also to enable > the left and the right DSC engines. This happens as part of > the DSC enabling routine in the source in at

[PULL] drm-intel-next-fixes

2018-10-18 Thread Joonas Lahtinen
Hi Dave, Here comes the final set of fixes under -next-fixes umbrella. Next one will be then from -fixes, assuming a release next Sun. Fixes for bunch of display related issues reported by users, then the MST fixes that were dropped from Rodrigos PR + further Icelake fixes and proactive improveme

[PULL] drm-misc-fixes

2018-10-18 Thread Maarten Lankhorst
drm-misc-fixes-2018-10-18: drm-misc-fixes for v4.19: - Fix use of freed memory in drm_mode_setcrtc. - Reject pixel format changing requests in fb helper. - Add 6 bpc quirk for HP Pavilion 15-n233sl - Fix VSDB yCBCr420 Deep Color mode bit definitions The following changes since commit 4d4c2d89913e2d

[PATCH 3/3] drm/radeon: Replace TTM initialization/release with ttm_global

2018-10-18 Thread Thomas Zimmermann
Unified initialization and release of the global TTM state is provided by struct ttm_global and its interfaces. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/radeon/radeon.h | 4 +-- drivers/gpu/drm/radeon/radeon_ttm.c | 40 - 2 files changed, 7 insertions

[PATCH 1/3] drm/ttm: Provide struct ttm_global for referencing TTM global state

2018-10-18 Thread Thomas Zimmermann
The new struct ttm_global provides drivers with TTM's global memory and BO in a unified way. Initialization and release is handled internally. The functionality provided by struct ttm_global is currently re-implemented by a dozen individual DRM drivers using struct drm_global. The implementation o

[PATCH 2/3] drm/amdgpu: Replace TTM initialization/release with ttm_global

2018-10-18 Thread Thomas Zimmermann
Unified initialization and relesae of the global TTM state is provided by struct ttm_global and its interfaces. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 63 ++--- drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.h | 4 +- 2 files changed, 7 inserti

[PATCH 0/3] Provide struct ttm_global for TTM global state

2018-10-18 Thread Thomas Zimmermann
TTM provides global memory and a global BO that is shared by all TTM-based drivers. The data structures are provided by struct drm_global and its helpers. All TTM-based DRM drivers copy the initialization and clean-up code for the global TTM state from each other; leading to code duplication. The

[Bug 108484] wrong

2018-10-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108484 Chris Wilson changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[radeon-alex:amd-staging-dkms-4.18 839/1379] htmldocs: drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c:1542: warning: Function parameter or member 'mem' not described in 'amdgpu_vm_bo_split_mapping'

2018-10-18 Thread kbuild test robot
tree: git://people.freedesktop.org/~agd5f/linux.git amd-staging-dkms-4.18 head: 937a0a9df97d4c1c7cecf6faa2f501d175bc3c2a commit: 3a357f8fc9464b3341938cb3b68930989cb0e064 [839/1379] drm/amdgpu: [hybrid] add direct gma(dgma) support reproduce: make htmldocs All warnings (new ones prefixed by >>

[Bug 108486] Incompatible result.json generation between igt_runner and piglit summary

2018-10-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108486 Andi changed: What|Removed |Added Summary|Iincompatible result.json |Incompatible result.json |gene

[Bug 108486] Iincompatible result.json generation between igt_runner and piglit summary

2018-10-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108486 Bug ID: 108486 Summary: Iincompatible result.json generation between igt_runner and piglit summary Product: DRI Version: DRI git Hardware: x86-64 (AMD64) O

[PATCH] drm/scheduler: use hw_rq_count for load calculation

2018-10-18 Thread Nayan Deshmukh
If the hardware queue for a scheduler is empty then we don't need to the shift the entities from their current scheduler as they are not getting scheduled because of some dependency. Signed-off-by: Nayan Deshmukh --- drivers/gpu/drm/scheduler/sched_entity.c | 14 ++ 1 file changed, 1

[PATCH 0/1] drm/vboxvideo: Move vboxvideo driver out of staging

2018-10-18 Thread Hans de Goede
Hi All, Here is a patch wich adds a copy of the vboxvideo driver from staging (as it will look like after the cleanup series I just posted). The purpose of posting this as a copy rather then a patch just moving the files is to give the DRM maintainers a chance to do a review before it gets merged

[PATCH 3/6] staging: vboxvideo: Drop duplicate vbox_err.h file

2018-10-18 Thread Hans de Goede
Switch to the more complete vbox_err.h file from include/linux which got added with the merging of the vboxguest driver. Signed-off-by: Hans de Goede --- drivers/staging/vboxvideo/hgsmi_base.c | 4 +- drivers/staging/vboxvideo/modesetting.c | 4 +- drivers/staging/vboxvideo/vbox_err.h| 50

[PATCH 6/6] staging: vboxvideo: Stop disabling/enabling accel support on master set / drop

2018-10-18 Thread Hans de Goede
Userspace mode-setting (and thus also VESA) is not supported together with modesetting. KMS userspace apps not properly marking the framebuffer as dirty are also not supported. So stop trying to accommodate this and simply enable accel once at driver init. Signed-off-by: Hans de Goede --- drive

[PATCH 4/6] staging: vboxvideo: Cleanup the comments

2018-10-18 Thread Hans de Goede
Some comments where still using docbook style comments, move these either over to kerneldoc, or just make them regular comments. Also remove a bunch of obsolete comments. Signed-off-by: Hans de Goede --- drivers/staging/vboxvideo/hgsmi_base.c | 72 +--- drivers/staging/vboxvideo/h

[PATCH 5/6] staging: vboxvideo: Change licence headers over to SPDX

2018-10-18 Thread Hans de Goede
All the files contain a MIT license header, replace this with: SPDX-License-Identifier: MIT Signed-off-by: Hans de Goede --- drivers/staging/vboxvideo/hgsmi_base.c | 23 ++-- drivers/staging/vboxvideo/hgsmi_ch_setup.h | 23 ++-- drivers/staging/vboxvideo/hgs

[PATCH 1/6] staging: vboxvideo: Stop accessing crtc_state->active

2018-10-18 Thread Hans de Goede
Atomic modesetting drivers should never check crtc_state->active directly, instead check crtc_state->enable. Signed-off-by: Hans de Goede --- drivers/staging/vboxvideo/vbox_mode.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/vboxvideo/vbox_mode.c b/drivers

[PATCH 2/6] staging: vboxvideo: Keep old mode when disable crtc

2018-10-18 Thread Hans de Goede
On DPMS off we get a call to vbx_crtc_atomic_disable, followed by a call to vbox_primary_atomic_update, at which point crtc_state->enable is 0 and crtc_state->mode has been zero-ed. On a 0 width/height vbox_do_modeset() falls back to 640x480, so this causes the guest Window (its "monitor") to resiz

Re: [PATCH] drm/panel: Add simple panel mode for the ARM RTSMv8

2018-10-18 Thread Linus Walleij
On Thu, Oct 18, 2018 at 3:56 PM Robin Murphy wrote: > On 18/10/18 13:27, Linus Walleij wrote: > > Having failed any attempts at a more generic solution, > > I fall back to the very specific solution: define a simple > > panel for the ARM RTSMv8 emulated platform. > > From the bits of the discuss

[Bug 106490] VA-API video deconding broken for Chromium on Mesa 18.0.3

2018-10-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=106490 --- Comment #6 from Tomáš Chvátal --- This might be possible duplicate of: https://bugs.freedesktop.org/show_bug.cgi?id=104597 I can reproduce this too on openSUSE with Mesa 18.1. The issue itself is exposed only on Radeon driver not on intel/n

Re: [PATCH 9/9] [DO NOT MERGE] drm/sun4i: rgb: Add 5% tolerance to dot clock frequency check

2018-10-18 Thread Laurent Pinchart
Hi Maxime, On Thursday, 18 October 2018 15:18:19 EEST Maxime Ripard wrote: > On Thu, Oct 18, 2018 at 02:31:01PM +0300, Laurent Pinchart wrote: > > On Thursday, 18 October 2018 12:42:58 EEST Maxime Ripard wrote: > >> On Thu, Oct 18, 2018 at 11:18:06AM +0200, Daniel Vetter wrote: > >>> On Thu, Oct 1

Re: [PATCH] drm/panel: Add simple panel mode for the ARM RTSMv8

2018-10-18 Thread Robin Murphy
Hi Linus, On 18/10/18 13:27, Linus Walleij wrote: Having failed any attempts at a more generic solution, I fall back to the very specific solution: define a simple panel for the ARM RTSMv8 emulated platform. From the bits of the discussion that I've caught it does seem like this framebuffer-g

RE: [PATCH v2 2/2] drm/selftest: Add drm damage helper selftest

2018-10-18 Thread Deepak Singh Rawat
> On Tue, Oct 16, 2018 at 01:46:09PM -0700, Deepak Rawat wrote: > > Selftest for drm damage helper iterator functions. > > > > Cc: Daniel Vetter > > Cc: alexandru-cosmin.gheor...@arm.com > > Signed-off-by: Deepak Rawat > > Reviewed-by: Daniel Vetter > > Reviewed-by: Thomas Hellstrom > > I gues

[radeon-alex:amd-staging-dkms-4.18 805/1379] htmldocs: drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c:1542: warning: Function parameter or member 'vram_base_offset' not described in 'amdgpu_vm_bo_split_mappin

2018-10-18 Thread kbuild test robot
tree: git://people.freedesktop.org/~agd5f/linux.git amd-staging-dkms-4.18 head: 937a0a9df97d4c1c7cecf6faa2f501d175bc3c2a commit: 67f5976ee4842d01af79144a8355a040ed36b6d2 [805/1379] drm/amdkfd: Copy in non-KFD changes reproduce: make htmldocs All warnings (new ones prefixed by >>): WARNING

[Bug 201273] Fatal error during GPU init amdgpu RX560

2018-10-18 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=201273 quirin.blae...@freenet.de changed: What|Removed |Added Hardware|S390-31 |x86-64 -- You are receiving

[Bug 201273] Fatal error during GPU init amdgpu RX560

2018-10-18 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=201273 --- Comment #6 from quirin.blae...@freenet.de --- Created attachment 279089 --> https://bugzilla.kernel.org/attachment.cgi?id=279089&action=edit dmesg + amdgpu_pm_info New Monitor and HDMI-cable. Bug is not impressed - i.e. System hangs at boot

Re: [PATCH 3/3] arm64: dts: renesas: r8a77965: Add LVDS support

2018-10-18 Thread Simon Horman
On Thu, Oct 18, 2018 at 12:17:14PM +0100, Kieran Bingham wrote: > Hi Laurent, > > On 18/10/18 01:41, Laurent Pinchart wrote: > > The M3-N (r8a77965) platform has oneLVDS encoder connected to the DU. > > s/oneLVDS/one LVDS/ > > > Add the corresponding DT node and wire it up. > > > > Signed-off-b

Re: [PATCH 9/9] [DO NOT MERGE] drm/sun4i: rgb: Add 5% tolerance to dot clock frequency check

2018-10-18 Thread Icenowy Zheng
在 2018-10-18四的 14:18 +0200,Maxime Ripard写道: > On Thu, Oct 18, 2018 at 02:31:01PM +0300, Laurent Pinchart wrote: > > Hello, > > > > On Thursday, 18 October 2018 12:42:58 EEST Maxime Ripard wrote: > > > On Thu, Oct 18, 2018 at 11:18:06AM +0200, Daniel Vetter wrote: > > > > On Thu, Oct 18, 2018 at 10

[PATCH 1/9] drm/bridge: move ANA78xx driver to analogix subdirectory

2018-10-18 Thread Icenowy Zheng
As ANA78xx chips are designed and produced by Analogix Semiconductor, Inc, move their driver codes into analogix subdirectory. Signed-off-by: Icenowy Zheng --- drivers/gpu/drm/bridge/Kconfig | 10 -- drivers/gpu/drm/bridge/Makefile| 4 ++--

[PATCH v3 1/2] drm/exynos: decon: Make plane alpha configurable

2018-10-18 Thread Christoph Manszewski
The decon hardware supports variable plane alpha. Currently planes are opaque, make this configurable. Tested on TM2 with Exynos 5433 CPU, on top of exynos-drm-next, commit: c530174b90fa Signed-off-by: Christoph Manszewski --- v2 changes: - remove window blend property for the first (0) layer

Re: [PATCH 5/5] ARM: shmobile: defconfig: Enable SII902X

2018-10-18 Thread Simon Horman
On Mon, Sep 24, 2018 at 11:08:51AM +0200, Simon Horman wrote: > On Fri, Sep 21, 2018 at 07:08:31PM +0100, Fabrizio Castro wrote: > > The iwg23s board comes with the SiI9022ACNU HDMI transmitter, > > this patch makes sure the corresponding driver gets built. > > > > Signed-off-by: Fabrizio Castro

[PATCH 2/9] drm/bridge: split some definitions of ANX78xx to dedicated headers

2018-10-18 Thread Icenowy Zheng
Some definitions currently in analogix-anx78xx.h are not restricted to the ANX78xx series, but also applicable to other DisplayPort transmitters by Analogix. Split out them to dedicated headers, and make analogix-anx78xx.h include them. Signed-off-by: Icenowy Zheng --- .../drm/bridge/analogix/a

[PATCH v2 1/2] drm/exynos: decon: Make plane alpha configurable

2018-10-18 Thread Christoph Manszewski
The decon hardware supports variable plane alpha. Currently planes are opaque, make this configurable. Tested on TM2 with Exynos 5433 CPU, on top of exynos-drm-next, commit: c530174b90fa Signed-off-by: Christoph Manszewski --- v2 changes: - remove window blend property for the first (0) layer

Re: [PATCH 4/9] dt-bindings: Add ANX6345 DP/eDP transmitter binding

2018-10-18 Thread Icenowy Zheng
在 2018-10-18四的 14:23 +0300,Laurent Pinchart写道: > Hi Icenowy, > > On Thursday, 18 October 2018 13:00:05 EEST Icenowy Zheng wrote: > > 在 2018-10-18四的 11:53 +0300,Laurent Pinchart写道: > > > On Thursday, 18 October 2018 10:33:22 EEST Icenowy Zheng wrote: > > > > The ANX6345 is an ultra-low power Displa

  1   2   >