diff options
| author | Liao Chang <liaochang1@huawei.com> | 2023-08-08 09:29:50 +0800 |
|---|---|---|
| committer | Andi Shyti <andi.shyti@kernel.org> | 2023-08-10 10:13:24 +0200 |
| commit | 605efbf43813857d8110ca0b5bda75f93426a789 (patch) | |
| tree | 7c09a4cf68334edc0e99c18a5663e1a9d8eca37c /drivers/i2c/busses | |
| parent | 3c5e6ae40164ba6af1efaa1ca94e2cdea0c8f25e (diff) | |
| download | linux-605efbf43813857d8110ca0b5bda75f93426a789.tar.gz | |
i2c: qcom-cci: Use dev_err_probe in probe function
Use the dev_err_probe function instead of dev_err in the probe function
so that the printed message includes the return value and also handles
-EPROBE_DEFER nicely.
Signed-off-by: Liao Chang <liaochang1@huawei.com>
Reviewed-by: Andi Shyti <andi.shyti@kernel.org>
Link: https://lore.kernel.org/r/20230808012954.1643834-6-liaochang1@huawei.com
Signed-off-by: Andi Shyti <andi.shyti@kernel.org>
Diffstat (limited to 'drivers/i2c/busses')
| -rw-r--r-- | drivers/i2c/busses/i2c-qcom-cci.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/i2c/busses/i2c-qcom-cci.c b/drivers/i2c/busses/i2c-qcom-cci.c index 622dc14add9d7a..cf13abec05f15a 100644 --- a/drivers/i2c/busses/i2c-qcom-cci.c +++ b/drivers/i2c/busses/i2c-qcom-cci.c @@ -588,10 +588,8 @@ static int cci_probe(struct platform_device *pdev) /* Clocks */ ret = devm_clk_bulk_get_all(dev, &cci->clocks); - if (ret < 1) { - dev_err(dev, "failed to get clocks %d\n", ret); - return ret; - } + if (ret < 1) + return dev_err_probe(dev, ret, "failed to get clocks\n"); cci->nclocks = ret; /* Retrieve CCI clock rate */ |