diff options
| author |   Uwe Kleine-König <u.kleine-koenig@pengutronix.de> | 2022-01-23 18:51:59 +0100 |
|---|---|---|
| committer |   Mark Brown <broonie@kernel.org> | 2022-02-09 13:00:42 +0000 |
| commit | 316f569df766df9a49c36c052ec6afaf19cb6933 (patch) | |
| tree | 56e577291a422a53f6fd7cd677e0c60d9119e529 /drivers/char/tpm/st33zp24/st33zp24.c | |
| parent | staging: fbtft: Deduplicate driver registration macros (diff) | |
| download | linux-dev-316f569df766df9a49c36c052ec6afaf19cb6933.tar.xz linux-dev-316f569df766df9a49c36c052ec6afaf19cb6933.zip | |
tpm: st33zp24: Make st33zp24_remove() a void function
Up to now st33zp24_remove() returns zero unconditionally. Make it return
no value instead which makes it easier to see in the callers that there is
no error to handle.
Also the return value of i2c and spi remove callbacks is ignored anyway.
Link: https://lore.kernel.org/r/20220104231103.227924-1-u.kleine-koenig@pengutronix.de
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Acked-by: Lee Jones <lee.jones@linaro.org>
Link: https://lore.kernel.org/r/20220123175201.34839-4-u.kleine-koenig@pengutronix.de
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/char/tpm/st33zp24/st33zp24.c')
| -rw-r--r-- | drivers/char/tpm/st33zp24/st33zp24.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/char/tpm/st33zp24/st33zp24.c b/drivers/char/tpm/st33zp24/st33zp24.c index ce9efb73c144..15b393e92c8e 100644 --- a/drivers/char/tpm/st33zp24/st33zp24.c +++ b/drivers/char/tpm/st33zp24/st33zp24.c @@ -511,10 +511,9 @@ _tpm_clean_answer: } EXPORT_SYMBOL(st33zp24_probe); -int st33zp24_remove(struct tpm_chip *chip) +void st33zp24_remove(struct tpm_chip *chip) { tpm_chip_unregister(chip); - return 0; } EXPORT_SYMBOL(st33zp24_remove); |