diff options
author | Kevin K Wong <kevin.k.wong@intel.com> | 2015-03-03 08:58:56 -0800 |
---|---|---|
committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | 2015-03-06 19:28:46 +0000 |
commit | 36f26030987367e60a8d97dbce9f47f7fda14729 (patch) | |
tree | 8ba221bc81fb12f0dc21ff11a47ef9e609b2adce | |
parent | adfae2c63ff6f683f38e7c9be974d752eddc0a43 (diff) | |
download | chrome-ec-36f26030987367e60a8d97dbce9f47f7fda14729.tar.gz |
mec1322: Updated code to only clear the interrupt status bit of LPC_RESET#.
Interrupt Source register is R/WC, so the specific bit should be cleared
by directly writing that bit instead of |=.
BUG=none
TEST=Verified LPC_RESET# is detected by interrupt handler via EC console message.
BRANCH=none
Change-Id: Ib553c839e1311538b17a4d9fbc10c9df5b7e6b44
Signed-off-by: Kevin K Wong <kevin.k.wong@intel.com>
Reviewed-on: https://chromium-review.googlesource.com/256854
Reviewed-by: Shawn N <shawnn@chromium.org>
Reviewed-by: Vic Yang <victoryang@chromium.org>
-rw-r--r-- | chip/mec1322/lpc.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/chip/mec1322/lpc.c b/chip/mec1322/lpc.c index b3c3da9ff0..421330246c 100644 --- a/chip/mec1322/lpc.c +++ b/chip/mec1322/lpc.c @@ -195,7 +195,7 @@ static void setup_lpc(void) gpio_config_module(MODULE_LPC, 1); /* Set up interrupt on LRESET# deassert */ - MEC1322_INT_SOURCE(19) |= 1 << 1; + MEC1322_INT_SOURCE(19) = 1 << 1; MEC1322_INT_ENABLE(19) |= 1 << 1; MEC1322_INT_BLK_EN |= 1 << 19; task_enable_irq(MEC1322_IRQ_GIRQ19); @@ -281,7 +281,7 @@ void girq19_interrupt(void) lpc_get_pltrst_asserted() ? "" : "de"); /* Clear interrupt source */ - MEC1322_INT_SOURCE(19) |= 1 << 1; + MEC1322_INT_SOURCE(19) = 1 << 1; } } DECLARE_IRQ(MEC1322_IRQ_GIRQ19, girq19_interrupt, 1); |