On Fri, Jun 3, 2016 at 9:30 PM, Crestez Dan Leonard <[email protected]> wrote: > In some cases this can result in incorrectly returning a negative value > from asus_acpi_get_sensor_info and the AK8963 magnetometer failing to > show up. > > Cc: Srinivas Pandruvada <[email protected]> > Signed-off-by: Crestez Dan Leonard <[email protected]>
Looks good to me. I had to look into the code to see that cpm is an alias for buffer.pointer, perhaps this should have been mentioned in the commit message. Acked-by: Daniel Baluta <[email protected]> > --- > drivers/iio/imu/inv_mpu6050/inv_mpu_acpi.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/iio/imu/inv_mpu6050/inv_mpu_acpi.c > b/drivers/iio/imu/inv_mpu6050/inv_mpu_acpi.c > index 2771106..f770472 100644 > --- a/drivers/iio/imu/inv_mpu6050/inv_mpu_acpi.c > +++ b/drivers/iio/imu/inv_mpu6050/inv_mpu_acpi.c > @@ -56,6 +56,7 @@ static int asus_acpi_get_sensor_info(struct acpi_device > *adev, > int i; > acpi_status status; > union acpi_object *cpm; > + int ret; > > status = acpi_evaluate_object(adev->handle, "CNF0", NULL, &buffer); > if (ACPI_FAILURE(status)) > @@ -82,10 +83,10 @@ static int asus_acpi_get_sensor_info(struct acpi_device > *adev, > } > } > } > - > + ret = cpm->package.count; > kfree(buffer.pointer); > > - return cpm->package.count; > + return ret; > } > > static int acpi_i2c_check_resource(struct acpi_resource *ares, void *data) > -- > 2.5.5 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to [email protected] > More majordomo info at http://vger.kernel.org/majordomo-info.html
