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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <202408162236.nuV8tt6o-lkp@intel.com>
Date: Fri, 16 Aug 2024 22:49:55 +0800
From: kernel test robot <lkp@...el.com>
To: Stuart Hayhurst <stuart.a.hayhurst@...il.com>,
	linux-input@...r.kernel.org
Cc: oe-kbuild-all@...ts.linux.dev,
	Stuart Hayhurst <stuart.a.hayhurst@...il.com>,
	Jiri Kosina <jikos@...nel.org>,
	Benjamin Tissoires <bentiss@...nel.org>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] HID: corsair-void: Add Corsair Void headset family driver

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@intel.com/config)
compiler: hppa-linux-gcc (GCC) 14.1.0
reproduce: (https://download.01.org/0day-ci/archive/20240816/202408162236.nuV8tt6o-lkp@intel.com/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@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202408162236.nuV8tt6o-lkp@intel.com/

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