[PATCH 1/4] arm: msm: gpio support

Pavel Machek pavel at ucw.cz
Tue Apr 6 09:32:16 EDT 2010


> > >> >> +     spin_lock_irqsave(&msm_chip->lock, irq_flags);
> > >> >> +
> > >> >> +     b = 1U << (irq - FIRST_GPIO_IRQ - msm_chip->chip.base);
> > >> >
> > >> > Urk... That's a bit confusing...
> > >> >
> > >> > You might want to make this a macro or an inline function with some kind
> > >> > of comment.
> > >> >
> > >>
> > >> On a related note, why did you inline msm_gpio_clear_detect_status? It
> > >> is used from two other functions.
> > >>
> > >
> > > You mean other places in the Android tree?
> > 
> > No, you copied it three times in this file.
> 
> Oh, well it was part of the API elimination , but it's helpful when
> doing clean up to have stuff all unrolled so optimization possibilities
> present themselves more easily .. You think I should convert it into an
> inline?

I guess so. Premature optimalization is sqrt(all evil).

Really, leave that to compiler.
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html



More information about the linux-arm-kernel mailing list