If ACPI table provides an Interrupt() resource we may consider to use it
instead of GpioInt() one.

Here we leave an error condition, when getting IRQ resource, to the driver
to decide how to proceed, because some drivers may consider IRQ resource
optional.

Signed-off-by: Andy Shevchenko <[email protected]>
Reviewed-by: Hans de Goede <[email protected]>
---
 drivers/platform/x86/i2c-multi-instantiate.c | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/drivers/platform/x86/i2c-multi-instantiate.c 
b/drivers/platform/x86/i2c-multi-instantiate.c
index efee77a11419..a1e6ec12ef73 100644
--- a/drivers/platform/x86/i2c-multi-instantiate.c
+++ b/drivers/platform/x86/i2c-multi-instantiate.c
@@ -18,6 +18,7 @@
 #define IRQ_RESOURCE_TYPE      GENMASK(1, 0)
 #define IRQ_RESOURCE_NONE      0
 #define IRQ_RESOURCE_GPIO      1
+#define IRQ_RESOURCE_APIC      2
 
 struct i2c_inst_data {
        const char *type;
@@ -104,6 +105,14 @@ static int i2c_multi_inst_probe(struct platform_device 
*pdev)
                        }
                        board_info.irq = ret;
                        break;
+               case IRQ_RESOURCE_APIC:
+                       ret = platform_get_irq(pdev, inst_data[i].irq_idx);
+                       if (ret < 0) {
+                               dev_dbg(dev, "Error requesting irq at index %d: 
%d\n",
+                                       inst_data[i].irq_idx, ret);
+                       }
+                       board_info.irq = ret;
+                       break;
                default:
                        board_info.irq = 0;
                        break;
-- 
2.19.2

Reply via email to