On 15 May 2014 12:14, Arnd Bergmann [off-list ref] wrote:

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

quoted

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.

Only in header files, no?

I've been chastised previously for putting this in .c files.

-Christoffer