From 13b79a0d88e350ec874e45fdbd08cb6f37fa4a48 Mon Sep 17 00:00:00 2001 From: Amitoj Kaur Chawla Date: Sun, 28 Feb 2016 21:22:00 +0530 Subject: [PATCH] staging: sm750fb: Use pcim_enable_device() Devm_ functions allocate memory that is released when a driver detaches. Replace pci_enable_device with the managed pcim_enable_device and remove corresponding pci_disable_device from probe and suspend functions of a pci_dev. Also, an unnecessary label has been removed by replacing it with a direct return statement. Signed-off-by: Amitoj Kaur Chawla Signed-off-by: Greg Kroah-Hartman --- drivers/staging/sm750fb/sm750.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/staging/sm750fb/sm750.c b/drivers/staging/sm750fb/sm750.c index 8ec81bed8b54..aac74a7495f6 100644 --- a/drivers/staging/sm750fb/sm750.c +++ b/drivers/staging/sm750fb/sm750.c @@ -422,7 +422,6 @@ static int lynxfb_suspend(struct pci_dev *pdev, pm_message_t mesg) return ret; } - pci_disable_device(pdev); ret = pci_set_power_state(pdev, pci_choose_state(pdev, mesg)); if (ret) { pr_err("error:%d occurred in pci_set_power_state\n", ret); @@ -1053,14 +1052,14 @@ static int lynxfb_pci_probe(struct pci_dev *pdev, int err; /* enable device */ - err = pci_enable_device(pdev); + err = pcim_enable_device(pdev); if (err) return err; err = -ENOMEM; sm750_dev = devm_kzalloc(&pdev->dev, sizeof(*sm750_dev), GFP_KERNEL); if (!sm750_dev) - goto disable_pci; + return err; sm750_dev->fbinfo[0] = sm750_dev->fbinfo[1] = NULL; sm750_dev->devid = pdev->device; @@ -1115,8 +1114,6 @@ static int lynxfb_pci_probe(struct pci_dev *pdev, release_fb: sm750fb_frambuffer_release(sm750_dev); -disable_pci: - pci_disable_device(pdev); return err; } -- 2.39.5