> +static inline struct cxl_mem_command *cxl_mem_find_command(u16 opcode) > +{ > + struct cxl_mem_command *c; > + > + cxl_for_each_cmd(c) Would you be amenable to adding { > + if (c->opcode == opcode) > + return c; > + and } here (and in the code below as well where cxl_for_each_cmd is used?) Regardless of that: Reviewed-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>