From: Tony Lindgren <tony@atomide.com> To: linux-omap@vger.kernel.org Cc: Nishanth Menon <nm@ti.com>, Tero Kristo <t-kristo@ti.com>, Santosh Shilimkar <santosh.shilimkar@oracle.com>, linux-arm-kernel@lists.infradead.org, Dave Gerlach <d-gerlach@ti.com> Subject: Re: [PATCH 1/3] ARM: OMAP2+: Separate dra7 cpuidle from omap5 Date: Thu, 17 Aug 2017 16:42:19 -0700 [thread overview] Message-ID: <20170817234218.GC6008@atomide.com> (raw) In-Reply-To: <20170817230122.30655-2-tony@atomide.com> * Tony Lindgren <tony@atomide.com> [170817 16:01]: > As omap5 supports deeper idle states compared to dra7, let's > separate dra7 from omap5 in preparation for adding more support > for omap5. Oops, sorry this first patch can be ignored. I forgot we already separated out things for dra7 when we added omap5 cpu retention mode support earlir. So there's nothing left to move away for dra7. > @@ -303,6 +342,9 @@ int __init omap4_idle_init(void) > if (soc_is_omap54xx()) { > state_ptr = &omap5_idle_data[0]; > idle_driver = &omap5_idle_driver; > + } else if (soc_is_dra7xx()) { > + state_ptr = &dra7_idle_data[0]; > + idle_driver = &dra7_idle_driver; > } else { > state_ptr = &omap4_idle_data[0]; > idle_driver = &omap4_idle_driver; And then this part now wrongly enables things for dra7 while it can't be done. 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 1/3] ARM: OMAP2+: Separate dra7 cpuidle from omap5 Date: Thu, 17 Aug 2017 16:42:19 -0700 [thread overview] Message-ID: <20170817234218.GC6008@atomide.com> (raw) In-Reply-To: <20170817230122.30655-2-tony@atomide.com> * Tony Lindgren <tony@atomide.com> [170817 16:01]: > As omap5 supports deeper idle states compared to dra7, let's > separate dra7 from omap5 in preparation for adding more support > for omap5. Oops, sorry this first patch can be ignored. I forgot we already separated out things for dra7 when we added omap5 cpu retention mode support earlir. So there's nothing left to move away for dra7. > @@ -303,6 +342,9 @@ int __init omap4_idle_init(void) > if (soc_is_omap54xx()) { > state_ptr = &omap5_idle_data[0]; > idle_driver = &omap5_idle_driver; > + } else if (soc_is_dra7xx()) { > + state_ptr = &dra7_idle_data[0]; > + idle_driver = &dra7_idle_driver; > } else { > state_ptr = &omap4_idle_data[0]; > idle_driver = &omap4_idle_driver; And then this part now wrongly enables things for dra7 while it can't be done. Regards, Tony
next prev parent reply other threads:[~2017-08-17 23:42 UTC|newest] Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-08-17 23:01 [PATCH 0/3] omap5 cpu off mode support Tony Lindgren 2017-08-17 23:01 ` Tony Lindgren 2017-08-17 23:01 ` [PATCH 1/3] ARM: OMAP2+: Separate dra7 cpuidle from omap5 Tony Lindgren 2017-08-17 23:01 ` Tony Lindgren 2017-08-17 23:42 ` Tony Lindgren [this message] 2017-08-17 23:42 ` Tony Lindgren 2017-08-17 23:01 ` [PATCH 2/3] ARM: OMAP5: Add cpuidle assembly code Tony Lindgren 2017-08-17 23:01 ` Tony Lindgren 2017-08-18 1:38 ` Nishanth Menon 2017-08-18 1:38 ` Nishanth Menon 2017-08-17 23:01 ` [PATCH 3/3] ARM: OMAP5: Enable CPU off idle states Tony Lindgren 2017-08-17 23:01 ` Tony Lindgren 2017-08-18 1:29 ` Nishanth Menon 2017-08-18 1:29 ` Nishanth Menon 2017-08-18 15:26 ` Tony Lindgren 2017-08-18 15:26 ` Tony Lindgren 2017-08-18 22:27 ` Nishanth Menon 2017-08-18 22:27 ` Nishanth Menon
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=20170817234218.GC6008@atomide.com \ --to=tony@atomide.com \ --cc=d-gerlach@ti.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-omap@vger.kernel.org \ --cc=nm@ti.com \ --cc=santosh.shilimkar@oracle.com \ --cc=t-kristo@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: linkBe 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.