diff mbox series

power: supply: sc27xx: Delete superfluous error message

Message ID 20210805125233.16268-1-tangbin@cmss.chinamobile.com (mailing list archive)
State Not Applicable, archived
Headers show
Series power: supply: sc27xx: Delete superfluous error message | expand

Commit Message

Tang Bin Aug. 5, 2021, 12:52 p.m. UTC
In the function sc27xx_fgu_probe(), when get irq failed,
platform_get_irq() logs an error message, so remove
redundant message here.

Signed-off-by: Tang Bin <tangbin@cmss.chinamobile.com>
---
 drivers/power/supply/sc27xx_fuel_gauge.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

Comments

Sebastian Reichel Aug. 5, 2021, 3:59 p.m. UTC | #1
Hi,

On Thu, Aug 05, 2021 at 08:52:33PM +0800, Tang Bin wrote:
> In the function sc27xx_fgu_probe(), when get irq failed,
> platform_get_irq() logs an error message, so remove
> redundant message here.
> 
> Signed-off-by: Tang Bin <tangbin@cmss.chinamobile.com>
> ---

Thanks, queued.

--Sebastian

>  drivers/power/supply/sc27xx_fuel_gauge.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/drivers/power/supply/sc27xx_fuel_gauge.c b/drivers/power/supply/sc27xx_fuel_gauge.c
> index 9c627618c..5e7f7a19e 100644
> --- a/drivers/power/supply/sc27xx_fuel_gauge.c
> +++ b/drivers/power/supply/sc27xx_fuel_gauge.c
> @@ -1229,10 +1229,8 @@ static int sc27xx_fgu_probe(struct platform_device *pdev)
>  	}
>  
>  	irq = platform_get_irq(pdev, 0);
> -	if (irq < 0) {
> -		dev_err(dev, "no irq resource specified\n");
> +	if (irq < 0)
>  		return irq;
> -	}
>  
>  	ret = devm_request_threaded_irq(data->dev, irq, NULL,
>  					sc27xx_fgu_interrupt,
> -- 
> 2.20.1.windows.1
> 
> 
>
diff mbox series

Patch

diff --git a/drivers/power/supply/sc27xx_fuel_gauge.c b/drivers/power/supply/sc27xx_fuel_gauge.c
index 9c627618c..5e7f7a19e 100644
--- a/drivers/power/supply/sc27xx_fuel_gauge.c
+++ b/drivers/power/supply/sc27xx_fuel_gauge.c
@@ -1229,10 +1229,8 @@  static int sc27xx_fgu_probe(struct platform_device *pdev)
 	}
 
 	irq = platform_get_irq(pdev, 0);
-	if (irq < 0) {
-		dev_err(dev, "no irq resource specified\n");
+	if (irq < 0)
 		return irq;
-	}
 
 	ret = devm_request_threaded_irq(data->dev, irq, NULL,
 					sc27xx_fgu_interrupt,