From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Pitcher Subject: Re: Switcheroo support in i915 Date: Mon, 08 Sep 2014 22:41:46 +0100 Message-ID: <9598117.BCh740qaWU@tomac> References: <17412193.9crM6G9hXq@tomac> <20140908080650.GC19343@nuc-i3427.alporthouse.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from s1.neomailbox.net (s1.neomailbox.net [5.148.176.57]) by gabe.freedesktop.org (Postfix) with ESMTP id 8CF286E037 for ; Mon, 8 Sep 2014 14:41:58 -0700 (PDT) In-Reply-To: <20140908080650.GC19343@nuc-i3427.alporthouse.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Chris Wilson Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Monday 08 Sep 2014 09:06:50 Chris Wilson wrote: > On Sat, Sep 06, 2014 at 02:01:31PM +0100, Tom Pitcher wrote: > > I've been attempting to solve a bug with i915 & switcheroo on many Macbook > > Pros (some with nvidia discrete cards, others radeon), report here: > > > > https://bugs.freedesktop.org/show_bug.cgi?id=61115 > > > > Basically, after switching to i915 using switcheroo the display isn't > > connected. It was mentioned that some kind of reprobing was needed, so I > > tried > > the following: > Right, I guess the output states do need to be repolled as well. But we > need to effectively re-initialised the LVDS (and possibly all the other > connectors, but it is likely to only be the panel connectors that are > muxed). > > This will require some rejigging of intel_lvds_init(), though it should > be safe enough to destroy and attempt to recreate it here: > > for_each_connector_safe() > if (connector->type == DRM_MODE_CONNECOR_LVDS) > drm_connector_destroy(connector); > > for_each_encoder_safe() > if (encoder->type == DRM_MODE_ENCODER_LVDS) > drm_encoder_destroy(encoder); > > intel_lvds_init(); > -Chris Thanks Chris, I turned what you provided into this: diff --git a/drivers/gpu/drm/i915/i915_dma.c b/drivers/gpu/drm/i915/i915_dma.c index d443441..edcecc9 100644 --- a/drivers/gpu/drm/i915/i915_dma.c +++ b/drivers/gpu/drm/i915/i915_dma.c @@ -1281,6 +1281,26 @@ static void i915_switcheroo_set_state(struct pci_dev *pdev, enum vga_switcheroo_ } } +static void +i915_switcheroo_reprobe(struct pci_dev *pdev) +{ + struct drm_device *dev = pci_get_drvdata(pdev); + struct drm_encoder *encoder, *encoder_tmp; + struct drm_connector *connector, *connector_tmp; + + intel_fbdev_output_poll_changed(dev); + + list_for_each_entry_safe(connector, connector_tmp, &dev- >mode_config.connector_list, head) + if (connector->connector_type == DRM_MODE_CONNECTOR_LVDS) + connector->funcs->destroy(connector); + + list_for_each_entry_safe(encoder, encoder_tmp, &dev- >mode_config.encoder_list, head) + if (encoder->encoder_type == DRM_MODE_ENCODER_LVDS) + intel_encoder_destroy(encoder); + + intel_lvds_init(dev); +} + static bool i915_switcheroo_can_switch(struct pci_dev *pdev) { struct drm_device *dev = pci_get_drvdata(pdev); @@ -1295,7 +1315,7 @@ static bool i915_switcheroo_can_switch(struct pci_dev *pdev) static const struct vga_switcheroo_client_ops i915_switcheroo_ops = { .set_gpu_state = i915_switcheroo_set_state, - .reprobe = NULL, + .reprobe = i915_switcheroo_reprobe, .can_switch = i915_switcheroo_can_switch, }; This doesn't seem to have altered the behaviour - though my code is probably bad. Any ideas? (Apologies if this message comes through twice - my mail host is playing up).