Re: [PATCH 26/28] ARM: mmu: Simplify the use of dma_flush_range()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Am Dienstag, den 08.05.2018, 14:29 -0700 schrieb Andrey Smirnov:
> Simplify the use of dma_flush_range() by changing its signature to
> accept pointer to start of the data and data size. This change allows
> us to avoid a whole bunch of repetitive arithmetic currently done by
> all of the callers.
> 
> Signed-off-by: Andrey Smirnov <andrew.smirnov@xxxxxxxxx>

One nit below, otherwise
Reviewed-by: Lucas Stach <l.stach@xxxxxxxxxxxxxx>

> ---
>  arch/arm/cpu/mmu.c | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/arch/arm/cpu/mmu.c b/arch/arm/cpu/mmu.c
> index b14a869af..fc7102700 100644
> --- a/arch/arm/cpu/mmu.c
> +++ b/arch/arm/cpu/mmu.c
> @@ -142,8 +142,11 @@ static u32 *find_pte(unsigned long adr)
> >  	return &table[(adr >> PAGE_SHIFT) & 0xff];
>  }
>  
> -static void dma_flush_range(unsigned long start, unsigned long end)
> +static void dma_flush_range(void *ptr, size_t size)
>  {
> > +	unsigned long start = (unsigned long)ptr;
> +	unsigned long end   = start + size;

Personal taste, but I don't like those extra spaces. They tend to not
line up as well after a few future code changes.

> +
> >  	__dma_flush_range(start, end);
> >  	if (outer_cache.flush_range)
> >  		outer_cache.flush_range(start, end);
> @@ -170,9 +173,7 @@ static int __remap_range(void *_start, size_t size, u32 pte_flags)
> >  		p[i] |= pte_flags | PTE_TYPE_SMALL;
> >  	}
>  
> > -	dma_flush_range((unsigned long)p,
> > -			(unsigned long)p + numentries * sizeof(u32));
> -
> > +	dma_flush_range(p, numentries * sizeof(u32));
> >  	tlb_invalidate();
>  
> >  	return 0;
> @@ -203,7 +204,7 @@ void *map_io_sections(unsigned long phys, void *_start, size_t size)
> >  	for (sec = start; sec < start + size; sec += PGDIR_SIZE, phys += PGDIR_SIZE)
> >  		ttb[pgd_index(sec)] = phys | PMD_SECT_DEF_UNCACHED;
>  
> > -	dma_flush_range((unsigned long)ttb, (unsigned long)ttb + 0x4000);
> > +	dma_flush_range(ttb, 0x4000);
> >  	tlb_invalidate();
> >  	return _start;
>  }
> @@ -249,9 +250,8 @@ static int arm_mmu_remap_sdram(struct memory_bank *bank)
> >  		pte += PTRS_PER_PTE;
> >  	}
>  
> > -	dma_flush_range((unsigned long)ttb, (unsigned long)ttb + 0x4000);
> > -	dma_flush_range((unsigned long)ptes,
> > -			(unsigned long)ptes + num_ptes * sizeof(u32));
> > +	dma_flush_range(ttb, 0x4000);
> > +	dma_flush_range(ptes, num_ptes * sizeof(u32));
>  
> >  	tlb_invalidate();
>  

_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox




[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux