diff options
author | Anda-Maria Nicolae <anda-maria.nicolae@intel.com> | 2015-03-05 06:23:56 -0500 |
---|---|---|
committer | Sebastian Reichel <sre@kernel.org> | 2015-03-07 13:36:29 -0500 |
commit | 42d0631bb6c9bd730437cca19e5b870ecd24215d (patch) | |
tree | 11638727cda66151de65eb2e470a864f4362077c /drivers/power/bq2415x_charger.c | |
parent | 1d93b850296339082b7ace26ed20385d3bc5e2c6 (diff) |
bq2415x_charger: Remove unnecessary else after return
Fix coding style to comply with checkpatch.pl
Signed-off-by: Anda-Maria Nicolae <anda-maria.nicolae@intel.com>
Signed-off-by: Sebastian Reichel <sre@kernel.org>
Diffstat (limited to 'drivers/power/bq2415x_charger.c')
-rw-r--r-- | drivers/power/bq2415x_charger.c | 24 |
1 files changed, 8 insertions, 16 deletions
diff --git a/drivers/power/bq2415x_charger.c b/drivers/power/bq2415x_charger.c index 727740066754..4976c1b4bc82 100644 --- a/drivers/power/bq2415x_charger.c +++ b/drivers/power/bq2415x_charger.c | |||
@@ -346,8 +346,7 @@ static int bq2415x_exec_command(struct bq2415x_device *bq, | |||
346 | BQ2415X_BIT_CE); | 346 | BQ2415X_BIT_CE); |
347 | if (ret < 0) | 347 | if (ret < 0) |
348 | return ret; | 348 | return ret; |
349 | else | 349 | return ret > 0 ? 0 : 1; |
350 | return ret > 0 ? 0 : 1; | ||
351 | case BQ2415X_CHARGER_ENABLE: | 350 | case BQ2415X_CHARGER_ENABLE: |
352 | return bq2415x_i2c_write_bit(bq, BQ2415X_REG_CONTROL, | 351 | return bq2415x_i2c_write_bit(bq, BQ2415X_REG_CONTROL, |
353 | 0, BQ2415X_BIT_CE); | 352 | 0, BQ2415X_BIT_CE); |
@@ -420,20 +419,17 @@ static enum bq2415x_chip bq2415x_detect_chip(struct bq2415x_device *bq) | |||
420 | case 0: | 419 | case 0: |
421 | if (bq->chip == BQ24151A) | 420 | if (bq->chip == BQ24151A) |
422 | return bq->chip; | 421 | return bq->chip; |
423 | else | 422 | return BQ24151; |
424 | return BQ24151; | ||
425 | case 1: | 423 | case 1: |
426 | if (bq->chip == BQ24150A || | 424 | if (bq->chip == BQ24150A || |
427 | bq->chip == BQ24152 || | 425 | bq->chip == BQ24152 || |
428 | bq->chip == BQ24155) | 426 | bq->chip == BQ24155) |
429 | return bq->chip; | 427 | return bq->chip; |
430 | else | 428 | return BQ24150; |
431 | return BQ24150; | ||
432 | case 2: | 429 | case 2: |
433 | if (bq->chip == BQ24153A) | 430 | if (bq->chip == BQ24153A) |
434 | return bq->chip; | 431 | return bq->chip; |
435 | else | 432 | return BQ24153; |
436 | return BQ24153; | ||
437 | default: | 433 | default: |
438 | return BQUNKNOWN; | 434 | return BQUNKNOWN; |
439 | } | 435 | } |
@@ -444,8 +440,7 @@ static enum bq2415x_chip bq2415x_detect_chip(struct bq2415x_device *bq) | |||
444 | case 0: | 440 | case 0: |
445 | if (bq->chip == BQ24156A) | 441 | if (bq->chip == BQ24156A) |
446 | return bq->chip; | 442 | return bq->chip; |
447 | else | 443 | return BQ24156; |
448 | return BQ24156; | ||
449 | case 2: | 444 | case 2: |
450 | return BQ24158; | 445 | return BQ24158; |
451 | default: | 446 | default: |
@@ -474,8 +469,7 @@ static int bq2415x_detect_revision(struct bq2415x_device *bq) | |||
474 | case BQ24152: | 469 | case BQ24152: |
475 | if (ret >= 0 && ret <= 3) | 470 | if (ret >= 0 && ret <= 3) |
476 | return ret; | 471 | return ret; |
477 | else | 472 | return -1; |
478 | return -1; | ||
479 | case BQ24153: | 473 | case BQ24153: |
480 | case BQ24153A: | 474 | case BQ24153A: |
481 | case BQ24156: | 475 | case BQ24156: |
@@ -485,13 +479,11 @@ static int bq2415x_detect_revision(struct bq2415x_device *bq) | |||
485 | return 0; | 479 | return 0; |
486 | else if (ret == 1) | 480 | else if (ret == 1) |
487 | return 1; | 481 | return 1; |
488 | else | 482 | return -1; |
489 | return -1; | ||
490 | case BQ24155: | 483 | case BQ24155: |
491 | if (ret == 3) | 484 | if (ret == 3) |
492 | return 3; | 485 | return 3; |
493 | else | 486 | return -1; |
494 | return -1; | ||
495 | case BQUNKNOWN: | 487 | case BQUNKNOWN: |
496 | return -1; | 488 | return -1; |
497 | } | 489 | } |