Claude review: drm/panel: Add driver for Novatek NT35532

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

Patch Review

**Init sequence uses generic writes for standard DCS commands.** At the end of `nt35532_on()`:

```c
mipi_dsi_generic_write_seq_multi(&dsi_ctx, 0x11, 0x00);
mipi_dsi_msleep(&dsi_ctx, 120);
mipi_dsi_generic_write_seq_multi(&dsi_ctx, 0x29, 0x00);
mipi_dsi_msleep(&dsi_ctx, 50);
```

`0x11` is `MIPI_DCS_EXIT_SLEEP_MODE` and `0x29` is `MIPI_DCS_SET_DISPLAY_ON`. These should use the proper DCS helpers, consistent with how `nt35532_off()` already uses them:

```c
mipi_dsi_dcs_exit_sleep_mode_multi(&dsi_ctx);
mipi_dsi_msleep(&dsi_ctx, 120);
mipi_dsi_dcs_set_display_on_multi(&dsi_ctx);
mipi_dsi_msleep(&dsi_ctx, 50);
```

This is a common artifact of auto-generated drivers, but reviewers will ask for it to be fixed.

**`#include <drm/drm_probe_helper.h>`** is included for `drm_connector_helper_get_modes_fixed()`. This is correct, but the corresponding `select DRM_KMS_HELPER` in Kconfig may be unnecessary — check whether `DRM_MIPI_DSI` already selects it transitively. Other recent panel drivers don't always add this select.

**Structure and APIs are good:**
- Uses `devm_drm_panel_alloc()` (the modern allocation API) 
- Uses `devm_regulator_bulk_get_const()` with a static supply array
- Uses `mipi_dsi_multi_context` for the init sequence
- Uses `drm_connector_helper_get_modes_fixed()` for mode reporting
- `prepare_prev_first = true` is set correctly

**Reset sequence looks correct** given `GPIO_ACTIVE_LOW` in the DTS:

```c
gpiod_set_value_cansleep(ctx->reset_gpio, 0);  // de-assert
usleep_range(10000, 11000);
gpiod_set_value_cansleep(ctx->reset_gpio, 1);  // assert (reset pulse)
usleep_range(5000, 6000);
gpiod_set_value_cansleep(ctx->reset_gpio, 0);  // de-assert
usleep_range(10000, 11000);
```

**Minor:** The `MODULE_AUTHOR()` macro is missing. Not strictly required but conventional.

---
Generated by Claude Code Patch Reviewer