| Message ID | 20201027160631.608503-1-fparent@baylibre.com |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
| Series | [1/2] dt-bindings: mediatek: mmsys: add mt1867 binding | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success |
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
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>
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
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
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
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 --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
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(+)