Re: [PATCH 2/2] dma-mapping: clearly mark DMA ops as an architecture feature

2024-08-28 Thread Christoph Hellwig
On Wed, Aug 28, 2024 at 08:21:14AM +0200, Andreas Larsson wrote: > On 2024-08-28 08:10, Christoph Hellwig wrote: > > --- a/drivers/xen/Kconfig > > +++ b/drivers/xen/Kconfig > > @@ -177,8 +177,8 @@ config XEN_GRANT_DMA_ALLOC > > > > config SWIOTLB_XEN > > def_bool y > > + depends on ARCH_DM

Re: [PATCH 2/2] dma-mapping: clearly mark DMA ops as an architecture feature

2024-08-28 Thread Andreas Larsson
On 2024-08-28 08:10, Christoph Hellwig wrote: > --- a/drivers/xen/Kconfig > +++ b/drivers/xen/Kconfig > @@ -177,8 +177,8 @@ config XEN_GRANT_DMA_ALLOC > > config SWIOTLB_XEN > def_bool y > + depends on ARCH_DMA_OPS Rename to ARCH_HAS_DMA_OPS in v2 is missing here > depends on X

[PATCH 2/2] dma-mapping: clearly mark DMA ops as an architecture feature

2024-08-27 Thread Christoph Hellwig
DMA ops are a helper for architectures and not for drivers to override the DMA implementation. Unfortunately driver authors keep ignoring this. Make the fact more clear by renaming the symbol to ARCH_HAS_DMA_OPS and having the two drivers overriding their dma_ops depend on that. These drivers sh