Re: [PATCH 0/3] MediaTek MMSYS: Split out MT8173 and add MT6795

[Matthias Brugger]

On 09/03/2023 11:26, AngeloGioacchino Del Regno wrote:
> This series migrates MT8173 to use its own mmsys DDP routing table
> and introduces support for the MediaTek Helio X10 MT6795 using the
> same routing table as MT8173.
>
> Broader explanation goes to that MT6795 addition:
>
> Both MT6795 and MT8173 can support more usecases than the ones in
> the current routing tables and the ones that are actually supported
> in mediatek-drm!
> This is mostly about MERGE, dual-DSI, WDMA (for command mode panels)
> and others, but, again, right now, these are not supported.
>
> I could have created a mt6795-mmsys.h, but that would've been a 1:1
> clone of mt8173-mmsys.h, creating unnecessary code duplication hence
> raising code (and kernel) size for no practical reason.
>
> As a side note, if more MTK Smartphone SoCs land upstream, I expect
> to see more re-using of MTK Chromebook SoCs mmsys routing tables.
>
> P.S.: MT6795 dt-bindings are already upstream in v6.1 [1].
>
> This series was tested on:
>   - MT8173 Acer Chromebook R13 (Elm)
>   - MT6795 Sony Xperia M5 (Holly)

Applied, thanks!

> Also, this series depends on [2]
>
> [1]: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/tree/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.yaml?h=v6.1.15#n28
> [2]: https://patchwork.kernel.org/project/linux-mediatek/list/?series=724014
>
> AngeloGioacchino Del Regno (3):
>    soc: mediatek: mtk-mmsys: Split out MT8173 mmsys DDP routing table
>    soc: mediatek: mtk-mmsys: Change MT8173 num_resets to 64
>    soc: mediatek: mtk-mmsys: Add support for MT6795 Helio X10
>
>   drivers/soc/mediatek/mt8173-mmsys.h | 95 +++++++++++++++++++++++++++++
>   drivers/soc/mediatek/mtk-mmsys.c    | 16 ++++-
>   drivers/soc/mediatek/mtk-mmsys.h    |  2 +-
>   3 files changed, 109 insertions(+), 4 deletions(-)
>   create mode 100644 drivers/soc/mediatek/mt8173-mmsys.h