Skip to content

Conversation

@jvdillon
Copy link

No description provided.

@jvdillon
Copy link
Author

jvdillon commented Sep 19, 2025

Note: while this allows me to build the modules, I still cannot insmod them.

$ make
make -C /lib/modules/`uname -r`/build M=$PWD modules
make[1]: Entering directory '/usr/src/linux-headers-6.14.0-29-generic'
make[2]: Entering directory '/home/jvdillon/apml_modules'
warning: the compiler differs from the one used to build the kernel
  The kernel was built by: x86_64-linux-gnu-gcc-13 (Ubuntu 13.3.0-6ubuntu2~24.04) 13.3.0
  You are using:           gcc-13 (Ubuntu 13.3.0-6ubuntu2~24.04) 13.3.0
  CC [M]  sbrmi.o
  CC [M]  sbrmi-common.o
  LD [M]  apml_sbrmi.o
  CC [M]  apml_sbtsi.o
  CC [M]  apml_alertl.o
  MODPOST Module.symvers
  CC [M]  apml_sbrmi.mod.o
  CC [M]  .module-common.o
  LD [M]  apml_sbrmi.ko
  BTF [M] apml_sbrmi.ko
Skipping BTF generation for apml_sbrmi.ko due to unavailability of vmlinux
  CC [M]  apml_sbtsi.mod.o
  LD [M]  apml_sbtsi.ko
  BTF [M] apml_sbtsi.ko
Skipping BTF generation for apml_sbtsi.ko due to unavailability of vmlinux
  CC [M]  apml_alertl.mod.o
  LD [M]  apml_alertl.ko
  BTF [M] apml_alertl.ko
Skipping BTF generation for apml_alertl.ko due to unavailability of vmlinux
make[2]: Leaving directory '/home/jvdillon/apml_modules'
make[1]: Leaving directory '/usr/src/linux-headers-6.14.0-29-generic'
sudo modprobe i3c

sudo insmod ./apml_sbrmi.ko
# insmod: ERROR: could not insert module ./apml_sbrmi.ko: Unknown symbol in module

sudo insmod ./apml_sbtsi.ko
# insmod: ERROR: could not insert module ./apml_sbtsi.ko: Unknown symbol in module

sudo dmesg -T|tail -2
# [Fri Sep 19 11:05:26 2025] apml_sbrmi: Unknown symbol __devm_regmap_init_i3c (err -2)
# [Fri Sep 19 11:05:26 2025] apml_sbtsi: Unknown symbol __devm_regmap_init_i3c (err -2)


regmap = devm_regmap_init_i3c(i3cdev, &sbtsi_i3c_regmap_config);
regmap = devm_regmap_init_i2c(i3cdev, &sbtsi_i3c_regmap_config);
if (IS_ERR(regmap)) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

why this change?

sbrmi.c Outdated

if (rmi_dev->i3cdev) {
regmap = devm_regmap_init_i3c(rmi_dev->i3cdev,
regmap = devm_regmap_init_i2c(rmi_dev->i3cdev,
Copy link
Collaborator

Choose a reason for hiding this comment

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

why this change?

}

static int alert_remove(struct platform_device *pdev)
static void alert_remove(struct platform_device *pdev)
Copy link
Collaborator

Choose a reason for hiding this comment

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

This change should be on basis of Linux kernel version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants