Commit 21b56ec4 authored by Alan Cox's avatar Alan Cox Committed by Linus Torvalds

neo: Remove a bogus NULL check

Julia Lawall found an un-needed check in the neo driver. Her patch moves
the check to cover the code dereferencing it, however it cannot be NULL
anyway so remove the NULL check instead.
Signed-off-by: default avatarAlan Cox <alan@redhat.com>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent c774bda2
...@@ -577,9 +577,6 @@ static void neo_parse_modem(struct jsm_channel *ch, u8 signals) ...@@ -577,9 +577,6 @@ static void neo_parse_modem(struct jsm_channel *ch, u8 signals)
jsm_printk(MSIGS, INFO, &ch->ch_bd->pci_dev, jsm_printk(MSIGS, INFO, &ch->ch_bd->pci_dev,
"neo_parse_modem: port: %d msignals: %x\n", ch->ch_portnum, msignals); "neo_parse_modem: port: %d msignals: %x\n", ch->ch_portnum, msignals);
if (!ch)
return;
/* Scrub off lower bits. They signify delta's, which I don't care about */ /* Scrub off lower bits. They signify delta's, which I don't care about */
/* Keep DDCD and DDSR though */ /* Keep DDCD and DDSR though */
msignals &= 0xf8; msignals &= 0xf8;
......
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