[PATCH] drm/crtc-helper: Fixup error handling in drm_helper_crtc_mode_set

2015-07-02 Thread John Hunter
ailing list > dri-devel at lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel > -- Best regards Junwang Zhao Microprocessor Research and Develop Center Department of Computer Science Peking University Beijing, 100871, PRC -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/b45330b4/attachment.html>

[PATCH] drm: reset empty state in transitional helpers

2015-07-02 Thread John Hunter
op.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel > -- Best regards Junwang Zhao Microprocessor Research and Develop Center Department of Computer Science Peking University Beijing, 100871, PRC -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/a78dc094/attachment-0001.html>

[PATCH 3/3] drm/exynos: remove drm_iommu_attach_device_if_possible

2015-07-02 Thread Joonyoung Shim
Already drm_iommu_attach_device checks whether support iommu internally. It should clear channels always regardless iommu support. We didn't know because we can detect the problem when iommu is enabled, so we don't have to use drm_iommu_attach_device_if_possible and then we can remove

[PATCH 2/3] drm/exynos: remove unnecessary checking to support iommu

2015-07-02 Thread Joonyoung Shim
Already drm_iommu_attach_device and drm_iommu_detach_device check whether support iommu internally, so we don't have to call is_drm_iommu_supported before call them. Signed-off-by: Joonyoung Shim --- drivers/gpu/drm/exynos/exynos5433_drm_decon.c | 3 +--

[PATCH 1/3] drm/exynos: remove to use ifdef CONFIG_ARM_DMA_USE_IOMMU

2015-07-02 Thread Joonyoung Shim
If CONFIG_ARM_DMA_USE_IOMMU is disable, CONFIG_DRM_EXYNOS_IOMMU also is disable. When CONFIG_DRM_EXYNOS_IOMMU is disable, is_drm_iommu_supported() returns always false, so we can remove to use ifdef CONFIG_ARM_DMA_USE_IOMMU in is_drm_iommu_supported(). Signed-off-by: Joonyoung Shim ---

[PATCH] drm/exynos: fix vsync interrupt clear routine of mixer

2015-07-02 Thread Joonyoung Shim
INT_EN_VSYNC bit is not used when we clear vsync interrupt but INT_STATUS_VSYNC bit should be related. Also, if we want to enable vsync interrupt, we should write 1 in INT_CLEAR_VSYNC bit before we set INT_EN_VSYNC bit. It will clear prior vsync interrupt. You can check it from exynos mixer user

[PATCH] drm/exynos: fix compile error and warning of g2d

2015-07-02 Thread Joonyoung Shim
Should be !g2d_userptr->vec, not !vec. This will fix below compile error. drivers/gpu/drm/exynos/exynos_drm_g2d.c: In function ‘g2d_userptr_get_dma_addr’: drivers/gpu/drm/exynos/exynos_drm_g2d.c:465:7: error: ‘vec’ undeclared (first use in this function) if (!vec) Also, if

[PATCH] drm/exynos: iommu: fix potential NULL pointer dereference

2015-07-02 Thread Joonyoung Shim
On 06/29/2015 03:35 PM, Hyungwon Hwang wrote: > Dear Marek, > > On Thu, 25 Jun 2015 15:10:12 +0200 > Marek Szyprowski wrote: > >> Some drivers (like Exynos mixer) calls >> drm_iommu_attach_device_if_possible before registering crtc, so >> additional check for NULL crtc pointer is required. > >

[PATCH 00/23] drm/exynos: atomic improvements + exynos_encoder removal

2015-07-02 Thread Joonyoung Shim
Hi Gustavo, On 06/24/2015 06:35 AM, Gustavo Padovan wrote: > From: Gustavo Padovan > > Hi, > > This set improves exynos in a number of ways. The first five patches are > general clean up/fixes. > > Patches 06 to 12 are improvements on top of the newly added atomic modesetting > support. I'm

[PATCH 19/23] drm/exynos: remove extra call to exynos_dp_commit()

2015-07-02 Thread Joonyoung Shim
+Cc Hyungwon, On 06/24/2015 06:35 AM, Gustavo Padovan wrote: > From: Gustavo Padovan > > exynos_dp_commit() was getting called twice by exynos encoder core, once > inside the .enable() call and another time by .commit() itself. > > The remove of the second call caused the wake of a bug, the

[PATCH 12/23] drm/exynos: don't track enabled state at exynos_crtc

2015-07-02 Thread Joonyoung Shim
On 06/24/2015 06:35 AM, Gustavo Padovan wrote: > 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

[PATCH 08/23] drm/exynos: pass struct exynos_drm_plane in update/enable

2015-07-02 Thread Joonyoung Shim
On 06/24/2015 06:35 AM, Gustavo Padovan wrote: > From: Gustavo Padovan > > We already have the plane pointer in before calling .update_plane() or > disable_plane() so pass it directly to those calls avoiding a new > conversion from zpos to struct exynos_drm_plane. > > Signed-off-by: Gustavo

[PATCH 02/23] drm/exynos: calculate vrefresh instead of use a fixed value

2015-07-02 Thread Joonyoung Shim
On 06/24/2015 06:35 AM, Gustavo Padovan wrote: > From: Gustavo Padovan > > When mode's vrefresh is zero we should ask DRM core to calculate vrefresh > for us so we can get the correct value instead of relying on fixed value > defined in a macro. But if vrefresh is still zero we should fail the >

[Bug 66963] Rv6xx dpm problems

2015-07-02 Thread bugzilla-dae...@freedesktop.org
it now. I'll do it right this time and it won't be long before I finish it. -- 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

[Bug 66963] Rv6xx dpm problems

2015-07-02 Thread bugzilla-dae...@freedesktop.org
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/5388e197/attachment.html>

[Bug 66963] Rv6xx dpm problems

2015-07-02 Thread bugzilla-dae...@freedesktop.org
-- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/409222c3/attachment-0001.html>

[Bug 66963] Rv6xx dpm problems

2015-07-02 Thread bugzilla-dae...@freedesktop.org
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/20150702/518788b5/attachment.html>

[Bug 61529] [r600g][kms][ATI RV710] r600_pcie_gart_tlb_flush+0xf5/0x110

2015-07-02 Thread bugzilla-dae...@freedesktop.org
4a8e0699. I didn't realize it had already been applied. -- 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/20150702/237c530d/attachment.html>

[Intel-gfx] [PATCH 04/12] drm: Add structures for querying color capabilities

2015-07-02 Thread Ville Syrjälä
On Thu, Jul 02, 2015 at 05:45:32PM +0100, Damien Lespiau wrote: > On Thu, Jul 02, 2015 at 05:20:45PM +0100, Damien Lespiau wrote: > > On Wed, Jul 01, 2015 at 09:18:14PM +0530, Kausal Malladi wrote: > > > From: Kausal Malladi > > > > > > The DRM color management framework is targeting various

[Bug 61529] [r600g][kms][ATI RV710] r600_pcie_gart_tlb_flush+0xf5/0x110

2015-07-02 Thread bugzilla-dae...@freedesktop.org
You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/3135898c/attachment.html>

[Bug 91172] [4.1.0][radeonsi][BONAIRE] Playing Ark Survival Evolved (new game) - IH ring buffer overflow

2015-07-02 Thread bugzilla-dae...@freedesktop.org
: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/38214ba3/attachment.html>

[Bug 66963] Rv6xx dpm problems

2015-07-02 Thread bugzilla-dae...@freedesktop.org
e assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/b8df9410/attachment.html>

[Bug 66963] Rv6xx dpm problems

2015-07-02 Thread bugzilla-dae...@freedesktop.org
are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/f341e46e/attachment-0001.html>

[Bug 91197] Radeon Trinity 7400G fails with 4K UHD screen - atombios stuck executing BF64

2015-07-02 Thread bugzilla-dae...@freedesktop.org
You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/f7da85bb/attachment.html>

[Bug 91197] Radeon Trinity 7400G fails with 4K UHD screen - atombios stuck executing BF64

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

[Bug 91197] Radeon Trinity 7400G fails with 4K UHD screen - atombios stuck executing BF64

2015-07-02 Thread bugzilla-dae...@freedesktop.org
attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/fb9627d1/attachment.html>

[PATCH 02/23] drm/exynos: calculate vrefresh instead of use a fixed value

2015-07-02 Thread Gustavo Padovan
Hi Joonyoung, 2015-07-02 Joonyoung Shim : > On 06/24/2015 06:35 AM, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > When mode's vrefresh is zero we should ask DRM core to calculate vrefresh > > for us so we can get the correct value instead of relying on fixed value > > defined in a

[Bug 91197] Radeon Trinity 7400G fails with 4K UHD screen - atombios stuck executing BF64

2015-07-02 Thread bugzilla-dae...@freedesktop.org
: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/0943e936/attachment.html>

[Bug 91197] Radeon Trinity 7400G fails with 4K UHD screen - atombios stuck executing BF64

2015-07-02 Thread bugzilla-dae...@freedesktop.org
ext part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/4b755d05/attachment.html>

[Bug 91197] Radeon Trinity 7400G fails with 4K UHD screen - atombios stuck executing BF64

2015-07-02 Thread bugzilla-dae...@freedesktop.org
-- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/f6512a87/attachment.html>

[Bug 91197] Radeon Trinity 7400G fails with 4K UHD screen - atombios stuck exeuting BF64

2015-07-02 Thread bugzilla-dae...@freedesktop.org
ext part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/0cb71c5b/attachment-0001.html>

[Bug 91197] Radeon Trinity 7400G fails with 4K UHD screen - atombios stuck exeuting BF64

2015-07-02 Thread bugzilla-dae...@freedesktop.org
ause: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/4ea69b2c/attachment.html>

[Bug 91197] Radeon Trinity 7400G fails with 4K UHD screen - atombios stuck exeuting BF64

2015-07-02 Thread bugzilla-dae...@freedesktop.org
ttp://lists.freedesktop.org/archives/dri-devel/attachments/20150702/58d9ec24/attachment.html>

[Bug 91197] Radeon Trinity 7400G fails with 4K UHD screen - atombios stuck exeuting BF64

2015-07-02 Thread bugzilla-dae...@freedesktop.org
ext part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/9989a8da/attachment.html>

[Bug 91197] Radeon Trinity 7400G fails with 4K UHD screen - atombios stuck exeuting BF64

2015-07-02 Thread bugzilla-dae...@freedesktop.org
art -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/544a8afd/attachment.html>

[Bug 91197] Radeon Trinity 7400G fails with 4K UHD screen - atombios stuck exeuting BF64

2015-07-02 Thread bugzilla-dae...@freedesktop.org
attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/70cb7bff/attachment.html>

[Bug 61529] [r600g][kms][ATI RV710] r600_pcie_gart_tlb_flush+0xf5/0x110

2015-07-02 Thread bugzilla-dae...@freedesktop.org
are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/33bc3baa/attachment.html>

[Intel-gfx] [PATCH 04/12] drm: Add structures for querying color capabilities

2015-07-02 Thread Damien Lespiau
On Thu, Jul 02, 2015 at 05:20:45PM +0100, Damien Lespiau wrote: > On Wed, Jul 01, 2015 at 09:18:14PM +0530, Kausal Malladi wrote: > > From: Kausal Malladi > > > > The DRM color management framework is targeting various hardware > > platforms and drivers. Different platforms can have different

[PATCH 03/12] drm/i915: Attach color properties to CRTC

2015-07-02 Thread Damien Lespiau
On Wed, Jul 01, 2015 at 09:18:13PM +0530, Kausal Malladi wrote: > From: Kausal Malladi > > This patch does the following: > 1. Adds new files intel_color_manager(.c/.h) > 2. Attaches color properties to CRTC while initialization We usually order the series so that "it always works". In this

[PATCH 08/12] drm: Export drm_property_replace_global_blob function

2015-07-02 Thread Damien Lespiau
On Wed, Jul 01, 2015 at 09:18:18PM +0530, Kausal Malladi wrote: > From: Kausal Malladi > > drm_property_replace_global_blob() is getting used by many wrapper > functions to replace an existing blob with new values. Because this > function was static, modules are forced to create wrapper

[PATCH 07/12] drm: Add structures to set/get a palette color property

2015-07-02 Thread Damien Lespiau
On Wed, Jul 01, 2015 at 09:18:17PM +0530, Kausal Malladi wrote: > From: Kausal Malladi > > This patch adds new structures in DRM layer for Palette color correction. > These structures will be used by user space agents to configure > appropriate number of samples and Palette LUT for a platform. >

[PATCH 03/12] drm/i915: Attach color properties to CRTC

2015-07-02 Thread Damien Lespiau
On Wed, Jul 01, 2015 at 09:18:13PM +0530, Kausal Malladi wrote: > From: Kausal Malladi > > This patch does the following: > 1. Adds new files intel_color_manager(.c/.h) > 2. Attaches color properties to CRTC while initialization A small note that we could remove manager from the whole API name,

[PATCH 04/12] drm: Add structures for querying color capabilities

2015-07-02 Thread Damien Lespiau
On Wed, Jul 01, 2015 at 09:18:14PM +0530, Kausal Malladi wrote: > From: Kausal Malladi > > The DRM color management framework is targeting various hardware > platforms and drivers. Different platforms can have different color > correction and enhancement capabilities. > > A commom user space

[PATCH] drm/tegra: dc: Reset VBLANK to off

2015-07-02 Thread Thierry Reding
From: Thierry Reding Upon driver load, reset the VBLANK machinery to off to reflect the hardware state. Since the ->reset() callback is called from the initial drm_mode_config_reset() call, move the latter after the VBLANK machinery initialization by drm_vblank_init().

[Bug 61529] [r600g][kms][ATI RV710] r600_pcie_gart_tlb_flush+0xf5/0x110

2015-07-02 Thread bugzilla-dae...@freedesktop.org
es/dri-devel/attachments/20150702/3245f643/attachment.html>

[PATCH 09/12] drm/i915: Add pipe level Gamma correction for CHV/BSW

2015-07-02 Thread Damien Lespiau
On Wed, Jul 01, 2015 at 09:18:19PM +0530, Kausal Malladi wrote: > From: Kausal Malladi > > CHV/BSW platform supports various Gamma correction modes, which are: > 1. Legacy 8-bit mode > 2. 10-bit CGM (Color Gamut Mapping) mode > > This patch does the following: > 1. Adds the core function to

[Bug 91192] ring test failed/si startup failed on resume

2015-07-02 Thread bugzilla-dae...@freedesktop.org
-- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/e89fcd2f/attachment.html>

[Bug 91141] Lots of *ERROR* Couldn't update BO_VA (-22) since drm/radeon: stop using addr to check for BO move

2015-07-02 Thread bugzilla-dae...@freedesktop.org
eedesktop.org/archives/dri-devel/attachments/20150702/432de19a/attachment.html>

[Bug 91193] [290x] Dota2 reborn ingame rendering breaks with git-af4b9c7

2015-07-02 Thread bugzilla-dae...@freedesktop.org
=e4339bc9886a26d75b924ad045c3ddd003f802c3 it seems to render fine. -- 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/20150702/c68e8

[Bug 91192] ring test failed/si startup failed on resume

2015-07-02 Thread bugzilla-dae...@freedesktop.org
nee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/a53bd810/attachment.html>

[Bug 91192] ring test failed/si startup failed on resume

2015-07-02 Thread bugzilla-dae...@freedesktop.org
dri-devel/attachments/20150702/1d9233a2/attachment.html>

[PATCH 01/18] drm/atomic: add connectors_changed to separate it from mode_changed

2015-07-02 Thread Maarten Lankhorst
This can be a separate case from mode_changed, when connectors stay the same but only the mode is different. Drivers may choose to implement specific optimizations to prevent a full modeset for this case. Cc: dri-devel at lists.freedesktop.org Signed-off-by: Maarten Lankhorst ---

[PATCH] drm: reset empty state in transitional helpers

2015-07-02 Thread Daniel Vetter
Transitional drivers might not have all the state frobbing lined up yet. But since the initial code has been merged a lot more state was added, so we really need this. Cc: Daniel Stone Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_crtc_helper.c | 8 +---

[PATCH] drm/crtc-helper: Fixup error handling in drm_helper_crtc_mode_set

2015-07-02 Thread Daniel Vetter
On Thu, Jul 02, 2015 at 02:27:30PM +0100, Daniel Stone wrote: > Hi, > > > On 2 Jul 2015, at 2:16 pm, Daniel Vetter wrote: > > > > In > > > > commit 9f658b7b62e7aefc1ee067136126eca3f58cabfd > > Author: Daniel Stone > > Date: Fri May 22 13:34:45 2015 +0100 > > > >drm/crtc_helper: Replace

[PATCH] drm: remove redundant code form drm_ioc32.c

2015-07-02 Thread Daniel Vetter
On Thu, Jul 02, 2015 at 11:10:06AM +0300, Jani Nikula wrote: > On Wed, 01 Jul 2015, Jarkko Sakkinen > wrote: > > The compat ioctl handler ends up calling access_ok() twice: first > > indirectly inside compat_alloc_user_space() and then after returning > > from that function. This patch fixes

[PATCH v2 2/5] drm/rockchip: vop: fix yuv plane support

2015-07-02 Thread Tomasz Figa
On Thu, Jul 2, 2015 at 3:53 PM, Mark yao wrote: > Hi Tomasz > Thanks for your review, I will fix it soon. > > On 2015年07月02日 14:00, Tomasz Figa wrote: >> >> Hi Mark, >> >> Please see my comments inline. >> >> On Fri, Jun 26, 2015 at 7:07 PM, Mark Yao wrote: >>> >>> vop support yuv with

[PATCH 3/3] drm/exynos: remove drm_iommu_attach_device_if_possible

2015-07-02 Thread Marek Szyprowski
Hello, On 2015-07-02 14:49, Joonyoung Shim wrote: > Already drm_iommu_attach_device checks whether support iommu internally. > It should clear channels always regardless iommu support. We didn't know > because we can detect the problem when iommu is enabled, so we don't > have to use

[PATCH 2/3] drm/exynos: remove unnecessary checking to support iommu

2015-07-02 Thread Marek Szyprowski
Hello, On 2015-07-02 14:49, Joonyoung Shim wrote: > Already drm_iommu_attach_device and drm_iommu_detach_device check > whether support iommu internally, so we don't have to call > is_drm_iommu_supported before call them. > > Signed-off-by: Joonyoung Shim Tested-by: Marek Szyprowski > --- >

[PATCH 2/2] drm/plane-helper: Use init state when freshly allocating them

2015-07-02 Thread Daniel Vetter
Daniel fixed the same issue for crtc states in commit 9f658b7b62e7aefc1ee067136126eca3f58cabfd Author: Daniel Stone Date: Fri May 22 13:34:45 2015 +0100 drm/crtc_helper: Replace open-coded CRTC state helpers Follow suite. Cc: Daniel Stone Signed-off-by: Daniel Vetter ---

[PATCH 1/2] drm/crtc-helper: Fixup error handling in drm_helper_crtc_mode_set

2015-07-02 Thread Daniel Vetter
In commit 9f658b7b62e7aefc1ee067136126eca3f58cabfd Author: Daniel Stone Date: Fri May 22 13:34:45 2015 +0100 drm/crtc_helper: Replace open-coded CRTC state helpers error handling code was broken, resulting in the first path not being checked correctly. Fix this by using the same pattern

[PATCH 1/3] drm/exynos: remove to use ifdef CONFIG_ARM_DMA_USE_IOMMU

2015-07-02 Thread Marek Szyprowski
Hello, On 2015-07-02 14:49, Joonyoung Shim wrote: > If CONFIG_ARM_DMA_USE_IOMMU is disable, CONFIG_DRM_EXYNOS_IOMMU also is > disable. When CONFIG_DRM_EXYNOS_IOMMU is disable, > is_drm_iommu_supported() returns always false, so we can remove to use > ifdef CONFIG_ARM_DMA_USE_IOMMU in

[PATCHv2 2/7] drm/omap: check that plane is inside crtc

2015-07-02 Thread Laurent Pinchart
Hi Tomi, Thank you for the patch. On Thursday 02 July 2015 15:35:31 Tomi Valkeinen wrote: > DRM allows planes to be partially off-screen, but DSS hardware does not. > This patch adds the necessary check to reject plane configs if the plane > is not fully inside the crtc. > > Signed-off-by: Tomi

[PATCHv2 7/7] drm/omap: replace ALIGN(PAGE_SIZE) by PAGE_ALIGN

2015-07-02 Thread Tomi Valkeinen
From: Fabian Frederick use mm.h definition Cc: David Airlie Cc: Tomi Valkeinen Cc: dri-devel at lists.freedesktop.org Signed-off-by: Fabian Frederick Signed-off-by: Tomi Valkeinen --- drivers/gpu/drm/omapdrm/omap_fbdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCHv2 6/7] drm/omap: fix align_pitch() for 24 bits per pixel

2015-07-02 Thread Tomi Valkeinen
align_pitch() uses ALIGN() to ensure the pitch is aligned to SGX's requirement of 8 pixels. However, ALIGN() expects the alignment value to be a power of two, which is not the case for 24 bits per pixels. Use roundup() instead, which works for all alignments. This fixes the error seen with 24

[PATCHv2 5/7] drm/omap: fix omap_gem_put_paddr() error handling

2015-07-02 Thread Tomi Valkeinen
If tiler_unpin() call in omap_gem_put_paddr() fails, omap_gem_put_paddr() will immediately stop processing and return an error. This patch remoes that error checking, and also removes omap_gem_put_paddr()'s return value, because: * The caller of omap_gem_put_paddr() can do nothing if an error

[PATCHv2 4/7] drm/omap: fix omap_framebuffer_unpin() error handling

2015-07-02 Thread Tomi Valkeinen
omap_framebuffer_unpin() check the return value of omap_gem_put_paddr() and return immediately if omap_gem_put_paddr() fails. This patch removes the check for the return value, and also removes the return value of omap_framebuffer_unpin(), because: * Nothing checks the return value of

[PATCHv2 3/7] drm/omap: increase DMM transaction timeout

2015-07-02 Thread Tomi Valkeinen
The DMM driver uses a timeout of 1 ms to wait for DMM transaction to finish. While DMM should always finish the operation within that time, the timeout is rather strict. Small misbehavior of the system (e.g. an irq taking too long) could trigger the timeout. As the DMM is a critical piece of code

[PATCHv2 2/7] drm/omap: check that plane is inside crtc

2015-07-02 Thread Tomi Valkeinen
DRM allows planes to be partially off-screen, but DSS hardware does not. This patch adds the necessary check to reject plane configs if the plane is not fully inside the crtc. Signed-off-by: Tomi Valkeinen --- drivers/gpu/drm/omapdrm/omap_plane.c | 26 ++ 1 file changed,

[PATCHv2 1/7] drm/omap: return error if dma_alloc_writecombine fails

2015-07-02 Thread Tomi Valkeinen
On a platform with no TILER (e.g. omap3, am43xx), when the user wants to allocate buffer with flag OMAP_BO_SCANOUT, the buffer needs to be allocated with dma_alloc_writecombine. For some reason the driver does not return an error if that alloc fails, instead it continues without backing memory.

[PATCHv2 0/7] drm/omap: various small fixes

2015-07-02 Thread Tomi Valkeinen
Hi, This is v2 of this series. Changes to v1: * updated according to the comments * added fix for 24 bpp formats * added PAGE_ALIGN cleanup that I have missed Tomi Fabian Frederick (1): drm/omap: replace ALIGN(PAGE_SIZE) by PAGE_ALIGN Tomi Valkeinen (6): drm/omap: return error if

[PATCH] drm/crtc-helper: Fixup error handling in drm_helper_crtc_mode_set

2015-07-02 Thread Daniel Vetter
In commit 9f658b7b62e7aefc1ee067136126eca3f58cabfd Author: Daniel Stone Date: Fri May 22 13:34:45 2015 +0100 drm/crtc_helper: Replace open-coded CRTC state helpers error handling code was broken, resulting in the first path not being checked correctly. Fix this by using the same pattern

[PATCH v2] drm/rockchip: vop: remove hardware cursor window

2015-07-02 Thread Mark Yao
hardware cursor windows only have some fixed size, and not support width virtual, when move hardware cursor windows outside of left, the display would be wrong, so this window can't for cursor now. And Tag hardware cursor window as a overlay is wrong, will make userspace wrong behaviour. So just

[PATCH] drm/rockchip: vop: remove hardware cursor window

2015-07-02 Thread Mark yao
On 2015年07月01日 19:52, Heiko Stübner wrote: > Hi Mark, > > Am Mittwoch, 1. Juli 2015, 17:49:33 schrieb Mark Yao: >> hardware cursor windows only have some fixed size, and not support >> width virtual, when move hardware cursor windows outside of left, >> the display would be wrong, so this

[PATCH v2 2/5] drm/rockchip: vop: fix yuv plane support

2015-07-02 Thread Tomasz Figa
Hi Mark, Please see my comments inline. On Fri, Jun 26, 2015 at 7:07 PM, Mark Yao wrote: > vop support yuv with NV12, NV16 and NV24, only 2 plane yuv. > > Signed-off-by: Mark Yao > > Changes in v2: > - Uv buffer not support odd offset, align it. > - Fix error display when move yuv image. > >

[DRM] Fix a regression caused by a typo

2015-07-02 Thread Shixin Zeng
From: Shixin Zeng The length of each EDID block is EDID_LENGTH, and number of blocks is (1 + edid->extensions) This causes wrong EDID to be passed on, and is a regression introduced by d2ed34362a52 (drm: Introduce helper for replacing blob properties) Signed-off-by:

[PATCH v2 2/5] drm/rockchip: vop: fix yuv plane support

2015-07-02 Thread Mark yao
Hi Tomasz Thanks for your review, I will fix it soon. On 2015年07月02日 14:00, Tomasz Figa wrote: > Hi Mark, > > Please see my comments inline. > > On Fri, Jun 26, 2015 at 7:07 PM, Mark Yao wrote: >> vop support yuv with NV12, NV16 and NV24, only 2 plane yuv. >> >> Signed-off-by: Mark Yao

[PATCH] drm/crtc-helper: Fixup error handling in drm_helper_crtc_mode_set

2015-07-02 Thread Daniel Stone
Hi, > On 2 Jul 2015, at 2:16 pm, Daniel Vetter wrote: > > In > > commit 9f658b7b62e7aefc1ee067136126eca3f58cabfd > Author: Daniel Stone > Date: Fri May 22 13:34:45 2015 +0100 > >drm/crtc_helper: Replace open-coded CRTC state helpers > > error handling code was broken, resulting in the

[PULL] drm-intel-next-fixes

2015-07-02 Thread Jani Nikula
Hi Linus - I hear Dave is on vacation, so please pull this first batch of Intel fixes for v4.2 directly. (Dave, do correct me if I'm wrong!) Almost all of it is regression fixes all around, with cc: stable, and then there's Ander's fix for one of the warnings you reported. We're still working

[PATCH] gpu: host1x: Fix MLOCK's debug info

2015-07-02 Thread Terje Bergstrom
On 06/28/2015 12:27 PM, Dmitry Osipenko wrote: > MLOCK's debug info, spewed on CDMA timeout, contains meaningless MLOCK > owner channel ID because HOST1X_SYNC_MLOCK_OWNER_CHID_F() returns shifted > value, while unshifted should be used. Fix it by changing '_F' to '_V'. > > Signed-off-by: Dmitry

[PATCH v2 1/5] drm/rockchip: vop: optimize virtual stride calculate

2015-07-02 Thread Mark yao
On 2015年07月02日 12:59, Tomasz Figa wrote: > On Fri, Jun 26, 2015 at 7:07 PM, Mark Yao wrote: >> vir_stride need number words of the virtual width, and fb->pitches >> save bytes_per_pixel, so just div 4 switch to stride. >> >> Signed-off-by: Mark Yao >> --- >> Changes in v2: None >> >>

[PATCH v2 1/5] drm/rockchip: vop: optimize virtual stride calculate

2015-07-02 Thread Tomasz Figa
On Fri, Jun 26, 2015 at 7:07 PM, Mark Yao wrote: > vir_stride need number words of the virtual width, and fb->pitches > save bytes_per_pixel, so just div 4 switch to stride. > > Signed-off-by: Mark Yao > --- > Changes in v2: None > > drivers/gpu/drm/rockchip/rockchip_drm_vop.c |2 +- > 1

[pull] radeon and amdgpu drm-next-4.2

2015-07-02 Thread Michel Dänzer
On 02.07.2015 06:20, Alex Deucher wrote: > > Alex Deucher (4): [...] > Revert "drm/radeon: dont switch vt on suspend" Do we have a plan for re-enabling this? It seems to work fine on my machines, and it's kinda nice not to see any console spam during suspend/resume. -- Earthling Michel

[Bug 61529] [r600g][kms][ATI RV710] r600_pcie_gart_tlb_flush+0xf5/0x110

2015-07-02 Thread bugzilla-dae...@freedesktop.org
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/20150702/6e9aa318/attachment.html>

[PATCH libdrm v2 1/2] intel: Add EXEC_OBJECT_SUPPORTS_48B_ADDRESS flag.

2015-07-02 Thread Michel Thierry
On 7/1/2015 6:06 PM, Emil Velikov wrote: > Hi Michel, > > Although I cannot comment on the exact implementation I can give you > general some tips which you might find useful. > Hi Emil, > On 1 July 2015 at 16:28, Michel Thierry wrote: >> Gen8+ supports 48-bit virtual addresses, but some objects

[PATCH v2] drm/radeon: default to 2048 MB GART size on SI+

2015-07-02 Thread Grigori Goronzy
Newer ASICs have more VRAM on average and allocating more GART as well can have advantages. Also see commit edcd26e8. Ideally, we should scale GART size based on actual VRAM size, but that requires significant restructuring of initialization. v2: extract small helper, apply to error paths ---

[PATCH 3/3] drm/radeon: default to 2048 MB GART size on SI+

2015-07-02 Thread Grigori Goronzy
Newer ASICs have more VRAM on average and allocating more GART as well can have advantages. Also see commit edcd26e8. Ideally, we should scale GART size based on actual VRAM size, but that requires significant restructuring of initialization. --- drivers/gpu/drm/radeon/radeon_device.c | 4 +++-

[PATCH 2/3] drm/radeon: fix HDP flushing

2015-07-02 Thread Grigori Goronzy
This was regressed by commit 39e7f6f8, although I don't know of any actual issues caused by it. The storage domain is read without TTM locking now, but the lock never helped to prevent any actual races. Cc: stable at vger.kernel.org --- drivers/gpu/drm/radeon/radeon_gem.c | 1 + 1 file changed,

[PATCH 1/3] drm/radeon: use RCU query for GEM_BUSY syscall

2015-07-02 Thread Grigori Goronzy
We don't need to call the (expensive) radeon_bo_wait, checking the fences via RCU is much faster. The reservation done by radeon_bo_wait does not save us from any race conditions. --- drivers/gpu/drm/radeon/radeon_gem.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git

[PATCH] drm: remove redundant code form drm_ioc32.c

2015-07-02 Thread Jani Nikula
On Wed, 01 Jul 2015, Jarkko Sakkinen wrote: > The compat ioctl handler ends up calling access_ok() twice: first > indirectly inside compat_alloc_user_space() and then after returning > from that function. This patch fixes issue. > > Signed-off-by: Jarkko Sakkinen > --- >

[pull] radeon and amdgpu drm-next-4.2

2015-07-02 Thread Alex Deucher
Hi Linus, Dave is on vacation, so please pull this directly. First round of fixes for 4.2 for radeon and amdgpu. Stuff all over the place: - hibernation, suspend fixes for radeon and amdgpu - radeon audio fix - amdgpu ioctl optimzations and fixes - amdgpu VCE cs checker improvements - misc bug

[Bug 73528] Deferred lighting in Second Life causes system hiccups and screen flickering

2015-07-02 Thread bugzilla-dae...@freedesktop.org
nts/20150702/08c7a913/attachment.html>

[Bug 82186] [r600g] BARTS GPU lockup with minecraft shaders

2015-07-02 Thread bugzilla-dae...@freedesktop.org
ttp://lists.freedesktop.org/archives/dri-devel/attachments/20150702/1791dbcc/attachment.html>

[Bug 73528] Deferred lighting in Second Life causes system hiccups and screen flickering

2015-07-02 Thread bugzilla-dae...@freedesktop.org
bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150702/579cfb2d/attachment-0001.html>

[PATCH 1/3] drm/radeon: use RCU query for GEM_BUSY syscall

2015-07-02 Thread Alex Deucher
On Thu, Jul 2, 2015 at 5:41 AM, Grigori Goronzy wrote: > We don't need to call the (expensive) radeon_bo_wait, checking the > fences via RCU is much faster. The reservation done by radeon_bo_wait > does not save us from any race conditions. Can you add your signed-off by to these patches? Alex

[PATCH 1/2] drm/atomic: Update old_fb after setting a property.

2015-07-02 Thread Rob Clark
On Wed, Jun 24, 2015 at 2:59 AM, Maarten Lankhorst wrote: > This change updates the old_fb pointer only after acquiring the plane lock, > if there are no properties the fb cannot have been changed either, so > this works out correctly. > > Found in a discussion with Rob Clark. > > Cc: Rob Clark

[pull] radeon and amdgpu drm-next-4.2

2015-07-02 Thread Alex Deucher
On Wed, Jul 1, 2015 at 11:08 PM, Michel Dänzer wrote: > On 02.07.2015 06:20, Alex Deucher wrote: >> >> Alex Deucher (4): > [...] >> Revert "drm/radeon: dont switch vt on suspend" > > Do we have a plan for re-enabling this? It seems to work fine on my > machines, and it's kinda nice not to

[PATCH] drm/crtc-helper: Fixup error handling in drm_helper_crtc_mode_set

2015-07-02 Thread Daniel Stone
Hi, On Thu, 2015-07-02 at 09:23 +0200, Daniel Vetter wrote: > In > > commit 9f658b7b62e7aefc1ee067136126eca3f58cabfd > Author: Daniel Stone > Date: Fri May 22 13:34:45 2015 +0100 > > drm/crtc_helper: Replace open-coded CRTC state helpers > > error handling code was broken, resulting in

[PATCH] drm: remove redundant code form drm_ioc32.c

2015-07-02 Thread Daniel Vetter
On Wed, Jul 01, 2015 at 12:24:37PM +0300, Jarkko Sakkinen wrote: > The compat ioctl handler ends up calling access_ok() twice: first > indirectly inside compat_alloc_user_space() and then after returning > from that function. This patch fixes issue. > > Signed-off-by: Jarkko Sakkinen Applied to

[PATCH] drm/crtc-helper: Fixup error handling in drm_helper_crtc_mode_set

2015-07-02 Thread Daniel Vetter
In commit 9f658b7b62e7aefc1ee067136126eca3f58cabfd Author: Daniel Stone Date: Fri May 22 13:34:45 2015 +0100 drm/crtc_helper: Replace open-coded CRTC state helpers error handling code was broken, resulting in the first path not being checked correctly. Fix this by using the same pattern

[PATCH] drm/crtc-helper: Fixup error handling in drm_helper_crtc_mode_set

2015-07-02 Thread Daniel Vetter
In commit 9f658b7b62e7aefc1ee067136126eca3f58cabfd Author: Daniel Stone Date: Fri May 22 13:34:45 2015 +0100 drm/crtc_helper: Replace open-coded CRTC state helpers error handling code was broken, resulting in the first path not being checked correctly. Fix this by using the same pattern

  1   2   >