diff options
| author | Harry Yang <harryy@codeaurora.org> | 2016-06-23 17:37:21 -0700 |
|---|---|---|
| committer | Kyle Yan <kyan@codeaurora.org> | 2016-06-28 17:01:43 -0700 |
| commit | 88196d66781e7b593ff64fc2d34bcc8151d8e310 (patch) | |
| tree | 43bd214173437c753ffcb63b5df5370ec8843f6f | |
| parent | f6f4829b35699ba8cb7532a58953eb498fff80f0 (diff) | |
qcom-charger: qpnp-smb2: Remove unnecessary watchdog interrupts
Watchdog interrupts bark and snarl get triggered frequently, causing
flood messages when enabled in debug_mask. However, neither irq is
essentially being used in charger driver.
Remove both interrupts by setting their irq handlers to NULL, and
skipping irq registration in case of NULL handlers.
CRs-Fixed: 1033724
Change-Id: I9414f3238d1568ec29973a7ec74054c779a7a196
Signed-off-by: Harry Yang <harryy@codeaurora.org>
| -rw-r--r-- | drivers/power/qcom-charger/qpnp-smb2.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/power/qcom-charger/qpnp-smb2.c b/drivers/power/qcom-charger/qpnp-smb2.c index 9e752799affe..dbdcd9026541 100644 --- a/drivers/power/qcom-charger/qpnp-smb2.c +++ b/drivers/power/qcom-charger/qpnp-smb2.c @@ -642,8 +642,8 @@ static const struct smb2_irq_info smb2_irqs[] = { { "div2-en-dg", smblib_handle_debug }, { "dcin-icl-change", smblib_handle_debug }, /* MISCELLANEOUS IRQs */ - { "wdog-snarl", smblib_handle_debug }, - { "wdog-bark", smblib_handle_debug }, + { "wdog-snarl", NULL }, + { "wdog-bark", NULL }, { "aicl-fail", smblib_handle_debug }, { "aicl-done", smblib_handle_debug }, { "high-duty-cycle", smblib_handle_debug }, @@ -683,6 +683,9 @@ static int smb2_request_interrupt(struct smb2 *chip, return irq_index; } + if (!smb2_irqs[irq_index].handler) + return 0; + irq_data = devm_kzalloc(chg->dev, sizeof(*irq_data), GFP_KERNEL); if (!irq_data) return -ENOMEM; |