From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 0AF70606A899 for ; Sun, 15 Nov 2026 20:27:51 +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: by kanga.kvack.org (Postfix) id D15966B00EF; Sun, 15 Nov 2026 15:27:49 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id CC67D6B05B0; Sun, 15 Nov 2026 15:27:49 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id BB5FE6B05B1; Sun, 15 Nov 2026 15:27:49 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id 8B4966B00EF for ; Sun, 15 Nov 2026 15:27:49 -0500 (EST) Received: from smtpin23.hostedemail.com (lb01a-stub [10.200.18.249]) by unirelay07.hostedemail.com (Postfix) with ESMTP id 7142A160711 for ; Sun, 15 Nov 2026 20:27:46 +0000 (UTC) X-FDA: 85438489332.23.B60A277 Received: from tor.source.kernel.org (tor.source.kernel.org [172.105.4.254]) by imf30.hostedemail.com (Postfix) with ESMTP id C5B8580002 for ; Sun, 15 Nov 2026 20:27:44 +0000 (UTC) Authentication-Results: imf30.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20260515 header.b=R9K+GxFc; dmarc=pass (policy=quarantine) header.from=kernel.org; spf=pass (imf30.hostedemail.com: domain of sashiko-bot@kernel.org designates 172.105.4.254 as permitted sender) smtp.mailfrom=sashiko-bot@kernel.org ARC-Seal: i=1; a=rsa-sha256; d=hostedemail.com; s=arc-20220608; cv=none; t=1794774464; b=JIMiGNv9Br2oGUaeIR/U3W5kArY04Hb/7Ji+kHQNt8lIYNzBMbZPpi+XZgWm+ImTdnyX5N X9Rl8D3nTKgoFRQtF0UP/FYpPW9bz+VlPX+Xw5bLJuRw+JanJosC6nVjBK7kl/oHL96IB1 l/r8zNk7u13FsBiICRpOtMgafvLFcp4= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1794774464; h=from:from:sender:reply-to:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=22XwY66KnSOtO1/d/kMDoLBAuqZ542YRW7O2pDyTx1w=; b=qSGnBV12is2BqnJgDaaFPw7gskEjkacO5Pac07mYFsc6hixVfsRseOicU9tod3oHB8YjxR AvxdDzgxn5subRBm7l/ZkmSkRjQBJr4ys8pq8szgy+2AGOFA822J/nodTK+LYYM4IGRGa/ l2jSiELWBU/L6wrdRuai+aimsviFIqc= ARC-Authentication-Results: i=1; imf30.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20260515 header.b=R9K+GxFc; dmarc=pass (policy=quarantine) header.from=kernel.org; spf=pass (imf30.hostedemail.com: domain of sashiko-bot@kernel.org designates 172.105.4.254 as permitted sender) smtp.mailfrom=sashiko-bot@kernel.org 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 Reply-To: sashiko-reviews@lists.linux.dev 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-Rspamd-Queue-Id: C5B8580002 X-Rspamd-Server: rspam12 X-Stat-Signature: ksydjdeah1dabycyuoi38hu5yiamzcpm X-Rspam-User: 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_PASS,SPF_PASS autolearn=ham autolearn_force=no version=4.0.1 X-HE-Tag: 1794774464-131523 X-HE-Meta: U2FsdGVkX19aJH4q5BA9MbdKtmy1QJYX1KT+ALEbjPWXO8NshWMfIEF7jD7rd4qs2reJA5AzYa/y3Q5duWr030PkpWiMiaWujxbpZceeKhXnJIcB9Q1b9lmEOj57p4LVuDpgDFbDZP8wYQb4dSOUcRj9X29W2RA9517JV6Z3CiFoV1CEsjcD/nYEUYf1F/pmHGzvc1uLPFk7vnysiwLXI5A9KmfTAPPPEI+Ve2881qYvpHsvREbNh8A3wwu1b5M+SdO8QiZYOqW3Wd7aysx1WYHigDSbeNvdd6ByGr93XoLci/SUHFkJQ0ilta1SAYaYK8kM/tkzElmbRh1srvxtZJhNrabShaHpt0N/8fHT8d32jbZixfkBfvfvNuzANXvklQHcc2niEksJDY60RlHSlC5t7kaPiF7olHGDh/81t/uATemB8qjQe8W66x5/DOs0zXebd/PzTpCzFPHEMJPsINae/gv6Luztj+ox59vP3m6ZIngERsEanIz1q394oERZtEcPFllSxbWyjb/LLH9MB+vLVHTBbGvXqHPHWzkSYA1pT6udiWy4m2MnqgDD7c9pGo6LhbsviWjCQ5iY+8YMw5LFccgxkZUQUVCrxgVD9ThUVb7l0SzaiLilkUKhpwHZ0yoBDF0HLTU8rJRyI+Dq1qL+s+dQGc+1miDniR4EOA8oUAD294yvzAm3qJ1ms6BbTQhKAl41Kh26oAYcfQZce0Z+nuXqbWbPJqw9G/yCnTmv635Y3zMvs8OwiyviiuU4WRn2X6sLo/XRqh766s34lf5O1C05pJQZhxPrDPAjboBG2GXSjsqbz5ao+CFJFc2On1vP+rijwVdhyeCRqwvklOqdh3QhB/sJn+8SgMwrb5IQOy9YpRlDBWN2Ej5axw/VWjqJPMU/YT/9HtpC6+jcH6PjVOnJ3Bhym7a1xnxbAUrhm2Ibxo+wPiX3RUJwTnyO5RxtEcSzBjCAcMaKJW1 feTGUlAH jR0+7EW4foGcutCrD0l1C/kQ7VtxN4LNDrit7jhPnKXa7ns33NBY6DVe7b36HF5hCmwcbdMeIazKcyqQOjgFs2E4kmXjhQPGk6AMuWBQaBLZ5iztNHa48iK9Dkfa19AdquhsMZi3gmnalOxk3uJZHVHeF9UK/BdT8XbSR3aq0Q16JBe15bcLV1fipBBc381XpSo+8mCUpRfDpAEcn7Aqa6KNVTRJ6rri5UHiK6aK3mT22UBZuHtQVnRaHhtyNSnncv2VBeNLEV5ZmVD+4LtPtT+q4i9x6uZj2ur20wd5rfltJYHk613jRD2lLUx4BITqNyhWMFKZcbEwnIfY/h0HE1HJcH3w3sFXZY5Zfv6+73cQokyMSIek9oJGgsOwXYhxae+xSc+qL5Y1a6c0Ga2q/KdAod8weIFu5w8M6V4n9wnC2CRAQOEPB542fsffAdt+ohyjEmh14KYZiLM9vAwe/D5YFrkv9x4ZyR7MiU53m4CobcSnLWlqxMhNuSMkjsTApR8Wvnlyo8rD2JuXy1LBfwyfT5bZ5jbv4IrSNtq+B+8WuUfsqUrYUvaBLqP9q4nC/I7d0rt7tNrtDoTknRHv3phspfwQ8diKT2Zc63IzJVoZaflGorEaNZixSpQSh1Cc3Qbvxsje5kbYmwuXLcZLyRe8LE0yxXRKexIihm2RCgFBcF+86Rvtj2GQYEpuqoi3YhWl0A7DkqR6jyHcd+Rk6WD4c2YFRI0zZgpVMB1LByOFE9bN5I05aftgKFyKkC81b0zOVYQ3GoO1xAz2gir+neG03XxtpHSaAkMgsydSOoUjQdYrPeGNHkGjIsfsfVzvnYMiB64t8piNoS5+7Sg62PaAUoYpzkGOiJv+s Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: 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