[PATCH v3 03/10] dt-bindings: display: rockchip,dw-mipi-dsi: Drop address/size cells

[Rob Herring]

On Sun, Jun 29, 2025 at 02:34:44PM +0200, Diederik de Haas wrote:
> When the dw-mipi-dsi binding was initially added in commit
> a20d86e7f964 ("Documentation: dt-bindings: Add bindings for rk3288 DW MIPI DSI driver")
> the #address-cells and #size-cells were added as required properties.
> 
> When the binding was converted to yaml format in commit
> 0dac2102cf6b ("dt-bindings: display: rockchip: convert dw_mipi_dsi_rockchip.txt to yaml")
> those properties were demoted to optional and removed from the binding
> example.
> 
> As for the compatibles:
> - rockchip,px30-mipi-dsi      removed in this patch set
> - rockchip,rk3128-mipi-dsi    never used
> - rockchip,rk3288-mipi-dsi    added (invalid); later removed [1]
> - rockchip,rk3399-mipi-dsi    removed in this patch set
> - rockchip,rk3568-mipi-dsi    never used
> - rockchip,rv1126-mipi-dsi    proposed (invalid); never accepted [2]
> 
> [1] 282e2e078ba5 ("ARM: dts: rockchip: Remove #address/#size-cells from rk3288 mipi_dsi")
> [2] https://lore.kernel.org/all/20230731110012.2913742-12-jagan@edgeble.ai/
> 
> The #address-cells and #size-cells are useful (and required) in the
> ports node and for panel(s), but those properties are declared in their
> schemas already. Now that there are no remaining users, remove these
> properties from the Rockchip specific extensions of the Synopsys
> DesignWare MIPI DSI host controller.

The change is fine, but your reasoning is flawed. These properties are 
used if you define DSI devices on the "DSI bus" where the address of the 
child devices are the DSI virtual channel. Often though that's just 0 
and not really used, so you don't need it. 

The change is fine because these properties are defined in 
dsi-controller.yaml, so specifying them here is redundant.

> 
> Signed-off-by: Diederik de Haas <didi.debian at cknow.org>
> ---
>  .../bindings/display/rockchip/rockchip,dw-mipi-dsi.yaml     | 6 ------
>  1 file changed, 6 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/display/rockchip/rockchip,dw-mipi-dsi.yaml b/Documentation/devicetree/bindings/display/rockchip/rockchip,dw-mipi-dsi.yaml
> index ccd71c5324af..0881e82deb11 100644
> --- a/Documentation/devicetree/bindings/display/rockchip/rockchip,dw-mipi-dsi.yaml
> +++ b/Documentation/devicetree/bindings/display/rockchip/rockchip,dw-mipi-dsi.yaml
> @@ -58,12 +58,6 @@ properties:
>    power-domains:
>      maxItems: 1
>  
> -  "#address-cells":
> -    const: 1
> -
> -  "#size-cells":
> -    const: 0
> -
>  required:
>    - compatible
>    - clocks
> -- 
> 2.50.0
>