Hello.
On 12/17/2014 08:21 AM, Suman Tripathi wrote:
This patch addresses the issue with ATA_CMD_PACKET pio mode
command for enumeration and device detection with ATAPI devices.It is the
same issue as in patch
Ugh, HTML... Please stick to plain text when posting to the
'vger.kernel.org' mailing lists (HTML mails are supposed to be ignored).
www.spinics.net/lists/linux-__ide/msg49092.html
<http://www.spinics.net/lists/linux-ide/msg49092.html>
Signed-off-by: Suman Tripathi <stripathi@xxxxxxx <mailto:stripathi@xxxxxxx>>
---
drivers/ata/ahci_xgene.c | 12 +++++++-----
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/drivers/ata/ahci_xgene.c b/drivers/ata/ahci_xgene.c
index 0ffd3c9..9730907 100644
--- a/drivers/ata/ahci_xgene.c
+++ b/drivers/ata/ahci_xgene.c
@@ -125,10 +125,11 @@ static int xgene_ahci_restart_engine(__struct
ata_port *ap)
* xgene_ahci_qc_issue - Issue commands to the device
* @qc: Command to issue
*
- * Due to Hardware errata for IDENTIFY DEVICE command, the controller cannot
- * clear the BSY bit after receiving the PIO setup FIS. This results in
the dma
- * state machine goes into the CMFatalErrorUpdate state and locks up. By
- * restarting the dma engine, it removes the controller out of lock up state.
+ * Due to Hardware errata for IDENTIFY DEVICE command and COMMAND PACKET
Just PACKET.
+ * command of ATAPI protocol set, the controller cannot clear the BSY bit
+ * after receiving the PIO setup FIS. This results in the dma state machine
DMA.
+ * goes into the CMFatalErrorUpdate state and locks up. By restarting the
s/goes/going/.
[suman] : Didn't understand this. Do you mean to change to going ?
It's a 'sed' command to replace "goes" by "going" IIRC. Yes. Also,
s/locks/locking/.
It is already goes there.
[...]
Thanks,
with regards,
suman
MBR, Sergei
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
