summaryrefslogtreecommitdiffstats
path: root/drivers/hid/hid-quirks.c
diff options
context:
space:
mode:
authorHyungwoo Yang <hyungwoo.yang@intel.com>2019-06-06 06:52:27 +0200
committerJiri Kosina <jkosina@suse.cz>2019-06-26 14:08:11 +0200
commitb12bbdc5dd883f6575f57e529af26cd2c521b320 (patch)
tree99fe288f374305a9fd20c3dfb9d14bf8dc3b1ed1 /drivers/hid/hid-quirks.c
parentHID: multitouch: Add pointstick support for ALPS Touchpad (diff)
downloadlinux-b12bbdc5dd883f6575f57e529af26cd2c521b320.tar.xz
linux-b12bbdc5dd883f6575f57e529af26cd2c521b320.zip
HID: intel-ish-hid: fix wrong driver_data usage
Currently, in suspend() and resume(), ishtp client drivers are using driver_data to get "struct ishtp_cl_device" object which is set by bus driver. It's wrong since the driver_data should not be owned bus. driver_data should be owned by the corresponding ishtp client driver. Due to this, some ishtp client driver like cros_ec_ishtp which uses its driver_data to transfer its data to its child doesn't work correctly. So this patch removes setting driver_data in bus drier and instead of using driver_data to get "struct ishtp_cl_device", since "struct device" is embedded in "struct ishtp_cl_device", we introduce a helper function that returns "struct ishtp_cl_device" from "struct device". Signed-off-by: Hyungwoo Yang <hyungwoo.yang@intel.com> Acked-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Diffstat (limited to 'drivers/hid/hid-quirks.c')
0 files changed, 0 insertions, 0 deletions