On 2020-03-12 04:39, Andy Shevchenko wrote: > +static inline u32 get_unaligned_be24(const u8 *buf) > +{ > + return (u32)p[0] << 16 | (u32)p[1] << 8 | (u32)p[2]; > +} The argument is called 'buf' and the function body dereferences a pointer called 'p'. Does this even compile? Bart.