On Fri, Jan 31, 2025 at 09:37:04AM +0200, Sakari Ailus wrote: > Hi folks, > > One patch turned into a set, the second patch being the original one. > > since v3: > > - Add a patch (first one) to switch to unsigned long type for polarity > field and renames it as gpio_flags. > > - Use a polarity_low boolean field for indicating the GPIO polarity. This > is turned to gpio_flags in int3472_get_con_id_and_polarity(). > > - Put the new polarity_low bool field next to the u8's for better struct > packing. > > - Add a patch (last one) to rename "func" as "con_id" which is used > throughout the GPIO framework. With a comment addressed in patch 2, Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> -- With Best Regards, Andy Shevchenko