Hiremath, Vaibhav wrote:
>> -----Original Message-----
>> From: [email protected] [mailto:linux-omap-
>> [email protected]] On Behalf Of Nilofer, Samreen
>> Sent: Thursday, October 14, 2010 4:57 PM
>> To: [email protected]
>> Cc: Kishore Y; Mittal, Mukund; Rajkumar N; Nilofer, Samreen
>> Subject: [PATCH 1/2] OMAP3: ZOOM2/3/3630SDP: Add display board file for
>> OMAP3
>> 
>> From: Kishore Y <[email protected]>
>> 
>> Board-zoom-display.c added as a common file for display functionality
>> on boards zoom2, zoom3 and 3630sdp
>> 
>> Signed-off-by: Mukund Mittal <[email protected]>
>> Signed-off-by: Kishore Y <[email protected]>
>> Signed-off-by: Rajkumar N <[email protected]>
>> Signed-off-by: Samreen <[email protected]>
>> ---
>>  arch/arm/mach-omap2/Makefile                  |    3 +
>>  arch/arm/mach-omap2/board-zoom-display.c      |  165
>>  +++++++++++++++++++++++++ arch/arm/mach-omap2/board-zoom-peripherals.c  |  
>>  49 +++++++- arch/arm/mach-omap2/include/mach/board-zoom.h |    2 +
>>  4 files changed, 217 insertions(+), 2 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 88d3a1e..95e0ecf 100644
>> --- a/arch/arm/mach-omap2/Makefile
>> +++ b/arch/arm/mach-omap2/Makefile
>> @@ -132,16 +132,19 @@ obj-$(CONFIG_MACH_NOKIA_RX51) += board- rx51.o \
>>                                         hsmmc.o
>>  obj-$(CONFIG_MACH_OMAP_ZOOM2)               += board-zoom2.o \
>>                                         board-zoom-peripherals.o \
>> +                                       board-zoom-display.o \
>>                                         board-flash.o \
>>                                         hsmmc.o \
>>                                         board-zoom-debugboard.o
>>  obj-$(CONFIG_MACH_OMAP_ZOOM3)               += board-zoom3.o \
>>                                         board-zoom-peripherals.o \
>> +                                       board-zoom-display.o \
>>                                         board-flash.o \
>>                                         hsmmc.o \
>>                                         board-zoom-debugboard.o
>>  obj-$(CONFIG_MACH_OMAP_3630SDP)             += board-3630sdp.o \
>>                                         board-zoom-peripherals.o \
>> +                                       board-zoom-display.o \
>>                                         board-flash.o \
>>                                         hsmmc.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..1e0b3da ---
>> /dev/null +++ b/arch/arm/mach-omap2/board-zoom-display.c
>> @@ -0,0 +1,165 @@
>> +/*
>> + * Copyright (C) 2010 Texas Instruments Inc.
>> + *
>> + * 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/mcspi.h>
>> +#include <plat/display.h>
>> +
>> +#define LCD_PANEL_ENABLE_GPIO               (7 +
> OMAP_MAX_GPIO_LINES)
> [Hiremath, Vaibhav] Are you sure that you don't want to use
> this GPIO in your enable/disable callback API?
> 
[Samreen]
   This will be taken care through the Backlight function,
Where it would be enabled/disabled accordingly.
   I will remove this define from this file as it is not used.

>> +#define LCD_PANEL_RESET_GPIO_PROD   96
>> +#define LCD_PANEL_RESET_GPIO_PILOT  55
>> +#define LCD_PANEL_QVGA_GPIO         56
>> +
>> +static void zoom_lcd_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);
>> +
>> +    return;
>> +err0:
>> +    gpio_free(lcd_panel_reset_gpio);
>> +}
>> +
>> +static int zoom_panel_enable_lcd(struct omap_dss_device *dssdev) { +        
>> return
>> 0; +}
>> +
>> +static void zoom_panel_disable_lcd(struct omap_dss_device *dssdev) { +}
>> +
>> +/*
>> + * PWMA/B register offsets (TWL4030_MODULE_PWMA)  */ +#define
>> TWL_INTBR_PMBR1      0xD +#define TWL_INTBR_GPBR1    0xC
>> +#define TWL_LED_PWMON       0x0
>> +#define TWL_LED_PWMOFF      0x1
>> +
>> +static int zoom_set_bl_intensity(struct omap_dss_device *dssdev, int level)
>> +{
>> +    unsigned char c;
>> +    u8 mux_pwm, enb_pwm;
>> +
>> +    if (level > 100)
>> +            return -1;
>> +
>> +    twl_i2c_read_u8(TWL4030_MODULE_INTBR, &mux_pwm, TWL_INTBR_PMBR1);
>> +    twl_i2c_read_u8(TWL4030_MODULE_INTBR, &enb_pwm, TWL_INTBR_GPBR1); +
>> +    if (level == 0) {
>> +            /* disable pwm1 output and clock */
>> +            enb_pwm = enb_pwm & 0xF5;
>> +            /* change pwm1 pin to gpio pin */
> [Hiremath, Vaibhav] If I understand correctly you don't have
> to configure this to GPIO mode and again to PWM mode every
> time. In case of oMAP3EVM we have something -
> 
> 
>                 c = ((125 * (100 - intensity)) / 100) + 1;
>                 twl_i2c_write_u8(TWL4030_MODULE_PWMA, 0x7F,
>                                 TWL_LED_PWMOFF);
>                 twl_i2c_write_u8(TWL4030_MODULE_PWMA, c,
>                                 TWL_LED_PWMON);
> 
> This should be enough to handle all the
>
> Thanks,
> Vaibhav
[Samreen]
 I will modify this code to change gpio mode to PWM1 mode only when
it is not in gpio mode. Instead of it being repeatedly configured
> 
>> +            mux_pwm = mux_pwm & 0xCF;
>> +            twl_i2c_write_u8(TWL4030_MODULE_INTBR,
>> +                                    enb_pwm, TWL_INTBR_GPBR1);
>> +            twl_i2c_write_u8(TWL4030_MODULE_INTBR,
>> +                                    mux_pwm, TWL_INTBR_PMBR1);
>> +            return 0;
>> +    }
>> +
>> +    /* change gpio pin to pwm1 pin */
>> +    mux_pwm = mux_pwm | 0x30;
>> +    /* enable pwm1 output and clock*/
>> +    enb_pwm = enb_pwm | 0x0A;
>> +    twl_i2c_write_u8(TWL4030_MODULE_INTBR, mux_pwm, TWL_INTBR_PMBR1);
>> +    twl_i2c_write_u8(TWL4030_MODULE_INTBR, enb_pwm, TWL_INTBR_GPBR1); +
>> +    c = ((50 * (100 - level)) / 100) + 1;
>> +    twl_i2c_write_u8(TWL4030_MODULE_PWM1, 0x7F, TWL_LED_PWMOFF);
>> +    twl_i2c_write_u8(TWL4030_MODULE_PWM1, c, TWL_LED_PWMON); +
>> +    return 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,
>> +    .max_backlight_level    = 100,
>> +    .set_backlight          = zoom_set_bl_intensity,
>> +};
>> +
>> +static struct omap_dss_device *zoom_dss_devices[] = { +     
>> &zoom_lcd_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(void)
>> +{
>> +    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_panel_init();
>> +}
>> +
>> diff --git a/arch/arm/mach-omap2/board-zoom-peripherals.c
>> b/arch/arm/mach- omap2/board-zoom-peripherals.c index 6b39849..ebcfbed 100644
>> --- a/arch/arm/mach-omap2/board-zoom-peripherals.c
>> +++ b/arch/arm/mach-omap2/board-zoom-peripherals.c
>> @@ -27,6 +27,8 @@
>>  #include "mux.h"
>>  #include "hsmmc.h"
>> 
>> +#define LCD_PANEL_ENABLE_GPIO               (7 + OMAP_MAX_GPIO_LINES) +
>>  /* Zoom2 has Qwerty keyboard*/
>>  static int board_keymap[] = {
>>      KEY(0, 0, KEY_E),
>> @@ -171,9 +173,43 @@ static struct omap2_hsmmc_info mmc[] __initdata = { 
>>  {}      /* Terminator */ };
>> 
>> +static struct regulator_consumer_supply zoom_vpll2_supply =
>> +    REGULATOR_SUPPLY("vdds_dsi", "omapdss");
>> +
>> +static struct regulator_consumer_supply zoom_vdda_dac_supply =
>> +    REGULATOR_SUPPLY("vdda_dac", "omapdss");
>> +
>> +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)
>>  {
>> +    int ret;
>> +
>>      /* gpio + 0 is "mmc0_cd" (input/IRQ) */
>>      mmc[0].gpio_cd = gpio + 0;
>>      omap2_hsmmc_init(mmc);
>> @@ -185,7 +221,15 @@ static int zoom_twl_gpio_setup(struct device *dev,
>>      zoom_vsim_supply.dev = mmc[0].dev;
>>      zoom_vmmc2_supply.dev = mmc[1].dev;
>> 
>> -    return 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, 0);
>> +
>> +    return ret;
>>  }
>> 
>> 
>> @@ -243,7 +287,8 @@ static struct twl4030_platform_data zoom_twldata = {
>>      .vmmc1          = &zoom_vmmc1,
>>      .vmmc2          = &zoom_vmmc2,
>>      .vsim           = &zoom_vsim,
>> -
>> +    .vpll2          = &zoom_vpll2,
>> +    .vdac           = &zoom_vdac,
>>  };
>> 
>>  static struct i2c_board_info __initdata zoom_i2c_boardinfo[] = { diff
>> --git a/arch/arm/mach-omap2/include/mach/board-zoom.h
>> b/arch/arm/mach-omap2/include/mach/board-zoom.h
>> index 3af69d2..8d8eeb2 100644
>> --- a/arch/arm/mach-omap2/include/mach/board-zoom.h
>> +++ b/arch/arm/mach-omap2/include/mach/board-zoom.h @@ -3,9 +3,11 @@
>>   */
>>  #include <linux/mtd/mtd.h>
>>  #include <linux/mtd/partitions.h>
>> +#include <plat/display.h>
>> 
>>  #define ZOOM_NAND_CS    0
>> 
>>  extern void __init board_nand_init(struct mtd_partition *, u8 nr_parts,
>> u8 cs);
>>  extern int __init zoom_debugboard_init(void);  extern void __init
>> zoom_peripherals_init(void); +extern void __init zoom_display_init(void);
>> --
>> 1.5.6.3
>> 
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-omap"
>> in the body of a message to [email protected] 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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to