staging: dgnc: remove redundant NULL check for brd
authorDaeseok Youn <daeseok.youn@gmail.com>
Wed, 6 Jul 2016 06:15:34 +0000 (15:15 +0900)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 15 Aug 2016 17:13:25 +0000 (19:13 +0200)
the "brd" value cannot be NULL in dgnc_finalize_board_init().
Because "brd" as a parameter of this function was already
checked for NULL.

the dgnc_finalize_board_init() as a static function was called
only from dgnc_found_board() function and brd->magic value
was assigned once in dgnc_found_board(). So it doesn't need to
check for DGNC_BOARD_MAGIC value.

Signed-off-by: Daeseok Youn <daeseok.youn@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/dgnc/dgnc_driver.c

index 2c3fb2ab5a04aade904923edd2777bc90f6f825c..e6993752345b934adf79f5fb854e14d0904e4fc5 100644 (file)
@@ -579,9 +579,6 @@ static int dgnc_finalize_board_init(struct dgnc_board *brd)
 {
        int rc = 0;
 
-       if (!brd || brd->magic != DGNC_BOARD_MAGIC)
-               return -ENODEV;
-
        if (brd->irq) {
                rc = request_irq(brd->irq, brd->bd_ops->intr,
                                 IRQF_SHARED, "DGNC", brd);