Re: [PATCH] virtio: fix kernel-doc warnings for virtio_device

["Michael S. Tsirkin" <mst@redhat.com>]

On Thu, Mar 12, 2026 at 06:51:39AM -0400, Kenner de Azevedo dos Santos Miranda wrote:
> The following warnings were reported when running 'make htmldocs':
> 
>   WARNING: ./include/linux/virtio.h:188 struct member 'map'
>   not described in 'virtio_device'
> 
>   WARNING: ./include/linux/virtio.h:188 struct member
>   'VIRTIO_DECLARE_FEATURES(features' not described in
>   'virtio_device'
> 
>   WARNING: ./include/linux/virtio.h:188 struct member 'vmap'
>   not described in 'virtio_device'
> 
> Document the map and vmap fields in struct virtio_device.
> 
> Also avoid kernel-doc confusion caused by the VIRTIO_DECLARE_FEATURES()
> macro by documenting the logical @features field.
> 
> After these changes, running 'make htmldocs' no longer reports warnings
> for map, vmap, or VIRTIO_DECLARE_FEATURES().
> 
> Signed-off-by: Kenner de Azevedo dos Santos Miranda <kenner.linuxdev@gmail.com>


I will queue this. Thanks!

> ---
>  include/linux/virtio.h | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/include/linux/virtio.h b/include/linux/virtio.h
> index 3bbc4cb6a672..074d36453177 100644
> --- a/include/linux/virtio.h
> +++ b/include/linux/virtio.h
> @@ -157,11 +157,13 @@ struct virtio_admin_cmd {
>   * @id: the device type identification (used to match it with a driver).
>   * @config: the configuration ops for this device.
>   * @vringh_config: configuration ops for host vrings.
> + * @map: virtio specific mapping operations used by the device or transport.
>   * @vqs: the list of virtqueues for this device.
>   * @features: the 64 lower features supported by both driver and device.
>   * @features_array: the full features space supported by both driver and
> - *		    device.
> + *                device.
>   * @priv: private pointer for the driver's use.
> + * @vmap: mapping token passed to virtio mapping helpers and transport ops.
>   * @debugfs_dir: debugfs directory entry.
>   * @debugfs_filter_features: features to be filtered set by debugfs.
>   */
> @@ -179,6 +181,7 @@ struct virtio_device {
>  	const struct vringh_config_ops *vringh_config;
>  	const struct virtio_map_ops *map;
>  	struct list_head vqs;
> +       /* See @features */
>  	VIRTIO_DECLARE_FEATURES(features);
>  	void *priv;
>  	union virtio_map vmap;
> -- 
> 2.43.0