parisc: Make struct parisc_driver::remove() return void
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Sat, 7 Aug 2021 09:19:27 +0000 (11:19 +0200)
committerHelge Deller <deller@gmx.de>
Mon, 30 Aug 2021 08:18:25 +0000 (10:18 +0200)
commit87875c1084a28364dad8cd4f9ecbfdfe0b845ad5
treecdb664f154ac23a7053af31401d56a92c8b953d3
parentd220da0967dbda232350c5dc39317e04e0892743
parisc: Make struct parisc_driver::remove() return void

The caller of this function (parisc_driver_remove() in
arch/parisc/kernel/drivers.c) ignores the return value, so better don't
return any value at all to not wake wrong expectations in driver authors.

The only function that could return a non-zero value before was
ipmi_parisc_remove() which returns the return value of
ipmi_si_remove_by_dev(). Make this function return void, too, as for all
other callers the value is ignored, too.

Also fold in a small checkpatch fix for:

WARNING: Unnecessary space before function pointer arguments
+ void (*remove) (struct parisc_device *dev);

Acked-by: Dmitry Torokhov <dmitry.torokhov@gmail.com> (for drivers/input)
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Acked-by: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
Acked-by: Jiri Slaby <jirislaby@kernel.org>
Signed-off-by: Helge Deller <deller@gmx.de>
13 files changed:
arch/parisc/include/asm/parisc-device.h
drivers/char/ipmi/ipmi_si.h
drivers/char/ipmi/ipmi_si_intf.c
drivers/char/ipmi/ipmi_si_parisc.c
drivers/char/ipmi/ipmi_si_platform.c
drivers/input/keyboard/hilkbd.c
drivers/input/serio/gscps2.c
drivers/net/ethernet/i825xx/lasi_82596.c
drivers/parport/parport_gsc.c
drivers/scsi/lasi700.c
drivers/scsi/zalon.c
drivers/tty/serial/mux.c
sound/parisc/harmony.c