Claude review: dt-bindings: display: msm: qcm2290: Add Shikra MDSS

[Claude Code Review Bot <claude-review@example.com>]

Patch Review

This is the substantive patch of the series. The changes are well-structured.

**select block (qcom,qcm2290-mdss.yaml):** Adding the `select` block is correct and necessary. Since `mdss-common.yaml` uses `select: false`, and none of the other MDSS binding files in the tree use an explicit `select`, the default filename-based schema matching would fail for DT nodes whose first compatible is `qcom,shikra-mdss`. The `contains: enum: [qcom,qcm2290-mdss]` approach ensures this schema is applied whenever `qcom,qcm2290-mdss` appears anywhere in the compatible list, covering both the standalone and fallback cases.

**Title fix:** The title change from `"Qualcomm QCM220 Display MDSS"` to `"Qualcomm QCM2290 and Shikra Display MDSS"` is a nice drive-by fix of a pre-existing typo ("QCM220" → "QCM2290").

**DPU binding (qcom,qcm2290-dpu.yaml):** Minor nit — the diff removes the blank line that separated `compatible:` from `reg:`:

```diff
   compatible:
-    const: qcom,qcm2290-dpu
-
+    oneOf:
+      - const: qcom,qcm2290-dpu
+      - items:
+          - const: qcom,shikra-dpu
+          - const: qcom,qcm2290-dpu
   reg:
```

The blank line before `reg:` was a visual separator between properties. Its removal won't break validation, but it's inconsistent with the style in the rest of the file and other binding files. Worth keeping for readability.

**DSI compatible chain (qcom,qcm2290-mdss.yaml):** The Shikra DSI compatible is defined as a 3-item chain: `qcom,shikra-dsi-ctrl`, `qcom,qcm2290-dsi-ctrl`, `qcom,mdss-dsi-ctrl`. This is correct — the kernel DSI driver will match on the `qcom,qcm2290-dsi-ctrl` or `qcom,mdss-dsi-ctrl` fallbacks.

**dsi-controller-main.yaml:** Adding `qcom,shikra-dsi-ctrl` in alphabetical order between `qcom,sdm845-dsi-ctrl` and `qcom,sm6115-dsi-ctrl` is correct.

No functional issues found.

---
Generated by Claude Code Patch Reviewer