From: khilman@kernel•org (Kevin Hilman)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH] ARM: EXYNOS: reset KFC cores when cpu is up
Date: Tue, 01 Sep 2015 12:33:06 -0700 [thread overview]
Message-ID: <7hy4gq6ikt.fsf@deeprootsystems.com> (raw)
In-Reply-To: <CAM4voamGcz_g+ONa-YaYMs0ZGgcPiXGGOeCV47U5nw_uu48+Jg@mail.gmail.com> (Abhilash Kesavan's message of "Tue, 1 Sep 2015 20:17:14 +0530")
Abhilash Kesavan <kesavan.abhilash@gmail•com> writes:
> On Tue, Sep 1, 2015 at 5:51 AM, Krzysztof Kozlowski
> <k.kozlowski@samsung•com> wrote:
>> On 01.09.2015 07:46, Javier Martinez Canillas wrote:
[...]
>>> I posted a similar patch that instead disabling CCI for the XU3 board,
>>> it disables in exynos5422-odroidxu3-common.dtsi since all Exynos5422
>>> Odroid boards have the same broken firmware and so the same issue:
>>>
>>> https://lkml.org/lkml/2015/8/29/59
OK, that makes more sense. Thanks.
>>> Krzysztof tested it on an Odroid XU3 Lite and reported that disabling
>>> CCI caused some CPUs to fail to boot even with $subject applied:
>>>
>>> https://lkml.org/lkml/2015/8/29/65
>>>
>>> Did you succeed booting all CPUs with CONFIG_ARM_BIG_LITTLE_CPUIDLE
>>> enabled and CCI disabled in the the Odroid XU3 DTS?
I thought so, but re-testing I'm seein the same results as Krzysztof:
>> Indeed. On Odroid XU3 Chanho's patch gives me 8 CPUs up. Disabling CCI
>> causes fails on CPU{5,6,7} (Cortex-A15).
>
> Chanho's patch is for the exynos mcpm back-end. However, when we
> disable CCI the mcpm code is bypassed and we default to the code in
> exynos' platsmp.c/firmware.c. If the A7s were failing to boot-up then
> the reason could have been that Chanho's workaround was not being
> executed after applying the CCI disablement patch.
> According to the comments the A15s are not booting and so the
> exynos_boot_secondary function needs to be checked further.
Thanks for the explanation. That makes sense. $SUBJECT fix should
be made so it works for both scenarios.
Kevin
next prev parent reply other threads:[~2015-09-01 19:33 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-31 14:25 [PATCH] ARM: EXYNOS: reset KFC cores when cpu is up Chanho Park
2015-08-31 22:11 ` Kevin Hilman
2015-08-31 22:46 ` Javier Martinez Canillas
2015-09-01 0:21 ` Krzysztof Kozlowski
2015-09-01 14:47 ` Abhilash Kesavan
2015-09-01 19:33 ` Kevin Hilman [this message]
2015-09-01 0:08 ` Krzysztof Kozlowski
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=7hy4gq6ikt.fsf@deeprootsystems.com \
--to=khilman@kernel$(echo .)org \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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