]> www.infradead.org Git - users/hch/misc.git/commitdiff
ALSA: usb-audio: us144mkii: Use le16_to_cpu() for product ID comparison
authorŠerif Rami <ramiserifpersia@gmail.com>
Tue, 19 Aug 2025 07:51:17 +0000 (09:51 +0200)
committerTakashi Iwai <tiwai@suse.de>
Tue, 19 Aug 2025 09:29:03 +0000 (11:29 +0200)
The `us144mkii.c` driver was generating sparse warnings due to direct
comparisons of `dev->descriptor.idProduct` (a `__le16` type) with
integer constants. This commit resolves these warnings by explicitly
converting `idProduct` to a CPU-endian integer using `le16_to_cpu()`
before comparison.

This ensures correct handling of endianness and eliminates the sparse
warnings:
- `restricted __le16 degrades to integer`

Fixes: dee1bcf28a3d ("ALSA: usb-audio: Add initial driver for TASCAM US-144MKII")
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202508190811.tjQJZI1X-lkp@intel.com/
Signed-off-by: Šerif Rami <ramiserifpersia@gmail.com>
Link: https://patch.msgid.link/20250819075118.9221-1-ramiserifpersia@gmail.com
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/usb/usx2y/us144mkii.c

index da53c1908c4aa6b2a47e84402a254d51d003cb61..3127a3206370119b469586410c9a0e8cdb9a745d 100644 (file)
@@ -514,10 +514,10 @@ static int tascam_probe(struct usb_interface *intf,
        }
 
        strscpy(card->driver, DRIVER_NAME, sizeof(card->driver));
-       if (dev->descriptor.idProduct == USB_PID_TASCAM_US144) {
+       if (le16_to_cpu(dev->descriptor.idProduct) == USB_PID_TASCAM_US144) {
                strscpy(card->shortname, "TASCAM US-144",
                        sizeof(card->shortname));
-       } else if (dev->descriptor.idProduct == USB_PID_TASCAM_US144MKII) {
+       } else if (le16_to_cpu(dev->descriptor.idProduct) == USB_PID_TASCAM_US144MKII) {
                strscpy(card->shortname, "TASCAM US-144MKII",
                        sizeof(card->shortname));
        } else {