Claude review: drm/bridge: tc358762: Improve LCDCTRL defines

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

Patch Review

This is the most important cleanup patch. The old LCDCTRL defines were significantly wrong:

- Old `LCDCTRL_VTGEN BIT(4)` → New `LCDCTRL_VTGEN BIT(1)` (the old "VTGEN" was actually part of the pixel format field)
- Old `LCDCTRL_UNK6 BIT(6)` → part of `LCDCTRL_PXLFORM GENMASK(6,4)` 
- Old `LCDCTRL_RGB888 BIT(8)` → New `LCDCTRL_DPI_EN BIT(8)`
- Old `LCDCTRL_VSDELAY(v) (v<<20)` → New `LCDCTRL_DCLK_POL BIT(20)`

The commit message clearly explains the mapping. I verified: old value `LCDCTRL_VSDELAY(1)|LCDCTRL_RGB888|LCDCTRL_UNK6|LCDCTRL_VTGEN` = `BIT(20)|BIT(8)|BIT(6)|BIT(4)` = 0x100150, new value `FIELD_PREP(GENMASK(6,4),5)|LCDCTRL_DPI_EN|LCDCTRL_DCLK_POL` = `0x50|0x100|0x100000` = 0x100150. Matches perfectly.

Also correctly removes the "based on guesswork" comment, suggesting Tomi has verified these against actual documentation. The polarity defines are renamed from `HSPOL`/`DEPOL`/`VSPOL` to `HSYNC_POL`/`DE_POL`/`VSYNC_POL` for clarity.

No issues. **Reviewed-by worthy.**

---
Generated by Claude Code Patch Reviewer