* [PATCH] ARM: OMAP2+ Add Primview dispalys to panel-generic
@ 2011-07-08 10:47 Jan Weitzel
2011-08-04 12:18 ` Tomi Valkeinen
0 siblings, 1 reply; 2+ messages in thread
From: Jan Weitzel @ 2011-07-08 10:47 UTC (permalink / raw)
To: linux-arm-kernel
Add displays to panel-generic-dpi.c
Prime View PD035VL1 (640 x 480)
Prime View PD050VL1 (640 x 480)
Prime View PD104SLF (800 x 600)
Prime View PM070WL4 (800 x 480)
Signed-off-by: Jan Weitzel <j.weitzel@phytec.de>
---
drivers/video/omap2/displays/panel-generic-dpi.c | 100 ++++++++++++++++++++++
1 files changed, 100 insertions(+), 0 deletions(-)
diff --git a/drivers/video/omap2/displays/panel-generic-dpi.c b/drivers/video/omap2/displays/panel-generic-dpi.c
index 9c90f75..87d9ba1 100644
--- a/drivers/video/omap2/displays/panel-generic-dpi.c
+++ b/drivers/video/omap2/displays/panel-generic-dpi.c
@@ -232,6 +232,106 @@ static struct panel_config generic_dpi_panels[] = {
.power_off_delay = 0,
.name = "powertip_ph480272t",
},
+
+ /* Prime-View PD035VL1 */
+ {
+ {
+ .x_res = 640,
+ .y_res = 480,
+
+ .pixel_clock = 25000,
+
+ .hsw = 96,
+ .hfp = 18,
+ .hbp = 46,
+
+ .vsw = 2,
+ .vfp = 10,
+ .vbp = 33,
+ },
+ .acbi = 0x0,
+ .acb = 0x28,
+ .config = OMAP_DSS_LCD_TFT | OMAP_DSS_LCD_IVS |
+ OMAP_DSS_LCD_IHS | OMAP_DSS_LCD_IPC,
+ .power_on_delay = 50,
+ .power_off_delay = 100,
+ .name = "pd035vl1",
+ },
+
+ /* Prime-View PD050VL1 */
+ {
+ {
+ .x_res = 640,
+ .y_res = 480,
+
+ .pixel_clock = 25000,
+
+ .hsw = 96,
+ .hfp = 18,
+ .hbp = 46,
+
+ .vsw = 2,
+ .vfp = 10,
+ .vbp = 33,
+ },
+ .acbi = 0x0,
+ .acb = 0x28,
+ .config = OMAP_DSS_LCD_TFT | OMAP_DSS_LCD_IVS |
+ OMAP_DSS_LCD_IHS | OMAP_DSS_LCD_IPC,
+ .power_on_delay = 50,
+ .power_off_delay = 100,
+ .name = "pd050vl1",
+ },
+
+ /* Prime-View PM070WL4 */
+ {
+ {
+ .x_res = 800,
+ .y_res = 480,
+
+ .pixel_clock = 32000,
+
+ .hsw = 128,
+ .hfp = 42,
+ .hbp = 86,
+
+ .vsw = 2,
+ .vfp = 10,
+ .vbp = 33,
+ },
+ .acbi = 0x0,
+ .acb = 0x28,
+ .config = OMAP_DSS_LCD_TFT | OMAP_DSS_LCD_IVS |
+ OMAP_DSS_LCD_IHS | OMAP_DSS_LCD_IPC,
+ .power_on_delay = 50,
+ .power_off_delay = 100,
+ .name = "pm070wl4",
+ },
+
+ /* Prime-View PD104SLF */
+ {
+ {
+ .x_res = 800,
+ .y_res = 600,
+
+ .pixel_clock = 40000,
+
+ .hsw = 128,
+ .hfp = 42,
+ .hbp = 86,
+
+ .vsw = 4,
+ .vfp = 1,
+ .vbp = 23,
+ },
+ .acbi = 0x0,
+ .acb = 0x28,
+ .config = OMAP_DSS_LCD_TFT | OMAP_DSS_LCD_IVS |
+ OMAP_DSS_LCD_IHS | OMAP_DSS_LCD_IPC,
+ .power_on_delay = 50,
+ .power_off_delay = 100,
+ .name = "pd104slf",
+ },
};
struct panel_drv_data {
--
1.7.0.4
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [PATCH] ARM: OMAP2+ Add Primview dispalys to panel-generic
2011-07-08 10:47 [PATCH] ARM: OMAP2+ Add Primview dispalys to panel-generic Jan Weitzel
@ 2011-08-04 12:18 ` Tomi Valkeinen
0 siblings, 0 replies; 2+ messages in thread
From: Tomi Valkeinen @ 2011-08-04 12:18 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On Fri, 2011-07-08 at 12:47 +0200, Jan Weitzel wrote:
> Add displays to panel-generic-dpi.c
> Prime View PD035VL1 (640 x 480)
> Prime View PD050VL1 (640 x 480)
> Prime View PD104SLF (800 x 600)
> Prime View PM070WL4 (800 x 480)
Did you copy the acb and power_on/off_delay values from the sharp panel?
acbi and acb are not needed at all, and you should check if
power_on/off_delay are needed and if they are, what they should be.
Also, the first two panels look identical. While there's nothing wrong
with that, I wonder if could be a more generic "family" name for the
panels, and there would be need to define the timings only once.
Tomi
> Signed-off-by: Jan Weitzel <j.weitzel@phytec.de>
> ---
> drivers/video/omap2/displays/panel-generic-dpi.c | 100 ++++++++++++++++++++++
> 1 files changed, 100 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/video/omap2/displays/panel-generic-dpi.c b/drivers/video/omap2/displays/panel-generic-dpi.c
> index 9c90f75..87d9ba1 100644
> --- a/drivers/video/omap2/displays/panel-generic-dpi.c
> +++ b/drivers/video/omap2/displays/panel-generic-dpi.c
> @@ -232,6 +232,106 @@ static struct panel_config generic_dpi_panels[] = {
> .power_off_delay = 0,
> .name = "powertip_ph480272t",
> },
> +
> + /* Prime-View PD035VL1 */
> + {
> + {
> + .x_res = 640,
> + .y_res = 480,
> +
> + .pixel_clock = 25000,
> +
> + .hsw = 96,
> + .hfp = 18,
> + .hbp = 46,
> +
> + .vsw = 2,
> + .vfp = 10,
> + .vbp = 33,
> + },
> + .acbi = 0x0,
> + .acb = 0x28,
> + .config = OMAP_DSS_LCD_TFT | OMAP_DSS_LCD_IVS |
> + OMAP_DSS_LCD_IHS | OMAP_DSS_LCD_IPC,
> + .power_on_delay = 50,
> + .power_off_delay = 100,
> + .name = "pd035vl1",
> + },
> +
> + /* Prime-View PD050VL1 */
> + {
> + {
> + .x_res = 640,
> + .y_res = 480,
> +
> + .pixel_clock = 25000,
> +
> + .hsw = 96,
> + .hfp = 18,
> + .hbp = 46,
> +
> + .vsw = 2,
> + .vfp = 10,
> + .vbp = 33,
> + },
> + .acbi = 0x0,
> + .acb = 0x28,
> + .config = OMAP_DSS_LCD_TFT | OMAP_DSS_LCD_IVS |
> + OMAP_DSS_LCD_IHS | OMAP_DSS_LCD_IPC,
> + .power_on_delay = 50,
> + .power_off_delay = 100,
> + .name = "pd050vl1",
> + },
> +
> + /* Prime-View PM070WL4 */
> + {
> + {
> + .x_res = 800,
> + .y_res = 480,
> +
> + .pixel_clock = 32000,
> +
> + .hsw = 128,
> + .hfp = 42,
> + .hbp = 86,
> +
> + .vsw = 2,
> + .vfp = 10,
> + .vbp = 33,
> + },
> + .acbi = 0x0,
> + .acb = 0x28,
> + .config = OMAP_DSS_LCD_TFT | OMAP_DSS_LCD_IVS |
> + OMAP_DSS_LCD_IHS | OMAP_DSS_LCD_IPC,
> + .power_on_delay = 50,
> + .power_off_delay = 100,
> + .name = "pm070wl4",
> + },
> +
> + /* Prime-View PD104SLF */
> + {
> + {
> + .x_res = 800,
> + .y_res = 600,
> +
> + .pixel_clock = 40000,
> +
> + .hsw = 128,
> + .hfp = 42,
> + .hbp = 86,
> +
> + .vsw = 4,
> + .vfp = 1,
> + .vbp = 23,
> + },
> + .acbi = 0x0,
> + .acb = 0x28,
> + .config = OMAP_DSS_LCD_TFT | OMAP_DSS_LCD_IVS |
> + OMAP_DSS_LCD_IHS | OMAP_DSS_LCD_IPC,
> + .power_on_delay = 50,
> + .power_off_delay = 100,
> + .name = "pd104slf",
> + },
> };
>
> struct panel_drv_data {
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-08-04 12:18 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-07-08 10:47 [PATCH] ARM: OMAP2+ Add Primview dispalys to panel-generic Jan Weitzel
2011-08-04 12:18 ` Tomi Valkeinen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).