Commit 13c7faa5 authored by Emmanuil Chatzipetru's avatar Emmanuil Chatzipetru Committed by Greg Kroah-Hartman

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: default avatarEmmanuil Chatzipetru <chatzi.emanuel@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 5f6791ec
...@@ -53,9 +53,8 @@ static int dcon_was_irq(void) ...@@ -53,9 +53,8 @@ static int dcon_was_irq(void)
/* irq status will appear in PMIO_Rx50[6] on gpio12 */ /* irq status will appear in PMIO_Rx50[6] on gpio12 */
tmp = inb(VX855_GPI_STATUS_CHG); 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) static int dcon_init_xo_1_5(struct dcon_priv *dcon)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment