Re: [PATCH 1/2] libata: kill ATA_FLAG_SRST

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

 



Tejun Heo wrote:
As EDD support is removed, a LLD must implement either SRST or SATA
reset.  ATA_FLAG_SATA_RESET is enough to represent which one a LLD
implements.  Kill ATA_FLAG_SRST.

Note that both flags are already marked obsolete.  ATA_FLAG_SATA_RESET
will be removed too once all LLDs are converted to new probing
mechanism.  This partial removal is to make room for a new flag needed
for new PM.

Signed-off-by: Tejun Heo <htejun@xxxxxxxxx>

NAK. This will break too many drivers right now (e.g. even more in the #pata-drivers branch).

My suggestion would be to create a patch that converts as many LLDDs as you can to the new probing/EH. Ripping through the bmdma+PHY style controllers should be easy, for example.

Additionally, I would appreciate it if you would patch the #pata-drivers branch (in a separate patch), if you make a change across all libata LLDDs. Otherwise the pata-drivers branch will _rapidly_ fall behind, which is something that I don't want to happen.

Finally, please update Documentation/DocBook/libata.tmpl, at least for the ata_port_operations changes made by you. I can work on updating the EH and other sections myself, if you still dislike DocBook :) But I really want the libata driver API reference to stay current. (Alan grumbled at me about this, which I agree)

	Jeff



-
: 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