Any comments would be, as always, appreciated. -Nish Description: Reorder add_wait_queue() and set_current_state() as a signal could be lost in between the two functions. Signed-off-by: Nishanth Aravamudan Signed-off-by: Domen Puncer --- kj-domen/drivers/video/sa1100fb.c | 2 +- 1 files changed, 1 insertion(+), 1 deletion(-) diff -puN drivers/video/sa1100fb.c~reorder-state-drivers_video_sa1100fb drivers/video/sa1100fb.c --- kj/drivers/video/sa1100fb.c~reorder-state-drivers_video_sa1100fb 2004-12-25 01:35:19.000000000 +0100 +++ kj-domen/drivers/video/sa1100fb.c 2004-12-25 01:35:19.000000000 +0100 @@ -1044,8 +1044,8 @@ static void sa1100fb_disable_controller( GPCR |= SHANNON_GPIO_DISP_EN; } - add_wait_queue(&fbi->ctrlr_wait, &wait); set_current_state(TASK_UNINTERRUPTIBLE); + add_wait_queue(&fbi->ctrlr_wait, &wait); LCSR = 0xffffffff; /* Clear LCD Status Register */ LCCR0 &= ~LCCR0_LDM; /* Enable LCD Disable Done Interrupt */ _