Subject: Re: checkpatch nits ...

From: David Howells <[email-masked]>
Date: Sun, 24 Aug 2008 13:23:05 +0100

Kevin Diggs <[email-masked]> wrote:

> The entire block is:
> 
> 	__asm__ __volatile__ (
> 		"addi %0,%3,-1\n"
> 		"andc %1,%3,%0\n"
> 		"cntlzw %1,%1\n"
> 		"subfic %1,%1,31\n"
> 		"cntlzw %0,%2\n":
> 		"=r"(cntlz), "=r"(cnttz):
> 		"r"(tmp), "b"(cnttz)
> 	);

As long as this has no side effects, the __volatile__ isn't necessary.  If the
only effect is to produce the specified outputs based on the specified inputs,
the code can be moved or eliminated if the outputs aren't used.

David