diff options
| author | Zhang Zekun <zhangzekun11@huawei.com> | 2024-09-09 20:29:21 +0800 |
|---|---|---|
| committer | Bjorn Andersson <andersson@kernel.org> | 2024-10-29 15:04:31 -0500 |
| commit | cd8ec43758a5ca91c5503d1cde2df95e05b968ca (patch) | |
| tree | 4ef7904659abfb8a12f18c71e88b95e59b02010b | |
| parent | 876b405004842b10a684f06f967502d20d64384f (diff) | |
| download | lwn-cd8ec43758a5ca91c5503d1cde2df95e05b968ca.tar.gz lwn-cd8ec43758a5ca91c5503d1cde2df95e05b968ca.zip | |
soc: qcom: smp2p: Simplify code with dev_err_probe()
Use dev_err_probe() directly in the driver probe phase, and we
don't need to judge if the error code is not equal to -EPROBE_DEFER.
This can simplify the code a bit.
Signed-off-by: Zhang Zekun <zhangzekun11@huawei.com>
Link: https://lore.kernel.org/r/20240909122921.12627-5-zhangzekun11@huawei.com
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
| -rw-r--r-- | drivers/soc/qcom/smp2p.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/drivers/soc/qcom/smp2p.c b/drivers/soc/qcom/smp2p.c index cefcbd61c628..2731449e2201 100644 --- a/drivers/soc/qcom/smp2p.c +++ b/drivers/soc/qcom/smp2p.c @@ -467,12 +467,9 @@ static int qcom_smp2p_alloc_outbound_item(struct qcom_smp2p *smp2p) int ret; ret = qcom_smem_alloc(pid, smem_id, sizeof(*out)); - if (ret < 0 && ret != -EEXIST) { - if (ret != -EPROBE_DEFER) - dev_err(smp2p->dev, - "unable to allocate local smp2p item\n"); - return ret; - } + if (ret < 0 && ret != -EEXIST) + return dev_err_probe(smp2p->dev, ret, + "unable to allocate local smp2p item\n"); out = qcom_smem_get(pid, smem_id, NULL); if (IS_ERR(out)) { |