On Wed, Oct 26, 2016 at 12:25:51PM +0100, Mark Rutland wrote:

> > +Required properties:
> > +
> > +- compatible: should specify the actual hardware chip followed by the
> > +  generic interface name "tcg,tpm_tis-spi";
> 
> Please use '-' in comaptible strings rather than '_'.

Erk, there is also a typo spi vs mmio.

Note that the SPI patches were already Ack'd by DT so there is now
precedent:

Documentation/devicetree/bindings/security/tpm/tpm_tis_spi.txt:    
"tcg,tpm_tis-spi"

Do you still want to change? Should we change spi too?

> Please also describe chip strings, even if those aren't use by the
> driver, e.g.

Sure, I can give one or two, but the list will not be exhaustive,
there are many different TPM chips that adhere to this standard.

Thanks,
Jason

------------------------------------------------------------------------------
The Command Line: Reinvented for Modern Developers
Did the resurgence of CLI tooling catch you by surprise?
Reconnect with the command line and become more productive. 
Learn the new .NET and ASP.NET CLI. Get your free copy!
http://sdm.link/telerik
_______________________________________________
tpmdd-devel mailing list
tpmdd-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tpmdd-devel

Reply via email to