On Sun, Sep 24, 2017 at 01:29:09PM +0000, Amrani, Ram wrote: > > static struct verbs_device *qedr_device_alloc(struct verbs_sysfs_dev *sysfs_dev) > > { > > struct qelr_device *dev; > > @@ -269,7 +238,7 @@ static const struct verbs_device_ops qelr_dev_ops = { > > .name = "qelr", > > Should this be the driver name or the library name? > The driver name is qedr and the library name is libqedr. It is usually abbreviated to qelr. > I reckon that this should be the driver name (otherwise auto-load will fail?!). I will fix all the names, it is confusing as all the other drivers use their lib name as prefix.. The above string is only used for diagnostic prints. Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html