ata: Drop unnecessary static
authorJulia Lawall <Julia.Lawall@lip6.fr>
Sat, 15 Jul 2017 20:07:43 +0000 (22:07 +0200)
committerTejun Heo <tj@kernel.org>
Wed, 2 Aug 2017 15:11:53 +0000 (08:11 -0700)
Drop static on a local variable, when the variable is initialized before
any possible use.  Thus, the static has no benefit.

The semantic patch that fixes this problem is as follows:
(http://coccinelle.lip6.fr/)

// <smpl>
@bad exists@
position p;
identifier x;
type T;
@@
static T x@p;
...
x = <+...x...+>

@@
identifier x;
expression e;
type T;
position p != bad.p;
@@
-static
 T x@p;
 ... when != x
     when strict
?x = e;
// </smpl>

Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Tejun Heo <tj@kernel.org>
drivers/ata/sata_gemini.c

index 8c70452..ab97be6 100644 (file)
@@ -305,7 +305,7 @@ static int gemini_sata_probe(struct platform_device *pdev)
        struct device *dev = &pdev->dev;
        struct device_node *np = dev->of_node;
        struct sata_gemini *sg;
-       static struct regmap *map;
+       struct regmap *map;
        struct resource *res;
        enum gemini_muxmode muxmode;
        u32 gmode;