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/gma500: fix kernel-doc warning for framebuffer init/create functions Date: Fri, 27 Feb 2026 12:40:47 +1000 Message-ID: In-Reply-To: <20260226030038.1182961-4-yujie.liu@intel.com> References: <20260226030038.1182961-1-yujie.liu@intel.com> <20260226030038.1182961-4-yujie.liu@intel.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review Adds missing `@info` parameter documentation for two functions in the same file, both stemming from the same commit 4a792c59203b that added the `info` parameter. ```diff * psb_framebuffer_init - initialize a framebuffer * @dev: our DRM device * @fb: framebuffer to set up + * @info: pixel format information * @mode_cmd: mode description ``` ```diff * psb_user_framebuffer_create - create framebuffer * @dev: our DRM device * @filp: client file + * @info: pixel format information * @cmd: mode request ``` **Minor nit:** The description "pixel format information" is adequate but could be slightly more precise (e.g., "format info from drm_get_format_info()"). Not worth a respin though. --- --- Generated by Claude Code Patch Reviewer