On 2009.08.25 21:20:18 -0700, Linus Torvalds wrote: > > And how about MacBook 2.1, which apparently also goes black? >
Linus, sorry for reply this old thread. I just handed on one MacBook with 945GM, and tried to find out the reason of black screen. It is TV detection that caused it. MacBook routes integrated TV DAC to mini-DVI port too, like for VGA. Our TV detect does load detect, which trys to set a mode onto TV port and checked back its status. This TV load-detect makes the LVDS black on MacBook. It looks our load detect function has problem that when we pick up one crtc, we don't check if its has a fb. During first TV detect, choosed crtc obviously has no fb, it would fail in modesetting when setting plane surface. This might have effect on the purpose of load- detect process. I'll see how it should be fixed. But even if I hacked plane setting with all zeros, screen still goes black on my MacBook, although TV detect return is right that there's none found. So this blank issue is within detect process, if you try to load fbcon or X later, setting mode is fine on LVDS. That's why build fbcon in kernel might workaround this issue. I don't know much about our TV detect method details, but looks like here we should check current TV status when system start. Otherwise it seems we can't stopping blank the screen...So I did a temp hacking like, diff --git a/drivers/gpu/drm/i915/intel_tv.c b/drivers/gpu/drm/i915/intel_tv.c index 5b1c9e9..f3ecbaa 100644 --- a/drivers/gpu/drm/i915/intel_tv.c +++ b/drivers/gpu/drm/i915/intel_tv.c @@ -1451,8 +1451,18 @@ intel_tv_detect(struct drm_connector *connector) struct intel_output *intel_output = to_intel_output(connector); struct intel_tv_priv *tv_priv = intel_output->dev_priv; struct drm_encoder *encoder = &intel_output->enc; + struct drm_i915_private *dev_priv = connector->dev->dev_private; int dpms_mode; int type = tv_priv->type; + static int first_detect = 1; + + if (first_detect) { + first_detect = 0; + if (!(I915_READ(TV_CTL) & TV_ENC_ENABLE)) + return connector_status_disconnected; + else + return connector_status_connected; + } mode = reported_modes[0]; drm_mode_set_crtcinfo(&mode, CRTC_INTERLACE_HALVE_V); which makes LVDS working, but I haven't tested with mini-DVI <-> S-Video. I'm not sure if this is ideal enough for this, but first get port states in system startup for initial setup, and userspace could check and set required config later seems sane to me, as you have also suggested to do. This patch doesn't aim to be upstream, we may set the state in priv struct to check boot state, but I'd like to hear others' comment about this. thanks. -- Open Source Technology Center, Intel ltd. $gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
signature.asc
Description: Digital signature
------------------------------------------------------------------------------ Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day trial. Simplify your report design, integration and deployment - and focus on what you do best, core application coding. Discover what's new with Crystal Reports now. http://p.sf.net/sfu/bobj-july
-- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel