* [PATCH 8/9] [SCSI] mvsas: update comments
[not found] <mvsas-comments>
@ 2011-05-24 14:38 ` yxlraid
2011-05-25 1:41 ` Jack Wang
0 siblings, 1 reply; 5+ messages in thread
From: yxlraid @ 2011-05-24 14:38 UTC (permalink / raw)
To: James.Bottomley; +Cc: jslaby, linux-scsi, linux-kernel, yuxiangl, jfeng
From: Xiangliang Yu <yuxiangl@marvell.com>
-- Update comments: remove obsolete comments and add new comments
Signed-off-by: Xiangliang Yu <yuxiangl@marvell.com>
---
drivers/scsi/mvsas/mv_64xx.c | 17 ++++-------------
drivers/scsi/mvsas/mv_94xx.c | 19 ++++++-------------
drivers/scsi/mvsas/mv_94xx.h | 24 ++++++++++++------------
drivers/scsi/mvsas/mv_chips.h | 2 --
drivers/scsi/mvsas/mv_defs.h | 2 +-
drivers/scsi/mvsas/mv_init.c | 1 -
drivers/scsi/mvsas/mv_sas.c | 14 +-------------
7 files changed, 24 insertions(+), 55 deletions(-)
diff --git a/drivers/scsi/mvsas/mv_64xx.c b/drivers/scsi/mvsas/mv_64xx.c
index bc75ba7..dec5f96 100644
--- a/drivers/scsi/mvsas/mv_64xx.c
+++ b/drivers/scsi/mvsas/mv_64xx.c
@@ -33,7 +33,6 @@ static void mvs_64xx_detect_porttype(struct mvs_info *mvi, int i)
u32 reg;
struct mvs_phy *phy = &mvi->phy[i];
- /* TODO check & save device type */
reg = mr32(MVS_GBL_PORT_TYPE);
phy->phy_type &= ~(PORT_TYPE_SAS | PORT_TYPE_SATA);
if (reg & MODE_SAS_SATA & (1 << i))
@@ -63,7 +62,6 @@ static void __devinit mvs_64xx_phy_hacks(struct mvs_info *mvi)
mvs_phy_hacks(mvi);
if (!(mvi->flags & MVF_FLAG_SOC)) {
- /* TEST - for phy decoding error, adjust voltage levels */
for (i = 0; i < MVS_SOC_PORTS; i++) {
mvs_write_port_vsr_addr(mvi, i, VSR_PHY_MODE8);
mvs_write_port_vsr_data(mvi, i, 0x2F0);
@@ -375,13 +373,7 @@ static int __devinit mvs_64xx_init(struct mvs_info *mvi)
mvs_update_phyinfo(mvi, i, 1);
}
- /* FIXME: update wide port bitmaps */
-
/* little endian for open address and command table, etc. */
- /*
- * it seems that ( from the spec ) turning on big-endian won't
- * do us any good on big-endian machines, need further confirmation
- */
cctl = mr32(MVS_CTL);
cctl |= CCTL_ENDIAN_CMD;
cctl |= CCTL_ENDIAN_DATA;
@@ -394,8 +386,8 @@ static int __devinit mvs_64xx_init(struct mvs_info *mvi)
tmp |= PCS_CMD_RST;
tmp &= ~PCS_SELF_CLEAR;
mw32(MVS_PCS, tmp);
- /* interrupt coalescing may cause missing HW interrput in some case,
- * and the max count is 0x1ff, while our max slot is 0x200,
+ /*
+ * the max count is 0x1ff, while our max slot is 0x200,
* it will make count 0.
*/
tmp = 0;
@@ -632,7 +624,6 @@ static void mvs_64xx_phy_work_around(struct mvs_info *mvi, int i)
{
u32 tmp;
struct mvs_phy *phy = &mvi->phy[i];
- /* workaround for HW phy decoding error on 1.5g disk drive */
mvs_write_port_vsr_addr(mvi, i, VSR_PHY_MODE6);
tmp = mvs_read_port_vsr_data(mvi, i);
if (((phy->phy_status & PHY_NEG_SPP_PHYS_LINK_RATE_MASK) >>
@@ -765,8 +756,8 @@ static void mvs_64xx_tune_interrupt(struct mvs_info *mvi, u32 time)
{
void __iomem *regs = mvi->regs;
u32 tmp = 0;
- /* interrupt coalescing may cause missing HW interrput in some case,
- * and the max count is 0x1ff, while our max slot is 0x200,
+ /*
+ * the max count is 0x1ff, while our max slot is 0x200,
* it will make count 0.
*/
if (time == 0) {
diff --git a/drivers/scsi/mvsas/mv_94xx.c b/drivers/scsi/mvsas/mv_94xx.c
index 1276e49..f4a995c 100644
--- a/drivers/scsi/mvsas/mv_94xx.c
+++ b/drivers/scsi/mvsas/mv_94xx.c
@@ -460,13 +460,7 @@ static int __devinit mvs_94xx_init(struct mvs_info *mvi)
mvs_update_phyinfo(mvi, i, 1);
}
- /* FIXME: update wide port bitmaps */
-
/* little endian for open address and command table, etc. */
- /*
- * it seems that ( from the spec ) turning on big-endian won't
- * do us any good on big-endian machines, need further confirmation
- */
cctl = mr32(MVS_CTL);
cctl |= CCTL_ENDIAN_CMD;
cctl &= ~CCTL_ENDIAN_OPEN;
@@ -478,8 +472,8 @@ static int __devinit mvs_94xx_init(struct mvs_info *mvi)
tmp |= PCS_CMD_RST;
tmp &= ~PCS_SELF_CLEAR;
mw32(MVS_PCS, tmp);
- /* interrupt coalescing may cause missing HW interrput in some case,
- * and the max count is 0x1ff, while our max slot is 0x200,
+ /*
+ * the max count is 0x1ff, while our max slot is 0x200,
* it will make count 0.
*/
tmp = 0;
@@ -488,6 +482,7 @@ static int __devinit mvs_94xx_init(struct mvs_info *mvi)
else
mw32(MVS_INT_COAL, MVS_CHIP_SLOT_SZ | COAL_EN);
+ /* default interrupt coalescing time is 128us */
tmp = 0x10000 | interrupt_coalescing;
mw32(MVS_INT_COAL_TMOUT, tmp);
@@ -745,7 +740,7 @@ static int mvs_94xx_oob_done(struct mvs_info *mvi, int i)
{
u32 phy_st;
phy_st = mvs_read_phy_ctl(mvi, i);
- if (phy_st & PHY_READY_MASK) /* phy ready */
+ if (phy_st & PHY_READY_MASK)
return 1;
return 0;
}
@@ -770,7 +765,6 @@ static void mvs_94xx_get_att_identify_frame(struct mvs_info *mvi, int port_id,
int i;
u32 id_frame[7];
- /* mvs_hexdump(28, (u8 *)id_frame, 0); */
for (i = 0; i < 7; i++) {
mvs_write_port_cfg_addr(mvi, port_id,
CONFIG_ATT_ID_FRAME0 + i * 4);
@@ -778,7 +772,6 @@ static void mvs_94xx_get_att_identify_frame(struct mvs_info *mvi, int port_id,
mv_dprintk("94xx phy %d atta frame %d %x.\n",
port_id + mvi->id * mvi->chip->n_phy, i, id_frame[i]);
}
- /* mvs_hexdump(28, (u8 *)id_frame, 0); */
memcpy(id, id_frame, 28);
}
@@ -962,8 +955,8 @@ static void mvs_94xx_tune_interrupt(struct mvs_info *mvi, u32 time)
{
void __iomem *regs = mvi->regs;
u32 tmp = 0;
- /* interrupt coalescing may cause missing HW interrput in some case,
- * and the max count is 0x1ff, while our max slot is 0x200,
+ /*
+ * the max count is 0x1ff, while our max slot is 0x200,
* it will make count 0.
*/
if (time == 0) {
diff --git a/drivers/scsi/mvsas/mv_94xx.h b/drivers/scsi/mvsas/mv_94xx.h
index b42532b..77a1e52 100644
--- a/drivers/scsi/mvsas/mv_94xx.h
+++ b/drivers/scsi/mvsas/mv_94xx.h
@@ -121,18 +121,18 @@ enum pci_cfg_registers {
/* SAS/SATA Vendor Specific Port Registers */
enum sas_sata_vsp_regs {
- VSR_PHY_STAT = 0x00 * 4, /* Phy Status */
- VSR_PHY_MODE1 = 0x01 * 4, /* phy tx */
- VSR_PHY_MODE2 = 0x02 * 4, /* tx scc */
- VSR_PHY_MODE3 = 0x03 * 4, /* pll */
- VSR_PHY_MODE4 = 0x04 * 4, /* VCO */
- VSR_PHY_MODE5 = 0x05 * 4, /* Rx */
- VSR_PHY_MODE6 = 0x06 * 4, /* CDR */
- VSR_PHY_MODE7 = 0x07 * 4, /* Impedance */
- VSR_PHY_MODE8 = 0x08 * 4, /* Voltage */
- VSR_PHY_MODE9 = 0x09 * 4, /* Test */
- VSR_PHY_MODE10 = 0x0A * 4, /* Power */
- VSR_PHY_MODE11 = 0x0B * 4, /* Phy Mode */
+ VSR_PHY_STAT = 0x00 * 4, /* Phy Interrupt Status */
+ VSR_PHY_MODE1 = 0x01 * 4, /* phy Interrupt Enable */
+ VSR_PHY_MODE2 = 0x02 * 4, /* Phy Configuration */
+ VSR_PHY_MODE3 = 0x03 * 4, /* Phy Status */
+ VSR_PHY_MODE4 = 0x04 * 4, /* Phy Counter 0 */
+ VSR_PHY_MODE5 = 0x05 * 4, /* Phy Counter 1 */
+ VSR_PHY_MODE6 = 0x06 * 4, /* Event Counter Control */
+ VSR_PHY_MODE7 = 0x07 * 4, /* Event Counter Select */
+ VSR_PHY_MODE8 = 0x08 * 4, /* Event Counter 0 */
+ VSR_PHY_MODE9 = 0x09 * 4, /* Event Counter 1 */
+ VSR_PHY_MODE10 = 0x0A * 4, /* Event Counter 2 */
+ VSR_PHY_MODE11 = 0x0B * 4, /* Event Counter 3 */
VSR_PHY_ACT_LED = 0x0C * 4, /* Activity LED control */
VSR_PHY_FFE_CONTROL = 0x10C,
diff --git a/drivers/scsi/mvsas/mv_chips.h b/drivers/scsi/mvsas/mv_chips.h
index 0a11bc7..bcc4080 100644
--- a/drivers/scsi/mvsas/mv_chips.h
+++ b/drivers/scsi/mvsas/mv_chips.h
@@ -164,7 +164,6 @@ static inline void __devinit mvs_phy_hacks(struct mvs_info *mvi)
{
u32 tmp;
- /* workaround for SATA R-ERR, to ignore phy glitch */
tmp = mvs_cr32(mvi, CMD_PHY_TIMER);
tmp &= ~(1 << 9);
tmp |= (1 << 10);
@@ -179,7 +178,6 @@ static inline void __devinit mvs_phy_hacks(struct mvs_info *mvi)
tmp |= 0x3fff;
mvs_cw32(mvi, CMD_SAS_CTL0, tmp);
- /* workaround for WDTIMEOUT , set to 550 ms */
mvs_cw32(mvi, CMD_WD_TIMER, 0x7a0000);
/* not to halt for different port op during wideport link change */
diff --git a/drivers/scsi/mvsas/mv_defs.h b/drivers/scsi/mvsas/mv_defs.h
index 1927e1b..dec7cad 100644
--- a/drivers/scsi/mvsas/mv_defs.h
+++ b/drivers/scsi/mvsas/mv_defs.h
@@ -160,7 +160,7 @@ enum hw_register_bits {
TXQ_CMD_SSP = 1, /* SSP protocol */
TXQ_CMD_SMP = 2, /* SMP protocol */
TXQ_CMD_STP = 3, /* STP/SATA protocol */
- TXQ_CMD_SSP_FREE_LIST = 4, /* add to SSP targ free list */
+ TXQ_CMD_SSP_FREE_LIST = 4, /* add to SSP target free list */
TXQ_CMD_SLOT_RESET = 7, /* reset command slot */
TXQ_MODE_I = (1U << 28), /* mode: 0=target,1=initiator */
TXQ_MODE_TARGET = 0,
diff --git a/drivers/scsi/mvsas/mv_init.c b/drivers/scsi/mvsas/mv_init.c
index b28ee5b..cf4aaa9 100644
--- a/drivers/scsi/mvsas/mv_init.c
+++ b/drivers/scsi/mvsas/mv_init.c
@@ -405,7 +405,6 @@ err_out:
return NULL;
}
-/* move to PCI layer or libata core? */
static int pci_go_64(struct pci_dev *pdev)
{
int rc;
diff --git a/drivers/scsi/mvsas/mv_sas.c b/drivers/scsi/mvsas/mv_sas.c
index 31ca8fe..4958fef 100644
--- a/drivers/scsi/mvsas/mv_sas.c
+++ b/drivers/scsi/mvsas/mv_sas.c
@@ -102,7 +102,6 @@ struct mvs_info *mvs_find_dev_mvi(struct domain_device *dev)
}
-/* FIXME */
int mvs_find_dev_phyno(struct domain_device *dev, int *phyno)
{
unsigned long i = 0, j = 0, n = 0, num = 0;
@@ -177,7 +176,6 @@ void mvs_phys_reset(struct mvs_info *mvi, u32 phy_mask, int hard)
}
}
-/* FIXME: locking? */
int mvs_phy_control(struct asd_sas_phy *sas_phy, enum phy_func func,
void *funcdata)
{
@@ -504,11 +502,8 @@ static int mvs_task_prep_ata(struct mvs_info *mvi,
flags |= MCH_ATAPI;
}
- /* FIXME: fill in port multiplier number */
-
hdr->flags = cpu_to_le32(flags);
- /* FIXME: the low order order 5 bits for the TAG if enable NCQ */
if (task->ata_task.use_ncq && mvs_get_ncq_tag(task, &hdr_tag))
task->ata_task.fis.sector_count |= (u8) (hdr_tag << 3);
else
@@ -552,9 +547,6 @@ static int mvs_task_prep_ata(struct mvs_info *mvi,
buf_tmp_dma += i;
/* region 4: status buffer (larger the PRD, smaller this buf) ****** */
- /* FIXME: probably unused, for SATA. kept here just in case
- * we get a STP/SATA error information record
- */
slot->response = buf_tmp;
hdr->status_buf = cpu_to_le64(buf_tmp_dma);
if (mvi->flags & MVF_FLAG_SOC)
@@ -1126,7 +1118,6 @@ static void *mvs_get_d2h_reg(struct mvs_info *mvi, int i, void *buf)
MVS_CHIP_DISP->write_port_cfg_addr(mvi, i, PHYR_SATA_SIG0);
s[0] = cpu_to_le32(MVS_CHIP_DISP->read_port_cfg_data(mvi, i));
- /* Workaround: take some ATAPI devices for ATA */
if (((s[1] & 0x00FFFFFF) == 0x00EB1401) && (*(u8 *)&s[3] == 0x01))
s[1] = 0x00EB1401 | (*((u8 *)&s[1] + 3) & 0x10);
@@ -1433,7 +1424,6 @@ static void mvs_tmf_timedout(unsigned long data)
complete(&task->completion);
}
-/* XXX */
#define MVS_TASK_TIMEOUT 20
static int mvs_exec_internal_tmf_task(struct domain_device *dev,
void *parameter, u32 para_len, struct mvs_tmf_task *tmf)
@@ -1577,7 +1567,6 @@ int mvs_I_T_nexus_reset(struct domain_device *dev)
mv_printk("%s for device[%x]:rc= %d\n",
__func__, mvi_dev->device_id, rc);
- /* housekeeper */
spin_lock_irqsave(&mvi->lock, flags);
mvs_release_task(mvi, dev);
spin_unlock_irqrestore(&mvi->lock, flags);
@@ -1681,7 +1670,6 @@ int mvs_abort_task(struct sas_task *task)
} else if (task->task_proto & SAS_PROTOCOL_SATA ||
task->task_proto & SAS_PROTOCOL_STP) {
- /* to do free register_set */
if (SATA_DEV == dev->dev_type) {
struct mvs_slot_info *slot = task->lldd_task;
u32 slot_idx = (u32)(slot - mvi->slot_info);
@@ -1901,6 +1889,7 @@ int mvs_slot_complete(struct mvs_info *mvi, u32 rx_desc, u32 flags)
return -1;
}
+ /* when no device attaching, go ahead and complete by error handling*/
if (unlikely(!mvi_dev || flags)) {
if (!mvi_dev)
mv_dprintk("port has not device.\n");
@@ -2017,7 +2006,6 @@ void mvs_release_task(struct mvs_info *mvi,
struct domain_device *dev)
{
int i, phyno[WIDE_PORT_MAX_PHY], num;
- /* housekeeper */
num = mvs_find_dev_phyno(dev, phyno);
for (i = 0; i < num; i++)
mvs_do_release_task(mvi, phyno[i], dev);
--
1.7.4.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
* RE: [PATCH 8/9] [SCSI] mvsas: update comments
2011-05-24 14:38 ` [PATCH 8/9] [SCSI] mvsas: update comments yxlraid
@ 2011-05-25 1:41 ` Jack Wang
2011-05-25 2:05 ` Xiangliang Yu
0 siblings, 1 reply; 5+ messages in thread
From: Jack Wang @ 2011-05-25 1:41 UTC (permalink / raw)
To: yxlraid; +Cc: linux-scsi, yuxiangl
Where is the [PATCH 9/9]?
> From: Xiangliang Yu <yuxiangl@marvell.com>
>
> -- Update comments: remove obsolete comments and add new comments
>
> Signed-off-by: Xiangliang Yu <yuxiangl@marvell.com>
> ---
> drivers/scsi/mvsas/mv_64xx.c | 17 ++++-------------
> drivers/scsi/mvsas/mv_94xx.c | 19 ++++++-------------
> drivers/scsi/mvsas/mv_94xx.h | 24 ++++++++++++------------
> drivers/scsi/mvsas/mv_chips.h | 2 --
> drivers/scsi/mvsas/mv_defs.h | 2 +-
> drivers/scsi/mvsas/mv_init.c | 1 -
> drivers/scsi/mvsas/mv_sas.c | 14 +-------------
> 7 files changed, 24 insertions(+), 55 deletions(-)
>
> diff --git a/drivers/scsi/mvsas/mv_64xx.c b/drivers/scsi/mvsas/mv_64xx.c
> index bc75ba7..dec5f96 100644
> --- a/drivers/scsi/mvsas/mv_64xx.c
> +++ b/drivers/scsi/mvsas/mv_64xx.c
> @@ -33,7 +33,6 @@ static void mvs_64xx_detect_porttype(struct mvs_info
*mvi,
> int i)
> u32 reg;
> struct mvs_phy *phy = &mvi->phy[i];
>
> - /* TODO check & save device type */
> reg = mr32(MVS_GBL_PORT_TYPE);
> phy->phy_type &= ~(PORT_TYPE_SAS | PORT_TYPE_SATA);
> if (reg & MODE_SAS_SATA & (1 << i))
> @@ -63,7 +62,6 @@ static void __devinit mvs_64xx_phy_hacks(struct mvs_info
> *mvi)
> mvs_phy_hacks(mvi);
>
> if (!(mvi->flags & MVF_FLAG_SOC)) {
> - /* TEST - for phy decoding error, adjust voltage levels */
> for (i = 0; i < MVS_SOC_PORTS; i++) {
> mvs_write_port_vsr_addr(mvi, i, VSR_PHY_MODE8);
> mvs_write_port_vsr_data(mvi, i, 0x2F0);
> @@ -375,13 +373,7 @@ static int __devinit mvs_64xx_init(struct mvs_info
*mvi)
> mvs_update_phyinfo(mvi, i, 1);
> }
>
> - /* FIXME: update wide port bitmaps */
> -
> /* little endian for open address and command table, etc. */
> - /*
> - * it seems that ( from the spec ) turning on big-endian won't
> - * do us any good on big-endian machines, need further confirmation
> - */
> cctl = mr32(MVS_CTL);
> cctl |= CCTL_ENDIAN_CMD;
> cctl |= CCTL_ENDIAN_DATA;
> @@ -394,8 +386,8 @@ static int __devinit mvs_64xx_init(struct mvs_info
*mvi)
> tmp |= PCS_CMD_RST;
> tmp &= ~PCS_SELF_CLEAR;
> mw32(MVS_PCS, tmp);
> - /* interrupt coalescing may cause missing HW interrput in some case,
> - * and the max count is 0x1ff, while our max slot is 0x200,
> + /*
> + * the max count is 0x1ff, while our max slot is 0x200,
> * it will make count 0.
> */
> tmp = 0;
> @@ -632,7 +624,6 @@ static void mvs_64xx_phy_work_around(struct mvs_info
*mvi,
> int i)
> {
> u32 tmp;
> struct mvs_phy *phy = &mvi->phy[i];
> - /* workaround for HW phy decoding error on 1.5g disk drive */
> mvs_write_port_vsr_addr(mvi, i, VSR_PHY_MODE6);
> tmp = mvs_read_port_vsr_data(mvi, i);
> if (((phy->phy_status & PHY_NEG_SPP_PHYS_LINK_RATE_MASK) >>
> @@ -765,8 +756,8 @@ static void mvs_64xx_tune_interrupt(struct mvs_info
*mvi,
> u32 time)
> {
> void __iomem *regs = mvi->regs;
> u32 tmp = 0;
> - /* interrupt coalescing may cause missing HW interrput in some case,
> - * and the max count is 0x1ff, while our max slot is 0x200,
> + /*
> + * the max count is 0x1ff, while our max slot is 0x200,
> * it will make count 0.
> */
> if (time == 0) {
> diff --git a/drivers/scsi/mvsas/mv_94xx.c b/drivers/scsi/mvsas/mv_94xx.c
> index 1276e49..f4a995c 100644
> --- a/drivers/scsi/mvsas/mv_94xx.c
> +++ b/drivers/scsi/mvsas/mv_94xx.c
> @@ -460,13 +460,7 @@ static int __devinit mvs_94xx_init(struct mvs_info
*mvi)
> mvs_update_phyinfo(mvi, i, 1);
> }
>
> - /* FIXME: update wide port bitmaps */
> -
> /* little endian for open address and command table, etc. */
> - /*
> - * it seems that ( from the spec ) turning on big-endian won't
> - * do us any good on big-endian machines, need further confirmation
> - */
> cctl = mr32(MVS_CTL);
> cctl |= CCTL_ENDIAN_CMD;
> cctl &= ~CCTL_ENDIAN_OPEN;
> @@ -478,8 +472,8 @@ static int __devinit mvs_94xx_init(struct mvs_info
*mvi)
> tmp |= PCS_CMD_RST;
> tmp &= ~PCS_SELF_CLEAR;
> mw32(MVS_PCS, tmp);
> - /* interrupt coalescing may cause missing HW interrput in some case,
> - * and the max count is 0x1ff, while our max slot is 0x200,
> + /*
> + * the max count is 0x1ff, while our max slot is 0x200,
> * it will make count 0.
> */
> tmp = 0;
> @@ -488,6 +482,7 @@ static int __devinit mvs_94xx_init(struct mvs_info
*mvi)
> else
> mw32(MVS_INT_COAL, MVS_CHIP_SLOT_SZ | COAL_EN);
>
> + /* default interrupt coalescing time is 128us */
> tmp = 0x10000 | interrupt_coalescing;
> mw32(MVS_INT_COAL_TMOUT, tmp);
>
> @@ -745,7 +740,7 @@ static int mvs_94xx_oob_done(struct mvs_info *mvi, int
i)
> {
> u32 phy_st;
> phy_st = mvs_read_phy_ctl(mvi, i);
> - if (phy_st & PHY_READY_MASK) /* phy ready */
> + if (phy_st & PHY_READY_MASK)
> return 1;
> return 0;
> }
> @@ -770,7 +765,6 @@ static void mvs_94xx_get_att_identify_frame(struct
> mvs_info *mvi, int port_id,
> int i;
> u32 id_frame[7];
>
> - /* mvs_hexdump(28, (u8 *)id_frame, 0); */
> for (i = 0; i < 7; i++) {
> mvs_write_port_cfg_addr(mvi, port_id,
> CONFIG_ATT_ID_FRAME0 + i * 4);
> @@ -778,7 +772,6 @@ static void mvs_94xx_get_att_identify_frame(struct
> mvs_info *mvi, int port_id,
> mv_dprintk("94xx phy %d atta frame %d %x.\n",
> port_id + mvi->id * mvi->chip->n_phy, i,
id_frame[i]);
> }
> - /* mvs_hexdump(28, (u8 *)id_frame, 0); */
> memcpy(id, id_frame, 28);
> }
>
> @@ -962,8 +955,8 @@ static void mvs_94xx_tune_interrupt(struct mvs_info
*mvi,
> u32 time)
> {
> void __iomem *regs = mvi->regs;
> u32 tmp = 0;
> - /* interrupt coalescing may cause missing HW interrput in some case,
> - * and the max count is 0x1ff, while our max slot is 0x200,
> + /*
> + * the max count is 0x1ff, while our max slot is 0x200,
> * it will make count 0.
> */
> if (time == 0) {
> diff --git a/drivers/scsi/mvsas/mv_94xx.h b/drivers/scsi/mvsas/mv_94xx.h
> index b42532b..77a1e52 100644
> --- a/drivers/scsi/mvsas/mv_94xx.h
> +++ b/drivers/scsi/mvsas/mv_94xx.h
> @@ -121,18 +121,18 @@ enum pci_cfg_registers {
>
> /* SAS/SATA Vendor Specific Port Registers */
> enum sas_sata_vsp_regs {
> - VSR_PHY_STAT = 0x00 * 4, /* Phy Status */
> - VSR_PHY_MODE1 = 0x01 * 4, /* phy tx */
> - VSR_PHY_MODE2 = 0x02 * 4, /* tx scc */
> - VSR_PHY_MODE3 = 0x03 * 4, /* pll */
> - VSR_PHY_MODE4 = 0x04 * 4, /* VCO */
> - VSR_PHY_MODE5 = 0x05 * 4, /* Rx */
> - VSR_PHY_MODE6 = 0x06 * 4, /* CDR */
> - VSR_PHY_MODE7 = 0x07 * 4, /* Impedance */
> - VSR_PHY_MODE8 = 0x08 * 4, /* Voltage */
> - VSR_PHY_MODE9 = 0x09 * 4, /* Test */
> - VSR_PHY_MODE10 = 0x0A * 4, /* Power */
> - VSR_PHY_MODE11 = 0x0B * 4, /* Phy Mode */
> + VSR_PHY_STAT = 0x00 * 4, /* Phy Interrupt Status */
> + VSR_PHY_MODE1 = 0x01 * 4, /* phy Interrupt Enable */
> + VSR_PHY_MODE2 = 0x02 * 4, /* Phy Configuration */
> + VSR_PHY_MODE3 = 0x03 * 4, /* Phy Status */
> + VSR_PHY_MODE4 = 0x04 * 4, /* Phy Counter 0 */
> + VSR_PHY_MODE5 = 0x05 * 4, /* Phy Counter 1 */
> + VSR_PHY_MODE6 = 0x06 * 4, /* Event Counter Control */
> + VSR_PHY_MODE7 = 0x07 * 4, /* Event Counter Select */
> + VSR_PHY_MODE8 = 0x08 * 4, /* Event Counter 0 */
> + VSR_PHY_MODE9 = 0x09 * 4, /* Event Counter 1 */
> + VSR_PHY_MODE10 = 0x0A * 4, /* Event Counter 2 */
> + VSR_PHY_MODE11 = 0x0B * 4, /* Event Counter 3 */
> VSR_PHY_ACT_LED = 0x0C * 4, /* Activity LED control */
>
> VSR_PHY_FFE_CONTROL = 0x10C,
> diff --git a/drivers/scsi/mvsas/mv_chips.h b/drivers/scsi/mvsas/mv_chips.h
> index 0a11bc7..bcc4080 100644
> --- a/drivers/scsi/mvsas/mv_chips.h
> +++ b/drivers/scsi/mvsas/mv_chips.h
> @@ -164,7 +164,6 @@ static inline void __devinit mvs_phy_hacks(struct
mvs_info
> *mvi)
> {
> u32 tmp;
>
> - /* workaround for SATA R-ERR, to ignore phy glitch */
> tmp = mvs_cr32(mvi, CMD_PHY_TIMER);
> tmp &= ~(1 << 9);
> tmp |= (1 << 10);
> @@ -179,7 +178,6 @@ static inline void __devinit mvs_phy_hacks(struct
mvs_info
> *mvi)
> tmp |= 0x3fff;
> mvs_cw32(mvi, CMD_SAS_CTL0, tmp);
>
> - /* workaround for WDTIMEOUT , set to 550 ms */
> mvs_cw32(mvi, CMD_WD_TIMER, 0x7a0000);
>
> /* not to halt for different port op during wideport link change */
> diff --git a/drivers/scsi/mvsas/mv_defs.h b/drivers/scsi/mvsas/mv_defs.h
> index 1927e1b..dec7cad 100644
> --- a/drivers/scsi/mvsas/mv_defs.h
> +++ b/drivers/scsi/mvsas/mv_defs.h
> @@ -160,7 +160,7 @@ enum hw_register_bits {
> TXQ_CMD_SSP = 1, /* SSP protocol */
> TXQ_CMD_SMP = 2, /* SMP protocol */
> TXQ_CMD_STP = 3, /* STP/SATA protocol */
> - TXQ_CMD_SSP_FREE_LIST = 4, /* add to SSP targ free list
*/
> + TXQ_CMD_SSP_FREE_LIST = 4, /* add to SSP target free
list */
> TXQ_CMD_SLOT_RESET = 7, /* reset command slot */
> TXQ_MODE_I = (1U << 28), /* mode:
0=target,1=initiator */
> TXQ_MODE_TARGET = 0,
> diff --git a/drivers/scsi/mvsas/mv_init.c b/drivers/scsi/mvsas/mv_init.c
> index b28ee5b..cf4aaa9 100644
> --- a/drivers/scsi/mvsas/mv_init.c
> +++ b/drivers/scsi/mvsas/mv_init.c
> @@ -405,7 +405,6 @@ err_out:
> return NULL;
> }
>
> -/* move to PCI layer or libata core? */
> static int pci_go_64(struct pci_dev *pdev)
> {
> int rc;
> diff --git a/drivers/scsi/mvsas/mv_sas.c b/drivers/scsi/mvsas/mv_sas.c
> index 31ca8fe..4958fef 100644
> --- a/drivers/scsi/mvsas/mv_sas.c
> +++ b/drivers/scsi/mvsas/mv_sas.c
> @@ -102,7 +102,6 @@ struct mvs_info *mvs_find_dev_mvi(struct domain_device
> *dev)
>
> }
>
> -/* FIXME */
> int mvs_find_dev_phyno(struct domain_device *dev, int *phyno)
> {
> unsigned long i = 0, j = 0, n = 0, num = 0;
> @@ -177,7 +176,6 @@ void mvs_phys_reset(struct mvs_info *mvi, u32
phy_mask,
> int hard)
> }
> }
>
> -/* FIXME: locking? */
> int mvs_phy_control(struct asd_sas_phy *sas_phy, enum phy_func func,
> void *funcdata)
> {
> @@ -504,11 +502,8 @@ static int mvs_task_prep_ata(struct mvs_info *mvi,
> flags |= MCH_ATAPI;
> }
>
> - /* FIXME: fill in port multiplier number */
> -
> hdr->flags = cpu_to_le32(flags);
>
> - /* FIXME: the low order order 5 bits for the TAG if enable NCQ */
> if (task->ata_task.use_ncq && mvs_get_ncq_tag(task, &hdr_tag))
> task->ata_task.fis.sector_count |= (u8) (hdr_tag << 3);
> else
> @@ -552,9 +547,6 @@ static int mvs_task_prep_ata(struct mvs_info *mvi,
> buf_tmp_dma += i;
>
> /* region 4: status buffer (larger the PRD, smaller this buf) ******
*/
> - /* FIXME: probably unused, for SATA. kept here just in case
> - * we get a STP/SATA error information record
> - */
> slot->response = buf_tmp;
> hdr->status_buf = cpu_to_le64(buf_tmp_dma);
> if (mvi->flags & MVF_FLAG_SOC)
> @@ -1126,7 +1118,6 @@ static void *mvs_get_d2h_reg(struct mvs_info *mvi,
int
> i, void *buf)
> MVS_CHIP_DISP->write_port_cfg_addr(mvi, i, PHYR_SATA_SIG0);
> s[0] = cpu_to_le32(MVS_CHIP_DISP->read_port_cfg_data(mvi, i));
>
> - /* Workaround: take some ATAPI devices for ATA */
> if (((s[1] & 0x00FFFFFF) == 0x00EB1401) && (*(u8 *)&s[3] == 0x01))
> s[1] = 0x00EB1401 | (*((u8 *)&s[1] + 3) & 0x10);
>
> @@ -1433,7 +1424,6 @@ static void mvs_tmf_timedout(unsigned long data)
> complete(&task->completion);
> }
>
> -/* XXX */
> #define MVS_TASK_TIMEOUT 20
> static int mvs_exec_internal_tmf_task(struct domain_device *dev,
> void *parameter, u32 para_len, struct mvs_tmf_task
*tmf)
> @@ -1577,7 +1567,6 @@ int mvs_I_T_nexus_reset(struct domain_device *dev)
> mv_printk("%s for device[%x]:rc= %d\n",
> __func__, mvi_dev->device_id, rc);
>
> - /* housekeeper */
> spin_lock_irqsave(&mvi->lock, flags);
> mvs_release_task(mvi, dev);
> spin_unlock_irqrestore(&mvi->lock, flags);
> @@ -1681,7 +1670,6 @@ int mvs_abort_task(struct sas_task *task)
>
> } else if (task->task_proto & SAS_PROTOCOL_SATA ||
> task->task_proto & SAS_PROTOCOL_STP) {
> - /* to do free register_set */
> if (SATA_DEV == dev->dev_type) {
> struct mvs_slot_info *slot = task->lldd_task;
> u32 slot_idx = (u32)(slot - mvi->slot_info);
> @@ -1901,6 +1889,7 @@ int mvs_slot_complete(struct mvs_info *mvi, u32
rx_desc,
> u32 flags)
> return -1;
> }
>
> + /* when no device attaching, go ahead and complete by error
handling*/
> if (unlikely(!mvi_dev || flags)) {
> if (!mvi_dev)
> mv_dprintk("port has not device.\n");
> @@ -2017,7 +2006,6 @@ void mvs_release_task(struct mvs_info *mvi,
> struct domain_device *dev)
> {
> int i, phyno[WIDE_PORT_MAX_PHY], num;
> - /* housekeeper */
> num = mvs_find_dev_phyno(dev, phyno);
> for (i = 0; i < num; i++)
> mvs_do_release_task(mvi, phyno[i], dev);
> --
> 1.7.4.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH 8/9] [SCSI] mvsas: update comments
2011-05-25 1:41 ` Jack Wang
@ 2011-05-25 2:05 ` Xiangliang Yu
2011-06-29 15:46 ` James Bottomley
0 siblings, 1 reply; 5+ messages in thread
From: Xiangliang Yu @ 2011-05-25 2:05 UTC (permalink / raw)
To: Jack Wang, yxlraid; +Cc: linux-scsi
I find a issue in [PATCH 9/9] before committing. I will commit the patch when the issue is fixed
-----Original Message-----
From: Jack Wang [mailto:jack_wang@usish.com]
Sent: Wednesday, May 25, 2011 9:42 AM
To: yxlraid@gmail.com
Cc: linux-scsi@vger.kernel.org; Xiangliang Yu
Subject: RE: [PATCH 8/9] [SCSI] mvsas: update comments
Where is the [PATCH 9/9]?
> From: Xiangliang Yu <yuxiangl@marvell.com>
>
> -- Update comments: remove obsolete comments and add new comments
>
> Signed-off-by: Xiangliang Yu <yuxiangl@marvell.com>
> ---
> drivers/scsi/mvsas/mv_64xx.c | 17 ++++-------------
> drivers/scsi/mvsas/mv_94xx.c | 19 ++++++-------------
> drivers/scsi/mvsas/mv_94xx.h | 24 ++++++++++++------------
> drivers/scsi/mvsas/mv_chips.h | 2 --
> drivers/scsi/mvsas/mv_defs.h | 2 +-
> drivers/scsi/mvsas/mv_init.c | 1 -
> drivers/scsi/mvsas/mv_sas.c | 14 +-------------
> 7 files changed, 24 insertions(+), 55 deletions(-)
>
> diff --git a/drivers/scsi/mvsas/mv_64xx.c b/drivers/scsi/mvsas/mv_64xx.c
> index bc75ba7..dec5f96 100644
> --- a/drivers/scsi/mvsas/mv_64xx.c
> +++ b/drivers/scsi/mvsas/mv_64xx.c
> @@ -33,7 +33,6 @@ static void mvs_64xx_detect_porttype(struct mvs_info
*mvi,
> int i)
> u32 reg;
> struct mvs_phy *phy = &mvi->phy[i];
>
> - /* TODO check & save device type */
> reg = mr32(MVS_GBL_PORT_TYPE);
> phy->phy_type &= ~(PORT_TYPE_SAS | PORT_TYPE_SATA);
> if (reg & MODE_SAS_SATA & (1 << i))
> @@ -63,7 +62,6 @@ static void __devinit mvs_64xx_phy_hacks(struct mvs_info
> *mvi)
> mvs_phy_hacks(mvi);
>
> if (!(mvi->flags & MVF_FLAG_SOC)) {
> - /* TEST - for phy decoding error, adjust voltage levels */
> for (i = 0; i < MVS_SOC_PORTS; i++) {
> mvs_write_port_vsr_addr(mvi, i, VSR_PHY_MODE8);
> mvs_write_port_vsr_data(mvi, i, 0x2F0);
> @@ -375,13 +373,7 @@ static int __devinit mvs_64xx_init(struct mvs_info
*mvi)
> mvs_update_phyinfo(mvi, i, 1);
> }
>
> - /* FIXME: update wide port bitmaps */
> -
> /* little endian for open address and command table, etc. */
> - /*
> - * it seems that ( from the spec ) turning on big-endian won't
> - * do us any good on big-endian machines, need further confirmation
> - */
> cctl = mr32(MVS_CTL);
> cctl |= CCTL_ENDIAN_CMD;
> cctl |= CCTL_ENDIAN_DATA;
> @@ -394,8 +386,8 @@ static int __devinit mvs_64xx_init(struct mvs_info
*mvi)
> tmp |= PCS_CMD_RST;
> tmp &= ~PCS_SELF_CLEAR;
> mw32(MVS_PCS, tmp);
> - /* interrupt coalescing may cause missing HW interrput in some case,
> - * and the max count is 0x1ff, while our max slot is 0x200,
> + /*
> + * the max count is 0x1ff, while our max slot is 0x200,
> * it will make count 0.
> */
> tmp = 0;
> @@ -632,7 +624,6 @@ static void mvs_64xx_phy_work_around(struct mvs_info
*mvi,
> int i)
> {
> u32 tmp;
> struct mvs_phy *phy = &mvi->phy[i];
> - /* workaround for HW phy decoding error on 1.5g disk drive */
> mvs_write_port_vsr_addr(mvi, i, VSR_PHY_MODE6);
> tmp = mvs_read_port_vsr_data(mvi, i);
> if (((phy->phy_status & PHY_NEG_SPP_PHYS_LINK_RATE_MASK) >>
> @@ -765,8 +756,8 @@ static void mvs_64xx_tune_interrupt(struct mvs_info
*mvi,
> u32 time)
> {
> void __iomem *regs = mvi->regs;
> u32 tmp = 0;
> - /* interrupt coalescing may cause missing HW interrput in some case,
> - * and the max count is 0x1ff, while our max slot is 0x200,
> + /*
> + * the max count is 0x1ff, while our max slot is 0x200,
> * it will make count 0.
> */
> if (time == 0) {
> diff --git a/drivers/scsi/mvsas/mv_94xx.c b/drivers/scsi/mvsas/mv_94xx.c
> index 1276e49..f4a995c 100644
> --- a/drivers/scsi/mvsas/mv_94xx.c
> +++ b/drivers/scsi/mvsas/mv_94xx.c
> @@ -460,13 +460,7 @@ static int __devinit mvs_94xx_init(struct mvs_info
*mvi)
> mvs_update_phyinfo(mvi, i, 1);
> }
>
> - /* FIXME: update wide port bitmaps */
> -
> /* little endian for open address and command table, etc. */
> - /*
> - * it seems that ( from the spec ) turning on big-endian won't
> - * do us any good on big-endian machines, need further confirmation
> - */
> cctl = mr32(MVS_CTL);
> cctl |= CCTL_ENDIAN_CMD;
> cctl &= ~CCTL_ENDIAN_OPEN;
> @@ -478,8 +472,8 @@ static int __devinit mvs_94xx_init(struct mvs_info
*mvi)
> tmp |= PCS_CMD_RST;
> tmp &= ~PCS_SELF_CLEAR;
> mw32(MVS_PCS, tmp);
> - /* interrupt coalescing may cause missing HW interrput in some case,
> - * and the max count is 0x1ff, while our max slot is 0x200,
> + /*
> + * the max count is 0x1ff, while our max slot is 0x200,
> * it will make count 0.
> */
> tmp = 0;
> @@ -488,6 +482,7 @@ static int __devinit mvs_94xx_init(struct mvs_info
*mvi)
> else
> mw32(MVS_INT_COAL, MVS_CHIP_SLOT_SZ | COAL_EN);
>
> + /* default interrupt coalescing time is 128us */
> tmp = 0x10000 | interrupt_coalescing;
> mw32(MVS_INT_COAL_TMOUT, tmp);
>
> @@ -745,7 +740,7 @@ static int mvs_94xx_oob_done(struct mvs_info *mvi, int
i)
> {
> u32 phy_st;
> phy_st = mvs_read_phy_ctl(mvi, i);
> - if (phy_st & PHY_READY_MASK) /* phy ready */
> + if (phy_st & PHY_READY_MASK)
> return 1;
> return 0;
> }
> @@ -770,7 +765,6 @@ static void mvs_94xx_get_att_identify_frame(struct
> mvs_info *mvi, int port_id,
> int i;
> u32 id_frame[7];
>
> - /* mvs_hexdump(28, (u8 *)id_frame, 0); */
> for (i = 0; i < 7; i++) {
> mvs_write_port_cfg_addr(mvi, port_id,
> CONFIG_ATT_ID_FRAME0 + i * 4);
> @@ -778,7 +772,6 @@ static void mvs_94xx_get_att_identify_frame(struct
> mvs_info *mvi, int port_id,
> mv_dprintk("94xx phy %d atta frame %d %x.\n",
> port_id + mvi->id * mvi->chip->n_phy, i,
id_frame[i]);
> }
> - /* mvs_hexdump(28, (u8 *)id_frame, 0); */
> memcpy(id, id_frame, 28);
> }
>
> @@ -962,8 +955,8 @@ static void mvs_94xx_tune_interrupt(struct mvs_info
*mvi,
> u32 time)
> {
> void __iomem *regs = mvi->regs;
> u32 tmp = 0;
> - /* interrupt coalescing may cause missing HW interrput in some case,
> - * and the max count is 0x1ff, while our max slot is 0x200,
> + /*
> + * the max count is 0x1ff, while our max slot is 0x200,
> * it will make count 0.
> */
> if (time == 0) {
> diff --git a/drivers/scsi/mvsas/mv_94xx.h b/drivers/scsi/mvsas/mv_94xx.h
> index b42532b..77a1e52 100644
> --- a/drivers/scsi/mvsas/mv_94xx.h
> +++ b/drivers/scsi/mvsas/mv_94xx.h
> @@ -121,18 +121,18 @@ enum pci_cfg_registers {
>
> /* SAS/SATA Vendor Specific Port Registers */
> enum sas_sata_vsp_regs {
> - VSR_PHY_STAT = 0x00 * 4, /* Phy Status */
> - VSR_PHY_MODE1 = 0x01 * 4, /* phy tx */
> - VSR_PHY_MODE2 = 0x02 * 4, /* tx scc */
> - VSR_PHY_MODE3 = 0x03 * 4, /* pll */
> - VSR_PHY_MODE4 = 0x04 * 4, /* VCO */
> - VSR_PHY_MODE5 = 0x05 * 4, /* Rx */
> - VSR_PHY_MODE6 = 0x06 * 4, /* CDR */
> - VSR_PHY_MODE7 = 0x07 * 4, /* Impedance */
> - VSR_PHY_MODE8 = 0x08 * 4, /* Voltage */
> - VSR_PHY_MODE9 = 0x09 * 4, /* Test */
> - VSR_PHY_MODE10 = 0x0A * 4, /* Power */
> - VSR_PHY_MODE11 = 0x0B * 4, /* Phy Mode */
> + VSR_PHY_STAT = 0x00 * 4, /* Phy Interrupt Status */
> + VSR_PHY_MODE1 = 0x01 * 4, /* phy Interrupt Enable */
> + VSR_PHY_MODE2 = 0x02 * 4, /* Phy Configuration */
> + VSR_PHY_MODE3 = 0x03 * 4, /* Phy Status */
> + VSR_PHY_MODE4 = 0x04 * 4, /* Phy Counter 0 */
> + VSR_PHY_MODE5 = 0x05 * 4, /* Phy Counter 1 */
> + VSR_PHY_MODE6 = 0x06 * 4, /* Event Counter Control */
> + VSR_PHY_MODE7 = 0x07 * 4, /* Event Counter Select */
> + VSR_PHY_MODE8 = 0x08 * 4, /* Event Counter 0 */
> + VSR_PHY_MODE9 = 0x09 * 4, /* Event Counter 1 */
> + VSR_PHY_MODE10 = 0x0A * 4, /* Event Counter 2 */
> + VSR_PHY_MODE11 = 0x0B * 4, /* Event Counter 3 */
> VSR_PHY_ACT_LED = 0x0C * 4, /* Activity LED control */
>
> VSR_PHY_FFE_CONTROL = 0x10C,
> diff --git a/drivers/scsi/mvsas/mv_chips.h b/drivers/scsi/mvsas/mv_chips.h
> index 0a11bc7..bcc4080 100644
> --- a/drivers/scsi/mvsas/mv_chips.h
> +++ b/drivers/scsi/mvsas/mv_chips.h
> @@ -164,7 +164,6 @@ static inline void __devinit mvs_phy_hacks(struct
mvs_info
> *mvi)
> {
> u32 tmp;
>
> - /* workaround for SATA R-ERR, to ignore phy glitch */
> tmp = mvs_cr32(mvi, CMD_PHY_TIMER);
> tmp &= ~(1 << 9);
> tmp |= (1 << 10);
> @@ -179,7 +178,6 @@ static inline void __devinit mvs_phy_hacks(struct
mvs_info
> *mvi)
> tmp |= 0x3fff;
> mvs_cw32(mvi, CMD_SAS_CTL0, tmp);
>
> - /* workaround for WDTIMEOUT , set to 550 ms */
> mvs_cw32(mvi, CMD_WD_TIMER, 0x7a0000);
>
> /* not to halt for different port op during wideport link change */
> diff --git a/drivers/scsi/mvsas/mv_defs.h b/drivers/scsi/mvsas/mv_defs.h
> index 1927e1b..dec7cad 100644
> --- a/drivers/scsi/mvsas/mv_defs.h
> +++ b/drivers/scsi/mvsas/mv_defs.h
> @@ -160,7 +160,7 @@ enum hw_register_bits {
> TXQ_CMD_SSP = 1, /* SSP protocol */
> TXQ_CMD_SMP = 2, /* SMP protocol */
> TXQ_CMD_STP = 3, /* STP/SATA protocol */
> - TXQ_CMD_SSP_FREE_LIST = 4, /* add to SSP targ free list
*/
> + TXQ_CMD_SSP_FREE_LIST = 4, /* add to SSP target free
list */
> TXQ_CMD_SLOT_RESET = 7, /* reset command slot */
> TXQ_MODE_I = (1U << 28), /* mode:
0=target,1=initiator */
> TXQ_MODE_TARGET = 0,
> diff --git a/drivers/scsi/mvsas/mv_init.c b/drivers/scsi/mvsas/mv_init.c
> index b28ee5b..cf4aaa9 100644
> --- a/drivers/scsi/mvsas/mv_init.c
> +++ b/drivers/scsi/mvsas/mv_init.c
> @@ -405,7 +405,6 @@ err_out:
> return NULL;
> }
>
> -/* move to PCI layer or libata core? */
> static int pci_go_64(struct pci_dev *pdev)
> {
> int rc;
> diff --git a/drivers/scsi/mvsas/mv_sas.c b/drivers/scsi/mvsas/mv_sas.c
> index 31ca8fe..4958fef 100644
> --- a/drivers/scsi/mvsas/mv_sas.c
> +++ b/drivers/scsi/mvsas/mv_sas.c
> @@ -102,7 +102,6 @@ struct mvs_info *mvs_find_dev_mvi(struct domain_device
> *dev)
>
> }
>
> -/* FIXME */
> int mvs_find_dev_phyno(struct domain_device *dev, int *phyno)
> {
> unsigned long i = 0, j = 0, n = 0, num = 0;
> @@ -177,7 +176,6 @@ void mvs_phys_reset(struct mvs_info *mvi, u32
phy_mask,
> int hard)
> }
> }
>
> -/* FIXME: locking? */
> int mvs_phy_control(struct asd_sas_phy *sas_phy, enum phy_func func,
> void *funcdata)
> {
> @@ -504,11 +502,8 @@ static int mvs_task_prep_ata(struct mvs_info *mvi,
> flags |= MCH_ATAPI;
> }
>
> - /* FIXME: fill in port multiplier number */
> -
> hdr->flags = cpu_to_le32(flags);
>
> - /* FIXME: the low order order 5 bits for the TAG if enable NCQ */
> if (task->ata_task.use_ncq && mvs_get_ncq_tag(task, &hdr_tag))
> task->ata_task.fis.sector_count |= (u8) (hdr_tag << 3);
> else
> @@ -552,9 +547,6 @@ static int mvs_task_prep_ata(struct mvs_info *mvi,
> buf_tmp_dma += i;
>
> /* region 4: status buffer (larger the PRD, smaller this buf) ******
*/
> - /* FIXME: probably unused, for SATA. kept here just in case
> - * we get a STP/SATA error information record
> - */
> slot->response = buf_tmp;
> hdr->status_buf = cpu_to_le64(buf_tmp_dma);
> if (mvi->flags & MVF_FLAG_SOC)
> @@ -1126,7 +1118,6 @@ static void *mvs_get_d2h_reg(struct mvs_info *mvi,
int
> i, void *buf)
> MVS_CHIP_DISP->write_port_cfg_addr(mvi, i, PHYR_SATA_SIG0);
> s[0] = cpu_to_le32(MVS_CHIP_DISP->read_port_cfg_data(mvi, i));
>
> - /* Workaround: take some ATAPI devices for ATA */
> if (((s[1] & 0x00FFFFFF) == 0x00EB1401) && (*(u8 *)&s[3] == 0x01))
> s[1] = 0x00EB1401 | (*((u8 *)&s[1] + 3) & 0x10);
>
> @@ -1433,7 +1424,6 @@ static void mvs_tmf_timedout(unsigned long data)
> complete(&task->completion);
> }
>
> -/* XXX */
> #define MVS_TASK_TIMEOUT 20
> static int mvs_exec_internal_tmf_task(struct domain_device *dev,
> void *parameter, u32 para_len, struct mvs_tmf_task
*tmf)
> @@ -1577,7 +1567,6 @@ int mvs_I_T_nexus_reset(struct domain_device *dev)
> mv_printk("%s for device[%x]:rc= %d\n",
> __func__, mvi_dev->device_id, rc);
>
> - /* housekeeper */
> spin_lock_irqsave(&mvi->lock, flags);
> mvs_release_task(mvi, dev);
> spin_unlock_irqrestore(&mvi->lock, flags);
> @@ -1681,7 +1670,6 @@ int mvs_abort_task(struct sas_task *task)
>
> } else if (task->task_proto & SAS_PROTOCOL_SATA ||
> task->task_proto & SAS_PROTOCOL_STP) {
> - /* to do free register_set */
> if (SATA_DEV == dev->dev_type) {
> struct mvs_slot_info *slot = task->lldd_task;
> u32 slot_idx = (u32)(slot - mvi->slot_info);
> @@ -1901,6 +1889,7 @@ int mvs_slot_complete(struct mvs_info *mvi, u32
rx_desc,
> u32 flags)
> return -1;
> }
>
> + /* when no device attaching, go ahead and complete by error
handling*/
> if (unlikely(!mvi_dev || flags)) {
> if (!mvi_dev)
> mv_dprintk("port has not device.\n");
> @@ -2017,7 +2006,6 @@ void mvs_release_task(struct mvs_info *mvi,
> struct domain_device *dev)
> {
> int i, phyno[WIDE_PORT_MAX_PHY], num;
> - /* housekeeper */
> num = mvs_find_dev_phyno(dev, phyno);
> for (i = 0; i < num; i++)
> mvs_do_release_task(mvi, phyno[i], dev);
> --
> 1.7.4.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH 8/9] [SCSI] mvsas: update comments
2011-05-25 2:05 ` Xiangliang Yu
@ 2011-06-29 15:46 ` James Bottomley
2011-06-30 2:00 ` Xiangliang Yu
0 siblings, 1 reply; 5+ messages in thread
From: James Bottomley @ 2011-06-29 15:46 UTC (permalink / raw)
To: Xiangliang Yu; +Cc: Jack Wang, yxlraid, linux-scsi
On Tue, 2011-05-24 at 19:05 -0700, Xiangliang Yu wrote:
> I find a issue in [PATCH 9/9] before committing. I will commit the
> patch when the issue is fixed
So what do you want me to do here? You still need to add the ABI file
for GregKH in 3/9 and 9/9 still hasn't appeared yet.
James
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH 8/9] [SCSI] mvsas: update comments
2011-06-29 15:46 ` James Bottomley
@ 2011-06-30 2:00 ` Xiangliang Yu
0 siblings, 0 replies; 5+ messages in thread
From: Xiangliang Yu @ 2011-06-30 2:00 UTC (permalink / raw)
To: James Bottomley; +Cc: Jack Wang, yxlraid, linux-scsi, Jacky Feng
>> I find a issue in [PATCH 9/9] before committing. I will commit the
>> patch when the issue is fixed
>So what do you want me to do here? You still need to add the ABI file
>for GregKH in 3/9
Don't need to add the ABI file.
The mail is here: http://marc.info/?l=linux-scsi&m=130683194120196&w=2
>and 9/9 still hasn't appeared yet.
I will commit the patch tonight.
If any question, please let me known.
Thank you very much.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2011-06-30 2:05 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <mvsas-comments>
2011-05-24 14:38 ` [PATCH 8/9] [SCSI] mvsas: update comments yxlraid
2011-05-25 1:41 ` Jack Wang
2011-05-25 2:05 ` Xiangliang Yu
2011-06-29 15:46 ` James Bottomley
2011-06-30 2:00 ` Xiangliang Yu
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.