Messages in this thread | | | Date | Thu, 24 Jan 2013 11:26:27 +0530 | From | Santosh Shilimkar <> | Subject | Re: [v3 2/2] ARM: tegra: Skip scu_enable(scu_base) if not Cortex A9 |
| |
Stephen,
On Thursday 24 January 2013 10:34 AM, Stephen Warren wrote: > On 01/23/2013 01:58 AM, Santosh Shilimkar wrote: >> On Tuesday 22 January 2013 10:34 PM, Olof Johansson wrote: >>> On Tue, Jan 22, 2013 at 8:57 AM, Stephen Warren >>> <swarren@wwwdotorg.org> wrote: >>>> On 01/21/2013 11:07 PM, Santosh Shilimkar wrote: >>>>> On Tuesday 22 January 2013 11:22 AM, Hiroshi Doyu wrote: >>>>>> Skip scu_enable(scu_base) if CPU is not Cortex A9 with SCU. >>>>>> >>>>>> Signed-off-by: Hiroshi Doyu <hdoyu@nvidia.com> >>>>>> --- >>>>> Looks fine. I will also update OMAP code with the new >>>>> interface. Thanks. >>>> >>>> OK, so patch 1/2 at least needs to get into a stable arm-soc branch >>>> then. Unless there are violent objections, I'll forward patch 1/2 to >>>> arm-soc and request it be added into a branch so that Tegra and OMAP can >>>> both merge it into their branches as a dependency. I guess patch 2/2 >>>> could also be included; I don't think it has any complex dependencies >>>> that'd prevent that, and would help to show how patch 1/2 gets used. >>>> >>>> Hiroshi, is this series the only dependency you need for your Tegra114 >>>> series? So, I could merge your Tegra114 series once this series is >>>> applied? >>> >>> For something like this, it might make more sense for us to just apply >>> the patches for OMAP on top, i.e. we'll pull the short branch from >>> you, and then we can just apply patches (with maintainer acks) on top, >>> instead of doing a bunch of single-patch pulls. >> >> In case you decide to apply patches, you can use patch in the end >> of the email for OMAP. Attached the same in case mailer damages it. >> >> Btw, I noticed the build error with patch 1/1. Since I wasn't using >> the first interface in OMAP code, I just bypassed it for testing. >> I might be missing some dependent patch which added >> read_cpuid_part_number(). > > Thanks for the OMAP patch. I have pushed a couple of temporary and > non-stable branches to: > > git://git.kernel.org/pub/scm/linux/kernel/git/swarren/linux-tegra.git > > Branch scu-base-rework contains just the SCU base address patches. After > the few things listed below, I'll rename this branch and send a pull > request to arm-soc. > > 1) Olof asked that Russell Ack or otherwise OK Hiroshi's latest versions > of the following two patches since he commented on previous versions: > > ARM: tegra: Skip scu_enable(scu_base) if not Cortex A9 > ARM: Add API to detect SCU base address from CP15 > > 2) Lorenzo's last comment on "ARM: tegra: Use DT /cpu node to detect > number of CPU core" was: > > Please add missing punctuation, reword the commit log to make it clearer. > > ... so that patch needs a reworded commit log. > > 3) This branch needs testing on both Tegra30 and OMAP (I'm away from the > office at the moment and can only test on Tegra20 here), > Tested 'scu-base-rework' branch on OMAP and it works just fine. I noticed that the OMAP patch subject got line wrapped. Can you please fix that before sending pull request ? Thanks for picking up the patch.
Regards, Santosh
| |