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 Fri, Oct 20, 2017 at 04:20:33PM +0200, Christian Gromm wrote:
> On 18.10.2017 16:29, Greg KH wrote:
> > On Wed, Oct 18, 2017 at 04:02:33PM +0200, Christian Gromm wrote:
> > > On 18.10.2017 14:12, Greg KH wrote:
> > > > 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.
> > > > 
> > > 
> > > This is a huge reconstruction of the driver architecture that
> > > requires more than one thing be done at a time. Breaking this
> > > up into tiny patches might render the sources broken, if you
> > > don't apply the complete series. (Which is also a no-go, right?)
> > > 
> > > Since staging is meant to be the place to fix things up,
> > > I thought I'd get away with this. Anyways, I'll try...
> > 
> > Try to make it into something that you would want to be able to review :)
> > 
> 
> Well... ok then.
> 
> Do think you can spend an extra minute in Prague next week, so we can
> talk about what's next?

Sure, but to start with, getting this patch series into a mergable state
is "what is next" :)

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