Claude review: arm64: dts: qcom: sm6125-xiaomi-laurel-sprout: Enable MDSS and add panel

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

Patch Review

1. **`panel_vci_3p0` voltage mismatch** (lines 99-110 of the DTS): The regulator node is named `panel_vci_3p0` and `regulator-name` is `"panel_vci_3p0"`, implying 3.0V, but the actual voltage is set to **1800000 µV (1.8V)**:
   ```
   panel_vci_3p0: regulator-panel-vci {
       regulator-name = "panel_vci_3p0";
       regulator-min-microvolt = <1800000>;
       regulator-max-microvolt = <1800000>;
   ```
   This is likely a bug — either the voltage should be 3000000 (matching the name), or the name should be changed to `panel_vci_1p8`. VCI supplies for Samsung AMOLED panels are typically 3.0V. **This should be clarified/fixed.**

2. **No `vin-supply` on `panel_vci_3p0`**: The `panel_vdd_1p8` regulator has `vin-supply = <&vreg_l9a>`, but `panel_vci_3p0` has no `vin-supply`. This may be intentional if it's directly driven by the GPIO, but worth confirming.

3. **Extra blank line** (between `&mdss_dsi0_out` and `&mdss_dsi0_phy` blocks): There's a double blank line at the junction. Very minor style nit.

4. **`regulator-boot-on` on both regulators**: This keeps regulators enabled during boot, preventing screen flicker. Reasonable for a display panel.

5. **`reset-gpios` polarity**: The panel node uses `GPIO_ACTIVE_LOW`, and the driver's reset sequence asserts with `gpiod_set_value_cansleep(ctx->reset_gpio, 0)` → deassert, `1` → assert. With `GPIOD_OUT_HIGH` at probe (asserting reset = holding panel in reset), then the prepare sequence deasserts, pulses assert, deasserts. This is consistent — the GPIO descriptor layer handles the inversion. Fine.

**Summary**: The main actionable item is the `panel_vci_3p0` regulator voltage — the name says 3.0V but the value is 1.8V. This should be resolved before merging.

---
Generated by Claude Code Patch Reviewer