]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
drm: fourcc: Convert drm_format_info kerneldoc to in-line member documentation
authorAlexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
Fri, 19 Oct 2018 10:57:44 +0000 (11:57 +0100)
committerAlexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
Mon, 22 Oct 2018 09:30:26 +0000 (10:30 +0100)
In-line member documentation seems to be desired way of documenting
structure members.

This change had been suggested by Daniel Vetter here:
https://lists.freedesktop.org/archives/dri-devel/2018-October/192176.html

Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>
Reviewed-by: Maxime Ripard <maxime.ripard@bootlin.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20181019105752.17741-2-alexandru-cosmin.gheorghe@arm.com
include/drm/drm_fourcc.h

index 865ef60c17af385d2eb55db2d8c53a2f08d7ca07..345f11227e9efc66be22ad1c2bc38b2d90640e81 100644 (file)
@@ -52,25 +52,35 @@ struct drm_mode_fb_cmd2;
 
 /**
  * struct drm_format_info - information about a DRM format
- * @format: 4CC format identifier (DRM_FORMAT_*)
- * @depth: Color depth (number of bits per pixel excluding padding bits),
- *     valid for a subset of RGB formats only. This is a legacy field, do not
- *     use in new code and set to 0 for new formats.
- * @num_planes: Number of color planes (1 to 3)
- * @cpp: Number of bytes per pixel (per plane)
- * @hsub: Horizontal chroma subsampling factor
- * @vsub: Vertical chroma subsampling factor
- * @has_alpha: Does the format embeds an alpha component?
- * @is_yuv: Is it a YUV format?
  */
 struct drm_format_info {
+       /** @format: 4CC format identifier (DRM_FORMAT_*) */
        u32 format;
+
+       /**
+        * @depth:
+        *
+        * Color depth (number of bits per pixel excluding padding bits),
+        * valid for a subset of RGB formats only. This is a legacy field, do
+        * not use in new code and set to 0 for new formats.
+        */
        u8 depth;
+
+       /** @num_planes: Number of color planes (1 to 3) */
        u8 num_planes;
+
+       /** @cpp: Number of bytes per pixel (per plane) */
        u8 cpp[3];
+
+       /** @hsub: Horizontal chroma subsampling factor */
        u8 hsub;
+       /** @vsub: Vertical chroma subsampling factor */
        u8 vsub;
+
+       /** @has_alpha: Does the format embeds an alpha component? */
        bool has_alpha;
+
+       /** @is_yuv: Is it a YUV format? */
        bool is_yuv;
 };