Re: [PATCH 1/4] firmware: arm_scmi: bus: Bypass setting fwnode for scmi cpufreq

Sudeep Holla
2025-02-18T10:29:35Z

On Tue, Feb 18, 2025 at 09:09:49AM +0800, Peng Fan wrote:
> A potential solution is not using reg in the protocol nodes. Define nodes
> as below:
> devperf {
> compatible ="arm,scmi-devperf";
> }
>
> cpuperf {
> compatible ="arm,scmi-cpuperf";
> }
>
> pinctrl {
> compatible ="arm,scmi-pinctrl";
> }
>
> The reg is coded in driver.
>
> But the upper requires restruction of scmi framework.
>
> Put the above away, could we first purse a simple way first to address
> the current bug in kernel? Just as I prototyped here:
> https://github.com/MrVan/linux/tree/b4/scmi-fwdevlink-v2
>

Good luck getting these bindings merged. I don't like it as it is pushing
software policy or issues into to the devicetree. What we have as SCMI
binding is more than required for a firmware interface IMO. So, you are
on your own to get these bindings approved as I am not on board with
these but if you convince DT maintainers, I will have a look at it then
to see if we can make that work really.

--
Regards,
Sudeep