[0/7] Clean up TPM compatibles

Message

Alexander Steffen Sept. 30, 2022, 4:51 p.m. UTC

Not all TPM compatibles were documented correctly so far. Some were not
documented at all, others missed some properties, most of them did not use the
latest YAML format. But by design most TPMs are very similar, so this allows to
document them in a single trivial-tpms.yaml, similar to how
trivial-devices.yaml documents simple I2C and SPI devices. This fixes all the
documentation issues and ensures that dtbs_check can be used in the future to
detect incorrect configurations automatically.

Alexander Steffen (7):
  dt-bindings: tpm: Introduce trivial-tpms.yaml
  dt-bindings: tpm: Improve documentation of compatibles
  dt-bindings: tpm: Add previously undocumented properties
  dt-bindings: tpm: Merge existing trivial TPMs to YAML
  dt-bindings: tpm: Document previously undocumented compatibles
  dt-bindings: tpm: Add fallbacks for all TCG-compliant TPM variants
  tpm: Add missing compatibles to drivers

 .../bindings/security/tpm/google,cr50.txt     |  19 ---
 .../bindings/security/tpm/tpm-i2c.txt         |  26 ----
 .../bindings/security/tpm/tpm_tis_mmio.txt    |  25 ----
 .../bindings/security/tpm/tpm_tis_spi.txt     |  23 ----
 .../bindings/security/tpm/trivial-tpms.yaml   | 118 ++++++++++++++++++
 .../devicetree/bindings/trivial-devices.yaml  |  16 ---
 drivers/char/tpm/tpm_tis_i2c.c                |   2 +
 drivers/char/tpm/tpm_tis_spi_main.c           |   1 +
 8 files changed, 121 insertions(+), 109 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/security/tpm/google,cr50.txt
 delete mode 100644 Documentation/devicetree/bindings/security/tpm/tpm-i2c.txt
 delete mode 100644 Documentation/devicetree/bindings/security/tpm/tpm_tis_mmio.txt
 delete mode 100644 Documentation/devicetree/bindings/security/tpm/tpm_tis_spi.txt
 create mode 100644 Documentation/devicetree/bindings/security/tpm/trivial-tpms.yaml

Comments

Rob Herring Sept. 30, 2022, 10:09 p.m. UTC | #1

+Joel

On Fri, Sep 30, 2022 at 12:03 PM Alexander Steffen
<[e-mail]> wrote:
>
> Not all TPM compatibles were documented correctly so far. Some were not
> documented at all, others missed some properties, most of them did not use the
> latest YAML format. But by design most TPMs are very similar, so this allows to
> document them in a single trivial-tpms.yaml, similar to how
> trivial-devices.yaml documents simple I2C and SPI devices. This fixes all the
> documentation issues and ensures that dtbs_check can be used in the future to
> detect incorrect configurations automatically.
>
> Alexander Steffen (7):
>   dt-bindings: tpm: Introduce trivial-tpms.yaml
>   dt-bindings: tpm: Improve documentation of compatibles
>   dt-bindings: tpm: Add previously undocumented properties
>   dt-bindings: tpm: Merge existing trivial TPMs to YAML
>   dt-bindings: tpm: Document previously undocumented compatibles
>   dt-bindings: tpm: Add fallbacks for all TCG-compliant TPM variants
>   tpm: Add missing compatibles to drivers
>
>  .../bindings/security/tpm/google,cr50.txt     |  19 ---
>  .../bindings/security/tpm/tpm-i2c.txt         |  26 ----
>  .../bindings/security/tpm/tpm_tis_mmio.txt    |  25 ----
>  .../bindings/security/tpm/tpm_tis_spi.txt     |  23 ----
>  .../bindings/security/tpm/trivial-tpms.yaml   | 118 ++++++++++++++++++
>  .../devicetree/bindings/trivial-devices.yaml  |  16 ---
>  drivers/char/tpm/tpm_tis_i2c.c                |   2 +
>  drivers/char/tpm/tpm_tis_spi_main.c           |   1 +
>  8 files changed, 121 insertions(+), 109 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/security/tpm/google,cr50.txt
>  delete mode 100644 Documentation/devicetree/bindings/security/tpm/tpm-i2c.txt
>  delete mode 100644 Documentation/devicetree/bindings/security/tpm/tpm_tis_mmio.txt
>  delete mode 100644 Documentation/devicetree/bindings/security/tpm/tpm_tis_spi.txt
>  create mode 100644 Documentation/devicetree/bindings/security/tpm/trivial-tpms.yaml

Thanks, but please coordinate your work with this[1].

Rob

[1] https://lore.kernel.org/all/[e-mail]/

>