Claude review: ARM: dts: microchip: sama7d65: add LCD controller

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

Patch Review

**Critical issue — missing fallback compatible will break DRM probe:**

The DRM display controller driver at `drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c:766` does:
```c
match = of_match_node(atmel_hlcdc_of_match, dev->dev->parent->of_node);
```
The `atmel_hlcdc_of_match` table (line 569–598) contains `"microchip,sam9x75-xlcdc"` but **not** `"microchip,sama7d65-xlcdc"`. This means the DRM probe will fail with `"invalid compatible string"` and the display won't work.

The LVDS node in patch 3 correctly uses a fallback compatible (`"microchip,sama7d65-lvds", "microchip,sam9x75-lvds"`), but this LCD node does not. The fix is either:
- Add a fallback compatible: `compatible = "microchip,sama7d65-xlcdc", "microchip,sam9x75-xlcdc";` (and update the MFD binding to match this pattern), or
- Include a separate patch adding `sama7d65-xlcdc` to the DRM driver's match table with appropriate data

**Style issue — missing space before `{`:**
```
+		xlcdc: lcd-controller@e1400000{
```
Should be:
```
		xlcdc: lcd-controller@e1400000 {
```

The node content itself looks correct: reg, interrupts, clocks, clock-names, display-controller subnode, and pwm subnode all follow the existing pattern from `sam9x7.dtsi`.

---
Generated by Claude Code Patch Reviewer