This seems fine to me, I don't see any reason we would want to build the modules as arm, and if we did for a one off test, this one module being broken wouldn't be a big deal.
Okay, once the D6 stuff is in trunk, this can follow.
We have a general rule about not having ifdefs in models, but IMO the target CPU architecture should be an exception, because the resulting modules are incompatible anyway.
Modules are built separately for the v5 and v7 architectures, the main 'switch' is the THUMB_FW define.
Something different: A module I'm planning would use the hexadecimal input box (hexbox) and the HP timer functions. May I go ahead adding them to the export list? The HP timer functions will need wrappers too.
If agreed, I'll do this after the D6 merge.