stop using C bitfield access in SolarFlare driver
explicitly pass self when calling member functions
remove timer.init() call
revert to using unions and uints of proper length in ef_event
move locals back into inner scope

There is no visible performance benefit from keeping them in
module global scope.