diff options
| author |   Sergio Paracuellos <sergio.paracuellos@gmail.com> | 2018-05-20 15:00:25 +0200 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2018-05-25 18:49:32 +0200 |
| commit | b36dd3af48f65c0e498d62bb5e6c388b03f77608 (patch) | |
| tree | e525c3b58352df3928c345dc83484464502253ac /drivers/staging/mt7621-gpio | |
| parent | staging: mt7621-gpio: avoid devm_kzalloc() hidden inside declarations and refactor function a bit (diff) | |
| download | linux-dev-b36dd3af48f65c0e498d62bb5e6c388b03f77608.tar.xz linux-dev-b36dd3af48f65c0e498d62bb5e6c388b03f77608.zip | |
staging: mt7621-gpio: use ternary operator in return in mediatek_gpio_get_direction
This commits replaces if statement and two returns in favour
of a only one return using a ternary operator.
Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Reviewed-by: NeilBrown <neil@brown.name>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/mt7621-gpio')
| -rw-r--r-- | drivers/staging/mt7621-gpio/gpio-mt7621.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/staging/mt7621-gpio/gpio-mt7621.c b/drivers/staging/mt7621-gpio/gpio-mt7621.c index bbe6cce56d90..15a1003017f8 100644 --- a/drivers/staging/mt7621-gpio/gpio-mt7621.c +++ b/drivers/staging/mt7621-gpio/gpio-mt7621.c @@ -135,10 +135,7 @@ mediatek_gpio_get_direction(struct gpio_chip *chip, unsigned int offset) t = mtk_gpio_r32(rg, GPIO_REG_CTRL); spin_unlock_irqrestore(&rg->lock, flags); - if (t & BIT(offset)) - return 0; - - return 1; + return (t & BIT(offset)) ? 0 : 1; } static int |