Commit 4c6dc531 authored by Antonino Daplas's avatar Antonino Daplas Committed by Linus Torvalds

[PATCH] pxafb: Reorder add_wait_queue() and set_current_state()

Reorder set_current_state() and add_wait_queue().  There is a chance that a
signal could be missed in between the two functions currently.
Signed-off-by: default avatarNishanth Aravamudan <nacc@us.ibm.com>
Signed-off-by: default avatarDomen Puncer <domen@coderock.org>
Signed-off-by: default avatarAntonino Daplas <adaplas@pol.net>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 9db52e2f
...@@ -740,8 +740,8 @@ static void pxafb_disable_controller(struct pxafb_info *fbi) ...@@ -740,8 +740,8 @@ static void pxafb_disable_controller(struct pxafb_info *fbi)
DPRINTK("Disabling LCD controller\n"); DPRINTK("Disabling LCD controller\n");
add_wait_queue(&fbi->ctrlr_wait, &wait);
set_current_state(TASK_UNINTERRUPTIBLE); set_current_state(TASK_UNINTERRUPTIBLE);
add_wait_queue(&fbi->ctrlr_wait, &wait);
LCSR = 0xffffffff; /* Clear LCD Status Register */ LCSR = 0xffffffff; /* Clear LCD Status Register */
LCCR0 &= ~LCCR0_LDM; /* Enable LCD Disable Done Interrupt */ LCCR0 &= ~LCCR0_LDM; /* Enable LCD Disable Done Interrupt */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment