>-----Original Message----- >From: weiping zhang [mailto:zhangweiping@xxxxxxxxxxxxxxx] >Sent: Tuesday, August 08, 2017 10:46 AM >To: kashyap.desai@xxxxxxxxxxxx; sumit.saxena@xxxxxxxxxxxx; >shivasharan.srikanteshwara@xxxxxxxxxxxx >Cc: megaraidlinux.pdl@xxxxxxxxxxxx; linux-scsi@xxxxxxxxxxxxxxx >Subject: [PATCH v2] scsi: megaraid_sas: fix allocate instance->pd_info twice > >fix allocate instance->pd_info twice which was introduced by 96188a89cc6d. > >Signed-off-by: weiping zhang <zhangweiping@xxxxxxxxxxxxxxx> >--- > drivers/scsi/megaraid/megaraid_sas_base.c | 8 +++----- > 1 file changed, 3 insertions(+), 5 deletions(-) > >diff --git a/drivers/scsi/megaraid/megaraid_sas_base.c >b/drivers/scsi/megaraid/megaraid_sas_base.c >index 71c4746..a0f057c 100644 >--- a/drivers/scsi/megaraid/megaraid_sas_base.c >+++ b/drivers/scsi/megaraid/megaraid_sas_base.c >@@ -6096,14 +6096,12 @@ static int megasas_probe_one(struct pci_dev >*pdev, > instance->pd_info = pci_alloc_consistent(pdev, > sizeof(struct MR_PD_INFO), &instance->pd_info_h); > >- instance->pd_info = pci_alloc_consistent(pdev, >- sizeof(struct MR_PD_INFO), &instance->pd_info_h); >- instance->tgt_prop = pci_alloc_consistent(pdev, >- sizeof(struct MR_TARGET_PROPERTIES), &instance- >>tgt_prop_h); >- > if (!instance->pd_info) > dev_err(&instance->pdev->dev, "Failed to alloc mem >for pd_info\n"); > >+ instance->tgt_prop = pci_alloc_consistent(pdev, >+ sizeof(struct MR_TARGET_PROPERTIES), &instance- >>tgt_prop_h); >+ > if (!instance->tgt_prop) > dev_err(&instance->pdev->dev, "Failed to alloc mem >for tgt_prop\n"); Acked-by: Sumit Saxena <sumit.saxena@xxxxxxxxxxxx> > >-- >2.9.4
