Hi, On Sat, Feb 1, 2020 at 8:39 PM Stephen Boyd <swboyd@xxxxxxxxxxxx> wrote: > > This allows us to validate the dt binding to the implementation. Add the > interrupt property too, because that's required but nobody noticed when > the non-YAML binding was introduced. > > Cc: Andrey Pronin <apronin@xxxxxxxxxxxx> > Cc: Douglas Anderson <dianders@xxxxxxxxxxxx> > Signed-off-by: Stephen Boyd <swboyd@xxxxxxxxxxxx> > --- > > Changes from v1: > * Dropped spi-max-frequency as required > * Capped spi-max-frequency at 1MHz > * Added interrupt-parent to example to be realistic > > .../bindings/security/tpm/google,cr50.txt | 19 ------- > .../bindings/security/tpm/google,cr50.yaml | 50 +++++++++++++++++++ > 2 files changed, 50 insertions(+), 19 deletions(-) This seems sane to me. Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx>