since we are using device-managed function, it is unnecessary
to free in probe.

Signed-off-by: Barry Song <song.bao....@hisilicon.com>
---
 drivers/net/ethernet/hisilicon/hns3/hns3_enet.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3_enet.c 
b/drivers/net/ethernet/hisilicon/hns3/hns3_enet.c
index b14f2abc2425..1817d7f2e5f6 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hns3_enet.c
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3_enet.c
@@ -2097,10 +2097,8 @@ static int hns3_probe(struct pci_dev *pdev, const struct 
pci_device_id *ent)
        pci_set_drvdata(pdev, ae_dev);
 
        ret = hnae3_register_ae_dev(ae_dev);
-       if (ret) {
-               devm_kfree(&pdev->dev, ae_dev);
+       if (ret)
                pci_set_drvdata(pdev, NULL);
-       }
 
        return ret;
 }
@@ -2157,7 +2155,6 @@ static void hns3_shutdown(struct pci_dev *pdev)
        struct hnae3_ae_dev *ae_dev = pci_get_drvdata(pdev);
 
        hnae3_unregister_ae_dev(ae_dev);
-       devm_kfree(&pdev->dev, ae_dev);
        pci_set_drvdata(pdev, NULL);
 
        if (system_state == SYSTEM_POWER_OFF)
-- 
2.23.0


Reply via email to