Re: [PATCH] HID: corsair-void: Add Corsair Void headset family driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Stuart,

kernel test robot noticed the following build warnings:

[auto build test WARNING on hid/for-next]
[also build test WARNING on linus/master v6.11-rc3 next-20240816]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Stuart-Hayhurst/HID-corsair-void-Add-Corsair-Void-headset-family-driver/20240815-004208
base:   https://git.kernel.org/pub/scm/linux/kernel/git/hid/hid.git for-next
patch link:    https://lore.kernel.org/r/20240813153819.840275-3-stuart.a.hayhurst%40gmail.com
patch subject: [PATCH] HID: corsair-void: Add Corsair Void headset family driver
config: parisc-randconfig-r131-20240816 (https://download.01.org/0day-ci/archive/20240816/202408162236.nuV8tt6o-lkp@xxxxxxxxx/config)
compiler: hppa-linux-gcc (GCC) 14.1.0
reproduce: (https://download.01.org/0day-ci/archive/20240816/202408162236.nuV8tt6o-lkp@xxxxxxxxx/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@xxxxxxxxx>
| Closes: https://lore.kernel.org/oe-kbuild-all/202408162236.nuV8tt6o-lkp@xxxxxxxxx/

sparse warnings: (new ones prefixed by >>)
>> drivers/hid/hid-corsair-void.c:405:23: sparse: sparse: incorrect type in assignment (different base types) @@     expected unsigned short [usertype] send_sidetone @@     got restricted __le16 [usertype] @@
   drivers/hid/hid-corsair-void.c:405:23: sparse:     expected unsigned short [usertype] send_sidetone
   drivers/hid/hid-corsair-void.c:405:23: sparse:     got restricted __le16 [usertype]

vim +405 drivers/hid/hid-corsair-void.c

   395	
   396	static int corsair_void_send_sidetone_wired(struct device *dev, const char *buf,
   397						    unsigned int sidetone)
   398	{
   399		struct usb_interface *usb_if = to_usb_interface(dev->parent);
   400		struct usb_device *usb_dev = interface_to_usbdev(usb_if);
   401		u16 send_sidetone;
   402		int ret = 0;
   403	
   404		/* Packet format to set sidetone for wired headsets */
 > 405		send_sidetone = cpu_to_le16(sidetone);
   406		ret = usb_control_msg_send(usb_dev, 0,
   407					   CORSAIR_VOID_USB_SIDETONE_REQUEST,
   408					   CORSAIR_VOID_USB_SIDETONE_REQUEST_TYPE,
   409					   CORSAIR_VOID_USB_SIDETONE_VALUE,
   410					   CORSAIR_VOID_USB_SIDETONE_INDEX,
   411					   &send_sidetone, 2, USB_CTRL_SET_TIMEOUT,
   412					   GFP_KERNEL);
   413	
   414		return ret;
   415	}
   416	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki




[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux