On Sun, Jun 14, 2009 at 11:30:27PM +0200, ext Krzysztof Helt wrote: > On Thu, 4 Jun 2009 20:52:31 +0300 > Imre Deak <imre.deak@xxxxxxxxx> wrote: > > > From: arun c <arun.edarath@xxxxxxxxx> > > > > omap2evm LCD supports VGA and QVGA resolution, by default its in VGA mode. > > > > Signed-off-by: Arun C <arunedarath@xxxxxxxxxxxxxxxxxxxx> > > Signed-off-by: Tony Lindgren <tony@xxxxxxxxxxx> > > Fixed-by: Jarkko Nikula <jarkko.nikula@xxxxxxxxx> > > Fixed-by: David Brownell <dbrownell@xxxxxxxxxxxxxxxxxxxxx> > > Signed-off-by: Imre Deak <imre.deak@xxxxxxxxx> > > --- > > drivers/video/omap/Makefile | 1 + > > drivers/video/omap/lcd_omap2evm.c | 189 +++++++++++++++++++++++++++++++++++++ > > 2 files changed, 190 insertions(+), 0 deletions(-) > > create mode 100644 drivers/video/omap/lcd_omap2evm.c > > > > diff --git a/drivers/video/omap/Makefile b/drivers/video/omap/Makefile > > index 7a37b03..c2475e3 100644 > > --- a/drivers/video/omap/Makefile > > +++ b/drivers/video/omap/Makefile > > @@ -27,6 +27,7 @@ objs-y$(CONFIG_MACH_OMAP_OSK) += lcd_osk.o > > > > objs-y$(CONFIG_MACH_OMAP_APOLLON) += lcd_apollon.o > > objs-y$(CONFIG_MACH_OMAP_2430SDP) += lcd_2430sdp.o > > +objs-y$(CONFIG_MACH_OMAP2EVM) += lcd_omap2evm.o > > objs-y$(CONFIG_FB_OMAP_LCD_MIPID) += lcd_mipid.o > > > > omapfb-objs := $(objs-yy) > diff --git a/drivers/video/omap/lcd_omap2evm.c b/drivers/video/omap/lcd_omap2evm.c > new file mode 100644 > index 0000000..2fc46c2 > --- /dev/null > +++ b/drivers/video/omap/lcd_omap2evm.c > @@ -0,0 +1,189 @@ > +/* > + * LCD panel support for the MISTRAL OMAP2EVM board > + * > + * Author: Arun C <arunedarath@xxxxxxxxxxxxxxxxxxxx> > + * > + * Derived from drivers/video/omap/lcd_omap3evm.c > + * Derived from drivers/video/omap/lcd-apollon.c > + * > + * This program is free software; you can redistribute it and/or modify it > + * under the terms of the GNU General Public License as published by the > + * Free Software Foundation; either version 2 of the License, or (at your > + * option) any later version. > + * > + * This program is distributed in the hope that it will be useful, but > + * WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License along > + * with this program; if not, write to the Free Software Foundation, Inc., > + * 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. > + */ > + > +#include <linux/module.h> > +#include <linux/platform_device.h> > +#include <linux/gpio.h> > +#include <linux/i2c/twl4030.h> > + > +#include <mach/mux.h> > +#include <mach/omapfb.h> > +#include <asm/mach-types.h> > + > +#define LCD_PANEL_ENABLE_GPIO 154 > +#define LCD_PANEL_LR 128 > +#define LCD_PANEL_UD 129 > +#define LCD_PANEL_INI 152 > +#define LCD_PANEL_QVGA 148 > +#define LCD_PANEL_RESB 153 > + > +#define LCD_XRES 480 > +#define LCD_YRES 640 > +#define LCD_PIXCLOCK_MAX 20000 /* in kHz */ > > You can drop defines used only once. There are no such defines for some other panels > in your patches. Ok, fixing it. --Imre -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html