static void __iomem *base;I do not understand you wanting a module parameter as well as a sysfs
+static unsigned int events = PVPANIC_PANICKED | PVPANIC_CRASH_LOADED;
+module_param(events, uint, 0644);
+MODULE_PARM_DESC(events, "set event limitation of pvpanic device");
file. Why is this needed? Why are you spreading this information out
across different apis and locations?
Again, adding module parameters is almost never a good idea anymore,
they are a pain to manage and use.