Messages in this thread | | | Date | Tue, 30 Nov 2004 01:10:46 +0100 (CET) | From | Jesper Juhl <> | Subject | Re: ds1620: replace schedule_timeout() with msleep() |
| |
On Mon, 29 Nov 2004, Russell King wrote:
> On Mon, Nov 29, 2004 at 11:37:48PM +0100, Jesper Juhl wrote: > > On Mon, 29 Nov 2004, Domen Puncer wrote: > > > It's right: > > > schedule_timeout(2*HZ) sleeps for 2 seconds; > > > msleep(2000) sleeps for 2000 miliseconds, and does not depend on what > > > HZ is. > > > > It seems I didn't understand schedule_timeout() properly, thank you for > > the clarification. > > As part-author of this driver, and actually of this particular bit > of code, a 2 second delay is intented here. The fan needs to be run > at full power in order to start running, so the idea here is to give > it full power for 2 seconds and then to restore the temperature trip > points to the configured values. > That makes sense - thanks.
-- Jesper
- 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/
| |