From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomi Valkeinen Subject: Re: [PATCH 2/2] omap3: devkit8000: Add and use 4.3 inch display Date: Fri, 18 Feb 2011 15:07:46 +0200 Message-ID: <1298034466.24062.99.camel@deskari> References: <1297530130-10355-1-git-send-email-daniel.morsing@gmail.com> <1297530130-10355-3-git-send-email-daniel.morsing@gmail.com> <4D5E67E7.7040603@gmail.com> <1298033744.2323.9.camel@morsing> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: Received: from bear.ext.ti.com ([192.94.94.41]:45115 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752292Ab1BRNHw (ORCPT ); Fri, 18 Feb 2011 08:07:52 -0500 In-Reply-To: <1298033744.2323.9.camel@morsing> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Daniel Morsing Cc: Thomas Weber , Tony Lindgren , Bryan Wu , "linux-fbdev@vger.kernel.org" , "linux-omap@vger.kernel.org" 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