[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