On Fri, Nov 10, 2017 at 03:40:28PM +0200, Andy Shevchenko wrote: > There is no need, since we preserve firmware settings, to override > polarity for GpioInt() resources. > > While Documentation/gpio-properties.txt refers to any from GpioIo() / > GpioInt() resources, the active_low flag has been introduced to fill the > gap only for GpioIo() which lacks of that information. > > Moreover, in case of GpioInt() existed solution was broken anyway, it > overrides only in one direction, i.e. from 0 to 1, otherwise it would be > still 1 as defined in the resource macro. > > So, move the assignment to a right place and forbid to (semi-)override > polarity for GpioInt() type of resources. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> Reviewed-by: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html