Re: [PATCH] drm/mediatek: Remove dependency on GEM DMA helper

From: Doug Anderson
Date: Fri Jan 27 2023 - 10:58:23 EST


Hi,

On Thu, Jan 19, 2023 at 2:40 PM Rob Clark <robdclark@xxxxxxxxx> wrote:
>
> From: Rob Clark <robdclark@xxxxxxxxxxxx>
>
> It appears that the dependency on the DMA helpers was only for
> drm_gem_dma_vm_ops.
>
> Signed-off-by: Rob Clark <robdclark@xxxxxxxxxxxx>
> ---
> drivers/gpu/drm/mediatek/Kconfig | 1 -
> drivers/gpu/drm/mediatek/mtk_drm_gem.c | 7 ++++++-
> 2 files changed, 6 insertions(+), 2 deletions(-)

Drive-by comment: can you post a follow-up patch that also removes the
"#include <drm/drm_gem_dma_helper.h>" from "mtk_drm_gem.c", assuming
that still compiles? If it doesn't compile, that might be a bad sign
that we missed something. ;-)

-Doug