Re: [PATCH v3 02/26] staging: most: integrate driver in kernel's device model

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

 



On Mon, Oct 16, 2017 at 10:46:09AM +0200, Christian Gromm wrote:
> The following patch adapts the driver to use the device model by:
> 
> 	- adopting the MOST bus_type
> 	- registering the core as a busdriver
> 	- removing private kobject/kset usage
> 	- removing private lists and structures to track registered modules
> 	  and making use of the device model API
> 	- removing prefix of modules
> 	- allowing adapter drivers (a.k.a. HDM) to register MOST devices
> 	- registering AIM modules as components with the core to build
> 	  the user space experience of the driver stack
> 	- using attribute groups to create the sysfs files
> 	- renaming variables to prevent collision with the introduced device
> 	  structs.

Hint, when you have to enumerate a list of different things you do in a
single patch, that is a _huge_ sign that the patch needs to be broken up
into smaller pieces, each piece only doing one logical thing.

So please do that here, as it is, this patch is unreviewable because it
does too much at once.

thanks,

greg k-h
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux