On Thu, 12 Jun 2008 05:24:32 -0500, [EMAIL PROTECTED] wrote:
From: Iqbal Shareef [EMAIL PROTECTED]
Separate out 3430 LCD panel support from 2430 SDP.
Currently the support for the LCD on 3430SDP is added in the
lcd_2430sdp.c
file. This patch separates it out to a new file.
You're at least missing 3430sdp defconfig updates, Kconfig changes
in drivers/media/video/omap and the removal of conditional code in
lcd_2430sdp.c
Some more comments below
Signed-off-by: Iqbal Shareef [EMAIL PROTECTED]
---
arch/arm/mach-omap2/board-3430sdp.c |2 +-
drivers/video/omap/Makefile |2 +-
drivers/video/omap/lcd_3430sdp.c| 161
+++
3 files changed, 163 insertions(+), 2 deletions(-)
create mode 100644 drivers/video/omap/lcd_3430sdp.c
diff --git a/arch/arm/mach-omap2/board-3430sdp.c
b/arch/arm/mach-omap2/board-3430sdp.c
index f6cd08b..7a216cc 100644
--- a/arch/arm/mach-omap2/board-3430sdp.c
+++ b/arch/arm/mach-omap2/board-3430sdp.c
@@ -204,7 +204,7 @@ static struct spi_board_info sdp3430_spi_board_info[]
__initdata = {
};
static struct platform_device sdp3430_lcd_device = {
- .name = sdp2430_lcd,
+ .name = sdp3430_lcd,
.id = -1,
};
diff --git a/drivers/video/omap/Makefile b/drivers/video/omap/Makefile
index fe7ee5d..a72a2d9 100644
--- a/drivers/video/omap/Makefile
+++ b/drivers/video/omap/Makefile
@@ -30,7 +30,7 @@ objs-y$(CONFIG_MACH_SX1) += lcd_sx1.o
objs-y$(CONFIG_MACH_OMAP_APOLLON) += lcd_apollon.o
objs-y$(CONFIG_MACH_OMAP_2430SDP) += lcd_2430sdp.o
-objs-y$(CONFIG_MACH_OMAP_3430SDP) += lcd_2430sdp.o
+objs-y$(CONFIG_MACH_OMAP_3430SDP) += lcd_3430sdp.o
objs-y$(CONFIG_MACH_OMAP3EVM) += lcd_omap3evm.o
objs-y$(CONFIG_MACH_OMAP3_BEAGLE) += lcd_omap3beagle.o
objs-y$(CONFIG_FB_OMAP_LCD_MIPID) += lcd_mipid.o
diff --git a/drivers/video/omap/lcd_3430sdp.c
b/drivers/video/omap/lcd_3430sdp.c
new file mode 100644
index 000..a8e4366
--- /dev/null
+++ b/drivers/video/omap/lcd_3430sdp.c
@@ -0,0 +1,161 @@
+/*
+ * LCD panel support for the TI 3430SDP board
+ *
+ * Copyright (C) 2008 Texas Instruments
+ * Author: Iqbal Shareef [EMAIL PROTECTED]
+ *
+ * Derived from drivers/video/omap/lcd_2430sdp.c
+ * Copyright (C) 2007 MontaVista
+ * Author: Hunyue Yau [EMAIL PROTECTED]
+ *
+ * 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/i2c/twl4030.h
+
+#include asm/arch/gpio.h
+#include asm/arch/omapfb.h
+
+#define SDP3430_LCD_PANEL_BACKLIGHT_GPIO 8
+#define SDP3430_LCD_PANEL_ENABLE_GPIO5
+
+static unsigned backlight_gpio;
+static unsigned enable_gpio;
+
+#define LCD_PIXCLOCK_MAX 167000
+#define PM_RECEIVER TWL4030_MODULE_PM_RECEIVER
PM_RECEIVER has no prefix. It's better to use TWL4030_MODULE_PM_RECEIVER
to avoid namespace conflicts
+#define ENABLE_VAUX2_DEDICATED 0x03
+#define ENABLE_VAUX2_DEV_GRP 0x20
+#define ENABLE_VAUX3_DEDICATED 0x05
+#define ENABLE_VAUX3_DEV_GRP 0xE0
+
+static int sdp3430_panel_init(struct lcd_panel *panel,
+ struct omapfb_device *fbdev)
+{
+ enable_gpio= SDP3430_LCD_PANEL_ENABLE_GPIO;
+ backlight_gpio = SDP3430_LCD_PANEL_BACKLIGHT_GPIO;
+
+ omap_request_gpio(enable_gpio); /* LCD panel */
+ omap_request_gpio(backlight_gpio); /* LCD backlight */
+ omap_set_gpio_direction(enable_gpio, 0);/* output */
+ omap_set_gpio_direction(backlight_gpio, 0); /* output */
better using gpiolib
+
+ return 0;
+}
+
+static void sdp3430_panel_cleanup(struct lcd_panel *panel)
+{
+ omap_free_gpio(backlight_gpio);
+ omap_free_gpio(enable_gpio);
gpiolib
+}
+
+static int sdp3430_panel_enable(struct lcd_panel *panel)
+{
+ omap_set_gpio_dataout(enable_gpio, 1);
+ omap_set_gpio_dataout(backlight_gpio, 1);
gpiolib
+
+ if (0 != twl4030_i2c_write_u8(PM_RECEIVER, ENABLE_VAUX3_DEDICATED,
+ TWL4030_VAUX3_DEDICATED))
+ return -EIO;
+ if (0 !=