brcmfmac: of: remove redundant variable len
authorColin Ian King <colin.i.king@gmail.com>
Mon, 7 Feb 2022 13:33:29 +0000 (13:33 +0000)
committerKalle Valo <kvalo@kernel.org>
Thu, 10 Feb 2022 08:46:48 +0000 (10:46 +0200)
The variable len is being assigned bit is never used. The variable
and the strlen call are redundant and can be removed.

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Signed-off-by: Kalle Valo <kvalo@kernel.org>
Link: https://lore.kernel.org/r/20220207133329.336664-1-colin.i.king@gmail.com
drivers/net/wireless/broadcom/brcm80211/brcmfmac/of.c

index 5708de1..8623bde 100644 (file)
@@ -71,14 +71,13 @@ void brcmf_of_probe(struct device *dev, enum brcmf_bus_type bus_type,
        /* Set board-type to the first string of the machine compatible prop */
        root = of_find_node_by_path("/");
        if (root) {
-               int i, len;
+               int i;
                char *board_type;
                const char *tmp;
 
                of_property_read_string_index(root, "compatible", 0, &tmp);
 
                /* get rid of '/' in the compatible string to be able to find the FW */
-               len = strlen(tmp) + 1;
                board_type = devm_kstrdup(dev, tmp, GFP_KERNEL);
                if (!board_type) {
                        of_node_put(root);