spi/pl022: fix stop queue procedure
This fix prevents queue being marked as "stopped", if data exists in the queue list. Signed-off-by: Grzegorz Sygieda <grzegorz.sygieda@tieto.com> Signed-off-by: Lukasz Baj <lukasz.baj@tieto.com> Signed-off-by: Linus Walleij <linus.walleij@stericsson.com> Signed-off-by: Grant Likely <grant.likely@secretlab.ca>
This commit is contained in:
parent
781c7b129b
commit
4a12404dda
|
@ -1211,7 +1211,6 @@ static int stop_queue(struct pl022 *pl022)
|
||||||
* A wait_queue on the pl022->busy could be used, but then the common
|
* A wait_queue on the pl022->busy could be used, but then the common
|
||||||
* execution path (pump_messages) would be required to call wake_up or
|
* execution path (pump_messages) would be required to call wake_up or
|
||||||
* friends on every SPI message. Do this instead */
|
* friends on every SPI message. Do this instead */
|
||||||
pl022->run = QUEUE_STOPPED;
|
|
||||||
while (!list_empty(&pl022->queue) && pl022->busy && limit--) {
|
while (!list_empty(&pl022->queue) && pl022->busy && limit--) {
|
||||||
spin_unlock_irqrestore(&pl022->queue_lock, flags);
|
spin_unlock_irqrestore(&pl022->queue_lock, flags);
|
||||||
msleep(10);
|
msleep(10);
|
||||||
|
@ -1220,6 +1219,7 @@ static int stop_queue(struct pl022 *pl022)
|
||||||
|
|
||||||
if (!list_empty(&pl022->queue) || pl022->busy)
|
if (!list_empty(&pl022->queue) || pl022->busy)
|
||||||
status = -EBUSY;
|
status = -EBUSY;
|
||||||
|
else pl022->run = QUEUE_STOPPED;
|
||||||
|
|
||||||
spin_unlock_irqrestore(&pl022->queue_lock, flags);
|
spin_unlock_irqrestore(&pl022->queue_lock, flags);
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue