Re: [PATCH] mm: memblock: fix minor typo and unclear comment

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

 



On Thu, May 07, 2020 at 06:22:32PM +0800, qiwuchen55@xxxxxxxxx wrote:
> From: chenqiwu <chenqiwu@xxxxxxxxxx>
> 
> Fix a minor typo "usabe->usable" for the current discription
> of member variable "memory" in struct memblock.
> 
> BTW, I think it's unclear the member variable "base" in
> struct memblock_type is currently described as the physical
> address of memory region, change it to base address of the
> region is clearer since the variable is decorated as phys_addr_t.
> 
> Signed-off-by: chenqiwu <chenqiwu@xxxxxxxxxx>

Reviewed-by: Mike Rapoport <rppt@xxxxxxxxxxxxx>

> ---
>  include/linux/memblock.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/linux/memblock.h b/include/linux/memblock.h
> index 6bc37a7..b594e0b 100644
> --- a/include/linux/memblock.h
> +++ b/include/linux/memblock.h
> @@ -41,7 +41,7 @@ enum memblock_flags {
>  
>  /**
>   * struct memblock_region - represents a memory region
> - * @base: physical address of the region
> + * @base: base address of the region
>   * @size: size of the region
>   * @flags: memory region attributes
>   * @nid: NUMA node id
> @@ -75,7 +75,7 @@ struct memblock_type {
>   * struct memblock - memblock allocator metadata
>   * @bottom_up: is bottom up direction?
>   * @current_limit: physical address of the current allocation limit
> - * @memory: usabe memory regions
> + * @memory: usable memory regions
>   * @reserved: reserved memory regions
>   * @physmem: all physical memory
>   */
> -- 
> 1.9.1
> 

-- 
Sincerely yours,
Mike.




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux