diff options
| author | Uwe Kleine-König <[email protected]> | 2021-01-22 10:24:49 +0100 |
|---|---|---|
| committer | Takashi Iwai <[email protected]> | 2021-01-26 07:42:27 +0100 |
| commit | 30e88d017fcbeb50c4b07577fe059558361067e7 (patch) | |
| tree | 971c2212b650bbb811473bc7e55db671c9a57cfc /include/linux | |
| parent | 7797b4e00faf3229ef67ac37015dd2ee8e4c7901 (diff) | |
isa: Make the remove callback for isa drivers return void
The driver core ignores the return value of the remove callback, so
don't give isa drivers the chance to provide a value.
Adapt all isa_drivers with a remove callbacks accordingly; they all
return 0 unconditionally anyhow.
Acked-by: Marc Kleine-Budde <[email protected]> # for drivers/net/can/sja1000/tscan1.c
Acked-by: William Breathitt Gray <[email protected]>
Acked-by: Wolfram Sang <[email protected]> # for drivers/i2c/
Reviewed-by: Takashi Iway <[email protected]> # for sound/
Reviewed-by: Hans Verkuil <[email protected]> # for drivers/media/
Signed-off-by: Uwe Kleine-König <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Takashi Iwai <[email protected]>
Diffstat (limited to 'include/linux')
| -rw-r--r-- | include/linux/isa.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/isa.h b/include/linux/isa.h index 41336da0f4e7..e30963190968 100644 --- a/include/linux/isa.h +++ b/include/linux/isa.h @@ -13,7 +13,7 @@ struct isa_driver { int (*match)(struct device *, unsigned int); int (*probe)(struct device *, unsigned int); - int (*remove)(struct device *, unsigned int); + void (*remove)(struct device *, unsigned int); void (*shutdown)(struct device *, unsigned int); int (*suspend)(struct device *, unsigned int, pm_message_t); int (*resume)(struct device *, unsigned int); |