|
|
Subscribe / Log in / New account

ROHM BD96801 Support ERRB IRQ

From:  Matti Vaittinen <mazziesaccount-AT-gmail.com>
To:  Matti Vaittinen <mazziesaccount-AT-gmail.com>, Matti Vaittinen <matti.vaittinen-AT-fi.rohmeurope.com>
Subject:  [PATCH 0/2] ROHM BD96801 Support ERRB IRQ
Date:  Mon, 26 Aug 2024 11:14:21 +0300
Message-ID:  <cover.1724655894.git.mazziesaccount@gmail.com>
Cc:  Matti Vaittinen <mazziesaccount-AT-gmail.com>, Lee Jones <lee-AT-kernel.org>, Liam Girdwood <lgirdwood-AT-gmail.com>, Mark Brown <broonie-AT-kernel.org>, linux-kernel-AT-vger.kernel.org
Archive-link:  Article

Add ERRB IRQ support to ROHM BD96801 PMIC driver.

The BD96801 has two IRQ pins, ERRB and INTB. ERRB is used to notify
'fatal' events so handling it has not been mandatory. However, there
are use-cases where handling the ERRB interrupts may be beneficial.

---

Supporting more than one IRQ pin in one device, using the regmap-IRQ,
caused an IRQ domain naming collision in the debugFS when device-tree
was used.

In order to avoid the collision, support for adding a name suffix to
IRQ domain was merged in regmap tree at
commit c69bb91c47e8 ("regmap IRQ support for devices with multiple IRQs")
This is required for the MFD driver to compile. Hence this series is
based on the regmap/for-6.12.

Furthermore, there has been a few (trivial) fixes to the BD96801 driver
in the regulator tree, and thus merge conflicts are possible. I can
re-spin the regulator driver based on the regulator tree when there is
a branch with the required IRQ domain and regmap commits as well as the
MFD driver. The MFD driver with ERRB support is needed so that the
regulator probe won't break for the users with the "errb" in
device-tree.

The ERRB patches were originally part of this series:
https://lore.kernel.org/all/cover.1717486682.git.mazziesa...

Revision history in the patches refers to the old versions in that
series.

Matti Vaittinen (2):
  mfd: bd96801: Add ERRB IRQ
  regulator: bd96801: Add ERRB IRQ

 drivers/mfd/rohm-bd96801.c            | 276 ++++++++++++++++++++++----
 drivers/regulator/bd96801-regulator.c | 130 ++++++++++--
 2 files changed, 356 insertions(+), 50 deletions(-)

-- 
2.45.2


-- 
Matti Vaittinen, Linux device drivers
ROHM Semiconductors, Finland SWDC
Kiviharjunlenkki 1E
90220 OULU
FINLAND

~~~ "I don't think so," said Rene Descartes. Just then he vanished ~~~
Simon says - in Latin please.
~~~ "non cogito me" dixit Rene Descarte, deinde evanescavit ~~~
Thanks to Simon Glass for the translation =] 


Attachment: signature.asc (type=application/pgp-signature)

-----BEGIN PGP SIGNATURE----- iQEzBAEBCAAdFiEEIx+f8wZb28fLKEhTeFA3/03aocUFAmbMOVgACgkQeFA3/03a ocUjGQgArSbKLvStp10MxHlBvSaNSy/uMRzpOSdSNO+7wzcfbQJ4oY+lDETamYfQ rynk/XvC3q3oOPDPVXrGNUYTqP0FouylzFs6bUNxm6SwdGzXSRL2YDj9fEnzjvfe xUm2yXmGtBCyGzJyhNqLalVDwsNR+XXZCBz0vSO72OW1LQuZfcyC6pzlYky+VC0F 4IbIkDAlDr9XC5CTQUEl8GXEykgQZOlK3xsEXqNrydmJThjtO73FIW2sPYL7e+gA waiPH99DwvL77ZBZmy+Cjfa+R0iqRH68N+N1cPxsVOGoEBbvNmncC404Ya1tWGQ7 jwuy9B/yMpdmreINmc7V83WQ9D3oLg== =zCKK -----END PGP SIGNATURE-----


Copyright © 2024, Eklektix, Inc.
Comments and public postings are copyrighted by their creators.
Linux is a registered trademark of Linus Torvalds