diff options
| author | Colin Ian King <colin.king@canonical.com> | 2016-09-06 14:44:09 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2016-10-24 21:04:27 +0200 |
| commit | 69117994aa7d9dba8ea3bcdde7676b123d975877 (patch) | |
| tree | aef0a0b977f2051bcec3e351072a67c06b3cf567 | |
| parent | [media] coda: add missing header dependencies (diff) | |
| download | linux-69117994aa7d9dba8ea3bcdde7676b123d975877.tar.xz linux-69117994aa7d9dba8ea3bcdde7676b123d975877.zip | |
[media] variable name is never null, so remove null check
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>
| -rw-r--r-- | drivers/media/usb/pvrusb2/pvrusb2-sysfs.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/media/usb/pvrusb2/pvrusb2-sysfs.c b/drivers/media/usb/pvrusb2/pvrusb2-sysfs.c index 06fe63ced58c..d977976b8d91 100644 --- a/drivers/media/usb/pvrusb2/pvrusb2-sysfs.c +++ b/drivers/media/usb/pvrusb2/pvrusb2-sysfs.c @@ -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); } |