diff options
author | Takashi Iwai <tiwai@suse.de> | 2014-10-31 11:24:32 +0100 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2014-11-04 15:09:10 +0100 |
commit | a6cece9d81990e729c1f9da2a5bff2d29f7df649 (patch) | |
tree | 46746842e5039a1ce4e01e1e9de58ccb46f234de /sound/usb/endpoint.c | |
parent | ALSA: usb-audio: Flatten probe and disconnect functions (diff) | |
download | linux-a6cece9d81990e729c1f9da2a5bff2d29f7df649.tar.xz linux-a6cece9d81990e729c1f9da2a5bff2d29f7df649.zip |
ALSA: usb-audio: Pass direct struct pointer instead of list_head
Some functions in mixer.c and endpoint.c receive list_head instead of
the object itself. This is not obvious and rather error-prone. Let's
pass the proper object directly instead.
The functions in midi.c still receive list_head and this can't be
changed since the object definition isn't exposed to the outside of
midi.c, so left as is.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/usb/endpoint.c')
-rw-r--r-- | sound/usb/endpoint.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/sound/usb/endpoint.c b/sound/usb/endpoint.c index 114e3e7ff511..167d0c1643e1 100644 --- a/sound/usb/endpoint.c +++ b/sound/usb/endpoint.c @@ -1002,15 +1002,12 @@ void snd_usb_endpoint_release(struct snd_usb_endpoint *ep) /** * snd_usb_endpoint_free: Free the resources of an snd_usb_endpoint * - * @ep: the list header of the endpoint to free + * @ep: the endpoint to free * * This free all resources of the given ep. */ -void snd_usb_endpoint_free(struct list_head *head) +void snd_usb_endpoint_free(struct snd_usb_endpoint *ep) { - struct snd_usb_endpoint *ep; - - ep = list_entry(head, struct snd_usb_endpoint, list); kfree(ep); } |