Re: [PATCH 1/5] clock: introduce non interruptible timeout

[Sascha Hauer <s.hauer@pengutronix.de>]

On Sat, Oct 27, 2012 at 10:03:41PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> is_non_timeout call poller_call if the timeout is >= 100us
> 
> but on 1-wire bus we need to wait 500us and not more than 930us
> for the bus reset. So if the poller_call is caller we can not guarantee it.
> 
> So for this introduce is_non_interruptible_timeout than we only wait.
> 
> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> ---
>  common/clock.c  |   14 ++++++++++----
>  include/clock.h |    1 +
>  2 files changed, 11 insertions(+), 4 deletions(-)
> 
> diff --git a/common/clock.c b/common/clock.c
> index 8adbeaf..a358b78 100644
> --- a/common/clock.c
> +++ b/common/clock.c
> @@ -135,16 +135,22 @@ uint32_t clocksource_hz2mult(uint32_t hz, uint32_t shift_constant)
>          return (uint32_t)tmp;
>  }
>  
> -int is_timeout(uint64_t start_ns, uint64_t time_offset_ns)
> +int is_non_interruptible_timeout(uint64_t start_ns, uint64_t time_offset_ns)

Please rename to is_timeout_non_interruptible.

Otherwise ok.

Sascha


-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox