Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider: - [Medium] The compatible string `qcom,qcm2290-dsi-ctrl` (and the newly added `qcom,shikra-dsi-ctrl` fallback) is missing from the conditional `if-then` constraint blocks in `dsi-controller-main.yaml`, preventing strict validation. --
commit 605ca9e1f594a8d64bb53c70c6ef6319fc95eeed Author: Nabige Aala <[email protected]> dt-bindings: display: msm: qcm2290: Add Shikra MDSS Shikra reuses the same MDSS/DPU 6.5 hardware as QCM2290. Extend the existing qcm2290 bindings to cover Shikra by adding fallback compatible chains for MDSS, DPU and DSI controller nodes rather than introducing a separate binding file. > diff --git > a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml > b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml > index a24fcb9144181..08a6668d3ca78 100644 > --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml > +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml > @@ -46,6 +46,10 @@ properties: > - qcom,sm8650-dsi-ctrl > - qcom,sm8750-dsi-ctrl > - const: qcom,mdss-dsi-ctrl > + - items: > + - const: qcom,shikra-dsi-ctrl > + - const: qcom,qcm2290-dsi-ctrl > + - const: qcom,mdss-dsi-ctrl [Severity: Medium] Should qcom,shikra-dsi-ctrl and qcom,qcm2290-dsi-ctrl be added to the conditional constraint blocks lower down in this file? Because they are omitted from the conditional blocks governing 6-clock setups, the schema falls back to permissive defaults (3 to 12 clocks). This allows incomplete or invalid device tree nodes for Shikra to silently pass validation. While the missing qcom,qcm2290-dsi-ctrl constraint was a pre-existing issue, adding the new qcom,shikra-dsi-ctrl compatible without ensuring it has strict validation introduces this permissive validation bug for the new platform. The block enforcing these strict limits starts like this: - if: properties: compatible: contains: enum: - qcom,msm8998-dsi-ctrl - qcom,sa8775p-dsi-ctrl - qcom,sar2130p-dsi-ctrl > - items: > - enum: > - qcom,qcs8300-dsi-ctrl -- Sashiko AI review ยท https://sashiko.dev/#/patchset/[email protected]?part=1
