2021-07-05 03:42:48

by Chun-Jie Chen

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

This patch adds the mmsys document binding for MT8192 SoC.

Signed-off-by: Chun-Jie Chen <[email protected]>
---
.../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


2021-07-05 12:56:35

by Chun-Kuang Hu

[permalink] [raw]
Subject: Re: [v12 02/20] dt-bindings: ARM: Mediatek: Add mmsys document binding for MT8192

Chun-Jie Chen <[email protected]> 於 2021年7月5日 週一 上午11:44寫道:
>
> This patch adds the mmsys document binding for MT8192 SoC.

Reviewed-by: Chun-Kuang Hu <[email protected]>

>
> Signed-off-by: Chun-Jie Chen <[email protected]>
> ---
> .../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
> [email protected]
> http://lists.infradead.org/mailman/listinfo/linux-mediatek

2021-07-05 15:41:57

by Matthias Brugger

[permalink] [raw]
Subject: Re: [v12 02/20] dt-bindings: ARM: Mediatek: Add mmsys document binding for MT8192



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 <[email protected]>

Reviewed-by: Matthias Brugger <[email protected]>

> ---
> .../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
>

2021-07-05 15:46:38

by Matthias Brugger

[permalink] [raw]
Subject: Re: [v12 02/20] dt-bindings: ARM: Mediatek: Add mmsys document binding for MT8192

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 <[email protected]>
>
> Reviewed-by: Matthias Brugger <[email protected]>
>

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
>>

2021-07-06 02:09:45

by Chun-Jie Chen

[permalink] [raw]
Subject: Re: [v12 02/20] dt-bindings: ARM: Mediatek: Add mmsys document binding for MT8192

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 <[email protected]>
> >
> > Reviewed-by: Matthias Brugger <[email protected]>
> >
>
> 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/[email protected]/

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
> > >
>
>

2021-08-05 19:05:30

by Matthias Brugger

[permalink] [raw]
Subject: Re: [v12 02/20] dt-bindings: ARM: Mediatek: Add mmsys document binding for MT8192

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 <[email protected]>
>>
>> Reviewed-by: Matthias Brugger <[email protected]>
>>
>
> 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
>>>
>