__initcalls are supposed to return a value.
Statically initialize the struct instead of doing strcpy() at runtime.