diff options
| author | Zhen Lei <thunder.leizhen@huawei.com> | 2021-05-11 17:18:43 +0800 |
|---|---|---|
| committer | Guenter Roeck <linux@roeck-us.net> | 2021-06-17 04:21:45 -0700 |
| commit | 4943c6039d4ac1ae8535786da7c2a28c376c589c (patch) | |
| tree | 34cb667b39556bf676d8e19a8fd122b4a462077a /drivers/hwmon/bt1-pvt.c | |
| parent | ea541c185c358f870ccb0d5fce6f726c5146daae (diff) | |
hwmon: (bt1-pvt) Remove redundant error printing in pvt_request_regs()
When devm_ioremap_resource() fails, a clear enough error message will be
printed by its subfunction __devm_ioremap_resource(). The error
information contains the device name, failure cause, and possibly resource
information.
Therefore, remove the error printing here to simplify code and reduce the
binary size.
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
Link: https://lore.kernel.org/r/20210511091843.4561-1-thunder.leizhen@huawei.com
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers/hwmon/bt1-pvt.c')
| -rw-r--r-- | drivers/hwmon/bt1-pvt.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/hwmon/bt1-pvt.c b/drivers/hwmon/bt1-pvt.c index 3e1d56585b91..74ce5211eb75 100644 --- a/drivers/hwmon/bt1-pvt.c +++ b/drivers/hwmon/bt1-pvt.c @@ -924,10 +924,8 @@ static int pvt_request_regs(struct pvt_hwmon *pvt) } pvt->regs = devm_ioremap_resource(pvt->dev, res); - if (IS_ERR(pvt->regs)) { - dev_err(pvt->dev, "Couldn't map PVT registers\n"); + if (IS_ERR(pvt->regs)) return PTR_ERR(pvt->regs); - } return 0; } |