* [PATCH 0/2] AT91 Clock Adjustments
@ 2025-02-06 18:14 Ryan.Wanner
2025-02-06 18:14 ` [PATCH 1/2] clk: at91: sckc: Fix parent_data struct for slow osc Ryan.Wanner
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Ryan.Wanner @ 2025-02-06 18:14 UTC (permalink / raw)
To: mturquette, sboyd, nicolas.ferre, alexandre.belloni,
claudiu.beznea
Cc: linux-clk, linux-arm-kernel, linux-kernel, mripard, Ryan Wanner
From: Ryan Wanner <Ryan.Wanner@microchip.com>
This set has clock system adjustments for the AT91 clock system.
The first is to adjust the slow clock driver to account for the updated
DT node-naming for clocks and xtals and ensuring the driver is still
backwards compatible.
The second is a adding a missing clk_hw struct that is not added into
parent_data struct causing a incorrect parent for main_osc.
Ryan Wanner (2):
clk: at91: sckc: Fix parent_data struct for slow osc
clk: at91: sama7d65: Add missing clk_hw to parent_data
drivers/clk/at91/sama7d65.c | 1 +
drivers/clk/at91/sckc.c | 12 ++++++------
2 files changed, 7 insertions(+), 6 deletions(-)
--
2.43.0
^ permalink raw reply [flat|nested] 6+ messages in thread* [PATCH 1/2] clk: at91: sckc: Fix parent_data struct for slow osc 2025-02-06 18:14 [PATCH 0/2] AT91 Clock Adjustments Ryan.Wanner @ 2025-02-06 18:14 ` Ryan.Wanner 2025-02-06 18:14 ` [PATCH 2/2] clk: at91: sama7d65: Add missing clk_hw to parent_data Ryan.Wanner 2025-02-26 9:06 ` [PATCH 0/2] AT91 Clock Adjustments Claudiu Beznea 2 siblings, 0 replies; 6+ messages in thread From: Ryan.Wanner @ 2025-02-06 18:14 UTC (permalink / raw) To: mturquette, sboyd, nicolas.ferre, alexandre.belloni, claudiu.beznea Cc: linux-clk, linux-arm-kernel, linux-kernel, mripard, Ryan Wanner From: Ryan Wanner <Ryan.Wanner@microchip.com> The slow xtal is not described correctly as a parent, the driver looks for a "slow_xtal" string which is incorrect and will not work with the new formating of xtals. To avoid this and keep this driver backwards compatible the parent_data.fw_name is replaced with parent_data.name and the original parent_data.name is replaced with parent_data.index. Using the index is safe due to the driver requiring only 1 xtal. Fixes: 8aa1db9ccee0e ("clk: at91: sckc: switch to parent_data/parent_hw") Signed-off-by: Ryan Wanner <Ryan.Wanner@microchip.com> --- drivers/clk/at91/sckc.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/clk/at91/sckc.c b/drivers/clk/at91/sckc.c index 021d1b412af4c..952a805b6f7e1 100644 --- a/drivers/clk/at91/sckc.c +++ b/drivers/clk/at91/sckc.c @@ -374,7 +374,7 @@ static void __init at91sam9x5_sckc_register(struct device_node *np, const char *xtal_name; struct clk_hw *slow_rc, *slow_osc, *slowck; static struct clk_parent_data parent_data = { - .name = "slow_xtal", + .index = 0, }; const struct clk_hw *parent_hws[2]; bool bypass; @@ -407,7 +407,7 @@ static void __init at91sam9x5_sckc_register(struct device_node *np, if (!xtal_name) goto unregister_slow_rc; - parent_data.fw_name = xtal_name; + parent_data.name = xtal_name; slow_osc = at91_clk_register_slow_osc(regbase, "slow_osc", &parent_data, 1200000, bypass, bits); @@ -476,7 +476,7 @@ static void __init of_sam9x60_sckc_setup(struct device_node *np) const char *xtal_name; const struct clk_hw *parent_hws[2]; static struct clk_parent_data parent_data = { - .name = "slow_xtal", + .index = 0, }; bool bypass; int ret; @@ -494,7 +494,7 @@ static void __init of_sam9x60_sckc_setup(struct device_node *np) if (!xtal_name) goto unregister_slow_rc; - parent_data.fw_name = xtal_name; + parent_data.name = xtal_name; bypass = of_property_read_bool(np, "atmel,osc-bypass"); slow_osc = at91_clk_register_slow_osc(regbase, "slow_osc", &parent_data, 5000000, bypass, @@ -592,7 +592,7 @@ static void __init of_sama5d4_sckc_setup(struct device_node *np) const char *xtal_name; const struct clk_hw *parent_hws[2]; static struct clk_parent_data parent_data = { - .name = "slow_xtal", + .index = 0, }; int ret; @@ -609,7 +609,7 @@ static void __init of_sama5d4_sckc_setup(struct device_node *np) xtal_name = of_clk_get_parent_name(np, 0); if (!xtal_name) goto unregister_slow_rc; - parent_data.fw_name = xtal_name; + parent_data.name = xtal_name; osc = kzalloc(sizeof(*osc), GFP_KERNEL); if (!osc) -- 2.43.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/2] clk: at91: sama7d65: Add missing clk_hw to parent_data 2025-02-06 18:14 [PATCH 0/2] AT91 Clock Adjustments Ryan.Wanner 2025-02-06 18:14 ` [PATCH 1/2] clk: at91: sckc: Fix parent_data struct for slow osc Ryan.Wanner @ 2025-02-06 18:14 ` Ryan.Wanner 2025-02-26 9:06 ` [PATCH 0/2] AT91 Clock Adjustments Claudiu Beznea 2 siblings, 0 replies; 6+ messages in thread From: Ryan.Wanner @ 2025-02-06 18:14 UTC (permalink / raw) To: mturquette, sboyd, nicolas.ferre, alexandre.belloni, claudiu.beznea Cc: linux-clk, linux-arm-kernel, linux-kernel, mripard, Ryan Wanner From: Ryan Wanner <Ryan.Wanner@microchip.com> The main_xtal clk_hw struct is not passed into parent_data.hw causing an issue with main_osc parent. Passing the main_xtal struct into the parent_data struct will ensure the correct parent structure. Signed-off-by: Ryan Wanner <Ryan.Wanner@microchip.com> --- drivers/clk/at91/sama7d65.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/clk/at91/sama7d65.c b/drivers/clk/at91/sama7d65.c index a5d40df8b2f27..08306261c9c7e 100644 --- a/drivers/clk/at91/sama7d65.c +++ b/drivers/clk/at91/sama7d65.c @@ -1138,6 +1138,7 @@ static void __init sama7d65_pmc_setup(struct device_node *np) parent_data.name = main_xtal_name; parent_data.fw_name = main_xtal_name; + parent_data.hw = main_xtal_hw; main_osc_hw = at91_clk_register_main_osc(regmap, "main_osc", NULL, &parent_data, bypass); if (IS_ERR(main_osc_hw)) -- 2.43.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 0/2] AT91 Clock Adjustments 2025-02-06 18:14 [PATCH 0/2] AT91 Clock Adjustments Ryan.Wanner 2025-02-06 18:14 ` [PATCH 1/2] clk: at91: sckc: Fix parent_data struct for slow osc Ryan.Wanner 2025-02-06 18:14 ` [PATCH 2/2] clk: at91: sama7d65: Add missing clk_hw to parent_data Ryan.Wanner @ 2025-02-26 9:06 ` Claudiu Beznea [not found] ` <b76108c1-3282-4b98-a210-b350e8ec0ec2@microchip.com> 2 siblings, 1 reply; 6+ messages in thread From: Claudiu Beznea @ 2025-02-26 9:06 UTC (permalink / raw) To: Ryan.Wanner, mturquette, sboyd, nicolas.ferre, alexandre.belloni Cc: linux-clk, linux-arm-kernel, linux-kernel, mripard On 06.02.2025 20:14, Ryan.Wanner@microchip.com wrote: > From: Ryan Wanner <Ryan.Wanner@microchip.com> > > This set has clock system adjustments for the AT91 clock system. > > The first is to adjust the slow clock driver to account for the updated > DT node-naming for clocks and xtals and ensuring the driver is still > backwards compatible. > > The second is a adding a missing clk_hw struct that is not added into > parent_data struct causing a incorrect parent for main_osc. > > Ryan Wanner (2): > clk: at91: sckc: Fix parent_data struct for slow osc > clk: at91: sama7d65: Add missing clk_hw to parent_data > > drivers/clk/at91/sama7d65.c | 1 + > drivers/clk/at91/sckc.c | 12 ++++++------ > 2 files changed, 7 insertions(+), 6 deletions(-) > There are boot failures on sama7g5 with this series on top of at91-next and the following diff on sama7g5 dts to mimic the sama7d65 use case: diff --git a/arch/arm/boot/dts/microchip/at91-sama7g5ek.dts b/arch/arm/boot/dts/microchip/at91-sama7g5ek.dts index 0f5e6ad438dd..1bce9f999431 100644 --- a/arch/arm/boot/dts/microchip/at91-sama7g5ek.dts +++ b/arch/arm/boot/dts/microchip/at91-sama7g5ek.dts @@ -35,16 +35,6 @@ aliases { i2c2 = &i2c9; }; - clocks { - slow_xtal { - clock-frequency = <32768>; - }; - - main_xtal { - clock-frequency = <24000000>; - }; - }; - gpio-keys { compatible = "gpio-keys"; @@ -132,6 +122,15 @@ spdif_out: spdif-out { }; }; +&slow_xtal { + clock-frequency = <32768>; +}; + +&main_xtal { + clock-frequency = <24000000>; +}; + + &adc { vddana-supply = <&vddout25>; vref-supply = <&vddout25>; diff --git a/arch/arm/boot/dts/microchip/sama7g5.dtsi b/arch/arm/boot/dts/microchip/sama7g5.dtsi index 17bcdcf0cf4a..3158c2f0b4c0 100644 --- a/arch/arm/boot/dts/microchip/sama7g5.dtsi +++ b/arch/arm/boot/dts/microchip/sama7g5.dtsi @@ -117,12 +117,12 @@ map1 { }; clocks { - slow_xtal: slow_xtal { + slow_xtal: clk-slowxtal { compatible = "fixed-clock"; #clock-cells = <0>; }; - main_xtal: main_xtal { + main_xtal: clk-mainxtal { compatible = "fixed-clock"; #clock-cells = <0>; }; I've identified the following: cpu cpu0: _opp_config_clk_single: failed to set clock rate: -22 cpufreq: __target_index: Failed to change cpu frequency: -22 ------------[ cut here ]------------ kernel BUG at drivers/cpufreq/cpufreq.c:1523! Internal error: Oops - BUG: 0 [#1] ARM Modules linked in: CPU: 0 UID: 0 PID: 1 Comm: swapper Not tainted 6.14.0-rc1+ #11 Hardware name: Microchip SAMA7 PC is at cpufreq_online+0x8d8/0xa30 LR is at __wake_up+0x20/0x30 pc : [<c063cca0>] lr : [<c0151254>] psr: a0000013 sp : e0821d30 ip : 00000000 fp : c0e50508 r10: c434830c r9 : c4348204 r8 : c0e97230 r7 : c4348264 r6 : 00000000 r5 : 000ca511 r4 : c4348200 r3 : 047e0d67 r2 : 047e0d67 r1 : 00000003 r0 : ffffffea Flags: NzCv IRQs on FIQs on Mode SVC_32 ISA ARM Segment none Control: 10c53c7d Table: 60004059 DAC: 00000051 Register r0 information: non-paged memory Register r1 information: non-paged memory Register r2 information: non-paged memory Register r3 information: non-paged memory Register r4 information: slab kmalloc-512 start c4348200 pointer offset 0 size 512 Register r5 information: non-paged memory Register r6 information: NULL pointer Register r7 information: slab kmalloc-512 start c4348200 pointer offset 100 size 512 Register r8 information: non-slab/vmalloc memory Register r9 information: slab kmalloc-512 start c4348200 pointer offset 4 size 512 Register r10 information: slab kmalloc-512 start c4348200 pointer offset 268 size 512 Register r11 information: non-slab/vmalloc memory Register r12 information: NULL pointer Process swapper (pid: 1, stack limit = 0x08d7ca6b) Stack: (0xe0821d30 to 0xe0822000) 1d20: 00000000 e0821d98 c4348208 c0c00250 1d40: 00000001 00000000 e0821d98 c0e50508 c4048000 00000000 c406f480 c4048000 1d60: c4341ec0 c0e50508 c4341ec8 c063ce8c c0e593a8 c4048000 c406f4bc c406f480 1d80: c4048000 c04c8bb0 c406f4cc c40704b0 00000000 047e0d67 3b9aca03 c0e5986c 1da0: c0e5931c c0e97230 00000000 c063a618 00000000 c4048000 c0e59858 c40d0a10 1dc0: c0e597f0 c063f80c c0d31830 00000000 00000000 00000000 00000000 00000000 1de0: 00000000 00000000 e0821df4 00000000 00000000 c0ba32fc 00000000 047e0d67 1e00: c40d75d8 00000000 c40d0a10 c0e59804 00000000 00000000 c0d31830 c0c2b5a0 1e20: c0c776b0 c04cc8dc c40d0a10 00000000 c0e59804 c04ca1f4 c40d0a10 c0e59804 1e40: c40d0a10 00000000 00000000 c04ca47c 00000000 c0be8598 c0e94b28 c0e59804 1e60: c40d0a10 00000000 00000000 c04ca658 c40d0a10 c0e59804 c40d0a54 c4048000 1e80: 00000000 c04ca8c4 c0e59804 c04ca834 c406f3c0 c04c8494 c4048000 c406f40c 1ea0: c4070d30 047e0d67 c0e59804 c4341e00 00000000 c406f3c0 c4341e34 c04c959c 1ec0: c0c00c14 c0e6c000 c0e59804 c0d2113c c4070800 00000000 c0e6c000 c04cb598 1ee0: c4048000 c0d2113c c4070800 c010d21c 000003c0 00000000 c407084e c4070800 1f00: c407084d c013e9e0 c0bee764 000000c0 00000000 00000000 c0d004d0 c4048000 1f20: 00000006 00000006 373126cc c407086a 00000000 047e0d67 00000012 047e0d67 1f40: 00000007 c0d42bd8 00000007 c4070800 000000c0 c0d31850 c0d31830 c0d011f4 1f60: 00000006 00000006 00000000 c0d004d0 e0821f6c c0d004d0 00000000 c0e03380 1f80: c0905828 00000000 00000000 00000000 00000000 00000000 00000000 c0905844 1fa0: 00000000 c0905828 00000000 c010014c 00000000 00000000 00000000 00000000 1fc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 1fe0: 00000000 00000000 00000000 00000000 00000013 00000000 00000000 00000000 Call trace: cpufreq_online from cpufreq_add_dev+0x84/0x94 cpufreq_add_dev from subsys_interface_register+0xf0/0x108 subsys_interface_register from cpufreq_register_driver+0x13c/0x2e4 cpufreq_register_driver from dt_cpufreq_probe+0xb4/0x380 dt_cpufreq_probe from platform_probe+0x5c/0xb8 platform_probe from really_probe+0xc8/0x2c8 really_probe from __driver_probe_device+0x88/0x19c __driver_probe_device from driver_probe_device+0x30/0x104 driver_probe_device from __driver_attach+0x90/0x178 __driver_attach from bus_for_each_dev+0x6c/0xb4 bus_for_each_dev from bus_add_driver+0xcc/0x1ec bus_add_driver from driver_register+0x7c/0x114 driver_register from do_one_initcall+0x40/0x21c do_one_initcall from kernel_init_freeable+0x194/0x1f8 kernel_init_freeable from kernel_init+0x1c/0x128 kernel_init from ret_from_fork+0x14/0x28 Exception stack(0xe0821fb0 to 0xe0821ff8) 1fa0: 00000000 00000000 00000000 00000000 1fc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 1fe0: 00000000 00000000 00000000 00000000 00000013 00000000 Code: e3a02000 ebfff87e e3500000 0a000043 (e7f001f2) ---[ end trace 0000000000000000 ]--- note: swapper[1] exited with irqs disabled Kernel panic - not syncing: Attempted to kill init! exitcode=0x0000000b ---[ end Kernel panic - not syncing: Attempted to kill init! exitcode=0x0000000b ]--- And this one: Starting kernel ... Booting Linux on physical CPU 0x0 Linux version 6.14.0-rc1+ (claudiu@claudiu-X670E-Pro-RS) (arm-none-linux-gnueabihf-gcc (GNU Toolchain for the A-profile Architecture 10.3-2021.07 (arm-10.29)) 10.3.1 20210621, GNU ld (GNU Toolchain for the A-profile Architecture 10.3-2021.07 (arm-10.29)) 2.36.1.20210621) #11 Wed Feb 26 11:01:10 EET 2025 CPU: ARMv7 Processor [410fc075] revision 5 (ARMv7), cr=10c53c7d CPU: PIPT / VIPT nonaliasing data cache, VIPT aliasing instruction cache OF: fdt: Machine model: Microchip SAMA7G5-EK printk: legacy bootconsole [earlycon0] enabled Memory policy: Data cache writeback cma: Reserved 192 MiB at 0x70000000 on node -1 Zone ranges: Normal [mem 0x0000000060000000-0x000000007fffffff] Movable zone start for each node Early memory node ranges node 0: [mem 0x0000000060000000-0x000000007fffffff] Initmem setup node 0 [mem 0x0000000060000000-0x000000007fffffff] OF: reserved mem: Reserved memory: No reserved-memory node in the DT CPU: All CPU(s) started in SVC mode. Kernel command line: console=ttyS0,115200 root=/dev/mmcblk1p2 rootfstype=ext4 rw rootwait cma=192m atmel.pm_modes=standby,ulp0 earlyprintk printk: log buffer data + meta data: 65536 + 204800 = 270336 bytes Dentry cache hash table entries: 65536 (order: 6, 262144 bytes, linear) Inode-cache hash table entries: 32768 (order: 5, 131072 bytes, linear) Built 1 zonelists, mobility grouping on. Total pages: 131072 mem auto-init: stack:off, heap alloc:off, heap free:off SLUB: HWalign=64, Order=0-3, MinObjects=0, CPUs=1, Nodes=1 NR_IRQS: 16, nr_irqs: 16, preallocated irqs: 16 Main crystal frequency not set, using approximate value timer_probe: no matching timers found Console: colour dummy device 80x30 sched_clock: 32 bits at 100 Hz, resolution 10000000ns, wraps every 21474836475000000ns I suspect parent_data in sama7g5_pmc_setup() need to be initialized with zero. Thank you, Claudiu ^ permalink raw reply related [flat|nested] 6+ messages in thread
[parent not found: <b76108c1-3282-4b98-a210-b350e8ec0ec2@microchip.com>]
* Re: [PATCH 0/2] AT91 Clock Adjustments [not found] ` <b76108c1-3282-4b98-a210-b350e8ec0ec2@microchip.com> @ 2025-03-18 17:33 ` Ryan Wanner 2025-03-19 17:50 ` Claudiu Beznea 0 siblings, 1 reply; 6+ messages in thread From: Ryan Wanner @ 2025-03-18 17:33 UTC (permalink / raw) To: Claudiu Beznea, mturquette, sboyd, nicolas.ferre, alexandre.belloni Cc: linux-clk, linux-arm-kernel, linux-kernel, mripard On 2/26/25 08:36, Ryan wrote: > On 2/26/25 02:06, Claudiu Beznea wrote: >> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe >> >> On 06.02.2025 20:14, Ryan.Wanner@microchip.com wrote: >>> From: Ryan Wanner <Ryan.Wanner@microchip.com> >>> >>> This set has clock system adjustments for the AT91 clock system. >>> >>> The first is to adjust the slow clock driver to account for the updated >>> DT node-naming for clocks and xtals and ensuring the driver is still >>> backwards compatible. >>> >>> The second is a adding a missing clk_hw struct that is not added into >>> parent_data struct causing a incorrect parent for main_osc. >>> >>> Ryan Wanner (2): >>> clk: at91: sckc: Fix parent_data struct for slow osc >>> clk: at91: sama7d65: Add missing clk_hw to parent_data >>> >>> drivers/clk/at91/sama7d65.c | 1 + >>> drivers/clk/at91/sckc.c | 12 ++++++------ >>> 2 files changed, 7 insertions(+), 6 deletions(-) >>> >> >> There are boot failures on sama7g5 with this series on top of at91-next and >> the following diff on sama7g5 dts to mimic the sama7d65 use case: >> >> diff --git a/arch/arm/boot/dts/microchip/at91-sama7g5ek.dts >> b/arch/arm/boot/dts/microchip/at91-sama7g5ek.dts >> index 0f5e6ad438dd..1bce9f999431 100644 >> --- a/arch/arm/boot/dts/microchip/at91-sama7g5ek.dts >> +++ b/arch/arm/boot/dts/microchip/at91-sama7g5ek.dts >> @@ -35,16 +35,6 @@ aliases { >> i2c2 = &i2c9; >> }; >> >> - clocks { >> - slow_xtal { >> - clock-frequency = <32768>; >> - }; >> - >> - main_xtal { >> - clock-frequency = <24000000>; >> - }; >> - }; >> - >> gpio-keys { >> compatible = "gpio-keys"; >> >> @@ -132,6 +122,15 @@ spdif_out: spdif-out { >> }; >> }; >> >> +&slow_xtal { >> + clock-frequency = <32768>; >> +}; >> + >> +&main_xtal { >> + clock-frequency = <24000000>; >> +}; >> + >> + >> &adc { >> vddana-supply = <&vddout25>; >> vref-supply = <&vddout25>; >> diff --git a/arch/arm/boot/dts/microchip/sama7g5.dtsi >> b/arch/arm/boot/dts/microchip/sama7g5.dtsi >> index 17bcdcf0cf4a..3158c2f0b4c0 100644 >> --- a/arch/arm/boot/dts/microchip/sama7g5.dtsi >> +++ b/arch/arm/boot/dts/microchip/sama7g5.dtsi >> @@ -117,12 +117,12 @@ map1 { >> }; >> >> clocks { >> - slow_xtal: slow_xtal { >> + slow_xtal: clk-slowxtal { >> compatible = "fixed-clock"; >> #clock-cells = <0>; >> }; >> >> - main_xtal: main_xtal { >> + main_xtal: clk-mainxtal { >> compatible = "fixed-clock"; >> #clock-cells = <0>; >> }; >> >> I've identified the following: >> >> >> cpu cpu0: _opp_config_clk_single: failed to set clock rate: -22 >> cpufreq: __target_index: Failed to change cpu frequency: -22 >> ------------[ cut here ]------------ >> kernel BUG at drivers/cpufreq/cpufreq.c:1523! >> Internal error: Oops - BUG: 0 [#1] ARM >> Modules linked in: >> CPU: 0 UID: 0 PID: 1 Comm: swapper Not tainted 6.14.0-rc1+ #11 >> Hardware name: Microchip SAMA7 >> PC is at cpufreq_online+0x8d8/0xa30 >> LR is at __wake_up+0x20/0x30 >> pc : [<c063cca0>] lr : [<c0151254>] psr: a0000013 >> sp : e0821d30 ip : 00000000 fp : c0e50508 >> r10: c434830c r9 : c4348204 r8 : c0e97230 >> r7 : c4348264 r6 : 00000000 r5 : 000ca511 r4 : c4348200 >> r3 : 047e0d67 r2 : 047e0d67 r1 : 00000003 r0 : ffffffea >> Flags: NzCv IRQs on FIQs on Mode SVC_32 ISA ARM Segment none >> Control: 10c53c7d Table: 60004059 DAC: 00000051 >> Register r0 information: non-paged memory >> Register r1 information: non-paged memory >> Register r2 information: non-paged memory >> Register r3 information: non-paged memory >> Register r4 information: slab kmalloc-512 start c4348200 pointer offset 0 >> size 512 >> Register r5 information: non-paged memory >> Register r6 information: NULL pointer >> Register r7 information: slab kmalloc-512 start c4348200 pointer offset 100 >> size 512 >> Register r8 information: non-slab/vmalloc memory >> Register r9 information: slab kmalloc-512 start c4348200 pointer offset 4 >> size 512 >> Register r10 information: slab kmalloc-512 start c4348200 pointer offset >> 268 size 512 >> Register r11 information: non-slab/vmalloc memory >> Register r12 information: NULL pointer >> Process swapper (pid: 1, stack limit = 0x08d7ca6b) >> Stack: (0xe0821d30 to 0xe0822000) >> 1d20: 00000000 e0821d98 c4348208 c0c00250 >> 1d40: 00000001 00000000 e0821d98 c0e50508 c4048000 00000000 c406f480 c4048000 >> 1d60: c4341ec0 c0e50508 c4341ec8 c063ce8c c0e593a8 c4048000 c406f4bc c406f480 >> 1d80: c4048000 c04c8bb0 c406f4cc c40704b0 00000000 047e0d67 3b9aca03 c0e5986c >> 1da0: c0e5931c c0e97230 00000000 c063a618 00000000 c4048000 c0e59858 c40d0a10 >> 1dc0: c0e597f0 c063f80c c0d31830 00000000 00000000 00000000 00000000 00000000 >> 1de0: 00000000 00000000 e0821df4 00000000 00000000 c0ba32fc 00000000 047e0d67 >> 1e00: c40d75d8 00000000 c40d0a10 c0e59804 00000000 00000000 c0d31830 c0c2b5a0 >> 1e20: c0c776b0 c04cc8dc c40d0a10 00000000 c0e59804 c04ca1f4 c40d0a10 c0e59804 >> 1e40: c40d0a10 00000000 00000000 c04ca47c 00000000 c0be8598 c0e94b28 c0e59804 >> 1e60: c40d0a10 00000000 00000000 c04ca658 c40d0a10 c0e59804 c40d0a54 c4048000 >> 1e80: 00000000 c04ca8c4 c0e59804 c04ca834 c406f3c0 c04c8494 c4048000 c406f40c >> 1ea0: c4070d30 047e0d67 c0e59804 c4341e00 00000000 c406f3c0 c4341e34 c04c959c >> 1ec0: c0c00c14 c0e6c000 c0e59804 c0d2113c c4070800 00000000 c0e6c000 c04cb598 >> 1ee0: c4048000 c0d2113c c4070800 c010d21c 000003c0 00000000 c407084e c4070800 >> 1f00: c407084d c013e9e0 c0bee764 000000c0 00000000 00000000 c0d004d0 c4048000 >> 1f20: 00000006 00000006 373126cc c407086a 00000000 047e0d67 00000012 047e0d67 >> 1f40: 00000007 c0d42bd8 00000007 c4070800 000000c0 c0d31850 c0d31830 c0d011f4 >> 1f60: 00000006 00000006 00000000 c0d004d0 e0821f6c c0d004d0 00000000 c0e03380 >> 1f80: c0905828 00000000 00000000 00000000 00000000 00000000 00000000 c0905844 >> 1fa0: 00000000 c0905828 00000000 c010014c 00000000 00000000 00000000 00000000 >> 1fc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 >> 1fe0: 00000000 00000000 00000000 00000000 00000013 00000000 00000000 00000000 >> Call trace: >> cpufreq_online from cpufreq_add_dev+0x84/0x94 >> cpufreq_add_dev from subsys_interface_register+0xf0/0x108 >> subsys_interface_register from cpufreq_register_driver+0x13c/0x2e4 >> cpufreq_register_driver from dt_cpufreq_probe+0xb4/0x380 >> dt_cpufreq_probe from platform_probe+0x5c/0xb8 >> platform_probe from really_probe+0xc8/0x2c8 >> really_probe from __driver_probe_device+0x88/0x19c >> __driver_probe_device from driver_probe_device+0x30/0x104 >> driver_probe_device from __driver_attach+0x90/0x178 >> __driver_attach from bus_for_each_dev+0x6c/0xb4 >> bus_for_each_dev from bus_add_driver+0xcc/0x1ec >> bus_add_driver from driver_register+0x7c/0x114 >> driver_register from do_one_initcall+0x40/0x21c >> do_one_initcall from kernel_init_freeable+0x194/0x1f8 >> kernel_init_freeable from kernel_init+0x1c/0x128 >> kernel_init from ret_from_fork+0x14/0x28 >> Exception stack(0xe0821fb0 to 0xe0821ff8) >> 1fa0: 00000000 00000000 00000000 00000000 >> 1fc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 >> 1fe0: 00000000 00000000 00000000 00000000 00000013 00000000 >> Code: e3a02000 ebfff87e e3500000 0a000043 (e7f001f2) >> ---[ end trace 0000000000000000 ]--- >> note: swapper[1] exited with irqs disabled >> Kernel panic - not syncing: Attempted to kill init! exitcode=0x0000000b >> ---[ end Kernel panic - not syncing: Attempted to kill init! >> exitcode=0x0000000b ]--- >> >> >> And this one: >> >> Starting kernel ... >> >> Booting Linux on physical CPU 0x0 >> Linux version 6.14.0-rc1+ (claudiu@claudiu-X670E-Pro-RS) >> (arm-none-linux-gnueabihf-gcc (GNU Toolchain for the A-profile Architecture >> 10.3-2021.07 (arm-10.29)) 10.3.1 20210621, GNU ld (GNU Toolchain for the >> A-profile Architecture 10.3-2021.07 (arm-10.29)) 2.36.1.20210621) #11 Wed >> Feb 26 11:01:10 EET 2025 >> CPU: ARMv7 Processor [410fc075] revision 5 (ARMv7), cr=10c53c7d >> CPU: PIPT / VIPT nonaliasing data cache, VIPT aliasing instruction cache >> OF: fdt: Machine model: Microchip SAMA7G5-EK >> printk: legacy bootconsole [earlycon0] enabled >> Memory policy: Data cache writeback >> cma: Reserved 192 MiB at 0x70000000 on node -1 >> Zone ranges: >> Normal [mem 0x0000000060000000-0x000000007fffffff] >> Movable zone start for each node >> Early memory node ranges >> node 0: [mem 0x0000000060000000-0x000000007fffffff] >> Initmem setup node 0 [mem 0x0000000060000000-0x000000007fffffff] >> OF: reserved mem: Reserved memory: No reserved-memory node in the DT >> CPU: All CPU(s) started in SVC mode. >> Kernel command line: console=ttyS0,115200 root=/dev/mmcblk1p2 >> rootfstype=ext4 rw rootwait cma=192m atmel.pm_modes=standby,ulp0 earlyprintk >> printk: log buffer data + meta data: 65536 + 204800 = 270336 bytes >> Dentry cache hash table entries: 65536 (order: 6, 262144 bytes, linear) >> Inode-cache hash table entries: 32768 (order: 5, 131072 bytes, linear) >> Built 1 zonelists, mobility grouping on. Total pages: 131072 >> mem auto-init: stack:off, heap alloc:off, heap free:off >> SLUB: HWalign=64, Order=0-3, MinObjects=0, CPUs=1, Nodes=1 >> NR_IRQS: 16, nr_irqs: 16, preallocated irqs: 16 >> Main crystal frequency not set, using approximate value >> timer_probe: no matching timers found >> Console: colour dummy device 80x30 >> sched_clock: 32 bits at 100 Hz, resolution 10000000ns, wraps every >> 21474836475000000ns >> >> >> I suspect parent_data in sama7g5_pmc_setup() need to be initialized with zero. > > With this series I did not see any boot issues with the sama7g54. But > when doing the DTS and DTSI changes that you made above I did see boot > issues with the sama7g54. > > It seems that the 2/2 patch also needs to have a similar patch for the > sama7g54 clock driver. Would it be better to have this change to all the SoCs and their clock drivers to match the correct naming? Since this set is mainly to adjust for sama7d65 and a separate change to sama7g5.c clock driver would be needed to fix the error that you are seeing above. > > Best, > Ryan >> >> Thank you, >> Claudiu >> >> >> > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/2] AT91 Clock Adjustments 2025-03-18 17:33 ` Ryan Wanner @ 2025-03-19 17:50 ` Claudiu Beznea 0 siblings, 0 replies; 6+ messages in thread From: Claudiu Beznea @ 2025-03-19 17:50 UTC (permalink / raw) To: Ryan Wanner, mturquette, sboyd, nicolas.ferre, alexandre.belloni Cc: linux-clk, linux-arm-kernel, linux-kernel, mripard On 18.03.2025 19:33, Ryan Wanner wrote: > On 2/26/25 08:36, Ryan wrote: >> On 2/26/25 02:06, Claudiu Beznea wrote: >>> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe >>> >>> On 06.02.2025 20:14, Ryan.Wanner@microchip.com wrote: >>>> From: Ryan Wanner <Ryan.Wanner@microchip.com> >>>> >>>> This set has clock system adjustments for the AT91 clock system. >>>> >>>> The first is to adjust the slow clock driver to account for the updated >>>> DT node-naming for clocks and xtals and ensuring the driver is still >>>> backwards compatible. >>>> >>>> The second is a adding a missing clk_hw struct that is not added into >>>> parent_data struct causing a incorrect parent for main_osc. >>>> >>>> Ryan Wanner (2): >>>> clk: at91: sckc: Fix parent_data struct for slow osc >>>> clk: at91: sama7d65: Add missing clk_hw to parent_data >>>> >>>> drivers/clk/at91/sama7d65.c | 1 + >>>> drivers/clk/at91/sckc.c | 12 ++++++------ >>>> 2 files changed, 7 insertions(+), 6 deletions(-) >>>> >>> >>> There are boot failures on sama7g5 with this series on top of at91-next and >>> the following diff on sama7g5 dts to mimic the sama7d65 use case: >>> >>> diff --git a/arch/arm/boot/dts/microchip/at91-sama7g5ek.dts >>> b/arch/arm/boot/dts/microchip/at91-sama7g5ek.dts >>> index 0f5e6ad438dd..1bce9f999431 100644 >>> --- a/arch/arm/boot/dts/microchip/at91-sama7g5ek.dts >>> +++ b/arch/arm/boot/dts/microchip/at91-sama7g5ek.dts >>> @@ -35,16 +35,6 @@ aliases { >>> i2c2 = &i2c9; >>> }; >>> >>> - clocks { >>> - slow_xtal { >>> - clock-frequency = <32768>; >>> - }; >>> - >>> - main_xtal { >>> - clock-frequency = <24000000>; >>> - }; >>> - }; >>> - >>> gpio-keys { >>> compatible = "gpio-keys"; >>> >>> @@ -132,6 +122,15 @@ spdif_out: spdif-out { >>> }; >>> }; >>> >>> +&slow_xtal { >>> + clock-frequency = <32768>; >>> +}; >>> + >>> +&main_xtal { >>> + clock-frequency = <24000000>; >>> +}; >>> + >>> + >>> &adc { >>> vddana-supply = <&vddout25>; >>> vref-supply = <&vddout25>; >>> diff --git a/arch/arm/boot/dts/microchip/sama7g5.dtsi >>> b/arch/arm/boot/dts/microchip/sama7g5.dtsi >>> index 17bcdcf0cf4a..3158c2f0b4c0 100644 >>> --- a/arch/arm/boot/dts/microchip/sama7g5.dtsi >>> +++ b/arch/arm/boot/dts/microchip/sama7g5.dtsi >>> @@ -117,12 +117,12 @@ map1 { >>> }; >>> >>> clocks { >>> - slow_xtal: slow_xtal { >>> + slow_xtal: clk-slowxtal { >>> compatible = "fixed-clock"; >>> #clock-cells = <0>; >>> }; >>> >>> - main_xtal: main_xtal { >>> + main_xtal: clk-mainxtal { >>> compatible = "fixed-clock"; >>> #clock-cells = <0>; >>> }; >>> >>> I've identified the following: >>> >>> >>> cpu cpu0: _opp_config_clk_single: failed to set clock rate: -22 >>> cpufreq: __target_index: Failed to change cpu frequency: -22 >>> ------------[ cut here ]------------ >>> kernel BUG at drivers/cpufreq/cpufreq.c:1523! >>> Internal error: Oops - BUG: 0 [#1] ARM >>> Modules linked in: >>> CPU: 0 UID: 0 PID: 1 Comm: swapper Not tainted 6.14.0-rc1+ #11 >>> Hardware name: Microchip SAMA7 >>> PC is at cpufreq_online+0x8d8/0xa30 >>> LR is at __wake_up+0x20/0x30 >>> pc : [<c063cca0>] lr : [<c0151254>] psr: a0000013 >>> sp : e0821d30 ip : 00000000 fp : c0e50508 >>> r10: c434830c r9 : c4348204 r8 : c0e97230 >>> r7 : c4348264 r6 : 00000000 r5 : 000ca511 r4 : c4348200 >>> r3 : 047e0d67 r2 : 047e0d67 r1 : 00000003 r0 : ffffffea >>> Flags: NzCv IRQs on FIQs on Mode SVC_32 ISA ARM Segment none >>> Control: 10c53c7d Table: 60004059 DAC: 00000051 >>> Register r0 information: non-paged memory >>> Register r1 information: non-paged memory >>> Register r2 information: non-paged memory >>> Register r3 information: non-paged memory >>> Register r4 information: slab kmalloc-512 start c4348200 pointer offset 0 >>> size 512 >>> Register r5 information: non-paged memory >>> Register r6 information: NULL pointer >>> Register r7 information: slab kmalloc-512 start c4348200 pointer offset 100 >>> size 512 >>> Register r8 information: non-slab/vmalloc memory >>> Register r9 information: slab kmalloc-512 start c4348200 pointer offset 4 >>> size 512 >>> Register r10 information: slab kmalloc-512 start c4348200 pointer offset >>> 268 size 512 >>> Register r11 information: non-slab/vmalloc memory >>> Register r12 information: NULL pointer >>> Process swapper (pid: 1, stack limit = 0x08d7ca6b) >>> Stack: (0xe0821d30 to 0xe0822000) >>> 1d20: 00000000 e0821d98 c4348208 c0c00250 >>> 1d40: 00000001 00000000 e0821d98 c0e50508 c4048000 00000000 c406f480 c4048000 >>> 1d60: c4341ec0 c0e50508 c4341ec8 c063ce8c c0e593a8 c4048000 c406f4bc c406f480 >>> 1d80: c4048000 c04c8bb0 c406f4cc c40704b0 00000000 047e0d67 3b9aca03 c0e5986c >>> 1da0: c0e5931c c0e97230 00000000 c063a618 00000000 c4048000 c0e59858 c40d0a10 >>> 1dc0: c0e597f0 c063f80c c0d31830 00000000 00000000 00000000 00000000 00000000 >>> 1de0: 00000000 00000000 e0821df4 00000000 00000000 c0ba32fc 00000000 047e0d67 >>> 1e00: c40d75d8 00000000 c40d0a10 c0e59804 00000000 00000000 c0d31830 c0c2b5a0 >>> 1e20: c0c776b0 c04cc8dc c40d0a10 00000000 c0e59804 c04ca1f4 c40d0a10 c0e59804 >>> 1e40: c40d0a10 00000000 00000000 c04ca47c 00000000 c0be8598 c0e94b28 c0e59804 >>> 1e60: c40d0a10 00000000 00000000 c04ca658 c40d0a10 c0e59804 c40d0a54 c4048000 >>> 1e80: 00000000 c04ca8c4 c0e59804 c04ca834 c406f3c0 c04c8494 c4048000 c406f40c >>> 1ea0: c4070d30 047e0d67 c0e59804 c4341e00 00000000 c406f3c0 c4341e34 c04c959c >>> 1ec0: c0c00c14 c0e6c000 c0e59804 c0d2113c c4070800 00000000 c0e6c000 c04cb598 >>> 1ee0: c4048000 c0d2113c c4070800 c010d21c 000003c0 00000000 c407084e c4070800 >>> 1f00: c407084d c013e9e0 c0bee764 000000c0 00000000 00000000 c0d004d0 c4048000 >>> 1f20: 00000006 00000006 373126cc c407086a 00000000 047e0d67 00000012 047e0d67 >>> 1f40: 00000007 c0d42bd8 00000007 c4070800 000000c0 c0d31850 c0d31830 c0d011f4 >>> 1f60: 00000006 00000006 00000000 c0d004d0 e0821f6c c0d004d0 00000000 c0e03380 >>> 1f80: c0905828 00000000 00000000 00000000 00000000 00000000 00000000 c0905844 >>> 1fa0: 00000000 c0905828 00000000 c010014c 00000000 00000000 00000000 00000000 >>> 1fc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 >>> 1fe0: 00000000 00000000 00000000 00000000 00000013 00000000 00000000 00000000 >>> Call trace: >>> cpufreq_online from cpufreq_add_dev+0x84/0x94 >>> cpufreq_add_dev from subsys_interface_register+0xf0/0x108 >>> subsys_interface_register from cpufreq_register_driver+0x13c/0x2e4 >>> cpufreq_register_driver from dt_cpufreq_probe+0xb4/0x380 >>> dt_cpufreq_probe from platform_probe+0x5c/0xb8 >>> platform_probe from really_probe+0xc8/0x2c8 >>> really_probe from __driver_probe_device+0x88/0x19c >>> __driver_probe_device from driver_probe_device+0x30/0x104 >>> driver_probe_device from __driver_attach+0x90/0x178 >>> __driver_attach from bus_for_each_dev+0x6c/0xb4 >>> bus_for_each_dev from bus_add_driver+0xcc/0x1ec >>> bus_add_driver from driver_register+0x7c/0x114 >>> driver_register from do_one_initcall+0x40/0x21c >>> do_one_initcall from kernel_init_freeable+0x194/0x1f8 >>> kernel_init_freeable from kernel_init+0x1c/0x128 >>> kernel_init from ret_from_fork+0x14/0x28 >>> Exception stack(0xe0821fb0 to 0xe0821ff8) >>> 1fa0: 00000000 00000000 00000000 00000000 >>> 1fc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 >>> 1fe0: 00000000 00000000 00000000 00000000 00000013 00000000 >>> Code: e3a02000 ebfff87e e3500000 0a000043 (e7f001f2) >>> ---[ end trace 0000000000000000 ]--- >>> note: swapper[1] exited with irqs disabled >>> Kernel panic - not syncing: Attempted to kill init! exitcode=0x0000000b >>> ---[ end Kernel panic - not syncing: Attempted to kill init! >>> exitcode=0x0000000b ]--- >>> >>> >>> And this one: >>> >>> Starting kernel ... >>> >>> Booting Linux on physical CPU 0x0 >>> Linux version 6.14.0-rc1+ (claudiu@claudiu-X670E-Pro-RS) >>> (arm-none-linux-gnueabihf-gcc (GNU Toolchain for the A-profile Architecture >>> 10.3-2021.07 (arm-10.29)) 10.3.1 20210621, GNU ld (GNU Toolchain for the >>> A-profile Architecture 10.3-2021.07 (arm-10.29)) 2.36.1.20210621) #11 Wed >>> Feb 26 11:01:10 EET 2025 >>> CPU: ARMv7 Processor [410fc075] revision 5 (ARMv7), cr=10c53c7d >>> CPU: PIPT / VIPT nonaliasing data cache, VIPT aliasing instruction cache >>> OF: fdt: Machine model: Microchip SAMA7G5-EK >>> printk: legacy bootconsole [earlycon0] enabled >>> Memory policy: Data cache writeback >>> cma: Reserved 192 MiB at 0x70000000 on node -1 >>> Zone ranges: >>> Normal [mem 0x0000000060000000-0x000000007fffffff] >>> Movable zone start for each node >>> Early memory node ranges >>> node 0: [mem 0x0000000060000000-0x000000007fffffff] >>> Initmem setup node 0 [mem 0x0000000060000000-0x000000007fffffff] >>> OF: reserved mem: Reserved memory: No reserved-memory node in the DT >>> CPU: All CPU(s) started in SVC mode. >>> Kernel command line: console=ttyS0,115200 root=/dev/mmcblk1p2 >>> rootfstype=ext4 rw rootwait cma=192m atmel.pm_modes=standby,ulp0 earlyprintk >>> printk: log buffer data + meta data: 65536 + 204800 = 270336 bytes >>> Dentry cache hash table entries: 65536 (order: 6, 262144 bytes, linear) >>> Inode-cache hash table entries: 32768 (order: 5, 131072 bytes, linear) >>> Built 1 zonelists, mobility grouping on. Total pages: 131072 >>> mem auto-init: stack:off, heap alloc:off, heap free:off >>> SLUB: HWalign=64, Order=0-3, MinObjects=0, CPUs=1, Nodes=1 >>> NR_IRQS: 16, nr_irqs: 16, preallocated irqs: 16 >>> Main crystal frequency not set, using approximate value >>> timer_probe: no matching timers found >>> Console: colour dummy device 80x30 >>> sched_clock: 32 bits at 100 Hz, resolution 10000000ns, wraps every >>> 21474836475000000ns >>> >>> >>> I suspect parent_data in sama7g5_pmc_setup() need to be initialized with zero. >> >> With this series I did not see any boot issues with the sama7g54. But >> when doing the DTS and DTSI changes that you made above I did see boot >> issues with the sama7g54. >> >> It seems that the 2/2 patch also needs to have a similar patch for the >> sama7g54 clock driver. > > Would it be better to have this change to all the SoCs and their clock > drivers to match the correct naming? That would be good. Thank you, Claudiu > Since this set is mainly to adjust > for sama7d65 and a separate change to sama7g5.c clock driver would be > needed to fix the error that you are seeing above. >> >> Best, >> Ryan >>> >>> Thank you, >>> Claudiu >>> >>> >>> >> > ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2025-03-19 17:52 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-02-06 18:14 [PATCH 0/2] AT91 Clock Adjustments Ryan.Wanner
2025-02-06 18:14 ` [PATCH 1/2] clk: at91: sckc: Fix parent_data struct for slow osc Ryan.Wanner
2025-02-06 18:14 ` [PATCH 2/2] clk: at91: sama7d65: Add missing clk_hw to parent_data Ryan.Wanner
2025-02-26 9:06 ` [PATCH 0/2] AT91 Clock Adjustments Claudiu Beznea
[not found] ` <b76108c1-3282-4b98-a210-b350e8ec0ec2@microchip.com>
2025-03-18 17:33 ` Ryan Wanner
2025-03-19 17:50 ` Claudiu Beznea
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).