mirror of
https://github.com/AuxXxilium/linux_dsm_epyc7002.git
synced 2024-11-24 04:00:52 +07:00
thermal: Delete an error message in four functions
The function “platform_get_irq” can log an error already. Thus omit redundant messages for the exception handling in the calling functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> Reviewed-by: Amit Kucheria <amit.kucheria@linaro.org> Reviewed-by: Keerthy <j-keerthy@ti.com> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org> Link: https://lore.kernel.org/r/05f49ae7-5cc7-d6a0-fc3d-abaf2a0b373c@web.de
This commit is contained in:
parent
0145f67866
commit
8cb775bb00
@ -1241,10 +1241,8 @@ static int rockchip_thermal_probe(struct platform_device *pdev)
|
||||
return -ENXIO;
|
||||
|
||||
irq = platform_get_irq(pdev, 0);
|
||||
if (irq < 0) {
|
||||
dev_err(&pdev->dev, "no irq resource?\n");
|
||||
if (irq < 0)
|
||||
return -EINVAL;
|
||||
}
|
||||
|
||||
thermal = devm_kzalloc(&pdev->dev, sizeof(struct rockchip_thermal_data),
|
||||
GFP_KERNEL);
|
||||
|
@ -94,10 +94,8 @@ static int st_mmap_register_enable_irq(struct st_thermal_sensor *sensor)
|
||||
int ret;
|
||||
|
||||
sensor->irq = platform_get_irq(pdev, 0);
|
||||
if (sensor->irq < 0) {
|
||||
dev_err(dev, "failed to register IRQ\n");
|
||||
if (sensor->irq < 0)
|
||||
return sensor->irq;
|
||||
}
|
||||
|
||||
ret = devm_request_threaded_irq(dev, sensor->irq,
|
||||
NULL, st_mmap_thermal_trip_handler,
|
||||
|
@ -385,10 +385,8 @@ static int stm_register_irq(struct stm_thermal_sensor *sensor)
|
||||
int ret;
|
||||
|
||||
sensor->irq = platform_get_irq(pdev, 0);
|
||||
if (sensor->irq < 0) {
|
||||
dev_err(dev, "%s: Unable to find IRQ\n", __func__);
|
||||
if (sensor->irq < 0)
|
||||
return sensor->irq;
|
||||
}
|
||||
|
||||
ret = devm_request_threaded_irq(dev, sensor->irq,
|
||||
NULL,
|
||||
|
@ -772,10 +772,9 @@ static int ti_bandgap_talert_init(struct ti_bandgap *bgp,
|
||||
int ret;
|
||||
|
||||
bgp->irq = platform_get_irq(pdev, 0);
|
||||
if (bgp->irq < 0) {
|
||||
dev_err(&pdev->dev, "get_irq failed\n");
|
||||
if (bgp->irq < 0)
|
||||
return bgp->irq;
|
||||
}
|
||||
|
||||
ret = request_threaded_irq(bgp->irq, NULL,
|
||||
ti_bandgap_talert_irq_handler,
|
||||
IRQF_TRIGGER_HIGH | IRQF_ONESHOT,
|
||||
|
Loading…
Reference in New Issue
Block a user