[PATCH 3/4] drm/fb-helper: Add module option to disable fbdev emulation

2015-08-26 Thread Archit Taneja
On 08/25/2015 07:15 PM, Daniel Vetter wrote: > Faster than recompiling. > > Note that restore_fbdev_mode_unlocked is a bit special and the only > one which returns an error code when fbdev isn't there - i915 needs > that one to not fall over with some additional fbcon related restore > code. Ever

[PATCH] drm/msm/mdp5: enable clocks in hw_init and set_irqmask

2015-08-26 Thread Archit Taneja
mdp5_hw_init and mdp5_set_irqmask configure registers but may not have clocks enabled. Add mdp5_enable/disable calls in these funcs to ensure clocks are enabled. We need this until we get proper runtime pm support. Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/mdp/mdp5/mdp5_irq.c | 10 ++

[PATCH 3/3] modetest: add atomic page flip support

2015-08-26 Thread Hyungwon Hwang
This patch adds support for atomic page flip. User can specify -V option with the plane id for testing atomic page flipping. --- tests/modetest/modetest.c | 153 -- 1 file changed, 149 insertions(+), 4 deletions(-) diff --git a/tests/modetest/modetest.c

[PATCH 1/3] modetest: introduce get_prop_info() for getting property id and type

2015-08-26 Thread Hyungwon Hwang
Modetest gets the property name from user to set it. So the name must be converted to its id. Until now, this is done in the set_property(). But to support atomic modeset in modetest, this logic should be separated from the fuction, because atomic modeset and legacy modeset use different IOCTLs. S

[PATCH 2/3] modetest: add atomic modeset support

2015-08-26 Thread Hyungwon Hwang
This patch adds support for atomic modeset. Using -a option, user can make modeset to use DRM_IOCTL_MODE_ATOMIC instead of legacy IOCTLs. Also, by using -w option, user can set the property as before. Signed-off-by: Hyungwon Hwang --- tests/modetest/modetest.c | 221 +

[GIT PULL] drm/rockchip: fixes and new features

2015-08-26 Thread Mark yao
Hi Dave Here are some fixes and some new features for rockchip drm, tested on popmetal rk3288 board, can you land them? The following changes since commit db56176025cee5e242dfeed5f4e304d095d29fa3: Revert "drm/atomic: Call ww_acquire_done after check phase is complete" (2015-08-25 17

[PATCH v4] dma-buf: Add ioctls to allow userspace to flush

2015-08-26 Thread Thomas Hellstrom
Hi, Tiago. On 08/26/2015 02:02 AM, Tiago Vignatti wrote: > From: Daniel Vetter > > The userspace might need some sort of cache coherency management e.g. when CPU > and GPU domains are being accessed through dma-buf at the same time. To > circumvent this problem there are begin/end coherency marke

[Intel-gfx] [PATCH] drm/i915: Fix build warning on 32-bit

2015-08-26 Thread Daniel Vetter
On Mon, Aug 17, 2015 at 05:19:09PM +, Zanoni, Paulo R wrote: > Em Sex, 2015-08-14 às 12:35 +0200, Thierry Reding escreveu: > > From: Thierry Reding > > > > The gtt.stolen_size field is of type size_t, and so should be printed > > using %zu to avoid build warnings on either 32-bit and 64-bit

[Intel-gfx] [PATCH] drm/i915: Fix build warning on 32-bit

2015-08-26 Thread Jani Nikula
On Wed, 26 Aug 2015, Daniel Vetter wrote: > On Mon, Aug 17, 2015 at 05:19:09PM +, Zanoni, Paulo R wrote: >> Em Sex, 2015-08-14 às 12:35 +0200, Thierry Reding escreveu: >> > From: Thierry Reding >> > >> > The gtt.stolen_size field is of type size_t, and so should be printed >> > using %zu to

[Intel-gfx] [PATCH] drm/i915: Fix build warning on 32-bit

2015-08-26 Thread Daniel Vetter
On Wed, Aug 26, 2015 at 10:26:35AM +0300, Jani Nikula wrote: > On Wed, 26 Aug 2015, Daniel Vetter wrote: > > On Mon, Aug 17, 2015 at 05:19:09PM +, Zanoni, Paulo R wrote: > >> Em Sex, 2015-08-14 às 12:35 +0200, Thierry Reding escreveu: > >> > From: Thierry Reding > >> > > >> > The gtt.stolen

[PATCH libdrm] omap: Initialize DMA BUF file descriptor to -1

2015-08-26 Thread Thierry Reding
From: Thierry Reding Commit c86dabfc9f04 ("omap: zero is a valid fd number, treat it as such") corrected checks for valid file descriptors, but the OMAP buffer object code initializes the DMA-BUF file descriptor to 0 (as a result of calloc()'ing the structure). Obviously this isn't going to work

[PATCH 3/4] drm/fb-helper: Add module option to disable fbdev emulation

2015-08-26 Thread Archit Taneja
On 08/26/2015 10:42 AM, Archit Taneja wrote: > > > On 08/25/2015 07:15 PM, Daniel Vetter wrote: >> Faster than recompiling. >> >> Note that restore_fbdev_mode_unlocked is a bit special and the only >> one which returns an error code when fbdev isn't there - i915 needs >> that one to not fall over

[Intel-gfx] [PATCH 4/5] Documentation: drm: Convert KMS Properties HTML table to CALS

2015-08-26 Thread Daniel Vetter
On Tue, Aug 25, 2015 at 05:10:54PM +0100, Graham Whaley wrote: > On Tue, 2015-08-25 at 16:29 +0200, Daniel Vetter wrote: > > On Tue, Aug 25, 2015 at 10:26:44AM +0100, Graham Whaley wrote: > > > The KMS Properties table is in HTML format, which is not supported > > > for building pdfdocs, resulting

[PATCH v2 1/7] drm/vc4: Add devicetree bindings for VC4.

2015-08-26 Thread Luc Verhaegen
On Tue, Aug 18, 2015 at 02:54:02PM -0700, Eric Anholt wrote: > VC4 is the GPU (display and 3D) subsystem present on the 2835 and some > other Broadcom SoCs. > ... > +Broadcom VC4 GPU > + > +The VC4 device present on the Raspberry Pi includes a display system > +with HDMI output and the HVS scaler f

[RFC 04/13] drm/dp: Enhanced framing capability is DP 1.1+

2015-08-26 Thread Jani Nikula
On Wed, 12 Aug 2015, Thierry Reding wrote: > From: Thierry Reding > > The enhanced framing capability was added in DisplayPort 1.1, so any > code dealing with it needs to be protected accordingly. > > Signed-off-by: Thierry Reding > --- > drivers/gpu/drm/drm_dp_helper.c | 5 +++-- > 1 file chan

[PATCH] drm/crtc: Add a helper func to get a registered crtc from its index

2015-08-26 Thread Thierry Reding
h the drm_driver functions over. Thierry -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150826/b4ba366e/attachment.sig>

[PATCH 1/3] drm/dp: add drm_dp_tps3_supported helper

2015-08-26 Thread Jani Nikula
Cc: Thierry Reding Signed-off-by: Jani Nikula --- include/drm/drm_dp_helper.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/include/drm/drm_dp_helper.h b/include/drm/drm_dp_helper.h index 499e9f625aef..8c52d0ef1fc9 100644 --- a/include/drm/drm_dp_helper.h +++ b/include/drm/drm_dp_he

[PATCH 2/3] drm/i915/dp: use the drm dp helper for determining sink tps3 support

2015-08-26 Thread Jani Nikula
No functional changes. Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/intel_dp.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c index f10a4724b841..60dca34d2f0f 100644 --- a/drivers/gpu/drm/i915/intel_d

[PATCH 3/3] drm/i915/dp: move TPS3 logic to where it's used

2015-08-26 Thread Jani Nikula
There is no need to have a separate flag for tps3 as the information is only used at one location. Move the logic there to make it easier to follow. Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/intel_dp.c | 31 +-- drivers/gpu/drm/i915/intel_drv.h | 1 - 2 fi

[PATCH 3/4] drm/fb-helper: Add module option to disable fbdev emulation

2015-08-26 Thread Daniel Vetter
On Wed, Aug 26, 2015 at 02:14:37PM +0530, Archit Taneja wrote: > > > On 08/26/2015 10:42 AM, Archit Taneja wrote: > > > > > >On 08/25/2015 07:15 PM, Daniel Vetter wrote: > >>Faster than recompiling. > >> > >>Note that restore_fbdev_mode_unlocked is a bit special and the only > >>one which returns

[PATCH] drm/fb-helper: Use -errno return in restore_mode_unlocked

2015-08-26 Thread Daniel Vetter
On Tue, Aug 25, 2015 at 03:20:02PM -0400, Rob Clark wrote: > On Tue, Aug 25, 2015 at 11:20 AM, Daniel Vetter > wrote: > > Using bool and returning true upon error is very uncommon. Also an int > > return value is actually what all the callers which did check it seem > > to have expected. > > > >

[PATCH 3/4] drm/fb-helper: Add module option to disable fbdev emulation

2015-08-26 Thread Daniel Vetter
On Wed, Aug 26, 2015 at 01:34:58PM +0200, Daniel Vetter wrote: > On Wed, Aug 26, 2015 at 02:14:37PM +0530, Archit Taneja wrote: > > > > > > On 08/26/2015 10:42 AM, Archit Taneja wrote: > > > > > > > > >On 08/25/2015 07:15 PM, Daniel Vetter wrote: > > >>Faster than recompiling. > > >> > > >>Note t

[PATCH] drm/dp: Adjust i2c-over-aux retry count based on message size and i2c bus speed

2015-08-26 Thread Ville Syrjälä
On Wed, Aug 26, 2015 at 12:11:56PM +0100, simon.farnsworth at onelan.com wrote: > On Tuesday 25 August 2015 18:10:14 ville.syrjala at linux.intel.com wrote: > > From: Ville Syrjälä > > > > Calculate the number of retries we should do for each i2c-over-aux > > message based on the time it takes

[PATCH 2/4] drm/fb-helper: atomic restore_fbdev_mode()..

2015-08-26 Thread Daniel Vetter
On Tue, Aug 25, 2015 at 03:35:58PM -0400, Rob Clark wrote: > Add support for using atomic code-paths for restore_fbdev_mode(). > > Signed-off-by: Rob Clark > --- > drivers/gpu/drm/drm_atomic_helper.c | 131 > +--- > drivers/gpu/drm/drm_fb_helper.c | 74 +

[PATCH 1/7] drm/vc4: Add devicetree bindings for VC4.

2015-08-26 Thread Daniel Vetter
On Tue, Aug 25, 2015 at 04:42:18PM -0400, Rob Clark wrote: > On Mon, Aug 24, 2015 at 9:47 AM, Rob Herring wrote: > > On Mon, Aug 17, 2015 at 1:30 PM, Eric Anholt wrote: > >> Stephen Warren writes: > >> > >>> On 08/12/2015 06:56 PM, Eric Anholt wrote: > Signed-off-by: Eric Anholt > >>> > >>

[PATCH 1/7] drm/vc4: Add devicetree bindings for VC4.

2015-08-26 Thread Thierry Reding
next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150826/6ac61672/attachment.sig>

[PATCH 2/3] drm:msm: Initial Add Writeback Support (V2)

2015-08-26 Thread Daniel Vetter
On Tue, Aug 25, 2015 at 9:11 PM, Rob Clark wrote: > On Tue, Aug 25, 2015 at 3:05 AM, Daniel Vetter wrote: >> On Wed, Aug 19, 2015 at 03:00:04PM -0400, Rob Clark wrote: >>> On Tue, Apr 7, 2015 at 2:09 PM, Jilai Wang wrote: >>> So one thing that I wanted sorting out before we let userspace see >>>

[PATCH v4] dma-buf: Add ioctls to allow userspace to flush

2015-08-26 Thread Daniel Vetter
On Wed, Aug 26, 2015 at 08:49:00AM +0200, Thomas Hellstrom wrote: > Hi, Tiago. > > On 08/26/2015 02:02 AM, Tiago Vignatti wrote: > > From: Daniel Vetter > > > > The userspace might need some sort of cache coherency management e.g. when > > CPU > > and GPU domains are being accessed through dma-b

[PATCH 1/7] drm/vc4: Add devicetree bindings for VC4.

2015-08-26 Thread Thierry Reding
me principles that the host1x infrastructure uses could be implemented in a similar way for other DRM drivers. You can easily collect information about subdevices by walking the device tree and matching on known compatible strings. And you can also instantiate the top-level device from driver code rather than have it in DT. It should still be possible to make this work without an artificial device node in DT. The component and master infrastructure is largely orthogonal to that, and as far as I remember the only blocker is the need for a top-level device. I wonder if perhaps this could be made to work by binding the master to the top- level SoC device. Obviously adding the node in DT is easier, but to my knowledge easy has never been a good excuse for mangling DT. Perhaps that's different these days... Thierry -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150826/23936e24/attachment-0001.sig>

[PATCH 3/4] drm/fb-helper: Add module option to disable fbdev emulation

2015-08-26 Thread Archit Taneja
On 08/26/2015 05:04 PM, Daniel Vetter wrote: > On Wed, Aug 26, 2015 at 02:14:37PM +0530, Archit Taneja wrote: >> >> >> On 08/26/2015 10:42 AM, Archit Taneja wrote: >>> >>> >>> On 08/25/2015 07:15 PM, Daniel Vetter wrote: Faster than recompiling. Note that restore_fbdev_mode_unlocke

[PATCH v4] dma-buf: Add ioctls to allow userspace to flush

2015-08-26 Thread Thomas Hellstrom
On 08/26/2015 02:10 PM, Daniel Vetter wrote: > On Wed, Aug 26, 2015 at 08:49:00AM +0200, Thomas Hellstrom wrote: >> Hi, Tiago. >> >> On 08/26/2015 02:02 AM, Tiago Vignatti wrote: >>> From: Daniel Vetter >>> >>> The userspace might need some sort of cache coherency management e.g. when >>> CPU >>>

[PATCH 3/4] drm/fb-helper: Add module option to disable fbdev emulation

2015-08-26 Thread Archit Taneja
On 08/26/2015 05:07 PM, Daniel Vetter wrote: > On Wed, Aug 26, 2015 at 01:34:58PM +0200, Daniel Vetter wrote: >> On Wed, Aug 26, 2015 at 02:14:37PM +0530, Archit Taneja wrote: >>> >>> >>> On 08/26/2015 10:42 AM, Archit Taneja wrote: On 08/25/2015 07:15 PM, Daniel Vetter wrote:

[PATCH 3/4] drm/fb-helper: Add module option to disable fbdev emulation

2015-08-26 Thread Daniel Vetter
On Wed, Aug 26, 2015 at 05:59:14PM +0530, Archit Taneja wrote: > > > On 08/26/2015 05:07 PM, Daniel Vetter wrote: > >On Wed, Aug 26, 2015 at 01:34:58PM +0200, Daniel Vetter wrote: > >>On Wed, Aug 26, 2015 at 02:14:37PM +0530, Archit Taneja wrote: > >>> > >>> > >>>On 08/26/2015 10:42 AM, Archit Ta

[PATCH v4] dma-buf: Add ioctls to allow userspace to flush

2015-08-26 Thread Daniel Vetter
On Wed, Aug 26, 2015 at 02:28:30PM +0200, Thomas Hellstrom wrote: > On 08/26/2015 02:10 PM, Daniel Vetter wrote: > > On Wed, Aug 26, 2015 at 08:49:00AM +0200, Thomas Hellstrom wrote: > >> Hi, Tiago. > >> > >> On 08/26/2015 02:02 AM, Tiago Vignatti wrote: > >>> From: Daniel Vetter > >>> > >>> The u

[PATCH libdrm] omap: Initialize DMA BUF file descriptor to -1

2015-08-26 Thread Emil Velikov
On 26 August 2015 at 09:16, Thierry Reding wrote: > From: Thierry Reding > > Commit c86dabfc9f04 ("omap: zero is a valid fd number, treat it as > such") corrected checks for valid file descriptors, but the OMAP buffer > object code initializes the DMA-BUF file descriptor to 0 (as a result of > ca

[PATCH 1/3] drm/dp: add drm_dp_tps3_supported helper

2015-08-26 Thread Alex Deucher
On Wed, Aug 26, 2015 at 7:33 AM, Jani Nikula wrote: > Cc: Thierry Reding > Signed-off-by: Jani Nikula Reviewed-by: Alex Deucher > --- > include/drm/drm_dp_helper.h | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/include/drm/drm_dp_helper.h b/include/drm/drm_dp_helper.h > inde

[PATCH] drm/msm/mdp5: enable clocks in hw_init and set_irqmask

2015-08-26 Thread h...@codeaurora.org
Hi Archit, > mdp5_hw_init and mdp5_set_irqmask configure registers but may not have > clocks enabled. > > Add mdp5_enable/disable calls in these funcs to ensure clocks are > enabled. We need this until we get proper runtime pm support. > > Signed-off-by: Archit Taneja > --- > drivers/gpu/drm/msm

[PATCH v2 00/22] Enable gpu switching on the MacBook Pro

2015-08-26 Thread Lukas Wunner
Hi Daniel, On Tue, Aug 25, 2015 at 10:21:20AM +0200, Daniel Vetter wrote: > On Tue, Aug 25, 2015 at 09:36:47AM +0200, Lukas Wunner wrote: > > I've overhauled locking once more because previously all EDID reads > > were serialized even on machines which don't use vga_switcheroo at all. > > Seems it

[PATCH] drm/atomic-helper: Add option to update planes only on active crtc

2015-08-26 Thread Daniel Vetter
With drivers supporting runtime pm it's generally not a good idea to touch the hardware when it's off. Add an option to the commit_planes helper to support this case. Note that the helpers already add all planes on a crtc when a modeset happens, hence plane updates will not be lost if drivers set

[Bug 84614] radeon gpu crash /kernel crash when using dynamic light in borderlands 2

2015-08-26 Thread bugzilla-dae...@freedesktop.org
ubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150826/846a7a0d/attachment-0001.html>

[PATCH] drm/msm/mdp5: enable clocks in hw_init and set_irqmask

2015-08-26 Thread Rob Clark
2015-08-26 9:55 GMT-04:00 : > Hi Archit, > >> mdp5_hw_init and mdp5_set_irqmask configure registers but may not have >> clocks enabled. >> >> Add mdp5_enable/disable calls in these funcs to ensure clocks are >> enabled. We need this until we get proper runtime pm support. >> >> Signed-off-by: Arch

[PATCH v4] dma-buf: Add ioctls to allow userspace to flush

2015-08-26 Thread Tiago Vignatti
On 08/26/2015 09:58 AM, Daniel Vetter wrote: > The other is that right now there's no user nor implementation in sight > which actually does range-based flush optimizations, so I'm pretty much > expecting we'll get it wrong. Maybe instead we should go one step further > and remove the range from th

[PATCH v4] dma-buf: Add ioctls to allow userspace to flush

2015-08-26 Thread Thomas Hellstrom
On 08/26/2015 04:32 PM, Tiago Vignatti wrote: > On 08/26/2015 09:58 AM, Daniel Vetter wrote: >> The other is that right now there's no user nor implementation in sight >> which actually does range-based flush optimizations, so I'm pretty much >> expecting we'll get it wrong. Maybe instead we should

[PATCH v4] dma-buf: Add ioctls to allow userspace to flush

2015-08-26 Thread Daniel Vetter
On Wed, Aug 26, 2015 at 11:32:30AM -0300, Tiago Vignatti wrote: > On 08/26/2015 09:58 AM, Daniel Vetter wrote: > >The other is that right now there's no user nor implementation in sight > >which actually does range-based flush optimizations, so I'm pretty much > >expecting we'll get it wrong. Maybe

[PATCH v4] dma-buf: Add ioctls to allow userspace to flush

2015-08-26 Thread Tiago Vignatti
On 08/26/2015 11:51 AM, Daniel Vetter wrote: > On Wed, Aug 26, 2015 at 11:32:30AM -0300, Tiago Vignatti wrote: >> On 08/26/2015 09:58 AM, Daniel Vetter wrote: >>> The other is that right now there's no user nor implementation in sight >>> which actually does range-based flush optimizations, so I'm

[PATCH] drm/msm/mdp5: enable clocks in hw_init and set_irqmask

2015-08-26 Thread h...@codeaurora.org
> 2015-08-26 9:55 GMT-04:00 : >> Hi Archit, >> >>> mdp5_hw_init and mdp5_set_irqmask configure registers but may not have >>> clocks enabled. >>> >>> Add mdp5_enable/disable calls in these funcs to ensure clocks are >>> enabled. We need this until we get proper runtime pm support. >>> >>> Signed-o

drm/msm/dsi: hs_zero timing

2015-08-26 Thread Hai Li
Hi Werner, Thanks for sharing this. The DPHY timings in downstream dtsi are exactly the same as the excel calculation, but slightly different from the output of drm code as you posted. (e.g hs_zero is 116 vs 118) I think it is caused by some precision loss during driver calculation, but I need

[PATCH v4] dma-buf: Add ioctls to allow userspace to flush

2015-08-26 Thread Thomas Hellstrom
> 26 aug 2015 kl. 16:58 skrev Tiago Vignatti : > >> On 08/26/2015 11:51 AM, Daniel Vetter wrote: >>> On Wed, Aug 26, 2015 at 11:32:30AM -0300, Tiago Vignatti wrote: On 08/26/2015 09:58 AM, Daniel Vetter wrote: The other is that right now there's no user nor implementation in sight >>

[PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Daniel Vetter
Very strictly speaking this is possible if you have special hw and genlocked CRTCs. In general switching a plane between two active CRTC just won't work so well and is probably not tested at all. Just forbid it. I've put this into the core since I really couldn't come up with a case where we don't

[PATCH 03/11] drm/exynos: add prepare and cleanup phases for planes

2015-08-26 Thread Gustavo Padovan
Hi Inki, 2015-08-24 Inki Dae : > On 2015년 08월 16일 01:26, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > .prepare_plane() and .cleanup_plane() allows to perform extra operations > > before and after the update of planes. For FIMD for example this will > > be used to enable disable

[PATCH 1/2] drm/exynos: remove legacy ->suspend()/resume()

2015-08-26 Thread Gustavo Padovan
Hi, What about this patch? We need it to avoid the WARN_ON added by patch 2/2 that was already picked up by Daniel. Gustavo 2015-08-13 Gustavo Padovan : > From: Gustavo Padovan > > These legacy helpers should only be used by shadow-attaching drivers. > KMS drivers has its own way to h

[Intel-gfx] [PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Ville Syrjälä
On Wed, Aug 26, 2015 at 05:41:23PM +0200, Daniel Vetter wrote: > Very strictly speaking this is possible if you have special hw and > genlocked CRTCs. In general switching a plane between two active CRTC > just won't work so well and is probably not tested at all. Just forbid > it. > > I've put th

[PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Rob Clark
On Wed, Aug 26, 2015 at 11:41 AM, Daniel Vetter wrote: > Very strictly speaking this is possible if you have special hw and > genlocked CRTCs. In general switching a plane between two active CRTC > just won't work so well and is probably not tested at all. Just forbid > it. So, I expect msm shou

[PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Rob Clark
On Wed, Aug 26, 2015 at 12:03 PM, Rob Clark wrote: > On Wed, Aug 26, 2015 at 11:41 AM, Daniel Vetter > wrote: >> Very strictly speaking this is possible if you have special hw and >> genlocked CRTCs. In general switching a plane between two active CRTC >> just won't work so well and is probably

[PATCH] drm/edid: Allow comma separated edid binaries. (v2)

2015-08-26 Thread Bob Paauwe
Allow comma separated filenames in the edid_firmware parameter. For example: edid_firmware=eDP-1:edid/1280x480.bin,DP-2:edid/1920x1080.bin v2: Use strsep() to simplify parsing of comma seperated string. (Matt) Move initial bail before strdup. (Matt) Reviewed-by: Matt Roper Signed-off-by: B

[PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Ville Syrjälä
On Wed, Aug 26, 2015 at 12:07:30PM -0400, Rob Clark wrote: > On Wed, Aug 26, 2015 at 12:03 PM, Rob Clark wrote: > > On Wed, Aug 26, 2015 at 11:41 AM, Daniel Vetter > > wrote: > >> Very strictly speaking this is possible if you have special hw and > >> genlocked CRTCs. In general switching a plan

[Intel-gfx] [PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Daniel Stone
On Wed, 2015-08-26 at 18:53 +0300, Ville Syrjälä wrote: > On Wed, Aug 26, 2015 at 05:41:23PM +0200, Daniel Vetter wrote: > > Very strictly speaking this is possible if you have special hw and > > genlocked CRTCs. In general switching a plane between two active > > CRTC > > just won't work so well

[RFC][PATCH 1/2] dt-bindings: drm/mediatek: Add Mediatek DRM dts binding

2015-08-26 Thread Philipp Zabel
Hi, overall it looks to me like this binding is modeled after the Linux DRM abstractions. Instead, the device nodes should be modeled after the hardware. Describing each function block as a separate device node is probably not the best choice, as would be combining all devices in the mmsys range

[PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Rob Clark
On Wed, Aug 26, 2015 at 12:30 PM, Ville Syrjälä wrote: > On Wed, Aug 26, 2015 at 12:07:30PM -0400, Rob Clark wrote: >> On Wed, Aug 26, 2015 at 12:03 PM, Rob Clark wrote: >> > On Wed, Aug 26, 2015 at 11:41 AM, Daniel Vetter > > ffwll.ch> wrote: >> >> Very strictly speaking this is possible if yo

[PATCH v2 00/11] drm/exynos: improve atomic modesetting

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan Hi, This patchset adds a couple of changes to improve atomic modesetting: * add check for the START shadow register for FIMD to only finish the update when the screen was actually updated. * add asynchronous atomic commit, so now page flips can be run asynchronously. I

[PATCH v2 01/11] drm/exynos: don't track enabled state at exynos_crtc

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan struct drm_crtc already stores the enabled state of the crtc thus we don't need to replicate enabled in exynos_drm_crtc. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/exynos/exynos_drm_crtc.c | 16 drivers/gpu/drm/exynos/exynos_drm_drv.h | 1 - 2 f

[PATCH] drm/exynos: fimd: move window protect code to prepare/atomic_flush

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan Only set/clear the update bit in the CRTC's .atomic_begin()/flush() so all planes are really committed at the same time. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/exynos/exynos_drm_fimd.c | 57 +++- 1 file changed, 34 insertions(+), 23

[PATCH v2 02/11] drm/exynos: fimd: unify call to exynos_drm_crtc_finish_pageflip()

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan Unify handling of finished plane update to prepare for a following patch that will check for the START and START_S regs to really make sure that the plane was updated. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/exynos/exynos_drm_fimd.c | 10 +- 1 file chan

[PATCH v2 03/11] drm/exynos: add begin and flush phases for planes

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan .atomic_begin() and .atomic_flush() allows to perform extra operations before and after the update of planes. For FIMD for example this will be used to enable disable the shadow protection bit. Signed-off-by: Gustavo Padovan --- v2: rename prepare_plane/cleanup_plane to a

[PATCH v2 04/11] drm/exynos: fimd: move window protect code to atomic_begin/flush

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan Only set/clear the update bit in the CRTC's .atomic_begin()/flush() so all planes are really committed at the same time. Signed-off-by: Gustavo Padovan --- v2: rename prepare_plane/cleanup_plane to atomic_begin/atomic_flush --- drivers/gpu/drm/exynos/exynos_drm_fimd.c |

[PATCH v2 05/11] drm/exynos: check for pending fb before finish update

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan The current code was ignoring the end of update for all overlay planes, caring only for the primary plane update in case of pageflip. This change adds a change to start to check for pending updates for all planes through exynos_plane->pending_fb. At the start of plane updat

[PATCH v2 06/11] drm/exynos: add macro to get the address of START_S reg

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan This macro is need to get the value of the START shadow register, that will tell if an framebuffer is currently displayed on the screen or not. Signed-off-by: Gustavo Padovan --- include/video/samsung_fimd.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/vide

[PATCH v2 07/11] drm/exynos: fimd: only finish update if START == START_S

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan fimd_update_plane() programs BUF_START[win] and during the update BUF_START[win] is copied to BUF_START_S[win] (its shadow register) and starts scanning out, then it raises a irq. The fimd_irq_handler, in the case we have a pending_fb, will check the fb value was copied to

[PATCH v2 08/11] drm/exynos: add atomic asynchronous commit

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan The atomic modesetting interfaces supports async commits that should be implemented by the drivers. If drm core requests an async commit exynos_atomic_commit() will now schedule a work task to run the update later. It also serializes commits that needs to run on the same cr

[PATCH v2 09/11] drm/exynos: wait all planes updates to finish

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan Add infrastructure to wait for all planes updates to finish by using an atomic_t variable to track how many pending updates we are waiting plus a wait_queue for the wait part. It also changes vblank behaviour and keeps it enabled for all types of updates Signed-off-by: Gus

[PATCH v2 10/11] drm/exynos: remove wait queue for pending page flip

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan Exynos atomic commit procedures already does this job of waiting for pending updates to finish, that means using pending_flip_queue is pointless now because the disable CRTC procedure will never happen during a page_flip. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm

[PATCH v2 11/11] drm/exynos: Enable atomic modesetting feature

2015-08-26 Thread Gustavo Padovan
From: Gustavo Padovan Now that atomic modesetting is implemented for exynos enable the DRIVER_ATOMIC flag on the driver's features. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/exynos/exynos_drm_drv.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/e

drm/msm/dsi: hs_zero timing

2015-08-26 Thread Rob Clark
On Wed, Aug 26, 2015 at 1:39 PM, Werner Johansson wrote: > > On Aug 26, 2015 08:34, "Hai Li" wrote: >> >> Hi Werner, >> >> Thanks for sharing this. The DPHY timings in downstream dtsi are exactly >> the same as the excel calculation, but slightly different from the output of >> drm code as you po

drm/msm/dsi: hs_zero timing

2015-08-26 Thread Hai Li
-- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150826/91186538/attachment.html>

[PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Ville Syrjälä
On Wed, Aug 26, 2015 at 01:38:36PM -0400, Rob Clark wrote: > On Wed, Aug 26, 2015 at 12:30 PM, Ville Syrjälä > wrote: > > On Wed, Aug 26, 2015 at 12:07:30PM -0400, Rob Clark wrote: > >> On Wed, Aug 26, 2015 at 12:03 PM, Rob Clark wrote: > >> > On Wed, Aug 26, 2015 at 11:41 AM, Daniel Vetter >>

[PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Rob Clark
On Wed, Aug 26, 2015 at 1:53 PM, Ville Syrjälä wrote: > On Wed, Aug 26, 2015 at 01:38:36PM -0400, Rob Clark wrote: >> On Wed, Aug 26, 2015 at 12:30 PM, Ville Syrjälä >> wrote: >> > On Wed, Aug 26, 2015 at 12:07:30PM -0400, Rob Clark wrote: >> >> On Wed, Aug 26, 2015 at 12:03 PM, Rob Clark >

drm/msm/dsi: hs_zero timing

2015-08-26 Thread Rob Clark
On Wed, Aug 26, 2015 at 2:22 PM, Werner Johansson wrote: > > On Aug 26, 2015 10:46, "Rob Clark" wrote: >> >> btw, w/ some of these clk rounding issues, I suspect we need 'struct >> drm_display_mode' to be able to represent mode clock with greater >> accuracy than Khz.. > > Interesting point! Howe

[PATCH 0/6] Properly detect swiotlb.

2015-08-26 Thread jgli...@redhat.com
So this is only build tested as i am away from hardware right now. Idea is to provide reliable way to check if swiotlb is in use for a device or not. It seems swiotlb_nr_tbl() is no longer reliable for that. Please test. Cheers, Jérôme Glisse

[PATCH 1/6] swiotlb: Add helper to know if it is in use for a specific device.

2015-08-26 Thread jgli...@redhat.com
From: Jérôme Glisse Some device like GPU do things differently if swiotlb is in use. We use to rely on swiotlb_nr_tbl() to know if swiotlb was enabled or not but this is unreliable. Patch add a simple helpers to check if any of the dma_ops associated with a device points to the swiotlb function

[PATCH 2/6] drm/radeon: Use swiotlb_in_use() to know if swiotlb is enabled or not.

2015-08-26 Thread jgli...@redhat.com
From: Jérôme Glisse We can not rely on swiotlb_nr_tbl() to know if swiotlb is in use or not for our device. Use the new helper to determine that. Signed-off-by: Jérôme Glisse Cc: Konrad Rzeszutek Wilk Cc: Alex Deucher Cc: Ben Skeggs Cc: Dave Airlie Cc: lkml at vger.kernel.org Cc: Daniel

[PATCH 3/6] drm/nouveau: Use swiotlb_in_use() to know if swiotlb is enabled or not.

2015-08-26 Thread jgli...@redhat.com
From: Jérôme Glisse We can not rely on swiotlb_nr_tbl() to know if swiotlb is in use or not for our device. Use the new helper to determine that. Signed-off-by: Jérôme Glisse Cc: Konrad Rzeszutek Wilk Cc: Alex Deucher Cc: Ben Skeggs Cc: Dave Airlie Cc: lkml at vger.kernel.org Cc: Daniel

[PATCH 5/6] drm/i915: Use swiotlb_in_use() to know if swiotlb is enabled.

2015-08-26 Thread jgli...@redhat.com
From: Jérôme Glisse We can not rely on swiotlb_nr_tbl() to know if swiotlb is in use or not for our device. Use the new helper to determine that. Signed-off-by: Jérôme Glisse Cc: Konrad Rzeszutek Wilk Cc: Alex Deucher Cc: Ben Skeggs Cc: Dave Airlie Cc: lkml at vger.kernel.org Cc: Daniel

[PATCH 6/6] drm/amdgpu: Use swiotlb_in_use() to know if swiotlb is enabled.

2015-08-26 Thread jgli...@redhat.com
From: Jérôme Glisse We can not rely on swiotlb_nr_tbl() to know if swiotlb is in use or not for our device. Use the new helper to determine that. Signed-off-by: Jérôme Glisse Cc: Konrad Rzeszutek Wilk Cc: Alex Deucher Cc: Ben Skeggs Cc: Dave Airlie Cc: lkml at vger.kernel.org Cc: Daniel

[PATCH 4/6] drm/vmwgfx: Use swiotlb_in_use() to know if swiotlb is enabled.

2015-08-26 Thread jgli...@redhat.com
From: Jérôme Glisse We can not rely on swiotlb_nr_tbl() to know if swiotlb is in use or not for our device. Use the new helper to determine that. Signed-off-by: Jérôme Glisse Cc: Konrad Rzeszutek Wilk Cc: Alex Deucher Cc: Ben Skeggs Cc: Dave Airlie Cc: lkml at vger.kernel.org Cc: Daniel

[PATCH 1/6] swiotlb: Add helper to know if it is in use for a specific device.

2015-08-26 Thread Konrad Rzeszutek Wilk
On Wed, Aug 26, 2015 at 02:52:02PM -0400, jglisse at redhat.com wrote: > From: Jérôme Glisse > > Some device like GPU do things differently if swiotlb is in use. We > use to rely on swiotlb_nr_tbl() to know if swiotlb was enabled or not > but this is unreliable. Patch add a simple helpers to ch

[PATCH 1/2] drm/exynos: remove legacy ->suspend()/resume()

2015-08-26 Thread Daniel Vetter
On Wed, Aug 26, 2015 at 12:50:44PM -0300, Gustavo Padovan wrote: > Hi, > > What about this patch? We need it to avoid the WARN_ON added by patch > 2/2 that was already picked up by Daniel. That patch is only for 4.4, so not too time critical to get the exynos one in. But might be good to get it i

[Bug 86351] HDMI audio garbled output on Radeon R9 280X

2015-08-26 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=86351 --- Comment #16 from Lorenzo Bona --- Any hint on this? -- You are receiving this mail because: You are watching the assignee of the bug.

[PATCH 1/6] swiotlb: Add helper to know if it is in use for a specific device.

2015-08-26 Thread Jerome Glisse
On Wed, Aug 26, 2015 at 03:02:31PM -0400, Konrad Rzeszutek Wilk wrote: > On Wed, Aug 26, 2015 at 02:52:02PM -0400, jglisse at redhat.com wrote: > > From: Jérôme Glisse > > > > Some device like GPU do things differently if swiotlb is in use. We > > use to rely on swiotlb_nr_tbl() to know if swio

[PATCH 1/6] swiotlb: Add helper to know if it is in use for a specific device.

2015-08-26 Thread Konrad Rzeszutek Wilk
On Wed, Aug 26, 2015 at 03:26:42PM -0400, Jerome Glisse wrote: > On Wed, Aug 26, 2015 at 03:02:31PM -0400, Konrad Rzeszutek Wilk wrote: > > On Wed, Aug 26, 2015 at 02:52:02PM -0400, jglisse at redhat.com wrote: > > > From: Jérôme Glisse > > > > > > Some device like GPU do things differently if

[PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Daniel Vetter
Very strictly speaking this is possible if you have special hw and genlocked CRTCs. In general switching a plane between two active CRTC just won't work so well and is probably not tested at all. Just forbid it. I've put this into the core since right now no helper or driver copes with it, no user

[PATCH 1/3] drm/dp: Define AUX_RETRY_INTERVAL as 500 us

2015-08-26 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Currently we react to native and i2c defers by waiting either 400-500 us or 500-600 us, depending on which code path we take. Consolidate them all to one define AUX_RETRY_INTERVAL which defines the minimum interval. Since we've been using two different intervals pick the lon

[PATCH v2 2/3] drm/dp: Adjust i2c-over-aux retry count based on message size and i2c bus speed

2015-08-26 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Calculate the number of retries we should do for each i2c-over-aux message based on the time it takes to perform the i2c transfer vs. the aux transfer. We assume the shortest possible length for the aux transfer, and the longest possible (exluding clock stretching) for the i

[PATCH 3/3] drm/dp: Add dp_aux_i2c_speed_khz module param to set the assume i2c bus speed

2015-08-26 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä To help with debugging i2c-over-aux issues, add a module parameter than can be used to tweak the assumed i2c bus speed, and thus the maximum number of retries we will do for each aux message. Cc: Simon Farnsworth Cc: moosotc at gmail.com Signed-off-by: Ville Syrjälä ---

[Bug 87856] Driver load fails with no error on ppc64 host

2015-08-26 Thread bugzilla-dae...@freedesktop.org
attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150826/cb5c74d7/attachment.html>

[PATCH 1/6] swiotlb: Add helper to know if it is in use for a specific device.

2015-08-26 Thread Jerome Glisse
On Wed, Aug 26, 2015 at 03:44:52PM -0400, Konrad Rzeszutek Wilk wrote: > On Wed, Aug 26, 2015 at 03:26:42PM -0400, Jerome Glisse wrote: > > On Wed, Aug 26, 2015 at 03:02:31PM -0400, Konrad Rzeszutek Wilk wrote: > > > On Wed, Aug 26, 2015 at 02:52:02PM -0400, jglisse at redhat.com wrote: > > > > Fro

[PATCH 1/6] swiotlb: Add helper to know if it is in use for a specific device.

2015-08-26 Thread Konrad Rzeszutek Wilk
On Wed, Aug 26, 2015 at 04:31:50PM -0400, Jerome Glisse wrote: > On Wed, Aug 26, 2015 at 03:44:52PM -0400, Konrad Rzeszutek Wilk wrote: > > On Wed, Aug 26, 2015 at 03:26:42PM -0400, Jerome Glisse wrote: > > > On Wed, Aug 26, 2015 at 03:02:31PM -0400, Konrad Rzeszutek Wilk wrote: > > > > On Wed, Aug

[Bug 91676] Random TONGA lockups during normal usage

2015-08-26 Thread bugzilla-dae...@freedesktop.org
il because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150826/067af358/attachment-0001.html>

[PATCH 1/6] swiotlb: Add helper to know if it is in use for a specific device.

2015-08-26 Thread Jerome Glisse
On Wed, Aug 26, 2015 at 04:38:14PM -0400, Konrad Rzeszutek Wilk wrote: > On Wed, Aug 26, 2015 at 04:31:50PM -0400, Jerome Glisse wrote: > > On Wed, Aug 26, 2015 at 03:44:52PM -0400, Konrad Rzeszutek Wilk wrote: > > > On Wed, Aug 26, 2015 at 03:26:42PM -0400, Jerome Glisse wrote: > > > > On Wed, Aug

[Intel-gfx] [PATCH] drm/atomic: refuse changing CRTC for planes directly

2015-08-26 Thread Rob Clark
On Wed, Aug 26, 2015 at 3:49 PM, Daniel Vetter wrote: > Very strictly speaking this is possible if you have special hw and > genlocked CRTCs. In general switching a plane between two active CRTC > just won't work so well and is probably not tested at all. Just forbid > it. > > I've put this into

[Bug 86351] HDMI audio garbled output on Radeon R9 280X

2015-08-26 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=86351 --- Comment #17 from Alex Deucher --- Maybe something with the audio driver? This doesn't seem to be a GPU driver issue. -- You are receiving this mail because: You are watching the assignee of the bug.

  1   2   >