From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jonathan Cameron Date: Tue, 18 Sep 2012 20:52:38 +0000 Subject: Re: [patch 2/2] HID: sensors: remove some unneeded checks Message-Id: <5058DF16.20002@kernel.org> List-Id: References: <20120914065346.GE11886@elgon.mountain> <4FA419E87744DF4DAECD5BCE1214B7A919D527E5@ORSMSX108.amr.corp.intel.com> In-Reply-To: <4FA419E87744DF4DAECD5BCE1214B7A919D527E5@ORSMSX108.amr.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: "Pandruvada, Srinivas" Cc: Jiri Kosina , Dan Carpenter , "linux-input@vger.kernel.org" , "kernel-janitors@vger.kernel.org" I've add these both to the togreg branch of git://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git Will send onwards in a few days. > Tested this and it works. > > -----Original Message----- > From: Jiri Kosina [mailto:jkosina@suse.cz] > Sent: Monday, September 17, 2012 4:00 AM > To: Dan Carpenter > Cc: Pandruvada, Srinivas; linux-input@vger.kernel.org; kernel-janitors@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 > > Signed-off-by: Jiri Kosina > > 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 >