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 <stripa...@apm.com <mailto:stripa...@apm.com>>
    ---
       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 linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to