Re: [PATCH] Video : Amba: Use in_interrupt() in clcdfb_sleep().

From
Russell King - ARM Linux <hidden>
Date
2012-03-11 13:28:00
Also in
kernel-janitors, lkml
Reply to
santosh nayak, 2012-03-11
On Sun, Mar 11, 2012 at 06:44:25PM +0530, santosh nayak wrote:
From: Santosh Nayak <redacted>

Instead of "in_atomic()", we can use in_interrupt() to check whether
its an interrupt context.
What are you trying to fix?

Your description is an example of a bad commit comment.  It merely
describes the change, which anyone can see by looking at the diff.
What it totally and utterly fails to do is to describe _why_ the
change is necessary or what the problem is.

So, until it does, this patch gets a definite NAK.
Thread (14 messages) 14 messages, 4 authors, last reply 2012-03-11
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help