Hi Brandon, On 12/15/23 09:17, Brandon Cheo Fusi wrote: > On Thu, Dec 14, 2023 at 12:17 PM Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote: >> >> On 14-12-23, 11:33, Brandon Cheo Fusi wrote: >>> Add Kconfig file for cpufreq scaling drivers that can handle RISC-V >>> CPUs. An entry is included for the Allwinner H6 cpufreq driver that >>> works with D1. >>> >>> Signed-off-by: Brandon Cheo Fusi <fusibrandon13@xxxxxxxxx> >>> --- >>> drivers/cpufreq/Kconfig | 4 ++++ >>> drivers/cpufreq/Kconfig.riscv | 16 ++++++++++++++++ >>> 2 files changed, 20 insertions(+) >>> create mode 100644 drivers/cpufreq/Kconfig.riscv >> >> We don't have a separate kconfig file for each architecture. Only if >> there are too many entries for an architecture, we add a new file. >> >> -- >> viresh > > The sun50i cpufreq driver is currently only available when CONFIG_ARM or > CONFIG_ARM64 is selected, so this was the only decent way I could think > of making it accessible on either one of CONFIG_(ARM | ARM64 | RISC-V). > Any suggestions for a better workaround ? Move the option to the main drivers/cpufreq/Kconfig, like QORIQ_CPUFREQ, which is also used with multiple architectures (PowerPC and ARM, in that case). We don't want two options for the same driver. Regards, Samuel
