Re: [PATCH v16 RESEND 0/7] of: add display helper

2013-01-23 Thread Steffen Trumtrar
On Thu, Jan 24, 2013 at 10:15:54AM +1000, Dave Airlie wrote: > >> > Hi! > >> > > >> > There was still no maintainer, that commented, ack'd, nack'd, apply'd the > >> > series. So, this is just a resend. > >> > The patches were tested with: > >> > > >> > - v15 on Tegra by Thierry > >> >

[PATCH] fbcon: don't lose the console font across generic->chip driver switch

2013-01-23 Thread Dave Airlie
From: Dave Airlie If grub2 loads efifb/vesafb, then when systemd starts it can set the console font on that framebuffer device, however when we then load the native KMS driver, the first thing it does is tear down the generic framebuffer driver. The thing is the generic code is doing the right t

Re: [PATCH] vgacon/vt: clear buffer attributes when we load a 512 character font

2013-01-23 Thread H. Peter Anvin
We should clear this bit presumably on switching either from or to 512-char mode, since the bit doesn't really make sense either way. Dave Airlie wrote: >From: Dave Airlie > >When we switch from 256->512 byte font rendering mode, it means the >current contents of the screen is being reinterpre

[PATCH] vgacon/vt: clear buffer attributes when we load a 512 character font

2013-01-23 Thread H. Peter Anvin
We should clear this bit presumably on switching either from or to 512-char mode, since the bit doesn't really make sense either way. Dave Airlie wrote: >From: Dave Airlie > >When we switch from 256->512 byte font rendering mode, it means the >current contents of the screen is being reinterpre

Re: [PATCH v16 RESEND 0/7] of: add display helper

2013-01-23 Thread Leela Krishna Amudala
Steffen, You can add my tested-by for this series.. :) I have been using them for Exynos: smdk5250 board. On Wed, Jan 23, 2013 at 2:42 PM, Steffen Trumtrar wrote: > On Tue, Jan 22, 2013 at 03:50:48PM -0600, Rob Clark wrote: >> On Mon, Jan 21, 2013 at 5:07 AM, Steffen Trumtrar >> wrote: >> > Hi!

[PATCH] drm/exynos: add check for the device power status

2013-01-23 Thread Shirish S
V2: Add mutex protection, while read. The hdmi and mixer win_commit calls currently are not checking the status of IP before updating the respective registers, this patch adds this check. Signed-off-by: Shirish S --- drivers/gpu/drm/exynos/exynos_hdmi.c | 7 +++ drivers/gpu/drm/exynos/exyn

[PATCH V3] drm_crtc: check if fb_create return NULL

2013-01-23 Thread Su, Xuemin
From: xueminsu Date: Tue, 22 Jan 2013 22:39:39 +0800 Subject: [PATCH] drm_crtc: check if fb_create return NULL Some buggy driver may still return NULL in fb_create, which leads to kernel panic. Signed-off-by: xueminsu --- drivers/gpu/drm/drm_crtc.c |2 ++ 1 files changed, 2 insertions(+),

RE: [PATCH V2 RESEND] drm_crtc: check if fb_create return NULL

2013-01-23 Thread Su, Xuemin
-Original Message- >From: Daniel Vetter [mailto:daniel.vet...@ffwll.ch] On Behalf Of Daniel Vetter >Sent: Wednesday, January 23, 2013 5:54 PM >To: Su, Xuemin >Cc: airl...@linux.ie; dri-devel@lists.freedesktop.org; >linux-ker...@vger.kernel.org; yanmin_zh...@linux.intel.com; He, Bo >Subject

Re: [Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-23 Thread Francesco Lavra
On 01/23/2013 03:56 PM, Maarten Lankhorst wrote: > Thanks for the review, how does this delta look? > > diff --git a/include/linux/fence.h b/include/linux/fence.h > index d9f091d..831ed0a 100644 > --- a/include/linux/fence.h > +++ b/include/linux/fence.h > @@ -42,7 +42,10 @@ struct fence_cb; > *

[PATCH] drm/exynos: add check for the device power status

2013-01-23 Thread Shirish S
V2: Add mutex protection, while read. The hdmi and mixer win_commit calls currently are not checking the status of IP before updating the respective registers, this patch adds this check. Signed-off-by: Shirish S --- drivers/gpu/drm/exynos/exynos_hdmi.c | 7 +++ drivers/gpu/drm/exynos/exyn

[Bug 59431] [snb efi gmux] imac 12,1 blank screen upon booting

2013-01-23 Thread bugzilla-dae...@freedesktop.org
because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20130123/29ee980d/attachment.html>

[Bug 59588] llvm rv790 etqw gpu lock since r600g/llvm: tgsi to llvm emits store.swizzle intrinsic for vs/fs output

2013-01-23 Thread bugzilla-dae...@freedesktop.org
e: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20130123/f05adcca/attachment.html>

[Bug 59431] [snb efi gmux] imac 12,1 blank screen upon booting

2013-01-23 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/20130123/a8029fe0/attachment.html>

[PATCH] vgacon/vt: clear buffer attributes when we load a 512 character font

2013-01-23 Thread Dave Airlie
From: Dave Airlie When we switch from 256->512 byte font rendering mode, it means the current contents of the screen is being reinterpreted. The bit that holds the high bit of the 9-bit font, may have been previously set, and thus the new font misrenders. The problem case we see is grub2 writes

[Bug 59431] [snb efi gmux] imac 12,1 blank screen upon booting

2013-01-23 Thread bugzilla-dae...@freedesktop.org
TML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20130123/a3c1e53d/attachment.html>

Re: [PATCH] drm/exynos: add check for the device power status

2013-01-23 Thread Shirish S
On Wed, Jan 23, 2013 at 8:18 PM, Sean Paul wrote: > On Wed, Jan 23, 2013 at 5:10 AM, Shirish S wrote: > > The hdmi and mixer win_commit calls currently are > > not checking the status of IP before updating the > > respective registers, this patch adds this check. > > > > Signed-off-by: Shirish S

[Bug 59431] [snb efi gmux] imac 12,1 blank screen upon booting

2013-01-23 Thread bugzilla-dae...@freedesktop.org
. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20130123/0b0451b9/attachment.html>

[Bug 59431] [snb efi gmux] imac 12,1 blank screen upon booting

2013-01-23 Thread bugzilla-dae...@freedesktop.org
-- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20130123/40424a1a/attachment-0001.html>

[Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-23 Thread Francesco Lavra
On 01/23/2013 03:56 PM, Maarten Lankhorst wrote: > Thanks for the review, how does this delta look? > > diff --git a/include/linux/fence.h b/include/linux/fence.h > index d9f091d..831ed0a 100644 > --- a/include/linux/fence.h > +++ b/include/linux/fence.h > @@ -42,7 +42,10 @@ struct fence_cb; > *

[PATCH 1/2] fb: Rework locking to fix lock ordering on takeover

2013-01-23 Thread Daniel Vetter
On Wed, Jan 23, 2013 at 5:38 PM, Takashi Iwai wrote: > At Wed, 23 Jan 2013 17:25:08 +0100, > Daniel Vetter wrote: >> >> From: Alan Cox >> >> Adjust the console layer to allow a take over call where the caller already >> holds the locks. Make the fb layer lock in order. >> >> This s partly a band

[PATCH 1/2] fb: Rework locking to fix lock ordering on takeover

2013-01-23 Thread Takashi Iwai
At Wed, 23 Jan 2013 17:25:08 +0100, Daniel Vetter wrote: > > From: Alan Cox > > Adjust the console layer to allow a take over call where the caller already > holds the locks. Make the fb layer lock in order. > > This s partly a band aid, the fb layer is terminally confused about the > locking r

[Bug 59431] [snb efi gmux] imac 12,1 blank screen upon booting

2013-01-23 Thread bugzilla-dae...@freedesktop.org
: <http://lists.freedesktop.org/archives/dri-devel/attachments/20130123/c332b96d/attachment.html>

[PATCH 2/2] drm/i915: fixup per-crtc locking in intel_release_load_detect_pipe

2013-01-23 Thread Daniel Vetter
One of the early return cases missed the mutex unlocking. Hilarity ensued. This regression has been introduced in commit 7b24056be6db7ce907baffdd4cf142ab774ea60c Author: Daniel Vetter Date: Wed Dec 12 00:35:33 2012 +0100 drm: don't hold crtc mutexes for connector ->detect callbacks Bugzi

[PATCH 1/2] fb: Rework locking to fix lock ordering on takeover

2013-01-23 Thread Daniel Vetter
From: Alan Cox Adjust the console layer to allow a take over call where the caller already holds the locks. Make the fb layer lock in order. This s partly a band aid, the fb layer is terminally confused about the locking rules it uses for its notifiers it seems. Signed-off-by: Alan Cox [danvet

[PATCH 0/2] fbcon+i915 locking fixes

2013-01-23 Thread Daniel Vetter
Hi Dave, First patch is a resend of Alan's fbcon vs. fb notifier deadlock fix. Without this lockdep is pretty much useless for debugging drm locking issues, and it looks like quite a few bootup hangs could be blamed on this one, too. Since the fbdev maintainer seems to be awol, please merge this

[PATCH 4/5] drm/tegra: Implement VBLANK support

2013-01-23 Thread Mark Zhang
On 01/23/2013 02:21 AM, Jon Mayo wrote: > On Mon, Jan 14, 2013 at 7:55 AM, Thierry Reding > wrote: >> Implement support for the VBLANK IOCTL. Note that Tegra is somewhat >> special in this case because it doesn't use the generic IRQ support >> provided by the DRM core (DRIVER_HAVE_IRQ) but rather

[Bug 43751] [TTM] Out of kernel memory

2013-01-23 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=43751 am.devel at gmail.com changed: What|Removed |Added CC||am.devel at gmail.com --- Comm

Re: [PATCH v16 RESEND 0/7] of: add display helper

2013-01-23 Thread Dave Airlie
>> > Hi! >> > >> > There was still no maintainer, that commented, ack'd, nack'd, apply'd the >> > series. So, this is just a resend. >> > The patches were tested with: >> > >> > - v15 on Tegra by Thierry >> > - sh-mobile-lcdcfb by Laurent >> > - MX53QSB by Marek >> >

[Bug 59588] llvm rv790 etqw gpu lock since r600g/llvm: tgsi to llvm emits store.swizzle intrinsic for vs/fs output

2013-01-23 Thread bugzilla-dae...@freedesktop.org
-- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20130123/b16afee9/attachment.html>

[Bug 59588] llvm rv790 etqw gpu lock since r600g/llvm: tgsi to llvm emits store.swizzle intrinsic for vs/fs output

2013-01-23 Thread bugzilla-dae...@freedesktop.org
-- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20130123/a4c3cc6a/attachment.html>

[Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-23 Thread Maarten Lankhorst
Op 22-01-13 16:13, Francesco Lavra schreef: > Hi, > > On 01/15/2013 01:34 PM, Maarten Lankhorst wrote: > [...] >> diff --git a/include/linux/fence.h b/include/linux/fence.h >> new file mode 100644 >> index 000..d9f091d >> --- /dev/null >> +++ b/include/linux/fence.h >> @@ -0,0 +1,347 @@ >> +/*

[Bug 59431] [snb efi gmux] imac 12,1 blank screen upon booting

2013-01-23 Thread bugzilla-dae...@freedesktop.org
eceiving 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/20130123/e9a8d622/attachment.html>

[PATCH] drm/radeon: fix NULL pointer dereference in UMS mode in radeon_cs_parser_fini()

2013-01-23 Thread Herton Ronaldo Krzesinski
On Wed, Jan 23, 2013 at 12:21:29PM -0500, Ilija Hadzic wrote: > On Wed, Jan 23, 2013 at 11:07 AM, Herton Ronaldo Krzesinski > wrote: > > On Thu, Jan 17, 2013 at 10:09:42AM -0700, Shuah Khan wrote: > >> On Wed, 2013-01-16 at 21:06 -0600, Ilija Hadzic wrote: > >> > Actually, the code path affected b

[PATCH] drm/radeon: fix a rare case of double kfree

2013-01-23 Thread Alex Deucher
On Wed, Jan 23, 2013 at 1:59 PM, Ilija Hadzic wrote: > If one (but not both) allocations of p->chunks[].kpage[] > in radeon_cs_parser_init fail, the error path will free > the successfully allocated page, but leave a stale pointer > value in the kpage[] field. This will later cause a > double-free

[PATCH] drm/radeon: fix NULL pointer dereference in UMS mode in radeon_cs_parser_fini()

2013-01-23 Thread Herton Ronaldo Krzesinski
On Thu, Jan 17, 2013 at 10:09:42AM -0700, Shuah Khan wrote: > On Wed, 2013-01-16 at 21:06 -0600, Ilija Hadzic wrote: > > Actually, the code path affected by your patch is not executed in UMS mode > > at all. Notice that radeon_cs_parser_fini is only called from > > radeon_cs_ioctl which is a KMS-

[Bug 59431] [snb efi gmux] imac 12,1 blank screen upon booting

2013-01-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=59431 --- Comment #24 from Seth Forshee --- Created attachment 73542 --> https://bugs.freedesktop.org/attachment.cgi?id=73542&action=edit Add quirk to efifb for iMac 12,1 Here's a patch to try. But after some more poking around I'm wondering why thi

[PATCH] drm/radeon: fix a rare case of double kfree

2013-01-23 Thread Ilija Hadzic
If one (but not both) allocations of p->chunks[].kpage[] in radeon_cs_parser_init fail, the error path will free the successfully allocated page, but leave a stale pointer value in the kpage[] field. This will later cause a double-free when radeon_cs_parser_fini is called. This patch fixes the issu

[Bug 59588] llvm rv790 etqw gpu lock since r600g/llvm: tgsi to llvm emits store.swizzle intrinsic for vs/fs output

2013-01-23 Thread bugzilla-dae...@freedesktop.org
dummy one. -- 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/20130123/917f5472/attachment.html>

[PATCH 1/4] drm/tilcdc: add TI LCD Controller DRM driver (v3)

2013-01-23 Thread Russell King - ARM Linux
On Wed, Jan 23, 2013 at 07:24:33AM -0600, Rob Clark wrote: > On Wed, Jan 23, 2013 at 3:42 AM, Jean-Francois Moine > wrote: > > Hi Rob, > > > > As I wanted to re-use your nxp-tda998x driver for the Marvell Dove SoC, > > I had a look at your IT LCD driver. Comments below. > > Just fyi, you can re-

[Bug 59588] llvm rv790 etqw gpu lock since r600g/llvm: tgsi to llvm emits store.swizzle intrinsic for vs/fs output

2013-01-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=59588 --- Comment #12 from Andy Furniss --- (In reply to comment #10) > Can you test with this new patch ? (Remove the previous one) > It adds dummy export to vs outputs It works OK with the new patch. -- You are receiving this mail because: You are

[Bug 59431] [snb efi gmux] imac 12,1 blank screen upon booting

2013-01-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=59431 --- Comment #23 from William --- Here the information from the output: <"Apple Inc."> <"iMac12,1"> FrameBuff erBase: 0x9001 PixelsPerScanLine: 1920 HorizontalResolution: 1920 VerticalResolution: 1080 -- You are receiving this mail becaus

[patch] drm/nouveau/disp: sizeof() wrong pointer

2013-01-23 Thread Bernd Petrovitsch
On Mit, 2013-01-23 at 11:38 +0300, Dan Carpenter wrote: > On Tue, Jan 22, 2013 at 10:42:25AM +0100, Paul Menzel wrote: > > > > Did you find this by manual inspection or did you use some tool? > > I found this because it caused a problem in a parser I was working > on but Sparse warns about "warni

[PATCH] drm/radeon: fix NULL pointer dereference in UMS mode in radeon_cs_parser_fini()

2013-01-23 Thread Ilija Hadzic
On Wed, Jan 23, 2013 at 11:07 AM, Herton Ronaldo Krzesinski wrote: > On Thu, Jan 17, 2013 at 10:09:42AM -0700, Shuah Khan wrote: >> On Wed, 2013-01-16 at 21:06 -0600, Ilija Hadzic wrote: >> > Actually, the code path affected by your patch is not executed in UMS mode >> > at all. Notice that radeon

linux-3.7.4: Regression causing a kmemleak in drm_pci_set_busid()

2013-01-23 Thread Martin Mokrejs
Hi, I have kmemleak detector enabled since 3.7.1 but now, with 3.7.4 I get the following: comm "X", pid 5475, jiffies 4294992244 (age 133695.910s) hex dump (first 32 bytes): 69 39 31 35 40 70 63 69 3a 30 30 30 30 3a 30 30 i915 at pci::00 3a 30 32 2e 30 00 6b 6b 6b 6b 6b 6b 6b 6

Re: [PATCH] drm/radeon: fix a rare case of double kfree

2013-01-23 Thread Alex Deucher
On Wed, Jan 23, 2013 at 1:59 PM, Ilija Hadzic wrote: > If one (but not both) allocations of p->chunks[].kpage[] > in radeon_cs_parser_init fail, the error path will free > the successfully allocated page, but leave a stale pointer > value in the kpage[] field. This will later cause a > double-free

[Bug 59431] [snb efi gmux] imac 12,1 blank screen upon booting

2013-01-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=59431 --- Comment #22 from Seth Forshee --- (In reply to comment #21) > Hi Seth, > > the output is: > bios vendor: Apple Inc. > system product name: iMac12,1 Rats, it turns out there's more information required. If you can boot into OS X and run the

[patch] drm/nouveau/disp: sizeof() wrong pointer

2013-01-23 Thread Dan Carpenter
On Tue, Jan 22, 2013 at 10:42:25AM +0100, Paul Menzel wrote: > > Did you find this by manual inspection or did you use some tool? > I found this because it caused a problem in a parser I was working on but Sparse warns about "warning: expression using sizeof(void)". It's sort of hard to run Spa

[patch v2] drm/nouveau/disp: sizeof() wrong pointer

2013-01-23 Thread Dan Carpenter
"data" is a void pointer and "args" is "data" after we have casted it to a struct. We care about the size of the struct here. Btw, sizeof(*data) is 1. Signed-off-by: Dan Carpenter --- v2: tweaked the commit message diff --git a/drivers/gpu/drm/nouveau/core/engine/disp/nv50.c b/drivers/gpu/drm

[PATCH V2 RESEND] drm_crtc: check if fb_create return NULL

2013-01-23 Thread Su, Xuemin
From: xueminsu Date: Tue, 22 Jan 2013 22:39:39 +0800 Subject: [PATCH] drm_crtc: check if fb_create return NULL Some buggy driver may still return NULL in fb_create, which leads to kernel panic. Signed-off-by: xueminsu --- drivers/gpu/drm/drm_crtc.c |7 +++ 1 files changed, 7 insertions

[Bug 58272] Rv670 AGP drm-next ttm errors

2013-01-23 Thread bugzilla-dae...@freedesktop.org
or the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20130123/c2c778d6/attachment.html>

[PATCH] drm/radeon: fix a rare case of double kfree

2013-01-23 Thread Ilija Hadzic
If one (but not both) allocations of p->chunks[].kpage[] in radeon_cs_parser_init fail, the error path will free the successfully allocated page, but leave a stale pointer value in the kpage[] field. This will later cause a double-free when radeon_cs_parser_fini is called. This patch fixes the issu

[PATCH V2 RESEND] drm_crtc: check if fb_create return NULL

2013-01-23 Thread Daniel Vetter
On Wed, Jan 23, 2013 at 11:19:27AM +0800, Su, Xuemin wrote: > From: xueminsu > Date: Tue, 22 Jan 2013 22:39:39 +0800 > Subject: [PATCH] drm_crtc: check if fb_create return NULL > > Some buggy driver may still return NULL in fb_create, > which leads to kernel panic. > > Signed-off-by: xueminsu

[PATCH 2/4] drm/i2c: nxp-tda998x (v2)

2013-01-23 Thread Jean-Francois Moine
On Tue, 22 Jan 2013 16:36:23 -0600 Rob Clark wrote: > Driver for the NXP TDA998X i2c hdmi encoder slave. > > v1: original > v2: fix npix/nline programming > > Signed-off-by: Rob Clark > --- > drivers/gpu/drm/i2c/Makefile | 3 + > drivers/gpu/drm/i2c/tda998x_drv.c | 908 > +

[PATCH 1/4] drm/tilcdc: add TI LCD Controller DRM driver (v3)

2013-01-23 Thread Jean-Francois Moine
Hi Rob, As I wanted to re-use your nxp-tda998x driver for the Marvell Dove SoC, I had a look at your IT LCD driver. Comments below. On Tue, 22 Jan 2013 16:36:22 -0600 Rob Clark wrote: > A simple DRM/KMS driver for the TI LCD Controller found in various > smaller TI parts (AM33xx, OMAPL138, etc)

[Bug 59431] [snb efi gmux] imac 12,1 blank screen upon booting

2013-01-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=59431 --- Comment #21 from William --- Hi Seth, the output is: bios vendor: Apple Inc. system product name: iMac12,1 Could that also explain why X is not working? -- You are receiving this mail because: You are the assignee for the bug. ___

[PATCH] drm_crtc: check if fb_create return NULL

2013-01-23 Thread Su, Xuemin
From: xueminsu Date: Tue, 22 Jan 2013 22:39:39 +0800 Subject: [PATCH] drm_crtc: check if fb_create return NULL Some buggy driver may still return NULL in fb_create, which leads to kernel panic. Signed-off-by: xueminsu --- drivers/gpu/drm/drm_crtc.c |6 ++ 1 files changed, 6 insertions(

[PATCH] radeon_display: Use pointer return error codes

2013-01-23 Thread Su, Xuemin
From: xueminsu Date: Tue, 22 Jan 2013 22:16:53 +0800 Subject: [PATCH] radeon_display: Use pointer return error codes drm_mode_addfb() expects fb_create return error code instead of NULL. Signed-off-by: xueminsu --- drivers/gpu/drm/radeon/radeon_display.c |2 +- 1 files changed, 1 insertion

[Bug 59431] [snb efi gmux] imac 12,1 blank screen upon booting

2013-01-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=59431 --- Comment #20 from Seth Forshee --- Okay, thanks. I suspect at least part of the problem may be related to efifb. I don't see that it's being used on your machine, and I note that it has a bunch of quirks for making it work with various Mac ha

[PATCH v16 RESEND 0/7] of: add display helper

2013-01-23 Thread Steffen Trumtrar
On Tue, Jan 22, 2013 at 03:50:48PM -0600, Rob Clark wrote: > On Mon, Jan 21, 2013 at 5:07 AM, Steffen Trumtrar > wrote: > > Hi! > > > > There was still no maintainer, that commented, ack'd, nack'd, apply'd the > > series. So, this is just a resend. > > The patches were tested with: > > > >

[PATCH v2 4/5] drm/tegra: Implement VBLANK support

2013-01-23 Thread Terje Bergström
On 22.01.2013 21:59, Mario Kleiner wrote: > The current swap scheduling is based on having an accurate software > vblank counter. Atm. that counter is maintained in software, incremented > during vblank irq. At irq off -> on transition we need a hw counter to > reinitialize. And there is a timeo

[PATCH] drm/exynos: add check for the device power status

2013-01-23 Thread Sean Paul
On Wed, Jan 23, 2013 at 5:10 AM, Shirish S wrote: > The hdmi and mixer win_commit calls currently are > not checking the status of IP before updating the > respective registers, this patch adds this check. > > Signed-off-by: Shirish S > --- > drivers/gpu/drm/exynos/exynos_hdmi.c | 3 +++ > driv

Re: [PATCH] drm/radeon: fix NULL pointer dereference in UMS mode in radeon_cs_parser_fini()

2013-01-23 Thread Herton Ronaldo Krzesinski
On Wed, Jan 23, 2013 at 12:21:29PM -0500, Ilija Hadzic wrote: > On Wed, Jan 23, 2013 at 11:07 AM, Herton Ronaldo Krzesinski > wrote: > > On Thu, Jan 17, 2013 at 10:09:42AM -0700, Shuah Khan wrote: > >> On Wed, 2013-01-16 at 21:06 -0600, Ilija Hadzic wrote: > >> > Actually, the code path affected b

[Bug 59431] [snb efi gmux] imac 12,1 blank screen upon booting

2013-01-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=59431 --- Comment #19 from William --- Hi Seth, i have been testing with raring using 3.8 kernel. On 64 bit kernel i get a console screen using the i915.disable=1 but not on the 32bit kernel. see also the dmesg files attached. When using the 3.5 ker

Re: [PATCH] drm/radeon: fix NULL pointer dereference in UMS mode in radeon_cs_parser_fini()

2013-01-23 Thread Ilija Hadzic
On Wed, Jan 23, 2013 at 11:07 AM, Herton Ronaldo Krzesinski wrote: > On Thu, Jan 17, 2013 at 10:09:42AM -0700, Shuah Khan wrote: >> On Wed, 2013-01-16 at 21:06 -0600, Ilija Hadzic wrote: >> > Actually, the code path affected by your patch is not executed in UMS mode >> > at all. Notice that radeon

[PATCH] radeon_display: Use pointer return error codes

2013-01-23 Thread Alex Deucher
On Tue, Jan 22, 2013 at 9:27 PM, Su, Xuemin wrote: > From: xueminsu > Date: Tue, 22 Jan 2013 22:16:53 +0800 > Subject: [PATCH] radeon_display: Use pointer return error codes > > drm_mode_addfb() expects fb_create return error code > instead of NULL. > > Signed-off-by: xueminsu Thanks. Added to

[Bug 43751] [TTM] Out of kernel memory

2013-01-23 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=43751 am.de...@gmail.com changed: What|Removed |Added CC||am.de...@gmail.com --- Comment #7

Re: [PATCH 1/2] fb: Rework locking to fix lock ordering on takeover

2013-01-23 Thread Daniel Vetter
On Wed, Jan 23, 2013 at 5:38 PM, Takashi Iwai wrote: > At Wed, 23 Jan 2013 17:25:08 +0100, > Daniel Vetter wrote: >> >> From: Alan Cox >> >> Adjust the console layer to allow a take over call where the caller already >> holds the locks. Make the fb layer lock in order. >> >> This s partly a band

[PATCH 0/4] TI LCDC DRM driver

2013-01-23 Thread Sascha Hauer
Hi Rob, On Tue, Jan 22, 2013 at 04:36:21PM -0600, Rob Clark wrote: > > drivers/gpu/drm/Kconfig| 2 + > drivers/gpu/drm/Makefile | 1 + > drivers/gpu/drm/i2c/Makefile | 3 + > drivers/gpu/drm/i2c/tda998x_drv.c | 908 > +++

[PATCH 2/4] drm/i2c: nxp-tda998x (v2)

2013-01-23 Thread Koen Kooi
Op 22 jan. 2013, om 23:36 heeft Rob Clark het volgende geschreven: > Driver for the NXP TDA998X i2c hdmi encoder slave. > > v1: original > v2: fix npix/nline programming > > Signed-off-by: Rob Clark Tested on a beaglebone-black: Tested-by: Koen Kooi

[PATCH 1/4] drm/tilcdc: add TI LCD Controller DRM driver (v3)

2013-01-23 Thread Koen Kooi
Op 22 jan. 2013, om 23:36 heeft Rob Clark het volgende geschreven: > A simple DRM/KMS driver for the TI LCD Controller found in various > smaller TI parts (AM33xx, OMAPL138, etc). This driver uses the > CMA helpers. Currently only the TFP410 DVI encoder is supported > (tested with beaglebone

thoughts on requiring multi-arch support for arm drm drivers?

2013-01-23 Thread Sascha Hauer
On Wed, Jan 23, 2013 at 02:29:25AM +0100, Laurent Pinchart wrote: > Hi Rob, > > On Monday 21 January 2013 09:54:01 Rob Clark wrote: > > On Mon, Jan 21, 2013 at 9:47 AM, Laurent Pinchart wrote: > > > On Sunday 20 January 2013 09:08:34 Rob Clark wrote: > > >> One thing I've run into in the past when

Re: [PATCH 1/2] fb: Rework locking to fix lock ordering on takeover

2013-01-23 Thread Takashi Iwai
At Wed, 23 Jan 2013 17:25:08 +0100, Daniel Vetter wrote: > > From: Alan Cox > > Adjust the console layer to allow a take over call where the caller already > holds the locks. Make the fb layer lock in order. > > This s partly a band aid, the fb layer is terminally confused about the > locking r

[PATCH 1/4] drm/tilcdc: add TI LCD Controller DRM driver (v3)

2013-01-23 Thread Rob Clark
On Wed, Jan 23, 2013 at 8:13 AM, Rob Clark wrote: > On Wed, Jan 23, 2013 at 7:36 AM, Russell King - ARM Linux > wrote: >> On Wed, Jan 23, 2013 at 07:24:33AM -0600, Rob Clark wrote: >>> On Wed, Jan 23, 2013 at 3:42 AM, Jean-Francois Moine >>> wrote: >>> > Hi Rob, >>> > >>> > As I wanted to re-us

[PATCH 2/2] drm/i915: fixup per-crtc locking in intel_release_load_detect_pipe

2013-01-23 Thread Daniel Vetter
One of the early return cases missed the mutex unlocking. Hilarity ensued. This regression has been introduced in commit 7b24056be6db7ce907baffdd4cf142ab774ea60c Author: Daniel Vetter Date: Wed Dec 12 00:35:33 2012 +0100 drm: don't hold crtc mutexes for connector ->detect callbacks Bugzi

[PATCH 1/2] fb: Rework locking to fix lock ordering on takeover

2013-01-23 Thread Daniel Vetter
From: Alan Cox Adjust the console layer to allow a take over call where the caller already holds the locks. Make the fb layer lock in order. This s partly a band aid, the fb layer is terminally confused about the locking rules it uses for its notifiers it seems. Signed-off-by: Alan Cox [danvet

[PATCH 0/2] fbcon+i915 locking fixes

2013-01-23 Thread Daniel Vetter
Hi Dave, First patch is a resend of Alan's fbcon vs. fb notifier deadlock fix. Without this lockdep is pretty much useless for debugging drm locking issues, and it looks like quite a few bootup hangs could be blamed on this one, too. Since the fbdev maintainer seems to be awol, please merge this

[PATCH 1/4] drm/tilcdc: add TI LCD Controller DRM driver (v3)

2013-01-23 Thread Rob Clark
On Wed, Jan 23, 2013 at 7:36 AM, Russell King - ARM Linux wrote: > On Wed, Jan 23, 2013 at 07:24:33AM -0600, Rob Clark wrote: >> On Wed, Jan 23, 2013 at 3:42 AM, Jean-Francois Moine >> wrote: >> > Hi Rob, >> > >> > As I wanted to re-use your nxp-tda998x driver for the Marvell Dove SoC, >> > I ha

Re: [PATCH] drm/radeon: fix NULL pointer dereference in UMS mode in radeon_cs_parser_fini()

2013-01-23 Thread Herton Ronaldo Krzesinski
On Thu, Jan 17, 2013 at 10:09:42AM -0700, Shuah Khan wrote: > On Wed, 2013-01-16 at 21:06 -0600, Ilija Hadzic wrote: > > Actually, the code path affected by your patch is not executed in UMS mode > > at all. Notice that radeon_cs_parser_fini is only called from > > radeon_cs_ioctl which is a KMS-

[Bug 59588] llvm rv790 etqw gpu lock since r600g/llvm: tgsi to llvm emits store.swizzle intrinsic for vs/fs output

2013-01-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=59588 --- Comment #11 from vincent --- Created attachment 73532 --> https://bugs.freedesktop.org/attachment.cgi?id=73532&action=edit Add dummy vs export -- You are receiving this mail because: You are the assignee for the bug. _

[Bug 59588] llvm rv790 etqw gpu lock since r600g/llvm: tgsi to llvm emits store.swizzle intrinsic for vs/fs output

2013-01-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=59588 --- Comment #10 from vincent --- Can you test with this new patch ? (Remove the previous one) It adds dummy export to vs outputs -- You are receiving this mail because: You are the assignee for the bug. _

[Bug 59431] [snb efi gmux] imac 12,1 blank screen upon booting

2013-01-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=59431 Seth Forshee changed: What|Removed |Added CC||seth.fors...@canonical.com --- Comment #1

[PATCH 2/4] drm/i2c: nxp-tda998x (v2)

2013-01-23 Thread Rob Clark
On Wed, Jan 23, 2013 at 3:42 AM, Jean-Francois Moine wrote: > On Tue, 22 Jan 2013 16:36:23 -0600 > Rob Clark wrote: > >> Driver for the NXP TDA998X i2c hdmi encoder slave. >> >> v1: original >> v2: fix npix/nline programming >> >> Signed-off-by: Rob Clark >> --- >> drivers/gpu/drm/i2c/Makefile

[PATCH 1/4] drm/tilcdc: add TI LCD Controller DRM driver (v3)

2013-01-23 Thread Rob Clark
On Wed, Jan 23, 2013 at 3:42 AM, Jean-Francois Moine wrote: > Hi Rob, > > As I wanted to re-use your nxp-tda998x driver for the Marvell Dove SoC, > I had a look at your IT LCD driver. Comments below. Just fyi, you can re-use the nxp-tda998x part independently of tilcdc (just in case that wasn't c

[Bug 58667] Random crashes on CAYMAN

2013-01-23 Thread bugzilla-dae...@freedesktop.org
... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20130123/0b848160/attachment.html>

Re: [Linaro-mm-sig] [PATCH 4/7] fence: dma-buf cross-device synchronization (v11)

2013-01-23 Thread Maarten Lankhorst
Op 22-01-13 16:13, Francesco Lavra schreef: > Hi, > > On 01/15/2013 01:34 PM, Maarten Lankhorst wrote: > [...] >> diff --git a/include/linux/fence.h b/include/linux/fence.h >> new file mode 100644 >> index 000..d9f091d >> --- /dev/null >> +++ b/include/linux/fence.h >> @@ -0,0 +1,347 @@ >> +/*

Re: [PATCH] drm/exynos: add check for the device power status

2013-01-23 Thread Sean Paul
On Wed, Jan 23, 2013 at 5:10 AM, Shirish S wrote: > The hdmi and mixer win_commit calls currently are > not checking the status of IP before updating the > respective registers, this patch adds this check. > > Signed-off-by: Shirish S > --- > drivers/gpu/drm/exynos/exynos_hdmi.c | 3 +++ > driv

Re: [PATCH 1/4] drm/tilcdc: add TI LCD Controller DRM driver (v3)

2013-01-23 Thread Rob Clark
On Wed, Jan 23, 2013 at 8:13 AM, Rob Clark wrote: > On Wed, Jan 23, 2013 at 7:36 AM, Russell King - ARM Linux > wrote: >> On Wed, Jan 23, 2013 at 07:24:33AM -0600, Rob Clark wrote: >>> On Wed, Jan 23, 2013 at 3:42 AM, Jean-Francois Moine >>> wrote: >>> > Hi Rob, >>> > >>> > As I wanted to re-us

Re: [PATCH 1/4] drm/tilcdc: add TI LCD Controller DRM driver (v3)

2013-01-23 Thread Russell King - ARM Linux
On Wed, Jan 23, 2013 at 07:24:33AM -0600, Rob Clark wrote: > On Wed, Jan 23, 2013 at 3:42 AM, Jean-Francois Moine wrote: > > Hi Rob, > > > > As I wanted to re-use your nxp-tda998x driver for the Marvell Dove SoC, > > I had a look at your IT LCD driver. Comments below. > > Just fyi, you can re-use

Re: [patch] drm/nouveau/disp: sizeof() wrong pointer

2013-01-23 Thread Bernd Petrovitsch
On Mit, 2013-01-23 at 11:38 +0300, Dan Carpenter wrote: > On Tue, Jan 22, 2013 at 10:42:25AM +0100, Paul Menzel wrote: > > > > Did you find this by manual inspection or did you use some tool? > > I found this because it caused a problem in a parser I was working > on but Sparse warns about "warni

[PATCH] drm/exynos: add check for the device power status

2013-01-23 Thread Shirish S
The hdmi and mixer win_commit calls currently are not checking the status of IP before updating the respective registers, this patch adds this check. Signed-off-by: Shirish S --- drivers/gpu/drm/exynos/exynos_hdmi.c | 3 +++ drivers/gpu/drm/exynos/exynos_mixer.c | 3 +++ 2 files changed, 6 inse

Re: [PATCH 2/4] drm/i2c: nxp-tda998x (v2)

2013-01-23 Thread Jean-Francois Moine
On Tue, 22 Jan 2013 16:36:23 -0600 Rob Clark wrote: > Driver for the NXP TDA998X i2c hdmi encoder slave. > > v1: original > v2: fix npix/nline programming > > Signed-off-by: Rob Clark > --- > drivers/gpu/drm/i2c/Makefile | 3 + > drivers/gpu/drm/i2c/tda998x_drv.c | 908 > +

Re: [PATCH 1/4] drm/tilcdc: add TI LCD Controller DRM driver (v3)

2013-01-23 Thread Jean-Francois Moine
Hi Rob, As I wanted to re-use your nxp-tda998x driver for the Marvell Dove SoC, I had a look at your IT LCD driver. Comments below. On Tue, 22 Jan 2013 16:36:22 -0600 Rob Clark wrote: > A simple DRM/KMS driver for the TI LCD Controller found in various > smaller TI parts (AM33xx, OMAPL138, etc)

Re: [PATCH 2/4] drm/i2c: nxp-tda998x (v2)

2013-01-23 Thread Koen Kooi
Op 22 jan. 2013, om 23:36 heeft Rob Clark het volgende geschreven: > Driver for the NXP TDA998X i2c hdmi encoder slave. > > v1: original > v2: fix npix/nline programming > > Signed-off-by: Rob Clark Tested on a beaglebone-black: Tested-by: Koen Kooi _

Re: [PATCH 1/4] drm/tilcdc: add TI LCD Controller DRM driver (v3)

2013-01-23 Thread Koen Kooi
Op 22 jan. 2013, om 23:36 heeft Rob Clark het volgende geschreven: > A simple DRM/KMS driver for the TI LCD Controller found in various > smaller TI parts (AM33xx, OMAPL138, etc). This driver uses the > CMA helpers. Currently only the TFP410 DVI encoder is supported > (tested with beaglebone

Re: [PATCH] radeon_display: Use pointer return error codes

2013-01-23 Thread Alex Deucher
On Tue, Jan 22, 2013 at 9:27 PM, Su, Xuemin wrote: > From: xueminsu > Date: Tue, 22 Jan 2013 22:16:53 +0800 > Subject: [PATCH] radeon_display: Use pointer return error codes > > drm_mode_addfb() expects fb_create return error code > instead of NULL. > > Signed-off-by: xueminsu Thanks. Added to

Re: [PATCH 1/4] drm/tilcdc: add TI LCD Controller DRM driver (v3)

2013-01-23 Thread Rob Clark
On Wed, Jan 23, 2013 at 7:36 AM, Russell King - ARM Linux wrote: > On Wed, Jan 23, 2013 at 07:24:33AM -0600, Rob Clark wrote: >> On Wed, Jan 23, 2013 at 3:42 AM, Jean-Francois Moine wrote: >> > Hi Rob, >> > >> > As I wanted to re-use your nxp-tda998x driver for the Marvell Dove SoC, >> > I had a

[Bug 59492] piglit dlist-color-material test fail

2013-01-23 Thread bugzilla-dae...@freedesktop.org
... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20130123/e7b568bb/attachment-0001.html>

[Bug 59588] llvm rv790 etqw gpu lock since r600g/llvm: tgsi to llvm emits store.swizzle intrinsic for vs/fs output

2013-01-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=59588 --- Comment #9 from Alex Deucher --- Regarding the conversation on IRC, the vertex shader has to export at least one generic param (not counting special exports like position). So if the vertex shader doesn't export any params, you need a dummy

Re: [PATCH 2/4] drm/i2c: nxp-tda998x (v2)

2013-01-23 Thread Rob Clark
On Wed, Jan 23, 2013 at 3:42 AM, Jean-Francois Moine wrote: > On Tue, 22 Jan 2013 16:36:23 -0600 > Rob Clark wrote: > >> Driver for the NXP TDA998X i2c hdmi encoder slave. >> >> v1: original >> v2: fix npix/nline programming >> >> Signed-off-by: Rob Clark >> --- >> drivers/gpu/drm/i2c/Makefile

Re: [PATCH 1/4] drm/tilcdc: add TI LCD Controller DRM driver (v3)

2013-01-23 Thread Rob Clark
On Wed, Jan 23, 2013 at 3:42 AM, Jean-Francois Moine wrote: > Hi Rob, > > As I wanted to re-use your nxp-tda998x driver for the Marvell Dove SoC, > I had a look at your IT LCD driver. Comments below. Just fyi, you can re-use the nxp-tda998x part independently of tilcdc (just in case that wasn't c

  1   2   >