From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: fbdev: au1100fb: Replace custom printk wrappers by pr_* Date: Mon, 16 Feb 2026 11:57:00 +1000 Message-ID: In-Reply-To: =?utf-8?q?=3C6c72b8bcd1b608d1ec7227b8d1aa6c7a73148a05=2E1771163?= =?utf-8?q?316=2Egit=2Eu=2Ekleine-koenig=40baylibre=2Ecom=3E?= References: =?utf-8?q??= =?utf-8?q?=3C6c72b8bcd1b608d1ec7227b8d1aa6c7a73148a05=2E1771163316=2Egit=2E?= =?utf-8?q?u=2Ekleine-koenig=40baylibre=2Ecom=3E?= X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review **Minor issue found**: The `pr_fmt` definition is: ```c #define pr_fmt(fmt) "au1100fb:" fmt "\n" ``` The old macros used `DRIVER_NAME ": "` (with a space after the colon). The ne= w `pr_fmt` has `"au1100fb:"` with no space. This changes all message output f= rom `"au1100fb: some message"` to `"au1100fb:some message"`. This is likely a= n oversight - the standard kernel convention for `pr_fmt` includes a space (e= .g., `KBUILD_MODNAME ": " fmt`). --- Generated by Claude Code Patch Reviewer