mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-02-23 20:01:47 -05:00
platform/x86/amd/pmf: Use dev_err_probe() to simplify error handling
To simplify error handling in the amd_pmf probe function and reduce code size, dev_err() is replaced with dev_err_probe(). Reviewed-by: Mario Limonciello <mario.limonciello@amd.com> Co-developed-by: Patil Rajesh Reddy <Patil.Reddy@amd.com> Signed-off-by: Patil Rajesh Reddy <Patil.Reddy@amd.com> Signed-off-by: Shyam Sundar S K <Shyam-sundar.S-k@amd.com> Link: https://lore.kernel.org/r/20241104054829.620858-2-Shyam-sundar.S-k@amd.com Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com> Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
This commit is contained in:
committed by
Ilpo Järvinen
parent
ab49d7bf99
commit
cc8e2dbf99
@@ -429,18 +429,18 @@ static int amd_pmf_probe(struct platform_device *pdev)
|
||||
|
||||
err = amd_smn_read(0, AMD_PMF_BASE_ADDR_LO, &val);
|
||||
if (err) {
|
||||
dev_err(dev->dev, "error in reading from 0x%x\n", AMD_PMF_BASE_ADDR_LO);
|
||||
pci_dev_put(rdev);
|
||||
return pcibios_err_to_errno(err);
|
||||
return dev_err_probe(dev->dev, pcibios_err_to_errno(err),
|
||||
"error in reading from 0x%x\n", AMD_PMF_BASE_ADDR_LO);
|
||||
}
|
||||
|
||||
base_addr_lo = val & AMD_PMF_BASE_ADDR_HI_MASK;
|
||||
|
||||
err = amd_smn_read(0, AMD_PMF_BASE_ADDR_HI, &val);
|
||||
if (err) {
|
||||
dev_err(dev->dev, "error in reading from 0x%x\n", AMD_PMF_BASE_ADDR_HI);
|
||||
pci_dev_put(rdev);
|
||||
return pcibios_err_to_errno(err);
|
||||
return dev_err_probe(dev->dev, pcibios_err_to_errno(err),
|
||||
"error in reading from 0x%x\n", AMD_PMF_BASE_ADDR_HI);
|
||||
}
|
||||
|
||||
base_addr_hi = val & AMD_PMF_BASE_ADDR_LO_MASK;
|
||||
|
||||
Reference in New Issue
Block a user