Re: ds1620: replace schedule_timeout() with msleep()
From: Jesper Juhl (juhl-lkml_at_dif.dk)
Date: 11/30/04
- Previous message: Jesper Juhl: "[PATCH] net/via-rhine: convert MODULE_PARM to module_param"
- In reply to: Russell King: "Re: ds1620: replace schedule_timeout() with msleep()"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Date: Tue, 30 Nov 2004 01:10:46 +0100 (CET) To: Russell King <rmk+lkml@arm.linux.org.uk>
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/
- Previous message: Jesper Juhl: "[PATCH] net/via-rhine: convert MODULE_PARM to module_param"
- In reply to: Russell King: "Re: ds1620: replace schedule_timeout() with msleep()"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Relevant Pages
|
