Re: [PATCH 3/3] iommu/mediatek: Add support for mt8189

["Yong Wu (吴勇)" <Yong.Wu@mediatek.com>]

On Thu, 2025-08-07 at 17:57 +0800, zhengnan chen wrote:
> From: "zhengnan chen" <zhengnan.chen@mediatek.com>
> 
> Add support for mt8189 INFRA & APU & MM IOMMUs.
> 
> Signed-off-by: zhengnan chen <zhengnan.chen@mediatek.com>

Splitting this into three patches(apu/mm/infra) may make the patch more
readable.

> ---
>  drivers/iommu/mtk_iommu.c | 76
> +++++++++++++++++++++++++++++++++++++++
>  1 file changed, 76 insertions(+)
> 

[snip]

> +
> +static const struct mtk_iommu_plat_data mt8189_data_mm = {
> +	.m4u_plat	= M4U_MT8189,
> +	.flags		= HAS_BCLK | HAS_SUB_COMM_3BITS |
> OUT_ORDER_WR_EN |
> +			  WR_THROT_EN | IOVA_34_EN | MTK_IOMMU_TYPE_MM
> |
> +			  PGTABLE_PA_35_EN | DL_WITH_MULTI_LARB,
> +	.hw_list	= &m4ulist,
> +	.inv_sel_reg	= REG_MMU_INV_SEL_GEN2,
> +	.banks_num	= 5,
> +	.banks_enable	= {true, false, false, false, true},
> +	.iova_region	= mt8192_multi_dom,
> +	.iova_region_nr	= ARRAY_SIZE(mt8192_multi_dom),
> +	.iova_region_larb_msk = mt8189_larb_region_msk,
> +	.larbid_remap	= {{0}, {1}, {21/*GCE_D*/, 21/*GCE_M*/, 2},

Take care the format, Add space after '/*'

> +			   {19, 20, 9, 11}, {7}, {4},
> +			   {13, 17}, {14, 16}},
> +};
> +
>  static const struct mtk_iommu_plat_data mt8192_data = {
>  	.m4u_plat       = M4U_MT8192,
>  	.flags          = HAS_BCLK | HAS_SUB_COMM_2BITS |
> OUT_ORDER_WR_EN |
> @@ -1826,6 +1899,9 @@ static const struct of_device_id
> mtk_iommu_of_ids[] = {
>  	{ .compatible = "mediatek,mt8188-iommu-infra", .data =
> &mt8188_data_infra},
>  	{ .compatible = "mediatek,mt8188-iommu-vdo",   .data =
> &mt8188_data_vdo},
>  	{ .compatible = "mediatek,mt8188-iommu-vpp",   .data =
> &mt8188_data_vpp},
> +	{ .compatible = "mediatek,mt8189-iommu-apu",   .data =
> &mt8189_data_apu},
> +	{ .compatible = "mediatek,mt8189-iommu-infra", .data =
> &mt8189_data_infra},
> +	{ .compatible = "mediatek,mt8189-iommu-mm",    .data =
> &mt8189_data_mm},
>  	{ .compatible = "mediatek,mt8192-m4u", .data = &mt8192_data},
>  	{ .compatible = "mediatek,mt8195-iommu-infra", .data =
> &mt8195_data_infra},
>  	{ .compatible = "mediatek,mt8195-iommu-vdo",   .data =
> &mt8195_data_vdo},