Hi folks,

is there a way to replace obsolete #ifdef's via spatch ?

Here's my scenario:

In the linux kernel we have many places where drivers statically assign
pointers to match tables (eg. for oftree or acpi) into a global driver
struct - sometimes this is enclosed into some #ifdef CONFIG_..., when
things like oftree or acpi are optional.

Example patch for illustrating what I'd like to do:

--- a/sound/soc/codecs/pcm3060-i2c.c
+++ b/sound/soc/codecs/pcm3060-i2c.c
@@ -45,9 +45,7 @@ static int pcm3060_i2c_probe(struct i2c_client *i2c,
 static struct i2c_driver pcm3060_i2c_driver = {
        .driver = {
                .name = "pcm3060",
-#ifdef CONFIG_OF
-               .of_match_table = pcm3060_of_match,
-#endif /* CONFIG_OF */
+               .of_match_table = of_match_ptr(pcm3060_of_match),
        },
        .id_table = pcm3060_i2c_id,
        .probe = pcm3060_i2c_probe,


Just wrapping the assignment into the macro call turned out to be easy.
But I haven't found a way to remove the now #ifdef :(

Any idea how I could solve this ?


--mtx

--
Enrico Weigelt, metux IT consult
Free software and Linux embedded engineering
i...@metux.net -- +49-151-27565287
_______________________________________________
Cocci mailing list
Cocci@systeme.lip6.fr
https://systeme.lip6.fr/mailman/listinfo/cocci

Reply via email to