[PATCH 5/8] drm: make the connector properties code use the object properties code

2012-03-29 Thread Eugeni Dodonov
bed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20120329/446224ba/attachment.htm>

[PATCH 4/8] drm: add generic ioctls to get/set properties on any object

2012-03-29 Thread Eugeni Dodonov
hment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20120329/1d71378a/attachment.htm>

[PATCH 3/8] drm: create struct drm_object_properties and use it

2012-03-29 Thread Eugeni Dodonov
;http://lists.freedesktop.org/archives/dri-devel/attachments/20120329/ed096a8f/attachment.htm>

[PATCH 1/8] drm: add drm_property_change_is_valid

2012-03-29 Thread Eugeni Dodonov
-- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20120329/223c84fb/attachment.htm>

[Bug 42876] System doesn't boot with GTX 550 Ti

2012-03-29 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=42876 --- Comment #8 from ArTourter 2012-03-29 23:36:40 --- (In reply to comment #7) > (In reply to comment #6) > > I have tried booting with both nouveau and nvidia blacklisted in console > > mode > > and still get the same behaviour. > > > > As

[PATCH libdrm 5/5] modetest: print CRTC properties

2012-03-29 Thread Eugeni Dodonov
bed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20120329/73aa5902/attachment.htm>

[PATCH libdrm 4/5] Add support for generic object properties IOCTLs

2012-03-29 Thread Eugeni Dodonov
-- Eugeni Dodonov <http://eugeni.dodonov.net/> -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20120329/b004e0c9/attachment.htm>

[PATCH RFC xf86-video-intel 2/2] Add underscan properties

2012-03-29 Thread Chris Wilson
On Thu, 29 Mar 2012 18:30:20 -0300, Paulo Zanoni wrote: > From: Paulo Zanoni > > In the Kernel side, these are crtc properties (since in the hardware, > underscan use the panel fitters, which are attached to the pipes). > Ideally we should make these as crtc properties too, but since xrandr > do

[PATCH libdrm 3/5] modetest: print more about our properties

2012-03-29 Thread Eugeni Dodonov
HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20120329/9a487b5b/attachment.html>

[PATCH RFC xf86-video-intel 1/2] Update the rotation property whenever we change the rotation.

2012-03-29 Thread Chris Wilson
On Thu, 29 Mar 2012 18:30:19 -0300, Paulo Zanoni wrote: > From: Paulo Zanoni > > Don't worry if that fails: only the KVMr feature will be affected. > > We still need to change the sna/ code. > We also need to add the dependency on libdrm. > > Signed-off-by: Paulo Zanoni > --- > src/intel.h

[PATCH libdrm 2/5] modetest: fix memory leak

2012-03-29 Thread Eugeni Dodonov
Paulo Zanoni > Reviewed-by: Eugeni Dodonov -- Eugeni Dodonov <http://eugeni.dodonov.net/> -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20120329/0c458a63/attachment.htm>

Re: [PATCH] drm/radeon/kms: fix fans after resume

2012-03-29 Thread Michel Dänzer
On Don, 2012-03-29 at 19:04 -0400, alexdeuc...@gmail.com wrote: > From: Alex Deucher > > On pre-R600 asics, the SpeedFanControl table is not > executed as part of ASIC_Init as it is on newer asics. > > Fixes: > https://bugzilla.kernel.org/show_bug.cgi?id=29412 > > Signed-off-by: Alex Deucher

[PATCH libdrm 1/5] modetest: fix some compiler warnings

2012-03-29 Thread Eugeni Dodonov
ill get used again in the future (but I might be wrong). -- Eugeni Dodonov <http://eugeni.dodonov.net/> -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20120329/a01de1e0/attachment.htm>

[PATCH] drm: add valid pixel formats for fb

2012-03-29 Thread Seung-Woo Kim
This patch adds multi buffer plane pixel formats into valid pixel format list for fb since they are missed from the list. Signed-off-by: Seung-Woo Kim Signed-off-by: Inki Dae Signed-off-by: Kyungmin Park --- drivers/gpu/drm/drm_crtc.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-

[Bug 29412] fans running at full-speed after resume from suspend with radeon and KMS

2012-03-29 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=29412 --- Comment #15 from Alex Deucher 2012-03-29 23:01:52 --- Perfect. No need for the vbios. I'll send the patch to Dave. -- Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because:

[Bug 42678] [3.3-rc1] radeon stuck in kernel after lockup

2012-03-29 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=42678 --- Comment #10 from J?r?me Glisse 2012-03-29 22:57:06 --- Adrien question is: Is Xorg stuck inside the kernel ? Fixing root cause of GPU lockup is a different matter (basicly you have to go though several G of datas and there is no tools to

[PATCH 2/8] drm: WARN() when drm_connector_attach_property fails

2012-03-29 Thread Chris Wilson
On Thu, 29 Mar 2012 18:27:20 -0300, Paulo Zanoni wrote: > From: Paulo Zanoni > > Also return void instead of int. We have more than 100 callers and > no one checks for the return value. > > If this function fails the property won't be exposed by the get/set > ioctls, but we should probably surv

Re: [git pull] drm intel hibernation fix

2012-03-29 Thread richard -rw- weinberger
On Thu, Mar 29, 2012 at 9:34 AM, Keith Packard wrote: > <#part sign=pgpmime> > On Thu, 29 Mar 2012 08:14:08 +0100 (IST), Dave Airlie > wrote: > >> Dave Airlie (1): >>       drm/i915: suspend fbdev device around suspend/hibernate > I've reported the issue one year ago and kept the discussion ali

Re: [git pull] drm main pull for 3.4-rc1

2012-03-29 Thread Tony Vroon
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 29/03/12 13:39, Daniel Vetter wrote: > Is the for-chainsaw branch plus intel_iommu=igfx_off now stable for > you? It appears to crash when I shut my machine down, but has otherwise behaved quite well. Regards, - -- Tony Vroon UNIX systems adminis

[Bug 42678] [3.3-rc1] radeon stuck in kernel after lockup

2012-03-29 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=42678 Adrien Nader changed: What|Removed |Added CC||camaradetux at gmail.Com --- Comment #

[git pull] drm intel hibernation fix

2012-03-29 Thread richard -rw- weinberger
On Thu, Mar 29, 2012 at 9:34 AM, Keith Packard wrote: > <#part sign=pgpmime> > On Thu, 29 Mar 2012 08:14:08 +0100 (IST), Dave Airlie > wrote: > >> Dave Airlie (1): >> ? ? ? drm/i915: suspend fbdev device around suspend/hibernate > I've reported the issue one year ago and kept the discussion ali

[PATCH libdrm] omap: fix license header

2012-03-29 Thread Rob Clark
From: Rob Clark In syncing with the corresponding kernel header, the wrong license header was inadvertantly copied over. The intention was for the userspace headers to have a MIT license following the convention of the rest of libdrm, xorg, etc. Signed-off-by: Rob Clark --- omap/omap_drm.h |

[PATCH 04/16] drm: initial multiple nodes ioctl work.

2012-03-29 Thread Ville Syrjälä
On Thu, Mar 29, 2012 at 12:22:20PM -0500, Ilija Hadzic wrote: > > > On Thu, 29 Mar 2012, Ville [iso-8859-1] Syrj?l? wrote: > > >> +/* render node create and remove functions > >> + if crtc/encoders/connectors all == 0 then gpgpu node */ > >> +struct drm_render_node_create { > >> + __u32 node_

[Bug 47955] Celestia hit fallback on r300g from git?

2012-03-29 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=47955 --- Comment #15 from Andrew Randrianasulu 2012-03-29 20:54:13 PDT --- .. and spriteblast mesa demo surely don't work correctly here. (I see something remotely like _giant_ sprites flashing on screen) -- Configure bugmail: https://bugs.freedesk

[RFC] Revive the work on render-nodes branch

2012-03-29 Thread Ville Syrjälä
On Thu, Mar 29, 2012 at 12:41:22PM -0400, Ilija Hadzic wrote: > The following set of patches will revive the drm-render-nodes [1] > branch that has been dormant in Dave Airlie's repository for some > time. I rebased this branch to the latest drm-core-next and did some > (hopefully useful) follow-up

[PATCH RFC 1/2] drm: add bitmask property type

2012-03-29 Thread Rob Clark
On Thu, Mar 29, 2012 at 8:02 PM, Rob Clark wrote: > From: Rob Clark > > A bitmask property is similar to an enum. ?The enum value is a bit > position (0-63), and valid property values consist of a mask of > zero or more of (1 << enum_val[n]). > > TODO: word commit msg better > TODO: maybe "flags"

[PATCH 04/16] drm: initial multiple nodes ioctl work.

2012-03-29 Thread Ville Syrjälä
On Thu, Mar 29, 2012 at 12:41:26PM -0400, Ilija Hadzic wrote: > diff --git a/include/drm/drm_mode.h b/include/drm/drm_mode.h > index 2a2acda..2a403bb 100644 > --- a/include/drm/drm_mode.h > +++ b/include/drm/drm_mode.h > @@ -441,4 +441,18 @@ struct drm_mode_destroy_dumb { > uint32_t handle; >

[PATCH RFC 2/2] drm: add plane properties

2012-03-29 Thread Rob Clark
From: Rob Clark The omapdrm driver uses this for setting per-overlay rotation. It is likely also useful for setting YUV->RGB colorspace conversion matrix, etc. --- drivers/gpu/drm/drm_crtc.c | 19 +++ include/drm/drm_crtc.h |5 + 2 files changed, 24 insertions(+),

[PATCH RFC 1/2] drm: add bitmask property type

2012-03-29 Thread Rob Clark
From: Rob Clark A bitmask property is similar to an enum. The enum value is a bit position (0-63), and valid property values consist of a mask of zero or more of (1 << enum_val[n]). TODO: word commit msg better TODO: maybe "flags" would be a better name for the property type? --- See https://g

[PATCH libdrm] libdrm: update drm/drm_fourcc.h from kernel to add multi plane formats

2012-03-29 Thread Seung-Woo Kim
Multi buffer plane pixel formats are added as like kernel header. Signed-off-by: Seung-Woo Kim --- include/drm/drm_fourcc.h |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/include/drm/drm_fourcc.h b/include/drm/drm_fourcc.h index 85facb0..7cfd95a 100644 --- a/inclu

Re: [PATCH 5/8] drm: make the connector properties code use the object properties code

2012-03-29 Thread Eugeni Dodonov
On Thu, Mar 29, 2012 at 18:27, Paulo Zanoni wrote: > From: Paulo Zanoni > > Signed-off-by: Paulo Zanoni > Reviewed-by: Eugeni Dodonov -- Eugeni Dodonov ___ dri-devel mailing list dri-devel@lists.freedesktop.org http://

Re: [PATCH 4/8] drm: add generic ioctls to get/set properties on any object

2012-03-29 Thread Eugeni Dodonov
On Thu, Mar 29, 2012 at 18:27, Paulo Zanoni wrote: > + switch (arg_obj->type) { > + case DRM_MODE_OBJECT_CONNECTOR: > + ret = drm_mode_connector_set_obj_prop(arg_obj, property, > + arg->value); > + break;

Re: [PATCH 3/8] drm: create struct drm_object_properties and use it

2012-03-29 Thread Eugeni Dodonov
On Thu, Mar 29, 2012 at 18:27, Paulo Zanoni wrote: > From: Paulo Zanoni > > For now, only connectors have it. In the future, all objects that need > properties should use it. Since the strucutre is referenced inside > *structure > struct drm_mode_object, we will be able to deal with object pr

Re: [PATCH 1/8] drm: add drm_property_change_is_valid

2012-03-29 Thread Eugeni Dodonov
On Thu, Mar 29, 2012 at 18:27, Paulo Zanoni wrote: > + if (property->flags & DRM_MODE_PROP_RANGE) { > + if (value < property->values[0]) > + return false; > + if (value > property->values[1]) > + return false; > Those

[PATCH libdrm] omap: fix license header

2012-03-29 Thread Rob Clark
From: Rob Clark In syncing with the corresponding kernel header, the wrong license header was inadvertantly copied over. The intention was for the userspace headers to have a MIT license following the convention of the rest of libdrm, xorg, etc. Signed-off-by: Rob Clark --- omap/omap_drm.h |

Re: [PATCH libdrm 5/5] modetest: print CRTC properties

2012-03-29 Thread Eugeni Dodonov
On Thu, Mar 29, 2012 at 18:28, Paulo Zanoni wrote: > From: Paulo Zanoni > > Signed-off-by: Paulo Zanoni > Reviewed-by: Eugeni Dodonov -- Eugeni Dodonov ___ dri-devel mailing list dri-devel@lists.freedesktop.org http://

Re: [PATCH libdrm 4/5] Add support for generic object properties IOCTLs

2012-03-29 Thread Eugeni Dodonov
On Thu, Mar 29, 2012 at 18:28, Paulo Zanoni wrote: > From: Paulo Zanoni > > New library calls: > - drmModeObjectGetProperties > - drmModeFreeObjectProperties > - drmModeObjectSetProperties > > Signed-off-by: Paulo Zanoni > Reviewed-by: Eugeni Dodonov -- Eugeni Dodonov

Re: [PATCH libdrm 3/5] modetest: print more about our properties

2012-03-29 Thread Eugeni Dodonov
On Thu, Mar 29, 2012 at 18:28, Paulo Zanoni wrote: > From: Paulo Zanoni > > In the future we'll have more than just connector properties, so create > a dump_prop function that can handle any property (instead of the > current dump_props function that only handles connector properties). > > Also,

Re: [PATCH libdrm 2/5] modetest: fix memory leak

2012-03-29 Thread Eugeni Dodonov
On Thu, Mar 29, 2012 at 18:28, Paulo Zanoni wrote: > From: Paulo Zanoni > > Don't "continue" without freeing the connector. > > 192 bytes in 6 blocks are indirectly lost in loss record 6 of 12 > at 0x4C2779D: malloc (in > /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so) > by 0x4E30DD8: d

Re: [PATCH libdrm 1/5] modetest: fix some compiler warnings

2012-03-29 Thread Eugeni Dodonov
On Thu, Mar 29, 2012 at 18:28, Paulo Zanoni wrote: > From: Paulo Zanoni > > Use unsigned int instead of int: > - modetest.c:89:1: warning: comparison between signed and unsigned integer > expressions [-Wsign-compare] > - modetest.c:97:1: warning: comparison between signed and unsigned integer >

[PATCH] drm/radeon/kms: fix fans after resume

2012-03-29 Thread alexdeuc...@gmail.com
From: Alex Deucher On pre-R600 asics, the SpeedFanControl table is not executed as part of ASIC_Init as it is on newer asics. Fixes: https://bugzilla.kernel.org/show_bug.cgi?id=29412 Signed-off-by: Alex Deucher Cc: stable at vger.kernel.org --- drivers/gpu/drm/radeon/atom.c | 15 +++

[PATCH RFC xf86-video-intel 2/2] Add underscan properties

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni In the Kernel side, these are crtc properties (since in the hardware, underscan use the panel fitters, which are attached to the pipes). Ideally we should make these as crtc properties too, but since xrandr doesn't have support for them, we expose the atoms as output properties

[PATCH RFC xf86-video-intel 1/2] Update the rotation property whenever we change the rotation.

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni Don't worry if that fails: only the KVMr feature will be affected. We still need to change the sna/ code. We also need to add the dependency on libdrm. Signed-off-by: Paulo Zanoni --- src/intel.h |3 ++ src/intel_display.c | 80

[PATCH libdrm 5/5] modetest: print CRTC properties

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni Signed-off-by: Paulo Zanoni --- tests/modetest/modetest.c | 15 +++ 1 files changed, 15 insertions(+), 0 deletions(-) diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c index 02ea579..a6a2a8c 100644 --- a/tests/modetest/modetest.c +++ b/tests/mo

[PATCH libdrm 4/5] Add support for generic object properties IOCTLs

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni New library calls: - drmModeObjectGetProperties - drmModeFreeObjectProperties - drmModeObjectSetProperties Signed-off-by: Paulo Zanoni --- include/drm/drm.h |2 + include/drm/drm_mode.h | 24 ++ xf86drmMode.c | 83 +++

[PATCH libdrm 3/5] modetest: print more about our properties

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni In the future we'll have more than just connector properties, so create a dump_prop function that can handle any property (instead of the current dump_props function that only handles connector properties). Also, make this function print a lot more information about the existi

[PATCH libdrm 2/5] modetest: fix memory leak

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni Don't "continue" without freeing the connector. 192 bytes in 6 blocks are indirectly lost in loss record 6 of 12 at 0x4C2779D: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so) by 0x4E30DD8: drmMalloc (xf86drm.c:147) by 0x4E35024: drmAllocCpy (xf86drmMod

[PATCH libdrm 1/5] modetest: fix some compiler warnings

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni Use unsigned int instead of int: - modetest.c:89:1: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] - modetest.c:97:1: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] - modetest.c:117:1: warning: compar

[PATCH 8/8] drm/i915: add overscan compensation CRTC properties

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni They're named "underscan hborder" and "underscan vborder". The properties accept values from 0 to 100, where 0 is "don't compensate" and 100 is "shrink the screen as much as possible" (not necessarily 100 pixels). V2: Rename to "underscan hborder" and "underscan vborder" to ma

[PATCH RFC 7/8] drm/i915: add 'rotation' CRTC property

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni This property is needed so we can inform the KVMr feature about our current rotation: whenever we change the rotation, we should change that property so that the KVMr knows that the screen is rotated. How to reproduce the problem: - on an AMT machine, enable KVM - boot the mac

[PATCH 6/8] drm: add CRTC properties

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni The i915 driver needs this for the rotation and overscan compensation properties. Other drivers might need this too. Signed-off-by: Paulo Zanoni --- drivers/gpu/drm/drm_crtc.c | 20 include/drm/drm_crtc.h |9 - 2 files changed, 28 inser

[PATCH 5/8] drm: make the connector properties code use the object properties code

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni Signed-off-by: Paulo Zanoni --- drivers/gpu/drm/drm_crtc.c | 96 +-- 1 files changed, 12 insertions(+), 84 deletions(-) diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c index 12f93e4..b21bfcd 100644 --- a/drivers/

[PATCH 4/8] drm: add generic ioctls to get/set properties on any object

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni Useless for connector properties (since they already have their own ioctls), but useful when we add properties to CRTCs, planes and other objects. Signed-off-by: Paulo Zanoni --- drivers/gpu/drm/drm_crtc.c | 180 drivers/gpu/drm/

[PATCH 3/8] drm: create struct drm_object_properties and use it

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni For now, only connectors have it. In the future, all objects that need properties should use it. Since the strucutre is referenced inside struct drm_mode_object, we will be able to deal with object properties without knowing the real type of the object. Signed-off-by: Paulo Za

[PATCH 2/8] drm: WARN() when drm_connector_attach_property fails

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni Also return void instead of int. We have more than 100 callers and no one checks for the return value. If this function fails the property won't be exposed by the get/set ioctls, but we should probably survive. If this starts happening, the solution will be to increase DRM_CON

[PATCH 1/8] drm: add drm_property_change_is_valid

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni Move code from drm_mode_connector_property_set_ioctl to a new function, so we can reuse this code when we add crtc properties. v2: use bool instead of int Signed-off-by: Paulo Zanoni --- drivers/gpu/drm/drm_crtc.c | 41 + 1 files ch

CRTC properties patches

2012-03-29 Thread Paulo Zanoni
Hi Here is a new version of the patches. Summary: almost all patches changed at least one line, Kernel patches 7 and 8 are RFCs. The main difference is that now the ioctls are not for CRTC properties, but for generic "object" properties. I decided to write it like this after Rob Clark's review.

Re: regression(?) 3.3-rc4 -> 3.3-rc5: drm intel hangs

2012-03-29 Thread Norbert Preining
Hi everyone, is there anything else I can provide? On Fr, 09 Mär 2012, Norbert Preining wrote: > Hi everyone, > > currently 3.3.0-rc6+ I just was hit by that after wake up from ram: > > On Di, 28 Feb 2012, Daniel Vetter wrote: > > > Feb 28 11:42:47 mithrandir kernel: [15627.756071] > > > [drm:

Re: [PATCH RFC 1/2] drm: add bitmask property type

2012-03-29 Thread Rob Clark
On Thu, Mar 29, 2012 at 8:02 PM, Rob Clark wrote: > From: Rob Clark > > A bitmask property is similar to an enum.  The enum value is a bit > position (0-63), and valid property values consist of a mask of > zero or more of (1 << enum_val[n]). > > TODO: word commit msg better > TODO: maybe "flags"

[Bug 29412] fans running at full-speed after resume from suspend with radeon and KMS

2012-03-29 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=29412 --- Comment #14 from Jon Dowland 2012-03-29 18:08:59 --- That second patch sorts it! Thanks! I booted to single-user; modprobe radeon; pm-suspend; resumed: fine. Moved to multi-user; logged into GNOME; suspended via GNOME menu; resumed: fine

[PATCH RFC 2/2] drm: add plane properties

2012-03-29 Thread Rob Clark
From: Rob Clark The omapdrm driver uses this for setting per-overlay rotation. It is likely also useful for setting YUV->RGB colorspace conversion matrix, etc. --- drivers/gpu/drm/drm_crtc.c | 19 +++ include/drm/drm_crtc.h |5 + 2 files changed, 24 insertions(+),

[PATCH RFC 1/2] drm: add bitmask property type

2012-03-29 Thread Rob Clark
From: Rob Clark A bitmask property is similar to an enum. The enum value is a bit position (0-63), and valid property values consist of a mask of zero or more of (1 << enum_val[n]). TODO: word commit msg better TODO: maybe "flags" would be a better name for the property type? --- See https://g

[PATCH] drm/radeon: Don't dereference possibly-NULL pointer.

2012-03-29 Thread Michel Dänzer
From: Michel D?nzer Reported-by: Dan Carpenter Signed-off-by: Michel D?nzer --- Third time's the charm, I hope... drivers/gpu/drm/radeon/radeon_object.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_object.c b/drivers/gpu/drm/radeon

[PATCH 2/7 v6] drm/i915/intel_i2c: use double-buffered writes

2012-03-29 Thread Daniel Kurtz
On Thu, Mar 29, 2012 at 2:41 AM, Chris Wilson wrote: > On Thu, 29 Mar 2012 02:26:34 +0800, Daniel Kurtz > wrote: >> The GMBUS controller GMBUS3 register is double-buffered. ?Take advantage >> of this ?by writing two 4-byte words before the first wait for HW_RDY. >> This helps keep the GMBUS con

[PATCH 4/7 v6] drm/i915/intel_i2c: use WAIT cycle, not STOP

2012-03-29 Thread Daniel Kurtz
On Thu, Mar 29, 2012 at 2:48 AM, Chris Wilson wrote: > On Thu, 29 Mar 2012 02:26:36 +0800, Daniel Kurtz > wrote: >> The i915 is only able to generate a STOP cycle (i.e. finalize an i2c >> transaction) during a DATA or WAIT phase. ?In other words, the >> controller rejects a STOP requested as pa

[Bug 42876] System doesn't boot with GTX 550 Ti

2012-03-29 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=42876 --- Comment #8 from ArTourter 2012-03-29 23:36:40 --- (In reply to comment #7) > (In reply to comment #6) > > I have tried booting with both nouveau and nvidia blacklisted in console > > mode > > and still get the same behaviour. > > > > As

[PATCH 5/7 v6] drm/i915/intel_i2c: use INDEX cycles for i2c read transactions

2012-03-29 Thread Daniel Kurtz
On Thu, Mar 29, 2012 at 2:52 AM, Chris Wilson wrote: > On Thu, 29 Mar 2012 02:26:37 +0800, Daniel Kurtz > wrote: >> It is very common for an i2c device to require a small 1 or 2 byte write >> followed by a read. ?For example, when reading from an i2c EEPROM it is >> common to write and address,

[PATCH] drm/radeon/kms: fix fans after resume

2012-03-29 Thread alexdeucher
From: Alex Deucher On pre-R600 asics, the SpeedFanControl table is not executed as part of ASIC_Init as it is on newer asics. Fixes: https://bugzilla.kernel.org/show_bug.cgi?id=29412 Signed-off-by: Alex Deucher Cc: sta...@vger.kernel.org --- drivers/gpu/drm/radeon/atom.c | 15 ++

[Bug 29412] fans running at full-speed after resume from suspend with radeon and KMS

2012-03-29 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=29412 --- Comment #15 from Alex Deucher 2012-03-29 23:01:52 --- Perfect. No need for the vbios. I'll send the patch to Dave. -- Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because:

drm/i915:Fix array bounds warnings

2012-03-29 Thread majianpeng

[Bug 42678] [3.3-rc1] radeon stuck in kernel after lockup

2012-03-29 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=42678 --- Comment #10 from Jérôme Glisse 2012-03-29 22:57:06 --- Adrien question is: Is Xorg stuck inside the kernel ? Fixing root cause of GPU lockup is a different matter (basicly you have to go though several G of datas and there is no tools to

[PATCH] drm/i915:Fix array bounds warnings.

2012-03-29 Thread majianpeng
Signed-off-by: majianpeng --- drivers/gpu/drm/i915/i915_debugfs.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c index fdb7cce..8ad59dd 100644 --- a/drivers/gpu/drm/i915/i915_debugfs.c +++ b/driv

drm/i915:fix for some -Wuninitialized compiler warnings

2012-03-29 Thread majianpeng

Re: [PATCH RFC xf86-video-intel 2/2] Add underscan properties

2012-03-29 Thread Chris Wilson
On Thu, 29 Mar 2012 18:30:20 -0300, Paulo Zanoni wrote: > From: Paulo Zanoni > > In the Kernel side, these are crtc properties (since in the hardware, > underscan use the panel fitters, which are attached to the pipes). > Ideally we should make these as crtc properties too, but since xrandr > do

[PATCH] drm/i915:fix for some -Wuninitialized compiler warnings.

2012-03-29 Thread majianpeng
Signed-off-by: majianpeng --- drivers/gpu/drm/i915/i915_gem.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c index 1f441f5..c071b66 100644 --- a/drivers/gpu/drm/i915/i915_gem.c +++ b/drivers/gpu/drm/i9

Re: [PATCH RFC xf86-video-intel 1/2] Update the rotation property whenever we change the rotation.

2012-03-29 Thread Chris Wilson
On Thu, 29 Mar 2012 18:30:19 -0300, Paulo Zanoni wrote: > From: Paulo Zanoni > > Don't worry if that fails: only the KVMr feature will be affected. > > We still need to change the sna/ code. > We also need to add the dependency on libdrm. > > Signed-off-by: Paulo Zanoni > --- > src/intel.h

[Bug 42678] [3.3-rc1] radeon stuck in kernel after lockup

2012-03-29 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=42678 Adrien Nader changed: What|Removed |Added CC||camarade...@gmail.com --- Comment #9 f

drm/radeon: Only warn if the intra-domain offset actually exceeds the limit.

2012-03-29 Thread Dan Carpenter
Hello Michel D?nzer, This is a semi-automatic email about new static checker warnings. The patch d936622c3627: "drm/radeon: Only warn if the intra-domain offset actually exceeds the limit." from Mar 28, 2012, leads to the following Smatch complaint: drivers/gpu/drm/radeon/radeon_object.c:244 r

Re: [PATCH 2/8] drm: WARN() when drm_connector_attach_property fails

2012-03-29 Thread Chris Wilson
On Thu, 29 Mar 2012 18:27:20 -0300, Paulo Zanoni wrote: > From: Paulo Zanoni > > Also return void instead of int. We have more than 100 callers and > no one checks for the return value. > > If this function fails the property won't be exposed by the get/set > ioctls, but we should probably surv

[git pull] drm main pull for 3.4-rc1

2012-03-29 Thread Daniel Vetter
On Wed, Mar 28, 2012 at 09:55:09AM +0100, Tony Vroon wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 27/03/12 21:39, Daniel Vetter wrote: > > Yet another thing to try: Can you append intel_iommu=igfx_off to > > your kernel cmdline? If you can, try this both on mainline kernels > >

[PATCH RFC xf86-video-intel 2/2] Add underscan properties

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni In the Kernel side, these are crtc properties (since in the hardware, underscan use the panel fitters, which are attached to the pipes). Ideally we should make these as crtc properties too, but since xrandr doesn't have support for them, we expose the atoms as output properties

[PATCH RFC xf86-video-intel 1/2] Update the rotation property whenever we change the rotation.

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni Don't worry if that fails: only the KVMr feature will be affected. We still need to change the sna/ code. We also need to add the dependency on libdrm. Signed-off-by: Paulo Zanoni --- src/intel.h |3 ++ src/intel_display.c | 80

[PATCH libdrm 5/5] modetest: print CRTC properties

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni Signed-off-by: Paulo Zanoni --- tests/modetest/modetest.c | 15 +++ 1 files changed, 15 insertions(+), 0 deletions(-) diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c index 02ea579..a6a2a8c 100644 --- a/tests/modetest/modetest.c +++ b/tests/mo

[PATCH libdrm 4/5] Add support for generic object properties IOCTLs

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni New library calls: - drmModeObjectGetProperties - drmModeFreeObjectProperties - drmModeObjectSetProperties Signed-off-by: Paulo Zanoni --- include/drm/drm.h |2 + include/drm/drm_mode.h | 24 ++ xf86drmMode.c | 83 +++

[PATCH libdrm 3/5] modetest: print more about our properties

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni In the future we'll have more than just connector properties, so create a dump_prop function that can handle any property (instead of the current dump_props function that only handles connector properties). Also, make this function print a lot more information about the existi

[PATCH libdrm 2/5] modetest: fix memory leak

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni Don't "continue" without freeing the connector. 192 bytes in 6 blocks are indirectly lost in loss record 6 of 12 at 0x4C2779D: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so) by 0x4E30DD8: drmMalloc (xf86drm.c:147) by 0x4E35024: drmAllocCpy (xf86drmMod

[PATCH libdrm 1/5] modetest: fix some compiler warnings

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni Use unsigned int instead of int: - modetest.c:89:1: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] - modetest.c:97:1: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] - modetest.c:117:1: warning: compar

[PATCH 8/8] drm/i915: add overscan compensation CRTC properties

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni They're named "underscan hborder" and "underscan vborder". The properties accept values from 0 to 100, where 0 is "don't compensate" and 100 is "shrink the screen as much as possible" (not necessarily 100 pixels). V2: Rename to "underscan hborder" and "underscan vborder" to ma

[PATCH RFC 7/8] drm/i915: add 'rotation' CRTC property

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni This property is needed so we can inform the KVMr feature about our current rotation: whenever we change the rotation, we should change that property so that the KVMr knows that the screen is rotated. How to reproduce the problem: - on an AMT machine, enable KVM - boot the mac

[PATCH 6/8] drm: add CRTC properties

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni The i915 driver needs this for the rotation and overscan compensation properties. Other drivers might need this too. Signed-off-by: Paulo Zanoni --- drivers/gpu/drm/drm_crtc.c | 20 include/drm/drm_crtc.h |9 - 2 files changed, 28 inser

[PATCH 5/8] drm: make the connector properties code use the object properties code

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni Signed-off-by: Paulo Zanoni --- drivers/gpu/drm/drm_crtc.c | 96 +-- 1 files changed, 12 insertions(+), 84 deletions(-) diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c index 12f93e4..b21bfcd 100644 --- a/drivers/

[PATCH 4/8] drm: add generic ioctls to get/set properties on any object

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni Useless for connector properties (since they already have their own ioctls), but useful when we add properties to CRTCs, planes and other objects. Signed-off-by: Paulo Zanoni --- drivers/gpu/drm/drm_crtc.c | 180 drivers/gpu/drm/

[PATCH 3/8] drm: create struct drm_object_properties and use it

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni For now, only connectors have it. In the future, all objects that need properties should use it. Since the strucutre is referenced inside struct drm_mode_object, we will be able to deal with object properties without knowing the real type of the object. Signed-off-by: Paulo Za

[PATCH 2/8] drm: WARN() when drm_connector_attach_property fails

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni Also return void instead of int. We have more than 100 callers and no one checks for the return value. If this function fails the property won't be exposed by the get/set ioctls, but we should probably survive. If this starts happening, the solution will be to increase DRM_CON

[PATCH 1/8] drm: add drm_property_change_is_valid

2012-03-29 Thread Paulo Zanoni
From: Paulo Zanoni Move code from drm_mode_connector_property_set_ioctl to a new function, so we can reuse this code when we add crtc properties. v2: use bool instead of int Signed-off-by: Paulo Zanoni --- drivers/gpu/drm/drm_crtc.c | 41 + 1 files ch

Re: CRTC properties patches

2012-03-29 Thread Paulo Zanoni
Hi Here is a new version of the patches. Summary: almost all patches changed at least one line, Kernel patches 7 and 8 are RFCs. The main difference is that now the ioctls are not for CRTC properties, but for generic "object" properties. I decided to write it like this after Rob Clark's review.

[git pull] drm main pull for 3.4-rc1

2012-03-29 Thread Tony Vroon
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 29/03/12 13:39, Daniel Vetter wrote: > Is the for-chainsaw branch plus intel_iommu=igfx_off now stable for > you? It appears to crash when I shut my machine down, but has otherwise behaved quite well. Regards, - -- Tony Vroon UNIX systems adminis

[PATCH 2/2] tests: add render_node tests

2012-03-29 Thread Ilija Hadzic
Add two simple programs (render_node_add and render_node_rm) that can be used to add and remove render nodes from a shell command or a script. Signed-off-by: Ilija Hadzic --- .gitignore |2 + configure.ac |1 + tests/Makefile.am

[PATCH 1/2] libdrm: add functions for render node manipulation

2012-03-29 Thread Ilija Hadzic
Implement the user-space side of drm_render_node_create and drm_render_node_remove ioctls. The new functions are drmCreateRenderNode and drmRemoveRenderNode. Signed-off-by: Ilija Hadzic --- include/drm/drm.h |2 + include/drm/drm_mode.h | 16 + xf86drm.c | 5

libdrm part of the render-node patch series

2012-03-29 Thread Ilija Hadzic
The following two patches add libdrm support for render node manipulation. The first patch adds the functions for render node creation and removal that an application can call. The second patch add two small test utilities that allow render node creation and removal from shell. For example, this w

  1   2   >