Messages in this thread Patch in this message | | | Date | Thu, 7 Feb 2013 23:57:38 +0100 | From | Samuel Ortiz <> | Subject | Re: [char-misc-next 05/11] mei: bus: Call bus routines from the core code |
| |
Hi Arnd,
On Thu, Feb 07, 2013 at 10:37:30PM +0000, Arnd Bergmann wrote: > On Thursday 07 February 2013, Tomas Winkler wrote: > > @@ -197,6 +197,9 @@ static int mei_probe(struct pci_dev *pdev, const struct pci_device_id *ent) > > mei_pdev = pdev; > > pci_set_drvdata(pdev, dev); > > > > + err = mei_bus_init(mei_pdev); > > + if (err) > > + goto deregister_mei; > > > > schedule_delayed_work(&dev->timer_work, HZ); > > > > This is fairly unusual, and will break if you ever have multiple mei devices > in one system, because you end up registering the bus type for each > device. I think it would be more logical to register/unregister > the bus_type from the module_init/exit functions of the module > that contains the bus_type object. That makes sense, I'll fix that.
Cheers, Samuel.
-- Intel Open Source Technology Centre http://oss.intel.com/
| |