David,
But once it gets exported out of that file, you have to give it an appropriate name. Probably with a "batman_adv_" prefix or similar.
Otherwise, for one thing, if this code is statically built into the tree it can conflict with symbol names elsewhere and break the build.
thanks for pointing this out. I wasn't aware of the potential conflict created by our naming scheme. Obviously, we will change the proposed patch. We are going to review all other exported function names and send appropriate patches later on.
Regards, Marek