diff --git a/libs/gst/controller/gstdirectcontrolbinding.c b/libs/gst/controller/gstdirectcontrolbinding.c
index 05964890f9..7787e238ab 100644
--- a/libs/gst/controller/gstdirectcontrolbinding.c
+++ b/libs/gst/controller/gstdirectcontrolbinding.c
@@ -478,7 +478,7 @@ gst_direct_control_binding_get_g_value_array (GstControlBinding * _self,
  * gst_direct_control_binding_new:
  * @object: the object of the property
  * @property_name: the property-name to attach the control source
- * @csource: the control source
+ * @cs: the control source
  *
  * Create a new control-binding that attaches the #GstControlSource to the
  * #GObject property.
diff --git a/libs/gst/controller/gstlfocontrolsource.c b/libs/gst/controller/gstlfocontrolsource.c
index 819518db65..20ee0fd4b1 100644
--- a/libs/gst/controller/gstlfocontrolsource.c
+++ b/libs/gst/controller/gstlfocontrolsource.c
@@ -556,7 +556,7 @@ gst_lfo_control_source_class_init (GstLFOControlSourceClass * klass)
   gobject_class->get_property = gst_lfo_control_source_get_property;
 
   /**
-   * GstLFOControlSource:waveform
+   * GstLFOControlSource:waveform:
    *
    * Specifies the waveform that should be used for this #GstLFOControlSource.
    */
@@ -566,7 +566,7 @@ gst_lfo_control_source_class_init (GstLFOControlSourceClass * klass)
           G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
 
   /**
-   * GstLFOControlSource:frequency
+   * GstLFOControlSource:frequency:
    *
    * Specifies the frequency that should be used for the waveform
    * of this #GstLFOControlSource. It should be large enough
@@ -578,7 +578,7 @@ gst_lfo_control_source_class_init (GstLFOControlSourceClass * klass)
           G_PARAM_READWRITE | GST_PARAM_CONTROLLABLE | G_PARAM_STATIC_STRINGS));
 
   /**
-   * GstLFOControlSource:timeshift
+   * GstLFOControlSource:timeshift:
    *
    * Specifies the timeshift to the right that should be used for the waveform
    * of this #GstLFOControlSource in nanoseconds.
@@ -593,7 +593,7 @@ gst_lfo_control_source_class_init (GstLFOControlSourceClass * klass)
           G_PARAM_READWRITE | GST_PARAM_CONTROLLABLE | G_PARAM_STATIC_STRINGS));
 
   /**
-   * GstLFOControlSource:amplitude
+   * GstLFOControlSource:amplitude:
    *
    * Specifies the amplitude for the waveform of this #GstLFOControlSource.
    */
@@ -603,7 +603,7 @@ gst_lfo_control_source_class_init (GstLFOControlSourceClass * klass)
           G_PARAM_READWRITE | GST_PARAM_CONTROLLABLE | G_PARAM_STATIC_STRINGS));
 
   /**
-   * GstLFOControlSource:offset
+   * GstLFOControlSource:offset:
    *
    * Specifies the value offset for the waveform of this #GstLFOControlSource.
    */
diff --git a/libs/gst/controller/gsttimedvaluecontrolsource.h b/libs/gst/controller/gsttimedvaluecontrolsource.h
index e1dde55b0e..d6624ed5c3 100644
--- a/libs/gst/controller/gsttimedvaluecontrolsource.h
+++ b/libs/gst/controller/gsttimedvaluecontrolsource.h
@@ -48,6 +48,7 @@ G_BEGIN_DECLS
 typedef struct _GstTimedValueControlSource GstTimedValueControlSource;
 typedef struct _GstTimedValueControlSourceClass GstTimedValueControlSourceClass;
 typedef struct _GstTimedValueControlSourcePrivate GstTimedValueControlSourcePrivate;
+typedef struct _GstControlPoint GstControlPoint;
 
 /**
  * GstControlPoint:
@@ -56,7 +57,7 @@ typedef struct _GstTimedValueControlSourcePrivate GstTimedValueControlSourcePriv
  * values used for interpolation. This "inherits" from
  * GstTimedValue.
  */
-typedef struct _GstControlPoint
+struct _GstControlPoint
 {
   /* fields from GstTimedValue. DO NOT CHANGE! */
   GstClockTime timestamp;       /* timestamp of the value change */
@@ -73,7 +74,7 @@ typedef struct _GstControlPoint
     } cubic;
   } cache;
 
-} GstControlPoint;
+};
 
 /**
  * GstTimedValueControlSource:
@@ -90,6 +91,7 @@ struct _GstTimedValueControlSource {
   gint nvalues;                 /* Number of control points */
   gboolean valid_cache;
 
+  /*< private >*/
   GstTimedValueControlSourcePrivate *priv;
   gpointer _gst_reserved[GST_PADDING];
 };