firmware: cleanup - cleaned up some documentation

Cleaned up some documentation and moved some methods around to group
some implementation.

Change-Id: I25f1c8428f3ee679b5b3f3e34f2d60864626f89f
Signed-off-by: Patrick Venture <venture@google.com>
diff --git a/firmware_handler.hpp b/firmware_handler.hpp
index 30b450b..e3f9f3b 100644
--- a/firmware_handler.hpp
+++ b/firmware_handler.hpp
@@ -18,12 +18,15 @@
  */
 struct Session
 {
-    /** Pointer to the correct Data handler interface. (nullptr on BT (or KCS))
+    /**
+     * Pointer to the correct Data handler interface. (nullptr on BT (or KCS))
      */
     DataInterface* dataHandler;
 
-    /** Pointer to the correct image handler interface.  (nullptr on hash
-     * blob_id) */
+    /**
+     * Pointer to the correct image handler interface.  (nullptr on hash
+     * blob_id)
+     */
     ImageHandlerInterface* imageHandler;
 
     /** The flags used to open the session. */