* [RESEND PATCH net-next 0/5] dt-bindings: net: realtek,rtl9301-switch
@ 2025-02-18 19:52 Chris Packham
2025-02-18 19:52 ` [RESEND PATCH net-next 1/5] dt-bindings: net: Move realtek,rtl9301-switch to net Chris Packham
` (5 more replies)
0 siblings, 6 replies; 12+ messages in thread
From: Chris Packham @ 2025-02-18 19:52 UTC (permalink / raw)
To: lee, robh, krzk+dt, conor+dt, andrew+netdev, davem, edumazet,
kuba, pabeni, tsbogend
Cc: devicetree, linux-kernel, netdev, linux-mips, Chris Packham
This is my attempt at trying to sort out the mess I've created with the RTL9300
switch dt-bindings. Some context is available on [1] and [2].
The first patch just moves the binding from mfd/ to net/ (with an adjustment of
the internal path name). The next two patches are successors to patches already
sent as part of the series [3]. Finally the last two patches are needed to
update the in-tree dts to pass `make dtbs_check`, I couldn't figure out a clean
way of decoupling these `make dtbs_check` will either complain that the dts has
unexpected properties or that it is missing required properties.
[1] - https://lore.kernel.org/lkml/20250204-eccentric-deer-of-felicity-02b7ee@krzk-bin/
[2] - https://lore.kernel.org/lkml/4e3c5d83-d215-4eff-bf02-6d420592df8f@alliedtelesis.co.nz/
[3] - https://lore.kernel.org/lkml/20250204030249.1965444-1-chris.packham@alliedtelesis.co.nz/
Chris Packham (5):
dt-bindings: net: Move realtek,rtl9301-switch to net
dt-bindings: net: Add switch ports and interrupts to RTL9300
dt-bindings: net: Add Realtek MDIO controller
mips: dts: realtek: Add switch interrupts
mips: dts: cameo-rtl9302c: Add switch block
.../bindings/net/realtek,rtl9301-mdio.yaml | 86 +++++++++++++++++++
.../{mfd => net}/realtek,rtl9301-switch.yaml | 63 +++++++++++++-
.../cameo-rtl9302c-2x-rtl8224-2xge.dts | 48 +++++++++++
arch/mips/boot/dts/realtek/rtl930x.dtsi | 3 +
4 files changed, 199 insertions(+), 1 deletion(-)
create mode 100644 Documentation/devicetree/bindings/net/realtek,rtl9301-mdio.yaml
rename Documentation/devicetree/bindings/{mfd => net}/realtek,rtl9301-switch.yaml (66%)
--
2.48.1
^ permalink raw reply [flat|nested] 12+ messages in thread
* [RESEND PATCH net-next 1/5] dt-bindings: net: Move realtek,rtl9301-switch to net
2025-02-18 19:52 [RESEND PATCH net-next 0/5] dt-bindings: net: realtek,rtl9301-switch Chris Packham
@ 2025-02-18 19:52 ` Chris Packham
2025-02-20 10:57 ` Lee Jones
2025-02-21 21:05 ` Rob Herring (Arm)
2025-02-18 19:52 ` [RESEND PATCH net-next 2/5] dt-bindings: net: Add switch ports and interrupts to RTL9300 Chris Packham
` (4 subsequent siblings)
5 siblings, 2 replies; 12+ messages in thread
From: Chris Packham @ 2025-02-18 19:52 UTC (permalink / raw)
To: lee, robh, krzk+dt, conor+dt, andrew+netdev, davem, edumazet,
kuba, pabeni, tsbogend
Cc: devicetree, linux-kernel, netdev, linux-mips, Chris Packham
Initially realtek,rtl9301-switch was placed under mfd/ because it had
some non-switch related blocks (specifically i2c and reset) but with a
bit more review it has become apparent that this was wrong and the
binding should live under net/.
Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
---
Notes:
It would probably be simplest for this to come in via net-next with an
ack from Lee.
.../bindings/{mfd => net}/realtek,rtl9301-switch.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
rename Documentation/devicetree/bindings/{mfd => net}/realtek,rtl9301-switch.yaml (97%)
diff --git a/Documentation/devicetree/bindings/mfd/realtek,rtl9301-switch.yaml b/Documentation/devicetree/bindings/net/realtek,rtl9301-switch.yaml
similarity index 97%
rename from Documentation/devicetree/bindings/mfd/realtek,rtl9301-switch.yaml
rename to Documentation/devicetree/bindings/net/realtek,rtl9301-switch.yaml
index f053303ab1e6..d0a93d5f03aa 100644
--- a/Documentation/devicetree/bindings/mfd/realtek,rtl9301-switch.yaml
+++ b/Documentation/devicetree/bindings/net/realtek,rtl9301-switch.yaml
@@ -1,7 +1,7 @@
# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
%YAML 1.2
---
-$id: http://devicetree.org/schemas/mfd/realtek,rtl9301-switch.yaml#
+$id: http://devicetree.org/schemas/net/realtek,rtl9301-switch.yaml#
$schema: http://devicetree.org/meta-schemas/core.yaml#
title: Realtek Switch with Internal CPU
--
2.48.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [RESEND PATCH net-next 2/5] dt-bindings: net: Add switch ports and interrupts to RTL9300
2025-02-18 19:52 [RESEND PATCH net-next 0/5] dt-bindings: net: realtek,rtl9301-switch Chris Packham
2025-02-18 19:52 ` [RESEND PATCH net-next 1/5] dt-bindings: net: Move realtek,rtl9301-switch to net Chris Packham
@ 2025-02-18 19:52 ` Chris Packham
2025-02-21 21:06 ` Rob Herring (Arm)
2025-02-18 19:52 ` [RESEND PATCH net-next 3/5] dt-bindings: net: Add Realtek MDIO controller Chris Packham
` (3 subsequent siblings)
5 siblings, 1 reply; 12+ messages in thread
From: Chris Packham @ 2025-02-18 19:52 UTC (permalink / raw)
To: lee, robh, krzk+dt, conor+dt, andrew+netdev, davem, edumazet,
kuba, pabeni, tsbogend
Cc: devicetree, linux-kernel, netdev, linux-mips, Chris Packham
Add bindings for the ethernet-switch and interrupt properties for the
RTL9300.
Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
---
Notes:
This is tecnically v7 of [1] changes from that are:
- Use ethernet-switch.yaml#/$defs/ethernet-ports
- Enforce "ethernet-ports" be used instead of just "ports"
- Add interrupts for the switch block
[1] - https://lore.kernel.org/lkml/20250204030249.1965444-2-chris.packham@alliedtelesis.co.nz/
.../bindings/net/realtek,rtl9301-switch.yaml | 30 +++++++++++++++++++
1 file changed, 30 insertions(+)
diff --git a/Documentation/devicetree/bindings/net/realtek,rtl9301-switch.yaml b/Documentation/devicetree/bindings/net/realtek,rtl9301-switch.yaml
index d0a93d5f03aa..5d29647bea2d 100644
--- a/Documentation/devicetree/bindings/net/realtek,rtl9301-switch.yaml
+++ b/Documentation/devicetree/bindings/net/realtek,rtl9301-switch.yaml
@@ -14,6 +14,8 @@ description:
number of different peripherals are accessed through a common register block,
represented here as a syscon node.
+$ref: ethernet-switch.yaml#/$defs/ethernet-ports
+
properties:
compatible:
items:
@@ -28,12 +30,23 @@ properties:
reg:
maxItems: 1
+ interrupts:
+ maxItems: 2
+
+ interrupt-names:
+ items:
+ - const: switch
+ - const: nic
+
'#address-cells':
const: 1
'#size-cells':
const: 1
+ ethernet-ports:
+ type: object
+
patternProperties:
'reboot@[0-9a-f]+$':
$ref: /schemas/power/reset/syscon-reboot.yaml#
@@ -44,6 +57,8 @@ patternProperties:
required:
- compatible
- reg
+ - interrupts
+ - interrupt-names
additionalProperties: false
@@ -52,6 +67,9 @@ examples:
ethernet-switch@1b000000 {
compatible = "realtek,rtl9301-switch", "syscon", "simple-mfd";
reg = <0x1b000000 0x10000>;
+ interrupt-parent = <&intc>;
+ interrupts = <23>, <24>;
+ interrupt-names = "switch", "nic";
#address-cells = <1>;
#size-cells = <1>;
@@ -110,5 +128,17 @@ examples:
};
};
};
+
+ ethernet-ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ port@0 {
+ reg = <0>;
+ };
+ port@1 {
+ reg = <1>;
+ };
+ };
};
--
2.48.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [RESEND PATCH net-next 3/5] dt-bindings: net: Add Realtek MDIO controller
2025-02-18 19:52 [RESEND PATCH net-next 0/5] dt-bindings: net: realtek,rtl9301-switch Chris Packham
2025-02-18 19:52 ` [RESEND PATCH net-next 1/5] dt-bindings: net: Move realtek,rtl9301-switch to net Chris Packham
2025-02-18 19:52 ` [RESEND PATCH net-next 2/5] dt-bindings: net: Add switch ports and interrupts to RTL9300 Chris Packham
@ 2025-02-18 19:52 ` Chris Packham
2025-02-19 14:06 ` Rob Herring (Arm)
2025-02-18 19:52 ` [RESEND PATCH net-next 4/5] mips: dts: realtek: Add switch interrupts Chris Packham
` (2 subsequent siblings)
5 siblings, 1 reply; 12+ messages in thread
From: Chris Packham @ 2025-02-18 19:52 UTC (permalink / raw)
To: lee, robh, krzk+dt, conor+dt, andrew+netdev, davem, edumazet,
kuba, pabeni, tsbogend
Cc: devicetree, linux-kernel, netdev, linux-mips, Chris Packham
Add dtschema for the MDIO controller found in the RTL9300 Ethernet
switch. The controller is slightly unusual in that direct MDIO
communication is not possible. We model the MDIO controller with the
MDIO buses as child nodes and the PHYs as children of the buses. The
mapping of switch port number to MDIO bus/addr requires the
ethernet-ports sibling to provide the mapping via the phy-handle
property.
Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
---
Notes:
This is technically v7 of [1] and [2] which are combined now that
rtl9301-switch.yaml under net/ the only change from those is that the
$ref: in rtl9301-switch.yaml can now use a relative path
I could technically do away with the reg property on the mdio-controller
node. I don't currently need to use it in my driver and it looks like
the register offsets are the same between the RTL9300 and RTL9310.
[1] - https://lore.kernel.org/lkml/20250204030249.1965444-6-chris.packham@alliedtelesis.co.nz/
[2] - https://lore.kernel.org/lkml/20250204030249.1965444-4-chris.packham@alliedtelesis.co.nz/
.../bindings/net/realtek,rtl9301-mdio.yaml | 86 +++++++++++++++++++
.../bindings/net/realtek,rtl9301-switch.yaml | 31 +++++++
2 files changed, 117 insertions(+)
create mode 100644 Documentation/devicetree/bindings/net/realtek,rtl9301-mdio.yaml
diff --git a/Documentation/devicetree/bindings/net/realtek,rtl9301-mdio.yaml b/Documentation/devicetree/bindings/net/realtek,rtl9301-mdio.yaml
new file mode 100644
index 000000000000..02e4e33e9969
--- /dev/null
+++ b/Documentation/devicetree/bindings/net/realtek,rtl9301-mdio.yaml
@@ -0,0 +1,86 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/net/realtek,rtl9301-mdio.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Realtek RTL9300 MDIO Controller
+
+maintainers:
+ - Chris Packham <chris.packham@alliedtelesis.co.nz>
+
+properties:
+ compatible:
+ oneOf:
+ - items:
+ - enum:
+ - realtek,rtl9302b-mdio
+ - realtek,rtl9302c-mdio
+ - realtek,rtl9303-mdio
+ - const: realtek,rtl9301-mdio
+ - const: realtek,rtl9301-mdio
+
+ '#address-cells':
+ const: 1
+
+ '#size-cells':
+ const: 0
+
+ reg:
+ maxItems: 1
+
+patternProperties:
+ '^mdio-bus@[0-3]$':
+ $ref: mdio.yaml#
+
+ properties:
+ reg:
+ maxItems: 1
+
+ required:
+ - reg
+
+ patternProperties:
+ '^ethernet-phy@[a-f0-9]+$':
+ type: object
+ $ref: ethernet-phy.yaml#
+ unevaluatedProperties: false
+
+ unevaluatedProperties: false
+
+required:
+ - compatible
+ - reg
+
+unevaluatedProperties: false
+
+examples:
+ - |
+ mdio-controller@ca00 {
+ compatible = "realtek,rtl9301-mdio";
+ reg = <0xca00 0x200>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ mdio-bus@0 {
+ reg = <0>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ ethernet-phy@0 {
+ compatible = "ethernet-phy-ieee802.3-c45";
+ reg = <0>;
+ };
+ };
+
+ mdio-bus@1 {
+ reg = <1>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ ethernet-phy@0 {
+ compatible = "ethernet-phy-ieee802.3-c45";
+ reg = <0>;
+ };
+ };
+ };
diff --git a/Documentation/devicetree/bindings/net/realtek,rtl9301-switch.yaml b/Documentation/devicetree/bindings/net/realtek,rtl9301-switch.yaml
index 5d29647bea2d..80eabc170669 100644
--- a/Documentation/devicetree/bindings/net/realtek,rtl9301-switch.yaml
+++ b/Documentation/devicetree/bindings/net/realtek,rtl9301-switch.yaml
@@ -54,6 +54,9 @@ patternProperties:
'i2c@[0-9a-f]+$':
$ref: /schemas/i2c/realtek,rtl9301-i2c.yaml#
+ 'mdio-controller@[0-9a-f]+$':
+ $ref: realtek,rtl9301-mdio.yaml#
+
required:
- compatible
- reg
@@ -129,15 +132,43 @@ examples:
};
};
+ mdio-controller@ca00 {
+ compatible = "realtek,rtl9301-mdio";
+ reg = <0xca00 0x200>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ mdio-bus@0 {
+ reg = <0>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ phy1: ethernet-phy@0 {
+ reg = <0>;
+ };
+ };
+ mdio-bus@1 {
+ reg = <1>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ phy2: ethernet-phy@0 {
+ reg = <0>;
+ };
+ };
+ };
+
ethernet-ports {
#address-cells = <1>;
#size-cells = <0>;
port@0 {
reg = <0>;
+ phy-handle = <&phy1>;
};
port@1 {
reg = <1>;
+ phy-handle = <&phy2>;
};
};
};
--
2.48.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [RESEND PATCH net-next 4/5] mips: dts: realtek: Add switch interrupts
2025-02-18 19:52 [RESEND PATCH net-next 0/5] dt-bindings: net: realtek,rtl9301-switch Chris Packham
` (2 preceding siblings ...)
2025-02-18 19:52 ` [RESEND PATCH net-next 3/5] dt-bindings: net: Add Realtek MDIO controller Chris Packham
@ 2025-02-18 19:52 ` Chris Packham
2025-02-18 19:52 ` [RESEND PATCH net-next 5/5] mips: dts: cameo-rtl9302c: Add switch block Chris Packham
2025-02-21 23:30 ` [RESEND PATCH net-next 0/5] dt-bindings: net: realtek,rtl9301-switch patchwork-bot+netdevbpf
5 siblings, 0 replies; 12+ messages in thread
From: Chris Packham @ 2025-02-18 19:52 UTC (permalink / raw)
To: lee, robh, krzk+dt, conor+dt, andrew+netdev, davem, edumazet,
kuba, pabeni, tsbogend
Cc: devicetree, linux-kernel, netdev, linux-mips, Chris Packham
Add interrupts for the rtl9301-switch.
Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
---
arch/mips/boot/dts/realtek/rtl930x.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch/mips/boot/dts/realtek/rtl930x.dtsi b/arch/mips/boot/dts/realtek/rtl930x.dtsi
index 17577457d159..beb5ab41cb64 100644
--- a/arch/mips/boot/dts/realtek/rtl930x.dtsi
+++ b/arch/mips/boot/dts/realtek/rtl930x.dtsi
@@ -33,6 +33,9 @@ lx_clk: clock-175mhz {
switch0: switch@1b000000 {
compatible = "realtek,rtl9301-switch", "syscon", "simple-mfd";
reg = <0x1b000000 0x10000>;
+ interrupt-parent = <&intc>;
+ interrupts = <23>, <24>;
+ interrupt-names = "switch", "nic";
#address-cells = <1>;
#size-cells = <1>;
--
2.48.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [RESEND PATCH net-next 5/5] mips: dts: cameo-rtl9302c: Add switch block
2025-02-18 19:52 [RESEND PATCH net-next 0/5] dt-bindings: net: realtek,rtl9301-switch Chris Packham
` (3 preceding siblings ...)
2025-02-18 19:52 ` [RESEND PATCH net-next 4/5] mips: dts: realtek: Add switch interrupts Chris Packham
@ 2025-02-18 19:52 ` Chris Packham
2025-02-21 23:30 ` [RESEND PATCH net-next 0/5] dt-bindings: net: realtek,rtl9301-switch patchwork-bot+netdevbpf
5 siblings, 0 replies; 12+ messages in thread
From: Chris Packham @ 2025-02-18 19:52 UTC (permalink / raw)
To: lee, robh, krzk+dt, conor+dt, andrew+netdev, davem, edumazet,
kuba, pabeni, tsbogend
Cc: devicetree, linux-kernel, netdev, linux-mips, Chris Packham
Add the switch port and phys to the cameo-rtl9302c-2x-rtl8224-2xge
board.
Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
---
Notes:
This is technically v7 of [1]. Differences from that are that I've
omitted the MDIO busses and PHYs while I'm trying to sort out the switch
bindings. I'll add these back in a follow up series.
[1] - https://lore.kernel.org/lkml/20250204030249.1965444-7-chris.packham@alliedtelesis.co.nz/
.../cameo-rtl9302c-2x-rtl8224-2xge.dts | 48 +++++++++++++++++++
1 file changed, 48 insertions(+)
diff --git a/arch/mips/boot/dts/realtek/cameo-rtl9302c-2x-rtl8224-2xge.dts b/arch/mips/boot/dts/realtek/cameo-rtl9302c-2x-rtl8224-2xge.dts
index 6789bf374044..51306c104b01 100644
--- a/arch/mips/boot/dts/realtek/cameo-rtl9302c-2x-rtl8224-2xge.dts
+++ b/arch/mips/boot/dts/realtek/cameo-rtl9302c-2x-rtl8224-2xge.dts
@@ -71,3 +71,51 @@ partition@1180000 {
};
};
};
+
+&switch0 {
+ ethernet-ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ port@0 {
+ reg = <0>;
+ phy-mode = "usxgmii";
+ };
+ port@1 {
+ reg = <1>;
+ phy-mode = "usxgmii";
+ };
+ port@2 {
+ reg = <2>;
+ phy-mode = "usxgmii";
+ };
+ port@3 {
+ reg = <3>;
+ phy-mode = "usxgmii";
+ };
+ port@16 {
+ reg = <16>;
+ phy-mode = "usxgmii";
+ };
+ port@17 {
+ reg = <17>;
+ phy-mode = "usxgmii";
+ };
+ port@18 {
+ reg = <18>;
+ phy-mode = "usxgmii";
+ };
+ port@19 {
+ reg = <19>;
+ phy-mode = "usxgmii";
+ };
+ port@24{
+ reg = <24>;
+ phy-mode = "10gbase-r";
+ };
+ port@25{
+ reg = <25>;
+ phy-mode = "10gbase-r";
+ };
+ };
+};
--
2.48.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [RESEND PATCH net-next 3/5] dt-bindings: net: Add Realtek MDIO controller
2025-02-18 19:52 ` [RESEND PATCH net-next 3/5] dt-bindings: net: Add Realtek MDIO controller Chris Packham
@ 2025-02-19 14:06 ` Rob Herring (Arm)
2025-02-19 20:42 ` Chris Packham
0 siblings, 1 reply; 12+ messages in thread
From: Rob Herring (Arm) @ 2025-02-19 14:06 UTC (permalink / raw)
To: Chris Packham
Cc: conor+dt, krzk+dt, linux-mips, tsbogend, andrew+netdev, pabeni,
linux-kernel, devicetree, netdev, kuba, edumazet, lee, davem
On Wed, 19 Feb 2025 08:52:14 +1300, Chris Packham wrote:
> Add dtschema for the MDIO controller found in the RTL9300 Ethernet
> switch. The controller is slightly unusual in that direct MDIO
> communication is not possible. We model the MDIO controller with the
> MDIO buses as child nodes and the PHYs as children of the buses. The
> mapping of switch port number to MDIO bus/addr requires the
> ethernet-ports sibling to provide the mapping via the phy-handle
> property.
>
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
> ---
>
> Notes:
> This is technically v7 of [1] and [2] which are combined now that
> rtl9301-switch.yaml under net/ the only change from those is that the
> $ref: in rtl9301-switch.yaml can now use a relative path
>
> I could technically do away with the reg property on the mdio-controller
> node. I don't currently need to use it in my driver and it looks like
> the register offsets are the same between the RTL9300 and RTL9310.
>
> [1] - https://lore.kernel.org/lkml/20250204030249.1965444-6-chris.packham@alliedtelesis.co.nz/
> [2] - https://lore.kernel.org/lkml/20250204030249.1965444-4-chris.packham@alliedtelesis.co.nz/
>
> .../bindings/net/realtek,rtl9301-mdio.yaml | 86 +++++++++++++++++++
> .../bindings/net/realtek,rtl9301-switch.yaml | 31 +++++++
> 2 files changed, 117 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/net/realtek,rtl9301-mdio.yaml
>
Please add Acked-by/Reviewed-by tags when posting new versions. However,
there's no need to repost patches *only* to add the tags. The upstream
maintainer will do that for acks received on the version they apply.
If a tag was not added on purpose, please state why and what changed.
Missing tags:
Reviewed-by: Rob Herring (Arm) <robh@kernel.org>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [RESEND PATCH net-next 3/5] dt-bindings: net: Add Realtek MDIO controller
2025-02-19 14:06 ` Rob Herring (Arm)
@ 2025-02-19 20:42 ` Chris Packham
0 siblings, 0 replies; 12+ messages in thread
From: Chris Packham @ 2025-02-19 20:42 UTC (permalink / raw)
To: Rob Herring (Arm)
Cc: conor+dt, krzk+dt, linux-mips, tsbogend, andrew+netdev, pabeni,
linux-kernel, devicetree, netdev, kuba, edumazet, lee, davem
Hi Rob,
On 20/02/2025 03:06, Rob Herring (Arm) wrote:
> On Wed, 19 Feb 2025 08:52:14 +1300, Chris Packham wrote:
>> Add dtschema for the MDIO controller found in the RTL9300 Ethernet
>> switch. The controller is slightly unusual in that direct MDIO
>> communication is not possible. We model the MDIO controller with the
>> MDIO buses as child nodes and the PHYs as children of the buses. The
>> mapping of switch port number to MDIO bus/addr requires the
>> ethernet-ports sibling to provide the mapping via the phy-handle
>> property.
>>
>> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
>> ---
>>
>> Notes:
>> This is technically v7 of [1] and [2] which are combined now that
>> rtl9301-switch.yaml under net/ the only change from those is that the
>> $ref: in rtl9301-switch.yaml can now use a relative path
>>
>> I could technically do away with the reg property on the mdio-controller
>> node. I don't currently need to use it in my driver and it looks like
>> the register offsets are the same between the RTL9300 and RTL9310.
>>
>> [1] - https://lore.kernel.org/lkml/20250204030249.1965444-6-chris.packham@alliedtelesis.co.nz/
>> [2] - https://lore.kernel.org/lkml/20250204030249.1965444-4-chris.packham@alliedtelesis.co.nz/
>>
>> .../bindings/net/realtek,rtl9301-mdio.yaml | 86 +++++++++++++++++++
>> .../bindings/net/realtek,rtl9301-switch.yaml | 31 +++++++
>> 2 files changed, 117 insertions(+)
>> create mode 100644 Documentation/devicetree/bindings/net/realtek,rtl9301-mdio.yaml
>>
>
> Please add Acked-by/Reviewed-by tags when posting new versions. However,
> there's no need to repost patches *only* to add the tags. The upstream
> maintainer will do that for acks received on the version they apply.
>
> If a tag was not added on purpose, please state why and what changed.
>
> Missing tags:
>
> Reviewed-by: Rob Herring (Arm) <robh@kernel.org>
Sorry about that. I wasn't sure since this was spun off from the other
series and because it now includes changes to
realtek,rtl9301-switch.yaml. I should have at least mentioned that.
If there is another round for this one I'll include your r-by.
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [RESEND PATCH net-next 1/5] dt-bindings: net: Move realtek,rtl9301-switch to net
2025-02-18 19:52 ` [RESEND PATCH net-next 1/5] dt-bindings: net: Move realtek,rtl9301-switch to net Chris Packham
@ 2025-02-20 10:57 ` Lee Jones
2025-02-21 21:05 ` Rob Herring (Arm)
1 sibling, 0 replies; 12+ messages in thread
From: Lee Jones @ 2025-02-20 10:57 UTC (permalink / raw)
To: Chris Packham
Cc: robh, krzk+dt, conor+dt, andrew+netdev, davem, edumazet, kuba,
pabeni, tsbogend, devicetree, linux-kernel, netdev, linux-mips
On Wed, 19 Feb 2025, Chris Packham wrote:
> Initially realtek,rtl9301-switch was placed under mfd/ because it had
> some non-switch related blocks (specifically i2c and reset) but with a
> bit more review it has become apparent that this was wrong and the
> binding should live under net/.
>
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
> ---
>
> Notes:
> It would probably be simplest for this to come in via net-next with an
> ack from Lee.
>
> .../bindings/{mfd => net}/realtek,rtl9301-switch.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> rename Documentation/devicetree/bindings/{mfd => net}/realtek,rtl9301-switch.yaml (97%)
Acked-by: Lee Jones <lee@kernel.org>
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [RESEND PATCH net-next 1/5] dt-bindings: net: Move realtek,rtl9301-switch to net
2025-02-18 19:52 ` [RESEND PATCH net-next 1/5] dt-bindings: net: Move realtek,rtl9301-switch to net Chris Packham
2025-02-20 10:57 ` Lee Jones
@ 2025-02-21 21:05 ` Rob Herring (Arm)
1 sibling, 0 replies; 12+ messages in thread
From: Rob Herring (Arm) @ 2025-02-21 21:05 UTC (permalink / raw)
To: Chris Packham
Cc: andrew+netdev, linux-mips, tsbogend, edumazet, conor+dt, krzk+dt,
devicetree, pabeni, kuba, lee, netdev, linux-kernel, davem
On Wed, 19 Feb 2025 08:52:12 +1300, Chris Packham wrote:
> Initially realtek,rtl9301-switch was placed under mfd/ because it had
> some non-switch related blocks (specifically i2c and reset) but with a
> bit more review it has become apparent that this was wrong and the
> binding should live under net/.
>
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
> ---
>
> Notes:
> It would probably be simplest for this to come in via net-next with an
> ack from Lee.
>
> .../bindings/{mfd => net}/realtek,rtl9301-switch.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> rename Documentation/devicetree/bindings/{mfd => net}/realtek,rtl9301-switch.yaml (97%)
>
Acked-by: Rob Herring (Arm) <robh@kernel.org>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [RESEND PATCH net-next 2/5] dt-bindings: net: Add switch ports and interrupts to RTL9300
2025-02-18 19:52 ` [RESEND PATCH net-next 2/5] dt-bindings: net: Add switch ports and interrupts to RTL9300 Chris Packham
@ 2025-02-21 21:06 ` Rob Herring (Arm)
0 siblings, 0 replies; 12+ messages in thread
From: Rob Herring (Arm) @ 2025-02-21 21:06 UTC (permalink / raw)
To: Chris Packham
Cc: conor+dt, tsbogend, davem, pabeni, netdev, kuba, andrew+netdev,
devicetree, lee, linux-mips, linux-kernel, krzk+dt, edumazet
On Wed, 19 Feb 2025 08:52:13 +1300, Chris Packham wrote:
> Add bindings for the ethernet-switch and interrupt properties for the
> RTL9300.
>
> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
> ---
>
> Notes:
> This is tecnically v7 of [1] changes from that are:
> - Use ethernet-switch.yaml#/$defs/ethernet-ports
> - Enforce "ethernet-ports" be used instead of just "ports"
> - Add interrupts for the switch block
>
> [1] - https://lore.kernel.org/lkml/20250204030249.1965444-2-chris.packham@alliedtelesis.co.nz/
>
> .../bindings/net/realtek,rtl9301-switch.yaml | 30 +++++++++++++++++++
> 1 file changed, 30 insertions(+)
>
Reviewed-by: Rob Herring (Arm) <robh@kernel.org>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [RESEND PATCH net-next 0/5] dt-bindings: net: realtek,rtl9301-switch
2025-02-18 19:52 [RESEND PATCH net-next 0/5] dt-bindings: net: realtek,rtl9301-switch Chris Packham
` (4 preceding siblings ...)
2025-02-18 19:52 ` [RESEND PATCH net-next 5/5] mips: dts: cameo-rtl9302c: Add switch block Chris Packham
@ 2025-02-21 23:30 ` patchwork-bot+netdevbpf
5 siblings, 0 replies; 12+ messages in thread
From: patchwork-bot+netdevbpf @ 2025-02-21 23:30 UTC (permalink / raw)
To: Chris Packham
Cc: lee, robh, krzk+dt, conor+dt, andrew+netdev, davem, edumazet,
kuba, pabeni, tsbogend, devicetree, linux-kernel, netdev,
linux-mips
Hello:
This series was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:
On Wed, 19 Feb 2025 08:52:11 +1300 you wrote:
> This is my attempt at trying to sort out the mess I've created with the RTL9300
> switch dt-bindings. Some context is available on [1] and [2].
>
> The first patch just moves the binding from mfd/ to net/ (with an adjustment of
> the internal path name). The next two patches are successors to patches already
> sent as part of the series [3]. Finally the last two patches are needed to
> update the in-tree dts to pass `make dtbs_check`, I couldn't figure out a clean
> way of decoupling these `make dtbs_check` will either complain that the dts has
> unexpected properties or that it is missing required properties.
>
> [...]
Here is the summary with links:
- [RESEND,net-next,1/5] dt-bindings: net: Move realtek,rtl9301-switch to net
https://git.kernel.org/netdev/net-next/c/3fa337651df5
- [RESEND,net-next,2/5] dt-bindings: net: Add switch ports and interrupts to RTL9300
https://git.kernel.org/netdev/net-next/c/92575a218237
- [RESEND,net-next,3/5] dt-bindings: net: Add Realtek MDIO controller
https://git.kernel.org/netdev/net-next/c/96757457da0e
- [RESEND,net-next,4/5] mips: dts: realtek: Add switch interrupts
(no matching commit)
- [RESEND,net-next,5/5] mips: dts: cameo-rtl9302c: Add switch block
(no matching commit)
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2025-02-21 23:29 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-02-18 19:52 [RESEND PATCH net-next 0/5] dt-bindings: net: realtek,rtl9301-switch Chris Packham
2025-02-18 19:52 ` [RESEND PATCH net-next 1/5] dt-bindings: net: Move realtek,rtl9301-switch to net Chris Packham
2025-02-20 10:57 ` Lee Jones
2025-02-21 21:05 ` Rob Herring (Arm)
2025-02-18 19:52 ` [RESEND PATCH net-next 2/5] dt-bindings: net: Add switch ports and interrupts to RTL9300 Chris Packham
2025-02-21 21:06 ` Rob Herring (Arm)
2025-02-18 19:52 ` [RESEND PATCH net-next 3/5] dt-bindings: net: Add Realtek MDIO controller Chris Packham
2025-02-19 14:06 ` Rob Herring (Arm)
2025-02-19 20:42 ` Chris Packham
2025-02-18 19:52 ` [RESEND PATCH net-next 4/5] mips: dts: realtek: Add switch interrupts Chris Packham
2025-02-18 19:52 ` [RESEND PATCH net-next 5/5] mips: dts: cameo-rtl9302c: Add switch block Chris Packham
2025-02-21 23:30 ` [RESEND PATCH net-next 0/5] dt-bindings: net: realtek,rtl9301-switch patchwork-bot+netdevbpf
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).