From: khilman@ti.com (Kevin Hilman)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH/RFT 0/8] ARM: OMAP: remove IP checks from SoC family detection
Date: Mon, 30 Apr 2012 13:48:39 -0700 [thread overview]
Message-ID: <87d36pt07s.fsf@ti.com> (raw)
In-Reply-To: <20120430162510.GA18669@animalcreek.com> (Mark A. Greer's message of "Mon, 30 Apr 2012 09:25:10 -0700")
"Mark A. Greer" <mgreer@animalcreek.com> writes:
> On Fri, Apr 27, 2012 at 03:25:38PM -0700, Kevin Hilman wrote:
>> "Mark A. Greer" <mgreer@animalcreek.com> writes:
>>
>> > On Thu, Apr 26, 2012 at 04:29:45PM -0700, Kevin Hilman wrote:
>> >
>> > Hi Kevin.
>> >
>> >> This is a rebased version of this series which is ready for broader
>> >> testing. I'd especially appreciate testing from those of you with
>> >> AM35x platforms.
>> >>
>> >> Currently, our SoC detection is based on SoC family detection
>> >> (using die ID) and the presence of specific IP blocks (or feature.)
>> >>
>> >> This series begins the separation of the SoC family detection and
>> >> specific IP detection by completely removing IP detection from the SoC
>> >> family detection (cpu_is_*.)
>> >>
>> >> Applies on top of v3.4-rc4, boot tested on AM3517 EVM.
>> >
>> > Your series applied and booted without issue until the point that
>> > pm_idle is called. At that point it hung because its doing a wfi
>> > and not getting an interrupt to wake up back up. That's expected
>> > behaviour so your patches are good (IMHO) but I need to finish and
>> > submit a new version of my patches to fix that issue.
>>
>> For the benefit of others wanting to test this:
>>
>> You have to use 'nohlt' on the cmdline on AM35x in order to avoid WFI
>> and the wakeup problems that are still being worked on.
>
> Oops, yes. I meant to add that and say that my am3517evm boots fine
> with your patches and 'nohlt' on the cmdline. I'm using omap2plus_defconfig
> with CONFIG_TI_DAVINCI_EMAC enabled.
Thanks for testing, will add a Tested-by from you.
Kevin
next prev parent reply other threads:[~2012-04-30 20:48 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-04-26 23:29 [PATCH/RFT 0/8] ARM: OMAP: remove IP checks from SoC family detection Kevin Hilman
2012-04-26 23:29 ` [PATCH/RFT 1/8] ARM: OMAP: remove unused cpu_is macros that depend on specific IP checks Kevin Hilman
2012-04-26 23:29 ` [PATCH/RFT 2/8] ARM: OMAP3: clock data: replace 3503/3517 flag with AM35x flag for UART4 Kevin Hilman
2012-04-26 23:29 ` [PATCH/RFT 3/8] ARM: OMAP3: clock data: treat all AM35x devices the same Kevin Hilman
2012-04-26 23:29 ` [PATCH/RFT 4/8] ARM: OMAP: AM35x: remove redunant cpu_is checks for AM3505 Kevin Hilman
2012-04-26 23:29 ` [PATCH/RFT 5/8] ARM: OMAP: clock: remove unused CK_3505 flag Kevin Hilman
2012-04-30 9:05 ` Jean Pihet
2012-04-30 17:19 ` Kevin Hilman
2012-04-26 23:29 ` [PATCH/RFT 6/8] ARM: OMAP: remove unused cpu_is_omap3505() Kevin Hilman
2012-04-30 9:07 ` Jean Pihet
2012-04-30 9:35 ` Koen Kooi
2012-04-30 17:20 ` Kevin Hilman
2012-04-26 23:29 ` [PATCH/RFT 7/8] ARM: OMAP: remove unused cpu_is_omap3530() Kevin Hilman
2012-04-26 23:29 ` [PATCH/RFT 8/8] ARM: OMAP: AM35xx: convert 3517 detection/flags to AM35xx Kevin Hilman
2012-04-27 22:08 ` [PATCH/RFT 0/8] ARM: OMAP: remove IP checks from SoC family detection Mark A. Greer
2012-04-27 22:25 ` Kevin Hilman
2012-04-30 16:25 ` Mark A. Greer
2012-04-30 20:48 ` Kevin Hilman [this message]
2012-04-30 23:47 ` Kevin Hilman
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=87d36pt07s.fsf@ti.com \
--to=khilman@ti.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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 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).