Re: [PATCH] sh: remove no-op macro VMLINUX_SYMBOL()

[Masahiro Yamada]

2018-05-09 16:53 GMT+09:00 Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx>:
> VMLINUX_SYMBOL() is no-op unless CONFIG_HAVE_UNDERSCORE_SYMBOL_PREFIX
> is defined.  It has ever been selected only by BLACKFIN and METAG.
> VMLINUX_SYMBOL() is unneeded for SuperH-specific code.
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx>
> ---



For information for SH maintainers just in case.

I already picked up this patch to my Kbuild tree.

Thanks.



>  arch/sh/include/asm/vmlinux.lds.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/sh/include/asm/vmlinux.lds.h b/arch/sh/include/asm/vmlinux.lds.h
> index f312813..9929556 100644
> --- a/arch/sh/include/asm/vmlinux.lds.h
> +++ b/arch/sh/include/asm/vmlinux.lds.h
> @@ -7,9 +7,9 @@
>  #ifdef CONFIG_DWARF_UNWINDER
>  #define DWARF_EH_FRAME                                                 \
>         .eh_frame : AT(ADDR(.eh_frame) - LOAD_OFFSET) {                 \
> -                 VMLINUX_SYMBOL(__start_eh_frame) = .;                 \
> +                 __start_eh_frame = .;                                 \
>                   *(.eh_frame)                                          \
> -                 VMLINUX_SYMBOL(__stop_eh_frame) = .;                  \
> +                 __stop_eh_frame = .;                                  \
>         }
>  #else
>  #define DWARF_EH_FRAME
> --
> 2.7.4
>



-- 
Best Regards
Masahiro Yamada