On Sun, Aug 02, 2020 at 07:35:56PM +0300, Mike Rapoport wrote:
> From: Mike Rapoport <r...@linux.ibm.com>
> 
> There are several occurrences of the following pattern:
> 
>       for_each_memblock(memory, reg) {
>               start = __pfn_to_phys(memblock_region_memory_base_pfn(reg);
>               end = __pfn_to_phys(memblock_region_memory_end_pfn(reg));
> 
>               /* do something with start and end */
>       }
> 
> Using for_each_mem_range() iterator is more appropriate in such cases and
> allows simpler and cleaner code.
> 
> Signed-off-by: Mike Rapoport <r...@linux.ibm.com>
> ---
>  arch/mips/cavium-octeon/dma-octeon.c     | 12 +++---
>  arch/mips/kernel/setup.c                 | 31 +++++++--------

Acked-by: Thomas Bogendoerfer <tsbog...@alpha.franken.de>

Thomas.

-- 
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea.                                                [ RFC1925, 2.3 ]

Reply via email to