Re: [PATCH 2/2] ahci_xgene: Fix the dma state machine lockup for the ATA_CMD_PACKET PIO mode command.

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

 



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 linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux