Dan Williams <dan.j.willi...@intel.com> writes:

> @@ -312,8 +312,9 @@ static ssize_t flags_show(struct device *dev,
>  {
>       struct nvdimm *nvdimm = to_nvdimm(dev);
>  
> -     return sprintf(buf, "%s%s\n",
> +     return sprintf(buf, "%s%s%s\n",
>                       test_bit(NDD_ALIASING, &nvdimm->flags) ? "alias " : "",
> +                     test_bit(NDD_LABELING, &nvdimm->flags) ? "label" : "",
                                                                       ^

Missing a space.

The rest looks sane.

Reviewed-by: Jeff Moyer <jmo...@redhat.com>

Reply via email to