> +#ifdef CONFIG_SWIOTLB
> + if (swiotlb_nr_tbl()) {
> + return ttm_dma_populate(ttm, rdev->dev);
> + }
> +#endif
..
> +#ifdef CONFIG_SWIOTLB
> + if (swiotlb_nr_tbl()) {
> + ttm_dma_unpopulate(ttm, rdev->dev);
> + return;
> + }
> +#endif
> +#ifdef CONFIG_SWIOTLB
> + if (rdev->need_dma32 && swiotlb_nr_tbl()) {
> + sprintf(radeon_mem_types_names[i], "ttm_dma_page_pool");
> + radeon_mem_types_list[i].name = radeon_mem_types_names[i];
> + radeon_mem_types_list[i].show = &ttm_dma_page_alloc_debugfs;
> + radeon_mem_types_list[i].driver_features = 0;
> + radeon_mem_types_list[i++].data = NULL;
> + }
> +#endif
So I think the last one does not need the rdev->need_dma32 as you had
removed it from the other sites.
