diff mbox series

[v12,02/20] dt-bindings: ARM: Mediatek: Add mmsys document binding for MT8192

Message ID 20210705033824.1934-3-chun-jie.chen@mediatek.com
State Superseded, archived
Headers show
Series [v12,01/20] dt-bindings: ARM: Mediatek: Add new document bindings of MT8192 clock | expand

Checks

Context Check Description
robh/checkpatch success

Commit Message

Chun-Jie Chen July 5, 2021, 3:38 a.m. UTC
This patch adds the mmsys document binding for MT8192 SoC.

Signed-off-by: Chun-Jie Chen <chun-jie.chen@mediatek.com>
---
 .../devicetree/bindings/arm/mediatek/mediatek,mmsys.txt          | 1 +
 1 file changed, 1 insertion(+)

Comments

Chun-Kuang Hu July 5, 2021, 12:55 p.m. UTC | #1
Chun-Jie Chen <chun-jie.chen@mediatek.com> 於 2021年7月5日 週一 上午11:44寫道:
>
> This patch adds the mmsys document binding for MT8192 SoC.

Reviewed-by: Chun-Kuang Hu <chunkuang.hu@kernel.org>

>
> Signed-off-by: Chun-Jie Chen <chun-jie.chen@mediatek.com>
> ---
>  .../devicetree/bindings/arm/mediatek/mediatek,mmsys.txt          | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
> index 78c50733985c..9712a6831fab 100644
> --- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
> +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
> @@ -16,6 +16,7 @@ Required Properties:
>         - "mediatek,mt8167-mmsys", "syscon"
>         - "mediatek,mt8173-mmsys", "syscon"
>         - "mediatek,mt8183-mmsys", "syscon"
> +       - "mediatek,mt8192-mmsys", "syscon"
>  - #clock-cells: Must be 1
>
>  For the clock control, the mmsys controller uses the common clk binding from
> --
> 2.18.0
> _______________________________________________
> Linux-mediatek mailing list
> Linux-mediatek@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-mediatek
Matthias Brugger July 5, 2021, 3:40 p.m. UTC | #2
On 05/07/2021 05:38, Chun-Jie Chen wrote:
> This patch adds the mmsys document binding for MT8192 SoC.
> 
> Signed-off-by: Chun-Jie Chen <chun-jie.chen@mediatek.com>

Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com>

> ---
>  .../devicetree/bindings/arm/mediatek/mediatek,mmsys.txt          | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
> index 78c50733985c..9712a6831fab 100644
> --- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
> +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
> @@ -16,6 +16,7 @@ Required Properties:
>  	- "mediatek,mt8167-mmsys", "syscon"
>  	- "mediatek,mt8173-mmsys", "syscon"
>  	- "mediatek,mt8183-mmsys", "syscon"
> +	- "mediatek,mt8192-mmsys", "syscon"
>  - #clock-cells: Must be 1
>  
>  For the clock control, the mmsys controller uses the common clk binding from
>
Matthias Brugger July 5, 2021, 3:45 p.m. UTC | #3
On 05/07/2021 17:40, Matthias Brugger wrote:
> 
> 
> On 05/07/2021 05:38, Chun-Jie Chen wrote:
>> This patch adds the mmsys document binding for MT8192 SoC.
>>
>> Signed-off-by: Chun-Jie Chen <chun-jie.chen@mediatek.com>
> 
> Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com>
> 

Of course, that's not what we need here...

Now pushed to v5.14-tmp/dts64 until v5.14-rc1 is released.

Thanks!
Matthias

>> ---
>>  .../devicetree/bindings/arm/mediatek/mediatek,mmsys.txt          | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
>> index 78c50733985c..9712a6831fab 100644
>> --- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
>> +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
>> @@ -16,6 +16,7 @@ Required Properties:
>>  	- "mediatek,mt8167-mmsys", "syscon"
>>  	- "mediatek,mt8173-mmsys", "syscon"
>>  	- "mediatek,mt8183-mmsys", "syscon"
>> +	- "mediatek,mt8192-mmsys", "syscon"
>>  - #clock-cells: Must be 1
>>  
>>  For the clock control, the mmsys controller uses the common clk binding from
>>
Chun-Jie Chen July 6, 2021, 2:05 a.m. UTC | #4
On Mon, 2021-07-05 at 17:45 +0200, Matthias Brugger wrote:
> On 05/07/2021 17:40, Matthias Brugger wrote:
> > 
> > 
> > On 05/07/2021 05:38, Chun-Jie Chen wrote:
> > > This patch adds the mmsys document binding for MT8192 SoC.
> > > 
> > > Signed-off-by: Chun-Jie Chen <chun-jie.chen@mediatek.com>
> > 
> > Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com>
> > 
> 
> Of course, that's not what we need here...
> 
> Now pushed to v5.14-tmp/dts64 until v5.14-rc1 is released.
> 
> Thanks!
> Matthias
> 

I see some discussion in [1] and check the implement of afe
driver(soc/mediatek/mt8192/mt8192-afe-pcm.c)
The audsys shoud be documented in mediatek,audsys.txt
I will send v13 to apply it and update dts [2],
sorry for the inconvenience.

[1]
https://patchwork.kernel.org/project/linux-mediatek/patch/20210630180652.1.I88a52644e47e88b15f5db9841cb084dc53c5875c@changeid/
[2]
https://patchwork.kernel.org/project/linux-mediatek/cover/20210616011551.29654-1-chun-jie.chen@mediatek.com/

Best Regards,
Chun-Jie
> > > ---
> > >  .../devicetree/bindings/arm/mediatek/mediatek,mmsys.txt         
> > >  | 1 +
> > >  1 file changed, 1 insertion(+)
> > > 
> > > diff --git
> > > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.t
> > > xt
> > > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.t
> > > xt
> > > index 78c50733985c..9712a6831fab 100644
> > > ---
> > > a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.t
> > > xt
> > > +++
> > > b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.t
> > > xt
> > > @@ -16,6 +16,7 @@ Required Properties:
> > >  	- "mediatek,mt8167-mmsys", "syscon"
> > >  	- "mediatek,mt8173-mmsys", "syscon"
> > >  	- "mediatek,mt8183-mmsys", "syscon"
> > > +	- "mediatek,mt8192-mmsys", "syscon"
> > >  - #clock-cells: Must be 1
> > >  
> > >  For the clock control, the mmsys controller uses the common clk
> > > binding from
> > > 
> 
>
Matthias Brugger Aug. 5, 2021, 3:41 p.m. UTC | #5
On 05/07/2021 17:45, Matthias Brugger wrote:
> On 05/07/2021 17:40, Matthias Brugger wrote:
>>
>>
>> On 05/07/2021 05:38, Chun-Jie Chen wrote:
>>> This patch adds the mmsys document binding for MT8192 SoC.
>>>
>>> Signed-off-by: Chun-Jie Chen <chun-jie.chen@mediatek.com>
>>
>> Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com>
>>
> 
> Of course, that's not what we need here...
> 
> Now pushed to v5.14-tmp/dts64 until v5.14-rc1 is released.
> 

Given that the change to yaml was taken by Rob Herring I drop this patch from my
queue.

Regards,
Matthias

> Thanks!
> Matthias
> 
>>> ---
>>>  .../devicetree/bindings/arm/mediatek/mediatek,mmsys.txt          | 1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
>>> index 78c50733985c..9712a6831fab 100644
>>> --- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
>>> +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
>>> @@ -16,6 +16,7 @@ Required Properties:
>>>  	- "mediatek,mt8167-mmsys", "syscon"
>>>  	- "mediatek,mt8173-mmsys", "syscon"
>>>  	- "mediatek,mt8183-mmsys", "syscon"
>>> +	- "mediatek,mt8192-mmsys", "syscon"
>>>  - #clock-cells: Must be 1
>>>  
>>>  For the clock control, the mmsys controller uses the common clk binding from
>>>
>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
index 78c50733985c..9712a6831fab 100644
--- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
+++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
@@ -16,6 +16,7 @@  Required Properties:
 	- "mediatek,mt8167-mmsys", "syscon"
 	- "mediatek,mt8173-mmsys", "syscon"
 	- "mediatek,mt8183-mmsys", "syscon"
+	- "mediatek,mt8192-mmsys", "syscon"
 - #clock-cells: Must be 1
 
 For the clock control, the mmsys controller uses the common clk binding from