diff options
author | Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> | 2016-11-12 17:04:26 +0100 |
---|---|---|
committer | Geert Uytterhoeven <geert+renesas@glider.be> | 2016-11-16 10:28:58 +0100 |
commit | 42831cf96531bf4dd9035fb8a94afe8e1fc235b3 (patch) | |
tree | 80437d1d5de8b70cfd71510334dc824703401c1e | |
parent | c314c9f15aa5f43f0e5c0e2602cc65798dbd1598 (diff) | |
download | op-kernel-dev-42831cf96531bf4dd9035fb8a94afe8e1fc235b3.zip op-kernel-dev-42831cf96531bf4dd9035fb8a94afe8e1fc235b3.tar.gz |
pinctrl: sh-pfc: r8a7795: Simplify get bias logic
The last else statement is missing braces, and the indentation level can
be reduced.
Suggested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
-rw-r--r-- | drivers/pinctrl/sh-pfc/pfc-r8a7795.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/drivers/pinctrl/sh-pfc/pfc-r8a7795.c b/drivers/pinctrl/sh-pfc/pfc-r8a7795.c index 2e8cc2a..6c58563 100644 --- a/drivers/pinctrl/sh-pfc/pfc-r8a7795.c +++ b/drivers/pinctrl/sh-pfc/pfc-r8a7795.c @@ -5367,13 +5367,12 @@ static unsigned int r8a7795_pinmux_get_bias(struct sh_pfc *pfc, reg = pullups[pin].reg; bit = BIT(pullups[pin].bit); - if (sh_pfc_read_reg(pfc, PUEN + reg, 32) & bit) { - if (sh_pfc_read_reg(pfc, PUD + reg, 32) & bit) - return PIN_CONFIG_BIAS_PULL_UP; - else - return PIN_CONFIG_BIAS_PULL_DOWN; - } else + if (!(sh_pfc_read_reg(pfc, PUEN + reg, 32) & bit)) return PIN_CONFIG_BIAS_DISABLE; + else if (sh_pfc_read_reg(pfc, PUD + reg, 32) & bit) + return PIN_CONFIG_BIAS_PULL_UP; + else + return PIN_CONFIG_BIAS_PULL_DOWN; } static void r8a7795_pinmux_set_bias(struct sh_pfc *pfc, unsigned int pin, |