[git pull] drm fixes

2015-09-24 Thread Dave Airlie
Hi Linus, Most of the GPU drivers people were at XDC last week, so I didn't get much to send, so I let it rollover until this week. Also Alex was away for 3 weeks so amdgpu/radeon got a bit more stuff than usual in one go. I've been trying to figure out some 4.2 issues with i915 still (that

linux-next: manual merge of the drm-intel tree with the drm-intel-fixes tree

2015-09-24 Thread Stephen Rothwell
Hi Jani, On Thu, 24 Sep 2015 11:57:05 +0300 Jani Nikula wrote: > > That was the right thing to do. > > The former commit is headed for v4.3, and there will have to be another > version of it for -next. This will cause you another conflict, and you > should resolve it with the same approach.

[PATCH] drm/radeon: add quirck for MSI R7 370

2015-09-24 Thread mrader3...@yandex.ru
From: Maxim Sheviakov Just adds the quirck for MSI R7 370 Armor 2X Bug: https://bugs.freedesktop.org/show_bug.cgi?id=91294 Signed-off-by: Maxim Sheviakov --- --- /home/maxik/linux/drivers/gpu/drm/radeon/si_dpm.c.orig 2015-09-23 22:49:45.462513212 + +++

[PATCH] drm/nouveau: fix memory leak

2015-09-24 Thread Sudip Mukherjee
On Fri, Sep 11, 2015 at 03:00:56PM +0530, Sudip Mukherjee wrote: > If pm_runtime_get_sync() we were going to "out" but we missed freeing > vma. > > Signed-off-by: Sudip Mukherjee > --- Hi Ben, Another gentle ping for another patch. regards sudip

[PATCH] drm/gma500: fix double freeing

2015-09-24 Thread Sudip Mukherjee
On Wed, Sep 09, 2015 at 06:20:40PM +0530, Sudip Mukherjee wrote: > If backing->stolen is true then we were freeing backing by calling > psb_gtt_free_range() but we called it again after unlocking the mutex. > Lets make it NULL after freeing in psb_gtt_free_range() and check for > NULL before

[PATCH] drm/mgag200: fix memory leak

2015-09-24 Thread Sudip Mukherjee
On Mon, Sep 07, 2015 at 05:28:30PM +0530, Sudip Mukherjee wrote: > If drm_fb_helper_alloc_fbi() fails then we were directly returning > without freeing sysram. Also if drm_fb_helper_alloc_fbi() succeeds but > mgag200_framebuffer_init() fails then we were not releasing sysram and > we were not

[PATCH] drm/nouveau: remove unused function

2015-09-24 Thread Sudip Mukherjee
On Wed, Sep 02, 2015 at 12:08:08PM +0530, Sudip Mukherjee wrote: > coverity.com reported that memset was using a buffer of size 0, on > checking the code it turned out that the function was not being used. So > remove it. > > Signed-off-by: Sudip Mukherjee > --- Hi Ben, A gentle ping. regards

[PATCH 14/16] drm/irq: Add drm_crtc_vblank_count_and_time()

2015-09-24 Thread Daniel Vetter
On Thu, Sep 24, 2015 at 06:35:36PM +0200, Thierry Reding wrote: > From: Thierry Reding > > This function is the KMS native variant of drm_vblank_count_and_time(). > It takes a struct drm_crtc * instead of a struct drm_device * and an > index of the CRTC. > > Eventually the goal is to access

[PATCH 13/16] drm: Move ->get_scanout_position() to struct drm_crtc_funcs

2015-09-24 Thread Daniel Vetter
On Thu, Sep 24, 2015 at 06:35:35PM +0200, Thierry Reding wrote: > From: Thierry Reding > > None of the drivers use this in legacy mode, so it can be converted to > use struct drm_crtc * directly. While at it, also make the sole user of > the callback, drm_calc_vbltimestamp_from_scanoutpos(),

[PATCH 09/16] drm/irq: Use unsigned int pipe in public API

2015-09-24 Thread Russell King - ARM Linux
On Thu, Sep 24, 2015 at 06:35:31PM +0200, Thierry Reding wrote: > This continues the pattern started in commit cc1ef118fc09 ("drm/irq: > Make pipe unsigned and name consistent"). This is applied to the public > APIs and driver callbacks, so pretty much all drivers need to be updated > to match the

[PATCH 08/16] drm/irq: Rename drm_crtc -> crtc

2015-09-24 Thread Daniel Vetter
On Thu, Sep 24, 2015 at 06:35:30PM +0200, Thierry Reding wrote: > From: Thierry Reding > > Since the original crtc parameter was renamed to pipe, there is no > longer a need to artificially prefix the CRTC parameter. > > Signed-off-by: Thierry Reding Applied to drm-misc, thanks. -Daniel >

No more new fbdev drivers, please

2015-09-24 Thread Daniel Vetter
On Thu, Sep 24, 2015 at 07:12:27PM +0200, Ondrej Zary wrote: > On Thursday 24 September 2015 17:59:12 Daniel Vetter wrote: > > On Thu, Sep 24, 2015 at 11:21:15AM -0400, Austin S Hemmelgarn wrote: > > > On 2015-09-24 08:46, Thomas Petazzoni wrote: > > > >Hello, > > > > > > > >On Thu, 24 Sep 2015

No more new fbdev drivers, please

2015-09-24 Thread Ondrej Zary
On Thursday 24 September 2015 17:59:12 Daniel Vetter wrote: > On Thu, Sep 24, 2015 at 11:21:15AM -0400, Austin S Hemmelgarn wrote: > > On 2015-09-24 08:46, Thomas Petazzoni wrote: > > >Hello, > > > > > >On Thu, 24 Sep 2015 15:27:01 +0300, Tomi Valkeinen wrote: > > >>fbdev is (more or less)

[PATCH 3/5] drm: omapdrm: tiler: Fix module autoload for OF platform driver

2015-09-24 Thread Tomi Valkeinen
.freedesktop.org/archives/dri-devel/attachments/20150924/16ad1383/attachment.sig>

[PATCH 6/6] drm/sti: Do not export symbols

2015-09-24 Thread Thierry Reding
From: Thierry Reding None of these exported symbols are used outside of the drm-sti driver, so there is no reason to export them. Cc: Benjamin Gaignard Cc: Vincent Abriou Signed-off-by: Thierry Reding --- drivers/gpu/drm/sti/sti_crtc.c | 3 ---

[PATCH 5/6] drm/sti: Build monolithic driver

2015-09-24 Thread Thierry Reding
From: Thierry Reding There's no use building the individual drivers as separate modules because they are all only useful if combined into a single DRM/KMS device. Cc: Benjamin Gaignard Cc: Vincent Abriou Signed-off-by: Thierry Reding --- drivers/gpu/drm/sti/Makefile

[PATCH 4/6] drm/imx: Do not export symbols

2015-09-24 Thread Thierry Reding
From: Thierry Reding Now that the driver is monolithic there is no longer any need to export these symbols. Cc: Philipp Zabel Signed-off-by: Thierry Reding --- drivers/gpu/drm/imx/imx-drm-core.c | 12 1 file changed, 12 deletions(-) diff --git

[PATCH 3/6] drm/imx: Build monolithic driver

2015-09-24 Thread Thierry Reding
From: Thierry Reding There's no use building the individual drivers as separate modules because they are all only useful if combined into a single DRM/KMS device. Cc: Philipp Zabel Signed-off-by: Thierry Reding --- drivers/gpu/drm/imx/Kconfig| 15

[PATCH 2/6] drm/tegra: Use new multi-driver module helpers

2015-09-24 Thread Thierry Reding
From: Thierry Reding Use the new multi-driver module helpers to get rid of some boilerplate in the module initialization and cleanup functions. Signed-off-by: Thierry Reding --- drivers/gpu/drm/tegra/drm.c | 56 ++---

[PATCH 1/6] driver-core: platform: Provide helpers for multi-driver modules

2015-09-24 Thread Thierry Reding
From: Thierry Reding Some modules register several sub-drivers. Provide a helper that makes it easy to register and unregister a list of sub-drivers, as well as unwind properly on error. Cc: Greg Kroah-Hartman Signed-off-by: Thierry Reding ---

[PATCH 2/2] drm/edid: Implement SCDC support detection

2015-09-24 Thread Thierry Reding
From: Thierry Reding Sinks compliant with the HDMI 2.0 specification may support SCDC, a mechanism for the source and the sink to communicate. Sinks advertise support for this feature in the HDMI Forum Vendor Specific Data Block as defined in the HDMI 2.0 specification,

[PATCH 1/2] drm: Add SCDC helpers

2015-09-24 Thread Thierry Reding
From: Thierry Reding SCDC is a mechanism defined in the HDMI 2.0 specification that allows the source and sink devices to communicate. This commit introduces helpers to access the SCDC and provides the symbolic names for the various registers defined in the specification.

[PATCH 3/5] drm: omapdrm: tiler: Fix module autoload for OF platform driver

2015-09-24 Thread Tomi Valkeinen
--- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: OpenPGP digital signature URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150924/0a0823ec/attachment.sig>

[PATCH 16/16] drm/sti: Use drm_crtc_vblank_*() API

2015-09-24 Thread Thierry Reding
From: Thierry Reding Non-legacy drivers should only use this API to allow per-CRTC data to be eventually moved into struct drm_crtc. Cc: Benjamin Gaignard Cc: Vincent Abriou Signed-off-by: Thierry Reding --- drivers/gpu/drm/sti/sti_crtc.c | 37

[PATCH 15/16] drm/armada: Use drm_crtc_vblank_*() API

2015-09-24 Thread Thierry Reding
From: Thierry Reding Non-legacy drivers should only use this API to allow per-CRTC data to be eventually moved into struct drm_crtc. Cc: Russell King Signed-off-by: Thierry Reding --- drivers/gpu/drm/armada/armada_crtc.c | 17

[PATCH 14/16] drm/irq: Add drm_crtc_vblank_count_and_time()

2015-09-24 Thread Thierry Reding
From: Thierry Reding This function is the KMS native variant of drm_vblank_count_and_time(). It takes a struct drm_crtc * instead of a struct drm_device * and an index of the CRTC. Eventually the goal is to access vblank data through the CRTC only so that the per-CRTC data

[PATCH 13/16] drm: Move ->get_scanout_position() to struct drm_crtc_funcs

2015-09-24 Thread Thierry Reding
From: Thierry Reding None of the drivers use this in legacy mode, so it can be converted to use struct drm_crtc * directly. While at it, also make the sole user of the callback, drm_calc_vbltimestamp_from_scanoutpos(), pass through the CRTC directly. v2: use standard

[PATCH 12/16] drm/msm: Use unsigned int pipe consistently

2015-09-24 Thread Thierry Reding
From: Thierry Reding Use unsigned int pipe consistently to denote the CRTC index. Cc: Rob Clark Signed-off-by: Thierry Reding --- drivers/gpu/drm/msm/msm_drv.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/msm/msm_drv.c

[PATCH 11/16] drm/imx: Use unsigned int for CRTC index

2015-09-24 Thread Thierry Reding
From: Thierry Reding The CRTC index can never be negative, so use an unsigned rather than a signed integer for the loop variable. Cc: Philipp Zabel Signed-off-by: Thierry Reding --- drivers/gpu/drm/imx/imx-drm-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH 10/16] drm/gma500: Use unsigned int pipe consistently

2015-09-24 Thread Thierry Reding
From: Thierry Reding The pipe number can never be negative, so turn variables referring to it to unsigned int. Cc: Patrik Jakobsson Signed-off-by: Thierry Reding --- drivers/gpu/drm/gma500/psb_drv.h | 8 drivers/gpu/drm/gma500/psb_irq.c | 30

[PATCH 09/16] drm/irq: Use unsigned int pipe in public API

2015-09-24 Thread Thierry Reding
From: Thierry Reding This continues the pattern started in commit cc1ef118fc09 ("drm/irq: Make pipe unsigned and name consistent"). This is applied to the public APIs and driver callbacks, so pretty much all drivers need to be updated to match the new prototypes. Cc:

[PATCH 08/16] drm/irq: Rename drm_crtc -> crtc

2015-09-24 Thread Thierry Reding
From: Thierry Reding Since the original crtc parameter was renamed to pipe, there is no longer a need to artificially prefix the CRTC parameter. Signed-off-by: Thierry Reding --- drivers/gpu/drm/drm_irq.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff

[PATCH 07/16] drm/sti: Store correct CRTC index in events

2015-09-24 Thread Thierry Reding
From: Thierry Reding A negative pipe causes a special case to be triggered for drivers that don't have proper VBLANK support. STi does support VBLANKs, so there is no need for the fallback code. Cc: Benjamin Gaignard Cc: Vincent Abriou Signed-off-by: Thierry Reding ---

[PATCH 06/16] drm/rockchip: Store correct CRTC index in events

2015-09-24 Thread Thierry Reding
From: Thierry Reding A negative pipe causes a special case to be triggered for drivers that don't have proper VBLANK support. Rockchip does support VBLANKs, so there is no need for the fallback code. Cc: Mark Yao Signed-off-by: Thierry Reding ---

[PATCH 05/16] drm/imx: Store correct CRTC index in events

2015-09-24 Thread Thierry Reding
From: Thierry Reding A negative pipe causes a special case to be triggered for drivers that don't have proper VBLANK support. i.MX does support VBLANKs, so there is no need for the fallback code. Cc: Philipp Zabel Acked-by: Philipp Zabel Signed-off-by: Thierry Reding ---

[PATCH 04/16] drm/imx: Drop pipe field from struct imx_drm_crtc

2015-09-24 Thread Thierry Reding
From: Thierry Reding Use the drm_crtc_index() helper to determine the pipe number of the CRTC instead. Cc: Philipp Zabel Signed-off-by: Thierry Reding --- drivers/gpu/drm/imx/imx-drm-core.c | 17 - 1 file changed, 8 insertions(+), 9 deletions(-) diff

[PATCH 03/16] drm/imx: Make pipe number unsigned

2015-09-24 Thread Thierry Reding
From: Thierry Reding There's no reason whatsoever why this should ever be negative. The same goes for the number of pipes added to the DRM device. Cc: Philipp Zabel Acked-by: Philipp Zabel Signed-off-by: Thierry Reding --- drivers/gpu/drm/imx/imx-drm-core.c | 6 +++---

[PATCH 02/16] drm/bochs: Store correct CRTC index in events

2015-09-24 Thread Thierry Reding
From: Thierry Reding Previously a negative pipe caused a special case to be triggered for drivers that didn't have proper VBLANK support. The trigger for this special case is now independent of the pipe, so the correct CRTC index can now be stored in events. v2: convert to

[PATCH 01/16] drm/gma500: Sanity-check pipe index

2015-09-24 Thread Thierry Reding
From: Thierry Reding If the DSI output isn't connected, then mdfld_dsi_encoder_get_pipe() will return -1. The mdfld_dsi_dp_mode_set() function doesn't properly check for this condition and causes the following compiler warnings: CC

[PATCH 3/5] drm: omapdrm: tiler: Fix module autoload for OF platform driver

2015-09-24 Thread Luis de Bethencourt
On Thu, Sep 24, 2015 at 06:43:33PM +0300, Tomi Valkeinen wrote: > > On 24/09/15 18:36, Luis de Bethencourt wrote: > > > I am a bit confused. > > Yes, it's an interesting mess due to legacy reasons. Maybe we manage to > fix it some day... > > > So how the OMAP DRM auto loading is supposed to

No more new fbdev drivers, please

2015-09-24 Thread Daniel Vetter
On Thu, Sep 24, 2015 at 11:21:15AM -0400, Austin S Hemmelgarn wrote: > On 2015-09-24 08:46, Thomas Petazzoni wrote: > >Hello, > > > >On Thu, 24 Sep 2015 15:27:01 +0300, Tomi Valkeinen wrote: > > > >>fbdev is (more or less) maintained, but it's a deprecated framework. All > >>new Linux display

[Bug 91921] radeon: command stream overflowed with current git

2015-09-24 Thread bugzilla-dae...@freedesktop.org
ttp://lists.freedesktop.org/archives/dri-devel/attachments/20150924/c2d9393d/attachment-0001.html>

[Bug 90595] [drm:si_dpm_set_power_state [radeon]] *ERROR* si_set_sw_state failed

2015-09-24 Thread bugzilla-dae...@freedesktop.org
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150924/5955cb32/attachment.html>

[PATCH 3/5] drm: omapdrm: tiler: Fix module autoload for OF platform driver

2015-09-24 Thread Luis de Bethencourt
On Thu, Sep 24, 2015 at 01:41:56PM +0300, Tomi Valkeinen wrote: > > On 17/09/15 17:21, Luis de Bethencourt wrote: > > This platform driver has a OF device ID table but the OF module > > alias information is not created so module autoloading won't work. > > > > Signed-off-by: Luis de Bethencourt

[PATCH] drm: omapdrm: tiler: Remove unneded module alias for tiler

2015-09-24 Thread Luis de Bethencourt
omap_dmm_tiler.c can't be compiled as a module and it is built unconditionally as part of omapdrm. Since it can't be used as a module, there is no need for it to have an unused MODULE_ALIAS(). Signed-off-by: Luis de Bethencourt --- Hi, Submitting the removal of MODULE_ALIAS("platform:"

No more new fbdev drivers, please

2015-09-24 Thread Daniel Vetter
On Thu, Sep 24, 2015 at 02:46:21PM +0200, Thomas Petazzoni wrote: > Hello, > > On Thu, 24 Sep 2015 15:27:01 +0300, Tomi Valkeinen wrote: > > > fbdev is (more or less) maintained, but it's a deprecated framework. All > > new Linux display drivers should be done on DRM. > > > > So let's not add

PROBLEM: Intel VGA output busticated on 4.3-rc2 (regression)

2015-09-24 Thread Nick Bowler
-- next part -- A non-text attachment was scrubbed... Name: i915-novga-dmesg.log.gz Type: application/x-gzip Size: 12717 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150924/6e00b20a/attachment-0001.bin>

[PATCH] drm: drm_atomic_crtc_get_property should be static

2015-09-24 Thread Daniel Vetter
On Thu, Sep 24, 2015 at 03:01:03AM -0700, Geliang Tang wrote: > Fixes the following sparse warning: > drivers/gpu/drm/drm_atomic.c:442:5: warning: symbol > 'drm_atomic_crtc_get_property' was not declared. Should it be static? > > Signed-off-by: Geliang Tang Applied to drm-misc, thansk.

[PATCH] drm/gma500: Remove DP_LINK_STATUS_SIZE redefinition

2015-09-24 Thread Daniel Vetter
On Thu, Sep 24, 2015 at 01:44:08PM +0200, Patrik Jakobsson wrote: > On Thu, Sep 24, 2015 at 10:07 AM, Thierry Reding > wrote: > > From: Thierry Reding > > > > The DRM/DP helpers already contain a definition for this macro. Remove > > the duplicate in the GMA500 driver to avoid having to keep

[PATCH v1.1 03/15] vga_switcheroo: Set active attribute to false for audio clients

2015-09-24 Thread Daniel Vetter
On Thu, Sep 24, 2015 at 11:40:52AM +0200, Takashi Iwai wrote: > On Wed, 23 Sep 2015 09:13:28 +0200, > Daniel Vetter wrote: > > > > On Thu, Aug 27, 2015 at 04:43:43PM +0200, Lukas Wunner wrote: > > > The active attribute in struct vga_switcheroo_client denotes whether > > > the outputs are

[PATCH 1/5] ASoC : dwc : support dw i2s in slave mode

2015-09-24 Thread Alex Deucher
On Wed, Sep 23, 2015 at 1:03 PM, Alex Deucher wrote: > From: Maruthi Srinivas Bayyavarapu > > dw i2s controller can work in slave mode, codec being master. > Added a device caps (DW_I2S_SLAVE) to support slave mode operation. > This can be added to platform data to support slave mode. > >

[regression] [git pull] drm for 4.3

2015-09-24 Thread Jani Nikula
On Thu, 24 Sep 2015, "davej at codemonkey.org.uk" wrote: > On Wed, Sep 23, 2015 at 11:07:56AM +, Lankhorst, Maarten wrote: > > Hey, > > > > Dave Jones schreef op di 22-09-2015 om 21:49 [-0400]: > > > On Tue, Sep 22, 2015 at 09:15:58AM -0700, Matt Roper wrote: > > > > On Tue, Sep 22,

[Bug 92087] [Radeon/Tonga] lockup in valley demo

2015-09-24 Thread bugzilla-dae...@freedesktop.org
ttp://lists.freedesktop.org/archives/dri-devel/attachments/20150924/df346fde/attachment.html>

[Bug 91278] Tonga GPU lock/reset fail with Unigine Valley

2015-09-24 Thread bugzilla-dae...@freedesktop.org
: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150924/80f67165/attachment.html>

[Bug 92087] [Radeon/Tonga] lockup in valley demo

2015-09-24 Thread bugzilla-dae...@freedesktop.org
/show_bug.cgi?id=91790 -- 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-devel/attachments/20150924/6bdf986a/attachment-0001.html>

No more new fbdev drivers, please

2015-09-24 Thread Tomi Valkeinen
cation/pgp-signature Size: 819 bytes Desc: OpenPGP digital signature URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150924/688b4354/attachment.sig>

[Bug 92101] face culling and alpha composition broken with Radeon driver in Kerbal Space Program 1.0.4

2015-09-24 Thread bugzilla-dae...@freedesktop.org
.ca/3165739 http://pastebin.ca/3165740 -- 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-devel/attachments/20150924/7150d6ec/attachment.html>

[Bug 92101] face culling and alpha composition broken with Radeon driver in Kerbal Space Program 1.0.4

2015-09-24 Thread bugzilla-dae...@freedesktop.org
at lists.freedesktop ||.org -- 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-devel/attachments/20150

No more new fbdev drivers, please

2015-09-24 Thread Thomas Petazzoni
Hello, On Thu, 24 Sep 2015 15:27:01 +0300, Tomi Valkeinen wrote: > fbdev is (more or less) maintained, but it's a deprecated framework. All > new Linux display drivers should be done on DRM. > > So let's not add any more new fbdev drivers. > > I will continue to maintain the current fbdev

[PATCH v2 2/2] drm/qxl: avoid dependency lock

2015-09-24 Thread Frediano Ziglio
qxl_bo_unref calls drm_gem_object_unreference_unlocked which locks dev->struct_mutex. However this lock could be already locked if the call came from qxl_gem_object_free. As we don't need to call qxl_bo_ref/qxl_bo_unref cause qxl_release_list_add will hold a reference by itself avoid to call them

[PATCH v2 1/2] drm/qxl: avoid buffer reservation in qxl_crtc_page_flip

2015-09-24 Thread Frediano Ziglio
This avoid a dependency lock error. According to https://lwn.net/Articles/548909/ users of WW mutex API should avoid using different context. When a buffer is reserved with qxl_bo_reserve a ww_mutex_lock without context is used. However during qxl_draw_dirty_fb different locks with specific

[PATCH v2 0/2] Dependency locks fixes for QXL driver

2015-09-24 Thread Frediano Ziglio
These two patches fixes two lock dependencies issues. Changes from v1: Sent a wrong patch. Frediano Ziglio (2): drm/qxl: avoid buffer reservation in qxl_crtc_page_flip drm/qxl: avoid dependency lock drivers/gpu/drm/qxl/qxl_display.c | 10 +- drivers/gpu/drm/qxl/qxl_release.c | 4

[PATCH] fbdev: put module after running driver callback

2015-09-24 Thread Tomi Valkeinen
me to trigger this bug. Tomi -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: OpenPGP digital signature URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150924/8a12adba/attachment-0001.sig>

[PATCH 2/2] drm/qxl: avoid dependency lock

2015-09-24 Thread Frediano Ziglio
qxl_bo_unref calls drm_gem_object_unreference_unlocked which locks dev->struct_mutex. However this lock could be already locked if the call came from qxl_gem_object_free. As we don't need to call qxl_bo_ref/qxl_bo_unref cause qxl_release_list_add will hold a reference by itself avoid to call them

[PATCH 1/2] drm/qxl: use to_qxl_bo macro

2015-09-24 Thread Frediano Ziglio
Instead of using container_of directly use to_qxl_bo macro. Signed-off-by: Frediano Ziglio --- drivers/gpu/drm/qxl/qxl_object.c | 2 +- drivers/gpu/drm/qxl/qxl_ttm.c| 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/qxl/qxl_object.c

[PATCH 0/2] Dependency locks fixes for QXL driver

2015-09-24 Thread Frediano Ziglio
These two patches fixes two lock dependencies issues. Frediano Ziglio (2): drm/qxl: use to_qxl_bo macro drm/qxl: avoid dependency lock drivers/gpu/drm/qxl/qxl_object.c | 2 +- drivers/gpu/drm/qxl/qxl_release.c | 4 +--- drivers/gpu/drm/qxl/qxl_ttm.c | 4 ++-- 3 files changed, 4

[PATCH] drm/qxl: use to_qxl_bo macro

2015-09-24 Thread Frediano Ziglio
Instead of using container_of directly use to_qxl_bo macro. Signed-off-by: Frediano Ziglio --- drivers/gpu/drm/qxl/qxl_object.c | 2 +- drivers/gpu/drm/qxl/qxl_ttm.c| 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/qxl/qxl_object.c

4.3-rc2 on radeon: new backtraces during resume

2015-09-24 Thread Pavel Machek
On Wed 2015-09-23 11:26:18, Alex Deucher wrote: > On Wed, Sep 23, 2015 at 7:51 AM, Pavel Machek wrote: > > HI! > > > > > > I suspended T40p by mistake, and I got some lovely backtraces as a > > result: > > > > Any ideas? > > Please see this thread: > https://lkml.org/lkml/2015/9/23/361 Yes,

[PATCH 4/4] fbdev: Debug knob to register without holding console_lock

2015-09-24 Thread Tomi Valkeinen
-- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: OpenPGP digital signature URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150924/2bc37f70/attachment.sig>

[PATCH] drm/gma500: Remove DP_LINK_STATUS_SIZE redefinition

2015-09-24 Thread Patrik Jakobsson
On Thu, Sep 24, 2015 at 10:07 AM, Thierry Reding wrote: > From: Thierry Reding > > The DRM/DP helpers already contain a definition for this macro. Remove > the duplicate in the GMA500 driver to avoid having to keep both updated > synchronously. > > Cc: Patrik Jakobsson > Signed-off-by: Thierry

[PATCH 3/5] drm: omapdrm: tiler: Fix module autoload for OF platform driver

2015-09-24 Thread Tomi Valkeinen
bed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: OpenPGP digital signature URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150924/aa07aa8d/attachment.sig>

[patch 1/4] drm/amdgpu: unwind properly in amdgpu_cs_parser_init()

2015-09-24 Thread Deucher, Alexander
> -Original Message- > From: Dan Carpenter [mailto:dan.carpenter at oracle.com] > Sent: Thursday, September 24, 2015 3:57 AM > To: Alex Deucher > Cc: Koenig, Christian; David Airlie; Ilja Van Sprundel; security at > kernel.org; > Olsak, Marek; Maling list - DRI developers; Deucher,

[PATCH RFC v4 4/8] drm/i2c: tda998x: Add support of a DT graph of ports

2015-09-24 Thread Jean-Francois Moine
On Mon, 21 Sep 2015 10:19:18 -0500 Rob Herring wrote: > On 09/18/2015 06:06 AM, Jyri Sarha wrote: > > From: Jean-Francois Moine > > > > Two kinds of ports may be declared in a DT graph of ports: video and audio. > > This patch accepts the port value from a video port as an alternative > > to

[PATCH 2/2] fixup! drm/atomic: Make prepare_fb/cleanup_fb only take state, v3.

2015-09-24 Thread Thierry Reding
From: Thierry Reding --- drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_plane.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_plane.c b/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_plane.c index 82be6b86a168..d78f8df1fa75 100644 ---

[PATCH 1/2] fixup! drm: Stop using linedur_ns and pixeldur_ns for vblank timestamps

2015-09-24 Thread Thierry Reding
From: Thierry Reding --- drivers/gpu/drm/amd/amdgpu/amdgpu_display.c | 11 ++- drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h| 3 ++- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c

[PATCH 0/2] Fixup patches to fix warnings in recent linux-next

2015-09-24 Thread Thierry Reding
From: Thierry Reding Hi Daniel, here are a couple of patches that fixup some warnings that have started to show up in linux-next recently. They are somewhat unconventionally named, but since these are fixups to patches in your topic/drm-misc branch I thought you might find

[regression] Re: [Linux-v4.2-10463-g9a9952bbd76a] i915: WARNING: intel_display.c:1377 assert_planes_disabled

2015-09-24 Thread Jani Nikula
On Thu, 24 Sep 2015, Sedat Dilek wrote: > On Wed, Sep 23, 2015 at 9:18 AM, Daniel Vetter wrote: >> Adding Jairo to track this regression. > > Hi, > > commit 721a09f7393de6c28a07516dccd654c6e995944a > "drm/i915: Add primary plane to mask if it's visible" > > ...pending in

No more new fbdev drivers, please

2015-09-24 Thread Austin S Hemmelgarn
ome good helper functions to simplify writing basic drivers would be absolutely wonderful. As far as not considering it 'because I don't have a desktop GPU' goes, I agree, that is silly, although for some people it may be 'because my chip doesn't do any "rendering"', which brings up the rather complicated discussion of what constitutes a GPU and what 'rendering' means. -- next part -- A non-text attachment was scrubbed... Name: smime.p7s Type: application/pkcs7-signature Size: 3019 bytes Desc: S/MIME Cryptographic Signature URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150924/bb15aa49/attachment-0001.bin>

[PATCH] drm/sti: Select FW_LOADER

2015-09-24 Thread Thierry Reding
From: Thierry Reding Select FW_LOADER explicitly to satify the direct dependency of FW_LOADER_USER_HELPER_FALLBACK. Acked-by: Benjamin Gaignard Signed-off-by: Thierry Reding --- drivers/gpu/drm/sti/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 3/3] drm/exynos: rotator: Clock control is unused if !PM

2015-09-24 Thread Thierry Reding
From: Thierry Reding Protect the rotator_clk_crtl() function with an #ifdef CONFIG_PM guard to avoid "defined but not used" warnings. Signed-off-by: Thierry Reding --- drivers/gpu/drm/exynos/exynos_drm_rotator.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH 2/3] drm/exynos: fimc: Clock control is unused if !PM

2015-09-24 Thread Thierry Reding
From: Thierry Reding Protect the fimc_clk_ctrl() function with an #ifdef CONFIG_PM guard to avoid "defined but not used" warnings. Signed-off-by: Thierry Reding --- drivers/gpu/drm/exynos/exynos_drm_fimc.c | 36 1 file changed, 18

[PATCH 1/3] drm/exynos: Suspend/resume is unused if !PM

2015-09-24 Thread Thierry Reding
From: Thierry Reding Protect the suspend and resume callbacks with an #ifdef CONFIG_PM_SLEEP guard to avoid "defined but not used" warnings. Signed-off-by: Thierry Reding --- drivers/gpu/drm/exynos/exynos_drm_drv.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

[PULL] drm-intel-fixes

2015-09-24 Thread Jani Nikula
Hi Dave, a few drm/i915 fixes, including a fix to the recent regression reported by Sedat Dilek [1]. BR, Jani. [1] http://mid.gmane.org/CA+icZUUwi96RV5pfdG3FEmG95VFPfjBmUBiwrDzgduyJvt1e3g at mail.gmail.com The following changes since commit 1f93e4a96c9109378204c147b3eec0d0e8100fde: Linux

[Bug 92082] Build on Solaris: xf86drm.c:3021: error: 'PATH_MAX' undeclared (first use in this function)

2015-09-24 Thread bugzilla-dae...@freedesktop.org
-- 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-devel/attachments/20150924/b467fc00/attachment.html>

linux-next: manual merge of the drm-intel tree with the drm-intel-fixes tree

2015-09-24 Thread Jani Nikula
On Thu, 24 Sep 2015, Stephen Rothwell wrote: > Hi all, > > Today's linux-next merge of the drm-intel tree got a conflict in: > > drivers/gpu/drm/i915/intel_display.c > > between commit: > > 721a09f7393d ("drm/i915: Add primary plane to mask if it's visible") > > from the drm-intel-fixes tree

drm_device from another device driver?

2015-09-24 Thread Christian König
fs you should rather go with something like vram_volume_size. E.g. setting the size makes the volume available, setting it to zero tries to free it again. If anybody is using it while you try to change the size return -EBUSY. Regards, Christian. > > > ___ > dri-devel mailing list > dri-devel at lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150924/9a2aca13/attachment.html>

[PATCH v1.1 03/15] vga_switcheroo: Set active attribute to false for audio clients

2015-09-24 Thread Takashi Iwai
On Wed, 23 Sep 2015 09:13:28 +0200, Daniel Vetter wrote: > > On Thu, Aug 27, 2015 at 04:43:43PM +0200, Lukas Wunner wrote: > > The active attribute in struct vga_switcheroo_client denotes whether > > the outputs are currently switched to this client. The attribute is > > only meaningful for vga

[PATCH 03/15] vga_switcheroo: Set active attribute to false for audio clients

2015-09-24 Thread Takashi Iwai
On Sat, 19 Sep 2015 19:50:12 +0200, Lukas Wunner wrote: > > Hi Takashi, > > On Sat, Sep 19, 2015 at 06:44:58PM +0200, Takashi Iwai wrote: > > Without the cover letter I can't judge why this change is needed, so > > at the next time, please put relevant people to Cc of the cover letter > > (or

No more new fbdev drivers, please

2015-09-24 Thread Alex Deucher
On Thu, Sep 24, 2015 at 11:21 AM, Austin S Hemmelgarn wrote: > On 2015-09-24 08:46, Thomas Petazzoni wrote: >> >> Hello, >> >> On Thu, 24 Sep 2015 15:27:01 +0300, Tomi Valkeinen wrote: >> >>> fbdev is (more or less) maintained, but it's a deprecated framework. All >>> new Linux display drivers

[PATCH v1.1 03/15] vga_switcheroo: Set active attribute to false for audio clients

2015-09-24 Thread Takashi Iwai
On Thu, 27 Aug 2015 16:43:43 +0200, Lukas Wunner wrote: > > The active attribute in struct vga_switcheroo_client denotes whether > the outputs are currently switched to this client. The attribute is > only meaningful for vga clients. It is never used for audio clients. > > The function

[PATCH RFC 0/8] Add New DRM Driver for Hisilicon's Hi6220 SoC

2015-09-24 Thread Xinliang Liu
n, but detailed > constraint checking would still be done by the kernel. > > The weston patches from Daniel Stone have a similar design, would be worth > checking those out. > ​Oh, that's great!​ This sounds a great way. That might solve my problem. I'll try. Thanks. ​Best, -Xinliang ​ > > Cheers, Daniel > -- > Daniel Vetter > Software Engineer, Intel Corporation > http://blog.ffwll.ch > -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150924/ffb7c090/attachment.html>

[PATCH RFC v4 4/8] drm/i2c: tda998x: Add support of a DT graph of ports

2015-09-24 Thread Rob Herring
On Thu, Sep 24, 2015 at 5:36 AM, Jean-Francois Moine wrote: > On Mon, 21 Sep 2015 10:19:18 -0500 > Rob Herring wrote: > >> On 09/18/2015 06:06 AM, Jyri Sarha wrote: >> > From: Jean-Francois Moine >> > >> > Two kinds of ports may be declared in a DT graph of ports: video and audio. >> > This

linux-next: manual merge of the drm-intel tree with the drm-intel-fixes tree

2015-09-24 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the drm-intel tree got a conflict in: drivers/gpu/drm/i915/intel_display.c between commit: 721a09f7393d ("drm/i915: Add primary plane to mask if it's visible") from the drm-intel-fixes tree and commit: f9cd7b881a04 ("drm/i915: Move sprite/cursor

No more new fbdev drivers, please

2015-09-24 Thread Austin S Hemmelgarn
ramework is by no-means lightweight (TBH, fbdev isn't really either, but it's much more light weight than DRM). -- next part -- A non-text attachment was scrubbed... Name: smime.p7s Type: application/pkcs7-signature Size: 3019 bytes Desc: S/MIME Cryptographic Signature URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150924/faacc56a/attachment.bin>

[patch 1/4] drm/amdgpu: unwind properly in amdgpu_cs_parser_init()

2015-09-24 Thread Dan Carpenter
On Wed, Sep 23, 2015 at 01:13:21PM -0400, Alex Deucher wrote: > Applied. thanks! Ugh. I'm sorry, I have a signedness bug in this patch. That was really sloppy of me. Should a send a v2 or a fixup? regards, dan carpenter

[PATCH] drm/gma500: Remove DP_LINK_STATUS_SIZE redefinition

2015-09-24 Thread Thierry Reding
From: Thierry Reding The DRM/DP helpers already contain a definition for this macro. Remove the duplicate in the GMA500 driver to avoid having to keep both updated synchronously. Cc: Patrik Jakobsson Signed-off-by: Thierry Reding --- drivers/gpu/drm/gma500/cdv_intel_dp.c

[PATCH 09/14] drm/exynos: remove call to drm_gem_free_mmap_offset()

2015-09-24 Thread Joonyoung Shim
Hi Inki, On 08/17/2015 06:03 PM, Inki Dae wrote: > On 2015년 08월 17일 17:17, Joonyoung Shim wrote: >> On 08/17/2015 04:52 PM, Inki Dae wrote: >>> On 2015년 08월 17일 14:29, Joonyoung Shim wrote: On 08/16/2015 02:07 PM, Inki Dae wrote: > On 2015년 07월 28일 17:53, Joonyoung Shim

[Bug 90595] [drm:si_dpm_set_power_state [radeon]] *ERROR* si_set_sw_state failed

2015-09-24 Thread bugzilla-dae...@freedesktop.org
kernel. -- 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-devel/attachments/20150924/002d1dc6/attachment-0001.html>

[PATCH] drm: drm_atomic_crtc_get_property should be static

2015-09-24 Thread Geliang Tang
Fixes the following sparse warning: drivers/gpu/drm/drm_atomic.c:442:5: warning: symbol 'drm_atomic_crtc_get_property' was not declared. Should it be static? Signed-off-by: Geliang Tang --- drivers/gpu/drm/drm_atomic.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[git pull] vmwgfx-fixes-4.3-150924

2015-09-24 Thread Thomas Hellstrom
Dave, Vmwgfx fixes for 4.3: - A couple of uninitialized variable fixes by Christian Engelmayer - A TTM fix for a bug that causes problems with the new vmwgfx device init - A vmwgfx refcounting fix - A vmwgfx iomem caching fix - A DRM change to allow also control clients to read the drm

  1   2   >