On Tue, Jan 08, 2019 at 05:48:43PM -0800, Matthew Garrett wrote: > +static inline int _calc_tpm2_event_size(struct tcg_pcr_event2 *event, > + struct tcg_pcr_event *event_header) I'm more used to having two underscores i.e. __calc_tpm2_event_size() for functions that are wrapped. Otherwise, looks good to me. /Jarkko