[PATCH] drm/nouveau/kms/nvd9-: Remove unused header in crc.c

[PATCH] drm/nouveau/kms/nvd9-: Remove unused header in crc.c

[Lyude Paul]

No functional changes.

Signed-off-by: Lyude Paul <lyude@redhat.com>
---
 drivers/gpu/drm/nouveau/dispnv50/crc.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/gpu/drm/nouveau/dispnv50/crc.c b/drivers/gpu/drm/nouveau/dispnv50/crc.c
index deb6af40ef328..42a6ebbc2e3c3 100644
--- a/drivers/gpu/drm/nouveau/dispnv50/crc.c
+++ b/drivers/gpu/drm/nouveau/dispnv50/crc.c
@@ -16,7 +16,6 @@
 #include "nouveau_drv.h"
 #include "core.h"
 #include "head.h"
-#include "wndw.h"
 #include "handles.h"
 #include "crc.h"
 

base-commit: 29d6da40d0b8bf3bbc3dcd1d2198434a0e1f71b0
-- 
2.54.0

Claude review: drm/nouveau/kms/nvd9-: Remove unused header in crc.c

[Claude Code Review Bot]

Overall Series Review

Subject: drm/nouveau/kms/nvd9-: Remove unused header in crc.c
Author: Lyude Paul <lyude@redhat.com>
Patches: 1
Reviewed: 2026-05-05T09:07:11.262906

---

This is a single trivial cleanup patch that removes an unused `#include "wndw.h"` from `drivers/gpu/drm/nouveau/dispnv50/crc.c`. The change is correct and harmless.

**Verdict: Looks good.** No functional impact, no risk.

---
Generated by Claude Code Patch Reviewer

Claude review: drm/nouveau/kms/nvd9-: Remove unused header in crc.c

[Claude Code Review Bot]

Patch Review

**Status: Clean**

The patch removes the unused include:

```c
-#include "wndw.h"
```

I verified that `crc.c` contains zero references to any symbol exported by `wndw.h` — no use of `struct nv50_wndw`, `struct nv50_wndw_func`, `nv50_wndw()`, or any of the `wndwc*` / `base507c*` function declarations. The remaining includes (`nouveau_drv.h`, `core.h`, `head.h`, `handles.h`, `crc.h`) are sufficient for everything `crc.c` needs.

**Minor nit on the subject line:** The subject says "nvd9-" (suggesting GF119+), but the file path is `dispnv50/` which covers NV50 and later. The "nvd9-" scope qualifier in the subject seems to refer to the CRC feature's hardware support range rather than the file location, which is fine, but could be slightly confusing since the file itself is shared across the dispnv50 code.

**No issues found.** Reviewed-by worthy as-is.

---
Generated by Claude Code Patch Reviewer