diff mbox series

rtc: rv3028: Use IRQ flags obtained from device tree if available

Message ID 20221208133605.4193907-1-w.egorov@phytec.de
State New
Headers
Series rtc: rv3028: Use IRQ flags obtained from device tree if available |

Commit Message

Wadim Egorov Dec. 8, 2022, 1:36 p.m. UTC 
  Make the interrupt pin of the RV3028 usable with GPIO controllers
without level type IRQs support, such as the TI Davinci GPIO controller.

Therefore, allow the IRQ type to be passed from the device tree if
available.

Based on commit d4785b46345c ("rtc: pcf2127: use IRQ flags obtained from device tree if available")

Signed-off-by: Wadim Egorov <w.egorov@phytec.de>
---
 drivers/rtc/rtc-rv3028.c | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

Comments

Alexandre Belloni Dec. 13, 2022, 5:10 p.m. UTC | #1 
On Thu, 8 Dec 2022 14:36:05 +0100, Wadim Egorov wrote:
> Make the interrupt pin of the RV3028 usable with GPIO controllers
> without level type IRQs support, such as the TI Davinci GPIO controller.
> 
> Therefore, allow the IRQ type to be passed from the device tree if
> available.
> 
> Based on commit d4785b46345c ("rtc: pcf2127: use IRQ flags obtained from device tree if available")
> 
> [...]

Applied, thanks!

[1/1] rtc: rv3028: Use IRQ flags obtained from device tree if available
      commit: 2d620d98d74311ac06719adf08dc9a695bd1a10b

Best regards,
diff mbox series

Patch

diff --git a/drivers/rtc/rtc-rv3028.c b/drivers/rtc/rtc-rv3028.c
index dd170e3efd83..b0099e26e3b0 100644
--- a/drivers/rtc/rtc-rv3028.c
+++ b/drivers/rtc/rtc-rv3028.c
@@ -902,9 +902,20 @@  static int rv3028_probe(struct i2c_client *client)
 		return PTR_ERR(rv3028->rtc);
 
 	if (client->irq > 0) {
+		unsigned long flags;
+
+		/*
+		 * If flags = 0, devm_request_threaded_irq() will use IRQ flags
+		 * obtained from device tree.
+		 */
+		if (dev_fwnode(&client->dev))
+			flags = 0;
+		else
+			flags = IRQF_TRIGGER_LOW;
+
 		ret = devm_request_threaded_irq(&client->dev, client->irq,
 						NULL, rv3028_handle_irq,
-						IRQF_TRIGGER_LOW | IRQF_ONESHOT,
+						flags | IRQF_ONESHOT,
 						"rv3028", rv3028);
 		if (ret) {
 			dev_warn(&client->dev, "unable to request IRQ, alarms disabled\n");