diff mbox series

dt-bindings: net: dpmac: Add phys property

Message ID 20220801181347.3873041-1-sean.anderson@seco.com
State Not Applicable, archived
Headers show
Series dt-bindings: net: dpmac: Add phys property | expand

Checks

Context Check Description
robh/checkpatch warning total: 0 errors, 1 warnings, 9 lines checked
robh/patch-applied fail build log

Commit Message

Sean Anderson Aug. 1, 2022, 6:13 p.m. UTC
This silences warnings like

arch/arm64/boot/dts/freescale/fsl-lx2160a-honeycomb.dtb: ethernet@8: 'phys', 'sfp' do not match any of the regexes: 'pinctrl-[0-9]+'
	From schema: Documentation/devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml

Fixes: f978fe85b8d1 ("dpaa2-mac: configure the SerDes phy on a protocol change")
Signed-off-by: Sean Anderson <sean.anderson@seco.com>
---

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

Comments

Krzysztof Kozlowski Aug. 2, 2022, 7:51 a.m. UTC | #1
On 01/08/2022 20:13, Sean Anderson wrote:
> This silences warnings like
> 
> arch/arm64/boot/dts/freescale/fsl-lx2160a-honeycomb.dtb: ethernet@8: 'phys', 'sfp' do not match any of the regexes: 'pinctrl-[0-9]+'
> 	From schema: Documentation/devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml
> 
> Fixes: f978fe85b8d1 ("dpaa2-mac: configure the SerDes phy on a protocol change")
> Signed-off-by: Sean Anderson <sean.anderson@seco.com>

Don't  we have phy-handle for this?

Best regards,
Krzysztof
Sean Anderson Aug. 2, 2022, 3:04 p.m. UTC | #2
On 8/2/22 3:51 AM, Krzysztof Kozlowski wrote:
> On 01/08/2022 20:13, Sean Anderson wrote:
>> This silences warnings like
>> 
>> arch/arm64/boot/dts/freescale/fsl-lx2160a-honeycomb.dtb: ethernet@8: 'phys', 'sfp' do not match any of the regexes: 'pinctrl-[0-9]+'
>> 	From schema: Documentation/devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml
>> 
>> Fixes: f978fe85b8d1 ("dpaa2-mac: configure the SerDes phy on a protocol change")
>> Signed-off-by: Sean Anderson <sean.anderson@seco.com>
> 
> Don't  we have phy-handle for this?

This is for serdes phys. See dtschema/schemas/phy/phy-consumer.yaml

--Sean
Rob Herring Aug. 3, 2022, 10:48 p.m. UTC | #3
On Mon, 01 Aug 2022 14:13:46 -0400, Sean Anderson wrote:
> This silences warnings like
> 
> arch/arm64/boot/dts/freescale/fsl-lx2160a-honeycomb.dtb: ethernet@8: 'phys', 'sfp' do not match any of the regexes: 'pinctrl-[0-9]+'
> 	From schema: Documentation/devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml
> 
> Fixes: f978fe85b8d1 ("dpaa2-mac: configure the SerDes phy on a protocol change")
> Signed-off-by: Sean Anderson <sean.anderson@seco.com>
> ---
> 
>  Documentation/devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml | 3 +++
>  1 file changed, 3 insertions(+)
> 

Reviewed-by: Rob Herring <robh@kernel.org>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml b/Documentation/devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml
index 600240281e8c..b46c7e174a03 100644
--- a/Documentation/devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml
+++ b/Documentation/devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml
@@ -30,6 +30,9 @@  properties:
 
   phy-mode: true
 
+  phys:
+    maxItems: 1
+
   pcs-handle:
     maxItems: 1
     description: