Re: [PATCH net-next] dt-bindings: net: Convert snps,dwc-qos-ethernet to DT schema

[Rob Herring <robh@kernel.org>]

On Tue, Nov 03, 2026 at 08:34:17PM +0000, sashiko-bot@kernel.org wrote:
> Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
> - [Low] dt-bindings: net: snps,dwc-qos-ethernet-4.10: redundant maxItems on clocks
> --
> 
> commit ccf9541bde2e61e4e9ad2f4d7325ff792d7181d2
> Author: Rob Herring (Arm) <robh@kernel.org>
> 
> dt-bindings: net: Convert snps,dwc-qos-ethernet to DT schema
> 
> This commit converts the snps,dwc-qos-ethernet-4.10 binding to DT schema
> format and moves shared properties to snps,dwmac-common.yaml. It also drops
> unused compatibles and updates the example to use Tegra186.
> 
> [Severity: Low]
> In Documentation/devicetree/bindings/net/snps,dwc-qos-ethernet-4.10.yaml at
> line 39 under properties:clocks, maxItems: 5 is specified alongside a
> 5-element items list without additionalItems: true.
> 
> In the devicetree meta-schema (meta-schemas/items.yaml), when items is a
> fixed-order list, maxItems is implicitly set to the length of the items list
> and explicitly specifying maxItems is disallowed.
> 
> Does this cause dt_binding_check / dt-doc-validate to fail on
> properties:clocks:maxItems, and should maxItems: 5 be removed here?

Yes, I left this fix in my tree uncommitted before sending this...

pw-bot: cr