And another issue...
What is drm_crtc_helper_set_mode() passed as the fb argument? Is it
the old fb, or the new fb?
bool drm_crtc_helper_set_mode(struct drm_crtc *crtc,
struct drm_display_mode *mode,
int x, int y,
On Thu, Jun 13, 2013 at 12:19:03PM +0100, Russell King - ARM Linux wrote:
> The deeper I look, the more bugs there seem to be in this DRM stuff,
> and I'm continuing to look because I'm chasing a framebuffer refcount
> bug.
So, this refcount bug - I think I've just found it. This is the flow of
r
Hi,
On Thursday 13 June 2013 04:51 PM, Inki Dae wrote:
-Original Message-
From: Sylwester Nawrocki [mailto:s.nawro...@samsung.com]
Sent: Thursday, June 13, 2013 5:56 PM
To: Rahul Sharma
Cc: Rahul Sharma; Inki Dae; linux-samsung-...@vger.kernel.org; devicetree-
disc...@lists.ozlabs.org
On Thu, Jun 13, 2013 at 12:50:16PM +0100, Russell King - ARM Linux wrote:
> On Thu, Jun 13, 2013 at 12:19:03PM +0100, Russell King - ARM Linux wrote:
> > The deeper I look, the more bugs there seem to be in this DRM stuff,
> > and I'm continuing to look because I'm chasing a framebuffer refcount
>
On Thu, Jun 13, 2013 at 05:28:08PM +0900, Inki Dae wrote:
> This patch adds a buffer synchronization framework based on DMA BUF[1]
> and reservation[2] to use dma-buf resource, and based on ww-mutexes[3]
> for lock mechanism.
>
> The purpose of this framework is not only to couple cache operations
On Wed, Jun 12, 2013 at 6:56 AM, Jerome Glisse wrote:
> On Wed, Jun 12, 2013 at 6:26 AM, Michel Dänzer wrote:
>> On Die, 2013-06-11 at 16:23 -0700, Andy Lutomirski wrote:
>>> If the device is idle for over ten seconds, then the next attempt to do
>>> anything can race with the lockup detector and
Hello Kishon,
On 2013년 06월 13일 21:54, Kishon Vijay Abraham I wrote:
> Hi,
>
> On Thursday 13 June 2013 04:51 PM, Inki Dae wrote:
>>
>>
>>> -Original Message-
>>> From: Sylwester Nawrocki [mailto:s.nawro...@samsung.com]
>>> Sent: Thursday, June 13, 2013 5:56 PM
>>> To: Rahul Sharma
>>> Cc:
Hello Rahul,
On 2013년 06월 11일 23:11, Rahul Sharma wrote:
> Exynos hdmi IP version is named after hdmi specification version i.e.
> 1.3 and 1.4. This versioning mechanism is not sufficient to handle
> the diversity in the hdmi/phy IPs which are present across the exynos
> SoC family.
>
> This patc
Hello Rahul,
This patch looks good to me just with mixer part of 2nd patch because
there is no hdmiphy related things.
On 2013년 06월 11일 23:11, Rahul Sharma wrote:
> Add support for exynos5420 mixer IP in the drm mixer driver.
>
> Signed-off-by: Rahul Sharma
> ---
> drivers/gpu/drm/exynos/exyno
Hello Rahul,
this patch is not related with others and it looks good to me.
On 2013년 06월 11일 23:11, Rahul Sharma wrote:
> Cleanup by removing flags variable from drm_hdmi_dt_parse_pdata
> which is not used anywhere. Swtiching to of_get_named_gpio instead
> of of_get_named_gpio_flags solved this.
https://bugs.freedesktop.org/show_bug.cgi?id=64257
--- Comment #64 from Marc Dietrich ---
patch in comment 60 does not fix the issues (no change).
--
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri
https://bugs.freedesktop.org/show_bug.cgi?id=65438
Michel Dänzer changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=65730
--- Comment #5 from Andy Furniss ---
(In reply to comment #4)
> Before I being, I must make a disclaimer that I am not a Linux genius by any
> means, but a moderate user, able and willing to find my way around things.
> I do occasionally assist
vm_mmap function returns unsigned long so addr type should be unsigned long.
a pointer or address variable is required to use unsigned long or uint64_t
type for 64bits address support.
So this patch makes sure that addr has unsigned long type and also
exynos_drm_gem_mmap_ioctl returns correct erro
https://bugs.freedesktop.org/show_bug.cgi?id=65730
--- Comment #6 from Andy Furniss ---
(In reply to comment #5)
> option radeon audio=1
Oops, should be
options radeon audio=1
--
You are receiving this mail because:
You are the assignee for the bug.
Hi Dave,
Could you please take this patch in your tree for v3.11 ?
On Thursday 25 April 2013 12:14:54 Laurent Pinchart wrote:
> This simplifies cleanup paths and fixes a probe time crash in the error
> path when trying to cleanup mode setting before it was initialized.
>
> Signed-off-by: Laurent
Hi,
Tested this series on snow board and is working fine.
Tested-by: Arun Kumar K
Regards
Arun
On Mon, Jun 10, 2013 at 4:30 PM, Rahul Sharma wrote:
> Add clock changes for hdmi subsystem for exynos5250 SoC. These
> include addition of new clocks like mout_hdmi and smmu_tv, associating
> ID to
https://bugs.freedesktop.org/show_bug.cgi?id=65730
--- Comment #7 from Michel Dänzer ---
The kernel command line works regardless of whether the driver was built into
the kernel or as a module.
--
You are receiving this mail because:
You are the assignee for the bug.
___
https://bugs.freedesktop.org/show_bug.cgi?id=65730
--- Comment #8 from Andy Furniss ---
(In reply to comment #7)
> The kernel command line works regardless of whether the driver was built
> into the kernel or as a module.
Oh OK, sorry for the noise.
--
You are receiving this mail because:
You
https://bugs.freedesktop.org/show_bug.cgi?id=65723
Michel Dänzer changed:
What|Removed |Added
Summary|Xonotic glsl 1.30 broken|Xonotic glsl 1.30 broken
On Thu, Jun 13, 2013 at 12:50:16PM +0100, Russell King - ARM Linux wrote:
> On Thu, Jun 13, 2013 at 12:19:03PM +0100, Russell King - ARM Linux wrote:
> > The deeper I look, the more bugs there seem to be in this DRM stuff,
> > and I'm continuing to look because I'm chasing a framebuffer refcount
>
On Fri, Jun 14, 2013 at 02:54:04AM +0200, Laurent Pinchart wrote:
> Hi Daniel,
>
> On Friday 07 June 2013 10:50:55 Daniel Vetter wrote:
> > On Fri, Jun 07, 2013 at 09:44:45AM +0200, Laurent Pinchart wrote:
> > > On Wednesday 05 June 2013 10:55:05 Daniel Vetter wrote:
> > > > On Wed, Jun 05, 2013 a
On Thu, Jun 13, 2013 at 3:03 PM, Russell King - ARM Linux
wrote:
> On Thu, Jun 13, 2013 at 12:50:16PM +0100, Russell King - ARM Linux wrote:
>> On Thu, Jun 13, 2013 at 12:19:03PM +0100, Russell King - ARM Linux wrote:
>> > The deeper I look, the more bugs there seem to be in this DRM stuff,
>> > a
On 06/12/2013 06:16 AM, Joonyoung Shim wrote:
This adds to call low-level mmap() from prime helpers.
Signed-off-by: Joonyoung Shim
---
drivers/gpu/drm/drm_prime.c | 5 -
include/drm/drmP.h | 2 ++
2 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/drm
https://bugs.freedesktop.org/show_bug.cgi?id=65730
--- Comment #9 from Jason ---
Well, I downloaded the latest stable 3.9.6 kernel from www.kernel.org and
applied the patch. Apparently my system didn't like the new kernel because it
broke my X. I'm just not going to worry about this one for now
https://bugs.freedesktop.org/show_bug.cgi?id=65723
--- Comment #5 from Rafael Castillo ---
many thanks for your answer, i will try those patches and report back
--
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel ma
https://bugs.freedesktop.org/show_bug.cgi?id=65438
--- Comment #13 from Rafael Castillo ---
many thanks for the help very appreciated
--
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.
On Fri, Jun 14, 2013 at 4:42 PM, Russell King - ARM Linux
wrote:
> On Fri, Jun 14, 2013 at 04:23:22PM +0200, Daniel Vetter wrote:
>> On Thu, Jun 13, 2013 at 3:03 PM, Russell King - ARM Linux
>> wrote:
>> > There's a bigger issue here - if it's possible for
>> > drm_crtc_helper_set_config()
>> >
Hi Dave,
I added one more patch on top if it's not too late. It's a fix for
UVD on big endian:
drm/radeon: fix UVD on big endian
This fixes the kernel side so that the ring should come
up and ring and IB tests should work. The userspace
UVD drivers will also need big endian fixes.
Signed-off
Hello,
After a (too) long delay here's the fifth version of my modetest enhancements
patch set.
Beside various cleanups, these patches allow dropping master after mode set,
configuring more than two pipes and planes, setting properties from the
command line, setting plane positions and configurin
Enable all standard automake warnings except for -Wpointer-arith (as the
test pattern generation code uses void pointer arithmetics) and fix
them.
Signed-off-by: Laurent Pinchart
---
tests/modetest/Makefile.am | 4 +++-
tests/modetest/buffers.c | 13 +++--
tests/modetest/buffers.h |
Those variables are declared in unistd.h, there's no need to redeclare
them here.
Signed-off-by: Laurent Pinchart
Reviewed-by: Jani Nikula
---
tests/modetest/modetest.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
index 47704db..e0d7
The current mostly random sort order hinders code readability. Sort the
options alphabetically in the code, and by group in the help message.
Signed-off-by: Laurent Pinchart
Reviewed-by: Jani Nikula
---
tests/modetest/modetest.c | 49 ++-
1 file chang
If the -d parameter is specified, modetest will drop master permissions
after setting the mode.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
inde
If the -M parameter is specified, modetest will use the requested device
name instead of trying its builtin list of device names.
Signed-off-by: Laurent Pinchart
Reviewed-by: Jani Nikula
---
tests/modetest/modetest.c | 50 ---
1 file changed, 34 inser
Instead of retrieving resources as they are needed, retrieve them all
(except property blobs) in one go at startup.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 408 +-
1 file changed, 261 insertions(+), 147 deletions(-)
diff --git
Configuring mode on more than two connectors or two planes is perfectly
valid. Support it.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 20 ++--
1 file changed, 18 insertions(+), 2 deletions(-)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
The -w parameter can be used to set a property value from the command
line, using the target object ID and the property name.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 108 +-
1 file changed, 106 insertions(+), 2 deletions(-)
dif
Extend the -P option to allow specifying the plane x and y offsets. The
position is optional, if not specified the plane will be positioned at
the center of the screen as before.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 67 ---
1
The argument isn't used, remove it.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 10 +++---
1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
index 258b958..b736f2b 100644
--- a/tests/modetest/modetest.c
+++
As modetest automatically selects an unused plan, providing the plane ID
allows modifying plane properties for the selected planes.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tests/modetest/modetest.c
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 29 +
1 file changed, 17 insertions(+), 12 deletions(-)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
index 4a6566f..c4e4dda 100644
--- a/tests/modetest/modetest.c
+++ b/tests/modetes
Instead of passing the device fd and resources as global variables group
them in a device structure and pass it explictly to all functions that
need it.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 199 --
1 file changed, 102 inserti
This prepares the code for the split in separate functions of CRTC and
planes setup.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 58 ++-
1 file changed, 37 insertions(+), 21 deletions(-)
diff --git a/tests/modetest/modetest.c b/tes
The field is no needed, make it a local variable where used.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
index c4e4dda..a7b8b43 100644
--- a/test
This prepares the code for the split in separate functions of CRTC and
planes setup.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
index e5f12ce..c
Planes are associated with CRTCs, not connectors. Don't try to be too
clever, use the CRTC ID in the -P option. This prepares for splitting
CRTC and planes setup.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 32 +---
1 file changed, 17 insertions(+)
There's not reason to require setting a mode to test planes. Split the
two operations.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 106 +++---
1 file changed, 73 insertions(+), 33 deletions(-)
diff --git a/tests/modetest/modetest.c b/t
This prepares the code for handling multiple connectors in a single
pipeline in a cloned configuration.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 162 --
1 file changed, 85 insertions(+), 77 deletions(-)
diff --git a/tests/modete
The -s argument can now take a list of connectors. Configure all of them
in cloned mode using a single CRTC.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 211 ++
1 file changed, 157 insertions(+), 54 deletions(-)
diff --git a/tests/
When building the pipeline, instead of using only the encoders attached
to a connector, take all possible encoders into account to locate a
CRTC.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 35 +--
1 file changed, 25 insertions(+), 10 deletions
Multiple the image height by 1.5 for NV12/NV21 and by 2 for NV16/NV61 to
make room for the chroma plane.
Signed-off-by: Laurent Pinchart
---
tests/modetest/buffers.c | 19 ++-
1 file changed, 18 insertions(+), 1 deletion(-)
diff --git a/tests/modetest/buffers.c b/tests/modetest/
The line stride passed to the function is expressed in bytes, there's no
need to multiply it by 2.
Signed-off-by: Laurent Pinchart
---
tests/modetest/buffers.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/tests/modetest/buffers.c b/tests/modetest/buffers.c
ind
On Friday 14 June 2013 23:21:11 Laurent Pinchart wrote:
> Hello,
>
> After a (too) long delay here's the fifth version of my modetest
> enhancements patch set.
And I've of course sent the wrong version :-/ v6 (hopefully final) is on the
way. Sorry for the noise.
> Beside various cleanups, these
Hello,
Here's the sixth (and hopefully final) version of my modeset enhancements
patch set.
Beside various cleanups, these patches allow dropping master after mode set,
configuring more than two pipes and planes, setting properties from the
command line, setting plane positions and configuring pi
Enable all standard automake warnings except for -Wpointer-arith (as the
test pattern generation code uses void pointer arithmetics) and fix
them.
Signed-off-by: Laurent Pinchart
---
tests/modetest/Makefile.am | 4 +++-
tests/modetest/buffers.c | 13 +++--
tests/modetest/buffers.h |
Those variables are declared in unistd.h, there's no need to redeclare
them here.
Signed-off-by: Laurent Pinchart
Reviewed-by: Jani Nikula
---
tests/modetest/modetest.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
index 47704db..e0d7
If the -M parameter is specified, modetest will use the requested device
name instead of trying its builtin list of device names.
Signed-off-by: Laurent Pinchart
Reviewed-by: Jani Nikula
---
tests/modetest/modetest.c | 50 +--
1 file changed, 35 inser
The current mostly random sort order hinders code readability. Sort the
options alphabetically in the code, and by group in the help message.
Signed-off-by: Laurent Pinchart
Reviewed-by: Jani Nikula
---
tests/modetest/modetest.c | 49 ++-
1 file chang
If the -d parameter is specified, modetest will drop master permissions
after setting the mode.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
inde
Instead of retrieving resources as they are needed, retrieve them all
(except property blobs) in one go at startup.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 408 +-
1 file changed, 261 insertions(+), 147 deletions(-)
diff --git
Configuring mode on more than two connectors or two planes is perfectly
valid. Support it.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 20 ++--
1 file changed, 18 insertions(+), 2 deletions(-)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
The -w parameter can be used to set a property value from the command
line, using the target object ID and the property name.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 108 +-
1 file changed, 106 insertions(+), 2 deletions(-)
dif
Extend the -P option to allow specifying the plane x and y offsets. The
position is optional, if not specified the plane will be positioned at
the center of the screen as before.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 67 ---
1
As modetest automatically selects an unused plan, providing the plane ID
allows modifying plane properties for the selected planes.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tests/modetest/modetest.c
The argument isn't used, remove it.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 10 +++---
1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
index 7f7a3a2..89bfc53 100644
--- a/tests/modetest/modetest.c
+++
Instead of passing the device fd and resources as global variables group
them in a device structure and pass it explictly to all functions that
need it.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 201 --
1 file changed, 103 inserti
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 29 +
1 file changed, 17 insertions(+), 12 deletions(-)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
index cfcb989..a1a683f 100644
--- a/tests/modetest/modetest.c
+++ b/tests/modetes
The field is no needed, make it a local variable where used.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
index a1a683f..2c3e093 100644
--- a/test
This prepares the code for the split in separate functions of CRTC and
planes setup.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 58 ++-
1 file changed, 37 insertions(+), 21 deletions(-)
diff --git a/tests/modetest/modetest.c b/tes
This prepares the code for the split in separate functions of CRTC and
planes setup.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
index 6fbaf09..3
There's not reason to require setting a mode to test planes. Split the
two operations.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 104 --
1 file changed, 72 insertions(+), 32 deletions(-)
diff --git a/tests/modetest/modetest.c b/t
Planes are associated with CRTCs, not connectors. Don't try to be too
clever, use the CRTC ID in the -P option. This prepares for splitting
CRTC and planes setup.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 32 +---
1 file changed, 17 insertions(+)
This prepares the code for handling multiple connectors in a single
pipeline in a cloned configuration.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 162 --
1 file changed, 85 insertions(+), 77 deletions(-)
diff --git a/tests/modete
The -s argument can now take a list of connectors. Configure all of them
in cloned mode using a single CRTC.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 211 ++
1 file changed, 157 insertions(+), 54 deletions(-)
diff --git a/tests/
When building the pipeline, instead of using only the encoders attached
to a connector, take all possible encoders into account to locate a
CRTC.
Signed-off-by: Laurent Pinchart
---
tests/modetest/modetest.c | 35 +--
1 file changed, 25 insertions(+), 10 deletions
The line stride passed to the function is expressed in bytes, there's no
need to multiply it by 2.
Signed-off-by: Laurent Pinchart
---
tests/modetest/buffers.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/tests/modetest/buffers.c b/tests/modetest/buffers.c
ind
Multiple the image height by 1.5 for NV12/NV21 and by 2 for NV16/NV61 to
make room for the chroma plane.
Signed-off-by: Laurent Pinchart
---
tests/modetest/buffers.c | 19 ++-
1 file changed, 18 insertions(+), 1 deletion(-)
diff --git a/tests/modetest/buffers.c b/tests/modetest/
Hi all,
Russell found a refcount bug in my fb refcounting conversion, but to also fix
i915.ko I've opted for a slightly different approach.
While at it I've thought it would be good to backport some of the semantic
changes we've implented in i915's ->set_config callback since we've forked our
own
There's no point in trying to clean up after driver-bugs, so just blow
up. Furthermore it's an interface abuse to set no mode but have an fb
and aslo to try to set an fb without enough connectors. These two
spefici cases of interface abuse have been committed by the fb helper,
but that's been fixed
... since we already check for fb->pixel_format, which encodes all
this. The other two fields are only for backwards compat of older
drivers (and we might want to look into eventually just killing them).
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/drm_crtc_helper.c |5 -
1 file chan
Atm the crtc helper implementation of set_config has really
inconsisten semantics: If just an fb update is good enough, dpms state
will be left as-is, but if we do a full modeset we force everything to
dpms on.
This change has already been applied to the i915 modeset code in
commit e3de42b68478a8
This has originally been added in
commit a3a0544b2c84e1d7a2022b558ecf66d8c6a8dd93
Author: Dave Airlie
Date: Mon Aug 31 15:16:30 2009 +1000
drm/kms: add explicit encoder disable function and detach harder.
I think it's the wrong thing to do for a few reasons:
- It's policy in the kernel. U
Historically drm lacked fb refcounting, so the updating of crtc->fb
was done by the lower levels at a point convenient to get their own
refcounting (e.g. refcounts for the underlying gem bo, pinning
refcounts) right. With the introduction of refcounted fbs the drm core
handled the fb refcounts, but
Drivers are allowed (actually have to) disable unrelated crtcs in
their ->set_config callback (when we steal all the connectors from
that crtc). If they do that they'll clear crtc->fb to NULL.
Which results in a refcount leak, since the drm core is keeping track
of that reference.
To fix this tra
On Sat, Jun 15, 2013 at 12:15 AM, Russell King - ARM Linux
wrote:
> On Fri, Jun 14, 2013 at 09:50:22PM +0200, Daniel Vetter wrote:
>> On Fri, Jun 14, 2013 at 4:42 PM, Russell King - ARM Linux
>> wrote:
>> > If you're happy with the patch I supplied, that's probably the minimal fix
>> > which shou
On Sat, Jun 15, 2013 at 12:14:42PM +0800, Aaron Lu wrote:
> On 06/15/2013 09:38 AM, Matthew Garrett wrote:
> > Well, Windows 8 will only use the ACPI backlight interface if the GPU
> > driver decides to, right? So the logic for deciding whether to remove
> > the ACPI backlight control or not should
venenience.
--
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/20130614/285c88b3/attachment.html>
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130614/77405a83/attachment.html>
Hi Daniel,
On Friday 07 June 2013 10:50:55 Daniel Vetter wrote:
> On Fri, Jun 07, 2013 at 09:44:45AM +0200, Laurent Pinchart wrote:
> > On Wednesday 05 June 2013 10:55:05 Daniel Vetter wrote:
> > > On Wed, Jun 05, 2013 at 03:51:53AM +0200, Laurent Pinchart wrote:
> > > > On Tuesday 04 June 2013 20
Hi Russell,
> -Original Message-
> From: Russell King - ARM Linux [mailto:linux at arm.linux.org.uk]
> Sent: Friday, June 14, 2013 2:26 AM
> To: Inki Dae
> Cc: maarten.lankhorst at canonical.com; daniel at ffwll.ch; robdclark at
> gmail.com;
> linux-fbdev at vger.kernel.org; dri-devel at
Hello Kishon,
On 2013? 06? 13? 21:54, Kishon Vijay Abraham I wrote:
> Hi,
>
> On Thursday 13 June 2013 04:51 PM, Inki Dae wrote:
>>
>>
>>> -Original Message-
>>> From: Sylwester Nawrocki [mailto:s.nawrocki at samsung.com]
>>> Sent: Thursday, June 13, 2013 5:56 PM
>>> To: Rahul Sharma
>>>
Hello Rahul,
On 2013? 06? 11? 23:11, Rahul Sharma wrote:
> Exynos hdmi IP version is named after hdmi specification version i.e.
> 1.3 and 1.4. This versioning mechanism is not sufficient to handle
> the diversity in the hdmi/phy IPs which are present across the exynos
> SoC family.
>
> This patc
Hello Rahul,
This patch looks good to me just with mixer part of 2nd patch because
there is no hdmiphy related things.
On 2013? 06? 11? 23:11, Rahul Sharma wrote:
> Add support for exynos5420 mixer IP in the drm mixer driver.
>
> Signed-off-by: Rahul Sharma
> ---
> drivers/gpu/drm/exynos/exyno
Hello Rahul,
this patch is not related with others and it looks good to me.
On 2013? 06? 11? 23:11, Rahul Sharma wrote:
> Cleanup by removing flags variable from drm_hdmi_dt_parse_pdata
> which is not used anywhere. Swtiching to of_get_named_gpio instead
> of of_get_named_gpio_flags solved this.
...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130614/50422e87/attachment.html>
org/archives/dri-devel/attachments/20130614/5a4c817a/attachment.html>
somewhere and regenerating something ...
--
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/20130614/d4bfa6cc/attachment.html>
vm_mmap function returns unsigned long so addr type should be unsigned long.
a pointer or address variable is required to use unsigned long or uint64_t
type for 64bits address support.
So this patch makes sure that addr has unsigned long type and also
exynos_drm_gem_mmap_ioctl returns correct erro
art --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130614/ea4cde3f/attachment-0001.html>
1 - 100 of 171 matches
Mail list logo
