diff options
| author | Colin Ian King <colin.king@canonical.com> | 2016-09-06 15:44:09 +0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2016-10-24 22:04:27 +0300 |
| commit | 69117994aa7d9dba8ea3bcdde7676b123d975877 (patch) | |
| tree | aef0a0b977f2051bcec3e351072a67c06b3cf567 /drivers/media/usb/pvrusb2/pvrusb2-sysfs.c | |
| parent | 06c24f6779ea863a9410c872c577d8ae986a4b60 (diff) | |
| download | linux-69117994aa7d9dba8ea3bcdde7676b123d975877.tar.xz | |
[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>
Diffstat (limited to 'drivers/media/usb/pvrusb2/pvrusb2-sysfs.c')
| -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); } |