[PATCH 5/5] drm/atomic-helpers: make mode_set hooks optional

2015-02-22 Thread Laurent Pinchart
Hi Daniel, On Sunday 22 February 2015 19:53:23 Laurent Pinchart wrote: > On Sunday 22 February 2015 12:24:20 Daniel Vetter wrote: > > With runtime PM the hw might still be off while doing the ->mode_set > > callbacks - runtime PM get/put should only happen in the > > enable/disable hooks to proper

[PATCH 4/5] drm/atomic-helper: Rename commmit_post/pre_planes

2015-02-22 Thread Laurent Pinchart
Hi Daniel, Thank you for the patch. On Sunday 22 February 2015 12:24:19 Daniel Vetter wrote: > These names only make sense because of backwards compatability with > the order used by the crtc helper library. There's not really any real > requirement in the ordering here. > > So rename them to so

[PATCH 5/5] drm/atomic-helpers: make mode_set hooks optional

2015-02-22 Thread Laurent Pinchart
Hi Daniel, Thank you for the patch. On Sunday 22 February 2015 12:24:20 Daniel Vetter wrote: > With runtime PM the hw might still be off while doing the ->mode_set > callbacks - runtime PM get/put should only happen in the > enable/disable hooks to properly support DPMS. Which essentially makes >

[PATCHv2 1/2] drm: atmel-hlcdc: Add PM suspend/resume support

2015-02-22 Thread Sylvain Rochet
Hello Andrzej, On Tue, Feb 17, 2015 at 10:47:23AM +0100, Andrzej Hajda wrote: > On 02/12/2015 09:49 PM, Sylvain Rochet wrote: > > +static int atmel_hlcdc_dc_resume(struct drm_device *dev) > > +{ > > + struct drm_crtc *crtc; > > + > > + drm_modeset_lock_all(dev); > > + > > + list_for_each_ent

[PATCH] drm/msm/atomic: Don't leak atomic commit object when commit fails

2015-02-22 Thread Rob Clark
On Sun, Feb 22, 2015 at 5:58 PM, Laurent Pinchart wrote: > If the atomic commit fails due to completion wait interruption the > atomic commit object is not freed and is thus leaked. Free it. > > Signed-off-by: Laurent Pinchart thx, queued up on my fixes branch BR, -R > --- > drivers/gpu/drm/m

[PATCHv3 2/2] drm: atmel-hlcdc: Add pinctrl PM select sleep, default state in CRTC suspend/resume

2015-02-22 Thread Sylvain Rochet
Some LCD panels have back-powering issue when un-powered, allows users to use an alternate pinctrl "sleep" in order to clamp outputs to a wanted state at suspend. Signed-off-by: Sylvain Rochet Acked-by: Boris Brezillon --- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | 3 +++ 1 file changed,

[PATCHv3 1/2] drm: atmel-hlcdc: Add PM suspend/resume support

2015-02-22 Thread Sylvain Rochet
On suspend: switch off CRTC if not already suspended with runtime PM On resume: switch on CRTC if we were not already suspended from runtime PM while suspending. Signed-off-by: Sylvain Rochet --- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 47 1 file changed, 47

[PATCHv3 0/2] drm: atmel-hlcdc: PM support

2015-02-22 Thread Sylvain Rochet
This series adds basic PM support for Atmel HLCDC. This series depends on: [PATCH] drm: atmel-hlcdc: remove useless pm_runtime_put_sync in probe <1423841785-23105-1-git-send-email-boris.brezillon at free-electrons.com> [PATCH v3] drm: atmel-hlcdc: Atomic mode-setting conversion <1423842415-23

[PATCH] drm: Kconfig: Let all DRM_GEM_CMA_HELPER related macros depend on HAVE_DMA_ATTRS

2015-02-22 Thread Chen Gang S
On 2/3/15 05:26, Chen Gang S wrote: [...] > > - Xtensa gcc5 cross compiler has issues: > >it causes more than 10 broken areas with allmodconfig (but no issues >with defconfig). I guess, in fact, it is only 1 real issue (which can >cause all breaks), and I shall try to fix it within

[PATCH] Fix i2c table parsing for dcb 4.1

2015-02-22 Thread Stefan Huehner
Code before looked only at bit 31 to decide if a port is unused. However dcb 4.1 spec says 0x1F in bits 31-27 and 26-22 means unused. This fixed hdmi monitor detection on GM206. --- drm/nouveau/nvkm/subdev/bios/i2c.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drm/nou

[PATCH 2/2] drm: WARN if drm_handle_vblank is called errornously

2015-02-22 Thread Daniel Vetter
KMS drivers are in full control of their irq and vblank handling, if they get a vblank interrupt before drm_vblank_init or after drm_vblank_cleanup that's just a driver bug. For ums driver there's only r128 and radeon which support vblank, and they call drm_vblank_init in their driver load functio

[PATCH 1/2] drm/irq: Don't call ->get_vblank_counter directly from irq_uninstall/cleanup

2015-02-22 Thread Daniel Vetter
The pipe might already have been shut down, and then it's not a good idea to call hw accessor functions. Instead use the same logic as drm_vblank_off which has all the necessary checks to avoid troubles or inconsistency. Noticed by Imre while reviewing my patches to remove some sanity checks from

[Bug 72575] Several games are unable to detect VRAM size

2015-02-22 Thread bugzilla-dae...@freedesktop.org
. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150222/e4fe480f/attachment.html>

[GIT PULL]: few dma-buf updates for 3.20-rc1

2015-02-22 Thread Sumit Semwal
Hi Linus, On 22 February 2015 at 01:42, Linus Torvalds wrote: > On Fri, Feb 20, 2015 at 8:27 AM, Sumit Semwal > wrote: >> >> Could you please pull a few dma-buf changes for 3.20-rc1? Nothing >> fancy, minor cleanups. > > No. > > I pulled, and immediately unpulled again. > > This is complete shi

[PATCH] drm: Fix the CRTC_STEREO_DOUBLE_ONLY define to include stero modes

2015-02-22 Thread Daniel Vetter
On Tue, Feb 17, 2015 at 10:05:27AM -0800, Matt Roper wrote: > On Mon, Feb 16, 2015 at 03:12:31PM +, Damien Lespiau wrote: > > The CRTC_STEREO_DOUBLE_ONLY define was introduced in commit: > > > > commit ecb7e16bf187bc369cf6a5cd108582c01329980d > > Author: Gustavo Padovan > > Date: Mon

Update front buffer without CPU interaction?

2015-02-22 Thread Daniel Vetter
On Mon, Feb 16, 2015 at 01:43:07PM +0100, Volker Vogelhuber wrote: > I'm currently trying to setup a rendering pipe on an Intel Baytrail E3845 > cpu. > In our product we want to have an FPGA streaming video images to a > predefined memory area using bus master dma and render those images using > Op

[PATCH] drm/udl: add enable/disable_vblank stubs

2015-02-22 Thread Daniel Vetter
On Sun, Feb 15, 2015 at 04:52:39PM -0800, Aaron Plattner wrote: > vblank_disable_and_save calls the driver's disable_vblank hook > unconditionally, > which crashes the udl driver since it doesn't implement it. Fix this by > adding > stub implementations of these functions identical to the qxl on

[PATCH 1/3] drm/layerscape: Add fsl dcu DRM driver

2015-02-22 Thread Daniel Vetter
On Fri, Feb 13, 2015 at 07:03:54PM +0800, Jianwei Wang wrote: > This patch add support for Two Dimensional Animation and Compositing > Engine (2D-ACE) on the Freescale LS102x SoCs. > > 2D-ACE is a Freescale display controller. It supports at most four > plane and provide an hardware cursor. > > T

[PATCH 5/5] drm/atomic-helpers: make mode_set hooks optional

2015-02-22 Thread Daniel Vetter
With runtime PM the hw might still be off while doing the ->mode_set callbacks - runtime PM get/put should only happen in the enable/disable hooks to properly support DPMS. Which essentially makes these callbacks useless for drivers support runtime PM, so make them optional. Again motivated by disc

[PATCH 4/5] drm/atomic-helper: Rename commmit_post/pre_planes

2015-02-22 Thread Daniel Vetter
These names only make sense because of backwards compatability with the order used by the crtc helper library. There's not really any real requirement in the ordering here. So rename them to something more descriptive and update the kerneldoc a bit. Motivated in a discussion with Laurent about how

[PATCH 3/5] drm/atomic: Rename drm_atomic_helper_commit_pre_planes() state argument

2015-02-22 Thread Daniel Vetter
From: Laurent Pinchart The argument contains a pointer to the old state, rename it to old_state like in all other commit helper functions. Signed-off-by: Laurent Pinchart Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_atomic_helper.c | 10 +- 1 file changed, 5 insertions(+), 5 d

[PATCH 2/5] drm: If available use atomic state in getcrtc ioctl

2015-02-22 Thread Daniel Vetter
This way drivers fully converted to atomic don't need to update these legacy state variables in their modeset code any more. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_crtc.c | 25 ++--- 1 file changed, 18 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/

[PATCH 1/5] drm: Add DRM_DEBUG_ATOMIC

2015-02-22 Thread Daniel Vetter
Atomic state handling adds a lot of indirection and complexity between simple updates and drivers. For easier debugging the diagnostic output is therefore rather chatty. Which is great for tracking down atomic issues, but really annoying otherwise. Add a new DRM_DEBUG_ATOMIC to be able to filter t

[PATCH] drm/irq: Don't call ->get_vblank_counter directly from irq_uninstall/cleanup

2015-02-22 Thread Daniel Vetter
The pipe might already have been shut down, and then it's not a good idea to call hw accessor functions. Instead use the same logic as drm_vblank_off which has all the necessary checks to avoid troubles or inconsistency. Noticed by Imre while reviewing my patches to remove some sanity checks from

[PATCH] drm/radeon: Fix regression with suspend/resume

2015-02-22 Thread Christian König
On 22.02.2015 00:18, Ross Zwisler wrote: > On Wed, 2015-02-18 at 13:02 +0100, Christian König wrote: >> Well, what the patch does is just changing where buffers are placed in >> memory. E.g. now we place the buffer at the end of memory as well. >> >> So I can imagine at least three possible causes

[PATCH] drm: Fix deadlock due to getconnector locking changes

2015-02-22 Thread Daniel Vetter
In daniel at phenom:~/linux/src$ git show ccfc08655 commit ccfc08655d5fd5076828f45fb09194c070f2f63a Author: Rob Clark Date: Thu Dec 18 16:01:48 2014 -0500 drm: tweak getconnector locking We need to extend the locking to cover connector->state reading for atomic drivers, but the above comm

[PATCH 2/2] drm/msm: Support NV12MT format in mdp4

2015-02-22 Thread Daniel Vetter
On Fri, Feb 20, 2015 at 3:42 PM, Beeresh Gopal wrote: > +/* Samsung framebuffer modifiers */ > + > +/* > + * NV12 64x32 Tiled > + * > + * 2 planes Y and CbCr, grouped into 64x32 macro tiles, > + * with a non-standard order in memory (Z-shape). > + * > + * Pixel layout identical to DRM_FORMAT_NV21

[KERNEL] Re: [KERNEL] Regression bug in drm/i915, Wrong assumption in commit e11aa36 breaks suspend on at least lenovo x61

2015-02-22 Thread Daniel Vetter
On Wed, Feb 18, 2015 at 8:03 PM, Klaus Ethgen wrote: > Am Mi den 18. Feb 2015 um 16:39 schrieb Jani Nikula: >> On Tue, 17 Feb 2015, Klaus Ethgen wrote: >> > After solving the conflicts, I applied the revert (see attachment) to >> > v3.18.7. I think it should also apply to the current head. With

[PATCH 4/5] drm/atomic-helper: Rename commmit_post/pre_planes

2015-02-22 Thread Rob Clark
On Sun, Feb 22, 2015 at 6:24 AM, Daniel Vetter wrote: > These names only make sense because of backwards compatability with > the order used by the crtc helper library. There's not really any real > requirement in the ordering here. > > So rename them to something more descriptive and update the

[PATCH 3/5] drm/atomic: Rename drm_atomic_helper_commit_pre_planes() state argument

2015-02-22 Thread Rob Clark
On Sun, Feb 22, 2015 at 6:24 AM, Daniel Vetter wrote: > From: Laurent Pinchart > > The argument contains a pointer to the old state, rename it to old_state > like in all other commit helper functions. > > Signed-off-by: Laurent Pinchart > Signed-off-by: Daniel Vetter Reviewed-by: Rob Clark

[Intel-gfx] [PATCH 2/5] drm: If available use atomic state in getcrtc ioctl

2015-02-22 Thread Rob Clark
On Sun, Feb 22, 2015 at 6:24 AM, Daniel Vetter wrote: > This way drivers fully converted to atomic don't need to update these > legacy state variables in their modeset code any more. > > Signed-off-by: Daniel Vetter Reviewed-by: Rob Clark > --- > drivers/gpu/drm/drm_crtc.c | 25 +

[PATCH 1/5] drm: Add DRM_DEBUG_ATOMIC

2015-02-22 Thread Rob Clark
On Sun, Feb 22, 2015 at 6:24 AM, Daniel Vetter wrote: > Atomic state handling adds a lot of indirection and complexity between > simple updates and drivers. For easier debugging the diagnostic output > is therefore rather chatty. Which is great for tracking down atomic > issues, but really annoyi

[Bug 89275] PC sometimes hangs on suspend

2015-02-22 Thread bugzilla-dae...@freedesktop.org
for errors to post here, so if you'll tell me where to look i'll post them. -- You are receiving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-dev

[Bug 83226] Allow use of ColorRange and ColorSpace in xorg.conf.d files

2015-02-22 Thread bugzilla-dae...@freedesktop.org
: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150222/85e32090/attachment.html>

[Bug 42960] Display does not work when resuming from suspend

2015-02-22 Thread bugzilla-dae...@freedesktop.org
iving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150222/76efb16d/attachment.html>