Re: [PATCH v8 07/14] clk: en7523: Add clock driver for Airoha EN7523 SoC

From: Felix Fietkau
Date: Wed Dec 22 2021 - 10:35:18 EST


On 2021-12-21 15:06, Arnd Bergmann wrote:
On Mon, Dec 20, 2021 at 10:18 PM Felix Fietkau <nbd@xxxxxxxx> wrote:

This driver only registers fixed rate clocks, since the clocks are fully
initialized by the boot loader and should not be changed later, according
to Airoha.

Signed-off-by: Felix Fietkau <nbd@xxxxxxxx>

I don't have any particular comments on the driver itself. I'm happy
to merge it through
the soc tree for the initial support of the clk maintainers are ok
with it and have reviewed
the actual code.

drivers/clk/Kconfig | 9 +
drivers/clk/Makefile | 1 +
drivers/clk/clk-en7523.c | 350 +++++++++++++++++++++++++++++++++++++++
3 files changed, 360 insertions(+)
create mode 100644 drivers/clk/clk-en7523.c

I think this should go into drivers/clk/mediatek, as it probably shares
some of the logic with the mt762x drivers in there. I'd suggest adding the
maintainers for those drivers to Cc for additional review.
I don't think it shares any logic with the other drivers, but since Airoha is a subsidiary of Mediatek, I guess it makes sense to move it there.

- Felix