On Wed, 8 Sep 2021 22:11:53 -0700 Dan Williams <[email protected]> wrote:
> Clean up existing kernel-doc warnings before adding new CXL label data > structures. > > drivers/nvdimm/label.h:66: warning: Function parameter or member 'labelsize' > not described in 'nd_namespace_index' > drivers/nvdimm/label.h:66: warning: Function parameter or member 'free' not > described in 'nd_namespace_index' > drivers/nvdimm/label.h:103: warning: Function parameter or member 'align' not > described in 'nd_namespace_label' > drivers/nvdimm/label.h:103: warning: Function parameter or member 'reserved' > not described in 'nd_namespace_label' > drivers/nvdimm/label.h:103: warning: Function parameter or member 'type_guid' > not described in 'nd_namespace_label' > drivers/nvdimm/label.h:103: warning: Function parameter or member > 'abstraction_guid' not described in 'nd_namespace_label' > drivers/nvdimm/label.h:103: warning: Function parameter or member 'reserved2' > not described in 'nd_namespace_label' > drivers/nvdimm/label.h:103: warning: Function parameter or member 'checksum' > not described in 'nd_namespace_label' > > Reported-by: Jonathan Cameron <[email protected]> > Signed-off-by: Dan Williams <[email protected]> LGTM Reviewed-by: Jonathan Cameron <[email protected]> > --- > drivers/nvdimm/label.h | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/drivers/nvdimm/label.h b/drivers/nvdimm/label.h > index 31f94fad7b92..7fa757d47846 100644 > --- a/drivers/nvdimm/label.h > +++ b/drivers/nvdimm/label.h > @@ -34,6 +34,7 @@ enum { > * struct nd_namespace_index - label set superblock > * @sig: NAMESPACE_INDEX\0 > * @flags: placeholder > + * @labelsize: log2 size (v1 labels 128 bytes v2 labels 256 bytes) > * @seq: sequence number for this index > * @myoff: offset of this index in label area > * @mysize: size of this index struct > @@ -43,7 +44,7 @@ enum { > * @major: label area major version > * @minor: label area minor version > * @checksum: fletcher64 of all fields > - * @free[0]: bitmap, nlabel bits > + * @free: bitmap, nlabel bits > * > * The size of free[] is rounded up so the total struct size is a > * multiple of NSINDEX_ALIGN bytes. Any bits this allocates beyond > @@ -77,7 +78,12 @@ struct nd_namespace_index { > * @dpa: DPA of NVM range on this DIMM > * @rawsize: size of namespace > * @slot: slot of this label in label area > - * @unused: must be zero > + * @align: physical address alignment of the namespace > + * @reserved: reserved > + * @type_guid: copy of struct acpi_nfit_system_address.range_guid > + * @abstraction_guid: personality id (btt, btt2, fsdax, devdax....) > + * @reserved2: reserved > + * @checksum: fletcher64 sum of this object > */ > struct nd_namespace_label { > u8 uuid[NSLABEL_UUID_LEN]; >
