2023-06-01 20:47:45

by Nícolas F. R. A. Prado

[permalink] [raw]
Subject: [PATCH] arm64: dts: mediatek: mt8192: Add missing dma-ranges to soc node

In the series "Adjust the dma-ranges for MTK IOMMU", the mtk-iommu
driver was adapted to separate the iova range based on the larb used,
and a dma-ranges property was added to the soc node in the devicetree of
the affected SoCs allowing the whole 16GB iova range to be used. Except
that for mt8192, there was no patch adding dma-ranges.

Add the missing dma-ranges property to the soc node like was done for
mt8195 and mt8186. This fixes the usage of the vcodec, which would
otherwise trigger iommu faults.

Signed-off-by: Nícolas F. R. A. Prado <[email protected]>

---
Matthias,

this is a very simple but important fix for the vcodec functionality.
Since the vcodec node for mt8192 was just merged, can we please get this
in in this merge window as well?

Thanks,
Nícolas

arch/arm64/boot/dts/mediatek/mt8192.dtsi | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/mediatek/mt8192.dtsi b/arch/arm64/boot/dts/mediatek/mt8192.dtsi
index 7ff183c35494..a3612de8e9d1 100644
--- a/arch/arm64/boot/dts/mediatek/mt8192.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt8192.dtsi
@@ -401,6 +401,7 @@ soc {
#address-cells = <2>;
#size-cells = <2>;
compatible = "simple-bus";
+ dma-ranges = <0x0 0x0 0x0 0x0 0x4 0x0>;
ranges;

gic: interrupt-controller@c000000 {
--
2.40.1



Subject: Re: [PATCH] arm64: dts: mediatek: mt8192: Add missing dma-ranges to soc node

Il 01/06/23 22:32, Nícolas F. R. A. Prado ha scritto:
> In the series "Adjust the dma-ranges for MTK IOMMU", the mtk-iommu
> driver was adapted to separate the iova range based on the larb used,
> and a dma-ranges property was added to the soc node in the devicetree of
> the affected SoCs allowing the whole 16GB iova range to be used. Except
> that for mt8192, there was no patch adding dma-ranges.
>
> Add the missing dma-ranges property to the soc node like was done for
> mt8195 and mt8186. This fixes the usage of the vcodec, which would
> otherwise trigger iommu faults.
>
> Signed-off-by: Nícolas F. R. A. Prado <[email protected]>
>
> ---
> Matthias,
>
> this is a very simple but important fix for the vcodec functionality.
> Since the vcodec node for mt8192 was just merged, can we please get this
> in in this merge window as well?

+1

Reviewed-by: AngeloGioacchino Del Regno <[email protected]>

>
> Thanks,
> Nícolas
>
> arch/arm64/boot/dts/mediatek/mt8192.dtsi | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8192.dtsi b/arch/arm64/boot/dts/mediatek/mt8192.dtsi
> index 7ff183c35494..a3612de8e9d1 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8192.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8192.dtsi
> @@ -401,6 +401,7 @@ soc {
> #address-cells = <2>;
> #size-cells = <2>;
> compatible = "simple-bus";
> + dma-ranges = <0x0 0x0 0x0 0x0 0x4 0x0>;
> ranges;
>
> gic: interrupt-controller@c000000 {


2023-06-06 06:59:55

by Matthias Brugger

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: mediatek: mt8192: Add missing dma-ranges to soc node



On 05/06/2023 11:45, AngeloGioacchino Del Regno wrote:
> Il 01/06/23 22:32, Nícolas F. R. A. Prado ha scritto:
>> In the series "Adjust the dma-ranges for MTK IOMMU", the mtk-iommu
>> driver was adapted to separate the iova range based on the larb used,
>> and a dma-ranges property was added to the soc node in the devicetree of
>> the affected SoCs allowing the whole 16GB iova range to be used. Except
>> that for mt8192, there was no patch adding dma-ranges.
>>
>> Add the missing dma-ranges property to the soc node like was done for
>> mt8195 and mt8186. This fixes the usage of the vcodec, which would
>> otherwise trigger iommu faults.
>>
>> Signed-off-by: Nícolas F. R. A. Prado <[email protected]>
>>
>> ---
>> Matthias,
>>
>> this is a very simple but important fix for the vcodec functionality.
>> Since the vcodec node for mt8192 was just merged, can we please get this
>> in in this merge window as well?
>
> +1
>
> Reviewed-by: AngeloGioacchino Del Regno <[email protected]>
>

Applied, thanks!

>>
>> Thanks,
>> Nícolas
>>
>>   arch/arm64/boot/dts/mediatek/mt8192.dtsi | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8192.dtsi
>> b/arch/arm64/boot/dts/mediatek/mt8192.dtsi
>> index 7ff183c35494..a3612de8e9d1 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8192.dtsi
>> +++ b/arch/arm64/boot/dts/mediatek/mt8192.dtsi
>> @@ -401,6 +401,7 @@ soc {
>>           #address-cells = <2>;
>>           #size-cells = <2>;
>>           compatible = "simple-bus";
>> +        dma-ranges = <0x0 0x0 0x0 0x0 0x4 0x0>;
>>           ranges;
>>           gic: interrupt-controller@c000000 {
>