Re: [PATCH] dt-bindings: tpm: Convert cr50 binding to YAML

From: Stephen Boyd
Date: Fri Jan 31 2020 - 20:35:34 EST


Quoting Stephen Boyd (2020-01-09 08:08:11)
> Quoting Doug Anderson (2019-12-17 09:45:02)
> > On Mon, Dec 16, 2019 at 4:54 PM Stephen Boyd <swboyd@xxxxxxxxxxxx> wrote:
> >
> > > + spi-max-frequency:
> > > + maxItems: 1
> >
> > This is not an array type. Why do you need maxItems? Should treat
> > like an int? Do we have any ranges of sane values we can put here?
> > I'm sure that there is a maximum that Cr50 can talk at.
>
> From what I see looking through downstream sources my best guess for a
> max frequency is 1 MHz.
>

I'm leaning towards dropping this property. Is there any benefit? The
driver should know the max anyway.