[RFC] Stereo 3D modes support

2012-09-12 Thread Damien Lespiau
Hi, This series introduces stereo 3D modes support and is split in 3 chunks: 1. 3 kernel patches to parse the 3D_present flag of the HDMI CEA vendor block, to expose 3D formats flags in modes and to add a new property on connectors supporting stereo 3D, 2. Sync the new mode flags in libdrm

[PATCH 1/3] drm: Parse the HDMI cea vendor block for 3D present

2012-09-12 Thread Damien Lespiau
From: Damien Lespiau For now, let's just look at the 3D_present flag of the CEA HDMI vendor block to detect if the sink supports a small list of then mandatory 3D formats. See the HDMI 1.4a 3D extraction for detail: http://www.hdmi.org/manufacturer/specification.aspx Signed-off-by: D

[PATCH 2/3] drm: Add Stereo 3D properties

2012-09-12 Thread Damien Lespiau
From: Damien Lespiau The "select 3D mode" property can be connected to connectors to signal user space that 3D framebuffers can be scanned out to the said connector. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_crtc.c | 32 include/drm/

[PATCH 3/3] drm/i915: Add HDMI vendor info frame support

2012-09-12 Thread Damien Lespiau
From: Damien Lespiau When scanning out a 3D framebuffer, send the corresponding infoframe to the HDMI sink. See http://www.hdmi.org/manufacturer/specification.aspx for details. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/i915/intel_drv.h | 14 drivers/gpu/drm/i915

[PATCH] Sync the mode flags for the stereo 3D formats

2012-09-12 Thread Damien Lespiau
From: Damien Lespiau Signed-off-by: Damien Lespiau --- include/drm/drm_mode.h | 35 +-- xf86drmMode.h | 35 +-- 2 files changed, 42 insertions(+), 28 deletions(-) diff --git a/include/drm/drm_mode.h b/include/drm

[PATCH 1/2] lib: Dump information about the supported 3D stereo formats

2012-09-12 Thread Damien Lespiau
From: Damien Lespiau When dumping the details of a mode, let's add the 3D formats the mode supports. Signed-off-by: Damien Lespiau --- lib/drmtest.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/lib/drmtest.c b/lib/drmtest.c index 8df9797..4d5a67c 1

[PATCH 2/2] tests/testdisplay: Test the stereo 3D modes

2012-09-12 Thread Damien Lespiau
From: Damien Lespiau Now that modes have flags to describe which 3d formats the sink supports, it's time to test them. The new test cycles through the supported 3D formats and paint 3D stereoscopic images taken from publicly available samples: http://www.quantumdata.com/apps/3D/sample_BM

[PATCH] drm: Make the HPD status updates debug logs more readable

2012-12-04 Thread Damien Lespiau
From: Damien Lespiau Instead of just printing "status updated from 1 to 2", make those enum numbers immediately readable. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_crtc_helper.c | 17 +++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/drive

[PATCH 1/2] drm: Make the HPD status updates debug logs more readable

2012-12-10 Thread Damien Lespiau
From: Damien Lespiau Instead of just printing "status updated from 1 to 2", make those enum numbers immediately readable. v2: Also patch output_poll_execute() (Daniel Vetter) Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_crtc_helper.c | 22 ++ 1 file c

[PATCH 2/2] drm: Don't prune modes loudly when a connector is disconnected

2012-12-10 Thread Damien Lespiau
From: Damien Lespiau drm_helper_probe_single_connector_modes() is responsible for pruning the previously detected modes on a disconnected connector. We don't really need to log, again, the full list of modes that used to be valid when connected. Signed-off-by: Damien Lespiau --- driver

Re: [PATCH] drm: Make the HPD status updates debug logs more readable

2012-12-10 Thread Damien Lespiau
On Tue, Dec 4, 2012 at 5:39 PM, Daniel Vetter wrote: > On Tue, Dec 04, 2012 at 04:00:17PM +0000, Damien Lespiau wrote: >> From: Damien Lespiau >> >> Instead of just printing "status updated from 1 to 2", make those enum >> numbers immediately readable. >&

[PATCH] drm: Remove two unused fields from struct drm_display_mode

2012-08-17 Thread Damien Lespiau
From: Damien Lespiau Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_modes.c |3 --- include/drm/drm_crtc.h |2 -- 2 files changed, 0 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c index b7adb4a..28637c1 100644 --- a

[RFC] Stereo 3D modes support

2012-09-12 Thread Damien Lespiau
Hi, This series introduces stereo 3D modes support and is split in 3 chunks: 1. 3 kernel patches to parse the 3D_present flag of the HDMI CEA vendor block, to expose 3D formats flags in modes and to add a new property on connectors supporting stereo 3D, 2. Sync the new mode flags in libdrm

[PATCH 1/3] drm: Parse the HDMI cea vendor block for 3D present

2012-09-12 Thread Damien Lespiau
From: Damien Lespiau For now, let's just look at the 3D_present flag of the CEA HDMI vendor block to detect if the sink supports a small list of then mandatory 3D formats. See the HDMI 1.4a 3D extraction for detail: http://www.hdmi.org/manufacturer/specification.aspx Signed-off-by: D

[PATCH 2/3] drm: Add Stereo 3D properties

2012-09-12 Thread Damien Lespiau
From: Damien Lespiau The "select 3D mode" property can be connected to connectors to signal user space that 3D framebuffers can be scanned out to the said connector. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_crtc.c | 32 include/drm/

[PATCH 3/3] drm/i915: Add HDMI vendor info frame support

2012-09-12 Thread Damien Lespiau
From: Damien Lespiau When scanning out a 3D framebuffer, send the corresponding infoframe to the HDMI sink. See http://www.hdmi.org/manufacturer/specification.aspx for details. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/i915/intel_drv.h | 14 drivers/gpu/drm/i915

[PATCH] Sync the mode flags for the stereo 3D formats

2012-09-12 Thread Damien Lespiau
From: Damien Lespiau Signed-off-by: Damien Lespiau --- include/drm/drm_mode.h | 35 +-- xf86drmMode.h | 35 +-- 2 files changed, 42 insertions(+), 28 deletions(-) diff --git a/include/drm/drm_mode.h b/include/drm

[PATCH 1/2] lib: Dump information about the supported 3D stereo formats

2012-09-12 Thread Damien Lespiau
From: Damien Lespiau When dumping the details of a mode, let's add the 3D formats the mode supports. Signed-off-by: Damien Lespiau --- lib/drmtest.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/lib/drmtest.c b/lib/drmtest.c index 8df9797..4d5a67c 1

Stereo 3D modes support v2

2012-09-27 Thread Damien Lespiau
This series is the second revison of: http://lists.freedesktop.org/archives/intel-gfx/2012-September/020457.html It changes the way 3d modes are exposed to user-space: - An "expose 3D modes" property can be installed on connectors that support stereo 3D - User space can indicate through that

[PATCH 1/3] drm: Add an "expose 3d modes" property

2012-09-27 Thread Damien Lespiau
From: Damien Lespiau The "expose 3D modes" property can be attached to connectors to allow user space to indicate it can deal with 3D modes and that the drm driver should expose those 3D modes. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_c

[PATCH 2/3] drm: Parse the HDMI cea vendor block for 3D present

2012-09-27 Thread Damien Lespiau
From: Damien Lespiau For now, let's just look at the 3D_present flag of the CEA HDMI vendor block to detect if the sink supports a small list of then mandatory 3D formats. See the HDMI 1.4a 3D extraction for detail: http://www.hdmi.org/manufacturer/specification.aspx Signed-off-by: D

[PATCH 3/3] drm/i915: Add HDMI vendor info frame support

2012-09-27 Thread Damien Lespiau
From: Damien Lespiau When scanning out a 3D framebuffer, send the corresponding infoframe to the HDMI sink. See http://www.hdmi.org/manufacturer/specification.aspx for details. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/i915/intel_drv.h | 14 +++ drivers/gpu/drm/i915

[PATCH 1/2] lib: Dump information about the supported 3D stereo formats

2012-09-27 Thread Damien Lespiau
From: Damien Lespiau When dumping the details of a mode, let's add the 3D formats the mode supports. Signed-off-by: Damien Lespiau --- lib/drmtest.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/lib/drmtest.c b/lib/drmtest.c index 8df9797..4d5a67c 1

[PATCH 2/2] tests/testdisplay: Test the stereo 3D modes

2012-09-27 Thread Damien Lespiau
From: Damien Lespiau Now that modes have flags to describe which 3d formats the sink supports, it's time to test them. The new test cycles through the supported 3D formats and paint 3D stereoscopic images taken from publicly available samples: http://www.quantumdata.com/apps/3D/sample_BM

[PATCH] drm: Add missing break in the command line mode parsing code

2012-10-09 Thread Damien Lespiau
From: Damien Lespiau As we parse the string given on the command line one char at a time, it seems that we do want a break at every case. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_modes.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/gpu/drm

[PATCH] drm: Fix a typo in the struct drm_plane_funcs documentation

2013-02-13 Thread Damien Lespiau
Signed-off-by: Damien Lespiau --- include/drm/drm_crtc.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h index e90c8dc..b248382 100644 --- a/include/drm/drm_crtc.h +++ b/include/drm/drm_crtc.h @@ -659,7 +659,7 @@ struct

Re: [PATCH] drm/i915: Fix missing variable initilization

2013-03-08 Thread Damien Lespiau
27;ing Paulo in case I've missed > something. FWIW, I posted a similar patch (without moving the variable around) on intel-gfx yesterday, so: Reviewed-by: Damien Lespiau -- Damien ___ dri-devel mailing list dri-devel@lists.freedesktop.org http://li

Re: [Intel-gfx] [PATCH v3] drm/i915: bounds check execbuffer relocation count

2013-03-15 Thread Damien Lespiau
On Thu, Mar 14, 2013 at 12:32:00PM -0700, Kees Cook wrote: > On Thu, Mar 14, 2013 at 9:57 AM, Daniel Vetter wrote: > > On Wed, Mar 13, 2013 at 9:28 PM, Daniel Vetter wrote: > >> On Tue, Mar 12, 2013 at 09:07:46AM +, Chris Wilson wrote: > >>> On Mon, Mar 11, 2013 at 05:31:45PM -0700, Kees Cook

Some lost drm patches

2013-05-08 Thread Damien Lespiau
I just collected a few sad patches crying in the dark alley of the Internet that dri-devel can sometimes be. A few of them even have a r-b tag by benevolent and fatherly figures. All they want is to find a new home! -- Damien ___ dri-devel mailing list

[PATCH 1/5] drm: Add missing break in the command line mode parsing code

2013-05-08 Thread Damien Lespiau
As we parse the string given on the command line one char at a time, it seems that we do want a break at every case. Signed-off-by: Damien Lespiau Reviewed-by: Rodrigo Vivi --- drivers/gpu/drm/drm_modes.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/drm_modes.c b/drivers

[PATCH 2/5] drm: Make the HPD status updates debug logs more readable

2013-05-08 Thread Damien Lespiau
Instead of just printing "status updated from 1 to 2", make those enum numbers immediately readable. v2: Also patch output_poll_execute() (Daniel Vetter) Signed-off-by: Damien Lespiau Reviewed-by: Jesse Barnes --- drivers/gpu/drm/drm_crtc_helper.c | 22 ++ 1 fi

[PATCH 3/5] drm: Don't prune modes loudly when a connector is disconnected

2013-05-08 Thread Damien Lespiau
drm_helper_probe_single_connector_modes() is responsible for pruning the previously detected modes on a disconnected connector. We don't really need to log, again, the full list of modes that used to be valid when connected. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_crtc_hel

[PATCH 4/5] drm: Fix a typo in the struct drm_plane_funcs documentation

2013-05-08 Thread Damien Lespiau
From: "Lespiau, Damien" Signed-off-by: Damien Lespiau --- include/drm/drm_crtc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h index e3e0d65..23fb185 100644 --- a/include/drm/drm_crtc.h +++ b/include/drm/drm_crtc.h

[PATCH 5/5] drm: Use names of ioctls in debug traces

2013-05-08 Thread Damien Lespiau
From: Chris Cummins The intention here is to make the output of dmesg with full verbosity a bit easier for a human to parse. This commit transforms: [drm:drm_ioctl], pid=699, cmd=0x6458, nr=0x58, dev 0xe200, auth=1 [drm:drm_ioctl], pid=699, cmd=0xc010645b, nr=0x5b, dev 0xe200, auth=1 [drm:drm_io

Some lost drm patches v2

2013-05-10 Thread Damien Lespiau
Updated series with: - Reviews applied, - Drop a patch that would conflict with Ville's latest changes, - Drop Chris (Cummins)'s patch as he rebased it on top of drm-next and sent it back himself, - Add a suggestion from Chris (Wilson) to not repeat the message in the poll handler. I did not d

[PATCH 1/4] drm: Add missing break in the command line mode parsing code

2013-05-10 Thread Damien Lespiau
As we parse the string given on the command line one char at a time, it seems that we do want a break at every case. Signed-off-by: Damien Lespiau Reviewed-by: Rodrigo Vivi --- drivers/gpu/drm/drm_modes.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/drm_modes.c b/drivers

[PATCH 2/4] drm: Make the HPD status updates debug logs more readable

2013-05-10 Thread Damien Lespiau
Instead of just printing "status updated from 1 to 2", make those enum numbers immediately readable. v2: Also patch output_poll_execute() (Daniel Vetter) v3: Use drm_get_connector_status_name (Ville Syrjälä) Signed-off-by: Damien Lespiau Reviewed-by: Jesse Barnes (for v1) --- drive

[PATCH 3/4] drm: Don't prune modes loudly when a connector is disconnected

2013-05-10 Thread Damien Lespiau
drm_helper_probe_single_connector_modes() is responsible for pruning the previously detected modes on a disconnected connector. We don't really need to log, again, the full list of modes that used to be valid when connected. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_crtc_hel

[PATCH 4/4] drm: Only print a debug message when the polled connector has changed

2013-05-10 Thread Damien Lespiau
Suggested-by: Chris Wilson Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_crtc_helper.c | 19 +-- 1 file changed, 13 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/drm_crtc_helper.c b/drivers/gpu/drm/drm_crtc_helper.c index 9085db6..ed1334e 100644 --- a

[PATCH] drm: Remove two unused fields from struct drm_display_mode

2012-08-17 Thread Damien Lespiau
From: Damien Lespiau Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_modes.c |3 --- include/drm/drm_crtc.h |2 -- 2 files changed, 0 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c index b7adb4a..28637c1 100644 --- a

[PATCH] drm: Make the HPD status updates debug logs more readable

2012-12-04 Thread Damien Lespiau
From: Damien Lespiau Instead of just printing "status updated from 1 to 2", make those enum numbers immediately readable. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_crtc_helper.c | 17 +++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/drive

[PATCH 1/2] drm: Make the HPD status updates debug logs more readable

2012-12-10 Thread Damien Lespiau
From: Damien Lespiau Instead of just printing "status updated from 1 to 2", make those enum numbers immediately readable. v2: Also patch output_poll_execute() (Daniel Vetter) Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_crtc_helper.c | 22 ++ 1 file c

[PATCH 2/2] drm: Don't prune modes loudly when a connector is disconnected

2012-12-10 Thread Damien Lespiau
From: Damien Lespiau drm_helper_probe_single_connector_modes() is responsible for pruning the previously detected modes on a disconnected connector. We don't really need to log, again, the full list of modes that used to be valid when connected. Signed-off-by: Damien Lespiau --- driver

[PATCH] drm: Make the HPD status updates debug logs more readable

2012-12-10 Thread Damien Lespiau
On Tue, Dec 4, 2012 at 5:39 PM, Daniel Vetter wrote: > On Tue, Dec 04, 2012 at 04:00:17PM +0000, Damien Lespiau wrote: >> From: Damien Lespiau >> >> Instead of just printing "status updated from 1 to 2", make those enum >> numbers immediately readable. >&

Re: [Intel-gfx] [PATCH] drm: add support for additional stereo 3D modes

2013-10-11 Thread Damien Lespiau
On Fri, Oct 11, 2013 at 02:12:14PM +0300, Ville Syrjälä wrote: > On Thu, Oct 10, 2013 at 02:19:15PM +0100, Thomas Wood wrote: > > +static int add_3d_struct_modes(struct drm_connector *connector, u16 > > structure, > > + const u8 *video_db, u8 video_len, u8 video_index) > >

Re: [PATCH] drm: add support for additional stereo 3D modes

2013-10-11 Thread Damien Lespiau
On Thu, Oct 10, 2013 at 02:19:15PM +0100, Thomas Wood wrote: > + if ((multi_present == 1 || multi_present == 2) && You could use the awesome binary literals gcc extension here and 0b01 and 0b10 to be even closer to the spec wording. There's a precedent in drivers/watchdog/sunxi_wdt.c! -- Da

Bumping libdrm's version right after a release

2013-10-11 Thread Damien Lespiau
Hi, Would anyone be opposed to bumping the version number of libdrm right after a release? That would allow us to be able to depend on new APIs that are not yet in a released libdrm. -- Damien ___ dri-devel mailing list dri-devel@lists.freedesktop.org

[PATCH 1/4] drm: Add support for CRTC primary planes

2014-03-03 Thread Damien Lespiau
On Thu, Feb 27, 2014 at 02:14:40PM -0800, Matt Roper wrote: > Allow drivers to provide a drm_plane structure corresponding to a CRTC's > primary plane. These planes will be included in the plane list for any > clients setting the DRM_CLIENT_CAP_EXPOSE_PRIMARY_PLANES capability bit. > > Signed-off

[PATCH 2/4] drm: Add plane type property

2014-03-03 Thread Damien Lespiau
On Thu, Feb 27, 2014 at 11:03:07PM -0500, Rob Clark wrote: > >> > @@ -1114,6 +1126,10 @@ int drm_plane_set_primary(struct drm_device *dev, > >> > struct drm_plane *plane, > >> > >> > >> fwiw, this comment probably belongs in #1/4 but: > >> > >> you probably don't need to introduce drm_plane_set_pr

[PATCH 1/4] drm: Add support for CRTC primary planes

2014-03-03 Thread Damien Lespiau
On Mon, Mar 03, 2014 at 09:45:53AM -0800, Matt Roper wrote: > On Mon, Mar 03, 2014 at 03:47:43PM +0000, Damien Lespiau wrote: > > On Thu, Feb 27, 2014 at 02:14:40PM -0800, Matt Roper wrote: > > > Allow drivers to provide a drm_plane structure corresponding to a CRTC's >

[PATCH] drm: Check if the allocation has succeeded before dereferencing newmode

2014-03-03 Thread Damien Lespiau
We allocate memory in drm_display_mode_from_vic_index() and use it without checking the pointer is valid. Fix that. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_edid.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index

[Intel-gfx] [PATCH 1/4] drm: Added plane alpha and color blending property

2014-03-20 Thread Damien Lespiau
On Sat, Mar 08, 2014 at 01:51:16PM +0530, sagar.a.kamble at intel.com wrote: > From: Sagar Kamble > > This patch creates a generic blending enum property. > Drivers may support subset of these values. > > Cc: airlied at linux.ie > Cc: dri-devel at lists.freedesktop.org > Cc: linux-kernel at vger

[Intel-gfx] [PATCH 1/4] drm: Added plane alpha and color blending property

2014-03-20 Thread Damien Lespiau
On Sat, Mar 08, 2014 at 01:51:16PM +0530, sagar.a.kamble at intel.com wrote: > From: Sagar Kamble > > This patch creates a generic blending enum property. > Drivers may support subset of these values. > > Cc: airlied at linux.ie > Cc: dri-devel at lists.freedesktop.org > Cc: linux-kernel at vger

[PATCH 01/11] drm: Refresh the explanation of debug categories

2014-03-24 Thread Damien Lespiau
That comment wasn't super-readable, so I tried to improve it: - Put the comment before the values it's documenting - Add a mention to PRIME - Reword things a bit to be a lighter read - Add a note about the option to set the debug value at run-time Signed-off-by: Damien Lespiau --- i

[PATCH 02/11] drm: Remove the unused (and unusable) DRM_LOG_MODE()

2014-03-24 Thread Damien Lespiau
This macro was trying to use the non existing DRM_UT_MODE debug category and looks like it should be covered by DRM_LOG_KMS(). Signed-off-by: Damien Lespiau --- include/drm/drmP.h | 6 -- 1 file changed, 6 deletions(-) diff --git a/include/drm/drmP.h b/include/drm/drmP.h index 97900b7

[PATCH 03/11] drm/exynos: Replace DRM_LOG_KMS() by DRM_DEBUG_KMS()

2014-03-24 Thread Damien Lespiau
There are only a few users of the DRM_LOG_KMS() macro. We can simplify the DRM code a bit by replacing them by DRM_DEBUG_KMS(). Cc: Inki Dae Signed-off-by: Damien Lespiau --- drivers/gpu/drm/exynos/exynos_drm_fbdev.c | 2 +- drivers/gpu/drm/exynos/exynos_drm_plane.c | 2 +- 2 files changed, 2

[PATCH 00/11] A few patches around DRM logging

2014-03-24 Thread Damien Lespiau
river-specific patches (why some of you are Cced in that series), I'd love if you could take the time to throw a Acked-by/Reviewed-by tag. Also, do you have any objection if the driver specific patch go through the DRM tree? should people judge that series worthwhile, of course. -- Damien Dami

[PATCH 04/11] drm/gma500: Replace DRM_LOG_KMS() by DRM_DEBUG_KMS()

2014-03-24 Thread Damien Lespiau
There are only a few users of the DRM_LOG_KMS() macro. We can simplify the DRM code a bit by replacing them by DRM_DEBUG_KMS(). Cc: Patrik Jakobsson Signed-off-by: Damien Lespiau --- drivers/gpu/drm/gma500/psb_intel_sdvo.c | 20 ++-- 1 file changed, 10 insertions(+), 10

[PATCH 07/11] drm: Remove the now unused DRM_LOG* macros

2014-03-24 Thread Damien Lespiau
Signed-off-by: Damien Lespiau --- include/drm/drmP.h | 19 --- 1 file changed, 19 deletions(-) diff --git a/include/drm/drmP.h b/include/drm/drmP.h index 1455e58..3055b36 100644 --- a/include/drm/drmP.h +++ b/include/drm/drmP.h @@ -228,30 +228,11 @@ int drm_err(const char *func

[PATCH 08/11] drm: Pull the test on drm_debug in the logging macros

2014-03-24 Thread Damien Lespiau
unlikely() as well for good measure. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_stub.c | 26 -- include/drm/drmP.h | 27 +++ 2 files changed, 27 insertions(+), 26 deletions(-) diff --git a/drivers/gpu/drm/drm_stub.c b/drivers/gpu/d

[PATCH 09/11] drm: drm_ut_debug_printk() isn't called with NULL anywmore

2014-03-24 Thread Damien Lespiau
The DRM_LOG* macros where the only sites where drm_ut_debug_printk was called with NULL arguments for prefix and function_name. Now that they are gone, we can remove that case. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_stub.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions

[PATCH 05/11] drm/i915: Replace DRM_LOG_KMS() by DRM_DEBUG_KMS()

2014-03-24 Thread Damien Lespiau
There are only a few users of the DRM_LOG_KMS() macro. We can simplify the DRM code a bit by replacing them by DRM_DEBUG_KMS(). Signed-off-by: Damien Lespiau --- drivers/gpu/drm/i915/dvo_ch7xxx.c | 4 ++-- drivers/gpu/drm/i915/dvo_ivch.c | 30 +++--- drivers/gpu/drm

[PATCH 06/11] staging: imx-drm: Replace DRM_LOG_KMS() by DRM_DEBUG_KMS()

2014-03-24 Thread Damien Lespiau
There are only a few users of the DRM_LOG_KMS() macro. We can simplify the DRM code a bit by replacing them by DRM_DEBUG_KMS(). Cc: Philipp Zabel Cc: Lucas Stach Signed-off-by: Damien Lespiau --- drivers/staging/imx-drm/ipuv3-plane.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH 10/11] drm: Remove the prefix argument of drm_ut_debug_printk()

2014-03-24 Thread Damien Lespiau
This is always DRM_NAME, so we can just make it part of the format string instead of asking prink to do it for us. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_stub.c | 6 ++ include/drm/drmP.h | 17 ++--- 2 files changed, 8 insertions(+), 15 deletions(-) diff

[PATCH 11/11] drm: Remove the ', ' after the function name in debug logs

2014-03-24 Thread Damien Lespiau
fo to have here, let's just say I'd happily review a patch removing it. If I don't get annoyed enough to submit a patch, that is. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_stub.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/drm_stub.

[Intel-gfx] [PATCH 1/3] drm/i915: Don't set mode_config's cursor size

2014-03-25 Thread Damien Lespiau
On Tue, Mar 25, 2014 at 07:23:26PM +0100, Daniel Vetter wrote: > Or we simply do this per-pixel format with one for each framebuffer plane, > i.e. > > struct drm_get_plane_fb_limits { > uint32_t plane_id; /* in */ > uint32_t fourcc; /* in */ > struct drm_plane_limits limits[MAX_F

[PATCH] drm: Specify a bit more the DRM_CAP_CURSOR_{WIDTH, HEIGHT} caps

2014-03-28 Thread Damien Lespiau
ursor size - other drivers fall back to returning 64x64 The common contract is to return a valid cursor size. Cc: Sagar Kamble Cc: Chris Wilson Cc: Alex Deucher Cc: Imre Deak Signed-off-by: Damien Lespiau --- include/uapi/drm/drm.h | 9 + 1 file changed, 9 insertions(+) diff --

Stereo 3D modes support v2

2012-09-27 Thread Damien Lespiau
This series is the second revison of: http://lists.freedesktop.org/archives/intel-gfx/2012-September/020457.html It changes the way 3d modes are exposed to user-space: - An "expose 3D modes" property can be installed on connectors that support stereo 3D - User space can indicate through that

[PATCH 1/3] drm: Add an "expose 3d modes" property

2012-09-27 Thread Damien Lespiau
From: Damien Lespiau The "expose 3D modes" property can be attached to connectors to allow user space to indicate it can deal with 3D modes and that the drm driver should expose those 3D modes. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_c

[PATCH 2/3] drm: Parse the HDMI cea vendor block for 3D present

2012-09-27 Thread Damien Lespiau
From: Damien Lespiau For now, let's just look at the 3D_present flag of the CEA HDMI vendor block to detect if the sink supports a small list of then mandatory 3D formats. See the HDMI 1.4a 3D extraction for detail: http://www.hdmi.org/manufacturer/specification.aspx Signed-off-by: D

[PATCH 3/3] drm/i915: Add HDMI vendor info frame support

2012-09-27 Thread Damien Lespiau
From: Damien Lespiau When scanning out a 3D framebuffer, send the corresponding infoframe to the HDMI sink. See http://www.hdmi.org/manufacturer/specification.aspx for details. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/i915/intel_drv.h | 14 +++ drivers/gpu/drm/i915

[PATCH 1/2] lib: Dump information about the supported 3D stereo formats

2012-09-27 Thread Damien Lespiau
From: Damien Lespiau When dumping the details of a mode, let's add the 3D formats the mode supports. Signed-off-by: Damien Lespiau --- lib/drmtest.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/lib/drmtest.c b/lib/drmtest.c index 8df9797..4d5a67c 1

[PATCH 2/2] tests/testdisplay: Test the stereo 3D modes

2012-09-27 Thread Damien Lespiau
From: Damien Lespiau Now that modes have flags to describe which 3d formats the sink supports, it's time to test them. The new test cycles through the supported 3D formats and paint 3D stereoscopic images taken from publicly available samples: http://www.quantumdata.com/apps/3D/sample_BM

[PATCH] drm: Don't generate invalid AVI infoframes for CEA modes

2013-08-05 Thread Damien Lespiau
rry Reding Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_edid.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 95d6f4b..8d1139f 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c @@ -3107,6 +310

[PATCH] drm: Don't generate invalid AVI infoframes for CEA modes

2013-08-05 Thread Damien Lespiau
On Mon, Aug 05, 2013 at 04:54:06PM +0300, Ville Syrj?l? wrote: > On Mon, Aug 05, 2013 at 02:36:47PM +0100, Damien Lespiau wrote: > > >From CEA-861: > > > > Data Byte 1, bit A0 indicates whether Active Format Data is present in > > Data Byte 2 bits R3 through R0.

[PATCH] drm: Don't generate invalid AVI infoframes for CEA modes

2013-08-05 Thread Damien Lespiau
On Mon, Aug 05, 2013 at 09:07:05PM +0300, Ville Syrj?l? wrote: > On Mon, Aug 05, 2013 at 02:36:47PM +0100, Damien Lespiau wrote: > > >From CEA-861: > > > > Data Byte 1, bit A0 indicates whether Active Format Data is present in > > Data Byte 2 bits R3 through R0.

Port the i915 HDMI infoframe code to the common infrastructure v2

2013-08-06 Thread Damien Lespiau
Re-posting the whole series because I forgot dri-devel in the first version and also added a few patches from the review. Version 2 of the series: http://lists.freedesktop.org/archives/intel-gfx/2013-August/031183.html With Ville's comments so far addressed. I've also added the already posted

[PATCH 01/12] video/hdmi: Replace the payload length by their defines

2013-08-06 Thread Damien Lespiau
Cc: Thierry Reding Reviewed-by: Ville Syrj?l? Signed-off-by: Damien Lespiau --- drivers/video/hdmi.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/video/hdmi.c b/drivers/video/hdmi.c index 4017833..dbd882f 100644 --- a/drivers/video/hdmi.c +++ b/drivers

[PATCH 03/12] video/hdmi: Add a macro to return the size of a full infoframe

2013-08-06 Thread Damien Lespiau
Cc: Thierry Reding Reviewed-by: Ville Syrj?l? Signed-off-by: Damien Lespiau --- include/linux/hdmi.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/include/linux/hdmi.h b/include/linux/hdmi.h index 0f3f82e..bc6743e 100644 --- a/include/linux/hdmi.h +++ b/include/linux/hdmi.h @@ -23,6

[PATCH 02/12] video/hdmi: Introduce a generic hdmi_infoframe union

2013-08-06 Thread Damien Lespiau
And a way to pack hdmi_infoframe generically. Cc: Thierry Reding Reviewed-by: Ville Syrj?l? Signed-off-by: Damien Lespiau --- drivers/video/hdmi.c | 43 +++ include/linux/hdmi.h | 17 + 2 files changed, 60 insertions(+) diff --git a

[PATCH 04/12] video/hmdi: Clear the whole incoming buffer, not just the infoframe size

2013-08-06 Thread Damien Lespiau
x27;s zero the whole incoming buffer as those extra bytes are also used to compute the ECC and need to be 0. Signed-off-by: Damien Lespiau --- drivers/video/hdmi.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/video/hdmi.c b/drivers/video/hdmi.c index f7

[PATCH 05/12] drm: Don't generate invalid AVI infoframes for CEA modes

2013-08-06 Thread Damien Lespiau
rry Reding Reviewed-by: Ville Syrj?l? Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_edid.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 95d6f4b..8d1139f 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edi

[PATCH 06/12] drm/i915/hdmi: Change the write_infoframe vfunc to take a buffer and a type

2013-08-06 Thread Damien Lespiau
mix signs (Ville Syrj?l?) Signed-off-by: Damien Lespiau Signed-off-by: Paulo Zanoni Signed-off-by: Thierry Reding --- drivers/gpu/drm/i915/intel_drv.h | 4 +- drivers/gpu/drm/i915/intel_hdmi.c | 106 -- 2 files changed, 59 insertions(+), 51 deletions(-) di

[PATCH 07/12] drm/i915/hdmi: Port the infoframe code to the common hdmi helpers

2013-08-06 Thread Damien Lespiau
Ville Syrj?l?) Signed-off-by: Damien Lespiau --- drivers/gpu/drm/i915/intel_hdmi.c | 82 +++ 1 file changed, 58 insertions(+), 24 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_hdmi.c b/drivers/gpu/drm/i915/intel_hdmi.c index ee67e23..455dfa7 100644

[PATCH 08/12] drm/i915/sdvo: Port the infoframe code to the shared infrastructure

2013-08-06 Thread Damien Lespiau
Reviewed-by: Ville Syrj?l? Signed-off-by: Damien Lespiau Signed-off-by: Paulo Zanoni Signed-off-by: Thierry Reding --- drivers/gpu/drm/i915/intel_sdvo.c | 38 -- 1 file changed, 20 insertions(+), 18 deletions(-) diff --git a/drivers/gpu/drm/i915

[PATCH 09/12] drm/i915: Remove the now obsolete infoframe definitions

2013-08-06 Thread Damien Lespiau
All the HDMI infoframe code has been ported to use video/hdmi.c, so it's time to say bye bye to this code. Reviewed-by: Ville Syrj?l? Signed-off-by: Damien Lespiau --- drivers/gpu/drm/i915/intel_drv.h | 61 --- drivers/gpu/drm/i915/intel_hdmi.c

[PATCH 10/12] drm: Handle the DBLCLK flag in the common infoframe helper

2013-08-06 Thread Damien Lespiau
Suggested-by: Ville Syrj?l? Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_edid.c| 3 +++ drivers/gpu/drm/i915/intel_hdmi.c | 3 --- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 8d1139f..a9c8980

[PATCH 11/12] drm: Set aspect ratio fields in the AVI infoframe even for non CEA modes

2013-08-06 Thread Damien Lespiau
I cannot find any evidence what we shouldn't try to set those fields when setting a non-CEA mode on an HDMI sink. So just kill that return. Suggested-by: Ville Syrj?l? Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_edid.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/driver

[PATCH 12/12] drm/i915/hmdi: Rename set_infoframe() to write_infoframe()

2013-08-06 Thread Damien Lespiau
set_frame() wraps the write_frame() vfunc. Be consistent and name the wrapping function like the vfunc being called. It's doubly confusing as we also have a set_infoframes() vfunc and set_infoframe() doesn't wrap it. Reviewed-by: Ville Syrj?l? Signed-off-by: Damien Lespiau --- drive

[Intel-gfx] [PATCH 04/12] video/hmdi: Clear the whole incoming buffer, not just the infoframe size

2013-08-07 Thread Damien Lespiau
On Wed, Aug 07, 2013 at 01:56:58PM +0300, Ville Syrj?l? wrote: > On Tue, Aug 06, 2013 at 08:32:16PM +0100, Damien Lespiau wrote: > > If the user if this API is providing a bigger buffer than the infoframe > > size, it could be for a could reason. For instance it could be because &

Some edid-decode patches

2013-08-07 Thread Damien Lespiau
A bit more context than the previous patch that was a bit alone. This series parses yet a bit more of the EDID: - The HDMI VICs in the HDMI vendor specific block (HDMI 1.4, 4k modes) - The CEA Video Capability Data Block It also includes 2 EDIDs of TVs that have those bits. -- Damien

[edid-decode 1/7] Print the resolutions next to the CEA VICs

2013-08-07 Thread Damien Lespiau
Signed-off-by: Damien Lespiau --- edid-decode.c | 84 +-- 1 file changed, 82 insertions(+), 2 deletions(-) diff --git a/edid-decode.c b/edid-decode.c index 9840db6..7515181 100644 --- a/edid-decode.c +++ b/edid-decode.c @@ -32,6 +32,8

[edid-decode 2/7] Add Skyworth 50E780U 50" edid (4k TV)

2013-08-07 Thread Damien Lespiau
Signed-off-by: Damien Lespiau --- data/skyworth-50e780u-hdmi | Bin 0 -> 256 bytes 1 file changed, 0 insertions(+), 0 deletions(-) create mode 100644 data/skyworth-50e780u-hdmi diff --git a/data/skyworth-50e780u-hdmi b/data/skyworth-50e780u-hdmi new file mode 100644 in

[edid-decode 3/7] Decode HDMI 1.4 4k VICs

2013-08-07 Thread Damien Lespiau
Signed-off-by: Damien Lespiau --- edid-decode.c | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/edid-decode.c b/edid-decode.c index 7515181..55e48a7 100644 --- a/edid-decode.c +++ b/edid-decode.c @@ -705,7 +705,7 @@ cea_hdmi_block(unsigned char *x) if

[edid-decode 4/7] Print the HDMI resolution next to the HDMI VICs

2013-08-07 Thread Damien Lespiau
Signed-off-by: Damien Lespiau --- edid-decode.c | 21 +++-- 1 file changed, 19 insertions(+), 2 deletions(-) diff --git a/edid-decode.c b/edid-decode.c index 55e48a7..5061228 100644 --- a/edid-decode.c +++ b/edid-decode.c @@ -659,6 +659,13 @@ cea_video_block(unsigned char *x

[edid-decode 5/7] Add the EDID of a Samsung TV that has a VCDB

2013-08-07 Thread Damien Lespiau
--- data/samsung-UE40D8000YU-hmdi | Bin 0 -> 256 bytes 1 file changed, 0 insertions(+), 0 deletions(-) create mode 100644 data/samsung-UE40D8000YU-hmdi diff --git a/data/samsung-UE40D8000YU-hmdi b/data/samsung-UE40D8000YU-hmdi new file mode 100644 index

[edid-decode 6/7] Add a small framework to decode fields generically

2013-08-07 Thread Damien Lespiau
Signed-off-by: Damien Lespiau --- edid-decode.c | 69 +-- 1 file changed, 67 insertions(+), 2 deletions(-) diff --git a/edid-decode.c b/edid-decode.c index 5061228..7aed3c6 100644 --- a/edid-decode.c +++ b/edid-decode.c @@ -32,8 +32,6

[edid-decode 7/7] Decode the Video Capability Data Block

2013-08-07 Thread Damien Lespiau
Signed-off-by: Damien Lespiau --- edid-decode.c | 39 +++ 1 file changed, 39 insertions(+) diff --git a/edid-decode.c b/edid-decode.c index 7aed3c6..58297c9 100644 --- a/edid-decode.c +++ b/edid-decode.c @@ -861,6 +861,44 @@ cea_hdmi_block(unsigned char *x

[edid-decode] Add a small framework to decode fields generically

2013-08-07 Thread Damien Lespiau
v2: Fix rebase fail that removed a necessary hunk Signed-off-by: Damien Lespiau --- edid-decode.c | 67 +++ 1 file changed, 67 insertions(+) diff --git a/edid-decode.c b/edid-decode.c index 5061228..083ddd9 100644 --- a/edid-decode.c

HDMI 4k support

2013-08-07 Thread Damien Lespiau
(and in 2.21.14) Author: Damien Lespiau Date: Wed Jul 31 18:50:51 2013 +0100 uxa/display: Keep the EDID blob around for the lifetime of an output * My "Use the TMDS maximum frequency to check mode dot clock" xserver series: http://lists.x.org/archiv

<    1   2   3   4   5   6   >