On 10/04/14 03:26, Michael Welling wrote:
Signed-off-by: Michael Welling <mwelling@xxxxxxxx>
Hmm. I wonder if this is a little too tight on the range, but given it is in a function that is likely to be called infrequently it doesn't really matter. Note this is where we find out whether the datasheet is actually accurate is saying 3 msecs is long enough by whether anyone screams :) Also cc'd Jon as if anyone screams about this they are likely to do it at him. As a general rule, do cc the original author unless you know there email bounces (and if so mention it in the cover letter as someone might know where to get hold of them). Applied to the togreg branch of iio.git - initially pushed out as testing. Thanks, Jonathan
--- drivers/staging/iio/light/tsl2583.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/iio/light/tsl2583.c b/drivers/staging/iio/light/tsl2583.c index f015fb4..fa96498 100644 --- a/drivers/staging/iio/light/tsl2583.c +++ b/drivers/staging/iio/light/tsl2583.c @@ -451,7 +451,7 @@ static int taos_chip_on(struct iio_dev *indio_dev) } } - msleep(3); + usleep_range(3000, 3500); /* NOW enable the ADC * initialize the desired mode of operation */ utmp = TSL258X_CNTL_PWR_ON | TSL258X_CNTL_ADC_ENBL;
-- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
