Re: [RFC PATCH] firmware: scmi: Make SCMI arch independent
Michal Simek <[email protected]> Fri, 10 Jul 2026 08:01:56 +0200
| Newsgroups | org.kernel.vger.arm-scmi,dev.linux.lists.imx,org.infradead.lists.linux-arm-kernel,org.kernel.vger.linux-clk,org.kernel.vger.linux-doc,org.kernel.vger.linux-gpio,org.kernel.vger.linux-hwmon,org.kernel.vger.linux-iio,org.kernel.vger.linux-kernel,org.kernel.vger.linux-pm |
|---|---|
| Message-ID | <[email protected]> |
On 7/9/26 18:07, Guenter Roeck wrote: > On Thu, Jul 09, 2026 at 03:27:00PM +0200, Michal Simek wrote: >> On heterogenious systems like AMD/Xilinx FPGA there is a need to talk to >> SCMI server from different architectures than ARM that's why remove >> ARM/ARM64 Kconfig dependency with also remove ARM from description and >> rename folder to reflect it. >> From specification perspective only ARM specific transport layers should be >> available on ARM/ARM64 architectures. >> >> That's why get rid of ARM prefix and description from documentation, file >> names, folder names, MODULE description, module names, Kconfig and >> comments. >> But keep origin Kconfig symbols not to break existing users. >> >> Hwmon, pinctrl, powercap, regulator, reset, clk and cpufreq scmi drivers >> already miss arm prefix that's why synchronize all of them to be without >> arm prefix. >> >> Signed-off-by: Michal Simek <[email protected]> >> --- >> >> Likely I missed other locations but I wanted to send this RFC to have a >> discussion about it. >> --- > ... >> drivers/hwmon/Kconfig | 2 +- > ... >> diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig >> index 2bfbcc033d59..cda0aedb36ca 100644 >> --- a/drivers/hwmon/Kconfig >> +++ b/drivers/hwmon/Kconfig >> @@ -401,7 +401,7 @@ config SENSORS_ARCTIC_FAN_CONTROLLER >> will be called arctic_fan_controller. >> >> config SENSORS_ARM_SCMI >> - tristate "ARM SCMI Sensors" >> + tristate "SCMI Sensors" >> depends on ARM_SCMI_PROTOCOL >> depends on THERMAL || !THERMAL_OF >> help > > I agree in principle, but I don't see why this would have to be done > in a single patch. Also, both the Kconfig file (SENSORS_ARM_SCMI, > ARM_SCMI_PROTOCOL) and the driver itself (""ARM SCMI HWMON interface > driver") still reference ARM. It should be split for sure. Just wanted to get initial feedback about this change to show that it targets multiple locations. Thanks, Michal