[PATCH v8 1/2] dt-bindings: net: Add FSD EQoS device tree bindings

[Krzysztof Kozlowski]

On 06/03/2025 08:44, Swathi K S wrote:
> 
> 
>> -----Original Message-----
>> From: Krzysztof Kozlowski <krzk at kernel.org>
>> Sent: 06 March 2025 12:45
>> To: Swathi K S <swathi.ks at samsung.com>; krzk+dt at kernel.org;
>> andrew+netdev at lunn.ch; davem at davemloft.net; edumazet at google.com;
>> kuba at kernel.org; pabeni at redhat.com; robh at kernel.org;
>> conor+dt at kernel.org; richardcochran at gmail.com;
>> mcoquelin.stm32 at gmail.com; alexandre.torgue at foss.st.com
>> Cc: rmk+kernel at armlinux.org.uk; netdev at vger.kernel.org;
>> devicetree at vger.kernel.org; linux-stm32 at st-md-mailman.stormreply.com;
>> linux-arm-kernel at lists.infradead.org; linux-kernel at vger.kernel.org;
>> pankaj.dubey at samsung.com; ravi.patel at samsung.com;
>> gost.dev at samsung.com
>> Subject: Re: [PATCH v8 1/2] dt-bindings: net: Add FSD EQoS device tree
>> bindings
>>
>> On 05/03/2025 10:12, Swathi K S wrote:
>>> Add FSD Ethernet compatible in Synopsys dt-bindings document. Add FSD
>>> Ethernet YAML schema to enable the DT validation.
>>>
>>> Signed-off-by: Pankaj Dubey <pankaj.dubey at samsung.com>
>>> Signed-off-by: Ravi Patel <ravi.patel at samsung.com>
>>> Signed-off-by: Swathi K S <swathi.ks at samsung.com>
>>> ---
>>>  .../devicetree/bindings/net/snps,dwmac.yaml   |   5 +-
>>>  .../bindings/net/tesla,fsd-ethqos.yaml        | 118 ++++++++++++++++++
>>>  2 files changed, 121 insertions(+), 2 deletions(-)  create mode
>>> 100644 Documentation/devicetree/bindings/net/tesla,fsd-ethqos.yaml
>>>
>>
>> I tried and did not see any differences, so point me exactly to any difference
>> in the binding (binding!) which would justify dropping review?
> 
> Added the following in the example given in DT binding doc:

OK, but that's an example, not the binding, so no new properties in the
binding.

> 
> assigned-clocks = <&clock_peric PERIC_EQOS_PHYRXCLK_MUX>,
>                                 <&clock_peric PERIC_EQOS_PHYRXCLK>;
> assigned-clock-parents = <&clock_peric PERIC_EQOS_PHYRXCLK>;
> 
> Given the significance of these changes, I assumed the changes need to be reviewed again.

Adding standard properties to the example is not significant change. It
is really a minor change.



Best regards,
Krzysztof