Re: [PATCH v3 0/3] OMAP4+: Get rid of internal SRAM handling
* Dave Gerlach [140912 11:48]: > On 09/10/2014 11:56 AM, Nishanth Menon wrote: > > On 11:04-20140910, Dave Gerlach wrote: > >> v3: > >> Fix minor issue in last patch to check for null sram_pool if no sram > >> phandle is given in DT. > >> > >> Make all OMAP DT only platforms (am33xx, am43xx, omap4 and omap5) > >> use drivers/misc/sram.c driver instead of the omap internal > >> implementation for SRAM handling. > >> > >> Previous discussion can be found at [1]. > >> > >> Regards, > >> Dave > >> > >> [1] > >> http://lists.infradead.org/pipermail/linux-arm-kernel/2013-August/195588.html > >> > >> Rajendra Nayak (3): > >> ARM: AM335x: Get rid of unused sram init function > >> ARM: OMAP4+: Move SRAM data to DT > >> ARM: OMAP4+: Remove static iotable mappings for SRAM > >> > >> Documentation/devicetree/bindings/arm/omap/mpu.txt | 3 ++ > >> arch/arm/boot/dts/am33xx.dtsi | 5 ++- > >> arch/arm/boot/dts/am4372.dtsi | 5 +++ > >> arch/arm/boot/dts/omap4.dtsi | 6 > >> arch/arm/boot/dts/omap5.dtsi | 8 - > >> arch/arm/configs/omap2plus_defconfig | 1 + > >> arch/arm/mach-omap2/io.c | 17 -- > >> arch/arm/mach-omap2/omap4-common.c | 22 +++- > >> arch/arm/mach-omap2/sram.c | 39 > >> +- > >> arch/arm/mach-omap2/sram.h | 7 > >> 10 files changed, 46 insertions(+), 67 deletions(-) > >> > > > > Could you please provide logs for the following: > > a) Low power transition tests for OMAP3,4 on all available platforms as > > well? > > b) provide bootlogs on all omap2plus platforms to ensure we have no > > regressions. > > > > Here are logs for low power transition, all platforms passed, properly > transitioned power domains with mem sleep using wakeup_timer to wake. > > Low Power transition on v3.17-rc4 with patches applied, omap2plus_defconfig > (OMAP3 and OMAP4): > > 1: BeagleBoard-XM: http://fpaste.org/133215/ > 2: OMAP3430-Labrador(LDP): http://fpaste.org/133225/ > 3: n900: http://fpaste.org/133244/ > 4: pandaboard-es: http://fpaste.org/133213/ > 5: pandaboard-vanilla: http://fpaste.org/133214/ > 6: sdp3430: http://fpaste.org/133246/ > > Boot on v3.17-rc4 with patches applied, omap2plus_defconfig: > > 1: am335x-evm: Boot PASS: http://fpaste.org/133179/ > 2: am335x-sk: Boot PASS: http://fpaste.org/133180/ > 3: am3517-evm: Boot PASS: http://fpaste.org/133181/ > 4: am37x-evm: Boot PASS: http://fpaste.org/133182/ > 5: am43xx-epos: Boot PASS: http://fpaste.org/133183/ > 6: am43xx-gpevm: Boot PASS: http://fpaste.org/133184/ > 7: BeagleBoard-XM: Boot PASS: http://fpaste.org/133191/ > 8: beagleboard-vanilla: Boot PASS: http://fpaste.org/133192/ > 9: beaglebone-black: Boot PASS: http://fpaste.org/133193/ > 10: beaglebone: Boot PASS: http://fpaste.org/133194/ > 11: craneboard: Boot PASS: http://fpaste.org/133186/ > 12: dra7xx-evm: Boot PASS: http://fpaste.org/131396/ > 13: OMAP3430-Labrador(LDP): Boot PASS: http://fpaste.org/133206/ > 14: n900: Boot PASS: http://fpaste.org/133207/ > 15: omap5-evm: Boot PASS: http://fpaste.org/133208/ > 16: pandaboard-es: Boot PASS: http://fpaste.org/133187/ > 17: pandaboard-vanilla: Boot PASS: http://fpaste.org/133188/ > 18:sdp2430: Boot PASS: http://fpaste.org/133189/ > 19:sdp3430: Boot PASS: http://fpaste.org/133212/ > TOTAL = 19 boards, Booted Boards = 19, No Boot boards = 0 > > Again, all pass. I have also tested with next AM335x Suspend/Resume > implementation, next version will use generic sram driver to allocate for > low-level assembly code, and that works fine as well. Great, thanks for doing that. Applying all into omap-for-v3.18/soc. Regards, Tony -- 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
Re: [PATCH v3 0/3] OMAP4+: Get rid of internal SRAM handling
On 09/10/2014 11:56 AM, Nishanth Menon wrote: > On 11:04-20140910, Dave Gerlach wrote: >> v3: >> Fix minor issue in last patch to check for null sram_pool if no sram >> phandle is given in DT. >> >> Make all OMAP DT only platforms (am33xx, am43xx, omap4 and omap5) >> use drivers/misc/sram.c driver instead of the omap internal >> implementation for SRAM handling. >> >> Previous discussion can be found at [1]. >> >> Regards, >> Dave >> >> [1] >> http://lists.infradead.org/pipermail/linux-arm-kernel/2013-August/195588.html >> >> Rajendra Nayak (3): >> ARM: AM335x: Get rid of unused sram init function >> ARM: OMAP4+: Move SRAM data to DT >> ARM: OMAP4+: Remove static iotable mappings for SRAM >> >> Documentation/devicetree/bindings/arm/omap/mpu.txt | 3 ++ >> arch/arm/boot/dts/am33xx.dtsi | 5 ++- >> arch/arm/boot/dts/am4372.dtsi | 5 +++ >> arch/arm/boot/dts/omap4.dtsi | 6 >> arch/arm/boot/dts/omap5.dtsi | 8 - >> arch/arm/configs/omap2plus_defconfig | 1 + >> arch/arm/mach-omap2/io.c | 17 -- >> arch/arm/mach-omap2/omap4-common.c | 22 +++- >> arch/arm/mach-omap2/sram.c | 39 >> +- >> arch/arm/mach-omap2/sram.h | 7 >> 10 files changed, 46 insertions(+), 67 deletions(-) >> > > Could you please provide logs for the following: > a) Low power transition tests for OMAP3,4 on all available platforms as > well? > b) provide bootlogs on all omap2plus platforms to ensure we have no > regressions. > Here are logs for low power transition, all platforms passed, properly transitioned power domains with mem sleep using wakeup_timer to wake. Low Power transition on v3.17-rc4 with patches applied, omap2plus_defconfig (OMAP3 and OMAP4): 1: BeagleBoard-XM: http://fpaste.org/133215/ 2: OMAP3430-Labrador(LDP): http://fpaste.org/133225/ 3: n900: http://fpaste.org/133244/ 4: pandaboard-es: http://fpaste.org/133213/ 5: pandaboard-vanilla: http://fpaste.org/133214/ 6: sdp3430: http://fpaste.org/133246/ Boot on v3.17-rc4 with patches applied, omap2plus_defconfig: 1: am335x-evm: Boot PASS: http://fpaste.org/133179/ 2: am335x-sk: Boot PASS: http://fpaste.org/133180/ 3: am3517-evm: Boot PASS: http://fpaste.org/133181/ 4: am37x-evm: Boot PASS: http://fpaste.org/133182/ 5: am43xx-epos: Boot PASS: http://fpaste.org/133183/ 6: am43xx-gpevm: Boot PASS: http://fpaste.org/133184/ 7: BeagleBoard-XM: Boot PASS: http://fpaste.org/133191/ 8: beagleboard-vanilla: Boot PASS: http://fpaste.org/133192/ 9: beaglebone-black: Boot PASS: http://fpaste.org/133193/ 10: beaglebone: Boot PASS: http://fpaste.org/133194/ 11: craneboard: Boot PASS: http://fpaste.org/133186/ 12: dra7xx-evm: Boot PASS: http://fpaste.org/131396/ 13: OMAP3430-Labrador(LDP): Boot PASS: http://fpaste.org/133206/ 14: n900: Boot PASS: http://fpaste.org/133207/ 15: omap5-evm: Boot PASS: http://fpaste.org/133208/ 16: pandaboard-es: Boot PASS: http://fpaste.org/133187/ 17: pandaboard-vanilla: Boot PASS: http://fpaste.org/133188/ 18:sdp2430: Boot PASS: http://fpaste.org/133189/ 19:sdp3430: Boot PASS: http://fpaste.org/133212/ TOTAL = 19 boards, Booted Boards = 19, No Boot boards = 0 Again, all pass. I have also tested with next AM335x Suspend/Resume implementation, next version will use generic sram driver to allocate for low-level assembly code, and that works fine as well. Regards, Dave -- 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
Re: [PATCH v3 0/3] OMAP4+: Get rid of internal SRAM handling
On 11:04-20140910, Dave Gerlach wrote: > v3: > Fix minor issue in last patch to check for null sram_pool if no sram > phandle is given in DT. > > Make all OMAP DT only platforms (am33xx, am43xx, omap4 and omap5) > use drivers/misc/sram.c driver instead of the omap internal > implementation for SRAM handling. > > Previous discussion can be found at [1]. > > Regards, > Dave > > [1] > http://lists.infradead.org/pipermail/linux-arm-kernel/2013-August/195588.html > > Rajendra Nayak (3): > ARM: AM335x: Get rid of unused sram init function > ARM: OMAP4+: Move SRAM data to DT > ARM: OMAP4+: Remove static iotable mappings for SRAM > > Documentation/devicetree/bindings/arm/omap/mpu.txt | 3 ++ > arch/arm/boot/dts/am33xx.dtsi | 5 ++- > arch/arm/boot/dts/am4372.dtsi | 5 +++ > arch/arm/boot/dts/omap4.dtsi | 6 > arch/arm/boot/dts/omap5.dtsi | 8 - > arch/arm/configs/omap2plus_defconfig | 1 + > arch/arm/mach-omap2/io.c | 17 -- > arch/arm/mach-omap2/omap4-common.c | 22 +++- > arch/arm/mach-omap2/sram.c | 39 > +- > arch/arm/mach-omap2/sram.h | 7 > 10 files changed, 46 insertions(+), 67 deletions(-) > Could you please provide logs for the following: a) Low power transition tests for OMAP3,4 on all available platforms as well? b) provide bootlogs on all omap2plus platforms to ensure we have no regressions. -- Regards, Nishanth Menon -- 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
[PATCH v3 0/3] OMAP4+: Get rid of internal SRAM handling
v3: Fix minor issue in last patch to check for null sram_pool if no sram phandle is given in DT. Make all OMAP DT only platforms (am33xx, am43xx, omap4 and omap5) use drivers/misc/sram.c driver instead of the omap internal implementation for SRAM handling. Previous discussion can be found at [1]. Regards, Dave [1] http://lists.infradead.org/pipermail/linux-arm-kernel/2013-August/195588.html Rajendra Nayak (3): ARM: AM335x: Get rid of unused sram init function ARM: OMAP4+: Move SRAM data to DT ARM: OMAP4+: Remove static iotable mappings for SRAM Documentation/devicetree/bindings/arm/omap/mpu.txt | 3 ++ arch/arm/boot/dts/am33xx.dtsi | 5 ++- arch/arm/boot/dts/am4372.dtsi | 5 +++ arch/arm/boot/dts/omap4.dtsi | 6 arch/arm/boot/dts/omap5.dtsi | 8 - arch/arm/configs/omap2plus_defconfig | 1 + arch/arm/mach-omap2/io.c | 17 -- arch/arm/mach-omap2/omap4-common.c | 22 +++- arch/arm/mach-omap2/sram.c | 39 +- arch/arm/mach-omap2/sram.h | 7 10 files changed, 46 insertions(+), 67 deletions(-) -- 1.9.0 -- 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