Index: /trunk/psModules/src/camera/pmFPAWrite.c
===================================================================
--- /trunk/psModules/src/camera/pmFPAWrite.c	(revision 9601)
+++ /trunk/psModules/src/camera/pmFPAWrite.c	(revision 9602)
@@ -15,8 +15,5 @@
 #include "pmFPAWrite.h"
 
-bool pmReadoutWriteNext(pmReadout *readout, // Readout to write
-                        psFits *fits,   // FITS file to which to write
-                        int z           // Image plane to write
-                       )
+bool pmReadoutWriteNext(pmReadout *readout, psFits *fits, int z)
 {
     PS_ASSERT_PTR_NON_NULL(readout, false);
@@ -133,10 +130,5 @@
 
 
-bool pmChipWrite(pmChip *chip,          // Chip to write
-                 psFits *fits,          // FITS file to which to write
-                 psDB *db,              // Database handle for "concepts" update
-                 bool blank,            // Write a blank PHU?
-                 bool recurse           // Recurse to lower levels?
-                )
+bool pmChipWrite(pmChip *chip, psFits *fits, psDB *db, bool blank, bool recurse)
 {
     PS_ASSERT_PTR_NON_NULL(chip, false);
@@ -193,10 +185,5 @@
 
 
-bool pmFPAWrite(pmFPA *fpa,             // FPA to write
-                psFits *fits,           // FITS file to which to write
-                psDB *db,               // Database handle for "concepts" update
-                bool blank,             // Write a blank PHU?
-                bool recurse            // Recurse to lower levels?
-               )
+bool pmFPAWrite(pmFPA *fpa, psFits *fits, psDB *db, bool blank, bool recurse)
 {
     PS_ASSERT_PTR_NON_NULL(fpa, false);
Index: /trunk/psModules/src/camera/pmFPAWrite.h
===================================================================
--- /trunk/psModules/src/camera/pmFPAWrite.h	(revision 9601)
+++ /trunk/psModules/src/camera/pmFPAWrite.h	(revision 9602)
@@ -1,29 +1,67 @@
+/// @file pmFPAWrite.h
+///
+/// @brief Write FPA components to a FITS file
+///
+/// @ingroup Camera
+///
+/// @author Paul Price, IfA
+///
+/// @version $Revision: 1.3 $ $Name: not supported by cvs2svn $
+/// @date $Date: 2006-10-17 03:16:59 $
+///
+/// Copyright 2005-2006 Institute for Astronomy, University of Hawaii
+///
+
 #ifndef PM_FPA_WRITE_H
 #define PM_FPA_WRITE_H
 
-#include "pslib.h"
+#include <pslib.h>
 #include "pmFPA.h"
 
-bool pmReadoutWriteNext(pmReadout *readout, // Readout to write
-                        psFits *fits,   // FITS file to which to write
-                        int z           // Image plane to write
+
+/// Write a readout incrementally
+///
+/// Writes a readout to a FITS file, perhaps incrementally (if it has been read in using pmReadoutReadNext).
+/// Relies on the FITS header to specify how many image planes there are, and the width and height.
+bool pmReadoutWriteNext(pmReadout *readout, ///< Readout to write
+                        psFits *fits,   ///<  FITS file to which to write
+                        int z           ///<  Image plane to write
                        );
 
-bool pmCellWrite(pmCell *cell,          // Cell to write
-                 psFits *fits,          // FITS file to which to write
-                 psDB *db,              // Database handle for "concepts" update
-                 bool pixels            // Write the pixels, or only the PHU header?
+/// Write a cell to a FITS file
+///
+/// Generates CELL.TRIMSEC, CELL.BIASSEC and the HDU pixels if required.  A blank (i.e., image-less header) is
+/// written only if specifically requested.  Writes the concepts to the various locations, and then the HDU to
+/// the FITS file.  This function should be called at the beginning of the output cell loop with blank=true in
+/// order to produce the correct file structure.
+bool pmCellWrite(pmCell *cell,          ///<  Cell to write
+                 psFits *fits,          ///<  FITS file to which to write
+                 psDB *db,              ///<  Database handle for "concepts" update
+                 bool blank             ///<  Write a blank PHU?
                 );
-bool pmChipWrite(pmChip *chip,          // Chip to write
-                 psFits *fits,          // FITS file to which to write
-                 psDB *db,              // Database handle for "concepts" update
-                 bool pixels,           // Write the pixels, or only the PHU header?
-                 bool recurse           // Recurse to lower levels?
+
+/// Write a chip to a FITS file
+///
+/// Generates CELL.TRIMSEC, CELL.BIASSEC and the HDU pixels if required.  A blank (i.e., image-less header) is
+/// written only if specifically requested.  Writes the concepts to the various locations, and then the HDU to
+/// the FITS file, optionally recursing to lower levels.  This function should be called at the beginning of
+/// the output chip loop with blank=true and recurse=false in order to produce the correct file structure.
+bool pmChipWrite(pmChip *chip,          ///<  Chip to write
+                 psFits *fits,          ///<  FITS file to which to write
+                 psDB *db,              ///<  Database handle for "concepts" update
+                 bool blank,            ///<  Write a blank PHU?
+                 bool recurse           ///<  Recurse to lower levels?
                 );
-bool pmFPAWrite(pmFPA *fpa,             // FPA to write
-                psFits *fits,           // FITS file to which to write
-                psDB *db,               // Database handle for "concepts" update
-                bool pixels,            // Write the pixels, or only the PHU header?
-                bool recurse            // Recurse to lower levels?
+/// Write a cell to a FITS file
+///
+/// Generates CELL.TRIMSEC, CELL.BIASSEC and the HDU pixels if required.  A blank (i.e., image-less header) is
+/// written only if specifically requested.  Writes the concepts to the various locations, and then the HDU to
+/// the FITS file, optionally recursing to lower levels.  This function should be called at the beginning of
+/// the output FPA loop with blank=true and recurse=false in order to produce the correct file structure.
+bool pmFPAWrite(pmFPA *fpa,             ///<  FPA to write
+                psFits *fits,           ///<  FITS file to which to write
+                psDB *db,               ///<  Database handle for "concepts" update
+                bool blank,             ///<  Write a blank PHU?
+                bool recurse            ///<  Recurse to lower levels?
                );
 
