Messages in this thread | | | Date | Sat, 1 Mar 2014 20:13:34 -0800 (PST) | From | Chase Southwood <> | Subject | Re: [PATCH] Staging: comedi: add timeouts to while loops in s626.c |
| |
>On Friday, February 28, 2014 11:49 PM, Chase Southwood <chase.southwood@yahoo.com> wrote:
>>On Friday, February 28, 2014 11:26 AM, Ian Abbott <abbotti@mev.co.uk> wrote: >>>On 2014-02-28 07:35, Chase Southwood wrote:
[snip]
>>In the case of s626_send_dac(), it doesn't seem to be used in any >>critical sections, so it could make use of Hartley's comedi_timeout(). >> >>Some of the timeout errors could be propagated, especially for >>s626_send_dac() which is only reachable from very few paths. > > >Awesome, I'll swap all of my timeouts out for comedi_timeout() in s626_send_dac().
Actually, after taking another look at this, I don't think that using comedi_timeout() here is going to work, actually. The context from which s626_send_dac() is called allows sleep all right, but readl() isn't a comedi function and therefore it doesn't behave (in parameters or return values) as the callback function parameter to comedi_timeout() requires. So unless I'm missing something particularly large here, I believe we'll have to do the timeouts here manually as well. Am I correct here, and if so, would you like the iteration based timeouts here as well, or a sleep-based timeout similar to that employed by comedi_timeout()?
Thanks, Chase -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |