Re: [PATCH 4/6] PalmLD IDE

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tuesday 21 of April 2009 17:19:33 Alan Cox wrote:
> > http://www.palm.com/us/support/contact/environment/disassem_inst_Lifedriv
> >e.pdf , see slide 10 and 15 for yourself, I think noone would want to
> > solder anything into that device ;-)
>
> Agreed but if the hardware can do it you don't need the flag. Obviously
> not very important in this case.

OK, we can remove that, I'll resend the patch.
>
> > > +	/* we'd better wait for drive's ready signal here
> > > +	  (if we knew where it will come from) */
> > > +	msleep(300);
> > >
> > > Our probe code should already be waiting for ready signals  ?
> >
> > The drive asserts some GPIO when it becomes ready, dunno which one though
> > so we just wait here.
>
> Ok so the ATA bus side ready isn't sufficient for your hardware ?

I can retry without the delay, but I'm quite afraid I ran into problems 
without it. Btw. I rewrote that driver more than half year ago, so I don't 
really recall the details, sorry, I don't want to misinform you here.
>
> > > The only other question is a general architectural one as to whether it
> > > would be better to set up the GPIO etc and create a pata_platform
> > > platform device (possibly tweaking pata_platform flags to allow the
> > > caller to indicate generic pio with mode setting by set features
> > > command)
> >
> > I already explained this to Eric, there isn't any other obscure hardware
> > I think. Or if this is concerning something else, I'll recheck later.


--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux