Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

3.7: Beaglebone HDMI: Refresh patches #14

Closed
wants to merge 1 commit into from

2 participants

@joelagnel
Collaborator

Adding 1280x720 resolution to HDMI and other changes.

Joel A Fernandes 3.7: Beaglebone HDMI: Refresh patches
Signed-off-by: Joel A Fernandes <joelagnel@ti.com>
daacb54
@koenkooi

Applied, but still no picture on my screen:

root@bonelt:~# uname -a
Linux bonelt 3.7.0-rc8 #328 SMP Thu Dec 6 23:03:17 CET 2012 armv7l GNU/Linux
root@bonelt:~# dmesg | grep -i hdmi
[ 1.411833] capebus bone:0: Slot #1 id='NXP HDMI on CapeBus,00A1,Beagleboardtoys,Override Part#'
[ 1.564359] bonegeneric bone-0:1: NXP HDMI on CapeBus: V=00A1 'bone-generic-cape'
[ 1.691428] hdmitx(tda19989) 1.3.0 compiled: Dec 6 2012 23:03:02 -ioctl (2009-10-15)
[ 1.974223] HDMI TX SW Version:5.3 compatibility:0
[ 2.999455] hdmi EDID received
[ 3.002718] tmdlHdmiTxGetEdidSourceAddress 3813
root@bonelt:~# fbset

mode "1280x720-75"
# D: 92.756 MHz, H: 56.318 kHz, V: 75.191 Hz
geometry 1280 720 1280 720 16
timings 10781 219 109 19 5 39 5
rgba 5/11,6/5,5/0,0/0
endmode

@koenkooi koenkooi closed this
@joelagnel
Collaborator

I'm sorry I didn't realise it was merged, apologise on my part. Thank you Koen and Robert. I've also cleared my comments to avoid confusing others. New to GitHub pull requests!

Coming back to the issue, Koen can you try different monitors? Also what beaglebone rev is this? Do you see a pixclk signal on the expansion header? I tried 3 monitors here and all work.

@koenkooi

The part that stands out in the past about is that it is using a 92mhz pixclock instead of the 72 needed for 720p60

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Dec 6, 2012
  1. 3.7: Beaglebone HDMI: Refresh patches

    Joel A Fernandes authored
    Signed-off-by: Joel A Fernandes <joelagnel@ti.com>
This page is out of date. Refresh to see the latest.
View
8 patches/hdmi/0001-Import-working-HDMI-driver-from-3.2-kernel.patch
@@ -1,7 +1,7 @@
-From d036ea43d380dcf93857a809a4774db33d6b8189 Mon Sep 17 00:00:00 2001
+From b6fa181e6981fbc51cb6996e922d565e3f73a74f Mon Sep 17 00:00:00 2001
From: Joel A Fernandes <joelagnel@ti.com>
Date: Tue, 13 Nov 2012 11:41:09 -0500
-Subject: [PATCH 1/7] Import working HDMI driver from 3.2 kernel
+Subject: [PATCH 1/8] Import working HDMI driver from 3.2 kernel
Fixes ontop of the original driver:
6a3b828 hdmi: Fix Video port mapping in NXP driver
@@ -62,7 +62,7 @@ Signed-off-by: Joel A Fernandes <joelagnel@ti.com>
drivers/video/nxp/tda998x_version.h | 17 +
drivers/video/nxp/test/Makefile | 28 +
drivers/video/nxp/test/demo_tda.c | 758 ++
- 53 files changed, 54063 insertions(+), 0 deletions(-)
+ 53 files changed, 54063 insertions(+)
create mode 100755 drivers/video/nxp/MakeModules
create mode 100755 drivers/video/nxp/Makefile
create mode 100755 drivers/video/nxp/comps/tmbslHdmiTx/inc/tmbslHdmiTx_types.h
@@ -72647,5 +72647,5 @@ index 0000000..0365a6c
+ return 0;
+}
--
-1.7.7.6
+1.7.9.5
View
8 patches/hdmi/0002-Added-DT-binding-to-NXP-driver.patch
@@ -1,12 +1,12 @@
-From 10c746eaa24f4fe5312292f4d36931942137bff3 Mon Sep 17 00:00:00 2001
+From cade52843489c0321388b60e590dccfe8d24bbfc Mon Sep 17 00:00:00 2001
From: Joel A Fernandes <joelagnel@ti.com>
Date: Thu, 15 Nov 2012 16:45:47 -0600
-Subject: [PATCH 2/7] Added DT binding to NXP driver
+Subject: [PATCH 2/8] Added DT binding to NXP driver
Signed-off-by: Joel A Fernandes <joelagnel@ti.com>
---
drivers/video/nxp/tda998x.c | 7 +++++++
- 1 files changed, 7 insertions(+), 0 deletions(-)
+ 1 file changed, 7 insertions(+)
diff --git a/drivers/video/nxp/tda998x.c b/drivers/video/nxp/tda998x.c
index 4b45e41..2864221 100755
@@ -34,5 +34,5 @@ index 4b45e41..2864221 100755
.probe = this_i2c_probe,
.remove = this_i2c_remove,
--
-1.7.7.6
+1.7.9.5
View
58 patches/hdmi/0003-da8xx-fb-Add-timings-for-720x480-60.patch
@@ -1,18 +1,20 @@
-From dd6a7e89930e18f5c776c7708d5d964a9d6cc356 Mon Sep 17 00:00:00 2001
+From c242a9a36af2f76755d878c3f41547ee3657b07e Mon Sep 17 00:00:00 2001
From: Joel A Fernandes <joelagnel@ti.com>
Date: Tue, 20 Nov 2012 11:17:33 -0600
-Subject: [PATCH 3/7] da8xx-fb: Add timings for 720x480@60
+Subject: [PATCH 3/8] da8xx-fb: Add timings for 720x480@60
+
+Also add timings for 1080p, 720p and 480p
Signed-off-by: Joel A Fernandes <joelagnel@ti.com>
---
- drivers/video/da8xx-fb.c | 14 ++++++++++++++
- 1 files changed, 14 insertions(+), 0 deletions(-)
+ drivers/video/da8xx-fb.c | 56 ++++++++++++++++++++++++++++++++++++++++++++++
+ 1 file changed, 56 insertions(+)
diff --git a/drivers/video/da8xx-fb.c b/drivers/video/da8xx-fb.c
-index 6d98184..91061e6 100644
+index 6d98184..d6b228f 100644
--- a/drivers/video/da8xx-fb.c
+++ b/drivers/video/da8xx-fb.c
-@@ -312,6 +312,20 @@ static struct da8xx_panel known_lcd_panels[] = {
+@@ -312,6 +312,62 @@ static struct da8xx_panel known_lcd_panels[] = {
.pxl_clk = 30000000,
.invert_pxl_clk = 0,
},
@@ -30,9 +32,51 @@ index 6d98184..91061e6 100644
+ .pxl_clk = 27027000,
+ .invert_pxl_clk = 0,
+ },
++ [7] = {
++ /* 1024 x 768 @ 60 Hz Reduced blanking VESA CVT 0.79M3-R */
++ .name = "nxp-1280x720@60",
++ .width = 1280,
++ .height = 720,
++ .hfp = 109, // 20
++ .hbp = 219, // 54
++ .hsw = 39,
++ .vfp = 5,
++ .vbp = 19,
++ .vsw = 5,
++ .pxl_clk = 74250000,
++ .invert_pxl_clk = 0,
++ },
++ [8] = {
++ /* 1024 x 768 @ 60 Hz Reduced blanking VESA CVT 0.79M3-R */
++ .name = "nxp-640x480@60",
++ .width = 640,
++ .height = 480,
++ .hfp = 19, // 20 /* Need more changes later */
++ .hbp = 79, // 54
++ .hsw = 59,
++ .vfp = 9,
++ .vbp = 30,
++ .vsw = 6,
++ .pxl_clk = 25200000,
++ .invert_pxl_clk = 0,
++ },
++ [9] = {
++ /* 1024 x 768 @ 60 Hz Reduced blanking VESA CVT 0.79M3-R */
++ .name = "nxp-1920x1080@24",
++ .width = 1920,
++ .height = 1080,
++ .hfp = 103, // 20
++ .hbp = 311, // 54
++ .hsw = 31,
++ .vfp = 37,
++ .vbp = 60,
++ .vsw = 8,
++ .pxl_clk = 96000000,
++ .invert_pxl_clk = 0,
++ },
};
/* Enable the Raster Engine of the LCD Controller */
--
-1.7.7.6
+1.7.9.5
View
53 ...us-override-and-pinmux-for-da8xx-dt.patch → ...us-override-and-pinmux-for-da8xx-dt.patch
@@ -1,22 +1,21 @@
-From 8dc8c9075c40f1f239f917f902afad61a30418a5 Mon Sep 17 00:00:00 2001
+From ad5d61fec15720d63330c3611d43900d8b27df74 Mon Sep 17 00:00:00 2001
From: Joel A Fernandes <joelagnel@ti.com>
-Date: Wed, 21 Nov 2012 12:19:29 -0600
-Subject: [PATCH 6/7] Add capebus override and pinmux for da8xx-dt
+Date: Thu, 6 Dec 2012 02:57:00 -0600
+Subject: [PATCH 4/8] Add capebus override and pinmux for da8xx-dt
-Capebus override is left for LT only, rest is kept in common
-as there is an HDMI cape as well that could reuse in future.
+Config da8xx-fb for 1280x720 with disp-pll settings
+Add commented disp-pll settings for other freqs
Signed-off-by: Joel A Fernandes <joelagnel@ti.com>
---
- arch/arm/boot/dts/am335x-bone-common.dtsi | 48 +++++++++++++++++++++++++++++
- arch/arm/boot/dts/am335x-bonelt.dts | 10 ++++++
- 2 files changed, 58 insertions(+), 0 deletions(-)
+ arch/arm/boot/dts/am335x-bone-common.dtsi | 54 +++++++++++++++++++++++++++++
+ 1 file changed, 54 insertions(+)
diff --git a/arch/arm/boot/dts/am335x-bone-common.dtsi b/arch/arm/boot/dts/am335x-bone-common.dtsi
-index 0378046..543365d 100644
+index 46d5f27..69c6ce0 100644
--- a/arch/arm/boot/dts/am335x-bone-common.dtsi
+++ b/arch/arm/boot/dts/am335x-bone-common.dtsi
-@@ -127,6 +127,32 @@
+@@ -119,6 +119,32 @@
>;
};
@@ -49,7 +48,7 @@ index 0378046..543365d 100644
bone_geiger_cape_led_pins: pinmux_bone_geiger_cape_led_pins {
pinctrl-single,pins = <
0xe4 0x07 /* lcd_hsync.gpio2_23, OUTPUT | MODE7 */
-@@ -275,6 +301,10 @@
+@@ -267,6 +293,10 @@
compatible = "bone-generic-cape";
};
@@ -60,7 +59,7 @@ index 0378046..543365d 100644
/* overrides; no EEPROM (prototyping) */
// override@3 {
// compatible = "bone-capebus-slot-override";
-@@ -439,6 +469,24 @@
+@@ -426,6 +456,30 @@
slots = <&cape_eeprom_0 &cape_eeprom_1 &cape_eeprom_2 &cape_eeprom_3>;
};
@@ -76,8 +75,14 @@ index 0378046..543365d 100644
+ pinctrl-0 = <&nxp_hdmi_cape_pins>;
+
+ ti,hwmods = "lcdc";
-+ disp-pll = <135000000>;
-+ panel-type = "nxp-720x480@60";
++/*
++ 126000000 - 640x480
++ 135000000 - 720x480
++ 371000000 - 1280x720
++ 192000000 - 1920x1080
++*/
++ disp-pll = <371000000>;
++ panel-type = "nxp-1280x720@60";
+ };
+ };
+};
@@ -85,24 +90,6 @@ index 0378046..543365d 100644
&bone_dvi_cape {
board-name = "BeagleBone DVI-D CAPE";
-diff --git a/arch/arm/boot/dts/am335x-bonelt.dts b/arch/arm/boot/dts/am335x-bonelt.dts
-index 037594d..44814b4 100644
---- a/arch/arm/boot/dts/am335x-bonelt.dts
-+++ b/arch/arm/boot/dts/am335x-bonelt.dts
-@@ -37,3 +37,13 @@
- reg = <0x34>;
- };
- };
-+
-+&capebus {
-+ override@0 {
-+ board-name = "NXP HDMI on CapeBus";
-+ compatible = "bone-capebus-slot-override";
-+ slot = <1>;
-+ version = "00A1";
-+ manufacturer = "Beagleboardtoys";
-+ };
-+};
--
-1.7.7.6
+1.7.9.5
View
28 patches/hdmi/0005-am335x-bonelt-dts-Add-DT-node-to-probe-NXP-driver.patch
@@ -1,28 +0,0 @@
-From b8997f9a3e9b459ece8d754d53edf46ae1015061 Mon Sep 17 00:00:00 2001
-From: Joel A Fernandes <joelagnel@ti.com>
-Date: Tue, 20 Nov 2012 14:18:08 -0600
-Subject: [PATCH 5/7] am335x-bonelt/dts: Add DT node to probe NXP driver
-
-Signed-off-by: Joel A Fernandes <joelagnel@ti.com>
----
- arch/arm/boot/dts/am335x-bonelt.dts | 7 +++++++
- 1 files changed, 7 insertions(+), 0 deletions(-)
-
-diff --git a/arch/arm/boot/dts/am335x-bonelt.dts b/arch/arm/boot/dts/am335x-bonelt.dts
-index 87257ac..037594d 100644
---- a/arch/arm/boot/dts/am335x-bonelt.dts
-+++ b/arch/arm/boot/dts/am335x-bonelt.dts
-@@ -30,3 +30,10 @@
- ti,non-removable;
- status = "okay";
- };
-+
-+&i2c0 {
-+ tda998X@34 {
-+ compatible = "nxp,tda988x";
-+ reg = <0x34>;
-+ };
-+};
---
-1.7.7.6
-
View
8 ...efile-Add-new-Kconfig-for-old-drive.patch → ...efile-Add-new-Kconfig-for-old-drive.patch
@@ -1,14 +1,14 @@
-From 269de74f2ba0e45b18f7b4cc28452c967549c135 Mon Sep 17 00:00:00 2001
+From 15e33db742f876c9a67f78dd127b13c274f882e6 Mon Sep 17 00:00:00 2001
From: Joel A Fernandes <joelagnel@ti.com>
Date: Tue, 20 Nov 2012 14:14:27 -0600
-Subject: [PATCH 4/7] video/Kconfig+Makefile: Add new Kconfig for old driver
+Subject: [PATCH 5/8] video/Kconfig+Makefile: Add new Kconfig for old driver
Signed-off-by: Joel A Fernandes <joelagnel@ti.com>
---
drivers/video/Kconfig | 2 ++
drivers/video/Makefile | 2 ++
drivers/video/nxp/Kconfig | 7 +++++++
- 3 files changed, 11 insertions(+), 0 deletions(-)
+ 3 files changed, 11 insertions(+)
create mode 100644 drivers/video/nxp/Kconfig
diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig
@@ -48,5 +48,5 @@ index 0000000..1315c5f
+ Say Y here if you want to support NXP's TDA998X HDMI driver
+ To compile this driver as a module, choose M here.
--
-1.7.7.6
+1.7.9.5
View
38 patches/hdmi/0006-am335x-bonelt-dts-Add-DT-node-to-probe-NXP-driver.patch
@@ -0,0 +1,38 @@
+From b4b7e4ae83639f5d7919485d42d501e63b5f5bc3 Mon Sep 17 00:00:00 2001
+From: Joel A Fernandes <joelagnel@ti.com>
+Date: Tue, 20 Nov 2012 14:18:08 -0600
+Subject: [PATCH 6/8] am335x-bonelt/dts: Add DT node to probe NXP driver
+
+Signed-off-by: Joel A Fernandes <joelagnel@ti.com>
+---
+ arch/arm/boot/dts/am335x-bonelt.dts | 17 +++++++++++++++++
+ 1 file changed, 17 insertions(+)
+
+diff --git a/arch/arm/boot/dts/am335x-bonelt.dts b/arch/arm/boot/dts/am335x-bonelt.dts
+index 87257ac..44814b4 100644
+--- a/arch/arm/boot/dts/am335x-bonelt.dts
++++ b/arch/arm/boot/dts/am335x-bonelt.dts
+@@ -30,3 +30,20 @@
+ ti,non-removable;
+ status = "okay";
+ };
++
++&i2c0 {
++ tda998X@34 {
++ compatible = "nxp,tda988x";
++ reg = <0x34>;
++ };
++};
++
++&capebus {
++ override@0 {
++ board-name = "NXP HDMI on CapeBus";
++ compatible = "bone-capebus-slot-override";
++ slot = <1>;
++ version = "00A1";
++ manufacturer = "Beagleboardtoys";
++ };
++};
+--
+1.7.9.5
+
View
29 patches/hdmi/0007-tda-driver-enable-1280x720.patch
@@ -0,0 +1,29 @@
+From 5c2c648a72ebb0306fdcd1075380751b7aa3e09b Mon Sep 17 00:00:00 2001
+From: Joel A Fernandes <joelagnel@ti.com>
+Date: Mon, 3 Dec 2012 11:08:39 -0600
+Subject: [PATCH 7/8] tda driver: enable 1280x720
+
+Signed-off-by: Joel A Fernandes <joelagnel@ti.com>
+---
+ drivers/video/nxp/tda998x.c | 4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/drivers/video/nxp/tda998x.c b/drivers/video/nxp/tda998x.c
+index 2864221..de9ce6d 100755
+--- a/drivers/video/nxp/tda998x.c
++++ b/drivers/video/nxp/tda998x.c
+@@ -1034,9 +1034,9 @@ static int hdmi_tx_init(tda_instance *this)
+ /* this->tda.setio.video_out.format = TMDL_HDMITX_VFMT_31_1920x1080p_50Hz; */
+ /* this->tda.setio.video_out.format = TMDL_HDMITX_VFMT_PC_640x480p_60Hz; */
+ /* this->tda.setio.video_out.format = TMDL_HDMITX_VFMT_PC_640x480p_72Hz; */
+- // this->tda.setio.video_out.format = TMDL_HDMITX_VFMT_04_1280x720p_60Hz;
++ this->tda.setio.video_out.format = TMDL_HDMITX_VFMT_04_1280x720p_60Hz;
+ /* this->tda.setio.video_out.format = TMDL_HDMITX_VFMT_19_1280x720p_50Hz; */
+- this->tda.setio.video_out.format = TMDL_HDMITX_VFMT_02_720x480p_60Hz;
++ // this->tda.setio.video_out.format = TMDL_HDMITX_VFMT_02_720x480p_60Hz;
+
+ this->tda.setio.video_in.mode = TMDL_HDMITX_VINMODE_RGB444;
+ /* this->tda.setio.video_in.mode = TMDL_HDMITX_VINMODE_CCIR656; */
+--
+1.7.9.5
+
View
13 ...fine-TDA_CEC-we-don-t-use-CEC-featu.patch → patches/hdmi/0008-Makefile-Disable-CEC.patch
@@ -1,15 +1,12 @@
-From 728be73967a74447dccc54c29723640ea9c6879d Mon Sep 17 00:00:00 2001
+From ef496dda5c4c3f07f571ddcfaa3814a7c7047177 Mon Sep 17 00:00:00 2001
From: Joel A Fernandes <joelagnel@ti.com>
-Date: Wed, 21 Nov 2012 13:59:09 -0600
-Subject: [PATCH 7/7] nxp:Makefile Undefine TDA_CEC, we don't use CEC features
-
-CEC code is forced as a module, lets exclude it from build because its
-based on a prehistoric kernel and will break our build.
+Date: Mon, 3 Dec 2012 11:09:29 -0600
+Subject: [PATCH 8/8] Makefile: Disable CEC
Signed-off-by: Joel A Fernandes <joelagnel@ti.com>
---
drivers/video/nxp/Makefile | 2 +-
- 1 files changed, 1 insertions(+), 1 deletions(-)
+ 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/nxp/Makefile b/drivers/video/nxp/Makefile
index f583226..05dc816 100755
@@ -25,5 +22,5 @@ index f583226..05dc816 100755
# add this if INTERRUPT is wired, otherwise polling with timer is used
#EXTRA_CFLAGS += -DIRQ
--
-1.7.7.6
+1.7.9.5
Something went wrong with that request. Please try again.