* 2.2.18: e100.c (SuSE 7.1): udelay() used in a wrong way?
@ 2001-03-22 12:59 Ulrich Windl
2001-03-22 13:26 ` Jeff Garzik
0 siblings, 1 reply; 2+ messages in thread
From: Ulrich Windl @ 2001-03-22 12:59 UTC (permalink / raw)
To: linux-kernel
>From the source code of drivers/net/e100.c:
/****************************************************************************
* Name: Phy82562EHDelayMilliseconds
*
* Description: Stalls execution for a specified number of milliseconds.
*
* Arguments: Time - milliseconds to delay
*
* Returns: Nothing
*
************************************************************************
***/
void
Phy82562EHDelayMilliseconds(int Time)
{
udelay(Time);
}
AFAIK, udelay() delays microseconds, not milliseconds.
Ulrich
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: 2.2.18: e100.c (SuSE 7.1): udelay() used in a wrong way?
2001-03-22 12:59 2.2.18: e100.c (SuSE 7.1): udelay() used in a wrong way? Ulrich Windl
@ 2001-03-22 13:26 ` Jeff Garzik
0 siblings, 0 replies; 2+ messages in thread
From: Jeff Garzik @ 2001-03-22 13:26 UTC (permalink / raw)
To: Ulrich Windl; +Cc: linux-kernel
Ulrich Windl wrote:
> void
> Phy82562EHDelayMilliseconds(int Time)
> {
> udelay(Time);
> }
>
> AFAIK, udelay() delays microseconds, not milliseconds.
Yep, you are correct, and the code is incorrect.
mdelay() delays milliseconds.
--
Jeff Garzik | May you have warm words on a cold evening,
Building 1024 | a full mooon on a dark night,
MandrakeSoft | and a smooth road all the way to your door.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2001-03-22 13:28 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2001-03-22 12:59 2.2.18: e100.c (SuSE 7.1): udelay() used in a wrong way? Ulrich Windl
2001-03-22 13:26 ` Jeff Garzik
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox