Message ID | 20220908182433.466908-1-stephan.gerhold@kernkonzept.com |
---|---|
Headers | show |
Series | remoteproc: qcom_q6v5_mss: Add MSM8909 and MSM8953 | expand |
On 08/09/2022 20:24, Stephan Gerhold wrote: > qcom,q6v5.txt covers multiple SoCs with quite different binding > requirements. Converting this into one DT schema would require > several if statements, making the DT schema overall harder to > read and understand. > > To avoid this, follow the example of SC7180/SC7280 and split > "qcom,msm8916-mss-pil" (and the equivalent deprecated "qcom,q6v5-pil" > compatible) into a separate DT schema. The schema is somewhat based > on the one for SC7180/SC7280 but adjusted for the old platforms. > > Compared to the old plain text bindings, add missing documentation for > the "bam-dmux" subnode and recommend one particular approach to specify > the MBA/MPSS "memory-region" (the other one is marked as deprecated). > > Signed-off-by: Stephan Gerhold <stephan.gerhold@kernkonzept.com> > --- > "dt-bindings: remoteproc: qcom,smd-edge: Add APR/FastRPC" [1] should be > applied additionally to avoid a dtbs_check warning related to fastrpc in > msm8916.dtsi. > > Changes in v4: Add "unevaluatedProperties: false" to smd-edge > Changes in v3: None > Changes in v2: > - Add blank lines between top-level properties > - Drop "deprecated" in "oneOf" list, it is not clear if this is valid > and it should be redundant since the properties itself are already > marked as "deprecated" > > [1]: https://lore.kernel.org/linux-arm-msm/20220908181432.458900-1-stephan.gerhold@kernkonzept.com/ Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On Thu, 8 Sep 2022 20:24:26 +0200, Stephan Gerhold wrote: > Convert parts of the qcom,q6v5.txt binding documentation to a DT schema > (together with some DT fixes) in preparation of adding new compatibles > for MSM8909 and MSM8953. The schema can be easily shared between > MSM8916, MSM8974, MSM8909 and MSM8953 because they have almost the same > requirements. The other SoCs are quite different and are likely more > clearly described in separate DT schema(s) to avoid a lot of complicated > if statements. > > [...] Applied, thanks! [1/7] arm64: dts: qcom: msm8916: Drop MSS fallback compatible commit: ff02ac621634e82c0c34d02a79d402ae700cdfd0 Best regards,