From: Arnd Bergmann <arnd@arndb.de>
To: HC Yen <hc.yen@mediatek.com>
Cc: Matthias Brugger <matthias.bgg@gmail.com>,
"Joe. C" <yingjoe.chen@mediatek.com>,
Rob Herring <robh+dt@kernel.org>,
arm@kernel.org, Pawel Moll <pawel.moll@arm.com>,
Mark Rutland <mark.rutland@arm.com>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
Kumar Gala <galak@codeaurora.org>,
Russell King <linux@arm.linux.org.uk>,
Olof Johansson <olof@lixom.net>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
srv_heupstream <srv_heupstream@mediatek.com>,
Yingjoe Chen <yingjoe.chen@gmail.com>,
huang eddie <eddie.huang@mediatek.com>,
Nathan Chung <nathan.chung@mediatek.com>,
Yuhau Chen <yh.chen@mediatek.com>,
Sascha Hauer <kernel@pengutronix.de>
Subject: Re: [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:41 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1413973797-17619-1-git-send-email-yingjoe.chen@mediatek.com>
[not found] ` <1413973797-17619-9-git-send-email-yingjoe.chen@mediatek.com>
2014-10-22 13:18 ` [PATCH v4 8/8] ARM: mediatek: Add config option for mediatek SoCs 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
[not found] ` <1413973797-17619-2-git-send-email-yingjoe.chen@mediatek.com>
2014-11-03 17:53 ` [PATCH v4 1/8] ARM: mediatek: Add basic support for mt8127 Matthias Brugger
[not found] ` <1415083006.10186.85.camel@mtksdaap41>
2014-11-04 7:39 ` Arnd Bergmann [this message]
2014-11-04 8:36 ` Matthias Brugger
2014-11-04 9:45 ` Arnd Bergmann
[not found] ` <1413973797-17619-3-git-send-email-yingjoe.chen@mediatek.com>
2014-11-03 17:55 ` [PATCH v4 2/8] ARM: mediatek: add dts for 8127 Moose board Matthias Brugger
2014-11-04 5:44 ` Yingjoe Chen
[not found] ` <1413973797-17619-4-git-send-email-yingjoe.chen@mediatek.com>
2014-11-03 17:58 ` [PATCH v4 3/8] ARM: mediatek: Add basic support for mt8135 Matthias Brugger
[not found] ` <1413973797-17619-5-git-send-email-yingjoe.chen@mediatek.com>
2014-11-03 17:58 ` [PATCH v4 4/8] ARM: mediatek: add dts for MT8135 evaluation board 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=arm@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=eddie.huang@mediatek.com \
--cc=galak@codeaurora.org \
--cc=hc.yen@mediatek.com \
--cc=ijc+devicetree@hellion.org.uk \
--cc=kernel@pengutronix.de \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=mark.rutland@arm.com \
--cc=matthias.bgg@gmail.com \
--cc=nathan.chung@mediatek.com \
--cc=olof@lixom.net \
--cc=pawel.moll@arm.com \
--cc=robh+dt@kernel.org \
--cc=srv_heupstream@mediatek.com \
--cc=yh.chen@mediatek.com \
--cc=yingjoe.chen@gmail.com \
--cc=yingjoe.chen@mediatek.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: 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