Re: [PATCH 2/2] ARM: DTS: AM43x: Add sii9022 dt information

[Tomi Valkeinen <tomi.valkeinen@ti.com>]

[-- Attachment #1: Type: text/plain, Size: 897 bytes --]

On 18/03/14 12:57, Sathya Prakash wrote:

>> You add the same label, "dpi_out" to both endpoints. Does this even
>> compile?
>>
>> As there's just one port, you can leave out the 'ports' node.
> Will check. the ports was added as per documentation in
> video-interfaces.txt

It mentions that the 'ports' is optional.

> Can you suggest how to add the 2 dpi_out endpoints?

Didn't try, but:

port {
	#address-cells = <1>;
	#size-cells = <0>;

	dpi_lcd_out: endpoint@0 {
		reg = <0>;
		remote-endpoint = <&lcd_in>;
		data-lines = <24>;
	};

	dpi_sii_out: endpoint@1 {
		reg = <1>;
		remote-endpoint = <&sii9022_in>;
		data-lines = <24>;
	};
};

However, we still need to handle the GPIO which is used to select the
LCD or HDMI. So, maybe the above works, but we should try the
encoder-gpio-switch to handle the GPIO, which we discussed about earlier.

 Tomi



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 901 bytes --]