Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <20220912132501.1812575-1-georgepee@gmail.com>
In-Reply-To: <20220912132501.1812575-1-georgepee@gmail.com>
From:   George Pee <georgepee@gmail.com>
Date:   Thu, 3 Nov 2022 08:40:07 -0500
Message-ID: <CAKj0CMvyYvp=LjGmG3dxzN4mjE76zBzjiLkCe-tK0bvsM0JDzA@mail.gmail.com>
Subject: Re: [PATCH v2] ARM : Support for optional ARMv8.2 half-precision
 floating point extension
Cc:     Russell King <linux@armlinux.org.uk>,
        Ard Biesheuvel <ardb@kernel.org>,
        "Russell King (Oracle)" <rmk+kernel@armlinux.org.uk>,
        Linus Walleij <linus.walleij@linaro.org>,
        Nicolas Pitre <nico@fluxnic.net>,
        Keith Packard <keithpac@amazon.com>,
        Austin Kim <austindh.kim@gmail.com>,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
To:     unlisted-recipients:; (no To-header on input)
Precedence: bulk

On Mon, Sep 12, 2022 at 8:25 AM george pee <georgepee@gmail.com> wrote:
>
> Report feature /proc/cpuinfo as fphp to be consistent with arm64
>
> Signed-off-by: george pee <georgepee@gmail.com>
> ---
>  arch/arm/include/uapi/asm/hwcap.h | 1 +
>  arch/arm/kernel/entry-armv.S      | 3 ++-
>  arch/arm/kernel/setup.c           | 1 +
>  arch/arm/vfp/vfpmodule.c          | 2 ++
>  4 files changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/include/uapi/asm/hwcap.h b/arch/arm/include/uapi/asm/hwcap.h
> index 990199d8b7c6..5d635dce8853 100644
> --- a/arch/arm/include/uapi/asm/hwcap.h
> +++ b/arch/arm/include/uapi/asm/hwcap.h
> @@ -37,5 +37,6 @@
>  #define HWCAP2_SHA1    (1 << 2)
>  #define HWCAP2_SHA2    (1 << 3)
>  #define HWCAP2_CRC32   (1 << 4)
> +#define HWCAP2_FPHP    (1 << 5)
>
>  #endif /* _UAPI__ASMARM_HWCAP_H */
> diff --git a/arch/arm/kernel/entry-armv.S b/arch/arm/kernel/entry-armv.S
> index c39303e5c234..161f8df852e1 100644
> --- a/arch/arm/kernel/entry-armv.S
> +++ b/arch/arm/kernel/entry-armv.S
> @@ -625,11 +625,12 @@ call_fpe:
>         ret.w   lr                              @ CP#6
>         ret.w   lr                              @ CP#7
>         ret.w   lr                              @ CP#8
> -       ret.w   lr                              @ CP#9
>  #ifdef CONFIG_VFP
> +       W(b)    do_vfp                          @ CP#9  (VFP/FP16)
>         W(b)    do_vfp                          @ CP#10 (VFP)
>         W(b)    do_vfp                          @ CP#11 (VFP)
>  #else
> +       ret.w   lr                              @ CP#9
>         ret.w   lr                              @ CP#10 (VFP)
>         ret.w   lr                              @ CP#11 (VFP)
>  #endif
> diff --git a/arch/arm/kernel/setup.c b/arch/arm/kernel/setup.c
> index 1e8a50a97edf..8887d0f447d6 100644
> --- a/arch/arm/kernel/setup.c
> +++ b/arch/arm/kernel/setup.c
> @@ -1258,6 +1258,7 @@ static const char *hwcap2_str[] = {
>         "sha1",
>         "sha2",
>         "crc32",
> +       "fphp",
>         NULL
>  };
>
> diff --git a/arch/arm/vfp/vfpmodule.c b/arch/arm/vfp/vfpmodule.c
> index 2cb355c1b5b7..0806b0b1f2c7 100644
> --- a/arch/arm/vfp/vfpmodule.c
> +++ b/arch/arm/vfp/vfpmodule.c
> @@ -831,6 +831,8 @@ static int __init vfp_init(void)
>
>                         if ((fmrx(MVFR1) & 0xf0000000) == 0x10000000)
>                                 elf_hwcap |= HWCAP_VFPv4;
> +                       if ((fmrx(MVFR1) & 0x0f000000) == 0x03000000)
> +                               elf_hwcap2 |= HWCAP2_FPHP;
>                 }
>         /* Extract the architecture version on pre-cpuid scheme */
>         } else {
> --
> 2.37.3
>

Any concerns with this?