[PATCH net-next] dt-bindings: net: aspeed: add AST2700 MDIO compatible

[PATCH net-next] dt-bindings: net: aspeed: add AST2700 MDIO compatible

[Jacky Chou]

Add "aspeed,ast2700-mdio" compatible to the binding schema with a fallback
to "aspeed,ast2600-mdio".

Although the MDIO controller on AST2700 is functionally the same as the
one on AST2600, it's good practice to add a SoC-specific compatible for
new silicon. This allows future driver updates to handle any 2700-specific
integration issues without requiring devicetree changes or complex
runtime detection logic.

For now, the driver continues to bind via the existing
"aspeed,ast2600-mdio" compatible, so no driver changes are needed.

Signed-off-by: Jacky Chou <jacky_chou@aspeedtech.com>
---
 Documentation/devicetree/bindings/net/aspeed,ast2600-mdio.yaml | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/net/aspeed,ast2600-mdio.yaml b/Documentation/devicetree/bindings/net/aspeed,ast2600-mdio.yaml
index d6ef468495c5..1c90e7c15a44 100644
--- a/Documentation/devicetree/bindings/net/aspeed,ast2600-mdio.yaml
+++ b/Documentation/devicetree/bindings/net/aspeed,ast2600-mdio.yaml
@@ -13,13 +13,19 @@ description: |+
   The ASPEED AST2600 MDIO controller is the third iteration of ASPEED's MDIO
   bus register interface, this time also separating out the controller from the
   MAC.
+  The ASPEED AST2700 MDIO controller is similar to the AST2600's.
 
 allOf:
   - $ref: mdio.yaml#
 
 properties:
   compatible:
-    const: aspeed,ast2600-mdio
+    oneOf:
+      - const: aspeed,ast2600-mdio
+      - items:
+          - enum:
+              - aspeed,ast2700-mdio
+          - const: aspeed,ast2600-mdio
 
   reg:
     maxItems: 1

---
base-commit: c9dfb92de0738eb7fe6a591ad1642333793e8b6e
change-id: 20251117-aspeed_mdio_ast2700-aa089c4f0474

Best regards,
-- 
Jacky Chou <jacky_chou@aspeedtech.com>

Re: [PATCH net-next] dt-bindings: net: aspeed: add AST2700 MDIO compatible

[Conor Dooley]

[-- Attachment #1: Type: text/plain, Size: 2311 bytes --]

On Mon, Nov 17, 2025 at 03:30:18PM +0800, Jacky Chou wrote:
> Add "aspeed,ast2700-mdio" compatible to the binding schema with a fallback
> to "aspeed,ast2600-mdio".
> 
> Although the MDIO controller on AST2700 is functionally the same as the
> one on AST2600, it's good practice to add a SoC-specific compatible for
> new silicon. This allows future driver updates to handle any 2700-specific
> integration issues without requiring devicetree changes or complex
> runtime detection logic.
> 
> For now, the driver continues to bind via the existing
> "aspeed,ast2600-mdio" compatible, so no driver changes are needed.
> 
> Signed-off-by: Jacky Chou <jacky_chou@aspeedtech.com>
> ---
>  Documentation/devicetree/bindings/net/aspeed,ast2600-mdio.yaml | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/net/aspeed,ast2600-mdio.yaml b/Documentation/devicetree/bindings/net/aspeed,ast2600-mdio.yaml
> index d6ef468495c5..1c90e7c15a44 100644
> --- a/Documentation/devicetree/bindings/net/aspeed,ast2600-mdio.yaml
> +++ b/Documentation/devicetree/bindings/net/aspeed,ast2600-mdio.yaml
> @@ -13,13 +13,19 @@ description: |+
>    The ASPEED AST2600 MDIO controller is the third iteration of ASPEED's MDIO
>    bus register interface, this time also separating out the controller from the
>    MAC.
> +  The ASPEED AST2700 MDIO controller is similar to the AST2600's.

This statement disagrees with your commit message that claims
functionally identical, and implies that the 2700 supports some extra
features or whatever.
I think I'd drop this entirely from the patch, rather than try to reword
it. Remove it and then:
Acked-by: Conor Dooley <conor.dooley@microchip.com>
pw-bot: changes-requested

>  allOf:
>    - $ref: mdio.yaml#
>  
>  properties:
>    compatible:
> -    const: aspeed,ast2600-mdio
> +    oneOf:
> +      - const: aspeed,ast2600-mdio
> +      - items:
> +          - enum:
> +              - aspeed,ast2700-mdio
> +          - const: aspeed,ast2600-mdio
>  
>    reg:
>      maxItems: 1
> 
> ---
> base-commit: c9dfb92de0738eb7fe6a591ad1642333793e8b6e
> change-id: 20251117-aspeed_mdio_ast2700-aa089c4f0474
> 
> Best regards,
> -- 
> Jacky Chou <jacky_chou@aspeedtech.com>
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

[PATCH net-next] dt-bindings: net: aspeed: add AST2700 MDIO compatible

[Jacky Chou]

Hi Conor Dooley,

Thank your for your reply.

> >    The ASPEED AST2600 MDIO controller is the third iteration of ASPEED's
> MDIO
> >    bus register interface, this time also separating out the controller from
> the
> >    MAC.
> > +  The ASPEED AST2700 MDIO controller is similar to the AST2600's.
> 
> This statement disagrees with your commit message that claims functionally
> identical, and implies that the 2700 supports some extra features or whatever.
> I think I'd drop this entirely from the patch, rather than try to reword it.
> Remove it and then:
> Acked-by: Conor Dooley <conor.dooley@microchip.com>
> pw-bot: changes-requested
> 

Agreed.
I will remove this description in next version.

Thanks,
Jacky