summaryrefslogtreecommitdiffstats
path: root/drivers/hid
diff options
context:
space:
mode:
authorAlan Cox <alan@linux.intel.com>2012-11-20 17:33:14 +0100
committerJiri Kosina <jkosina@suse.cz>2012-11-20 17:35:57 +0100
commite3bbe8755c9c947d80f88c8a2a81073dbc01bd1f (patch)
tree5056ee0fbca7e8c573a63b65aac4f2a465fd4977 /drivers/hid
parentHID: hiddev: Remove redundant check on unsigned variable (diff)
downloadlinux-e3bbe8755c9c947d80f88c8a2a81073dbc01bd1f.tar.xz
linux-e3bbe8755c9c947d80f88c8a2a81073dbc01bd1f.zip
HID: sensor-hub: Remove pointless NULL check
report cannot be NULL, fortunately as we use it before we check ! Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Diffstat (limited to 'drivers/hid')
-rw-r--r--drivers/hid/hid-sensor-hub.c4
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/hid/hid-sensor-hub.c b/drivers/hid/hid-sensor-hub.c
index d9d73e9163eb..4ff24977544d 100644
--- a/drivers/hid/hid-sensor-hub.c
+++ b/drivers/hid/hid-sensor-hub.c
@@ -437,9 +437,6 @@ static int sensor_hub_raw_event(struct hid_device *hdev,
ptr = raw_data;
ptr++; /*Skip report id*/
- if (!report)
- goto err_report;
-
spin_lock_irqsave(&pdata->lock, flags);
for (i = 0; i < report->maxfield; ++i) {
@@ -485,7 +482,6 @@ static int sensor_hub_raw_event(struct hid_device *hdev,
callback->pdev);
spin_unlock_irqrestore(&pdata->lock, flags);
-err_report:
return 1;
}