On Thu, Apr 9, 2020 at 8:44 AM Stephen Boyd <sboyd@xxxxxxxxxx> wrote:
This patch series cleans up a handful of selects that were redundant and deletes presumably dead code with the goal of making it possible to add kunit tests for the CCF in the future. To do that, we introduce a "legacy" clk Kconfig option to mark code that hasn't migrated to the common clk framework and then make the COMMON_CLK config option visible in the menuconfig as long as that legacy option isn't enabled. I've also included a couple patches at the end that may be more controversial but helped me consolidate all this logic/code. I haven't done more than compile test a few configs for arm, arm64, h8300, and mips. More testing is welcome. The plan is that I'll just merge the whole pile through the clk tree. If the first five patches or the last three patches are better going through another tree like arm-soc or architecture trees that's fine too, but there are potential conflicts between trees so maybe it's better to just leave it all in one tree. Changes from v1: * Fixed MIPS ralink build problem pointed out by Arnd * Fixed meson mx sdio build due to bad Kconfig exposed by this change * Picked up acks
Whole series Reviewed-by: Arnd Bergmann <arnd@xxxxxxxx> I also made a patch to completely remove the private clk implementation for mips/loongson2f, but it obviously clashes with your series at the moment. I'll send you what I have today and you can decide if you want to add it to your series after it gets an Ack from the maintainers, or I'll send it separately later. Arnd