linux-kernel - Re: [PATCH net-next 01/12] dt-bindings: ptp: add bindings for NETC Timer

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <836c9f0b-2b73-4b36-8105-db1ae59b799c@kernel.org>
Date: Mon, 14 Jul 2025 12:09:13 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Wei Fang <wei.fang@....com>
Cc: "F.S. Peng" <fushi.peng@....com>,
 "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
 "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
 "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
 "imx@...ts.linux.dev" <imx@...ts.linux.dev>,
 "robh@...nel.org" <robh@...nel.org>, "krzk+dt@...nel.org"
 <krzk+dt@...nel.org>, "conor+dt@...nel.org" <conor+dt@...nel.org>,
 "richardcochran@...il.com" <richardcochran@...il.com>,
 Claudiu Manoil <claudiu.manoil@....com>,
 Vladimir Oltean <vladimir.oltean@....com>, Clark Wang
 <xiaoning.wang@....com>, "andrew+netdev@...n.ch" <andrew+netdev@...n.ch>,
 "davem@...emloft.net" <davem@...emloft.net>,
 "edumazet@...gle.com" <edumazet@...gle.com>,
 "kuba@...nel.org" <kuba@...nel.org>, "pabeni@...hat.com" <pabeni@...hat.com>
Subject: Re: [PATCH net-next 01/12] dt-bindings: ptp: add bindings for NETC
 Timer

On 14/07/2025 11:56, Wei Fang wrote:
> 
>>
>> How does the other consumer - ethernet - reference this one here? Paste
>> complete DTS of this and users, otherwise it is just ping-pong
>> discussion where you put just a little effort to bounce back my question.
> 
> Below is the DTS node of enetc (ethernet device) and timer node.
> 
> enetc_port0: ethernet@0,0 {
> 	compatible = "pci1131,e101";
> 	reg = <0x000000 0 0 0 0>;
> 	pinctrl-names = "default";
> 	pinctrl-0 = <&pinctrl_enetc0>;
> 	phy-handle = <&ethphy0>;
> 	phy-mode = "rgmii-id";
> 	status = "okay";

How do you use netc_timer in such case?

> };
> 
> netc_timer: ethernet@18,0 {
> 	compatible = "pci1131,ee02";
> 	reg = <0x00c000 0 0 0 0>;
> 	clocks = <&netc_system333m>;
> 	clock-names = "system";
> };
> 
> Currently, the enetc driver uses the PCIe device number and function number
> of the Timer to obtain the Timer device, so there is no related binding in DTS.

So you just tightly coupled these devices. Looks poor design for me, but
your choice. Anyway, then use that channel as information to pass the
pin/timer/channel number. You do not get a new property for that.

> In the future, we plan to add phandle to the enetc document to bind enetc
> and Timer, because there will be multiple Timer instances on subsequent
> platforms.

Bindings must be complete, not "in the future" but now. Start sending
complete work, so we won't have to guess it.

> 
> But what I want to say is that "nxp,pps-channel" is used to specify which

There is no user in your DTS of nxp,pps-channel.

Best regards,
Krzysztof