Re: PATCH: __bad_udelay fixes(?) for linux-2.4.0-prerelease

From: Alan Cox (alan@lxorguk.ukuu.org.uk)
Date: Mon Jan 01 2001 - 11:04:32 EST


> (that is, 20 milliseconds). I guess the purpose of this change is
> to tell driver maintainers to either take a harder look at whether they
> really need to do a busy sleep for that long (you can still do it with

It is primarily there because udelay() that long will fail. An mdelay is
an acceptable substitution. mdelay knows the delays will be bigger so the
time to do the math and a loop doesnt throw small delays

> drivers/video/atyfb.c - An intentional 50ms delay.
> drivers/video/clgenfb.c - An intentional 100ms delay.
> I've changed both files to keep the
> delays by using mdelay instead of udelay.
> Perhaps somebody could check the
> approaprirate documentation and test
> on real hardware to determine if the
> delays really need to be this long.

Seems sane. For toshoboe in my tree I just switched it to mdelay()

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sun Jan 07 2001 - 21:00:11 EST