From: Emmanuil Chatzipetru Date: Sun, 8 Jan 2017 13:37:05 +0000 (+0100) Subject: staging: olpc_dcon: olpc_dcon_xo_1_5: Remove redundant return statement. X-Git-Tag: v5.15~11841^2~389 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=13c7faa54091e8036c814f84aaf69045138ec9b6;p=platform%2Fkernel%2Flinux-starfive.git staging: olpc_dcon: olpc_dcon_xo_1_5: Remove redundant return statement. dcon_was_irq(); should return a boolean value if PMIO_Rx50[6] is either set or unset, which is evaluated in the first return statement. Therefore, the following return statement is redundant and thus, removed. Signed-off-by: Emmanuil Chatzipetru Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/olpc_dcon/olpc_dcon_xo_1_5.c b/drivers/staging/olpc_dcon/olpc_dcon_xo_1_5.c index 1e23ef1..e7a3957 100644 --- a/drivers/staging/olpc_dcon/olpc_dcon_xo_1_5.c +++ b/drivers/staging/olpc_dcon/olpc_dcon_xo_1_5.c @@ -53,9 +53,8 @@ static int dcon_was_irq(void) /* irq status will appear in PMIO_Rx50[6] on gpio12 */ tmp = inb(VX855_GPI_STATUS_CHG); - return !!(tmp & BIT_GPIO12); - return 0; + return !!(tmp & BIT_GPIO12); } static int dcon_init_xo_1_5(struct dcon_priv *dcon)