Based on comments from hch and jejb, an update to the ibmvscsi
driver patch.  Primarily style issues, though the removal of
gratuitous casting lead to some significant refactoring
of functions.

- Moved #include <asm/...> after #include <linux/...>
- removed the map_direct_data and map_indirect_data and moved
   the function inline
- Fixed function declaration style issues (static on a line by
   itself)
- Removed bogus cast from dma_map_single
- Moved setting of data transfer direction to a single helper
   function
- Fixed up some gratuitous casting of types
- Collapsed a couple of other functions inline
- Removed bogus check for NULL from remove
- Removed the "evt_struct_for" routine as being needlessly obfuscatory
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>