lkml.org 
[lkml]   [2003]   [Feb]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: PATCH: clean up the IDE iops, add ones for a dead iface
From
Date
On Wed, 2003-02-19 at 02:53, Alan Cox wrote:
> On Tue, 2003-02-18 at 23:09, Russell King wrote:
> > On Tue, Feb 18, 2003 at 06:03:21PM +0000, Alan Cox wrote:
> > > +static u8 ide_unplugged_inb (unsigned long port)
> > > +{
> > > + return 0xFF;
> > > +}
> >
> > Shouldn't this return 0x7f, ie bit 7 clear, as if we have an interface
> > without drive attached?
>
> 0xFF is what most hardware returns for an unclaimed pci address. I'd rather
> make sure it also works when we don't get the event, or the device
> summarily explodes than fake up something to make it look good. If
> 0xff doesnt work we have a bug anyway, lets find them.

Hrm... I tend to agree with Russell here... 0x7f is the "safe" value
for IDE. IDE controllers with nothing wired shall have a pull down
on D7. The reason is simple: busy loops in the IDE code waiting for
BSY to go down.

Now, if your point is to keep BSY up and have wait loops timeout,
then 0xff may actually make some sense ;)

Ben.
-
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/

\
 
 \ /
  Last update: 2005-03-22 13:33    [W:0.063 / U:5.396 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site