[RFT/PATCH 7/7] arm: omap: smartreflex: micro-optimization for sanity check

Felipe Balbi balbi at ti.com
Tue Oct 11 08:19:23 EDT 2011


On Tue, Oct 11, 2011 at 03:22:33PM +0400, Sergei Shtylyov wrote:
> Hello.
> 
> On 10-10-2011 16:26, Felipe Balbi wrote:
> 
> >val&&  (val != 1) == val>  1
> 
> >Signed-off-by: Felipe Balbi<balbi at ti.com>
> >---
> >  arch/arm/mach-omap2/smartreflex.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> >diff --git a/arch/arm/mach-omap2/smartreflex.c b/arch/arm/mach-omap2/smartreflex.c
> >index 7bdabfa..4b0d6a8 100644
> >--- a/arch/arm/mach-omap2/smartreflex.c
> >+++ b/arch/arm/mach-omap2/smartreflex.c
> >@@ -866,7 +866,7 @@ static int omap_sr_autocomp_store(void *data, u64 val)
> >  	}
> >
> >  	/* Sanity check */
> >-	if (val&&  (val != 1)) {
> >+	if (val>  1) {
> >  		pr_warning("%s: Invalid argument %lld\n", __func__, val);
> 
>    The format specified should be "%llu"?

and what does that have to do with $SUBJECT ??

-- 
balbi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20111011/f99ff706/attachment-0001.sig>


More information about the linux-arm-kernel mailing list