* 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@vger.kernel.org, Russell King, srv_heupstream, Pawel Moll, Ian Campbell, HC Yen, Yuhau Chen, linux-kernel@vger.kernel.org, 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@vger.kernel.org, Russell King, srv_heupstream, Pawel Moll, Ian Campbell, HC Yen, Yuhau Chen, linux-kernel@vger.kernel.org, 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