Hi Jason, Am Donnerstag, 12. September 2013, 01:08:41 schrieb Jason Gunthorpe: > On Wed, Sep 11, 2013 at 02:58:07PM -0600, Jason Gunthorpe wrote: > > I'm not yet 100% certain this driver works perfectly, for some reason > > my TPM will not create an EK, which is either due to a larger sent TPM > > messages becoming corrupt (combined with a TPM bug to not report an > > error code) or something wrong with the TPM itself. > > > > + > > + status = i2c_smbus_write_i2c_block_data(client, buf[0], len - 1, > > + buf + 1); > > Thankfully Peter pointed out that these calls only transfer 32 bytes, > which causes the problem I observed. The TPM returns success with a > corrupted response when you truncate the request, apparently. > > Fixing it with: > > + status = i2c_master_send(client, buf, len); > > Has the driver pass all my tests. I will post a v1 after a few days > and collect any other comments. Yes this should probably do the thing ;) Thanks for porting yet another driver. I think I'll stage them (nuvoton, st and atmel) all in a temporary branch "testing-and-review" so it's easier for people to try them out. This branch might rebase and change all the time, but maybe it makes things easier (and patches can be squashed afterwards easily if needed). But since you'll be sending a v2, I'll wait until the v2. Thanks, Peter -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
