Skip to content

Review apml modules venice a0 #135

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
akky16 wants to merge 10 commits into
base: dev-a1
Choose a base branch
from
Open

Review apml modules venice a0 #135

Show file tree
Hide file tree
Changes from 9 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
bcc5de5
amd-apml: sbrmi: Add sbrmi device as per the SBRMI device instance ID
akky16 Jun 2, 2025
c77f047
amd-apml: sbtsi: Add sbtsi device as per the SBTSI device instance ID
akky16 Jun 2, 2025
416a90d
amd-apml: sbtsi: Add device IDs for socket 0 and Socket 1 for Fam:0x1…
akky16 Jun 2, 2025
3cbafb7
amd-apml: sbrmi: Add device IDs for socket 0 and Socket 1 for Fam:0x1…
akky16 Jun 2, 2025
0666518
linux: i3c: Extra info should not be updated in the driver
akky16 Jun 2, 2025
ca272c9
amd-apml: sbrmi: update label for hwmon entry to include socket_iod n…
akky16 Jun 2, 2025
2495233
amd_apml: sbtsi: update label for hwmon entry to include socket & die…
akky16 Jun 2, 2025
1a104c9
amd-apml: alertl: Add support for AMD APML Alert_L platform driver
sathyapk Jun 2, 2025
df7cc4b
amd-apml: alertl: dts changes for Morocco & Congo
akky16 May 30, 2025
0f54429
amd-apml: doc: Documentation for dts changes for Alert_L module
akky16 Jun 3, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 11 additions & 0 deletions arch/arm64/boot/dts/aspeed/aspeed-bmc-amd-congo.dts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -894,6 +894,17 @@
#endif
};

/ {
/* Alert_L associated with socket 0 */
alertl_sock0 {
compatible = "apml-alertl";
status = "okay";
gpios = <&ltpi0_gpio 20 GPIO_ACTIVE_LOW>;
sbrmi = <&sbrmi_p0_iod0>;
sbtsi = <&sbtsi_p0_iod0>;
};
};

#ifdef I3C_HUB

#define JESD300_SPD_I3C_MODE(bus, index, addr) \
Expand Down
11 changes: 11 additions & 0 deletions arch/arm64/boot/dts/aspeed/aspeed-bmc-amd-morocco.dts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -984,6 +984,17 @@
#endif
};

/ {
/* Alert_L associated with socket 0 */
alertl_sock0 {
compatible = "apml-alertl";
status = "okay";
gpios = <&ltpi0_gpio 20 GPIO_ACTIVE_LOW>;
sbrmi = <&sbrmi_p0_iod0>;
sbtsi = <&sbtsi_p0_iod0>;
};
};

Copy link
Copy Markdown
Collaborator

@abinayaddhandapani abinayaddhandapani Jun 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add Alert_L node for socket 1.
You can get the LTPI GPIO number from the PR #139

Copy link
Copy Markdown
Collaborator

@nchatrad nchatrad Jun 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done in PR #143

#ifdef I3C_HUB

#define JESD300_SPD_I3C_MODE(bus, index, addr) \
Expand Down
1 change: 0 additions & 1 deletion drivers/i3c/device.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -366,7 +366,6 @@ i3c_device_match_id(struct i3c_device *i3cdev,
part = I3C_PID_PART_ID(devinfo.pid);
ext_info = I3C_PID_EXTRA_INFO(devinfo.pid);
inst = I3C_PID_INSTANCE_ID(devinfo.pid);
ext_info = ext_info | (inst << 12);
rndpid = I3C_PID_RND_LOWER_32BITS(devinfo.pid);

printk(" i3c_device_match_id: manuf 0x%x, part 0x%x, inst 0x%x, ext_info 0x%x \n", manuf, part, inst, ext_info);
Expand Down
11 changes: 11 additions & 0 deletions drivers/misc/amd-apml/Kconfig
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,3 +24,14 @@ config APML_SBTSI

This driver can also be built as a module. If so, the module will
be called apml_sbtsi.

config APML_ALERTL
tristate "Emulated apml alertl interface driver over i3c bus"
depends on APML_SBRMI && APML_SBTSI
default n
help
If you say yes here you get support for emulated alertl
interface on AMD SoCs with APML interface connected to a BMC device.

This driver can also be built as a module. If so, the module will
be called apml_alertl.
1 change: 1 addition & 0 deletions drivers/misc/amd-apml/Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,3 +6,4 @@

obj-$(CONFIG_APML_SBRMI) += sbrmi.o sbrmi-common.o
obj-$(CONFIG_APML_SBTSI) += apml_sbtsi.o
obj-$(CONFIG_APML_ALERTL) += apml_alertl.o
Loading