pinctrl: bcm: remove redundant assignment to pointer log
authorColin Ian King <colin.king@canonical.com>
Thu, 5 Sep 2019 14:09:19 +0000 (15:09 +0100)
committerLinus Walleij <linus.walleij@linaro.org>
Thu, 12 Sep 2019 09:25:53 +0000 (10:25 +0100)
The pointer log is being initialized with a value that is never read
and is being re-assigned a little later on. The assignment is
redundant and hence can be removed.

Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Link: https://lore.kernel.org/r/20190905140919.29283-1-colin.king@canonical.com
Reviewed-by: Ray Jui <ray.jui@broadcom.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/bcm/pinctrl-cygnus-mux.c

index 44df359..dcab220 100644 (file)
@@ -923,7 +923,6 @@ static int cygnus_mux_log_init(struct cygnus_pinctrl *pinctrl)
        if (!pinctrl->mux_log)
                return -ENOMEM;
 
-       log = pinctrl->mux_log;
        for (i = 0; i < CYGNUS_NUM_IOMUX_REGS; i++) {
                for (j = 0; j < CYGNUS_NUM_MUX_PER_REG; j++) {
                        log = &pinctrl->mux_log[i * CYGNUS_NUM_MUX_PER_REG