On Thursday 15 May 2014 10:34:35 Marc Zyngier wrote:

quoted

+static inline bool gic_is_standard(struct gic_chip_data *gic)

Please loose all of the inlines. The compiler can do this by itself.

quoted

+{
+       return (gic->nr_cpu_if == 8);
+}
+

I think it's actually the common style to spell out the 'inline'
for trivial helpers like this. The compiler doesn't need it, but
it also doesn't hurt and most people write it.

	Arnd