Re: [PATCH 0/2] drm/exynos/mixer: fixes for interlaced mode

2018-02-19 Thread Andrzej Hajda
Hi Inki, On 02.02.2018 16:11, Andrzej Hajda wrote: > Hi Inki, Tobias, > > This is reviewed/updated/tested Tobias's patch addressing page-faults > in Video Processor in interlaced mode. Plus preliminary patch fixing Mixer > interlaced mode. > > Regards > Andrzej Gentle ping. Regards Andrzej > >

Re: [PATCHv2 4/8] dt-bindings: panel: common: document orientation property

2018-02-19 Thread Sebastian Reichel
Hi, On Sun, Feb 18, 2018 at 05:24:24PM -0600, Rob Herring wrote: > On Thu, Feb 08, 2018 at 07:30:31PM +0100, Sebastian Reichel wrote: > > Introduce new "orientation" property for describing in which > > orientation a panel has been mounted to the device. This can > > be used by the operating

Re: [PATCH] s390/console: enable dummy console for vt

2018-02-19 Thread Farhan Ali
On 02/15/2018 07:02 AM, Christian Borntraeger wrote: On 02/15/2018 12:57 PM, Thomas Huth wrote: On 15.02.2018 12:26, Geert Uytterhoeven wrote: Hi Christian, On Thu, Feb 15, 2018 at 12:14 PM, Christian Borntraeger wrote: To enable the virtual terminal layer with

Re: [PATCH] drm/simple_kms_helper: Fix NULL pointer dereference with no active CRTC

2018-02-19 Thread Oleksandr Andrushchenko
On 02/19/2018 04:30 PM, Daniel Vetter wrote: On Tue, Feb 13, 2018 at 10:44:16AM +0200, Oleksandr Andrushchenko wrote: From: Oleksandr Andrushchenko It is possible that drm_simple_kms_plane_atomic_check called with no CRTC set, e.g. when user-space application

[PATCH v2] drm: drm_memory: fix compiler warning on 32-bit targets

2018-02-19 Thread David Lechner
The end field of struct resource is resource_size_t, so on 32-bit targets, this is a 32-bit value. This results in a compiler warning of "warning: comparison of distinct pointer types lacks a cast". Fix it by making max_iomem resource_size_t to match the end field. Fixes: (82626363a217 "drm: add

Re: [PATCH] drm: drm_memory: fix compiler warning on 32-bit targets

2018-02-19 Thread David Lechner
On 02/19/2018 05:00 PM, David Lechner wrote: The end field of struct resource is unsigned long, so on 32-bit targets, this is a 32-bit value. This results in a compiler warning of "warning: comparison of distinct pointer types lacks a cast". Fix it by making max_iomem unsigned long to match the

Re: [PATCH] s390/console: enable dummy console for vt

2018-02-19 Thread Farhan Ali
On 02/19/2018 08:37 AM, Christian Borntraeger wrote: On 02/19/2018 02:35 PM, Farhan Ali wrote: On 02/15/2018 07:02 AM, Christian Borntraeger wrote: On 02/15/2018 12:57 PM, Thomas Huth wrote: On 15.02.2018 12:26, Geert Uytterhoeven wrote: Hi Christian, On Thu, Feb 15, 2018 at 12:14

Re: [PATCH v3 03/12] drm: rcar-du: Fix legacy DT to create LVDS encoder nodes

2018-02-19 Thread Rob Herring
On Wed, Feb 14, 2018 at 6:04 PM, Laurent Pinchart wrote: > The internal LVDS encoders now have their own DT bindings. Before > switching the driver infrastructure to those new bindings, implement > backward-compatibility through live DT patching. > >

[Bug 99353] Kaveri 7400K shows random colored noise instead of GUI in X or Wayland

2018-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99353 --- Comment #35 from Bong Cosca --- Created attachment 137448 --> https://bugs.freedesktop.org/attachment.cgi?id=137448=edit Proposed patch -- You are receiving this mail because: You are the assignee for the

Re: [PATCH 2/8] media: s5p-jpeg: Use bulk clk API

2018-02-19 Thread kbuild test robot
Hi Maciej, Thank you for the patch! Yet something to improve: [auto build test ERROR on linuxtv-media/master] [also build test ERROR on v4.16-rc2 next-20180219] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day

[Bug 105166] account request for drm-misc

2018-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105166 --- Comment #1 from David Lechner --- Created attachment 137446 --> https://bugs.freedesktop.org/attachment.cgi?id=137446=edit GPG Key -- You are receiving this mail because: You are the assignee for the

[Bug 104194] HDMI with RX460 not working on linux 4.15-rc2 with DC enabled and linux-amd-staging-drm-next

2018-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104194 --- Comment #5 from saunders...@wright.edu --- Is there any info that would be useful to provide in order to help debug this issue? It's occurring on a stable build for me, unlike the other person with this problem. -- You are receiving this

Re: [PATCH RFC] drm/bridge: panel: Add module_get/but calls to attached panel driver

2018-02-19 Thread Daniel Vetter
On Mon, Feb 19, 2018 at 10:06:22PM +0200, Jyri Sarha wrote: > Currently there is no way for a master drm driver to protect against an > attached panel driver from being unloaded while it is in use. The > least we can do is to indicate the usage by incrementing the module > reference count. > >

Re: [PATCH 8/8] [media] s5p-mfc: Use clk bulk API

2018-02-19 Thread kbuild test robot
Hi Maciej, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on linuxtv-media/master] [also build test WARNING on v4.16-rc2 next-20180219] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https

Re: linux-next: Tree for Feb 19 (gpu/drm/pl111/)

2018-02-19 Thread Randy Dunlap
[add Eric + dri-devel] On 02/19/18 09:26, Randy Dunlap wrote: > On 02/18/18 18:14, Stephen Rothwell wrote: >> Hi all, >> >> Changes since 20180216: >> >> The nand tree gained a conflict against Linus' tree. >> >> The drm tree gained conflicts against Linus' tree. > > on i386 (randconfig): > >

[Bug 105166] account request for drm-misc

2018-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105166 Bug ID: 105166 Summary: account request for drm-misc Product: DRI Version: DRI git Hardware: Other OS: All Status: NEW Severity: normal

[Bug 105170] Radeon R9 Nano failed testing IB on GFX ring (-110). [amd64, kernel 4.15.4]

2018-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105170 --- Comment #1 from erhar...@mailbox.org --- Created attachment 137451 --> https://bugs.freedesktop.org/attachment.cgi?id=137451=edit kernel config used (kernel 4.15.4) -- You are receiving this mail because: You are the assignee for the

[Bug 102820] [bisected][DC] commit ebbf7337e2daacacef3e01114e6be68a2a4f11b4 prevents X11 from starting

2018-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102820 --- Comment #18 from russianneuroman...@ya.ru --- Discussion: https://www.phoronix.com/forums/forum/phoronix/latest-phoronix-articles/1008686-amd-s-raven-ridge-botchy-linux-support-appears-worse-with-some-motherboards-bios/page8 -- You are

[PATCH] drm/tinydrm: Allocate dummy SPI RX buffer if needed

2018-02-19 Thread David Lechner
This adds a new feature to allocate a dummy RX buffer for SPI controllers that require an RX buffer even when not receiving. If an RX buffer is not supplied, the SPI controller will reallocate a new buffer on each transfer. On systems with limited memory (e.g. 64MB and CONFIG_SWAP=n), this

[GIT PULL] exynos-drm-fixes

2018-02-19 Thread Inki Dae
Hi Dave, Just several fixups - timestamp issue, register definition and setting value, and cleanups - removing unnecessary messages and file. Please kindly let me know if there is any problem. Thanks, Inki Dae The following changes since commit

Re: [PATCH] fix double ;;s in code

2018-02-19 Thread Rob Clark
On Mon, Feb 19, 2018 at 2:33 PM, Pavel Machek wrote: > On Mon 2018-02-19 16:41:35, Daniel Vetter wrote: >> On Sun, Feb 18, 2018 at 11:00:56AM +0100, Christophe LEROY wrote: >> > >> > >> > Le 17/02/2018 à 22:19, Pavel Machek a écrit : >> > > >> > > Fix double ;;'s in code. >> > > >>

Re: [PATCH 4/8] drm/exynos/dsi: Use clk bulk API

2018-02-19 Thread kbuild test robot
Hi Maciej, Thank you for the patch! Yet something to improve: [auto build test ERROR on linuxtv-media/master] [also build test ERROR on v4.16-rc2 next-20180219] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day

[Bug 105170] Radeon R9 Nano failed testing IB on GFX ring (-110). [amd64, kernel 4.15.4]

2018-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105170 Bug ID: 105170 Summary: Radeon R9 Nano failed testing IB on GFX ring (-110). [amd64, kernel 4.15.4] Product: DRI Version: unspecified Hardware: x86-64 (AMD64)

[PATCH] drm: drm_memory: fix compiler warning on 32-bit targets

2018-02-19 Thread David Lechner
The end field of struct resource is unsigned long, so on 32-bit targets, this is a 32-bit value. This results in a compiler warning of "warning: comparison of distinct pointer types lacks a cast". Fix it by making max_iomem unsigned long to match the end field. Fixes: (82626363a217 "drm: add func

[Bug 105170] Radeon R9 Nano failed testing IB on GFX ring (-110). [amd64, kernel 4.15.4]

2018-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105170 --- Comment #2 from erhar...@mailbox.org --- Created attachment 137452 --> https://bugs.freedesktop.org/attachment.cgi?id=137452=edit xorg.log -- You are receiving this mail because: You are the assignee for the

[PATCH libdrm] configure.ac: pthread-stubs not present on OpenBSD

2018-02-19 Thread Jonathan Gray
pthread-stubs is no longer required on OpenBSD and has been removed. libpthread parts involved moved to libc. Signed-off-by: Jonathan Gray --- configure.ac | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index

Re: amdgpu hangs on boot or shutdown on AMD Raven Ridge CPU (Engineer Sample)

2018-02-19 Thread Daniel Drake
Hi, > >>> We are working with new laptops that have the AMD Ravenl Ridge > >>> chipset with this `/proc/cpuinfo` > >>> https://gist.github.com/mschiu77/b06dba574e89b9a30cf4c450eaec49bc > >>> > >>> With the latest kernel 4.15, there're lots of different > >>> panics/oops during boot so no

[PATCH] drm: simple_kms_helper: Add mode_valid() callback support

2018-02-19 Thread Linus Walleij
The PL111 needs to filter valid modes based on memory bandwidth. I guess it is a pretty simple operation, so we can still claim the DRM KMS helper pipeline is simple after adding this (optional) vtable callback. Reviewed-by: Eric Anholt Reviewed-by: Daniel Vetter

Re: [PATCH] drm: add documentation for tv connector state margins

2018-02-19 Thread Jani Nikula
On Mon, 19 Feb 2018, Daniel Vetter wrote: > On Mon, Feb 19, 2018 at 12:40:09PM +0200, Jani Nikula wrote: >> A bit boring documentation fix, but gets rid of the warnings: >> >> ./include/drm/drm_connector.h:370: warning: Function parameter or member >> 'margins.left' not

[Bug 198123] Console is the wrong color at boot with radeon 6670

2018-02-19 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=198123 --- Comment #30 from Daniel Vetter (dan...@ffwll.ch) --- There was not a single backtrace in that attachement. Was this only up to fbcon, or did you start X too? Where the colors correct afterwards? It's also strange that it fails to compile on

[PATCH 1/2] drm/mm: Fix caching of leftmost node in the interval tree

2018-02-19 Thread Chris Wilson
When we descend the tree to find our slot, if we step to the right, we are no longer the leftmost node. Fixes: f808c13fd373 ("lib/interval_tree: fast overlap detection") Signed-off-by: Chris Wilson Cc: Davidlohr Bueso Cc: Jérôme Glisse

[PATCH 2/2] drm/mm: Micro-optimise updating the upper layers of the interval tree

2018-02-19 Thread Chris Wilson
When we insert a node into a hole inside the interval tree, we need to climb the layers above us to update the cached interval. When doing so, we know that the initial node exists as it is our starting hole. add/remove: 0/0 grow/shrink: 0/1 up/down: 0/-10 (-10) Function

Re: [PATCH 1/3] drm: add func to get max iomem address

2018-02-19 Thread Daniel Vetter
On Thu, Feb 08, 2018 at 02:00:15PM +0800, Chunming Zhou wrote: > it will be used to check if the driver needs swiotlb > > Change-Id: Idbe47af8f12032d4803bb3d47273e807f19169c3 > Signed-off-by: Chunming Zhou > --- > include/drm/drm_cache.h | 13 + > 1 file

Re: [PATCH V3 09/29] drm/i915: deprecate pci_get_bus_and_slot()

2018-02-19 Thread Joonas Lahtinen
Quoting Jani Nikula (2018-02-19 11:34:34) > On Fri, 16 Feb 2018, Bjorn Helgaas wrote: > > On Mon, Nov 27, 2017 at 11:57:46AM -0500, Sinan Kaya wrote: > >> pci_get_bus_and_slot() is restrictive such that it assumes domain=0 as > >> where a PCI device is present. This restricts

Re: [PATCH 1/3] drm: add func to get max iomem address v2

2018-02-19 Thread Daniel Vetter
On Fri, Feb 16, 2018 at 02:40:11PM +, Chris Wilson wrote: > Quoting Chunming Zhou (2018-02-09 02:44:08) > > it will be used to check if the driver needs swiotlb > > v2: Don't use inline, instead, move function to drm_memory.c (Mechel > > Daenzer ) > > > > Change-Id:

[PATCH 2/4] drm/amdgpu: use new ww_mutex_is_owned_by function

2018-02-19 Thread Christian König
Instead of accessing ww_mutex internals directly use the provided function to check if the ww_mutex was indeed locked by the current command submission. Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 3 ++- 1 file changed, 2 insertions(+),

[PATCH 4/4] drm/ttm: keep BOs reserved until end of eviction

2018-02-19 Thread Christian König
This avoids problems when BOs are evicted but directly moved back into the domain from other threads. Signed-off-by: Christian König --- drivers/gpu/drm/ttm/ttm_bo.c | 37 + 1 file changed, 29 insertions(+), 8 deletions(-) diff

[PATCH 1/4] locking/ww_mutex: add ww_mutex_is_owned_by function

2018-02-19 Thread Christian König
amdgpu needs to verify if userspace sends us valid addresses and the simplest way of doing this is to check if the buffer object is locked with the ticket of the current submission. Clean up the access to the ww_mutex internals by providing a function for this and extend the check to the thread

[PATCH 3/4] drm/ttm: handle already locked BOs during eviction and swapout.

2018-02-19 Thread Christian König
This solves the problem that when we swapout a BO from a domain we sometimes couldn't make room for it because holding the lock blocks all other BOs with this reservation object. Signed-off-by: Christian König --- drivers/gpu/drm/ttm/ttm_bo.c | 33

[4.9-stable 2/9] drm/i915: hide unused intel_panel_set_backlight function

2018-02-19 Thread Arnd Bergmann
commit fd94d53e55bd487368dfee9f1af24da78b2bb582 upstream. Building i915 without backlight support results in a harmless warning for intel_panel_set_backlight: drivers/gpu/drm/i915/intel_panel.c:653:13: error: 'intel_panel_set_backlight' defined but not used [-Werror=unused-function] This moves

Re: [PATCH] gpu/drm/udl: Replace struct_mutex with driver private lock

2018-02-19 Thread Daniel Vetter
On Sat, Feb 10, 2018 at 06:47:31PM +0530, Shreeya Patel wrote: > On Fri, 2018-02-09 at 12:18 +, Chris Wilson wrote: > > Quoting Shreeya Patel (2018-02-09 12:10:56) > > > > > > dev->struct_mutex is the Big DRM Lock and the only bit where > > > it’s mandatory is serializing GEM buffer object

Re: kdg2kfd_probe failed

2018-02-19 Thread Oded Gabbay
HI Sen, Those messages are expected and you can ignore them. In kernel 4.14 a couple of warning/error messages were added to amdkfd. If you use the amdgpu driver you will not see them but because you use radeon you are seeing them. In kernel 4.15 we removed support of amdkfd from radeon, so once

Re: [RFC v2 02/10] drm: drm_fourcc: Introduce macro-pixel info to drm_format_info

2018-02-19 Thread Daniel Vetter
On Thu, Feb 08, 2018 at 06:19:32PM -0800, Hyun Kwon wrote: > Hi Daniel, > > On Tue, 2018-01-30 at 02:22:40 -0800, Daniel Vetter wrote: > > On Thu, Jan 25, 2018 at 06:03:59PM -0800, Hyun Kwon wrote: > > > Multiple pixels can be grouped as a single unit and form a 'macro-pixel'. > > > This is to

Re: [PATCH] drm/ast: Add option to initialize palette on driver load

2018-02-19 Thread Daniel Vetter
On Fri, Feb 02, 2018 at 01:59:51AM -0600, Timothy Pearson wrote: > > Non-x86, such as OpenPOWER and ARM machines, do not execute the > ASPEED-provided > option ROM on system start. As a result, the VGA palette registers remain > uninitialized, leading to odd colors and generally hard to read

Re: [PATCH V3 09/29] drm/i915: deprecate pci_get_bus_and_slot()

2018-02-19 Thread Jani Nikula
On Fri, 16 Feb 2018, Bjorn Helgaas wrote: > On Mon, Nov 27, 2017 at 11:57:46AM -0500, Sinan Kaya wrote: >> pci_get_bus_and_slot() is restrictive such that it assumes domain=0 as >> where a PCI device is present. This restricts the device drivers to be >> reused for other

Re: [PATCH] dma-buf/sw_sync: Fix kerneldoc warnings

2018-02-19 Thread Daniel Vetter
On Thu, Feb 08, 2018 at 11:38:16AM +, Chris Wilson wrote: > drivers/dma-buf/sw_sync.c:248: warning: No description found for parameter > 'obj' > drivers/dma-buf/sw_sync.c:248: warning: No description found for parameter > 'value' > drivers/dma-buf/sw_sync.c:248: warning: Excess function

Re: [PATCH 0/6] Add support for in-line nested struct comments

2018-02-19 Thread Jani Nikula
On Sun, 18 Feb 2018, Jonathan Corbet wrote: > On Fri, 16 Feb 2018 11:48:14 -0200 > Mauro Carvalho Chehab wrote: > >> his series fix two bugs at kernel-doc.rst examples and add support >> for in-line nested struct comments. >> >> It also converts one

[Bug 198123] Console is the wrong color at boot with radeon 6670

2018-02-19 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=198123 --- Comment #31 from Daniel Vetter (dan...@ffwll.ch) --- Created attachment 274229 --> https://bugzilla.kernel.org/attachment.cgi?id=274229=edit another test patch for Deposite Pirate -- You are receiving this mail because: You are watching

Re: [PATCH] drm/bochs: make structure bochs_bo_driver static

2018-02-19 Thread Daniel Vetter
On Wed, Feb 07, 2018 at 11:13:53AM +, Colin King wrote: > From: Colin Ian King > > The structure bochs_bo_driver is local to the source and does not need to > be in global scope, so make it static. > > Cleans up sparse warning: >

[Bug 105156] kdg2kfd_probe failed

2018-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105156 Oded Gabbay changed: What|Removed |Added Status|NEW |ASSIGNED -- You

[Bug 105156] kdg2kfd_probe failed

2018-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105156 Oded Gabbay changed: What|Removed |Added Assignee|dri-devel@lists.freedesktop

Re: [RFC v2 02/10] drm: drm_fourcc: Introduce macro-pixel info to drm_format_info

2018-02-19 Thread Daniel Vetter
On Thu, Feb 08, 2018 at 06:19:42PM -0800, Hyun Kwon wrote: > Hi Daniel, > > On Tue, 2018-01-30 at 02:27:07 -0800, Daniel Vetter wrote: > > On Thu, Jan 25, 2018 at 06:03:59PM -0800, Hyun Kwon wrote: > > > Multiple pixels can be grouped as a single unit and form a 'macro-pixel'. > > > This is to

Re: [PATCH] drm/nouveau: prefer XBGR2101010 for addfb ioctl

2018-02-19 Thread Daniel Vetter
On Mon, Feb 05, 2018 at 09:10:12AM -0500, Ilia Mirkin wrote: > On Mon, Feb 5, 2018 at 8:58 AM, Ville Syrjälä > wrote: > > On Sat, Feb 03, 2018 at 02:11:23PM -0500, Ilia Mirkin wrote: > >> Nouveau only exposes support for XBGR2101010. Prior to the atomic > >>

Re: [PATCH] drm/nouveau: prefer XBGR2101010 for addfb ioctl

2018-02-19 Thread Daniel Vetter
On Mon, Feb 19, 2018 at 10:21:54AM +0100, Daniel Vetter wrote: > On Mon, Feb 05, 2018 at 09:10:12AM -0500, Ilia Mirkin wrote: > > On Mon, Feb 5, 2018 at 8:58 AM, Ville Syrjälä > > wrote: > > > On Sat, Feb 03, 2018 at 02:11:23PM -0500, Ilia Mirkin wrote: > > >>

Re: [PATCH 1/3] drm: simple_kms_helper: Add mode_valid() callback support

2018-02-19 Thread Daniel Vetter
On Sat, Feb 10, 2018 at 05:08:34PM +, Eric Anholt wrote: > Linus Walleij writes: > > > The PL111 needs to filter valid modes based on memory bandwidth. > > I guess it is a pretty simple operation, so we can still claim > > the DRM KMS helper pipeline is simple after

Re: [PATCH] drm: Fix kerneldoc warnings for drm_lease

2018-02-19 Thread Daniel Vetter
On Thu, Feb 08, 2018 at 11:08:17AM +, Chris Wilson wrote: > drivers/gpu/drm/drm_lease.c:56: warning: No description found for parameter > 'lessee_id' > drivers/gpu/drm/drm_lease.c:56: warning: Excess function parameter 'id' > description in '_drm_find_lessee' >

Re: [PATCH 1/7] PCI: Restore BARs on runtime resume despite being unbound

2018-02-19 Thread Rafael J. Wysocki
On Sun, Feb 18, 2018 at 9:38 AM, Lukas Wunner wrote: > PCI devices not bound to a driver are supposed to stay in D0 during > runtime suspend. But they may have a parent which is bound and can be > transitioned to D3cold at runtime. Once the parent goes to D3cold, the > unbound

kdg2kfd_probe failed

2018-02-19 Thread Sen Dion
For some unknown reason, I started to get the "kgd2kfd_probe failed" message after the update of Fedora-27 (limited to security advisories). Here is dmesg output: Console: switching to colour frame buffer device 240x67 radeon :01:05.0: fb0: radeondrmfb frame buffer device kfd kfd: DID

Re: [PATCH 1/4] drm/simple_kms_helper: Add {enable|disable}_vblank callback support

2018-02-19 Thread Oleksandr Andrushchenko
ping On 02/12/2018 10:52 AM, Oleksandr Andrushchenko wrote: From: Oleksandr Andrushchenko If simple_kms_helper based driver needs to work with vblanks, then it has to provide drm_driver.{enable|disable}_vblank callbacks, because

Re: [PATCH] drm/fb-helper: Scale back depth to supported maximum

2018-02-19 Thread Daniel Vetter
On Fri, Feb 02, 2018 at 04:03:43PM +0200, Ville Syrjälä wrote: > On Fri, Feb 02, 2018 at 02:56:30PM +0100, Linus Walleij wrote: > > On Thu, Feb 1, 2018 at 2:19 PM, Ville Syrjälä > > wrote: > > > [Me] > > >> + /* > > >> + * If we run into a situation where,

Re: Questions on page flips and atomic modeset

2018-02-19 Thread Daniel Vetter
On Tue, Feb 06, 2018 at 11:59:37AM +0200, Oleksandr Andrushchenko wrote: > Hello, Ville! > > Thank you very much for such a comprehensive answer. Just noticed a few of your questions threads. We'd very much appreciate (for the next person creating an atomic driver) if you check out our kernel

Re: Thinkpad X1 Carbon 3rd - Reducing the compressed framebuffer size

2018-02-19 Thread Pali Rohár
On Tuesday 13 February 2018 19:45:56 Ville Syrjälä wrote: > On Tue, Feb 13, 2018 at 06:43:41PM +0100, Pali Rohár wrote: > > On Tuesday 13 February 2018 18:12:21 Ville Syrjälä wrote: > > > On Tue, Feb 13, 2018 at 05:04:37PM +0100, Pali Rohár wrote: > > > > So it can be done only once after reboot?

Re: [PATCH v5 1/5] drm: xlnx: Xilinx DRM KMS module

2018-02-19 Thread Daniel Vetter
On Tue, Feb 06, 2018 at 05:36:36PM -0800, Hyun Kwon wrote: > Xilinx has various platforms for display, where users can create > using multiple IPs in the programmable FPGA fabric, or where > some hardened piepline is available on the chip. Furthermore, > hardened pipeline can also interact with

Re: Questions on page flips and atomic modeset

2018-02-19 Thread Oleksandr Andrushchenko
On 02/19/2018 11:36 AM, Daniel Vetter wrote: On Tue, Feb 06, 2018 at 11:59:37AM +0200, Oleksandr Andrushchenko wrote: Hello, Ville! Thank you very much for such a comprehensive answer. Just noticed a few of your questions threads. We'd very much appreciate (for the next person creating an

Re: [PATCH v3 33/43] drm/panel: simple: Change mode for Sharp lq123p1jx31

2018-02-19 Thread Enric Balletbo Serra
Hi, 2018-02-16 21:54 GMT+01:00 Doug Anderson : > Hi, > > On Fri, Feb 16, 2018 at 4:34 AM, Enric Balletbo Serra > wrote: >> Hi, >> >> 2018-01-31 17:52 GMT+01:00 Doug Anderson : >>> Hi, >>> >>> >>> On Wed, Jan 31, 2018 at 7:16 AM,

Re: [PATCH RFC v3 2/6] drm: drm_fourcc: Introduce macro-pixel info to drm_format_info

2018-02-19 Thread Daniel Vetter
On Fri, Feb 09, 2018 at 05:35:52PM -0800, Hyun Kwon wrote: > Multiple pixels can be grouped as a single unit and form a 'macro-pixel'. > This is to model formats where multiple non-byte aligned pixels are stored > together in a byte-aligned way. For example, if 3 - 10 bit > pixels are stored in 32

Re: [PATCH v2 2/4] drm: NULL pointer dereference [null-pointer-deref] (CWE 476) problem

2018-02-19 Thread Daniel Vetter
On Mon, Feb 12, 2018 at 02:51:42PM -0500, Joe Moriarty wrote: > The Parfait (version 2.1.0) static code analysis tool found the > following NULL pointer derefernce problem. > > - drivers/gpu/drm/drm_dp_mst_topology.c > The call to drm_dp_calculate_rad() in function drm_dp_port_setup_pdt() > could

Re: [PATCH RFC 5/9] drm/omap: move common stuff from dss.h to omapdss.h

2018-02-19 Thread Tomi Valkeinen
Hi Jyri, On 16/02/18 13:25, Jyri Sarha wrote: > The new DSS6 driver needs some structs and defines which are currently > in dss.h, which is for the old DSS driver. > > Move the required structs and defines from dss.h to omapdss.h. > > Signed-off-by: Tomi Valkeinen >

Re: [Intel-gfx] [Nouveau] [PATCH 0/5] Fix deadlock on runtime suspend in DRM drivers

2018-02-19 Thread Daniel Vetter
On Mon, Feb 19, 2018 at 12:58:17PM +0100, Lukas Wunner wrote: > On Mon, Feb 19, 2018 at 12:34:43PM +0100, Daniel Vetter wrote: > > On Sun, Feb 11, 2018 at 10:38:28AM +0100, Lukas Wunner wrote: > > > Fix a deadlock on hybrid graphics laptops that's been present since 2013: > > > > > > DRM drivers

Re: [PATCH] gpu: ipu-v3: prg: fix device node leak in ipu_prg_lookup_by_phandle

2018-02-19 Thread Philipp Zabel
Hi Tobias, On Thu, 2018-02-15 at 15:35 +0100, Tobias Jordan wrote: > Before returning, call of_node_put() for the device node returned by > of_parse_phandle(). > > Fixes: ea9c260514c1 ("gpu: ipu-v3: add driver for Prefetch Resolve Gasket") > Signed-off-by: Tobias Jordan

Re: [PATCH] gpu: ipu-v3: make const arrays int_reg static, shrinks object size

2018-02-19 Thread Philipp Zabel
Hi Colin, On Wed, 2018-02-14 at 18:45 +, Colin King wrote: > From: Colin Ian King > > Don't populate the const read-only arrays int_reg on the stack but instead > make them static. Makes the object code smaller by over 80 bytes: > > Before: >text data

Re: [PATCH] drm/simple_kms_helper: Fix NULL pointer dereference with no active CRTC

2018-02-19 Thread Daniel Vetter
On Tue, Feb 13, 2018 at 10:44:16AM +0200, Oleksandr Andrushchenko wrote: > From: Oleksandr Andrushchenko > > It is possible that drm_simple_kms_plane_atomic_check called > with no CRTC set, e.g. when user-space application sets CRTC_ID/FB_ID > to 0 before doing

Re: [RFC PATCH v2 2/6] dt-bindings: add bindings for Samsung micro-USB 11-pin connector

2018-02-19 Thread Rob Herring
On Thu, Feb 15, 2018 at 11:39:16AM +0100, Andrzej Hajda wrote: > Samsung micro-USB 11-pin connector beside standard micro-USB pins, > has pins dedicated to route MHL traffic. > > Signed-off-by: Andrzej Hajda > --- > .../connector/samsung,usb-connector-11pin.txt | 51 >

Re: [PATCH] drm: Handle unexpected holes in color-eviction

2018-02-19 Thread Chris Wilson
Quoting Joonas Lahtinen (2018-02-19 14:40:32) > Quoting Chris Wilson (2018-02-19 13:35:43) > > +++ b/drivers/gpu/drm/drm_mm.c > > @@ -836,9 +836,24 @@ struct drm_mm_node *drm_mm_scan_color_evict(struct > > drm_mm_scan *scan) > > if (!mm->color_adjust) > > return NULL; > >

Re: [Intel-gfx] [Nouveau] [PATCH 0/5] Fix deadlock on runtime suspend in DRM drivers

2018-02-19 Thread Lukas Wunner
On Mon, Feb 19, 2018 at 03:05:53PM +0100, Daniel Vetter wrote: > On Mon, Feb 19, 2018 at 12:58:17PM +0100, Lukas Wunner wrote: > > On Mon, Feb 19, 2018 at 12:34:43PM +0100, Daniel Vetter wrote: > > > Well, userspace expects hotplug events, even when we runtime suspend > > > stuff. Hence waking

Re: [PATCH 3/8] drm/atomic: Include color encoding/range in plane state dump

2018-02-19 Thread Daniel Vetter
On Wed, Feb 14, 2018 at 09:23:22PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > Include color_enconding and color_range in the plane state dump. > > Cc: Harry Wentland > Cc: Daniel Vetter > Cc: Daniel Stone

Re: [PATCH 0/8] drm: Add COLOR_ENCODING and COLOR_RANGE plane properties

2018-02-19 Thread Daniel Vetter
On Wed, Feb 14, 2018 at 09:23:19PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > Here's a refresh of Jyri's COLOR_ENCODING and COLOR_RANGE properties, > and the i915 implementation I did on top. I tossed in a few core > updates as well: plane state dump,

Re: [PATCH v3 3/6] dt-bindings: Add panel-timing subnode to simple-panel

2018-02-19 Thread Thierry Reding
On Thu, Feb 08, 2018 at 12:48:50PM -0500, Sean Paul wrote: > This patch adds a new subnode to simple-panel allowing us to override > the typical timing expressed in the panel's display_timing. > > Changes in v2: > - Split out the binding into a new patch (Rob) > - display-timings is a new

Re: [PATCH] drm/meson: fix vsync buffer update

2018-02-19 Thread Daniel Vetter
On Thu, Feb 15, 2018 at 11:19:36AM +0100, Neil Armstrong wrote: > The plane buffer address/stride/height was incorrectly updated in the > plane_atomic_update operation instead of the vsync irq. > This patch delays this operation in the vsync irq along with the > other plane delayed setup. > >

Re: [PATCH RFC v3 3/6] drm: fourcc: Add drm_format_plane_width_bytes()

2018-02-19 Thread Daniel Vetter
On Fri, Feb 09, 2018 at 05:35:53PM -0800, Hyun Kwon wrote: > drm_format_plane_width_bytes() calculates and returns the number of bytes > for given width of specified format. The calculation uses @cpp > in drm format info for byte-aligned formats. If the format isn't > byte-aligned, @cpp should 0,

Re: [PATCH v2 7/8] drm/rockchip: dw_hdmi: store rockchip_hdmi reference in phy_data object

2018-02-19 Thread Robin Murphy
On 16/02/18 20:41, Heiko Stuebner wrote: When using special phy handling operations we'll often need access to the rockchip_hdmi struct. As the chip-data that occupies the phy_data pointer initially gets assigned to the rockchip_hdmi struct we can not s/not/now/ (or s/not//) ? Again, this

Re: [Nouveau] [PATCH 0/5] Fix deadlock on runtime suspend in DRM drivers

2018-02-19 Thread Lukas Wunner
On Mon, Feb 19, 2018 at 12:34:43PM +0100, Daniel Vetter wrote: > On Sun, Feb 11, 2018 at 10:38:28AM +0100, Lukas Wunner wrote: > > Fix a deadlock on hybrid graphics laptops that's been present since 2013: > > > > DRM drivers poll connectors in 10 sec intervals. The poll worker is > > stopped on

Re: [PATCH libdrm] meson: do not use cairo/valgrind if disabled

2018-02-19 Thread Eric Engestrom
On Sunday, 2018-02-18 14:00:50 +0100, Igor Gnatenko wrote: > -Dcairo-tests=false currently results into enabling cairo support if it > was found. Same for valgrind. Indeed, this was wrong; thanks for the fix! Reviewed-by: Eric Engestrom Do you have commit access, or

[PATCH libdrm] meson: do not use cairo/valgrind if disabled

2018-02-19 Thread Igor Gnatenko
-Dcairo-tests=false currently results into enabling cairo support if it was found. Same for valgrind. v2: * Use underscore-prefixed variables to not change type of variable * Use empty array for "fake" dependency instead of real empty object v3: * Fix typo Reviewed-by: Eric Engestrom

Re: [PATCH v5 05/12] drm/bridge/synopsys: dw-hdmi: don't clobber drvdata

2018-02-19 Thread Philipp Zabel
On Wed, 2018-02-14 at 21:08 +0100, Jernej Skrabec wrote: > dw_hdmi shouldn't set drvdata since some drivers might need to store > it's own data there. Rework dw_hdmi in a way to return struct dw_hdmi > instead to store it in drvdata. This way drivers are responsible to > store and pass structure

Re: [PATCH v3 5/6] drm/panel: simple: Use display_timing for lq123p1jx31

2018-02-19 Thread Enric Balletbo Serra
Hi, 2018-02-08 18:48 GMT+01:00 Sean Paul : > Convert the sharp lq123p1jx31 from using a fixed mode to specifying a > display timing with min/typ/max values. This allows us to capture the > timings set forth in the datasheet as well as the additional values that > we've

Re: [RFC PULL] Add Display Support for Qualcomm SDM845

2018-02-19 Thread Daniel Vetter
On Tue, Feb 13, 2018 at 03:00:09PM -0500, Rob Clark wrote: > On Tue, Feb 13, 2018 at 2:18 PM, Sean Paul wrote: > > Hi dri-devel, > > Qualcomm has been working for the past few weeks on forward porting their > > downstream drm driver from 4.14 to mainline. Please consider

Re: [Intel-gfx] [Nouveau] [PATCH 0/5] Fix deadlock on runtime suspend in DRM drivers

2018-02-19 Thread Daniel Vetter
On Mon, Feb 19, 2018 at 03:47:42PM +0100, Lukas Wunner wrote: > On Mon, Feb 19, 2018 at 03:05:53PM +0100, Daniel Vetter wrote: > > On Mon, Feb 19, 2018 at 12:58:17PM +0100, Lukas Wunner wrote: > > > On Mon, Feb 19, 2018 at 12:34:43PM +0100, Daniel Vetter wrote: > > > > Well, userspace expects

[PATCH] drm/todo: Add idr_init_base todo

2018-02-19 Thread Daniel Vetter
Suggested-by: Chris Wilson Cc: Chris Wilson Acked-by: Chris Wilson Signed-off-by: Daniel Vetter --- Documentation/gpu/todo.rst | 10 ++ 1 file changed, 10 insertions(+) diff --git

Re: [PATCH 1/8] drm: Add optional COLOR_ENCODING and COLOR_RANGE properties to drm_plane

2018-02-19 Thread Daniel Vetter
On Wed, Feb 14, 2018 at 09:23:20PM +0200, Ville Syrjala wrote: > From: Jyri Sarha > > Add a standard optional properties to support different non RGB color > encodings in DRM planes. COLOR_ENCODING select the supported non RGB > color encoding, for instance ITU-R BT.709 YCbCr.

Re: [PATCH 02/10] drm: Add Plane Degamma properties

2018-02-19 Thread Daniel Vetter
On Thu, Feb 15, 2018 at 12:32:52AM -0500, Daniele Castagna wrote: > From: "uma.shankar at intel.com (Uma Shankar)" > > Add Plane Degamma as a blob property and plane > degamma size as a range property. > > (am from https://patchwork.kernel.org/patch/10046515/) > >

[Bug 198123] Console is the wrong color at boot with radeon 6670

2018-02-19 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=198123 --- Comment #32 from Deposite Pirate (dpir...@metalpunks.info) --- Hi, Yeah, I wasn't sure because I never enabled backtraces before. But it didn't seem to me that there were many backtraces either. Though I did double check the patch was

Re: [PATCH] drm: Handle unexpected holes in color-eviction

2018-02-19 Thread Joonas Lahtinen
Quoting Chris Wilson (2018-02-19 13:35:43) > +++ b/drivers/gpu/drm/drm_mm.c > @@ -836,9 +836,24 @@ struct drm_mm_node *drm_mm_scan_color_evict(struct > drm_mm_scan *scan) > if (!mm->color_adjust) > return NULL; > > - hole = list_first_entry(>hole_stack,

Re: [PATCH] drm: Use idr_init_base(1) when using id==0 for invalid

2018-02-19 Thread Daniel Vetter
On Mon, Feb 19, 2018 at 02:35:38PM +, Chris Wilson wrote: > Quoting Christian König (2018-02-13 13:48:24) > > Am 12.02.2018 um 18:14 schrieb Ville Syrjälä: > > > On Mon, Feb 12, 2018 at 02:55:33PM +, Chris Wilson wrote: > > >> Use the new idr_init_base() function to create an IDR that

Re: [PATCH 00/10] drm: Add plane color matrix on rockchip

2018-02-19 Thread Daniel Vetter
On Thu, Feb 15, 2018 at 12:32:50AM -0500, Daniele Castagna wrote: > Hello, > this patch series adds a per plane color matrix property as well as an > implementation for rockchip. > > This patch series builds on and includes previous work done by uma.shankar: >

Re: [PATCH RFC v3 4/6] drm: drm_fourcc: Add new formats for Xilinx IPs

2018-02-19 Thread Daniel Vetter
On Fri, Feb 09, 2018 at 05:35:54PM -0800, Hyun Kwon wrote: > This patch adds new formats needed by Xilinx IP. Pixels are not > byte-aligned in these formats, and the drm_format_info for these > formats has macro-pixel information. > > Signed-off-by: Jeffrey Mouroux >

Re: [RFC PATCH v2 1/6] dt-bindings: add bindings for USB physical connector

2018-02-19 Thread Rob Herring
On Thu, Feb 15, 2018 at 11:39:15AM +0100, Andrzej Hajda wrote: > These bindings allow to describe most known standard USB connectors > and it should be possible to extend it if necessary. > USB connectors, beside USB can be used to route other protocols, > for example UART, Audio, MHL. In such

Re: [PATCH v3 4/6] drm/panel: simple: Add ability to override typical timing

2018-02-19 Thread Enric Balletbo Serra
Hi, 2018-02-08 18:48 GMT+01:00 Sean Paul : > This patch adds the ability to override the typical display timing for a > given panel. This is useful for devices which have timing constraints > that do not apply across the entire display driver (eg: to avoid > crosstalk

[Bug 105163] *-symbol-checks hard-code the nm executable

2018-02-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105163 Bug ID: 105163 Summary: *-symbol-checks hard-code the nm executable Product: DRI Version: XOrg git Hardware: All OS: Linux (All) Status: NEW Severity:

Re: [PATCH v3 6/6] arm64: dts: rockchip: Specify override mode for kevin panel

2018-02-19 Thread Enric Balletbo Serra
2018-02-08 18:48 GMT+01:00 Sean Paul : > This patch adds an override mode for kevin devices. The mode increases > both back porches to allow a pixel clock of 2kHz as opposed to the > 'typical' value of 252750kHz. This is needed to avoid interference with > the touch

  1   2   >