From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: drivers: ALSA: use rest_of_page() macro where appropriate Date: Thu, 05 Mar 2026 13:59:12 +1000 Message-ID: In-Reply-To: <20260304012717.201797-8-ynorov@nvidia.com> References: <20260304012717.201797-1-ynorov@nvidia.com> <20260304012717.201797-8-ynorov@nvidia.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review **Subject line is misleading.** The patch touches: - `drivers/block/null_blk/main.c` (block layer) - `drivers/gpu/drm/i915/gt/shmem_utils.c` (DRM/i915) - `drivers/md/dm-pcache/backing_dev.h` (device-mapper) - `sound/virtio/virtio_pcm_msg.c` (virtio sound) Only the last one is ALSA/sound-related. The subject should be something like "drivers/sound: use rest_of_page() macro where appropriate" or the patch should be split by subsystem to match the granularity of the rest of the series. Given that patches 2-6 are carefully split by subsystem (fs, net, core, spi, KVM), lumping these four unrelated subsystems together under "ALSA" is inconsistent and will make it harder for subsystem maintainers to pick up or ack. The code changes themselves are all correct. --- Generated by Claude Code Patch Reviewer