[PATCH 00/50] staging: most: rework driver architecture and fix defects

Greg KH gregkh at linuxfoundation.org
Fri Nov 24 15:31:03 UTC 2017


On Tue, Nov 21, 2017 at 03:04:34PM +0100, Christian Gromm wrote:
> This patch set fixes bugs and integrates the driver to the kernel's device
> model by revising its architecture. Part of this change is rearranging the
> directory layout, renaming of files and folders and thereby creating a
> clean driver structure.

Ok I've now applied all of these to my testing branch.  I think you need
to make some follow-up patches:
	- fix the build to actually compile the individual drivers :)
	- make DEVICE_ATTR structures static
	- use ATTRIBUTE_GROUPS where you can
	- use DEVICE_ATTR_RW() and _RO and _WO instead of a "raw"
	  DEVICE_ATTR() call.  There should not be any need to be having
	  any "special" file permissions for all of these attributes.
	- make sure you do not have any "empty" release functions.  I
	  saw one you removed, verify you don't have any others.

thanks,

greg k-h


More information about the devel mailing list