staging: dgap: remove bogus null test in dgap_tty_init()
authorDaeseok Youn <daeseok.youn@gmail.com>
Mon, 2 Jun 2014 05:06:53 +0000 (14:06 +0900)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 19 Jun 2014 22:21:47 +0000 (15:21 -0700)
- The channels array were set to NULL in dgap_found_board().
- Removes redundant null check for channels array in for loop,
if one of the channel cannot be allocated, dgap_tty_init() just returns
an error.

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

index 5556a6ec8a124e2643dfcdd73c821d886e788d38..fcfa061cb5e2ec8f08dd24bb37745b4a4fdfb1e8 100644 (file)
@@ -1379,12 +1379,10 @@ static int dgap_tty_init(struct board_t *brd)
         * when the driver was first loaded.
         */
        for (i = 0; i < brd->nasync; i++) {
-               if (!brd->channels[i]) {
-                       brd->channels[i] =
-                               kzalloc(sizeof(struct channel_t), GFP_KERNEL);
-                       if (!brd->channels[i])
-                               return -ENOMEM;
-               }
+               brd->channels[i] =
+                       kzalloc(sizeof(struct channel_t), GFP_KERNEL);
+               if (!brd->channels[i])
+                       return -ENOMEM;
        }
 
        ch = brd->channels[0];
@@ -1398,9 +1396,6 @@ static int dgap_tty_init(struct board_t *brd)
        /* Set up channel variables */
        for (i = 0; i < brd->nasync; i++, ch = brd->channels[i], bs++) {
 
-               if (!brd->channels[i])
-                       continue;
-
                spin_lock_init(&ch->ch_lock);
 
                /* Store all our magic numbers */