Re: [PATCH for-5.3] drm/omap: ensure we have a valid dma_mask

2019-08-11 Thread Peter Ujfalusi
On 09/08/2019 13.00, Tomi Valkeinen wrote: > Here's my version. > > From c258309e36fc86076db155aead03a3900b96c3d4 Mon Sep 17 00:00:00 2001 > From: Tomi Valkeinen > Date: Fri, 9 Aug 2019 09:54:49 +0300 > Subject: [PATCH] drm/omap: ensure we have a valid dma_mask > > The omapdrm driver uses

Re: [PATCH for-5.3] drm/omap: ensure we have a valid dma_mask

2019-08-09 Thread Christoph Hellwig
On Fri, Aug 09, 2019 at 01:00:38PM +0300, Tomi Valkeinen wrote: > Alright, thanks for the clarification! > > Here's my version. Looks god to me: Reviewed-by: Christoph Hellwig

Re: [PATCH for-5.3] drm/omap: ensure we have a valid dma_mask

2019-08-09 Thread Tomi Valkeinen
On 09/08/2019 11:07, Christoph Hellwig wrote: > On Fri, Aug 09, 2019 at 09:40:32AM +0300, Tomi Valkeinen wrote: >> We do call dma_set_coherent_mask() in omapdrm's probe() (in omap_drv.c), >> but apparently that's not enough anymore. Changing that call to >> dma_coerce_mask_and_coherent() removes

Re: [PATCH for-5.3] drm/omap: ensure we have a valid dma_mask

2019-08-09 Thread Christoph Hellwig
On Fri, Aug 09, 2019 at 09:40:32AM +0300, Tomi Valkeinen wrote: > We do call dma_set_coherent_mask() in omapdrm's probe() (in omap_drv.c), > but apparently that's not enough anymore. Changing that call to > dma_coerce_mask_and_coherent() removes the WARN. I can create a patch for > that, or

Re: [PATCH for-5.3] drm/omap: ensure we have a valid dma_mask

2019-08-09 Thread Tomi Valkeinen
Hi, On 08/08/2019 13:10, Christoph Hellwig wrote: The omapfb platform devices does not have a DMA mask set. The traditional arm DMA code ignores, but the generic dma-direct/swiotlb has stricter checks and thus fails mappings without a DMA mask. As we use swiotlb for arm with LPAE now, omap

[PATCH for-5.3] drm/omap: ensure we have a valid dma_mask

2019-08-08 Thread Christoph Hellwig
The omapfb platform devices does not have a DMA mask set. The traditional arm DMA code ignores, but the generic dma-direct/swiotlb has stricter checks and thus fails mappings without a DMA mask. As we use swiotlb for arm with LPAE now, omap needs to catch up and actually set a DMA mask. Fixes: