From: Arnd Bergmann <arnd@arndb.de>
To: Tushar Behera <tushar.behera@linaro.org>
Cc: Tomasz Figa <t.figa@samsung.com>,
Kyungmin Park <kyungmin.park@samsung.com>,
Kgene Kim <kgene.kim@samsung.com>,
linux-samsung-soc <linux-samsung-soc@vger.kernel.org>
Subject: Re: Boot hang on Origen with (!SMP && CPU_IDLE)
Date: Mon, 6 Jan 2014 16:30:56 +0100 [thread overview]
Message-ID: <201401061630.56740.arnd@arndb.de> (raw)
In-Reply-To: <CAHbNUh18WkH4CFcaeP1Tdjm3L-Rr0xVzgCnAHoPbbCFRWjKyxg@mail.gmail.com>
On Monday 06 January 2014, Tushar Behera wrote:
> The device tree node for l2x0 device was missing. After adding a node
> as below I can start booting Origen board.
>
> diff --git a/arch/arm/boot/dts/exynos4210-origen.dts
> b/arch/arm/boot/dts/exynos4210-origen.dts
> index 1a12fb2..675f323 100644
> --- a/arch/arm/boot/dts/exynos4210-origen.dts
> +++ b/arch/arm/boot/dts/exynos4210-origen.dts
> @@ -32,6 +32,13 @@
>
> + l2-cache-controller@10502000 {
> + compatible = "arm,pl310-cache";
> + reg = <0x10502000 0x1000>;
> + cache-unified;
> + cache-level = <2>;
> + };
> +
Ok, very good!
> >> be a good time to get rid of the L2_AUX_VAL and L2_AUX_MASK defines and
> >> just read the respective settings from DT.
>
> Ok.
Does the node you list above have the right settings for this?
> > 2) There is no L2 cache controller node in Exynos4*.dtsi.
> >
> > It should be added, but L2 cache can't be enabled on all boards yet,
> > since on boards where secure firmware is enabled, special configuration
> > involving SMC calls is required. Patches for this are queued on my work
> > queue, but it's quite tricky due to 1), which needs to consider whether
> > secure firmware is enabled or not.
> >
>
> In that case, would it be ok to add the node for Origen board only?
Wouldn't that leave other systems still broken? I'm particularly worried
about what patch to backport to linux-stable. We should definitely add
the node for Origen, but we may also have to revert my broken patch
in all affected kernel versions.
Are there any systems that may or may not have secure firmware enabled
depending on the boot loader, or do we always know whether secure firmware
is there or not?
Arnd
next prev parent reply other threads:[~2014-01-06 15:31 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-03 11:49 Boot hang on Origen with (!SMP && CPU_IDLE) Tushar Behera
2014-01-03 13:37 ` Arnd Bergmann
2014-01-03 13:58 ` Tomasz Figa
2014-01-06 10:31 ` Tushar Behera
2014-01-06 15:30 ` Arnd Bergmann [this message]
2014-01-06 16:41 ` Tomasz Figa
2014-01-07 7:03 ` Tushar Behera
2014-01-07 8:51 ` Arnd Bergmann
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=201401061630.56740.arnd@arndb.de \
--to=arnd@arndb.de \
--cc=kgene.kim@samsung.com \
--cc=kyungmin.park@samsung.com \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=t.figa@samsung.com \
--cc=tushar.behera@linaro.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 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.