public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <greg@kroah.com>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Patrik Jakobsson <patrik.r.jakobsson@gmail.com>,
	alan@linux.intel.com, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] gma500: Add option to disable LVDS
Date: Tue, 10 May 2011 14:59:21 -0700	[thread overview]
Message-ID: <20110510215921.GB12854@kroah.com> (raw)
In-Reply-To: <20110510224602.73b24d0a@lxorguk.ukuu.org.uk>

On Tue, May 10, 2011 at 10:46:02PM +0100, Alan Cox wrote:
> On Tue, 10 May 2011 23:20:38 +0200
> Patrik Jakobsson <patrik.r.jakobsson@gmail.com> wrote:
> 
> > This patch adds the option to load the driver without LVDS support. Some platforms with the gma500 have nothing hooked up to LVDS but only uses SDVO. LVDS is still detected and added as an encoder which causes SDVO mode-setting to fail.
> > 
> > Signed-off-by: Patrik Jakobsson <patrik.r.jakobsson@gmail.com>
> 
> What platform is this - I'd really like to keep such things automatic and
> also if possible make the code handle the case not blow up. The user
> shouldn't have to deal with such funnies.

I agree, there should be a way to automatically detect this, we don't
need another-module-parameter for something like this if at all
possible.

thanks,

greg k-h

  reply	other threads:[~2011-05-10 21:59 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-10 21:20 [PATCH] gma500: Add option to disable LVDS Patrik Jakobsson
2011-05-10 21:46 ` Alan Cox
2011-05-10 21:59   ` Greg KH [this message]
2011-05-10 23:18   ` Patrik Jakobsson
2011-05-11  9:10     ` Alan Cox
2011-05-11 12:41       ` Patrik Jakobsson
2011-05-11 16:59         ` Patrik Jakobsson
2011-05-11 13:02     ` Matthew Garrett
2011-05-11 13:35     ` Kristoffer Ericson

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=20110510215921.GB12854@kroah.com \
    --to=greg@kroah.com \
    --cc=alan@linux.intel.com \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=patrik.r.jakobsson@gmail.com \
    /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