[src/trunk]: src/sys/arch/arm/vfp We KASSERT((fregs->vfp_fpexc & VFP_FPEXC_EN...

[bouyer <bouyer%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/8629ea75040d
branches:  trunk
changeset: 827143:8629ea75040d
user:      bouyer <bouyer%NetBSD.org@localhost>
date:      Mon Oct 16 15:13:00 2017 +0000

description:
We KASSERT((fregs->vfp_fpexc & VFP_FPEXC_EN) == 0) just before, so
enabled is always false. remove.

diffstat:

 sys/arch/arm/vfp/vfp_init.c |  11 +----------
 1 files changed, 1 insertions(+), 10 deletions(-)

diffs (27 lines):

diff -r fc0f633f6e11 -r 8629ea75040d sys/arch/arm/vfp/vfp_init.c
--- a/sys/arch/arm/vfp/vfp_init.c       Mon Oct 16 15:08:24 2017 +0000
+++ b/sys/arch/arm/vfp/vfp_init.c       Mon Oct 16 15:13:00 2017 +0000
@@ -1,4 +1,4 @@
-/*      $NetBSD: vfp_init.c,v 1.54 2017/10/16 15:08:24 bouyer Exp $ */
+/*      $NetBSD: vfp_init.c,v 1.55 2017/10/16 15:13:00 bouyer Exp $ */
 
 /*
  * Copyright (c) 2008 ARM Ltd
@@ -535,17 +535,8 @@
        /*
         * Load and Enable the VFP (so that we can write the registers).
         */
-       bool enabled = fregs->vfp_fpexc & VFP_FPEXC_EN;
        fregs->vfp_fpexc |= VFP_FPEXC_EN;
        armreg_fpexc_write(fregs->vfp_fpexc);
-       if (enabled) {
-               /*
-                * If we think the VFP is enabled, it must have be
-                * disabled by vfp_state_release for another LWP so
-                * we can now just return.
-                */
-               return;
-       }
        KASSERT(curcpu()->ci_pcu_curlwp[PCU_FPU] == NULL);
        KASSERT(l->l_pcu_cpu[PCU_FPU] == NULL);