summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>2017-03-22 20:20:58 +0100
committerSamuel Ortiz <sameo@linux.intel.com>2017-04-05 10:15:17 +0200
commit3267183c3d1a6612c00961d7334413f9976c751a (patch)
treea026d6eb5c803cd5a08ce22736b47c0867d6ec5d
parentNFC: st21nfca: Use unified device property API meaningfully (diff)
downloadlinux-3267183c3d1a6612c00961d7334413f9976c751a.tar.xz
linux-3267183c3d1a6612c00961d7334413f9976c751a.zip
NFC: netlink: Use error code from nfc_activate_target()
It looks like a typo to assign a return code to a variable which is not used. Found due to a compiler warning: net/nfc/netlink.c: In function ‘nfc_genl_activate_target’: net/nfc/netlink.c:903:6: warning: variable ‘rc’ set but not used [-Wunused-but-set-variable] int rc; ^~ Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
-rw-r--r--net/nfc/netlink.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/nfc/netlink.c b/net/nfc/netlink.c
index 4c95319dc22b..23a6e01f35ed 100644
--- a/net/nfc/netlink.c
+++ b/net/nfc/netlink.c
@@ -922,7 +922,7 @@ static int nfc_genl_activate_target(struct sk_buff *skb, struct genl_info *info)
rc = nfc_activate_target(dev, target_idx, protocol);
nfc_put_device(dev);
- return 0;
+ return rc;
}
static int nfc_genl_dep_link_up(struct sk_buff *skb, struct genl_info *info)