]> git.baikalelectronics.ru Git - kernel.git/commit
Merge series "ASoC: wcd9335: Firx some resources leak in the probe and remove functio...
authorMark Brown <broonie@kernel.org>
Thu, 26 Aug 2021 18:15:14 +0000 (19:15 +0100)
committerMark Brown <broonie@kernel.org>
Thu, 26 Aug 2021 18:15:14 +0000 (19:15 +0100)
commit2b6a7730274e542acd68b5026e03af22a18e1fbc
tree8f11397cd94f31edb286e26578d7d91cc382cfcc
parent700d9b8388b4bf2cba1a6ef239e74b7e4636683b
parent8a399e9b9cfb646df74443189d82d4ce339b66b5
Merge series "ASoC: wcd9335: Firx some resources leak in the probe and remove function" from Christophe JAILLET <christophe.jaillet@wanadoo.fr>:

The first 2 patches are sraightforward and look logical to me.

However, the 3rd one in purely speculative. It is based on the fact that a
comment states that we enable some irqs on some slave ports. That said, it writes
0xFF in some registers.

So, I guess that we should disable these irqs when the driver is removed. That
said, writing 0x00 at the same place looks logical to me.

This cis untested and NOT based on any documentation. Just a blind fix.
Review with care.
You'll be warned :)

Christophe JAILLET (3):
  ASoC: wcd9335: Fix a double irq free in the remove function
  ASoC: wcd9335: Fix a memory leak in the error handling path of the
    probe function
  ASoC: wcd9335: Disable irq on slave ports in the remove function

 sound/soc/codecs/wcd9335.c | 23 +++++++++++++++++++++--
 1 file changed, 21 insertions(+), 2 deletions(-)

--
2.30.2