On 26.09.24 15:15, Sascha Hauer wrote: > We currently only have dma_alloc_writecombine() for aarch32. Implement > it for aarch64 as it is useful for mapping framebuffer memory. > > Signed-off-by: Sascha Hauer <s.ha...@pengutronix.de>
Reviewed-by: Ahmad Fatoum <a.fat...@pengutronix.de> Thanks, Ahmad > --- > arch/arm/cpu/mmu_64.c | 11 ++++++++++- > arch/arm/cpu/mmu_64.h | 15 ++++++--------- > 2 files changed, 16 insertions(+), 10 deletions(-) > > diff --git a/arch/arm/cpu/mmu_64.c b/arch/arm/cpu/mmu_64.c > index 7c7834201b..7854f71f4c 100644 > --- a/arch/arm/cpu/mmu_64.c > +++ b/arch/arm/cpu/mmu_64.c > @@ -24,6 +24,8 @@ > > #include "mmu_64.h" > > +#define ARCH_MAP_WRITECOMBINE ((unsigned)-1) > + > static uint64_t *get_ttb(void) > { > return (uint64_t *)get_ttbr(current_el()); > @@ -172,9 +174,11 @@ static unsigned long get_pte_attrs(unsigned flags) > case MAP_CACHED: > return CACHED_MEM; > case MAP_UNCACHED: > - return attrs_uncached_mem(); > + return attrs_xn() | UNCACHED_MEM; > case MAP_FAULT: > return 0x0; > + case ARCH_MAP_WRITECOMBINE: > + return attrs_xn() | MEM_ALLOC_WRITECOMBINE; > default: > return ~0UL; > } > @@ -295,6 +299,11 @@ void dma_flush_range(void *ptr, size_t size) > v8_flush_dcache_range(start, end); > } > > +void *dma_alloc_writecombine(size_t size, dma_addr_t *dma_handle) > +{ > + return dma_alloc_map(size, dma_handle, ARCH_MAP_WRITECOMBINE); > +} > + > static void init_range(size_t total_level0_tables) > { > uint64_t *ttb = get_ttb(); > diff --git a/arch/arm/cpu/mmu_64.h b/arch/arm/cpu/mmu_64.h > index e3959e4407..d3c39dabb5 100644 > --- a/arch/arm/cpu/mmu_64.h > +++ b/arch/arm/cpu/mmu_64.h > @@ -8,22 +8,19 @@ > #define UNCACHED_MEM (PTE_BLOCK_MEMTYPE(MT_DEVICE_nGnRnE) | \ > PTE_BLOCK_OUTER_SHARE | \ > PTE_BLOCK_AF) > +#define MEM_ALLOC_WRITECOMBINE (PTE_BLOCK_MEMTYPE(MT_NORMAL_NC) | \ > + PTE_BLOCK_OUTER_SHARE | \ > + PTE_BLOCK_AF) > > -static inline unsigned long attrs_uncached_mem(void) > +static inline unsigned long attrs_xn(void) > { > - unsigned long attrs = UNCACHED_MEM; > - > switch (current_el()) { > case 3: > case 2: > - attrs |= PTE_BLOCK_UXN; > - break; > + return PTE_BLOCK_UXN; > default: > - attrs |= PTE_BLOCK_UXN | PTE_BLOCK_PXN; > - break; > + return PTE_BLOCK_UXN | PTE_BLOCK_PXN; > } > - > - return attrs; > } > > /* > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |