diff mbox series

riscv: Remove unnecessary declaration

Message ID 20210207081053.94133-1-wangkefeng.wang@huawei.com (mailing list archive)
State New, archived
Headers show
Series riscv: Remove unnecessary declaration | expand

Commit Message

Kefeng Wang Feb. 7, 2021, 8:10 a.m. UTC
max_low_pfn and min_low_pfn are declared in linux/memblock.h,
and it also is included in arch/riscv/mm/init.c, drop unnecessary
declaration.

Signed-off-by: Kefeng Wang <wangkefeng.wang@huawei.com>
---
 arch/riscv/include/asm/page.h | 3 ---
 1 file changed, 3 deletions(-)

Comments

Atish Patra Feb. 8, 2021, 9:17 p.m. UTC | #1
On Sun, Feb 7, 2021 at 12:07 AM Kefeng Wang <wangkefeng.wang@huawei.com> wrote:
>
> max_low_pfn and min_low_pfn are declared in linux/memblock.h,
> and it also is included in arch/riscv/mm/init.c, drop unnecessary
> declaration.
>
> Signed-off-by: Kefeng Wang <wangkefeng.wang@huawei.com>
> ---
>  arch/riscv/include/asm/page.h | 3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/arch/riscv/include/asm/page.h b/arch/riscv/include/asm/page.h
> index 64a675c5c30a..adc9d26f3d75 100644
> --- a/arch/riscv/include/asm/page.h
> +++ b/arch/riscv/include/asm/page.h
> @@ -97,9 +97,6 @@ extern unsigned long pfn_base;
>  #define ARCH_PFN_OFFSET                (PAGE_OFFSET >> PAGE_SHIFT)
>  #endif /* CONFIG_MMU */
>
> -extern unsigned long max_low_pfn;
> -extern unsigned long min_low_pfn;
> -
>  #define __pa_to_va_nodebug(x)  ((void *)((unsigned long) (x) + va_pa_offset))
>  #define __va_to_pa_nodebug(x)  ((unsigned long)(x) - va_pa_offset)
>
> --
> 2.26.2
>
>
> _______________________________________________
> linux-riscv mailing list
> linux-riscv@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-riscv



Reviewed-by: Atish Patra <atish.patra@wdc.com>
Palmer Dabbelt Feb. 19, 2021, 7:15 a.m. UTC | #2
On Sun, 07 Feb 2021 00:10:53 PST (-0800), wangkefeng.wang@huawei.com wrote:
> max_low_pfn and min_low_pfn are declared in linux/memblock.h,
> and it also is included in arch/riscv/mm/init.c, drop unnecessary
> declaration.
>
> Signed-off-by: Kefeng Wang <wangkefeng.wang@huawei.com>
> ---
>  arch/riscv/include/asm/page.h | 3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/arch/riscv/include/asm/page.h b/arch/riscv/include/asm/page.h
> index 64a675c5c30a..adc9d26f3d75 100644
> --- a/arch/riscv/include/asm/page.h
> +++ b/arch/riscv/include/asm/page.h
> @@ -97,9 +97,6 @@ extern unsigned long pfn_base;
>  #define ARCH_PFN_OFFSET		(PAGE_OFFSET >> PAGE_SHIFT)
>  #endif /* CONFIG_MMU */
>
> -extern unsigned long max_low_pfn;
> -extern unsigned long min_low_pfn;
> -
>  #define __pa_to_va_nodebug(x)	((void *)((unsigned long) (x) + va_pa_offset))
>  #define __va_to_pa_nodebug(x)	((unsigned long)(x) - va_pa_offset)

Thanks, this is on for-next.
diff mbox series

Patch

diff --git a/arch/riscv/include/asm/page.h b/arch/riscv/include/asm/page.h
index 64a675c5c30a..adc9d26f3d75 100644
--- a/arch/riscv/include/asm/page.h
+++ b/arch/riscv/include/asm/page.h
@@ -97,9 +97,6 @@  extern unsigned long pfn_base;
 #define ARCH_PFN_OFFSET		(PAGE_OFFSET >> PAGE_SHIFT)
 #endif /* CONFIG_MMU */
 
-extern unsigned long max_low_pfn;
-extern unsigned long min_low_pfn;
-
 #define __pa_to_va_nodebug(x)	((void *)((unsigned long) (x) + va_pa_offset))
 #define __va_to_pa_nodebug(x)	((unsigned long)(x) - va_pa_offset)