From: "Yong Wu (吴勇)" <Yong.Wu@mediatek.com> To: "joro@8bytes.org" <joro@8bytes.org>, "will@kernel.org" <will@kernel.org>, "krzysztof.kozlowski@linaro.org" <krzysztof.kozlowski@linaro.org>, "matthias.bgg@gmail.com" <matthias.bgg@gmail.com>, "robh+dt@kernel.org" <robh+dt@kernel.org> Cc: "linux-mediatek@lists.infradead.org" <linux-mediatek@lists.infradead.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "Youlin Pei (裴友林)" <youlin.pei@mediatek.com>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, "Jianjiao Zeng (曾健姣)" <Jianjiao.Zeng@mediatek.com>, "robin.murphy@arm.com" <robin.murphy@arm.com>, "Chengci Xu (许承赐)" <Chengci.Xu@mediatek.com>, "YF Wang (王云飞)" <YF.Wang@mediatek.com>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "krzysztof.kozlowski+dt@linaro.org" <krzysztof.kozlowski+dt@linaro.org>, "iommu@lists.linux.dev" <iommu@lists.linux.dev>, "Mingyuan Ma (马鸣远)" <Mingyuan.Ma@mediatek.com>, "angelogioacchino.delregno@collabora.com" <angelogioacchino.delregno@collabora.com> Subject: Re: [PATCH v8 1/7] dt-bindings: mediatek: mt8188: Add binding for MM & INFRA IOMMU Date: Tue, 14 Mar 2023 02:41:54 +0000 [thread overview] Message-ID: <aba29e7fd9a461f2d94c83cf2c7496521433847d.camel@mediatek.com> (raw) In-Reply-To: <388e2274-d46a-68a4-3e85-5bd597085041@linaro.org> Hi Krzysztof, Thanks very much for your help review. On Wed, 2023-03-08 at 11:35 +0100, Krzysztof Kozlowski wrote: > On 07/03/2023 09:05, Yong Wu wrote: > > From: "Chengci.Xu" <chengci.xu@mediatek.com> > > > > Add descriptions for mt8188 IOMMU which also use ARM Short- > > Descriptor > > translation table format. > > > > In mt8188, there are two smi-common HW and IOMMU, one is for > > vdo(video > > output), the other is for vpp(video processing pipe). They connects > > with different smi-larbs, then some setting(larbid_remap) is > > different. > > Differentiate them with the compatible string. > > > > Something like this: > > > > IOMMU(VDO) IOMMU(VPP) > > | | > > SMI_COMMON_VDO SMI_COMMON_VPP > > > > --------------- ---------------- > > | | ... | | ... > > larb0 larb2 ... larb1 larb3 ... > > > > We also have an IOMMU that is for infra master like PCIe. > > And infra master don't have the larb and ports. > > > > Signed-off-by: Chengci.Xu <chengci.xu@mediatek.com> > > Reviewed-by: Yong Wu <yong.wu@mediatek.com> > > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > Indeed, I missed broken checkpatch/DCO chain. Just confirm what this means. Do you mean MAINTAINERS need to be updated since we add a prefix "mediatek," for the header file? > > NAK. Please fix the chain. > > Best regards, > Krzysztof >
WARNING: multiple messages have this Message-ID (diff)
From: "Yong Wu (吴勇)" <Yong.Wu@mediatek.com> To: "joro@8bytes.org" <joro@8bytes.org>, "will@kernel.org" <will@kernel.org>, "krzysztof.kozlowski@linaro.org" <krzysztof.kozlowski@linaro.org>, "matthias.bgg@gmail.com" <matthias.bgg@gmail.com>, "robh+dt@kernel.org" <robh+dt@kernel.org> Cc: "linux-mediatek@lists.infradead.org" <linux-mediatek@lists.infradead.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "Youlin Pei (裴友林)" <youlin.pei@mediatek.com>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, "Jianjiao Zeng (曾健姣)" <Jianjiao.Zeng@mediatek.com>, "robin.murphy@arm.com" <robin.murphy@arm.com>, "Chengci Xu (许承赐)" <Chengci.Xu@mediatek.com>, "YF Wang (王云飞)" <YF.Wang@mediatek.com>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "krzysztof.kozlowski+dt@linaro.org" <krzysztof.kozlowski+dt@linaro.org>, "iommu@lists.linux.dev" <iommu@lists.linux.dev>, "Mingyuan Ma (马鸣远)" <Mingyuan.Ma@mediatek.com>, "angelogioacchino.delregno@collabora.com" <angelogioacchino.delregno@collabora.com> Subject: Re: [PATCH v8 1/7] dt-bindings: mediatek: mt8188: Add binding for MM & INFRA IOMMU Date: Tue, 14 Mar 2023 02:41:54 +0000 [thread overview] Message-ID: <aba29e7fd9a461f2d94c83cf2c7496521433847d.camel@mediatek.com> (raw) In-Reply-To: <388e2274-d46a-68a4-3e85-5bd597085041@linaro.org> Hi Krzysztof, Thanks very much for your help review. On Wed, 2023-03-08 at 11:35 +0100, Krzysztof Kozlowski wrote: > On 07/03/2023 09:05, Yong Wu wrote: > > From: "Chengci.Xu" <chengci.xu@mediatek.com> > > > > Add descriptions for mt8188 IOMMU which also use ARM Short- > > Descriptor > > translation table format. > > > > In mt8188, there are two smi-common HW and IOMMU, one is for > > vdo(video > > output), the other is for vpp(video processing pipe). They connects > > with different smi-larbs, then some setting(larbid_remap) is > > different. > > Differentiate them with the compatible string. > > > > Something like this: > > > > IOMMU(VDO) IOMMU(VPP) > > | | > > SMI_COMMON_VDO SMI_COMMON_VPP > > > > --------------- ---------------- > > | | ... | | ... > > larb0 larb2 ... larb1 larb3 ... > > > > We also have an IOMMU that is for infra master like PCIe. > > And infra master don't have the larb and ports. > > > > Signed-off-by: Chengci.Xu <chengci.xu@mediatek.com> > > Reviewed-by: Yong Wu <yong.wu@mediatek.com> > > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > Indeed, I missed broken checkpatch/DCO chain. Just confirm what this means. Do you mean MAINTAINERS need to be updated since we add a prefix "mediatek," for the header file? > > NAK. Please fix the chain. > > Best regards, > Krzysztof > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2023-03-14 2:42 UTC|newest] Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-03-07 8:05 [PATCH v8 0/7] MT8188 IOMMU SUPPORT Yong Wu 2023-03-07 8:05 ` Yong Wu 2023-03-07 8:05 ` [PATCH v8 1/7] dt-bindings: mediatek: mt8188: Add binding for MM & INFRA IOMMU Yong Wu 2023-03-07 8:05 ` Yong Wu 2023-03-07 9:51 ` AngeloGioacchino Del Regno 2023-03-07 9:51 ` AngeloGioacchino Del Regno 2023-03-08 10:35 ` Krzysztof Kozlowski 2023-03-08 10:35 ` Krzysztof Kozlowski 2023-03-14 2:41 ` Yong Wu (吴勇) [this message] 2023-03-14 2:41 ` Yong Wu (吴勇) 2023-03-14 6:46 ` Krzysztof Kozlowski 2023-03-14 6:46 ` Krzysztof Kozlowski 2023-03-14 10:14 ` Yong Wu (吴勇) 2023-03-14 14:08 ` Krzysztof Kozlowski 2023-03-14 14:08 ` Krzysztof Kozlowski 2023-03-07 8:05 ` [PATCH v8 2/7] iommu/mediatek: Set dma_mask for PGTABLE_PA_35_EN Yong Wu 2023-03-07 8:05 ` Yong Wu 2023-03-07 9:53 ` AngeloGioacchino Del Regno 2023-03-07 9:53 ` AngeloGioacchino Del Regno 2023-03-07 8:05 ` [PATCH v8 3/7] iommu/mediatek: Fix two IOMMU share pagetable issue Yong Wu 2023-03-07 8:05 ` Yong Wu 2023-03-07 8:05 ` [PATCH v8 4/7] iommu/mediatek: Adjust mtk_iommu_config flow Yong Wu 2023-03-07 8:05 ` Yong Wu 2023-03-07 8:05 ` [PATCH v8 5/7] iommu/mediatek: Add enable IOMMU SMC command for INFRA master Yong Wu 2023-03-07 8:05 ` Yong Wu 2023-03-07 8:05 ` [PATCH v8 6/7] iommu/mediatek: Add MT8188 IOMMU Support Yong Wu 2023-03-07 8:05 ` Yong Wu 2023-03-07 8:05 ` [PATCH v8 7/7] iommu/mediatek: mt8188: Add iova_region_larb_msk Yong Wu 2023-03-07 8:05 ` Yong Wu 2023-03-07 9:56 ` AngeloGioacchino Del Regno 2023-03-07 9:56 ` AngeloGioacchino Del Regno
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=aba29e7fd9a461f2d94c83cf2c7496521433847d.camel@mediatek.com \ --to=yong.wu@mediatek.com \ --cc=Chengci.Xu@mediatek.com \ --cc=Jianjiao.Zeng@mediatek.com \ --cc=Mingyuan.Ma@mediatek.com \ --cc=YF.Wang@mediatek.com \ --cc=angelogioacchino.delregno@collabora.com \ --cc=devicetree@vger.kernel.org \ --cc=iommu@lists.linux.dev \ --cc=joro@8bytes.org \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=krzysztof.kozlowski@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mediatek@lists.infradead.org \ --cc=matthias.bgg@gmail.com \ --cc=robh+dt@kernel.org \ --cc=robin.murphy@arm.com \ --cc=will@kernel.org \ --cc=youlin.pei@mediatek.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.