From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f52.google.com (mail-wm1-f52.google.com [209.85.128.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3DDE947CC6D for ; Wed, 17 Jun 2026 16:43:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.52 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781714628; cv=none; b=h90c2wc6WT/iBGDMG5cdtVthBSwQnILzKzh1un5Vf1FSJf6MZUFK2GgcsrcjquoXZQx+rRpO6Mf9S1J3mLvZEPtCWEUzp3l5M1kIEa5DjOqdM/F8VviHiYjfthcH7p9V3BZjA4h7figBgWj55Hyl1AatW5wkJvyDFEnvuQUqT+M= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781714628; c=relaxed/simple; bh=D4YdmYqvXb/Xngj2moqIU1HFb/wEpXGadyA0iAx2G9Y=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=l/ga2Bl+g7QiUxn0WtdRvNDu3+HEWd54bGDiojL44BllSbI+bzp9V/oIbcPLB4UnUb9S1N1BrRTOUC1JjoQMpWxCyZM5W2QuH7b8Sk0RB9m1qS6ZWl9HBbeMQCn+UWJW+GaPdqZ3gnHCM/pvy48N1Onqj1eUKPr+SWDarTzCX70= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=j8TUZfJX; arc=none smtp.client-ip=209.85.128.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="j8TUZfJX" Received: by mail-wm1-f52.google.com with SMTP id 5b1f17b1804b1-4908b92904fso74428495e9.0 for ; Wed, 17 Jun 2026 09:43:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1781714625; x=1782319425; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=91ni4OHOJEhYMTKX+2kMkaj6csE1qduQg5o/8P82bfQ=; b=j8TUZfJX+NeyknFnN7fOELfyvNORzV5Sa9Ymz3rOCf2taVXN3rCywu5llezyo5PYk7 wzO5r0zHUKTZyXuYPWi3UAOn1jFPQ+bUZmDxeOmkuIYrLXRv4rSMLSGyuFeBMSbz93Uw Mtz16+ny3kmKfB4+zB85AQDiCABL8N5LnuhVpEQQBJg4kQobpI0VBrO98IdnFBOKOfT5 L1E9XVkp9S86FUuFGFNeZSlq7K/t76XfYTo0EsGGtQk3jJQr7B240gx67TIaoljxVOjn udSolD/6XrkRYdjCDIweVVLKGRKb4tuzAmny0gBcJW6i5/pBUOTdpWfMl8CPx2lSKcUK mKyg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1781714625; x=1782319425; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=91ni4OHOJEhYMTKX+2kMkaj6csE1qduQg5o/8P82bfQ=; b=iFZITCqQBIkJInzRedKO41eWlY8VoeyFpPoYYuW+8tG5w4A/D+FPX733KT9vIyDxoz +fcRhHRAqPamQlfcswj77yFXldujIjEz4co3LHDahhoKVVmqQu/43ko6EvxayV0uEseg u5MPZ78ot2ho68tZEf6txQ/Hy1aEMVnW9Fo31stb4bHssg+jGmF57qbAKqLjqmROIdTS nYvFHEv64I4RxP8KCItyv+6iY60PztPNxW5O+eqMrRhN3U7jC1GJZbRgYxKiCT/IDhYx DRKIpNpS3wDFUTij7Gs8eXqusGZIaBos5KjP1c8w75uh5oxzECF1cvh6V8wQzJe4/UtP Qe3w== X-Forwarded-Encrypted: i=1; AFNElJ+d1jH/qZVV9Qxkjej7WWkLkRpkqX+NEiMFrgQnIZdfmzZbeAlcr5NKYq4Mqb/6f7kw5pJo/PCxnz09@vger.kernel.org X-Gm-Message-State: AOJu0Yz2fVS5ssKWhLk3hdMM6Jms/+wxG4PK6hBHtfvhAhMzOPHUXs/m VAy7opBOpdaBOefPOojBbiW9XwwMaq7gcVGKyCz2Bju0KYuKxInmRZ9c X-Gm-Gg: Acq92OGRjfo8oB2JjicufRQXPGfF4Pu3hWo9a/xRYGniIRx+xjHWpgmxpYvCID7tv5U r7hTq2tEe4dEtStHojAHfkI1Z6m3LOs7cJ9f4IJ43jnME5CfDfkeZndSpUd2CALaxulu+2mNtX1 8Ju/sbFUEbQWNqQVTLVYZu1x7jwP5gYQ/dfFo5wCUlzLcKH1nlhy2FpagSnpKvnvQ/QeKbr4WXm ihgPqHYFZyxnANZuNl/40tmK8xn8apHDXOGmB7RfHH1zH/BSTyC2mIr1t4L6LI3ru81EoCKLON9 6Ag5YuS3dy+ArvLKXAIKDDCTuBkFdLdQSMVrp+ENEyx6nnLVD3y3sZo1Hy6q0j+sRrPNryWTcTX Bv/adZfLcwtGCrG/XEmaqkZs9jOFn6p00NyL2FqquKFRygZT9XexRYp7qaNJZFbJPCyRQvLYqtP tqxn0bkST/eKMYiMrlzRv4oX6Xuck7Z78sh59zF31fj4ApRLqD39smsZZGDVSbJhauvlnPvk69H nXKXs8qcDczFcx3FH2h4neXwP/fOFFlq0X4q83+LH94G7f8lQt/2pA= X-Received: by 2002:a05:600c:a15:b0:491:91cc:d12d with SMTP id 5b1f17b1804b1-49234131f7dmr59507685e9.25.1781714625451; Wed, 17 Jun 2026 09:43:45 -0700 (PDT) Received: from ?IPV6:2003:ea:8f4a:8700:58d1:af07:762f:9d17? (p200300ea8f4a870058d1af07762f9d17.dip0.t-ipconnect.de. [2003:ea:8f4a:8700:58d1:af07:762f:9d17]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-49230a8ec56sm156440695e9.9.2026.06.17.09.43.44 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 17 Jun 2026 09:43:44 -0700 (PDT) Message-ID: <876a38f8-75ea-4b32-bb65-216cb3adb436@gmail.com> Date: Wed, 17 Jun 2026 18:43:42 +0200 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v4 1/3] dt-bindings: net: add Realtek RTL8125 PCIe Ethernet To: ricardo@pardini.net, nic_swsd@realtek.com, Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner Cc: Sebastian Reichel , netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org References: <20260617-rk3588-dts-rtl-eth-describe-dt-alias-v4-0-2bd38922d129@pardini.net> <20260617-rk3588-dts-rtl-eth-describe-dt-alias-v4-1-2bd38922d129@pardini.net> Content-Language: en-US From: Heiner Kallweit In-Reply-To: <20260617-rk3588-dts-rtl-eth-describe-dt-alias-v4-1-2bd38922d129@pardini.net> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 17.06.2026 14:58, Ricardo Pardini via B4 Relay wrote: > From: Ricardo Pardini > > Add a binding for fixed/soldered Realtek RTL8125 PCIe Ethernet > controller. > > The "pciVVVV,DDDD" compatibles are the Open Firmware PCI Bus Binding > spelling, auto-derived from PCI-SIG vendor/device IDs, but they still > need a binding when used in a board DT - analogous to "usbVVVV,PPPP" > compatibles documented in their own bindings (e.g. microchip,lan95xx) > so board DTs attaching properties (fixed MAC, nvmem cell, ...) to > these PCI function nodes can be validated. > > Suggested-by: Sebastian Reichel > Signed-off-by: Ricardo Pardini > --- > .../devicetree/bindings/net/realtek,rtl8125.yaml | 43 ++++++++++++++++++++++ > MAINTAINERS | 1 + > 2 files changed, 44 insertions(+) > > diff --git a/Documentation/devicetree/bindings/net/realtek,rtl8125.yaml b/Documentation/devicetree/bindings/net/realtek,rtl8125.yaml > new file mode 100644 > index 0000000000000..eee13fbc1e6a6 > --- /dev/null > +++ b/Documentation/devicetree/bindings/net/realtek,rtl8125.yaml > @@ -0,0 +1,43 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/net/realtek,rtl8125.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Realtek RTL8125 2.5 Gigabit PCIe Ethernet Controller > + > +maintainers: > + - Heiner Kallweit > + > +description: > + The Realtek RTL8125 is a 2.5GBASE-T Ethernet controller with a PCIe host > + interface. > + > +allOf: > + - $ref: ethernet-controller.yaml# > + > +properties: > + compatible: > + const: pci10ec,8125 IIRC we came to the conclusion that the compatible string isn't used in the relevant code path. Then why add it here? Is there an alignment on this? If it should be added here, then an explaining comment would be helpful. > + > + reg: > + maxItems: 1 > + > +required: > + - compatible > + - reg > + > +unevaluatedProperties: false > + > +examples: > + - | > + pcie { > + #address-cells = <3>; > + #size-cells = <2>; > + > + ethernet@0,0 { > + compatible = "pci10ec,8125"; > + reg = <0x10000 0 0 0 0>; > + local-mac-address = [00 00 00 00 00 00]; > + }; > + }; > diff --git a/MAINTAINERS b/MAINTAINERS > index c8d4b913f26c1..e5fbd82946aec 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -134,6 +134,7 @@ M: Heiner Kallweit > M: nic_swsd@realtek.com > L: netdev@vger.kernel.org > S: Maintained > +F: Documentation/devicetree/bindings/net/realtek,rtl8125.yaml > F: drivers/net/ethernet/realtek/r8169* > > 8250/16?50 (AND CLONE UARTS) SERIAL DRIVER >