Re: [PATCHv5 00/35] ARM: OMAP2+: PRCM/SCM cleanups against 4.0-rc
On 03/31/2015 04:25 AM, Tony Lindgren wrote: * Tony Lindgren [150330 17:15]: Hi Tero, * Tero Kristo [150320 11:45]: Hi, v5 contains the following changes still: - re-ordered patches a bit, the single clock driver fix moved to beginning of the set, waiting for a separate merge from Mike - Changed patch #23 to fix the slightly misleading logic (removed the extra +1) - Changed patches #16 and #18 to fix OMAP2/3/4 etc. only builds - Fixed ti81xx boot issues (hopefully, I don't have access to hardware to test it) - Changed control module DTS layout based on discussions with Tony, this includes addition of minimal l4 bus (patch #25+) Testing done for v5: 1: am335x-evm : boot 2: am335x-evmsk: boot 3: am3517-evm : boot 4: am43x-epos-evm : boot 5: am437x-gp-evm : boot 6: omap3-beagle-xm : boot 7: omap3-beagle: boot, suspend (ret/off), cpuidle (ret/off) 8: am335x-boneblack: boot 9: am335x-bone : boot 10: dra7xx-evm : boot 11: omap3-n900 : boot 12: omap5-uevm : boot 13: omap4-panda-es : boot, suspend (ret), cpuidle (ret) 14: omap4-panda : boot 15: omap2430-sdp: boot 16: omap3430-sdp: boot 17: omap4-sdp-es23plus: boot Branch available at: - tree: https://github.com/t-kristo/linux-pm.git - branch: 4.0-rc1-prcm-cleanup-v5 I found few more issues regarding diff of the dmesg before and after, you may want to diff also dra7 before and after. Then I just noticed this series won't boot on omap3 with the legacy mode. You can test this by building a uImage with the following command: $ mkimage -A arm -O linux -T kernel -C none -a 0x80008000 -e 0x80008000 \ -n "Linux" -d arch/arm/boot/zImage /tmp/uImage Then make sure you're not passing a .dtb file. Below is the error I'm getting with debug_ll + earlyprintk enabled. Also noticed a make randconfig build warning: arch/arm/mach-omap2/prm_common.c:702:35: warning: ‘scrm_data’ defined but not used [-Wunused-variable] That seems to happen at least with the following selection: # CONFIG_ARCH_OMAP2 is not set # CONFIG_ARCH_OMAP3 is not set # CONFIG_ARCH_OMAP4 is not set # CONFIG_SOC_OMAP5 is not set # CONFIG_SOC_AM33XX is not set CONFIG_SOC_AM43XX=y # CONFIG_SOC_DRA7XX is not set CONFIG_ARCH_OMAP2PLUS=y Ok will fix that warning also and repost in a bit. -Tero Regards, Tony [0.00] PC is at regmap_read+0x10/0x60 [0.00] LR is at clk_memmap_readl+0x34/0x54 [0.00] pc : []lr : []psr: 21d3 [0.00] sp : c08d1e90 ip : fp : [0.00] r10: c07efaa8 r9 : 0040 r8 : c601c5c0 [0.00] r7 : c601bbc0 r6 : c601c5c0 r5 : 0040 r4 : 0001 [0.00] r3 : fa004000 r2 : c08d1ea4 r1 : 0040 r0 : 0040 [0.00] Flags: nzCv IRQs off FIQs off Mode SVC_32 ISA ARM Segment kernel [0.00] Control: 10c5387d Table: 80004019 DAC: 0015 [0.00] Process swapper/0 (pid: 0, stack limit = 0xc08d0218) [0.00] Stack: (0xc08d1e90 to 0xc08d2000) [0.00] 1e80: c601bbc0 c601c5c0 c0035694 [0.00] 1ea0: c6001c8c c6001c40 c07f c04d7d78 0001 c07efaa8 c04d4dd0 [0.00] 1ec0: c04d08c4 c115c5bc 61d3 c601bbc0 0020 0001 [0.00] 1ee0: 0003 0013 0040 c07efaa8 c04d8284 c07f [0.00] 1f00: c094e4e8 c07efaa8 c06683ec c08d1efc c7eff101 0020 c0951fe4 0001 [0.00] 1f20: c0951fcc c7eff140 c0965000 c04d840c 0013 0003 [0.00] 1f40: 0001 039457c8 0040 c601bb80 c0951fcc c0945dac c094e504 [0.00] 1f60: c08b3280 c08a7694 c601bb80 c094e4e8 c601bb80 c094e508 c08a77a8 [0.00] 1f80: c08aa480 c0965000 c08d28c0 c08b3280 c7eff140 c0965000 [0.00] 1fa0: c08aa490 c08aa480 c0965000 c086f7a4 c08b1e58 c0863aec [0.00] 1fc0: c0863678 c08b3280 c0965214 c08d296c [0.00] 1fe0: c08b327c c08d7a0c 80004059 411fc083 8000807c [0.00] [] (regmap_read) from [] (clk_memmap_readl+0x34/0x54) [0.00] [] (clk_memmap_readl) from [] (ti_clk_divider_recalc_rate+0x20/0xf8) [0.00] [] (ti_clk_divider_recalc_rate) from [] (clk_register+0x360/0x6f8) [0.00] [] (clk_register) from [] (_register_divider.constprop.5+0xb8/0x120) [0.00] [] (_register_divider.constprop.5) from [] (ti_clk_register_divider+0x80/0xa4) [0.00] [] (ti_clk_register_divider) from [] (ti_clk_register_clk+0x88/0x17c) [0.00] [] (ti_clk_register_clk) from [] (ti_clk_register_legacy_clks+0x20/0x158) [0.00] [] (ti_clk_register_legacy_clks) from [] (omap3430_clk_legacy_init+0x10/0x58) [0.00] [] (omap3430_clk_legacy_init) from [] (omap3_sync32k_timer_init+0x8/0x58) [0.00] [] (omap3_sync32k_timer_init) from [] (st
Re: [PATCHv5 00/35] ARM: OMAP2+: PRCM/SCM cleanups against 4.0-rc
On 03/31/2015 03:10 AM, Tony Lindgren wrote: Hi Tero, * Tero Kristo [150320 11:45]: Hi, v5 contains the following changes still: - re-ordered patches a bit, the single clock driver fix moved to beginning of the set, waiting for a separate merge from Mike - Changed patch #23 to fix the slightly misleading logic (removed the extra +1) - Changed patches #16 and #18 to fix OMAP2/3/4 etc. only builds - Fixed ti81xx boot issues (hopefully, I don't have access to hardware to test it) - Changed control module DTS layout based on discussions with Tony, this includes addition of minimal l4 bus (patch #25+) Testing done for v5: 1: am335x-evm : boot 2: am335x-evmsk: boot 3: am3517-evm : boot 4: am43x-epos-evm : boot 5: am437x-gp-evm : boot 6: omap3-beagle-xm : boot 7: omap3-beagle: boot, suspend (ret/off), cpuidle (ret/off) 8: am335x-boneblack: boot 9: am335x-bone : boot 10: dra7xx-evm : boot 11: omap3-n900 : boot 12: omap5-uevm : boot 13: omap4-panda-es : boot, suspend (ret), cpuidle (ret) 14: omap4-panda : boot 15: omap2430-sdp: boot 16: omap3430-sdp: boot 17: omap4-sdp-es23plus: boot Branch available at: - tree: https://github.com/t-kristo/linux-pm.git - branch: 4.0-rc1-prcm-cleanup-v5 I found few more issues regarding diff of the dmesg before and after, you may want to diff also dra7 before and after. Then I just noticed this series won't boot on omap3 with the legacy mode. You can test this by building a uImage with the following command: $ mkimage -A arm -O linux -T kernel -C none -a 0x80008000 -e 0x80008000 \ -n "Linux" -d arch/arm/boot/zImage /tmp/uImage Then make sure you're not passing a .dtb file. Below is the error I'm getting with debug_ll + earlyprintk enabled. Yeah, I forgot to check the legacy boot and of course broke it, kind of annoying to test it atm. >.< Made a simple fix, will repost. -Tero Regards, Tony [0.00] PC is at regmap_read+0x10/0x60 [0.00] LR is at clk_memmap_readl+0x34/0x54 [0.00] pc : []lr : []psr: 21d3 [0.00] sp : c08d1e90 ip : fp : [0.00] r10: c07efaa8 r9 : 0040 r8 : c601c5c0 [0.00] r7 : c601bbc0 r6 : c601c5c0 r5 : 0040 r4 : 0001 [0.00] r3 : fa004000 r2 : c08d1ea4 r1 : 0040 r0 : 0040 [0.00] Flags: nzCv IRQs off FIQs off Mode SVC_32 ISA ARM Segment kernel [0.00] Control: 10c5387d Table: 80004019 DAC: 0015 [0.00] Process swapper/0 (pid: 0, stack limit = 0xc08d0218) [0.00] Stack: (0xc08d1e90 to 0xc08d2000) [0.00] 1e80: c601bbc0 c601c5c0 c0035694 [0.00] 1ea0: c6001c8c c6001c40 c07f c04d7d78 0001 c07efaa8 c04d4dd0 [0.00] 1ec0: c04d08c4 c115c5bc 61d3 c601bbc0 0020 0001 [0.00] 1ee0: 0003 0013 0040 c07efaa8 c04d8284 c07f [0.00] 1f00: c094e4e8 c07efaa8 c06683ec c08d1efc c7eff101 0020 c0951fe4 0001 [0.00] 1f20: c0951fcc c7eff140 c0965000 c04d840c 0013 0003 [0.00] 1f40: 0001 039457c8 0040 c601bb80 c0951fcc c0945dac c094e504 [0.00] 1f60: c08b3280 c08a7694 c601bb80 c094e4e8 c601bb80 c094e508 c08a77a8 [0.00] 1f80: c08aa480 c0965000 c08d28c0 c08b3280 c7eff140 c0965000 [0.00] 1fa0: c08aa490 c08aa480 c0965000 c086f7a4 c08b1e58 c0863aec [0.00] 1fc0: c0863678 c08b3280 c0965214 c08d296c [0.00] 1fe0: c08b327c c08d7a0c 80004059 411fc083 8000807c [0.00] [] (regmap_read) from [] (clk_memmap_readl+0x34/0x54) [0.00] [] (clk_memmap_readl) from [] (ti_clk_divider_recalc_rate+0x20/0xf8) [0.00] [] (ti_clk_divider_recalc_rate) from [] (clk_register+0x360/0x6f8) [0.00] [] (clk_register) from [] (_register_divider.constprop.5+0xb8/0x120) [0.00] [] (_register_divider.constprop.5) from [] (ti_clk_register_divider+0x80/0xa4) [0.00] [] (ti_clk_register_divider) from [] (ti_clk_register_clk+0x88/0x17c) [0.00] [] (ti_clk_register_clk) from [] (ti_clk_register_legacy_clks+0x20/0x158) [0.00] [] (ti_clk_register_legacy_clks) from [] (omap3430_clk_legacy_init+0x10/0x58) [0.00] [] (omap3430_clk_legacy_init) from [] (omap3_sync32k_timer_init+0x8/0x58) [0.00] [] (omap3_sync32k_timer_init) from [] (start_kernel+0x238/0x3e8) [0.00] [] (start_kernel) from [<8000807c>] (0x8000807c) [0.00] Code: e92d4070 e1a04000 e1a05001 e1a1 (e594117c) [0.00] ---[ end trace cb88537fdc8fa200 ]--- -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo
Re: [PATCHv5 00/35] ARM: OMAP2+: PRCM/SCM cleanups against 4.0-rc
* Tony Lindgren [150330 17:15]: > Hi Tero, > > * Tero Kristo [150320 11:45]: > > Hi, > > > > v5 contains the following changes still: > > > > - re-ordered patches a bit, the single clock driver fix moved to beginning > > of the set, waiting for a separate merge from Mike > > - Changed patch #23 to fix the slightly misleading logic (removed the extra > > +1) > > - Changed patches #16 and #18 to fix OMAP2/3/4 etc. only builds > > - Fixed ti81xx boot issues (hopefully, I don't have access to hardware > > to test it) > > - Changed control module DTS layout based on discussions with Tony, this > > includes addition of minimal l4 bus (patch #25+) > > > > Testing done for v5: > > 1: am335x-evm : boot > > 2: am335x-evmsk: boot > > 3: am3517-evm : boot > > 4: am43x-epos-evm : boot > > 5: am437x-gp-evm : boot > > 6: omap3-beagle-xm : boot > > 7: omap3-beagle: boot, suspend (ret/off), cpuidle (ret/off) > > 8: am335x-boneblack: boot > > 9: am335x-bone : boot > > 10: dra7xx-evm : boot > > 11: omap3-n900 : boot > > 12: omap5-uevm : boot > > 13: omap4-panda-es : boot, suspend (ret), cpuidle (ret) > > 14: omap4-panda : boot > > 15: omap2430-sdp: boot > > 16: omap3430-sdp: boot > > 17: omap4-sdp-es23plus: boot > > > > Branch available at: > > - tree: https://github.com/t-kristo/linux-pm.git > > - branch: 4.0-rc1-prcm-cleanup-v5 > > I found few more issues regarding diff of the dmesg before and after, > you may want to diff also dra7 before and after. > > Then I just noticed this series won't boot on omap3 with the legacy mode. > You can test this by building a uImage with the following command: > > $ mkimage -A arm -O linux -T kernel -C none -a 0x80008000 -e 0x80008000 \ > -n "Linux" -d arch/arm/boot/zImage /tmp/uImage > > Then make sure you're not passing a .dtb file. > > Below is the error I'm getting with debug_ll + earlyprintk enabled. Also noticed a make randconfig build warning: arch/arm/mach-omap2/prm_common.c:702:35: warning: ‘scrm_data’ defined but not used [-Wunused-variable] That seems to happen at least with the following selection: # CONFIG_ARCH_OMAP2 is not set # CONFIG_ARCH_OMAP3 is not set # CONFIG_ARCH_OMAP4 is not set # CONFIG_SOC_OMAP5 is not set # CONFIG_SOC_AM33XX is not set CONFIG_SOC_AM43XX=y # CONFIG_SOC_DRA7XX is not set CONFIG_ARCH_OMAP2PLUS=y Regards, Tony > [0.00] PC is at regmap_read+0x10/0x60 > [0.00] LR is at clk_memmap_readl+0x34/0x54 > [0.00] pc : []lr : []psr: 21d3 > [0.00] sp : c08d1e90 ip : fp : > [0.00] r10: c07efaa8 r9 : 0040 r8 : c601c5c0 > [0.00] r7 : c601bbc0 r6 : c601c5c0 r5 : 0040 r4 : 0001 > [0.00] r3 : fa004000 r2 : c08d1ea4 r1 : 0040 r0 : 0040 > [0.00] Flags: nzCv IRQs off FIQs off Mode SVC_32 ISA ARM Segment > kernel > [0.00] Control: 10c5387d Table: 80004019 DAC: 0015 > [0.00] Process swapper/0 (pid: 0, stack limit = 0xc08d0218) > [0.00] Stack: (0xc08d1e90 to 0xc08d2000) > [0.00] 1e80: c601bbc0 > c601c5c0 c0035694 > [0.00] 1ea0: c6001c8c c6001c40 c07f c04d7d78 0001 > c07efaa8 c04d4dd0 > [0.00] 1ec0: c04d08c4 c115c5bc 61d3 c601bbc0 > 0020 0001 > [0.00] 1ee0: 0003 0013 0040 c07efaa8 c04d8284 > c07f > [0.00] 1f00: c094e4e8 c07efaa8 c06683ec c08d1efc c7eff101 0020 > c0951fe4 0001 > [0.00] 1f20: c0951fcc c7eff140 c0965000 c04d840c > 0013 0003 > [0.00] 1f40: 0001 039457c8 0040 c601bb80 c0951fcc > c0945dac c094e504 > [0.00] 1f60: c08b3280 c08a7694 c601bb80 c094e4e8 c601bb80 > c094e508 c08a77a8 > [0.00] 1f80: c08aa480 c0965000 c08d28c0 c08b3280 > c7eff140 c0965000 > [0.00] 1fa0: c08aa490 c08aa480 c0965000 c086f7a4 > c08b1e58 c0863aec > [0.00] 1fc0: c0863678 c08b3280 > c0965214 c08d296c > [0.00] 1fe0: c08b327c c08d7a0c 80004059 411fc083 8000807c > > [0.00] [] (regmap_read) from [] > (clk_memmap_readl+0x34/0x54) > [0.00] [] (clk_memmap_readl) from [] > (ti_clk_divider_recalc_rate+0x20/0xf8) > [0.00] [] (ti_clk_divider_recalc_rate) from [] > (clk_register+0x360/0x6f8) > [0.00] [] (clk_register) from [] > (_register_divider.constprop.5+0xb8/0x120) > [0.00] [] (_register_divider.constprop.5) from [] > (ti_clk_register_divider+0x80/0xa4) > [0.00] [] (ti_clk_register_divider) from [] > (ti_clk_register_clk+0x88/0x17c) > [0.00] [] (ti_clk_register_clk) from [] > (ti_clk_register_legacy_clks+0x20/0x158) > [0.00] [] (ti_clk_register_legacy_clks) from [] > (omap3430_clk_legacy_init+0x10
Re: [PATCHv5 00/35] ARM: OMAP2+: PRCM/SCM cleanups against 4.0-rc
Hi Tero, * Tero Kristo [150320 11:45]: > Hi, > > v5 contains the following changes still: > > - re-ordered patches a bit, the single clock driver fix moved to beginning > of the set, waiting for a separate merge from Mike > - Changed patch #23 to fix the slightly misleading logic (removed the extra > +1) > - Changed patches #16 and #18 to fix OMAP2/3/4 etc. only builds > - Fixed ti81xx boot issues (hopefully, I don't have access to hardware > to test it) > - Changed control module DTS layout based on discussions with Tony, this > includes addition of minimal l4 bus (patch #25+) > > Testing done for v5: > 1: am335x-evm : boot > 2: am335x-evmsk: boot > 3: am3517-evm : boot > 4: am43x-epos-evm : boot > 5: am437x-gp-evm : boot > 6: omap3-beagle-xm : boot > 7: omap3-beagle: boot, suspend (ret/off), cpuidle (ret/off) > 8: am335x-boneblack: boot > 9: am335x-bone : boot > 10: dra7xx-evm : boot > 11: omap3-n900 : boot > 12: omap5-uevm : boot > 13: omap4-panda-es : boot, suspend (ret), cpuidle (ret) > 14: omap4-panda : boot > 15: omap2430-sdp: boot > 16: omap3430-sdp: boot > 17: omap4-sdp-es23plus: boot > > Branch available at: > - tree: https://github.com/t-kristo/linux-pm.git > - branch: 4.0-rc1-prcm-cleanup-v5 I found few more issues regarding diff of the dmesg before and after, you may want to diff also dra7 before and after. Then I just noticed this series won't boot on omap3 with the legacy mode. You can test this by building a uImage with the following command: $ mkimage -A arm -O linux -T kernel -C none -a 0x80008000 -e 0x80008000 \ -n "Linux" -d arch/arm/boot/zImage /tmp/uImage Then make sure you're not passing a .dtb file. Below is the error I'm getting with debug_ll + earlyprintk enabled. Regards, Tony [0.00] PC is at regmap_read+0x10/0x60 [0.00] LR is at clk_memmap_readl+0x34/0x54 [0.00] pc : []lr : []psr: 21d3 [0.00] sp : c08d1e90 ip : fp : [0.00] r10: c07efaa8 r9 : 0040 r8 : c601c5c0 [0.00] r7 : c601bbc0 r6 : c601c5c0 r5 : 0040 r4 : 0001 [0.00] r3 : fa004000 r2 : c08d1ea4 r1 : 0040 r0 : 0040 [0.00] Flags: nzCv IRQs off FIQs off Mode SVC_32 ISA ARM Segment kernel [0.00] Control: 10c5387d Table: 80004019 DAC: 0015 [0.00] Process swapper/0 (pid: 0, stack limit = 0xc08d0218) [0.00] Stack: (0xc08d1e90 to 0xc08d2000) [0.00] 1e80: c601bbc0 c601c5c0 c0035694 [0.00] 1ea0: c6001c8c c6001c40 c07f c04d7d78 0001 c07efaa8 c04d4dd0 [0.00] 1ec0: c04d08c4 c115c5bc 61d3 c601bbc0 0020 0001 [0.00] 1ee0: 0003 0013 0040 c07efaa8 c04d8284 c07f [0.00] 1f00: c094e4e8 c07efaa8 c06683ec c08d1efc c7eff101 0020 c0951fe4 0001 [0.00] 1f20: c0951fcc c7eff140 c0965000 c04d840c 0013 0003 [0.00] 1f40: 0001 039457c8 0040 c601bb80 c0951fcc c0945dac c094e504 [0.00] 1f60: c08b3280 c08a7694 c601bb80 c094e4e8 c601bb80 c094e508 c08a77a8 [0.00] 1f80: c08aa480 c0965000 c08d28c0 c08b3280 c7eff140 c0965000 [0.00] 1fa0: c08aa490 c08aa480 c0965000 c086f7a4 c08b1e58 c0863aec [0.00] 1fc0: c0863678 c08b3280 c0965214 c08d296c [0.00] 1fe0: c08b327c c08d7a0c 80004059 411fc083 8000807c [0.00] [] (regmap_read) from [] (clk_memmap_readl+0x34/0x54) [0.00] [] (clk_memmap_readl) from [] (ti_clk_divider_recalc_rate+0x20/0xf8) [0.00] [] (ti_clk_divider_recalc_rate) from [] (clk_register+0x360/0x6f8) [0.00] [] (clk_register) from [] (_register_divider.constprop.5+0xb8/0x120) [0.00] [] (_register_divider.constprop.5) from [] (ti_clk_register_divider+0x80/0xa4) [0.00] [] (ti_clk_register_divider) from [] (ti_clk_register_clk+0x88/0x17c) [0.00] [] (ti_clk_register_clk) from [] (ti_clk_register_legacy_clks+0x20/0x158) [0.00] [] (ti_clk_register_legacy_clks) from [] (omap3430_clk_legacy_init+0x10/0x58) [0.00] [] (omap3430_clk_legacy_init) from [] (omap3_sync32k_timer_init+0x8/0x58) [0.00] [] (omap3_sync32k_timer_init) from [] (start_kernel+0x238/0x3e8) [0.00] [] (start_kernel) from [<8000807c>] (0x8000807c) [0.00] Code: e92d4070 e1a04000 e1a05001 e1a1 (e594117c) [0.00] ---[ end trace cb88537fdc8fa200 ]--- -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCHv5 00/35] ARM: OMAP2+: PRCM/SCM cleanups against 4.0-rc
Hi, v5 contains the following changes still: - re-ordered patches a bit, the single clock driver fix moved to beginning of the set, waiting for a separate merge from Mike - Changed patch #23 to fix the slightly misleading logic (removed the extra +1) - Changed patches #16 and #18 to fix OMAP2/3/4 etc. only builds - Fixed ti81xx boot issues (hopefully, I don't have access to hardware to test it) - Changed control module DTS layout based on discussions with Tony, this includes addition of minimal l4 bus (patch #25+) Testing done for v5: 1: am335x-evm : boot 2: am335x-evmsk: boot 3: am3517-evm : boot 4: am43x-epos-evm : boot 5: am437x-gp-evm : boot 6: omap3-beagle-xm : boot 7: omap3-beagle: boot, suspend (ret/off), cpuidle (ret/off) 8: am335x-boneblack: boot 9: am335x-bone : boot 10: dra7xx-evm : boot 11: omap3-n900 : boot 12: omap5-uevm : boot 13: omap4-panda-es : boot, suspend (ret), cpuidle (ret) 14: omap4-panda : boot 15: omap2430-sdp: boot 16: omap3430-sdp: boot 17: omap4-sdp-es23plus: boot Branch available at: - tree: https://github.com/t-kristo/linux-pm.git - branch: 4.0-rc1-prcm-cleanup-v5 -Tero -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html