Re: [PATCH 17/17] OMAPDSS: DSI: Remove redundant fields in omap_dss_dsi_videomode_data

2012-06-27 Thread Tomi Valkeinen
On Tue, 2012-06-26 at 15:06 +0530, Archit Taneja wrote: The struct omap_dss_dsi_videomode_data holds polaritiy/logic level information of the DISPC video port signals DE, HSYNC and VSYNC. This information already exists in the omap_video_timings struct. Use the fields in omap_video_timings

Re: [PATCH 17/17] OMAPDSS: DSI: Remove redundant fields in omap_dss_dsi_videomode_data

2012-06-27 Thread Archit Taneja
On Wednesday 27 June 2012 05:35 PM, Tomi Valkeinen wrote: On Tue, 2012-06-26 at 15:06 +0530, Archit Taneja wrote: The struct omap_dss_dsi_videomode_data holds polaritiy/logic level information of the DISPC video port signals DE, HSYNC and VSYNC. This information already exists in the

Re: [PATCH 17/17] OMAPDSS: DSI: Remove redundant fields in omap_dss_dsi_videomode_data

2012-06-27 Thread Tomi Valkeinen
On Wed, 2012-06-27 at 17:48 +0530, Archit Taneja wrote: On Wednesday 27 June 2012 05:35 PM, Tomi Valkeinen wrote: The sync polarities between DISPC and DSI do not matter elsewhere, they do not affect the DSI output, so why do we have them in the panel data? Why doesn't dsi.c just use some

[PATCH 17/17] OMAPDSS: DSI: Remove redundant fields in omap_dss_dsi_videomode_data

2012-06-26 Thread Archit Taneja
The struct omap_dss_dsi_videomode_data holds polaritiy/logic level information of the DISPC video port signals DE, HSYNC and VSYNC. This information already exists in the omap_video_timings struct. Use the fields in omap_video_timings to program VP_DE_POL, VP_HSYNC_POL and VP_VSYNC_POL in