On 11/12/2019 6:10 PM, Mark Brown wrote:

On Tue, Nov 12, 2019 at 02:02:37PM +0100, Pawel Harlozinski wrote:

...

/* Disable jack detection */
regmap_update_bits(rt274->regmap, RT274_EAPD_GPIO_IRQ_CTRL,
			RT274_IRQ_EN, RT274_IRQ_DIS);

• disable_irq(rt274->i2c->irq);
  

  return 0;

Shouldn't the register update above be suppressing interrupts?

For rt274 disable_irq is also needed, otherwise we're getting flood of irq's in case of not loaded machine board.

disable_irq() is a bit of a hammer and interferes with things like possible share use.

This irq should be handled in codec code anyway - control of jack detect events from non-codec code is done with set_jack. Similar solutions for jack report irq enable/disable flow are implemented in rt5640 and rt5651.

---

Alsa-devel mailing list Alsa-devel@alsa-project.org https://mailman.alsa-project.org/mailman/listinfo/alsa-devel