[PATCH] scsi: core: update cdrom.h coments

[PATCH] scsi: core: update cdrom.h coments

[Randy Dunlap]

Update the location and ending value of the cdrom ioctls.

Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
---
Cc: "James E.J. Bottomley" <James.Bottomley@HansenPartnership.com>
Cc: "Martin K. Petersen" <mkp@kernel.org>
Cc: Phillip Potter <phil@philpotter.co.uk>
Cc: Jens Axboe <axboe@kernel.dk>
Cc: linux-block@vger.kernel.org

 include/scsi/scsi.h |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- linux-next-20261019.orig/include/scsi/scsi.h
+++ linux-next-20261019/include/scsi/scsi.h
@@ -176,7 +176,7 @@ enum scsi_qc_status {
 /*
  * Here are some scsi specific ioctl commands which are sometimes useful.
  *
- * Note that include/linux/cdrom.h also defines IOCTL 0x5300 - 0x5395
+ * Note that include/uapi/linux/cdrom.h also defines IOCTL 0x5300 - 0x5396.
  */
 
 /* Used to obtain PUN and LUN info.  Conflicts with CDROMAUDIOBUFSIZ */

Re: [PATCH] scsi: core: update cdrom.h coments

[Phillip Potter]

On Tue, Oct 20, 2026 at 10:06:03AM -0700, Randy Dunlap wrote:
> Update the location and ending value of the cdrom ioctls.
> 
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> ---
> Cc: "James E.J. Bottomley" <James.Bottomley@HansenPartnership.com>
> Cc: "Martin K. Petersen" <mkp@kernel.org>
> Cc: Phillip Potter <phil@philpotter.co.uk>
> Cc: Jens Axboe <axboe@kernel.dk>
> Cc: linux-block@vger.kernel.org
> 
>  include/scsi/scsi.h |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> --- linux-next-20261019.orig/include/scsi/scsi.h
> +++ linux-next-20261019/include/scsi/scsi.h
> @@ -176,7 +176,7 @@ enum scsi_qc_status {
>  /*
>   * Here are some scsi specific ioctl commands which are sometimes useful.
>   *
> - * Note that include/linux/cdrom.h also defines IOCTL 0x5300 - 0x5395
> + * Note that include/uapi/linux/cdrom.h also defines IOCTL 0x5300 - 0x5396.
>   */
>  
>  /* Used to obtain PUN and LUN info.  Conflicts with CDROMAUDIOBUFSIZ */

Hi Randy,

Thanks for the patch. That said, without meaning to be too picky,
strictly speaking the quoted UAPI header file actually defines from
0x5301 onwards unless I'm missing something - perhaps we should change
the start of the range as well? Happy to adjust and forward on with your
other patch if you're happy with that?

Regards,
Phil

Re: [PATCH] scsi: core: update cdrom.h coments

[Martin K. Petersen]

Randy,

> Update the location and ending value of the cdrom ioctls.

Adjusted start value and applied to 7.4/scsi-staging, thanks!

-- 
Martin K. Petersen

Re: [PATCH] scsi: core: update cdrom.h coments

[Martin K. Petersen (Oracle)]

On Tue, 20 Oct 2026 10:06:03 -0700, Randy Dunlap wrote:

> Update the location and ending value of the cdrom ioctls.

Applied to 7.4/scsi-queue, thanks!

[1/1] scsi: core: update cdrom.h coments
      https://git.kernel.org/mkp/scsi/c/5b0d568e183e

-- 
Martin K. Petersen