> > > +subsys_initcall(hnae3_init);
> > 
> > And the point of this is?
> > 
> >     Andrew
> This looks weird but not sure how we could have made sure HNAE is available 
> before other
> 2 drivers could have loaded. Changing into module_init() means two other 
> modules (dependent
> upon hnae) if loaded first will experience load time linking problems. Do you 
> think this
> is okay? 

Often the best way to decide if something is sensible is to see if
lots of other drivers do the same. Can you point to other drivers
doing this?

depmod/modprobe will actually sort this out for you. depmod looks at
the symbols exported and required for each module and builds a
dependency tree. When you modprobe a driver, it looks at the
dependency tree, and loads any drivers needed in order to fulfil the
dependencies.

        Andrew

Reply via email to