Re: [RFC PATCH 6/6] rust: cpufreq: Add rust implementation of cppc_cpufreq driver

[Pierre Gondois]

Hello Greg, Miguel,

On 8/15/24 15:04, Miguel Ojeda wrote:
> On Thu, Aug 15, 2024 at 10:31 AM Pierre Gondois <pierre.gondois@xxxxxxx> wrote:
> > In an effort to add test/support the cpufreq framework in rust,
> > add a rust implementation of the cppc_cpufreq driver named:
> > `rcppc_cpufreq`.
>
> Similar to what Greg said -- is this intended to be something like a
> "Rust reference driver" [1] for the subsystem?

The initial intent was to review/test Viresh's patchset [1]. I then
thought it would be a good idea to implement another cpufreq driver
to see if the provided interface would work.
As the cpufreq-dt driver is re-implemented in Viresh's patchset,
I thought it was also ok to have this driver.

> [1] https://rust-for-linux.com/rust-reference-drivers
>
> > +       depends on ACPI_PROCESSOR
> > +       depends on ARM || ARM64 || RISCV

Yes right

> `depends on RUST`?
>
> Also, I imagine you skipped all safety comments etc. since it is an
> RFC, but I thought I would mention it nevertheless.

Ok, if it is decided not to drop this patchset, I ll add some comments,

> Thanks for experimenting with Rust!
>
> Cheers,
> Miguel


Regards,
Pierre