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

2012-08-17 Thread Damien Lespiau
From: Damien Lespiau damien.lesp...@intel.com Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- 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

[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

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

2012-09-12 Thread Damien Lespiau
From: Damien Lespiau damien.lesp...@intel.com 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

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

2012-09-12 Thread Damien Lespiau
From: Damien Lespiau damien.lesp...@intel.com 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 damien.lesp...@intel.com --- drivers/gpu/drm/drm_crtc.c | 32

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

2012-09-12 Thread Damien Lespiau
From: Damien Lespiau damien.lesp...@intel.com 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 damien.lesp...@intel.com --- drivers/gpu/drm/i915/intel_drv.h

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

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

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

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

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 damien.lesp...@intel.com 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 damien.lesp...@intel.com --- drivers/gpu/drm

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

2012-09-27 Thread Damien Lespiau
From: Damien Lespiau damien.lesp...@intel.com 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

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

2012-09-27 Thread Damien Lespiau
From: Damien Lespiau damien.lesp...@intel.com 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 damien.lesp...@intel.com --- drivers/gpu/drm/i915/intel_drv.h

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

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

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

2012-09-27 Thread Damien Lespiau
From: Damien Lespiau damien.lesp...@intel.com 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

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

2012-10-09 Thread Damien Lespiau
From: Damien Lespiau damien.lesp...@intel.com 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 damien.lesp...@intel.com --- drivers/gpu/drm/drm_modes.c |1 + 1 files changed, 1 insertions(+), 0

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

2012-12-04 Thread Damien Lespiau
From: Damien Lespiau damien.lesp...@intel.com Instead of just printing status updated from 1 to 2, make those enum numbers immediately readable. Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/drm/drm_crtc_helper.c | 17 +++-- 1 file changed, 15 insertions

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

2012-12-10 Thread Damien Lespiau
From: Damien Lespiau damien.lesp...@intel.com 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 damien.lesp...@intel.com --- drivers/gpu/drm/drm_crtc_helper.c

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

2012-12-10 Thread Damien Lespiau
From: Damien Lespiau damien.lesp...@intel.com 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

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 dan...@ffwll.ch wrote: On Tue, Dec 04, 2012 at 04:00:17PM +, Damien Lespiau wrote: From: Damien Lespiau damien.lesp...@intel.com Instead of just printing status updated from 1 to 2, make those enum numbers immediately readable. Signed-off

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

2013-02-13 Thread Damien Lespiau
Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- 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

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

2013-03-08 Thread Damien Lespiau
missed something. FWIW, I posted a similar patch (without moving the variable around) on intel-gfx yesterday, so: Reviewed-by: Damien Lespiau damien.lesp...@intel.com -- Damien ___ dri-devel mailing list dri-devel@lists.freedesktop.org http

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 daniel.vet...@ffwll.ch wrote: On Wed, Mar 13, 2013 at 9:28 PM, Daniel Vetter dan...@ffwll.ch wrote: On Tue, Mar 12, 2013 at 09:07:46AM +, Chris Wilson wrote: On Mon, Mar 11, 2013 at

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

[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 damien.lesp...@intel.com Reviewed-by: Rodrigo Vivi rodrigo.v...@gmail.com --- drivers/gpu/drm/drm_modes.c | 1 + 1 file changed, 1 insertion(+) diff

[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 damien.lesp...@intel.com Reviewed-by: Jesse Barnes jbar...@virtuousgeek.org --- drivers/gpu/drm/drm_crtc_helper.c

[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 damien.lesp...@intel.com --- drivers/gpu

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

2013-05-08 Thread Damien Lespiau
From: Lespiau, Damien damien.lesp...@intel.com Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- 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

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

2013-05-08 Thread Damien Lespiau
From: Chris Cummins christopher.e.cumm...@intel.com 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,

[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 damien.lesp...@intel.com Reviewed-by: Jesse Barnes jbar

[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 damien.lesp...@intel.com --- drivers/gpu

[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 ch...@chris-wilson.co.uk Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- 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

[edid-decode] Decode HDMI 1.4 4k VICs

2013-07-30 Thread Damien Lespiau
Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- edid-decode.c | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/edid-decode.c b/edid-decode.c index 9840db6..f74bbe4 100644 --- a/edid-decode.c +++ b/edid-decode.c @@ -625,7 +625,7 @@ cea_hdmi_block

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

2013-08-05 Thread Damien Lespiau
Reding thierry.red...@avionic-design.de Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- 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

Re: [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. A source device shall set A0=1 when

Re: [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. A source device shall set A0=1 when

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 thierry.red...@avionic-design.de Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/video/hdmi.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/video/hdmi.c b/drivers

[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 thierry.red...@avionic-design.de Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/video/hdmi.c | 43 +++ include

[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 thierry.red...@avionic-design.de Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- include/linux/hdmi.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/include/linux/hdmi.h b/include/linux/hdmi.h index

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

2013-08-06 Thread Damien Lespiau
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 damien.lesp...@intel.com --- drivers/video/hdmi.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/video/hdmi.c b/drivers/video

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

2013-08-06 Thread Damien Lespiau
Reding thierry.red...@avionic-design.de Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- 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

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

2013-08-06 Thread Damien Lespiau
(Ville Syrjälä) Signed-off-by: Damien Lespiau damien.lesp...@intel.com Signed-off-by: Paulo Zanoni paulo.r.zanoni at intel.com Signed-off-by: Thierry Reding thierry.reding at avionic-design.de --- drivers/gpu/drm/i915/intel_drv.h | 4 +- drivers/gpu/drm/i915/intel_hdmi.c | 106

[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ä ville.syrj...@linux.intel.com Signed-off-by: Damien Lespiau damien.lesp...@intel.com Signed-off-by: Paulo Zanoni paulo.r.zanoni at intel.com Signed-off-by: Thierry Reding thierry.reding at avionic-design.de --- drivers/gpu/drm/i915/intel_sdvo.c | 38

[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ä ville.syrj...@linux.intel.com Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/drm/drm_edid.c | 2 -- 1

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

2013-08-06 Thread Damien Lespiau
Syrjälä) Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- 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

[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ä ville.syrj...@linux.intel.com Signed-off-by: Damien

[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ä ville.syrj...@linux.intel.com Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/drm/i915/intel_drv.h | 61

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

2013-08-06 Thread Damien Lespiau
Suggested-by: Ville Syrjälä ville.syrj...@linux.intel.com Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- 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

Re: [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 it gives the buffer

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 damien.lesp...@intel.com --- 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

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

2013-08-07 Thread Damien Lespiau
Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- 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

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

2013-08-07 Thread Damien Lespiau
Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- 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

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

2013-08-07 Thread Damien Lespiau
Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- 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

[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 damien.lesp...@intel.com --- 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

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

2013-08-07 Thread Damien Lespiau
Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- 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

[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 damien.lesp...@intel.com --- edid-decode.c | 67 +++ 1 file changed, 67 insertions(+) diff --git a/edid-decode.c b/edid-decode.c index 5061228..083ddd9 100644

HDMI 4k support

2013-08-07 Thread Damien Lespiau
merged (and in 2.21.14) Author: Damien Lespiau damien.lesp...@intel.com 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

[PATCH 1/2] drm/edid: Fix add_cea_modes() style issues

2013-08-07 Thread Damien Lespiau
A few styles issues have creept in here, fix them before touching this code again. Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/drm/drm_edid.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm

[PATCH 2/2] drm/edid: Parse the HDMI CEA block and look for 4k modes

2013-08-07 Thread Damien Lespiau
HDMI 1.4 adds 4 4k x 2k modes in the the CEA vendor specific block. With this commit, we now parse this block and expose the 4k modes that we find there. Signed-off-by: Damien Lespiau damien.lesp...@intel.com Tested-by: Cancan Feng cancan.f...@intel.com Bugzilla: https://bugs.freedesktop.org

HDMI 4k support v2

2013-08-13 Thread Damien Lespiau
Following up the first instance of this series: http://lists.freedesktop.org/archives/dri-devel/2013-August/043125.html Here is a v2 with Ville's review pass and a few additions: - Alternate clock modes for 4k resolutions - HDMI vendor specific infoframe support in drivers/video/hdmi.c -

[PATCH 01/12] drm: Don't export drm_find_cea_extension() any more

2013-08-13 Thread Damien Lespiau
This function is only used inside drm_edid.c. Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/drm/drm_edid.c | 5 ++--- include/drm/drm_crtc.h | 1 - 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c

[PATCH 02/12] drm/edid: Fix add_cea_modes() style issues

2013-08-13 Thread Damien Lespiau
A few styles issues have crept in here, fix them before touching this code again. v2: constify arguments that can be (Ville Syrjälä) Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/drm/drm_edid.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff

[PATCH 03/12] drm/edid: Parse the HDMI CEA block and look for 4k modes

2013-08-13 Thread Damien Lespiau
is relative to the end of the required fields of the HDMI VSDB (Ville Syrjälä) Signed-off-by: Damien Lespiau damien.lesp...@intel.com Tested-by: Cancan Feng cancan.f...@intel.com Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=67030 --- drivers/gpu/drm/drm_edid.c | 124

[PATCH 04/12] drm: Add support for alternate clocks of 4k modes

2013-08-13 Thread Damien Lespiau
Suggested-by: Ville Syrjälä ville.syrj...@linux.intel.com Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/drm/drm_edid.c | 68 ++ 1 file changed, 62 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers

[PATCH 05/12] video/hdmi: Don't let the user of this API create invalid infoframes

2013-08-13 Thread Damien Lespiau
To set the active aspect ratio value in the AVI infoframe today, you not only have to set the active_aspect field, but also the active_info_valid bit. Out of the 1 user of this API, we had 100% misuse, forgetting the _valid bit. This was fixed in: Author: Damien Lespiau damien.lesp...@intel.com

[PATCH 06/12] video/hdmi: Derive the bar data valid bit from the bar data fields

2013-08-13 Thread Damien Lespiau
Just like: Author: Damien Lespiau damien.lesp...@intel.com Date: Mon Aug 12 11:53:24 2013 +0100 video/hdmi: Don't let the user of this API create invalid infoframes But this time for the horizontal/vertical bar data present bits. Signed-off-by: Damien Lespiau damien.lesp

[PATCH 07/12] video/hdmi: Introduce helpers for the HDMI vendor specific infoframe

2013-08-13 Thread Damien Lespiau
Provide the programming model than the other infoframe types. The generic _pack() function can't handle those yet as we need to move the vendor OUI in the generic hdmi_vendor_infoframe structure to know which kind of vendor infoframe we are dealing with. Signed-off-by: Damien Lespiau damien.lesp

[PATCH 08/12] gpu: host1x: Port the HDMI vendor infoframe code the common helpers

2013-08-13 Thread Damien Lespiau
-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/host1x/drm/hdmi.c | 24 1 file changed, 4 insertions(+), 20 deletions(-) diff --git a/drivers/gpu/host1x/drm/hdmi.c b/drivers/gpu/host1x/drm/hdmi.c index 01097da..b548918 100644 --- a/drivers/gpu/host1x/drm/hdmi.c +++ b

[PATCH 09/12] drm/edid: Move HDMI_IDENTIFIER to hdmi.h

2013-08-13 Thread Damien Lespiau
, but details..) Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/drm/drm_edid.c | 1 - include/linux/hdmi.h | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index ec47eb8..9a07a33 100644 --- a/drivers/gpu

[PATCH 10/12] video/hdmi: Hook the HDMI vendor infoframe with the generic _pack()

2013-08-13 Thread Damien Lespiau
of this API has been ported in: Author: Damien Lespiau damien.lesp...@intel.com Date: Mon Aug 12 18:08:37 2013 +0100 gpu: host1x: Port the HDMI vendor infoframe code the common helpers Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/video/hdmi.c | 45

[PATCH 11/12] drm: Add a helper to forge HDMI vendor infoframes

2013-08-13 Thread Damien Lespiau
This can then be used by DRM drivers to setup their vendor infoframes. Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/drm/drm_edid.c | 36 include/drm/drm_edid.h | 4 2 files changed, 40 insertions(+) diff --git a/drivers

[PATCH 12/12] drm/i915/hdmi: Write HDMI vendor specific infoframes

2013-08-13 Thread Damien Lespiau
With all the common infoframe bits now in place, we can finally write the vendor specific infoframes in our driver. Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/drm/i915/i915_reg.h | 2 ++ drivers/gpu/drm/i915/intel_hdmi.c | 28 2 files

HDMI 4k support v3

2013-08-14 Thread Damien Lespiau
Follow up on v2: http://lists.freedesktop.org/archives/dri-devel/2013-August/043604.html With the quick and nice reviews from Ville and Simon, it's time for a v3: - Fix embarrassing hmdi typo - Fix the sending of vendor specific infoframes for side-by-side half modes - Smaller changes

[PATCH 01/12] drm: Don't export drm_find_cea_extension() any more

2013-08-14 Thread Damien Lespiau
This function is only used inside drm_edid.c. Signed-off-by: Damien Lespiau damien.lesp...@intel.com Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com --- drivers/gpu/drm/drm_edid.c | 5 ++--- include/drm/drm_crtc.h | 1 - 2 files changed, 2 insertions(+), 4 deletions(-) diff --git

[PATCH 02/12] drm/edid: Fix add_cea_modes() style issues

2013-08-14 Thread Damien Lespiau
A few styles issues have crept in here, fix them before touching this code again. v2: constify arguments that can be (Ville Syrjälä) v3: constify, but better (Ville Syrjälä) Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/drm/drm_edid.c | 12 +++- 1 file changed

[PATCH 03/12] drm/edid: Parse the HDMI CEA block and look for 4k modes

2013-08-14 Thread Damien Lespiau
is relative to the end of the required fields of the HDMI VSDB (Ville Syrjälä) v3: Fix 'Unknow' typo (Simon Farnsworth) Signed-off-by: Damien Lespiau damien.lesp...@intel.com Tested-by: Cancan Feng cancan.f...@intel.com Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=67030 Reviewed-by: Simon

[PATCH 04/12] drm: Add support for alternate clocks of 4k modes

2013-08-14 Thread Damien Lespiau
v2: Fix hmdi typo (Simon Farnsworth, Ville Syrjälä) Suggested-by: Ville Syrjälä ville.syrj...@linux.intel.com Signed-off-by: Damien Lespiau damien.lesp...@intel.com Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com Reviewed-by: Simon Farnsworth simon.farnswo...@onelan.co.uk --- drivers

[PATCH 05/12] video/hdmi: Don't let the user of this API create invalid infoframes

2013-08-14 Thread Damien Lespiau
To set the active aspect ratio value in the AVI infoframe today, you not only have to set the active_aspect field, but also the active_info_valid bit. Out of the 1 user of this API, we had 100% misuse, forgetting the _valid bit. This was fixed in: Author: Damien Lespiau damien.lesp...@intel.com

[PATCH 06/12] video/hdmi: Derive the bar data valid bit from the bar data fields

2013-08-14 Thread Damien Lespiau
Just like: Author: Damien Lespiau damien.lesp...@intel.com Date: Mon Aug 12 11:53:24 2013 +0100 video/hdmi: Don't let the user of this API create invalid infoframes But this time for the horizontal/vertical bar data present bits. Signed-off-by: Damien Lespiau damien.lesp

[PATCH 08/12] gpu: host1x: Port the HDMI vendor infoframe code the common helpers

2013-08-14 Thread Damien Lespiau
message with the note about the length (Ville Syrjälä) Cc: Thierry Reding thierry.red...@gmail.com Cc: Terje Bergström tbergst...@nvidia.com Cc: linux-te...@vger.kernel.org Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/host1x/drm/hdmi.c | 24

[PATCH 07/12] video/hdmi: Introduce helpers for the HDMI vendor specific infoframe

2013-08-14 Thread Damien Lespiau
(half), hmdi typo, pack 3D_Ext_Data (Ville Syrjälä) Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/video/hdmi.c | 88 include/linux/hdmi.h | 26 2 files changed, 114 insertions(+) diff --git a/drivers

[PATCH 10/12] video/hdmi: Hook the HDMI vendor infoframe with the generic _pack()

2013-08-14 Thread Damien Lespiau
of this API has been ported in: Author: Damien Lespiau damien.lesp...@intel.com Date: Mon Aug 12 18:08:37 2013 +0100 gpu: host1x: Port the HDMI vendor infoframe code the common helpers v2: Change oui to be an unsigned int (Ville Syrjälä) Signed-off-by: Damien Lespiau damien.lesp

[PATCH 09/12] drm/edid: Move HDMI_IDENTIFIER to hdmi.h

2013-08-14 Thread Damien Lespiau
, but details..) Signed-off-by: Damien Lespiau damien.lesp...@intel.com Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com --- drivers/gpu/drm/drm_edid.c | 1 - include/linux/hdmi.h | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm

[PATCH 11/12] drm: Add a helper to forge HDMI vendor infoframes

2013-08-14 Thread Damien Lespiau
This can then be used by DRM drivers to setup their vendor infoframes. v2: Fix hmdi typo (Simon Farnsworth) Signed-off-by: Damien Lespiau damien.lesp...@intel.com Reviewed-by: Simon Farnsworth simon.farnswo...@onelan.co.uk Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com --- drivers/gpu

[PATCH 12/12] drm/i915/hdmi: Write HDMI vendor specific infoframes

2013-08-14 Thread Damien Lespiau
With all the common infoframe bits now in place, we can finally write the vendor specific infoframes in our driver. Signed-off-by: Damien Lespiau damien.lesp...@intel.com Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com --- drivers/gpu/drm/i915/i915_reg.h | 2 ++ drivers/gpu/drm/i915

Re: [Intel-gfx] [PATCH] i915: Add a Kconfig option to turn on i915.preliminary_hw_support by default

2013-08-15 Thread Damien Lespiau
similar in mind for this on other little parameters/debug features). Reviewed-by: Damien Lespiau damien.lesp...@intel.com It doesn't apply cleanly to drm-intel/drm-intel-nightly which is the preferred branch to base patches on. Daniel might fix this himself as this is rather trivial to solve

Re: [PATCH 09/12] drm/edid: Move HDMI_IDENTIFIER to hdmi.h

2013-08-19 Thread Damien Lespiau
On Thu, Aug 15, 2013 at 05:12:00PM +0200, Thierry Reding wrote: On Wed, Aug 14, 2013 at 06:19:12PM +0100, Damien Lespiau wrote: [...] +#define HDMI_IDENTIFIER 0x000c03 HDMI_IDENTIFIER sounds really generic. Perhaps HDMI_INFOFRAME_OUI_HDMI? This identifier is not infoframe specific, it's

HDMI 4k support v4

2013-08-19 Thread Damien Lespiau
Follow up on v3: http://lists.freedesktop.org/archives/dri-devel/2013-August/043696.html Changes between v3 and v4: - Future proof the sending of 3D_Ext_Data - Renamed struct hdmi_hdmi_infoframe to hdmi_vendor_infoframe (by, in turn, renaming the hdmi_vendor_infoframe enum to

[PATCH 01/14] drm: Don't export drm_find_cea_extension() any more

2013-08-19 Thread Damien Lespiau
This function is only used inside drm_edid.c. Signed-off-by: Damien Lespiau damien.lesp...@intel.com Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com --- drivers/gpu/drm/drm_edid.c | 5 ++--- include/drm/drm_crtc.h | 1 - 2 files changed, 2 insertions(+), 4 deletions(-) diff --git

[PATCH 02/14] drm/edid: Fix add_cea_modes() style issues

2013-08-19 Thread Damien Lespiau
A few styles issues have crept in here, fix them before touching this code again. v2: constify arguments that can be (Ville Syrjälä) v3: constify, but better (Ville Syrjälä) Signed-off-by: Damien Lespiau damien.lesp...@intel.com Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com

[PATCH 03/14] drm/edid: Parse the HDMI CEA block and look for 4k modes

2013-08-19 Thread Damien Lespiau
is relative to the end of the required fields of the HDMI VSDB (Ville Syrjälä) v3: Fix 'Unknow' typo (Simon Farnsworth) Signed-off-by: Damien Lespiau damien.lesp...@intel.com Tested-by: Cancan Feng cancan.f...@intel.com Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=67030 Reviewed-by: Simon

[PATCH 04/14] drm: Add support for alternate clocks of 4k modes

2013-08-19 Thread Damien Lespiau
v2: Fix hmdi typo (Simon Farnsworth, Ville Syrjälä) Suggested-by: Ville Syrjälä ville.syrj...@linux.intel.com Signed-off-by: Damien Lespiau damien.lesp...@intel.com Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com Reviewed-by: Simon Farnsworth simon.farnswo...@onelan.co.uk --- drivers

[PATCH 05/14] video/hdmi: Don't let the user of this API create invalid infoframes

2013-08-19 Thread Damien Lespiau
To set the active aspect ratio value in the AVI infoframe today, you not only have to set the active_aspect field, but also the active_info_valid bit. Out of the 1 user of this API, we had 100% misuse, forgetting the _valid bit. This was fixed in: Author: Damien Lespiau damien.lesp...@intel.com

[PATCH 06/14] video/hdmi: Derive the bar data valid bit from the bar data fields

2013-08-19 Thread Damien Lespiau
Just like: Author: Damien Lespiau damien.lesp...@intel.com Date: Mon Aug 12 11:53:24 2013 +0100 video/hdmi: Don't let the user of this API create invalid infoframes But this time for the horizontal/vertical bar data present bits. Signed-off-by: Damien Lespiau damien.lesp

[PATCH 07/14] video/hdmi: Introduce helpers for the HDMI vendor specific infoframe

2013-08-19 Thread Damien Lespiau
(half), hmdi typo, pack 3D_Ext_Data (Ville Syrjälä) v3: Future proof the sending of 3D_Ext_Data (Ville Syrjälä), Fix multi-lines comment style (Thierry Reding) Signed-off-by: Damien Lespiau damien.lesp...@intel.com Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com --- drivers/video

[PATCH 08/14] gpu: host1x: Port the HDMI vendor infoframe code the common helpers

2013-08-19 Thread Damien Lespiau
message with the note about the length (Ville Syrjälä) Cc: Thierry Reding thierry.red...@gmail.com Cc: Terje Bergström tbergst...@nvidia.com Cc: linux-te...@vger.kernel.org Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/host1x/drm/hdmi.c | 24

[PATCH 10/14] video/hdmi: Hook the HDMI vendor infoframe with the generic _pack()

2013-08-19 Thread Damien Lespiau
of this API has been ported in: Author: Damien Lespiau damien.lesp...@intel.com Date: Mon Aug 12 18:08:37 2013 +0100 gpu: host1x: Port the HDMI vendor infoframe code the common helpers v2: Change oui to be an unsigned int (Ville Syrjälä) Signed-off-by: Damien Lespiau damien.lesp

[PATCH 09/14] drm/edid: Move HDMI_IDENTIFIER to hdmi.h

2013-08-19 Thread Damien Lespiau
, but details..) Signed-off-by: Damien Lespiau damien.lesp...@intel.com Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com --- drivers/gpu/drm/drm_edid.c | 1 - include/linux/hdmi.h | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm

[PATCH 11/14] video/hdmi: Use hdmi_vendor_infoframe for the HDMI specific infoframe

2013-08-19 Thread Damien Lespiau
-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/host1x/drm/hdmi.c | 6 +++--- drivers/video/hdmi.c | 25 + include/linux/hdmi.h | 15 --- 3 files changed, 24 insertions(+), 22 deletions(-) diff --git a/drivers/gpu/host1x/drm/hdmi.c b

  1   2   3   4   5   6   >