On 13/06/2025 11:33, Vincent Knecht via B4 Relay wrote: > From: Vincent Knecht <vincent.kne...@mailoo.org> > > Add bindings for qcom,msm8939-camss in order to support the camera > subsystem for MSM8939. > > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlow...@linaro.org>
Drop, you did here some unusual changes. > Signed-off-by: Vincent Knecht <vincent.kne...@mailoo.org> > --- > .../bindings/media/qcom,msm8939-camss.yaml | 254 > +++++++++++++++++++++ > 1 file changed, 254 insertions(+) > > diff --git a/Documentation/devicetree/bindings/media/qcom,msm8939-camss.yaml > b/Documentation/devicetree/bindings/media/qcom,msm8939-camss.yaml > new file mode 100644 > index > 0000000000000000000000000000000000000000..9fbb4b204ac8728b822864ad8336aa9d826d6b5b > --- /dev/null > +++ b/Documentation/devicetree/bindings/media/qcom,msm8939-camss.yaml > @@ -0,0 +1,254 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/media/qcom,msm8939-camss.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Qualcomm MSM8939 Camera Subsystem (CAMSS) > + > +maintainers: > + - Vincent Knecht <vincent.kne...@mailoo.org> > + > +description: > + The CAMSS IP is a CSI decoder and ISP present on Qualcomm platforms > + > +properties: > + compatible: > + const: qcom,msm8939-camss > + > + reg: > + maxItems: 11 > + > + reg-names: > + items: > + - const: csi_clk_mux No, I already provided arguments in two lengthy discussions - this is not sorted by name. Keep the same order as in previous device, so msm8916 for example. Or any other, but listen to some requests to sort it by some arbitrary rule which was never communicated by DT maintainers. > + - const: csid0 > + - const: csid1 > + - const: csid2 > + - const: csiphy0 > + - const: csiphy0_clk_mux > + - const: csiphy1 > + - const: csiphy1_clk_mux > + - const: ispif > + - const: vfe0 > + - const: vfe0_vbif > + > + clocks: > + maxItems: 24 > + > + clock-names: > + items: > + - const: ahb > + - const: csi0 > + - const: csi0_ahb > + - const: csi0_phy > + - const: csi0_pix > + - const: csi0_rdi > + - const: csi1 > + - const: csi1_ahb > + - const: csi1_phy > + - const: csi1_pix > + - const: csi1_rdi > + - const: csi2 > + - const: csi2_ahb > + - const: csi2_phy > + - const: csi2_pix > + - const: csi2_rdi > + - const: csi_vfe0 > + - const: csiphy0_timer > + - const: csiphy1_timer > + - const: ispif_ahb > + - const: top_ahb > + - const: vfe0 > + - const: vfe_ahb > + - const: vfe_axi > + Also messed up order. > + interrupts: > + maxItems: 7 > + > + interrupt-names: > + items: > + - const: csid0 > + - const: csid1 > + - const: csid2 > + - const: csiphy0 > + - const: csiphy1 > + - const: ispif > + - const: vfe0 As well. Best regards, Krzysztof
