Commit 85ebe0af authored by William Breathitt Gray's avatar William Breathitt Gray Committed by Bartosz Golaszewski

isa: Introduce the module_isa_driver_with_irq helper macro

Several ISA drivers feature IRQ support that can configured via an "irq"
array module parameter. This array typically matches directly with the
respective "base" array module parameter. To reduce code repetition, a
module_isa_driver_with_irq helper macro is introduced to provide a check
ensuring that the number of "irq" passed to the module matches with the
respective number of "base".
Signed-off-by: default avatarWilliam Breathitt Gray <william.gray@linaro.org>
Acked-by: default avatarWilliam Breathitt Gray <william.gray@linaro.org>
Signed-off-by: default avatarBartosz Golaszewski <brgl@bgdev.pl>
parent 13c5d4ce
...@@ -38,6 +38,32 @@ static inline void isa_unregister_driver(struct isa_driver *d) ...@@ -38,6 +38,32 @@ static inline void isa_unregister_driver(struct isa_driver *d)
} }
#endif #endif
#define module_isa_driver_init(__isa_driver, __num_isa_dev) \
static int __init __isa_driver##_init(void) \
{ \
return isa_register_driver(&(__isa_driver), __num_isa_dev); \
} \
module_init(__isa_driver##_init)
#define module_isa_driver_with_irq_init(__isa_driver, __num_isa_dev, __num_irq) \
static int __init __isa_driver##_init(void) \
{ \
if (__num_irq != __num_isa_dev) { \
pr_err("%s: Number of irq (%u) does not match number of base (%u)\n", \
__isa_driver.driver.name, __num_irq, __num_isa_dev); \
return -EINVAL; \
} \
return isa_register_driver(&(__isa_driver), __num_isa_dev); \
} \
module_init(__isa_driver##_init)
#define module_isa_driver_exit(__isa_driver) \
static void __exit __isa_driver##_exit(void) \
{ \
isa_unregister_driver(&(__isa_driver)); \
} \
module_exit(__isa_driver##_exit)
/** /**
* module_isa_driver() - Helper macro for registering a ISA driver * module_isa_driver() - Helper macro for registering a ISA driver
* @__isa_driver: isa_driver struct * @__isa_driver: isa_driver struct
...@@ -48,16 +74,22 @@ static inline void isa_unregister_driver(struct isa_driver *d) ...@@ -48,16 +74,22 @@ static inline void isa_unregister_driver(struct isa_driver *d)
* use this macro once, and calling it replaces module_init and module_exit. * use this macro once, and calling it replaces module_init and module_exit.
*/ */
#define module_isa_driver(__isa_driver, __num_isa_dev) \ #define module_isa_driver(__isa_driver, __num_isa_dev) \
static int __init __isa_driver##_init(void) \ module_isa_driver_init(__isa_driver, __num_isa_dev); \
{ \ module_isa_driver_exit(__isa_driver)
return isa_register_driver(&(__isa_driver), __num_isa_dev); \
} \ /**
module_init(__isa_driver##_init); \ * module_isa_driver_with_irq() - Helper macro for registering an ISA driver with irq
static void __exit __isa_driver##_exit(void) \ * @__isa_driver: isa_driver struct
{ \ * @__num_isa_dev: number of devices to register
isa_unregister_driver(&(__isa_driver)); \ * @__num_irq: number of IRQ to register
} \ *
module_exit(__isa_driver##_exit); * Helper macro for ISA drivers with irq that do not do anything special in
* module init/exit. Each module may only use this macro once, and calling it
* replaces module_init and module_exit.
*/
#define module_isa_driver_with_irq(__isa_driver, __num_isa_dev, __num_irq) \
module_isa_driver_with_irq_init(__isa_driver, __num_isa_dev, __num_irq); \
module_isa_driver_exit(__isa_driver)
/** /**
* max_num_isa_dev() - Maximum possible number registered of an ISA device * max_num_isa_dev() - Maximum possible number registered of an ISA device
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment