diff mbox series

arm: dts: aspeed: Enable PECI and LPC Snoop

Message ID 20240917-dts-aspeed-system1-peci-snoop-v1-1-b967fb3a6735@gmail.com
State New
Headers show
Series arm: dts: aspeed: Enable PECI and LPC Snoop | expand

Commit Message

Manojkiran Eda via B4 Relay Sept. 17, 2024, 11:57 a.m. UTC
From: Manojkiran Eda <manojkiran.eda@gmail.com>

This patch enables the PECI interface and configures the LPC Snoop for
ports 0x80 and 0x81 in the ASPEED BMC for IBM System1.

Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>
---
This patch enables PECI and LPC snoop functionality on the IBM System1
BMC in the device tree.

The following changes have been made:

1. Enabled the PECI controller (peci0) by marking its status to "okay".
2. Enabled the LPC snoop engine, configuring snoop ports at 0x80 and
   0x81.

These changes are required to support PECI communication and LPC
snooping for system monitoring and debugging purposes.
---
 arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts | 9 +++++++++
 1 file changed, 9 insertions(+)


---
base-commit: ca2478a7d974f38d29d27acb42a952c7f168916e
change-id: 20240917-dts-aspeed-system1-peci-snoop-725c70f4a485

Best regards,

Comments

Andrew Jeffery Sept. 18, 2024, 12:39 a.m. UTC | #1
Hi Manoj,

Can you please mention mentions the specific platform in the subject
(IBM System1)? Otherwise the single-line log view gives the impression
that you're enabling them for all Aspeed-based platforms.

Perhaps:

   ARM: dts: aspeed: Enable PECI and LPC snoop for IBM System1

Also, can you please send this to the upstream lists? From there I'll
backport it to the openbmc tree. `b4 prep --auto-to-cc` should help.

Andrew

On Tue, 2024-09-17 at 17:27 +0530, Manojkiran Eda via B4 Relay wrote:
> From: Manojkiran Eda <manojkiran.eda@gmail.com>
> 
> This patch enables the PECI interface and configures the LPC Snoop for
> ports 0x80 and 0x81 in the ASPEED BMC for IBM System1.
> 
> Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>
> ---
> This patch enables PECI and LPC snoop functionality on the IBM System1
> BMC in the device tree.
> 
> The following changes have been made:
> 
> 1. Enabled the PECI controller (peci0) by marking its status to "okay".
> 2. Enabled the LPC snoop engine, configuring snoop ports at 0x80 and
>    0x81.
> 
> These changes are required to support PECI communication and LPC
> snooping for system monitoring and debugging purposes.
> ---
>  arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts
> index cb3063413d1f..738a86c787c0 100644
> --- a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts
> +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts
> @@ -464,6 +464,15 @@ &kcs3 {
>  	aspeed,lpc-interrupts = <11 IRQ_TYPE_LEVEL_LOW>;
>  };
>  
> +&peci0 {
> +	status = "okay";
> +};
> +
> +&lpc_snoop {
> +	status = "okay";
> +	snoop-ports = <0x80>, <0x81>;
> +};
> +
>  &i2c0 {
>  	status = "okay";
>  
> 
> ---
> base-commit: ca2478a7d974f38d29d27acb42a952c7f168916e
> change-id: 20240917-dts-aspeed-system1-peci-snoop-725c70f4a485
> 
> Best regards,
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts
index cb3063413d1f..738a86c787c0 100644
--- a/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts
+++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-system1.dts
@@ -464,6 +464,15 @@  &kcs3 {
 	aspeed,lpc-interrupts = <11 IRQ_TYPE_LEVEL_LOW>;
 };
 
+&peci0 {
+	status = "okay";
+};
+
+&lpc_snoop {
+	status = "okay";
+	snoop-ports = <0x80>, <0x81>;
+};
+
 &i2c0 {
 	status = "okay";