Re: [RFC] etnaviv: missing dma_mask

From: Lucas Stach
Date: Thu Aug 23 2018 - 04:54:25 EST


Am Freitag, den 17.08.2018, 08:42 +0200 schrieb Christoph Hellwig:
> On Tue, Aug 14, 2018 at 05:12:25PM +0300, Eugeniy Paltsev wrote:
> > Hi Lucas, Christoph,
> >
> > After switching ARC to generic dma_noncoherent cache opsÂ
> > etnaviv driver start failing on dma maping functions because of
> > dma_mask lack.
> >
> > So I'm wondering is it valid case to have device which is
> > DMA capable and doesn't have dma_mask set?
> >
> > If not, then I guess something like that should work
> > (at least it works for ARC):
>
> This looks ok is a minimal fix:
>
> Reviewed-by: Christoph Hellwig <hch@xxxxxx>
>
> But why doesn't this device have a dma-range property in DT?

Because the etnaviv device is a virtual device not represented in DT,
as it is only used to expose the DRM device, which may cover multiple
GPU core devices. The GPU core devices are properly configured from DT,
but unfortunately many of the dma related operations happen through the
DRM device. We could fix this by replacing many of the DRM helpers with
etnaviv specific functions handling dma per GPU core, but it isn't a
clear win right now, as generally on SoCs with multiple GPU cores, the
devices are on the same bus and have the same dma requirements.

Regards,
Lucas