On Sun, Dec 17, 2023 at 11:13:30AM +0100, Lukas Wunner wrote: > TPM dt-bindings cleanup, take three. > > The existing devicetrees in arch/arm and arch/arm64 contain a few > violations of the schemas contained herein. E.g. the nodename is > not "tpm@..." but "tpmdev@..." in some cases. I've got patches > lined up to fix these issues and will submit them separately. > > Changes v2 -> v3: > * [PATCH 1/4]: > * Drop clock-frequency property from tcg,tpm-tis-i2c.yaml. > It doesn't exist at the peripheral level, only at the > controller level. Document maximum clock frequency for > infineon,slb9635tt and infineon,slb9645tt in textual form > (as was previously done in trivial-devices.yaml). > * Document reset-gpios property as used by: > arch/arm64/boot/dts/freescale/imx8mp-beacon-kit.dts > > * [PATCH 2/4]: > * Introduce new schema specifically for google,cr50 (Rob). > Note that I can't use "oneOf" for SPI and I²C properties > because a node with only a "reg" property is a valid SPI > and I²C node and would match both subschemas. > So "anyOf" it is. > > * [PATCH 4/4]: > * Use "const" instead of "enum" for singleton compatible string. > > Link to v2: > https://lore.kernel.org/linux-devicetree/cover.1701093036.git.lukas@xxxxxxxxx/ > > > Lukas Wunner (4): > dt-bindings: tpm: Consolidate TCG TIS bindings > dt-bindings: tpm: Convert Google Cr50 bindings to DT schema > dt-bindings: tpm: Convert IBM vTPM bindings to DT schema > dt-bindings: tpm: Document Microsoft fTPM bindings Reviewed-by: Rob Herring <robh@xxxxxxxxxx> Great to have this done finally! Thanks for picking it up. If the TPM maintainers don't pick it up for 6.8, I will. Rob