diff --git a/paddle/function/Im2Col.h b/paddle/function/Im2Col.h
index d461ec7510b482cd2ce7b7748eb9a9e057f1a8f4..6d76e229bfc21f7e0c3c681c30ec189b70144be0 100644
--- a/paddle/function/Im2Col.h
+++ b/paddle/function/Im2Col.h
@@ -24,10 +24,11 @@ enum ColFormat { kCFO = 0, kOCF = 1 };
  *        five dimensions in the Im2ColFunctor calculation,
  *        And in the Col2ImFunctor calculation, it is reversed.
  *
- * \param imData  Image data of NCHW format.
- *                The shape of imData is:
- *                [inputChannels, inputHeight, inputWidth].
- * \param colData colData data.
+ * \param imData   Image data.
+ * \param imShape  The shape of imData,
+ *                 [inputChannels, inputHeight, inputWidth].
+ * \param colData  Column data.
+ * \param colShape The shape of colData.
  *
  * If the template argument Format is kCFO, the shape of colData is:
  * [inputChannels, filterHeight, filterWidth, outputHeight, outputWidth]
@@ -38,10 +39,10 @@ enum ColFormat { kCFO = 0, kOCF = 1 };
  * outputHeight * outputWidth.
  *
  * Reshape:
- *     shape of colData                shape of sequence
+ *     shape of colData           shape of convolution matrix
  *     [inputChannels,
  *      filterHeight,
- *      filterWidth,      ======>    [seqLength, stepSize]
+ *      filterWidth,      ======>      [height, width]
  *      outputHeight,
  *      outputWidth]
  *
@@ -53,7 +54,7 @@ enum ColFormat { kCFO = 0, kOCF = 1 };
  * inputChannels * filterHeight * filterWidth.
  *
  * Reshape:
- *     shape of colData                shape of sequence
+ *     shape of colData             shape of sequence matrix
  *     [outputHeight,
  *      outputWidth,
  *      inputChannels,    ======>    [seqLength, stepSize]