Commit 35cfd1d9 authored by Michael Tokarev's avatar Michael Tokarev Committed by Jiri Kosina
Browse files

HID: blacklist Powercom USB UPS 



For quite some time users with various UPSes from Powercom were forced to play
magic with bind/unbind in /sys in order to be able to see the UPSes.  The
beasts does not work as HID devices, even if claims to do so.  cypress_m8
driver works with the devices instead, creating a normal serial port with which
normal UPS controlling software works.

The manufacturer confirmed the upcoming models with proper HID support will
have different device IDs.  In any way, it's wrong to have two completely
different modules for one device in kernel.

Blacklist the device in HID (add it to hid_ignore_list) to stop this mess,
finally.

Signed-off-By: default avatarMichael Tokarev <mjt@tls.msk.ru>
Signed-off-by: default avatarJiri Kosina <jkosina@suse.cz>
parent d2f8d7ee

drivers/hid/hid-core.c

+1 −0
Original line number Diff line number Diff line
@@ -1605,6 +1605,7 @@ static const struct hid_device_id hid_ignore_list[] = { 
	{ HID_USB_DEVICE(USB_VENDOR_ID_PANJIT, 0x0002) },

	{ HID_USB_DEVICE(USB_VENDOR_ID_PANJIT, 0x0003) },

	{ HID_USB_DEVICE(USB_VENDOR_ID_PANJIT, 0x0004) },

	{ HID_USB_DEVICE(USB_VENDOR_ID_POWERCOM, USB_DEVICE_ID_POWERCOM_UPS) },

	{ HID_USB_DEVICE(USB_VENDOR_ID_SOUNDGRAPH, USB_DEVICE_ID_SOUNDGRAPH_IMON_LCD) },

	{ HID_USB_DEVICE(USB_VENDOR_ID_SOUNDGRAPH, USB_DEVICE_ID_SOUNDGRAPH_IMON_LCD2) },

	{ HID_USB_DEVICE(USB_VENDOR_ID_SOUNDGRAPH, USB_DEVICE_ID_SOUNDGRAPH_IMON_LCD3) },

drivers/hid/hid-ids.h

+3 −0
Original line number Diff line number Diff line
@@ -348,6 +348,9 @@ 
#define USB_VENDOR_ID_PLAYDOTCOM	0x0b43

#define USB_DEVICE_ID_PLAYDOTCOM_EMS_USBII	0x0003



#define USB_VENDOR_ID_POWERCOM		0x0d9f

#define USB_DEVICE_ID_POWERCOM_UPS	0x0002



#define USB_VENDOR_ID_SAITEK		0x06a3

#define USB_DEVICE_ID_SAITEK_RUMBLEPAD	0xff17

CRA Git | Maintained and supported by SUSTech CRA and CCSE