Re: [Intel-gfx] [PATCH] drm/i915: Valleyview doesn't have rc6+ or rc6++

2012-09-18 Thread Rodrigo Vivi
Feel free to use: Reviewed-by: Rodrigo Vivi On Mon, Sep 17, 2012 at 9:10 PM, Ben Widawsky wrote: > I do not currently have a VLV to test this on, but hopefully it only > removes information from debugfs, sysfs, and prevents enabling an > unsupported mode. > > CC: Jesse Barnes > Signed-off-by: B

Re: [Intel-gfx] [PATCH 7/8] drm/i915: extract pch_pll_set from ironlake_crtc_mode_set

2012-09-18 Thread Paulo Zanoni
Hi 2012/9/12 Daniel Vetter : > On Wed, Sep 12, 2012 at 10:06:35AM -0300, Paulo Zanoni wrote: >> From: Paulo Zanoni [ ... ] >> + if (HAS_PCH_IBX(dev) || HAS_PCH_CPT(dev)) { > > Since the plan is to move all the hsw_crtc_stuff out into it's own > function I'd prefer a !HAS_PCH_LPT check here.

Re: [Intel-gfx] [PATCH 4/4] drm/i915/dp: Be smarter about connection sense for branch devices

2012-09-18 Thread Adam Jackson
On Tue, 2012-09-18 at 10:58 -0400, Adam Jackson wrote: > If there's no downstream device, DPCD success is good enough. If > there's a hotplug-capable downstream device, count the number of > connected sinks in DP_SINK_STATUS and return success if it's non-zero. > Otherwise, probe DDC and report ap

Re: [Intel-gfx] Valid DP connection without EDID?

2012-09-18 Thread Takashi Iwai
At Tue, 18 Sep 2012 10:48:11 -0400, Adam Jackson wrote: > > On Tue, 2012-09-18 at 16:32 +0200, Takashi Iwai wrote: > > > Woohoo, the patch works! > > \o/ > > > Could you brush up and resend the patches for merging to 3.7 kernel? > > Or, would you like to fix the multiple branch devices first? >

[Intel-gfx] [PATCH 4/4] drm/i915/dp: Be smarter about connection sense for branch devices

2012-09-18 Thread Adam Jackson
If there's no downstream device, DPCD success is good enough. If there's a hotplug-capable downstream device, count the number of connected sinks in DP_SINK_STATUS and return success if it's non-zero. Otherwise, probe DDC and report appropriately. v2: Check DP_SINK_STATUS instead of something unr

[Intel-gfx] [PATCH 3/4] drm/i915/dp: Fetch downstream port info if needed during DPCD fetch

2012-09-18 Thread Adam Jackson
v2: Fix parenthesis mismatch, spotted by Jani Nikula Tested-by: Takashi Iwai Signed-off-by: Adam Jackson --- drivers/gpu/drm/i915/intel_dp.c | 25 - 1 files changed, 20 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/i

[Intel-gfx] [PATCH 2/4] drm/dp: Update DPCD defines

2012-09-18 Thread Adam Jackson
Sources: DP, eDP, and DP interop specs, and a VESA slideshow about DP 1.2 for the MST bits. Tested-by: Takashi Iwai Signed-off-by: Adam Jackson --- include/drm/drm_dp_helper.h | 60 --- 1 files changed, 56 insertions(+), 4 deletions(-) diff --git a/inc

[Intel-gfx] [PATCH 1/4] drm: Export drm_probe_ddc()

2012-09-18 Thread Adam Jackson
Tested-by: Takashi Iwai Signed-off-by: Adam Jackson --- drivers/gpu/drm/drm_edid.c |3 ++- include/drm/drm_crtc.h |1 + 2 files changed, 3 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index b7ee230..a1895ba 100644 --- a/drivers/g

Re: [Intel-gfx] Valid DP connection without EDID?

2012-09-18 Thread Adam Jackson
On Tue, 2012-09-18 at 16:32 +0200, Takashi Iwai wrote: > Woohoo, the patch works! \o/ > Could you brush up and resend the patches for merging to 3.7 kernel? > Or, would you like to fix the multiple branch devices first? I'm not too worried about branch devices with multiple downstreams just yet

Re: [Intel-gfx] Valid DP connection without EDID?

2012-09-18 Thread Takashi Iwai
At Tue, 18 Sep 2012 10:04:33 -0400, Adam Jackson wrote: > > On Tue, 2012-09-18 at 13:01 +0200, Takashi Iwai wrote: > > > > I started a patch series for this a bit ago, I'll send it on > > > momentarily. > > > > Thanks! I evaluated it now (with a typo fix suggested by Jani). > > Unfortunately, i

Re: [Intel-gfx] Valid DP connection without EDID?

2012-09-18 Thread Adam Jackson
On Tue, 2012-09-18 at 13:01 +0200, Takashi Iwai wrote: > > I started a patch series for this a bit ago, I'll send it on > > momentarily. > > Thanks! I evaluated it now (with a typo fix suggested by Jani). > Unfortunately, it doesn't improve the situation. > > The fetch of downstream ports succe

Re: [Intel-gfx] Valid DP connection without EDID?

2012-09-18 Thread Takashi Iwai
At Mon, 17 Sep 2012 12:24:36 -0400, Adam Jackson wrote: > > On Mon, 2012-09-17 at 11:32 +0200, Takashi Iwai wrote: > > At Mon, 17 Sep 2012 19:11:34 +1000, > > Dave Airlie wrote: > > > On Mon, Sep 17, 2012 at 6:28 PM, Takashi Iwai wrote: > > > > At Mon, 17 Sep 2012 18:21:47 +1000, > > > > Dave Air

Re: [Intel-gfx] [PATCH] drm/i915: fix OOPS in lid_notify

2012-09-18 Thread Chris Wilson
On Mon, 17 Sep 2012 14:38:14 -0700, Jesse Barnes wrote: > On Mon, 17 Sep 2012 22:27:21 +0200 > Daniel Vetter wrote: > > > This goes back to > > > > commit c1c7af60892070e4b82ad63bbfb95ae745056de0 > > Author: Jesse Barnes > > Date: Thu Sep 10 15:28:03 2009 -0700 > > > > drm/i915: force

[Intel-gfx] [PATCH] drm/i915: ioremap the right range for the registers on gen2

2012-09-18 Thread Chris Wilson
Fixes regression from commit 1ca673954355eb4105ef7fccdd770e69d4d3d1ef Author: Chris Wilson Date: Fri Sep 14 11:57:46 2012 +0100 drm/i915: Limit the ioremap of the PCI bar to the registers which only mapped 64k of the register BAR when the full 512k was required. Signed-off-by: Chris Wilso

[Intel-gfx] [PATCH] drm/i915: Assert that the exec object lookup table is a power-of-two

2012-09-18 Thread Chris Wilson
As we make the simplification of using a power-of-two size for the execbuffer handle-to-object TLB, we should validate that this is actually true and so clarify that premise. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/i915_gem_execbuffer.c |1 + 1 file changed, 1 insertion(+) diff

Re: [Intel-gfx] [PATCH 3/4] drm/i915/dp: Fetch downstream port info if needed during DPCD fetch

2012-09-18 Thread Jani Nikula
On Mon, 17 Sep 2012, Adam Jackson wrote: > Signed-off-by: Adam Jackson > --- > drivers/gpu/drm/i915/intel_dp.c | 25 - > 1 files changed, 20 insertions(+), 5 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c > index ace757a.

Re: [Intel-gfx] [PATCH 2/2] [RESEND] console: implement lockdep support for console_lock

2012-09-18 Thread Jani Nikula
On Tue, 18 Sep 2012, Daniel Vetter wrote: > +#ifdef CONFIG_LOCKDEP > +struct lockdep_map console_lock_dep_map = { > + .name = "console_lock" > +}; > +#endif static? BR, Jani. ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.

Re: [Intel-gfx] ironlake wierdness bisection result

2012-09-18 Thread Dave Airlie
On Tue, Sep 18, 2012 at 4:43 PM, Dave Airlie wrote: > Okay I spent today bisecting this and although I'm not 100% sure I got > it right the whole way down, I think where I landed seems sane, so I > said I'd share > > This seems to cause the blank rendering zones people have been reporting. Though

[Intel-gfx] ironlake wierdness bisection result

2012-09-18 Thread Dave Airlie
Okay I spent today bisecting this and although I'm not 100% sure I got it right the whole way down, I think where I landed seems sane, so I said I'd share This seems to cause the blank rendering zones people have been reporting. Dave. commit c07496fa61f4c5cb2addd1c57f6b22fcaeea2eeb Author: Danie