Subject: Re: [PATCH v1 1/2] dt-binding: ptp: add bindings document for dte
 based ptp clock
To: Arun Parameswaran <arun.parameswaran@broadcom.com>,
        Richard Cochran <richardcochran@gmail.com>
Cc: Rob Herring <robh+dt@kernel.org>, Mark Rutland <mark.rutland@arm.com>,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        netdev@vger.kernel.org, bcm-kernel-feedback-list@broadcom.com
References: <1497299161-6458-1-git-send-email-arun.parameswaran@broadcom.com>
 <1497299161-6458-2-git-send-email-arun.parameswaran@broadcom.com>
 <20170613050947.GA1684@localhost.localdomain>
 <cb008f0f-1585-653e-0b89-47478f9f4734@broadcom.com>
From: Ray Jui <ray.jui@broadcom.com>
Message-ID: <a4586d75-2aca-94f3-ab7a-fc94c9dc5a4a@broadcom.com>
Date: Wed, 14 Jun 2017 11:18:54 -0700
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:54.0) Gecko/20100101
 Thunderbird/54.0
MIME-Version: 1.0
In-Reply-To: <cb008f0f-1585-653e-0b89-47478f9f4734@broadcom.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1024
Lines: 32


On 6/13/17 10:46 AM, Arun Parameswaran wrote:
> 
> Hi Richard,
> 
> On 17-06-12 10:09 PM, Richard Cochran wrote:
>> On Mon, Jun 12, 2017 at 01:26:00PM -0700, Arun Parameswaran wrote:
>>> +Example:
>>> +
>>> +ptp_dte: ptp_dte@180af650 {
>>> +	compatible = "brcm,ptp-dte";
>>> +	reg = <0x180af650 0x10>;
>>> +	status = "okay";
>>> +};
>> This patch set looks okay, as far as it goes, but how does one
>> actually use the clock?
>>
>> How does one connect this node to an Ethernet MAC, for example?
> In some of the SoC's, the DTE's NCO is synced in hardware with the Ethernet's
> 1588 Shim NCO (used for time-stamping the packets). We are working on getting
> all the future SoC's synced this way (for Ethernet).
> 

So apparently you do not need any phandle based DT binding defined for
the connection between the Ethernet device node and the PTP device node?

> In other scenarios, we use an user-space app to perform the sync based on
> time-stamps from external hardware sources like the Wi-Fi.
>> Thanks,
>> Richard
>