EDAC/aspeed: Remove set but not used variable ‘np’ [Linux 5.3]

This Linux kernel change "EDAC/aspeed: Remove set but not used variable ‘np’" is included in the Linux 5.3 release. This change is authored by YueHaibing <yuehaibing [at] huawei.com> on Sat May 25 22:41:53 2019 +0800. The commit for this change in Linux stable tree is ff70cac (patch).

EDAC/aspeed: Remove set but not used variable 'np'

Fix the following -Wunused-but-set-variable warning:

  drivers/edac/aspeed_edac.c: In function aspeed_probe:
  drivers/edac/aspeed_edac.c:284:22: warning: variable np set but not used [-Wunused-but-set-variable]

It is never used and can be removed.

Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Signed-off-by: Borislav Petkov <bp@suse.de>
Reviewed-by: Andrew Jeffery <andrew@aj.id.au>
Reviewed-by: Stefan Schaeckeler <sschaeck@cisco.com>
Cc: James Morse <james.morse@arm.com>
Cc: Joel Stanley <joel@jms.id.au>
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-aspeed@lists.ozlabs.org
Cc: linux-edac <linux-edac@vger.kernel.org>
Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
Link: https://lkml.kernel.org/r/20190525144153.2028-1-yuehaibing@huawei.com

There are 4 lines of Linux source code added/deleted in this change. Code changes to Linux kernel are as follows.

 drivers/edac/aspeed_edac.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/edac/aspeed_edac.c b/drivers/edac/aspeed_edac.c
index 11833c0..5634437 100644
--- a/drivers/edac/aspeed_edac.c
+++ b/drivers/edac/aspeed_edac.c
@@ -281,15 +281,11 @@ static int aspeed_probe(struct platform_device *pdev)
    struct device *dev = &pdev->dev;
    struct edac_mc_layer layers[2];
    struct mem_ctl_info *mci;
-   struct device_node *np;
    struct resource *res;
    void __iomem *regs;
    u32 reg04;
    int rc;

-   /* setup regmap */
-   np = dev->of_node;
-
    res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
    if (!res)
        return -ENOENT;

Leave a Reply

Your email address will not be published. Required fields are marked *