From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753017AbbF2DDw (ORCPT ); Sun, 28 Jun 2015 23:03:52 -0400 Received: from mailgw02.mediatek.com ([210.61.82.184]:50614 "EHLO mailgw02.mediatek.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1752621AbbF2DDp (ORCPT ); Sun, 28 Jun 2015 23:03:45 -0400 X-Listener-Flag: 11101 Message-ID: <1435547019.20584.13.camel@mtksdaap41> Subject: Re: [RESEND PATCH 1/6] Document: bindings: DT: Add SMP enable method for MT6580 SoC platform From: Yingjoe Chen To: Scott Shu , Matthias Brugger , Arnd Bergmann , CC: , , , , , , , , , , , Date: Mon, 29 Jun 2015 11:03:39 +0800 In-Reply-To: <1434650481-39421-2-git-send-email-scott.shu@mediatek.com> References: <1434650481-39421-1-git-send-email-scott.shu@mediatek.com> <1434650481-39421-2-git-send-email-scott.shu@mediatek.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.2.3-0ubuntu6 Content-Transfer-Encoding: 7bit MIME-Version: 1.0 X-MTK: N Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 2015-06-19 at 02:01 +0800, Scott Shu wrote: > For MT6580 SoC platform, the secondary cores are in powered off state > as default, so compared with MT65xx series SoC, one new enable method > is needed. This method using the SPM (System Power Manager) inside > the SCYSYS to control the CPU power. > --- > Documentation/devicetree/bindings/arm/cpus.txt | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/arm/cpus.txt b/Documentation/devicetree/bindings/arm/cpus.txt > index ac2903d..fb80b2e 100644 > --- a/Documentation/devicetree/bindings/arm/cpus.txt > +++ b/Documentation/devicetree/bindings/arm/cpus.txt > @@ -194,6 +194,7 @@ nodes to be present and contain the properties described below. > "marvell,armada-380-smp" > "marvell,armada-390-smp" > "marvell,armada-xp-smp" > + "mediatek,mt6580-smp" > "mediatek,mt65xx-smp" > "mediatek,mt81xx-tz-smp" > "qcom,gcc-msm8660" Hi It seems we have 3 different kinds of cpu enable method now, and mt65xx-smp doesn't cover all mt65xx series. So maybe it make sense to change naming before it got merged. Short summary for these methods: mt65xx-smp: For mt65xx socs which wakeup all cores at boot. Tested on mt6589 by Matthias. mt6580-smp: Only first core is alive at boot, so need to wakeup other cores using SPM. AFAIK only for mt6580 now. mt81xx-tz-smp: For soc which wakeup all cores at boot, and have trustzone firmware. Suitable for mt8127, mt8135. I'm not sure about smp for mt6592, maybe Howard or Scott can comment on it. Any suggestion on how we should organize these methods? Joe.C From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yingjoe Chen Subject: Re: [RESEND PATCH 1/6] Document: bindings: DT: Add SMP enable method for MT6580 SoC platform Date: Mon, 29 Jun 2015 11:03:39 +0800 Message-ID: <1435547019.20584.13.camel@mtksdaap41> References: <1434650481-39421-1-git-send-email-scott.shu@mediatek.com> <1434650481-39421-2-git-send-email-scott.shu@mediatek.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1434650481-39421-2-git-send-email-scott.shu@mediatek.com> Sender: linux-kernel-owner@vger.kernel.org To: Scott Shu , Matthias Brugger , Arnd Bergmann , ibanezchen@gmail.com Cc: robh+dt@kernel.org, linux@arm.linux.org.uk, rchintakuntla@cavium.com, lorenzo.pieralisi@arm.com, marc.ceeeee@gmail.com, heiko@sntech.de, catalin.marinas@arm.com, galak@codeaurora.org, srv_wsdupstream@mediatek.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org List-Id: linux-mediatek@lists.infradead.org On Fri, 2015-06-19 at 02:01 +0800, Scott Shu wrote: > For MT6580 SoC platform, the secondary cores are in powered off state > as default, so compared with MT65xx series SoC, one new enable method > is needed. This method using the SPM (System Power Manager) inside > the SCYSYS to control the CPU power. > --- > Documentation/devicetree/bindings/arm/cpus.txt | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/arm/cpus.txt b/Documentation/devicetree/bindings/arm/cpus.txt > index ac2903d..fb80b2e 100644 > --- a/Documentation/devicetree/bindings/arm/cpus.txt > +++ b/Documentation/devicetree/bindings/arm/cpus.txt > @@ -194,6 +194,7 @@ nodes to be present and contain the properties described below. > "marvell,armada-380-smp" > "marvell,armada-390-smp" > "marvell,armada-xp-smp" > + "mediatek,mt6580-smp" > "mediatek,mt65xx-smp" > "mediatek,mt81xx-tz-smp" > "qcom,gcc-msm8660" Hi It seems we have 3 different kinds of cpu enable method now, and mt65xx-smp doesn't cover all mt65xx series. So maybe it make sense to change naming before it got merged. Short summary for these methods: mt65xx-smp: For mt65xx socs which wakeup all cores at boot. Tested on mt6589 by Matthias. mt6580-smp: Only first core is alive at boot, so need to wakeup other cores using SPM. AFAIK only for mt6580 now. mt81xx-tz-smp: For soc which wakeup all cores at boot, and have trustzone firmware. Suitable for mt8127, mt8135. I'm not sure about smp for mt6592, maybe Howard or Scott can comment on it. Any suggestion on how we should organize these methods? Joe.C From mboxrd@z Thu Jan 1 00:00:00 1970 From: yingjoe.chen@mediatek.com (Yingjoe Chen) Date: Mon, 29 Jun 2015 11:03:39 +0800 Subject: [RESEND PATCH 1/6] Document: bindings: DT: Add SMP enable method for MT6580 SoC platform In-Reply-To: <1434650481-39421-2-git-send-email-scott.shu@mediatek.com> References: <1434650481-39421-1-git-send-email-scott.shu@mediatek.com> <1434650481-39421-2-git-send-email-scott.shu@mediatek.com> Message-ID: <1435547019.20584.13.camel@mtksdaap41> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, 2015-06-19 at 02:01 +0800, Scott Shu wrote: > For MT6580 SoC platform, the secondary cores are in powered off state > as default, so compared with MT65xx series SoC, one new enable method > is needed. This method using the SPM (System Power Manager) inside > the SCYSYS to control the CPU power. > --- > Documentation/devicetree/bindings/arm/cpus.txt | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/arm/cpus.txt b/Documentation/devicetree/bindings/arm/cpus.txt > index ac2903d..fb80b2e 100644 > --- a/Documentation/devicetree/bindings/arm/cpus.txt > +++ b/Documentation/devicetree/bindings/arm/cpus.txt > @@ -194,6 +194,7 @@ nodes to be present and contain the properties described below. > "marvell,armada-380-smp" > "marvell,armada-390-smp" > "marvell,armada-xp-smp" > + "mediatek,mt6580-smp" > "mediatek,mt65xx-smp" > "mediatek,mt81xx-tz-smp" > "qcom,gcc-msm8660" Hi It seems we have 3 different kinds of cpu enable method now, and mt65xx-smp doesn't cover all mt65xx series. So maybe it make sense to change naming before it got merged. Short summary for these methods: mt65xx-smp: For mt65xx socs which wakeup all cores at boot. Tested on mt6589 by Matthias. mt6580-smp: Only first core is alive at boot, so need to wakeup other cores using SPM. AFAIK only for mt6580 now. mt81xx-tz-smp: For soc which wakeup all cores at boot, and have trustzone firmware. Suitable for mt8127, mt8135. I'm not sure about smp for mt6592, maybe Howard or Scott can comment on it. Any suggestion on how we should organize these methods? Joe.C