diff mbox

[U-Boot,6/6] ARM: zynq: Enable EDID for zybo

Message ID 25e615eb98e1d86b54ddc30f36192e9c348a9679.1455359979.git.monstr@monstr.eu
State Accepted
Delegated to: Michal Simek
Headers show

Commit Message

Michal Simek Feb. 13, 2016, 10:39 a.m. UTC
Zybo contains on board HDMI that's why enable EDID.
Doing it via config because zynq i2c driver hasn't been moved to DM yet
and enabling via Kconfig requires DM_I2C.
This will be moved that driver is moved to DM.

Signed-off-by: Michal Simek <monstr@monstr.eu>
---

Currently the biggest problem is EEPROM support with DM together.

Log for zybo:
Zynq> i2c dev 1
Setting bus to 1
Zynq> i2c probe
Valid chip addresses: 50
Zynq> i2c edid 50
EDID version: 1.3
Product ID code: 05cd
Manufacturer: SAM
Serial number: 30303033
Manufactured in week: 42 year: 2010
Video input definition: digital signal, voltage level 0
Monitor is non-RGB
Maximum visible display size: 16 cm x 9 cm
Power management features: active off, no suspend, no standby
Estabilished timings:
	640x480		60 Hz (VGA)
	800x600		56 Hz (VESA)
	800x600		60 Hz (VESA)
	1024x768	60 Hz (VESA)
Standard timings:
	1600x1200 	60 Hz
	1280x1024 	60 Hz
	1280x960 	60 Hz
	1280x800 	60 Hz
	1440x900 	60 Hz
	1680x1050 	60 Hz
	1920x1080 	60 Hz (detailed)
	1280x720 	60 Hz (detailed)
Monitor range limits, horizontal sync: 30-81 kHz, vertical refresh:
50-60 Hz, max pixel clock: 170 MHz
Monitor name: SyncMaster

---
 include/configs/zynq_zybo.h | 2 ++
 1 file changed, 2 insertions(+)

Comments

Michal Simek Feb. 22, 2016, 3:16 p.m. UTC | #1
On 13.2.2016 11:39, Michal Simek wrote:
> Zybo contains on board HDMI that's why enable EDID.
> Doing it via config because zynq i2c driver hasn't been moved to DM yet
> and enabling via Kconfig requires DM_I2C.
> This will be moved that driver is moved to DM.
> 
> Signed-off-by: Michal Simek <monstr@monstr.eu>
> ---
> 
> Currently the biggest problem is EEPROM support with DM together.
> 
> Log for zybo:
> Zynq> i2c dev 1
> Setting bus to 1
> Zynq> i2c probe
> Valid chip addresses: 50
> Zynq> i2c edid 50
> EDID version: 1.3
> Product ID code: 05cd
> Manufacturer: SAM
> Serial number: 30303033
> Manufactured in week: 42 year: 2010
> Video input definition: digital signal, voltage level 0
> Monitor is non-RGB
> Maximum visible display size: 16 cm x 9 cm
> Power management features: active off, no suspend, no standby
> Estabilished timings:
> 	640x480		60 Hz (VGA)
> 	800x600		56 Hz (VESA)
> 	800x600		60 Hz (VESA)
> 	1024x768	60 Hz (VESA)
> Standard timings:
> 	1600x1200 	60 Hz
> 	1280x1024 	60 Hz
> 	1280x960 	60 Hz
> 	1280x800 	60 Hz
> 	1440x900 	60 Hz
> 	1680x1050 	60 Hz
> 	1920x1080 	60 Hz (detailed)
> 	1280x720 	60 Hz (detailed)
> Monitor range limits, horizontal sync: 30-81 kHz, vertical refresh:
> 50-60 Hz, max pixel clock: 170 MHz
> Monitor name: SyncMaster
> 
> ---
>  include/configs/zynq_zybo.h | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/include/configs/zynq_zybo.h b/include/configs/zynq_zybo.h
> index 644d462c01c9..3db3c13e8a3e 100644
> --- a/include/configs/zynq_zybo.h
> +++ b/include/configs/zynq_zybo.h
> @@ -24,6 +24,8 @@
>  #define CONFIG_CMD_EEPROM
>  #define CONFIG_ZYNQ_GEM_EEPROM_ADDR	0x50
>  #define CONFIG_ZYNQ_GEM_I2C_MAC_OFFSET	0xFA
> +#define CONFIG_DISPLAY
> +#define CONFIG_I2C_EDID
>  
>  /* Define ZYBO PS Clock Frequency to 50MHz */
>  #define CONFIG_ZYNQ_PS_CLK_FREQ	50000000UL
> 

Applied to zynq repo.

Thanks,
Michal
diff mbox

Patch

diff --git a/include/configs/zynq_zybo.h b/include/configs/zynq_zybo.h
index 644d462c01c9..3db3c13e8a3e 100644
--- a/include/configs/zynq_zybo.h
+++ b/include/configs/zynq_zybo.h
@@ -24,6 +24,8 @@ 
 #define CONFIG_CMD_EEPROM
 #define CONFIG_ZYNQ_GEM_EEPROM_ADDR	0x50
 #define CONFIG_ZYNQ_GEM_I2C_MAC_OFFSET	0xFA
+#define CONFIG_DISPLAY
+#define CONFIG_I2C_EDID
 
 /* Define ZYBO PS Clock Frequency to 50MHz */
 #define CONFIG_ZYNQ_PS_CLK_FREQ	50000000UL