diff options
| author |   Dan Carpenter <dan.carpenter@oracle.com> | 2018-05-31 09:25:07 +0300 |
|---|---|---|
| committer |   Takashi Iwai <tiwai@suse.de> | 2018-05-31 09:11:03 +0200 |
| commit | 0d5bcfc9974ad8ce0fc77a95a53ab704faa0dfb7 (patch) | |
| tree | a725d490e20d6de26780076066a416286e6d25f3 /sound | |
| parent | ALSA: xen-front: freeing an error pointer (diff) | |
| download | linux-dev-0d5bcfc9974ad8ce0fc77a95a53ab704faa0dfb7.tar.xz linux-dev-0d5bcfc9974ad8ce0fc77a95a53ab704faa0dfb7.zip | |
ALSA: xen-front: fix a loop timeout
We want the loop to exit when "to" is set to zero, but in the current
code it's set to -1. Also I tweaked the indenting so it doesn't look
like we're passing "--to" to xenbus_read_unsigned().
Fixes: cc3196ae197c ("ALSA: xen-front: Introduce Xen para-virtualized sound frontend driver")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Oleksandr Andrushchenko <oleksandr_andrushchenko@epam.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound')
| -rw-r--r-- | sound/xen/xen_snd_front.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/xen/xen_snd_front.c b/sound/xen/xen_snd_front.c index c18973a9bc9b..b089b13b5160 100644 --- a/sound/xen/xen_snd_front.c +++ b/sound/xen/xen_snd_front.c @@ -334,7 +334,7 @@ static int xen_drv_remove(struct xenbus_device *dev) */ while ((xenbus_read_unsigned(front_info->xb_dev->otherend, "state", XenbusStateUnknown) != XenbusStateInitWait) && - to--) + --to) msleep(10); if (!to) { |