powerpc: don't export static symbol
authorJulia Lawall <Julia.Lawall@lip6.fr>
Wed, 11 Mar 2015 16:56:24 +0000 (17:56 +0100)
committerScott Wood <scottwood@freescale.com>
Wed, 1 Apr 2015 03:23:18 +0000 (22:23 -0500)
The semantic patch that fixes this problem is as follows:
(http://coccinelle.lip6.fr/)

// <smpl>
@r@
type T;
identifier f;
@@

static T f (...) { ... }

@@
identifier r.f;
declarer name EXPORT_SYMBOL;
@@

-EXPORT_SYMBOL(f);
// </smpl>

Furthermore, the function is never used, so its definition is dropped as
well.

Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: Scott Wood <scottwood@freescale.com>
arch/powerpc/sysdev/qe_lib/qe_io.c

index d09994164dafd315027bd381209128e76fc79e15..7ea0174f6d3d7e9f805d7c2c092838032ab78e4d 100644 (file)
@@ -190,28 +190,3 @@ int par_io_of_config(struct device_node *np)
        return 0;
 }
 EXPORT_SYMBOL(par_io_of_config);
-
-#ifdef DEBUG
-static void dump_par_io(void)
-{
-       unsigned int i;
-
-       printk(KERN_INFO "%s: par_io=%p\n", __func__, par_io);
-       for (i = 0; i < num_par_io_ports; i++) {
-               printk(KERN_INFO "      cpodr[%u]=%08x\n", i,
-                       in_be32(&par_io[i].cpodr));
-               printk(KERN_INFO "      cpdata[%u]=%08x\n", i,
-                       in_be32(&par_io[i].cpdata));
-               printk(KERN_INFO "      cpdir1[%u]=%08x\n", i,
-                       in_be32(&par_io[i].cpdir1));
-               printk(KERN_INFO "      cpdir2[%u]=%08x\n", i,
-                       in_be32(&par_io[i].cpdir2));
-               printk(KERN_INFO "      cppar1[%u]=%08x\n", i,
-                       in_be32(&par_io[i].cppar1));
-               printk(KERN_INFO "      cppar2[%u]=%08x\n", i,
-                       in_be32(&par_io[i].cppar2));
-       }
-
-}
-EXPORT_SYMBOL(dump_par_io);
-#endif /* DEBUG */