Re: [PATCH] pata_cmd64x: Set up MWDMA modes properly

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

 



>     That function rocks (except I didn't get what the address setup timings 
> mean to SW/MW DMA)...

They don't, it merges the DMA and PIO ones and you end up with the PIO
ones.

> > + *	Used when configuring the devices ot set the PIO timings. All the
> 
>     What's "ot"?

to - will fix

> >  	regD |= 0x20 << adev->devno;
> 
>     Hm, is setting of "drive DMA capable" bits really needed in the driver. 
> Doesn't libata core itself do this?

For most hardware it doesn't matter. The CMD64x driver code I have all
bothers to set it. I don't know if its needed, someone with the bits
handy could find out I guess, but if we arent sure it does no harm.
-
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