Tested this and it works. -----Original Message----- From: Jiri Kosina [mailto:jkos...@suse.cz] Sent: Monday, September 17, 2012 4:00 AM To: Dan Carpenter Cc: Pandruvada, Srinivas; linux-input@vger.kernel.org; kernel-janit...@vger.kernel.org; Jonathan Cameron Subject: Re: [patch 2/2] HID: sensors: remove some unneeded checks
On Fri, 14 Sep 2012, Dan Carpenter wrote: > "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.carpen...@oracle.com> Signed-off-by: Jiri Kosina <jkos...@suse.cz> Adding Jonathan to CC as well. Thanks. > > diff --git a/drivers/hid/hid-sensor-hub.c > b/drivers/hid/hid-sensor-hub.c index 0c93b10..22ec3c6 100644 > --- a/drivers/hid/hid-sensor-hub.c > +++ b/drivers/hid/hid-sensor-hub.c > @@ -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); > -- Jiri Kosina SUSE Labs -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html