On Fri, Aug 10, 2012 at 2:32 PM, Daniel Vetter wrote:
> On Fri, Aug 10, 2012 at 04:57:43PM +0200, Maarten Lankhorst wrote:
>> Documentation says that code requiring dma-buf should add it to
>> select, so inline fallbacks are not going to be used. A link error
>> will make it obvious what went wron
On 08/11/2012 03:31 AM, Paul Menzel wrote:
> As a side note, could you also mention the patch iteration in the tag,
> that means [PATCH vN] so that I know what is the latest version. That
> would be great.
Can you (or anyone else reading this) point me to how to do this with
git send-email?
> I w
Hey,
Op 11-08-12 17:14, Rob Clark schreef:
> On Fri, Aug 10, 2012 at 3:29 PM, Daniel Vetter wrote:
>> On Fri, Aug 10, 2012 at 04:57:52PM +0200, Maarten Lankhorst wrote:
>>> A dma-fence can be attached to a buffer which is being filled or consumed
>>> by hw, to allow userspace to pass the buffer w
Op 10-08-12 21:57, Daniel Vetter schreef:
> On Fri, Aug 10, 2012 at 04:57:58PM +0200, Maarten Lankhorst wrote:
>> This type of fence can be used with hardware synchronization for simple
>> hardware that can block execution until the condition
>> (dma_buf[offset] - value) >= 0 has been met.
>>
>> A
On Sat, Aug 11, 2012 at 10:14:40AM -0500, Rob Clark wrote:
> On Fri, Aug 10, 2012 at 3:29 PM, Daniel Vetter wrote:
> > On Fri, Aug 10, 2012 at 04:57:52PM +0200, Maarten Lankhorst wrote:
> >> +
> >> + if (!ret) {
> >> + cb->base.flags = 0;
> >> + cb->base.func = __dma_fe
Alex Deucher (2):
radeon: add some missing evergreen pci ids
radeon: add some new SI pci ids
Chris Wilson (1):
intel: Bail gracefully if we encounter an unknown Intel device
Cooper Yuan (1):
libdrm/exynos: padding gem_mmap structure to 64-bit aligned
Damien Lespiau (1):
Dear Ian,
Am Freitag, den 10.08.2012, 13:44 -0500 schrieb Ian Pilcher:
[…]
> diff --git a/Documentation/EDID/edid_quirks.txt
> b/Documentation/EDID/edid_quirks.txt
> new file mode 100644
> index 000..256ded0
> --- /dev/null
> +++ b/Documentation/EDID/edid_quirks.txt
[…]
> +Overview
> +==
On Sat, Aug 11, 2012 at 06:00:46PM +0200, Maarten Lankhorst wrote:
> Op 11-08-12 17:14, Rob Clark schreef:
> > On Fri, Aug 10, 2012 at 3:29 PM, Daniel Vetter wrote:
> >>> +/**
> >>> + * dma_fence_signal - signal completion of a fence
> >>> + * @fence: the fence to signal
> >>> + *
> >>> + * All re
Dear Ian,
Am Samstag, den 11.08.2012, 10:38 -0500 schrieb Ian Pilcher:
> On 08/11/2012 03:31 AM, Paul Menzel wrote:
> > As a side note, could you also mention the patch iteration in the tag,
> > that means [PATCH vN] so that I know what is the latest version. That
> > would be great.
>
> Can you
Hello Adam,
On Fri, Jul 20, 2012 at 12:34:21PM +1000, Dave Airlie wrote:
> > Samsung 2443BW is 1920x1200 but reports 1920x1080 in the EDID. Attached
> > is a proof-of-concept implementation of a quirk. It works on my i686 PC.
...
> > This implementation matches the wrong mode by size. Other approa
On Sat, Aug 11, 2012 at 2:22 PM, Daniel Vetter wrote:
>> >> +
>> >> +/**
>> >> + * dma_fence_wait - wait for a fence to be signaled
>> >> + *
>> >> + * @fence: [in]The fence to wait on
>> >> + * @intr:[in]if true, do an interruptible wait
>> >> + * @timeout: [in]absolute time for
Updated patch set, based on Paul's feedback.
* Separate user-defined quirks stuff from new HDMI-related quirks
* (Hopefully) improve documentation
Also continuing to explore the wonders of git format-patch and git
send-email.
Ian Pilcher (2):
drm: Add user-defined EDID quirks capability
dr
Add the ability for users to define their own EDID quirks via a
module parameter or sysfs attribute.
Signed-off-by: Ian Pilcher
Acked-by: Paul Menzel
---
Documentation/EDID/edid_quirks.txt | 126 ++
drivers/gpu/drm/drm_drv.c | 2 +
drivers/gpu/drm/drm_edid.c | 500 +++
Add EDID quirk flags to disable HDMI audio and HDMI InfoFrames.
Add quirk for LG L246WP.
Signed-off-by: Ian Pilcher
Acked-by: Paul Menzel
---
drivers/gpu/drm/drm_edid.c | 24
1 file changed, 24 insertions(+)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm
Dear Ian,
thank you for your patch! The best thing is, I just asked Adam on
#intel-gfx the other day, if it was possible to have infrastructure to
test quirks without having to patch and build the Linux kernel. Nice!
As a side note, could you also mention the patch iteration in the tag,
that mea
Loongson has SWIOTLB configured, if without this patch kernel
compilation fails.
Signed-off-by: Huacai Chen
Signed-off-by: Hongliang Tao
Signed-off-by: Hua Yan
Cc: dri-devel@lists.freedesktop.org
---
drivers/gpu/drm/radeon/radeon_ttm.c |4
1 files changed, 4 insertions(+), 0 deletions
Signed-off-by: Huacai Chen
Signed-off-by: Hongliang Tao
Signed-off-by: Hua Yan
Cc: dri-devel@lists.freedesktop.org
---
drivers/gpu/drm/drm_vm.c |2 +-
drivers/gpu/drm/ttm/ttm_bo_util.c |2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/drm_vm.
Signed-off-by: Huacai Chen
Signed-off-by: Hongliang Tao
Signed-off-by: Hua Yan
Cc: dri-devel@lists.freedesktop.org
---
include/drm/drm_sarea.h |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/include/drm/drm_sarea.h b/include/drm/drm_sarea.h
index ee5389d..1d1a858 10064
From: Forest Bond
This causes the pipe to be forced off prior to initial mode set, which
roughly mirrors the behavior of the i915 driver. It fixes initial mode
setting on my Intel DN2800MT (Cedarview) board. Without it, mode
setting triggers an out-of-range error from the monitor for most modes
From: Forest Bond
Signed-off-by: Forest Bond
---
drivers/gpu/drm/gma500/cdv_intel_display.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/gpu/drm/gma500/cdv_intel_display.c
b/drivers/gpu/drm/gma500/cdv_intel_display.c
index a68509b..883a9f3 100644
--- a/driv
From: Forest Bond
This is set when setting DPMS on and off, but it isn't checked anywhere,
so just remove it.
Signed-off-by: Forest Bond
---
drivers/gpu/drm/gma500/cdv_intel_display.c |2 --
drivers/gpu/drm/gma500/psb_intel_drv.h |1 -
2 files changed, 0 insertions(+), 3 deletions(
Dear Linux folks,
where should I report the following warning
warning: (DRM_RADEON_KMS && DRM_I915 && STUB_POULSBO && FB_BACKLIGHT &&
USB_APPLEDISPLAY && FB_OLPC_DCON && ASUS_LAPTOP && SONY_LAPTOP && THINKPAD_ACPI
&& EEEPC_LAPTOP && ACPI_CMPC && SAMSUNG_Q10 && APPLE_GMUX) selects
BACK
On Fri, Aug 10, 2012 at 3:29 PM, Daniel Vetter wrote:
> On Fri, Aug 10, 2012 at 04:57:52PM +0200, Maarten Lankhorst wrote:
>> A dma-fence can be attached to a buffer which is being filled or consumed
>> by hw, to allow userspace to pass the buffer without waiting to another
>> device. For example
https://bugs.freedesktop.org/show_bug.cgi?id=45018
--- Comment #118 from Alexandre Demers
2012-08-11 04:49:31 UTC ---
Reproduced again with exactly the setup Alex told me to use (kernel
3.6-rc1+Jerome's patch v4 and latest mesa containing Christian's fix). To
reproduce, I clicked repeatedly on A
/drm_sysfs.c
> +++ b/drivers/gpu/drm/drm_sysfs.c
> @@ -84,6 +84,11 @@ static CLASS_ATTR_STRING(version, S_IRUGO,
> __stringify(CORE_PATCHLEVEL) " "
> CORE_DATE);
>
> +static CLASS_ATTR(edid_quirks_size, 0400, drm_edid_quirks_size_show, 0);
> +
> +static CLASS_ATTR(edid_quirks, 0600, drm_edid_quirks_show,
> + drm_edid_quirks_store);
> +
> /**
> * drm_sysfs_create - create a struct drm_sysfs_class structure
> * @owner: pointer to the module that is to "own" this struct drm_sysfs_class
> @@ -113,10 +118,22 @@ struct class *drm_sysfs_create(struct module *owner,
> char *name)
> if (err)
> goto err_out_class;
>
> + err = class_create_file(class, &class_attr_edid_quirks_size);
> + if (err)
> + goto err_out_version;
> +
> + err = class_create_file(class, &class_attr_edid_quirks);
> + if (err)
> + goto err_out_quirks_size;
> +
> class->devnode = drm_devnode;
>
> return class;
>
> +err_out_quirks_size:
> + class_remove_file(class, &class_attr_edid_quirks_size);
> +err_out_version:
> + class_remove_file(class, &class_attr_version.attr);
> err_out_class:
> class_destroy(class);
> err_out:
> @@ -132,6 +149,8 @@ void drm_sysfs_destroy(void)
> {
> if ((drm_class == NULL) || (IS_ERR(drm_class)))
> return;
> + class_remove_file(drm_class, &class_attr_edid_quirks);
> + class_remove_file(drm_class, &class_attr_edid_quirks_size);
> class_remove_file(drm_class, &class_attr_version.attr);
> class_destroy(drm_class);
> drm_class = NULL;
> diff --git a/include/drm/drmP.h b/include/drm/drmP.h
> index d6b67bb..c947f3e 100644
> --- a/include/drm/drmP.h
> +++ b/include/drm/drmP.h
> @@ -1501,6 +1501,7 @@ extern unsigned int drm_debug;
>
> extern unsigned int drm_vblank_offdelay;
> extern unsigned int drm_timestamp_precision;
> +extern char *drm_edid_quirks;
>
> extern struct class *drm_class;
> extern struct proc_dir_entry *drm_proc_root;
> @@ -1612,6 +1613,15 @@ void drm_gem_vm_open(struct vm_area_struct *vma);
> void drm_gem_vm_close(struct vm_area_struct *vma);
> int drm_gem_mmap(struct file *filp, struct vm_area_struct *vma);
>
> + /* EDID support (drm_edid.c) */
> +void drm_edid_quirks_param_process(void);
> +ssize_t drm_edid_quirks_size_show(struct class *class,
> + struct class_attribute *attr, char *buf);
> +ssize_t drm_edid_quirks_show(struct class *class, struct class_attribute
> *attr,
> + char *buf);
> +ssize_t drm_edid_quirks_store(struct class *class, struct class_attribute
> *attr,
> + const char *buf, size_t count);
> +
> #include "drm_global.h"
>
> static inline void
> diff --git a/include/drm/drm_edid.h b/include/drm/drm_edid.h
> index 0cac551..713229b 100644
> --- a/include/drm/drm_edid.h
> +++ b/include/drm/drm_edid.h
> @@ -202,11 +202,18 @@ struct detailed_timing {
> #define DRM_EDID_FEATURE_PM_SUSPEND (1 << 6)
> #define DRM_EDID_FEATURE_PM_STANDBY (1 << 7)
>
> +union edid_display_id {
> + struct {
> + __be16 mfg_id;
> + __le16 prod_code;
> + } __attribute__((packed)) s;
> + u32 u;
> +};
> +
> struct edid {
> u8 header[8];
> /* Vendor & product info */
> - u8 mfg_id[2];
> - u8 prod_code[2];
> + union edid_display_id display_id;
> u32 serial; /* FIXME: byte order */
> u8 mfg_week;
> u8 mfg_year;
> @@ -242,8 +249,6 @@ struct edid {
> u8 checksum;
> } __attribute__((packed));
>
> -#define EDID_PRODUCT_ID(e) ((e)->prod_code[0] | ((e)->prod_code[1] << 8))
> -
> struct drm_encoder;
> struct drm_connector;
> struct drm_display_mode;
Thanks again for that great patch. With the comments addressed above you
can add my acknowledgment.
Acked-by: Paul Menzel
I am going to try to test that patch too for a Philips and LG TV [2].
Thanks,
Paul
[2] https://bugs.freedesktop.org/show_bug.cgi?id=26294
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20120811/f5056002/attachment-0001.pgp>
Loongson has SWIOTLB configured, if without this patch kernel
compilation fails.
Signed-off-by: Huacai Chen
Signed-off-by: Hongliang Tao
Signed-off-by: Hua Yan
Cc: dri-devel at lists.freedesktop.org
---
drivers/gpu/drm/radeon/radeon_ttm.c |4
1 files changed, 4 insertions(+), 0 deleti
Signed-off-by: Huacai Chen
Signed-off-by: Hongliang Tao
Signed-off-by: Hua Yan
Cc: dri-devel at lists.freedesktop.org
---
drivers/gpu/drm/drm_vm.c |2 +-
drivers/gpu/drm/ttm/ttm_bo_util.c |2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/drm_
Signed-off-by: Huacai Chen
Signed-off-by: Hongliang Tao
Signed-off-by: Hua Yan
Cc: dri-devel at lists.freedesktop.org
---
include/drm/drm_sarea.h |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/include/drm/drm_sarea.h b/include/drm/drm_sarea.h
index ee5389d..1d1a858 10
Am Freitag, den 06.07.2012, 17:39 +0200 schrieb Paul Menzel:
> Am Freitag, den 06.07.2012, 17:34 +0200 schrieb Paul Menzel:
> > Am Donnerstag, den 05.07.2012, 08:58 -0400 schrieb Alex Deucher:
> > > On Wed, Jul 4, 2012 at 10:11 AM, Paul Menzel wrote:
> > > > Am Mittwoch, den 04.07.2012, 09:55 -0400
From: Forest Bond
This causes the pipe to be forced off prior to initial mode set, which
roughly mirrors the behavior of the i915 driver. It fixes initial mode
setting on my Intel DN2800MT (Cedarview) board. Without it, mode
setting triggers an out-of-range error from the monitor for most modes
From: Forest Bond
Signed-off-by: Forest Bond
---
drivers/gpu/drm/gma500/cdv_intel_display.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/gpu/drm/gma500/cdv_intel_display.c
b/drivers/gpu/drm/gma500/cdv_intel_display.c
index a68509b..883a9f3 100644
--- a/driv
From: Forest Bond
This is set when setting DPMS on and off, but it isn't checked anywhere,
so just remove it.
Signed-off-by: Forest Bond
---
drivers/gpu/drm/gma500/cdv_intel_display.c |2 --
drivers/gpu/drm/gma500/psb_intel_drv.h |1 -
2 files changed, 0 insertions(+), 3 deletions(
Merge: bf44ce8 5d299f3
Author: Linus Torvalds
Date: Wed Aug 8 20:06:43 2012 +0300
Thanks,
Paul
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally
On Fri, Aug 10, 2012 at 3:29 PM, Daniel Vetter wrote:
> On Fri, Aug 10, 2012 at 04:57:52PM +0200, Maarten Lankhorst wrote:
>> A dma-fence can be attached to a buffer which is being filled or consumed
>> by hw, to allow userspace to pass the buffer without waiting to another
>> device. For example
On Fri, Aug 10, 2012 at 2:32 PM, Daniel Vetter wrote:
> On Fri, Aug 10, 2012 at 04:57:43PM +0200, Maarten Lankhorst wrote:
>> Documentation says that code requiring dma-buf should add it to
>> select, so inline fallbacks are not going to be used. A link error
>> will make it obvious what went wron
On 08/11/2012 03:31 AM, Paul Menzel wrote:
> As a side note, could you also mention the patch iteration in the tag,
> that means [PATCH vN] so that I know what is the latest version. That
> would be great.
Can you (or anyone else reading this) point me to how to do this with
git send-email?
> I w
Hey,
Op 11-08-12 17:14, Rob Clark schreef:
> On Fri, Aug 10, 2012 at 3:29 PM, Daniel Vetter wrote:
>> On Fri, Aug 10, 2012 at 04:57:52PM +0200, Maarten Lankhorst wrote:
>>> A dma-fence can be attached to a buffer which is being filled or consumed
>>> by hw, to allow userspace to pass the buffer w
Op 10-08-12 21:57, Daniel Vetter schreef:
> On Fri, Aug 10, 2012 at 04:57:58PM +0200, Maarten Lankhorst wrote:
>> This type of fence can be used with hardware synchronization for simple
>> hardware that can block execution until the condition
>> (dma_buf[offset] - value) >= 0 has been met.
>>
>> A
On Sat, Aug 11, 2012 at 10:14:40AM -0500, Rob Clark wrote:
> On Fri, Aug 10, 2012 at 3:29 PM, Daniel Vetter wrote:
> > On Fri, Aug 10, 2012 at 04:57:52PM +0200, Maarten Lankhorst wrote:
> >> +
> >> + if (!ret) {
> >> + cb->base.flags = 0;
> >> + cb->base.func = __dma_fe
Alex Deucher (2):
radeon: add some missing evergreen pci ids
radeon: add some new SI pci ids
Chris Wilson (1):
intel: Bail gracefully if we encounter an unknown Intel device
Cooper Yuan (1):
libdrm/exynos: padding gem_mmap structure to 64-bit aligned
Damien Lespiau (1):
el.org/?p=linux/kernel/git/torvalds/linux.git;a=blob;f=drivers/gpu/drm/drm_edid.c;h=a8743c399e83234c976ebdb4b471542a0645c42d;hb=HEAD
[2]
https://en.wikipedia.org/wiki/Bitwise_operation#Shifts_in_C.2C_C.2B.2B.2C_C.23
[?]
Thanks,
Paul
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20120811/0f054575/attachment.pgp>
On Sat, Aug 11, 2012 at 06:00:46PM +0200, Maarten Lankhorst wrote:
> Op 11-08-12 17:14, Rob Clark schreef:
> > On Fri, Aug 10, 2012 at 3:29 PM, Daniel Vetter wrote:
> >>> +/**
> >>> + * dma_fence_signal - signal completion of a fence
> >>> + * @fence: the fence to signal
> >>> + *
> >>> + * All re
7095/git-how-to-split-up-a-commit-buried-in-history
[5] http://git.661346.n2.nabble.com/How-to-split-a-big-commit-td6238260.html
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20120811/e694b7c4/attachment-0001.pgp>
x-diff
Size: 1294 bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20120811/61baa9dd/attachment.patch>
-- next part --
A non-text attachment was scrubbed...
Name: 0002-drm-Add-quirk-for-Samsung-SyncMaster-2443BW.patch
Type:
On Sat, Aug 11, 2012 at 2:22 PM, Daniel Vetter wrote:
>> >> +
>> >> +/**
>> >> + * dma_fence_wait - wait for a fence to be signaled
>> >> + *
>> >> + * @fence: [in]The fence to wait on
>> >> + * @intr:[in]if true, do an interruptible wait
>> >> + * @timeout: [in]absolute time for
Updated patch set, based on Paul's feedback.
* Separate user-defined quirks stuff from new HDMI-related quirks
* (Hopefully) improve documentation
Also continuing to explore the wonders of git format-patch and git
send-email.
Ian Pilcher (2):
drm: Add user-defined EDID quirks capability
dr
Add the ability for users to define their own EDID quirks via a
module parameter or sysfs attribute.
Signed-off-by: Ian Pilcher
Acked-by: Paul Menzel
---
Documentation/EDID/edid_quirks.txt | 126 ++
drivers/gpu/drm/drm_drv.c | 2 +
drivers/gpu/drm/drm_edid.c | 500 +++
Add EDID quirk flags to disable HDMI audio and HDMI InfoFrames.
Add quirk for LG L246WP.
Signed-off-by: Ian Pilcher
Acked-by: Paul Menzel
---
drivers/gpu/drm/drm_edid.c | 24
1 file changed, 24 insertions(+)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm
Hi,
While looking at the kernel DRM code, I've noticed that in many places
we kmalloc() memory to store the raw EDID information, whether it be
from a DDC adapter, or loaded from firmware.
Nowhere can I find where this memory is freed. It seems in several
places that we assign it into connector-
We use __fls() to find the most significant bit. Using that, the
loop can be avoided. A second trick is to use the behaviour of the
rotate instructions to expand the range of the unsigned int to float
conversion to the full 32 bits in a branchless way.
The routine is now exact up to 2^24. Above
We use __fls() to find the most significant bit. Using that, the
loop can be avoided. A second trick is to use the behaviour of the
rotate instructions to expand the range of the unsigned int to float
conversion to the full 32 bits in a branchless way.
The routine is now exact up to 2^24. Above
Remove the copy of i2f() in r600_blit_kms.c
We rename the function to something longer now that it is a global
symbol. This reduces the likelyhood of unintended clashes later.
This might be a candidate for inclusion inside general drm infrastructure.
However, at the moment only the radeon driver
This allows gcc to fold duplicate calls into a single call. Since
the current users do actually call it multiple times with the
same arguments, this is an obvious win.
---
drivers/gpu/drm/radeon/r600_blit.c |2 +-
drivers/gpu/drm/radeon/r600_blit_shaders.h |2 +-
2 files changed,
Name: RADEON_I2C.diff
Type: text/x-patch
Size: 398 bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20120811/380ef43a/attachment-0001.bin>
54 matches
Mail list logo