| Message ID | Pine.WNT.4.64.0906291846100.2640@B00888-02.fsl.freescale.net (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Kumar Gala |
| Headers | show |
On Jun 29, 2009, at 8:22 AM, ashish kalra wrote: > Signed-off-by: Ashish Kalra <Ashish.Kalra@freescale.com> > --- > drivers/ata/sata_fsl.c | 10 ++++++++-- > 1 files changed, 8 insertions(+), 2 deletions(-) Can you provide more detail in the commit message about why this patch is needed or what 'asynchronous notification' gets us. - k > > > diff --git a/drivers/ata/sata_fsl.c b/drivers/ata/sata_fsl.c > index 94eaa43..5751145 100644 > --- a/drivers/ata/sata_fsl.c > +++ b/drivers/ata/sata_fsl.c > @@ -34,7 +34,7 @@ enum { > > SATA_FSL_HOST_FLAGS = (ATA_FLAG_SATA | ATA_FLAG_NO_LEGACY | > ATA_FLAG_MMIO | ATA_FLAG_PIO_DMA | > - ATA_FLAG_PMP | ATA_FLAG_NCQ), > + ATA_FLAG_PMP | ATA_FLAG_NCQ | ATA_FLAG_AN), > > SATA_FSL_MAX_CMDS = SATA_FSL_QUEUE_DEPTH, > SATA_FSL_CMD_HDR_SIZE = 16, /* 4 DWORDS */ > @@ -132,7 +132,7 @@ enum { > INT_ON_SINGL_DEVICE_ERR = (1 << 1), > INT_ON_CMD_COMPLETE = 1, > > - INT_ON_ERROR = INT_ON_FATAL_ERR | > + INT_ON_ERROR = INT_ON_FATAL_ERR | INT_ON_SNOTIFY_UPDATE | > INT_ON_PHYRDY_CHG | INT_ON_SINGL_DEVICE_ERR, > > /* > @@ -154,6 +154,7 @@ enum { > > DEFAULT_PORT_IRQ_ENABLE_MASK = IE_ON_FATAL_ERR | IE_ON_PHYRDY_CHG | > IE_ON_SIGNATURE_UPDATE | > + IE_ON_SNOTIFY_UPDATE | > IE_ON_SINGL_DEVICE_ERR | IE_ON_CMD_COMPLETE, > > EXT_INDIRECT_SEG_PRD_FLAG = (1 << 31), > @@ -1003,6 +1004,11 @@ static void sata_fsl_error_intr(struct > ata_port *ap) > freeze = 1; > } > > + /* Handle SDB FIS receive & notify update */ > + if (hstatus & INT_ON_SNOTIFY_UPDATE) { > + sata_async_notification(ap); > + } > + > /* Handle PHYRDY change notification */ > if (hstatus & INT_ON_PHYRDY_CHG) { > DPRINTK("SATA FSL: PHYRDY change indication\n"); > -- > 1.6.0 > > > _______________________________________________ > Linuxppc-dev mailing list > Linuxppc-dev@lists.ozlabs.org > https://lists.ozlabs.org/listinfo/linuxppc-dev
diff --git a/drivers/ata/sata_fsl.c b/drivers/ata/sata_fsl.c index 94eaa43..5751145 100644 --- a/drivers/ata/sata_fsl.c +++ b/drivers/ata/sata_fsl.c @@ -34,7 +34,7 @@ enum { SATA_FSL_HOST_FLAGS = (ATA_FLAG_SATA | ATA_FLAG_NO_LEGACY | ATA_FLAG_MMIO | ATA_FLAG_PIO_DMA | - ATA_FLAG_PMP | ATA_FLAG_NCQ), + ATA_FLAG_PMP | ATA_FLAG_NCQ | ATA_FLAG_AN), SATA_FSL_MAX_CMDS = SATA_FSL_QUEUE_DEPTH, SATA_FSL_CMD_HDR_SIZE = 16, /* 4 DWORDS */ @@ -132,7 +132,7 @@ enum { INT_ON_SINGL_DEVICE_ERR = (1 << 1), INT_ON_CMD_COMPLETE = 1, - INT_ON_ERROR = INT_ON_FATAL_ERR | + INT_ON_ERROR = INT_ON_FATAL_ERR | INT_ON_SNOTIFY_UPDATE | INT_ON_PHYRDY_CHG | INT_ON_SINGL_DEVICE_ERR, /* @@ -154,6 +154,7 @@ enum { DEFAULT_PORT_IRQ_ENABLE_MASK = IE_ON_FATAL_ERR | IE_ON_PHYRDY_CHG | IE_ON_SIGNATURE_UPDATE | + IE_ON_SNOTIFY_UPDATE | IE_ON_SINGL_DEVICE_ERR | IE_ON_CMD_COMPLETE, EXT_INDIRECT_SEG_PRD_FLAG = (1 << 31), @@ -1003,6 +1004,11 @@ static void sata_fsl_error_intr(struct ata_port *ap) freeze = 1; } + /* Handle SDB FIS receive & notify update */ + if (hstatus & INT_ON_SNOTIFY_UPDATE) { + sata_async_notification(ap); + } + /* Handle PHYRDY change notification */ if (hstatus & INT_ON_PHYRDY_CHG) { DPRINTK("SATA FSL: PHYRDY change indication\n");
Signed-off-by: Ashish Kalra <Ashish.Kalra@freescale.com> --- drivers/ata/sata_fsl.c | 10 ++++++++-- 1 files changed, 8 insertions(+), 2 deletions(-)