On Mon, Oct 05, 2020 at 08:35:58AM -0300, Fabio Estevam wrote: > On Mon, Oct 5, 2020 at 8:16 AM Primoz Fiser <primoz.fiser@xxxxxxxxx> wrote: > > > > Add devicetree bindings documentation for operating SSI in AC'97 > > variable/fixed mode of operation. > > > > Signed-off-by: Primoz Fiser <primoz.fiser@xxxxxxxxx> > > --- > > Documentation/devicetree/bindings/sound/fsl,ssi.txt | 5 +++++ > > 1 file changed, 5 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/sound/fsl,ssi.txt b/Documentation/devicetree/bindings/sound/fsl,ssi.txt > > index 7e15a85cecd2..abc5abe11fb9 100644 > > --- a/Documentation/devicetree/bindings/sound/fsl,ssi.txt > > +++ b/Documentation/devicetree/bindings/sound/fsl,ssi.txt > > @@ -43,6 +43,11 @@ Optional properties: > > - fsl,mode: The operating mode for the AC97 interface only. > > "ac97-slave" - AC97 mode, SSI is clock slave > > "ac97-master" - AC97 mode, SSI is clock master > > +- fsl,ac97-mode: SSI AC97 mode of operation. > > + "variable" - AC97 Variable Mode, SLOTREQ bits determine > > + next receive/transmit frame > > + "fixed" - AC97 Fixed Mode, SSI transmits in accordance with > > + AC97 Frame Rate Divider bits > > It would be good to mention what is the default mode when such > property is absent. Then perhaps it could be boolean? Rob