Message-ID: <1ee7386b-f42d-a182-f42d-c1f628fb4dda@linaro.org>
Date:   Tue, 14 Mar 2023 16:45:06 +0100
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.8.0
Subject: Re: [PATCH net-next V3] dt-bindings: net: ethernet-controller: Add
 ptp-hardware-clock
Content-Language: en-US
To:     Jakub Kicinski <kuba@kernel.org>, robh+dt@kernel.org,
        krzysztof.kozlowski+dt@linaro.org
Cc:     Sarath Babu Naidu Gaddam <sarath.babu.naidu.gaddam@amd.com>,
        davem@davemloft.net, edumazet@google.com, pabeni@redhat.com,
        richardcochran@gmail.com, netdev@vger.kernel.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        yangbo.lu@nxp.com, radhey.shyam.pandey@amd.com,
        anirudha.sarangi@amd.com, harini.katakam@amd.com, git@amd.com
References: <20230308054408.1353992-1-sarath.babu.naidu.gaddam@amd.com>
 <20230308054408.1353992-2-sarath.babu.naidu.gaddam@amd.com>
 <20230313153532.2ed45ddf@kernel.org>
From:   Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
In-Reply-To: <20230313153532.2ed45ddf@kernel.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 13/03/2023 23:35, Jakub Kicinski wrote:
> On Wed, 8 Mar 2023 11:14:08 +0530 Sarath Babu Naidu Gaddam wrote:
>> There is currently no standard property to pass PTP device index
>> information to ethernet driver when they are independent.
>>
>> ptp-hardware-clock property will contain phandle to PTP clock node.
>>
>> Its a generic (optional) property name to link to PTP phandle to
>> Ethernet node. Any future or current ethernet drivers that need
>> a reference to the PHC used on their system can simply use this
>> generic property name instead of using custom property
>> implementation in their device tree nodes."
>>
>> Signed-off-by: Sarath Babu Naidu Gaddam <sarath.babu.naidu.gaddam@amd.com>
>> Acked-by: Richard Cochran <richardcochran@gmail.com>
> 
> Rob, Krzysztof, any thoughts on this one?
> Looks like the v2 discussion was a bit inconclusive.

Anyway this did not implement changes I requested, so NAK.

Best regards,
Krzysztof