2012/8/20 Joonyoung Shim jy0922.s...@samsung.com:
On 08/20/2012 11:29 AM, InKi Dae wrote:
2012/8/20 Joonyoung Shim jy0922.s...@samsung.com:
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
On 08/20/2012 03:17 PM, InKi Dae wrote:
2012/8/20 Joonyoung Shim jy0922.s...@samsung.com:
On 08/20/2012 11:29 AM, InKi Dae wrote:
2012/8/20 Joonyoung Shim jy0922.s...@samsung.com:
On 08/17/2012 06:50 PM, Inki Dae wrote:
this patch changes ctx variable name in exynos_drm_hdmi_context
2012/8/20 Joonyoung Shim jy0922.s...@samsung.com:
On 08/20/2012 03:17 PM, InKi Dae wrote:
2012/8/20 Joonyoung Shim jy0922.s...@samsung.com:
On 08/20/2012 11:29 AM, InKi Dae wrote:
2012/8/20 Joonyoung Shim jy0922.s...@samsung.com:
On 08/17/2012 06:50 PM, Inki Dae wrote:
this patch
On Fri, 17 Aug 2012, Damien Lespiau damien.lesp...@gmail.com wrote:
From: Damien Lespiau damien.lesp...@intel.com
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
Reviewed-by: Jani Nikula jani.nik...@intel.com
---
drivers/gpu/drm/drm_modes.c |3 ---
include/drm/drm_crtc.h |
2012/8/20 InKi Dae inki@samsung.com:
2012/8/20 Joonyoung Shim jy0922.s...@samsung.com:
On 08/20/2012 03:17 PM, InKi Dae wrote:
2012/8/20 Joonyoung Shim jy0922.s...@samsung.com:
On 08/20/2012 11:29 AM, InKi Dae wrote:
2012/8/20 Joonyoung Shim jy0922.s...@samsung.com:
On 08/17/2012
https://bugs.freedesktop.org/show_bug.cgi?id=41265
--- Comment #51 from gy...@gmx.de 2012-08-20 07:32:20 UTC ---
With this xorg.conf i can use two external monitor connected to the
dockingstation.
Additionally the internal screen of the notebook is in textmode.
--
$cat /etc/X11/xorg.conf
Section
https://bugzilla.kernel.org/show_bug.cgi?id=46241
Summary: Resume to black screen on AMD A8-3500m/Radeon 6620G
Product: Drivers
Version: 2.5
Kernel Version: Linux version 3.6.0-030600rc1-generic (apw@gomeisa)
(gcc version 4.6.3
https://bugzilla.kernel.org/show_bug.cgi?id=46241
--- Comment #2 from Geoffrey Teale geoffrey.te...@canonical.com 2012-08-20
07:43:23 ---
Created an attachment (id=77951)
-- (https://bugzilla.kernel.org/attachment.cgi?id=77951)
modules
--
Configure bugmail:
https://bugzilla.kernel.org/show_bug.cgi?id=46241
--- Comment #3 from Geoffrey Teale geoffrey.te...@canonical.com 2012-08-20
07:44:08 ---
Created an attachment (id=77961)
-- (https://bugzilla.kernel.org/attachment.cgi?id=77961)
ver_linux
--
Configure bugmail:
https://bugzilla.kernel.org/show_bug.cgi?id=46241
--- Comment #4 from Geoffrey Teale geoffrey.te...@canonical.com 2012-08-20
07:44:55 ---
Created an attachment (id=77971)
-- (https://bugzilla.kernel.org/attachment.cgi?id=77971)
io
--
Configure bugmail:
https://bugzilla.kernel.org/show_bug.cgi?id=46241
--- Comment #5 from Geoffrey Teale geoffrey.te...@canonical.com 2012-08-20
07:55:10 ---
Sorry, should look like this:
[1.] Lenovo E525 screen black on resume
[2.] My setup is a Lenovo Thinkpad E525, AMD A8-3500m/Radeon 6620G Llano
series
From: Jerome Glisse jgli...@redhat.com
Virtual address need to be fenced to know when we can safely remove it.
This patch also properly clear the pagetable. Previously it was
serouisly broken.
Kernel 3.5/3.4 need a similar patch but adapted for difference in mutex locking.
v2: For to update
Second and hopefully last round for this patchset.
v2: Fix suspend/resume, and incorporate Jeromes comments.
Cheers,
Christian.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
Store a reference to the VM into the IB structure, that
makes calculating the IBs address a bit less complicated.
Signed-off-by: Christian König deathsim...@vodafone.de
Reviewed-by: Jerome Glisse jgli...@redhat.com
---
drivers/gpu/drm/radeon/ni.c |5 +++--
So it looks more like the rest of the driver.
Signed-off-by: Christian König deathsim...@vodafone.de
Reviewed-by: Jerome Glisse jgli...@redhat.com
---
drivers/gpu/drm/radeon/radeon.h | 35 +---
drivers/gpu/drm/radeon/radeon_asic.c | 50
It actually isn't very useful.
Signed-off-by: Christian König deathsim...@vodafone.de
Reviewed-by: Jerome Glisse jgli...@redhat.com
---
drivers/gpu/drm/radeon/ni.c | 11 ---
drivers/gpu/drm/radeon/radeon.h |2 --
drivers/gpu/drm/radeon/radeon_asic.c |3 ---
Signed-off-by: Christian König deathsim...@vodafone.de
Reviewed-by: Jerome Glisse jgli...@redhat.com
---
drivers/gpu/drm/radeon/radeon_cs.c | 20 ++--
1 file changed, 14 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/radeon/radeon_cs.c
Move flushing the VMs as function into the rings.
First step to make VM operations async.
Signed-off-by: Christian König deathsim...@vodafone.de
Reviewed-by: Jerome Glisse jgli...@redhat.com
---
drivers/gpu/drm/radeon/ni.c | 31 ---
Removing the need to wait for anything.
Still not ideal, since we need to free pt on va remove.
Signed-off-by: Christian König deathsim...@vodafone.de
Reviewed-by: Jerome Glisse jgli...@redhat.com
---
drivers/gpu/drm/radeon/radeon.h |4 +-
drivers/gpu/drm/radeon/radeon_cs.c | 28
Move binding onto the ring, simplifying handling a bit.
Signed-off-by: Christian König deathsim...@vodafone.de
Reviewed-by: Jerome Glisse jgli...@redhat.com
---
drivers/gpu/drm/radeon/ni.c| 20 ++---
drivers/gpu/drm/radeon/radeon.h| 30 +++-
Makes it easier to move it into the rings.
Signed-off-by: Christian König deathsim...@vodafone.de
Reviewed-by: Jerome Glisse jgli...@redhat.com
---
drivers/gpu/drm/radeon/ni.c | 22 +++---
drivers/gpu/drm/radeon/radeon.h | 12 ++--
From: Jerome Glisse jgli...@redhat.com
Make sure that the ib bo is bound and is page table is up to date
in the virtual address space.
Signed-off-by: Jerome Glisse jgli...@redhat.com
Reviewed-by: Christian König christian.koe...@amd.com
---
drivers/gpu/drm/radeon/radeon_cs.c |5 +
1
Currently doing the update with the CP.
v2: Rebased on Jeromes bugfix. Make validity comparison
more human readable.
Signed-off-by: Christian König deathsim...@vodafone.de
---
drivers/gpu/drm/radeon/ni.c | 20 ++
drivers/gpu/drm/radeon/nid.h |1 +
On Sat, 2012-08-18 at 03:16 +0200, Laurent Pinchart wrote:
Hi Tomi,
mipi-dbi-bus might not belong to include/video/panel/ though, as it can be
used for non-panel devices (at least in theory). The future mipi-dsi-bus
certainly will.
They are both display busses. So while they could be used
On Mon, Aug 20, 2012 at 7:35 AM, Paul Menzel
paulepan...@users.sourceforge.net 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
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 inki@samsung.com
Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com
---
drivers/gpu/drm/exynos/exynos_drm_drv.h |
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
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 inki@samsung.com
Signed-off-by: Kyungmin Park
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 inki@samsung.com
Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com
---
drivers/gpu/drm/exynos/exynos_drm_gem.c |2 +-
1 files changed, 1
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 inki@samsung.com
Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com
---
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 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
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 inki@samsung.com
Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com
---
drivers/gpu/drm/exynos/exynos_drm_drv.h
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 inki@samsung.com
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 inki@samsung.com
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 inki@samsung.com
Nobody uses it, so might as well simplify the code some.
Signed-off-by: Maarten Lankhorst maarten.lankho...@canonical.com
---
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
+++
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 deathsim...@vodafone.de
cc: sta...@kernel.org
---
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 maarten.lankho...@canonical.com
diff --git a/drivers/gpu/drm/ttm/ttm_execbuf_util.c
b/drivers/gpu/drm/ttm/ttm_execbuf_util.c
index
https://bugs.freedesktop.org/show_bug.cgi?id=53512
almos aaalmo...@gmail.com changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
https://bugs.freedesktop.org/show_bug.cgi?id=42373
--- Comment #32 from Alex Deucher ag...@yahoo.com 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
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 König
On Mon, Aug 20, 2012 at 8:30 AM, Luca Tettamanti kronos...@gmail.com 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:
From: Alan Cox a...@linux.intel.com
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
Signed-off-by: Maarten Lankhorst maarten.lankho...@canonical.com
---
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
---
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
Signed-off-by: Maarten Lankhorst maarten.lankho...@canonical.com
---
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
+++
Signed-off-by: Maarten Lankhorst maarten.lankho...@canonical.com
---
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
---
On Sun, Aug 19, 2012 at 5:58 PM, Dave Airlie airl...@gmail.com wrote:
On Mon, Aug 20, 2012 at 7:51 AM, Marek Olšák mar...@gmail.com 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
https://bugzilla.kernel.org/show_bug.cgi?id=46241
Alan a...@lxorguk.ukuu.org.uk changed:
What|Removed |Added
CC||a...@lxorguk.ukuu.org.uk
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 tvrtko.ursu...@onelan.co.uk
---
radeon_atombios.c |2 +-
1 file changed, 1 insertion(+), 1
https://bugzilla.kernel.org/show_bug.cgi?id=46241
Alex Deucher alexdeuc...@gmail.com changed:
What|Removed |Added
CC|
https://bugzilla.kernel.org/show_bug.cgi?id=30812
Alan a...@lxorguk.ukuu.org.uk changed:
What|Removed |Added
Status|NEW |RESOLVED
https://bugzilla.kernel.org/show_bug.cgi?id=30922
Alan a...@lxorguk.ukuu.org.uk changed:
What|Removed |Added
Status|NEW |RESOLVED
https://bugs.freedesktop.org/show_bug.cgi?id=53111
--- Comment #15 from Michel Dänzer mic...@daenzer.net 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
- Original Message -
From: Alan Cox a...@linux.intel.com
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
https://bugs.freedesktop.org/show_bug.cgi?id=53111
--- Comment #16 from Alexandre Demers alexandre.f.dem...@gmail.com 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
https://bugzilla.kernel.org/show_bug.cgi?id=31502
Alan a...@lxorguk.ukuu.org.uk changed:
What|Removed |Added
CC||a...@lxorguk.ukuu.org.uk
https://bugzilla.kernel.org/show_bug.cgi?id=31502
Alex Deucher alexdeuc...@gmail.com changed:
What|Removed |Added
CC|
https://bugzilla.kernel.org/show_bug.cgi?id=31682
Alan a...@lxorguk.ukuu.org.uk changed:
What|Removed |Added
Status|NEW |RESOLVED
On Mon, Aug 20, 2012 at 9:32 AM, Maarten Lankhorst
maarten.lankho...@canonical.com wrote:
Nobody uses it, so might as well simplify the code some.
Signed-off-by: Maarten Lankhorst maarten.lankho...@canonical.com
Reviewed-by: Jerome Glisse jgli...@redhat.com
---
diff --git
https://bugzilla.kernel.org/show_bug.cgi?id=31972
Alan a...@lxorguk.ukuu.org.uk changed:
What|Removed |Added
Status|NEW |RESOLVED
From: David Lamparter equi...@diac24.net
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
From: Alex Deucher alexander.deuc...@amd.com
We need it in the radeon drm module to fetch
and verify the vbios image on UEFI systems.
Signed-off-by: Alex Deucher alexander.deuc...@amd.com
Cc: sta...@vger.kernel.org
---
drivers/acpi/acpica/tbxface.c |1 +
1 files changed, 1 insertions(+), 0
From: Alex Deucher alexander.deuc...@amd.com
Allows us to verify the table size.
Signed-off-by: Alex Deucher alexander.deuc...@amd.com
Cc: sta...@vger.kernel.org
---
drivers/gpu/drm/radeon/radeon_bios.c |6 ++
1 files changed, 2 insertions(+), 4 deletions(-)
diff --git
From: Alex Deucher alexander.deuc...@amd.com
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 alexander.deuc...@amd.com
Cc: sta...@vger.kernel.org
---
https://bugzilla.kernel.org/show_bug.cgi?id=32072
Alan a...@lxorguk.ukuu.org.uk changed:
What|Removed |Added
Status|NEW |RESOLVED
https://bugzilla.kernel.org/show_bug.cgi?id=32162
Alex Deucher alexdeuc...@gmail.com changed:
What|Removed |Added
CC|
https://bugzilla.kernel.org/show_bug.cgi?id=33512
Alan a...@lxorguk.ukuu.org.uk changed:
What|Removed |Added
Status|NEW |RESOLVED
https://bugzilla.kernel.org/show_bug.cgi?id=33912
Alan a...@lxorguk.ukuu.org.uk changed:
What|Removed |Added
Status|NEW |RESOLVED
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, which
Hey,
Op 20-08-12 17:15, Jerome Glisse schreef:
On Mon, Aug 20, 2012 at 9:42 AM, Maarten Lankhorst
maarten.lankho...@canonical.com 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
On Mon, Aug 20, 2012 at 4:08 AM, Christian König
deathsim...@vodafone.de 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
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 just
https://bugzilla.kernel.org/show_bug.cgi?id=31862
Alan a...@lxorguk.ukuu.org.uk changed:
What|Removed |Added
Status|REOPENED|NEEDINFO
https://bugzilla.kernel.org/show_bug.cgi?id=31862
Alan a...@lxorguk.ukuu.org.uk changed:
What|Removed |Added
Summary|2.6.39.3 (and earlier): |3.4(and earlier): White
On 08/19/2012 10:22 PM, Dave Airlie wrote:
On Mon, Aug 20, 2012 at 3:13 PM, Randy Dunlap rdun...@xenotime.net wrote:
On 08/17/12 15:55, Dave Airlie wrote:
On Sat, Aug 18, 2012 at 8:54 AM, Dave Airlie airl...@gmail.com wrote:
On Sat, Aug 18, 2012 at 8:28 AM, Randy Dunlap rdun...@xenotime.net
Hi Tomi,
On Monday 20 August 2012 14:39:30 Tomi Valkeinen wrote:
On Sat, 2012-08-18 at 03:16 +0200, Laurent Pinchart wrote:
Hi Tomi,
mipi-dbi-bus might not belong to include/video/panel/ though, as it can be
used for non-panel devices (at least in theory). The future mipi-dsi-bus
On Tue, Aug 21, 2012 at 8:45 AM, Randy Dunlap rdun...@xenotime.net wrote:
On 08/19/2012 10:22 PM, Dave Airlie wrote:
On Mon, Aug 20, 2012 at 3:13 PM, Randy Dunlap rdun...@xenotime.net wrote:
On 08/17/12 15:55, Dave Airlie wrote:
On Sat, Aug 18, 2012 at 8:54 AM, Dave Airlie airl...@gmail.com
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 therefore GL3.0) for Mesa 9.0.
Marek
On Sun, Aug 19, 2012 at 2:22 AM, Marek Ol??k wrote:
> MSAA is impossible without
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 therefore GL3.0) for Mesa 9.0.
I'll try and get it merged to
https://bugs.freedesktop.org/show_bug.cgi?id=53111
--- Comment #13 from Alexandre Demers
2012-08-19 23:03:48 UTC ---
(In reply to comment #12)
> I tried to trace RenderFeatTest (one of the other applications locking my
> system). It did as with the piglit test: it didn't crash. However, the
>
https://bugs.freedesktop.org/show_bug.cgi?id=53111
--- Comment #14 from Alexandre Demers
2012-08-19 23:04:33 UTC ---
Created attachment 65813
--> https://bugs.freedesktop.org/attachment.cgi?id=65813
bad rendereing on test 7, where it used to lock
--
Configure bugmail:
https://bugzilla.kernel.org/show_bug.cgi?id=46231
Summary: Radeon NI: evergreen_resume fails after GPU lockup
Product: Drivers
Version: 2.5
Kernel Version: 3.6-rc2
Platform: All
OS/Version: Linux
Tree: Mainline
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 gives any benefit to us.
>
>
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
by drm_helper_connector_dpms.
>
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?
>
> Signed-off-by: Inki Dae
> Signed-off-by: Kyungmin Park
> ---
>
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 is not DRM_FORMAT_NV12MT then it
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 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: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
>>> ---
>>>
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/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.
>>
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/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:
>>
>> 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
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.
>
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
https://bugs.freedesktop.org/show_bug.cgi?id=43655
--- Comment #19 from Alexandre Demers
2012-08-20 03:00:22 UTC ---
Fixed by attachment 64759 (proposed in bug 42373 which is similar to this bug
but is not the same since it is not fixed by the attachment)
--
Configure bugmail:
https://bugs.freedesktop.org/show_bug.cgi?id=42373
--- Comment #31 from Alexandre Demers
2012-08-20 03:01:28 UTC ---
(In reply to comment #16)
> Created attachment 64759 [details] [review]
> Fixup mc programing
>
> This patch should fix your issue.
This patch doesn't apply correctly on kernel
1 - 100 of 211 matches
Mail list logo