Re: [PATCH v8 1/5] configfs: Allow dynamic group creation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Oct 20, 2015 at 01:53:42PM +0300, Daniel Baluta wrote:
> We don't want to hardcode default groups at subsystem
> creation time. We export:
> 	* configfs_register_group
> 	* configfs_unregister_group
> to allow drivers to programatically create/destroy groups
> later, after module init time.
> 
> This is needed for IIO configfs support.

This looks like a good start, and maybe we should merge it as is given
that merge window is about to end.  But I have two requests, one trivial
and one not so:

 a) please provide kerneldoc comments for the new functions
 b) just pass the name and item type and dynamically allocate the
    actua group. With that we can avoid a lot of boilerplate code and
    could also replace the horrible static sized defaults_groups array
    in current users with your new function.

--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux