On Mon, Nov 05, 2018 at 04:48:13PM -0500, Stefan Berger wrote: > > int tpm_send(struct tpm_chip *chip, void *cmd, size_t buflen) > > { > > + struct tpm_buf buf; > > int rc; > > > > chip = tpm_find_get_ops(chip); > > if (!chip) > > return -ENODEV; > > > > - rc = tpm_transmit_cmd(chip, NULL, cmd, buflen, 0, 0, > > + rc = tpm_buf_init(&buf, 0, 0); > > + if (rc) > > + goto out; > > + > > + memcpy(buf.data, cmd, buflen); > > > Nit: buflen -> cmd_len Agreed that it should but I try to keep the patch as minimal and mechanical as I can. Thank you. /Jarkko