On Tue, Nov 22, 2016 at 2:08 PM, Jason Gunthorpe
<jguntho...@obsidianresearch.com> wrote:
>
> On Tue, Nov 22, 2016 at 09:40:07AM -0300, Fabio Urquiza wrote:
> > If the variable expected_len is greater than 35 bytes, i2c_atmel_recv()
> > ignores the amount of data already read in i2c_atmel_read_status() and
> > request more data than what the device is ready to supply. As result the
> > TPM data sent to the upper layers will miss the first 35 bytes of the
> > response and will be filled with garbage in the end.
>
> I'm concerned your chip is not behaving the same as mine, I have to
> dig out my hardware and check. I'm fairly certain I would have hit
> this..
>
> Do you know the revision code for your chip?
>
> IIRC my TPM re-read the message starting from byte 0, which is why the
> code is like this.
>
> Jason

I'm working on an emulator for that chip and I don't have the real
hardware to check.

That behavior makes sense. I will change the emulator to follow that.

You can ignore that patch, then.

Thanks,

-Fabio Urquiza

------------------------------------------------------------------------------
_______________________________________________
tpmdd-devel mailing list
tpmdd-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tpmdd-devel

Reply via email to