diff mbox

[v2,1/4] dt-bindings: soc: add SCPSYS binding for MT7623 and MT7623A SoC

Message ID ecfc548fc3d22aa335b50e552ad2ad88427a0bb2.1517998367.git.sean.wang@mediatek.com (mailing list archive)
State Not Applicable, archived
Headers show

Commit Message

Sean Wang Feb. 7, 2018, 10:22 a.m. UTC
From: Sean Wang <sean.wang@mediatek.com>

document the binding for enabling SCPSYS on MediaTek MT7623 and MT7623A
SoC. Where MT7623 SoC has the same definition about power domains with
MT2701, so it's fine to using MT2701 ones as MT7623's fallback.

Signed-off-by: Sean Wang <sean.wang@mediatek.com>
Reviewed-by: Rob Herring <robh@kernel.org>
---
 Documentation/devicetree/bindings/soc/mediatek/scpsys.txt | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

Comments

Ulf Hansson Feb. 7, 2018, 8:19 p.m. UTC | #1
On 7 February 2018 at 11:22,  <sean.wang@mediatek.com> wrote:
> From: Sean Wang <sean.wang@mediatek.com>
>
> document the binding for enabling SCPSYS on MediaTek MT7623 and MT7623A
> SoC. Where MT7623 SoC has the same definition about power domains with
> MT2701, so it's fine to using MT2701 ones as MT7623's fallback.
>
> Signed-off-by: Sean Wang <sean.wang@mediatek.com>
> Reviewed-by: Rob Herring <robh@kernel.org>

For the series:

Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>

Kind regards
Uffe

> ---
>  Documentation/devicetree/bindings/soc/mediatek/scpsys.txt | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/soc/mediatek/scpsys.txt b/Documentation/devicetree/bindings/soc/mediatek/scpsys.txt
> index 76bf45b..d6fe16f 100644
> --- a/Documentation/devicetree/bindings/soc/mediatek/scpsys.txt
> +++ b/Documentation/devicetree/bindings/soc/mediatek/scpsys.txt
> @@ -21,6 +21,8 @@ Required properties:
>         - "mediatek,mt2712-scpsys"
>         - "mediatek,mt6797-scpsys"
>         - "mediatek,mt7622-scpsys"
> +       - "mediatek,mt7623-scpsys", "mediatek,mt2701-scpsys": For MT7623 SoC
> +       - "mediatek,mt7623a-scpsys": For MT7623A SoC
>         - "mediatek,mt8173-scpsys"
>  - #power-domain-cells: Must be 1
>  - reg: Address range of the SCPSYS unit
> @@ -28,10 +30,11 @@ Required properties:
>  - clock, clock-names: clocks according to the common clock binding.
>                        These are clocks which hardware needs to be
>                        enabled before enabling certain power domains.
> -       Required clocks for MT2701: "mm", "mfg", "ethif"
> +       Required clocks for MT2701 or MT7623: "mm", "mfg", "ethif"
>         Required clocks for MT2712: "mm", "mfg", "venc", "jpgdec", "audio", "vdec"
>         Required clocks for MT6797: "mm", "mfg", "vdec"
>         Required clocks for MT7622: "hif_sel"
> +       Required clocks for MT7622A: "ethif"
>         Required clocks for MT8173: "mm", "mfg", "venc", "venc_lt"
>
>  Optional properties:
> --
> 2.7.4
>
Sean Wang March 7, 2018, 9:17 a.m. UTC | #2
Hi, Matthias

just a gentle ping to the four related patches

	Sean

On Wed, 2018-02-07 at 18:22 +0800, sean.wang@mediatek.com wrote:
> From: Sean Wang <sean.wang@mediatek.com>
> 
> document the binding for enabling SCPSYS on MediaTek MT7623 and MT7623A
> SoC. Where MT7623 SoC has the same definition about power domains with
> MT2701, so it's fine to using MT2701 ones as MT7623's fallback.
> 
> Signed-off-by: Sean Wang <sean.wang@mediatek.com>
> Reviewed-by: Rob Herring <robh@kernel.org>
> ---
>  Documentation/devicetree/bindings/soc/mediatek/scpsys.txt | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/soc/mediatek/scpsys.txt b/Documentation/devicetree/bindings/soc/mediatek/scpsys.txt
> index 76bf45b..d6fe16f 100644
> --- a/Documentation/devicetree/bindings/soc/mediatek/scpsys.txt
> +++ b/Documentation/devicetree/bindings/soc/mediatek/scpsys.txt
> @@ -21,6 +21,8 @@ Required properties:
>  	- "mediatek,mt2712-scpsys"
>  	- "mediatek,mt6797-scpsys"
>  	- "mediatek,mt7622-scpsys"
> +	- "mediatek,mt7623-scpsys", "mediatek,mt2701-scpsys": For MT7623 SoC
> +	- "mediatek,mt7623a-scpsys": For MT7623A SoC
>  	- "mediatek,mt8173-scpsys"
>  - #power-domain-cells: Must be 1
>  - reg: Address range of the SCPSYS unit
> @@ -28,10 +30,11 @@ Required properties:
>  - clock, clock-names: clocks according to the common clock binding.
>                        These are clocks which hardware needs to be
>                        enabled before enabling certain power domains.
> -	Required clocks for MT2701: "mm", "mfg", "ethif"
> +	Required clocks for MT2701 or MT7623: "mm", "mfg", "ethif"
>  	Required clocks for MT2712: "mm", "mfg", "venc", "jpgdec", "audio", "vdec"
>  	Required clocks for MT6797: "mm", "mfg", "vdec"
>  	Required clocks for MT7622: "hif_sel"
> +	Required clocks for MT7622A: "ethif"
>  	Required clocks for MT8173: "mm", "mfg", "venc", "venc_lt"
>  
>  Optional properties:
Matthias Brugger March 11, 2018, 10:44 p.m. UTC | #3
On 03/07/2018 10:17 AM, Sean Wang wrote:
> Hi, Matthias
> 
> just a gentle ping to the four related patches
> 

All four pushed with Ulf's reviewed-by to v4.16-next/soc

Thanks a lot!

> 	Sean
> 
> On Wed, 2018-02-07 at 18:22 +0800, sean.wang@mediatek.com wrote:
>> From: Sean Wang <sean.wang@mediatek.com>
>>
>> document the binding for enabling SCPSYS on MediaTek MT7623 and MT7623A
>> SoC. Where MT7623 SoC has the same definition about power domains with
>> MT2701, so it's fine to using MT2701 ones as MT7623's fallback.
>>
>> Signed-off-by: Sean Wang <sean.wang@mediatek.com>
>> Reviewed-by: Rob Herring <robh@kernel.org>
>> ---
>>  Documentation/devicetree/bindings/soc/mediatek/scpsys.txt | 5 ++++-
>>  1 file changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/Documentation/devicetree/bindings/soc/mediatek/scpsys.txt b/Documentation/devicetree/bindings/soc/mediatek/scpsys.txt
>> index 76bf45b..d6fe16f 100644
>> --- a/Documentation/devicetree/bindings/soc/mediatek/scpsys.txt
>> +++ b/Documentation/devicetree/bindings/soc/mediatek/scpsys.txt
>> @@ -21,6 +21,8 @@ Required properties:
>>  	- "mediatek,mt2712-scpsys"
>>  	- "mediatek,mt6797-scpsys"
>>  	- "mediatek,mt7622-scpsys"
>> +	- "mediatek,mt7623-scpsys", "mediatek,mt2701-scpsys": For MT7623 SoC
>> +	- "mediatek,mt7623a-scpsys": For MT7623A SoC
>>  	- "mediatek,mt8173-scpsys"
>>  - #power-domain-cells: Must be 1
>>  - reg: Address range of the SCPSYS unit
>> @@ -28,10 +30,11 @@ Required properties:
>>  - clock, clock-names: clocks according to the common clock binding.
>>                        These are clocks which hardware needs to be
>>                        enabled before enabling certain power domains.
>> -	Required clocks for MT2701: "mm", "mfg", "ethif"
>> +	Required clocks for MT2701 or MT7623: "mm", "mfg", "ethif"
>>  	Required clocks for MT2712: "mm", "mfg", "venc", "jpgdec", "audio", "vdec"
>>  	Required clocks for MT6797: "mm", "mfg", "vdec"
>>  	Required clocks for MT7622: "hif_sel"
>> +	Required clocks for MT7622A: "ethif"
>>  	Required clocks for MT8173: "mm", "mfg", "venc", "venc_lt"
>>  
>>  Optional properties:
> 
>
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/soc/mediatek/scpsys.txt b/Documentation/devicetree/bindings/soc/mediatek/scpsys.txt
index 76bf45b..d6fe16f 100644
--- a/Documentation/devicetree/bindings/soc/mediatek/scpsys.txt
+++ b/Documentation/devicetree/bindings/soc/mediatek/scpsys.txt
@@ -21,6 +21,8 @@  Required properties:
 	- "mediatek,mt2712-scpsys"
 	- "mediatek,mt6797-scpsys"
 	- "mediatek,mt7622-scpsys"
+	- "mediatek,mt7623-scpsys", "mediatek,mt2701-scpsys": For MT7623 SoC
+	- "mediatek,mt7623a-scpsys": For MT7623A SoC
 	- "mediatek,mt8173-scpsys"
 - #power-domain-cells: Must be 1
 - reg: Address range of the SCPSYS unit
@@ -28,10 +30,11 @@  Required properties:
 - clock, clock-names: clocks according to the common clock binding.
                       These are clocks which hardware needs to be
                       enabled before enabling certain power domains.
-	Required clocks for MT2701: "mm", "mfg", "ethif"
+	Required clocks for MT2701 or MT7623: "mm", "mfg", "ethif"
 	Required clocks for MT2712: "mm", "mfg", "venc", "jpgdec", "audio", "vdec"
 	Required clocks for MT6797: "mm", "mfg", "vdec"
 	Required clocks for MT7622: "hif_sel"
+	Required clocks for MT7622A: "ethif"
 	Required clocks for MT8173: "mm", "mfg", "venc", "venc_lt"
 
 Optional properties: