On 12/06/2025 17:18, Kaustabh Chakraborty wrote: > Add compatible string for Exynos7870 DSIM bridge controller. The > devicetree node requires four clock sources, named: > - bus_clk > - phyclk_mipidphy0_bitclkdiv8 > - phyclk_mipidphy0_rxclkesc0 > - sclk_mipi > > Signed-off-by: Kaustabh Chakraborty <[email protected]> > --- > .../bindings/display/bridge/samsung,mipi-dsim.yaml | 26 > ++++++++++++++++++++++ > 1 file changed, 26 insertions(+) > > diff --git > a/Documentation/devicetree/bindings/display/bridge/samsung,mipi-dsim.yaml > b/Documentation/devicetree/bindings/display/bridge/samsung,mipi-dsim.yaml > index > 1acad99f396527192b6853f0096cfb8ae5669e6b..887f3ba1edd24a177a766b1b523d0c197ff1123a > 100644 > --- a/Documentation/devicetree/bindings/display/bridge/samsung,mipi-dsim.yaml > +++ b/Documentation/devicetree/bindings/display/bridge/samsung,mipi-dsim.yaml > @@ -24,6 +24,7 @@ properties: > - samsung,exynos5410-mipi-dsi > - samsung,exynos5422-mipi-dsi > - samsung,exynos5433-mipi-dsi > + - samsung,exynos7870-mipi-dsi > - fsl,imx8mm-mipi-dsim > - fsl,imx8mp-mipi-dsim > - items: > @@ -144,6 +145,31 @@ required: > > allOf: > - $ref: ../dsi-controller.yaml# > + - if: > + properties: > + compatible: > + contains: > + const: samsung,exynos7870-mipi-dsi > + > + then: > + properties: > + clocks: > + minItems: 4
maxItems: 4 > + > + clock-names: > + items: > + - const: bus_clk > + - const: phyclk_mipidphy0_bitclkdiv8 > + - const: phyclk_mipidphy0_rxclkesc0 > + - const: sclk_mipi Does any existing driver code actually depends on the names? If not, we switched in Samsung in general to names matching the input or the function, not the name of provider. bus, bit (or bitdiv?), rx or esc0, sclk Best regards, Krzysztof
