diff options
| author | Dominik Brodowski <linux@dominikbrodowski.net> | 2022-01-02 18:48:58 +0100 |
|---|---|---|
| committer | Dominik Brodowski <linux@dominikbrodowski.net> | 2022-01-02 19:01:37 +0100 |
| commit | fffbcee9335ccf97adf2b5b342099b6430e8dbfd (patch) | |
| tree | 910d44d10abfc4c5dea71edaa7f30494f710cb5b /drivers/pcmcia | |
| parent | pcmcia: rsrc_nonstatic: Fix a NULL pointer dereference in nonstatic_find_mem_... (diff) | |
| download | linux-fffbcee9335ccf97adf2b5b342099b6430e8dbfd.tar.xz linux-fffbcee9335ccf97adf2b5b342099b6430e8dbfd.zip | |
pcmcia: make pcmcia_release_io() void, as no-one is interested in return value
As the only user of pcmcia_release_io() is not interested in its return
value, and we cannot do anything on failure, convert the function to return
void.
Reported-by: Jason Wang <wangborong@cdjrlc.com>
Signed-off-by: Dominik Brodowski <linux@dominikbrodowski.net>
Diffstat (limited to 'drivers/pcmcia')
| -rw-r--r-- | drivers/pcmcia/pcmcia_resource.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/pcmcia/pcmcia_resource.c b/drivers/pcmcia/pcmcia_resource.c index c1c197292111..d78091e79a0f 100644 --- a/drivers/pcmcia/pcmcia_resource.c +++ b/drivers/pcmcia/pcmcia_resource.c @@ -390,10 +390,9 @@ int pcmcia_release_configuration(struct pcmcia_device *p_dev) * "stale", we don't bother checking the port ranges against the * current socket values. */ -static int pcmcia_release_io(struct pcmcia_device *p_dev) +static void pcmcia_release_io(struct pcmcia_device *p_dev) { struct pcmcia_socket *s = p_dev->socket; - int ret = -EINVAL; config_t *c; mutex_lock(&s->ops_mutex); @@ -412,8 +411,6 @@ static int pcmcia_release_io(struct pcmcia_device *p_dev) out: mutex_unlock(&s->ops_mutex); - - return ret; } /* pcmcia_release_io */ |