Re: [PATCH 3/3 v3] AT91: initialize Compact Flash on AT91SAM9263 cpu

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

 



Hi.

Thursday 12 February 2009 17:33:08 Sergei Shtylyov napisał(a):
> >>   I'm still not sure why this is needed. Do you think thta power-on 
> >>reset is not enough?
> 
> > HW reset is not needed when board is powered-on. But situation is
> > diffrent when we have reboot due to board reset booton press.
> > There are configuration where signal from reset button is connected
> > to CF card (or disk), in some cases device reset signal is connected to
> > GPIO with assumption to be asserted by software.
> 
>     OK, thanks for the explanation.
> 
> > I don't know if we can remove this.
> 
>     Well, seems worth keeping.
> 
> > In hw reset case system/driver will run with undefined device state,
> 
>     What do you mean by undefined? It could have not come out of reset which 
> shoiuld mean that the IDE probing code will have to wait some time before BSY 
> clears -- see ide_port_wait_ready().
> 
> > in example device can generate interrupts.
> 
>     Interrupts after any kind of reset? That just shouldn't happen -- it's 
> against the ATA spec at least.
I mean undefined state and interrupts where is _no_ IDE device reset
and system boots because of board button h/w reset (or watchdog reset).

Ok, IDE layer will cope with any registers state during initialization, but
when not handled interrupt is generated we have spurious interrupt
which is some trouble for kernel.

Cheers
Stanislaw Gruszka
--
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