[PATCH 1/2] drm/fsl-dcu: update the panel dt binding document

Stefan Agner stefan at agner.ch
Wed Jun 29 05:40:21 UTC 2016


On 2016-06-28 19:58, Meng Yi wrote:
>> >>  Optional properties:
>> >>  - fsl,tcon:		The phandle to the timing controller node.
>> >> @@ -24,6 +24,11 @@ dcu: dcu at 2ce0000 {
>> >>  	clocks = <&platform_clk 0>, <&platform_clk 0>;
>> >>  	clock-names = "dcu", "pix";
>> >>  	big-endian;
>> >> -	fsl,panel = <&panel>;
>> >>  	fsl,tcon = <&tcon>;
>> >> +
>> >> +	port: port at 0 {
>> >
>> > Drop the unit address. With that,
>>
>> I was going to say add a corresponding reg property and specify address-cells.
>>
> 
> For now, we only have one 'remote-endpoint', you mean maybe we should
> use 'ports' that prepare for multiple connections?
> 

I ment to add the address because most drivers do. But then, most
drivers support multiple ports.

>> So is it ok to also create a port without address?
>>
> 
> It works fine, I think it is ok.

I guess it is ok too since Rob suggested it. However, consequently I
also would drop the unit address of the port node inside the panel node.

--
Stefan


More information about the dri-devel mailing list