[PATCH 19/22] drm/i915: Use crtc_clock with the adjusted mode

2013-09-25 Thread Damien Lespiau
k. No functional change. v2: Rebased on drm-intel-queued-next Reviewed-by: Ville Syrjälä Signed-off-by: Damien Lespiau --- drivers/gpu/drm/i915/intel_crt.c | 2 +- drivers/gpu/drm/i915/intel_display.c | 34 +- drivers/gpu/drm/i915/intel_dp.c

[PATCH 20/22] drm/i915: Ask the DRM core do make stereo timings adjustements

2013-09-25 Thread Damien Lespiau
Syrjälä Signed-off-by: Damien Lespiau --- drivers/gpu/drm/i915/intel_display.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index 3c982a4..c25622d 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b

[PATCH 22/22] drm/i915: Allow stereo modes on HDMI

2013-09-25 Thread Damien Lespiau
Reviewed-by: Ville Syrjälä Signed-off-by: Damien Lespiau --- drivers/gpu/drm/i915/intel_hdmi.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/intel_hdmi.c b/drivers/gpu/drm/i915/intel_hdmi.c index 1a57758..6004f9c 100644 --- a/drivers/gpu/drm/i915/intel_hdmi.c +++ b

[PATCH 21/22] drm/i915: Prefer crtc_{h|v}display for pipe src dimensions

2013-09-25 Thread Damien Lespiau
Now that we ask to adjust the crtc timings for stereo modes, the correct pipe_src_w and pipe_src_h can be found in crtc_vdisplay and crtc_hdisplay. v2: Add comment about why pipe_src_w/h need to be set afert set_crtcinfo() (Daniel Vetter) Reviewed-by: Ville Syrjälä Signed-off-by: Damien

Upstreaming the stereo v6 series

2013-09-25 Thread Damien Lespiau
Hi, So this series looks like a good candidate to be merged in one tree. Beside the new 3d flags added to the mode structure, the other new API is the SET_CLIENT_CAP ioctl. It seems that this new ioctl could already be potentially useful for user space to tell us they want the "primary" plane exp

Code stereo layouts as an enum in the mode structure

2013-09-27 Thread Damien Lespiau
Daniel noticed that it wasn't very smart to keep using one bit per stereo layout now that we don't have to or them. It's a nice final touch to the new stereo mode API extension that we should fix before committing to it. -- Damien ___ dri-devel mailing

[PATCH 1/3] drm: Code stereo layouts as an enum rather than a bit field

2013-09-27 Thread Damien Lespiau
, reserving 32 values. Even with that reservation, we gain 3 bits from the previous encoding. The code adding the mandatory stereo modes needeed to be adapted as it was relying or being able to or stereo layouts together. Suggested-by: Daniel Vetter Signed-off-by: Damien Lespiau --- drivers/gpu/drm

[PATCH 2/3] drm: Revert "drm: Reject modes with more than 1 stereo flags set"

2013-09-27 Thread Damien Lespiau
Now that the coding of stereo layout has changed from a bit field to an enum, we need remove that check. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_crtc.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c index 807309f

[PATCH 3/3] drm: Reject stereo modes with an unknown layout

2013-09-27 Thread Damien Lespiau
The kernel shouldn't accept invalid modes, just say No. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_crtc.c | 3 +++ include/drm/drm_crtc.h | 2 ++ include/uapi/drm/drm_mode.h | 4 3 files changed, 9 insertions(+) diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gp

Re: [PATCH edid-decode 1/3] Add a missing comma to the list of CEA VICs

2013-09-27 Thread Damien Lespiau
On Fri, Sep 27, 2013 at 12:26:59PM +0100, Thomas Wood wrote: > Signed-off-by: Thomas Wood For the series (that fixes 2 embarassing bugs from me and 1 long standing one that wasn't parsing the last CEA VIC): Reviewed-by: Damien Lespiau --

Re: [PATCH 1/3] drm/edid: add drm_edid_duplicate

2013-09-27 Thread Damien Lespiau
On Fri, Sep 27, 2013 at 03:08:27PM +0300, Jani Nikula wrote: > We have some code duplication related to EDID duplication. Add a helper. git grep kmemdup.*edid drivers/gpu/drm/ also returns 3 hits in nouveau here, should anyone be bored. -- Damien ___ d

Re: [PATCH edid-decode 1/3] Add a missing comma to the list of CEA VICs

2013-09-27 Thread Damien Lespiau
On Fri, Sep 27, 2013 at 12:26:59PM +0100, Thomas Wood wrote: > Signed-off-by: Thomas Wood Thanks for the series, all pushed with the virtual nod from Ajax on IRC. -- Damien ___ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freede

[PATCH 1/5] drm: Remove unused variable in drm_global_item_ref()

2013-09-28 Thread Damien Lespiau
Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_global.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/drm_global.c b/drivers/gpu/drm/drm_global.c index f731116..3d2e91c 100644 --- a/drivers/gpu/drm/drm_global.c +++ b/drivers/gpu/drm/drm_global.c @@ -67,7 +67,6

[PATCH 2/5] drm: Remove unused variable in drm_prime_sg_to_page_addr_arrays()

2013-09-28 Thread Damien Lespiau
Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_prime.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/drm_prime.c b/drivers/gpu/drm/drm_prime.c index 276d470..56805c3 100644 --- a/drivers/gpu/drm/drm_prime.c +++ b/drivers/gpu/drm/drm_prime.c

[PATCH 3/5] drm: Remove unused variable in drm_fb_helper_pan_display()

2013-09-28 Thread Damien Lespiau
Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_fb_helper.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c index f6f6cc7..e69d26b 100644 --- a/drivers/gpu/drm/drm_fb_helper.c +++ b/drivers/gpu/drm/drm_fb_helper.c

[PATCH 4/5] drm: Remove unused variable in drm_pick_crtcs()

2013-09-28 Thread Damien Lespiau
Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_fb_helper.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c index e69d26b..f61ac23 100644 --- a/drivers/gpu/drm/drm_fb_helper.c +++ b/drivers/gpu/drm/drm_fb_helper.c

[PATCH 5/5] drm: Remove unused variables in drm_fb_helper_hotplug_event()

2013-09-28 Thread Damien Lespiau
Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_fb_helper.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c index f61ac23..21742a8 100644 --- a/drivers/gpu/drm/drm_fb_helper.c +++ b/drivers/gpu/drm

[libdrm] Stereo 3D v3

2013-09-30 Thread Damien Lespiau
v2 was: http://lists.freedesktop.org/archives/dri-devel/2013-September/045354.html Only one change, the stereo part for the flags is now an enum instead of being a bit field. The kernel series has been pushed to drm-intel, the libdrm patches should follow if there isn't any objection. -- Dami

[PATCH libdrm 1/3] drm: Synchronize the stereo 3D mode flags from the kernel headers

2013-09-30 Thread Damien Lespiau
v2: stereo layouts are now an enum (Daniel Vetter) Signed-off-by: Damien Lespiau --- include/drm/drm_mode.h | 39 +-- xf86drmMode.h | 38 -- 2 files changed, 49 insertions(+), 28 deletions(-) diff --git a/include

[PATCH libdrm 3/3] drm: Introduce a drmSetClientCap() wrapper

2013-09-30 Thread Damien Lespiau
That wraps around the new DRM_SET_CLIENT_CAP ioctl. v2: SET_CAP -> SET_CLIENT_CAP renaming Signed-off-by: Damien Lespiau --- xf86drm.c | 7 +++ xf86drm.h | 2 ++ 2 files changed, 9 insertions(+) diff --git a/xf86drm.c b/xf86drm.c index 4791a05..720952f 100644 --- a/xf86drm.c ++

[PATCH libdrm 2/3] drm: Sync the DRM_SET_CLIENT_CAP ioctl definition

2013-09-30 Thread Damien Lespiau
v2: SET_CAP -> SET_CLIENT_CAP renaming Signed-off-by: Damien Lespiau --- include/drm/drm.h | 16 1 file changed, 16 insertions(+) diff --git a/include/drm/drm.h b/include/drm/drm.h index f400642..725bf51 100644 --- a/include/drm/drm.h +++ b/include/drm/drm.h @@ -618,6 +618

Re: [libdrm] Stereo 3D v3

2013-10-01 Thread Damien Lespiau
On Mon, Sep 30, 2013 at 01:41:13PM +0100, Damien Lespiau wrote: > The kernel series has been pushed to drm-intel, the libdrm patches should > follow if there isn't any objection. Swiftly pushed. -- Damien ___ dri-devel mailing lis

[Intel-gfx] [PATCH 05/17] drm: Add atomic driver interface definitions for objects

2014-11-05 Thread Damien Lespiau
On Wed, Nov 05, 2014 at 06:04:59PM +0100, Daniel Vetter wrote: > On Wed, Nov 5, 2014 at 5:26 PM, Thierry Reding > wrote: > >> +struct drm_plane_state { > >> + struct drm_crtc *crtc; > >> + struct drm_framebuffer *fb; > >> + > >> + /* Signed dest location allows it to be partially off

[PATCH] Fix SIGSEGV in libdrm for heigth = 0 and width = 0

2014-11-20 Thread Damien Lespiau
On Fri, Nov 07, 2014 at 07:43:04PM +0100, Thomas Meyer wrote: > > drm_intel_gem_bo_free() crashes because the list bo_gem->vma_list is not > yet initialised, but the error path tries to free it. > > See also https://bugs.freedesktop.org/show_bug.cgi?id=75844 > > Reviewed-by: Chris Wilson > Sign

[PATCH 1/3] drm: Add a note to drm_property_create() about property lifetime

2014-10-31 Thread Damien Lespiau
Cc: Chandra Konduru Cc: Daniel Vetter Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_crtc.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c index 4081d7a..0f3c24c 100644 --- a/drivers/gpu/drm/drm_crtc.c +++ b/drivers/gpu

[PATCH 2/3] drm/i915: Don't destroy DRM properties in the driver

2014-10-31 Thread Damien Lespiau
When drm properties are created, they are added to mode_config.property_list, which is then used in drm_mode_config_cleanup() to destroy every single property created by the driver. Cc: Chandra Konduru Cc: Daniel Vetter Signed-off-by: Damien Lespiau --- drivers/gpu/drm/i915/intel_sdvo.c | 47

[PATCH 3/3] drm/gma500: Don't destroy DRM properties in the driver

2014-10-31 Thread Damien Lespiau
When drm properties are created, they are added to mode_config.property_list which is then used in drm_mode_config_cleanup() to destroy every single property created by the driver. Cc: Patrik Jakobsson Signed-off-by: Damien Lespiau --- drivers/gpu/drm/gma500/psb_intel_sdvo.c | 49

[PATCH] drm: Improve debug output for drm_wait_one_vblank

2014-09-15 Thread Damien Lespiau
On Mon, Sep 15, 2014 at 02:05:56PM +0200, Daniel Vetter wrote: > This replicates what we've done in i915 in > > commit 31e4b89acbd7b19c9a8557e6e660a583a0b97daa > Author: Damien Lespiau > Date: Mon Aug 18 13:51:00 2014 +0100 > > drm/i915: Print the pipe on which

Shareable bufmgr objects v4

2014-09-15 Thread Damien Lespiau
On Mon, Sep 15, 2014 at 02:12:25PM +0100, Chris Wilson wrote: > On Fri, Sep 12, 2014 at 01:48:34PM +0100, Lionel Landwerlin wrote: > > This is getting bigger than expected. Adding the locking that Chris > > suggested on IRC. > > > > Thanks for taking time to review Chris. > > I'm happy with this

[Intel-gfx] [PATCH] intel: Add support for userptr objects

2014-09-17 Thread Damien Lespiau
ns and uncertainty > about its usefulness. > > v2: Improved error handling in feature detection per review comments. > > Signed-off-by: Tvrtko Ursulin > --- Reviewed-by: Damien Lespiau Pushed a slightly modified version of this patch as libdrm now has explicit symb

[PATCH] intel: Don't leak the test page in an has_userptr() error path

2014-09-17 Thread Damien Lespiau
When handling the error on GEM_CLOSE, we weren't freeing the allocated page. Plug that. Signed-off-by: Damien Lespiau --- intel/intel_bufmgr_gem.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c index f378e5c..ce

[Intel-gfx] [PATCH] intel: Don't leak the test page in an has_userptr() error path

2014-09-19 Thread Damien Lespiau
On Fri, Sep 19, 2014 at 02:31:56PM +0100, Tvrtko Ursulin wrote: > > Reviewed-by: Tvrtko Ursulin > Thanks for the review, pushed the patch. -- Damien

[PATCH libdrm 1/3] intel/skl: Add SKL PCI ids

2014-09-26 Thread Damien Lespiau
v2: Add more PCI IDs (Michael H. Nguyen) v3: Synchronize one more with the kernel PCI IDs (Damien) Signed-off-by: Damien Lespiau Signed-off-by: Ben Widawsky Signed-off-by: Michael H. Nguyen --- intel/intel_chipset.h | 43 ++- 1 file changed, 42

[PATCH libdrm 2/3] intel/skl: Add gen9 to the buffer manager init

2014-09-26 Thread Damien Lespiau
Signed-off-by: Damien Lespiau Reviewed-by: Kenneth Graunke Signed-off-by: Ben Widawsky --- intel/intel_bufmgr_gem.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c index ba65527..a6fa224 100644 --- a/intel/intel_bufmgr_gem.c +++ b/intel

[PATCH libdrm 3/3] intel/skl: add gen9 to the CS decoding init

2014-09-26 Thread Damien Lespiau
Signed-off-by: Damien Lespiau Reviewed-by: Kenneth Graunke Signed-off-by: Ben Widawsky --- intel/intel_decode.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/intel/intel_decode.c b/intel/intel_decode.c index a5d6e04..7d5cbe5 100644 --- a/intel/intel_decode.c +++ b

[PATCH libdrm 1/3] intel/skl: Add SKL PCI ids

2014-09-30 Thread Damien Lespiau
On Tue, Sep 30, 2014 at 11:19:37AM +0100, Thomas Wood wrote: > On 26 September 2014 14:19, Damien Lespiau > wrote: > > v2: Add more PCI IDs (Michael H. Nguyen) > > v3: Synchronize one more with the kernel PCI IDs (Damien) > > > > Signed-off-by: Damien Lespiau &

HDMI 4k support v2

2013-08-14 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 - E

[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 --- 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 index dfc7a1b..e014785

[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?) Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_edid.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm

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

2013-08-14 Thread Damien Lespiau
earer that offset is relative to the end of the required fields of the HDMI VSDB (Ville Syrj?l?) Signed-off-by: Damien Lespiau Tested-by: Cancan Feng 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-14 Thread Damien Lespiau
Suggested-by: Ville Syrj?l? Signed-off-by: Damien Lespiau --- 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/gpu/drm/drm_edid.c index 0faa08e..606335f 100644

[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 Date: Tue Aug 6 20

[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 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 --- drivers/video/hdmi.c | 5 +++-- in

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

2013-08-14 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 Le

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

2013-08-14 Thread Damien Lespiau
I just wrote the bits to define and pack HDMI vendor specific infoframe. Port the host1x driver to use those so I can refactor the infoframe code a bit more. Cc: Thierry Reding Cc: Terje Bergstr?m Cc: linux-tegra at vger.kernel.org Signed-off-by: Damien Lespiau --- drivers/gpu/host1x/drm

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

2013-08-14 Thread Damien Lespiau
HDMI, but details..) Signed-off-by: Damien Lespiau --- 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/drm/drm_ed

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

2013-08-14 Thread Damien Lespiau
this API has been ported in: Author: Damien Lespiau 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 --- drivers/video/hdmi.c | 45 + include/linux/h

[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. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_edid.c | 36 include/drm/drm_edid.h | 4 2 files changed, 40 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b

[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 --- drivers/gpu/drm/i915/i915_reg.h | 2 ++ drivers/gpu/drm/i915/intel_hdmi.c | 28 2 files changed, 30 insertions

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 here

[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 Reviewed-by: Ville Syrj?l? --- 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

[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 --- drivers/gpu/drm/drm_edid.c | 12 +++- 1 file changed, 7 insertions(+), 5

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

2013-08-14 Thread Damien Lespiau
earer that offset 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 Tested-by: Cancan Feng Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=67030 Reviewed-by: Simon Farnsworth -

[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? Signed-off-by: Damien Lespiau Reviewed-by: Ville Syrj?l? Reviewed-by: Simon Farnsworth --- drivers/gpu/drm/drm_edid.c | 68 ++ 1 file changed, 62 insertions(+), 6

[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 Date: Tue Aug 6 20

[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 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 Reviewed-by: Ville Syrj?l? --- dr

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

2013-08-14 Thread Damien Lespiau
imp up the commit message with the note about the length (Ville Syrj?l?) Cc: Thierry Reding Cc: Terje Bergstr?m Cc: linux-tegra at vger.kernel.org Signed-off-by: Damien Lespiau --- drivers/gpu/host1x/drm/hdmi.c | 24 1 file changed, 4 insertions(+), 20 deletions(-) di

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

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

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

2013-08-14 Thread Damien Lespiau
this API has been ported in: Author: Damien Lespiau 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 Reviewed-by: Ville Syrj?l? --- drivers/

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

2013-08-14 Thread Damien Lespiau
HDMI, but details..) Signed-off-by: Damien Lespiau Reviewed-by: Ville Syrj?l? --- 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 d76d608..3aa653f 100644 ---

[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 Reviewed-by: Simon Farnsworth Reviewed-by: Ville Syrj?l? --- drivers/gpu/drm/drm_edid.c | 36 include/drm/drm_edid.h

[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 Reviewed-by: Ville Syrj?l? --- drivers/gpu/drm/i915/i915_reg.h | 2 ++ drivers/gpu/drm/i915/intel_hdmi.c | 28 2

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

2013-08-15 Thread Damien Lespiau
something similar in mind for this on other little parameters/debug features). Reviewed-by: Damien Lespiau 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. -- Damien &

[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

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 hdmi_vendor

[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 Reviewed-by: Ville Syrj?l? --- 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

[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 Reviewed-by: Ville Syrj?l? --- drivers/gpu/drm/drm_edid.c | 12 +++- 1 file

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

2013-08-19 Thread Damien Lespiau
earer that offset 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 Tested-by: Cancan Feng Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=67030 Reviewed-by: Simon Farnswo

[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? Signed-off-by: Damien Lespiau Reviewed-by: Ville Syrj?l? Reviewed-by: Simon Farnsworth --- drivers/gpu/drm/drm_edid.c | 68 ++ 1 file changed, 62 insertions(+), 6

[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 Date: Tue Aug 6 20

[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 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 Reviewed-by: Ville Syrj?l? --- dr

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

2013-08-19 Thread Damien Lespiau
-side (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 Reviewed-by: Ville Syrj?l? --- drivers/video/hdmi.c

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

2013-08-19 Thread Damien Lespiau
imp up the commit message with the note about the length (Ville Syrj?l?) Cc: Thierry Reding Cc: Terje Bergstr?m Cc: linux-tegra at vger.kernel.org Signed-off-by: Damien Lespiau --- drivers/gpu/host1x/drm/hdmi.c | 24 1 file changed, 4 insertions(+), 20 deletions(-) di

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

2013-08-19 Thread Damien Lespiau
this API has been ported in: Author: Damien Lespiau 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 Reviewed-by: Ville Syrj?l? --- drivers/

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

2013-08-19 Thread Damien Lespiau
HDMI, but details..) Signed-off-by: Damien Lespiau Reviewed-by: Ville Syrj?l? --- 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 d76d608..3aa653f 100644 ---

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

2013-08-19 Thread Damien Lespiau
We just got rid of the version of hdmi_vendor_infoframe that had a byte array for anyone to poke at. It's now time to shuffle around the naming of hdmi_hdmi_infoframe to make hdmi_vendor_infoframe become the HDMI vendor specific structure. Cc: Thierry Reding Signed-off-by: Damien Le

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

2013-08-19 Thread Damien Lespiau
This can then be used by DRM drivers to setup their vendor infoframes. v2: Fix hmdi typo (Simon Farnsworth) v3: Adapt to the hdmi_vendor_infoframe rename Signed-off-by: Damien Lespiau Reviewed-by: Simon Farnsworth Reviewed-by: Ville Syrj?l? --- drivers/gpu/drm/drm_edid.c | 36

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

2013-08-19 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 Reviewed-by: Ville Syrj?l? --- drivers/gpu/drm/i915/i915_reg.h | 2 ++ drivers/gpu/drm/i915/intel_hdmi.c | 28 2

[PATCH 14/14] video/hdmi: Rename HDMI_IDENTIFIER to HDMI_IEEE_OUI

2013-08-19 Thread Damien Lespiau
HDMI_IDENTIFIER was felt too generic, rename it to what it is, the IEEE OUI corresponding to HDMI Licensing, LLC. http://standards.ieee.org/develop/regauth/oui/oui.txt Cc: Thierry Reding Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_edid.c | 2 +- drivers/video/hdmi.c | 4

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

2013-08-19 Thread Damien Lespiau
On Thu, Aug 15, 2013 at 04:45:56PM +0200, Thierry Reding wrote: > On Wed, Aug 14, 2013 at 06:19:09PM +0100, Damien Lespiau wrote: > > Just like: > > > > Author: Damien Lespiau > > Date: Mon Aug 12 11:53:24 2013 +0100 > > > > video/hdmi: Don&#x

A gentle scrub

2013-08-20 Thread Damien Lespiau
A small pass on drm headers to remove some stale prototypes/functions/defines and to make static a few functions. drivers/gpu/drm/drm_crtc.c | 38 +++-- drivers/gpu/drm/drm_fb_cma_helper.c | 5 ++--- drivers/gpu/drm/drm_modes.c | 21 --

[PATCH 1/8] drm: Remove stale prototypes

2013-08-20 Thread Damien Lespiau
A few prototypes have been left in the headers, their function friends long gone. Signed-off-by: Damien Lespiau --- include/drm/drmP.h | 5 - include/drm/drm_crtc.h | 11 --- 2 files changed, 16 deletions(-) diff --git a/include/drm/drmP.h b/include/drm/drmP.h index 39911dc

[PATCH 2/8] drm: Remove drm_mode_create_dithering_property()

2013-08-20 Thread Damien Lespiau
This was last used by nouveau, replaced by a driver-specific property in: commit de69185573586302ada2e59ba41835df36986277 Author: Ben Skeggs Date: Mon Oct 17 12:23:41 2011 +1000 drm/nouveau: improve dithering properties, and implement proper auto mode Signed-off-by: Damien

[PATCH 3/8] drm: Remove drm_mode_list_concat()

2013-08-20 Thread Damien Lespiau
code adding modes. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_modes.c | 21 - include/drm/drm_crtc.h | 2 -- 2 files changed, 23 deletions(-) diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c index 504a602..fc2adb6 100644 --- a/drivers/gpu

[PATCH 4/8] drm: Make drm_mode_remove() static

2013-08-20 Thread Damien Lespiau
It's only used in drm_crtc.c. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_crtc.c | 7 +++ include/drm/drm_crtc.h | 1 - 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c index ffb791f..54b4169 100644

[PATCH 5/8] drm: Remove 2 unused defines

2013-08-20 Thread Damien Lespiau
These were introduced in the very first DRM commit: commit f453ba0460742ad027ae0c4c7d61e62817b3e7ef Author: Dave Airlie Date: Fri Nov 7 14:05:41 2008 -0800 DRM: add mode setting support Add mode setting support to the DRM layer. But are unused. Signed-off-by: Damien

[PATCH 6/8] drm: Make drm_fb_cma_describe() static

2013-08-20 Thread Damien Lespiau
This function is only used in drm_fb_cma_helper.c. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_fb_cma_helper.c | 5 ++--- include/drm/drm_fb_cma_helper.h | 1 - 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/drm_fb_cma_helper.c b/drivers/gpu/drm

[PATCH 7/8] drm: Remove unused PCI ids

2013-08-20 Thread Damien Lespiau
Signed-off-by: Damien Lespiau --- include/drm/drm_pciids.h | 30 -- 1 file changed, 30 deletions(-) diff --git a/include/drm/drm_pciids.h b/include/drm/drm_pciids.h index 34efaf6..0a85e5c 100644 --- a/include/drm/drm_pciids.h +++ b/include/drm/drm_pciids.h @@ -690,29

[PATCH 8/8] drm: Make drm_get_platform_dev() static

2013-08-20 Thread Damien Lespiau
It's only used in drm_platform.c. Signed-off-by: Damien Lespiau --- drivers/gpu/drm/drm_platform.c | 7 +++ include/drm/drmP.h | 3 --- 2 files changed, 3 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/drm_platform.c b/drivers/gpu/drm/drm_platform.c index b8

[PATCH 8/8] drm: Make drm_get_platform_dev() static

2013-08-20 Thread Damien Lespiau
On Tue, Aug 20, 2013 at 07:56:42AM -0700, Ian Romanick wrote: > On 08/19/2013 04:53 PM, Damien Lespiau wrote: > >It's only used in drm_platform.c. > > > >Signed-off-by: Damien Lespiau > >--- > > drivers/gpu/drm/drm_platform.c | 7 +++ > > include

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

2013-08-21 Thread Damien Lespiau
On Mon, Aug 19, 2013 at 09:31:42PM +0200, Thierry Reding wrote: > On Mon, Aug 19, 2013 at 02:49:50PM +0100, Damien Lespiau wrote: > > 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: > >

[PATCH] drm: Remove the dithering_mode_property field

2013-08-22 Thread Damien Lespiau
truct drm_mode_config. Signed-off-by: Damien Lespiau --- include/drm/drm_crtc.h | 1 - 1 file changed, 1 deletion(-) diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h index 0a9f73e..960a27c 100644 --- a/include/drm/drm_crtc.h +++ b/include/drm/drm_crtc.h @@ -837,7 +837,6 @@ struct drm_mode_c

[Linaro-mm-sig] [RFC 0/4] Atomic Display Framework

2013-08-29 Thread Damien Lespiau
On Thu, Aug 29, 2013 at 10:58:51AM +0300, Ville Syrj?l? wrote: > After that we need to figure out how we can expose them to userspace w/o > risking breaking stuff too much. Maybe a new ioctl to enumerate private > planes only? And the maybe only accept direct use of private planes via > the atomic

[PATCH 6/6] intel: Update package name and description in libdrm_intel.pc

2013-08-30 Thread Damien Lespiau
On Thu, Aug 29, 2013 at 09:31:54PM +0100, Emil Velikov wrote: > Currently the package name and description duplicate that of the > core libdrm. Update those to reflect reality. > > Cc: Daniel Vetter > Signed-off-by: Emil Velikov Looks good to me: Reviewed-by: Damien Lespi

[PATCH 1/2] gitignore: Ignore various generated files

2013-12-13 Thread Damien Lespiau
- *.log/*.trs are generated by make check - TAGS are generated by make tags - build-aux, config.h.in~ by autoconf - *.sw? are temporary files create by vim - name_from_fd wasn't ignored yet for some reason Signed-off-by: Damien Lespiau --- .gitignore | 8 +++- 1 file changed, 7 inser

[PATCH 2/2] intel/test_decode: Allow gen8 to be infered from the batch filenames

2013-12-13 Thread Damien Lespiau
Signed-off-by: Damien Lespiau --- intel/test_decode.c | 1 + 1 file changed, 1 insertion(+) diff --git a/intel/test_decode.c b/intel/test_decode.c index 0fcdf3b..b710f34 100644 --- a/intel/test_decode.c +++ b/intel/test_decode.c @@ -146,6 +146,7 @@ infer_devid(const char *batch_filename

[PATCH 1/2] gitignore: Ignore various generated files

2013-12-13 Thread Damien Lespiau
On Fri, Dec 13, 2013 at 07:03:11PM +0100, Daniel Vetter wrote: > On Fri, Dec 13, 2013 at 05:51:25PM +0000, Damien Lespiau wrote: > > - *.log/*.trs are generated by make check > > - TAGS are generated by make tags > > - build-aux, config.h.in~ by autoconf > > - *.sw? ar

[Intel-gfx] [PATCH -next] drm/i915: fix return value check of debugfs_create_file()

2013-12-16 Thread Damien Lespiau
> > Signed-off-by: Wei Yongjun Reviewed-by: Damien Lespiau -- Damien > --- > drivers/gpu/drm/i915/i915_debugfs.c | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_debugfs.c > b/drivers/gpu/drm/i915/i915_de

[PATCH edid-decode] Read extension blocks in xrandr EDID property

2013-12-17 Thread Damien Lespiau
On Tue, Nov 26, 2013 at 11:25:49AM +, Simon Farnsworth wrote: > Extend the parsing of the xrandr EDID property block to read extension > blocks, not just the basic block. > > Signed-off-by: Simon Farnsworth Hi, That patch seems to work as intended, so pushed it with my r-b tag. Sorry it too

[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

<    1   2   3   4   5   6   >