123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122 |
- ISA Drivers
- -----------
- The following text is adapted from the commit message of the initial
- commit of the ISA bus driver authored by Rene Herman.
- During the recent "isa drivers using platform devices" discussion it was
- pointed out that (ALSA) ISA drivers ran into the problem of not having
- the option to fail driver load (device registration rather) upon not
- finding their hardware due to a probe() error not being passed up
- through the driver model. In the course of that, I suggested a separate
- ISA bus might be best; Russell King agreed and suggested this bus could
- use the .match() method for the actual device discovery.
- The attached does this. For this old non (generically) discoverable ISA
- hardware only the driver itself can do discovery so as a difference with
- the platform_bus, this isa_bus also distributes match() up to the
- driver.
- As another difference: these devices only exist in the driver model due
- to the driver creating them because it might want to drive them, meaning
- that all device creation has been made internal as well.
- The usage model this provides is nice, and has been acked from the ALSA
- side by Takashi Iwai and Jaroslav Kysela. The ALSA driver module_init's
- now (for oldisa-only drivers) become:
- static int __init alsa_card_foo_init(void)
- {
- return isa_register_driver(&snd_foo_isa_driver, SNDRV_CARDS);
- }
- static void __exit alsa_card_foo_exit(void)
- {
- isa_unregister_driver(&snd_foo_isa_driver);
- }
- Quite like the other bus models therefore. This removes a lot of
- duplicated init code from the ALSA ISA drivers.
- The passed in isa_driver struct is the regular driver struct embedding a
- struct device_driver, the normal probe/remove/shutdown/suspend/resume
- callbacks, and as indicated that .match callback.
- The "SNDRV_CARDS" you see being passed in is a "unsigned int ndev"
- parameter, indicating how many devices to create and call our methods
- with.
- The platform_driver callbacks are called with a platform_device param;
- the isa_driver callbacks are being called with a "struct device *dev,
- unsigned int id" pair directly -- with the device creation completely
- internal to the bus it's much cleaner to not leak isa_dev's by passing
- them in at all. The id is the only thing we ever want other then the
- struct device * anyways, and it makes for nicer code in the callbacks as
- well.
- With this additional .match() callback ISA drivers have all options. If
- ALSA would want to keep the old non-load behaviour, it could stick all
- of the old .probe in .match, which would only keep them registered after
- everything was found to be present and accounted for. If it wanted the
- behaviour of always loading as it inadvertently did for a bit after the
- changeover to platform devices, it could just not provide a .match() and
- do everything in .probe() as before.
- If it, as Takashi Iwai already suggested earlier as a way of following
- the model from saner buses more closely, wants to load when a later bind
- could conceivably succeed, it could use .match() for the prerequisites
- (such as checking the user wants the card enabled and that port/irq/dma
- values have been passed in) and .probe() for everything else. This is
- the nicest model.
- To the code...
- This exports only two functions; isa_{,un}register_driver().
- isa_register_driver() register's the struct device_driver, and then
- loops over the passed in ndev creating devices and registering them.
- This causes the bus match method to be called for them, which is:
- int isa_bus_match(struct device *dev, struct device_driver *driver)
- {
- struct isa_driver *isa_driver = to_isa_driver(driver);
- if (dev->platform_data == isa_driver) {
- if (!isa_driver->match ||
- isa_driver->match(dev, to_isa_dev(dev)->id))
- return 1;
- dev->platform_data = NULL;
- }
- return 0;
- }
- The first thing this does is check if this device is in fact one of this
- driver's devices by seeing if the device's platform_data pointer is set
- to this driver. Platform devices compare strings, but we don't need to
- do that with everything being internal, so isa_register_driver() abuses
- dev->platform_data as a isa_driver pointer which we can then check here.
- I believe platform_data is available for this, but if rather not, moving
- the isa_driver pointer to the private struct isa_dev is ofcourse fine as
- well.
- Then, if the the driver did not provide a .match, it matches. If it did,
- the driver match() method is called to determine a match.
- If it did _not_ match, dev->platform_data is reset to indicate this to
- isa_register_driver which can then unregister the device again.
- If during all this, there's any error, or no devices matched at all
- everything is backed out again and the error, or -ENODEV, is returned.
- isa_unregister_driver() just unregisters the matched devices and the
- driver itself.
- module_isa_driver is a helper macro for ISA drivers which do not do
- anything special in module init/exit. This eliminates a lot of
- boilerplate code. Each module may only use this macro once, and calling
- it replaces module_init and module_exit.
- max_num_isa_dev is a macro to determine the maximum possible number of
- ISA devices which may be registered in the I/O port address space given
- the address extent of the ISA devices.
|