> -----Original Message----- > From: Hiremath, Vaibhav > Sent: Friday, February 26, 2010 11:37 AM > To: Y, Kishore; Tomi Valkeinen > Cc: linux-omap@vger.kernel.org > Subject: RE: [PATCH 1/2 V3] OMAP: DSS: Add display board file for zoom > boards > > > -----Original Message----- > > From: Y, Kishore > > Sent: Friday, February 26, 2010 10:59 AM > > To: Hiremath, Vaibhav; Tomi Valkeinen > > Cc: linux-omap@vger.kernel.org > > Subject: RE: [PATCH 1/2 V3] OMAP: DSS: Add display board file for zoom > > boards > > > > > -----Original Message----- > > > From: Hiremath, Vaibhav > > > Sent: Thursday, February 25, 2010 11:53 PM > > > To: Hiremath, Vaibhav; Y, Kishore; Tomi Valkeinen > > > Cc: linux-omap@vger.kernel.org > > > Subject: RE: [PATCH 1/2 V3] OMAP: DSS: Add display board file for zoom > > > boards > > > > > > > > > > -----Original Message----- > > > > From: Hiremath, Vaibhav > > > > Sent: Thursday, February 25, 2010 11:49 PM > > > > To: Y, Kishore; Tomi Valkeinen > > > > Cc: linux-omap@vger.kernel.org > > > > Subject: RE: [PATCH 1/2 V3] OMAP: DSS: Add display board file for > zoom > > > > boards > > > > > > > > > > > > > -----Original Message----- > > > > > From: linux-omap-ow...@vger.kernel.org [mailto:linux-omap- > > > > > ow...@vger.kernel.org] On Behalf Of Y, Kishore > > > > > Sent: Thursday, February 25, 2010 9:15 PM > > > > > To: Tomi Valkeinen > > > > > Cc: linux-omap@vger.kernel.org > > > > > Subject: [PATCH 1/2 V3] OMAP: DSS: Add display board file for zoom > > > boards > > > > > > > > > > From: Kishore Y <kishor...@ti.com> > > > > > > > > > > board-zoom-display.c added as a common file for display > fucntionality > > > > > on boards zoom2, zoom3 and 3630sdp > > > > > > > > > > Signed-off-by: Mukund Mittal <mmit...@ti.com> > > > > > Signed-off-by: Kishore Y <kishor...@ti.com> > > > > > Reviewed-by: Gadiyar, Anand <gadi...@ti.com> > > > > > --- > > > > > arch/arm/mach-omap2/Makefile | 3 + > > > > > arch/arm/mach-omap2/board-zoom-display.c | 185 > > > > > +++++++++++++++++++++++++ > > > > > arch/arm/mach-omap2/board-zoom-peripherals.c | 36 +++++ > > > > > arch/arm/mach-omap2/include/mach/board-zoom.h | 3 + > > > > > 4 files changed, 227 insertions(+), 0 deletions(-) > > > > > create mode 100644 arch/arm/mach-omap2/board-zoom-display.c > > > > > > > > > > diff --git a/arch/arm/mach-omap2/Makefile b/arch/arm/mach- > > > omap2/Makefile > > > > > index b32678b..3dbe694 100644 > > > > > --- a/arch/arm/mach-omap2/Makefile > > > > > +++ b/arch/arm/mach-omap2/Makefile > > > > > @@ -91,14 +91,17 @@ obj-$(CONFIG_MACH_NOKIA_RX51) += > board- > > > rx51.o > > > > > \ > > > > > board-rx51-peripherals.o \ > > > > > mmc-twl4030.o > > > > > obj-$(CONFIG_MACH_OMAP_ZOOM2) += board-zoom2.o \ > > > > > + board-zoom-display.o \ > > > > > board-zoom-peripherals.o \ > > > > > mmc-twl4030.o \ > > > > > board-zoom-debugboard.o > > > > > obj-$(CONFIG_MACH_OMAP_ZOOM3) += board-zoom3.o \ > > > > > + board-zoom-display.o \ > > > > > board-zoom-peripherals.o \ > > > > > mmc-twl4030.o \ > > > > > board-zoom-debugboard.o > > > > > obj-$(CONFIG_MACH_OMAP_3630SDP) += board-3630sdp.o \ > > > > > + board-zoom-display.o \ > > > > > board-zoom-peripherals.o \ > > > > > mmc-twl4030.o > > > > > obj-$(CONFIG_MACH_CM_T35) += board-cm-t35.o \ > > > > > diff --git a/arch/arm/mach-omap2/board-zoom-display.c > b/arch/arm/mach- > > > > > omap2/board-zoom-display.c > > > > > new file mode 100644 > > > > > index 0000000..fc5851f > > > > > --- /dev/null > > > > > +++ b/arch/arm/mach-omap2/board-zoom-display.c > > > > > @@ -0,0 +1,185 @@ > > > > > +/* > > > > > + * Copyright (C) 2009 Texas Instruments Inc. > > > > > + * > > > > [Hiremath, Vaibhav] 2010??? > > > > > > My bad. Manjunath pointed out same thing in earlier version. > > Somehow I missed it again. I will correct this. > > > > > > > + * Modified from mach-omap2/board-zoom-peripherals.c > > > > > + * > > > > > + * This program is free software; you can redistribute it and/or > > > modify > > > > > + * it under the terms of the GNU General Public License version 2 > as > > > > > + * published by the Free Software Foundation. > > > > > + */ > > > > > + > > > > > +#include <linux/kernel.h> > > > > > +#include <linux/init.h> > > > > > +#include <linux/platform_device.h> > > > > > +#include <linux/gpio.h> > > > > > +#include <linux/i2c/twl.h> > > > > > +#include <linux/spi/spi.h> > > > > > +#include <plat/common.h> > > > > > +#include <plat/control.h> > > > > > +#include <plat/mcspi.h> > > > > > +#include <plat/display.h> > > > > > + > > > > > +#define LCD_PANEL_ENABLE_GPIO (7 + > > > > > OMAP_MAX_GPIO_LINES) > > > > > +#define LCD_PANEL_RESET_GPIO_PROD 96 > > > > > +#define LCD_PANEL_RESET_GPIO_PILOT 55 > > > > > +#define LCD_PANEL_QVGA_GPIO 56 > > > > > +#define TV_PANEL_ENABLE_GPIO 95 > > > > > + > > > > > +struct zoom_dss_board_info { > > > > > + int gpio_flag; > > > > > +}; > > > > > + > > > > > +static void zoom_lcd_tv_panel_init(void) > > > > > +{ > > > > > + int ret; > > > > > + unsigned char lcd_panel_reset_gpio; > > > > > + > > > > > + lcd_panel_reset_gpio = (omap_rev() > OMAP3430_REV_ES3_0) ? > > > > > + LCD_PANEL_RESET_GPIO_PROD : > > > > > + LCD_PANEL_RESET_GPIO_PILOT; > > > > > + > > > > > + ret = gpio_request(lcd_panel_reset_gpio, "lcd reset"); > > > > > + if (ret) { > > > > > + pr_err("Failed to get LCD reset GPIO (gpio%d)\n", > > > > > + lcd_panel_reset_gpio); > > > > > + return; > > > > > + } > > > > > + gpio_direction_output(lcd_panel_reset_gpio, 1); > > > > > + > > > > > + ret = gpio_request(LCD_PANEL_QVGA_GPIO, "lcd qvga"); > > > > > + if (ret) { > > > > > + pr_err("Failed to get LCD_PANEL_QVGA_GPIO (gpio%d).\n", > > > > > + LCD_PANEL_QVGA_GPIO); > > > > > + goto err0; > > > > > + } > > > > > + gpio_direction_output(LCD_PANEL_QVGA_GPIO, 1); > > > > > + > > > > > + ret = gpio_request(TV_PANEL_ENABLE_GPIO, "tv panel"); > > > > > + if (ret) { > > > > > + pr_err("Failed to get TV_PANEL_ENABLE_GPIO (gpio%d).\n", > > > > > + TV_PANEL_ENABLE_GPIO); > > > > > + goto err1; > > > > > + } > > > > > + gpio_direction_output(TV_PANEL_ENABLE_GPIO, 0); > > > > > + > > > > > + return; > > > > > + > > > > > +err1: > > > > > + gpio_free(LCD_PANEL_QVGA_GPIO); > > > > > +err0: > > > > > + gpio_free(lcd_panel_reset_gpio); > > > > > +} > > > > > + > > > > > +static int zoom_panel_enable_lcd(struct omap_dss_device *dssdev) > > > > > +{ > > > > > + int ret; > > > > > + struct zoom_dss_board_info *pdata; > > > > > + > > > > > + pdata = dssdev->dev.platform_data; > > > > > + if (pdata->gpio_flag == 0) { > > > > > + ret = gpio_request(LCD_PANEL_ENABLE_GPIO, "lcd enable"); > > > > > + if (ret) { > > > > > + pr_err("Failed to get" > > > > > + " LCD_PANEL_ENABLE_GPIO (gpio%d).\n", > > > > > + LCD_PANEL_ENABLE_GPIO); > > > > > + return ret; > > > > > + } > > > > > + gpio_direction_output(LCD_PANEL_ENABLE_GPIO, 1); > > > > > + pdata->gpio_flag = 1; > > > > > + } else { > > > > > + gpio_set_value(LCD_PANEL_ENABLE_GPIO, 1); > > > > > + } > > > > > + > > > > [Hiremath, Vaibhav] I think I had mentioned this in your last post, > why > > > do > > > > you want to call gpio_request and gpio_direction_output from here > and > > > > encapsulate it under one variable. You can very well do this in > > > > zoom_display_init with default value to 0, which is what you are > doing > > > in > > > > disable function below. And panel_enable function would get called > from > > > init > > > > sequence anyway right? > > > > > > > > With this you can remove gpio_flag altogether and function will > become > > > > simple. > > > > > > > > Do you see any issues with it? > > > > > > > > LCD_PANEL_ENABLE_GPIO is not part of core gpios. It is part of twlxxxx > chip > > which gets initialized only after board initialization is complete. Due > to > > this gpio_request() would fail. So we had to add it in enable function > which > > will be called only at dss init and later on. > > > [Hiremath, Vaibhav] Kishor, I am also using TWL GPIO's for backlight > control and DVI enable. And I am doing same thing inside function > "omap3evm_twl_gpio_setup", which gets called later stage of boot sequence. > It is working perfectly fine for me. > > This API is part of "struct twl4030_gpio_platform_data", which sets up > GPIO when TWL gets initialized. > > You should be doing this under this function. > > Thanks, > Vaibhav > >
Thanks for the input. I will try this out. > > Regards, > > KK > > > > > > > + return 0; > > > > > +} > > > > > + > > > > > +static void zoom_panel_disable_lcd(struct omap_dss_device > *dssdev) > > > > > +{ > > > > > + gpio_set_value(LCD_PANEL_ENABLE_GPIO, 0); > > > > > +} > > > > > + > > > > > +static int zoom_panel_enable_tv(struct omap_dss_device *dssdev) > > > > > +{ > > > > > + gpio_set_value(TV_PANEL_ENABLE_GPIO, 0); > > > > > + > > > > > + return 0; > > > > > +} > > > > > + > > > > > +static void zoom_panel_disable_tv(struct omap_dss_device *dssdev) > > > > > +{ > > > > > + gpio_set_value(TV_PANEL_ENABLE_GPIO, 1); > > > > > +} > > > > > + > > > > > +static struct zoom_dss_board_info zoom_dss_lcd_data = { > > > > > + .gpio_flag = 0, > > > > > +}; > > > > > + > > > > > +static struct omap_dss_device zoom_lcd_device = { > > > > > + .name = "lcd", > > > > > + .driver_name = "NEC_8048_panel", > > > > > + .type = OMAP_DISPLAY_TYPE_DPI, > > > > > + .phy.dpi.data_lines = 24, > > > > > + .platform_enable = zoom_panel_enable_lcd, > > > > > + .platform_disable = zoom_panel_disable_lcd, > > > > > + .dev = { > > > > > + .platform_data = &zoom_dss_lcd_data, > > > > > + }, > > > > > +}; > > > > > + > > > > > +static struct omap_dss_device zoom_tv_device = { > > > > > + .name = "tv", > > > > > + .driver_name = "venc", > > > > > + .type = OMAP_DISPLAY_TYPE_VENC, > > > > > + .phy.venc.type = -1, > > > > > + .platform_enable = zoom_panel_enable_tv, > > > > > + .platform_disable = zoom_panel_disable_tv, > > > > > +}; > > > > > + > > > > > +static struct omap_dss_device *zoom_dss_devices[] = { > > > > > + &zoom_lcd_device, > > > > > + &zoom_tv_device, > > > > > +}; > > > > > + > > > > > +static struct omap_dss_board_info zoom_dss_data = { > > > > > + .num_devices = ARRAY_SIZE(zoom_dss_devices), > > > > > + .devices = zoom_dss_devices, > > > > > + .default_device = &zoom_lcd_device, > > > > > +}; > > > > > + > > > > > +static struct platform_device zoom_dss_device = { > > > > > + .name = "omapdss", > > > > > + .id = -1, > > > > > + .dev = { > > > > > + .platform_data = &zoom_dss_data, > > > > > + }, > > > > > +}; > > > > > + > > > > > +static struct omap2_mcspi_device_config dss_lcd_mcspi_config = { > > > > > + .turbo_mode = 1, > > > > > + .single_channel = 1, /* 0: slave, 1: master */ > > > > > +}; > > > > > + > > > > > +static struct spi_board_info nec_8048_spi_board_info[] __initdata > = { > > > > > + [0] = { > > > > > + .modalias = "nec_8048_spi", > > > > > + .bus_num = 1, > > > > > + .chip_select = 2, > > > > > + .max_speed_hz = 375000, > > > > > + .controller_data = &dss_lcd_mcspi_config, > > > > > + }, > > > > > +}; > > > > > + > > > > > +static struct platform_device *zoom_display_devices[] __initdata > = { > > > > > + &zoom_dss_device, > > > > > +}; > > > > > + > > > > > +void __init zoom_display_init(enum omap_dss_venc_type venc_type) > > > > > +{ > > > > > + zoom_tv_device.phy.venc.type = venc_type; > > > > > + platform_add_devices(zoom_display_devices, > > > > > + ARRAY_SIZE(zoom_display_devices)); > > > > > + spi_register_board_info(nec_8048_spi_board_info, > > > > > + ARRAY_SIZE(nec_8048_spi_board_info)); > > > > > + zoom_lcd_tv_panel_init(); > > > > > +} > > > > > + > > > > > diff --git a/arch/arm/mach-omap2/board-zoom-peripherals.c > > > b/arch/arm/mach- > > > > > omap2/board-zoom-peripherals.c > > > > > index 8dd277c..4145a40 100755 > > > > > --- a/arch/arm/mach-omap2/board-zoom-peripherals.c > > > > > +++ b/arch/arm/mach-omap2/board-zoom-peripherals.c > > > > > @@ -170,6 +170,40 @@ static struct twl4030_hsmmc_info mmc[] > __initdata > > > = { > > > > > {} /* Terminator */ > > > > > }; > > > > > > > > > > +static struct regulator_consumer_supply zoom_vpll2_supply = { > > > > > + .supply = "vdds_dsi", > > > > > +}; > > > > > + > > > > > +static struct regulator_consumer_supply zoom_vdda_dac_supply = { > > > > > + .supply = "vdda_dac", > > > > > +}; > > > > > + > > > > > +static struct regulator_init_data zoom_vpll2 = { > > > > > + .constraints = { > > > > > + .min_uV = 1800000, > > > > > + .max_uV = 1800000, > > > > > + .valid_modes_mask = REGULATOR_MODE_NORMAL > > > > > + | REGULATOR_MODE_STANDBY, > > > > > + .valid_ops_mask = REGULATOR_CHANGE_MODE > > > > > + | REGULATOR_CHANGE_STATUS, > > > > > + }, > > > > > + .num_consumer_supplies = 1, > > > > > + .consumer_supplies = &zoom_vpll2_supply, > > > > > +}; > > > > > + > > > > > +static struct regulator_init_data zoom_vdac = { > > > > > + .constraints = { > > > > > + .min_uV = 1800000, > > > > > + .max_uV = 1800000, > > > > > + .valid_modes_mask = REGULATOR_MODE_NORMAL > > > > > + | REGULATOR_MODE_STANDBY, > > > > > + .valid_ops_mask = REGULATOR_CHANGE_MODE > > > > > + | REGULATOR_CHANGE_STATUS, > > > > > + }, > > > > > + .num_consumer_supplies = 1, > > > > > + .consumer_supplies = &zoom_vdda_dac_supply, > > > > > +}; > > > > > + > > > > > static int zoom_twl_gpio_setup(struct device *dev, > > > > > unsigned gpio, unsigned ngpio) > > > > > { > > > > > @@ -242,6 +276,8 @@ static struct twl4030_platform_data > zoom_twldata = > > > { > > > > > .vmmc1 = &zoom_vmmc1, > > > > > .vmmc2 = &zoom_vmmc2, > > > > > .vsim = &zoom_vsim, > > > > > + .vpll2 = &zoom_vpll2, > > > > > + .vdac = &zoom_vdac, > > > > > > > > > > }; > > > > > > > > > > diff --git a/arch/arm/mach-omap2/include/mach/board-zoom.h > > > > b/arch/arm/mach- > > > > > omap2/include/mach/board-zoom.h > > > > > index c93b29e..aec73b4 100644 > > > > > --- a/arch/arm/mach-omap2/include/mach/board-zoom.h > > > > > +++ b/arch/arm/mach-omap2/include/mach/board-zoom.h > > > > > @@ -1,5 +1,8 @@ > > > > > /* > > > > > * Defines for zoom boards > > > > > */ > > > > > +#include <plat/display.h> > > > > > + > > > > > extern int __init zoom_debugboard_init(void); > > > > > extern void __init zoom_peripherals_init(void); > > > > > +extern void __init zoom_display_init(enum omap_dss_venc_type > > > venc_type); > > > > [Hiremath, Vaibhav] Just for my understanding, from where you are > > > calling > > > > this function? I believe you are making this extern just to define > > > venc.type > > > > to appropriate value, right? > > > > > > > [Hiremath, Vaibhav] Yes, you are calling this from your board_init > > > function, it is part of your last patch. > > > > > > Thanks, > > > Vaibhav > > > > > > > Thanks, > > > > Vaibhav > > > > > > > > > -- > > > > > 1.5.6.3 > > > > > > > > > > Regards, > > > > > Kishore Y > > > > > -- > > > > > To unsubscribe from this list: send the line "unsubscribe linux- > omap" > > > in > > > > > the body of a message to majord...@vger.kernel.org > > > > > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html