Re: [PATCH 08/12] dt-bindings: net: snps,dwmac: Add compatible for UltraRISC DP1000 SoC
From: Jia Wang <wangjia@ultrarisc.com>
Date: 2026-08-27 08:17:51
Also in:
dmaengine, linux-devicetree, linux-i2c, linux-riscv, linux-spi, lkml
On 2026-08-25 18:12 +0800, Jia Wang wrote:
On 2026-08-24 18:13 +0100, Conor Dooley wrote:quoted
On Mon, Aug 24, 2026 at 06:09:48PM +0100, Conor Dooley wrote:quoted
On Mon, Aug 24, 2026 at 01:54:21PM +0800, Jia Wang via B4 Relay wrote:quoted
From: Jia Wang <wangjia@ultrarisc.com> Add the SoC-specific compatible string for the UltraRISC DP1000 GMAC controller to the DesignWare MAC binding. Signed-off-by: Jia Wang <wangjia@ultrarisc.com>The dts may not be right, but I am also a bit concerned here. This binding permits 100s of combinations of properties, of which only a very limited set can be correct for your device. Just adding the compatible permits all of this variation. Please add some restrictions.In fact, it's particularly bad here because you don't even restrict which fallback compatibles are used, so there's no guidance on what version of the IP is even in use!I have rechecked the DTS against the hardware. The controller uses a DesignWare GMAC IP from the 5.20 series, with "snps,dwmac-5.20" as its version-specific fallback. In the next version, I will restrict the fallback compatibles and add DP1000-specific constraints for its hardware resources.
Following the discussion in patch 11, I will drop this patch from v2. I will first submit the dedicated DP1000 GMAC binding and DWMAC glue driver as a separate series, and then send v2 of this DTS series with that dependency recorded.
quoted
quoted
pw-bot: changes-requested Thanks, Conor.Best regards, Jia Wang
Best regards, Jia Wang
quoted
quoted
quoted
--- Documentation/devicetree/bindings/net/snps,dwmac.yaml | 1 + 1 file changed, 1 insertion(+)diff --git a/Documentation/devicetree/bindings/net/snps,dwmac.yaml b/Documentation/devicetree/bindings/net/snps,dwmac.yaml index 2449311c6d28..2ed4564ae99d 100644 --- a/Documentation/devicetree/bindings/net/snps,dwmac.yaml +++ b/Documentation/devicetree/bindings/net/snps,dwmac.yaml@@ -119,6 +119,7 @@ properties: - starfive,jh7110-dwmac - tesla,fsd-ethqos - thead,th1520-gmac + - ultrarisc,dp1000-gmac reg: minItems: 1-- 2.34.1 _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv