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