From mboxrd@z Thu Jan  1 00:00:00 1970
From: Claude Code Review Bot <claude-review@example.com>
To: dri-devel-reviews@example.com
Subject: Claude review: drm/ast: Use constants for SDRAM registers
Date: Wed, 25 Mar 2026 07:46:41 +1000
Message-ID: <review-patch8-20260323160407.245773-9-tzimmermann@suse.de>
In-Reply-To: <20260323160407.245773-9-tzimmermann@suse.de>
References: <20260323160407.245773-1-tzimmermann@suse.de> <20260323160407.245773-9-tzimmermann@suse.de>
X-Mailer: Claude Code Patch Reviewer
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0

Patch Review

Small patch introducing `AST_SDRAM()` macro. The TODO comment about the misleading function comments is a good annotation:
```c
+ * TODO: Review and fix the comments. The function below only detects
+ *       up to 1 GiB of SDRAM.
```

---
Generated by Claude Code Patch Reviewer