Merge tag 'fbdev-3.12' of git://git.kernel.org/pub/scm/linux/kernel/git/tomba/linux
authorLinus Torvalds <torvalds@linux-foundation.org>
Thu, 5 Sep 2013 16:49:32 +0000 (09:49 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Thu, 5 Sep 2013 16:49:32 +0000 (09:49 -0700)
Pull fbdev changes from Tomi Valkeinen:
 - Improvements to da8xx-fb to make it support v2 of the LCDC IP, used
   eg in BeagleBone
 - Himax HX8369 controller support
 - Various small fixes and cleanups

* tag 'fbdev-3.12' of git://git.kernel.org/pub/scm/linux/kernel/git/tomba/linux: (42 commits)
  video: da8xx-fb: fix the polarities of the hsync/vsync pulse
  video: da8xx-fb: support lcdc v2 timing register expansion
  video: da8xx-fb: fixing timing off by one errors
  video: da8xx-fb fixing incorrect porch mappings
  video: xilinxfb: replace devm_request_and_ioremap by devm_ioremap_resource
  fbmem: move EXPORT_SYMBOL annotation next to symbol declarations
  drivers: video: fbcmap: remove the redundency and incorrect checkings
  video: mxsfb: simplify use of devm_ioremap_resource
  Release efifb's colormap in efifb_destroy()
  at91/avr32/atmel_lcdfb: prepare clk before calling enable
  video: exynos: Ensure definitions match prototypes
  OMAPDSS: fix WARN_ON in 'alpha_blending_enabled' sysfs file
  OMAPDSS: HDMI: Fix possible NULL reference
  video: da8xx-fb: adding am33xx as dependency
  video: da8xx-fb: let compiler decide what to inline
  video: da8xx-fb: make clock naming consistent
  video: da8xx-fb: set upstream clock rate (if reqd)
  video: da8xx-fb: reorganize panel detection
  video: da8xx-fb: ensure non-null cfg in pdata
  video: da8xx-fb: use devres
  ...

19 files changed:
Documentation/devicetree/bindings/video/simple-framebuffer.txt
drivers/video/Kconfig
drivers/video/atmel_lcdfb.c
drivers/video/backlight/hx8357.c
drivers/video/backlight/lp855x_bl.c
drivers/video/da8xx-fb.c
drivers/video/efifb.c
drivers/video/exynos/exynos_mipi_dsi_lowlevel.c
drivers/video/fbcmap.c
drivers/video/fbmem.c
drivers/video/matrox/matroxfb_base.c
drivers/video/mxsfb.c
drivers/video/omap2/dss/hdmi.c
drivers/video/omap2/dss/manager-sysfs.c
drivers/video/omap2/dss/ti_hdmi_4xxx_ip.c
drivers/video/output.c
drivers/video/xilinxfb.c
include/linux/platform_data/simplefb.h
include/video/da8xx-fb.h

index 3ea4605..70c26f3 100644 (file)
@@ -12,6 +12,7 @@ Required properties:
 - stride: The number of bytes in each line of the framebuffer.
 - format: The format of the framebuffer surface. Valid values are:
   - r5g6b5 (16-bit pixels, d[15:11]=r, d[10:5]=g, d[4:0]=b).
+  - a8b8g8r8 (32-bit pixels, d[31:24]=a, d[23:16]=b, d[15:8]=g, d[7:0]=r).
 
 Example:
 
index 34c3d96..84b685f 100644 (file)
@@ -2100,13 +2100,6 @@ config GPM1040A0_320X240
         bool "Giantplus Technology GPM1040A0 320x240 Color TFT LCD"
         depends on FB_NUC900
 
-config FB_NUC900_DEBUG
-        bool "NUC900 lcd debug messages"
-        depends on FB_NUC900
-        help
-          Turn on debugging messages. Note that you can set/unset at run time
-          through sysfs
-
 config FB_SM501
        tristate "Silicon Motion SM501 framebuffer support"
        depends on FB && MFD_SM501
@@ -2228,15 +2221,17 @@ config FB_SH7760
          panels <= 320 pixel horizontal resolution.
 
 config FB_DA8XX
-       tristate "DA8xx/OMAP-L1xx Framebuffer support"
-       depends on FB && ARCH_DAVINCI_DA8XX
+       tristate "DA8xx/OMAP-L1xx/AM335x Framebuffer support"
+       depends on FB && (ARCH_DAVINCI_DA8XX || SOC_AM33XX)
        select FB_CFB_FILLRECT
        select FB_CFB_COPYAREA
        select FB_CFB_IMAGEBLIT
        select FB_CFB_REV_PIXELS_IN_BYTE
+       select FB_MODE_HELPERS
+       select VIDEOMODE_HELPERS
        ---help---
          This is the frame buffer device driver for the TI LCD controller
-         found on DA8xx/OMAP-L1xx SoCs.
+         found on DA8xx/OMAP-L1xx/AM335x SoCs.
          If unsure, say N.
 
 config FB_VIRTUAL
index effdb37..088511a 100644 (file)
@@ -902,14 +902,14 @@ static int __init atmel_lcdfb_init_fbinfo(struct atmel_lcdfb_info *sinfo)
 
 static void atmel_lcdfb_start_clock(struct atmel_lcdfb_info *sinfo)
 {
-       clk_enable(sinfo->bus_clk);
-       clk_enable(sinfo->lcdc_clk);
+       clk_prepare_enable(sinfo->bus_clk);
+       clk_prepare_enable(sinfo->lcdc_clk);
 }
 
 static void atmel_lcdfb_stop_clock(struct atmel_lcdfb_info *sinfo)
 {
-       clk_disable(sinfo->bus_clk);
-       clk_disable(sinfo->lcdc_clk);
+       clk_disable_unprepare(sinfo->bus_clk);
+       clk_disable_unprepare(sinfo->lcdc_clk);
 }
 
 
index a0482b5..c7af8c4 100644 (file)
 #define HX8357_SET_POWER_NORMAL                0xd2
 #define HX8357_SET_PANEL_RELATED       0xe9
 
+#define HX8369_SET_DISPLAY_BRIGHTNESS          0x51
+#define HX8369_WRITE_CABC_DISPLAY_VALUE                0x53
+#define HX8369_WRITE_CABC_BRIGHT_CTRL          0x55
+#define HX8369_WRITE_CABC_MIN_BRIGHTNESS       0x5e
+#define HX8369_SET_POWER                       0xb1
+#define HX8369_SET_DISPLAY_MODE                        0xb2
+#define HX8369_SET_DISPLAY_WAVEFORM_CYC                0xb4
+#define HX8369_SET_VCOM                                0xb6
+#define HX8369_SET_EXTENSION_COMMAND           0xb9
+#define HX8369_SET_GIP                         0xd5
+#define HX8369_SET_GAMMA_CURVE_RELATED         0xe0
+
 struct hx8357_data {
        unsigned                im_pins[HX8357_NUM_IM_PINS];
        unsigned                reset;
        struct spi_device       *spi;
        int                     state;
+       bool                    use_im_pins;
 };
 
 static u8 hx8357_seq_power[] = {
@@ -143,6 +156,61 @@ static u8 hx8357_seq_display_mode[] = {
        HX8357_SET_DISPLAY_MODE_RGB_INTERFACE,
 };
 
+static u8 hx8369_seq_write_CABC_min_brightness[] = {
+       HX8369_WRITE_CABC_MIN_BRIGHTNESS, 0x00,
+};
+
+static u8 hx8369_seq_write_CABC_control[] = {
+       HX8369_WRITE_CABC_DISPLAY_VALUE, 0x24,
+};
+
+static u8 hx8369_seq_set_display_brightness[] = {
+       HX8369_SET_DISPLAY_BRIGHTNESS, 0xFF,
+};
+
+static u8 hx8369_seq_write_CABC_control_setting[] = {
+       HX8369_WRITE_CABC_BRIGHT_CTRL, 0x02,
+};
+
+static u8 hx8369_seq_extension_command[] = {
+       HX8369_SET_EXTENSION_COMMAND, 0xff, 0x83, 0x69,
+};
+
+static u8 hx8369_seq_display_related[] = {
+       HX8369_SET_DISPLAY_MODE, 0x00, 0x2b, 0x03, 0x03, 0x70, 0x00,
+       0xff, 0x00, 0x00, 0x00, 0x00, 0x03, 0x03, 0x00, 0x01,
+};
+
+static u8 hx8369_seq_panel_waveform_cycle[] = {
+       HX8369_SET_DISPLAY_WAVEFORM_CYC, 0x0a, 0x1d, 0x80, 0x06, 0x02,
+};
+
+static u8 hx8369_seq_set_address_mode[] = {
+       HX8357_SET_ADDRESS_MODE, 0x00,
+};
+
+static u8 hx8369_seq_vcom[] = {
+       HX8369_SET_VCOM, 0x3e, 0x3e,
+};
+
+static u8 hx8369_seq_gip[] = {
+       HX8369_SET_GIP, 0x00, 0x01, 0x03, 0x25, 0x01, 0x02, 0x28, 0x70,
+       0x11, 0x13, 0x00, 0x00, 0x40, 0x26, 0x51, 0x37, 0x00, 0x00, 0x71,
+       0x35, 0x60, 0x24, 0x07, 0x0f, 0x04, 0x04,
+};
+
+static u8 hx8369_seq_power[] = {
+       HX8369_SET_POWER, 0x01, 0x00, 0x34, 0x03, 0x00, 0x11, 0x11, 0x32,
+       0x2f, 0x3f, 0x3f, 0x01, 0x3a, 0x01, 0xe6, 0xe6, 0xe6, 0xe6, 0xe6,
+};
+
+static u8 hx8369_seq_gamma_curve_related[] = {
+       HX8369_SET_GAMMA_CURVE_RELATED, 0x00, 0x0d, 0x19, 0x2f, 0x3b, 0x3d,
+       0x2e, 0x4a, 0x08, 0x0e, 0x0f, 0x14, 0x16, 0x14, 0x14, 0x14, 0x1e,
+       0x00, 0x0d, 0x19, 0x2f, 0x3b, 0x3d, 0x2e, 0x4a, 0x08, 0x0e, 0x0f,
+       0x14, 0x16, 0x14, 0x14, 0x14, 0x1e,
+};
+
 static int hx8357_spi_write_then_read(struct lcd_device *lcdev,
                                u8 *txbuf, u16 txlen,
                                u8 *rxbuf, u16 rxlen)
@@ -219,6 +287,10 @@ static int hx8357_enter_standby(struct lcd_device *lcdev)
        if (ret < 0)
                return ret;
 
+       /*
+        * The controller needs 120ms when entering in sleep mode before we can
+        * send the command to go off sleep mode
+        */
        msleep(120);
 
        return 0;
@@ -232,6 +304,10 @@ static int hx8357_exit_standby(struct lcd_device *lcdev)
        if (ret < 0)
                return ret;
 
+       /*
+        * The controller needs 120ms when exiting from sleep mode before we
+        * can send the command to enter in sleep mode
+        */
        msleep(120);
 
        ret = hx8357_spi_write_byte(lcdev, HX8357_SET_DISPLAY_ON);
@@ -241,18 +317,9 @@ static int hx8357_exit_standby(struct lcd_device *lcdev)
        return 0;
 }
 
-static int hx8357_lcd_init(struct lcd_device *lcdev)
+static void hx8357_lcd_reset(struct lcd_device *lcdev)
 {
        struct hx8357_data *lcd = lcd_get_data(lcdev);
-       int ret;
-
-       /*
-        * Set the interface selection pins to SPI mode, with three
-        * wires
-        */
-       gpio_set_value_cansleep(lcd->im_pins[0], 1);
-       gpio_set_value_cansleep(lcd->im_pins[1], 0);
-       gpio_set_value_cansleep(lcd->im_pins[2], 1);
 
        /* Reset the screen */
        gpio_set_value(lcd->reset, 1);
@@ -260,7 +327,25 @@ static int hx8357_lcd_init(struct lcd_device *lcdev)
        gpio_set_value(lcd->reset, 0);
        usleep_range(10000, 12000);
        gpio_set_value(lcd->reset, 1);
+
+       /* The controller needs 120ms to recover from reset */
        msleep(120);
+}
+
+static int hx8357_lcd_init(struct lcd_device *lcdev)
+{
+       struct hx8357_data *lcd = lcd_get_data(lcdev);
+       int ret;
+
+       /*
+        * Set the interface selection pins to SPI mode, with three
+        * wires
+        */
+       if (lcd->use_im_pins) {
+               gpio_set_value_cansleep(lcd->im_pins[0], 1);
+               gpio_set_value_cansleep(lcd->im_pins[1], 0);
+               gpio_set_value_cansleep(lcd->im_pins[2], 1);
+       }
 
        ret = hx8357_spi_write_array(lcdev, hx8357_seq_power,
                                ARRAY_SIZE(hx8357_seq_power));
@@ -341,6 +426,9 @@ static int hx8357_lcd_init(struct lcd_device *lcdev)
        if (ret < 0)
                return ret;
 
+       /*
+        * The controller needs 120ms to fully recover from exiting sleep mode
+        */
        msleep(120);
 
        ret = hx8357_spi_write_byte(lcdev, HX8357_SET_DISPLAY_ON);
@@ -356,6 +444,96 @@ static int hx8357_lcd_init(struct lcd_device *lcdev)
        return 0;
 }
 
+static int hx8369_lcd_init(struct lcd_device *lcdev)
+{
+       int ret;
+
+       ret = hx8357_spi_write_array(lcdev, hx8369_seq_extension_command,
+                               ARRAY_SIZE(hx8369_seq_extension_command));
+       if (ret < 0)
+               return ret;
+       usleep_range(10000, 12000);
+
+       ret = hx8357_spi_write_array(lcdev, hx8369_seq_display_related,
+                               ARRAY_SIZE(hx8369_seq_display_related));
+       if (ret < 0)
+               return ret;
+
+       ret = hx8357_spi_write_array(lcdev, hx8369_seq_panel_waveform_cycle,
+                               ARRAY_SIZE(hx8369_seq_panel_waveform_cycle));
+       if (ret < 0)
+               return ret;
+
+       ret = hx8357_spi_write_array(lcdev, hx8369_seq_set_address_mode,
+                               ARRAY_SIZE(hx8369_seq_set_address_mode));
+       if (ret < 0)
+               return ret;
+
+       ret = hx8357_spi_write_array(lcdev, hx8369_seq_vcom,
+                               ARRAY_SIZE(hx8369_seq_vcom));
+       if (ret < 0)
+               return ret;
+
+       ret = hx8357_spi_write_array(lcdev, hx8369_seq_gip,
+                               ARRAY_SIZE(hx8369_seq_gip));
+       if (ret < 0)
+               return ret;
+
+       ret = hx8357_spi_write_array(lcdev, hx8369_seq_power,
+                               ARRAY_SIZE(hx8369_seq_power));
+       if (ret < 0)
+               return ret;
+
+       ret = hx8357_spi_write_byte(lcdev, HX8357_EXIT_SLEEP_MODE);
+       if (ret < 0)
+               return ret;
+
+       /*
+        * The controller needs 120ms to fully recover from exiting sleep mode
+        */
+       msleep(120);
+
+       ret = hx8357_spi_write_array(lcdev, hx8369_seq_gamma_curve_related,
+                               ARRAY_SIZE(hx8369_seq_gamma_curve_related));
+       if (ret < 0)
+               return ret;
+
+       ret = hx8357_spi_write_byte(lcdev, HX8357_EXIT_SLEEP_MODE);
+       if (ret < 0)
+               return ret;
+       usleep_range(1000, 1200);
+
+       ret = hx8357_spi_write_array(lcdev, hx8369_seq_write_CABC_control,
+                               ARRAY_SIZE(hx8369_seq_write_CABC_control));
+       if (ret < 0)
+               return ret;
+       usleep_range(10000, 12000);
+
+       ret = hx8357_spi_write_array(lcdev,
+                       hx8369_seq_write_CABC_control_setting,
+                       ARRAY_SIZE(hx8369_seq_write_CABC_control_setting));
+       if (ret < 0)
+               return ret;
+
+       ret = hx8357_spi_write_array(lcdev,
+                       hx8369_seq_write_CABC_min_brightness,
+                       ARRAY_SIZE(hx8369_seq_write_CABC_min_brightness));
+       if (ret < 0)
+               return ret;
+       usleep_range(10000, 12000);
+
+       ret = hx8357_spi_write_array(lcdev, hx8369_seq_set_display_brightness,
+                               ARRAY_SIZE(hx8369_seq_set_display_brightness));
+       if (ret < 0)
+               return ret;
+
+       ret = hx8357_spi_write_byte(lcdev, HX8357_SET_DISPLAY_ON);
+       if (ret < 0)
+               return ret;
+
+       return 0;
+}
+
 #define POWER_IS_ON(pwr)       ((pwr) <= FB_BLANK_NORMAL)
 
 static int hx8357_set_power(struct lcd_device *lcdev, int power)
@@ -388,10 +566,24 @@ static struct lcd_ops hx8357_ops = {
        .get_power      = hx8357_get_power,
 };
 
+static const struct of_device_id hx8357_dt_ids[] = {
+       {
+               .compatible = "himax,hx8357",
+               .data = hx8357_lcd_init,
+       },
+       {
+               .compatible = "himax,hx8369",
+               .data = hx8369_lcd_init,
+       },
+       {},
+};
+MODULE_DEVICE_TABLE(of, hx8357_dt_ids);
+
 static int hx8357_probe(struct spi_device *spi)
 {
        struct lcd_device *lcdev;
        struct hx8357_data *lcd;
+       const struct of_device_id *match;
        int i, ret;
 
        lcd = devm_kzalloc(&spi->dev, sizeof(*lcd), GFP_KERNEL);
@@ -408,6 +600,10 @@ static int hx8357_probe(struct spi_device *spi)
 
        lcd->spi = spi;
 
+       match = of_match_device(hx8357_dt_ids, &spi->dev);
+       if (!match || !match->data)
+               return -EINVAL;
+
        lcd->reset = of_get_named_gpio(spi->dev.of_node, "gpios-reset", 0);
        if (!gpio_is_valid(lcd->reset)) {
                dev_err(&spi->dev, "Missing dt property: gpios-reset\n");
@@ -424,25 +620,32 @@ static int hx8357_probe(struct spi_device *spi)
                return -EINVAL;
        }
 
-       for (i = 0; i < HX8357_NUM_IM_PINS; i++) {
-               lcd->im_pins[i] = of_get_named_gpio(spi->dev.of_node,
-                                               "im-gpios", i);
-               if (lcd->im_pins[i] == -EPROBE_DEFER) {
-                       dev_info(&spi->dev, "GPIO requested is not here yet, deferring the probe\n");
-                       return -EPROBE_DEFER;
-               }
-               if (!gpio_is_valid(lcd->im_pins[i])) {
-                       dev_err(&spi->dev, "Missing dt property: im-gpios\n");
-                       return -EINVAL;
-               }
-
-               ret = devm_gpio_request_one(&spi->dev, lcd->im_pins[i],
-                                       GPIOF_OUT_INIT_LOW, "im_pins");
-               if (ret) {
-                       dev_err(&spi->dev, "failed to request gpio %d: %d\n",
-                               lcd->im_pins[i], ret);
-                       return -EINVAL;
+       if (of_find_property(spi->dev.of_node, "im-gpios", NULL)) {
+               lcd->use_im_pins = 1;
+
+               for (i = 0; i < HX8357_NUM_IM_PINS; i++) {
+                       lcd->im_pins[i] = of_get_named_gpio(spi->dev.of_node,
+                                                           "im-gpios", i);
+                       if (lcd->im_pins[i] == -EPROBE_DEFER) {
+                               dev_info(&spi->dev, "GPIO requested is not here yet, deferring the probe\n");
+                               return -EPROBE_DEFER;
+                       }
+                       if (!gpio_is_valid(lcd->im_pins[i])) {
+                               dev_err(&spi->dev, "Missing dt property: im-gpios\n");
+                               return -EINVAL;
+                       }
+
+                       ret = devm_gpio_request_one(&spi->dev, lcd->im_pins[i],
+                                                   GPIOF_OUT_INIT_LOW,
+                                                   "im_pins");
+                       if (ret) {
+                               dev_err(&spi->dev, "failed to request gpio %d: %d\n",
+                                       lcd->im_pins[i], ret);
+                               return -EINVAL;
+                       }
                }
+       } else {
+               lcd->use_im_pins = 0;
        }
 
        lcdev = lcd_device_register("mxsfb", &spi->dev, lcd, &hx8357_ops);
@@ -452,7 +655,9 @@ static int hx8357_probe(struct spi_device *spi)
        }
        spi_set_drvdata(spi, lcdev);
 
-       ret = hx8357_lcd_init(lcdev);
+       hx8357_lcd_reset(lcdev);
+
+       ret = ((int (*)(struct lcd_device *))match->data)(lcdev);
        if (ret) {
                dev_err(&spi->dev, "Couldn't initialize panel\n");
                goto init_error;
@@ -475,12 +680,6 @@ static int hx8357_remove(struct spi_device *spi)
        return 0;
 }
 
-static const struct of_device_id hx8357_dt_ids[] = {
-       { .compatible = "himax,hx8357" },
-       {},
-};
-MODULE_DEVICE_TABLE(of, hx8357_dt_ids);
-
 static struct spi_driver hx8357_driver = {
        .probe  = hx8357_probe,
        .remove = hx8357_remove,
index a0e1e02..c0b41f1 100644 (file)
@@ -246,7 +246,7 @@ static int lp855x_bl_update_status(struct backlight_device *bl)
 {
        struct lp855x *lp = bl_get_data(bl);
 
-       if (bl->props.state & BL_CORE_SUSPENDED)
+       if (bl->props.state & (BL_CORE_SUSPENDED | BL_CORE_FBBLANK))
                bl->props.brightness = 0;
 
        if (lp->mode == PWM_BASED) {
index 0810939..e030e17 100644 (file)
 
 #define WSI_TIMEOUT    50
 #define PALETTE_SIZE   256
-#define LEFT_MARGIN    64
-#define RIGHT_MARGIN   64
-#define UPPER_MARGIN   32
-#define LOWER_MARGIN   32
+
+#define        CLK_MIN_DIV     2
+#define        CLK_MAX_DIV     255
 
 static void __iomem *da8xx_fb_reg_base;
-static struct resource *lcdc_regs;
 static unsigned int lcd_revision;
 static irq_handler_t lcdc_irq_handler;
 static wait_queue_head_t frame_done_wq;
 static int frame_done_flag;
 
-static inline unsigned int lcdc_read(unsigned int addr)
+static unsigned int lcdc_read(unsigned int addr)
 {
        return (unsigned int)__raw_readl(da8xx_fb_reg_base + (addr));
 }
 
-static inline void lcdc_write(unsigned int val, unsigned int addr)
+static void lcdc_write(unsigned int val, unsigned int addr)
 {
        __raw_writel(val, da8xx_fb_reg_base + (addr));
 }
 
 struct da8xx_fb_par {
+       struct device           *dev;
        resource_size_t p_palette_base;
        unsigned char *v_palette_base;
        dma_addr_t              vram_phys;
@@ -164,7 +163,6 @@ struct da8xx_fb_par {
        struct clk *lcdc_clk;
        int irq;
        unsigned int palette_sz;
-       unsigned int pxl_clk;
        int blank;
        wait_queue_head_t       vsync_wait;
        int                     vsync_flag;
@@ -178,29 +176,15 @@ struct da8xx_fb_par {
        unsigned int            which_dma_channel_done;
 #ifdef CONFIG_CPU_FREQ
        struct notifier_block   freq_transition;
-       unsigned int            lcd_fck_rate;
 #endif
+       unsigned int            lcdc_clk_rate;
        void (*panel_power_ctrl)(int);
        u32 pseudo_palette[16];
+       struct fb_videomode     mode;
+       struct lcd_ctrl_config  cfg;
 };
 
-/* Variable Screen Information */
-static struct fb_var_screeninfo da8xx_fb_var = {
-       .xoffset = 0,
-       .yoffset = 0,
-       .transp = {0, 0, 0},
-       .nonstd = 0,
-       .activate = 0,
-       .height = -1,
-       .width = -1,
-       .accel_flags = 0,
-       .left_margin = LEFT_MARGIN,
-       .right_margin = RIGHT_MARGIN,
-       .upper_margin = UPPER_MARGIN,
-       .lower_margin = LOWER_MARGIN,
-       .sync = 0,
-       .vmode = FB_VMODE_NONINTERLACED
-};
+static struct fb_var_screeninfo da8xx_fb_var;
 
 static struct fb_fix_screeninfo da8xx_fb_fix = {
        .id = "DA8xx FB Drv",
@@ -219,7 +203,7 @@ static struct fb_videomode known_lcd_panels[] = {
                .name           = "Sharp_LCD035Q3DG01",
                .xres           = 320,
                .yres           = 240,
-               .pixclock       = 4608000,
+               .pixclock       = KHZ2PICOS(4607),
                .left_margin    = 6,
                .right_margin   = 8,
                .upper_margin   = 2,
@@ -234,7 +218,7 @@ static struct fb_videomode known_lcd_panels[] = {
                .name           = "Sharp_LK043T1DG01",
                .xres           = 480,
                .yres           = 272,
-               .pixclock       = 7833600,
+               .pixclock       = KHZ2PICOS(7833),
                .left_margin    = 2,
                .right_margin   = 2,
                .upper_margin   = 2,
@@ -249,7 +233,7 @@ static struct fb_videomode known_lcd_panels[] = {
                .name           = "SP10Q010",
                .xres           = 320,
                .yres           = 240,
-               .pixclock       = 7833600,
+               .pixclock       = KHZ2PICOS(7833),
                .left_margin    = 10,
                .right_margin   = 10,
                .upper_margin   = 10,
@@ -261,8 +245,13 @@ static struct fb_videomode known_lcd_panels[] = {
        },
 };
 
+static bool da8xx_fb_is_raster_enabled(void)
+{
+       return !!(lcdc_read(LCD_RASTER_CTRL_REG) & LCD_RASTER_ENABLE);
+}
+
 /* Enable the Raster Engine of the LCD Controller */
-static inline void lcd_enable_raster(void)
+static void lcd_enable_raster(void)
 {
        u32 reg;
 
@@ -284,7 +273,7 @@ static inline void lcd_enable_raster(void)
 }
 
 /* Disable the Raster Engine of the LCD Controller */
-static inline void lcd_disable_raster(bool wait_for_frame_done)
+static void lcd_disable_raster(enum da8xx_frame_complete wait_for_frame_done)
 {
        u32 reg;
        int ret;
@@ -296,7 +285,8 @@ static inline void lcd_disable_raster(bool wait_for_frame_done)
                /* return if already disabled */
                return;
 
-       if ((wait_for_frame_done == true) && (lcd_revision == LCD_VERSION_2)) {
+       if ((wait_for_frame_done == DA8XX_FRAME_WAIT) &&
+                       (lcd_revision == LCD_VERSION_2)) {
                frame_done_flag = 0;
                ret = wait_event_interruptible_timeout(frame_done_wq,
                                frame_done_flag != 0,
@@ -331,7 +321,7 @@ static void lcd_blit(int load_mode, struct da8xx_fb_par *par)
                        reg_int = lcdc_read(LCD_INT_ENABLE_SET_REG) |
                                LCD_V2_END_OF_FRAME0_INT_ENA |
                                LCD_V2_END_OF_FRAME1_INT_ENA |
-                               LCD_FRAME_DONE;
+                               LCD_FRAME_DONE | LCD_SYNC_LOST;
                        lcdc_write(reg_int, LCD_INT_ENABLE_SET_REG);
                }
                reg_dma |= LCD_DUAL_FRAME_BUFFER_ENABLE;
@@ -417,10 +407,25 @@ static void lcd_cfg_horizontal_sync(int back_porch, int pulse_width,
        u32 reg;
 
        reg = lcdc_read(LCD_RASTER_TIMING_0_REG) & 0xf;
-       reg |= ((back_porch & 0xff) << 24)
-           | ((front_porch & 0xff) << 16)
-           | ((pulse_width & 0x3f) << 10);
+       reg |= (((back_porch-1) & 0xff) << 24)
+           | (((front_porch-1) & 0xff) << 16)
+           | (((pulse_width-1) & 0x3f) << 10);
        lcdc_write(reg, LCD_RASTER_TIMING_0_REG);
+
+       /*
+        * LCDC Version 2 adds some extra bits that increase the allowable
+        * size of the horizontal timing registers.
+        * remember that the registers use 0 to represent 1 so all values
+        * that get set into register need to be decremented by 1
+        */
+       if (lcd_revision == LCD_VERSION_2) {
+               /* Mask off the bits we want to change */
+               reg = lcdc_read(LCD_RASTER_TIMING_2_REG) & ~0x780000ff;
+               reg |= ((front_porch-1) & 0x300) >> 8;
+               reg |= ((back_porch-1) & 0x300) >> 4;
+               reg |= ((pulse_width-1) & 0x3c0) << 21;
+               lcdc_write(reg, LCD_RASTER_TIMING_2_REG);
+       }
 }
 
 static void lcd_cfg_vertical_sync(int back_porch, int pulse_width,
@@ -431,7 +436,7 @@ static void lcd_cfg_vertical_sync(int back_porch, int pulse_width,
        reg = lcdc_read(LCD_RASTER_TIMING_1_REG) & 0x3ff;
        reg |= ((back_porch & 0xff) << 24)
            | ((front_porch & 0xff) << 16)
-           | ((pulse_width & 0x3f) << 10);
+           | (((pulse_width-1) & 0x3f) << 10);
        lcdc_write(reg, LCD_RASTER_TIMING_1_REG);
 }
 
@@ -488,12 +493,12 @@ static int lcd_cfg_display(const struct lcd_ctrl_config *cfg,
        else
                reg &= ~LCD_SYNC_EDGE;
 
-       if (panel->sync & FB_SYNC_HOR_HIGH_ACT)
+       if ((panel->sync & FB_SYNC_HOR_HIGH_ACT) == 0)
                reg |= LCD_INVERT_LINE_CLOCK;
        else
                reg &= ~LCD_INVERT_LINE_CLOCK;
 
-       if (panel->sync & FB_SYNC_VERT_HIGH_ACT)
+       if ((panel->sync & FB_SYNC_VERT_HIGH_ACT) == 0)
                reg |= LCD_INVERT_FRAME_CLOCK;
        else
                reg &= ~LCD_INVERT_FRAME_CLOCK;
@@ -565,10 +570,11 @@ static int lcd_cfg_frame_buffer(struct da8xx_fb_par *par, u32 width, u32 height,
                break;
        case 24:
                reg |= LCD_V2_TFT_24BPP_MODE;
+               break;
        case 32:
+               reg |= LCD_V2_TFT_24BPP_MODE;
                reg |= LCD_V2_TFT_24BPP_UNPACK;
                break;
-
        case 8:
                par->palette_sz = 256 * 2;
                break;
@@ -681,11 +687,8 @@ static int fb_setcolreg(unsigned regno, unsigned red, unsigned green,
 }
 #undef CNVT_TOHW
 
-static void lcd_reset(struct da8xx_fb_par *par)
+static void da8xx_fb_lcd_reset(void)
 {
-       /* Disable the Raster if previously Enabled */
-       lcd_disable_raster(false);
-
        /* DMA has to be disabled */
        lcdc_write(0, LCD_DMA_CTRL_REG);
        lcdc_write(0, LCD_RASTER_CTRL_REG);
@@ -698,21 +701,76 @@ static void lcd_reset(struct da8xx_fb_par *par)
        }
 }
 
-static void lcd_calc_clk_divider(struct da8xx_fb_par *par)
+static int da8xx_fb_config_clk_divider(struct da8xx_fb_par *par,
+                                             unsigned lcdc_clk_div,
+                                             unsigned lcdc_clk_rate)
 {
-       unsigned int lcd_clk, div;
+       int ret;
 
-       lcd_clk = clk_get_rate(par->lcdc_clk);
-       div = lcd_clk / par->pxl_clk;
+       if (par->lcdc_clk_rate != lcdc_clk_rate) {
+               ret = clk_set_rate(par->lcdc_clk, lcdc_clk_rate);
+               if (IS_ERR_VALUE(ret)) {
+                       dev_err(par->dev,
+                               "unable to set clock rate at %u\n",
+                               lcdc_clk_rate);
+                       return ret;
+               }
+               par->lcdc_clk_rate = clk_get_rate(par->lcdc_clk);
+       }
 
        /* Configure the LCD clock divisor. */
-       lcdc_write(LCD_CLK_DIVISOR(div) |
+       lcdc_write(LCD_CLK_DIVISOR(lcdc_clk_div) |
                        (LCD_RASTER_MODE & 0x1), LCD_CTRL_REG);
 
        if (lcd_revision == LCD_VERSION_2)
                lcdc_write(LCD_V2_DMA_CLK_EN | LCD_V2_LIDD_CLK_EN |
                                LCD_V2_CORE_CLK_EN, LCD_CLK_ENABLE_REG);
 
+       return 0;
+}
+
+static unsigned int da8xx_fb_calc_clk_divider(struct da8xx_fb_par *par,
+                                             unsigned pixclock,
+                                             unsigned *lcdc_clk_rate)
+{
+       unsigned lcdc_clk_div;
+
+       pixclock = PICOS2KHZ(pixclock) * 1000;
+
+       *lcdc_clk_rate = par->lcdc_clk_rate;
+
+       if (pixclock < (*lcdc_clk_rate / CLK_MAX_DIV)) {
+               *lcdc_clk_rate = clk_round_rate(par->lcdc_clk,
+                                               pixclock * CLK_MAX_DIV);
+               lcdc_clk_div = CLK_MAX_DIV;
+       } else if (pixclock > (*lcdc_clk_rate / CLK_MIN_DIV)) {
+               *lcdc_clk_rate = clk_round_rate(par->lcdc_clk,
+                                               pixclock * CLK_MIN_DIV);
+               lcdc_clk_div = CLK_MIN_DIV;
+       } else {
+               lcdc_clk_div = *lcdc_clk_rate / pixclock;
+       }
+
+       return lcdc_clk_div;
+}
+
+static int da8xx_fb_calc_config_clk_divider(struct da8xx_fb_par *par,
+                                           struct fb_videomode *mode)
+{
+       unsigned lcdc_clk_rate;
+       unsigned lcdc_clk_div = da8xx_fb_calc_clk_divider(par, mode->pixclock,
+                                                         &lcdc_clk_rate);
+
+       return da8xx_fb_config_clk_divider(par, lcdc_clk_div, lcdc_clk_rate);
+}
+
+static unsigned da8xx_fb_round_clk(struct da8xx_fb_par *par,
+                                         unsigned pixclock)
+{
+       unsigned lcdc_clk_div, lcdc_clk_rate;
+
+       lcdc_clk_div = da8xx_fb_calc_clk_divider(par, pixclock, &lcdc_clk_rate);
+       return KHZ2PICOS(lcdc_clk_rate / (1000 * lcdc_clk_div));
 }
 
 static int lcd_init(struct da8xx_fb_par *par, const struct lcd_ctrl_config *cfg,
@@ -721,10 +779,11 @@ static int lcd_init(struct da8xx_fb_par *par, const struct lcd_ctrl_config *cfg,
        u32 bpp;
        int ret = 0;
 
-       lcd_reset(par);
-
-       /* Calculate the divider */
-       lcd_calc_clk_divider(par);
+       ret = da8xx_fb_calc_config_clk_divider(par, panel);
+       if (IS_ERR_VALUE(ret)) {
+               dev_err(par->dev, "unable to configure clock\n");
+               return ret;
+       }
 
        if (panel->sync & FB_SYNC_CLK_INVERT)
                lcdc_write((lcdc_read(LCD_RASTER_TIMING_2_REG) |
@@ -739,10 +798,10 @@ static int lcd_init(struct da8xx_fb_par *par, const struct lcd_ctrl_config *cfg,
                return ret;
 
        /* Configure the vertical and horizontal sync properties. */
-       lcd_cfg_vertical_sync(panel->lower_margin, panel->vsync_len,
-                       panel->upper_margin);
-       lcd_cfg_horizontal_sync(panel->right_margin, panel->hsync_len,
-                       panel->left_margin);
+       lcd_cfg_vertical_sync(panel->upper_margin, panel->vsync_len,
+                       panel->lower_margin);
+       lcd_cfg_horizontal_sync(panel->left_margin, panel->hsync_len,
+                       panel->right_margin);
 
        /* Configure for disply */
        ret = lcd_cfg_display(cfg, panel);
@@ -773,7 +832,7 @@ static irqreturn_t lcdc_irq_handler_rev02(int irq, void *arg)
        u32 stat = lcdc_read(LCD_MASKED_STAT_REG);
 
        if ((stat & LCD_SYNC_LOST) && (stat & LCD_FIFO_UNDERFLOW)) {
-               lcd_disable_raster(false);
+               lcd_disable_raster(DA8XX_FRAME_NOWAIT);
                lcdc_write(stat, LCD_MASKED_STAT_REG);
                lcd_enable_raster();
        } else if (stat & LCD_PL_LOAD_DONE) {
@@ -783,7 +842,7 @@ static irqreturn_t lcdc_irq_handler_rev02(int irq, void *arg)
                 * interrupt via the following write to the status register. If
                 * this is done after then one gets multiple PL done interrupts.
                 */
-               lcd_disable_raster(false);
+               lcd_disable_raster(DA8XX_FRAME_NOWAIT);
 
                lcdc_write(stat, LCD_MASKED_STAT_REG);
 
@@ -836,7 +895,7 @@ static irqreturn_t lcdc_irq_handler_rev01(int irq, void *arg)
        u32 reg_ras;
 
        if ((stat & LCD_SYNC_LOST) && (stat & LCD_FIFO_UNDERFLOW)) {
-               lcd_disable_raster(false);
+               lcd_disable_raster(DA8XX_FRAME_NOWAIT);
                lcdc_write(stat, LCD_STAT_REG);
                lcd_enable_raster();
        } else if (stat & LCD_PL_LOAD_DONE) {
@@ -846,7 +905,7 @@ static irqreturn_t lcdc_irq_handler_rev01(int irq, void *arg)
                 * interrupt via the following write to the status register. If
                 * this is done after then one gets multiple PL done interrupts.
                 */
-               lcd_disable_raster(false);
+               lcd_disable_raster(DA8XX_FRAME_NOWAIT);
 
                lcdc_write(stat, LCD_STAT_REG);
 
@@ -888,6 +947,9 @@ static int fb_check_var(struct fb_var_screeninfo *var,
                        struct fb_info *info)
 {
        int err = 0;
+       struct da8xx_fb_par *par = info->par;
+       int bpp = var->bits_per_pixel >> 3;
+       unsigned long line_size = var->xres_virtual * bpp;
 
        if (var->bits_per_pixel > 16 && lcd_revision == LCD_VERSION_1)
                return -EINVAL;
@@ -955,6 +1017,23 @@ static int fb_check_var(struct fb_var_screeninfo *var,
        var->green.msb_right = 0;
        var->blue.msb_right = 0;
        var->transp.msb_right = 0;
+
+       if (line_size * var->yres_virtual > par->vram_size)
+               var->yres_virtual = par->vram_size / line_size;
+
+       if (var->yres > var->yres_virtual)
+               var->yres = var->yres_virtual;
+
+       if (var->xres > var->xres_virtual)
+               var->xres = var->xres_virtual;
+
+       if (var->xres + var->xoffset > var->xres_virtual)
+               var->xoffset = var->xres_virtual - var->xres;
+       if (var->yres + var->yoffset > var->yres_virtual)
+               var->yoffset = var->yres_virtual - var->yres;
+
+       var->pixclock = da8xx_fb_round_clk(par, var->pixclock);
+
        return err;
 }
 
@@ -966,10 +1045,10 @@ static int lcd_da8xx_cpufreq_transition(struct notifier_block *nb,
 
        par = container_of(nb, struct da8xx_fb_par, freq_transition);
        if (val == CPUFREQ_POSTCHANGE) {
-               if (par->lcd_fck_rate != clk_get_rate(par->lcdc_clk)) {
-                       par->lcd_fck_rate = clk_get_rate(par->lcdc_clk);
-                       lcd_disable_raster(true);
-                       lcd_calc_clk_divider(par);
+               if (par->lcdc_clk_rate != clk_get_rate(par->lcdc_clk)) {
+                       par->lcdc_clk_rate = clk_get_rate(par->lcdc_clk);
+                       lcd_disable_raster(DA8XX_FRAME_WAIT);
+                       da8xx_fb_calc_config_clk_divider(par, &par->mode);
                        if (par->blank == FB_BLANK_UNBLANK)
                                lcd_enable_raster();
                }
@@ -978,7 +1057,7 @@ static int lcd_da8xx_cpufreq_transition(struct notifier_block *nb,
        return 0;
 }
 
-static inline int lcd_da8xx_cpufreq_register(struct da8xx_fb_par *par)
+static int lcd_da8xx_cpufreq_register(struct da8xx_fb_par *par)
 {
        par->freq_transition.notifier_call = lcd_da8xx_cpufreq_transition;
 
@@ -986,7 +1065,7 @@ static inline int lcd_da8xx_cpufreq_register(struct da8xx_fb_par *par)
                                         CPUFREQ_TRANSITION_NOTIFIER);
 }
 
-static inline void lcd_da8xx_cpufreq_deregister(struct da8xx_fb_par *par)
+static void lcd_da8xx_cpufreq_deregister(struct da8xx_fb_par *par)
 {
        cpufreq_unregister_notifier(&par->freq_transition,
                                    CPUFREQ_TRANSITION_NOTIFIER);
@@ -1006,7 +1085,7 @@ static int fb_remove(struct platform_device *dev)
                if (par->panel_power_ctrl)
                        par->panel_power_ctrl(0);
 
-               lcd_disable_raster(true);
+               lcd_disable_raster(DA8XX_FRAME_WAIT);
                lcdc_write(0, LCD_RASTER_CTRL_REG);
 
                /* disable DMA  */
@@ -1018,12 +1097,9 @@ static int fb_remove(struct platform_device *dev)
                                  par->p_palette_base);
                dma_free_coherent(NULL, par->vram_size, par->vram_virt,
                                  par->vram_phys);
-               free_irq(par->irq, par);
                pm_runtime_put_sync(&dev->dev);
                pm_runtime_disable(&dev->dev);
                framebuffer_release(info);
-               iounmap(da8xx_fb_reg_base);
-               release_mem_region(lcdc_regs->start, resource_size(lcdc_regs));
 
        }
        return 0;
@@ -1122,7 +1198,7 @@ static int cfb_blank(int blank, struct fb_info *info)
                if (par->panel_power_ctrl)
                        par->panel_power_ctrl(0);
 
-               lcd_disable_raster(true);
+               lcd_disable_raster(DA8XX_FRAME_WAIT);
                break;
        default:
                ret = -EINVAL;
@@ -1183,9 +1259,50 @@ static int da8xx_pan_display(struct fb_var_screeninfo *var,
        return ret;
 }
 
+static int da8xxfb_set_par(struct fb_info *info)
+{
+       struct da8xx_fb_par *par = info->par;
+       int ret;
+       bool raster = da8xx_fb_is_raster_enabled();
+
+       if (raster)
+               lcd_disable_raster(DA8XX_FRAME_WAIT);
+
+       fb_var_to_videomode(&par->mode, &info->var);
+
+       par->cfg.bpp = info->var.bits_per_pixel;
+
+       info->fix.visual = (par->cfg.bpp <= 8) ?
+                               FB_VISUAL_PSEUDOCOLOR : FB_VISUAL_TRUECOLOR;
+       info->fix.line_length = (par->mode.xres * par->cfg.bpp) / 8;
+
+       ret = lcd_init(par, &par->cfg, &par->mode);
+       if (ret < 0) {
+               dev_err(par->dev, "lcd init failed\n");
+               return ret;
+       }
+
+       par->dma_start = info->fix.smem_start +
+                        info->var.yoffset * info->fix.line_length +
+                        info->var.xoffset * info->var.bits_per_pixel / 8;
+       par->dma_end   = par->dma_start +
+                        info->var.yres * info->fix.line_length - 1;
+
+       lcdc_write(par->dma_start, LCD_DMA_FRM_BUF_BASE_ADDR_0_REG);
+       lcdc_write(par->dma_end, LCD_DMA_FRM_BUF_CEILING_ADDR_0_REG);
+       lcdc_write(par->dma_start, LCD_DMA_FRM_BUF_BASE_ADDR_1_REG);
+       lcdc_write(par->dma_end, LCD_DMA_FRM_BUF_CEILING_ADDR_1_REG);
+
+       if (raster)
+               lcd_enable_raster();
+
+       return 0;
+}
+
 static struct fb_ops da8xx_fb_ops = {
        .owner = THIS_MODULE,
        .fb_check_var = fb_check_var,
+       .fb_set_par = da8xxfb_set_par,
        .fb_setcolreg = fb_setcolreg,
        .fb_pan_display = da8xx_pan_display,
        .fb_ioctl = fb_ioctl,
@@ -1195,33 +1312,38 @@ static struct fb_ops da8xx_fb_ops = {
        .fb_blank = cfb_blank,
 };
 
-/* Calculate and return pixel clock period in pico seconds */
-static unsigned int da8xxfb_pixel_clk_period(struct da8xx_fb_par *par)
+static struct fb_videomode *da8xx_fb_get_videomode(struct platform_device *dev)
 {
-       unsigned int lcd_clk, div;
-       unsigned int configured_pix_clk;
-       unsigned long long pix_clk_period_picosec = 1000000000000ULL;
+       struct da8xx_lcdc_platform_data *fb_pdata = dev->dev.platform_data;
+       struct fb_videomode *lcdc_info;
+       int i;
 
-       lcd_clk = clk_get_rate(par->lcdc_clk);
-       div = lcd_clk / par->pxl_clk;
-       configured_pix_clk = (lcd_clk / div);
+       for (i = 0, lcdc_info = known_lcd_panels;
+               i < ARRAY_SIZE(known_lcd_panels); i++, lcdc_info++) {
+               if (strcmp(fb_pdata->type, lcdc_info->name) == 0)
+                       break;
+       }
 
-       do_div(pix_clk_period_picosec, configured_pix_clk);
+       if (i == ARRAY_SIZE(known_lcd_panels)) {
+               dev_err(&dev->dev, "no panel found\n");
+               return NULL;
+       }
+       dev_info(&dev->dev, "found %s panel\n", lcdc_info->name);
 
-       return pix_clk_period_picosec;
+       return lcdc_info;
 }
 
 static int fb_probe(struct platform_device *device)
 {
        struct da8xx_lcdc_platform_data *fb_pdata =
                                                device->dev.platform_data;
+       static struct resource *lcdc_regs;
        struct lcd_ctrl_config *lcd_cfg;
        struct fb_videomode *lcdc_info;
        struct fb_info *da8xx_fb_info;
-       struct clk *fb_clk = NULL;
        struct da8xx_fb_par *par;
-       resource_size_t len;
-       int ret, i;
+       struct clk *tmp_lcdc_clk;
+       int ret;
        unsigned long ulcm;
 
        if (fb_pdata == NULL) {
@@ -1229,30 +1351,19 @@ static int fb_probe(struct platform_device *device)
                return -ENOENT;
        }
 
-       lcdc_regs = platform_get_resource(device, IORESOURCE_MEM, 0);
-       if (!lcdc_regs) {
-               dev_err(&device->dev,
-                       "Can not get memory resource for LCD controller\n");
-               return -ENOENT;
-       }
-
-       len = resource_size(lcdc_regs);
+       lcdc_info = da8xx_fb_get_videomode(device);
+       if (lcdc_info == NULL)
+               return -ENODEV;
 
-       lcdc_regs = request_mem_region(lcdc_regs->start, len, lcdc_regs->name);
-       if (!lcdc_regs)
-               return -EBUSY;
-
-       da8xx_fb_reg_base = ioremap(lcdc_regs->start, len);
-       if (!da8xx_fb_reg_base) {
-               ret = -EBUSY;
-               goto err_request_mem;
-       }
+       lcdc_regs = platform_get_resource(device, IORESOURCE_MEM, 0);
+       da8xx_fb_reg_base = devm_ioremap_resource(&device->dev, lcdc_regs);
+       if (IS_ERR(da8xx_fb_reg_base))
+               return PTR_ERR(da8xx_fb_reg_base);
 
-       fb_clk = clk_get(&device->dev, "fck");
-       if (IS_ERR(fb_clk)) {
+       tmp_lcdc_clk = devm_clk_get(&device->dev, "fck");
+       if (IS_ERR(tmp_lcdc_clk)) {
                dev_err(&device->dev, "Can not get device clock\n");
-               ret = -ENODEV;
-               goto err_ioremap;
+               return PTR_ERR(tmp_lcdc_clk);
        }
 
        pm_runtime_enable(&device->dev);
@@ -1275,22 +1386,12 @@ static int fb_probe(struct platform_device *device)
                break;
        }
 
-       for (i = 0, lcdc_info = known_lcd_panels;
-               i < ARRAY_SIZE(known_lcd_panels);
-               i++, lcdc_info++) {
-               if (strcmp(fb_pdata->type, lcdc_info->name) == 0)
-                       break;
-       }
+       lcd_cfg = (struct lcd_ctrl_config *)fb_pdata->controller_data;
 
-       if (i == ARRAY_SIZE(known_lcd_panels)) {
-               dev_err(&device->dev, "GLCD: No valid panel found\n");
-               ret = -ENODEV;
+       if (!lcd_cfg) {
+               ret = -EINVAL;
                goto err_pm_runtime_disable;
-       } else
-               dev_info(&device->dev, "GLCD: Found %s panel\n",
-                                       fb_pdata->type);
-
-       lcd_cfg = (struct lcd_ctrl_config *)fb_pdata->controller_data;
+       }
 
        da8xx_fb_info = framebuffer_alloc(sizeof(struct da8xx_fb_par),
                                        &device->dev);
@@ -1301,21 +1402,18 @@ static int fb_probe(struct platform_device *device)
        }
 
        par = da8xx_fb_info->par;
-       par->lcdc_clk = fb_clk;
-#ifdef CONFIG_CPU_FREQ
-       par->lcd_fck_rate = clk_get_rate(fb_clk);
-#endif
-       par->pxl_clk = lcdc_info->pixclock;
+       par->dev = &device->dev;
+       par->lcdc_clk = tmp_lcdc_clk;
+       par->lcdc_clk_rate = clk_get_rate(par->lcdc_clk);
        if (fb_pdata->panel_power_ctrl) {
                par->panel_power_ctrl = fb_pdata->panel_power_ctrl;
                par->panel_power_ctrl(1);
        }
 
-       if (lcd_init(par, lcd_cfg, lcdc_info) < 0) {
-               dev_err(&device->dev, "lcd_init failed\n");
-               ret = -EFAULT;
-               goto err_release_fb;
-       }
+       fb_videomode_to_var(&da8xx_fb_var, lcdc_info);
+       par->cfg = *lcd_cfg;
+
+       da8xx_fb_lcd_reset();
 
        /* allocate frame buffer */
        par->vram_size = lcdc_info->xres * lcdc_info->yres * lcd_cfg->bpp;
@@ -1363,27 +1461,10 @@ static int fb_probe(struct platform_device *device)
                goto err_release_pl_mem;
        }
 
-       /* Initialize par */
-       da8xx_fb_info->var.bits_per_pixel = lcd_cfg->bpp;
-
-       da8xx_fb_var.xres = lcdc_info->xres;
-       da8xx_fb_var.xres_virtual = lcdc_info->xres;
-
-       da8xx_fb_var.yres         = lcdc_info->yres;
-       da8xx_fb_var.yres_virtual = lcdc_info->yres * LCD_NUM_BUFFERS;
-
        da8xx_fb_var.grayscale =
            lcd_cfg->panel_shade == MONOCHROME ? 1 : 0;
        da8xx_fb_var.bits_per_pixel = lcd_cfg->bpp;
 
-       da8xx_fb_var.hsync_len = lcdc_info->hsync_len;
-       da8xx_fb_var.vsync_len = lcdc_info->vsync_len;
-       da8xx_fb_var.right_margin = lcdc_info->right_margin;
-       da8xx_fb_var.left_margin  = lcdc_info->left_margin;
-       da8xx_fb_var.lower_margin = lcdc_info->lower_margin;
-       da8xx_fb_var.upper_margin = lcdc_info->upper_margin;
-       da8xx_fb_var.pixclock = da8xxfb_pixel_clk_period(par);
-
        /* Initialize fbinfo */
        da8xx_fb_info->flags = FBINFO_FLAG_DEFAULT;
        da8xx_fb_info->fix = da8xx_fb_fix;
@@ -1433,8 +1514,8 @@ static int fb_probe(struct platform_device *device)
                lcdc_irq_handler = lcdc_irq_handler_rev02;
        }
 
-       ret = request_irq(par->irq, lcdc_irq_handler, 0,
-                       DRIVER_NAME, par);
+       ret = devm_request_irq(&device->dev, par->irq, lcdc_irq_handler, 0,
+                              DRIVER_NAME, par);
        if (ret)
                goto irq_freq;
        return 0;
@@ -1463,12 +1544,6 @@ err_pm_runtime_disable:
        pm_runtime_put_sync(&device->dev);
        pm_runtime_disable(&device->dev);
 
-err_ioremap:
-       iounmap(da8xx_fb_reg_base);
-
-err_request_mem:
-       release_mem_region(lcdc_regs->start, len);
-
        return ret;
 }
 
@@ -1546,7 +1621,7 @@ static int fb_suspend(struct platform_device *dev, pm_message_t state)
                par->panel_power_ctrl(0);
 
        fb_set_suspend(info, 1);
-       lcd_disable_raster(true);
+       lcd_disable_raster(DA8XX_FRAME_WAIT);
        lcd_context_save();
        pm_runtime_put_sync(&dev->dev);
        console_unlock();
index 2a8286e..7f9ff75 100644 (file)
@@ -72,6 +72,7 @@ static void efifb_destroy(struct fb_info *info)
        if (request_mem_succeeded)
                release_mem_region(info->apertures->ranges[0].base,
                                   info->apertures->ranges[0].size);
+       fb_dealloc_cmap(&info->cmap);
        framebuffer_release(info);
 }
 
index 15c5abd..c148d06 100644 (file)
@@ -27,6 +27,7 @@
 #include <video/exynos_mipi_dsim.h>
 
 #include "exynos_mipi_dsi_regs.h"
+#include "exynos_mipi_dsi_lowlevel.h"
 
 void exynos_mipi_dsi_func_reset(struct mipi_dsim_device *dsim)
 {
index 5c3960d..f89245b 100644 (file)
@@ -285,13 +285,8 @@ int fb_set_user_cmap(struct fb_cmap_user *cmap, struct fb_info *info)
                rc = -ENODEV;
                goto out;
        }
-       if (cmap->start < 0 || (!info->fbops->fb_setcolreg &&
-                               !info->fbops->fb_setcmap)) {
-               rc = -EINVAL;
-               goto out1;
-       }
+
        rc = fb_set_cmap(&umap, info);
-out1:
        unlock_fb_info(info);
 out:
        fb_dealloc_cmap(&umap);
index 36e1fe2..dacaf74 100644 (file)
 #define FBPIXMAPSIZE   (1024 * 8)
 
 static DEFINE_MUTEX(registration_lock);
+
 struct fb_info *registered_fb[FB_MAX] __read_mostly;
+EXPORT_SYMBOL(registered_fb);
+
 int num_registered_fb __read_mostly;
+EXPORT_SYMBOL(num_registered_fb);
 
 static struct fb_info *get_fb_info(unsigned int idx)
 {
@@ -182,6 +186,7 @@ char* fb_get_buffer_offset(struct fb_info *info, struct fb_pixmap *buf, u32 size
 
        return addr;
 }
+EXPORT_SYMBOL(fb_get_buffer_offset);
 
 #ifdef CONFIG_LOGO
 
@@ -669,6 +674,7 @@ int fb_show_logo(struct fb_info *info, int rotate)
 int fb_prepare_logo(struct fb_info *info, int rotate) { return 0; }
 int fb_show_logo(struct fb_info *info, int rotate) { return 0; }
 #endif /* CONFIG_LOGO */
+EXPORT_SYMBOL(fb_show_logo);
 
 static void *fb_seq_start(struct seq_file *m, loff_t *pos)
 {
@@ -909,6 +915,7 @@ fb_pan_display(struct fb_info *info, struct fb_var_screeninfo *var)
                info->var.vmode &= ~FB_VMODE_YWRAP;
        return 0;
 }
+EXPORT_SYMBOL(fb_pan_display);
 
 static int fb_check_caps(struct fb_info *info, struct fb_var_screeninfo *var,
                         u32 activate)
@@ -1042,6 +1049,7 @@ fb_set_var(struct fb_info *info, struct fb_var_screeninfo *var)
  done:
        return ret;
 }
+EXPORT_SYMBOL(fb_set_var);
 
 int
 fb_blank(struct fb_info *info, int blank)
@@ -1073,6 +1081,7 @@ fb_blank(struct fb_info *info, int blank)
 
        return ret;
 }
+EXPORT_SYMBOL(fb_blank);
 
 static long do_fb_ioctl(struct fb_info *info, unsigned int cmd,
                        unsigned long arg)
@@ -1745,6 +1754,7 @@ register_framebuffer(struct fb_info *fb_info)
 
        return ret;
 }
+EXPORT_SYMBOL(register_framebuffer);
 
 /**
  *     unregister_framebuffer - releases a frame buffer device
@@ -1773,6 +1783,7 @@ unregister_framebuffer(struct fb_info *fb_info)
 
        return ret;
 }
+EXPORT_SYMBOL(unregister_framebuffer);
 
 /**
  *     fb_set_suspend - low level driver signals suspend
@@ -1796,6 +1807,7 @@ void fb_set_suspend(struct fb_info *info, int state)
                fb_notifier_call_chain(FB_EVENT_RESUME, &event);
        }
 }
+EXPORT_SYMBOL(fb_set_suspend);
 
 /**
  *     fbmem_init - init frame buffer subsystem
@@ -1912,6 +1924,7 @@ int fb_get_options(const char *name, char **option)
 
        return retval;
 }
+EXPORT_SYMBOL(fb_get_options);
 
 #ifndef MODULE
 /**
@@ -1959,20 +1972,4 @@ static int __init video_setup(char *options)
 __setup("video=", video_setup);
 #endif
 
-    /*
-     *  Visible symbols for modules
-     */
-
-EXPORT_SYMBOL(register_framebuffer);
-EXPORT_SYMBOL(unregister_framebuffer);
-EXPORT_SYMBOL(num_registered_fb);
-EXPORT_SYMBOL(registered_fb);
-EXPORT_SYMBOL(fb_show_logo);
-EXPORT_SYMBOL(fb_set_var);
-EXPORT_SYMBOL(fb_blank);
-EXPORT_SYMBOL(fb_pan_display);
-EXPORT_SYMBOL(fb_get_buffer_offset);
-EXPORT_SYMBOL(fb_set_suspend);
-EXPORT_SYMBOL(fb_get_options);
-
 MODULE_LICENSE("GPL");
index 401a56e..2456529 100644 (file)
@@ -2029,10 +2029,9 @@ static int matroxfb_probe(struct pci_dev* pdev, const struct pci_device_id* dumm
                return -1;
        }
 
-       minfo = kmalloc(sizeof(*minfo), GFP_KERNEL);
+       minfo = kzalloc(sizeof(*minfo), GFP_KERNEL);
        if (!minfo)
                return -1;
-       memset(minfo, 0, sizeof(*minfo));
 
        minfo->pcidev = pdev;
        minfo->dead = 0;
index dc09ebe..d250ed0 100644 (file)
@@ -46,7 +46,6 @@
 #include <linux/clk.h>
 #include <linux/dma-mapping.h>
 #include <linux/io.h>
-#include <linux/pinctrl/consumer.h>
 #include <linux/fb.h>
 #include <linux/regulator/consumer.h>
 #include <video/of_display_timing.h>
@@ -851,18 +850,11 @@ static int mxsfb_probe(struct platform_device *pdev)
        struct mxsfb_info *host;
        struct fb_info *fb_info;
        struct fb_modelist *modelist;
-       struct pinctrl *pinctrl;
        int ret;
 
        if (of_id)
                pdev->id_entry = of_id->data;
 
-       res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-       if (!res) {
-               dev_err(&pdev->dev, "Cannot get memory IO resource\n");
-               return -ENODEV;
-       }
-
        fb_info = framebuffer_alloc(sizeof(struct mxsfb_info), &pdev->dev);
        if (!fb_info) {
                dev_err(&pdev->dev, "Failed to allocate fbdev\n");
@@ -871,6 +863,7 @@ static int mxsfb_probe(struct platform_device *pdev)
 
        host = to_imxfb_host(fb_info);
 
+       res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
        host->base = devm_ioremap_resource(&pdev->dev, res);
        if (IS_ERR(host->base)) {
                ret = PTR_ERR(host->base);
@@ -882,12 +875,6 @@ static int mxsfb_probe(struct platform_device *pdev)
 
        host->devdata = &mxsfb_devdata[pdev->id_entry->driver_data];
 
-       pinctrl = devm_pinctrl_get_select_default(&pdev->dev);
-       if (IS_ERR(pinctrl)) {
-               ret = PTR_ERR(pinctrl);
-               goto fb_release;
-       }
-
        host->clk = devm_clk_get(&host->pdev->dev, NULL);
        if (IS_ERR(host->clk)) {
                ret = PTR_ERR(host->clk);
index 75f3c74..82a9640 100644 (file)
@@ -612,10 +612,11 @@ static void hdmi_display_set_timing(struct omap_dss_device *dssdev,
        hdmi.ip_data.cfg.cm = cm;
 
        t = hdmi_get_timings();
-       if (t != NULL)
+       if (t != NULL) {
                hdmi.ip_data.cfg = *t;
 
-       dispc_set_tv_pclk(t->timings.pixel_clock * 1000);
+               dispc_set_tv_pclk(t->timings.pixel_clock * 1000);
+       }
 
        mutex_unlock(&hdmi.lock);
 }
index de7e7b5..37b59fe 100644 (file)
@@ -285,9 +285,10 @@ static ssize_t manager_alpha_blending_enabled_show(
 {
        struct omap_overlay_manager_info info;
 
-       mgr->get_manager_info(mgr, &info);
+       if(!dss_has_feature(FEAT_ALPHA_FIXED_ZORDER))
+               return -ENODEV;
 
-       WARN_ON(!dss_has_feature(FEAT_ALPHA_FIXED_ZORDER));
+       mgr->get_manager_info(mgr, &info);
 
        return snprintf(buf, PAGE_SIZE, "%d\n",
                info.partial_alpha_enabled);
@@ -301,7 +302,8 @@ static ssize_t manager_alpha_blending_enabled_store(
        bool enable;
        int r;
 
-       WARN_ON(!dss_has_feature(FEAT_ALPHA_FIXED_ZORDER));
+       if(!dss_has_feature(FEAT_ALPHA_FIXED_ZORDER))
+               return -ENODEV;
 
        r = strtobool(buf, &enable);
        if (r)
index e242ed8..3dfe009 100644 (file)
@@ -779,16 +779,14 @@ void ti_hdmi_4xxx_basic_configure(struct hdmi_ip_data *ip_data)
        struct omap_video_timings video_timing;
        struct hdmi_video_format video_format;
        /* HDMI core */
-       struct hdmi_core_infoframe_avi avi_cfg = ip_data->avi_cfg;
+       struct hdmi_core_infoframe_avi *avi_cfg = &ip_data->avi_cfg;
        struct hdmi_core_video_config v_core_cfg;
        struct hdmi_core_packet_enable_repeat repeat_cfg;
        struct hdmi_config *cfg = &ip_data->cfg;
 
        hdmi_wp_init(&video_timing, &video_format);
 
-       hdmi_core_init(&v_core_cfg,
-               &avi_cfg,
-               &repeat_cfg);
+       hdmi_core_init(&v_core_cfg, avi_cfg, &repeat_cfg);
 
        hdmi_wp_video_init_format(&video_format, &video_timing, cfg);
 
@@ -822,24 +820,24 @@ void ti_hdmi_4xxx_basic_configure(struct hdmi_ip_data *ip_data)
         * configure packet
         * info frame video see doc CEA861-D page 65
         */
-       avi_cfg.db1_format = HDMI_INFOFRAME_AVI_DB1Y_RGB;
-       avi_cfg.db1_active_info =
-               HDMI_INFOFRAME_AVI_DB1A_ACTIVE_FORMAT_OFF;
-       avi_cfg.db1_bar_info_dv = HDMI_INFOFRAME_AVI_DB1B_NO;
-       avi_cfg.db1_scan_info = HDMI_INFOFRAME_AVI_DB1S_0;
-       avi_cfg.db2_colorimetry = HDMI_INFOFRAME_AVI_DB2C_NO;
-       avi_cfg.db2_aspect_ratio = HDMI_INFOFRAME_AVI_DB2M_NO;
-       avi_cfg.db2_active_fmt_ar = HDMI_INFOFRAME_AVI_DB2R_SAME;
-       avi_cfg.db3_itc = HDMI_INFOFRAME_AVI_DB3ITC_NO;
-       avi_cfg.db3_ec = HDMI_INFOFRAME_AVI_DB3EC_XVYUV601;
-       avi_cfg.db3_q_range = HDMI_INFOFRAME_AVI_DB3Q_DEFAULT;
-       avi_cfg.db3_nup_scaling = HDMI_INFOFRAME_AVI_DB3SC_NO;
-       avi_cfg.db4_videocode = cfg->cm.code;
-       avi_cfg.db5_pixel_repeat = HDMI_INFOFRAME_AVI_DB5PR_NO;
-       avi_cfg.db6_7_line_eoftop = 0;
-       avi_cfg.db8_9_line_sofbottom = 0;
-       avi_cfg.db10_11_pixel_eofleft = 0;
-       avi_cfg.db12_13_pixel_sofright = 0;
+       avi_cfg->db1_format = HDMI_INFOFRAME_AVI_DB1Y_RGB;
+       avi_cfg->db1_active_info =
+                       HDMI_INFOFRAME_AVI_DB1A_ACTIVE_FORMAT_OFF;
+       avi_cfg->db1_bar_info_dv = HDMI_INFOFRAME_AVI_DB1B_NO;
+       avi_cfg->db1_scan_info = HDMI_INFOFRAME_AVI_DB1S_0;
+       avi_cfg->db2_colorimetry = HDMI_INFOFRAME_AVI_DB2C_NO;
+       avi_cfg->db2_aspect_ratio = HDMI_INFOFRAME_AVI_DB2M_NO;
+       avi_cfg->db2_active_fmt_ar = HDMI_INFOFRAME_AVI_DB2R_SAME;
+       avi_cfg->db3_itc = HDMI_INFOFRAME_AVI_DB3ITC_NO;
+       avi_cfg->db3_ec = HDMI_INFOFRAME_AVI_DB3EC_XVYUV601;
+       avi_cfg->db3_q_range = HDMI_INFOFRAME_AVI_DB3Q_DEFAULT;
+       avi_cfg->db3_nup_scaling = HDMI_INFOFRAME_AVI_DB3SC_NO;
+       avi_cfg->db4_videocode = cfg->cm.code;
+       avi_cfg->db5_pixel_repeat = HDMI_INFOFRAME_AVI_DB5PR_NO;
+       avi_cfg->db6_7_line_eoftop = 0;
+       avi_cfg->db8_9_line_sofbottom = 0;
+       avi_cfg->db10_11_pixel_eofleft = 0;
+       avi_cfg->db12_13_pixel_sofright = 0;
 
        hdmi_core_aux_infoframe_avi_config(ip_data);
 
index 6285b97..1446c49 100644 (file)
@@ -32,8 +32,8 @@ MODULE_DESCRIPTION("Display Output Switcher Lowlevel Control Abstraction");
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Luming Yu <luming.yu@intel.com>");
 
-static ssize_t video_output_show_state(struct device *dev,
-                                      struct device_attribute *attr, char *buf)
+static ssize_t state_show(struct device *dev, struct device_attribute *attr,
+                         char *buf)
 {
        ssize_t ret_size = 0;
        struct output_device *od = to_output_device(dev);
@@ -42,9 +42,8 @@ static ssize_t video_output_show_state(struct device *dev,
        return ret_size;
 }
 
-static ssize_t video_output_store_state(struct device *dev,
-                                       struct device_attribute *attr,
-                                       const char *buf,size_t count)
+static ssize_t state_store(struct device *dev, struct device_attribute *attr,
+                          const char *buf,size_t count)
 {
        char *endp;
        struct output_device *od = to_output_device(dev);
@@ -62,6 +61,7 @@ static ssize_t video_output_store_state(struct device *dev,
        }
        return count;
 }
+static DEVICE_ATTR_RW(state);
 
 static void video_output_release(struct device *dev)
 {
@@ -69,16 +69,16 @@ static void video_output_release(struct device *dev)
        kfree(od);
 }
 
-static struct device_attribute video_output_attributes[] = {
-       __ATTR(state, 0644, video_output_show_state, video_output_store_state),
-       __ATTR_NULL,
+static struct attribute *video_output_attrs[] = {
+       &dev_attr_state.attr,
+       NULL,
 };
-
+ATTRIBUTE_GROUPS(video_output);
 
 static struct class video_output_class = {
        .name = "video_output",
        .dev_release = video_output_release,
-       .dev_attrs = video_output_attributes,
+       .dev_groups = video_output_groups,
 };
 
 struct output_device *video_output_register(const char *name,
index 6629b29..84c664e 100644 (file)
@@ -259,12 +259,12 @@ static int xilinxfb_assign(struct platform_device *pdev,
                struct resource *res;
 
                res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-               drvdata->regs_phys = res->start;
-               drvdata->regs = devm_request_and_ioremap(&pdev->dev, res);
-               if (!drvdata->regs) {
-                       rc = -EADDRNOTAVAIL;
+               drvdata->regs = devm_ioremap_resource(&pdev->dev, res);
+               if (IS_ERR(drvdata->regs)) {
+                       rc = PTR_ERR(drvdata->regs);
                        goto err_region;
                }
+               drvdata->regs_phys = res->start;
        }
 
        /* Allocate the framebuffer memory */
index 53774b0..077303c 100644 (file)
@@ -25,6 +25,7 @@
        { "r8g8b8", 24, {16, 8}, {8, 8}, {0, 8}, {0, 0}, DRM_FORMAT_RGB888 }, \
        { "x8r8g8b8", 32, {16, 8}, {8, 8}, {0, 8}, {0, 0}, DRM_FORMAT_XRGB8888 }, \
        { "a8r8g8b8", 32, {16, 8}, {8, 8}, {0, 8}, {24, 8}, DRM_FORMAT_ARGB8888 }, \
+       { "a8b8g8r8", 32, {0, 8}, {8, 8}, {16, 8}, {24, 8}, DRM_FORMAT_ABGR8888 }, \
        { "x2r10g10b10", 32, {20, 10}, {10, 10}, {0, 10}, {0, 0}, DRM_FORMAT_XRGB2101010 }, \
        { "a2r10g10b10", 32, {20, 10}, {10, 10}, {0, 10}, {30, 2}, DRM_FORMAT_ARGB2101010 }, \
 }
index f888259..efed3c3 100644 (file)
@@ -23,6 +23,11 @@ enum raster_load_mode {
        LOAD_PALETTE,
 };
 
+enum da8xx_frame_complete {
+       DA8XX_FRAME_WAIT,
+       DA8XX_FRAME_NOWAIT,
+};
+
 struct da8xx_lcdc_platform_data {
        const char manu_name[10];
        void *controller_data;