On Wed, Feb 24, 2021 at 4:48 PM Viswas G <Viswas.G@xxxxxxxxxxxxx> wrote: > > Producer index(PI) outbound queue and consumer index(CI) > for Outbound queue are in DMA memory. These values should > be reset to 0 during driver reinitialization. Why "reinitialization", the function init_default_table_values is called from chip init? > > Signed-off-by: Viswas G <Viswas.G@xxxxxxxxxxxxx> > Signed-off-by: Ruksar Devadi <Ruksar.devadi@xxxxxxxxxxxxx> > Signed-off-by: Ashokkumar N <Ashokkumar.N@xxxxxxxxxxxxx> > --- > drivers/scsi/pm8001/pm8001_hwi.c | 2 ++ > drivers/scsi/pm8001/pm80xx_hwi.c | 2 ++ > 2 files changed, 4 insertions(+) > > diff --git a/drivers/scsi/pm8001/pm8001_hwi.c b/drivers/scsi/pm8001/pm8001_hwi.c > index 4e0ce044ac69..783149b8b127 100644 > --- a/drivers/scsi/pm8001/pm8001_hwi.c > +++ b/drivers/scsi/pm8001/pm8001_hwi.c > @@ -240,6 +240,7 @@ static void init_default_table_values(struct pm8001_hba_info *pm8001_ha) > pm8001_ha->memoryMap.region[ci_offset + i].phys_addr_lo; > pm8001_ha->inbnd_q_tbl[i].ci_virt = > pm8001_ha->memoryMap.region[ci_offset + i].virt_ptr; > + pm8001_write_32(pm8001_ha->inbnd_q_tbl[i].ci_virt, 0, 0); > offsetib = i * 0x20; > pm8001_ha->inbnd_q_tbl[i].pi_pci_bar = > get_pci_bar_index(pm8001_mr32(addressib, > @@ -268,6 +269,7 @@ static void init_default_table_values(struct pm8001_hba_info *pm8001_ha) > 0 | (10 << 16) | (i << 24); > pm8001_ha->outbnd_q_tbl[i].pi_virt = > pm8001_ha->memoryMap.region[pi_offset + i].virt_ptr; > + pm8001_write_32(pm8001_ha->outbnd_q_tbl[i].pi_virt, 0, 0); > offsetob = i * 0x24; > pm8001_ha->outbnd_q_tbl[i].ci_pci_bar = > get_pci_bar_index(pm8001_mr32(addressob, > diff --git a/drivers/scsi/pm8001/pm80xx_hwi.c b/drivers/scsi/pm8001/pm80xx_hwi.c > index 1aa3a499c85a..0f2c57e054ac 100644 > --- a/drivers/scsi/pm8001/pm80xx_hwi.c > +++ b/drivers/scsi/pm8001/pm80xx_hwi.c > @@ -787,6 +787,7 @@ static void init_default_table_values(struct pm8001_hba_info *pm8001_ha) > pm8001_ha->memoryMap.region[ci_offset + i].phys_addr_lo; > pm8001_ha->inbnd_q_tbl[i].ci_virt = > pm8001_ha->memoryMap.region[ci_offset + i].virt_ptr; > + pm8001_write_32(pm8001_ha->inbnd_q_tbl[i].ci_virt, 0, 0); > offsetib = i * 0x20; > pm8001_ha->inbnd_q_tbl[i].pi_pci_bar = > get_pci_bar_index(pm8001_mr32(addressib, > @@ -820,6 +821,7 @@ static void init_default_table_values(struct pm8001_hba_info *pm8001_ha) > pm8001_ha->outbnd_q_tbl[i].interrup_vec_cnt_delay = (i << 24); > pm8001_ha->outbnd_q_tbl[i].pi_virt = > pm8001_ha->memoryMap.region[pi_offset + i].virt_ptr; > + pm8001_write_32(pm8001_ha->outbnd_q_tbl[i].pi_virt, 0, 0); > offsetob = i * 0x24; > pm8001_ha->outbnd_q_tbl[i].ci_pci_bar = > get_pci_bar_index(pm8001_mr32(addressob, > -- > 2.16.3 >