diff mbox

power: max77693: fix simple_return.cocci warnings

Message ID 20150121041134.GA205386@athens (mailing list archive)
State Not Applicable, archived
Headers show

Commit Message

Fengguang Wu Jan. 21, 2015, 4:11 a.m. UTC
drivers/power/max77693_charger.c:615:1-4: WARNING: end returns can be simpified

 Simplify a trivial if-return sequence.  Possibly combine with a
 preceding function call.
Generated by: scripts/coccinelle/misc/simple_return.cocci

CC: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---

 max77693_charger.c |    6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

--
To unsubscribe from this list: send the line "unsubscribe linux-pm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Krzysztof Kozlowski Jan. 21, 2015, 7:59 a.m. UTC | #1
On ?ro, 2015-01-21 at 12:11 +0800, kbuild test robot wrote:
> drivers/power/max77693_charger.c:615:1-4: WARNING: end returns can be simpified
> 
>  Simplify a trivial if-return sequence.  Possibly combine with a
>  preceding function call.
> Generated by: scripts/coccinelle/misc/simple_return.cocci
> 
> CC: Krzysztof Kozlowski <k.kozlowski@samsung.com>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> ---
> 
>  max77693_charger.c |    6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)

Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>

> 
> --- a/drivers/power/max77693_charger.c
> +++ b/drivers/power/max77693_charger.c
> @@ -612,12 +612,8 @@ static int max77693_reg_init(struct max7
>  	if (ret)
>  		return ret;
>  
> -	ret = max77693_set_charge_input_threshold_volt(chg,
> +	return max77693_set_charge_input_threshold_volt(chg,
>  			chg->charge_input_threshold_volt);
> -	if (ret)
> -		return ret;
> -
> -	return 0;
>  }
>  
>  #ifdef CONFIG_OF

--
To unsubscribe from this list: send the line "unsubscribe linux-pm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Sebastian Reichel Jan. 21, 2015, 1:05 p.m. UTC | #2
Hi,

On Wed, Jan 21, 2015 at 12:11:34PM +0800, kbuild test robot wrote:
> drivers/power/max77693_charger.c:615:1-4: WARNING: end returns can be simpified
> 
>  Simplify a trivial if-return sequence.  Possibly combine with a
>  preceding function call.
> Generated by: scripts/coccinelle/misc/simple_return.cocci

Thanks, applied with Reviewed-By from Krzysztof.

-- Sebastian
diff mbox

Patch

--- a/drivers/power/max77693_charger.c
+++ b/drivers/power/max77693_charger.c
@@ -612,12 +612,8 @@  static int max77693_reg_init(struct max7
 	if (ret)
 		return ret;
 
-	ret = max77693_set_charge_input_threshold_volt(chg,
+	return max77693_set_charge_input_threshold_volt(chg,
 			chg->charge_input_threshold_volt);
-	if (ret)
-		return ret;
-
-	return 0;
 }
 
 #ifdef CONFIG_OF