From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 1/8] ARM: mediatek: Add basic support for mt8127
Date: Tue, 04 Nov 2014 08:39:55 +0100 [thread overview]
Message-ID: <13477264.GuOdydWub7@wuerfel> (raw)
In-Reply-To: <1415083006.10186.85.camel@mtksdaap41>
On Tuesday 04 November 2014 14:36:45 HC Yen wrote:
> > > +
> > > +#include <dt-bindings/interrupt-controller/irq.h>
> > > +#include <dt-bindings/interrupt-controller/arm-gic.h>
> > > +#include "skeleton64.dtsi"
> >
> > Cortex a7 is 32 bits, right? So why do you use skeleton64.dtsi?
>
> Cortex-A7 is 32-bit, but that doesn't mean it can only have 32-bit
> physical address. With LPAE enabled, we can have physical address more
> than 32 bits.
>
> The main difference between "skeleton64.dtsi" and "skeleton.dtsi" is
> "#address-cells" property set to 2. Although there are few sources
> using "skeleton64.dtsi", some of them write "#address-cells = <2>"
> directly in order to have 64-bit address space. ARM's TC2 reference
> platform (vexpress-v2p-ca15_a7.dts) is an example.
>
> Some of MediaTek ARMv7 SoCs support address space larger than 4GB. It
> will be convenient to share the sources if we all use 64-bit device
> tree.
Right, in general, I'd use #address-cells=<2> for Cortex-A7/A15/A17.
Arnd
next prev parent reply other threads:[~2014-11-04 7:39 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-22 10:29 [PATCH v4 0/8] arm: Add basic support for MediaTek MT8127 & MT8135 SoCs Joe.C
2014-10-22 10:29 ` [PATCH v4 1/8] ARM: mediatek: Add basic support for mt8127 Joe.C
2014-11-03 17:53 ` Matthias Brugger
2014-11-04 6:36 ` HC Yen
2014-11-04 7:39 ` Arnd Bergmann [this message]
2014-11-04 8:36 ` Matthias Brugger
2014-11-04 9:45 ` Arnd Bergmann
2014-10-22 10:29 ` [PATCH v4 2/8] ARM: mediatek: add dts for 8127 Moose board Joe.C
2014-11-03 17:55 ` Matthias Brugger
2014-11-04 5:44 ` Yingjoe Chen
2014-10-22 10:29 ` [PATCH v4 3/8] ARM: mediatek: Add basic support for mt8135 Joe.C
2014-11-03 17:58 ` Matthias Brugger
2014-10-22 10:29 ` [PATCH v4 4/8] ARM: mediatek: add dts for MT8135 evaluation board Joe.C
2014-11-03 17:58 ` Matthias Brugger
2014-10-22 10:29 ` [PATCH v4 5/8] dt-bindings: add more chips in documentation for Mediatek SoC Joe.C
2014-10-22 10:29 ` [PATCH v4 6/8] ARM: dts: Build dtb for mt8127 & mt8135 Joe.C
2014-10-22 10:29 ` [PATCH v4 7/8] ARM: mediatek: Add earlyprintk support " Joe.C
2014-10-22 10:29 ` [PATCH v4 8/8] ARM: mediatek: Add config option for mediatek SoCs Joe.C
2014-10-22 13:18 ` Matthias Brugger
2014-10-22 13:31 ` Yingjoe Chen
2014-10-22 13:53 ` Matthias Brugger
2014-10-22 14:19 ` Yingjoe Chen
2014-10-24 5:37 ` [PATCH v4 8/8] ARM: mediatek: Fix description " Yingjoe Chen
2014-10-31 1:37 ` Yingjoe Chen
2014-10-31 14:53 ` Matthias Brugger
2014-11-03 17:51 ` Matthias Brugger
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=13477264.GuOdydWub7@wuerfel \
--to=arnd@arndb.de \
--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