Re: [PATCH] clk: mvebu: armada-37xx-periph: Fix initialization for cpu clocks

Viresh Kumar
2019-04-24T09:30:26Z

On 18-03-19, 14:21, Ilias Apalodimas wrote:
> Hi Gregory,
>
> > >>
> > > [ 14.909524] clk_pm_cpu_set_parent old=28004840 -> val=0x6000000 load_level=0
> > > [ 14.916424] clk_pm_cpu_set_parent old=2E004840 -> val=0x600 load_level=1
> > > [ 14.923315] clk_pm_cpu_set_parent old=8880A8C0 -> val=0x6000000 load_level=2
> > > [ 14.930572] clk_pm_cpu_set_parent old=8E80A8C0 -> val=0x600 load_level=3
> > >
> > >
> > > Let me know if you need anything else
> >
> > Could you show me the output of "cat /sys/kernel/debug/clk/clk_summary"
> >
> root@localhost ~ # cat /sys/kernel/debug/clk/clk_summary
> enable prepare protect duty
> clock count count count rate accuracy phase cycle
> ---------------------------------------------------------------------------------------------
> xtal 2 3 0 25000000 0 0 50000
> i2c_1 0 0 0 25000000 0 0 50000
> i2c_2 0 0 0 25000000 0 0 50000
> avs 0 0 0 25000000 0 0 50000
> TBG-B-S 2 2 0 1000000000 0 0 50000
> cpu 0 0 0 200000000 0 0 50000
> usb32_ss_sys 1 1 0 125000000 0 0 50000
> usb32_usb2_sys 0 0 0 100000000 0 0 50000
> gbe_125 0 0 0 125000000 0 0 50000
> gbe0_125 0 0 0 125000000 0 0 50000
> gbe1_125 0 0 0 125000000 0 0 50000
> gbe_core 1 1 0 250000000 0 0 50000
> gbe_bm 0 0 0 250000000 0 0 50000
> gbe0_core 1 1 0 250000000 0 0 50000
> gbe1_core 0 0 0 250000000 0 0 50000
> eip97 0 0 0 500000000 0 0 50000
> trace 0 0 0 1000000000 0 0 50000
> ddr_fclk 0 0 0 125000000 0 0 50000
> pwm 0 0 0 50000000 0 0 50000
> tscem_tmx 0 0 0 1000000000 0 0 50000
> sec_dap 0 0 0 100000000 0 0 50000
> sec_at 0 0 0 200000000 0 0 50000
> TBG-A-S 0 0 0 1600000000 0 0 50000
> sdio 0 0 0 400000000 0 0 50000
> ddr_phy 0 0 0 400000000 0 0 50000
> mmc 0 0 0 400000000 0 0 50000
> TBG-B-P 0 0 0 500000000 0 0 50000
> gbe_50 0 0 0 50000000 0 0 50000
> gbe0_50 0 0 0 50000000 0 0 50000
> gbe1_50 0 0 0 50000000 0 0 50000
> TBG-A-P 0 1 0 800000000 0 0 50000
> counter 0 0 0 160000000 0 0 50000
> sqf 0 1 0 200000000 0 0 50000
> tscem 0 0 0 200000000 0 0 50000
> sata_host 0 0 0 200000000 0 0 50000
> root@localhost ~ #
>
> > Also, during this week-end, Christian suggested that the issue might
> > come from the AVS support.
> >
> > Could you disable it and check you still have the issue?
> >
> > For this, you just have to remove the avs node in
> > arch/arm64/boot/dts/marvell/armada-37xx.dtsi and rebuild the dtb.
> Sure. You'll have to wait for a week though. Currently on a trip. I'll run that
> once i return

@Ilias: Can you please try this now and confirm to Gregory ?

--
viresh