On Sat, Oct 13, 2018 at 05:51:26PM +0530, Shreyansh Jain wrote:
> In case RTE_LIBRTE_DPAA2_USE_PHYS_IOVA is enabled, only supported
> class is RTE_IOVA_PA.
>
> Fixes: f7768afac101 ("bus/fslmc: support dynamic IOVA")
> Cc: hemant.agra...@nxp.com
>
> Signed-off-by: Shreyansh Jain <shreyansh.j...@nxp.com>
> ---
> drivers/bus/fslmc/fslmc_bus.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/bus/fslmc/fslmc_bus.c b/drivers/bus/fslmc/fslmc_bus.c
> index 960f55071..19e33caf1 100644
> --- a/drivers/bus/fslmc/fslmc_bus.c
> +++ b/drivers/bus/fslmc/fslmc_bus.c
> @@ -493,6 +493,10 @@ rte_dpaa2_get_iommu_class(void)
> bool is_vfio_noiommu_enabled = 1;
> bool has_iova_va;
>
> +#ifdef RTE_LIBRTE_DPAA2_USE_PHYS_IOVA
> + return RTE_IOVA_PA;
> +#endif
> +
I think you forgot to move it below the device list check as discussed in
previous patchset? :).
> if (TAILQ_EMPTY(&rte_fslmc_bus.device_list))
> return RTE_IOVA_DC;
>
> --
> 2.17.1
>
