On Mon, Sep 28, 2015 at 01:46:57PM +0200, Andrzej Pietrasiewicz wrote: >> } >> >> -static struct f_lb_opts_attribute f_lb_opts_qlen = >> - __CONFIGFS_ATTR(qlen, S_IRUGO | S_IWUSR, >> - f_lb_opts_qlen_show, >> - f_lb_opts_qlen_store); >> - > In my opinion the below line belongs here: > > +CONFIGFS_ATTR(f_lb_opts_, qlen);
The idea is to keep all the attribute defintions near the attribute array, similar to how most drivers define their sysfs attributes. If you really don't like that way I'll move it back. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
