> You do not need this property, if driver assumes that. Just enable it > unconditionally. The goal of this patch series is to change exactly that: to prevent the driver from unconditionally enabling the irq for wake. The driver works across numerous buses (spi, uart, i2c, lpc) and supports DT and ACPI. SPI+DT systems all happen to need irq wake enabled. > I don't think anything from previous discussion was > resolved. Which previous discussion do you mean? In v1 it was suggested to split up the DTS changes by arch/soc and add a cover letter (which I did). Wrt to the binding discussion, Sudeep said the new documentation wording looked good to him [1]. [1]: https://lore.kernel.org/all/ZYAjxxHcCOgDVMTQ@bogus/