[media] variable name is never null, so remove null check
authorColin Ian King <colin.king@canonical.com>
Tue, 6 Sep 2016 12:44:09 +0000 (09:44 -0300)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Mon, 24 Oct 2016 19:04:27 +0000 (17:04 -0200)
The variable name is always assigned to a literal string in the
proceeding switch statement, so it is never null and hence the
null check is redundant. Remove null the check.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
drivers/media/usb/pvrusb2/pvrusb2-sysfs.c

index 06fe63c..d977976 100644 (file)
@@ -116,7 +116,6 @@ static ssize_t show_type(struct device *class_dev,
        }
        pvr2_sysfs_trace("pvr2_sysfs(%p) show_type(cid=%d) is %s",
                         cip->chptr, cip->ctl_id, name);
-       if (!name) return -EINVAL;
        return scnprintf(buf, PAGE_SIZE, "%s\n", name);
 }