Re: [PATCH v3] iio: adc: exynos5_adc: fix compilation warnings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/13/2013 07:23 PM, Doug Anderson wrote:
> Lars,
> 
> On Wed, Mar 13, 2013 at 11:11 AM, Lars-Peter Clausen <lars@xxxxxxxxxx> wrote:
>> Agreed. Adding the dependency on OF in Kconfig should be all that is needed.
> 
> I think changing the timeout from 'unsigned long' to 'long' is also
> legit (to match the actual type returned) and a good idea.
> 
> -Doug

Yes, but that's a different issue and to be honest I didn't even realize
that the patch was trying to fix this as well. In my opinion it's best to
split this up into two patches one which fixes the OF dependency. The other
fixing the timeout issue. Cause there is more to it than just changing the type.

wait_for_completion_interruptible_timeout() may return
	1) 0, if there was a timeout, waiting for the completion
	2) > 0, if the completion was completeted, the returned value
	   the  remaining time.
	3) < 0, If it was interrupt while waiting for the completion

The code currently only handles 1) and 2), but it also needs to handle 3).
Since the completion has not been completed in case 3.

E.g. something like this should work:

	if (timeout == 0)
		return -ETIMEDOUT;
	else if(timeout < 0)
		return timeout;
	return 0;

- Lars

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux