From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: drm: verisilicon: add max cursor size to HWDB Date: Thu, 07 May 2026 13:14:48 +1000 Message-ID: In-Reply-To: <20260506175610.2542888-2-zhengxingda@iscas.ac.cn> References: <20260506175610.2542888-1-zhengxingda@iscas.ac.cn> <20260506175610.2542888-2-zhengxingda@iscas.ac.cn> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review This patch is clean and already has Thomas Zimmermann's Reviewed-by. - The comment explaining the square-only constraint is good. - Using `int32_t` for `max_cursor_size` matches how the cursor plane code uses it. Though `u32` would arguably be more natural for a size value, it's not wrong. - All 4 HWDB entries are updated to `max_cursor_size = 64`. The cover letter says DC8200 variants support 32 and 64 while some DC8000 support up to 256, so 64 is a safe common value. No issues with this patch. --- Generated by Claude Code Patch Reviewer