From: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
To: HC Yen <hc.yen-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
Cc: Matthias Brugger
<matthias.bgg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
"Joe. C" <yingjoe.chen-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>,
Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org>,
Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
Ian Campbell
<ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>,
Kumar Gala <galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>,
Russell King <linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org>,
Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org>,
"devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
srv_heupstream
<srv_heupstream-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>,
Yingjoe Chen
<yingjoe.chen-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
huang eddie <eddie.huang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>,
Nathan Chung
<nathan.chung-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>,
Yuhau Chen <yh.chen-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>,
Sascha Hauer <kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
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
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
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
[not found] ` <1413973797-17619-2-git-send-email-yingjoe.chen-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
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
[not found] ` <CABuKBeL1JX88BBK7uXqpmaGzwCe5D6inB_n-NV6ajUt59V1swA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
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
[not found] ` <CABuKBe+ku=vWiWbvAm8M7QgyzvVWZ5sdkRMocuUym8JTyk-okg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
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
[not found] ` <CABuKBeLJuaDnCYuvvS6csNzvbTeEdgEUdZ4tBQeVPuw37JK11Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
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
[not found] ` <CABuKBeKFZQAHPR4xjCnM+nz67zE79rWPTHbc=m3VKcCX54Q8rQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
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-r2ngtmty4d4@public.gmane.org \
--cc=arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=eddie.huang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org \
--cc=galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \
--cc=hc.yen-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org \
--cc=ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org \
--cc=kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org \
--cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
--cc=matthias.bgg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=nathan.chung-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org \
--cc=olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org \
--cc=pawel.moll-5wv7dgnIgG8@public.gmane.org \
--cc=robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=srv_heupstream-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org \
--cc=yh.chen-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org \
--cc=yingjoe.chen-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org \
--cc=yingjoe.chen-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.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;
as well as URLs for NNTP newsgroup(s).