"Axel Dörfler" <axeld@xxxxxxxxxxxxxxxx> wrote: > > +++ src/system/kernel/arch/x86/timers/hpet.h (revision 0) > > +#endif > Please add the closing comment like: > +#endif // _KERNEL_ARCH_x86_TIMERS_HPET_H > > (this is not the only place you left that out) And since this is a plain C header, you would actually use: +#endif /* _KERNEL_ARCH_x86_TIMERS_HPET_H */ :-) Bye, Axel.