From: Tony Lindgren <tony@atomide.com>
To: Dave Gerlach <d-gerlach@ti.com>
Cc: Nishanth Menon <nm@ti.com>,
linux-arm-kernel@lists.infradead.org, linux-omap@vger.kernel.org,
Benoit Cousson <bcousson@baylibre.com>,
Sekhar Nori <nsekhar@ti.com>,
Santosh Shilimkar <santosh.shilimkar@ti.com>
Subject: Re: [PATCH v3 0/3] OMAP4+: Get rid of internal SRAM handling
Date: Thu, 18 Sep 2014 09:48:47 -0700 [thread overview]
Message-ID: <20140918164846.GG14505@atomide.com> (raw)
In-Reply-To: <54133FDB.9030706@ti.com>
* Dave Gerlach <d-gerlach@ti.com> [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
WARNING: multiple messages have this Message-ID (diff)
From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 0/3] OMAP4+: Get rid of internal SRAM handling
Date: Thu, 18 Sep 2014 09:48:47 -0700 [thread overview]
Message-ID: <20140918164846.GG14505@atomide.com> (raw)
In-Reply-To: <54133FDB.9030706@ti.com>
* Dave Gerlach <d-gerlach@ti.com> [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
next prev parent reply other threads:[~2014-09-18 16:49 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-10 16:04 [PATCH v3 0/3] OMAP4+: Get rid of internal SRAM handling Dave Gerlach
2014-09-10 16:04 ` Dave Gerlach
2014-09-10 16:04 ` [PATCH v3 1/3] ARM: AM335x: Get rid of unused sram init function Dave Gerlach
2014-09-10 16:04 ` Dave Gerlach
2014-09-10 16:04 ` [PATCH v3 2/3] ARM: OMAP4+: Move SRAM data to DT Dave Gerlach
2014-09-10 16:04 ` Dave Gerlach
2014-09-10 16:04 ` [PATCH v3 3/3] ARM: OMAP4+: Remove static iotable mappings for SRAM Dave Gerlach
2014-09-10 16:04 ` Dave Gerlach
2014-09-10 16:56 ` [PATCH v3 0/3] OMAP4+: Get rid of internal SRAM handling Nishanth Menon
2014-09-10 16:56 ` Nishanth Menon
2014-09-12 18:47 ` Dave Gerlach
2014-09-12 18:47 ` Dave Gerlach
2014-09-18 16:48 ` Tony Lindgren [this message]
2014-09-18 16:48 ` Tony Lindgren
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140918164846.GG14505@atomide.com \
--to=tony@atomide.com \
--cc=bcousson@baylibre.com \
--cc=d-gerlach@ti.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=nm@ti.com \
--cc=nsekhar@ti.com \
--cc=santosh.shilimkar@ti.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.