Re: [PATCH v7 0/8] of: add display helper

2012-11-09 Thread Steffen Trumtrar
On Thu, Nov 08, 2012 at 03:35:47PM -0600, Rob Herring wrote: > On 10/31/2012 04:28 AM, Steffen Trumtrar wrote: > > Hi! > > > > Finally, v7 of the series. > > > > Changes since v6: > > - get rid of some empty lines etc. > > - move functions to their subsystems > > - split of_ from non-

[PATCH] drm/exynos: add vm_ops to specific gem mmaper

2012-11-09 Thread Inki Dae
This patch makes it takes a reference to gem object when specific gem mmap is requested. For this, it sets dev->driver->gem_vm_ops to vma->vm_ops. And this patch is based on exynos-drm-next-iommu branch of git://git.kernel.org/pub/scm/linux/kernel/git/daeinki/drm-exynos Signed-off-by: Ink

Re: [PATCH] drm: fix drm_framebuffer cleanup.

2012-11-09 Thread Prathyush K
On Fri, Nov 9, 2012 at 1:09 PM, Inki Dae wrote: > This patch fixes access issue to invalid memory region. > > crtc had only one drm_framebuffer object so when framebuffer > cleanup was requested after page flip, it'd try to disable > hardware overlay to current crtc. > But if current crtc points

[PATCH -fixes RESEND] drm/vmwgfx: Fix hibernation device reset

2012-11-09 Thread Thomas Hellstrom
The device would not reset properly when resuming from hibernation. Signed-off-by: Thomas Hellstrom Reviewed-by: Brian Paul Reviewed-by: Dmitry Torokhov Cc: sta...@vger.kernel.org Cc: linux-graphics-maintai...@vmware.com --- drivers/gpu/drm/vmwgfx/vmwgfx_drv.c |5 + 1 files changed, 5

Re: [PATCH] drm: fix drm_framebuffer cleanup.

2012-11-09 Thread Inki Dae
2012/11/9 Prathyush K > > On Fri, Nov 9, 2012 at 1:09 PM, Inki Dae wrote: > >> This patch fixes access issue to invalid memory region. >> >> crtc had only one drm_framebuffer object so when framebuffer >> cleanup was requested after page flip, it'd try to disable >> hardware overlay to current c

[PATCH 0/6] alloc_apertures() handling fixes

2012-11-09 Thread Tommi Rantala
Greetings, The following patches fix a few memory leaks and other problems related to the alloc_apertures() call. Tommi Rantala (6): drm/cirrus: check alloc_apertures() success in cirrus_kick_out_firmware_fb() drm/mgag200: check alloc_apertures() success in mga_vram_init() drm/mgag200:

[PATCH 1/6] drm/cirrus: check alloc_apertures() success in cirrus_kick_out_firmware_fb()

2012-11-09 Thread Tommi Rantala
Check for alloc_apertures() memory allocation failure, and propagate an error code in case the allocation failed. Signed-off-by: Tommi Rantala --- drivers/gpu/drm/cirrus/cirrus_drv.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/cirrus/cirru

[PATCH 2/6] drm/mgag200: check alloc_apertures() success in mga_vram_init()

2012-11-09 Thread Tommi Rantala
Check for alloc_apertures() memory allocation failure, and propagate an error code in case the allocation failed. Signed-off-by: Tommi Rantala --- drivers/gpu/drm/mgag200/mgag200_main.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/mgag200/mgag200_main.c b/drivers/gpu

[PATCH 3/6] drm/mgag200: free memory allocated with alloc_apertures()

2012-11-09 Thread Tommi Rantala
Fix a memory leak by deallocating the memory we got from alloc_apertures(). Signed-off-by: Tommi Rantala --- drivers/gpu/drm/mgag200/mgag200_main.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/mgag200/mgag200_main.c b/drivers/gpu/drm/mgag200/mgag200_main.c index 07d4b2

[PATCH 4/6] drm/mgag200: remove unneeded aper->count assignment after alloc_apertures()

2012-11-09 Thread Tommi Rantala
alloc_apertures() already does the assignment for us, so assigning the count member after the alloc_apertures() call is not needed. Signed-off-by: Tommi Rantala --- drivers/gpu/drm/mgag200/mgag200_main.c |1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/mgag200/mgag200_main.c

[PATCH 5/6] drm/radeon: check alloc_apertures() success in radeon_kick_out_firmware_fb()

2012-11-09 Thread Tommi Rantala
Check for alloc_apertures() memory allocation failure, and propagate an error code in case the allocation failed. Signed-off-by: Tommi Rantala --- drivers/gpu/drm/radeon/radeon_drv.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeo

[PATCH 6/6] drm/nouveau: free memory allocated with alloc_apertures()

2012-11-09 Thread Tommi Rantala
Fix a memory leak by deallocating the memory we got from alloc_apertures(). Signed-off-by: Tommi Rantala --- drivers/gpu/drm/nouveau/nouveau_drm.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/nouveau/nouveau_drm.c b/drivers/gpu/drm/nouveau/nouveau_drm.c index 0910125..

[PATCH -fixes] drm/vmwgfx: Fix a case where the code would BUG when trying to pin GMR memory

2012-11-09 Thread Thomas Hellstrom
Signed-off-by: Thomas Hellstrom Reviewed-by: Brian Paul Reviewed-by: Dmitry Torokhov Cc: sta...@vger.kernel.org Cc: linux-graphics-maintai...@vmware.com --- drivers/gpu/drm/vmwgfx/vmwgfx_dmabuf.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/vmwgfx/vm

[Bug 17902] [830M missing dvo driver] need support for DVO-LVDS via na2501

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=17902 Cool Loki changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|WORKSFORME

[Bug 17902] [830M missing dvo driver] need support for DVO-LVDS via na2501

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=17902 --- Comment #90 from Cool Loki --- Created attachment 69801 --> https://bugs.freedesktop.org/attachment.cgi?id=69801&action=edit rom_loki.bin -- You are receiving this mail because: You are the assignee for the bug. __

[Bug 17902] [830M missing dvo driver] need support for DVO-LVDS via na2501

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=17902 --- Comment #91 from Cool Loki --- Created attachment 69802 --> https://bugs.freedesktop.org/attachment.cgi?id=69802&action=edit intel_drm_dumper -- You are receiving this mail because: You are the assignee for the bug. __

[Bug 17902] [830M missing dvo driver] need support for DVO-LVDS via na2501

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=17902 --- Comment #92 from Cool Loki --- Created attachment 69803 --> https://bugs.freedesktop.org/attachment.cgi?id=69803&action=edit lspci -v My laptop model: Fujitsu Siemens FMV-610NU2 -- You are receiving this mail because: You are the assigne

[git pull] drm vmwgfx

2012-11-09 Thread Dave Airlie
Hi Linus, dropped the ball on a vmware patch, so two more fixes for vmwgfx are here, one for hibernate issue, one for a BUG trigger. Dave. The following changes since commit 4a48ed2334b7ae61dd11bb114fa35bd4ebdc1ca0: Merge branch 'drm-nouveau-fixes' of git://anongit.freedesktop.org/git/nouve

[Bug 17902] [830M missing dvo driver] need support for DVO-LVDS via na2501

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=17902 --- Comment #93 from Cool Loki --- Created attachment 69804 --> https://bugs.freedesktop.org/attachment.cgi?id=69804&action=edit dmesg.txt dmseg.txt kernel 3.7.0-rc4 -- You are receiving this mail because: You are the assignee for the bug. _

Re: [PATCH] drm: fix drm_framebuffer cleanup.

2012-11-09 Thread Ville Syrjälä
On Fri, Nov 09, 2012 at 04:39:30PM +0900, Inki Dae wrote: > This patch fixes access issue to invalid memory region. > > crtc had only one drm_framebuffer object so when framebuffer > cleanup was requested after page flip, it'd try to disable > hardware overlay to current crtc. > But if current crt

Re: [PATCH] drm/exynos: Add exynos drm specific fb_mmap function

2012-11-09 Thread Rahul Sharma
Hi All, Please review the following patch. regards, Rahul Sharma On Wed, Nov 7, 2012 at 4:38 PM, Rahul Sharma wrote: > From: Prathyush K > > This patch adds a exynos drm specific implementation of fb_mmap > which supports mapping a non-contiguous buffer to user space. > > This new function doe

[Bug 54767] [r300g] 298 failures on WebGL Conformance Test

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54767 --- Comment #4 from madbiologist --- (In reply to comment #3) > r300 cannot pass some of the tests, because the hardware is too limited > (some features cannot be implemented on r300), while other features may > produce slightly different results

[PATCH 0/8] vmwgfx stuff for -next

2012-11-09 Thread Thomas Hellstrom
This patch series should be applied after the drm/ttm: Get rid of a number of atomic read-modify-write ops vX patch series. It contains various improvements. Patch 7 is a rewrite of the resource management and is intended to pave the way for an upcoming hardware version change, and to ease impleme

[PATCH 3/8] drm/vmwgfx: Make screen object code not require fifo at init time

2012-11-09 Thread Thomas Hellstrom
Signed-off-by: Thomas Hellstrom Reviewed-by: Brian Paul Reviewed-by: Dmitry Torokhov --- drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c b/drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c index 6deaf2f..60f

[PATCH 4/8] drm/vmwgfx: Refactor module load to not require fifo unless fbdev is loaded

2012-11-09 Thread Thomas Hellstrom
This also fixes a bug where the fence manager was left without irq enabled when waiting for fences, causing various errors at module load time Signed-off-by: Thomas Hellstrom Reviewed-by: Brian Paul Reviewed-by: Dmitry Torokhov --- drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 50 ---

[PATCH 8/8] drm/vmwgfx: Add and make use of a header for surface size calculation.

2012-11-09 Thread Thomas Hellstrom
Signed-off-by: Thomas Hellstrom Reviewed-by: Brian Paul Reviewed-by: Dmitry Torokhov --- drivers/gpu/drm/vmwgfx/svga3d_surfacedefs.h | 909 +++ drivers/gpu/drm/vmwgfx/vmwgfx_surface.c | 113 +--- 2 files changed, 928 insertions(+), 94 deletions(-) create mode 1006

[Bug 54767] [r300g] 298 failures on WebGL Conformance Test

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54767 --- Comment #5 from Marek Olšák --- (In reply to comment #4) > Does that include atan(vec4, vec4) in fragment shader? I have a Radeon 9800 > Pro 128MB (R350). I don't remember if that function works on r300. I guess you have to try and see if t

[PATCH 5/8] drm/vmwgfx: Make vmw_dmabuf_unreference handle NULL objects

2012-11-09 Thread Thomas Hellstrom
Signed-off-by: Thomas Hellstrom Reviewed-by: Brian Paul Reviewed-by: Dmitry Torokhov --- drivers/gpu/drm/vmwgfx/vmwgfx_drv.h |7 +-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_drv.h b/drivers/gpu/drm/vmwgfx/vmwgfx_drv.h index 88a179e..7

[PATCH 1/8] drm/vmwgfx: Enable traces *after* we've hidden SVGA

2012-11-09 Thread Thomas Hellstrom
Hiding SVGA seems to trigger a VGA screen clear, and with no traces dirty it doesn't seem to repaint Signed-off-by: Thomas Hellstrom Reviewed-by: Brian Paul Reviewed-by: Dmitry Torokhov --- drivers/gpu/drm/vmwgfx/vmwgfx_drv.c |8 1 files changed, 4 insertions(+), 4 deletions(-) d

Re: [PATCH] drm: fix drm_framebuffer cleanup.

2012-11-09 Thread Inki Dae
2012/11/9 Ville Syrjälä > On Fri, Nov 09, 2012 at 04:39:30PM +0900, Inki Dae wrote: > > This patch fixes access issue to invalid memory region. > > > > crtc had only one drm_framebuffer object so when framebuffer > > cleanup was requested after page flip, it'd try to disable > > hardware overlay

[PATCH 2/8] drm/vmwgfx: Make overlay code not require fifo at init time

2012-11-09 Thread Thomas Hellstrom
Signed-off-by: Thomas Hellstrom Reviewed-by: Brian Paul Reviewed-by: Dmitry Torokhov --- drivers/gpu/drm/vmwgfx/vmwgfx_overlay.c | 21 - 1 files changed, 12 insertions(+), 9 deletions(-) diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_overlay.c b/drivers/gpu/drm/vmwgfx/vmwgfx

[Bug 56918] New: [R350] Black labels on MapsGL

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=56918 Priority: medium Bug ID: 56918 Assignee: dri-devel@lists.freedesktop.org Summary: [R350] Black labels on MapsGL Severity: normal Classification: Unclassified OS: Linux (All)

[Bug 56918] [R350] Black labels on MapsGL

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=56918 Alex Deucher changed: What|Removed |Added Attachment #69809|text/plain |image/png mime type|

[Bug 56918] [R350] Black labels on MapsGL

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=56918 --- Comment #1 from Tom Stellard --- (In reply to comment #0) > Created attachment 69809 [details] > screenshot from Mesa 9.0 > > Using a Radeon 9800 Pro 128MB (R350) on both Mesa 8.04 and Mesa 9.0. All of > the labels in Google MapsGL are black

Re: [PATCH] drm: fix drm_framebuffer cleanup.

2012-11-09 Thread Ville Syrjälä
On Fri, Nov 09, 2012 at 09:41:19PM +0900, Inki Dae wrote: > 2012/11/9 Ville Syrjälä > > > On Fri, Nov 09, 2012 at 04:39:30PM +0900, Inki Dae wrote: > > > This patch fixes access issue to invalid memory region. > > > > > > crtc had only one drm_framebuffer object so when framebuffer > > > cleanup

Re: [PATCH] drm/exynos: Add exynos drm specific fb_mmap function

2012-11-09 Thread Inki Dae
sorry for being late. 2012/11/7 Rahul Sharma > From: Prathyush K > > This patch adds a exynos drm specific implementation of fb_mmap > which supports mapping a non-contiguous buffer to user space. > > This new function does not assume that the frame buffer is contiguous > and calls dma_mmap_wri

[PATCH 0/2] NVIDIA Tegra DRM driver

2012-11-09 Thread Thierry Reding
Hi, This small set of patches adds support for the host1x and a subset of the display controller hardware available on NVIDIA Tegra SoCs. The first patch makes the RGB output available, which is directly driven by its parent display controller and usually connected to an LVDS bridge in embedded a

Re: [PATCH] drm: fix drm_framebuffer cleanup.

2012-11-09 Thread Inki Dae
2012/11/9 Ville Syrjälä > On Fri, Nov 09, 2012 at 09:41:19PM +0900, Inki Dae wrote: > > 2012/11/9 Ville Syrjälä > > > > > On Fri, Nov 09, 2012 at 04:39:30PM +0900, Inki Dae wrote: > > > > This patch fixes access issue to invalid memory region. > > > > > > > > crtc had only one drm_framebuffer ob

Re: [PATCH] drm: fix drm_framebuffer cleanup.

2012-11-09 Thread Ville Syrjälä
On Fri, Nov 09, 2012 at 11:04:58PM +0900, Inki Dae wrote: > 2012/11/9 Ville Syrjälä > > > On Fri, Nov 09, 2012 at 09:41:19PM +0900, Inki Dae wrote: > > > 2012/11/9 Ville Syrjälä > > > > > > > On Fri, Nov 09, 2012 at 04:39:30PM +0900, Inki Dae wrote: > > > > > This patch fixes access issue to inv

Re: [PATCH] drm: fix drm_framebuffer cleanup.

2012-11-09 Thread Rob Clark
On Fri, Nov 9, 2012 at 1:39 AM, Inki Dae wrote: > This patch fixes access issue to invalid memory region. > > crtc had only one drm_framebuffer object so when framebuffer > cleanup was requested after page flip, it'd try to disable > hardware overlay to current crtc. > But if current crtc points t

[Bug 56918] [R350] Black labels on MapsGL

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=56918 --- Comment #2 from madbiologist --- How do I set that? In the GRUB kernel boot options? Or with "setenv"? And where do I look for the debugging output? dmesg? Xorg.log? Or somewhere else? -- You are receiving this mail because: You are t

[Bug 56918] [R350] Black labels on MapsGL

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=56918 --- Comment #3 from Tom Stellard --- (In reply to comment #2) > How do I set that? In the GRUB kernel boot options? Or with "setenv"? And > where do I look for the debugging output? dmesg? Xorg.log? Or somewhere > else? Open a terminal a s

[Bug 54767] [r300g] 298 failures on WebGL Conformance Test

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54767 --- Comment #6 from madbiologist --- Its totally wrong. See attachments. I just thought perhaps it wasn't implemented yet/correctly in the r300g driver. -- You are receiving this mail because: You are the assignee for the bug. ___

[Bug 54767] [r300g] 298 failures on WebGL Conformance Test

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54767 --- Comment #7 from madbiologist --- Created attachment 69818 --> https://bugs.freedesktop.org/attachment.cgi?id=69818&action=edit error messages from test -- You are receiving this mail because: You are the assignee for the bug.

[Bug 54767] [r300g] 298 failures on WebGL Conformance Test

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54767 --- Comment #8 from madbiologist --- Created attachment 69819 --> https://bugs.freedesktop.org/attachment.cgi?id=69819&action=edit atan(vec4, vec4) image result -- You are receiving this mail because: You are the assignee for the bug. ___

[Bug 56918] [R350] Black labels on MapsGL

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=56918 --- Comment #4 from madbiologist --- Created attachment 69823 --> https://bugs.freedesktop.org/attachment.cgi?id=69823&action=edit radeon debug output Thanks. Attached file is from using Google MapsGL on Firefox 16.0.2 with Mesa 9.0. -- You

Re: [PATCH 2/2] drm: tegra: Add HDMI support

2012-11-09 Thread Rafał Miłecki
2012/11/9 Thierry Reding : > +/* all fields little endian */ > +struct hdmi_audio_infoframe { > + /* PB0 */ > + u8 csum; > + > + /* PB1 */ > + unsigned cc:3; /* channel count */ > + unsigned res1:1; > + unsigned ct:4; /* coding type */ > + > + /* PB2 */ > +

Re: [PATCH 1/2] drm: Add NVIDIA Tegra20 support

2012-11-09 Thread Thierry Reding
On Fri, Nov 09, 2012 at 09:18:58AM -0600, Rob Clark wrote: > On Fri, Nov 9, 2012 at 7:59 AM, Thierry Reding > wrote: [...] > > +static int regs_open(struct inode *inode, struct file *file) > > +{ > > + return single_open(file, regs_show, inode->i_private); > > +} > > + > > +static const stru

Re: [PATCH 2/2] drm: tegra: Add HDMI support

2012-11-09 Thread Christian König
On 09.11.2012 16:45, Rafał Miłecki wrote: 2012/11/9 Thierry Reding : +/* all fields little endian */ +struct hdmi_audio_infoframe { + /* PB0 */ + u8 csum; + + /* PB1 */ + unsigned cc:3; /* channel count */ + unsigned res1:1; + unsigned ct:4; /* coding type */

Re: [PATCH 2/2] drm: tegra: Add HDMI support

2012-11-09 Thread Rafał Miłecki
2012/11/9 Christian König : > On 09.11.2012 16:45, Rafał Miłecki wrote: >> I was told it won't work on different endian devices. See >> [RFC][PATCH] drm/radeon/hdmi: define struct for AVI infoframe >> http://lists.freedesktop.org/archives/dri-devel/2012-May/022544.html > > > Yeah, that's indeed tru

Re: [PATCH 1/2] drm: Add NVIDIA Tegra20 support

2012-11-09 Thread Rob Clark
On Fri, Nov 9, 2012 at 10:00 AM, Thierry Reding wrote: > On Fri, Nov 09, 2012 at 09:18:58AM -0600, Rob Clark wrote: >> On Fri, Nov 9, 2012 at 7:59 AM, Thierry Reding >> wrote: > [...] >> > +static int regs_open(struct inode *inode, struct file *file) >> > +{ >> > + return single_open(file,

[PATCH] drm/omap: fix typo, s/GET_OMAP_REVISION()/GET_OMAP_TYPE/

2012-11-09 Thread Rob Clark
From: Rob Clark The patch 'drm/omap: add support for ARCH_MULTIPLATFORM' had a small mistake, using GET_OMAP_REVISION() instead of GET_OMAP_TYPE. Signed-off-by: Rob Clark --- Same as earlier patch, but updated subject line as Paul Menzel suggested. arch/arm/mach-omap2/drm.c | 2 +- 1 file cha

Re: [PATCH] drm: fix drm_framebuffer cleanup.

2012-11-09 Thread Inki Dae
2012/11/9 Ville Syrjälä > On Fri, Nov 09, 2012 at 11:04:58PM +0900, Inki Dae wrote: > > 2012/11/9 Ville Syrjälä > > > > > On Fri, Nov 09, 2012 at 09:41:19PM +0900, Inki Dae wrote: > > > > 2012/11/9 Ville Syrjälä > > > > > > > > > On Fri, Nov 09, 2012 at 04:39:30PM +0900, Inki Dae wrote: > > > >

Re: [PATCH] drm: fix drm_framebuffer cleanup.

2012-11-09 Thread Inki Dae
2012/11/9 Rob Clark > On Fri, Nov 9, 2012 at 1:39 AM, Inki Dae wrote: > > This patch fixes access issue to invalid memory region. > > > > crtc had only one drm_framebuffer object so when framebuffer > > cleanup was requested after page flip, it'd try to disable > > hardware overlay to current cr

Re: [PATCH] drm: fix drm_framebuffer cleanup.

2012-11-09 Thread Rob Clark
On Fri, Nov 9, 2012 at 10:50 AM, Inki Dae wrote: > I don't think so. let's see it again. below is my idea AND the reason I > posted this patch. > Original codes, > gem alloc(gem0); > -> gem0 refcount = 1 > gem0 mmap > -> gem0 refcount = 2 > gem alloc(gem1); > -> gem1 refcount =1 > gem1 mmap > -> g

Re: [PATCH] drm: fix drm_framebuffer cleanup.

2012-11-09 Thread Rob Clark
On Fri, Nov 9, 2012 at 10:56 AM, Inki Dae wrote: > > > 2012/11/9 Rob Clark >> >> On Fri, Nov 9, 2012 at 1:39 AM, Inki Dae wrote: >> > This patch fixes access issue to invalid memory region. >> > >> > crtc had only one drm_framebuffer object so when framebuffer >> > cleanup was requested after pa

Re: [PATCH] drm: fix drm_framebuffer cleanup.

2012-11-09 Thread Rob Clark
On Fri, Nov 9, 2012 at 1:39 AM, Inki Dae wrote: > diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h > index 3fa18b7..92889be 100644 > --- a/include/drm/drm_crtc.h > +++ b/include/drm/drm_crtc.h > @@ -256,6 +256,7 @@ struct drm_framebuffer { > struct kref refcount; > stru

Re: [PATCH] PCI: Fix bit definitions of PCI_EXP_LNKCAP2 register

2012-11-09 Thread Bjorn Helgaas
On Thu, Nov 8, 2012 at 11:56 PM, Jingoo Han wrote: > According to the PCIe 3.0 spec, PCI_EXP_LNKCAP2_SLS_2_5GB is > 1st bit of PCI_EXP_LNKCAP2 register, not 0th bit. So, the bit > definition of supported link speed vector should be fixed. > > Signed-off-by: Jingoo Han > --- > include/uapi/linux/

[PATCH] drm: fix documentation for drm_crtc_set_mode()

2012-11-09 Thread alexdeucher
From: Alex Deucher x and y parameters are offsets, not width/height Signed-off-by: Alex Deucher Cc: sta...@vger.kernel.org --- drivers/gpu/drm/drm_crtc_helper.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/drm_crtc_helper.c b/drivers/gpu/drm/drm_

Re: [PATCH] PCI: Fix bit definitions of PCI_EXP_LNKCAP2 register

2012-11-09 Thread Alex Deucher
On Fri, Nov 9, 2012 at 12:15 PM, Bjorn Helgaas wrote: > On Thu, Nov 8, 2012 at 11:56 PM, Jingoo Han wrote: >> According to the PCIe 3.0 spec, PCI_EXP_LNKCAP2_SLS_2_5GB is >> 1st bit of PCI_EXP_LNKCAP2 register, not 0th bit. So, the bit >> definition of supported link speed vector should be fixed.

Re: [PATCH] drm: fix drm_framebuffer cleanup.

2012-11-09 Thread Inki Dae
2012. 11. 10. 오전 2:10 Rob Clark 작성: > On Fri, Nov 9, 2012 at 1:39 AM, Inki Dae wrote: >> diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h >> index 3fa18b7..92889be 100644 >> --- a/include/drm/drm_crtc.h >> +++ b/include/drm/drm_crtc.h >> @@ -256,6 +256,7 @@ struct drm_framebuffer

Re: [PATCH] PCI: Fix bit definitions of PCI_EXP_LNKCAP2 register

2012-11-09 Thread Bjorn Helgaas
On Fri, Nov 9, 2012 at 10:37 AM, Alex Deucher wrote: > On Fri, Nov 9, 2012 at 12:15 PM, Bjorn Helgaas wrote: >> On Thu, Nov 8, 2012 at 11:56 PM, Jingoo Han wrote: >>> According to the PCIe 3.0 spec, PCI_EXP_LNKCAP2_SLS_2_5GB is >>> 1st bit of PCI_EXP_LNKCAP2 register, not 0th bit. So, the bit >>

[Bug 56405] Distorted graphics on Radeon HD 6620G

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=56405 --- Comment #15 from Michael Dressel --- I tried to compile c1f4867c89adb1a6b19d66ec8ad146115909f0a7 the commit taged with mesa-8.0.4 and I get the following compilation error: g++ -c -I. -I../../../src/gallium/include -I../../../src/gallium/aux

Re: [PATCH] drm: fix drm_framebuffer cleanup.

2012-11-09 Thread Ville Syrjälä
On Sat, Nov 10, 2012 at 01:50:53AM +0900, Inki Dae wrote: > 2012/11/9 Ville Syrjälä > > > On Fri, Nov 09, 2012 at 11:04:58PM +0900, Inki Dae wrote: > > > 2012/11/9 Ville Syrjälä > > > > > > > On Fri, Nov 09, 2012 at 09:41:19PM +0900, Inki Dae wrote: > > > > > 2012/11/9 Ville Syrjälä > > > > > >

Re: [PATCH v7 5/8] fbmon: add videomode helpers

2012-11-09 Thread Steffen Trumtrar
Hi! On Fri, Nov 09, 2012 at 04:54:16PM +, Manjunathappa, Prakash wrote: > Hi Steffen, > > On Fri, Nov 09, 2012 at 02:55:45, Steffen Trumtrar wrote: > > Hi! > > > > On Wed, Oct 31, 2012 at 03:30:03PM +, Manjunathappa, Prakash wrote: > > > Hi Steffen, > > > > > > On Wed, Oct 31, 2012 at 1

[Bug 56405] Distorted graphics on Radeon HD 6620G

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=56405 --- Comment #16 from Michael Dressel --- I compiled the commit 6671d0dad300e591ac7c0e5110c6778373d0149a which I picked to start bisecting. (I picked it because the date is related to the date of the 8.0 branch as mentioned above) It shows the sa

[Bug 56405] Distorted graphics on Radeon HD 6620G

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=56405 --- Comment #17 from Michael Dressel --- (In reply to comment #14) > (In reply to comment #13) > Maybe it would be easier to just do manual builds instead of using PKGBUILD > for the bisect? You can point to the directory containing the manuall

[Bug 17902] [830M missing dvo driver] need support for DVO-LVDS via na2501

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=17902 Daniel Vetter changed: What|Removed |Added Status|REOPENED|RESOLVED Resolution|---

Re: [PATCH 2/2] drm: tegra: Add HDMI support

2012-11-09 Thread Thierry Reding
On Fri, Nov 09, 2012 at 05:04:09PM +0100, Rafał Miłecki wrote: > 2012/11/9 Christian König : > > On 09.11.2012 16:45, Rafał Miłecki wrote: > >> I was told it won't work on different endian devices. See > >> [RFC][PATCH] drm/radeon/hdmi: define struct for AVI infoframe > >> http://lists.freedesktop.

[Bug 26345] [845G] CPU/GPU incoherency

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=26345 Chris Wilson changed: What|Removed |Added CC||tup...@gentoo.org --- Comment #151 from C

Re: [PATCH 1/2] drm: Add NVIDIA Tegra20 support

2012-11-09 Thread Thierry Reding
On Fri, Nov 09, 2012 at 10:26:27AM -0600, Rob Clark wrote: > On Fri, Nov 9, 2012 at 10:00 AM, Thierry Reding > wrote: > > On Fri, Nov 09, 2012 at 09:18:58AM -0600, Rob Clark wrote: > >> On Fri, Nov 9, 2012 at 7:59 AM, Thierry Reding > >> wrote: > > [...] > >> > +static int regs_open(struct inode

[PATCH] ARM: add get_user() support for 8 byte types

2012-11-09 Thread Rob Clark
From: Rob Clark A new atomic modeset/pageflip ioctl being developed in DRM requires get_user() to work for 64bit types (in addition to just put_user()). Signed-off-by: Rob Clark --- arch/arm/include/asm/uaccess.h | 25 - arch/arm/lib/getuser.S | 17 +

Re: [PATCH 46/51] drm/i915: Add support for atomic modesetting completion events

2012-11-09 Thread Daniel Vetter
On Wed, Nov 07, 2012 at 02:29:45PM -0600, Rob Clark wrote: > On Thu, Nov 1, 2012 at 5:39 PM, Daniel Vetter wrote: > > On Thu, Nov 01, 2012 at 10:12:21AM -0700, Jesse Barnes wrote: > >> On Thu, 1 Nov 2012 19:07:02 +0200 > >> Ville Syrjälä wrote: > >> > >> > On Thu, Nov 01, 2012 at 07:39:12AM -0700

Re: [PATCH 46/51] drm/i915: Add support for atomic modesetting completion events

2012-11-09 Thread Rob Clark
On Fri, Nov 9, 2012 at 3:20 PM, Daniel Vetter wrote: > On Wed, Nov 07, 2012 at 02:29:45PM -0600, Rob Clark wrote: >> On Thu, Nov 1, 2012 at 5:39 PM, Daniel Vetter wrote: >> > On Thu, Nov 01, 2012 at 10:12:21AM -0700, Jesse Barnes wrote: >> >> On Thu, 1 Nov 2012 19:07:02 +0200 >> >> Ville Syrjälä

Re: [PATCH] HACK: drm: Allow encoders to be reenabled

2012-11-09 Thread Daniel Vetter
On Thu, Nov 08, 2012 at 08:01:57PM +0100, Thierry Reding wrote: > On Thu, Nov 08, 2012 at 12:48:18PM -0500, Alex Deucher wrote: > > On Wed, Nov 7, 2012 at 5:06 PM, Thierry Reding > > wrote: > > > When running the xf86-video-modesetting driver on top of a KMS driver, > > > leaving X causes the acti

[Bug 56936] New: cube map array support breaks rendering on BARTS

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=56936 Priority: medium Bug ID: 56936 Assignee: dri-devel@lists.freedesktop.org Summary: cube map array support breaks rendering on BARTS Severity: major Classification: Unclassified

[Bug 56936] cube map array support breaks rendering on BARTS

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=56936 maxi...@free.fr changed: What|Removed |Added Attachment #69839|0 |1 is obsolete|

Re: [PATCH 1/2] drm: Add NVIDIA Tegra20 support

2012-11-09 Thread Stephen Warren
On 11/09/2012 06:59 AM, Thierry Reding wrote: > This commit adds a KMS driver for the Tegra20 SoC. This includes basic > support for host1x and the two display controllers found on the Tegra20 > SoC. Each display controller can drive a separate RGB/LVDS output. > diff --git > a/Documentation/devi

[Bug 56936] cube map array support breaks rendering on BARTS

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=56936 --- Comment #2 from Dave Airlie --- with unigine heaven I get black textures even before this, I'm not sure what I could be going wrong running it here. I'm testing on a caicos but I don't think barts will be too much different. -- You are rec

[Bug 54696] unstable frame rate in Quake 3

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54696 JS changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug 56936] cube map array support breaks rendering on BARTS

2012-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=56936 Dave Airlie changed: What|Removed |Added Status|NEW |NEEDINFO --- Comment #3 from Dave Airlie

Re: [PATCH 1/2] drm: Add NVIDIA Tegra20 support

2012-11-09 Thread Stephen Warren
s and the two arch/arm/mach-tegra patches you posted, directly on top of next-20121109, and I see the following build failure: > drivers/gpu/drm/tegra/output.c: In function 'tegra_output_init': > drivers/gpu/drm/tegra/output.c:166:9: error: 'struct tegra_output' has no

Re: [PATCH] drm: fix drm_framebuffer cleanup.

2012-11-09 Thread Inki Dae
2012/11/10 Ville Syrjälä > On Sat, Nov 10, 2012 at 01:50:53AM +0900, Inki Dae wrote: > > 2012/11/9 Ville Syrjälä > > > > > On Fri, Nov 09, 2012 at 11:04:58PM +0900, Inki Dae wrote: > > > > 2012/11/9 Ville Syrjälä > > > > > > > > > On Fri, Nov 09, 2012 at 09:41:19PM +0900, Inki Dae wrote: > > >

[Bug 56414] [Regression nv 0de9 ]System boot fails with -fixes kernel

2012-11-09 Thread bugzilla-dae...@freedesktop.org
org/archives/dri-devel/attachments/20121109/69dbd508/attachment.html>

[Bug 56414] [Regression nv 0de9 ]System boot fails with -fixes kernel

2012-11-09 Thread bugzilla-dae...@freedesktop.org
--- Verified.Fixed. -- 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/20121109/c3c482e3/attachment.html>

[git pull] drm radeon + nouveau fixes

2012-11-09 Thread Dave Airlie
Hi Linus, just radeon and nouveau, mostly regressions fixers, and a couple of radeon register checker fixes. Dave. The following changes since commit 695ddeb457584a602f2ba117d08ce37cf6ec1589: drm/radeon: fix typo in evergreen_mc_resume() (2012-11-07 10:53:49 +1000) are available in the git

[PATCH] drm: fix drm_framebuffer cleanup.

2012-11-09 Thread Inki Dae
This patch fixes access issue to invalid memory region. crtc had only one drm_framebuffer object so when framebuffer cleanup was requested after page flip, it'd try to disable hardware overlay to current crtc. But if current crtc points to another drm_framebuffer, This may induce invalid memory ac

[PATCH] drm/exynos: fix linux framebuffer address setting.

2012-11-09 Thread Inki Dae
With iommu, buffer->dma_addr has device addres so this patch fixes for physical address to be set to fix.smem_start always. Signed-off-by: Inki Dae Signed-off-by: Kyungmin Park --- drivers/gpu/drm/exynos/exynos_drm_fbdev.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git

[PATCH v7 0/8] of: add display helper

2012-11-09 Thread Steffen Trumtrar
On Thu, Nov 08, 2012 at 03:35:47PM -0600, Rob Herring wrote: > On 10/31/2012 04:28 AM, Steffen Trumtrar wrote: > > Hi! > > > > Finally, v7 of the series. > > > > Changes since v6: > > - get rid of some empty lines etc. > > - move functions to their subsystems > > - split of_ from non-

[PATCH] drm/exynos: add vm_ops to specific gem mmaper

2012-11-09 Thread Inki Dae
This patch makes it takes a reference to gem object when specific gem mmap is requested. For this, it sets dev->driver->gem_vm_ops to vma->vm_ops. And this patch is based on exynos-drm-next-iommu branch of git://git.kernel.org/pub/scm/linux/kernel/git/daeinki/drm-exynos Signed-off-by: Ink

[PATCH] drm: fix drm_framebuffer cleanup.

2012-11-09 Thread Prathyush K
struct drm_framebuffer_funcs *funcs; > unsigned int pitches[4]; > unsigned int offsets[4]; > -- > 1.7.4.1 > > ___ > 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/20121109/2a0513a6/attachment-0001.html>

[PATCH -fixes RESEND] drm/vmwgfx: Fix hibernation device reset

2012-11-09 Thread Thomas Hellstrom
The device would not reset properly when resuming from hibernation. Signed-off-by: Thomas Hellstrom Reviewed-by: Brian Paul Reviewed-by: Dmitry Torokhov Cc: stable at vger.kernel.org Cc: linux-graphics-maintainer at vmware.com --- drivers/gpu/drm/vmwgfx/vmwgfx_drv.c |5 + 1 files chang

[PATCH] drm: fix drm_framebuffer cleanup.

2012-11-09 Thread Inki Dae
uct drm_crtc *crtc; >> const struct drm_framebuffer_funcs *funcs; >> unsigned int pitches[4]; >> unsigned int offsets[4]; >> -- >> 1.7.4.1 >> >> ___ >> dri-devel mailing list >> dri-devel at lists.freedesktop.org >> http://lists.freedesktop.org/mailman/listinfo/dri-devel >> > > > ___ > 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/20121109/a7a753a7/attachment.html>

[PATCH 0/6] alloc_apertures() handling fixes

2012-11-09 Thread Tommi Rantala
Greetings, The following patches fix a few memory leaks and other problems related to the alloc_apertures() call. Tommi Rantala (6): drm/cirrus: check alloc_apertures() success in cirrus_kick_out_firmware_fb() drm/mgag200: check alloc_apertures() success in mga_vram_init() drm/mgag200:

[PATCH 1/6] drm/cirrus: check alloc_apertures() success in cirrus_kick_out_firmware_fb()

2012-11-09 Thread Tommi Rantala
Check for alloc_apertures() memory allocation failure, and propagate an error code in case the allocation failed. Signed-off-by: Tommi Rantala --- drivers/gpu/drm/cirrus/cirrus_drv.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/cirrus/cirru

[PATCH 2/6] drm/mgag200: check alloc_apertures() success in mga_vram_init()

2012-11-09 Thread Tommi Rantala
Check for alloc_apertures() memory allocation failure, and propagate an error code in case the allocation failed. Signed-off-by: Tommi Rantala --- drivers/gpu/drm/mgag200/mgag200_main.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/mgag200/mgag200_main.c b/drivers/gpu

[PATCH 3/6] drm/mgag200: free memory allocated with alloc_apertures()

2012-11-09 Thread Tommi Rantala
Fix a memory leak by deallocating the memory we got from alloc_apertures(). Signed-off-by: Tommi Rantala --- drivers/gpu/drm/mgag200/mgag200_main.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/mgag200/mgag200_main.c b/drivers/gpu/drm/mgag200/mgag200_main.c index 07d4b2

[PATCH 4/6] drm/mgag200: remove unneeded aper->count assignment after alloc_apertures()

2012-11-09 Thread Tommi Rantala
alloc_apertures() already does the assignment for us, so assigning the count member after the alloc_apertures() call is not needed. Signed-off-by: Tommi Rantala --- drivers/gpu/drm/mgag200/mgag200_main.c |1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/mgag200/mgag200_main.c

[PATCH 5/6] drm/radeon: check alloc_apertures() success in radeon_kick_out_firmware_fb()

2012-11-09 Thread Tommi Rantala
Check for alloc_apertures() memory allocation failure, and propagate an error code in case the allocation failed. Signed-off-by: Tommi Rantala --- drivers/gpu/drm/radeon/radeon_drv.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeo

[PATCH 6/6] drm/nouveau: free memory allocated with alloc_apertures()

2012-11-09 Thread Tommi Rantala
Fix a memory leak by deallocating the memory we got from alloc_apertures(). Signed-off-by: Tommi Rantala --- drivers/gpu/drm/nouveau/nouveau_drm.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/nouveau/nouveau_drm.c b/drivers/gpu/drm/nouveau/nouveau_drm.c index 0910125..

  1   2   >