public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Seth Forshee <seth.forshee@canonical.com>
To: David Woodhouse <dwmw2@infradead.org>
Cc: Andreas Heider <andreas@meetr.de>,
	Matthew Garrett <mjg59@srcf.ucam.org>,
	Arun Raghavan <arun.raghavan@collabora.co.uk>,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH] apple-gmux: Restore switch registers on suspend/resume
Date: Mon, 30 Jul 2012 09:05:33 -0500	[thread overview]
Message-ID: <20120730140533.GA18692@thinkpad-t410> (raw)
In-Reply-To: <1343601889.2487.3.camel@shinybook.infradead.org>

On Sun, Jul 29, 2012 at 11:44:49PM +0100, David Woodhouse wrote:
> On Sun, 2012-07-29 at 09:46 +0200, Andreas Heider wrote:
> > The gmux code you're using is a bit outdated, but if you use 
> > git://kernel.ubuntu.com/sforshee/linux.git gmux-switcheroo and connect 
> > an external display (I tested it with a DP one) you should be able to 
> > boot to DIS and switch to the IGD and get output on the external display. 
> 
> Matthew's version has changes to the core vga_switcheroo code too, to
> add the ->client_active() method which lets it pick the right client at
> init time. I'll let you work out how to merge the two, but in the
> meantime here's what I needed to do to Matthew's one to make it work...

But that code is only needed if you've hacked in the port writes to
switch to IGD in the early boot code, right? It's fine as a local hack
when trying to get things working, but I don't see it as something
that's going to be mainlined. Putting togethter something we can
mainline is my goal with the patches I've got queued up in the
referenced repository.

Fwiw on the Macbook Pro 8,2, using my tree with the patches from
http://www.codon.org.uk/~mjg59/tmp/radeon_efi/ and no port writes hacked
into early boot, the active client as seen by vga_switcheroo and
apple_gmux are the same.

Seth


  reply	other threads:[~2012-07-30 14:05 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-10  3:39 [PATCH] apple-gmux: Restore switch registers on suspend/resume Arun Raghavan
2012-07-10 13:35 ` Seth Forshee
2012-07-10 16:05 ` Matthew Garrett
2012-07-10 16:35   ` Seth Forshee
2012-07-11  0:25     ` Andreas Heider
2012-07-29  0:42       ` David Woodhouse
2012-07-29  1:18         ` David Woodhouse
2012-07-29  7:46           ` Andreas Heider
2012-07-29 19:05             ` David Woodhouse
2012-07-29 19:34               ` Andreas Heider
2012-07-29 22:44             ` David Woodhouse
2012-07-30 14:05               ` Seth Forshee [this message]
2012-07-29 19:39           ` Matthew Garrett
2012-07-29 20:33             ` David Woodhouse
2012-07-29 20:52             ` David Woodhouse
2012-07-29 20:59               ` Matthew Garrett
2012-07-31 15:18                 ` Seth Forshee
2012-07-31 17:07                   ` Seth Forshee
2012-08-01 15:35                   ` David Woodhouse
2012-08-01 15:59                     ` Seth Forshee
2012-08-01 16:06                       ` Andreas Heider
2012-08-01 19:41                       ` David Woodhouse
2012-08-01 19:52                         ` Seth Forshee
2012-08-01 19:43                       ` David Woodhouse
2012-08-01 19:52                         ` Matthew Garrett
2012-08-01 19:56                         ` Seth Forshee
2012-08-01 19:56                         ` Andreas Heider

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120730140533.GA18692@thinkpad-t410 \
    --to=seth.forshee@canonical.com \
    --cc=andreas@meetr.de \
    --cc=arun.raghavan@collabora.co.uk \
    --cc=dwmw2@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mjg59@srcf.ucam.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox