Re: [PATCH 1/3] tty: hvcs: Drop unnecessary if block
From: Tyrel Datwyler <tyreld@linux.ibm.com>
Date: 2021-01-20 17:18:27
Also in:
lkml, sparclinux
On 1/14/21 9:57 AM, Uwe Kleine-König wrote:
If hvcs_probe() succeeded dev_set_drvdata() is called with a non-NULL value, and if hvcs_probe() failed hvcs_remove() isn't called. So there is no way dev_get_drvdata() can return NULL in hvcs_remove() and the check can just go away. Signed-off-by: Uwe Kleine-König <redacted>
Reviewed-by: Tyrel Datwyler <tyreld@linux.ibm.com>
quoted hunk ↗ jump to hunk
--- drivers/tty/hvc/hvcs.c | 3 --- 1 file changed, 3 deletions(-)diff --git a/drivers/tty/hvc/hvcs.c b/drivers/tty/hvc/hvcs.c index 509d1042825a..3e0461285c34 100644 --- a/drivers/tty/hvc/hvcs.c +++ b/drivers/tty/hvc/hvcs.c@@ -825,9 +825,6 @@ static int hvcs_remove(struct vio_dev *dev) unsigned long flags; struct tty_struct *tty; - if (!hvcsd) - return -ENODEV; - /* By this time the vty-server won't be getting any more interrupts */ spin_lock_irqsave(&hvcsd->lock, flags);