Hi, > -----Original Message----- > From: Richard Cochran <richardcochran@xxxxxxxxx> > Sent: Friday, January 18, 2019 9:39 AM > To: Y.b. Lu <yangbo.lu@xxxxxxx> > Cc: netdev@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; > linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; David S . Miller <davem@xxxxxxxxxxxxx>; > Rob Herring <robh+dt@xxxxxxxxxx>; Shawn Guo <shawnguo@xxxxxxxxxx> > Subject: Re: [v2, 3/5] dt-binding: ptp_qoriq: document "fsl,extts-fifo" property > > On Thu, Jan 17, 2019 at 05:34:34PM -0800, Richard Cochran wrote: > > > @@ -17,6 +17,7 @@ Clock Properties: > > > - fsl,tmr-fiper1 Fixed interval period pulse generator. > > > - fsl,tmr-fiper2 Fixed interval period pulse generator. > > > - fsl,max-adj Maximum frequency adjustment in parts per billion. > > > + - fsl,extts-fifo Required if hardware uses external trigger stamp FIFO. > > > > Does this take a Boolean value? Please explain the semantics. > > How about this instead? > > The presence of this property indicates hardware support for > the external trigger stamp FIFO. [Y.b. Lu] Yes. This is a Boolean value. Thanks for good suggestion and I will use this instead. > > > Thanks, > Richard