thunderbolt: Remove redundant variable 'approved'
authorColin Ian King <colin.king@canonical.com>
Wed, 25 Jul 2018 08:03:21 +0000 (11:03 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 25 Jul 2018 08:15:46 +0000 (10:15 +0200)
Variable 'approved' is being assigned but is never used hence it is
redundant and can be removed.

Cleans up clang warning:
warning: variable 'approved' set but not used [-Wunused-but-set-variable]

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/thunderbolt/icm.c

index 5ed3f8a31b0a4045cca7bf978b3fa99fd27a5630..e0930dbbcf475a913e5511ace8e9ec1fe9243be4 100644 (file)
@@ -674,7 +674,6 @@ icm_fr_xdomain_connected(struct tb *tb, const struct icm_pkg_header *hdr)
        struct tb_xdomain *xd;
        struct tb_switch *sw;
        u8 link, depth;
-       bool approved;
        u64 route;
 
        /*
@@ -688,7 +687,6 @@ icm_fr_xdomain_connected(struct tb *tb, const struct icm_pkg_header *hdr)
        link = pkg->link_info & ICM_LINK_INFO_LINK_MASK;
        depth = (pkg->link_info & ICM_LINK_INFO_DEPTH_MASK) >>
                ICM_LINK_INFO_DEPTH_SHIFT;
-       approved = pkg->link_info & ICM_LINK_INFO_APPROVED;
 
        if (link > ICM_MAX_LINK || depth > ICM_MAX_DEPTH) {
                tb_warn(tb, "invalid topology %u.%u, ignoring\n", link, depth);