Staging: bcm2835: Fixed style of block comments
authorDerek Robson <robsonde@gmail.com>
Sat, 11 Mar 2017 03:01:59 +0000 (16:01 +1300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 12 Mar 2017 14:16:55 +0000 (15:16 +0100)
Fixed style of block comments across whole driver
Found using checkpatch

Signed-off-by: Derek Robson <robsonde@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c
drivers/staging/vc04_services/bcm2835-camera/controls.c
drivers/staging/vc04_services/bcm2835-camera/mmal-msg-port.h
drivers/staging/vc04_services/bcm2835-camera/mmal-msg.h
drivers/staging/vc04_services/bcm2835-camera/mmal-vchiq.c
drivers/staging/vc04_services/bcm2835-camera/mmal-vchiq.h

index 2302f4bc84df13645afc47f46f243189c41f6da7..86bbd6e899a88f7dd0bbafd3344456536e775e7a 100644 (file)
@@ -236,8 +236,9 @@ static struct mmal_fmt *get_format(struct v4l2_format *f)
 }
 
 /* ------------------------------------------------------------------
-       Videobuf queue operations
-   ------------------------------------------------------------------*/
+ *     Videobuf queue operations
+ * ------------------------------------------------------------------
+ */
 
 static int queue_setup(struct vb2_queue *vq,
                       unsigned int *nbuffers, unsigned int *nplanes,
@@ -665,8 +666,9 @@ static struct vb2_ops bm2835_mmal_video_qops = {
 };
 
 /* ------------------------------------------------------------------
-       IOCTL operations
-   ------------------------------------------------------------------*/
+ *     IOCTL operations
+ * ------------------------------------------------------------------
+ */
 
 static int set_overlay_params(struct bm2835_mmal_dev *dev,
                              struct vchiq_mmal_port *port)
@@ -828,7 +830,8 @@ static int vidioc_g_fbuf(struct file *file, void *fh,
                         struct v4l2_framebuffer *a)
 {
        /* The video overlay must stay within the framebuffer and can't be
-          positioned independently. */
+        * positioned independently.
+        */
        struct bm2835_mmal_dev *dev = video_drvdata(file);
        struct vchiq_mmal_port *preview_port =
                    &dev->component[MMAL_COMPONENT_CAMERA]->
@@ -1285,7 +1288,8 @@ static int vidioc_s_fmt_vid_cap(struct file *file, void *priv,
        }
 
        /* If the format is unsupported v4l2 says we should switch to
-        * a supported one and not return an error. */
+        * a supported one and not return an error.
+        */
        mfmt = get_format(f);
        if (!mfmt) {
                v4l2_dbg(1, bcm2835_v4l2_debug, &dev->v4l2_dev,
@@ -1479,7 +1483,8 @@ static const struct v4l2_ioctl_ops camera0_ioctl_ops_gstreamer = {
        .vidioc_qbuf = vb2_ioctl_qbuf,
        .vidioc_dqbuf = vb2_ioctl_dqbuf,
        /* Remove this function ptr to fix gstreamer bug
-       .vidioc_enum_framesizes = vidioc_enum_framesizes, */
+        * .vidioc_enum_framesizes = vidioc_enum_framesizes,
+        */
        .vidioc_enum_frameintervals = vidioc_enum_frameintervals,
        .vidioc_g_parm        = vidioc_g_parm,
        .vidioc_s_parm        = vidioc_s_parm,
@@ -1492,8 +1497,9 @@ static const struct v4l2_ioctl_ops camera0_ioctl_ops_gstreamer = {
 };
 
 /* ------------------------------------------------------------------
-       Driver init/finalise
-   ------------------------------------------------------------------*/
+ *     Driver init/finalise
+ * ------------------------------------------------------------------
+ */
 
 static const struct v4l2_file_operations camera0_fops = {
        .owner = THIS_MODULE,
index 341a072096a2eccb62661350ebacd059e7028eb2..77a5d6f4e1eb8fe4e2482bb62d429b04598a1759 100644 (file)
@@ -90,7 +90,8 @@ struct bm2835_mmal_v4l2_ctrl {
        u32 id; /* v4l2 control identifier */
        enum bm2835_mmal_ctrl_type type;
        /* control minimum value or
-        * mask for MMAL_CONTROL_TYPE_STD_MENU */
+        * mask for MMAL_CONTROL_TYPE_STD_MENU
+        */
        s32 min;
        s32 max; /* maximum value of control */
        s32 def;  /* default value of control */
@@ -396,10 +397,10 @@ static int ctrl_set_metering_mode(struct bm2835_mmal_dev *dev,
                break;
 
        /* todo matrix weighting not added to Linux API till 3.9
-       case V4L2_EXPOSURE_METERING_MATRIX:
-               dev->metering_mode = MMAL_PARAM_EXPOSUREMETERINGMODE_MATRIX;
-               break;
-       */
+        * case V4L2_EXPOSURE_METERING_MATRIX:
+        *      dev->metering_mode = MMAL_PARAM_EXPOSUREMETERINGMODE_MATRIX;
+        *      break;
+        */
        }
 
        if (dev->scene_mode == V4L2_SCENE_MODE_NONE) {
@@ -978,8 +979,9 @@ static const struct bm2835_mmal_v4l2_ctrl v4l2_ctrls[V4L2_CTRL_COUNT] = {
                false
        },
 /*     {
-               0, MMAL_CONTROL_TYPE_CLUSTER, 3, 1, 0, NULL, 0, NULL
-       }, */
+ *             0, MMAL_CONTROL_TYPE_CLUSTER, 3, 1, 0, NULL, 0, NULL
+ *     },
+ */
        {
                V4L2_CID_EXPOSURE_AUTO, MMAL_CONTROL_TYPE_STD_MENU,
                ~0x03, 3, V4L2_EXPOSURE_AUTO, 0, NULL,
@@ -988,9 +990,9 @@ static const struct bm2835_mmal_v4l2_ctrl v4l2_ctrls[V4L2_CTRL_COUNT] = {
                false
        },
 /* todo this needs mixing in with set exposure
      {
-              V4L2_CID_SCENE_MODE, MMAL_CONTROL_TYPE_STD_MENU,
      },
*     {
*             V4L2_CID_SCENE_MODE, MMAL_CONTROL_TYPE_STD_MENU,
*     },
  */
        {
                V4L2_CID_EXPOSURE_ABSOLUTE, MMAL_CONTROL_TYPE_STD,
index 369d5d02c350312dea1dd54c8b37b0607d2659bf..84a0f4b717ef36e0244da0e510534117fb44e5a0 100644 (file)
@@ -26,12 +26,14 @@ enum mmal_port_type {
 #define MMAL_PORT_CAPABILITY_PASSTHROUGH                       0x01
 /** The port wants to allocate the buffer payloads.
  * This signals a preference that payload allocation should be done
- * on this port for efficiency reasons. */
+ * on this port for efficiency reasons.
+ */
 #define MMAL_PORT_CAPABILITY_ALLOCATION                        0x02
 /** The port supports format change events.
  * This applies to input ports and is used to let the client know
  * whether the port supports being reconfigured via a format
- * change event (i.e. without having to disable the port). */
+ * change event (i.e. without having to disable the port).
+ */
 #define MMAL_PORT_CAPABILITY_SUPPORTS_EVENT_FORMAT_CHANGE      0x04
 
 /* mmal port structure (MMAL_PORT_T)
index ee074be1dbe8334e2e86ea2ac7d8cd90f8d05505..bf3d1f3f2c53c0159787438a5ef23b65f76835b3 100644 (file)
@@ -108,9 +108,9 @@ struct mmal_msg_component_create {
 
 /* reply from VC to component creation request */
 struct mmal_msg_component_create_reply {
-       u32 status; /** enum mmal_msg_status - how does this differ to
-                    * the one in the header?
-                    */
+       u32 status;     /* enum mmal_msg_status - how does this differ to
+                        * the one in the header?
+                        */
        u32 component_handle; /* VideoCore handle for component */
        u32 input_num;        /* Number of input ports */
        u32 output_num;       /* Number of output ports */
@@ -228,10 +228,12 @@ struct mmal_msg_port_action_reply {
 /** Signals that the current payload is a keyframe (i.e. self decodable) */
 #define MMAL_BUFFER_HEADER_FLAG_KEYFRAME               (1<<3)
 /** Signals a discontinuity in the stream of data (e.g. after a seek).
- * Can be used for instance by a decoder to reset its state */
+ * Can be used for instance by a decoder to reset its state
+ */
 #define MMAL_BUFFER_HEADER_FLAG_DISCONTINUITY          (1<<4)
 /** Signals a buffer containing some kind of config data for the component
- * (e.g. codec config data) */
+ * (e.g. codec config data)
+ */
 #define MMAL_BUFFER_HEADER_FLAG_CONFIG                 (1<<5)
 /** Signals an encrypted payload */
 #define MMAL_BUFFER_HEADER_FLAG_ENCRYPTED              (1<<6)
@@ -314,9 +316,9 @@ struct mmal_msg_port_parameter_set {
 };
 
 struct mmal_msg_port_parameter_set_reply {
-       u32 status; /** enum mmal_msg_status todo: how does this
-                    * differ to the one in the header?
-                    */
+       u32 status;     /* enum mmal_msg_status todo: how does this
+                        * differ to the one in the header?
+                        */
 };
 
 /* port parameter getting */
index 88b379f52e0b26c956a196dd58b1bfa1709b77c6..fc1076db0f82dbfdbbb9822e26f482dd5f9ceb88 100644 (file)
@@ -1559,7 +1559,8 @@ done:
 
 /* ------------------------------------------------------------------
  * Exported API
- *------------------------------------------------------------------*/
+ *------------------------------------------------------------------
+ */
 
 int vchiq_mmal_port_set_format(struct vchiq_mmal_instance *instance,
                               struct vchiq_mmal_port *port)
index 2842ac960c4ac9eeb245405935d363faef6dcb9d..5140b330325e6171cebe8ef3bc1ed37f8abec601 100644 (file)
@@ -108,8 +108,8 @@ int vchiq_mmal_init(struct vchiq_mmal_instance **out_instance);
 int vchiq_mmal_finalise(struct vchiq_mmal_instance *instance);
 
 /* Initialise a mmal component and its ports
-*
-*/
+ *
+ */
 int vchiq_mmal_component_init(
                struct vchiq_mmal_instance *instance,
                const char *name,