Re: [PATCH v2 2/2] dt-bindings: drm/bridge: Document Cadence DSI bridge bindings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 06/06/17 15:48, Boris Brezillon wrote:

> Okay. Thanks for the clarification. Can you confirm that this version
> is correct?
> 
>  	dsi@xxx {
>  		#address-cells = <1>;
>  		#size-cells = <0>;
>  
>  		ports {
>  			#address-cells = <1>;
>  			#size-cells = <0>;
> 			dpi_in: port@0 {
>  				reg = <0>;
>  				#address-cells = <1>;
>  				#size-cells = <0>;
>  
>  				endpoint@0 {
>  					remote-endpoint = <&dpi_out>;
>  				};
>  			};
>  
>  			dsi_out: port@1 {
>  				reg = <1>;
> 				#address-cells = <1>;
>  				#size-cells = <0>;
>  
>  				dsi_out_vc0: endpoint@0 {
> 					reg = <0>;
>  					remote-endpoint = <&dsi_panel0_in>;
> 				};
> 
>  				dsi_out_vc1: endpoint@1 {
> 					reg = <1>;
>  					remote-endpoint = <&dsi_panel1_in>;
>  				};
>  			};
>  		};
>  
>  		panel@0 {
>  			compatible = "...";
>  			reg = <0>;
>  			#address-cells = <1>;
>  			#size-cells = <0>;
>  
>  			port@0 {
>  				#address-cells = <1>;
>  		                #size-cells = <0>;
>  				reg = <0>;
>  
>  				dsi_panel0_in: endpoint@0 {
> 					reg = <0>;
>  					remote-endpoint = <&dsi_out_vc0>;
>  				};
>  			};
>  		};
>  
>  		panel@1 {
>  			compatible = "...";
>  			reg = <1>;
>  			#address-cells = <1>;
>  			#size-cells = <0>;
>  
>  			port@0 {
>  				#address-cells = <1>;
>  		                #size-cells = <0>;
>  				reg = <0>;
>  
>  				dsi_panel1_in: endpoint@0 {
> 					reg = <0>;
>  					remote-endpoint = <&dsi_out_vc1>;
>  				};
>  			};
>  		};
>  	};
> 

Looks correct to me. I think it can be a bit shorter though:

- You don't need #address-cells and #size-cells for all. I think those
are inherited from the parent.
- If there's just one port and one endpoint, you can leave the 'reg'
out, as it's considered to be 0 by default.

So for the panel, you can have just:

port {
	dsi_panel1_in: endpoint {
		remote-endpoint = <&dsi_out_vc1>;
	};
};

Attachment: signature.asc
Description: OpenPGP digital signature


[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux