[PATCH v2] ARM: dts: aspeed: clemente: add MCTP over I2C support

Leo Wang posted 1 patch 1 week, 2 days ago
arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-clemente.dts | 11 +++++++++--
1 file changed, 9 insertions(+), 2 deletions(-)
[PATCH v2] ARM: dts: aspeed: clemente: add MCTP over I2C support
Posted by Leo Wang 1 week, 2 days ago
From: Leo Wang <leo.jt.wang@gmail.com>

Enable MCTP communication on the Meta Clemente BMC board:

- Add an MCTP I2C controller node at address 0x10 on I2C0
- Mark selected I2C mux channels as MCTP controllers
- Remove unused "i2c-mux-idle-disconnect" properties

This allows MCTP devices to be discovered and used for
management communication on the Clemente platform.

Signed-off-by: Leo Wang <leo.jt.wang@gmail.com>
---
Please consider the patch "[PATCH v1 1/1] ARM: dts: aspeed: clemente: add MCTP over I2C support"
withdrawn.

After further review we determined that enabling MCTP in the Clemente device-tree is not
required; the affected devices are managed through other mechanisms and adding MCTP nodes
would be unnecessary. To avoid confusion and noise, please ignore the previous patch.

Original patch message-id: <20250918-leo-dts-transceiver-v1-1-3e2580325edc@gmail.com>
---
Changes in v2:
- Withdraw patch; no functional changes
- Link to v1: https://lore.kernel.org/r/20250918-leo-dts-transceiver-v1-1-3e2580325edc@gmail.com
---
 arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-clemente.dts | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-clemente.dts b/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-clemente.dts
index a614f947dafc9a6628c3f3d45b62ff3ca9d2b618..a2f0ed6e7bbefbdcec33f7ddebdf2de62adec164 100644
--- a/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-clemente.dts
+++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-clemente.dts
@@ -269,18 +269,23 @@ &gpio1 {
 
 &i2c0 {
 	status = "okay";
+	multi-master;
+	mctp@10 {
+		compatible = "mctp-i2c-controller";
+		reg = <(0x10 | I2C_OWN_SLAVE_ADDRESS)>;
+	};
 
 	i2c-mux@71 {
 		compatible = "nxp,pca9546";
 		reg = <0x71>;
 		#address-cells = <1>;
 		#size-cells = <0>;
-		i2c-mux-idle-disconnect;
 
 		i2c0mux0ch0: i2c@0 {
 			#address-cells = <1>;
 			#size-cells = <0>;
 			reg = <0>;
+			mctp-controller;
 		};
 
 		i2c0mux0ch1: i2c@1 {
@@ -332,6 +337,7 @@ i2c0mux0ch2: i2c@2 {
 			#address-cells = <1>;
 			#size-cells = <0>;
 			reg = <2>;
+			mctp-controller;
 		};
 
 		i2c0mux0ch3: i2c@3 {
@@ -457,12 +463,12 @@ i2c-mux@75 {
 		reg = <0x75>;
 		#address-cells = <1>;
 		#size-cells = <0>;
-		i2c-mux-idle-disconnect;
 
 		i2c0mux3ch0: i2c@0 {
 			#address-cells = <1>;
 			#size-cells = <0>;
 			reg = <0>;
+			mctp-controller;
 		};
 
 		i2c0mux3ch1: i2c@1 {
@@ -514,6 +520,7 @@ i2c0mux3ch2: i2c@2 {
 			#address-cells = <1>;
 			#size-cells = <0>;
 			reg = <2>;
+			mctp-controller;
 		};
 
 		i2c0mux3ch3: i2c@3 {

---
base-commit: 1937f781fe7fcde26e537e74dfc6097fc08fc9bb
change-id: 20250918-leo-dts-transceiver-edc31e3c21af

Best regards,
-- 
Leo Wang <leo.jt.wang@fii-foxconn.com>
Re: [PATCH v2] ARM: dts: aspeed: clemente: add MCTP over I2C support
Posted by Andrew Jeffery 1 week, 1 day ago
Hi Leo,

On Mon, 2025-09-22 at 15:02 +0800, Leo Wang wrote:
> From: Leo Wang <leo.jt.wang@gmail.com>
> 
> Enable MCTP communication on the Meta Clemente BMC board:
> 
> - Add an MCTP I2C controller node at address 0x10 on I2C0
> - Mark selected I2C mux channels as MCTP controllers
> - Remove unused "i2c-mux-idle-disconnect" properties
> 
> This allows MCTP devices to be discovered and used for
> management communication on the Clemente platform.
> 
> Signed-off-by: Leo Wang <leo.jt.wang@gmail.com>
> ---
> Please consider the patch "[PATCH v1 1/1] ARM: dts: aspeed: clemente: add MCTP over I2C support"
> withdrawn.
> 
> After further review we determined that enabling MCTP in the Clemente device-tree is not
> required; the affected devices are managed through other mechanisms and adding MCTP nodes
> would be unnecessary. To avoid confusion and noise, please ignore the previous patch.
> 
> Original patch message-id: <20250918-leo-dts-transceiver-v1-1-3e2580325edc@gmail.com>
> ---
> Changes in v2:
> - Withdraw patch; no functional changes
> - Link to v1: https://lore.kernel.org/r/20250918-leo-dts-transceiver-v1-1-3e2580325edc@gmail.com
> ---

Please do not post a new revision of a series to announce withdrawal of
the series.

Rather, reply to the original series saying as much.

Thanks.

Andrew