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
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
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
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
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
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
, 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
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
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
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
--
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
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
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
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
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
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
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
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
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
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
++
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
&
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
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
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
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 +++--
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
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
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
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
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
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
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
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
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
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
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
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
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
-
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
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
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
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
-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/
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/
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
---
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
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
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
&
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
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
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
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
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
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
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
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
-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
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
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/
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
---
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
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
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
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
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
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 --
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
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
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
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
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
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
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
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
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
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:
> >
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
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
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
- *.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
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
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
>
> 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
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
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
201 - 300 of 505 matches
Mail list logo