From: Kristoffer Ericson <kristoffer.ericson@gmail.com>
To: Patrik Jakobsson <patrik.r.jakobsson@gmail.com>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>,
alan@linux.intel.com, greg@kroah.com,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] gma500: Add option to disable LVDS
Date: Wed, 11 May 2011 15:35:00 +0200 [thread overview]
Message-ID: <20110511133500.GA5358@Boggieman> (raw)
In-Reply-To: <BANLkTikz+PrQY2egkPTiacDSH5-Gkqkbww@mail.gmail.com>
On Wed, May 11, 2011 at 01:18:07AM +0200, Patrik Jakobsson wrote:
> On Tue, May 10, 2011 at 11:46 PM, Alan Cox <alan@lxorguk.ukuu.org.uk> 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.
> >
> > Alan
> >
>
> This is for the FIT-PC2.
>
> I agree, it's not a good way to handle it, but since I haven't been
> able to pinpoint the problem (and this is staging) it's something that
> works for now. Perhaps a nicer way to get around it is by checking
> against the dmi system id. That way it would be transparent to the
> user. But you might prefer we fix the real problem instead of hacking
> around it.
>
> When both LVDS and SDVO is enabled the driver tries to set up both
> pipes and LVDS only finds a bogus mode of 0x0 pixels. Not sure why
> LVDS takes over instead of SDVO since it has valid modes, but my
> screen just goes into powersave when no proper PLL can be set for the
> bogus mode.
>
> I'll do some more debugging tomorrow.
Im getting similiar problems although not to that extreme. It defaults back to
lowest common resolution which seems to be 1024x768, so in that sense I would
appreciate being able to turn it off. That being said, its an issue which will
eventually go away and therefore theres little point in implementing a workaround.
Best wishes
Kristoffer
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
prev parent reply other threads:[~2011-05-11 16:20 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
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 [this message]
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=20110511133500.GA5358@Boggieman \
--to=kristoffer.ericson@gmail.com \
--cc=alan@linux.intel.com \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=greg@kroah.com \
--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