Re: [PATCH V1 05/15] spmi: pmic-arb: cleanup unrequested irqs

From	kgunda@codeaurora.org
Newsgroups	linux.kernel
Subject	Re: [PATCH V1 05/15] spmi: pmic-arb: cleanup unrequested irqs
Date	2017-06-06 13:00 +0200
Message-ID	<tPknq-7Bq-55@gated-at.bofh.it> (permalink)
References	<tMOKZ-7oW-9@gated-at.bofh.it> <tMOL1-7oW-49@gated-at.bofh.it> <tN15v-6AU-5@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 2017-05-31 07:27, Stephen Boyd wrote:
> On 05/30, Kiran Gunda wrote:
>> From: Abhijeet Dharmapurikar <adharmap@codeaurora.org>
>> 
>> We see a unmapped irqs trigger right around bootup. This could
>> likely be because the bootloader exited leaving the interrupts
>> in an unknown or unhandled state.  Ack and mask the interrupt
>> if one is found. A request_irq later will unmask it and also
>> setup proper mapping structures.
> 
> Do we have systems where this is causing an interrupt storm due
> to a level high interrupt or something? Just plain acking and
> masking irqs at boot if we don't have an irq descriptor created
> yet doesn't sound like a good idea, because we'll lose all
> interrupts that happen before this driver probes?
> 
Yes. There were instances of an interrupt storm without this patch.
There is an RT_STATUS register in the  peripheral address space which
maintains the real time state and can be read by the client driver
before it registers for the irq. Few client drivers such as charger
already doing this.
>> 
>> Also the current driver ensures that no read/write transaction
>> is in progress while it makes changes to the interrupt regions.
>> This is not necessary because read/writes over spmi and arbiter
>> interrupt control are independent operations. Hence, remove the
>> synchronized accesses to interrupt region.
> 
> That's another patch for this paragraph.
This patch is already pulled in to Greg's tree.

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH V1 05/15] spmi: pmic-arb: cleanup unrequested irqs Kiran Gunda <kgunda@codeaurora.org> - 2017-05-30 14:50 +0200
  Re: [PATCH V1 05/15] spmi: pmic-arb: cleanup unrequested irqs Stephen Boyd <sboyd@codeaurora.org> - 2017-05-31 04:00 +0200
    Re: [PATCH V1 05/15] spmi: pmic-arb: cleanup unrequested irqs kgunda@codeaurora.org - 2017-06-06 13:00 +0200

csiph-web