Re: [PATCH] Remove v850 from linux/elf-em.h

[David Daney]

On 03/15/2016 02:10 PM, Rob Landley wrote:
> From: Rob Landley <rob@xxxxxxxxxxx>
>
> The v850 port was removed by commits f606ddf42fd4 and 07a887d399b8 in 2008.
> These #defines are not used in the current kernel.
>
> Signed-off-by: Rob Landley <rob@xxxxxxxxxxx>
> ---
>   include/uapi/linux/elf-em.h |    3 ---
>   1 file changed, 3 deletions(-)
>
> diff --git a/include/uapi/linux/elf-em.h b/include/uapi/linux/elf-em.h
> index b56dfcf..c3fdfe7 100644
> --- a/include/uapi/linux/elf-em.h
> +++ b/include/uapi/linux/elf-em.h
> @@ -30,7 +30,6 @@
>   #define EM_X86_64	62	/* AMD x86-64 */
>   #define EM_S390		22	/* IBM S/390 */
>   #define EM_CRIS		76	/* Axis Communications 32-bit embedded processor */
> -#define EM_V850		87	/* NEC v850 */

Can you do this to userspace visible files?

I thought only additions and obvious corrections were allowed.  Removing 
symbols could cause build breakage for something.

>   #define EM_M32R		88	/* Renesas M32R */
>   #define EM_MN10300	89	/* Panasonic/MEI MN10300, AM33 */
>   #define EM_OPENRISC     92     /* OpenRISC 32-bit embedded processor */
> @@ -50,8 +49,6 @@
>    */
>   #define EM_ALPHA	0x9026
>
> -/* Bogus old v850 magic number, used by old tools. */
> -#define EM_CYGNUS_V850	0x9080
>   /* Bogus old m32r magic number, used by old tools. */
>   #define EM_CYGNUS_M32R	0x9041
>   /* This is the old interim value for S/390 architecture */