diff options
author | Hyungwoo Yang <hyungwoo.yang@intel.com> | 2019-05-08 16:26:22 -0700 |
---|---|---|
committer | Jett Rink <jettrink@chromium.org> | 2019-05-14 16:34:10 +0000 |
commit | 7671f9d7dbe7d5590fccd5a16b97d1c8d45ce3c7 (patch) | |
tree | 738f9b83604fb188e0411d848ce8c01145e2e0a0 | |
parent | 0775901e94904463328ded042bb53ee49d2c8c46 (diff) | |
download | chrome-ec-7671f9d7dbe7d5590fccd5a16b97d1c8d45ce3c7.tar.gz |
ish: fix bugs in fwst check functions
currently functions to check the status of ilup(IPC link up)
and hup(HECI up) are changing values of FW status which is bug.
BRANCH=none
BUG=none
TEST=tested on Arcada platform
Change-Id: I1839304af619a3ee5a0856fe83560aa21a99e60a
Signed-off-by: Hyungwoo Yang <hyungwoo.yang@intel.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1601779
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Reviewed-by: Jett Rink <jettrink@chromium.org>
-rw-r--r-- | chip/ish/ish_fwst.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/chip/ish/ish_fwst.h b/chip/ish/ish_fwst.h index 506740e51b..c114db3241 100644 --- a/chip/ish/ish_fwst.h +++ b/chip/ish/ish_fwst.h @@ -117,7 +117,7 @@ static inline void ish_fwst_clear_ilup(void) /* return IPC link up state */ static inline int ish_fwst_is_ilup_set(void) { - return !!(IPC_ISH_FWSTS &= IPC_ISH_FWSTS_ILUP_MASK); + return !!(IPC_ISH_FWSTS & IPC_ISH_FWSTS_ILUP_MASK); } /* set HECI up */ @@ -135,7 +135,7 @@ static inline void ish_fwst_clear_hup(void) /* get HECI up status */ static inline int ish_fwst_is_hup_set(void) { - return !!(IPC_ISH_FWSTS &= IPC_ISH_FWSTS_HUP_MASK); + return !!(IPC_ISH_FWSTS & IPC_ISH_FWSTS_HUP_MASK); } /* set fw failure reason */ |