Re: [Intel-gfx] [PATCH] drm/i915: rip out the HWSTAM missed irq workaround

2012-01-09 Thread Daniel Vetter
On Mon, Jan 09, 2012 at 06:09:13PM -0800, Keith Packard wrote: > We could leave the code for spinning in place and simply control that > with a module parameter. That would allow us to disable it now, and if > we find problems (or are particularly paranoid) we could disable it > before 3.3 ships wi

Re: [Intel-gfx] [PATCH] drm/i915: set AUD_CONFIG N_value_index for DisplayPort

2012-01-09 Thread Wu Fengguang
On Mon, Jan 09, 2012 at 09:22:47AM -0800, Keith Packard wrote: > On Mon, 9 Jan 2012 21:17:17 +0800, Wu Fengguang > wrote: > > > +#define IBX_AUD_CONFIG_A 0xe2000 > > +#define CPT_AUD_CONFIG_A 0xe5000 > > These register addresses match the docs. > > > + I915_WRITE(

[Intel-gfx] [PATCH] Android port of intel-gpu-tools

2012-01-09 Thread Sateesh Kavuri
Added support for Android. Changes include fixes for compilation issues related to Android using an older version of GCC compiler (ver 4.3.3) while the latest version of intel-gpu-tools confirms to GCC ver 4.5.2 (C99 standard functions), using functions like getline(). Fixed such functions, hea

Re: [Intel-gfx] [PATCH] drm/i915: rip out the HWSTAM missed irq workaround

2012-01-09 Thread Keith Packard
On Tue, 10 Jan 2012 00:39:52 +0100, Daniel Vetter wrote: > I honestly don't trust my patch, so I'd like to give it as much validation > as possible. Which means: > - Shove it into -next and beat on it there. We can ship current 3.3 with > Eric's workaround - it's not great but at least this wor

Re: [Intel-gfx] [PATCH] drm/i915: rip out the HWSTAM missed irq workaround

2012-01-09 Thread Daniel Vetter
On Mon, Jan 09, 2012 at 02:00:47PM -0800, Keith Packard wrote: > On Thu, 5 Jan 2012 23:11:53 +0100, Daniel Vetter > wrote: > > > With the new ducttape of much finer quality, this seems to be no > > longer necessary. > > > > Tested on my ivb and snb machine with the usual suspects of testcases.

Re: [Intel-gfx] [PATCH] drm/i915: Print debugfs object list sizes in kiB instead of bytes.

2012-01-09 Thread Keith Packard
On Tue, 20 Dec 2011 16:58:55 +, David Woodhouse wrote: > In both the seq_printf, and the subject line, you have different > misspellings of 'KiB'. I fixed the misspellings and merged this to my drm-intel-next-fixes pending branch. -- keith.pack...@intel.com pgpLCgolHijGT.pgp Description:

Re: [Intel-gfx] [PATCH] drm/i915: rip out the HWSTAM missed irq workaround

2012-01-09 Thread Keith Packard
On Thu, 5 Jan 2012 23:11:53 +0100, Daniel Vetter wrote: > With the new ducttape of much finer quality, this seems to be no > longer necessary. > > Tested on my ivb and snb machine with the usual suspects of testcases. Eric suggested that unless we have evidence that the new work-around fixes

Re: [Intel-gfx] [PATCH] drm/i915: set AUD_CONFIG N_value_index for DisplayPort

2012-01-09 Thread Keith Packard
On Mon, 9 Jan 2012 21:17:17 +0800, Wu Fengguang wrote: > +#define IBX_AUD_CONFIG_A 0xe2000 > +#define CPT_AUD_CONFIG_A 0xe5000 These register addresses match the docs. > + I915_WRITE(aud_config, 1 << 29); /* N value index, 0x1 = DP */ Please don't use consta

Re: [Intel-gfx] [PATCH] Add tools/intel_infoframes

2012-01-09 Thread Daniel Vetter
On Mon, Jan 09, 2012 at 12:42:06PM -0200, Eugeni Dodonov wrote: > On Mon, Jan 9, 2012 at 12:03, wrote: > > > From: Paulo Zanoni > > > > This is a command-line tool that allows us to display and modify the > > InfoFrames we send. > > > > Signed-off-by: Paulo Zanoni > > > > Very interesting! >

Re: [Intel-gfx] [PATCH] drm/i915: Compute a loadavg as the number of pending requests

2012-01-09 Thread Eugeni Dodonov
On Sun, Jan 8, 2012 at 16:17, Daniel Vetter wrote: > I think if this really should be merged upstream, it needs to be more > optional (i.e. only fire it up if someone reads out the load_avg). People > will complain if we wake up 60x per second on a mostly idle machine just > for statistics ... >

Re: [Intel-gfx] [PATCH] Add tools/intel_infoframes

2012-01-09 Thread Eugeni Dodonov
On Mon, Jan 9, 2012 at 12:03, wrote: > From: Paulo Zanoni > > This is a command-line tool that allows us to display and modify the > InfoFrames we send. > > Signed-off-by: Paulo Zanoni > Very interesting! I think that it would be more interesting to use command line options instead of interac

[Intel-gfx] [PATCH] Add tools/intel_infoframes

2012-01-09 Thread przanoni
From: Paulo Zanoni This is a command-line tool that allows us to display and modify the InfoFrames we send. Signed-off-by: Paulo Zanoni --- Hi Before we add Kernel interfaces to deal with the infoframes, I decided to write this tool. It allows us to quickly test everything without requiring a

[Intel-gfx] [PATCH] drm/i915: set AUD_CONFIG N_value_index for DisplayPort

2012-01-09 Thread Wu Fengguang
It should be programmed to "0" for HDMI or "1" for DisplayPort. This enables DisplayPort audio for - HP EliteBook 8460p (whose BIOS does not set the N_value_index bit for us) - DisplayPort monitor hot plugged after boot (otherwise most BIOS will fill the N_value_index bit for us) Signed-off