Hongbo Li <lihongbo22@xxxxxxxxxx> wrote: > I see reason is u8, so may I use min_t(u8, sp->ack.reason, > RXRPC_ACK__INVALID)? No, please don't use min_t(<unsigned type>, ...) if umin() will do. IIRC, some arches can't do byte-level arithmetic. Thanks, David