if old_crtc isn't same as encoder->crtc then it means that
user changed crtc id to another one so a plane to old_crtc
should be disabled so that current plane can be updated safely
and plane->crtc should be set to new crtc(encoder->crtc)
Signed-off-by: Inki Dae
Signed-off-by: Kyungmin Park
---
this patch adds buf_cnt variable in exynos_drm_fb structure and
that means a buffer count to drm framebuffer and also adds two
functions to get/set the buffer count from/to exynos_drm_fb structure.
if pixel format is not DRM_FORMAT_NV12MT then it gets a buffer count
to drm framebuffer refering to
the values set to registers will be updated into real registers
at vsync so dma operation could be malfunctioned when accessed
to memory after gem buffer was released. this patch makes sure
that hw overlay is disabled before the gem buffer is released.
Signed-off-by: Inki Dae
Signed-off-by:
the values set to registers will be updated into real registers
at vsync so dma operation could be malfunctioned when accessed
to memory after gem buffer was released. this patch makes sure
that hw overlay is disabled before the gem buffer is released.
Signed-off-by: Inki Dae
Signed-off-by:
this interface can be used to make sure that hardware overlay is disabled
to avoid that memory region is accessed by dma after gem buffer was released.
Signed-off-by: Inki Dae
Signed-off-by: Kyungmin Park
---
drivers/gpu/drm/exynos/exynos_drm_drv.h | 17 +
this patch fixes that when drm_crtc_helper_set_mode() is called,
mode data for hardware overlay and conntroller are updated two times.
for example, in case that drm_crtc_helper_set_mode() is called,
overlay_ops->commit() and manager_ops->commit() callbacks can be called
two times, first at
this patch separetes fimd_power_on into fimd_activate and fimd_clock and
fimd_activate function will call fimd_clock to control fimd power and
vsync interrupt.
Signed-off-by: Inki Dae
Signed-off-by: Kyungmin Park
---
drivers/gpu/drm/exynos/exynos_drm_fimd.c | 60
do not align in page unit at dumb creation. the align is done
by exynos_drm_gem_create() to be called commonly.
Signed-off-by: Inki Dae
Signed-off-by: Kyungmin Park
---
drivers/gpu/drm/exynos/exynos_drm_gem.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git
this patch separates exynos_drm_subdrv_probe function into sub driver's probe
call
and encoder/connector creation so that exynos drm core module can take exception
when some operation was failed properly.
Signed-off-by: Inki Dae
Signed-off-by: Kyungmin Park
---
when remove callback of exynos_drm_subdrv is called, it could need
device object for sub driver to control things specific to hw such as
runtime pm.
Signed-off-by: Inki Dae
Signed-off-by: Kyungmin Park
---
drivers/gpu/drm/exynos/exynos_drm_drv.h |2 +-
Hello all,
Changelog v2:
. fixed duplicated mode setting.
- this patch includes below three patches of v1 and fixes
the issue that drm_helper_connector_dpms() isn't called.
http://www.spinics.net/lists/dri-devel/msg26427.html
On Mon, Aug 20, 2012 at 10:24:01AM -0400, Alex Deucher wrote:
> > I just tested the rebased acpi_patches branch: ACPI part works fine, but
> > I see a big slow down during radeon driver initialization when the
> > screen goes black for a couple of seconds (with 3.5.0 + acpi patches the
> > screen
https://bugzilla.kernel.org/show_bug.cgi?id=31862
Alan changed:
What|Removed |Added
Summary|2.6.39.3 (and earlier): |3.4(and earlier): White
|White
https://bugzilla.kernel.org/show_bug.cgi?id=31862
Alan changed:
What|Removed |Added
Status|REOPENED|NEEDINFO
Component|Other
Hey,
Op 20-08-12 17:15, Jerome Glisse schreef:
> On Mon, Aug 20, 2012 at 9:42 AM, Maarten Lankhorst
> wrote:
>> How is this different from just calling with no_wait == false?
>> As far as I can tell, both paths end up with the same result..
>>
>> Signed-off-by: Maarten Lankhorst
> NAK this
On 08/20/2012 05:23 PM, Dave Airlie wrote:
> On Tue, Aug 21, 2012 at 8:45 AM, Randy Dunlap wrote:
>> On 08/19/2012 10:22 PM, Dave Airlie wrote:
>>
>>> On Mon, Aug 20, 2012 at 3:13 PM, Randy Dunlap
>>> wrote:
On 08/17/12 15:55, Dave Airlie wrote:
> On Sat, Aug 18, 2012 at 8:54 AM,
On Mon, Aug 20, 2012 at 11:24:44AM -0500, Seth Forshee wrote:
> I'm not sure how we support both of these cases without doing something
> more like what I originally proposed, i.e. registering the LVDS
> connector even if it doesn't look like a panel is attached. I still
> honestly favor that
I am not sure whether this is needed, but better be safe than sorry.
---
radeon/radeon_surface.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/radeon/radeon_surface.c b/radeon/radeon_surface.c
index 98f4aaf..4118a37 100644
--- a/radeon/radeon_surface.c
+++
On Mon, Aug 20, 2012 at 10:56:33AM -0500, Seth Forshee wrote:
> On Mon, Aug 20, 2012 at 04:36:40PM +0100, Matthew Garrett wrote:
> > Won't this break the multiple cards with independent outputs case?
>
> Yes, if they don't have a switcheroo handler. I only have experience
> with one such machine,
On Mon, Aug 20, 2012 at 10:31:04AM -0500, Seth Forshee wrote:
> + /*
> + * For secondary graphics devices shouldn't be initialized
> + * until the handler and primary graphics device have been
> + * registered with vga_switcheroo.
> + *
> + * FIXME: Is
The removed member is unneeded, an extra pass is done after all
buffers have been reserved. The behavior stays the same even without
the removed member, but this makes the code slightly more readable.
Depends on previous 2 execbuf-util patches.
Signed-off-by: Maarten Lankhorst
---
2012/8/20 InKi Dae :
> 2012/8/20 Joonyoung Shim :
>> On 08/20/2012 03:17 PM, InKi Dae wrote:
>>>
>>> 2012/8/20 Joonyoung Shim :
On 08/20/2012 11:29 AM, InKi Dae wrote:
>
> 2012/8/20 Joonyoung Shim :
>>
>> On 08/17/2012 06:50 PM, Inki Dae wrote:
>>>
>>> this patch
On Mon, 2012-08-20 at 15:38 +0200, Christian K?nig wrote:
> Reset the lockup timeout on ring (re-)initialisation.
>
> Otherwise we get error messages like this on gpu resets:
> [ 1559.949177] radeon :01:00.0: GPU lockup CP stall for more than
> 1482270msec
>
> Signed-off-by: Christian
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/mgag200/mgag200_drv.h |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/mgag200/mgag200_drv.h
b/drivers/gpu/drm/mgag200/mgag200_drv.h
index 6f13b35..d22cbbf 100644
--- a/drivers/gpu/drm/mgag200/mgag200_drv.h
+++
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/cirrus/cirrus_drv.h |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/cirrus/cirrus_drv.h
b/drivers/gpu/drm/cirrus/cirrus_drv.h
index 64ea597..5d04564 100644
--- a/drivers/gpu/drm/cirrus/cirrus_drv.h
+++
Signed-off-by: Maarten Lankhorst
---
drivers/gpu/drm/ast/ast_drv.h |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/ast/ast_drv.h b/drivers/gpu/drm/ast/ast_drv.h
index d4af9ed..20a437f 100644
--- a/drivers/gpu/drm/ast/ast_drv.h
+++ b/drivers/gpu/drm/ast/ast_drv.h
@@ -94,7
https://bugzilla.kernel.org/show_bug.cgi?id=33912
Alan changed:
What|Removed |Added
Status|NEW |RESOLVED
CC|
2012/8/20 Joonyoung Shim :
> On 08/20/2012 03:17 PM, InKi Dae wrote:
>>
>> 2012/8/20 Joonyoung Shim :
>>>
>>> On 08/20/2012 11:29 AM, InKi Dae wrote:
2012/8/20 Joonyoung Shim :
>
> On 08/17/2012 06:50 PM, Inki Dae wrote:
>>
>> this patch changes ctx variable name in
https://bugzilla.kernel.org/show_bug.cgi?id=33512
Alan changed:
What|Removed |Added
Status|NEW |RESOLVED
CC|
On 08/19/2012 10:22 PM, Dave Airlie wrote:
> On Mon, Aug 20, 2012 at 3:13 PM, Randy Dunlap wrote:
>> On 08/17/12 15:55, Dave Airlie wrote:
>>
>>> On Sat, Aug 18, 2012 at 8:54 AM, Dave Airlie wrote:
On Sat, Aug 18, 2012 at 8:28 AM, Randy Dunlap
wrote:
> On 08/17/2012 03:25 PM,
From: Alan Cox
If you do a page flip with no flags set then event is NULL. If event is
NULL then the vmw_gfx driver likes to go digging into NULL and extracts
NULL->base.file_priv.
On a modern kernel with NULL mapping protection it's just another oops,
without it there are
How is this different from just calling with no_wait == false?
As far as I can tell, both paths end up with the same result..
Signed-off-by: Maarten Lankhorst
diff --git a/drivers/gpu/drm/ttm/ttm_execbuf_util.c
b/drivers/gpu/drm/ttm/ttm_execbuf_util.c
index 3f48a46..4e7b596 100644
---
Reset the lockup timeout on ring (re-)initialisation.
Otherwise we get error messages like this on gpu resets:
[ 1559.949177] radeon :01:00.0: GPU lockup CP stall for more than
1482270msec
Signed-off-by: Christian K?nig
cc: stable at kernel.org
---
drivers/gpu/drm/radeon/radeon_ring.c |
On 08/20/2012 03:17 PM, InKi Dae wrote:
> 2012/8/20 Joonyoung Shim :
>> On 08/20/2012 11:29 AM, InKi Dae wrote:
>>> 2012/8/20 Joonyoung Shim :
On 08/17/2012 06:50 PM, Inki Dae wrote:
> this patch changes ctx variable name in exynos_drm_hdmi_context
> structure to client because the
Nobody uses it, so might as well simplify the code some.
Signed-off-by: Maarten Lankhorst
---
diff --git a/drivers/gpu/drm/ttm/ttm_bo.c b/drivers/gpu/drm/ttm/ttm_bo.c
index 36f4b28..ddfe393 100644
--- a/drivers/gpu/drm/ttm/ttm_bo.c
+++ b/drivers/gpu/drm/ttm/ttm_bo.c
@@ -141,7 +141,6 @@ static
https://bugzilla.kernel.org/show_bug.cgi?id=32162
Alex Deucher changed:
What|Removed |Added
CC||alexdeucher at gmail.com
--- Comment
https://bugzilla.kernel.org/show_bug.cgi?id=32162
Alan changed:
What|Removed |Added
Component|Video(AGP) |Video(DRI - non Intel)
On Mon, Aug 20, 2012 at 3:13 PM, Randy Dunlap wrote:
> On 08/17/12 15:55, Dave Airlie wrote:
>
>> On Sat, Aug 18, 2012 at 8:54 AM, Dave Airlie wrote:
>>> On Sat, Aug 18, 2012 at 8:28 AM, Randy Dunlap
>>> wrote:
On 08/17/2012 03:25 PM, Justin M. Forbes wrote:
> for , we have
https://bugzilla.kernel.org/show_bug.cgi?id=32072
Alan changed:
What|Removed |Added
Status|NEW |RESOLVED
CC|
https://bugzilla.kernel.org/show_bug.cgi?id=31972
Alan changed:
What|Removed |Added
Status|NEW |RESOLVED
CC|
2012/8/20 Joonyoung Shim :
> On 08/20/2012 11:29 AM, InKi Dae wrote:
>>
>> 2012/8/20 Joonyoung Shim :
>>>
>>> On 08/17/2012 06:50 PM, Inki Dae wrote:
this patch changes ctx variable name in exynos_drm_hdmi_context
structure to client because the use of ctx variable makes it
https://bugzilla.kernel.org/show_bug.cgi?id=31892
Alan changed:
What|Removed |Added
Status|NEW |RESOLVED
CC|
There is a more recent APU stepping with a new PCI ID
shipping in the same board by Fujitsu which needs the
same quirk to correctly mark the back plane connectors.
Signed-off-by: Tvrtko Ursulin
---
radeon_atombios.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
https://bugzilla.kernel.org/show_bug.cgi?id=31712
Alan changed:
What|Removed |Added
Status|NEW |RESOLVED
CC|
https://bugzilla.kernel.org/show_bug.cgi?id=31682
Alan changed:
What|Removed |Added
Status|NEW |RESOLVED
CC|
https://bugzilla.kernel.org/show_bug.cgi?id=31502
Alex Deucher changed:
What|Removed |Added
CC||alexdeucher at gmail.com
--- Comment
https://bugzilla.kernel.org/show_bug.cgi?id=31502
Alan changed:
What|Removed |Added
CC||alan at lxorguk.ukuu.org.uk
https://bugs.freedesktop.org/show_bug.cgi?id=53111
--- Comment #16 from Alexandre Demers
2012-08-20 15:05:03 UTC ---
(In reply to comment #15)
> (In reply to comment #10)
> > Well, it seems running it through qapitrace doesn't lock.
>
> The apitrace looks incomplete: it doesn't contain any
https://bugs.freedesktop.org/show_bug.cgi?id=53111
--- Comment #15 from Michel D?nzer 2012-08-20 14:59:34
UTC ---
(In reply to comment #10)
> Well, it seems running it through qapitrace doesn't lock.
The apitrace looks incomplete: it doesn't contain any actual rendering
operations.
--
On Mon, Aug 20, 2012 at 4:08 AM, Christian K?nig
wrote:
> Second and hopefully last round for this patchset.
>
> v2: Fix suspend/resume, and incorporate Jeromes comments.
Looks good to me. Can you put up a git branch somewhere?
Reviewed-by: Alex Deucher
>
> Cheers,
> Christian.
>
>
https://bugzilla.kernel.org/show_bug.cgi?id=30922
Alan changed:
What|Removed |Added
Status|NEW |RESOLVED
CC|
https://bugzilla.kernel.org/show_bug.cgi?id=30812
Alan changed:
What|Removed |Added
Status|NEW |RESOLVED
CC|
https://bugzilla.kernel.org/show_bug.cgi?id=46241
Alex Deucher changed:
What|Removed |Added
CC||alexdeucher at gmail.com
--- Comment
2012/8/20 Joonyoung Shim :
> On 08/20/2012 02:15 PM, InKi Dae wrote:
>>
>> sorry, again.
>>
>> 2012/8/20 InKi Dae :
>>>
>>> 2012/8/20 Joonyoung Shim :
On 08/20/2012 11:23 AM, InKi Dae wrote:
>
> 2012/8/20 Joonyoung Shim :
>>
>> On 08/17/2012 06:50 PM, Inki Dae wrote:
2012/8/20 Joonyoung Shim :
> On 08/20/2012 01:31 PM, InKi Dae wrote:
>>
>> 2012/8/20 Joonyoung Shim :
>>>
>>> On 08/20/2012 10:52 AM, InKi Dae wrote:
2012/8/20 Joonyoung Shim :
>
> On 08/17/2012 06:50 PM, Inki Dae wrote:
>>
>> this patch separates sub driver's probe call
https://bugzilla.kernel.org/show_bug.cgi?id=46241
Alan changed:
What|Removed |Added
CC||alan at lxorguk.ukuu.org.uk
Kernel
.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20120820/02d5305e/attachment.pgp>
On Mon, Aug 20, 2012 at 7:35 AM, Paul Menzel
wrote:
> Dear Marek,
>
>
> thank you for all your work on MSAA.
>
>
> Am Sonntag, den 19.08.2012, 21:23 +0200 schrieb Marek Ol??k:
>
> Unfortunately you do not provide any commit message. What is the problem
> and what are the symptoms? When was it
On Thu, Aug 16, 2012 at 11:51:10AM -0400, Alex Deucher wrote:
> I've gathered up the various outstanding radeon patches and put them
> up in a tree for testing:
> http://cgit.freedesktop.org/~agd5f/linux/log/?h=drm-next-3.7-wip
> Some of these may end up in -fixes, but most of them are -next
>
On 08/20/2012 02:15 PM, InKi Dae wrote:
> sorry, again.
>
> 2012/8/20 InKi Dae :
>> 2012/8/20 Joonyoung Shim :
>>> On 08/20/2012 11:23 AM, InKi Dae wrote:
2012/8/20 Joonyoung Shim :
> On 08/17/2012 06:50 PM, Inki Dae wrote:
>> this patch adds buf_cnt variable in exynos_drm_fb
sorry, again.
2012/8/20 InKi Dae :
> 2012/8/20 Joonyoung Shim :
>> On 08/20/2012 11:23 AM, InKi Dae wrote:
>>>
>>> 2012/8/20 Joonyoung Shim :
On 08/17/2012 06:50 PM, Inki Dae wrote:
>
> this patch adds buf_cnt variable in exynos_drm_fb structure and
> that means a buffer
2012/8/20 Joonyoung Shim :
> On 08/20/2012 11:23 AM, InKi Dae wrote:
>>
>> 2012/8/20 Joonyoung Shim :
>>>
>>> On 08/17/2012 06:50 PM, Inki Dae wrote:
this patch adds buf_cnt variable in exynos_drm_fb structure and
that means a buffer count to drm framebuffer and also adds two
On Mon, Aug 20, 2012 at 1:55 PM, Maarten Lankhorst
wrote:
> Hey,
>
> Op 20-08-12 17:15, Jerome Glisse schreef:
>> On Mon, Aug 20, 2012 at 9:42 AM, Maarten Lankhorst
>> wrote:
>>> How is this different from just calling with no_wait == false?
>>> As far as I can tell, both paths end up with the
https://bugs.freedesktop.org/show_bug.cgi?id=42373
--- Comment #32 from Alex Deucher 2012-08-20 14:03:44 UTC
---
A similar patch is already upstream:
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=commitdiff;h=81ee8fb6b52ec69eeed37fe7943446af1dccecc5
--
Configure bugmail:
https://bugs.freedesktop.org/show_bug.cgi?id=53512
almos changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
On 08/20/2012 11:29 AM, InKi Dae wrote:
> 2012/8/20 Joonyoung Shim :
>> On 08/17/2012 06:50 PM, Inki Dae wrote:
>>> this patch changes ctx variable name in exynos_drm_hdmi_context
>>> structure to client because the use of ctx variable makes it confused.
>>
>> I don't prefer "client" name. This is
On 08/20/2012 11:23 AM, InKi Dae wrote:
> 2012/8/20 Joonyoung Shim :
>> On 08/17/2012 06:50 PM, Inki Dae wrote:
>>> this patch adds buf_cnt variable in exynos_drm_fb structure and
>>> that means a buffer count to drm framebuffer and also adds two
>>> functions to get/set the buffer count from/to
On 08/20/2012 01:31 PM, InKi Dae wrote:
> 2012/8/20 Joonyoung Shim :
>> On 08/20/2012 10:52 AM, InKi Dae wrote:
>>> 2012/8/20 Joonyoung Shim :
On 08/17/2012 06:50 PM, Inki Dae wrote:
> this patch separates sub driver's probe call and encoder/connector
> creation
> so that exynos
2012/8/20 Joonyoung Shim :
> On 08/20/2012 10:52 AM, InKi Dae wrote:
>>
>> 2012/8/20 Joonyoung Shim :
>>>
>>> On 08/17/2012 06:50 PM, Inki Dae wrote:
this patch separates sub driver's probe call and encoder/connector
creation
so that exynos drm core module can take exception
2012/8/20 Joonyoung Shim :
> On 08/17/2012 06:50 PM, Inki Dae wrote:
>>
>> crtc and encoder's dpms callback will be called before connector's dpms
>> is called so drm_helper_connector_dpms doesn't need to be called.
>
>
> I can't understand this description. I know crtc and encoder dpms are called
On Mon, Aug 20, 2012 at 11:23:16AM -0700, Guenter Roeck wrote:
> Fix config warning:
>
> warning: ( ... && DRM_USB) selects USB which has unmet direct dependencies
> (USB_SUPPORT && USB_ARCH_HAS_HCD)
>
> and build error:
> ERROR: "usb_speed_string" [drivers/usb/core/usbcore.ko] undefined!
>
>
2012/8/20 Joonyoung Shim :
> On 08/17/2012 06:50 PM, Inki Dae wrote:
>>
>> this patch changes ctx variable name in exynos_drm_hdmi_context
>> structure to client because the use of ctx variable makes it confused.
>
>
> I don't prefer "client" name. This is not client and server relationship.
>
On Mon, Aug 20, 2012 at 04:57:41PM +0100, Matthew Garrett wrote:
> On Mon, Aug 20, 2012 at 10:56:33AM -0500, Seth Forshee wrote:
> > On Mon, Aug 20, 2012 at 04:36:40PM +0100, Matthew Garrett wrote:
> > > Won't this break the multiple cards with independent outputs case?
> >
> > Yes, if they don't
2012/8/20 Joonyoung Shim :
> On 08/17/2012 06:50 PM, Inki Dae wrote:
>>
>> this patch adds buf_cnt variable in exynos_drm_fb structure and
>> that means a buffer count to drm framebuffer and also adds two
>> functions to get/set the buffer count from/to exynos_drm_fb structure.
>> if pixel format
Fix config warning:
warning: ( ... && DRM_USB) selects USB which has unmet direct dependencies
(USB_SUPPORT && USB_ARCH_HAS_HCD)
and build error:
ERROR: "usb_speed_string" [drivers/usb/core/usbcore.ko] undefined!
by adding the missing dependency on USB_ARCH_HAS_HCD to DRM_UDL and DRM_USB.
This
From: Alex Deucher
There are systems that use ATRM, but not ATPX.
Fixes:
https://bugs.freedesktop.org/show_bug.cgi?id=41265
V2: fix #ifdefs as per Greg's comments
V3: fix it harder
Signed-off-by: Alex Deucher
Cc: stable at vger.kernel.org
---
From: Alex Deucher
Allows us to verify the table size.
Signed-off-by: Alex Deucher
Cc: stable at vger.kernel.org
---
drivers/gpu/drm/radeon/radeon_bios.c |6 ++
1 files changed, 2 insertions(+), 4 deletions(-)
diff --git
From: Alex Deucher
We need it in the radeon drm module to fetch
and verify the vbios image on UEFI systems.
Signed-off-by: Alex Deucher
Cc: stable at vger.kernel.org
---
drivers/acpi/acpica/tbxface.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff
From: David Lamparter
This is required for pure UEFI systems. The vbios is stored
in ACPI rather than at the legacy vga location.
Fixes:
https://bugs.freedesktop.org/show_bug.cgi?id=26891
V2: fix #ifdefs as per Greg's comments
V3: fix it harder
Signed-off-by: Alex Deucher
On Mon, Aug 20, 2012 at 10:36 AM, Maarten Lankhorst
wrote:
> The removed member is unneeded, an extra pass is done after all
> buffers have been reserved. The behavior stays the same even without
> the removed member, but this makes the code slightly more readable.
>
> Depends on previous 2
On Mon, Aug 20, 2012 at 9:32 AM, Maarten Lankhorst
wrote:
> Nobody uses it, so might as well simplify the code some.
>
> Signed-off-by: Maarten Lankhorst
Reviewed-by: Jerome Glisse
> ---
> diff --git a/drivers/gpu/drm/ttm/ttm_bo.c b/drivers/gpu/drm/ttm/ttm_bo.c
> index 36f4b28..ddfe393 100644
On Mon, Aug 20, 2012 at 9:42 AM, Maarten Lankhorst
wrote:
> How is this different from just calling with no_wait == false?
> As far as I can tell, both paths end up with the same result..
>
> Signed-off-by: Maarten Lankhorst
NAK this seriously modify the behavior. The ttm_eu_del_from_lru_locked
On 08/17/2012 06:50 PM, Inki Dae wrote:
> the values set to registers will be updated into real registers
> at vsync so dma operation could be malfunctioned when accessed
> to memory after gem buffer was released. this patch makes sure
> that hw overlay is disabled before the gem buffer is
2012/8/20 Joonyoung Shim :
> On 08/17/2012 06:50 PM, Inki Dae wrote:
>>
>> encoder's mode_set callback isn't specific to hardware so it doesn't
>> need to call exynos_drm_encoder_dpms().
>
>
> Then, where is exynos_drm_encoder_dpms() called?
>
with this patch series, exynos_drm_encoder_dpms()
On 08/20/2012 10:52 AM, InKi Dae wrote:
> 2012/8/20 Joonyoung Shim :
>> On 08/17/2012 06:50 PM, Inki Dae wrote:
>>> this patch separates sub driver's probe call and encoder/connector
>>> creation
>>> so that exynos drm core module can take exception when some operation was
>>> failed properly.
>>
On Mon, Aug 20, 2012 at 04:36:40PM +0100, Matthew Garrett wrote:
> On Mon, Aug 20, 2012 at 10:31:04AM -0500, Seth Forshee wrote:
> > + /*
> > +* For secondary graphics devices shouldn't be initialized
> > +* until the handler and primary graphics device have been
> > +* registered
2012/8/20 Joonyoung Shim :
> On 08/17/2012 06:50 PM, Inki Dae wrote:
>>
>> this patch separates sub driver's probe call and encoder/connector
>> creation
>> so that exynos drm core module can take exception when some operation was
>> failed properly.
>
>
> Which exceptions? I don't know this patch
On 08/17/2012 06:37 PM, Leela Krishna Amudala wrote:
> Hello,
>
> On Fri, Aug 17, 2012 at 6:55 AM, Joonyoung Shim wrote:
>> Hi,
>>
>> 2012/8/16 Leela Krishna Amudala :
>>> Add device tree based discovery support for DRM-FIMD driver.
>>>
>>> Signed-off-by: Leela Krishna Amudala
>>> ---
>>>
On Sun, Aug 19, 2012 at 5:58 PM, Dave Airlie wrote:
> On Mon, Aug 20, 2012 at 7:51 AM, Marek Ol??k wrote:
>> Please can someone tell me if this patch will end up in kernel 3.6 or 3.7.
>>
>> If it is merged in Linus's tree in the coming days, I'll try to
>> complete R600-R700 MSAA support (and
Deferring initiailzation of the secondary GPU until switcheroo is ready
will allow successfully reading the EDID in systems which support muxing
the DDC seperately from the display.
Signed-off-by: Seth Forshee
---
drivers/gpu/drm/drm_drv.c |3 +
drivers/gpu/drm/drm_pci.c | 141
Some dual graphics machines support muxing the DDC separately from the
display, so make use of this functionality when reading the EDID on the
inactive GPU. Also serialize drm_get_edid() with a mutex to avoid races
on the DDC mux state.
Signed-off-by: Seth Forshee
---
drivers/gpu/drm/drm_edid.c
The gmux allows muxing the DDC independently from the display, so
support this functionality. This will allow reading the EDID for the
inactive GPU, fixing issues with machines that either don't have a VBT
or have invalid mode data in the VBT.
Signed-off-by: Seth Forshee
---
DRM needs to be notified of client and handler registration in order to
defer initialization of the secondary GPU until the EDID can be read
from the LVDS panel. To support this add a notifier call chain to
vga_switcheroo for subscribing to switcheroo events. Events are
generated for registration
Add vga_switcheroo_get_active_client() to allow drivers to get the
active video client. This will be used by drivers wishing to temporarily
mux only the DDC to the inactive client.
Signed-off-by: Seth Forshee
---
drivers/gpu/vga/vga_switcheroo.c | 14 ++
During graphics driver initialization its useful to be able to mux only
the DDC to the inactive client in order to read the EDID. Add a
switch_ddc callback to allow capable handlers to provide this
functionality, and add vga_switcheroo_switch_ddc() to allow DRM to mux
only the DDC.
Signed-off-by:
On Fri, Aug 10, 2012 at 05:19:48PM -0500, Seth Forshee wrote:
> First, I don't have a solution for the ordering of initialization. It
> just happens to work out for me right now.
Okay, I've got a proof-of-concept implementation of delaying secondary
GPU initialization until the i2c can be muxed
On 08/17/2012 06:50 PM, Inki Dae wrote:
> this patch changes ctx variable name in exynos_drm_hdmi_context
> structure to client because the use of ctx variable makes it confused.
I don't prefer "client" name. This is not client and server relationship.
>
> Signed-off-by: Inki Dae
>
On 08/17/2012 06:50 PM, Inki Dae wrote:
> if old_crtc isn't same as encoder->crtc then it means that
> user changed crtc id to another one so a plane to old_crtc
> should be disabled so that current plane can be updated safely
> and plane->crtc should be set to new crtc(encoder->crtc)
>
>
On Mon, Aug 20, 2012 at 8:30 AM, Luca Tettamanti wrote:
> On Thu, Aug 16, 2012 at 11:51:10AM -0400, Alex Deucher wrote:
>> I've gathered up the various outstanding radeon patches and put them
>> up in a tree for testing:
>> http://cgit.freedesktop.org/~agd5f/linux/log/?h=drm-next-3.7-wip
>> Some
On Fri, 17 Aug 2012, Damien Lespiau wrote:
> From: Damien Lespiau
>
> Signed-off-by: Damien Lespiau
Reviewed-by: Jani Nikula
> ---
> drivers/gpu/drm/drm_modes.c |3 ---
> include/drm/drm_crtc.h |2 --
> 2 files changed, 0 insertions(+), 5 deletions(-)
>
> diff --git
1 - 100 of 211 matches
Mail list logo