From 9f20ecc51bc08c1ee9c1ed320ef6ca411eb154c6 Mon Sep 17 00:00:00 2001 From: Daeseok Youn Date: Mon, 18 Aug 2014 23:31:46 +0900 Subject: [PATCH] staging: dgap: remove redundant setting a variable The brd(board_t) is initialized with zero, so "intr_used" is not needed to set zero when request_irq() is failed. Signed-off-by: Daeseok Youn Signed-off-by: Greg Kroah-Hartman --- drivers/staging/dgap/dgap.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/staging/dgap/dgap.c b/drivers/staging/dgap/dgap.c index 45f20b4a0c14..ff953767014e 100644 --- a/drivers/staging/dgap/dgap.c +++ b/drivers/staging/dgap/dgap.c @@ -846,14 +846,9 @@ static int dgap_request_irq(struct board_t *brd) rc = request_irq(brd->irq, dgap_intr, IRQF_SHARED, "DGAP", brd); - if (rc) - brd->intr_used = 0; - else + if (!rc) brd->intr_used = 1; - } else { - brd->intr_used = 0; } - return 0; } -- 2.20.1