)]}' { "log": [ { "commit": "88fcd5627563722483427a55113c0a83f56e8080", "tree": "6a596e33d6497fbed58147e8fdb59b611d956c12", "parents": [ "071f44b1d2c051641b62a3571223314737ccbe59" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:22 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:25 2008 -0400" }, "message": "libata: make PMP support optional\n\nMake PMP support optional by adding CONFIG_SATA_PMP and leaving out\nlibata-pmp.c if it isn\u0027t set. PMP helpers return constant values if\nPMP support is not enabled and PMP declarations alias non-PMP\ncounterparts. This makes the compiler to leave out PMP related part\nout and LLDs to use non-PMP counterparts automatically.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "071f44b1d2c051641b62a3571223314737ccbe59", "tree": "a07794c8109e5d82a78223ae0159eadbf862c463", "parents": [ "48515f6c006c2a9d7b624ee8ad068018c2d3fe0e" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:22 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:25 2008 -0400" }, "message": "libata: implement PMP helpers\n\nImplement helpers to test whether PMP is supported, attached and\ndetermine pmp number to use when issuing SRST to a link. While at it,\nmove ata_is_host_link() so that it\u0027s together with the two new PMP\nhelpers.\n\nThis change simplifies LLDs and helps making PMP support optional.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "48515f6c006c2a9d7b624ee8ad068018c2d3fe0e", "tree": "0ade033552ccb297ac7736a6b2d77dd249be4eb8", "parents": [ "127102aea2ea9ec4e9ca233e2b1a75c8d3b058c4" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:21 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:25 2008 -0400" }, "message": "libata: separate PMP support code from core code\n\nMost of PMP support code is already in libata-pmp.c. All that are in\nlibata-core.c are sata_pmp_port_ops and EXPORTs. Move them to\nlibata-pmp.c. Also, collect PMP related prototypes and declarations\nin header files and move them right above of SFF stuff.\n\nThis change is to make PMP support optional.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "127102aea2ea9ec4e9ca233e2b1a75c8d3b058c4", "tree": "92fb528eac658adbc7307a009bd8a7558db2658a", "parents": [ "350756f6dab6d37ef9ed3f18dec520e88969ddac" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:21 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:24 2008 -0400" }, "message": "libata: make SFF support optional\n\nNow that SFF support is completely separated out from the core layer,\nit can be made optional. Add CONFIG_ATA_SFF and let SFF drivers\ndepend on it. If CONFIG_ATA_SFF isn\u0027t set, all codes in libata-sff.c\nand data structures for SFF support are disabled. This saves good\nnumber of bytes for small systems.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "c9f75b04ed5ed65a058d18a8a8dda50632a96de8", "tree": "e0100eae3b10006a8618ddb32bdf971e20db0dd8", "parents": [ "520d06f92b32d7abe5127d7cc46a819db0f384e6" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:21 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:24 2008 -0400" }, "message": "libata: kill ata_noop_dev_select()\n\nNow that SFF assumptions are separated out from non-SFF reset\nsequence, port_ops-\u003esff_dev_select() is no longer necessary for\nnon-SFF controllers. Kill ata_noop_dev_select() and -\u003esff_dev_select\ninitialization from base and other non-SFF port_ops.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "79f97dadfe9b4b561634d202225ba2fa910dc225", "tree": "7bea39a135c7c87b7430d6c071288a05990b9309", "parents": [ "22183bf569c8600ff414ac25f23134044e0ef453" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:20 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:23 2008 -0400" }, "message": "libata: drop @finish_qc from ata_qc_complete_multiple()\n\nata_qc_complete_multiple() took @finish_qc and called it on every qc\nbefore completing it. This was to give opportunity to update TF cache\nbefore ata_qc_complete() tries to fill result_tf. Now that result TF\nis a separate operation, this is no longer necessary.\n\nUpdate sata_sil24, which was the only user of this mechanism, such\nthat it implements its own ops-\u003eqc_fill_rtf() and drop @finish_qc from\nata_qc_complete_multiple().\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "22183bf569c8600ff414ac25f23134044e0ef453", "tree": "1dea79cdbe439637477e1c4214b600b6823cb866", "parents": [ "305d2a1ab137d11d573319c315748a87060fe82d" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:20 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:23 2008 -0400" }, "message": "libata: add qc_fill_rtf port operation\n\nOn command completion, ata_qc_complete() directly called ops-\u003etf_read\nto fill qc-\u003eresult_tf. This patch adds ops-\u003eqc_fill_rtf to replace\nhardcoded ops-\u003etf_read usage.\n\nata_sff_qc_fill_rtf() which uses ops-\u003etf_read to fill result_tf is\nimplemented and set in ata_base_port_ops and other ops tables which\ndon\u0027t inherit from ata_base_port_ops, so this patch doesn\u0027t introduce\nany behavior change.\n\nops-\u003eqc_fill_rtf() is similar to ops-\u003esff_tf_read() but can only be\ncalled when a command finishes. As some non-SFF controllers don\u0027t\nhave TF registers defined unless they\u0027re associated with in-flight\ncommands, this limited operation makes life easier for those drivers\nand help lifting SFF assumptions from libata core layer.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "5958e3025fd9d97429163e074d9cfa3848f51f28", "tree": "f20492dc35fc2651452c871fbc812bd4f8379b40", "parents": [ "ac371987a81c61c2efbd6931245cdcaf43baad89" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:20 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:23 2008 -0400" }, "message": "libata: move PMP SCR access failure during reset to ata_eh_reset()\n\nIf PMP fan-out reset fails and SCR isn\u0027t accessible, PMP should be\nreset. This used to be tested by sata_pmp_std_hardreset() and\ncommunicated to EH by -ERESTART. However, this logic is generic and\ndoesn\u0027t really have much to do with specific hardreset implementation.\n\nThis patch moves SCR access failure detection logic to ata_eh_reset()\nwhere it belongs. As this makes sata_pmp_std_hardreset() identical to\nsata_std_hardreset(), the function is killed and replaced with the\nstandard method.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "ac371987a81c61c2efbd6931245cdcaf43baad89", "tree": "f88970931b26d2ad344d7d67ddabc64d9b48181d", "parents": [ "57c9efdfb3cee5d4564fcb5f70555e2edb1bc52a" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:19 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:23 2008 -0400" }, "message": "libata: clear SError after link resume\n\nSError used to be cleared in -\u003epostreset. This has small hotplug race\ncondition. If a device is plugged in after reset is complete but\npostreset hasn\u0027t run yet, its hotplug event gets lost when SError is\ncleared. This patch makes sata_link_resume() clear SError. This\nkills the race condition and makes a lot of sense as some PMP and host\nPHYs don\u0027t work properly without SError cleared.\n\nThis change makes sata_pmp_std_{pre|post}_reset()\u0027s unnecessary as\nthey become identical to ata_std counterparts. It also simplifies\nsata_pmp_hardreset() and ahci_vt8251_hardreset().\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "57c9efdfb3cee5d4564fcb5f70555e2edb1bc52a", "tree": "c2289500f093736853a94d2d9577036658676498", "parents": [ "9dadd45b24145d6aee2fabb28d7aef972301892b" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:19 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:23 2008 -0400" }, "message": "libata: implement and use sata_std_hardreset()\n\nImplement sata_std_hardreset(), which simply wraps around\nsata_link_hardreset(). sata_std_hardreset() becomes new standard\nhardreset method for sata_port_ops and sata_sff_hardreset() moves from\nata_base_port_ops to ata_sff_port_ops, which is where it really\nbelongs.\n\nata_is_builtin_hardreset() is added so that both\nata_std_error_handler() and ata_sff_error_handler() skip both builtin\nhardresets if SCR isn\u0027t accessible.\n\npiix_sidpr_hardreset() in ata_piix.c is identical to\nsata_std_hardreset() in functionality and got replaced with the\nstandard function.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "9dadd45b24145d6aee2fabb28d7aef972301892b", "tree": "c97c323e2edd400bc94eaceddf20f84e9a6da005", "parents": [ "a89611e8489ac24f371c9fd6fef6605b170b16ba" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:19 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:22 2008 -0400" }, "message": "libata: move generic hardreset code from sata_sff_hardreset() to sata_link_hardreset()\n\nsata_sff_hardreset() contains link readiness wait logic which isn\u0027t\nSFF specific. Move that part into sata_link_hardreset(), which now\ntakes two more parameters - @online and @check_ready. Both are\noptional. The former is out parameter for link onlineness after\nreset. The latter is used to wait for link readiness after hardreset.\n\nUsers of sata_link_hardreset() is updated to use new funtionality and\nahci_hardreset() is updated to use sata_link_hardreset() instead of\nsata_sff_hardreset(). This doesn\u0027t really cause any behavior change.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "aa2731ad9ad80ac3fca48bd1c4cf0eceede4810e", "tree": "db6d3327017b9b84355aba49a5ccd5c322d3f248", "parents": [ "705e76beb90b97421e1f61e857c4246799781bb5" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:19 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:22 2008 -0400" }, "message": "libata: separate out ata_wait_ready() and implement ata_wait_after_reset()\n\nFactor out waiting logic (which is common to all ATA controllers) from\nata_sff_wait_ready() into ata_wait_ready(). ata_wait_ready() takes\n@check_ready function pointer and uses it to poll for readiness. This\nallows non-SFF controllers to use ata_wait_ready() to wait for link\nreadiness.\n\nThis patch also implements ata_wait_after_reset() - generic version of\nata_sff_wait_after_reset() - using ata_wait_ready().\n\nata_sff_wait_ready() is reimplemented using ata_wait_ready() and\nata_sff_check_ready(). Functionality remains the same.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "705e76beb90b97421e1f61e857c4246799781bb5", "tree": "e571ad9229d469cd73d1388c76823922400823d5", "parents": [ "203c75b8245c5386044721d9c5eda5c6b71b3d14" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:19 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:22 2008 -0400" }, "message": "libata: restructure SFF post-reset readiness waits\n\nPreviously, post-softreset readiness is waited as follows.\n\n1. ata_sff_wait_after_reset() waits for 150ms and then for\n ATA_TMOUT_FF_WAIT if status is 0xff and other conditions meet.\n\n2. ata_bus_softreset() finishes with -ENODEV if status is still 0xff.\n If not, continue to #3.\n\n3. ata_bus_post_reset() waits readiness of dev0 and/or dev1 depending\n on devmask using ata_sff_wait_ready().\n\nAnd for post-hardreset readiness,\n\n1. ata_sff_wait_after_reset() waits for 150ms and then for\n ATA_TMOUT_FF_WAIT if status is 0xff and other conditions meet.\n\n2. sata_sff_hardreset waits for device readiness using\n ata_sff_wait_ready().\n\nThis patch merges and unifies post-reset readiness waits into\nata_sff_wait_ready() and ata_sff_wait_after_reset().\n\nATA_TMOUT_FF_WAIT handling is merged into ata_sff_wait_ready(). If TF\nstatus is 0xff, link status is unknown and the port is SATA, it will\ncontinue polling till ATA_TMOUT_FF_WAIT.\n\nata_sff_wait_after_reset() is updated to perform the following steps.\n\n1. waits for 150ms.\n\n2. waits for dev0 readiness using ata_sff_wait_ready(). Note that\n this is done regardless of devmask, as ata_sff_wait_ready() handles\n 0xff status correctly, this preserves the original behavior except\n that it may wait longer after softreset if link is online but\n status is 0xff. This behavior change is very unlikely to cause any\n actual difference and is intended. It brings softreset behavior to\n that of hardreset.\n\n3. waits for dev1 readiness just the same way ata_bus_post_reset() did.\n\nNow both soft and hard resets call ata_sff_wait_after_reset() after\nreset to wait for readiness after resets. As\nata_sff_wait_after_reset() contains calls to -\u003esff_dev_select(),\nexplicit call near the end of sata_sff_hardreset() is removed.\n\nThis change makes reset implementation simpler and more consistent.\n\nWhile at it, make the magical 150ms wait post-reset wait duration a\nconstant and ata_sff_wait_ready() and ata_sff_wait_after_reset() take\n@link instead of @ap. This is to make them consistent with other\nreset helpers and ease core changes.\n\npata_scc is updated accordingly.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "203c75b8245c5386044721d9c5eda5c6b71b3d14", "tree": "5a8c446c483a77dc86aca145b0b38c4a2b410dfa", "parents": [ "0aa1113d544226bc2c4a20d6ac1d71170512a361" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:18 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:22 2008 -0400" }, "message": "libata: separate out ata_std_postreset() from ata_sff_postreset()\n\nSeparate out generic ATA portion from ata_sff_postreset() into\nata_std_postreset() and implement ata_sff_postreset() using the std\nversion.\n\nata_base_port_ops now has ata_std_postreset() for its postreset and\nata_sff_port_ops overrides it to ata_sff_postreset().\n\nThis change affects pdc_adma, ahci, sata_fsl and sata_sil24. pdc_adma\nnow specifies postreset to ata_sff_postreset() explicitly. sata_fsl\nand sata_sil24 now use ata_std_postreset() which makes no difference\nto them. ahci now calls ata_std_postreset() from its own postreset\nmethod, which causes no behavior difference.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "0aa1113d544226bc2c4a20d6ac1d71170512a361", "tree": "503b33b0805424d312abddd3535c941bb85c03bc", "parents": [ "288623a06c652239d2f57d271af12bb024cf7218" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:18 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:22 2008 -0400" }, "message": "libata: separate out ata_std_prereset() from ata_sff_prereset()\n\nSeparate out generic ATA portion from ata_sff_prereset() into\nata_std_prereset() and implement ata_sff_prereset() using the std\nversion. Waiting for device readiness is the only SFF specific part.\n\nata_base_port_ops now has ata_std_prereset() for its prereset and\nata_sff_port_ops overrides it to ata_sff_prereset(). This change can\naffect pdc_adma, ahci, sata_fsl and sata_sil24. pdc_adma implements\nits own prereset using ata_sff_prereset() and the rest has hardreset\nand thus are unaffected by this change.\n\nThis change reflects real world situation. There is no generic way to\nwait for device readiness for non-SFF controllers and some of them\ndon\u0027t have any mechanism for that. Non-sff drivers which don\u0027t have\nhardreset should wrap ata_std_prereset() and wait for device readiness\nitself but there\u0027s no such driver now and isn\u0027t likely to be popular\nin the future either.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "288623a06c652239d2f57d271af12bb024cf7218", "tree": "e693ca8a46cbedc522cb6f5ca3395d3dc692cfb3", "parents": [ "5682ed33aae05d10a25c95633ef9d9c062825888" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:17 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:22 2008 -0400" }, "message": "libata: clean up port_ops-\u003esff_irq_clear()\n\n-\u003esff_irq_clear() is called only from SFF interrupt handler, so there\nis no reason to initialize it for non-SFF controllers. Also,\nata_sff_irq_clear() can handle both BMDMA and non-BMDMA SFF\ncontrollers.\n\nThis patch kills ata_noop_irq_clear() and removes it from base\nport_ops and sets -\u003esff_irq_clear to ata_sff_irq_clear() in sff\nport_ops instead of bmdma port_ops.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "5682ed33aae05d10a25c95633ef9d9c062825888", "tree": "1632d4d70f4fd2dc25596a5cde1183f70f162ac3", "parents": [ "9363c3825ea9ad76561eb48a395349dd29211ed6" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:16 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:22 2008 -0400" }, "message": "libata: rename SFF port ops\n\nAdd sff_ prefix to SFF specific port ops.\n\nThis rename is in preparation of separating SFF support out of libata\ncore layer. This patch strictly renames ops and doesn\u0027t introduce any\nbehavior difference.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "9363c3825ea9ad76561eb48a395349dd29211ed6", "tree": "abe89a0f7c82b805d84b1a211c97b317f6628d5f", "parents": [ "b67a1064cb1c1d3b43e01e8b43a6a8dcdefed733" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:16 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:21 2008 -0400" }, "message": "libata: rename SFF functions\n\nSFF functions have confusing names. Some have sff prefix, some have\nbmdma, some std, some pci and some none. Unify the naming by...\n\n* SFF functions which are common to both BMDMA and non-BMDMA are\n prefixed with ata_sff_.\n\n* SFF functions which are specific to BMDMA are prefixed with\n ata_bmdma_.\n\n* SFF functions which are specific to PCI but apply to both BMDMA and\n non-BMDMA are prefixed with ata_pci_sff_.\n\n* SFF functions which are specific to PCI and BMDMA are prefixed with\n ata_pci_bmdma_.\n\n* Drop generic prefixes from LLD specific routines. For example,\n bfin_std_dev_select -\u003e bfin_dev_select.\n\nThe following renames are noteworthy.\n\n ata_qc_issue_prot() -\u003e ata_sff_qc_issue()\n ata_pci_default_filter() -\u003e ata_bmdma_mode_filter()\n ata_dev_try_classify() -\u003e ata_sff_dev_classify()\n\nThis rename is in preparation of separating SFF support out of libata\ncore layer. This patch strictly renames functions and doesn\u0027t\nintroduce any behavior difference.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "83c063dd730cb56bf3fc89b70250ff9a398fec1e", "tree": "4447914e4957f678a3085d45824c132dfc5c8b8b", "parents": [ "c30484d775eb0b899f2b6a8f5508a99ecaba59fc" ], "author": { "name": "Yoichi Yuasa", "email": "yoichi_yuasa@tripeaks.co.jp", "time": "Thu Feb 28 21:43:13 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:21 2008 -0400" }, "message": "use ATA_TAG_INTERNAL in ata_tag_internal()\n\nIt should be ATA_TAG_INTERNAL.\n\nSigned-off-by: Yoichi Yuasa \u003cyoichi_yuasa@tripeaks.co.jp\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "03faab7827e4e45823fd27c47b84c133e20a0cd0", "tree": "046b04c7e650f5318beb330d70e855a889750154", "parents": [ "a5987e0a1b569146ed9cfa0a8c275a21b344fcaa" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Thu Mar 27 19:14:24 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:20 2008 -0400" }, "message": "libata: implement ATA_QCFLAG_RETRY\n\nCurrently whether a command should be retried after failure is\ndetermined inside ata_eh_finish(). Add ATA_QCFLAG_RETRY and move the\nlogic into ata_eh_autopsy(). This makes things clearer and helps\nextending retry determination logic.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "6fd36390117f7844ad147377878ddb52088f583a", "tree": "c25d9a8357ff173ce46eb057cc50370b87b9cca8", "parents": [ "3d5a3d67a578d95ac3a11a862d16a054e2d4d9cd" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Mar 25 22:16:44 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:18 2008 -0400" }, "message": "libata: kill ata_chk_status()\n\nata_chk_status() just calls ops-\u003echeck_status and it only adds\nconfusion with other status functions. Kill it.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "071ce34d57924edb76b76f7de460eb4991463959", "tree": "10400b05bd2649fe25a8c3aa4d41c08bd29c6347", "parents": [ "624d5c514eed18d5a93062e9d86d67065175f30a" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Mar 25 22:16:42 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:18 2008 -0400" }, "message": "libata: move ata_pci_default_filter() out of CONFIG_PCI\n\nata_pci_default_filter() doesn\u0027t really have anything to do with PCI.\nIt\u0027s generally applicable to BMDMA controllers. Move it out of\nCONFIG_PCI.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "624d5c514eed18d5a93062e9d86d67065175f30a", "tree": "61f89b05b70e9b443cf09083b4ae3d102dd357b7", "parents": [ "272f7884e8c0effe594e5537092b9c0ccc0140b0" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Mar 25 22:16:41 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:18 2008 -0400" }, "message": "libata: reorganize SFF related stuff\n\n* Move SFF related functions from libata-core.c to libata-sff.c.\n\n ata_[bmdma_]sff_port_ops, ata_devchk(), ata_dev_try_classify(),\n ata_std_dev_select(), ata_tf_to_host(), ata_busy_sleep(),\n ata_wait_after_reset(), ata_wait_ready(), ata_bus_post_reset(),\n ata_bus_softreset(), ata_bus_reset(), ata_std_softreset(),\n sata_std_hardreset(), ata_fill_sg(), ata_fill_sg_dumb(),\n ata_qc_prep(), ata_dump_qc_prep(), ata_data_xfer(),\n ata_data_xfer_noirq(), ata_pio_sector(), ata_pio_sectors(),\n atapi_send_cdb(), __atapi_pio_bytes(), atapi_pio_bytes(),\n ata_hsm_ok_in_wq(), ata_hsm_qc_complete(), ata_hsm_move(),\n ata_pio_task(), ata_qc_issue_prot(), ata_host_intr(),\n ata_interrupt(), ata_std_ports()\n\n* Make ata_pio_queue_task() global as it\u0027s now called from\n libata-sff.c.\n\n* Move SFF related stuff in include/linux/libata.h and\n drivers/ata/libata.h into one place. While at it, move timing\n constants into the global enum definition and fortify comments a\n bit.\n\nThis patch strictly moves stuff around and as such doesn\u0027t cause any\nfunctional difference.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "a1efdaba2dbd6fb89e23a87b66d3f4dd92c9f5af", "tree": "6197c537892e0d887b2a90e369b74abf0500b9ac", "parents": [ "959471936241bd83da7d0a76411cef6772140fe6" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Mar 25 12:22:50 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:18 2008 -0400" }, "message": "libata: make reset related methods proper port operations\n\nCurrently reset methods are not specified directly in the\nata_port_operations table. If a LLD wants to use custom reset\nmethods, it should construct and use a error_handler which uses those\nreset methods. It\u0027s done this way for two reasons.\n\nFirst, the ops table already contained too many methods and adding\nfour more of them would noticeably increase the amount of necessary\nboilerplate code all over low level drivers.\n\nSecond, as -\u003eerror_handler uses those reset methods, it can get\nconfusing. ie. By overriding -\u003eerror_handler, those reset ops can be\nmade useless making layering a bit hazy.\n\nNow that ops table uses inheritance, the first problem doesn\u0027t exist\nanymore. The second isn\u0027t completely solved but is relieved by\nproviding default values - most drivers can just override what it has\nimplemented and don\u0027t have to concern itself about higher level\ncallbacks. In fact, there currently is no driver which actually\nmodifies error handling behavior. Drivers which override\n-\u003eerror_handler just wraps the standard error handler only to prepare\nthe controller for EH. I don\u0027t think making ops layering strict has\nany noticeable benefit.\n\nThis patch makes -\u003eprereset, -\u003esoftreset, -\u003ehardreset, -\u003epostreset and\ntheir PMP counterparts propoer ops. Default ops are provided in the\nbase ops tables and drivers are converted to override individual reset\nmethods instead of creating custom error_handler.\n\n* ata_std_error_handler() doesn\u0027t use sata_std_hardreset() if SCRs\n aren\u0027t accessible. sata_promise doesn\u0027t need to use separate\n error_handlers for PATA and SATA anymore.\n\n* softreset is broken for sata_inic162x and sata_sx4. As libata now\n always prefers hardreset, this doesn\u0027t really matter but the ops are\n forced to NULL using ATA_OP_NULL for documentation purpose.\n\n* pata_hpt374 needs to use different prereset for the first and second\n PCI functions. This used to be done by branching from\n hpt374_error_handler(). The proper way to do this is to use\n separate ops and port_info tables for each function. Converted.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "959471936241bd83da7d0a76411cef6772140fe6", "tree": "abccf1d3931c17b581ee2c85cda05a603adcfb0f", "parents": [ "887125e3740283be25564bfc6fb5d24974b651ab" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Mar 25 12:22:49 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:17 2008 -0400" }, "message": "libata: kill port_info-\u003esht and -\u003eirq_handler\n\nlibata core layer doesn\u0027t care about sht or -\u003eirq_handler. Those are\nonly of interest to the LLD during initialization. This is confusing\nand has caused several drivers to have duplicate unused initializers\nfor these fields.\n\nCurrently only sata_nv uses these fields. Make sata_nv use\n-\u003eprivate_data, which is supposed to carry LLD-specific information,\ninstead and kill -\u003esht and -\u003eirq_handler. nv_pi_priv structure is\ndefined and struct literals are used to initialize private_data.\nNotational overhead is negligible.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "887125e3740283be25564bfc6fb5d24974b651ab", "tree": "bd037e1c17a468fabe1834e707b70b4b43513e4a", "parents": [ "1bd5b715a305f6f13455e89becbd839010dd14b5" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Mar 25 12:22:49 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:17 2008 -0400" }, "message": "libata: stop overloading port_info-\u003eprivate_data\n\nport_info-\u003eprivate_data is currently used for two purposes - to record\nprivate data about the port_info or to specify host-\u003eprivate_data to\nuse when allocating ata_host.\n\nThis overloading is confusing and counter-intuitive in that\nport_info-\u003eprivate_data becomes host-\u003eprivate_data instead of\nport-\u003eprivate_data. In addition, port_info and host don\u0027t correspond\nto each other 1-to-1. Currently, the first non-NULL\nport_info-\u003eprivate_data is used.\n\nThis patch makes port_info-\u003eprivate_data just be what it is -\nprivate_data for the port_info where LLD can jot down extra info.\nlibata no longer sets host-\u003eprivate_data to the first non-NULL\nport_info-\u003eprivate_data, @host_priv argument is added to\nata_pci_init_one() instead. LLDs which use ata_pci_init_one() can use\nthis argument to pass in pointer to host private data. LLDs which\ndon\u0027t should use init-register model anyway and can initialize\nhost-\u003eprivate_data directly.\n\nAdding @host_priv instead of using init-register model for LLDs which\nuse ata_pci_init_one() is suggested by Alan Cox.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nCc: Alan Cox \u003calan@lxorguk.ukuu.org.uk\u003e\n" }, { "commit": "1bd5b715a305f6f13455e89becbd839010dd14b5", "tree": "22e74b38495c51cda92145b7ae31f504f80d9fd3", "parents": [ "029cfd6b74fc5c517865fad78cf4a3ea8d9b664a" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Mar 25 12:22:49 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:17 2008 -0400" }, "message": "libata: make ata_pci_init_one() not use ops-\u003eirq_handler and pi-\u003esht\n\nata_pci_init_one() is the only function which uses ops-\u003eirq_handler\nand pi-\u003esht. Other initialization functions take the same information\nas arguments. This causes confusion and duplicate unused entries in\nstructures.\n\nMake ata_pci_init_one() take sht as an argument and use ata_interrupt\nimplicitly. All current users use ata_interrupt and if different irq\nhandler is necessary open coding ata_pci_init_one() using\nata_prepare_sff_host() and ata_activate_sff_host can be done under ten\nlines including error handling and driver which requires custom\ninterrupt handler is likely to require custom initialization anyway.\n\nAs ata_pci_init_one() was the last user of ops-\u003eirq_handler, this\npatch also kills the field.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "029cfd6b74fc5c517865fad78cf4a3ea8d9b664a", "tree": "4a40f44f29321e433497a51d2f6cfe1922ae1d58", "parents": [ "68d1d07b510bb57a504588adc2bd2758adea0965" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Mar 25 12:22:49 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:17 2008 -0400" }, "message": "libata: implement and use ops inheritance\n\nlibata lets low level drivers build ata_port_operations table and\nregister it with libata core layer. This allows low level drivers\nhigh level of flexibility but also burdens them with lots of\nboilerplate entries.\n\nThis becomes worse for drivers which support related similar\ncontrollers which differ slightly. They share most of the operations\nexcept for a few. However, the driver still needs to list all\noperations for each variant. This results in large number of\nduplicate entries, which is not only inefficient but also error-prone\nas it becomes very difficult to tell what the actual differences are.\n\nThis duplicate boilerplates all over the low level drivers also make\nupdating the core layer exteremely difficult and error-prone. When\ncompounded with multi-branched development model, it ends up\naccumulating inconsistencies over time. Some of those inconsistencies\ncause immediate problems and fixed. Others just remain there dormant\nmaking maintenance increasingly difficult.\n\nTo rectify the problem, this patch implements ata_port_operations\ninheritance. To allow LLDs to easily re-use their own ops tables\noverriding only specific methods, this patch implements poor man\u0027s\nclass inheritance. An ops table has -\u003einherits field which can be set\nto any ops table as long as it doesn\u0027t create a loop. When the host\nis started, the inheritance chain is followed and any operation which\nisn\u0027t specified is taken from the nearest ancestor which has it\nspecified. This operation is called finalization and done only once\nper an ops table and the LLD doesn\u0027t have to do anything special about\nit other than making the ops table non-const such that libata can\nupdate it.\n\nlibata provides four base ops tables lower drivers can inherit from -\nbase, sata, pmp, sff and bmdma. To avoid overriding these ops\naccidentaly, these ops are declared const and LLDs should always\ninherit these instead of using them directly.\n\nAfter finalization, all the ops table are identical before and after\nthe patch except for setting .irq_handler to ata_interrupt in drivers\nwhich didn\u0027t use to. The .irq_handler doesn\u0027t have any actual effect\nand the field will soon be removed by later patch.\n\n* sata_sx4 is still using old style EH and currently doesn\u0027t take\n advantage of ops inheritance.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "68d1d07b510bb57a504588adc2bd2758adea0965", "tree": "94e8788a8ca40017b33044329f98bbe6552cc526", "parents": [ "6bd99b4e0998571808fc1f09d5162348f21ff8c1" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Mar 25 12:22:49 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:17 2008 -0400" }, "message": "libata: implement and use SHT initializers\n\nlibata lets low level drivers build scsi_host_template and register it\nto the SCSI layer. This allows low level drivers high level of\nflexibility but also burdens them with lots of boilerplate entries.\n\nThis patch implements SHT initializers which can be used to initialize\nall the boilerplate entries in a sht. Three variants of them are\nimplemented - BASE, BMDMA and NCQ - for different types of drivers.\nNote that entries can be overriden by putting individual initializers\nafter the helper macro.\n\nAll sht tables are identical before and after this patch.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "358f9a77a668660729e705fde9c3cf69f013aa98", "tree": "a249660fdd9f252925eff9eaac3ed7836c839f91", "parents": [ "c1bc899f5805771926c9198e2ab4d77122c356a1" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Mar 25 12:22:47 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:16 2008 -0400" }, "message": "libata: implement and use ata_noop_irq_clear()\n\n-\u003eirq_clear() is used to clear IRQ bit of a SFF controller and isn\u0027t\nuseful for drivers which don\u0027t use libata SFF HSM implementation.\nHowever, it\u0027s a required callback and many drivers implement their own\nnoop version as placeholder. This patch implements ata_noop_irq_clear\nand use it to replace those custom placeholders.\n\nAlso, SFF drivers which don\u0027t support BMDMA don\u0027t need to use\nata_bmdma_irq_clear(). It becomes noop if BMDMA address isn\u0027t\ninitialized. Convert them to use ata_noop_irq_clear().\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "c1bc899f5805771926c9198e2ab4d77122c356a1", "tree": "008dc8c45d55477ba3fc7a9c9abe474ecb5027af", "parents": [ "f08048e94564d009b19038cfbdd800aa83e79c7f" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Mar 25 12:22:47 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:16 2008 -0400" }, "message": "libata: reorganize ata_port_operations\n\nOver the time, ops in ata_port_operations has become a bit confusing.\nReorganize. SFF/BMDMA ops are separated into separate a group as they\nwill be taken out of ata_port_operations later.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "b558edddb1c42c70a30cfe494984d4be409f7b2b", "tree": "b03275d15675a519de7d4edde709892f885a5fa0", "parents": [ "8cebf274dd1c955a6e03385a85fd6569ce445946" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Thu Jan 24 00:05:14 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:16 2008 -0400" }, "message": "libata: kill ata_ehi_schedule_probe()\n\nata_ehi_schedule_probe() was created to hide details of link-resuming\nreset magic. Now that all the softreset workarounds are gone,\nscheduling probe is very simple - set probe_mask and request RESET.\nKill ata_ehi_schedule_probe() and open code it. This also increases\nconsistency as ata_ehi_schedule_probe() couldn\u0027t cover individual\ndevice probings so they were open-coded even when the helper existed.\n\nWhile at it, define ATA_ALL_DEVICES as mask of all possible devices on\na link and always use it when requesting probe on link level for\nsimplicity and consistency. Setting extra bits in the probe_mask\ndoesn\u0027t hurt anybody.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "8cebf274dd1c955a6e03385a85fd6569ce445946", "tree": "c459cd20ef93413e063826bb99ca0bcaf67ec8d2", "parents": [ "672b2d65ba83a6f3f801fd3d58851aa9c0725a54" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Thu Jan 24 00:05:14 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:16 2008 -0400" }, "message": "libata: kill ATA_LFLAG_SKIP_D2H_BSY\n\nSome controllers can\u0027t reliably record the initial D2H FIS after SATA\nlink is brought online for whatever reason. Advanced controllers\nwhich don\u0027t have traditional TF register based interface often have\nthis problem as they don\u0027t really have the TF registers to update\nwhile the controller and link are being initialized.\n\nSKIP_D2H_BSY works around the problem by skipping the wait for device\nreadiness before issuing SRST, so for such controllers libata issues\nSRST blindly and hopes for the best.\n\nNow that libata defaults to hardreset, this workaround is no longer\nnecessary. For controllers which have support for hardreset, SRST is\nnever issued by itself. It is only issued as follow-up SRST for\ndevice classification and PMP initialization, so there\u0027s no need to\nwait for it from prereset.\n\nKill ATA_LFLAG_SKIP_D2H_BSY.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "672b2d65ba83a6f3f801fd3d58851aa9c0725a54", "tree": "ed3a2fe8013bf8322df637838deccf9f2bc9c92c", "parents": [ "d692abd92f22a81b38d52c39601871003262841c" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Thu Jan 24 00:05:14 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:16 2008 -0400" }, "message": "libata: kill ATA_EHI_RESUME_LINK\n\nATA_EHI_RESUME_LINK has two functions - promote reset to hardreset if\nATA_LFLAG_HRST_TO_RESUME is set and preventing EH from shortcutting\nreset action when probing is requested. The former is gone now and\nthe latter can easily be achieved by making EH to perform at least one\nreset if reset is requested, which also makes more sense than\ndepending on RESUME_LINK flag.\n\nAs ATA_EHI_RESUME_LINK was the only EHI reset modifier, this also\nkills reset modifier handling.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "d692abd92f22a81b38d52c39601871003262841c", "tree": "4ff3f35d88762a52fb4d48adda1f39e1966b9f26", "parents": [ "cf48062658e7ab3bc55e10c65676c3c73c16f8bf" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Thu Jan 24 00:05:14 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:15 2008 -0400" }, "message": "libata: kill ATA_LFLAG_HRST_TO_RESUME\n\nNow that hardreset is the preferred method of resetting, there\u0027s no\nneed for ATA_LFLAG_HRST_TO_RESUME flag. Kill it.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "cf48062658e7ab3bc55e10c65676c3c73c16f8bf", "tree": "9a50a016ceb6051d8a30a43ea39f2a4d3d25570e", "parents": [ "4b119e21d0c66c22e8ca03df05d9de623d0eb50f" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Thu Jan 24 00:05:14 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:15 2008 -0400" }, "message": "libata: prefer hardreset\n\nWhen both soft and hard resets are available, libata preferred\nsoftreset till now. The logic behind it was to be softer to devices;\nhowever, this doesn\u0027t really help much. Rationales for the change:\n\n* BIOS may freeze lock certain things during boot and softreset can\u0027t\n unlock those. This by itself is okay but during operation PHY event\n or other error conditions can trigger hardreset and the device may\n end up with different configuration.\n\n For example, after a hardreset, previously unlockable HPA can be\n unlocked resulting in different device size and thus revalidation\n failure. Similar condition can occur during or after resume.\n\n* Certain ATAPI devices require hardreset to recover after certain\n error conditions. On PATA, this is done by issuing the DEVICE RESET\n command. On SATA, COMRESET has equivalent effect. The problem is\n that DEVICE RESET needs its own execution protocol.\n\n For SFF controllers with bare TF access, it can be easily\n implemented but more advanced controllers (e.g. ahci and sata_sil24)\n require specialized implementations. Simply using hardreset solves\n the problem nicely.\n\n* COMRESET initialization sequence is the norm in SATA land and many\n SATA devices don\u0027t work properly if only SRST is used. For example,\n some PMPs behave this way and libata works around by always issuing\n hardreset if the host supports PMP.\n\n Like the above example, libata has developed a number of mechanisms\n aiming to promote softreset to hardreset if softreset is not going\n to work. This approach is time consuming and error prone.\n\n Also, note that, dependingon how you read the specs, it could be\n argued that PMP fan-out ports require COMRESET to start operation.\n In fact, all the PMPs on the market except one don\u0027t work properly\n if COMRESET is not issued to fan-out ports after PMP reset.\n\n* COMRESET is an integral part of SATA connection and any working\n device should be able to handle COMRESET properly. After all, it\u0027s\n the way to signal hardreset during reboot. This is the most used\n and recommended (at least by the ahci spec) method of resetting\n devices.\n\nSo, this patch makes libata prefer hardreset over softreset by making\nthe following changes.\n\n* Rename ATA_EH_RESET_MASK to ATA_EH_RESET and use it whereever\n ATA_EH_{SOFT|HARD}RESET used to be used. ATA_EH_{SOFT|HARD}RESET is\n now only used to tell prereset whether soft or hard reset will be\n issued.\n\n* Strip out now unneeded promote-to-hardreset logics from\n ata_eh_reset(), ata_std_prereset(), sata_pmp_std_prereset() and\n other places.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "e52dcc4899cf1b7601379c31542bd91cd2997a64", "tree": "40bce50f8f912bbd2d988526f3f5c5763f67af62", "parents": [ "436d34b36202ef724778ded1e9cb10f8c37b32bc" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Wed Apr 02 17:35:19 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Apr 04 02:43:36 2008 -0400" }, "message": "libata: ATA_12/16 doesn\u0027t fall into ATAPI_MISC\n\nSAT passthrus don\u0027t really fit into ATAPI_MISC class. SAT passthru\ncommands always transfer multiple of 512 bytes and variable length\nresponse is not allowed. This patch creates a separate category -\nATAPI_PASS_THRU - for these.\n\nThis fixes HSM violation on \"hdparm -I\".\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "436d34b36202ef724778ded1e9cb10f8c37b32bc", "tree": "45671994cf049263a25e7f540ae488b97bb7d01f", "parents": [ "a4ba7fe2a6c2b61419b290035bff398ab2591c54" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Wed Apr 02 17:28:46 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Apr 04 02:43:35 2008 -0400" }, "message": "libata: uninline atapi_cmd_type()\n\nUninline atapi_cmd_type(). It doesn\u0027t really have to be inline and\nmore case will be added which need to access unexported libata\nvariable.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "3ec25ebd69dc120d0590e64caaf1477aa88c8a93", "tree": "e0ea13a1f1e1fb5b2fa158fdd2f5122d09ae1d7f", "parents": [ "0f436eff54f90419ac1b8accfb3e6e17c4b49a4e" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Thu Mar 27 18:37:14 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Sat Mar 29 12:21:31 2008 -0400" }, "message": "libata: ATA_EHI_LPM should be ATA_EH_LPM\n\nEH actions are ATA_EH_* not ATA_EHI_*. Rename ATA_EHI_LPM to\nATA_EH_LPM.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nCc: Kristen Carlson Accardi \u003ckristen.c.accardi@intel.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "aacda37538e7f9cf2148eedf3766239829e51ba4", "tree": "0206d2db8ce636aadabf8e3effd1ee3114ccff2a", "parents": [ "cc7feea39bed2951cc29af3ad642f39a99dfe8d3" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Mar 18 17:47:43 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Mon Mar 24 22:09:12 2008 -0400" }, "message": "libata: implement ata_qc_raw_nbytes()\n\nImplement ata_qc_raw_nbytes() which determines the raw user-requested\nsize of a PC command.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "9116300634c5c76cfcd0d2af689846e04d172256", "tree": "dfc527d28af35f19dcb9c4092757d6eabb8afb1d", "parents": [ "559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Thu Feb 21 13:25:50 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Sun Feb 24 00:27:33 2008 -0500" }, "message": "libata: automatically use DMADIR if drive/bridge requires it\n\nBack in 2.6.17-rc2, a libata module parameter was added for atapi_dmadir.\n\nThat\u0027s nice, but most SATA devices which need it will tell us about it\nin their IDENTIFY PACKET response, as bit-15 of word-62 of the\nreturned data (as per ATA7, ATA8 specifications).\n\nSo for those which specify it, we should automatically use the DMADIR bit.\nOtherwise, disc writing will fail by default on many SATA-ATAPI drives.\n\nThis patch adds ATA_DFLAG_DMADIR and make ata_dev_configure() set it\nif atapi_dmadir is set or identify data indicates DMADIR is necessary.\natapi_xlat() is converted to check ATA_DFLAG_DMADIR before setting\nDMADIR.\n\nOriginal patch is from Mark Lord.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nCc: Mark Lord \u003cmlord@pobox.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "5d44b414daa8c1d8551aed6130d86d54175db43f", "tree": "8d43b2f5fcdd228ecbf5c7e3fc550d49649ba84d", "parents": [ "332673257056d8be8a69d759eda90a799af5472d" ], "author": { "name": "Harvey Harrison", "email": "harvey.harrison@gmail.com", "time": "Fri Feb 15 13:41:32 2008 -0800" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Feb 20 12:12:47 2008 -0500" }, "message": "ata: fix sparse warning in libata.h\n\nAvoids lots of these, also is more readable.\ninclude/linux/libata.h:1210:13: warning: potentially expensive pointer subtraction\n\nChange the subtraction to addition on the other side of the comparison.\n\nThanks to Christer Weinigel for the suggestion.\n\nSigned-off-by: Harvey Harrison \u003charvey.harrison@gmail.com\u003e\nAcked-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "dde2020754aeb14e17052d61784dcb37f252aac2", "tree": "1b6d57c6eff2024fd13e4b3b115d0a6770d8cb80", "parents": [ "db0a2e0099be3a1cff55879225881465f16c67d3" ], "author": { "name": "James Bottomley", "email": "James.Bottomley@HansenPartnership.com", "time": "Tue Feb 19 11:36:56 2008 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Feb 19 11:36:56 2008 +0100" }, "message": "libata: eliminate the home grown dma padding in favour of\n\nthat provided by the block layer\n\nATA requires that all DMA transfers begin and end on word boundaries.\nBecause of this, a large amount of machinery grew up in ide to adjust\nscatterlists on this basis. However, as of 2.5, the block layer has a\ndma_alignment variable which ensures both the beginning and length of a\nDMA transfer are aligned on the dma_alignment boundary. Although the\nblock layer does adjust the beginning of the transfer to ensure this\nhappens, it doesn\u0027t actually adjust the length, it merely makes sure\nthat space is allocated for transfers beyond the declared length. The\nupshot of this is that scatterlists may be padded to any size between\nthe actual length and the length adjusted to the dma_alignment safely\nknowing that memory is allocated in this region.\n\nRight at the moment, SCSI takes the default dma_aligment which is on a\n512 byte boundary. Note that this aligment only applies to transfers\ncoming in from user space. However, since all kernel allocations are\nautomatically aligned on a minimum of 32 byte boundaries, it is safe to\nadjust them in this manner as well.\n\ntj: * Adjusting sg after padding is done in block layer. Make libata\n set queue alignment correctly for ATAPI devices and drop broken\n sg mangling from ata_sg_setup().\n * Use request-\u003eraw_data_len for ATAPI transfer chunk size.\n * Killed qc-\u003eraw_nbytes.\n * Separated out killing qc-\u003en_iter.\n\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "37198e3051b63d3184886e9bb8235e7578e82628", "tree": "3ecd4e8b6b7c710b808909adbd2bf643a10e85b0", "parents": [ "837f5f8fb98d4357d49e9631c9ee2815f3c328ca" ], "author": { "name": "James Bottomley", "email": "James.Bottomley@HansenPartnership.com", "time": "Tue Feb 05 14:06:27 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Feb 06 06:59:32 2008 -0500" }, "message": "libata: kill now unused n_iter and fix sata_fsl\n\nqc-\u003en_iter was used for libata\u0027s own sg walking before sg chaining\nreplaced it. During conversion, the field and its usage in sata_fsl\nwere left behind. Kill the filed and update sata_fsl.\n\ntj: This was part of James\u0027s libata-use-block-layer-padding patch.\n Separated out by me.\n\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nCc: Li Yang \u003cleoli@freescale.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "4e6b79fa61091a0ed9b0af0f573cc257772cd88d", "tree": "cd3e9dca0355741713d038b0068cd4ba32f87b56", "parents": [ "cadb7345d92628d46cccd3765cc15cb9cd6abccf" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Fri Jan 18 18:36:28 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Jan 23 05:24:16 2008 -0500" }, "message": "libata: factor out ata_pci_activate_sff_host() from ata_pci_one()\n\nFactor out ata_pci_activate_sff_host() from ata_pci_one(). This does\nabout the same thing as ata_host_activate() but needs to be separate\nbecause SFF controllers use different and multiple IRQs in legacy\nmode.\n\nThis will be used to make SFF LLD initialization more flexible.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "442eacc362c2576aac8ebfd41b99252e28e0f49c", "tree": "4a40c47cb1c9d87830a04027323a26fe99541927", "parents": [ "d7b174500e5750099537c7f0bc4873f06b6c1b9a" ], "author": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Dec 19 04:25:10 2007 -0500" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Jan 23 05:24:15 2008 -0500" }, "message": "libata: make ata_port_queue_task() an internal function\n\nata_port_queue_task() served a single user: ata_pio_task()\n\nRename to ata_pio_queue_task() and un-export it, as nobody outside of\nlibata-core.c uses it.\n\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "0bcc65ad78ae517de16b2ca07a2891f49d44d156", "tree": "87eecc4aec56500f3e29a1e82b631093f950ac85", "parents": [ "ff2aeb1eb64c8a4770a6304f9addbae9f9828646" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Wed Dec 05 16:43:12 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Jan 23 05:24:15 2008 -0500" }, "message": "libata: make qc-\u003enbytes include extra buffers\n\nqc-\u003enbytes didn\u0027t use to include extra buffers setup by libata core\nlayer and my be odd. This patch makes qc-\u003enbytes include any extra\nbuffers setup by libata core layer and guaranteed to be aligned on 4\nbyte boundary.\n\nThis value is to be used to program the host controller. As this\nrepresents the actual length of buffer available to the controller and\nthe controller must be able to deal with short transfers for ATAPI\ncommands which can transfer variable length, this shouldn\u0027t break any\ncontrollers while making problems like rounding-down and controllers\nchoking up on odd transfer bytes much less likely.\n\nThe unmodified value is stored in new field qc-\u003eraw_nbytes.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "ff2aeb1eb64c8a4770a6304f9addbae9f9828646", "tree": "c6febbec290ec6c40bf3abc7bcdb7188f5039443", "parents": [ "f92a26365a72333f418abe82700c6030d4a1a807" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Wed Dec 05 16:43:11 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Jan 23 05:24:14 2008 -0500" }, "message": "libata: convert to chained sg\n\nlibata used private sg iterator to handle padding sg. Now that sg can\nbe chained, padding can be handled using standard sg ops. Convert to\nchained sg.\n\n* s/qc-\u003e__sg/qc-\u003esg/\n\n* s/qc-\u003epad_sgent/qc-\u003eextra_sg[]/. Because chaining consumes one sg\n entry. There need to be two extra sg entries. The renaming is also\n for future addition of other extra sg entries.\n\n* Padding setup is moved into ata_sg_setup_extra() which is organized\n in a way that future addition of other extra sg entries is easy.\n\n* qc-\u003eorig_n_elem is unused and removed.\n\n* qc-\u003en_elem now contains the number of sg entries that LLDs should\n map. qc-\u003emapped_n_elem is added to carry the original number of\n mapped sgs for unmapping.\n\n* The last sg of the original sg list is used to chain to extra sg\n list. The original last sg is pointed to by qc-\u003elast_sg and the\n content is stored in qc-\u003esaved_last_sg. It\u0027s restored during\n ata_sg_clean().\n\n* All sg walking code has been updated. Unnecessary assertions and\n checks for conditions the core layer already guarantees are removed.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nCc: Jens Axboe \u003cjens.axboe@oracle.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "001102d7859be0e7f7b9f2d62b841f2c0f9c2640", "tree": "3ba29a129314307bafc5ac4e6cab49ee98eb8698", "parents": [ "55dba3120fbcbea6800f9a18503d25f73212a347" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Wed Dec 05 16:43:09 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Jan 23 05:24:14 2008 -0500" }, "message": "libata: kill non-sg DMA interface\n\nWith atapi_request_sense() converted to use sg, there\u0027s no user of\nnon-sg interface. Kill non-sg interface.\n\n* ATA_QCFLAG_SINGLE and ATA_QCFLAG_SG are removed. ATA_QCFLAG_DMAMAP\n is used instead. (this way no LLD change is necessary)\n\n* qc-\u003ebuf_virt is removed.\n\n* ata_sg_init_one() and ata_sg_setup_one() are removed.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nCc: Rusty Russel \u003crusty@rustcorp.com.au\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "55dba3120fbcbea6800f9a18503d25f73212a347", "tree": "1b23e606aad8bc58dbe68ca905c0658625fb176e", "parents": [ "ceb0c642624f634c5b4f46b0e22df19be87a2e53" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Wed Dec 05 16:43:07 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Jan 23 05:24:14 2008 -0500" }, "message": "libata: update -\u003edata_xfer hook for ATAPI\n\nDepending on how many bytes are transferred as a unit, PIO data\ntransfer may consume more bytes than requested. Knowing how much\ndata is consumed is necessary to determine how much is left for\ndraining. This patch update -\u003edata_xfer such that it returns the\nnumber of consumed bytes.\n\nWhile at it, it also makes the following changes.\n\n* s/adev/dev/\n* use READ/WRITE constants for rw indication\n* misc clean ups\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "ceb0c642624f634c5b4f46b0e22df19be87a2e53", "tree": "a55adf01b6c00b4724a468a52fce03704bf5109f", "parents": [ "0dc36888d4422140f9eaf50f24953ec109f750a3" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Wed Dec 05 16:43:06 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Jan 23 05:24:14 2008 -0500" }, "message": "libata: add ATAPI_* cmd types and implement atapi_cmd_type()\n\nAdd ATAPI command types - ATAPI_READ, WRITE, RW_BUF, READ_CD and MISC,\nand implement atapi_cmd_type() which takes SCSI opcode and returns to\nwhich class the opcode belongs. This will be used later to improve\nATAPI handling.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "021ee9a6da1cfc57f6a6c769c3c898bdd4753108", "tree": "bb8c0246ded5a1d99320055343c7d277208fadd8", "parents": [ "a0f79b929acaba10d4780acd2543eff20bf4b5b0" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Dec 18 16:33:06 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Jan 23 05:24:12 2008 -0500" }, "message": "libata: reimplement ata_acpi_cbl_80wire() using ata_acpi_gtm_xfermask()\n\nReimplement ata_acpi_cbl_80wire() using ata_acpi_gtm_xfermask() and\nwhile at it relocate the function below ata_acpi_gtm_xfermask().\n\nNew ata_acpi_cbl_80wire() implementation takes @gtm, in both pata_via\nand pata_amd, use the initial GTM value. Both are trying to peek\ninitial BIOS configuration, so using initial caching value makes\nsense. This fixes ACPI part of cable detection in pata_amd which\npreviously always returned 0 because configuring PIO0 during reset\nclears DMA configuration.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nCc: Alan Cox \u003calan@lxorguk.ukuu.org.uk\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "a0f79b929acaba10d4780acd2543eff20bf4b5b0", "tree": "e637c1d9388a3991cd71c5be339c2ead59c460a2", "parents": [ "5df91a25df08d85700fef5fd59bb1873273e5ef5" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Dec 18 16:33:05 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Jan 23 05:24:12 2008 -0500" }, "message": "libata: implement ata_timing_cycle2mode() and use it in libata-acpi and pata_acpi\n\nlibata-acpi is using separate timing tables for transfer modes\nalthough libata-core has the complete ata_timing table. Implement\nata_timing_cycle2mode() to look for matching mode given transfer type\nand cycle duration and use it in libata-acpi and pata_acpi to replace\nprivate timing tables.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nCc: Alan Cox \u003calan@lxorguk.ukuu.org.uk\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "7c77fa4d51b1480bcec2e898c94d6912fe063c16", "tree": "d7d18fb22c8df2425878d02f4b415c06efa56457", "parents": [ "9cde9ed151e170f2e2a530f7ec0032dfbe9f443b" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Dec 18 16:33:03 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Jan 23 05:24:12 2008 -0500" }, "message": "libata: separate out ata_acpi_gtm_xfermask() from pacpi_discover_modes()\n\nFinding out matching transfer mode from ACPI GTM values is useful for\nother purposes too. Separate out the function and timing tables from\npata_acpi::pacpi_discover_modes().\n\nOther than checking shared-configuration bit after doing\nata_acpi_gtm() in pacpi_discover_modes() which should be safe, this\npatch doesn\u0027t introduce any behavior change.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nCc: Alan Cox \u003calan@lxorguk.ukuu.org.uk\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "c88f90c3779cd5e710f2acdf59ad2bd0380de98d", "tree": "7b3d7464f03d2abdc105d07728cab0df67262a25", "parents": [ "7dc951aefdc1dc20228691b04867fb6195864d67" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Nov 27 19:43:48 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Jan 23 05:24:12 2008 -0500" }, "message": "libata: add ATA_CBL_PATA_IGN\n\nATA_CBL_PATA_UNK indicates that the cable type can\u0027t be determined\nfrom the host side and might be either 80c or 40c. libata applies\ndrive or other generic limit in this case. However, there are\ncontrollers where both host and drive side detections are\nmisimplemented and the driver has to rely solely on private method -\npeeking BIOS or ACPI configuration or using some other private\nmechanism.\n\nThis patch adds ATA_CBL_PATA_IGN which tells libata to ignore the\ncable type completely and just let the LLD determine the transfer mode\nvia host transfer mode masks and -\u003emode_filter().\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nCc: Alan Cox \u003calan@lxorguk.ukuu.org.uk\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "7dc951aefdc1dc20228691b04867fb6195864d67", "tree": "b2c3c1213831593feaf16c9fdc0c88f26d3c10f2", "parents": [ "9d3501ab962b1506d93974faf8509251b4a85fbc" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Nov 27 19:43:42 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Jan 23 05:24:12 2008 -0500" }, "message": "libata: xfer_mask is unsigned long not unsigned int\n\nJeff says xfer_mask is unsigned long not unsigned int. Convert all\nxfermask fields and handling functions to deal with unsigned longs.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "9d3501ab962b1506d93974faf8509251b4a85fbc", "tree": "a4785ca835e1b0ca5551167cf0ecf484eed8b71b", "parents": [ "70cd071e4ecc06c985189665af75c108601fd5a3" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Nov 27 19:43:41 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Jan 23 05:24:11 2008 -0500" }, "message": "libata: kill ata_id_to_dma_mode()\n\nata_id_to_dma_mode() isn\u0027t quite generic. The function is basically\nprivately implemented ata_id_xfermask() combined with hardcoded mode\nprinting and configuration which are specific to ata_generic.\n\nKill the function and open code it in generic_set_mode() using generic\nxfermode handling functions.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "70cd071e4ecc06c985189665af75c108601fd5a3", "tree": "f28aae811d01f8ef44f27e5d469fe330ff834980", "parents": [ "6357357cae7794dcb89cace758108dec612e7ed5" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Nov 27 19:43:40 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Jan 23 05:24:11 2008 -0500" }, "message": "libata: clean up xfermode / PATA timing related stuff\n\n* s/ATA_BITS_(PIO|MWDMA|UDMA)/ATA_NR_\\1_MODES/g\n\n* Consistently use 0xff to indicate invalid transfer mode (0x00 is\n valid for PIO_SLOW).\n\n* Make ata_xfer_mode2mask() return proper mode mask instead of just\n the highest bit.\n\n* Sort ata_timing table in increasing xfermode order and update\n ata_timing_find_mode() accordingly.\n\nThis patch doesn\u0027t introduce any behavior change.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "6357357cae7794dcb89cace758108dec612e7ed5", "tree": "6fc39d16ff795f34b4f8a987966fd5195b1d9039", "parents": [ "f8ab6d8e15a9b978f79aee794c263014c2959dfc" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Nov 27 19:43:39 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Jan 23 05:24:11 2008 -0500" }, "message": "libata: export xfermode / PATA timing related functions\n\nExport the following xfermode related functions.\n\n* ata_pack_xfermask()\n* ata_unpack_xfermask()\n* ata_xfer_mask2mode()\n* ata_xfer_mode2mask()\n* ata_xfer_mode2shift()\n* ata_mode_string()\n* ata_id_xfermask()\n* ata_timing_find_mode()\n\nThese functions will be used later by LLD updates. While at it,\nchange unsigned short @speed to u8 @xfer_mode in\nata_timing_find_mode() for consistency.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "00115e0f5bc3bfdf3f3855ad89c8895f10458f92", "tree": "5f174c1d68e1ec0770fefdb40b813f321a838095", "parents": [ "663f99b86ac7d4c0eed8c239638da0ea8849288b" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Nov 27 19:28:58 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Jan 23 05:24:11 2008 -0500" }, "message": "libata: implement ATA_DFLAG_DUBIOUS_XFER\n\nATA_DFLAG_DUBIOUS_XFER is set whenever data transfer speed or method\nchanges and gets cleared when data transfer command succeeds in the\nnewly configured transfer mode.\n\nThis will be used to improve speed down logic.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003c\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "3884f7b0a8382b89d8ca5da23bd98e3e15fc805b", "tree": "a62ec936db81b3018d072cb03a28dc1d4337dd4a", "parents": [ "6f1d1e3a03fd04a9d9c82fd3cf414020097bb142" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Nov 27 19:28:56 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Jan 23 05:24:10 2008 -0500" }, "message": "libata: clean up EH speed down implementation\n\nClean up EH speed down implementation.\n\n* is_io boolean variable is replaced eflags. is_io is ATA_EFLAG_IS_IO.\n\n* Error categories now have names.\n\n* Better comments.\n\n* Reorder 5min and 10min rules in ata_eh_speed_down_verdict()\n\n* Use local variable @link to cache @dev-\u003elink in ata_eh_speed_down()\n\nThese changes are to improve readability and ease further changes.\nThis patch doesn\u0027t introduce any behavior change.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "f20ded38aa54b92dd0af32578b8916d0aa2d9e05", "tree": "26ab6752bf8318a8ea3b30356ac055f5f30cc31e", "parents": [ "11b7becca9425aab50807503c8102b2db9e5ecf0" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Nov 27 19:28:52 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Jan 23 05:24:10 2008 -0500" }, "message": "libata: rearrange ATA_DFLAG_*\n\nArea for DFLAGs which are cleared on INIT is full. Extend it by 8\nbits.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "140b5e59119a172a91b5fa13d54ca4f79bbefee1", "tree": "ebc04fbd143756d7ef80e870cd9ae214d2607047", "parents": [ "f2dfc1a12bb1a029df62b018a8e1882e91041025" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Wed Dec 12 12:21:52 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Mon Dec 17 20:43:28 2007 -0500" }, "message": "libata: fix ATAPI draining\n\nWith ATAPI transfer chunk size properly programmed, libata PIO HSM\nshould be able to handle full spurious data chunks. Also, it\u0027s a good\nidea to suppress trailing data warning for misc ATAPI commands as\nthere can be many of them per command - for example, if the chunk size\nis 16 and the drive tries to transfer 510 bytes, there can be 31\ntrailing data messages.\n\nThis patch makes the following updates to libata ATAPI PIO HSM\nimplementation.\n\n* Make it drain full spurious chunks.\n\n* Suppress trailing data warning message for misc commands.\n\n* Put limit on how many bytes can be drained.\n\n* If odd, round up consumed bytes and the number of bytes to be\n drained. This gets the number of bytes to drain right for drivers\n which do 16bit PIO.\n\nThis patch is partial backport of improve-ATAPI-data-xfer patchset\npending for #upstream.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "398e07826b24cbeb5ff2f0a178367fc9d24cd475", "tree": "78191583f2e1e5bb0aba9fbee8016e72285f4b31", "parents": [ "c05e6ff035c1b25d17364a685432b33937d3dc23" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Sat Dec 15 15:05:03 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Mon Dec 17 20:33:14 2007 -0500" }, "message": "libata-acpi: implement dev-\u003egtf_cache and evaluate _GTF right after _STM during resume\n\nOn certain implementations, _GTF evaluation depends on preceding _STM\nand both can be pretty picky about the configuration. Using _GTM\nresult cached during controller initialization satisfies the most\nneurotic _STM implementation. However, libata evaluates _GTF after\nreset during device configuration and the hardware state can be\ndifferent from what _GTF expects and can cause evaluation failure.\n\nThis patch adds dev-\u003egtf_cache and updates ata_dev_get_GTF() such that\nit uses the cached value if available. Cache is cleared with a call\nto ata_acpi_clear_gtf().\n\nBecause for SATA ACPI nodes _GTF must be evaluated after _SDD which\ncan\u0027t be done till IDENTIFY is complete, _GTF caching from\nata_acpi_on_resume() is used only for IDE ACPI nodes.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "c05e6ff035c1b25d17364a685432b33937d3dc23", "tree": "08c9171d024b6659b29a4f9f7d95318430b75b6a", "parents": [ "562f0c2d771ee7be6b37fe015f94a929f8056120" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Sat Dec 15 15:05:02 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Mon Dec 17 20:33:14 2007 -0500" }, "message": "libata-acpi: implement and use ata_acpi_init_gtm()\n\n_GTM fetches currently configured transfer mode while _STM configures\ncontroller according to _GTM parameter and prepares transfer mode\nconfiguration TFs for _GTF. In many cases _GTM and _STM\nimplementations are quite brittle and can\u0027t cope with configuration\nchanged by libata.\n\nlibata does not depend on ATA ACPI to configure devices. The only\nreason libata performs _GTM and _STM are to make _GTF evaluation\nsucceed and libata also doesn\u0027t care about how _GTF TFs configure\ntransfer mode. It overrides that configuration anyway, so from\nlibata\u0027s POV, it doesn\u0027t matter what value is feeded to _STM as long\nas evaluation succeeds for _STM and following _GTF.\n\nThis patch adds dev-\u003e__acpi_init_gtm and store initial _GTM values on\nhost initialization before modified by reset and mode configuration.\nIf the field is valid, ata_acpi_init_gtm() returns pointer to the\nsaved _GTM structure; otherwise, NULL.\n\nThis saved value is used for _STM during resume and peek at\nBIOS/firmware programmed initial timing for later use. The accessor\nis there to make building w/o ACPI easy as dev-\u003e__acpi_init doesn\u0027t\nexist if ACPI is not enabled.\n\nOn driver detach, the initial BIOS configuration is restored by\nexecuting _STM with the initial _GTM values such that the next driver\ncan also use the initial BIOS configured values.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "c2e366a107e511ad00c2181c52e4150fc086ec0f", "tree": "1697b064e0e4c08680fa47daec8b7618ceb425f5", "parents": [ "0d02f0b22b678b9d6c8ac8cad7b4cfbbdf6fab18" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Sat Dec 15 15:04:58 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Mon Dec 17 20:33:12 2007 -0500" }, "message": "libata: update ata_*_printk() macros such that level can be a variable\n\nMake prink helpers format @lv together rather than prepending to the\nformat string as constant.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "0d02f0b22b678b9d6c8ac8cad7b4cfbbdf6fab18", "tree": "27b5f5afcff7450f37432f30661e56337313c5e9", "parents": [ "4e5200334e03e5620aa19d538300c13db270a063" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Sat Dec 15 15:04:57 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Mon Dec 17 20:33:12 2007 -0500" }, "message": "libata-acpi: adjust constness in ata_acpi_gtm/stm() parameters\n\n* No internal function uses const ata_port. Drop const from @ap.\n\n* Make ata_acpi_stm() copy @stm before using it and change @stm to\n const.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "2d3b8eea7f2fbafd5d779cc92f7aedbd1ef575e9", "tree": "292cd6197c457defc3e07d02a663cabb3dceb253", "parents": [ "21bef6dd2b419f28c8096a8e30ad86dcbff44c02" ], "author": { "name": "Albert Lee", "email": "albertcc@tw.ibm.com", "time": "Thu Nov 15 10:35:46 2007 +0900" }, "committer": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Nov 19 12:28:11 2007 +0900" }, "message": "libata: workaround DRQ\u003d1 ERR\u003d1 for ATAPI tape drives\n\nAfter an error condition, some ATAPI tape drives set DRQ\u003d1 together\nwith ERR\u003d1 when asking the host to transfer the CDB of the next packet\ncommand (i.e. request sense). This patch, a revised version of\nAlan/Mark\u0027s previous patch, adds ATA_HORKAGE_STUCK_ERR to workaround\nthe problem by ignoring the ERR bit and proceed sending the CDB.\n\nSigned-off-by: Albert Lee \u003calbertcc@tw.ibm.com\u003e\nCc: Alan Cox \u003calan@lxorguk.ukuu.org.uk\u003e\nCc: Mark Lord \u003climl@rtr.ca\u003e\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "21bef6dd2b419f28c8096a8e30ad86dcbff44c02", "tree": "704912dbb4c28fd1a49d9256193762c19255d7a4", "parents": [ "de753e5e8678d9674de0a3bda9ead9e770fdbf53" ], "author": { "name": "Adrian Bunk", "email": "bunk@kernel.org", "time": "Thu Nov 15 10:35:45 2007 +0900" }, "committer": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Nov 19 12:28:09 2007 +0900" }, "message": "libata: remove unused functions\n\nThis patch removes the following obsolete functions:\n- libata-core.c: __sata_phy_reset()\n- libata-core.c: sata_phy_reset()\n- libata-eh.c: ata_qc_timeout()\n- libata-eh.c: ata_eng_timeout()\n\nSigned-off-by: Adrian Bunk \u003cbunk@kernel.org\u003e\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "6bbfd53d47abd1fb20d7c93a9b19a75970b66f49", "tree": "9c28b3b033e32dd5a03c08b61bdddcd79283791f", "parents": [ "73946f9fc5be1433f1e182d11303188390ff242f" ], "author": { "name": "Alan Cox", "email": "alan@lxorguk.ukuu.org.uk", "time": "Mon Nov 05 22:58:58 2007 +0000" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Mon Nov 05 18:10:28 2007 -0500" }, "message": "libata: handle broken cable reporting\n\nOne or two ancient drives predated the cable spec and didn\u0027t sent the\nvalid bits for the field. I had hoped to leave this out of libata as a\npiece of historical annoyance but a recent CD drive shows the same bug so\nwe have to import support for it.\n\nSame concept as Bartlomiej\u0027s changes old IDE except that as we have\ncentralised blacklists we can avoid keeping another private table of stuff\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "e027bd36c146582cef382364e5c826db93d4427b", "tree": "45f3d3db848a5bd44bcb2191db17cbfd30f4911a", "parents": [ "f90f0828e57e97cb1ff19520d252882cfc6fb3c0" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Fri Oct 26 16:19:26 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Tue Oct 30 09:59:43 2007 -0400" }, "message": "libata: implement and use ATA_QCFLAG_QUIET\n\nImplement ATA_QCFLAG_QUIET which indicates that there\u0027s no need to\nreport if the command fails with AC_ERR_DEV and set it for passthrough\ncommands.\n\nCombined with previous changes, this now makes device errors for all\ndirect commands reported directly to the issuer without going through\nEH actions and reporting.\n\nNote that EH is still invoked after non-IO device errors to determine\nthe nature of the error and resume command execution (some controller\nrequires special care after error to continue). It just performs\ndefault maintenance after error, examines what\u0027s going on, realizes\nthat it\u0027s none of its business and reports the command failure without\nlogging any error messages.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "ca77329fb713b7fea6a307068e0dd0248e7aa640", "tree": "6a1b987f489d7c3f0bbe81647b4ee2b0216afe8a", "parents": [ "ab6fc95f609b372a19e18ea689986846ab1ba29c" ], "author": { "name": "Kristen Carlson Accardi", "email": "kristen.c.accardi@intel.com", "time": "Thu Oct 25 00:58:59 2007 -0400" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Mon Oct 29 11:00:35 2007 -0400" }, "message": "[libata] Link power management infrastructure\n\nDevice Initiated Power Management, which is defined\nin SATA 2.5 can be enabled for disks which support it.\nThis patch enables DIPM when the user sets the link\npower management policy to \"min_power\".\n\nAdditionally, libata drivers can define a function\n(enable_pm) that will perform hardware specific actions to\nenable whatever power management policy the user set up\nfor Host Initiated Power management (HIPM).\nThis power management policy will be activated after all\ndisks have been enumerated and intialized. Drivers should\nalso define disable_pm, which will turn off link power\nmanagement, but not change link power management policy.\n\nDocumentation/scsi/link_power_management_policy.txt has additional\ninformation.\n\nSigned-off-by: Kristen Carlson Accardi \u003ckristen.c.accardi@intel.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "88ff6eafbb2a1c55f0f0e2e16d72e7b10d8ae8a5", "tree": "ad6ab294a4f725540bfa24b7a451273b99fa71c1", "parents": [ "054a5fbaceb2eb3a31ea843c1cf0b8e10b91478c" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Oct 16 14:21:24 2007 -0700" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Mon Oct 29 06:15:27 2007 -0400" }, "message": "libata: implement ata_wait_after_reset()\n\nOn certain device/controller combination, 0xff status is asserted\nafter reset and doesn\u0027t get cleared during 150ms post-reset wait. As\n0xff status is interpreted as no device (for good reasons), this can\nlead to misdetection on such cases.\n\nThis patch implements ata_wait_after_reset() which replaces the 150ms\nsleep and waits upto ATA_TMOUT_FF_WAIT if status is 0xff.\nATA_TMOUT_FF_WAIT is currently 800ms which is enough for\nHHD424020F7SV00 to get detected but not enough for Quantum GoVault\ndrive which is known to take upto 2s.\n\nWithout parallel probing, spending 2s on 0xff port would incur too\nmuch delay on ata_piix\u0027s which use 0xff to indicate empty port and\ndoesn\u0027t have SCR register, so GoVault needs to wait till parallel\nprobing.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "054a5fbaceb2eb3a31ea843c1cf0b8e10b91478c", "tree": "cbf21985f3894b11fe00666cef8ac33bae4f007d", "parents": [ "4dbfa39b6c95eb9d0aedb5bd00bb552b91c31e3d" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Thu Oct 25 18:30:36 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Mon Oct 29 06:15:25 2007 -0400" }, "message": "libata: track SLEEP state and issue SRST to wake it up\n\nATA devices in SLEEP mode don\u0027t respond to any commands. SRST is\nnecessary to wake it up. Till now, when a command is issued to a\ndevice in SLEEP mode, the command times out, which makes EH reset the\ndevice and retry the command after that, causing a long delay.\n\nThis patch makes libata track SLEEP state and issue SRST automatically\nif a command is about to be issued to a device in SLEEP.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nCc: Bruce Allen \u003cballen@gravity.phys.uwm.edu\u003e\nCc: Andrew Paprocki \u003candrew@ishiboo.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "2dcb407e61458ded17503d6bd12b8c064965368b", "tree": "6044e032197b84f9943a385b0c9dbb6656c3f97f", "parents": [ "01e7ae8c13bb06a2ce622ebace33bb7e28ef596c" ], "author": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 19 06:42:56 2007 -0400" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Tue Oct 23 20:59:42 2007 -0400" }, "message": "[libata] checkpatch-inspired cleanups\n\nTackle the relatively sane complaints of checkpatch --file.\n\nThe vast majority is indentation and whitespace changes, the rest are\n\n* #include fixes\n* printk KERN_xxx prefix addition\n* BSS/initializer cleanups\n\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "3be6cbd73f74b4a3da82cc7d6e1688a4ae595fc7", "tree": "a077cf70d1cd438db34f5def23dd8f72f42b5a8c", "parents": [ "858c9c406688bc7244986b5836265071edfd1d3f" ], "author": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Thu Oct 18 16:21:18 2007 -0400" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Thu Oct 18 16:21:18 2007 -0400" }, "message": "[libata] kill ata_sg_is_last()\n\nShort term, this works around a bug introduced by early sg-chaining\nwork.\n\nLong term, removing this function eliminates a branch from a hot\npath loop in each scatter/gather table build. Also, as this code\ndemonstrates, we don\u0027t need to _track_ the end of the s/g list, as\nlong as we mark it in some way. And doing so programatically is nice.\nSo its a useful cleanup, regardless of its short term effects.\n\nBased conceptually on a quick patch by Jens Axboe.\n\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "8726021626780a73e795c9b939e1ee49ac8c9136", "tree": "7f42d69578ab8f286a4b66180c95864a60e95690", "parents": [ "a8474ce23a73185dd2bae4c884b1716474032d31" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Oct 16 11:14:12 2007 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Oct 16 11:14:12 2007 +0200" }, "message": "libata: convert to using sg helpers\n\nThis converts libata to using the sg helpers for looking up sg\nelements, instead of doing it manually.\n\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "2855568b1ee4f58ef2c0a13ddfceb4b0b216b7ed", "tree": "79262af301c2386c2c391942991742653bc7935e", "parents": [ "b06ce3e51e3df4394a584c234f11240b1c6f8d5b" ], "author": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Thu Oct 11 17:12:35 2007 -0400" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:47 2007 -0400" }, "message": "[libata] struct pci_dev related cleanups\n\n* remove pointless pci_dev_to_dev() wrapper. Just directly reference\n the embedded struct device like everyone else does.\n\n* pata_cs5520: delete cs5520_remove_one(), it was a duplicate of\n ata_pci_remove_one()\n\n* linux/libata.h: don\u0027t bother including linux/pci.h, we don\u0027t need it.\n Simply declare \u0027struct pci_dev\u0027 and assume interested parties will\n include the header, as they should be doing anyway.\n\n* linux/libata.h: consolidate all CONFIG_PCI declarations into a\n single location in the header.\n\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "b06ce3e51e3df4394a584c234f11240b1c6f8d5b", "tree": "6a6f5a1cccdc2972b236d376afeba4fd296d5400", "parents": [ "afaa5c373d2c49ee4865847031b82f1377f609d0" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Oct 09 15:06:48 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:47 2007 -0400" }, "message": "libata: use ata_exec_internal() for PMP register access\n\nPMP registers used to be accessed with dedicated accessors -\u003epmp_read\nand -\u003epmp_write. During reset, those callbacks are called with the\nport frozen so they should be able to run without depending on\ninterrupt delivery. To achieve this, they were implemented polling.\n\nHowever, as resetting the host port makes the PMP to isolate fan-out\nports until SError.X is cleared, resetting fan-out ports while port is\nfrozen doesn\u0027t buy much additional safety.\n\nThis patch updates libata PMP support such that PMP registers are\naccessed using regular ata_exec_internal() mechanism and kills\n-\u003epmp_read/write() callbacks. The following changes are made.\n\n* PMP access helpers - sata_pmp_read_init_tf(), sata_pmp_read_val(),\n sata_pmp_write_init_tf() are folded into sata_pmp_read/write() which\n are now standalone PMP register access functions.\n\n* sata_pmp_read/write() returns err_mask instead of rc. This is\n consistent with other functions which issue internal commands and\n allows more detailed error reporting.\n\n* ahci interrupt handler is modified to ignore BAD_PMP and\n spurious/illegal completion IRQs while reset is in progress. These\n conditions are expected during reset.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "afaa5c373d2c49ee4865847031b82f1377f609d0", "tree": "785203bd4538d2e3c36622ebc2eac93652c7e19e", "parents": [ "2b789108fc1dcba22050a7e6e29ae5ebaea427dd" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Oct 09 15:06:10 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:47 2007 -0400" }, "message": "libata: implement ATA_PFLAG_RESETTING\n\nImplement ATA_PFLAG_RESETTING. This flag is set while reset is in\nprogress. It\u0027s set before prereset is called and cleared after reset\nfails or postreset is finished.\n\nThis flag itself doesn\u0027t have any function. It will be used by LLDs\nto tell whether reset is in progress if it needs to behave differently\nduring reset.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "badff03df7a005d13ea2ae8ddc2f5ec0cfc049e8", "tree": "11f661e29c5991a9baf33382749ce9f8347da5cf", "parents": [ "70edb185dbaa8a9ec0d6f3e50bb5698a4e85ded6" ], "author": { "name": "Alan Cox", "email": "alan@lxorguk.ukuu.org.uk", "time": "Thu Oct 04 21:28:18 2007 +0100" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:46 2007 -0400" }, "message": "libata-core: Expose gtm methods for driver use\n\nTalk to the dark side our driver has to, yes. Much misleading is the\ndata. Store it in a structure we do so that it may be parsed.\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\n--\nWhats small, old and shouts phrases out of order across mountains ?\nYodla..\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "b3a706014e56b1356e7b275fd25b833c63175bf0", "tree": "1595a67c8b09f9222c4d59f466acca681a56a02c", "parents": [ "7100819f5f9b99eb7c7dd5597f293388a405bf7b" ], "author": { "name": "Alan Cox", "email": "alan@redhat.com", "time": "Tue Oct 02 12:38:26 2007 -0400" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:45 2007 -0400" }, "message": "libata: Add a drivers/ide style DMA disable\n\nThis is useful when debugging, handling problem systems, or for\ndistributions just to get the system installed so it can be sorted\nout later.\n\nThis is a bit smarter than the old IDE one and lets you do\n\nlibata.dma\u003d0\tDisable all PATA DMA like old IDE\nlibata.dma\u003d1\tDisk DMA only\nlibata.dma\u003d2\tATAPI DMA only\nlibata.dma\u003d4\tCF DMA only\n\n(or combinations thereof - 0,1,3 being the useful ones I suspect)\n\n(I\u0027ve split CF as it seems to be a seperate case of pain and suffering\ndifferent to the others and caused by assorted PIO wired adapters etc)\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\n\n[edited to work on SATA too, changing name from \u0027pata_dma\u0027 to \u0027dma\u0027]\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "31f88384443b3e0d7e2c9d36a96647b7e82edad3", "tree": "7bd6321d668f86adbf266513b83a75ae8d568772", "parents": [ "d0df8b5d0fb547a3351c2a4b1ded7f7cde5d713a" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Sun Sep 23 13:19:54 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:44 2007 -0400" }, "message": "libata-pmp: implement qc_defer for command switching PMP support\n\nImplement sata_pmp_qc_defer_cmd_switch() - standard qc_defer for\ncommand switching PMP support.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "3af9a77af9e2b72366363864bfcd3d51465ff98a", "tree": "f6a311a56353cf81d34d64cf1a3f86aa6f659246", "parents": [ "3495de733633d24ee97852080b737b436c110d6e" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Sun Sep 23 13:19:54 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:44 2007 -0400" }, "message": "libata-pmp: implement Port Multiplier support\n\nImplement Port Multiplier support. To support PMP, a LLDD has to\nsupply ops-\u003epmp_read() and pmp_write(). If non-null, -\u003epmp_attach and\n-\u003epmp_detach are called on PMP attach and detach, respectively.\n\n-\u003epmp_read/write() can be called while the port is frozen, so they\nmust be implemented by polling. This patch supplies several helpers\nto ease -\u003epmp_read/write() implementation.\n\nAlso, irq_handler and error_handler must be PMP aware. Most of PMP\naware EH can be done by calling ata_pmp_do_eh() with appropriate\nmethods. PMP EH uses separate set of reset methods and this patch\nimplements standard prereset, hardreset and postreset methods.\n\nThis patch only implements PMP support. The next patch will integrate\nPMP into the reset of libata and thus enable PMP support.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "93328e1145c1989d1a214d34ac4e968dea7f7ed7", "tree": "3b22b8453c9c3526abeb68efcc45eacff4e6574b", "parents": [ "54174db300ee1bac632d62e4ac37fe02e47d1f18" ], "author": { "name": "Alan Cox", "email": "alan@redhat.com", "time": "Sat Sep 29 04:06:48 2007 -0400" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:43 2007 -0400" }, "message": "[PATCH] libata: Fix HPA handling regression\n\nRestore the support for handling drives that report one sector too many\n(ie SCSI not ATA style). This worked before the HPA update but was\nremoved in that process.\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "7d77b247088fb360aa74bfdd9e19bce1e1987668", "tree": "add1b03309dd6fa82eb0f47e1a88766695f38f28", "parents": [ "e31e8531d668c9c4dc7883054788f89805188003" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Sun Sep 23 13:14:13 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:42 2007 -0400" }, "message": "libata-pmp-prep: implement sata_async_notification()\n\nAN serves multiple purposes. For ATAPI, it\u0027s used for media change\nnotification. For PMP, for downstream PHY status change notification.\nImplement sata_async_notification() which demultiplexes AN.\n\nTo avoid unnecessary port events, ATAPI AN is not enabled if PMP is\nattached but SNTF is not available.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nCc: Kriten Carlson Accardi \u003ckristen.c.accardi@intel.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "e31e8531d668c9c4dc7883054788f89805188003", "tree": "9c3762792344d24c996d6459409456664a2e381b", "parents": [ "668108d73bbb1ae85f01db38d6be822fd28ece1f" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Sun Sep 23 13:14:13 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:41 2007 -0400" }, "message": "libata-pmp-prep: implement ATA_HORKAGE_SKIP_PM\n\nSome pseudo devices fail PM commands unnecessarily aborting system\nsuspend. Implement ATA_HORKAGE_SKIP_PM which makes libata skip PM\ncommands for these devices.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "f9df58cb27dfd605eced643bb3aa599fe4feeee8", "tree": "23b5151855d07f3b25c081e29d14f5d353367248", "parents": [ "fd995f7039f1955ccc6b43e1e2d168060b31e4b2" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Sun Sep 23 13:14:13 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:41 2007 -0400" }, "message": "libata-pmp-prep: implement ATA_LFLAG_DISABLED\n\nImplement ATA_LFLAG_DISABLED. The flag indicates the link is disabled\ndue to EH recovery failure. While a link is disabled, no EH action is\ntaken on the link and suspend/resume become noop too.\n\nThis will be used by PMP links to manage failed links.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "fd995f7039f1955ccc6b43e1e2d168060b31e4b2", "tree": "1aac318f211493183508a97aa6c8b2b6cb41d0b6", "parents": [ "ae791c05694d7391ee9261a0450a50f7e95aedfd" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Sun Sep 23 13:14:12 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:41 2007 -0400" }, "message": "libata-pmp-prep: implement ATA_LFLAG_NO_RETRY\n\nSome PMP links are connected to internal pseudo devices which may come\nand go depending on situation. There\u0027s no reason to try hard to\nrecover them. ATA_LFLAG_NO_RETRY tells EH to not retry if the device\nattached to the link fails.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "ae791c05694d7391ee9261a0450a50f7e95aedfd", "tree": "73e3fbcd6cdfe667b5dcc512daaec7fb5941a132", "parents": [ "da917d69d0ea63f5390716cba6e77f490ce96df9" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Sun Sep 23 13:14:12 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:41 2007 -0400" }, "message": "libata-pmp-prep: implement ATA_LFLAG_NO_SRST, ASSUME_ATA and ASSUME_SEMB\n\nSome links on some PMPs locks up on SRST and/or report incorrect\ndevice signature. Implement ATA_LFLAG_NO_SRST, ASSUME_ATA and\nASSUME_SEMB to handle these quirky links. NO_SRST makes EH avoid\nSRST. ASSUME_ATA and SEMB forces class code to ATA and SEMB_UNSUP\nrespectively. Note that SEMB isn\u0027t currently supported yet so the\n_UNSUP variant is used.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "da917d69d0ea63f5390716cba6e77f490ce96df9", "tree": "214670f42be4c140bbd05226adc8f86db51ee725", "parents": [ "31cc23b34913bc173680bdc87af79e551bf8cc0d" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Sun Sep 23 13:14:12 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:41 2007 -0400" }, "message": "libata-pmp-prep: implement qc_defer helpers\n\nImplement ap-\u003enr_active_links (the number of links with active qcs),\nap-\u003eexcl_link (pointer to link which can be used by -\u003eqc_defer and is\ncleared when a qc with ATA_QCFLAG_CLEAR_EXCL completes), and\nata_link_active().\n\nThese can be used by -\u003eqc_defer() to implement proper command\nexclusion. This set of helpers seem enough for both sil24 (ATAPI\nexclusion needed) and cmd-switching PMP.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "31cc23b34913bc173680bdc87af79e551bf8cc0d", "tree": "ec64421ead9259174f0de8b22c36449ece6d69a4", "parents": [ "fb7fd61454c8681cd2621051a710b78a00369203" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Sun Sep 23 13:14:12 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:41 2007 -0400" }, "message": "libata-pmp-prep: implement ops-\u003eqc_defer()\n\nControllers which support PMP have various restrictions on which\ncombinations of commands are allowed to what number of devices\nconcurrently. This patch implements ops-\u003eqc_defer() which determines\nwhether a qc can be issued at the moment or should be deferred.\n\nIf the function returns ATA_DEFER_LINK, the qc will be deferred until\na qc completes on the link. If ATA_DEFER_PORT, until a qc completes\non any link. The defer conditions are advisory and in general\nATA_DEFER_LINK can be considered as lower priority deferring than\nATA_DEFER_PORT.\n\nops-\u003eqc_defer() replaces fixed ata_scmd_need_defer(). For standard\nNCQ/non-NCQ exclusion, ata_std_qc_defer() is implemented. ahci and\nsata_sil24 are converted to use ata_std_qc_defer().\n\nops-\u003eqc_defer() is heavier than the original mechanism because full qc\nis prepped before determining to defer it, but various information is\nneeded to determine defer conditinos and fully translating a qc is the\nonly way to supply such information in generic manner.\n\nIMHO, this shouldn\u0027t cause any noticeable performance issues as\n\n* for most cases deferring occurs rarely (except for NCQ-aware\n cmd-switching PMP)\n* translation itself isn\u0027t that expensive\n* once deferred the command won\u0027t be repeated until another command\n completes which usually is a very long time cpu-wise.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "e0a7175263db4a226558883a51a88a5d2bc5d9fe", "tree": "f9d23742df7f8a461ff63ad5be190f6a17742dd8", "parents": [ "854c73a2f1c3bcc4aa88c25e208dc597e8efb795" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Sun Sep 23 13:14:12 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:40 2007 -0400" }, "message": "libata-pmp-prep: add PMP related constants, fields, ops and update helpers\n\nAdd PMP related constants, fields and ops. Also, update\nata_class_enabled/disabled() such that PMP classes are considered.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "3f19859ee95a38c066a0420eb8a30c76ecd67a42", "tree": "0ad611a48ddb18be3b8ec35e4dc318e2fd344961", "parents": [ "7a234aff3d83728fd83cf19df32d3df52566d2ac" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Sun Sep 02 23:23:57 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:39 2007 -0400" }, "message": "libata: update ata_dev_try_classify() arguments\n\nMake ata_dev_try_classify() take a pointer to ata_device instead of\nata_port/port_number combination for consistency and add @present\nargument. @present indicates whether the device seems present during\nreset. It\u0027s the result of TF access during softreset and link\nonlineness during hardreset. @present will be used to improve\ndiagnostic failure handling.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "2557164e0b18e78a7772632a2a90832f56181fc5", "tree": "f8c9830e65393be8eb470d6300e1c2bc26399b08", "parents": [ "e2d352af6d8f1734394bc1bcf0afe345cc3ef235" ], "author": { "name": "Kristen Carlson Accardi", "email": "kristen.c.accardi@intel.com", "time": "Tue Sep 11 11:41:59 2007 -0700" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:39 2007 -0400" }, "message": "ata: increase allowed config flags\n\nIn anticipation of more features, increase number of config flags\nallowed, and move the init flags.\n\nSigned-off-by: Kristen Carlson Accardi \u003ckristen.c.accardi@intel.com\u003e\nCc: Hugh Dickens \u003chugh@veritas.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "a1e10f7e68a544c80081fee4fa550dc28389f44a", "tree": "cf6673989f360e0b12b2a658a7042d30420c14eb", "parents": [ "cbcdd87593a1d85c5c4b259945a3a09eee12814d" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Sat Aug 18 13:28:49 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:37 2007 -0400" }, "message": "libata: move EH repeat reporting into ata_eh_report()\n\nEH is sometimes repeated without any error or action. For example,\nthis happens when probing IDENTIFY fails because of a phantom device.\nIn these cases, all the repeated EH does is making sure there is no\nunhandled error or pending action and return. This repeation is\nnecessary to avoid losing any event which occurred while EH was in\nprogress.\n\nUnfortunately, this dry run causes annonying \"EH pending after\ncompletion\" message. This patch moves the repeat reporting into\nata_eh_report() such that it\u0027s more compact and skipped on dry runs.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nCc: Mikael Pettersson \u003cmikep@it.uu.se\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "cbcdd87593a1d85c5c4b259945a3a09eee12814d", "tree": "ee03df963a12ec7f30f6c3a8742421daf2c34f50", "parents": [ "e923090ddd9fef1d4e06dc6c5295e29baced19f3" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Sat Aug 18 13:14:55 2007 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:37 2007 -0400" }, "message": "libata: implement and use ata_port_desc() to report port configuration\n\nCurrently, port configuration reporting has the following problems.\n\n* iomapped address is reported instead of raw address\n* report contains irrelevant fields or lacks necessary fields for\n non-SFF controllers.\n* host-\u003eirq/irq2 are there just for reporting and hacky.\n\nThis patch implements and uses ata_port_desc() and\nata_port_pbar_desc(). ata_port_desc() is almost identical to\nata_ehi_push_desc() except that it takes @ap instead of @ehi, has no\nlocking requirement, can only be used during host initialization and \"\n\" is used as separator instead of \", \". ata_port_pbar_desc() is a\nhelper to ease reporting of a PCI BAR or an offsetted address into it.\n\nLLD pushes whatever description it wants using the above two\nfunctions. The accumulated description is printed on host\nregistration after \"[S/P]ATA max MAX_XFERMODE \".\n\nSFF init helpers and ata_host_activate() automatically add\ndescriptions for addresses and irq respectively, so only LLDs which\nisn\u0027t standard SFF need to add custom descriptions. In many cases,\nsuch controllers need to report different things anyway.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "ac8869d56d95a8c74403e6f7a47d74fcfcc1b988", "tree": "2f812aff4c6e559f3c093f9933960d09631375c8", "parents": [ "7d73a363dea186a864f6295bbe842da8044d42cd" ], "author": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Thu Aug 16 03:17:03 2007 -0400" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:35 2007 -0400" }, "message": "[libata] Remove -\u003eport_disable() hook\n\nIt was always set to ata_port_disable(). Removed the hook, and replaced\nthe very few ap-\u003eops-\u003eport_disable() callsites with direct calls to\nata_port_disable().\n\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "e1ddb4b6a2c9b2c72991eb8640ef2f50691ac502", "tree": "fd9d0f14c5a5e329f4c1d2ee737177c5caf24b02", "parents": [ "6d32d30f55020d766388df7515f771f68c973033" ], "author": { "name": "Alan Cox", "email": "alan@lxorguk.ukuu.org.uk", "time": "Thu Aug 16 02:33:36 2007 -0400" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:34 2007 -0400" }, "message": "[libata] add ACPI cable detect API\n\nCombined from two Alan Cox patches:\n\n1) libata: ACPI checks for 80wire cable\n\nWe can use the ACPI mode information with several drivers as a hint to\ncable type. If the ACPI mode set by the BIOS is faster than UDMA33 then\nwe know the BIOS thinks there are 80wire cables. If it doesn\u0027t set such a\nmode or it has no ACPI method then we get no further information and can\nrely on existing approaches\n\nIntroduce the function headers needed. Null it out for non ACPI boxes\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\n\n2) libata: ACPI checks for 80wire cable\n\nProvide actual methods for checking if the ACPI support thinks the cable\nis 80wire, or doesn\u0027t know\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\n\nCombined into a single changeset and\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" }, { "commit": "6d32d30f55020d766388df7515f771f68c973033", "tree": "ec0efe48460b5b3f177fed2698ccd04b7964359b", "parents": [ "cb94c1cf5a6beffbd8935eb91227df0dd1987644" ], "author": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Wed Aug 15 05:38:46 2007 -0400" }, "committer": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Oct 12 14:55:34 2007 -0400" }, "message": "[libata] Remove -\u003eirq_ack() hook, and ata_dummy_irq_on()\n\n* -\u003eirq_ack() is redundant to what the irq handler already\n performs... chk-status + irq-clear. Furthermore, it is only\n called in one place, when screaming-irq-debugging is enabled,\n so we don\u0027t want to bother with a hook just for that.\n\n* ata_dummy_irq_on() is only ever used in drivers that have\n no callpath reaching -\u003eirq_on(). Remove .irq_on hook from\n those drivers, and the now-unused ata_dummy_irq_on()\n\nSigned-off-by: Jeff Garzik \u003cjeff@garzik.org\u003e\n" } ], "next": "cb94c1cf5a6beffbd8935eb91227df0dd1987644" }