diff mbox series

[1/2] dt-bindings: mediatek: mmsys: add mt1867 binding

Message ID 20201027160631.608503-1-fparent@baylibre.com (mailing list archive)
State New, archived
Headers show
Series [1/2] dt-bindings: mediatek: mmsys: add mt1867 binding | expand

Commit Message

Fabien Parent Oct. 27, 2020, 4:06 p.m. UTC
Add binding documentation for MT8167 SoC.

Signed-off-by: Fabien Parent <fparent@baylibre.com>
---
 .../devicetree/bindings/arm/mediatek/mediatek,mmsys.txt          | 1 +
 1 file changed, 1 insertion(+)

Comments

Chun-Kuang Hu Oct. 27, 2020, 11:38 p.m. UTC | #1
Hi, Fabien:

Fabien Parent <fparent@baylibre.com> 於 2020年10月28日 週三 上午12:07寫道:
>
> Add binding documentation for MT8167 SoC.
>

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


> Signed-off-by: Fabien Parent <fparent@baylibre.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 d8c9108c3b4a..78c50733985c 100644
> --- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
> +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
> @@ -13,6 +13,7 @@ Required Properties:
>         - "mediatek,mt6779-mmsys", "syscon"
>         - "mediatek,mt6797-mmsys", "syscon"
>         - "mediatek,mt7623-mmsys", "mediatek,mt2701-mmsys", "syscon"
> +       - "mediatek,mt8167-mmsys", "syscon"
>         - "mediatek,mt8173-mmsys", "syscon"
>         - "mediatek,mt8183-mmsys", "syscon"
>  - #clock-cells: Must be 1
> --
> 2.28.0
>
>
> _______________________________________________
> Linux-mediatek mailing list
> Linux-mediatek@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-mediatek
Rob Herring (Arm) Oct. 30, 2020, 7:17 p.m. UTC | #2
On Tue, 27 Oct 2020 17:06:29 +0100, Fabien Parent wrote:
> Add binding documentation for MT8167 SoC.
> 
> Signed-off-by: Fabien Parent <fparent@baylibre.com>
> ---
>  .../devicetree/bindings/arm/mediatek/mediatek,mmsys.txt          | 1 +
>  1 file changed, 1 insertion(+)
> 

Acked-by: Rob Herring <robh@kernel.org>
Chun-Kuang Hu Jan. 9, 2021, 11:17 p.m. UTC | #3
Hi, Matthias:

Rob Herring <robh@kernel.org> 於 2020年10月31日 週六 上午3:17寫道:
>
> On Tue, 27 Oct 2020 17:06:29 +0100, Fabien Parent wrote:
> > Add binding documentation for MT8167 SoC.

Even though the title need to change to 'mt8167', this patch looks
good to me. How do you think about this patch? One drm patch [1]
depend on this patch, if you like this patch, could you applied this
patch first?

[1] https://patchwork.kernel.org/project/linux-mediatek/patch/20201023133130.194140-6-fparent@baylibre.com/

Regards,
Chun-Kuang.

> >
> > Signed-off-by: Fabien Parent <fparent@baylibre.com>
> > ---
> >  .../devicetree/bindings/arm/mediatek/mediatek,mmsys.txt          | 1 +
> >  1 file changed, 1 insertion(+)
> >
>
> Acked-by: Rob Herring <robh@kernel.org>
>
> _______________________________________________
> Linux-mediatek mailing list
> Linux-mediatek@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-mediatek
Fabien Parent Feb. 23, 2021, 10:21 p.m. UTC | #4
Hi Matthias,

Pinging you in case you didn't see @Chun-Kuang Hu's question for you.

Thanks,
Best regards,
Fabien

On Sun, Jan 10, 2021 at 12:18 AM Chun-Kuang Hu <chunkuang.hu@kernel.org> wrote:
>
> Hi, Matthias:
>
> Rob Herring <robh@kernel.org> 於 2020年10月31日 週六 上午3:17寫道:
> >
> > On Tue, 27 Oct 2020 17:06:29 +0100, Fabien Parent wrote:
> > > Add binding documentation for MT8167 SoC.
>
> Even though the title need to change to 'mt8167', this patch looks
> good to me. How do you think about this patch? One drm patch [1]
> depend on this patch, if you like this patch, could you applied this
> patch first?
>
> [1] https://patchwork.kernel.org/project/linux-mediatek/patch/20201023133130.194140-6-fparent@baylibre.com/
>
> Regards,
> Chun-Kuang.
>
> > >
> > > Signed-off-by: Fabien Parent <fparent@baylibre.com>
> > > ---
> > >  .../devicetree/bindings/arm/mediatek/mediatek,mmsys.txt          | 1 +
> > >  1 file changed, 1 insertion(+)
> > >
> >
> > Acked-by: Rob Herring <robh@kernel.org>
> >
> > _______________________________________________
> > Linux-mediatek mailing list
> > Linux-mediatek@lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/linux-mediatek
Chun-Kuang Hu March 18, 2021, 12:41 a.m. UTC | #5
Hi, Matthias:

Ping again.

Regards,
Chun-Kuang.

Fabien Parent <fparent@baylibre.com> 於 2021年2月24日 週三 上午6:21寫道:
>
> Hi Matthias,
>
> Pinging you in case you didn't see @Chun-Kuang Hu's question for you.
>
> Thanks,
> Best regards,
> Fabien
>
> On Sun, Jan 10, 2021 at 12:18 AM Chun-Kuang Hu <chunkuang.hu@kernel.org> wrote:
> >
> > Hi, Matthias:
> >
> > Rob Herring <robh@kernel.org> 於 2020年10月31日 週六 上午3:17寫道:
> > >
> > > On Tue, 27 Oct 2020 17:06:29 +0100, Fabien Parent wrote:
> > > > Add binding documentation for MT8167 SoC.
> >
> > Even though the title need to change to 'mt8167', this patch looks
> > good to me. How do you think about this patch? One drm patch [1]
> > depend on this patch, if you like this patch, could you applied this
> > patch first?
> >
> > [1] https://patchwork.kernel.org/project/linux-mediatek/patch/20201023133130.194140-6-fparent@baylibre.com/
> >
> > Regards,
> > Chun-Kuang.
> >
> > > >
> > > > Signed-off-by: Fabien Parent <fparent@baylibre.com>
> > > > ---
> > > >  .../devicetree/bindings/arm/mediatek/mediatek,mmsys.txt          | 1 +
> > > >  1 file changed, 1 insertion(+)
> > > >
> > >
> > > Acked-by: Rob Herring <robh@kernel.org>
> > >
> > > _______________________________________________
> > > Linux-mediatek mailing list
> > > Linux-mediatek@lists.infradead.org
> > > http://lists.infradead.org/mailman/listinfo/linux-mediatek
Matthias Brugger April 6, 2021, 2:11 p.m. UTC | #6
Hi Chun-Kuang,

On 10/01/2021 00:17, Chun-Kuang Hu wrote:
> Hi, Matthias:
> 
> Rob Herring <robh@kernel.org> 於 2020年10月31日 週六 上午3:17寫道:
>>
>> On Tue, 27 Oct 2020 17:06:29 +0100, Fabien Parent wrote:
>>> Add binding documentation for MT8167 SoC.
> 
> Even though the title need to change to 'mt8167', this patch looks
> good to me. How do you think about this patch? One drm patch [1]
> depend on this patch, if you like this patch, could you applied this
> patch first?
> 
> [1] https://patchwork.kernel.org/project/linux-mediatek/patch/20201023133130.194140-6-fparent@baylibre.com/
> 

I just pushed the latest version [1] of this to v5.12-next/soc-2

I'll try to get this in for v5.13, if not possible we can sync again on how to
fix this.

Regards,
Matthias

[1]
https://patchwork.kernel.org/project/linux-mediatek/patch/20210405200354.2194930-1-fparent@baylibre.com/

> Regards,
> Chun-Kuang.
> 
>>>
>>> Signed-off-by: Fabien Parent <fparent@baylibre.com>
>>> ---
>>>  .../devicetree/bindings/arm/mediatek/mediatek,mmsys.txt          | 1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>
>> Acked-by: Rob Herring <robh@kernel.org>
>>
>> _______________________________________________
>> Linux-mediatek mailing list
>> Linux-mediatek@lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/linux-mediatek
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 d8c9108c3b4a..78c50733985c 100644
--- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
+++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt
@@ -13,6 +13,7 @@  Required Properties:
 	- "mediatek,mt6779-mmsys", "syscon"
 	- "mediatek,mt6797-mmsys", "syscon"
 	- "mediatek,mt7623-mmsys", "mediatek,mt2701-mmsys", "syscon"
+	- "mediatek,mt8167-mmsys", "syscon"
 	- "mediatek,mt8173-mmsys", "syscon"
 	- "mediatek,mt8183-mmsys", "syscon"
 - #clock-cells: Must be 1