Commitd9e1757

authored

Merge pull request#1015 from dmazzella/patch-1

Arduino_H7_Video: handling errors at initialization time

2 parentsf9744ff +0255e38 commitd9e1757Copy full SHA for d9e1757

File tree

+28

-10

lines changed

+28

-10

lines changed

Lines changed: 4 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -95,7 +95,10 @@ int Arduino_H7_Video::begin() {`
`95`	`95`	`#endif`
`96`	`96`
`97`	`97`	`/* Video controller/bridge init*/`
`98`		`- _shield->init(_edidMode);`
	`98`	`+int err_code = _shield->init(_edidMode);`
	`99`	`+if (err_code <0) {`
	`100`	`+return3;/* Video controller fail init*/`
	`101`	`+ }`
`99`	`102`
`100`	`103`	`#if __has_include("lvgl.h")`
`101`	`104`	`/* Initiliaze LVGL library*/`

Lines changed: 12 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -31,13 +31,22 @@ int USBCVideoClass::init(int edidmode) {`
`31`	`31`	`}`
`32`	`32`
`33`	`33`	`//Checking HDMI plug event`
`34`		`-anx7625_wait_hpd_event(0);`
	`34`	`+ err_code =anx7625_wait_hpd_event(0);`
	`35`	`+if(err_code <0) {`
	`36`	`+return err_code;`
	`37`	`+ }`
`35`	`38`
`36`	`39`	`//Read EDID`
`37`		`-anx7625_dp_get_edid(0, &recognized_edid);`
	`40`	`+ err_code =anx7625_dp_get_edid(0, &recognized_edid);`
	`41`	`+if(err_code <0) {`
	`42`	`+return err_code;`
	`43`	`+ }`
`38`	`44`
`39`	`45`	`//DSI Configuration`
`40`		`-anx7625_dp_start(0, &recognized_edid, (enum edid_modes) edidmode);`
	`46`	`+ err_code =anx7625_dp_start(0, &recognized_edid, (enum edid_modes) edidmode);`
	`47`	`+if(err_code <0) {`
	`48`	`+return err_code;`
	`49`	`+ }`
`41`	`50`
`42`	`51`	`return0;`
`43`	`52`	`}`

Lines changed: 11 additions & 5 deletions

Original file line number	Diff line number	Diff line change
`@@ -520,15 +520,21 @@ int anx7625_init(uint8_t bus) {`
`520`	`520`	`return0;`
`521`	`521`	`}`
`522`	`522`
`523`		`-voidanx7625_wait_hpd_event(uint8_t bus) {`
	`523`	`+intanx7625_wait_hpd_event(uint8_t bus) {`
`524`	`524`	`ANXINFO("Waiting for HDMI hot plug event...\n");`
`525`		`-`
`526`		`-while (1) {`
	`525`	`+`
	`526`	`+int retry_hpd_change =10000;`
	`527`	`+while (--retry_hpd_change) {`
`527`	`528`	`mdelay(10);`
`528`	`529`	`int detected =anx7625_hpd_change_detect(bus);`
`529`		`-if (detected ==1)`
`530`		`-break;`
	`530`	`+if (detected <0)`
	`531`	`+return -1;`
	`532`	`+if (detected >0)`
	`533`	`+return0;`
`531`	`534`	`}`
	`535`	`+`
	`536`	`+ANXERROR("Timed out to detect HPD change on bus %d.\n", bus);`
	`537`	`+return -1;`
`532`	`538`	`}`
`533`	`539`
`534`	`540`	`intanx7625_get_cc_status(uint8_t bus,uint8_t *cc_status) {`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@ extern "C" {`
`20`	`20`	`intanx7625_dp_start(uint8_tbus,conststructedid*edid,enumedid_modesmode=EDID_MODE_AUTO);`
`21`	`21`	`intanx7625_dp_get_edid(uint8_tbus,structedid*out);`
`22`	`22`	`intanx7625_init(uint8_tbus);`
`23`		`-voidanx7625_wait_hpd_event(uint8_tbus);`
	`23`	`+intanx7625_wait_hpd_event(uint8_tbus);`
`24`	`24`	`intanx7625_get_cc_status(uint8_tbus,uint8_t*cc_status);`
`25`	`25`	`intanx7625_read_system_status(uint8_tbus,uint8_t*sys_status);`
`26`	`26`	`boolanx7625_is_power_provider(uint8_tbus);`

Comments

(0)