Commit 9e3440d2 authored by Raag Jadav's avatar Raag Jadav Committed by Uwe Kleine-König

pwm: dwc: simplify error handling

Simplify error handling in ->probe() function using dev_err_probe() helper
and while at it, drop error codes from the message to prevent duplication.
Signed-off-by: default avatarRaag Jadav <raag.jadav@intel.com>
Tested-by: default avatarJarkko Nikula <jarkko.nikula@linux.intel.com>
Link: https://lore.kernel.org/r/20240219033835.11369-5-raag.jadav@intel.comSigned-off-by: default avatarUwe Kleine-König <u.kleine-koenig@pengutronix.de>
parent ebf2c89e
...@@ -53,18 +53,14 @@ static int dwc_pwm_probe(struct pci_dev *pci, const struct pci_device_id *id) ...@@ -53,18 +53,14 @@ static int dwc_pwm_probe(struct pci_dev *pci, const struct pci_device_id *id)
int i, ret; int i, ret;
ret = pcim_enable_device(pci); ret = pcim_enable_device(pci);
if (ret) { if (ret)
dev_err(dev, "Failed to enable device (%pe)\n", ERR_PTR(ret)); return dev_err_probe(dev, ret, "Failed to enable device\n");
return ret;
}
pci_set_master(pci); pci_set_master(pci);
ret = pcim_iomap_regions(pci, BIT(0), pci_name(pci)); ret = pcim_iomap_regions(pci, BIT(0), pci_name(pci));
if (ret) { if (ret)
dev_err(dev, "Failed to iomap PCI BAR (%pe)\n", ERR_PTR(ret)); return dev_err_probe(dev, ret, "Failed to iomap PCI BAR\n");
return ret;
}
info = (const struct dwc_pwm_info *)id->driver_data; info = (const struct dwc_pwm_info *)id->driver_data;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment