Claude review: drm/panel: Add Novatek/Tianma NT37700F panel

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

Patch Review

**Status: Mostly good, minor observations.**

The driver is well-structured and uses modern APIs (`mipi_dsi_multi_context`, `drm_connector_helper_get_modes_fixed`). A few observations:

1. **Includes**: `<linux/of.h>` is included but not directly needed — the OF match table works through `<linux/module.h>` and the `MODULE_DEVICE_TABLE` macro. This is very minor and cosmetic.

2. **`prepare` error path missing regulator disable**: In `nt37700f_tianma_prepare()`:
   ```c
   ret = nt37700f_tianma_on(ctx);
   if (ret < 0) {
       dev_err(dev, "Failed to initialize panel: %d\n", ret);
       gpiod_set_value_cansleep(ctx->reset_gpio, 1);
       return ret;
   }
   ```
   When `nt37700f_tianma_on()` fails, the reset GPIO is asserted but the regulator is **not** disabled. This will leave the `power` supply enabled after a failed prepare. The error path should call `regulator_disable(ctx->supply)` as well to match what `unprepare` does.

3. **Backlight mode flag toggling**: The backlight `update_status` and `get_brightness` callbacks toggle `MIPI_DSI_MODE_LPM`:
   ```c
   dsi->mode_flags &= ~MIPI_DSI_MODE_LPM;
   ret = mipi_dsi_dcs_set_display_brightness_large(dsi, brightness);
   ...
   dsi->mode_flags |= MIPI_DSI_MODE_LPM;
   ```
   This is not thread-safe — if backlight updates could race with other DSI operations, the mode flags could be corrupted. This is a pattern seen in generated drivers, and in practice it may be fine for this single-panel setup, but it's worth noting.

4. **`MIPI_DSI_MODE_VIDEO_BURST` without `MIPI_DSI_MODE_VIDEO`**: The mode flags are:
   ```c
   dsi->mode_flags = MIPI_DSI_MODE_VIDEO_BURST |
                     MIPI_DSI_CLOCK_NON_CONTINUOUS | MIPI_DSI_MODE_LPM;
   ```
   `MODE_VIDEO_BURST` implies video mode, but `MIPI_DSI_MODE_VIDEO` itself is not set. This means the panel operates in **command mode** (since `MODE_VIDEO` is not set), and `MODE_VIDEO_BURST` has no effect. The MODULE_DESCRIPTION says "cmd mode" which is consistent, but having `MODE_VIDEO_BURST` set when not in video mode is misleading/unnecessary. Consider removing `MIPI_DSI_MODE_VIDEO_BURST` if this is indeed a command-mode panel.

---
Generated by Claude Code Patch Reviewer