On Mon 06 Nov 2023 at 10:26, Neil Armstrong <neil.armstrong@xxxxxxxxxx> wrote: > On 06/11/2023 10:22, Tomeu Vizoso wrote: patch title is misleading. It does not target the vim3 only but all g12 and sm1 HWs. It should start with "arm64: dts: amlogic: g12: " instead >> Otherwise they are left at 24MHz and the NPU runs very slowly. >> Signed-off-by: Tomeu Vizoso <tomeu@xxxxxxxxxxxxxxx> >> Suggested-by: Lucas Stach <l.stach@xxxxxxxxxxxxxx> >> --- >> arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi | 3 +++ >> 1 file changed, 3 insertions(+) >> diff --git a/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi >> b/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi >> index ff68b911b729..9d5eab6595d0 100644 >> --- a/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi >> +++ b/arch/arm64/boot/dts/amlogic/meson-g12-common.dtsi >> @@ -2502,6 +2502,9 @@ npu: npu@ff100000 { >> clocks = <&clkc CLKID_NNA_CORE_CLK>, >> <&clkc CLKID_NNA_AXI_CLK>; >> clock-names = "core", "bus"; >> + assigned-clocks = <&clkc CLKID_NNA_CORE_CLK>, >> + <&clkc CLKID_NNA_AXI_CLK>; >> + assigned-clock-rates = <800000000>, <800000000>; >> resets = <&reset RESET_NNA>; >> status = "disabled"; >> }; > > Reviewed-by: Neil Armstrong <neil.armstrong@xxxxxxxxxx>