On 7/21/26 10:21 AM, Randy Dunlap wrote:
> Add missing struct member, function parameters, and function return
> values to eliminate all kernel-doc warnings:
>
> Warning: include/linux/nd.h:124 struct member 'common' not described in
> 'nd_namespace_io'
> Warning: include/linux/nd.h:124 Excess struct member 'dev' description in
> 'nd_namespace_io'
> Warning: include/linux/nd.h:166 function parameter 'flags' not described in
> 'nvdimm_read_bytes'
> Warning: include/linux/nd.h:166 No description found for return value of
> 'nvdimm_read_bytes'
> Warning: include/linux/nd.h:185 function parameter 'flags' not described in
> 'nvdimm_write_bytes'
> Warning: include/linux/nd.h:185 No description found for return value of
> 'nvdimm_write_bytes'
>
> Signed-off-by: Randy Dunlap <[email protected]>
Reviewed-by: Dave Jiang <[email protected]>
> ---
> Cc: Dan Williams <[email protected]>
> Cc: Vishal Verma <[email protected]>
> Cc: Dave Jiang <[email protected]>
> Cc: Alison Schofield <[email protected]>
> Cc: Ira Weiny <[email protected]>
> Cc: [email protected]
>
> include/linux/nd.h | 8 +++++++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
>
> --- linux-next-20260720.orig/include/linux/nd.h
> +++ linux-next-20260720/include/linux/nd.h
> @@ -110,7 +110,7 @@ static inline struct nd_namespace_common
>
> /**
> * struct nd_namespace_io - device representation of a persistent memory
> range
> - * @dev: namespace device created by the nd region driver
> + * @common: namespace device core infrastructure created by the nd region
> driver
> * @res: struct resource conversion of a NFIT SPA table
> * @size: cached resource_size(@res) for fast path size checks
> * @addr: virtual address to access the namespace range
> @@ -158,8 +158,11 @@ static inline struct nd_namespace_pmem *
> * @offset: namespace-relative starting offset
> * @buf: buffer to fill
> * @size: transfer length
> + * @flags: process (0) or atomic (1) context
> *
> * @buf is up-to-date upon return from this routine.
> + *
> + * Returns: %0 on success or a negative error code on failure
> */
> static inline int nvdimm_read_bytes(struct nd_namespace_common *ndns,
> resource_size_t offset, void *buf, size_t size,
> @@ -174,11 +177,14 @@ static inline int nvdimm_read_bytes(stru
> * @offset: namespace-relative starting offset
> * @buf: buffer to drain
> * @size: transfer length
> + * @flags: process (0) or atomic (1) context
> *
> * NVDIMM Namepaces disks do not implement sectors internally. Depending on
> * the @ndns, the contents of @buf may be in cpu cache, platform buffers,
> * or on backing memory media upon return from this routine. Flushing
> * to media is handled internal to the @ndns driver, if at all.
> + *
> + * Returns: %0 on success or a negative error code on failure
> */
> static inline int nvdimm_write_bytes(struct nd_namespace_common *ndns,
> resource_size_t offset, void *buf, size_t size,