Re: [PATCH 4/4] drm/sun4i: make sure we don't have a commit pending

2017-07-18 Thread Maxime Ripard
On Mon, Jul 17, 2017 at 02:57:19PM +0800, Chen-Yu Tsai wrote: > On Mon, Jul 17, 2017 at 2:55 PM, Maxime Ripard > wrote: > > On Fri, Jul 14, 2017 at 04:56:01PM +0800, Chen-Yu Tsai wrote: > >> Hi, > >> > >> On Thu, Jul 13, 2017 at 10:41 PM, Maxime Ripard > >>

Re: [PATCH 4/4] drm/sun4i: make sure we don't have a commit pending

2017-07-18 Thread Daniel Vetter
On Tue, Jul 18, 2017 at 9:07 AM, Maxime Ripard wrote: > On Mon, Jul 17, 2017 at 02:57:19PM +0800, Chen-Yu Tsai wrote: >> On Mon, Jul 17, 2017 at 2:55 PM, Maxime Ripard >> wrote: >> > On Fri, Jul 14, 2017 at 04:56:01PM +0800,

[PATCH] drm/syncobj: Fix kerneldoc

2017-07-18 Thread Daniel Vetter
make htmldocs helps with catching these. Cc: Dave Airlie Signed-off-by: Daniel Vetter --- Documentation/gpu/drm-mm.rst | 2 +- drivers/gpu/drm/drm_syncobj.c | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git

Re: [PATCH] drm/i915: Explicit the connector name for DP link training result

2017-07-18 Thread Jani Nikula
On Mon, 17 Jul 2017, Paul Kocialkowski wrote: > This adds the connector name when printing a debug message about the DP > link training result. It is useful to figure out what connector is > failing when multiple DP connectors are used. Perhaps more consistent

Re: [PATCH v6 0/7] STM32 DSI HOST

2017-07-18 Thread Archit Taneja
Hi, On 07/17/2017 01:10 PM, Philippe CORNU wrote: Version 6: - bridge/synopsys/dw-mipi-dsi.c: drm_bridge_add update (Archit Taneja) queued patches #1 to #4 to drm-misc-next. Please let me know when you apply the remainder of the ST patches. Once that's done, we can add CONFIG_DRM_STM_DSI=y

[PULL] drm-intel-next

2017-07-18 Thread Daniel Vetter
Hi Dave, drm-intel-next-2017-07-17: 2nd round of 4.14 features: - prep for deferred fbdev setup - refactor fixed 16.16 computations and skl+ wm code (Mahesh Kumar) - more cnl paches (Rodrigo, Imre et al) - tighten context cleanup and handling (Chris Wilson) - fix interlaced handling on skl+

Re: [PATCH] drm: stm: remove "default y" in Kconfig

2017-07-18 Thread Benjamin Gaignard
2017-07-10 17:24 GMT+02:00 Emil Velikov : > On 10 July 2017 at 16:07, Benjamin Gaignard > wrote: >> To do not force stm driver to be build by default >> >> Signed-off-by: Benjamin Gaignard > Yes, please. You

Re: [PATCH v6 0/7] STM32 DSI HOST

2017-07-18 Thread Philippe CORNU
On 07/18/2017 09:57 AM, Archit Taneja wrote: > > Hi, > > On 07/17/2017 01:10 PM, Philippe CORNU wrote: >> Version 6: >> - bridge/synopsys/dw-mipi-dsi.c: drm_bridge_add update (Archit Taneja) > > queued patches #1 to #4 to drm-misc-next. Please let me know when you apply > the remainder of the

Re: [PATCH 1/4] drm/atomic: implement drm_atomic_helper_commit_tail for runtime_pm users

2017-07-18 Thread Maxime Ripard
Hi Laurent, On Fri, Jul 14, 2017 at 02:43:12AM +0300, Laurent Pinchart wrote: > Hi Maxime, > > Thank you for the patch. > > On Thursday 13 Jul 2017 16:41:13 Maxime Ripard wrote: > > The current drm_atomic_helper_commit_tail helper works only if the CRTC is > > accessible, and documents an

Re: [PATCH 2/5] drm: vmwgfx: constify pci_device_id.

2017-07-18 Thread Sinclair Yeh
Thanks. Queued. On Sat, Jul 15, 2017 at 12:44:53PM +0530, Arvind Yadav wrote: > pci_device_id are not supposed to change at runtime. All functions > working with pci_device_id provided by work with > const pci_device_id. So mark the non-const structs as const. > > File size before: >text

[PATCH] drm: Update docs around gem_free_object

2017-07-18 Thread Daniel Vetter
Not all places correctly stated that gem_free_object_unlocked is the one to use. Reported-by: Eric Anholt --- Documentation/gpu/drm-mm.rst | 2 +- drivers/gpu/drm/drm_gem_cma_helper.c

Re: [RFC][PATCH] drm: kirin: Restrict modes to known good mode clocks

2017-07-18 Thread Daniel Vetter
On Mon, Jul 17, 2017 at 04:20:23PM -0700, John Stultz wrote: > On Tue, Jul 11, 2017 at 9:27 AM, Daniel Vetter wrote: > > On Tue, Jul 11, 2017 at 5:44 PM, John Stultz wrote: > >> On Tue, Jul 11, 2017 at 8:12 AM, Daniel Vetter wrote: > >>>

Re: [PATCH 1/3] drm/mxsfb: Use gem_free_object_unlocked

2017-07-18 Thread Daniel Vetter
On Mon, Jul 17, 2017 at 11:01:33AM -0700, Eric Anholt wrote: > Daniel Vetter writes: > > > CMA helpers are struct_mutex free, and so is the mxsfb itself. And > > that's the only valid reason for using gem_free_object. Please read > > the kerneldoc for this stuff :-) > >

Re: [PATCH 6/7] docs: Do not include from include/drm/drm_color_mgmt.h

2017-07-18 Thread Daniel Vetter
On Mon, Jul 17, 2017 at 03:00:47PM -0600, Jonathan Corbet wrote: > Commit 8f2e045ec878 (drm/color: un-inline drm_color_lut_extract()) moved > the only kerneldoc comment out of include/drm/drm_color_mgmt.h, leading to > this warning: > > ./include/drm/drm_color_mgmt.h:1: warning: no structured

[PATCH] drm/syncobj: add sync obj wait interface. (v7)

2017-07-18 Thread Dave Airlie
From: Dave Airlie This interface will allow sync object to be used to back Vulkan fences. This API is pretty much the vulkan fence waiting API, and I've ported the code from amdgpu. v2: accept relative timeout, pass remaining time back to userspace. v3: return to absolute

Re: [PATCH v2] drm/i915: Consistently use enum pipe for PCH transcoders

2017-07-18 Thread Daniel Vetter
On Mon, Jul 17, 2017 at 11:14:03AM -0700, Matthias Kaehlcke wrote: > The current code uses in some instances enum transcoder for PCH > transcoders and enum pipe in others. This is error prone and clang > raises warnings like this: > > drivers/gpu/drm/i915/intel_dp.c:3546:51: warning: implicit

[Bug 101787] colours all messed up

2017-07-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101787 --- Comment #8 from 247 --- Created attachment 132742 --> https://bugs.freedesktop.org/attachment.cgi?id=132742=edit gst.debug transmaggeddon this is the log for transmaggeddon after that error was triggered...maybe

Re: [Intel-gfx] [PATCH] drm: Don't complain too much about struct_mutex.

2017-07-18 Thread Daniel Vetter
On Sat, Jul 15, 2017 at 12:02:47PM +0100, Chris Wilson wrote: > Quoting Daniel Vetter (2017-07-15 10:53:28) > > For modern drivers the DRM core doesn't use struct_mutex at all, which > > means it's defacto a driver-private lock. But since we still need it > > for legacy drivers we can't initialize

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

2017-07-18 Thread Chris Zhong
Hi Sean Thanks for your replying. On Tuesday, July 18, 2017 04:23 AM, Sean Paul wrote: On Sat, Jul 15, 2017 at 07:00:18PM +0800, Chris Zhong wrote: Some DP/HDMI sink need to receive the audio infoframe to play sound, especially some multi-channel AV receiver, they need the channel_allocation

Re: [PATCH libdrm 1/2] drm/amdgpu: add syncobj create/destroy/import/export apis

2017-07-18 Thread Christian König
Am 18.07.2017 um 02:48 schrieb Dave Airlie: From: Dave Airlie These are just wrappers using the amdgpu device handle. Signed-off-by: Dave Airlie Reviewed-by: Christian König for this one. --- amdgpu/amdgpu.h| 55

Re: [PATCH libdrm 2/2] drm/amdgpu: add new low overhead command submission API.

2017-07-18 Thread Christian König
Am 18.07.2017 um 02:48 schrieb Dave Airlie: From: Dave Airlie This just sends chunks to the kernel API for a single command stream. This should provide a more future proof and extensible API for command submission. Signed-off-by: Dave Airlie ---

Re: [PATCH libdrm] libdrm_amdgpu: add kernel semaphore support

2017-07-18 Thread Christian König
Am 18.07.2017 um 04:29 schrieb zhoucm1: On 2017年07月18日 01:35, Christian König wrote: Am 17.07.2017 um 19:22 schrieb Marek Olšák: On Sun, Jul 16, 2017 at 11:36 PM, Dave Airlie wrote: I can take a look at it, I just won't have time until next week most likely. I've taken

[Bug 101787] colours all messed up

2017-07-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101787 Julien Isorce changed: What|Removed |Added Attachment #132737|0 |1 is

Re: [PATCH 1/2] drm/ttm: Implement vm_operations_struct.access v2

2017-07-18 Thread Christian König
Am 18.07.2017 um 05:57 schrieb Felix Kuehling: Allows gdb to access contents of user mode mapped BOs. System memory is handled by TTM using kmap. Other memory pools require a new driver callback in ttm_bo_driver. v2: * kmap only one page at a time * swap in BO if needed * make driver callback

Re: [PATCH v1 2/2] drm/bridge/synopsys: dsi: Register list clean up

2017-07-18 Thread Laurent Pinchart
Hi Philippe, Thank you for the patch. On Tuesday 18 Jul 2017 13:43:52 Philippe CORNU wrote: > This patch cleans up the Synopsys mipi dsi register list: > - remove unused registers Is the documentation for the DSI transmitter core public ? If not, it could be useful to keep unused registers for

Re: [PATCH libdrm] drm/amdgpu: add new low overhead command submission API. (v2)

2017-07-18 Thread Christian König
Am 18.07.2017 um 05:52 schrieb Dave Airlie: From: Dave Airlie This just sends chunks to the kernel API for a single command stream. This should provide a more future proof and extensible API for command submission. v2: use amdgpu_bo_list_handle, add two helper functions

[Bug 92715] [IGT] [BYT-M/KBL/BXT/BDW/IVB/BSW] gem_reset_stats sub tests fail

2017-07-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92715 Humberto Israel Perez Rodriguez changed: What|Removed |Added

[PATCH v2] drm/i915: Explicit the connector name for DP link training result

2017-07-18 Thread Paul Kocialkowski
This adds the connector name when printing a debug message about the DP link training result. It is useful to figure out what connector is failing when multiple DP connectors are used. Signed-off-by: Paul Kocialkowski ---

Re: [PATCH 11/18] dt-bindings: display: Add Allwinner MIPI-DSI bindings

2017-07-18 Thread Laurent Pinchart
On Monday 17 Jul 2017 13:41:49 Rob Herring wrote: > On Thu, Jul 13, 2017 at 04:13:06PM +0200, Maxime Ripard wrote: > > The Allwinner SoCs usually come with a DSI encoder. Add a binding for it. > > > > Signed-off-by: Maxime Ripard > > --- > > > >

Re: [PATCH] drm/msm/mdp5: Fix compilation warnings

2017-07-18 Thread Rob Clark
2017-07-18 2:34 GMT-04:00 Viresh Kumar : > On 29-06-17, 14:49, Viresh Kumar wrote: >> Following compilation warnings were observed for these files: >> >> CC [M] drivers/gpu/drm/msm/mdp/mdp5/mdp5_mdss.o >> drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c: In function

[PATCH v1 1/7] drm/stm: drv: Rename platform driver name

2017-07-18 Thread Philippe CORNU
Rename the platform driver name from "stm" to "stm32-display" for a better readability in /sys/bus/platform/drivers entries. Note: We keep "stm" as drm_driver.name because it is better when using "modetest -M stm ..." (even if recent modetest patch avoids using -M). Signed-off-by: Philippe CORNU

[PATCH v1 7/7] drm/stm: dsi: Constify phy ops structure

2017-07-18 Thread Philippe CORNU
Signed-off-by: Philippe CORNU --- drivers/gpu/drm/stm/dw_mipi_dsi-stm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/stm/dw_mipi_dsi-stm.c b/drivers/gpu/drm/stm/dw_mipi_dsi-stm.c index 16ae00e..568c5d0 100644 ---

[PATCH v1 3/7] drm/stm: ltdc: Lindent and minor cleanups

2017-07-18 Thread Philippe CORNU
Lindent then checkpatch --strict cleanups Signed-off-by: Philippe CORNU --- drivers/gpu/drm/stm/ltdc.c | 172 ++--- 1 file changed, 85 insertions(+), 87 deletions(-) diff --git a/drivers/gpu/drm/stm/ltdc.c

[PATCH v1 4/7] drm/stm: ltdc: Constify funcs structures

2017-07-18 Thread Philippe CORNU
Constify drm funcs structures. Signed-off-by: Philippe CORNU --- drivers/gpu/drm/stm/ltdc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c index 628825b..92e58ba 100644 ---

[PATCH v1 6/7] drm/stm: ltdc: Cleanup rename returned value

2017-07-18 Thread Philippe CORNU
Signed-off-by: Philippe CORNU --- drivers/gpu/drm/stm/ltdc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c index f4ed21a..8cd1b9b 100644 --- a/drivers/gpu/drm/stm/ltdc.c +++

[PATCH v1 2/7] drm/stm: ltdc: Cleanup signal polarity defines

2017-07-18 Thread Philippe CORNU
The GCR_PCPOL/DEPOL/VSPOL/HSPOL defines are sufficient to describe the HS, VS, DE & PC signal polarities. Signed-off-by: Philippe CORNU --- drivers/gpu/drm/stm/ltdc.c | 28 ++-- 1 file changed, 10 insertions(+), 18 deletions(-) diff --git

[PATCH v1 0/7] drm/stm: Various cleanups

2017-07-18 Thread Philippe CORNU
Version 1: - Initial commit The purpose of this set of patches is to clean up the drm stm driver. Philippe CORNU (7): drm/stm: drv: Rename platform driver name drm/stm: ltdc: Cleanup signal polarity defines drm/stm: ltdc: Lindent and minor cleanups drm/stm: ltdc: Constify funcs

[PATCH v1 5/7] drm/stm: ltdc: add devm_reset_control & platform_get_ressource

2017-07-18 Thread Philippe CORNU
Use devm_reset_control_get to avoid resource leakage. Also use platform_get_resource, which is more usual and consistent with platform_get_irq called later. Signed-off-by: Fabien Dessenne Signed-off-by: Philippe CORNU ---

Re: [RFC][PATCH v2] drm: kirin: Add mode_valid logic to avoid mode clocks we can't generate

2017-07-18 Thread Jose Abreu
Hi John, On 18-07-2017 05:22, John Stultz wrote: > Currently the hikey dsi logic cannot generate accurate byte > clocks values for all pixel clock values. Thus if a mode clock > is selected that cannot match the calculated byte clock, the > device will boot with a blank screen. > > This patch

Re: [PATCH 1/4] drm/atomic: implement drm_atomic_helper_commit_tail for runtime_pm users

2017-07-18 Thread Laurent Pinchart
Hi Maxime, On Tuesday 18 Jul 2017 09:05:22 Maxime Ripard wrote: > On Fri, Jul 14, 2017 at 02:43:12AM +0300, Laurent Pinchart wrote: > > On Thursday 13 Jul 2017 16:41:13 Maxime Ripard wrote: > >> The current drm_atomic_helper_commit_tail helper works only if the CRTC > >> is accessible, and

Re: [PATCH] drm/i915: Explicit the connector name for DP link training result

2017-07-18 Thread Paul Kocialkowski
Hey, On Tue, 2017-07-18 at 10:38 +0300, Jani Nikula wrote: > On Mon, 17 Jul 2017, Paul Kocialkowski com> wrote: > > This adds the connector name when printing a debug message about the > > DP > > link training result. It is useful to figure out what connector is

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

2017-07-18 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

Re: [PATCH 1/4] drm/atomic: implement drm_atomic_helper_commit_tail for runtime_pm users

2017-07-18 Thread Daniel Vetter
On Tue, Jul 18, 2017 at 01:14:12PM +0300, Laurent Pinchart wrote: > Hi Maxime, > > On Tuesday 18 Jul 2017 09:05:22 Maxime Ripard wrote: > > On Fri, Jul 14, 2017 at 02:43:12AM +0300, Laurent Pinchart wrote: > > > On Thursday 13 Jul 2017 16:41:13 Maxime Ripard wrote: > > >> The current

Re: [PATCH] drm/i915: Synchronize connectors states when switching from poll to irq

2017-07-18 Thread Paul Kocialkowski
On Mon, 2017-06-26 at 15:32 +0300, Paul Kocialkowski wrote: > After detecting an IRQ storm, hotplug detection will switch from > irq-based detection to poll-based detection. After a short delay or > when resetting storm detection from debugfs, detection will switch > back to being irq-based. > >

Re: [PATCH 1/4] drm/atomic: implement drm_atomic_helper_commit_tail for runtime_pm users

2017-07-18 Thread Laurent Pinchart
Hi Daniel, On Tuesday 18 Jul 2017 14:08:39 Daniel Vetter wrote: > On Tue, Jul 18, 2017 at 01:14:12PM +0300, Laurent Pinchart wrote: > > On Tuesday 18 Jul 2017 09:05:22 Maxime Ripard wrote: > >> On Fri, Jul 14, 2017 at 02:43:12AM +0300, Laurent Pinchart wrote: > >>> On Thursday 13 Jul 2017

Re: [PATCH 1/4] drm/atomic: implement drm_atomic_helper_commit_tail for runtime_pm users

2017-07-18 Thread Daniel Vetter
On Tue, Jul 18, 2017 at 2:47 PM, Laurent Pinchart wrote: > On Tuesday 18 Jul 2017 14:08:39 Daniel Vetter wrote: >> On Tue, Jul 18, 2017 at 01:14:12PM +0300, Laurent Pinchart wrote: >> > On Tuesday 18 Jul 2017 09:05:22 Maxime Ripard wrote: >> >> On Fri, Jul 14,

Re: [PATCH v2 00/14] Renesas R-Car VSP: Add H3 ES2.0 support

2017-07-18 Thread Hans Verkuil
On 26/06/17 20:12, Laurent Pinchart wrote: > Hello, > > This patch series implements support for the R-Car H3 ES2.0 SoC in the VSP > and DU drivers. > > Compared to the H3 ES1.1, the H3 ES2.0 has a new VSP2-DL instance that > includes two blending units, a BRU and a BRS. The BRS is similar to

[PATCH v1 1/2] drm/bridge/synopsys: dsi: Constify funcs structures

2017-07-18 Thread Philippe CORNU
Signed-off-by: Philippe CORNU --- drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c b/drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c index 36f5ccb..63c7a01 100644

[PATCH v1 2/2] drm/bridge/synopsys: dsi: Register list clean up

2017-07-18 Thread Philippe CORNU
This patch cleans up the Synopsys mipi dsi register list: - remove unused registers - rename registers according to the Synopsys documentation (1.30 & 1.31) - fix typos - re-order registers for a better coherency Signed-off-by: Philippe CORNU ---

[PATCH v1 0/2] drm/bridge/synopsys: dsi: Various cleanups

2017-07-18 Thread Philippe CORNU
Version 1: - Initial commit The purpose of this set of patches is to clean up the mipi dsi dw Synopsys drm bridge. Philippe CORNU (2): drm/bridge/synopsys: dsi: Constify funcs structures drm/bridge/synopsys: dsi: Register list clean up drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 79

Re: [PATCH 6/7] docs: Do not include from include/drm/drm_color_mgmt.h

2017-07-18 Thread Jonathan Corbet
On Tue, 18 Jul 2017 08:42:42 +0200 Daniel Vetter wrote: > > Commit 8f2e045ec878 (drm/color: un-inline drm_color_lut_extract()) moved > > the only kerneldoc comment out of include/drm/drm_color_mgmt.h, leading to > > this warning: > > > > ./include/drm/drm_color_mgmt.h:1:

Re: [PATCH 01/18] regmap: mmio: Add function to attach a clock

2017-07-18 Thread Mark Brown
On Mon, Jul 17, 2017 at 11:01:07AM +0200, Maxime Ripard wrote: > On Thu, Jul 13, 2017 at 05:01:42PM +0100, Mark Brown wrote: > > > This might be problematic if the clock to enable is stored in another > > > node. > > > Let's add a function that allows to attach a clock that has already been > >

Re: [PATCH v1 1/2] drm/bridge/synopsys: dsi: Constify funcs structures

2017-07-18 Thread Laurent Pinchart
Hi Philippe, Thank you for the patch. On Tuesday 18 Jul 2017 13:43:51 Philippe CORNU wrote: > Signed-off-by: Philippe CORNU Reviewed-by: Laurent Pinchart > --- > drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 2 +- > 1 file changed,

[PATCH 00/14] mgag200 fixes

2017-07-18 Thread Takashi Iwai
Hi, this is a summer cleanup sale, a patchset containing various fixes for mgag200 driver taken from openSUSE / SUSE kernels. They have been in our kernels for ages, so at least they are supposed to be stable. Most of patches came from Egbert, and one PM patch from me that is a resubmission of

[PATCH 01/14] drm/mgag200: Add doublescan and interlace support

2017-07-18 Thread Takashi Iwai
From: Egbert Eich This code was ported from the xorg mga driver. The doublescreen_allowed and interlace_allowed flags are set unconditionally for all models for now. Signed-off-by: Egbert Eich Signed-off-by: Takashi Iwai ---

Re: [PATCH libdrm] libdrm_amdgpu: add kernel semaphore support

2017-07-18 Thread Marek Olšák
Hi Dave, If you just add "get" functions for what you need from amdgpu objects, that should be fine. Marek On Mon, Jul 17, 2017 at 11:00 PM, Dave Airlie wrote: > On 18 July 2017 at 03:02, Christian König wrote: >> Am 17.07.2017 um 05:36 schrieb Dave

[PATCH] drm/cirrus: Add missing drm_connector_unregister()

2017-07-18 Thread Takashi Iwai
From: Egbert Eich When destroying connector unregister it. Signed-off-by: Egbert Eich Signed-off-by: Takashi Iwai --- drivers/gpu/drm/cirrus/cirrus_mode.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/cirrus/cirrus_mode.c

[PATCH 07/14] drm/mgag200: Add support for MATROX PCI device IDs 0x520 and 0x521

2017-07-18 Thread Takashi Iwai
From: Egbert Eich Add two more models G200_PCI and G200 for PCI device IDs 0x520 and 0x521, respectively. They need to retrieve the reference clock and pclk min/max values from BIOS, and set up the PLLs accordingly. Signed-off-by: Egbert Eich Signed-off-by: Takashi

[PATCH 10/14] drm/mgag200: Don't use crtc_* parameters for validation

2017-07-18 Thread Takashi Iwai
From: Egbert Eich The crtc_* are wrong references as the mode parameters to validate, use the ones without crtc_ prefix instead. Signed-off-by: Egbert Eich Signed-off-by: Takashi Iwai --- drivers/gpu/drm/mgag200/mgag200_mode.c | 12 ++-- 1

[PATCH 03/14] drm/mgag200: Fix memleak in error path in mgag200_bo_create()

2017-07-18 Thread Takashi Iwai
From: Egbert Eich The allocated struct mgag200_bo was not freed in all error paths. This patch consolidates error handling and fixes this. Signed-off-by: Egbert Eich Signed-off-by: Takashi Iwai --- drivers/gpu/drm/mgag200/mgag200_ttm.c | 11

[PATCH 11/14] drm/mgag200: Consolidate depth/bpp handling

2017-07-18 Thread Takashi Iwai
From: Egbert Eich The depth/bpp handling for chips with limited memory in commit 918be888d613 ("drm/mgag200: on cards with < 2MB VRAM default to 16-bit") was incomplete: the bpp limits were applied to mode validation. This consolidates dpeth/bpp handling, adds it to mode

[PATCH 12/14] drm/mgag200: Add command line option to specify preferred depth

2017-07-18 Thread Takashi Iwai
From: Egbert Eich G200 is old hardware. When KMS was designed around 2007 none of the chipsets current at this time had any restrictions to video modes depending on the depth. Thus video modes are validated independent of the depth which is purely a property of the scanout buffer.

[PATCH 08/14] drm/mgag200: Cleanup cursor BOs properly

2017-07-18 Thread Takashi Iwai
From: Egbert Eich Cursor BOs should be cleaned up properly on error or when unloading the driver. Signed-off-by: Egbert Eich Signed-off-by: Takashi Iwai --- drivers/gpu/drm/mgag200/mgag200_main.c | 11 +-- 1 file changed, 9 insertions(+), 2

[PATCH 04/14] drm/mgag200: Free container instead of member in mga_user_framebuffer_destroy()

2017-07-18 Thread Takashi Iwai
From: Egbert Eich Technically freeing mga_fb->base is the same as freeing mga_fb as 'base' the first member of the data structure. Still this makes it cleaner. Signed-off-by: Egbert Eich Signed-off-by: Takashi Iwai ---

[PATCH 06/14] drm/mgag200: Simplify function mgag200_ttm_placement()

2017-07-18 Thread Takashi Iwai
From: Egbert Eich Just a code refactoring, no functional change. Signed-off-by: Egbert Eich Signed-off-by: Takashi Iwai --- drivers/gpu/drm/mgag200/mgag200_drv.h | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git

[PATCH 13/14] drm/mgag200: Add mode validation debugging code

2017-07-18 Thread Takashi Iwai
From: Egbert Eich Give more verbose debug message at mode bandwidth checks. Signed-off-by: Egbert Eich Signed-off-by: Takashi Iwai --- drivers/gpu/drm/mgag200/mgag200_mode.c | 51 +- 1 file changed, 31 insertions(+),

[PATCH 05/14] drm/mgag200: Initialize data needed to map fbdev memory

2017-07-18 Thread Takashi Iwai
From: Egbert Eich Due to a missing initialization there was no way to map fbdev memory. Thus for example using the Xserver with the fbdev driver failed. This fix adds initialization for fix.smem_start and fix.smem_len in the fb_info structure, which fixes this problem.

[PATCH 14/14] drm/mgag200: Implement basic PM support

2017-07-18 Thread Takashi Iwai
Add a basic PM support to mgag200 driver. As there is no hardware specific init codes for this device, the callbacks are written in a fairly simple way. Signed-off-by: Takashi Iwai --- drivers/gpu/drm/mgag200/mgag200_drv.c | 41 +++ 1 file changed,

[PATCH 09/14] drm/mgag200: Add missing drm_connector_unregister()

2017-07-18 Thread Takashi Iwai
From: Egbert Eich When destroying connector unregister it. Signed-off-by: Egbert Eich Signed-off-by: Takashi Iwai --- drivers/gpu/drm/mgag200/mgag200_mode.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/mgag200/mgag200_mode.c

[PATCH 02/14] drm/mgag200: Add additional limits for certain G200 variants

2017-07-18 Thread Takashi Iwai
From: Egbert Eich According to the use UMS X.Org driver G200 WB chips don't support doublescan Signed-off-by: Egbert Eich Signed-off-by: Takashi Iwai --- drivers/gpu/drm/mgag200/mgag200_mode.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)

[PATCH 5/5] drm/ast: Actually load DP501 firmware when required

2017-07-18 Thread Takashi Iwai
From: Egbert Eich The ast driver has a code to load the DP501 firmware, but it's never used. This patch implements its actual usage by requesting the firmware on demand, and release the firmware at exit as well. Also the path contains a few cleanups and makes relevant functions

[PATCH 0/5] ast fix patches

2017-07-18 Thread Takashi Iwai
Hi, here is another clearance sale, a patchset containing fixes for ast driver, dug from openSUSE / SUSE kernels. All fixes came from Egbert. thanks, Takashi === Egbert Eich (5): drm/ast: Simplify function ast_bo_unpin() drm/ast: Free container instead of member in

[PATCH 4/5] drm/ast: Add an crtc_disable callback to the crtc helper funcs

2017-07-18 Thread Takashi Iwai
From: Egbert Eich Implement the proper CRTC disablement, just like done in mgag200 driver. Signed-off-by: Egbert Eich Signed-off-by: Takashi Iwai --- drivers/gpu/drm/ast/ast_mode.c | 16 1 file changed, 16 insertions(+) diff --git

[PATCH 1/5] drm/ast: Simplify function ast_bo_unpin()

2017-07-18 Thread Takashi Iwai
From: Egbert Eich Just a code refactoring, no functional change. Signed-off-by: Egbert Eich Signed-off-by: Takashi Iwai --- drivers/gpu/drm/ast/ast_ttm.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git

Re: [PATCH v2] dt-bindings: display: sunxi: Improve endpoint ID scheme readability

2017-07-18 Thread Maxime Ripard
On Tue, Jul 18, 2017 at 12:02:01PM +0800, Chen-Yu Tsai wrote: > The explanation for the endpoint ID numbering scheme is convoluted > and hard to understand. > > This patch aims to improve the readability of it by combining the > existing two paragraphs, while also providing a diagram example, >

[PATCH i-g-t 0/2] Unrelated hotplug uevent masking out actual test result

2017-07-18 Thread Paul Kocialkowski
This patch introduces a workaround for a case where a uevent is issued by the kernel because of DP link training failing on a connector unrelated to the current test. Since the test depends on receiving a hotplug uevent, it previously passed even though it should not have. False positives also

[PATCH i-g-t 1/2] tests/chamelium: Skip suspend/resume test with unreliable hotplug event

2017-07-18 Thread Paul Kocialkowski
It may occur that a hotplug uevent is detected at resume, even though it does not indicate that an actual hotplug happened. This is the case when link training fails on any other connector. There is currently no way to distinguish what connector caused a hotplug uevent, nor what the reason for

[PATCH i-g-t 1/2] tests/chamelium: Skip suspend/resume test with unreliable hotplug event

2017-07-18 Thread Paul Kocialkowski
It may occur that a hotplug uevent is detected at resume, even though it does not indicate that an actual hotplug happened. This is the case when link training fails on any other connector. There is currently no way to distinguish what connector caused a hotplug uevent, nor what the reason for

[PATCH] drm/radeon: Set depth on low mem to 16 bpp instead of 8 bpp

2017-07-18 Thread Takashi Iwai
From: Egbert Eich The radeon driver reduces the framebuffer resolution to 8bpp if a device with less than 32MB VRAM is found. This causes the framebuffer to run in 8 bit paletted mode. For a text console this is not an issue as 256 different colors is more than one gets on a VGA

[PATCH 2/5] drm/ast: Free container instead of member in ast_user_framebuffer_destroy()

2017-07-18 Thread Takashi Iwai
From: Egbert Eich Technically freeing ast_fb->base is the same as freeing ast_fb as 'base' the first member of the data structure. Still this makes it cleaner. Signed-off-by: Egbert Eich Signed-off-by: Takashi Iwai --- drivers/gpu/drm/ast/ast_main.c

[PATCH 3/5] drm/ast: Fix memleak in error path in ast_bo_create()

2017-07-18 Thread Takashi Iwai
From: Egbert Eich The allocated struct ast_bo was not freed in all error paths. This patch consolidates error handling and fixes this. Signed-off-by: Egbert Eich Signed-off-by: Takashi Iwai --- drivers/gpu/drm/ast/ast_ttm.c | 11 ++- 1 file

[PATCH i-g-t 2/2] tests/chamelium: Catch and flush hotplug uevents after each plug

2017-07-18 Thread Paul Kocialkowski
This adds calls to igt_hotplug_detected and igt_flush_hotplugs to catch and flush hotplugs from connector unplug (due to chamelium reset) and plug. These need to be intercepted so that they are not delayed and issued after resume, providing a false positive for the test result. In addition, the

Re: [RFC][PATCH v2] drm: kirin: Add mode_valid logic to avoid mode clocks we can't generate

2017-07-18 Thread John Stultz
On Tue, Jul 18, 2017 at 4:10 AM, Jose Abreu wrote: > Hi John, > > > On 18-07-2017 05:22, John Stultz wrote: >> Currently the hikey dsi logic cannot generate accurate byte >> clocks values for all pixel clock values. Thus if a mode clock >> is selected that cannot match

Re: [RFC 2/7] drm: Add GEM backed framebuffer library

2017-07-18 Thread Noralf Trønnes
Den 12.07.2017 15.46, skrev Noralf Trønnes: Add a library for drivers that can use a simple representation of a GEM backed framebuffer. Signed-off-by: Noralf Trønnes --- This patch adds a gem backed drm_framebuffer like this: struct drm_fb_gem { /** * @base:

Re: [PATCH 00/11] drm/sun4i: add CEC support

2017-07-18 Thread Maxime Ripard
Hi, On Tue, Jul 11, 2017 at 11:06:52PM +0200, Hans Verkuil wrote: > On 11/07/17 22:39, Maxime Ripard wrote: > > On Tue, Jul 11, 2017 at 08:30:33AM +0200, Hans Verkuil wrote: > >> From: Hans Verkuil > >> > >> This patch series adds CEC support for the sun4i HDMI

Re: [PATCH] drm: Update docs around gem_free_object

2017-07-18 Thread Eric Anholt
Daniel Vetter writes: > Not all places correctly stated that gem_free_object_unlocked is the > one to use. > > Reported-by: Eric Anholt Cc: Eric Anholt signature.asc Description: PGP

Re: [PATCH 07/14] proc/kcore: hide a harmless warning

2017-07-18 Thread Arnd Bergmann
On Tue, Jul 18, 2017 at 9:55 PM, Ard Biesheuvel wrote: > On 18 July 2017 at 20:53, Arnd Bergmann wrote: >> On Fri, Jul 14, 2017 at 2:28 PM, Ard Biesheuvel >> wrote: >>> On 14 July 2017 at 10:25, Arnd Bergmann

Re: [PATCH 07/14] proc/kcore: hide a harmless warning

2017-07-18 Thread Arnd Bergmann
On Tue, Jul 18, 2017 at 10:07 PM, Ard Biesheuvel wrote: > On 18 July 2017 at 21:01, Arnd Bergmann wrote: >> On Tue, Jul 18, 2017 at 9:55 PM, Ard Biesheuvel > > Ah, now it makes sense. I was a bit surprised that > -Wtautological-compare complains about

Re: [PATCH v2] drm/i915: Consistently use enum pipe for PCH transcoders

2017-07-18 Thread Matthias Kaehlcke
Hi Daniel, El Tue, Jul 18, 2017 at 08:39:50AM +0200 Daniel Vetter ha dit: > On Mon, Jul 17, 2017 at 11:14:03AM -0700, Matthias Kaehlcke wrote: > > The current code uses in some instances enum transcoder for PCH > > transcoders and enum pipe in others. This is error prone and clang > > raises

Re: [Intel-gfx] [PATCH i-g-t 1/2] tests/chamelium: Skip suspend/resume test with unreliable hotplug event

2017-07-18 Thread Chris Wilson
Quoting Paul Kocialkowski (2017-07-18 16:16:26) > It may occur that a hotplug uevent is detected at resume, even though it > does not indicate that an actual hotplug happened. This is the case when > link training fails on any other connector. > > There is currently no way to distinguish what

Re: [PATCH 1/2] drm/ttm: Implement vm_operations_struct.access v2

2017-07-18 Thread Michel Dänzer
On 17/07/17 11:57 PM, Felix Kuehling wrote: Allows gdb to access contents of user mode mapped BOs. System memory is handled by TTM using kmap. Other memory pools require a new driver callback in ttm_bo_driver. v2: * kmap only one page at a time * swap in BO if needed * make driver callback more

Re: [RFC 1/7] drm/gem: Add drm_gem_dumb_map_offset()

2017-07-18 Thread Noralf Trønnes
Den 12.07.2017 15.45, skrev Noralf Trønnes: 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

Re: [PULL] drm-misc-next

2017-07-18 Thread Sean Paul
On Tue, Jul 18, 2017 at 2:42 PM, Sean Paul wrote: > Hi Dave, > Here's the first -misc-next pull for 4.13 err, 4.14. > , definitely the largest one I've > sent to you. There's nothing too disruptive, a bunch of clean-up series which > tidy up atomic macros, return value

Re: [PATCH v2] drm/i915: Explicit the connector name for DP link training result

2017-07-18 Thread Manasi Navare
On Tue, Jul 18, 2017 at 05:25:36PM +0300, Paul Kocialkowski wrote: > This adds the connector name when printing a debug message about the DP > link training result. It is useful to figure out what connector is > failing when multiple DP connectors are used. > Thanks for the patch, this does make

Re: [Intel-gfx] [PATCH i-g-t 0/2] Unrelated hotplug uevent masking out actual test result

2017-07-18 Thread Lyude Paul
For the whole series Reviewed-by: Lyude will push in just a sec On Tue, 2017-07-18 at 18:16 +0300, Paul Kocialkowski wrote: > This patch introduces a workaround for a case where a uevent is > issued > by the kernel because of DP link training failing on a connector >

Re: [PATCH 6/8] drm: Allow DSI devices to be registered before the host registers.

2017-07-18 Thread Eric Anholt
Archit Taneja writes: > On 07/15/2017 04:28 AM, Eric Anholt wrote: >> Archit Taneja writes: >> >>> On 06/28/2017 01:28 AM, Eric Anholt wrote: When a mipi_dsi_host is registered, the DT is walked to find any child nodes with compatible

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

2017-07-18 Thread Eric Anholt
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 --- drivers/gpu/drm/bridge/panel.c | 30

[PATCH v5 6/6] drm/panel: Add support for the Raspberry Pi 7" Touchscreen.

2017-07-18 Thread Eric Anholt
This driver communicates with the Atmel microcontroller for sequencing the poweron of the TC358762 DSI-DPI bridge and controlling the backlight PWM. v2: Set the same default orientation as the closed source firmware used, which is the best for viewing angle. v3: Rewrite as an i2c client

[PATCH v5 5/6] dt-bindings: Document the Raspberry Pi Touchscreen nodes.

2017-07-18 Thread Eric Anholt
This doesn't yet cover input, but the driver does get the display working when the firmware is disabled from talking to our I2C lines. Signed-off-by: Eric Anholt Acked-by: Rob Herring --- .../panel/raspberrypi,7inch-touchscreen.txt| 49

[PATCH v5 4/6] drm: Allow DSI devices to be registered before the host registers.

2017-07-18 Thread Eric Anholt
When a mipi_dsi_host is registered, the DT is walked to find any child nodes with compatible strings. Those get registered as DSI devices, and most DSI panel drivers are mipi_dsi_drivers that attach to those nodes. There is one special case currently, the adv7533 bridge, where the bridge probes

  1   2   >