netdev - Re: [PATCH] arm64: dts: fsl: DPAA FMan DMA operations are coherent

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 5 Oct 2020 10:07:40 -0500
From:   Li Yang <leoyang.li@....com>
To:     madalin.bucur@....nxp.com
Cc:     Shawn Guo <shawnguo@...nel.org>,
        "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" 
        <devicetree@...r.kernel.org>, Rob Herring <robh+dt@...nel.org>,
        "moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE" 
        <linux-arm-kernel@...ts.infradead.org>,
        Netdev <netdev@...r.kernel.org>, camelia.groza@....nxp.com
Subject: Re: [PATCH] arm64: dts: fsl: DPAA FMan DMA operations are coherent

On Mon, Oct 5, 2020 at 7:47 AM Madalin Bucur <madalin.bucur@....nxp.com> wrote:
>
> Although the DPAA 1 FMan operations are coherent, the device tree
> node for the FMan does not indicate that, resulting in a needless
> loss of performance. Adding the missing dma-coherent property.
>
> Fixes: 1ffbecdd8321 ("arm64: dts: add DPAA FMan nodes")
>
> Signed-off-by: Madalin Bucur <madalin.bucur@....nxp.com>
> Tested-by: Camelia Groza <camelia.groza@....nxp.com>

Acked-by: Li Yang <leoyang.li@....com>

> ---
>  arch/arm64/boot/dts/freescale/qoriq-fman3-0.dtsi | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/qoriq-fman3-0.dtsi b/arch/arm64/boot/dts/freescale/qoriq-fman3-0.dtsi
> index 8bc6caa9167d..4338db14c5da 100644
> --- a/arch/arm64/boot/dts/freescale/qoriq-fman3-0.dtsi
> +++ b/arch/arm64/boot/dts/freescale/qoriq-fman3-0.dtsi
> @@ -19,6 +19,7 @@ fman0: fman@...0000 {
>         clock-names = "fmanclk";
>         fsl,qman-channel-range = <0x800 0x10>;
>         ptimer-handle = <&ptp_timer0>;
> +       dma-coherent;
>
>         muram@0 {
>                 compatible = "fsl,fman-muram";
> --
> 2.1.0
>