> -----Original Message----- > From: Greg KH [mailto:[email protected]] > Sent: Monday, March 14, 2011 3:34 PM > To: KY Srinivasan > Cc: [email protected]; [email protected]; > [email protected]; [email protected]; Haiyang Zhang; > Mike > Sterling; Abhishek Kane (Mindtree Consulting PVT LTD) > Subject: Re: [PATCH 10/21] Staging: hv: Cleanup root device handling > > On Thu, Mar 10, 2011 at 02:08:06PM -0800, K. Y. Srinivasan wrote: > > Now we can complete the cleanup of the root device > > management. Use the preferred APIs for creating and > > managing the root device. As part of this cleanup get rid > > of the root device object from vmbus_driver_context. > > I don't understand, what is the "root device"?
This would be the device under /sys/devices that all other hyperv devices would be grouped under. This notion of the root device existed in the existing code; however its creation and management was unnecessarily complicated. Regards, K. Y > > The hyper-v "bus controller"? > > > > > Signed-off-by: K. Y. Srinivasan <[email protected]> > > Signed-off-by: Haiyang Zhang <[email protected]> > > Signed-off-by: Mike Sterling <[email protected]> > > Signed-off-by: Abhishek Kane <[email protected]> > > Signed-off-by: Hank Janssen <[email protected]> > > --- > > drivers/staging/hv/vmbus_drv.c | 126 > > ++++++---------------------------------- > > 1 files changed, 18 insertions(+), 108 deletions(-) > > > > diff --git a/drivers/staging/hv/vmbus_drv.c b/drivers/staging/hv/vmbus_drv.c > > index b473f46..8b9394a 100644 > > --- a/drivers/staging/hv/vmbus_drv.c > > +++ b/drivers/staging/hv/vmbus_drv.c > > @@ -19,6 +19,7 @@ > > * Hank Janssen <[email protected]> > > */ > > #include <linux/init.h> > > +#include <linux/err.h> > > #include <linux/module.h> > > #include <linux/device.h> > > #include <linux/irq.h> > > @@ -40,6 +41,8 @@ > > #define VMBUS_IRQ 0x5 > > #define VMBUS_IRQ_VECTOR IRQ5_VECTOR > > > > +static struct device *root_dev; /* Root device */ > > This shouldn't be a "raw" struct device, should it? It should be of a > type that shows exactly what it is. Is it a hyper_v device that talks > on the bus? > > Or is it a "platform" device that controls all of the devices on the > bus, and as such should be the "root" device of the bus tree? > > confused, > > greg k-h _______________________________________________ devel mailing list [email protected] http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
