Messages in this thread | | | Date | Thu, 21 Dec 2023 15:29:47 -0800 | From | Ira Weiny <> | Subject | Re: [PATCH 3/3] nvdimm/namespace: fix kernel-doc for function params |
| |
Randy Dunlap wrote: > > > On 12/21/23 14:32, Ira Weiny wrote: > > Randy Dunlap wrote: > > > > [snip] > > > >> @@ -1656,8 +1664,10 @@ static int select_pmem_id(struct nd_regi > >> /** > >> * create_namespace_pmem - validate interleave set labelling, retrieve label0 > >> * @nd_region: region with mappings to validate > >> - * @nspm: target namespace to create > >> + * @nd_mapping: container of dpa-resource-root + labels > >> * @nd_label: target pmem namespace label to evaluate > >> + * > >> + * Returns: the created &struct device on success or -errno on error > > > > NIT: should this be ERR_PTR(-errno) on error? > > Oh, for sure. Thanks for catching that.
I'll clean this up as well if you are good with me changing patch 2/3 as well.
Ira
| |