[edk2-devel] [PATCH edk2-platforms 1/1] Platform/DeveloperBox: drop dma-ranges property from DT root node

[Leif Lindholm]

On Fri, Feb 21, 2020 at 17:29:04 +0100, Ard Biesheuvel wrote:
> The dma-ranges DT property describes the DMA translation between a
> parent bus and its children, and so having a dma-ranges property in
> the root node makes little sense, but it doesn't harm either.
> 
> However, recent kernels (v5.5+) have started spewing warnings into
> the kernel log of the following nature:
> 
>   OF: translation of DMA address(0) to CPU address failed node(/uart at 2a400000)
>   OF: translation of DMA address(0) to CPU address failed node(/pmu)
>   OF: translation of DMA address(0) to CPU address failed node(/pcie at 60000000)
>   OF: translation of DMA address(0) to CPU address failed node(/pcie at 60000000)
>   OF: translation of DMA address(0) to CPU address failed node(/pcie at 60000000)
>   OF: translation of DMA address(0) to CPU address failed node(/pcie at 60000000)
>   OF: translation of DMA address(0) to CPU address failed node(/pcie at 60000000)
>   OF: translation of DMA address(0) to CPU address failed node(/pcie at 60000000)
>   OF: translation of DMA address(0) to CPU address failed node(/pcie at 70000000)
>   OF: translation of DMA address(0) to CPU address failed node(/uart at 51040000)
>   OF: translation of DMA address(0) to CPU address failed node(/gpio at 51000000)
>   OF: translation of DMA address(0) to CPU address failed node(/i2c at 51210000)
>   OF: translation of DMA address(0) to CPU address failed node(/gpio-keys)
>   OF: translation of DMA address(0) to CPU address failed node(/tpm_tis at 10000000)
>   OF: translation of DMA address(0) to CPU address failed node(/sdhci at 52300000)
>   OF: translation of DMA address(0) to CPU address failed node(/ethernet at 522d0000)
>   OF: translation of DMA address(0) to CPU address failed node(/pcie at 60000000)
>   OF: translation of DMA address(0) to CPU address failed node(/pcie at 60000000)
> 
> and so we should probably get rid of this bogus dma-ranges property.
> 
> Signed-off-by: Ard Biesheuvel <ard.biesheuvel at linaro.org>

Reviewed-by: Leif Lindholm <leif at nuviainc.com>

> ---
>  Silicon/Socionext/SynQuacer/DeviceTree/SynQuacer.dtsi | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/Silicon/Socionext/SynQuacer/DeviceTree/SynQuacer.dtsi b/Silicon/Socionext/SynQuacer/DeviceTree/SynQuacer.dtsi
> index afb9e2c779df..2ee3821fca0b 100644
> --- a/Silicon/Socionext/SynQuacer/DeviceTree/SynQuacer.dtsi
> +++ b/Silicon/Socionext/SynQuacer/DeviceTree/SynQuacer.dtsi
> @@ -21,7 +21,6 @@
>      #address-cells = <2>;
>      #size-cells = <2>;
>      interrupt-parent = <&gic>;
> -    dma-ranges = <0x0 0x0 0x0 0x0 0x100 0x0>;
>  
>      aliases {
>          serial0 = &soc_uart0;
> -- 
> 2.17.1
> 

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#55599): https://edk2.groups.io/g/devel/message/55599
Mute This Topic: https://groups.io/mt/71454056/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-