From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
To: "Yong Wu (吴勇)" <Yong.Wu@mediatek.com>,
"joro@8bytes.org" <joro@8bytes.org>,
"will@kernel.org" <will@kernel.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 07:46:07 +0100 [thread overview]
Message-ID: <4ac7e2e0-9b19-c911-7297-3510e55d6cfd@linaro.org> (raw)
In-Reply-To: <aba29e7fd9a461f2d94c83cf2c7496521433847d.camel@mediatek.com>
On 14/03/2023 03:41, Yong Wu (吴勇) wrote:
> 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?
No it means, run checkpatch.
Best regards,
Krzysztof
WARNING: multiple messages have this Message-ID (diff)
From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
To: "Yong Wu (吴勇)" <Yong.Wu@mediatek.com>,
"joro@8bytes.org" <joro@8bytes.org>,
"will@kernel.org" <will@kernel.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 07:46:07 +0100 [thread overview]
Message-ID: <4ac7e2e0-9b19-c911-7297-3510e55d6cfd@linaro.org> (raw)
In-Reply-To: <aba29e7fd9a461f2d94c83cf2c7496521433847d.camel@mediatek.com>
On 14/03/2023 03:41, Yong Wu (吴勇) wrote:
> 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?
No it means, run checkpatch.
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 6:46 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 (吴勇)
2023-03-14 2:41 ` Yong Wu (吴勇)
2023-03-14 6:46 ` Krzysztof Kozlowski [this message]
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=4ac7e2e0-9b19-c911-7297-3510e55d6cfd@linaro.org \
--to=krzysztof.kozlowski@linaro.org \
--cc=Chengci.Xu@mediatek.com \
--cc=Jianjiao.Zeng@mediatek.com \
--cc=Mingyuan.Ma@mediatek.com \
--cc=YF.Wang@mediatek.com \
--cc=Yong.Wu@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=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: link
Be 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.