mythripk | 2 Jan 2012 09:32
Picon
Favicon

[PATCH v4 0/2] OMAPDSS: HDMI: Boardfile cleanup

From: Mythri P K <mythripk <at> ti.com>

Clean up of boardfiles to have the functions common to differnt boards in a
common display file, Also in some OMAP4 boards external pull-up's are enabled
thus enabling internal pull up should be a parameter from the boardfile.

changes since v3 : Comment improvement to move some portion from boardfile to
display.
changes since v2 : Make the flag as enum.
changes since v1 : Make omap hdmi_init generic to handle different parameters
and pass the pull up as a flag instead of int. 

Mythri P K (2):
  OMAPDSS: HDMI: Move duplicate code from boardfile
  OMAPDSS: HDMI: Disable DDC internal pull up

 arch/arm/mach-omap2/board-4430sdp.c    |   23 ++++++------------
 arch/arm/mach-omap2/board-omap4panda.c |   25 +++++++-------------
 arch/arm/mach-omap2/display.c          |   39 ++++++++++++++++++++++++++++++++
 include/video/omapdss.h                |    6 +++++
 4 files changed, 62 insertions(+), 31 deletions(-)

--

-- 
1.7.5.4

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo <at> vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

(Continue reading)

mythripk | 2 Jan 2012 09:32
Picon
Favicon

[PATCH v4 1/2] OMAPDSS: HDMI: Move duplicate code from boardfile

From: Mythri P K <mythripk <at> ti.com>

Move duplicate HDMI mux_init code from omap4 and panda board file
to display file.

Signed-off-by: Mythri P K <mythripk <at> ti.com>
---
 arch/arm/mach-omap2/board-4430sdp.c    |   16 +---------------
 arch/arm/mach-omap2/board-omap4panda.c |   17 +----------------
 arch/arm/mach-omap2/display.c          |   23 +++++++++++++++++++++++
 include/video/omapdss.h                |    2 ++
 4 files changed, 27 insertions(+), 31 deletions(-)

diff --git a/arch/arm/mach-omap2/board-4430sdp.c b/arch/arm/mach-omap2/board-4430sdp.c
index 73b1e99..c3bd640 100644
--- a/arch/arm/mach-omap2/board-4430sdp.c
+++ b/arch/arm/mach-omap2/board-4430sdp.c
 <at>  <at>  -595,20 +595,6  <at>  <at>  static void __init omap_sfh7741prox_init(void)
 			__func__, OMAP4_SFH7741_ENABLE_GPIO, error);
 }

-static void sdp4430_hdmi_mux_init(void)
-{
-	/* PAD0_HDMI_HPD_PAD1_HDMI_CEC */
-	omap_mux_init_signal("hdmi_hpd",
-			OMAP_PIN_INPUT_PULLUP);
-	omap_mux_init_signal("hdmi_cec",
-			OMAP_PIN_INPUT_PULLUP);
-	/* PAD0_HDMI_DDC_SCL_PAD1_HDMI_DDC_SDA */
-	omap_mux_init_signal("hdmi_ddc_scl",
(Continue reading)

mythripk | 2 Jan 2012 09:32
Picon
Favicon

[PATCH v4 2/2] OMAPDSS: HDMI: Disable DDC internal pull up

From: Mythri P K <mythripk <at> ti.com>

Disables the internal pull resistor for SDA and SCL which are enabled by
default, as there are external pull up's in 4460 and 4430 ES2.3
SDP, Blaze and Panda Boards, It is done to avoid the EDID read failure.

Signed-off-by: Ricardo Salveti de Araujo <ricardo.salveti <at> linaro.org>
Signed-off-by: Mythri P K <mythripk <at> ti.com>
---
 arch/arm/mach-omap2/board-4430sdp.c    |    9 ++++++++-
 arch/arm/mach-omap2/board-omap4panda.c |   10 +++++++++-
 arch/arm/mach-omap2/display.c          |   22 +++++++++++++++++++---
 include/video/omapdss.h                |    6 +++++-
 4 files changed, 41 insertions(+), 6 deletions(-)

diff --git a/arch/arm/mach-omap2/board-4430sdp.c b/arch/arm/mach-omap2/board-4430sdp.c
index c3bd640..4af874a 100644
--- a/arch/arm/mach-omap2/board-4430sdp.c
+++ b/arch/arm/mach-omap2/board-4430sdp.c
 <at>  <at>  -826,7 +826,14  <at>  <at>  static void omap_4430sdp_display_init(void)
 	sdp4430_lcd_init();
 	sdp4430_picodlp_init();
 	omap_display_init(&sdp4430_dss_data);
-	omap_hdmi_init();
+	/*
+	 * OMAP4460SDP/Blaze and OMAP4430 ES2.3 SDP/Blaze boards and
+	 * later have external pull up on the HDMI I2C lines
+	 */
+	if (cpu_is_omap446x() || omap_rev() > OMAP4430_REV_ES2_2)
+		omap_hdmi_init(OMAP_HDMI_SDA_SCL_EXTERNAL_PULLUP);
(Continue reading)

mythripk | 2 Jan 2012 09:44
Picon
Favicon

[PATCH 0/4] OMAPDSS: HDMI: Improve the timings logic in HDMI

From: Mythri P K <mythripk <at> ti.com>

There are some duplicate timing structure which are not needed thus removing
them to clean the code.
Also the static mapped timing structure is quite complicated to add new
timings, so simplify it by using array indexed method.

changes since V1: change of hdmi_find_timing function to return pointer to the
timing struct rather than deep copy to timing structure passed as parameter
variable name change from temp to timing2 in comparator function.

Mythri P K (4):
  OMAPDSS: HDMI: remove duplicate video interface code
  OMAPDSS: HDMI: update static timing table
  OMAPDSS: HDMI: change the timing match logic
  OMAPDSS: HDMI: remove duplicate code and mode parameter

 drivers/video/omap2/dss/hdmi.c            |  254 +++++++++++++----------------
 drivers/video/omap2/dss/ti_hdmi.h         |   14 +-
 drivers/video/omap2/dss/ti_hdmi_4xxx_ip.c |   47 ++----
 drivers/video/omap2/dss/ti_hdmi_4xxx_ip.h |    7 -
 4 files changed, 134 insertions(+), 188 deletions(-)

--

-- 
1.7.5.4

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo <at> vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
(Continue reading)

mythripk | 2 Jan 2012 09:44
Picon
Favicon

[PATCH v2 1/4] OMAPDSS: HDMI: remove duplicate video interface code

From: Mythri P K <mythripk <at> ti.com>

video interface structure is a duplicate structure with parameters which are
already present in ip_data config structure, Thus removing the structure and
modifying corresponding code.

Signed-off-by: Mythri P K <mythripk <at> ti.com>
---
 drivers/video/omap2/dss/ti_hdmi_4xxx_ip.c |   31 +++++++---------------------
 drivers/video/omap2/dss/ti_hdmi_4xxx_ip.h |    7 ------
 2 files changed, 8 insertions(+), 30 deletions(-)

diff --git a/drivers/video/omap2/dss/ti_hdmi_4xxx_ip.c b/drivers/video/omap2/dss/ti_hdmi_4xxx_ip.c
index e1a6ce5..403d6fc 100644
--- a/drivers/video/omap2/dss/ti_hdmi_4xxx_ip.c
+++ b/drivers/video/omap2/dss/ti_hdmi_4xxx_ip.c
 <at>  <at>  -629,8 +629,7  <at>  <at>  static void hdmi_core_av_packet_config(struct hdmi_ip_data *ip_data,
 }

 static void hdmi_wp_init(struct omap_video_timings *timings,
-			struct hdmi_video_format *video_fmt,
-			struct hdmi_video_interface *video_int)
+			struct hdmi_video_format *video_fmt)
 {
 	pr_debug("Enter hdmi_wp_init\n");

 <at>  <at>  -645,12 +644,6  <at>  <at>  static void hdmi_wp_init(struct omap_video_timings *timings,
 	video_fmt->y_res = 0;
 	video_fmt->x_res = 0;

(Continue reading)

mythripk | 2 Jan 2012 09:44
Picon
Favicon

[PATCH v2 4/4] OMAPDSS: HDMI: remove duplicate code and mode parameter

From: Mythri P K <mythripk <at> ti.com>

code and mode parameters are already a part of the ip_data structure
so no need to keep the same parameters again in hdmi global structure.

Signed-off-by: Mythri P K <mythripk <at> ti.com>
---
 drivers/video/omap2/dss/hdmi.c |   18 +++++++-----------
 1 files changed, 7 insertions(+), 11 deletions(-)

diff --git a/drivers/video/omap2/dss/hdmi.c b/drivers/video/omap2/dss/hdmi.c
index 6892faa..34f3dae 100644
--- a/drivers/video/omap2/dss/hdmi.c
+++ b/drivers/video/omap2/dss/hdmi.c
 <at>  <at>  -66,8 +66,6  <at>  <at>  static struct {
 	struct omap_display_platform_data *pdata;
 	struct platform_device *pdev;
 	struct hdmi_ip_data ip_data;
-	int code;
-	int mode;

 	struct clk *sys_clk;
 } hdmi;
 <at>  <at>  -163,7 +161,7  <at>  <at>  static const struct hdmi_config *hdmi_find_timing(
 	const struct hdmi_config *timing, timing1 = { {0}, {0} };

 	for (i = 0; i < len; i++) {
-		if (timings_arr[i].cm.code == hdmi.code) {
+		if (timings_arr[i].cm.code == hdmi.ip_data.cfg.cm.code) {
 			timing = &timings_arr[i];
(Continue reading)

mythripk | 2 Jan 2012 09:44
Picon
Favicon

[PATCH v2 3/4] OMAPDSS: HDMI: change the timing match logic

From: Mythri P K <mythripk <at> ti.com>

Change the timing match logic, Instead of the statically mapped method
to get the corresponding timings for a given code and mode, move to a
simpler array indexed method. It  will help to scale up to add more
timings when needed.

Signed-off-by: Mythri P K <mythripk <at> ti.com>
---
 drivers/video/omap2/dss/hdmi.c |  174 +++++++++++++++++-----------------------
 1 files changed, 75 insertions(+), 99 deletions(-)

diff --git a/drivers/video/omap2/dss/hdmi.c b/drivers/video/omap2/dss/hdmi.c
index f76ae47..6892faa 100644
--- a/drivers/video/omap2/dss/hdmi.c
+++ b/drivers/video/omap2/dss/hdmi.c
 <at>  <at>  -58,8 +58,6  <at>  <at> 
 #define EDID_SIZE_BLOCK0_TIMING_DESCRIPTOR	4
 #define EDID_SIZE_BLOCK1_TIMING_DESCRIPTOR	4

-#define OMAP_HDMI_TIMINGS_NB			34
-
 #define HDMI_DEFAULT_REGN 16
 #define HDMI_DEFAULT_REGM2 1

 <at>  <at>  -88,7 +86,7  <at>  <at>  static struct {
  * map it to corresponding CEA or VESA index.
  */

-static const struct hdmi_config cea_vesa_timings[OMAP_HDMI_TIMINGS_NB] = {
(Continue reading)

mythripk | 2 Jan 2012 09:44
Picon
Favicon

[PATCH v2 2/4] OMAPDSS: HDMI: update static timing table

From: Mythri P K <mythripk <at> ti.com>

Add the vsync polarity, hsync polarity, interlace to hdmi_video_timings.
Remove the now duplicate structure hdmi_timings.
update the static table structure in HDMI with CEA/VESA code and mode.

Signed-off-by: Mythri P K <mythripk <at> ti.com>
---
 drivers/video/omap2/dss/hdmi.c            |   96 ++++++++++++++--------------
 drivers/video/omap2/dss/ti_hdmi.h         |   14 ++---
 drivers/video/omap2/dss/ti_hdmi_4xxx_ip.c |   20 +++---
 3 files changed, 63 insertions(+), 67 deletions(-)

diff --git a/drivers/video/omap2/dss/hdmi.c b/drivers/video/omap2/dss/hdmi.c
index c56378c..f76ae47 100644
--- a/drivers/video/omap2/dss/hdmi.c
+++ b/drivers/video/omap2/dss/hdmi.c
 <at>  <at>  -88,42 +88,42  <at>  <at>  static struct {
  * map it to corresponding CEA or VESA index.
  */

-static const struct hdmi_timings cea_vesa_timings[OMAP_HDMI_TIMINGS_NB] = {
-	{ {640, 480, 25200, 96, 16, 48, 2, 10, 33} , 0 , 0},
-	{ {1280, 720, 74250, 40, 440, 220, 5, 5, 20}, 1, 1},
-	{ {1280, 720, 74250, 40, 110, 220, 5, 5, 20}, 1, 1},
-	{ {720, 480, 27027, 62, 16, 60, 6, 9, 30}, 0, 0},
-	{ {2880, 576, 108000, 256, 48, 272, 5, 5, 39}, 0, 0},
-	{ {1440, 240, 27027, 124, 38, 114, 3, 4, 15}, 0, 0},
-	{ {1440, 288, 27000, 126, 24, 138, 3, 2, 19}, 0, 0},
-	{ {1920, 540, 74250, 44, 528, 148, 5, 2, 15}, 1, 1},
(Continue reading)

mythripk | 2 Jan 2012 09:58
Picon
Favicon

[PATCH 1/3 REPOST] OMAPDSS: HDMI: Move Avi infoframe structure to

From: Mythri P K <mythripk <at> ti.com>

With AVI infoframe various parameters of video stream such as
aspect ratio, quantization range, videocode etc will be indicated
from source to sink.Thus AVI information needs to be set/accessed
by the middle ware based on the video content.
Thus this parameter is now moved to the ip_data structure.

Signed-off-by: Mythri P K <mythripk <at> ti.com>
---
 drivers/video/omap2/dss/ti_hdmi.h         |   42 +++++++++++++++++++++++++++++
 drivers/video/omap2/dss/ti_hdmi_4xxx_ip.c |    8 +++---
 drivers/video/omap2/dss/ti_hdmi_4xxx_ip.h |   40 ---------------------------
 3 files changed, 46 insertions(+), 44 deletions(-)

diff --git a/drivers/video/omap2/dss/ti_hdmi.h b/drivers/video/omap2/dss/ti_hdmi.h
index 3cf5198..835cfb1 100644
--- a/drivers/video/omap2/dss/ti_hdmi.h
+++ b/drivers/video/omap2/dss/ti_hdmi.h
 <at>  <at>  -108,6 +108,47  <at>  <at>  struct ti_hdmi_ip_ops {

 };

+/*
+ * Refer to section 8.2 in HDMI 1.3 specification for
+ * details about infoframe databytes
+ */
+struct hdmi_core_infoframe_avi {
+	/* Y0, Y1 rgb,yCbCr */
+	u8	db1_format;
(Continue reading)

mythripk | 2 Jan 2012 09:58
Picon
Favicon

[PATCH 1/3 REPOST] OMAPDSS: HDMI: Move Avi infoframe structure to

From: Mythri P K <mythripk <at> ti.com>

With AVI infoframe various parameters of video stream such as
aspect ratio, quantization range, videocode etc will be indicated
from source to sink.Thus AVI information needs to be set/accessed
by the middle ware based on the video content.
Thus this parameter is now moved to the ip_data structure.

Signed-off-by: Mythri P K <mythripk <at> ti.com>
---
 drivers/video/omap2/dss/ti_hdmi.h         |   42 +++++++++++++++++++++++++++++
 drivers/video/omap2/dss/ti_hdmi_4xxx_ip.c |    8 +++---
 drivers/video/omap2/dss/ti_hdmi_4xxx_ip.h |   40 ---------------------------
 3 files changed, 46 insertions(+), 44 deletions(-)

diff --git a/drivers/video/omap2/dss/ti_hdmi.h b/drivers/video/omap2/dss/ti_hdmi.h
index 3cf5198..835cfb1 100644
--- a/drivers/video/omap2/dss/ti_hdmi.h
+++ b/drivers/video/omap2/dss/ti_hdmi.h
 <at>  <at>  -108,6 +108,47  <at>  <at>  struct ti_hdmi_ip_ops {

 };

+/*
+ * Refer to section 8.2 in HDMI 1.3 specification for
+ * details about infoframe databytes
+ */
+struct hdmi_core_infoframe_avi {
+	/* Y0, Y1 rgb,yCbCr */
+	u8	db1_format;
(Continue reading)


Gmane