Broken monitors and/or broken graphic boards may send erroneous or no
EDID data. This also applies to broken KVM devices that are unable to
correctly forward the EDID data of the connected monitor but invent
their own fantasy data.
This patch allows to specify an EDID data set to be used instead o
Changes from V5 to V6:
1. The use of a particular EDID data set can now be restricted to a
given connector using the syntax
edid_firmware=[:]
as suggested by Thomas Reim.
2. The patches to introduce a module parameter and a quirk to cope with
an inverted brightness variable of some i915 graphic
Dear Thomas,
your proposal will not lead to a satisfying solution for those people
that suffer from monitors and other HW devices that provide wrong
EDIDs.
Oops, why not?
Before fetching the display modes, all drm devices will try to detect
the monitor. If EDID data is not correct (checksum),
On Tue, Mar 13, 2012 at 11:17:02AM +0100, Tomasz Stanislawski wrote:
> This patch adds extension to V4L2 api. It allow to export a mmap buffer as
> file
> descriptor. New ioctl VIDIOC_EXPBUF is added. It takes a buffer offset used by
> mmap and return a file descriptor on success.
>
> Signed-off-
An embedded and charset-unspecified text was scrubbed...
Name: drivers-gpu-drm-allow-to-load-edid-firmware.patch
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20120318/3bf9523d/attachment.ksh>
Changes from V5 to V6:
1. The use of a particular EDID data set can now be restricted to a
given connector using the syntax
edid_firmware=[:]
as suggested by Thomas Reim.
2. The patches to introduce a module parameter and a quirk to cope with
an inverted brightness variable of some i915 graphic
Hi Dave,
QA beat on the -next tree in my absence and filed 3 new issue. Afaict
they're all new sightings of old issues (hooray for improved test-suites)
safe for a very funky sporadic failure on ilk - investigation is still
going on this one.
Otherwise the missed irq on snb poped up again, but Be
On Sun, Mar 18, 2012 at 01:00:11PM -0700, Joe Perches wrote:
> Use a more current logging style. Ensure that appropriate
> logging messages are prefixed with "i915: ".
>
> Convert printks to pr_. Align arguments.
>
> Signed-off-by: Joe Perches
Queued for -next, thanks for the patch.
-Daniel
-
Dear Thomas,
> your proposal will not lead to a satisfying solution for those people
> that suffer from monitors and other HW devices that provide wrong
> EDIDs.
Oops, why not?
> Before fetching the display modes, all drm devices will try to detect
> the monitor. If EDID data is not correct (chec
On Sun, Mar 18, 2012 at 6:34 PM, Daniel Vetter
wrote:
> Big differences to other contenders in the field (like ion) is
> that this also supports highmem, so we have to split up the cpu
> access from the kernel side into a prepare and a kmap step.
>
> Prepare is allowed to fail and should do every
On Sun, Mar 18, 2012 at 6:34 PM, Daniel Vetter
wrote:
> v2: Fix spelling issues noticed by Rob Clark.
>
> Signed-off-by: Daniel Vetter
Signed-off-by: Rob Clark
> ---
> ?Documentation/dma-buf-sharing.txt | ?102 +++-
> ?1 files changed, 99 insertions(+), 3 deleti
On Fri, Mar 16, 2012 at 09:43:49PM -0700, Joe Perches wrote:
> Use a more current logging style. Ensure that appropriate
> logging messages are prefixed with "i915: ".
>
> Convert printks to pr_. Align arguments.
>
> Signed-off-by: Joe Perches
I've tried to apply this to my -next branch, but
On Sun, Mar 18, 2012 at 08:04:53PM +0100, Daniel Vetter wrote:
> On Sun, Mar 18, 2012 at 01:12:22PM +0530, Sumit Semwal wrote:
> > On 16 March 2012 23:23, Dave Airlie wrote:
> > > On Fri, Mar 16, 2012 at 4:04 PM, Rob Clark
> > > wrote:
> > >> From: Rob Clark
> > >>
> > >> Works in a similar way
On Sun, Mar 18, 2012 at 01:12:22PM +0530, Sumit Semwal wrote:
> On 16 March 2012 23:23, Dave Airlie wrote:
> > On Fri, Mar 16, 2012 at 4:04 PM, Rob Clark wrote:
> >> From: Rob Clark
> >>
> >> Works in a similar way to get_file(), and is needed in cases such as
> >> when the exporter needs to als
On Thu, Mar 15, 2012 at 03:56:23PM +0100, Carsten Emde wrote:
> Changes from V4 to V5:
>
> 1. Split the previous patch 2/2 into three patches to separately i) add the
> module parameter, ii) add the quirk infrastructure, and iii) actually quirk
> the related machine.
>
> 2. Rename the module para
On Sat, Mar 10, 2012 at 02:48:14AM +0800, Daniel Kurtz wrote:
> This patchset addresses a couple of issues with the i915 gmbus implementation:
> * fixes misassigned pin port pair for HDMI-D
> * fixes write transactions when they are the only transaction requested
>(including large >4-byte wri
On Wed, Mar 07, 2012 at 07:50:46PM +0800, Daniel Kurtz wrote:
> The i915 has multiple i2c adapters. However, they all share a single
> single set of i2c control registers (algorithm). Thus, different threads
> trying to access different adapters could interfere with each other.
>
> Note: differe
Signed-off-by: Marek Olšák
---
drivers/gpu/drm/radeon/r600_cs.c | 270 +-
1 files changed, 150 insertions(+), 120 deletions(-)
diff --git a/drivers/gpu/drm/radeon/r600_cs.c b/drivers/gpu/drm/radeon/r600_cs.c
index d9ebec3..0ec3f20 100644
--- a/drivers/gpu/drm
and document the other unused ones.
Signed-off-by: Marek Olšák
---
drivers/gpu/drm/radeon/evergreen_cs.c | 58 +++--
drivers/gpu/drm/radeon/reg_srcs/cayman| 12 ++
drivers/gpu/drm/radeon/reg_srcs/evergreen | 12 ++
3 files changed, 30 insertions(+),
Signed-off-by: Marek Olšák
---
drivers/gpu/drm/radeon/r600_cs.c | 89 +-
1 files changed, 68 insertions(+), 21 deletions(-)
diff --git a/drivers/gpu/drm/radeon/r600_cs.c b/drivers/gpu/drm/radeon/r600_cs.c
index b3c40e0..d9ebec3 100644
--- a/drivers/gpu/drm/r
Signed-off-by: Marek Olšák
---
drivers/gpu/drm/radeon/evergreen_cs.c | 91 -
1 files changed, 66 insertions(+), 25 deletions(-)
diff --git a/drivers/gpu/drm/radeon/evergreen_cs.c
b/drivers/gpu/drm/radeon/evergreen_cs.c
index 0427b96..7327bc7 100644
--- a/driver
There are also two fixes:
- In DRAW_INDEX_2, we read idx_value, but should have read idx+1.
- When correcting SQ_VTX_CONSTANT_WORD1_0.SIZE, we should subtract
the offset.
Signed-off-by: Marek Olšák
---
drivers/gpu/drm/radeon/evergreen_cs.c | 130 ++---
1 files chan
Signed-off-by: Marek Olšák
---
drivers/gpu/drm/radeon/evergreen_cs.c | 37 ++--
1 files changed, 21 insertions(+), 16 deletions(-)
diff --git a/drivers/gpu/drm/radeon/evergreen_cs.c
b/drivers/gpu/drm/radeon/evergreen_cs.c
index b39a089..0427b96 100644
--- a/drivers
Signed-off-by: Marek Olšák
---
drivers/gpu/drm/radeon/evergreen_cs.c |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/radeon/evergreen_cs.c
b/drivers/gpu/drm/radeon/evergreen_cs.c
index 8bf576a..4674a68 100644
--- a/drivers/gpu/drm/radeon/evergreen_
On Sun, Mar 18, 2012 at 6:34 PM, Daniel Vetter wrote:
> Big differences to other contenders in the field (like ion) is
> that this also supports highmem, so we have to split up the cpu
> access from the kernel side into a prepare and a kmap step.
>
> Prepare is allowed to fail and should do everyt
On Sun, Mar 18, 2012 at 6:34 PM, Daniel Vetter wrote:
> v2: Fix spelling issues noticed by Rob Clark.
>
> Signed-off-by: Daniel Vetter
Signed-off-by: Rob Clark
> ---
> Documentation/dma-buf-sharing.txt | 102 +++-
> 1 files changed, 99 insertions(+), 3 deletio
Dear Carsten,
your proposal will not lead to a satisfying solution for those people
that suffer from monitors and other HW devices that provide wrong
EDIDs.
Before fetching the display modes, all drm devices will try to detect
the monitor. If EDID data is not correct (checksum), an error
notifi
https://bugs.freedesktop.org/show_bug.cgi?id=38471
Thomas Kowaliczek changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
https://bugs.freedesktop.org/show_bug.cgi?id=46323
Ben Gouhier changed:
What|Removed |Added
Resolution|FIXED |WORKSFORME
--
Configure bugmail: https://
https://bugs.freedesktop.org/show_bug.cgi?id=46323
Ben Gouhier changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
https://bugs.freedesktop.org/show_bug.cgi?id=46323
Ben Gouhier changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
https://bugs.freedesktop.org/show_bug.cgi?id=47475
Chris Wilson changed:
What|Removed |Added
AssignedTo|idr at freedesktop.org |dri-devel at
lists.freedesktop
Otherwise subsystems will get this wrong and end up with and second
export ioctl with the flag and O_CLOEXEC support added.
Signed-Off-by: Daniel Vetter
---
Documentation/dma-buf-sharing.txt |5 +
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/Documentation/dma-buf-sharin
v2: Fix spelling issues noticed by Rob Clark.
Signed-off-by: Daniel Vetter
---
Documentation/dma-buf-sharing.txt | 102 +++-
1 files changed, 99 insertions(+), 3 deletions(-)
diff --git a/Documentation/dma-buf-sharing.txt
b/Documentation/dma-buf-sharing.txt
ind
Big differences to other contenders in the field (like ion) is
that this also supports highmem, so we have to split up the cpu
access from the kernel side into a prepare and a kmap step.
Prepare is allowed to fail and should do everything required so that
the kmap calls can succeed (like swapin/ba
The mutex protects the attachment list and hence needs to be held
around the callbakc to the exporters (optional) attach/detach
functions.
Holding the mutex around the map/unmap calls doesn't protect any
dma_buf state. Exporters need to properly protect any of their own
state anyway (to protect ag
On Tue, Mar 13, 2012 at 11:17:02AM +0100, Tomasz Stanislawski wrote:
> This patch adds extension to V4L2 api. It allow to export a mmap buffer as
> file
> descriptor. New ioctl VIDIOC_EXPBUF is added. It takes a buffer offset used by
> mmap and return a file descriptor on success.
>
> Signed-off-
Hi Dave,
QA beat on the -next tree in my absence and filed 3 new issue. Afaict
they're all new sightings of old issues (hooray for improved test-suites)
safe for a very funky sporadic failure on ilk - investigation is still
going on this one.
Otherwise the missed irq on snb poped up again, but Be
On Sun, Mar 18, 2012 at 01:00:11PM -0700, Joe Perches wrote:
> Use a more current logging style. Ensure that appropriate
> logging messages are prefixed with "i915: ".
>
> Convert printks to pr_. Align arguments.
>
> Signed-off-by: Joe Perches
Queued for -next, thanks for the patch.
-Daniel
-
On Sat, Mar 17, 2012 at 14:40:04 -0400, Matt Turner wrote:
> On Sat, Mar 17, 2012 at 2:35 PM, Jakob Bornecrantz
> wrote:
> > With my limited knowledge of automake I'm going to have
> > to NACK this patch. Most of these programs are used during
> > driver bring up to test things out, often we als
On 16 March 2012 20:21, Rob Clark wrote:
> On Fri, Mar 16, 2012 at 5:34 AM, Dave Airlie wrote:
>> From: Dave Airlie
>>
>> We need to pass the flags into dma_buf_fd at this point,
>> so the flags end up doing the right thing for O_CLOEXEC.
>>
>> Signed-off-by: Dave Airlie
>
> Signed-off-by: Rob
On 16 March 2012 23:23, Dave Airlie wrote:
> On Fri, Mar 16, 2012 at 4:04 PM, Rob Clark wrote:
>> From: Rob Clark
>>
>> Works in a similar way to get_file(), and is needed in cases such as
>> when the exporter needs to also keep a reference to the dmabuf (that
>> is later released with a dma_buf
Use a more current logging style. Ensure that appropriate
logging messages are prefixed with "i915: ".
Convert printks to pr_. Align arguments.
Signed-off-by: Joe Perches
---
drivers/gpu/drm/i915/i915_dma.c |6 ++-
drivers/gpu/drm/i915/i915_irq.c | 70 +--
Use a more current logging style. Ensure that appropriate
logging messages are prefixed with "i915: ".
Convert printks to pr_. Align arguments.
Signed-off-by: Joe Perches
---
drivers/gpu/drm/i915/i915_dma.c |6 ++-
drivers/gpu/drm/i915/i915_irq.c | 70 +--
On Sunday, March 18, 2012, Hugh Dickins wrote:
> Added Rafael to the Cc: Rafael, we're pondering over one or more of these
> recurrent threads about corruption after resume, seemingly related to i915.
Thanks for letting me know. :-)
I actually have a confirmation that the issue isn't present if t
On Fri, Mar 16, 2012 at 09:43:49PM -0700, Joe Perches wrote:
> Use a more current logging style. Ensure that appropriate
> logging messages are prefixed with "i915: ".
>
> Convert printks to pr_. Align arguments.
>
> Signed-off-by: Joe Perches
I've tried to apply this to my -next branch, but
On Sun, Mar 18, 2012 at 08:04:53PM +0100, Daniel Vetter wrote:
> On Sun, Mar 18, 2012 at 01:12:22PM +0530, Sumit Semwal wrote:
> > On 16 March 2012 23:23, Dave Airlie wrote:
> > > On Fri, Mar 16, 2012 at 4:04 PM, Rob Clark wrote:
> > >> From: Rob Clark
> > >>
> > >> Works in a similar way to get
On Sun, Mar 18, 2012 at 01:12:22PM +0530, Sumit Semwal wrote:
> On 16 March 2012 23:23, Dave Airlie wrote:
> > On Fri, Mar 16, 2012 at 4:04 PM, Rob Clark wrote:
> >> From: Rob Clark
> >>
> >> Works in a similar way to get_file(), and is needed in cases such as
> >> when the exporter needs to als
On Sun, Mar 18, 2012 at 6:10 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> DisplayPort spec v1.1a, Table 2-52.
>
> Same commit to Intel by ajax in 9962c9252e46eda7058067cbe73bdf1ed74b0d37
>
> Signed-off-by: Dave Airlie
Reviewed-by: Alex Deucher
> ---
> ?drivers/gpu/drm/radeon/atombios_dp.c |
On Thu, Mar 15, 2012 at 03:56:23PM +0100, Carsten Emde wrote:
> Changes from V4 to V5:
>
> 1. Split the previous patch 2/2 into three patches to separately i) add the
> module parameter, ii) add the quirk infrastructure, and iii) actually quirk
> the related machine.
>
> 2. Rename the module para
gnome.org/show_bug.cgi?id=672129
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20120318/95a06331/attachment.pgp>
only
seems to use the file name.
Thanks,
Paul
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20120318/5192078e/attachment.pgp>
On Sat, Mar 10, 2012 at 02:48:14AM +0800, Daniel Kurtz wrote:
> This patchset addresses a couple of issues with the i915 gmbus implementation:
> * fixes misassigned pin port pair for HDMI-D
> * fixes write transactions when they are the only transaction requested
>(including large >4-byte wri
On Wed, Mar 07, 2012 at 07:50:46PM +0800, Daniel Kurtz wrote:
> The i915 has multiple i2c adapters. However, they all share a single
> single set of i2c control registers (algorithm). Thus, different threads
> trying to access different adapters could interfere with each other.
>
> Note: differe
From: Dave Airlie
DisplayPort spec v1.1a, Table 2-52.
Same commit to Intel by ajax in 9962c9252e46eda7058067cbe73bdf1ed74b0d37
Signed-off-by: Dave Airlie
---
drivers/gpu/drm/radeon/atombios_dp.c |3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/drivers/gpu/drm/radeon/
https://bugs.freedesktop.org/show_bug.cgi?id=47475
Chris Wilson changed:
What|Removed |Added
AssignedTo|i...@freedesktop.org |dri-devel@lists.freedesktop
Dear Carsten,
your proposal will not lead to a satisfying solution for those people
that suffer from monitors and other HW devices that provide wrong
EDIDs.
Before fetching the display modes, all drm devices will try to detect
the monitor. If EDID data is not correct (checksum), an error
notifi
On Sun, Mar 18, 2012 at 6:10 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> DisplayPort spec v1.1a, Table 2-52.
>
> Same commit to Intel by ajax in 9962c9252e46eda7058067cbe73bdf1ed74b0d37
>
> Signed-off-by: Dave Airlie
Reviewed-by: Alex Deucher
> ---
> drivers/gpu/drm/radeon/atombios_dp.c |
On 16 March 2012 20:21, Rob Clark wrote:
> On Fri, Mar 16, 2012 at 5:34 AM, Dave Airlie wrote:
>> From: Dave Airlie
>>
>> We need to pass the flags into dma_buf_fd at this point,
>> so the flags end up doing the right thing for O_CLOEXEC.
>>
>> Signed-off-by: Dave Airlie
>
> Signed-off-by: Rob
On 16 March 2012 23:23, Dave Airlie wrote:
> On Fri, Mar 16, 2012 at 4:04 PM, Rob Clark wrote:
>> From: Rob Clark
>>
>> Works in a similar way to get_file(), and is needed in cases such as
>> when the exporter needs to also keep a reference to the dmabuf (that
>> is later released with a dma_buf
Added Rafael to the Cc: Rafael, we're pondering over one or more of these
recurrent threads about corruption after resume, seemingly related to i915.
On Sat, 17 Mar 2012, Keith Packard wrote:
> On Sat, 17 Mar 2012 18:44:18 -0700 (PDT), Hugh Dickins
> wrote:
> > I keep worrying about the sequence
On Sat, 17 Mar 2012, Keith Packard wrote:
> On Sat, 17 Mar 2012 15:52:15 -0700, Linus Torvalds
> wrote:
>
> > I do not believe we actually ever uncovered the original problem with
> > _MOVABLE: the problem was bisected down to the stable-backported
> > version of commit 4bdadb978569 ("drm/i915:
On Sat, 17 Mar 2012, Linus Torvalds wrote:
> On Sat, Mar 17, 2012 at 1:23 PM, Dave Airlie wrote:
> >
> > I did however get a flashback in google to this:
> >
> > http://lists.fedoraproject.org/pipermail/scm-commits/2010-July/456636.html
> >
> > Linus don't think we ever did work out why that worke
From: Julia Lawall
Most of the error handling code in this function frees the buffers
kcmd_addr, kvb_addr, and kbox_addr allocated at the beginning of this
function. These two branches are changed to do the same.
Signed-off-by: Julia Lawall
---
drivers/gpu/drm/savage/savage_state.c |5 ++
From: Julia Lawall
The function radeon_cs_parser_init is only called from two places, in
drivers/gpu/drm/radeon/radeon_cs.c and drivers/gpu/drm/radeon/r600_cs.c.
In each case, if the call fails another function is called that frees all
of the kdata and dpage information in the chunks array. So t
On Sat, Mar 17, 2012 at 14:40:04 -0400, Matt Turner wrote:
> On Sat, Mar 17, 2012 at 2:35 PM, Jakob Bornecrantz wrote:
> > With my limited knowledge of automake I'm going to have
> > to NACK this patch. Most of these programs are used during
> > driver bring up to test things out, often we also m
On Sunday, March 18, 2012, Hugh Dickins wrote:
> Added Rafael to the Cc: Rafael, we're pondering over one or more of these
> recurrent threads about corruption after resume, seemingly related to i915.
Thanks for letting me know. :-)
I actually have a confirmation that the issue isn't present if t
Am Sonntag, den 18.03.2012, 11:54 +0100 schrieb Paul Menzel:
> Am Donnerstag, den 15.03.2012, 11:08 +0100 schrieb Thomas Gleixner:
> > On Thu, 15 Mar 2012, Paul Menzel wrote:
>
> > > Am Donnerstag, den 15.03.2012, 00:35 +0100 schrieb Carsten Emde:
> > > > Einfaches Textdokument-Anlage
> > > > (dri
Am Donnerstag, den 15.03.2012, 11:08 +0100 schrieb Thomas Gleixner:
> On Thu, 15 Mar 2012, Paul Menzel wrote:
> > Am Donnerstag, den 15.03.2012, 00:35 +0100 schrieb Carsten Emde:
> > > Einfaches Textdokument-Anlage
> > > (drivers-gpu-drm-allow-to-load-edid-firmware.patch)
> >
> > somehow your mes
From: Dave Airlie
DisplayPort spec v1.1a, Table 2-52.
Same commit to Intel by ajax in 9962c9252e46eda7058067cbe73bdf1ed74b0d37
Signed-off-by: Dave Airlie
---
drivers/gpu/drm/radeon/atombios_dp.c |3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/drivers/gpu/drm/radeon/
70 matches
Mail list logo
