Re: [PATCH] gma500: Add option to disable LVDS

[Greg KH]

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@xxxxxxxxx> 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@xxxxxxxxx>
> 
> 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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/