[PATCH 00/50] staging: most: rework driver architecture and fix defects
christian.gromm at microchip.com
Fri Nov 24 15:46:38 UTC 2017
On 24.11.2017 16:31, Greg KH wrote:
> 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.
Nice. This makes it a lot easier than reworking the series.
More information about the devel