[Intel-gfx] [PATCH xf86-video-intel] test if librt is required for clock_gettime

2013-06-18 Thread Jonathan Gray
clock_gettime is in libc not librt on OpenBSD so check to see if linking librt is required. Signed-off-by: Jonathan Gray --- configure.ac | 4 test/Makefile.am | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git configure.ac configure.ac index 3f6f9bd..26688a3 100644 ---

Re: [Intel-gfx] [PATCH] Fix intel_detect_pch() to work in xen environment.

2013-06-18 Thread Daniel Vetter
Can you please resubmit this as a proper patch with a committ message, signed-off-by line and everything else (see Documentation/SubmittingPatches). -Daniel On Tue, Jun 18, 2013 at 2:54 PM, G.R. wrote: > On Sun, Dec 23, 2012 at 2:51 PM, G.R. > wrote: Hi Jesse, I think I need to resend the

Re: [Intel-gfx] [PATCH 4/4] drm: Constify the pretty-print functions

2013-06-18 Thread Dave Airlie
On Wed, Jun 19, 2013 at 10:53 AM, Laurent Pinchart wrote: > Hi Ville, > > On Friday 07 June 2013 18:43:07 ville.syrj...@linux.intel.com wrote: >> From: Ville Syrjälä >> >> The structures and strings involved with various pretty-print functions >> aren't meant to be modified, so make them all cons

Re: [Intel-gfx] [PATCH v2] drm: Remove some unused stuff from drm_plane

2013-06-18 Thread Laurent Pinchart
Hello, On Wednesday 08 May 2013 15:52:10 Laurent Pinchart wrote: > On Wednesday 08 May 2013 16:40:54 ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > There's a bunch of unused members inside drm_plane, bloating the size of > > the structure needlessly. Eliminate them. > > >

Re: [Intel-gfx] [PATCH 4/4] drm: Constify the pretty-print functions

2013-06-18 Thread Laurent Pinchart
Hi Ville, On Friday 07 June 2013 18:43:07 ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > The structures and strings involved with various pretty-print functions > aren't meant to be modified, so make them all const. The exception is > drm_connector_enum_list which does get modifi

[Intel-gfx] [PATCH 4/5] intel: Wrap a long comment.

2013-06-18 Thread Eric Anholt
--- intel/intel_bufmgr_gem.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c index 5e087bc..b72a30b 100644 --- a/intel/intel_bufmgr_gem.c +++ b/intel/intel_bufmgr_gem.c @@ -200,7 +200,8 @@ struct _drm_intel_bo_gem {

[Intel-gfx] [PATCH 2/5] intel: Demote the stride field to 32 bits.

2013-06-18 Thread Eric Anholt
It comes in to us as a u32, and goes to the kernel as a u32, so storing as a long in between is silly. --- intel/intel_bufmgr_gem.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c index b06c99a..f068df6 100644 --- a/in

[Intel-gfx] [PATCH 3/5] intel: Pack the map_count in next to reloc_count on 64-bit.

2013-06-18 Thread Eric Anholt
Saves another 4 bytes on a drm_intel_bo. --- intel/intel_bufmgr_gem.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c index f068df6..5e087bc 100644 --- a/intel/intel_bufmgr_gem.c +++ b/intel/intel_bufmgr_gem.c @@ -174,11 +1

[Intel-gfx] [PATCH 5/5] intel: Mark checks for debug printing as unlikely.

2013-06-18 Thread Eric Anholt
There is a very tiny change (-120 bytes) in compiled code size as a result. --- intel/intel_bufmgr_gem.c | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c index b72a30b..ee73857 100644 --- a/intel/intel_bufmgr_gem.c

[Intel-gfx] [PATCH 1/5] intel: Stop storing two copies of the bo's gem handle.

2013-06-18 Thread Eric Anholt
--- intel/intel_bufmgr_gem.c | 91 +++- 1 file changed, 43 insertions(+), 48 deletions(-) diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c index 3c73068..b06c99a 100644 --- a/intel/intel_bufmgr_gem.c +++ b/intel/intel_bufmgr_gem.c @@ -14

Re: [Intel-gfx] [PATCH V1] ALSA: hda - Avoid choose same converter for unused pins

2013-06-18 Thread Wang, Xingchao
> -Original Message- > From: Takashi Iwai [mailto:ti...@suse.de] > Sent: Tuesday, June 18, 2013 10:14 PM > To: Wang Xingchao > Cc: daniel.vet...@ffwll.ch; alsa-de...@alsa-project.org; > intel-gfx@lists.freedesktop.org; Wang, Xingchao > Subject: Re: [PATCH V1] ALSA: hda - Avoid choose same

Re: [Intel-gfx] [PATCH V1] ALSA: hda - Avoid choose same converter for unused pins

2013-06-18 Thread Takashi Iwai
At Tue, 18 Jun 2013 21:42:14 +0800, Wang Xingchao wrote: > > For Intel Haswell HDMI codecs, the pins choose converter 0 by default. > This would cause conflict when playing audio on unused pins,the pin with > physical device connected would get audio data too. > i.e. Pin 0/1/2 default choose conve

[Intel-gfx] [PATCH V1] ALSA: hda - Avoid choose same converter for unused pins

2013-06-18 Thread Wang Xingchao
For Intel Haswell HDMI codecs, the pins choose converter 0 by default. This would cause conflict when playing audio on unused pins,the pin with physical device connected would get audio data too. i.e. Pin 0/1/2 default choose converter 0, pin 1 has HDMI monitor connected. when play audio on Pin 0 o

[Intel-gfx] Updated drm-intel-testing

2013-06-18 Thread Daniel Vetter
Hi all, So apparently I've failed to send the -testing update mail last Friday. But that tree turned out to be seriously broken anyway, so seems I've been lucky. Anyway, a big one with quite some patches: - more hangcheck work from Mika and Chris to prepare for arb robustness - trickle feed fixes

Re: [Intel-gfx] [PATCH] Fix intel_detect_pch() to work in xen environment.

2013-06-18 Thread G.R.
On Sun, Dec 23, 2012 at 2:51 PM, G.R. wrote: >>> Hi Jesse, I think I need to resend the patch with proper comment to >>> have it formally accepted. >>> Any guide line for formal patch submission? Do I need to start a >>> separate thread? >> >> No, just cc Daniel Vetter. >> >> -- >> Jesse Barnes, I

Re: [Intel-gfx] [PATCH] ALSA: hda - Avoid choose same converter for unused pins

2013-06-18 Thread Wang, Xingchao
Hi Takashi, > -Original Message- > From: Takashi Iwai [mailto:ti...@suse.de] > Sent: Tuesday, June 18, 2013 8:07 PM > To: Wang Xingchao > Cc: daniel.vet...@ffwll.ch; alsa-de...@alsa-project.org; > intel-gfx@lists.freedesktop.org; Wang, Xingchao > Subject: Re: [PATCH] ALSA: hda - Avoid choo

Re: [Intel-gfx] [PATCH] ALSA: hda - Avoid choose same converter for unused pins

2013-06-18 Thread Takashi Iwai
At Tue, 18 Jun 2013 16:32:01 +0800, Wang Xingchao wrote: > > For Intel Haswell HDMI codecs, the pins choose converter 0 by default. > This would cause conflict when playing audio on unused pins,the pin with > physical device connected would get audio data too. > i.e. Pin 0/1/2 default choose conve

Re: [Intel-gfx] [PATCH 1/3] drm/i915: Fix retrieval of hangcheck stats

2013-06-18 Thread Mika Kuoppala
Chris Wilson writes: > The default context is always supported (as it contains the global > hangcheck stats) and the contexts for hangcheck are not limited > to any ring. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=65845 > Signed-off-by: Chris Wilson Patches 1-3: Reviewed-by: Mi

Re: [Intel-gfx] [PATCH] ALSA: hda - Avoid choose same converter for unused pins

2013-06-18 Thread Daniel Vetter
On Tue, Jun 18, 2013 at 09:03:05AM +, Wang, Xingchao wrote: > Hi Takashi/Daniel, > > This patch could be used to replace below two: > http://lists.freedesktop.org/archives/intel-gfx/2013-June/029076.html > http://lists.freedesktop.org/archives/intel-gfx/2013-June/029077.html > > It could avoid

Re: [Intel-gfx] [patch] drm/i915: remove a superflous semi-colon

2013-06-18 Thread Daniel Vetter
On Tue, Jun 18, 2013 at 10:29:58AM +0300, Dan Carpenter wrote: > This macro doesn't need a semi-colon. > > Signed-off-by: Dan Carpenter Queued for -next, thanks for the patch. -Daniel > > diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h > index e5cc7c0..592b0eb9 1006

Re: [Intel-gfx] [PATCH 3/4] drm/i915: Add display audio routing APIs for ALSA

2013-06-18 Thread Wang, Xingchao
Hi Daniel, > -Original Message- > From: daniel.vet...@ffwll.ch [mailto:daniel.vet...@ffwll.ch] On Behalf Of > Daniel Vetter > Sent: Tuesday, June 18, 2013 3:13 PM > To: Wang, Xingchao > Cc: Daniel Vetter; Wang Xingchao; Takashi Iwai; alsa-de...@alsa-project.org; > intel-gfx; David Hennings

Re: [Intel-gfx] [PATCH] ALSA: hda - Avoid choose same converter for unused pins

2013-06-18 Thread Wang, Xingchao
Hi Takashi/Daniel, This patch could be used to replace below two: http://lists.freedesktop.org/archives/intel-gfx/2013-June/029076.html http://lists.freedesktop.org/archives/intel-gfx/2013-June/029077.html It could avoid lots of changes in gfx side. I test the patch and it could fix the same rout

[Intel-gfx] [PATCH] ALSA: hda - Avoid choose same converter for unused pins

2013-06-18 Thread Wang Xingchao
For Intel Haswell HDMI codecs, the pins choose converter 0 by default. This would cause conflict when playing audio on unused pins,the pin with physical device connected would get audio data too. i.e. Pin 0/1/2 default choose converter 0, pin 1 has HDMI monitor connected. when play audio on Pin 0 o

Re: [Intel-gfx] [PATCH 0/8] Don't let the ghost eDP haunt us

2013-06-18 Thread Zoltan Nyul
This patch series seems ok for me. Reviewed-by: Zoltan Nyul Hi This week I sent a patch called "drm/i915: propagate errors from intel_dp_init_connector" to fix a Haswell bug that was preventing my machine from booting. Chris provided a nice suggestion for it, so this series should implement h

Re: [Intel-gfx] [PATCH 3/4] drm/i915: Add display audio routing APIs for ALSA

2013-06-18 Thread Daniel Vetter
On Mon, Jun 17, 2013 at 12:52:41PM +, Wang, Xingchao wrote: > Hi Daniel, > > > -Original Message- > > From: Daniel Vetter [mailto:daniel.vet...@ffwll.ch] > > Sent: Saturday, June 15, 2013 3:18 AM > > To: Wang Xingchao > > Cc: Takashi Iwai; alsa-de...@alsa-project.org; intel-gfx; David H