* Re: [PATCH v5' 1/8] ARM: mediatek: Add basic support for mt8127
[not found] ` <1415692428-18416-1-git-send-email-yingjoe.chen@mediatek.com>
@ 2014-11-11 7:58 ` Yingjoe Chen
2014-11-12 17:08 ` Matthias Brugger
0 siblings, 1 reply; 3+ messages in thread
From: Yingjoe Chen @ 2014-11-11 7:58 UTC (permalink / raw)
To: Arnd Bergmann, Matthias Brugger
Cc: Mark Rutland, devicetree, Russell King, srv_heupstream,
Pawel Moll, Ian Campbell, hc.yen, yh.chen, linux-kernel,
Rob Herring, nathan.chung, Sascha Hauer, Kumar Gala,
Olof Johansson, eddie.huang, yingjoe.chen
Hi, Matthias, Arnd,
After previous discussion[1], I'm not sure which one do you preferred
for mt8127/mt8135, so I send 64bits address version again. Trying to not
SPAM too much, I only send the difference(first 4 patches) this time.
We do have SoC that need >4GB address, so at least those will need to
use 64bits address. If we want to have a consistent look, 64bits is the
way to go. Also we already have some patches under review now using
64bits address. So I preferred this one, but both versions are OK to me.
Please help to review this and let me know anything you want me to
change. Thanks.
Joe.C
[1]
http://lists.infradead.org/pipermail/linux-arm-kernel/2014-November/299598.html
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v5' 1/8] ARM: mediatek: Add basic support for mt8127
2014-11-11 7:58 ` [PATCH v5' 1/8] ARM: mediatek: Add basic support for mt8127 Yingjoe Chen
@ 2014-11-12 17:08 ` Matthias Brugger
2014-11-13 10:17 ` Yingjoe Chen
0 siblings, 1 reply; 3+ messages in thread
From: Matthias Brugger @ 2014-11-12 17:08 UTC (permalink / raw)
To: Yingjoe Chen
Cc: Arnd Bergmann, Mark Rutland, devicetree, Russell King,
srv_heupstream, Pawel Moll, Ian Campbell, HC Yen, Yuhau Chen,
linux-kernel, Rob Herring, Nathan Chung, Sascha Hauer,
Kumar Gala, Olof Johansson, huang eddie, Yingjoe Chen
Hi Joe,
2014-11-11 8:58 GMT+01:00 Yingjoe Chen <yingjoe.chen@mediatek.com>:
>
> Hi, Matthias, Arnd,
>
> After previous discussion[1], I'm not sure which one do you preferred
> for mt8127/mt8135, so I send 64bits address version again. Trying to not
> SPAM too much, I only send the difference(first 4 patches) this time.
>
> We do have SoC that need >4GB address, so at least those will need to
> use 64bits address. If we want to have a consistent look, 64bits is the
> way to go. Also we already have some patches under review now using
> 64bits address. So I preferred this one, but both versions are OK to me.
Regarding Arnd's last email, the 64 bit is the right way.
Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com>
>
> Please help to review this and let me know anything you want me to
> change. Thanks.
>
> Joe.C
>
> [1]
> http://lists.infradead.org/pipermail/linux-arm-kernel/2014-November/299598.html
>
>
>
--
motzblog.wordpress.com
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v5' 1/8] ARM: mediatek: Add basic support for mt8127
2014-11-12 17:08 ` Matthias Brugger
@ 2014-11-13 10:17 ` Yingjoe Chen
0 siblings, 0 replies; 3+ messages in thread
From: Yingjoe Chen @ 2014-11-13 10:17 UTC (permalink / raw)
To: Matthias Brugger
Cc: Arnd Bergmann, Mark Rutland, devicetree, Russell King,
srv_heupstream, Pawel Moll, Ian Campbell, HC Yen, Yuhau Chen,
linux-kernel, Rob Herring, Nathan Chung, Sascha Hauer,
Kumar Gala, Olof Johansson, huang eddie, Yingjoe Chen
On Wed, 2014-11-12 at 18:08 +0100, Matthias Brugger wrote:
> Hi Joe,
>
> 2014-11-11 8:58 GMT+01:00 Yingjoe Chen <yingjoe.chen@mediatek.com>:
> >
> > Hi, Matthias, Arnd,
> >
> > After previous discussion[1], I'm not sure which one do you preferred
> > for mt8127/mt8135, so I send 64bits address version again. Trying to not
> > SPAM too much, I only send the difference(first 4 patches) this time.
> >
> > We do have SoC that need >4GB address, so at least those will need to
> > use 64bits address. If we want to have a consistent look, 64bits is the
> > way to go. Also we already have some patches under review now using
> > 64bits address. So I preferred this one, but both versions are OK to me.
>
> Regarding Arnd's last email, the 64 bit is the right way.
Hi Matthias,
Ok, please use this v5' then.
Thanks,
Joe.C
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-11-13 10:17 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1415086255-5555-1-git-send-email-yingjoe.chen@mediatek.com>
[not found] ` <1415692428-18416-1-git-send-email-yingjoe.chen@mediatek.com>
2014-11-11 7:58 ` [PATCH v5' 1/8] ARM: mediatek: Add basic support for mt8127 Yingjoe Chen
2014-11-12 17:08 ` Matthias Brugger
2014-11-13 10:17 ` Yingjoe Chen
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).