Commit 071494d8 authored by Giedrius Statkevičius's avatar Giedrius Statkevičius Committed by Greg Kroah-Hartman

staging: dgnc: remove redundant !ch checks

Remove checks that are redundant since we don't have boards with partially
initialized ->channels[i].
Signed-off-by: default avatarGiedrius Statkevičius <giedrius.statkevicius@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent ce2927e9
...@@ -379,7 +379,7 @@ static inline void cls_parse_isr(struct dgnc_board *brd, uint port) ...@@ -379,7 +379,7 @@ static inline void cls_parse_isr(struct dgnc_board *brd, uint port)
return; return;
ch = brd->channels[port]; ch = brd->channels[port];
if (!ch || ch->magic != DGNC_CHANNEL_MAGIC) if (ch->magic != DGNC_CHANNEL_MAGIC)
return; return;
/* Here we try to figure out what caused the interrupt to happen */ /* Here we try to figure out what caused the interrupt to happen */
...@@ -714,8 +714,6 @@ static void cls_tasklet(unsigned long data) ...@@ -714,8 +714,6 @@ static void cls_tasklet(unsigned long data)
/* Loop on each port */ /* Loop on each port */
for (i = 0; i < ports; i++) { for (i = 0; i < ports; i++) {
ch = bd->channels[i]; ch = bd->channels[i];
if (!ch)
continue;
/* /*
* NOTE: Remember you CANNOT hold any channel * NOTE: Remember you CANNOT hold any channel
......
...@@ -395,7 +395,7 @@ static inline void neo_parse_isr(struct dgnc_board *brd, uint port) ...@@ -395,7 +395,7 @@ static inline void neo_parse_isr(struct dgnc_board *brd, uint port)
return; return;
ch = brd->channels[port]; ch = brd->channels[port];
if (!ch || ch->magic != DGNC_CHANNEL_MAGIC) if (ch->magic != DGNC_CHANNEL_MAGIC)
return; return;
/* Here we try to figure out what caused the interrupt to happen */ /* Here we try to figure out what caused the interrupt to happen */
......
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