Hi,

> -----Original Message-----
> From: Richard Cochran <richardcoch...@gmail.com>
> Sent: Friday, January 18, 2019 9:39 AM
> To: Y.b. Lu <yangbo...@nxp.com>
> Cc: netdev@vger.kernel.org; devicet...@vger.kernel.org;
> linux-arm-ker...@lists.infradead.org; David S . Miller <da...@davemloft.net>;
> Rob Herring <robh...@kernel.org>; Shawn Guo <shawn...@kernel.org>
> 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

Reply via email to