From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 351EE606A338 for ; Sun, 15 Nov 2026 20:28:01 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20260515 header.b=R9K+GxFc Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id C8C4D10E348; Sun, 15 Nov 2026 20:27:55 +0000 (UTC) Received: from tor.source.kernel.org (tor.source.kernel.org [172.105.4.254]) by gabe.freedesktop.org (Postfix) with ESMTPS id 4928A10E0C7; Sun, 15 Nov 2026 20:27:44 +0000 (UTC) Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by tor.source.kernel.org (Postfix) with ESMTP id 0E5C0624DD; Sun, 15 Nov 2026 20:27:43 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9A52E1F000C4; Sun, 15 Nov 2026 20:27:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1794774462; bh=22XwY66KnSOtO1/d/kMDoLBAuqZ542YRW7O2pDyTx1w=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=R9K+GxFcxklLWxKNOJ5lRWC5zIQSWncnMMkh8TmdSlBzjVeCIYok3BI+LiZIT9Js/ wuRlKxXZ9tfeGHjVjLVPj4KVIC2OmTkytp5uIluoKYZYWdSYEniQPIj5QQyxgVWJ8a /bOuIKw1JSpNOZ6GUsOkWzb/fPu5wkio1MAqhuM/AxZd79rmeJLPB6eZ+PNX5oxePa I4OAq3pr+5s5YjD2xmNd79jVJ5yHrBZtLZEQ00vdL+E1kl4UHJOq+coEs+/Uz0nYt6 lORfbphZZ9QOpkOn8tneNfDGO1qbDVyF4Aocafhn88MzuNMXyI5Q4SQtGN+PraJ4cu KIB9IJPamEnFA== Message-ID: From: sashiko-bot@kernel.org Subject: Re: [PATCH 02/12] docs: ioctl-number: update sound entries To: "Randy Dunlap" Cc: "Jonathan Corbet" , "Shuah Khan" , "David Hildenbrand" , "Andrew Morton" , "Jason Gunthorpe" , "Leon Romanovsky" , linux-mm@kvack.org, "Jaroslav Kysela" , "Takashi Iwai" , linux-sound@vger.kernel.org, "Jens Axboe" , "Marcel Holtmann" , "Luiz Augusto von Dentz" , "Willem de Bruijn" , "Jason Wang" , "David Heidelberg" , "Chas Williams" <3chas3@gmail.com>, "David S. Miller" , "Eric Dumazet" , "Jakub Kicinski" , "Paolo Abeni" , "Helge Deller" , dri-devel@lists.freedesktop.org, MPT-FusionLinux.pdl@broadcom.com, "James E.J. Bottomley" , "Martin K. Petersen" , linux-scsi@vger.kernel.org, "Miquel Raynal" , "Richard Weinberger" , "Vignesh Raghavendra" , "Zhihao Cheng" , linux-mtd@lists.infradead.org, "Neill Kapron" , "Andrey Konovalov" , "Greg Kroah-Hartman" , linux-usb@vger.kernel.org, "Juergen Gross" , "Stefano Stabellini" , "Hans de Goede" , "Arnd Bergmann" , "Michael S. Tsirkin" , =?utf-8?b?RXVnZW5pbyBQw6lyZXo=?= , "Sean Christopherson" , "Paolo Bonzini" , kvm@vger.kernel.org, virtualization@lists.linux.dev, "Jan Kara" , =?utf-8?b?VGhlb2RvcmUgVHMnbw==?= , linux-ext4@vger.kernel.org, "Mike Marshall" , devel@lists.orangefs.org, "Paulo Alcantara" , "Namjae Jeon" , "Jaegeuk Kim" , "Chao Yu" , "Florian Fainelli" , "Broadcom internal kernel review list" , "Tzung-Bi Shih" , "Benson Leung" , "David Airlie" , "Simona Vetter" , "Alex Deucher" , =?utf-8?b?Q2hyaXN0aWFuIEvDtm5pZw==?= , amd-gfx@lists.freedesktop.org, "Jani Nikula" , "Joonas Lahtinen" , "Rodrigo Vivi" , "Tvrtko Ursulin" , intel-gfx@lists.freedesktop.org, "Matthew Brost" , =?utf-8?b?VGhvbWFzIEhlbGxzdHLDtm0=?= , intel-xe@lists.freedesktop.org In-Reply-To: <20261114202653.3113381-3-rdunlap@infradead.org> References: <20261114202653.3113381-1-rdunlap@infradead.org> <20261114202653.3113381-3-rdunlap@infradead.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Date: Sun, 15 Nov 2026 20:27:39 +0000 X-BeenThere: intel-xe@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel Xe graphics driver List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: sashiko-reviews@lists.linux.dev Sender: "Intel-xe" X-Spam-Status: No, score=-10.8 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=4.0.1 X-Spam-Checker-Version: SpamAssassin 4.0.1 (2024-03-26) on aws-us-west-2-korg-listserv-alma10-1 X-Archived-By: Thank you for your contribution! Sashiko AI review found 5 potential issues to consider: - [Low] docs: ioctl-number: wrong code, range, and conflict marker for scarlett2.h - [Low] docs: ioctl-number: incomplete sequence range for sound/pcm.h - [Low] docs: ioctl-number: incomplete range and tab char for sound/core/pcm_compat.c - [Low] docs: ioctl-number: incomplete sequence range for sound/core/seq/seq_compat.c - [Low] docs: ioctl-number: missing conflict! marker on switchtec.c under code 'W' -- commit 0316dc2cd0f479a35471758d208f4335b8fbc617 Author: Randy Dunlap docs: ioctl-number: update sound entries This commit updates the ioctl-number documentation table with additional sound header and compat files and adjusts sequence number ranges for sound ioctls. > diff --git a/Documentation/userspace-api/ioctl/ioctl-number.rst b/Documen= tation/userspace-api/ioctl/ioctl-number.rst > index e1d56d5cccd14..dac65f59d05e2 100644 > --- a/Documentation/userspace-api/ioctl/ioctl-number.rst > +++ b/Documentation/userspace-api/ioctl/ioctl-number.rst > @@ -122,11 +122,14 @@ Code Seq# Include File = Comments > 'A' 00-0F linux/agpgart.h, c= onflict! > drivers/char/agp/compat_ioctl.h > 'A' 00-7F sound/asound.h c= onflict! > +'A' 20 sound/pcm.h c= onflict! [Severity: Low] Should the sequence range for sound/pcm.h be 20-24 instead of 20? In include/sound/pcm.h, SNDRV_PCM_IOCTL_STATUS_EXT64 and SNDRV_PCM_IOCTL_STATUS_EXT32 also define ioctls using _IOWR('A', 0x24, ...). > +'A' 20-53 sound/core/pcm_compat.c conf= lict! [Severity: Low] Should the sequence range for sound/core/pcm_compat.c start at 10 rather than 20? The compat ioctl enum in sound/core/pcm_compat.c starts with SNDRV_PCM_IOCTL_HW_REFINE32 (_IOWR('A', 0x10, ...)), SNDRV_PCM_IOCTL_HW_PARAMS32 (0x11), and SNDRV_PCM_IOCTL_SW_PARAMS32 (0x13). Also, there is a literal tab character right after sound/core/pcm_compat.c instead of spaces. [ ... ] > @@ -160,7 +164,9 @@ Code Seq# Include File = Comments > 'H' 10-1F sound/hda_hwdep.h c= onflict! > 'H' 40-4F sound/hdspm.h c= onflict! > 'H' 40-4F sound/hdsp.h c= onflict! > +'H' 60-63 sound/scarlett2.h [Severity: Low] Should sound/scarlett2.h be listed under code 'S' with range 60-64 and a conflict! marker instead of 'H' 60-63? In include/uapi/sound/scarlett2.h, all ioctls use code 'S' from SCARLETT2_IOCTL_PVERSION (_IOR('S', 0x60, int)) through SCARLETT2_IOCTL_GET_ERASE_PROGRESS (_IOR('S', 0x64, ...)), which overlaps with linux/cdrom.h, sound/asequencer.h, and sound/fcp.h. [ ... ] > @@ -204,6 +210,8 @@ Code Seq# Include File = Comments > 'S' 80-81 scsi/scsi_ioctl.h c= onflict! > 'S' 82-FF scsi/scsi.h c= onflict! > 'S' 00-7F sound/asequencer.h c= onflict! > +'S' 21-23 sound/core/seq/seq_compat.c c= onflict! [Severity: Low] Should the sequence range for sound/core/seq/seq_compat.c be 20-52 instead of 21-23? That file also defines SNDRV_SEQ_IOCTL_CREATE_PORT32 (_IOWR('S', 0x20, ...)) and SNDRV_SEQ_IOCTL_QUERY_NEXT_PORT32 (_IOWR('S', 0x52, ...)). [ ... ] > @@ -219,7 +227,7 @@ Code Seq# Include File = Comments > 'V' C0 media/si4713.h c= onflict! > 'W' 00-1F linux/watchdog.h c= onflict! > 'W' 00-1F linux/wanrouter.h c= onflict! (pre 3.9) > -'W' 00-3F sound/asound.h c= onflict! > +'W' 00-41 sound/asound.h c= onflict! > 'W' 40-5F drivers/pci/switch/switchtec.c [Severity: Low] Does the drivers/pci/switch/switchtec.c entry now need a conflict! marker as well, since extending sound/asound.h to 00-41 causes the two ranges to overlap on 40-41? --=20 Sashiko AI review =C2=B7 https://sashiko.dev/#/patchset/20261114202653.3113= 381-1-rdunlap@infradead.org?part=3D2