> > +static inline void *ERR_CAST(const void *ptr) > > +{ > > + return (void *) ptr; > > +} > > Just to nit, surely you don't need the cast inside the function. The > casting happens at the call site between the argument and returned pointer. The way it's written you kinda do, since it takes a const void * and returns a plain void *. But I don't think that's the best way to write it. - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html