All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
From: Yingjoe Chen <yingjoe.chen@mediatek.com>
To: Scott Shu <scott.shu@mediatek.com>,
	Matthias Brugger <matthias.bgg@gmail.com>,
	Arnd Bergmann <arnd@arndb.de>, <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>
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	[thread overview]
Message-ID: <1435547019.20584.13.camel@mtksdaap41> (raw)
In-Reply-To: <1434650481-39421-2-git-send-email-scott.shu@mediatek.com>

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



WARNING: multiple messages have this Message-ID (diff)
From: Yingjoe Chen <yingjoe.chen@mediatek.com>
To: Scott Shu <scott.shu@mediatek.com>,
	Matthias Brugger <matthias.bgg@gmail.com>,
	Arnd Bergmann <arnd@arndb.de>,
	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
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	[thread overview]
Message-ID: <1435547019.20584.13.camel@mtksdaap41> (raw)
In-Reply-To: <1434650481-39421-2-git-send-email-scott.shu@mediatek.com>

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

WARNING: multiple messages have this Message-ID (diff)
From: yingjoe.chen@mediatek.com (Yingjoe Chen)
To: linux-arm-kernel@lists.infradead.org
Subject: [RESEND PATCH 1/6] Document: bindings: DT: Add SMP enable method for MT6580 SoC platform
Date: Mon, 29 Jun 2015 11:03:39 +0800	[thread overview]
Message-ID: <1435547019.20584.13.camel@mtksdaap41> (raw)
In-Reply-To: <1434650481-39421-2-git-send-email-scott.shu@mediatek.com>

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

  reply	other threads:[~2015-06-29  3:03 UTC|newest]

Thread overview: 75+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <Scott Shu <srv_scott.shu@mediatek.com>
     [not found] ` <Scott Shu <srv_scott.shu-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
2015-06-18  2:46   ` [PATCH 0/6] This series adds SMP support for the MediaTek MT6580 Scott Shu
     [not found]     ` <1434595587-25466-1-git-send-email-scott.shu-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
2015-06-18  2:46       ` [PATCH 1/6] Document: bindings: DT: Add SMP enable method for MT6580 SoC platform Scott Shu
2015-06-18  2:46       ` [PATCH 2/6] soc: Mediatek: Add SCPSYS CPU power domain driver Scott Shu
2015-06-18  2:46       ` [PATCH 3/6] ARM: mediatek: add smp bringup code for MT6580 Scott Shu
2015-06-18  2:46       ` [PATCH 4/6] ARM: Mediatek: enable GPT6 on boot up to make arch timer working " Scott Shu
2015-06-18  2:46       ` [PATCH 5/6] ARM: dts: mt6580: Add device nodes to the MT6580 dtsi file Scott Shu
2015-06-18  2:46       ` [PATCH 6/6] ARM: dts: mt6580: enable basic SMP bringup for mt6580 Scott Shu
2015-06-18 16:12       ` [PATCH 0/6] This series adds SMP support for the MediaTek MT6580 Matthias Brugger
     [not found]         ` <CABuKBeK-WLFnrOnPYmGR1cTfecHhq+_QyAqQjJpYfuA_MNW1aw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-06-24  5:56           ` Scott Shu (舒祖襄)
2015-06-18  3:35 ` Scott Shu
2015-06-18  3:35   ` Scott Shu
2015-06-18  3:35   ` [PATCH 1/6] Document: bindings: DT: Add SMP enable method for MT6580 SoC platform Scott Shu
2015-06-18  3:35     ` Scott Shu
2015-06-18  3:35   ` [PATCH 2/6] soc: Mediatek: Add SCPSYS CPU power domain driver Scott Shu
2015-06-18  3:35     ` Scott Shu
2015-06-18  3:35   ` [PATCH 3/6] ARM: mediatek: add smp bringup code for MT6580 Scott Shu
2015-06-18  3:35     ` Scott Shu
2015-06-18  3:35   ` [PATCH 4/6] ARM: Mediatek: enable GPT6 on boot up to make arch timer working " Scott Shu
2015-06-18  3:35     ` Scott Shu
2015-06-18  3:35   ` [PATCH 5/6] ARM: dts: mt6580: Add device nodes to the MT6580 dtsi file Scott Shu
2015-06-18  3:35     ` Scott Shu
2015-06-18  3:35   ` [PATCH 6/6] ARM: dts: mt6580: enable basic SMP bringup for mt6580 Scott Shu
2015-06-18  3:35     ` Scott Shu
2015-06-18 18:01 ` [RESEND PATCH 0/6] This series adds SMP support for the MediaTek MT6580 Scott Shu
2015-06-18 18:01   ` Scott Shu
2015-06-18 18:01   ` Scott Shu
2015-06-18 18:01   ` [RESEND PATCH 1/6] Document: bindings: DT: Add SMP enable method for MT6580 SoC platform Scott Shu
2015-06-18 18:01     ` Scott Shu
2015-06-18 18:01     ` Scott Shu
2015-06-29  3:03     ` Yingjoe Chen [this message]
2015-06-29  3:03       ` Yingjoe Chen
2015-06-29  3:03       ` Yingjoe Chen
2015-07-11 10:38       ` Yingjoe Chen
2015-07-11 10:38         ` Yingjoe Chen
2015-07-11 10:38         ` Yingjoe Chen
2015-07-17 22:04         ` Matthias Brugger
2015-07-17 22:04           ` Matthias Brugger
2015-06-18 18:01   ` [RESEND PATCH 2/6] soc: Mediatek: Add SCPSYS CPU power domain driver Scott Shu
2015-06-18 18:01     ` Scott Shu
2015-06-18 18:01     ` Scott Shu
2015-06-23  5:53     ` Sascha Hauer
2015-06-23  5:53       ` Sascha Hauer
2015-06-23  5:53       ` Sascha Hauer
2015-07-03  5:58       ` Scott Shu
2015-07-03  5:58         ` Scott Shu
2015-07-03  5:58         ` Scott Shu
2015-07-24  2:02       ` Scott Shu
2015-07-24  2:02         ` Scott Shu
2015-07-24  2:02         ` Scott Shu
2015-07-24  5:50         ` Sascha Hauer
2015-07-24  5:50           ` Sascha Hauer
2015-07-25  9:13           ` Scott Shu
2015-07-25  9:13             ` Scott Shu
2015-07-25  9:13             ` Scott Shu
2015-07-27 10:22             ` Matthias Brugger
2015-07-27 10:22               ` Matthias Brugger
2015-06-18 18:01   ` [RESEND PATCH 3/6] ARM: mediatek: add smp bringup code for MT6580 Scott Shu
2015-06-18 18:01     ` Scott Shu
2015-06-18 18:01     ` Scott Shu
2015-06-18 18:01   ` [RESEND PATCH 4/6] ARM: Mediatek: enable GPT6 on boot up to make arch timer working " Scott Shu
2015-06-18 18:01     ` Scott Shu
2015-06-18 18:01     ` Scott Shu
2015-06-18 18:01   ` [RESEND PATCH 5/6] ARM: dts: mt6580: Add device nodes to the MT6580 dtsi file Scott Shu
2015-06-18 18:01     ` Scott Shu
2015-06-18 18:01     ` Scott Shu
2015-06-18 18:01   ` [RESEND PATCH 6/6] ARM: dts: mt6580: enable basic SMP bringup for mt6580 Scott Shu
2015-06-18 18:01     ` Scott Shu
2015-06-18 18:01     ` Scott Shu
2015-06-18 18:23   ` [RESEND PATCH 0/6] This series adds SMP support for the MediaTek MT6580 Alexey Klimov
2015-06-18 18:23     ` Alexey Klimov
2015-07-24 12:59   ` Matthias Brugger
2015-07-24 12:59     ` Matthias Brugger
2015-07-24 13:10     ` Scott Shu
2015-07-24 13:10       ` Scott Shu
2015-07-24 13:10       ` Scott Shu

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=1435547019.20584.13.camel@mtksdaap41 \
    --to=yingjoe.chen@mediatek.com \
    --cc=arnd@arndb.de \
    --cc=catalin.marinas@arm.com \
    --cc=galak@codeaurora.org \
    --cc=heiko@sntech.de \
    --cc=ibanezchen@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=linux@arm.linux.org.uk \
    --cc=lorenzo.pieralisi@arm.com \
    --cc=marc.ceeeee@gmail.com \
    --cc=matthias.bgg@gmail.com \
    --cc=rchintakuntla@cavium.com \
    --cc=robh+dt@kernel.org \
    --cc=scott.shu@mediatek.com \
    --cc=srv_wsdupstream@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.