diff options
| author | Yuan Can <yuancan@huawei.com> | 2022-09-24 04:50:41 +0200 |
|---|---|---|
| committer | Florian Fainelli <f.fainelli@gmail.com> | 2022-10-17 19:30:46 +0200 |
| commit | 117bd98daca0fb1f4264e711f73ab76739e83466 (patch) | |
| tree | 14ed1955d3ee088f7f1985eebd88cd37675ee293 /drivers/firmware | |
| parent | Linux 6.1-rc1 (diff) | |
| download | linux-117bd98daca0fb1f4264e711f73ab76739e83466.tar.xz linux-117bd98daca0fb1f4264e711f73ab76739e83466.zip | |
firmware: raspberrypi: Use dev_err_probe() to simplify code
In the probe path, dev_err() can be replaced with dev_err_probe()
which will check if error code is -EPROBE_DEFER and prints the
error name. It also sets the defer probe reason which can be
checked later through debugfs.
Signed-off-by: Yuan Can <yuancan@huawei.com>
Link: https://lore.kernel.org/r/20220924025041.29389-1-yuancan@huawei.com
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Diffstat (limited to 'drivers/firmware')
| -rw-r--r-- | drivers/firmware/raspberrypi.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/drivers/firmware/raspberrypi.c b/drivers/firmware/raspberrypi.c index 4b8978b254f9..a9d7aafc6309 100644 --- a/drivers/firmware/raspberrypi.c +++ b/drivers/firmware/raspberrypi.c @@ -268,12 +268,9 @@ static int rpi_firmware_probe(struct platform_device *pdev) fw->cl.tx_block = true; fw->chan = mbox_request_channel(&fw->cl, 0); - if (IS_ERR(fw->chan)) { - int ret = PTR_ERR(fw->chan); - if (ret != -EPROBE_DEFER) - dev_err(dev, "Failed to get mbox channel: %d\n", ret); - return ret; - } + if (IS_ERR(fw->chan)) + return dev_err_probe(dev, PTR_ERR(fw->chan), + "Failed to get mbox channel\n"); init_completion(&fw->c); kref_init(&fw->consumers); |