Jason Wang
2021-Jul-15 08:10 UTC
[PATCH 4/4] vdpa: Add documentation for vdpa_alloc_device() macro
? 2021/7/15 ??4:00, Xie Yongji ??:> The return value of vdpa_alloc_device() macro is not very > clear, so that most of callers did the wrong check. Let's > add some comments to better document it. > > Signed-off-by: Xie Yongji <xieyongji at bytedance.com>Acked-by: Jason Wang <jasowang at redhat.com>> --- > include/linux/vdpa.h | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/include/linux/vdpa.h b/include/linux/vdpa.h > index 3357ac98878d..8cfe49d201dd 100644 > --- a/include/linux/vdpa.h > +++ b/include/linux/vdpa.h > @@ -277,6 +277,17 @@ struct vdpa_device *__vdpa_alloc_device(struct device *parent, > const struct vdpa_config_ops *config, > size_t size, const char *name); > > +/** > + * vdpa_alloc_device - allocate and initilaize a vDPA device > + * > + * @dev_struct: the type of the parent structure > + * @member: the name of struct vdpa_device within the @dev_struct > + * @parent: the parent device > + * @config: the bus operations that is supported by this device > + * @name: name of the vdpa device > + * > + * Return allocated data structure or ERR_PTR upon error > + */ > #define vdpa_alloc_device(dev_struct, member, parent, config, name) \ > container_of(__vdpa_alloc_device( \ > parent, config, \