Re: [PATCH 2/2] omap3: devkit8000: Add and use 4.3 inch display

[Daniel Morsing <daniel.morsing@gmail.com>]

On Fri, 2011-02-18 at 15:07 +0200, Tomi Valkeinen wrote:
> On Fri, 2011-02-18 at 06:55 -0600, Daniel Morsing wrote:
> > On Fri, 2011-02-18 at 13:36 +0100, Thomas Weber wrote:
> > > Hello Daniel,
> > > 
> > > Am 12.02.2011 18:02, schrieb Daniel Morsing:
> > > > This patch adds a generic panel entry for the 4.3 inch display that is
> > > > sold with the devkit8000 and modifies the board file to use this
> > > > display.
> > > > 
> > > > 
> > > 
> > > So only the 4.3 inch display can be used with the devkit8000?
> > > There are also Devkit8000 with 5.6 and 7 inch displays.
> > > 
> > > Regards,
> > > Thomas
> > > 
> > 
> > Yeah, I'll admit that the patch is a bit one-sided. I couldn't find a
> > way to specify a board using a class of displays with separate timing
> > settings.
> > 
> > One way to do it would be to add the settings to modedb, but modedb is
> > really only for standardized display settings. The other way to do it
> > would be to add a kernel config for which display you have connected to
> > the board, but I think that is a bad idea, because it makes everything
> > more complicated and switching out displays would require recompiling
> > the kernel.
> > 
> > Maybe Tomi or the fbdev guys have some input on how to best do this?
> 
> There's no proper way for this currently. And I'm not even sure what
> would be a good way to do this.
> 
> Optimally the board code could read some configuration value from eeprom
> or where-ever, which would tell the configuration of the board. And the
> board file could then configure the correct display.
> 
> Another way would be to add a devkit8000 specific kernel parameter,
> given from bootloader, which would tell the configuration. The board
> file would use this parameter to configure the correct display.
> 
>  Tomi
> 
> 

How about making it possible to give a generic panel name to the
omapfb.mode boot parameter? This would also solve the problem for any
future boards with different non-standard displays.

Regards,
Daniel Morsing