On Fri, Feb 05, 2021 at 01:53:37PM -0700, Dave Jiang wrote: > -static const struct mdev_parent_ops idxd_vdcm_ops = { > +static ssize_t name_show(struct kobject *kobj, struct device *dev, char *buf) > +{ > + struct vdcm_idxd_type *type; > + > + type = idxd_vdcm_find_vidxd_type(dev, kobject_name(kobj)); > + > + if (type) > + return sprintf(buf, "%s\n", type->name); > + > + return -EINVAL; Success oriented flow Jason