HID: sensors: remove some unneeded checks
"report_id" is unsigned so it's never less than zero. These checks can be removed without any problem. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Jiri Kosina <jkosina@suse.cz> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
This commit is contained in:
parent
2b7c4b8e3e
commit
24db0d75d3
|
@ -214,9 +214,6 @@ int sensor_hub_set_feature(struct hid_sensor_hub_device *hsdev, u32 report_id,
|
|||
struct sensor_hub_data *data = hid_get_drvdata(hsdev->hdev);
|
||||
int ret = 0;
|
||||
|
||||
if (report_id < 0)
|
||||
return -EINVAL;
|
||||
|
||||
mutex_lock(&data->mutex);
|
||||
report = sensor_hub_report(report_id, hsdev->hdev, HID_FEATURE_REPORT);
|
||||
if (!report || (field_index >= report->maxfield)) {
|
||||
|
@ -241,9 +238,6 @@ int sensor_hub_get_feature(struct hid_sensor_hub_device *hsdev, u32 report_id,
|
|||
struct sensor_hub_data *data = hid_get_drvdata(hsdev->hdev);
|
||||
int ret = 0;
|
||||
|
||||
if (report_id < 0)
|
||||
return -EINVAL;
|
||||
|
||||
mutex_lock(&data->mutex);
|
||||
report = sensor_hub_report(report_id, hsdev->hdev, HID_FEATURE_REPORT);
|
||||
if (!report || (field_index >= report->maxfield)) {
|
||||
|
@ -271,9 +265,6 @@ int sensor_hub_input_attr_get_raw_value(struct hid_sensor_hub_device *hsdev,
|
|||
struct hid_report *report;
|
||||
int ret_val = 0;
|
||||
|
||||
if (report_id < 0)
|
||||
return -EINVAL;
|
||||
|
||||
mutex_lock(&data->mutex);
|
||||
memset(&data->pending, 0, sizeof(data->pending));
|
||||
init_completion(&data->pending.ready);
|
||||
|
|
Loading…
Reference in New Issue