Re: [PATCH] Configure IDE probe delays

Todd Poynor
2004-08-06T20:51:37Z

At the risk of flogging this horse after the death certificate was just
signed, there were two threads running through the discussion and I
wasn't sure where things stand. One thread, as I read it, was: "Don't
hack around the delays, let's change the probing and do it right". The
other thread was: "Don't mess around with IDE probing, leave that alone
and let's do it right for SATA".

If the latter route is how things are headed then I'd still suggest
replacing the scattered hardcoded "50" constants with a meaningful symbol.

If, on the other hand, IDE probe delays might someday become more
complicated, such that a single symbol is not sufficient (as Alan warned
against), then at least it would be easier to find 'em all. And
converting some of the calls to use a new symbol for a new class of
delays would be quite easy.

Anyhow, not a big deal; if there's anything I can help out with to
improve the situation then I'd be glad to do so.

--
Todd Poynor
MontaVista Software