Re: [PATCH 1/2] staging: fsl-dpaa2/eth: Fix address translations

From	Laurentiu Tudor <laurentiu.tudor@nxp.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 1/2] staging: fsl-dpaa2/eth: Fix address translations
Date	2017-05-24 14:40 +0200
Message-ID	<tKDK1-2mu-3@gated-at.bofh.it> (permalink)
References	<tKDK1-2mu-5@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi Ioana,

Debatable nit inline.

On 05/24/2017 03:13 PM, Ioana Radulescu wrote:
> Use the correct mechanisms for translating a DMA-mapped IOVA
> address into a virtual one. Without this fix, once SMMU is
> enabled on Layerscape platforms, the Ethernet driver throws
> IOMMU translation faults.
>
> Signed-off-by: Nipun Gupta <nipun.gupta@nxp.com>
> Signed-off-by: Ioana Radulescu <ruxandra.radulescu@nxp.com>
> ---
>   drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c | 25 +++++++++++++++++++------
>   drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.h |  1 +
>   2 files changed, 20 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c b/drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c
> index 6f9eed66c64d..3fee0d6f17e0 100644
> --- a/drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c
> +++ b/drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c
> @@ -37,6 +37,7 @@
>   #include <linux/interrupt.h>
>   #include <linux/msi.h>
>   #include <linux/kthread.h>
> +#include <linux/iommu.h>
>
>   #include "../../fsl-mc/include/mc.h"
>   #include "../../fsl-mc/include/mc-sys.h"
> @@ -54,6 +55,16 @@ MODULE_DESCRIPTION("Freescale DPAA2 Ethernet Driver");
>
>   const char dpaa2_eth_drv_version[] = "0.1";
>
> +static void *dpaa2_iova_to_virt(struct iommu_domain *domain,

if you pass a "struct dpaa2_eth_priv *priv" instead of "iommu_domain" 
you can move the priv->iommu_domain reference in the function and 
slightly simplify the call sites.

---
Best Regards, Laurentiu

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH 1/2] staging: fsl-dpaa2/eth: Fix address translations Laurentiu Tudor <laurentiu.tudor@nxp.com> - 2017-05-24 14:40 +0200
  RE: [PATCH 1/2] staging: fsl-dpaa2/eth: Fix address translations Ruxandra Ioana Radulescu <ruxandra.radulescu@nxp.com> - 2017-05-25 14:40 +0200
    Re: [PATCH 1/2] staging: fsl-dpaa2/eth: Fix address translations Laurentiu Tudor <laurentiu.tudor@nxp.com> - 2017-05-29 16:00 +0200

csiph-web