2023-06-17 13:52:36

by Josua Mayer

Subject: [PATCH v2 2/4] dt-bindings: net: dpaa2 mac: add phys property

dpaa2 mac supports runtime configuration of network protocols on a
generic phy object specified via the "phys" property.
It is currently used with the SerDes28G driver to switch SerDes
protocol between SGMII and USXGMII at runtime.

Support was introduced with:
f978fe8 "dpaa2-mac: configure the SerDes phy on a protocol change"

Add bindings description for the missing "phys" property, to fix dtc_chk
errors present in various layerscape based dts:

'phys' does not match any of the regexes: 'pinctrl-[0-9]+'

Signed-off-by: Josua Mayer <[email protected]>
---
V1 -> V2: this patch is newly added to patch-set

Documentation/devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml | 4 ++++
1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml b/Documentation/devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml
index a1b71b35319e..a40c7a84af2a 100644
--- a/Documentation/devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml
+++ b/Documentation/devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml
@@ -24,6 +24,10 @@ properties:
maxItems: 1
description: The DPMAC number

+ phys:
+ maxItems: 1
+ description: Generic PHY for SerDes runtime configuration.
+
phy-handle: true

phy-connection-type: true
--
2.35.3

2023-06-17 20:38:08

by Krzysztof Kozlowski

Subject: Re: [PATCH v2 2/4] dt-bindings: net: dpaa2 mac: add phys property

On 17/06/2023 15:40, Josua Mayer wrote:
> dpaa2 mac supports runtime configuration of network protocols on a
> generic phy object specified via the "phys" property.
> It is currently used with the SerDes28G driver to switch SerDes
> protocol between SGMII and USXGMII at runtime.
>
> Support was introduced with:
> f978fe8 "dpaa2-mac: configure the SerDes phy on a protocol change"
>
> Add bindings description for the missing "phys" property, to fix dtc_chk
> errors present in various layerscape based dts:
>
> 'phys' does not match any of the regexes: 'pinctrl-[0-9]+'
>
> Signed-off-by: Josua Mayer <[email protected]>

This was sent:

https://lore.kernel.org/all/[email protected]/

And obviously will get lost because Sean for some reason decided to not
Cc maintainers and lists... so sure, we can go with this one:

Reviewed-by: Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof