On Fri, Jan 18, 2019 at 04:55:41PM +0100, Greg KH wrote: >On Fri, Jan 18, 2019 at 10:34:59AM +0800, Wei Yang wrote: >> As the description of struct device_private says, it stores data which >> is private to driver core. And it already has similar fields like: >> knode_parent, knode_driver, knode_driver and knode_bus. This look it is >> more proper to put knode_class together with those fields to make it >> private to driver core. >> >> This patch move device->knode_class to device_private to make it comply >> with code convention. >> >> Signed-off-by: Wei Yang <richardw.y...@linux.intel.com> >> Reviewed-by: Rafael J. Wysocki <rafael.j.wyso...@intel.com> > >Thanks for this. There are a number of other variables that should move >into this structure, if you want to take a look into doing that.
Really? I may not notice that yet. I may try to take a look into this, but not sure I would have enough time :-) > >greg k-h -- Wei Yang Help you, Help me
