diff options
author | Guenter Roeck <linux@roeck-us.net> | 2021-05-10 18:25:19 -0700 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2021-05-13 20:58:57 +0200 |
commit | 7912146abf6705c771b28ea5b6d608ee2ea39dcd (patch) | |
tree | 41fd405f0a2b5ce097f724a042b534984ef43c71 /drivers/tty/hvc | |
parent | 7985723d21a15d4a237efa89e85c938875a3f9c3 (diff) | |
download | linux-7912146abf6705c771b28ea5b6d608ee2ea39dcd.tar.gz linux-7912146abf6705c771b28ea5b6d608ee2ea39dcd.tar.bz2 linux-7912146abf6705c771b28ea5b6d608ee2ea39dcd.zip |
tty: hvc_iucv: Drop unnecessary NULL check after container_of
The result of container_of() operations is never NULL unless the extracted
element is the first element of the embedded structure. This is not the
case here. The NULL check is therefore unnecessary and misleading.
Remove it.
This change was made automatically with the following Coccinelle script.
@@
type t;
identifier v;
statement s;
@@
<+...
(
t v = container_of(...);
|
v = container_of(...);
)
...
when != v
- if (\( !v \| v == NULL \) ) s
...+>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Link: https://lore.kernel.org/r/20210511012519.2359074-1-linux@roeck-us.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/tty/hvc')
-rw-r--r-- | drivers/tty/hvc/hvc_iucv.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/tty/hvc/hvc_iucv.c b/drivers/tty/hvc/hvc_iucv.c index 2af1e5751bd6..3bd03ae01bf5 100644 --- a/drivers/tty/hvc/hvc_iucv.c +++ b/drivers/tty/hvc/hvc_iucv.c @@ -438,8 +438,6 @@ static void hvc_iucv_sndbuf_work(struct work_struct *work) struct hvc_iucv_private *priv; priv = container_of(work, struct hvc_iucv_private, sndbuf_work.work); - if (!priv) - return; spin_lock_bh(&priv->lock); hvc_iucv_send(priv); |