Index: /trunk/psModules/src/camera/pmHDUGenerate.c
===================================================================
--- /trunk/psModules/src/camera/pmHDUGenerate.c	(revision 9602)
+++ /trunk/psModules/src/camera/pmHDUGenerate.c	(revision 9603)
@@ -449,7 +449,5 @@
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////
 
-// Generate an HDU from a cell with pixels
-bool pmHDUGenerateForCell(pmCell *cell  // The cell
-                         )
+bool pmHDUGenerateForCell(pmCell *cell)
 {
     PS_ASSERT_PTR_NON_NULL(cell, false);
@@ -496,12 +494,10 @@
         return pmHDUGenerateForChip(cell->parent);
     default:
-        psAbort(__func__, "Shouldn't ever get here.\n");
+        psAbort(__func__, "Shouldn't ever get here: check your camera format configuration.\n");
     }
     return false;
 }
 
-// Generate an HDU from a chip with pixels
-bool pmHDUGenerateForChip(pmChip *chip  // The chip
-                         )
+bool pmHDUGenerateForChip(pmChip *chip)
 {
     PS_ASSERT_PTR_NON_NULL(chip, false);
@@ -546,5 +542,5 @@
         return pmHDUGenerateForFPA(chip->parent);
     default:
-        psAbort(__func__, "Shouldn't ever get here.\n");
+        psAbort(__func__, "Shouldn't ever get here: check your camera format configuration.\n");
     }
     return false;
@@ -552,7 +548,5 @@
 
 
-// Generate an HDU from an FPA with pixels
-bool pmHDUGenerateForFPA(pmFPA *fpa     // The fpa
-                        )
+bool pmHDUGenerateForFPA(pmFPA *fpa)
 {
     PS_ASSERT_PTR_NON_NULL(fpa, false);
@@ -590,5 +584,5 @@
         }
     default:
-        psAbort(__func__, "Shouldn't ever get here.\n");
+        psAbort(__func__, "Shouldn't ever get here: check your camera format configuration.\n");
     }
     return false;
Index: /trunk/psModules/src/camera/pmHDUGenerate.h
===================================================================
--- /trunk/psModules/src/camera/pmHDUGenerate.h	(revision 9602)
+++ /trunk/psModules/src/camera/pmHDUGenerate.h	(revision 9603)
@@ -1,2 +1,16 @@
+/// @file pmHDUGenerate.h
+///
+/// @brief Generate HDU pixels from FPA components that have pixels
+///
+/// @ingroup Camera
+///
+/// @author Paul Price, IfA
+///
+/// @version $Revision: 1.2 $ $Name: not supported by cvs2svn $
+/// @date $Date: 2006-10-17 03:29:08 $
+///
+/// Copyright 2005-2006 Institute for Astronomy, University of Hawaii
+///
+
 #ifndef PM_HDU_GENERATE_H
 #define PM_HDU_GENERATE_H
@@ -4,14 +18,38 @@
 #include "pmFPA.h"
 
-// Generate an HDU for a cell with pixels
-bool pmHDUGenerateForCell(pmCell *cell  // The cell for which to generate an HDU
+/// Generate an HDU (with CELL.TRIMSEC, CELL.BIASSEC and pixels) for a cell with pixels
+///
+/// The write functions for the FPA hierarchy use pmHDUWrite, which assumes that the images in the readouts
+/// are subimages of the pixels in the HDU structure.  If this is not the case, the HDU pixels can be
+/// generated using some simple assumptions.  Splices the images and overscans together without regard for
+/// CELL.X0 and CELL.Y0 (for a proper mosaic, see pmFPAMosaic), though it should respect CELL.READDIR (so that
+/// the bias and trim sections match properly).  A warning may be generated after running this function if the
+/// bias and trim sections are specified in the camera format by default values rather than in the header.
+/// Failure of this function is often due to a bad camera format file.
+bool pmHDUGenerateForCell(pmCell *cell  ///< The cell for which to generate an HDU
                          );
 
-// Generate an HDU for a cell with pixels
-bool pmHDUGenerateForChip(pmChip *chip  // The chip for which to generate an HDU
+/// Generate an HDU (with CELL.TRIMSEC, CELL.BIASSEC and pixels) for a cell with pixels
+///
+/// The write functions for the FPA hierarchy use pmHDUWrite, which assumes that the images in the readouts
+/// are subimages of the pixels in the HDU structure.  If this is not the case, the HDU pixels can be
+/// generated using some simple assumptions.  Splices the images and overscans together without regard for
+/// CELL.X0 and CELL.Y0 (for a proper mosaic, see pmFPAMosaic), though it should respect CELL.READDIR (so that
+/// the bias and trim sections match properly).  A warning may be generated after running this function if the
+/// bias and trim sections are specified in the camera format by default values rather than in the header.
+/// Failure of this function is often due to a bad camera format file.
+bool pmHDUGenerateForChip(pmChip *chip  ///< The chip for which to generate an HDU
                          );
 
-// Generate an HDU from an FPA with pixels
-bool pmHDUGenerateForFPA(pmFPA *fpa     // The fpa for which to generate an HDU
+// Generate an HDU (with CELL.TRIMSEC, CELL.BIASSEC and pixels) from an FPA with pixels
+///
+/// The write functions for the FPA hierarchy use pmHDUWrite, which assumes that the images in the readouts
+/// are subimages of the pixels in the HDU structure.  If this is not the case, the HDU pixels can be
+/// generated using some simple assumptions.  Splices the images and overscans together without regard for
+/// CELL.X0 and CELL.Y0 (for a proper mosaic, see pmFPAMosaic), though it should respect CELL.READDIR (so that
+/// the bias and trim sections match properly).  A warning may be generated after running this function if the
+/// bias and trim sections are specified in the camera format by default values rather than in the header.
+/// Failure of this function is often due to a bad camera format file.
+bool pmHDUGenerateForFPA(pmFPA *fpa     ///< The fpa for which to generate an HDU
                         );