mbox series

[v8,0/2] arm64: dts: mt8195: Add Ethernet controller

Message ID 20230105010712.10116-1-biao.huang@mediatek.com (mailing list archive)
Headers show
Series arm64: dts: mt8195: Add Ethernet controller | expand

Message

Biao Huang (黄彪) Jan. 5, 2023, 1:07 a.m. UTC
Changes in v8:
1. add reviewed-by as Andrew's comments.

Changes in v7:
1. move mdio node to .dtsi, and remove the compatible
property in ethernet-phy node as Andrew's comments.
2. add netdev@ to cc list as Jakub's reminder.

Changes in v6:
1. add reviewed-by as Angelo's comments
2. remove fix_mac_speed in driver as Andrew's comments.

Changes in v5:
1. reorder the clocks as Angelo's comments
2. add a driver patch to fix rgmii-id issue, then we can
use a ususal way rgmii/rgmii-id as Andrew's comments.

Changes in v4:
1. remove {address,size}-cells = <0> to avoid warning as Angelo's feedback.
2. Add reviewd-by as Angelo's comments.

Changes in v3:
1. move stmmac-axi-config, rx-queues-config, tx-queues-configs inside ethernet
node as Angelo's comments.
2. add {address,size}-cells = <0> in ethernet node as Angelo's comments.

Changes in v2:
1. modify pinctrl node used by ethernet to match rules in pinctrl-mt8195.yaml,
which is pointed by Krzysztof.
2. remove "mac-address" property in ethernet node as comments of Krzysztof.

Changes in v1:
add dts node for MT8195 Ethernet controller

Biao Huang (2):
  stmmac: dwmac-mediatek: remove the dwmac_fix_mac_speed
  arm64: dts: mt8195: Add Ethernet controller

 arch/arm64/boot/dts/mediatek/mt8195-demo.dts  | 77 ++++++++++++++++
 arch/arm64/boot/dts/mediatek/mt8195.dtsi      | 92 +++++++++++++++++++
 .../ethernet/stmicro/stmmac/dwmac-mediatek.c  | 26 ------
 3 files changed, 169 insertions(+), 26 deletions(-)

Comments

Jakub Kicinski Jan. 6, 2023, 4:58 a.m. UTC | #1
On Thu, 5 Jan 2023 09:07:10 +0800 Biao Huang wrote:
> Changes in v8:
> 1. add reviewed-by as Andrew's comments.

You don't have to repost just to include review/ack tags.
They are automatically gathered by our patch application tooling
when we apply patches to the tree.

I will take patch 1 to the networking tree, I _think_ patch 2 is
supposed to go via Matthias?
patchwork-bot+netdevbpf@kernel.org Jan. 6, 2023, 5:10 a.m. UTC | #2
Hello:

This series was applied to netdev/net.git (master)
by Jakub Kicinski <kuba@kernel.org>:

On Thu, 5 Jan 2023 09:07:10 +0800 you wrote:
> Changes in v8:
> 1. add reviewed-by as Andrew's comments.
> 
> Changes in v7:
> 1. move mdio node to .dtsi, and remove the compatible
> property in ethernet-phy node as Andrew's comments.
> 2. add netdev@ to cc list as Jakub's reminder.
> 
> [...]

Here is the summary with links:
  - [v8,1/2] stmmac: dwmac-mediatek: remove the dwmac_fix_mac_speed
    https://git.kernel.org/netdev/net/c/c26de7507d1f
  - [v8,2/2] arm64: dts: mt8195: Add Ethernet controller
    (no matching commit)

You are awesome, thank you!
Biao Huang (黄彪) Jan. 11, 2023, 5:49 a.m. UTC | #3
On Thu, 2023-01-05 at 20:58 -0800, Jakub Kicinski wrote:
> On Thu, 5 Jan 2023 09:07:10 +0800 Biao Huang wrote:
> > Changes in v8:
> > 1. add reviewed-by as Andrew's comments.
> 
> You don't have to repost just to include review/ack tags.
> They are automatically gathered by our patch application tooling
> when we apply patches to the tree.
OK, thanks for your kindly reminder~
> 
> I will take patch 1 to the networking tree, I _think_ patch 2 is
> supposed to go via Matthias?
OK, @Matthias, please kindly review patch 2 (dts patch).

Best Regards!
Biao