Claude review: Add support to 2 panels in bonded-DSI mode

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

Overall Series Review

Subject: Add support to 2 panels in bonded-DSI mode
Author: Jun Nie <jun.nie@linaro.org>
Patches: 10
Reviewed: 2026-05-25T20:00:01.551184

---

This v4 series adds support for driving two independent physical DSI panels in bonded-DSI mode from a single CRTC, targeting VR headset use cases (left/right eye panels). It touches the MSM DSI host driver to support `slice_per_pkt > 1` for DSC and dual-panel frame width halving, adds a `dual_panel` flag to the core MIPI DSI struct, introduces a new DT binding, and adds a panel driver for the Synaptics R63455. 

The overall architecture — having the panel driver register the second DSI device and set `dual_panel` so the host halves the frame — is reasonable, though several issues need attention before this can be merged:

1. **Build breakage**: The panel driver uses `backlight_properties.pulse_offset_rows`, which does not exist in the kernel backlight struct. This will fail to compile.
2. **`dsc_slice_per_pkt` placement**: Adding a non-standard field to `struct drm_dsc_config` (a struct that maps to the DSC standard parameter set) is debatable and needs DRM maintainer buy-in.
3. **Missing division-by-zero guard**: `dsc_slice_per_pkt` is used as a divisor without validation.
4. **DT binding issues**: YAML schema errors and questionable use of vendor-prefixed supply names instead of generic ones.
5. **Panel driver quality**: Uses deprecated API (`of_device_get_match_data`), includes an unused header, and has error-handling gaps.

---
Generated by Claude Code Patch Reviewer